Chrome OS install plugin fail

I can't install this plugin on my chrome OS file manager.
When I click on install plugin, I have an alert windows tell me, install fail.

can't install , me too.
when add this software at web store, error  "chrome 45 is required" in japanese.

@Kleinast @hottool77 Probably, you are using ChromeOS 44 on your Chromebook. I implemented a new feature (auto-updating file list) which a new event supported by ChromeOS 45 was used in the previous version 2.0.3, therefore, your ChromeOS rejected installing my app. So, I just released a new version 2.0.4 including the diff #111 so that my app works on ChromeOS 44. I would be great if you could check whether or not you can install and work my app on your environment.

I close this issue, because I think this issue has been fixed. Thank you for the reporting.


[AutoRelease] t2-servicebus-2024-11-04-58886(can only be merged by SDK owner)

https://github.com/Azure/sdk-release-request/issues/5674
Recording test fail, detailed info is in pipeline log(search keyword FAILED)!!!
https://dev.azure.com/azure-sdk/internal/_build?definitionId=1031
BuildTargetingString
azure-mgmt-servicebus
Skip.CreateApiReview

issue link:https://github.com/Azure/sdk-release-request/issues/5674


Add landscape mode support

https://github.com/openedx/openedx-app-ios/assets/128456094/8c15b2a1-b33e-4e3c-94d0-96c01a223004

Thanks for the pull request, @IvanStepanok!  Please note that it may take us up to several weeks or months to complete a review and merge your PR.
Feel free to add as much of the following information to the ticket as you can:

supporting documentation
Open edX discussion forum threads
timeline information ("this must be merged by XX date", and why that is)
partner information ("this is a course on edx.org")
any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.
Please let us know once your PR is ready for our review and all tests are green.

The paddings to progress dots are added.

@IvanStepanok 🎉 Your pull request was merged!  Please take a moment  to answer a two question survey so we can improve your experience in the future.


Unable to make any chroot with an desktop envioment

Please paste the output of the following command here: sudo edit-chroot -all

name: gnome
encrypted: no
Entering /mnt/stateful_partition/crouton/chroots/gnome...
crouton: version 1-20221126232257~master:ae594340
release: xenial
architecture: amd64
targets: gnome
host: version 15509.81.0 (Official Build) stable-channel octopus
kernel: Linux localhost 4.14.254-20624-g81052f3cfe59 #1 SMP PREEMPT Mon Sep 11 18:02:31 PDT 2023 x86_64 x86_64 x86_64 GNU/Linux
freon: yes
Unmounting /mnt/stateful_partition/crouton/chroots/gnome...

Please describe your issue:
title,  it also gives me this error when making an chroot with an de
sed: can't read common/cras_messages.h: No such file or directory
Failed to complete chroot setup.

If known, describe the steps to reproduce the issue:
sudo crouton -t gnome -n gnome
very creative name

it also errors UID 1000 not found in gnome when I try to enter the chroot anyway

Remove existing chroots with
sudo rm -R /mnt/stateful_partition/crouton/chroots/*
and try again with these instructions.
You have to install using the silence branch:
sudo CROUTON_BRANCH=silence crouton -r xenial -t xfce
The silence branch prevents the attempt to install cras, which cannot be installed without the 3rd-party tool "autotools", which has been removed.


Allow updating maia in a backwards-compatible fashion

Build a map of taker daemon versions when they connect to the maker. This allows
us to determine a policy in regards to which maia version to use during payout curve
calculation.
In this case, if the taker version is below 0.4.15 or unknown, use legacy maia version.

marked it as 'draft' as @bonomat changes that would actually import 2 versions of maia need to come first and be used here :-) (either via duplication of API or via importing 2 versions of the crate).
BTW: should we bump maia's  version? it's a breaking change there!

@klochowicz : is this still needed? If we rollout rollover using libp2p everyone will need to be on the new protocol, meaning, we could make the distinguishing between legacy network stack and libp2p-based network stack

@klochowicz : is this still needed? If we rollout rollover using libp2p everyone will need to be on the new protocol, meaning, we could make the distinguishing between legacy network stack and libp2p-based network stack

yes, I like this idea! with libp2p, this mechanism will not be needed anyway. closing this PR.


Sync parboiled to d7ad77a5077

Decided to see if it had changed much. It hasn't, but it's easier to stay current.

This PR isn't very important, but gives us a chance to discuss bigger matters.
We have problems running publishLocal since we inhaled parboiled2.  We could solve that by publishing this as an actual module, but it looks like another dependency to those who count dependencies by number instead of by burden.  It's a good solution but bad marketing.
Because this almost never changes, we could also consider republishing this as a hard fork in a separate repo.  That would save us recompiling every time.

Can someone describe the publishLocal problem?  I've heard a couple complaints, but not seen it.  I just ran sbt publishLocal on master, and it was fine.

If you set version to something that's not published anywhere and run sbt publishLocal, it will complain that it can't find the parboiled2 dependency in that version.

If you set version to something that's not published anywhere and run sbt publishLocal, it will complain that it can't find the parboiled2 dependency in that version.

Oh, it worked for me because there's already an 0.18.0-SNAPSHOT?  But that version of parboiled2 shouldn't be published anywhere. 🤔

Current Master
Change version.sbt -> 0.19.0-SNAPSHOT
sbt publishLocal
switch dependent project to rely on 0.19.0-SNAPSHOT
coursier.ResolutionException: Encountered 1 error(s) in dependency resolution:
    org.http4s:http4s-parboiled2_2.12:0.19.0-SNAPSHOT:

Don't forget to clear local cache afterwards

Okay, so publishing works, but depending on it doesn't.  And we rewrite the POM, but Ivy pulls from its own metadata, which is why the dependency appears locally but not when resolved via Maven?

I'm punting and making parboiled2 a module: #1382.


Make test names optional

Test names should be optional. I know I can put an empty string, but it would be nice not to have to write it at all. Rather than needing to write a name for each test, some people may prefer not to have names and rather just refer to the failing test in the code itself when something fails.

We aren't planning on changing this API right now.


Implement correct WebFinger requests including hostmeta

https://github.com/felx/mastodon-documentation/blob/master/Running-Mastodon/Serving_a_different_domain.md#webfinger

This should now be fixed.


Use https://pkg.go.dev/github.com/rogpeppe/go-internal/testscript to write new cli tests

We need to have a way to run the apigear cli as tests to test all the different cmd options. The testscript approach seems to be fine and it is extracted from the go internals package (so used also to test go cmd line).

See also https://bitfieldconsulting.com/golang/test-scripts (+2 blog entries)


font awsome free icons support

will make a guide how the resources should be handled..

cool 😎 thanks!

it got also the js animation option and full support for pro version.
needs to tweak the metadata file in order for it to work but for the free version now its fully compatible, with the jadeliver resources i created from github

yes good work and great idea 👍

@choper725  fyi reassigned the font class to package 01


Update TypeScript READMEs

Public-Facing Changes
Minor updates to READMEs
Description
@mcap/support/nodejs was an outdated package name. This stuff is not in the @mcap/support package anymore but in a separate @mcap/nodejs package.
Resolves FG-5991

I agree that is a documentation gap but I don't think it is the @mcap/support or @mcap/nodejs package's responsibility to explain it – they are just showing how to create an McapWriter, not how to use it


edit editapplicant and editjob

Fix bug qhere qualification does not register
fix message for editjob

Codecov Report

Merging #285 (73212a0) into master (0f6f79a) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master     #285      +/-   ##
============================================
- Coverage     70.02%   70.00%   -0.03%
  Complexity      826      826
============================================
  Files           131      131
  Lines          2886     2887       +1
  Branches        375      376       +1
============================================
  Hits           2021     2021
  Misses          738      738
- Partials        127      128       +1

Impacted Files
Coverage Δ

...java/seedu/address/logic/commands/job/EditJob.java
94.80% <ø> (ø)

.../java/seedu/address/model/applicant/Applicant.java
83.63% <0.00%> (-0.77%)
:arrow_down:

Continue to review full report at Codecov.

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


Add a fault tolerance option for multi-line json parsing

Before opening a new issue, please make sure you've reviewed the troubleshooting guide:
https://github.com/brimdata/brim/wiki/Troubleshooting#opening-an-issue
Is your feature request related to a problem? Please describe.
Add a fault tolerance option for multi-line json parsing.
Throw a warning when one line of a multi-line json is parsed incorrectly, skip the line that cannot be parsed and continue parsing.
Instead of throwing a warning and stopping the parsing.
Describe the solution you'd like
Skip the line that cannot be parsed and continue parsing.

This one has been transferred to the Zed repo since that's where an enhancement would need to first exist. For instance, using Zed commit 28f95a3 and the threelines.ndjson.gz test as was used in https://github.com/brimdata/brim/issues/2554:
$ gzcat threelines.ndjson.gz
{"one": 1}
{"two : 2}
{"three": 3}

$ zq -version
Version: v1.2.0-64-g28f95a3c

$ zq threelines.ndjson.gz
threelines.ndjson.gz: parse error: string literal: unescaped line break

Once a mode exists to handle this at the Zed layer, an enhancement may also be required at the app layer, as it would likely not be functionality we'd enable by default and therefore requires a flag (and hence the user would need to toggle the same flag from within the app).
It should also be noted that this is not a limitation unique to Zed tooling. For instance, the jq tool that's very common for multi-line JSON processing also fails.
$ gzcat threelines.ndjson.gz | jq .
{
  "one": 1
}
parse error: Invalid string: control characters from U+0000 through U+001F must be escaped at line 3, column 2

@burpheart: I wanted to make you aware that we've come up with an idea to address this at the Zed layer that's now tracked in #4546. Another user had reported an issue similar to yours and I proposed some interim workarounds in https://github.com/brimdata/zui/issues/2756#issuecomment-1522290280 that might also work for your case. I'm going to go ahead and close this particular issue so please feel free to keep an eye on #4546 and see how you like it when the enhancement lands.


Determine a plan for when it's a good point in time for legal teams to review

My understanding is that the updates of the Governance docs requires a formal approval voting at some point in time. In order to do this, companies usually ask their legal teams for review of the new terms.
From experience, this requires a bit of time (what's usually requested is two weeks) plus ideally an advance notice/projection on when this is going to happen, roughly.

I will own this process and ensure that the GB will have at least 2 weeks for voting on the part they have responsibility to approve.

reviewed and approved by GB


Game allows for only two save slots, overwrites Save 1 if trying to create more

Hello again! I noticed that when trying to save more than a couple of times, it wouldn't create more save slots, but instead just saves over the Slot 1. At one point I was able to create three save slots by deleting Save 0, and saving again. Trying to save after that resulted in Save 3 being overwritten. I downloaded a fresh version of this and didn't do any changes besides adding more Text Events to have some more lenght of the game to test on. I detected that by adding more save folders in user://dialogic/, they would show normally in the Save/Load screens, so I guess the error happens whilst it's trying to create more than a couple of save slots?
I also noticed that right-clicking the save files won't actually delete them, even though there's a text box under them saying you can do that? I don't really see this as much of an issue because you can still delete them by pressing the little x's, but just letting you know.
And another thing I was wondering is that when you click Save/Load/Settings -buttons on the game view, it advances text at the same time. I'm not sure if this is on purpose. What I did back in Unity, when trying to make a VN, was to have little trigger boxes around the buttons, so that when your mouse is inside them, it wouldn't trigger the dialogue to advance. Maybe something similar is possible in Godot as well?

I've fixed the save file overwriting and removed the old label.
The thing with the buttons at the bottom is a bit harder (because dialogic uses a input action), but I'll see what I can do.


[RFC] treewide: remove useless ucitrack definitions

#3381

I think we could remove this file complete if the generic part

rm -f /tmp/luci-indexcache

could be moved into https://github.com/openwrt/luci/blob/master/luci.mk#L218
In files where other uci commands are issued, only the ucitrack part should of course be removed.

And we should split the into separate commits.
So we are able to cherry-pick them.

Any updates? What should I do with uci-defaults? @jow-

@jow- nlbwmon and vpnbypass are fixed, what's next?

Merged via 814ad96df9634f39cb793c31d55b0351f88fadff - thanks!


Add missing emoji 15 images for google

Currently the new emojis from emoji 15 are missing for the "indexed" google files as google was missing from the build script. This adds the missing emojis and fixes the build script.
Would be great if you could release a new 15.0.1 release so we can use emoji 15 with google style :)

Fixed in https://github.com/iamcal/emoji-data/pull/228 by @iamcal

Thank you!


Stop click event propagation.

Also what exact npm version are you guys using since you have lockfileVersion 2 on package-lock?


[Feature]: add verbosity control to main converter

While working on latest conversion project, the constant output from the main converter at each iteration was interfering with the normal in-place behavior of tqdm. This PR offers simple boolean control over whether statements like "Source data is valid!" show up during conversion.
In the code, the source_data related functions need the different structure because the main function is a classmethod (I don't recall why) that would attempt to otherwise reference the verbosity parameter prior to or independent of class creation.

This is good, we should not print messages when things go well unless required to do so. I swear I remembered and issue about logging that I could not find.

getting the source schema needs to be a class method but I don't remember if there was a good reason for making validate_source_schema a class method. Doesn't this break is as a class method? If we are going to do that we might was well turn it into a normal instance method.

P.D. I swear I remembered and issue about logging that I could not find.

https://github.com/catalystneuro/nwb-conversion-tools/issues/162, long-term low-priority

getting the source schema needs to be a class method but I don't remember if there was a good reason for making validate_source_schema a class method. Doesn't this break is as a class method? If we are going to do that we might was well turn it into a normal instance method.

The workaround to avoid breaking is to add verbose to both the instance __init__ and to that class method (it's the only class method that has a print statement in it). Otherwise, yeah it didn't work since there was no cls.verbose without instantiation.

This is good, we should not print messages when things go well unless required to do so. Maybe once we add logging capabilities to the conversions we should drop this type of verbose entirely?

Also, just to be clear, I don't mind our default verbosity in general, it can be nice to know that various steps are passing especially when only running a conversion for a single file (and good to know when any given file is done when running on a bunch of files).

This is good, we should not print messages when things go well unless required to do so. Maybe once we add logging capabilities to the conversions we should drop this type of verbose entirely?

Also, just to be clear, I don't mind our default verbosity in general, it can be nice to know that various steps are passing especially when only running a conversion for a single file (and good to know when any given file is done when running on a bunch of files).
In general it's just good to be able to turn it off and on whenever you do or don't want it for some reason or another.

Yeah, I wonder, in general I prefer programs not telling me when things go as expected. I rather have verbosity as an option for debugging or when I want to be careful. It is a minor issue though.

Yeah, I wonder, in general I prefer programs not telling me when things go as expected. I rather have verbosity as an option for debugging or when I want to be careful. It is a minor issue though.

Feel free to bring up the behavioral change of setting default verbosity to False as an Issue or in our meeting tomorrow, then. Our default has always been True, so that's why it's set to that in this PR.


Connection timed out

Hi Joe, got this little issue going on here. (traceback below).
I'm testing this behind a proxy (at work), so I don't know if that'd be the issue or not, as it requires authentication. However, my python scripts make their way through. Happens both with python 2.7 and 3.4. Does your script not work behind a proxy? Either way, there should be some error handeling. Want me to take care of it?
Pidgey@NorthernSec:~/Git/cve-search$ pwhois cryto.net
Traceback (most recent call last):
File "/usr/local/bin/pwhois", line 23, in
data, server_list = pythonwhois.net.get_whois_raw(args.domain[0], with_server_list=True)
File "/usr/local/lib/python2.7/dist-packages/pythonwhois/net.py", line 33, in get_whois_raw
target_server = get_root_server(domain)
File "/usr/local/lib/python2.7/dist-packages/pythonwhois/net.py", line 76, in get_root_server
data = whois_request(domain, "whois.iana.org")
File "/usr/local/lib/python2.7/dist-packages/pythonwhois/net.py", line 86, in whois_request
sock.connect((server, port))
File "/usr/lib/python2.7/socket.py", line 224, in meth
return getattr(self._sock,name)(*args)
socket.error: [Errno 110] Connection timed out

Yeah, I have no idea what could be causing this, to be honest. Could you have a look into why it's happening in your environment (if it is still happening), and whether it'd require changes to python-whois to make it work?


Skip the session nonce cookie validation for the secondary IS in integration tests.

$subject.

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/502170210

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/502170210

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/502170210
Status: success

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/502170210
Status: success

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/502798707

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/502798707

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/502798707
Status: failure

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/502798707
Status: failure

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/508418464

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/508418464

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/508418464
Status: failure

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/508418464
Status: failure

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/508698425

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/508698425

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/508698425
Status: failure

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/508698425
Status: failure


[BUG] Promise rejections missing stack trace

Stack trace used to show up in this "details" section from azure insights UI:

The user was using chrome 92, so shouldn't be a browser issue where it just provides a terrible error.stack value or something.
Let me know if you need any other information. For now, I'll see about adding the error.stack to my custom dimensions area.
And fwiw, that's a custom error that we explicitly throw from a js file: throw new Error('Should not call this with student persona.')

OS/Browser: chrome 92
SDK Version: 2.7.0
How we initialized the SDK:

appInsights = new ApplicationInsights({
    config: {
      instrumentationKey,
      enableUnhandledPromiseRejectionTracking: true,
      // We only want to log errors for debugging.
      disableAjaxTracking: true,
      disableFetchTracking: true,
    },
  })

  appInsights.loadAppInsights()

  appInsights.addTelemetryInitializer(item => {
    const customDimensions = buildCustomDimensions(true)
    const props = (item.baseData.properties = item.baseData.properties || {})
    for (const key in customDimensions) props[key] = customDimensions[key]
    if (isErrorBenign(item.data)) return false // Don't track item
  })

Additional context
Might be related to this issue's fix: https://github.com/microsoft/ApplicationInsights-JS/issues/1608. promise rejection stack traces used to show up in the "details" section.

Hi @JohnnyFun
it seems that the image is not attached to your comment, could you please update the image and we can better target the problem?
Thanks!

Oh, sorry about that--had copied/pasted my comment on the closed issue I linked to and didn't see I missed the image.

For now, I'm just tracking the lastStack by calling the following prior to setting up my ApplicationInsights instance and telemetry handler:
let lastStack
...
window.addEventListener('unhandledrejection', e => {
  lastStack = e?.reason?.stack
})

...instantiate app insights and telemetry handler...

And then adding lastStack into my customDimensions in my telemetry handler.
I confirmed that my lastStack is populated for that error, but that app insights telemetry envelope has hasFullStack set to false. So the error goes into app insights code with a stack trace and gets pooped out without one.
Fwiw, app insights seems to retain the rejected promise rejection's stack trace for simple tests like calling this function:
function intentionallyThrowInPromise() {
    new Promise(() => {
      setTimeout(() => {
        throw new Error('CN Dev tools button clicked (Test AppInsights unhandled Promise rejection)')
      }, 100)
    })
  }

But for some reason, it doesn't retain the stack trace for my example case where I throw a new Error from somewhere deeper.
I'll let you know if I notice anything else. Let me know if you need/want any other details of my scenario.

@JohnnyFun
Thanks for the details!
After investigation, the major problem that leads to no error trace might be the way we parse onunhandledrejection. Currently we throw  error.reason.toString() to Exception.CreateAutoException() function and the error.reason.toString() will expose no detailed trace stack to  Exception.CreateAutoException() .  Instead we should use error.reason.stack.toString() to enable full trace tracking.
and please let us know if you notice anything else!

Ah, looks like that CreateAutoException doesn't pass a stack parameter through:

So looks like you just need to pass that through with: error.reason?.stack

I'm pretty sure unhandled promise errors always put their exception on reason, so error.reason?.stack should suffice. But might want to double check that you don't need something like error.stack ?? error.reason?.stack.

Otherwise, yes looks like your strategy would also work since stackDetails null coalesce stack and fall back to error, but if that stack parameter is there and just not being passed, it would seem to me it should be passed? :shrug:


Clean decoder

Improve and clean the codebase.
Also set the command to generate a json file.

I still think the hardcoded files are a bad idea.
The files path should be either passed via an argument or use standard IO (STDIN, STDOUT)

We have 2 arguments to let the user provide his own files (smile-data and output-file), we only use the hardcoded ones if these arguments are not provided.
The smile-data argument may be a string as well so users can just use STDIN if they prefer, and the result is written to STDOUT no matter what.
Is it not enough, or maybe I am missing the point ?
I agree that the hardocded files are not beautiful, but I find them convenient for quick usage. Do you think we should force users into creating their own files ?

Read from stdin, output to stdout.
And add option for using an input file. See yaml linter of symfony

It's better but

There is not need to have an option to write to a file. We have > to do that
There is no support for reading from stdin

I have another question: if you don't use asJson, what kind of output do you get?


Amazon Lamba Execution time 5 minutes

Error:
Function exceeded maximum time of 295 seconds and was killed

However, the main README.md mentions the maximum time is 15 minutes which should be 3 times this amount at around 900 seconds.

15 minutes is the maximum execution time for AWS lambda.  You can increase the runtime_timeout upto 900 seconds by editing the ~/.lithops/config file.  I have found 5 minutes to be enough for most of the tasks. Remember, lambda pricing is based on GB/second so it's always good to configure your tasks to run as quickly as possible

Got it, I updated both the runtime_timeout and the --timeout flag in the command.
I found nuclei never finishes running on all templates.


fix(ui): add a tooltip for commonly truncated fields in the events pane

It's not uncommon for the involved object and/or message to be longer
than can be displayed in the events pane, and finding out what the full
string is has often involved going into the inspector to look at the contents
of the element.
Adding a tooltip on these will help mitigate that a bit.
Signed-off-by: Daniel Herman<EMAIL_ADDRESS>

Alternatively, we could also just always add tooltips regardless of whether or not the text is truncated.

I think it would be fine to mouse over and get a pop-up, I think @simster7 added something similar recently

All the time or conditionally?  If we want it all the time, I'll swap it to that

Will add some screenshots / GIFS later today

Here's a quick GIF demoing the transition from no tooltip, to having a tooltip, to not having one again.


MTDSA-12286: RAML for Create a Historic Non-FHL UK Property Income & Expenses Period Summary

Please create a new item to the list in the file docs/overview.md for this endpoint


Does bitproto support union?

Hi. First, thanks for this very cool project!
I have one question: does bitproto support union (or something like protobuf's oneof)? It would be useful for protocol frames where the deserialization of the remaining bytes depends on some kind of frame id in the header.
I tried to look into https://bitproto.readthedocs.io/en/latest/language.html, but unfortunately didn't find anything.

Hello, the union or oneof feature is not supported.

Hi. Thanks for the confirmation!
If there is no plan no support this feature, then I guess this issue can be closed.

Hmm, No such plan so far.


Python fixes

Motivation for this change
Things done

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

[x] NixOS
[ ] macOS
[x] Linux

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

flake8 and mccabe don't seem to need any patching, or is it related due to https://github.com/NixOS/nixpkgs/issues/29103?

Ah, RuntimeError: dictionary changed size during iteration is indeed what I ran into. I'll close this branch and re-retest now that #29103 is fixed.

Still, we may want those commits because I intend to get the distutils.cfg commit back in.


docs: improve the document

Requirements

Fill out the template below.
The pull request must update the test suite.
After you create the pull request, all status checks must be pass.

Type

[ ] Bug fix
[ ] Feature change
[ ] Performance
[ ] Refactor
[x] Documentation
[ ] CI/CD
[ ] Test
[ ] Build
[ ] Style
[ ] Chore

Issue (only for type bug fix/feature change)

Description of the Change

Possible Drawbacks

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

npm package (@latest dist-tag)
GitHub release
v2.0.1

Your semantic-release bot :package::rocket:


Using IPv6 with vpc_attachments[].vpc_route_table_ids fails

Description
ℹ This is a copy of #101 which was closed automatically while still valid.
The module generally supports the use of IPv6. But when adding routes to the tables provided via vpc_attachments[].vpc_route_table_ids, it always tries to assign the vpc_attachments[].tgw_destination_cidr to the IPv4 aws_route.destination_cidr_block argument. Correct would be using aws_route.destination_cidr_block for IPv4 and aws_route.destination_ipv6_cidr_block for IPv6.

[x] ✋ I have searched the open/closed issues and my issue is not listed.

Versions

Module version [Required]: 2.8.2
Terraform version:Terraform v1.3.7
on darwin_arm64

Provider version(s):provider registry.terraform.io/hashicorp/aws v4.46.0

Reproduction Code [Required]
Everything it needs is to configure IPv6 and provide VPC Route Tables:
module "my_vpc_ipv6" {
  source  = "terraform-aws-modules/vpc/aws"
  version = "~> 3.0"

  name = "my-vpc-ipv6"
  cidr = "<IP_ADDRESS>/16"

  azs             = ["${local.region}a", "${local.region}b"]
  private_subnets = ["<IP_ADDRESS>/24", "<IP_ADDRESS>/24"]
  public_subnets  = ["<IP_ADDRESS>/24", "<IP_ADDRESS>/24"]

  enable_nat_gateway = false

  enable_ipv6                     = true
  assign_ipv6_address_on_creation = true

  private_subnet_assign_ipv6_address_on_creation = false

  public_subnet_ipv6_prefixes  = [0, 1]
  private_subnet_ipv6_prefixes = [2, 3]
}

module "my_tgw_ipv6" {
  source  = "terraform-aws-modules/transit-gateway/aws"
  version = "~> 2.0"

  name = "my-tgw-ipv6"

  vpc_attachments = {
    vpc = {
      vpc_id     = module.my_vpc_ipv6.vpc_id
      subnet_ids = module.my_vpc_ipv6.private_subnets

      ipv6_support         = true
      tgw_destination_cidr = "::/0"
      vpc_route_table_ids  = module.my_vpc_ipv6.private_route_table_ids
    },
  }
}

Expected behavior
The code runs and IPv6 routes are being created.
Actual behavior
The code would fails because of issue https://github.com/terraform-aws-modules/terraform-aws-transit-gateway/issues/100, but else it would fail, trying to assign an IPv6 CIDR to the IPv4 aws_route.destination_cidr_block instead of using its IPv6 counterpart aws_route.destination_ipv6_cidr_block.

"Keep alive" post

"Keep alive" post

"Keep alive" post

"Keep alive" post

"Keep alive" post

This issue has been resolved in version 2.12.1 :tada:


MINOR: Note that most communication happens on github

Which issue does this PR close?
N/A
Rationale for this change
Someone asked me for an invite to the community, and when reviewing the communication page on the docs I didn't see any explicit mention about the github site
What changes are included in this PR?
Explicitly state that most communication happens via github tickets / issues
Are there any user-facing changes?
Docs

Codecov Report

Merging #3375 (36b5444) into master (827cab9) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #3375   +/-   ##
=======================================
  Coverage   85.50%   85.50%
=======================================
  Files         294      294
  Lines       54119    54119
=======================================
  Hits        46277    46277
  Misses       7842     7842

Impacted Files
Coverage Δ

datafusion/core/src/physical_plan/metrics/value.rs
86.93% <0.00%> (-0.51%)
:arrow_down:

datafusion/expr/src/logical_plan/plan.rs
77.61% <0.00%> (+0.16%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


RFC: distribution strategy that keeps node content out of my deno install?

Problem
The dep tree for exec ==> fs ==> (node-stuff) yields:
Warning The following packages contained npm lifecycle scripts (preinstall/install/postinstall) that were not executed:
┠─<EMAIL_ADDRESS>┃
┠─ This may cause the packages to not work correctly.
┠─ Lifecycle scripts are only supported when using a `node_modules` directory.
┠─ Enable it in your deno config

Discussion
I've been out of the deno game for a bit. Is there a lib structure strategy that isolates the node/deno content from one another? maybe lib/exec-{common|node|deno}/ style structures? In retrospect, that's probably not viable, as it inhibits you from abstracting the runtime in sibling packages (e.g. I can't just import @gnome/foo and expect it to work from @gnome/bar). 🤔

I suppose it's all working as intended, deno itself just has an un supressable warning


Cannot infer type of "Column" when server_default and default kwargs are set on a @declarative_attr

Describe the bug
When adding the server_default kwarg to a Column declarative_attr, mypy complains like this:
error: Cannot infer type argument 1 of "Column"

not really sure if I am doing something wrong but I am doing as stated here: https://docs.sqlalchemy.org/en/14/core/metadata.html?highlight=server_default#sqlalchemy.schema.Column.params.server_default
Expected behavior
mypy to not error
To Reproduce
This snippet should reproduce the error:
import sqlalchemy as sa
from sqlalchemy.orm import (
    Mapped,
    declarative_mixin,
    declared_attr,
)
from datetime import datetime

@declarative_mixin
class AuditMixin:
    @declared_attr
    def created_at(self) -> Mapped[datetime]:
        return sa.Column(
            sa.DateTime(timezone=True),
            server_default=sa.func.now(), # this is the culprit, commenting this line makes mypy pass
            default=datetime.utcnow,
            nullable=False,
        )

    @declared_attr
    def updated_at(self) -> Mapped[Optional[datetime]]:
        return sa.Column(
            sa.DateTime(timezone=True),
            onupdate=datetime.utcnow,
        )

Then run mypy with sqlalchemy2-stubs on this file.
Error
error: Cannot infer type argument 1 of "Column"

Versions.

OS: macOS Catalina 10.15.7
Python: Python 3.9.5
SQLAlchemy: 1.4.23
mypy: mypy 0.910
SQLAlchemy2-stubs: 0.0.2-alpha.18

Many thanks in advance.

Hi,
Thanks for reporting.
Just a guess. removing default=datetime.utcnow, improve things? Setting default when using server_default is probably not the very usual, since default would be used by sqlalchemy instead of server_default
In any case this is probably a bug in the stubs.

After trying, it does work so I guess that the bug is when setting both server_default and default at the same time. Setting only one of them makes mypy pass, so I guess this is rather strange to encounter (I did this from lack of experience with sqlalchemy).

Thanks for testing that change.
Settings both is supported, even if unusual, I think, so we should try to resolve the issue.
@zzzeek can you confirm that we support this case?


scene length setting referenced from DB record for 3dsMax

Changelog Description
Setting the timeline length based on DB record in 3dsMax Hosts

Additional info
N/A
Testing notes:

launch 3dsMax via Openpype
create new scene
check the timeline
locate "Set Frame Range" in openpype menu
click it
check the frame range or the timeline to see if it is same with the setting shown in DB

This PR seems to also include #4652 - is that correct?

This PR seems to also include #4652 - is that correct?

yes, but they are in a separate ticket, I will wait the old one to merge and update this ticket.  I convert this to draft first


Having issue with bars overlapping each other

I am using N3 to plot shift data as well as daily totals. the daily totals come up fine but the shifts come up as overlapping each other as seen below:

If i use the running totals (average line) it seems to space them correctly as seen below:

The only thing I can think of is I use the same value for X in both the totals and the shifts but I am not 100% sure why that would cause this. Any help is appreciated :)
Thanks,
Greg

Hi Greg,
thanks for reporting the issue! Could you create a Plunkr/CodePen to help me to debug this problem?
Best,
Christoph

This is as good as I could get it. I have options before generating and options2 is what is there after generating. the data is just a snapshot that I pasted in. They overlap different on codepen but still overlap:
http://codepen.io/BinaryWasteland/pen/qdQWjL

Thanks a lot,

I created a little snippet to convert your date strings into date objects and then filter for valid date objects.
var data = [...];
$scope.data = data.map(function(d){
  // Create a valid date
  d.x = new Date(d.x);
  return d;
}).filter(function(d){
  // Filter data points with valid dates
  return d.x;
});

Here is the working demo: http://codepen.io/chaosmail/pen/QbJLav
Best,
Christoph

Ok so all you need to do to see my issue is change options to options2 and unclick the average and my issue is reproduced on yours. Also, is here:
http://codepen.io/BinaryWasteland/pen/qdQWjL

Also, Thanks for the sweet function for Date objects! I didnt think of that

Thanks again for creating the CodePen - it was quite easy to find the error with your help. I fixed it and proposed a PR to solve this issue; it should be merged in the next version.
Here is a CodePen with the fixed PR (just for reference - not for production): http://codepen.io/chaosmail/pen/WvYNRJ

Thanks @chaosmail! Works awesome. Just copied the .min to my project until the next release incorporates it :)


Add markdown filter to the parsed yml data in speakers.yml

We’re adding html tags into the talk and workshop abstracts, which is kinda gross.

Derp, they’re already being parsed as markdown I think.


URI shown as relative and can't open

What steps will reproduce the problem?

Open image detail, click More detail, page.
The error says it can't open browser for the URI specificed.
What version of the program are you using? On what operating system?
Version 4.8.3 commit 1e24b8a3d17b9aec60f934dc0bd345f77d54a139 Archlinux

Indeed, I can reproduce this.
Seems like the site domain is not being properly added in this URL, thanks for the ticket!


add angular support and multiple runs

Add support to have dynamic routing output angular script which adds routes during the dymanic routing module config phase.
Also adds support for routing to be run multiple times with same output file.

I should mention that if run with angular support enabled, the need for the global variable foundationRoutes is removed.  Backwards compatibility is also maintained as angular support is disabled by default.

@soumak77 Thanks for putting this together :) Could you apply these features to the external Front Router plugin as well? We won't be dropping the built-in plugin until after 1.2, but new projects are referencing the external plugin.

Yes I can get to that at some point
On Fri, Dec 4, 2015 at 1:35 PM, Geoff Kimball<EMAIL_ADDRESS>wrote:

@soumak77 https://github.com/soumak77 Thanks for putting this together
:) Could you apply these features to the external Front Router plugin
https://github.com/zurb/front-router as well? We won't be dropping the
built-in plugin until after 1.2, but new projects are referencing the
external plugin.
—
Reply to this email directly or view it on GitHub
https://github.com/zurb/foundation-apps/pull/704#issuecomment-162088077.


🔐💻 hardcode mocked /auth endpoints

PR-Codex overview
This PR adds siwe library for nonce generation and verification, updates dependencies, and introduces new routes for nonce generation and token verification.
Detailed summary

Added siwe library for nonce generation
Updated dependencies
Added route for nonce generation
Added route for token verification

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

added the impl for /nonce here @stepandel


update markdown dependency to 6.0.1

I have updated markdown dependency to 6.0.1 to ensure compatibility with other packages.
Thank you.

Hi @zahniar88, have you check this?
Thank you.


Add boat simulator ROS parameters

Purpose

We want to be able to parameterize particular aspects of the boat simulator to allow other members to configure the simulator to test specific scenarios. In particular, we want to parameterize for the physics engine and the low level control module.
We want to hopefully get most of the parameters that we can foresee in this issue, but down the line we will probably gradually add more. The idea is to add most of the infrastructure in this issue!
Description

You will likely need to make a PR both for the boat simulator when reading the parameters and for sailbot workspace when adding variables to the config file.
Parameters
We have the following ROS parameters that we want to put inside the configuration file:
/**:
 logging_throttle_period_sec: 2.0

physics_engine_node:
 wind_sensor:
  generator_type: gaussian # One of "gaussian", "constant"
  gaussian_params:
   mean: [1.0, 0.0]
   stddev: [0.1, 0.1] # Standard deviation for x and y components
   corr_xy: 0.0
  constant_params:
   value: [1.0, 0.0]

low_level_control_node:
 rudder:
  disable_actuation: false
  fixed_angle_deg: 0.0 # Only used if rudder actuation is disabled
  actuation_request_period_sec: 10.0
  actuation_execution_period_sec: 0.5
  pid: # Only used if rudder actuation is enabled
   kp: 0.0
   ki: 0.0
   kd: 0.0
   buffer_size: 50
 wingsail:
  disable_actuation: false
  fixed_angle_deg: 0.0 # Only used if actuation is disabled
  actuation_request_period_sec: 10.0
  actuation_execution_period_sec: 0.5
  actuation_speed_deg_per_sec: 0.1 # Only used if actuation is enabled

Testing
All we care about for this issue is that the parameters are loaded into their respective ROS nodes by examining the debug logs.
Resources

ROS Parameters
ROS Launch Files

@DFriend01 What is your opinion on enforcing some of these expected values in code? Ex: physics_engine_node.wind_sensor.generator_type is expected to have one of “gaussian” or “constant”. Asserts in code could be added to enforce that this is the case.

@DFriend01 What is your opinion on enforcing some of these expected values in code? Ex: physics_engine_node.wind_sensor.generator_type is expected to have one of “gaussian” or “constant”. Asserts in code could be added to enforce that this is the case.

@chrischang5 If it is implemented elegantly, then I don't see why not. ROS supports this to some extent using something called the ParameterDescriptor class. There isn't any online documentation on it, but you can find the class definition in sailbot workspace:

In physics_engine_node.py, go to PhysicsEngineNode.__declare_ros_parameters()
CTRL + Click on the function delcare_parameters to go to its definition
CTRL + Click on ParameterDescriptor to go it its definition.

You can explore this class to see what the possibilities are.

@DFriend01 what is the logging_throttle_period_sec and why is it listed at the top level?

@DFriend01 what is the logging_throttle_period_sec and why is it listed at the top level?

Basically, there are some logs that are not needed to be displayed on standard out all the time. This makes the logs overwhelming, so rather than publishing all the time, you can set an interval to publish at instead. This is called throttling the logs and the throttle period sets the period.
I figured that everyone might want to use a common throttle period, but maybe that is not the case.
@patrick-5546 what do you think? Should everyone use the same throttling period?

@patrick-5546 what do you think? Should everyone use the same throttling period?

I think the throttle period should be message-dependent. You might want to throttle some messages more than others.

Yeah my concern was that we were setting it for everyone but not everyone was consulted.

ParameterDescriptor has some capabilities of what we want here. ParameterDescriptors can set one of attributes FloatingPointRange or IntegerRange to potentially enforce the values of ros parameters. However, anything that cannot be enforced using ranges cannot be enforced at all. There is this constraints attribute that we can set, but this is just a string that describes what you can or cannot do with the parameter.
Here is a page I found that talks about each attribute of the ParameterDescriptor (in the context of a message that makes use of ParameterDescriptor information)
ParameterDescriptor usage in Python

You can't do any enforcement using ParameterDescriptor in array parameters. ex: DOUBLE_ARRAY


fix(template): use new RivetMultiplayerManager

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

#253
#252
#251  👈
#250
#249
#248
main

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

Join @NathanFlurry and the rest of your teammates on  Graphite


Adding new Relationships in Edit Item is difficult for entities with many relationships - REST

References
Rest Contract: DSpace/RestContract#170
Related to the issue: DSpace/dspace-angular/issues/1148
Description
This PR introduce search methods intended to facilitate an efficient management of the edit relationship. It allows to verify in a single call which if any items in a list have already a specified relationship with an item.
Checklist
This checklist provides a reminder of what we are going to look for when reviewing your PR. You need not complete this checklist prior to creating your PR (draft PRs are always welcome). If you are unsure about an item in the checklist, don't hesitate to ask. We're here to help!

[x] My PR is small in size (e.g. less than 1,000 lines of code, not including comments & integration tests). Exceptions may be made if previously agreed upon.
[x] My PR passes Checkstyle validation based on the Code Style Guide.
[x] My PR includes Javadoc for all new (or modified) public methods and classes. It also includes Javadoc for large or complex private methods.
[x] My PR passes all tests and includes new/updated Unit or Integration Tests based on the Code Testing Guide.
[x] If my PR includes new, third-party dependencies (in any pom.xml), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.
[x] If my PR modifies the REST API, I've linked to the REST Contract page (or open PR) related to this change.

The PR works as expected (https://github.com/DSpace/RestContract/blob/main/relationships.md#search-methods)

So, to me it's ready to go as soon as you add the missing java docs. Thank you for this contribution!

@paulo-graca @tdonohue thanks for the review,
fixing the java docs I also found a pagination bug, so I added a test to prove it and then added the fix.


Backend Homework 4

Homework -4
description:
collections
genrics
drift link : https://video.drift.com/v/ab8bdPUht8nnyTRZzT2wMAR06veqrdjRvpH15c6zQJE8/
snapshots:

Metric
Value

alert_status
:white_check_mark: OK

bugs
0

code_smells
30

reliability_rating
A (0 Bugs)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0


metadata-cleaner: 2.4.0 -> 2.5.0

Description of changes
Diff: https://gitlab.com/rmnvgr/metadata-cleaner/-/compare/v2.4.0...v2.5.0
Changelog: https://gitlab.com/rmnvgr/metadata-cleaner/-/blob/v2.5.0/CHANGELOG.md

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.


[$1000] Request Money - Paid IOU preview showing $0 after re-login

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Login to staging.new.expensify.com or Expensify App on 2 devices - User A and User B
User A send several money requests to user B
User B paid this IOU elsewhere
Users A and B log out and log in again
Go to chat with IOU preview from step 2

Expected Result:
IOU shows the previously paid amount
Actual Result:
Paid IOU preview showing $0 after re-login
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[x] Android / Chrome
[ ] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/27236c38-922b-462f-8978-0f24de7b3303
https://github.com/Expensify/App/assets/78819774/e480bbb8-f489-4ec7-a18a-aa63e5850324
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01b4bd5aaac9493670
Upwork Job ID:<PHONE_NUMBER>533008896
Last Price Increase: 2023-08-07

Yikes, this is weird. Triaging!

@mollfpr what do you think about the proposals?

Fetching the IOU report seems like there are other options. We still have a delay before the openReport is finished.

If this is intentional @mollfpr , should we be closing out this issue?

I'll confirm first with the team, but this should be fixed. The ideal solution might be to need to BE changes instead fetching the unnecessary report each time.

The amount is correctly shown in the current staging. I guess it's already fixed somewhere.
https://github.com/Expensify/App/assets/25520267/91a6e6da-38a0-48b0-8843-69902f4f8742

Actually, it's paid. There's another bug showing owes instead paid.
https://github.com/Expensify/App/assets/25520267/6292be4c-935e-431c-b278-a2e2eb265550

That seems like a separate bug @mollfpr - if we want to report and fix that, let's do that in slack not as part of this issue.

Heading ooo we're still looking for proposals, thanks in advance for the help moving this along BZ team!

Issue not reproducible during KI retests. (First week)

Thank you for that update @mvtglobally

Thank you for the update here!

The issue with the IOU preview showing $0 is not reproduced, but we have another issue similar on the chat list in LHN.
After sign-in back, the IOU preview correctly showed the amount, but the alternate text on the chat list remains showing $0 before opening the IOU report.
https://github.com/Expensify/App/assets/25520267/45b42cee-9d9c-439d-a0ee-c4007f588c41
The requested amount is 100,000 JPY and is displayed as it is on the IOU report, but the conversion is missing two digits 00. So I think https://github.com/Expensify/App/issues/24051 is a different issue with this and shouldn't be closed.

I agree with @mollfpr an issue #24051 shouldn’t close because Japan, Korea currency doesn’t have decimal value and it’s a valid bug with countries has currency decimal is 0

If the issue https://github.com/Expensify/App/issues/24051 shouldn't have been closed, let's tackle that in that issue rather that cross-posting here. Thanks @suneox @mollfpr

@zanyrenney since you are back online, I'm going to unassign myself. The latest status update is that we are still waiting for proposals here.

So I am confused now what we are trying to resolve here, the original issue is already fixed right ?

@ShogunFire Let's wait for what the team says. Although, we still have a similar issue with the alternate text in the chat list.

I also now cannot reproduce this and this this is solved as part of another fix. Closing. Please DM me if you disagree and think this should be reopened.


Add some new iterators to reg set

I'd like to have iterators that will tell me the sse registers with any elements set and the gp registers with any elements set.
It's fine for me to get ymm0, ymm1, ymm2, etc.... instead of xmm0, ymm1, xmm2, etc...

Could you explain what these do?  Does this iterate over every gp/sse register that's in the regset, not just the biggest one?  I think that's what I want but I can't tell.  I'm happy to update the comments in the source.

These iterate over 64-bit registers / 256-bit sse registers that have ANY
of their subregisters set.
So if your set contains ax, rdi, esi / xmm0 ymm1, the iterators gives you
-- rax, rdi, rsi / ymm0, ymm1
On Sat, Feb 21, 2015 at 2:14 AM, Berkeley Churchill <
<EMAIL_ADDRESS>wrote:

Could you explain what these do? Does this iterate over every gp/sse
register that's in the regset, not just the biggest one? I think that's
what I want but I can't tell. I'm happy to update the comments in the
source.
—
Reply to this email directly or view it on GitHub
https://github.com/eschkufz/x64asm/issues/153#issuecomment-75365208.


Demos with box drawing render incorrectly in Android

Apparently the default monospace font on Android doesn't have consistently sized box drawing characters. Most fonts rely on these as fallbacks. Need to find a font that does implement these characters consistently that we can use for Android while keeping the system fonts for other systems.

Example screenshot


ensure correct parsing the query params with '='

Expected:
http://someurl/?param=http%3A%2F%2Fsomeurl%3Fid%3D2837
should be parsed to:
{ param: 'http://someurl?id=2837' }
actual:
{ param: 'http://someurl?id' }
This change fixes that issue.

No idea what your "fix" is doing, but it's not correct.
The reason it doesn't work is that supplying full URLs isn't supported yet: #15

@sindresorhus  Thanks for your quick response. I believe the issue is not related to the full url parsing: my example was a bit confusing... I would expect this test to pass:
it('query strings params including embedded `=`', function () {
	assert.deepEqual(qs.parse('#param=http%3A%2F%2Fsomeurl%3Fid%3D2837'), {param: 'http://someurl?id=2837'});
});

The actual result is { 'param': 'http://someurl?id' }.
I hope this clarifies the suggested fix.

You test seems to pass fine: https://travis-ci.org/sindresorhus/query-string/builds/64986110
Are you sure this is the query string lib you're using and you're using the latest version?

@sindresorhus thanks for responding and adding the test. The issue was on my end - I wasn't encoding the url properly in the href attribute. I owe you a beer ; )


chore(prow/config): delete prow job contexts in branch protection for tidb@master

No need explicit contexts for prow jobs.
Ref: https://docs.prow.k8s.io/docs/components/core/tide/config/#context-policy-options
We only need to ensure tide in required contexts.
Ref:

https://github.com/PingCAP-QE/ci/issues/2129
#812 (reverted before)

/approve


(fea) Semantic chunking

Pull Request Test Coverage Report for Build<PHONE_NUMBER>3
Details

76 of 151 (50.33%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+29.3%) to 29.298%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

scanner/scanner.go
58
72
80.56%

scanner/common.go
18
79
22.78%

Totals

Change from base Build<PHONE_NUMBER>9:
29.3%

Covered Lines:
121

Relevant Lines:
413

💛  - Coveralls


Use builtin scopes more

VSCode has added more builtin fallback scopes, so we can remove some of our fallback scopes by aligning with their conventions.
Note that the macro scope doesn't seem to actually work at the moment. I have filed a bug with VSCode: https://github.com/microsoft/vscode/issues/110150

OK, I've cut this down to just remove the macro fallback, now that VSCode has implemented a builtin fallback: https://github.com/microsoft/vscode/issues/110150

Ah, perfect!
bors r+


Consider taping out a test accelerator for no cost using open source tooling + PDK

Thanks to the funded provided by DARPA there is an increasingly capable, fully automated, RTL to GDS toolchain for creation ASICs called OpenROAD (https://theopenroadproject.org/). To help support the project, Google has partnered with SkyWater to release a fully open source, manufacturable 130nm PDK (https://github.com/google/skywater-pdk) and with efabless a run a regular no-cost MPW program for open source designs (https://efabless.com/).
It would be awesome to see a test accelerator created and taped out in this process! It could also become an excellent benchmark.
Some other interesting links;

https://j.mp/op22-gtools
http://cfu-playground.rtfd.io/
https://www.youtube.com/c/ZeroToASIC/videos
https://www.youtube.com/watch?v=EczW2IWdnOM
https://www.youtube.com/watch?v=lq2BpWwcyQM

Thank you very much for interesting proposal.
Using NNgen, we can generate a model-specific RTL design, which means we cannot change the model structure after we synthesize the RTL design. So the NNgen-based ASIC will be a specialized chip for a specific model. It might be still useful as the feature extractor of the following output layers.


Bug restart fallback

Proposed changes:

Fixes https://github.com/RasaHQ/rasa/issues/4601
Changed TwoStageFallbackPolicy._is_user_input_expected to return true only if user input has not been given since executing action_default_fallback.

When the two stage fallback policy is used together with the mapping policy with default priorities, and the user's implementation of action_default_fallback asks for user input to restart, the two stage fallback policy was still predicting action_listen with confidence 1.0 and therefore overriding the mapping policies prediction of action_restart
Status (please check what you already did):

[x] added some tests for the functionality
[ ] updated the documentation
[x] updated the changelog (please check changelog for instructions)
[x] reformat files using black (please check Readme for instructions)

@melindaloubser1 is this ready for another review? (cc @wochinge )

Yes


Add built-ins for workflow.NewUUID() and workflow.NewRandom()

Will just make it slightly more convenient for a workflow developer vs having to do their own sideeffect or local activity

Can you clarify what workflow.NewRandom() might do? Anyone can of course just math/rand.NewRandom(math/rand.NewSource(123)) to have deterministic random, no side effect needed.

@mastermanu - Bump. Gonna close this, but feel free to comment/reopen w/ more discussion.
Basically we encourage use of side effects for "new UUID" and similar randomness, but this isn't what we're doing in other SDKs (they use run-specific seeded pseudorandom to get deterministic UUIDs) and we fear codifying the difference in API.


Claude tried to use write_to_file for 'xyz.filext' without value for required parameter 'content'. This is likely due to output token limits. Retrying...

When claude gets in this loop due to a large file and to many output tokens the cost becomes quite large.  Please see log below
Time (CDT)	Model	Model latency	Input Tokens	Output Tokens	Type	Error
Aug 30, 2024, 9:15 AM	claude-3-5-sonnet-20240620	103.99	127194	8183	HTTP
Aug 30, 2024, 9:14 AM	claude-3-5-sonnet-20240620	98.93	127061	8183	HTTP
Aug 30, 2024, 9:12 AM	claude-3-5-sonnet-20240620	94.91	126920	8184	HTTP
Requesting user input here would solve a lot.  Additionally I have had a large amount of success by asking it to break the work up into X number of files and then combine them at the end.
While writing this the loop has spend 2$.

Good point, this shouldn't result in the request automatically retrying if sending feedback like 'break into smaller files' can resolve the issues.

Closing as duplicate of https://github.com/saoudrizwan/claude-dev/issues/127


Feature/redux opinionated

@maximblack Thanks for the pull. very helpful.
One issue though. Not sure what I am doing wrong but this pull as well as the previous intl branch does not work for me when trying to serve offline.
I noticed this only happens if I use react-intl library. probably it is related to code-split-component. Tried many solutions. The original redux-oponianated works fine.
Any idea what I am doing wrong?

Hi,
I'm not sure if this is a stupid question or not, but would it make sense to use polyfill.io for the polyfilling instead of dealing with all of this complexity?

Hey all; given our decision to reduce the feature branches I am going to close this for now.  Please feel free to maintain any forks off site.  I am happy to add them to our README, however, I'd like to keep this repo a bit more streamlined and maintainable.
Thanks! 💖


[automated release] v1.0.10

🤖 automated PR, merge to trigger release

We don't need to release this, it was just a detect-secrets update


SimpleSpawner: Rename frequency to period

This property is the time, in seconds, to wait between spawning scenes. This is a period/interval/wavelength, not a frequency, which would be measured in Hz, or 1/seconds.
Adjust the name of the property and its documentation.
The instances of “frequency” that I have not changed are in the example scene's on-screen documentation. In this case it was actually used correctly: the keyboard action to increase the frequency reduces the property which is now called period, and the decrease action respectively increases the period.
I also added an annotation I learned yesterday to put units in the inspector slider for spawn period and spawn limit.

Yes I think it's worth discussing a migration path. The alternatives 1. and 2. don't sound to me a proper migration, but more an effort to keep backwards compat with some overhead. Still we are considering this plugin as experimental and in early stages (as said in the README). So I'd say let's merge this for now.

We came to the conclusion in discussion today that we should instead remove this demo because the new one from godotcon is a better demo of how to make a runner.

@wjt maybe you wanted to close https://github.com/endlessm/godot-block-coding-demos/pull/13 instead?

Oops, yes I did. Well noticed!

I think I'll take a quick look at compatibility..

Let's just do this and ignore the compatibility for now.


x791 & x789 - Work progress/Dashboard enhancement

New columns in results - Work Type,Project,Last Date RNAScope/IHC stain, Last Date slide imaged,Last Date of RNA Analysis, Last Date TO (Stain with TO labware), Last Date LP (Stain with LP labware)

Filtering feature for search results

Chris observed that the state machine was redundant. Your response says that is NA. You might be right, but could you explain why that is NA?

Chris observed that the state machine was redundant. Your response says that is NA. You might be right, but could you explain why that is NA?
Fixed. I have understood it wrong. Chris has pointed out that the state machine is not required in the component as it is only keeping the form values.


Update Readme.md, add yarn

This isn't meant to be a comprehensive list of every way to install this module from the npm registry, rather a hint for people that have no idea how to install a npm module. The npm CLI was chosen for the example not because it is popular, but because that is what is bundled with Node.js. If someone has no idea how to install a npm module, is it reasonable that they would even be using yarn?
I'm :-1: on this change for the reasons above.

Thanks, but this PR doesn't add any significant improvement over the current content in a sustainable manner.


Events are working

The event replication isn't working for me.
The following prints the element and container properly to my console:
    dragulaService.options($scope, 'drag', {
      containers: Array.prototype.slice.call(document.querySelectorAll('ul.schedule-list'))
    });

    dragulaService.find($scope, 'drag').drake.on('drop', function(el, drop, source) {
      console.log(el, 'el');
      console.log(drop, 'drop');
    });

While this does nothing:
    dragulaService.options($scope, 'drag', {
      containers: Array.prototype.slice.call(document.querySelectorAll('ul.schedule-list'))
    });

    $scope.$on('drag.drop', function(el, drop, source) {
      console.log(el, 'el');
      console.log(drop, 'drop');
    });

Check out the demo, they're working in the demo. Your listener for $scope.$on isn't taking the Angular event argument. Listener should have (e, el, container, source) arguments

The problem was that I was creating the bag using the dragulaService.options method, and was not using the directive. I chose to do it that way because the directive takes whatever the current scope is and sets up the bag on that scope. I was attempting to use it inside an ng-repeat so I could only drag internal to that item in the ng-repeat. It would be great if bags created via the options method similarly replicated events.

That's doable. Want to make a pr?

Sure, I probably won't have time today, but I'm happy to get one put together this week


api: allow using paths/list when a path is being deleted

Codecov Report

Merging #1849 (c4c42fa) into main (766260f) will increase coverage by 0.21%.
The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main    #1849      +/-   ##
==========================================
+ Coverage   61.97%   62.19%   +0.21%
==========================================
  Files         107      107
  Lines       11225    11224       -1
==========================================
+ Hits         6957     6981      +24
+ Misses       3750     3725      -25
  Partials      518      518

Impacted Files
Coverage Δ

internal/core/path_manager.go
89.41% <100.00%> (+0.84%)
:arrow_up:

... and 5 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


toBeAnArrayWith doesn't handle repeated items as expected

Can easily be checked with the following two tests
describe('repeated items', () => {
  it('works', () => {
    earl([1, 1]).toBeAnArrayWith(1, 1)
  })

  // this test fails
  it('throws', () => {
    expect(() => earl([1]).toBeAnArrayWith(1, 1)).to.throw('[1] does not contain array [1, 1]')
  })
})

Released as https://github.com/dethcrypto/earl/releases/tag/earljs%400.2.0


sql_scanner.h cannot pass the local pre-commit

04-01-2017 15:01:12 [<module>:143] INFO : Scanning : /Users/angli/Documents/peloton/script/validators/../../src
04-01-2017 15:01:13 [validate_file:108] INFO : Invalid pattern --  free\( -- found in : /Users/angli/Documents/peloton/script/validators/../../src/include/parser/sql_scanner.h
04-01-2017 15:01:13 [validate_file:109] INFO : Line :: 	 * and can realloc() it to grow it, and should free() it to

04-01-2017 15:01:13 [<module>:147] INFO : Validation not successful

Looks like this file is already removed from the main repo. I will rebase my repo.

Already fixed in 834ccde09f458806319ed7d5e7202154d57dded7


feat: Pushing example server docker container to Dockerhub

Description
https://issues.jboss.org/browse/AEROGEAR-8437
Automated publishing of aerogear/voyager-server-example-task container to Dockerhub.
Tested in my fork https://circleci.com/gh/wojta/voyager-ionic-example/9

Small improvements suggested. It's more important if that works.
Feel free to create release from master after: 0.1.0

Please don't merge. It's not working yet.


Split body and else apart in exception handling

Split apart the body and else blocks when building the CFG, with the body transitioning either to the handlers or to else depending on the state of exceptionValue. Additionally, else will now unconditionally jump to ensure, modeling the control flow for when exceptions are raised in the else block.
Motivation

Closer modeling of the control flow during exception handling.
Test plan

See included automated tests.

(first commit LGTM)

Internal stripe build: http://go/builds/bui_HEPREEuLew5Vxu
There were two errors introduced by this, so I'm going to investigate them before merging this.


DuplicateKeyException under load

Hello,
when creating access token with same user/client there's possible issue which will result into DuplicateKeyException. I've tested this on Oracle, Postgre and H2 databses.
I've been trying to solve this with details in #502 . After this implementation Oracle and H2 works ok but there is strange issue with postgre because it has ?different semantics? of transaction.

there is a proxy self-invocation (the transactional method calls another method on the same object)
Postgres, quite reasonably, doesn't like that. It's telling you that the transaction is dead, so you need to start again.

So @dsyer suggested to implement workaround with @Retryable.
Can someone please look into this and probably find the way how to correctly implement this in spring-security-oauth project for all databses? If more details is needed, just let me know.
Thanks

I can confirm the issue exists, using PostgreSQL DB 9.6 and spring-security-oauth version 2.0.12.RELEASE.

I think the problem is https://github.com/spring-projects/spring-security-oauth/blob/master/spring-security-oauth2/src/main/java/org/springframework/security/oauth2/provider/token/store/JdbcTokenStore.java#L148 and 149 using token_id instead of authentication_id
This should be a workaround:
    @Bean
    public TokenStore tokenStore(final DataSource dataSource) {
        final JdbcTemplate jdbcTemplate = new JdbcTemplate(dataSource);
        final AuthenticationKeyGenerator authenticationKeyGenerator = new DefaultAuthenticationKeyGenerator();
        return new JdbcTokenStore(dataSource) {

            @Override
            public void storeAccessToken(final OAuth2AccessToken token, final OAuth2Authentication authentication) {
                final String key = authenticationKeyGenerator.extractKey(authentication);
                jdbcTemplate.update("delete from oauth_access_token where authentication_id = ?", key);
                super.storeAccessToken(token, authentication);
            }

        };
    }

Because the was in #502 the question about a use case: Browser renders a list and does an additional request for each item to the backend running on multiple nodes. The backend nodes use the same database but doesn't answer these requests directly and forwards them over an oauth2 client authentication request to a second backend service.


[recipes] flutter recipes tree is broken

Dependencies from fuchsia and chromium are blocking each other. We need to revert a few CLs and manually roll dependencies.

\cc @nturgut @christopherfujino @iskakaushik @digiter @keyonghan

The problem was the recipe's unit test was always checking out head instead of the cl under test. If there were failures with the expectations or the tests didn't fail during CQ but failed for all the next CLs:

https://flutter-review.googlesource.com/c/recipes/+/2823
https://flutter-review.googlesource.com/c/recipes/+/2783
https://flutter-review.googlesource.com/c/recipes/+/2824


Update link for Nimbus Concensus GitHub

This link:
https://github.com/etheralpha/clientdiversity-org/blob/45d39c8e467a35c7cc216bcfc6055d67144c96f8/_data/clients-consensus.yml#L42
redirects to the nimbus-eth1 execution client, and should instead be changed to https://github.com/status-im/nimbus-eth2

Thanks for reaching out, just update! https://github.com/etheralpha/clientdiversity-org/commit/3dedbb3015dd0f58f11364ef6c94de7452dccb35


fix(docs): fix cookie showBanner example

Thanks for this! I'm afraid I noticed and merged a different PR that fixed this first. Thanks anyway <3
(Now I'm going through PRs in ascending order so I merge the PR from the person who got it first 😅).


non-npm js files available as well in npm package

Is your feature request related to a problem? Please describe.
not a problem
Describe the solution you'd like
i'd like to have access to non-npm javascript files (update.js, update.show.js, update.show.min.js and possibly update.min.js too) within the npm package as well.
Additional context
i use npm but i cant use the browser-update js as modules (var browserUpdate = require('browser-update');). my current webpack/mix setup doesnt build a JS that works in IE11. but i want to display browser-update warning. so i need to init the browser-update in the old fashion. i know i could use local js file (or link the http://browser-update.org/update.min.js file directly), but i would like to use npm as a way to update your package too. thanks

Hi @s4muel,
Can I just add them and this will be fine? Or should I create a second npm with them?
I guess the same package is ok since webpack et al only bundle the actual needed files, right?

@josselex yeah, just adding them to the same npm package is OK

do you have any update on this, please?


Move all project IDs to lowercase

Similar to #78 but for project IDs not author IDs.

@samtstern can i work on this issue?


Update docs with new categorization

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/7342/
Test PASSed.

alluxio-bot, merge this please

alluxio-bot, feature-merge this please

alluxio-bot, cherry-pick this to branch-2.1 please

alluxio-bot, cherry-pick this to branch-2.1 please


Armis AlertsActivities and Device Data Connector Enhancements

Required items, please complete
Change(s):

Armis AlertsActivities and Device Data Connector
Checkpoint Functionality Updated

Reason for Change(s):

Data loss and Frequent 401 Authentication Issue

Version Updated:

No

Testing Completed:

Yes

Checked that the validations are passing and have addressed any issues that are present:

Yes

Armis Device Data Connector
1 Failure is due to testing side scenarios.


Add Dataset.from_variable_length to accept numpy arrays with varying length

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Mac OS x
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): v1.3.0-rc2-20-g0787eee 1.3.0
Python version: Python 3.5.2 :: Continuum Analytics, Inc.
Bazel version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:
Exact command to reproduce:

import numpy as np
from tensorflow.contrib.data import Dataset
a=[np.arange(3), np.arange(5)]
Dataset.from_tensor_slices(np.asarray(a))

Describe the problem
Dataset has a nice .padded_batch feature which allows padding batches to ensure all tensors have the same size, which is very nice for the common use-case of having sequences of different lengths.
However, it seems impossible to create such a dataset from a list of numpy array with different lengths. See command above.
It would be very nice to have a Dataset.from_variable_length_tensor_slices or such that could take such inputs.
I know I'm supposed to use a TextLineDataset or something similar, and then use a series of map functions, but I think tensorflow should support doing all of the pre-processing outside of tensorflow instead of forcing it into a delayed execution graph paradigm, which is harder to reason about and debug. Also, my data is stored in JSON files, and requires complex pre-processing, so the TextLineDataset/map approach doesn't cut it.

You can use the new Dataset.from_generator() to do this (and in general wrap any Python code written as a generator in a Dataset):
a = [np.arange(3), np.arange(5)]
dataset = tf.contrib.data.Dataset.from_generator(lambda: a, a[0].dtype)

Yes that works beautifully. Thanks!
One small change to the code:
a = [np.arange(3), np.arange(5)]
dataset = tf.contrib.data.Dataset.from_generator(lambda: a, tf.int64)

It doesn't seem to be in 1.3... when will this be released? What's the minimal thing I can do the get it now? :smile_cat:

@harpone: pip install tf-nightly should do it!
But as you observed in EDIT2, the code in Dataset.from_generator() is built entirely out of transformations that were available in TF 1.3, so you can copy its implementation into your project and use it directly.

Yeah already installed a nightly build but thanks! BTW I'm getting some weirdness with (yesterday's) nightly: I'm doing
    def input_fn():
        """

        Returns:
            features: A dict containing key/value pairs that map feature column names to Tensors
                        containing the corresponding feature data
            targets: A Tensor containing label (target) values

        """
        if y is not None:
            data_this = [(x_, y_) for x_, y_ in zip(x, y)]
        else:
            data_this = x

        dataset = tf.contrib.data.Dataset.from_generator(lambda: data_this, (tf.float32, tf.float32))
        dataset = dataset.map(cropper)
        dataset = dataset.repeat(epochs)
        if shuffle:
            dataset = dataset.shuffle(buffer_size=10000)
        dataset = dataset.batch(batch_size)
        iterator = dataset.make_one_shot_iterator()

        if y is not None:
            features, targets = iterator.get_next()
            return features, targets

with a dataset formed of lists of same but varying shape x, y tensors. I get
2017-09-23 12:26:55.844395: W tensorflow/core/framework/op_kernel.cc:1192] Out of range: StopIteration: Iteration finished. which repeats epoch number of before the actual training. If I just do .repeat() it just throws the Out of range message forever...
Anyway, I'm starting to really like the Dataset class (together with the Estimator). Really impressive work :)

`a = [np.arange(3), np.arange(5)]
dataset = tf.contrib.data.Dataset.from_generator(lambda: a, tf.int64)
ds1=dataset.repeat(10)
value = ds1.make_one_shot_iterator().get_next()
sess.run(value)   /// first element
sess.run(value)   /// second element
sess.run(value) // Out of range message
throws OutOfRangeError : End of sequence
`

@harpone did just copying
https://github.com/tensorflow/tensorflow/blob/master/tensorflow/contrib/data/python/ops/dataset_ops.py suffice when using 1.3 as the base installation ?

I actually got some other errors when I tried the copying, but yeah, no... I just created a new virtual env and installed the nightly.


Update pipeline.py

actually setting num_workers = num_cpus ()
Description
current warning log lies about setting num_workers = num_cpus, it's the same code block across 4 files. could be more DRY if needed
Fixes # (issue)
New Package?
Did I fill in the tool.llamahub section in the pyproject.toml and provide a detailed README.md for my new integration or package?

[ ] Yes
[x] No

Version Bump?
Did I bump the version in the pyproject.toml file of the package I am updating? (Except for the llama-index-core package)

[x] Yes
[ ] No

Type of Change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Your pull-request will likely not be merged unless it is covered by some form of impactful unit testing.

[ ] I added new unit tests to cover this change
[x] I believe this change is already covered by existing unit tests

Suggested Checklist:

[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added Google Colab support for the newly added notebooks.
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] I ran make format; make lint to appease the lint gods

oops. thanks for catching ❤

Latest main fixes the unrelated failing tests


Leverage Antora cache to locally cache images

Antora uses a cache directory to avoid fetching remote resources:

On subsequent runs, Antora will attempt to resolve remote resources from the cache instead.
https://docs.antora.org/antora/2.2/playbook/configure-runtime/#cache

Currently, the "cache" does not work with Antora because the output directory is deleted between runs.
What we should do:

Resolve the cache directory from Antora
Create a directory in the cache directory named kroki (for instance, if the cache directory is ./.cache/antora then we should create ./.cache/antora/kroki)
When fetching images from the Kroki server, write them in the cache directory
Copy them to the output directory

When running again we should check if the image exists in the cache directory, if that's the case we should copy the image into the output directory, if not we should fetch it.
Open questions

Should we include the diagram library version in the hash? If yes, how do we retrieve the current version of the diagram library from the server?
Can we fetch the images again when the Antora --fetch option is used? If not, how can we remove the images?

This is exactly, what I am looking for!

Currently, the "cache" does not work with Antora
Shouldn't that read "does not work with/for Kroki" (or I don't get the sentence)?

As far as I remember, this information is not available in the Antora context. Meaning that extensions cannot retrieve the location of the cache directory used by Antora.
We could use our own cache directory but I feel like it would be better if everything was inside the same cache directory.

I would prefer not to piggy-back on the Antora flag but create something like --kroki-fetch instead. Some users may need that flexibility.

We cannot do that because extensions cannot add additional command line options. In other words --kroki-fetch won't be recognized by Antora CLI.
As far as I know, the only solution is to use an environment variable: KROKI_FETCH=true antora antora-playbook.yml

Disclaimer: I probably don't know Kroki well enough to have a qualified opinion on this. However, wouldn't the fetch option be enough for system maintainers or doc managers to manually "uncache" diagrams when the diagram lib versions change? Ideally they will want to have control over those anyway, no?

I guess, that's a good point!
To clarify, the current blocker (for me) is that the location of the cache directory used by Antora is not available on the context.
@mojavelinux do you think it's bad if we start using our own cache directory?

As far as I remember, this information is not available in the Antora context.

Actually, it is. The cache is stored on the playbook (and defaults to the return value of getCacheDir if not set). You do need to resolve it, but so does every Antora component. See https://gitlab.com/antora/antora/-/blob/master/packages/content-aggregator/lib/aggregate-content.js#L944-957
The cache folder in Antora is designed so that extensions will make use of it.  For example, "kroki".

We cannot do that because extensions cannot add additional command line options. In other words --kroki-fetch won't be recognized by Antora CLI.

We are considering providing specific fetch categories, but that needs to be taken up with Antora.

Actually, it is. The cache is stored on the playbook (and defaults to the return value of getCacheDir if not set). You do need to resolve it, but so does every Antora component. See gitlab.com/antora/antora/-/blob/master/packages/content-aggregator/lib/aggregate-content.js#L944-957

🤯 🤯 🤯
Then, I think we are ready to implement this feature!

The cache folder in Antora is designed so that extensions will make use of it.  For example, "kroki".

Yes, that's what I had in mind 👍🏻

For a dirty personal caching PoC I tweaked fetch#save() (around https://github.com/Mogztter/asciidoctor-kroki/blob/master/src/fetch.js#L46).
It works fine but I noticed unhxr was still issuing GET requests. Debugging revealed they stem from preprocessPlantUML() -> preprocessPlantUmlIncludes() triggered at https://github.com/Mogztter/asciidoctor-kroki/blob/master/src/asciidoctor-kroki.js#L81. So, those would have to be silenced as well if the diagram is already in the cache.

@Mogztter FYI I'm strongly considering creating an Antora helper (like the @antora/expand-path-helper) to wrap up the logic of finding and creating the cache folder. That should make it easier in the future. Until then, you'll need to use the code I linked to. You can also find a similar example in the ui-loader.


Release changelogs should read release-notes from commit messages

Summary: The current process for writing changelogs is manual. You manually compile the changelogs and pipe it to the create_release_tag script to be used as the tag's message.
We've now standardized our commit message formats, so we can parse commits to determine the type of commit + if it has any release notes to include. This PR updates create_release_tag so that it:

Finds all commits which touch a dependency of the artifact being released.
Parses the commit message to find the kind of commit (bug, feature request, cleanup).
Parses the commit message to get the release notes, if any.
Takes the commits and compiles it into a markdown-formatted changelog.

Here's an example changelog (there were no changelog updates since the last release so I set it to gather changelogs from a much earlier prev version):
tag release/vizier/v0.13.1-pre-changelog5.1
Tagger: Michelle Nguyen<EMAIL_ADDRESS>
##Changes by Kind

###New Features
- New chart type: gauge
- New chart type: pie
- New chart type: Text. Literally just the text you type in the Vis spec, nothing more.
- New chart type: Stat. Shows a label, metric, and units for that metric.

###Bug Fixes
- Fixes a bug where postgres requests could be dropped causing missing protocol traces
update changelog generation

Relevant Issues: N/A
Type of change: /kind cleanup
Test Plan: Ran create_release_tag. See example output above.

Tested out the latest changes. Looks good, so merging it in. We can tackle the build deps afterwards.


Face Recognition- simple API example

Hi! :)
So, I'm looking for an already trained model for face recognition.
Your github readme jumps straight from how to do face alignment with your repo,
to how to train/validate. I'm not interested in training a model,but just use it :)
Can you add to your README a simple,user-friendly example of how to use the trained models to do face recognition,get the embeddings,compare between faces etc.?
I'm sure other people will appreciate that as well :)
Thanks in advnace guys,great work!

Hi! :)
So, I'm looking for an already trained model for face recognition.
Your github readme jumps straight from how to do face alignment with your repo,
to how to train/validate. I'm not interested in training a model,but just use it :)
Can you add to your README a simple,user-friendly example of how to use the trained models to do face recognition,get the embeddings,compare between faces etc.?
I'm sure other people will appreciate that as well :)
Thanks in advnace guys,great work!

Hi, actually we have provided the feature extraction API (extract features from pre-trained models) ./util/extract_feature_v1.py (implemented with PyTorch build-in functions) and ./util/extract_feature_v2.py (implemented with OpenCV). To compute the distance, you can simply refer to the "perform_val" function in ./util/utils.py :)

Hi @ZhaoJ9014  , I was  wondering on how we can generate carray and issame for my own dataset that you use in perform_eval function in utils.py ?
I need to perform evaluation of my model on my own dataset of facescrub images.


meaning of multi_index's constructor

what is the exact meaning of the code and scope as the arguments of multi_index's constructor. Is the owner the creater of the table which is also the account which the contract is set to? what is scope for? Please inform me, thx

Does  Multi-Index Constructor help?

@spartucus i cannot understand the purpose of scope. is the scope to divide the table into several subtable? can scope be any domain even not a member of the class underlying the multi_index table.

will the available primary keys in different scope collide with each other? if i want to move one item from one scope to the other, do i need to reassign the primary key value with available primary key?


chore(deps)!: Update Secrecy

Updates Secrecy to the latest version. Because the api changed, this is unfortunately a breaking change (swiftide included 🤷).
However, probably best to keep secret managing libraries up to date.
Thanks for the great library, we use it a lot.

Glad to learn its useful for you, Thank you for the PR!
There are few breaking changes in main already which are just updated types, those changes are easy for users to upgrade as compared to the changes in this PR - hence I'll plan to release it in subsequent releases (and  take opportunity to update all other dependencies as well) - please consider release in v0.28.0+ for this PR.


Pre-saved cookies don't work

I followed the instructions to export my cookies from the Alexa web app while logged in and save them to<EMAIL_ADDRESS>but it doesn't seem to work. Upon installing (or re-installing) the integration, the file is deleted and I am not logged in. I have verified the cookies are correct.

Please fill out the template and provide logs.

Closed due to lack of info. Also, cookie import it going to be deprecated soon so don't use it.


leshan demo server: endpoint in COAPLOG

For a university project, we wrote an application which hooks into the RESTful API of the leshan demo server.
We used the events emitted by the server.
We started with NOTIFICATION events, which contain the endpoint in their JSON object ("ep": "iot-device1")
Then we switched to COAPLOG events (because we needed also Read and the first Observe answers), but there we saw that there is no JSON property indicating from which endpoint this comes.
This might (or might not) be a little enhancement for the demo server to be more directly useful.
We (roughly) fixed it with this code in EventServlet.java:
 import com.google.gson.JsonElement;

@Override
        public void trace(CoapMessage message) {
	    JsonElement coapLog = EventServlet.this.gson.toJsonTree(message);
            coapLog.getAsJsonObject().addProperty("ep",this.endpoint);
            String coapLogWithEndPoint = EventServlet.this.gson.toJson(coapLog);
            sendEvent(EVENT_COAP_LOG, coapLogWithEndPoint, endpoint);
        }

where trace is the function we replaced
Greetings

I'm not sure to understand why you are now using NOTIFICATIONS events + read/observe RESPONSE from ClientServlet ? :thinking:
But looking at EVENT_COAP_LOG point of view I think it makes sense to add the "ep" property.
Do you want to provide a PR ? (in this case you should have a look at this process)
Or I can just copy/paste the code (as it is really short) but you will not be the author.

You can just copy/paste the code (I don't mind not being the author)
As for why: we're doing a project where we wanted to save all sensor measurements in a Solid pod, but if you want to read for example the Sensor Units, you won't get there with observe (since they don't change).
We wanted the leshan demo server web interface to remain the control interface and we just hook our application into the event stream.

You can just copy/paste the code

Ok I will add it.

We wanted the leshan demo server web interface to remain the control interface and we just hook our application into the event stream

Ok I don't know so much about Solid Pod, but keep in mind you actually plug on CoAP layer and not LWM2M one.
So payload are still encoded and if blockwise transfer is use you will see each block instead of the "rebuilt" payload.

(added in master : 02b3c5da94a877e8828f6aa24baa07c217aff815)

Thanks for the headsup around blockwise transfer.


Incompatible with latest BYG version

The latest version of Oh, the Biomes You Go (BYG) is incompatible with Better End, which throws a crash that seems to have something to do with its compatibility measures. Here is a crash report.
In addition to getting this working again, the latest version of BYG has made sweeping changes to its End biomes that should preferably be addressed in Better End's compatibility:

The Bulbis biomes have been reworked entirely with new plantlife and 'tree' shapes, the latter of which seem to take after your 'Old Bulbis Gardens' variant biome. Said biome will likely need some adjustment to work with the new plantlife.
Likewise, the Nightshade biomes have also seen reworks to the shape of their trees that will no longer match up with Better End's Nightshade Redwoods biome.
And, finally, BYG adds therium crystal deposits throughout its End biomes, which I heard on the Discord that Better End will likely need to explicitly support.

Bug was related to missing structure IDs, it will be fixed in the next version (I think it will be released during this week)

Shape of redwood trees will be updated later. Crystal deposits probably should be added from BYG side

Fixed in 0.9.0, other things will be added later