Rewrite capability conflict detection

The process of detecting capabiltity conflicts was previously very confusing. The logic was separated among several files and the reasoning for various edge cases were not well explained.
This commit migrates the majority of the logic into DefaultCapabilitiesConflictHandler. The logic is simplified, as now we keep track of nodes as they enter and leave the graph. Implicit and explicit capabilities for nodes are tracked by the conflict handler, so we no longer need to filter nodes from conflicts when they are resolved, as capabilities currently in the graph are constantly updated as nodes enter and leave the graph.
Further, we no longer need to iterate over every module of the graph each time we see a new implicit capability, as we already know which nodes contribute which capabilities. Keeping track of implicit capabilities here is much more sensible than iterating over the entire graph for each implicit capability that may be in conflict.
Reviewing cheatsheet
Before merging the PR, comments starting with

❌ ❓must be fixed
🤔 💅 should be fixed
💭 may be fixed
🎉 celebrate happy things

@bot-gradle test apt

I've triggered the following builds for you. Click here to see all build failures.

All Performance Tests (Trigger)


samples: Add PSK support for echo client/server

PSK support is added to echo-client.c and echo-server.c.
If enabled, a header is included which contains the PSK.
If the default dummy PSK header is used, a warning is issued.
The header can be changed via Kconfig.
Signed-off-by: Torstein Grindvik<EMAIL_ADDRESS>

ping @torsteingrindvik can you reformat the help text as suggested by @pfalcon, we could merge this after that

ping @torsteingrindvik can you reformat the help text as suggested by @pfalcon, we could merge this after that

Sorry for the delay, was away for a week.
Should be OK now.


Installation fails on first run with a fresh system

You're missing require statements linking the apt key, apt source and package installation stanzas in unifi::install

Thank you for the issue report!
I've pushed a commit to master (4ecd5cd9) that should fix it.
Try it out and let me know if that works for you.


Bumping version to 1.3.

Signed-off-by: AWSHurneyt<EMAIL_ADDRESS>Description
Bumping version to 1.3.
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #159 (b70558a) into main (e022b96) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #159   +/-   ##
=======================================
  Coverage   50.81%   50.81%
=======================================
  Files         200      200
  Lines        4668     4668
  Branches      594      594
=======================================
  Hits         2372     2372
  Misses       2294     2294
  Partials        2        2

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 e022b96...b70558a. Read the comment docs.


"Go to Definition" for Absolute path doesn't work with folder named "I18n"

VSCode Version: 1.38.1
OS Version: Windows 10.0.18362

Steps to Reproduce:

Create a React app with create react app
Create jsconfig.json with these settings

{
  "compilerOptions": {
    "baseUrl": "src"
  },
  "include": [
    "src"
  ]
}

Create a folder named I18n with some code to export
Import the code from I18n folder
Ctrl+Click(Go to definition) the code to navigate to the I18n definition

Expected Result

Navigated to the code definition in I18n folder

Actual Result

It doesn't Navigate at all

Does this issue occur when all extensions are disabled?: Yes

Is I18n located under src? If not, it will not be included in your project due to your include config

@mjbvz Yeah it is and I did include in the include config, my guess is something do with the number in the folder name.

Doesn't repo for me:

Please share a small example project


Added better documentation for Linux

In CI I always use pecl to install, what do you think about this?

Current coverage is 62.45% (diff: 100%)

Merging #121 into master will decrease coverage by 0.24%

@@             master       #121   diff @@
==========================================
  Files             5          5
  Lines           815        815
  Methods           5          5
  Messages          0          0
  Branches        134        134
==========================================
- Hits            511        509     -2
- Misses          250        252     +2
  Partials         54         54

Powered by Codecov. Last update 74f5662...6a784c8

I added a note about it


ASIM NetworkSession schema parser with its sample and test data for CrowdStrikeFalconHost

Change(s):

Added ASIM NetworkSession schema parser for CrowdStrikeFalconHost based on CEF data

Reason for Change(s):

Initial version ASIM NetworkSession Parser

Version Updated:

Initial version

Testing Completed:

Yes

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

Yes

Hi @niralishah-crest,
Thank you for raising Pull Request with us! We will review the Pull Request and get back to you by shortly. Thanks

Hi @niralishah-crest, please check comments above from @vakohl and act accordingly. Thanks

Hi @niralishah-crest, please act on @vakohl's comments. Thanks

Hi @niralishah-crest,
We're waiting for an update on this. Thanks

Hi @niralishah-crest,
To follow up appropriately, we would appreciate some indication of the timeframe for these changes. Thanks

Hi @niralishah-crest, To follow up appropriately, we would appreciate some indication of the timeframe for these changes. Thanks

@v-atulyadav We have made the suggested changes.
Thanks.

Noted @niralishah-crest. Thanks

Hi @niralishah-crest,
Please work on above comment.
Thanks

Hi @niralishah-crest, please act on @vakohl's comments. Thanks

@v-atulyadav
We have started working on this PR suggested changes. Will update this by next week.
Thanks

Noted @jayeshprajapaticrest. Thanks

@v-atulyadav We have addressed the review comments and resolved merge conflicts.
Thanks

Thanks @niralishah-crest, I can still see branch conflicts.

@v-atulyadav Merge conflicts are resolved now.
Thanks

Thanks @niralishah-crest.

@niralishah-crest @jayeshprajapaticrest Please look into conflicts


Deep recursion in RecipeHelper.generateMaterialListTuples()

https://github.com/turbodiesel4598/NuclearCraft/blob/48c50f5a44d9181779e3e0d2bb22835a40f45211/src/main/java/nc/recipe/RecipeHelper.java#L489
Over 700 recursive calls deep and it'll crash under -Xss256k. The function should be reconsidered or rewritten. Maybe in the overhaul of NC since that's what you're planning :) Just keep that in mind then.
Oracle JDK 8, MC 1.12.2, FTB Revelations 3.2.0

@VADemon You'll have to excuse my inexperience with programming in general for this question, but would using some iterative procedure rather than a recursive one avoid the stack overflow issue?

Not OP but yes, the stack overflow happens because each time you make a call into a function a new stack frame is created. It's part of what gives you that handy stack trace that lists all the function calls in the stack that led to an exception. If the number of stack frames gets too big it can overflow. Using an iterative method will prevent this.

Right, so recursion, although handy, piles up a large number of frames, which then all get taken off sequentially, while iteration is a process of pushing and pulling to/from the stack one at a time.

@VADemon (and @Andrew-Rominger if you're having the same issue), I have a new build here which I think should fix the issue - could you give it a run with your Revelations instance? If it works, I'll put the build up on CF ;)

I think it has become worse: iirc it ran with -Xss320k before, but now it needs ~ -Xss384k and the crash log shows 1026 lines upon StackOverflow.
PS: Looks like MC needs a bare minimum of -Xss192k to initialise (otherwise crashes somewhere in natives of LWJGL)
PPS: You shouldn't commit "versions", commit changes when using git. To publish a "version" use tags and Github's releases for compiled binaries. You bundle too many changes in a single commit, can't easily revert a single feature change this way.
PPPS: Yeah, btw it's still a low priority problem, just try to not make it worse than it was before ;)

@VADemon I'm not using Git properly - it's just a repository for people to look at the source code; I'm happy to just deal with changes locally :)
As for why it was worse than before, turns out I literally forgot to remove the recursive call, so that was happening as well as the iterative one... soz about that :P
Now hopefully this build should work!

Confirmed that build works! Even with -Xss200k
Close when you publish 2.18m

Huzzah! Thanks very much for your help and patience :P
Will get a build out shortly ;)

2.18m uploaded to CF!


updated dependencies

I am currently playing around with this repository, so I decided to give back with a brief npm update command, just for the sake of maintenance.
It seems to build/run, so yeah.

Hi, thanks for the PR.
unfortunately, this includes a major version update of critical, which seems to break the critical CSS inlining task.
I will update that accordingly later, but for now I'm closing this.

Hi, thanks for the PR.
unfortunately, this includes a major version update of critical, which seems to break the critical CSS inlining task.
I will update that accordingly later, but for now I'm closing this.


Reduce use of sleep

Try to use "condition" variables if appropriate to achieve the effects currently achieved by sleeping.

done


Update for Expo SDK v48

Version 48 was released in February
https://blog.expo.dev/expo-sdk-48-ccb8302e231
It's possible that there are no changes required for expo-datadog config plugin. If so it might be worth considering an alternative versioning scheme, to avoid confusion.
Incidentally, I think expo is releasing 49 in a week or two
Thanks!

Oh it looks like there was a release [changes]

it's just not registering to GH Releases

Thanks @louiszawadzki!


Hot reload doesn't apply changes to TextStyle from a variable

Hello,
I followed "Write Your First Flutter App" tutorial to the end. everything looks great. But while I'm in the debugging mode, changes applied to the _biggerFont style are not mirrored in the application by the Hot reload. However, if the TextStyle is applied directly (without a variable - like the attached file) to the Text widget, it will work fine. Also, if debug session is restarted, changes are also applied.
Steps to Reproduce

Follow "Write Your First Flutter App" to the end
Change the TextStyle _fontSize to a different number or change the color of the Text
Save changes while in debugging, see that Hot reload is not applying your changes

Logs
Logs were too big. I attached them.

Flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, v1.2.1, on Linux, locale en_US.UTF-8)
[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
[✓] Android Studio (version 3.3)
[✓] VS Code (version 1.32.3)
[✓] Connected device (1 available)

• No issues found!

logs.txt
main.dart.txt

In Dart, static fields are lazily initialized. This means that the first time you run a Flutter app and a static field is read, it is set to whatever value its initializer was evaluated to. Global variables and static fields are treated as state, and are therefore not reinitialized during hot reload.
fllow Recent code change is included but app state is excluded
You can use function,it work just fine
TextStyle getTextStyle() {
    return const TextStyle(fontSize: 10);
}

Thanks @huangyanxiong01 for the explanation. I'm wondering: wouldn't it slow down the app as at every render this function will be executed? Imagine I have 100+ styles; would these functions be called at every render?

Thanks a lot @LinuxUbuntuLearn u save my live with ur solution. 👍


[backport release-23.05] tuxedo-keyboard: 3.1.4 -> 3.2.7

Description of changes
This PR backports these PRs to fix building tuxedo-keyboard against linux_latest (6.4):

#236069
#243064

They patches apply cleanly. I'm running them right now on my Tuxedo laptop without issues.
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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 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

[ ] Fits CONTRIBUTING.md.

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

6 packages failed to build:

linuxKernel.packages.linux_4_14.tuxedo-keyboard
linuxKernel.packages.linux_4_14_hardened.tuxedo-keyboard
linuxKernel.packages.linux_4_19.tuxedo-keyboard
linuxKernel.packages.linux_4_19_hardened.tuxedo-keyboard
linuxKernel.packages.linux_5_4.tuxedo-keyboard
linuxKernel.packages.linux_5_4_hardened.tuxedo-keyboard

16 packages built:

linuxKernel.packages.linux_5_10.tuxedo-keyboard
linuxKernel.packages.linux_5_10_hardened.tuxedo-keyboard
linuxKernel.packages.linux_5_15.tuxedo-keyboard
linuxKernel.packages.linux_5_15_hardened.tuxedo-keyboard
linuxKernel.packages.linux_6_1.tuxedo-keyboard
linuxKernel.packages.linux_hardened.tuxedo-keyboard (linuxKernel.packages.linux_6_1_hardened.tuxedo-keyboard)
linuxKernel.packages.linux_6_3.tuxedo-keyboard
linuxKernel.packages.linux_6_3_hardened.tuxedo-keyboard
linuxKernel.packages.linux_6_4.tuxedo-keyboard
linuxKernel.packages.linux_latest_libre.tuxedo-keyboard
linuxKernel.packages.linux_libre.tuxedo-keyboard
linuxKernel.packages.linux_lqx.tuxedo-keyboard
linuxKernel.packages.linux_testing_bcachefs.tuxedo-keyboard
linuxKernel.packages.linux_xanmod.tuxedo-keyboard
linuxKernel.packages.linux_xanmod_latest.tuxedo-keyboard (linuxKernel.packages.linux_xanmod_stable.tuxedo-keyboard)
linuxKernel.packages.linux_zen.tuxedo-keyboard

Looks pretty good from my end. No one is going to run Linux <=5.4 on a laptop...

Hydra doesn't think it's good:
https://hydra.nixos.org/eval/1797579#tabs-now-fail

OK, I guess you can consider kernels <= 5.4 unimportant.  But then, please reflect that in meta.broken, so that Hydra doesn't need to try building it all the time.  (nixpkgs master + backport)

Going to make a new PR fixing this.

@vcunat PR is ready at https://github.com/NixOS/nixpkgs/pull/243594


Signature counter check

The U2F device protocol incorporates a usage counter to allow the origin to detect problems in some circumstances. The U2F device remembers a count of the number of signature operations it has performed -- either per key pair (if it has sufficient memory) or globally (if it has a memory constraint, this leaks some privacy across keys) or even something in between (e.g., buckets of keys sharing a counter, with a bit less privacy leakage). The U2F device sends the actual counter value back to the browser which relays it to the origin after every signing operation. The U2F device also concatenates the counter value on to the hash of the client data before signing so that the origin can strongly verify that the counter value was not tampered with (by the browser).
The server can compare the counter value that the U2F device sent it and compare it against the counter value it saw in earlier interactions with the same U2F device. If the counter value has moved backward, it signals that there is more than one U2F device with the same key pair for the origin (i.e., a clone of the U2F device has been created at some point).
The counter is a strong signal of cloning but cannot detect cloning in every case -- for example, if the clone is only one which is used after the cloning operation and the original is never used, this case cannot be detected.

Closed with #8


[docs] Add 'gardening' to Lexicon.md

I noticed this term was explicitly mentioned in Harlan and Robert's talk "Becoming An Effective Contributor to Swift" but this is not part of Lexicon.md.

@swift-ci please smoke test


React to HtmlString removal from Mvc.

https://github.com/aspnet/Mvc/issues/4558
Associated PR: https://github.com/aspnet/Mvc/pull/4651

Hi @NTaylorMullen, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

Note: The breaking change is in for HtmlAbstractions.

@NTaylorMullen this PR is not correct, prime cache helper require both:
using Microsoft.AspNetCore.Html;
using Microsoft.AspNetCore.Mvc.Rendering;

One for HtmlString, another one for IHtmlHelper (it still in namespace Microsoft.AspNetCore.Mvc.Rendering you can check here ).

:up: :date:

:up:

@SteveSandersonMS this PR is obsolete now, i accidently fix this issue with _https://github.com/aspnet/JavaScriptServices/pull/88/files#diff-f2b00f5837f8ffaf55aa82f56f176132R6_
@NTaylorMullen sorry for this :sob:

@laskoviymishka all good!


Unable to see baseband signal.

From readme:
./waveconverter.py -q ../input_files/fan_all_dip1101_pruned_dec_c304p55M_s830k.iq -o ../output_files/test.log -g -p 3
304550000.0
attempting to retrieve protocol 3 from database
Traceback (most recent call last):
File "/home/dima/src/waveconverter/src/waveconverter_gui.py", line 380, in on_transmissionNumberSelect_value_changed
self.changeTxNumberToView(txNum)
File "/home/dima/src/waveconverter/src/waveconverter_gui.py", line 330, in changeTxNumberToView
self.drawBasebandPlot(wcv.txList[wcv.txNum].waveformData,
IndexError: list index out of range
and GUI not work.

Thanks for the bug report. One clarifying question: did the error occur immediately after hitting enter on the command line? Or did you click anything in the GUI first?

First one.

I'm experiencing exactly the same issue, any workarounds yet?

I tried to reproduce this error with the following steps:

installing Ubuntu 16.01 LTS on a fresh partition
installing all system updates
installing gnuradio via pybombs (instructions at http://www.fieldxp.com/Chapter-2-Installing-the-Software.pdf)
installing other pre-reqs listed in README
running the following:
./waveconverter.py -q ../input_files/fan_all_dip1101_pruned_dec_c304p55M_s830k.iq -o ../output_files/test.log -g -p 3

I did see the "list index out of range" error, which must have slipped past my last regression. This error, however, did not impact anything, and I was able to hit the demod button and see the baseband waveform.
mnaujokas - does anything happen when you hit the demod button?
bdn76 - can you tell me anything about your setup? for example, how did you install gnuradio?
Thanks for your patience all.
PC


[Model] Implement CoGSL

I think you may do the following things:

Add from .cogsl import CoGSLDataset in the gammagl.datasets.__init__.py. Otherwise, you will not be able to run the example.
Add from .cogsl import CoGSLModel in the gammagl.models.__init__.py.
Refer to the docs/source/api to add the docstring of the model. I think you need to modify the `docs/source/api/gammagl.datasets.rst.
Modify the README.md to add your model name with the running status of the four backends.
Modify the CONTRIBUTING.md to add your name and your laboratory (or university) name.


Metabase Search Integration: API Ticket

Is your feature request related to a problem? Please describe.
Users want to filter by anything, and see it all on a map and otherwise see the forms and photos in-app.  This would require a lot of work to allow for filtering by anything (basically re-implementing metabase).
This ticket is to

add a parameter to the search endpoint for activities and points of interest to allow for a question ID.
get the api to fetch a token from metabase api
get the api to then take that token and pass it to the question query api. (will attach sample api requests to ticket below).
validate that the result of the query:

has activity or point of interest ids
(maybe later) is owned by an appropriate user

get the api to query for those activities/points of interest and return the result set.

Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Metabase Search Integration: UI Ticket

Metabase Search Integration: UI Ticket


LAG Doc update for LACP

This PR updates the LAG readme for the added support of LACP along with the active-backup mode.

Can you add to the documentation that active-active can be supported with LACP?


mac time.h functions

I used these in https://github.com/shua/graf because I'm used to them, and don't really want to pull in time or chrono just for some stuff already present in libc.h . Compilation worked fine on my linux target, but failed on mac osx because these weren't defined in rust-lang/libc (but these are all present in 13.1 SDK).

It's currently failing because the new functions are unused, but I don't know how to fix this. I'm not sure why these functions are unused but the others are considered to be used. I tried git grep for some of the other functions defined in src/unix/bsd/apple/mod.rs but never found where they're being used, only where they're defined.

Because they're already defined and available on macOS via reexports: https://github.com/rust-lang/libc/blob/bac14427831aaff9b79dcdd125c03109414215c5/src/unix/mod.rs#L1269
Have you tried importing them actually?

ah, thanks, I misunderstood the errors. I updated the PR to remove the time.h functions and struct that were already defined in src/unix/mod.rs and src/unix/linux_like/mod.rs..

@rustbot review

@JohnTitor are you the right person to ask for a review on this?

Yeah, could you squash commits into one?

The test failure seems unrelated to my changes. The test that failed was targeting freebsd 14, but my changes only touched two files, both specific to macos.

It's fixed on master :)
@bors r+

:pushpin: Commit 78244febfab8590aac58308e7b7c3153bc556c4a has been approved by JohnTitor
It is now in the queue for this repository.

:hourglass: Testing commit 78244febfab8590aac58308e7b7c3153bc556c4a with merge b6c2e00b8a601b38470df60ab8efaa8b08fbe22a...

@bors retry timeout

:hourglass: Testing commit 78244febfab8590aac58308e7b7c3153bc556c4a with merge 6b7643d0623def614d60776b670bd3f9513cf02b...

:broken_heart: Test failed - checks-actions

@bors retry

:hourglass: Testing commit 78244febfab8590aac58308e7b7c3153bc556c4a with merge 4c0ff2c36f51118a61e013797281bc7f6fb02a8c...

:broken_heart: Test failed - checks-actions

:broken_heart: Test failed - checks-actions

@bors retry

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13, checks-cirrus-freebsd-14
Approved by: JohnTitor
Pushing 21d90af7feff1c981f77cdb9d48d1fb3e55309d0 to master...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13, checks-cirrus-freebsd-14
Approved by: JohnTitor
Pushing 21d90af7feff1c981f77cdb9d48d1fb3e55309d0 to master...


Adding space before class='active'

Is there a way to add a space before the output of class="active"? Right now, I have to code in a space like this:

XYZ

If the link is not a matching class, however, there's an extra space like this, which is what I'd like to avoid:

XYZ

@jayhlee A few options:
Sprout Active has another function that just outputs the word active and can also be used in conditionals. In this case, the space has been moved to inside the conditional:

XYZ

I haven't explored it too closely, but it may be fairly tricky for us to make more assumptions about where and how the active class is being output and how to adjust spacing proactively. The conditional logic is probably the best solution alongside other general use cases.
There might be some other techniques you could consider as well like minifying all of your html and removing the whitespaces that way.


Include DSTU4145 OIDs

Would OpenSSL team consider including DSTU4145 OIDs into OpenSSL core?
DSTU4145 is important Ukrainian government crypto standard, which is required to be followed by every legal digital interaction in the country. If including the patches would be something OpenSSL core team is interested in, I'm more than willing to assist in submitting necessary patches via a pull request.
Right now there's an OpenSSL fork where an engine with DSTU4145 is being maintained – https://github.com/dstucrypt/openssl-dstu/tree/dstu-1_0_1h. I would very much like to see OIDs separated from the engine, and have OIDs merged into upstream OpenSSL. The patch with OIDs already exists, though it would have to be rebased & cleaned up before submitting in a form of PR: https://github.com/dstucrypt/openssl-dstu/commit/5ffb6d0018eb230acb38656a8ad1cee5da63b28f.
Having OIDs upstreamed would allow DSTU4145 engine maintainers to focus on the engine development and significantly simplify its support. A big issue right now is keeping up with CVE patches, for instance, where it is required that the DSTU4145 patches are constantly rebased.
Another pain is that due to the lack of native support of DSTU4145 in OpenSSL, developers (like myself) have to use local proprietary C libraries, code of which cannot be properly audited.
In essence the engine/OID separation step would somewhat follow scenario with GOST engine: OIDs remained in the OpenSSL tree, while the engine code is now being developed in a dedicated repository – https://github.com/gost-engine/engine.
In 2016 original author of DSTU4145 patches tried submitting them, but ultimately didn't follow up to provide OIDs as a separate PR – https://rt.openssl.org/Ticket/Display.html?id=3163&user=guest&pass=guest. I wonder if it can be done this time.

Closing since #5216 is now merged.


[issue] Unable to load lyrics 404

musickit.js:15918
   GET https://amp-api.music.apple.com/v1/catalog/cn/songs/1460622491/lyrics 404

I'm using HK Apple Music, but the loading of lyrics seems to be in CN area, I'm not sure if this is the reason
Logs: app.example-1632657282153.log

Thank you for your feedback! I'll fix it soon.

Could you please try LitoMusic.exe from this build and see if it has fixed this issue?

It works now :D


New protos for vesting accounts

Changed

Rebuild protos with cosmos-sdk v0.43.0 (added "proto/cosmos/vesting/v1beta1/vesting.proto" to proto paths)

Thank you!


XXXXXXX incorrectly inserted data/BIOINF_3000_2022-16

Hi, thanks for your project! I think I may have found an issue. I hope to cite your paper one day.
Here is my notice.
In the latest update that there are high severity bugs in the file data/BIOINF_3000_2022-16.
XXXXXXX incorrectly inserted by Meghan Phillips.
This bug was introduced on 6/10/2022
The git hash is: ff6c2c1
Line 5
Expected
role deep right somebody require party. YOUR she analysis cause several least green major. walk old design let ACCEPT.

Actual
role deep right XXXXXXX somebody require party. YOUR she analysis cause several least green XXXXXXX major. walk old design let ACCEPT.

Line 7
Expected
NOT might past. religious rather set billion church continue few say.

Actual
XXXXXXX NOT might past. religious rather set billion church continue few say.

I am fixing the incorrect inserted 'XXXXXXX' in this file. I'll plan to submit a pull request soon.
Thank you.

Dear Dave, I am fixing the incorrect inserted 'XXXXXXX' in this file. I'll plan to submit a pull request soon.
Thank you.


Bulk get by keys

Hi,
How can I get docs if I have ids (primary index) with one request?
Could not find this in docs.
Thank you

somethink like bulkGet

does this cover this question ?
https://dexie.org/docs/WhereClause/WhereClause.anyOf()
if so, I would still recommend to add bulkGet method as a shortcut

anyOf() is conceptually a bulkGet, but can be done more efficiently. It's prepared in DBCore, that already have that capability, so it would be easy to extend the Table class to use it.
Marking this issue with the backlog label as it is in the plan.

ok, will wait for more efficient solution
thanks

It's implemented and documented: https://dexie.org/docs/Table/Table.bulkGet()
CI tests are running still.
The feature will be part of 3.0.0-alpha.8 to be released within some days I hope.

Ah, this explains it. The docs already list this method but the 2.0.4, which is installed by default, does not contain this method.
Adding v next (3.0.0-alpha.8) works. Thanks a lot for this fantastic library.


mxnet is much slower than caffe on android

I predict a face image  using Lightened CNN model on my android phone.
Mxnet on i5 machine is about 80ms
Caffe on i5 machine is about 75ms
Mxnet on android is about 700ms (OpenBLAS arm)
caffe on android is adbout 200ms (OpenBLAS arm)
So what is the root cause?

I have a similar issue, but MXnet is far behind both on powerful PC and on Raspberry PI.
Namely, on PC I have 17.38ms per image for MXNet and around 4ms for Caffe. On Raspberry PI - 369.55ms per image for MXNet and 76.53ms for Caffe. The network is identical for both setups, I converted it from the Caffe model.
I am using MXNet.cpp API. The benchmark code looks like this:
        vector<float> img_data(img_size * batch_size, 0.5);
        img.SyncCopyFromCPU(img_data);
        NDArray::WaitAll();
        m_argsMap["data"] = img;

        Executor *executor =
            m_net.SimpleBind(global_ctx, m_argsMap, map<string, NDArray>(),
                             map<string, OpReqType>(), m_auxMap);

        CHECK_NOTNULL(executor);

        std::vector<long> timings;
        for (int i = 0; i < NUM_EVAL; ++i) {
            auto start = std::chrono::system_clock::now();

            executor->Forward(false);
            NDArray::WaitAll();
            auto end = std::chrono::system_clock::now();
            timings.push_back(
                std::chrono::duration_cast<std::chrono::milliseconds>(
                    end - start).count());
        }

        report_metrics(timings, batch_size);

Is it the correct way to measure the MXNet forward pass time?

try to set your env like export MXNET_ENGINE_TYPE=NaiveEngine or try to rebuild mxnet with MXNET_PREDICT_ONLY=1, this will make predict faster about 2times.

When I compile MXNet with ADD_CFLAGS = -DMXNET_PREDICT_ONLY=1 in config.mk, I am getting a segfault when running benchmark:
[10:44:24] src/engine/engine.cc:36: MXNet start using engine: NaiveEngine
Segmentation fault (core dumped)

If I use MXNet lib compiled without this flag, but use the env variable export MXNET_ENGINE_TYPE=NaiveEngine, the benchmark executes just fine, but I do not see any difference in the inference time.
Some more info: I am using OpenBLAS and CPU-only setup:
echo "USE_CUDA=0" >>config.mk
echo "USE_CUDNN=0" >>config.mk
echo "USE_BLAS=openblas" >>config.mk
echo "ADD_LDFLAGS += -lopencv_core -lopencv_imgproc -lopencv_imgcodecs" >>config.mk

i'm not familiar with ARM, in may pc cpu, this indeed speed up. but it maybe due to your model, such as  model-size.

@tornadomeet
I have tested what you said above.in mxnet_predict-all.cc MXNET_PREDICT_ONLY macro is already set as true.
I used openblas in android and it is truly utilizing four cores of CPU fully.
the below is the time cost of convolution and dot cost on my android phone of lightened CNN model.
unpack_patch2col 2 cost:21.904 ms
unpack_patch2col 2 cost:146.750 ms
unpack_patch2col 2 cost:76.812 ms
unpack_patch2col 2 cost:37.764 ms
304ms in total
dot cost:10.211 ms
dot cost:63.549 ms
dot cost:61.920 ms
dot cost:21.202 ms
168ms in total
swapaxis cost:27.053 ms
swapaxis cost:6.774 ms
swapaxis cost:14.554 ms
swapaxis cost:7.757 ms
61ms in total
533 ms

Mark

mark

This issue is closed due to lack of activity in the last 90 days. Feel free to ping me to reopen if this is still an active issue. Thanks!


feature: Support new OpenAI o1 reasoning models

Feature request
I would like if Avante could prompt against the OAI reasoning models
Motivation
These reasoning models are allegedly more capable at coding.
Other
There are a few differences in the API now:

temperature must be set to 1 if you're using either o1-preview or o1-mini models.
max_tokens is not used. Instead max_completion_tokens must be used in the config
These models do not support a system role/message. I think we need to remove this line
Streaming is not supported, so I believe we need to remove this line.

Even with these changes, I'm not getting a successful end to end flow. I've never contributed to the avante.nvim codebase and I'm not entirely sure what else to try at the moment to get things working.
So far, my total diffs look like so:
diff --git a/lua/avante/config.lua b/lua/avante/config.lua
index c1689d7..82d3477 100644
--- a/lua/avante/config.lua
+++ b/lua/avante/config.lua
@@ -30,8 +30,8 @@ You are an excellent programming expert.
     endpoint = "https://api.openai.com/v1",
     model = "gpt-4o",
     timeout = 30000, -- Timeout in milliseconds
-    temperature = 0,
-    max_tokens = 4096,
+    temperature = 1,
+    max_completion_tokens = 4096,
     ["local"] = false,
   },
   ---@type AvanteSupportedProvider
diff --git a/lua/avante/providers/openai.lua b/lua/avante/providers/openai.lua
index 52e62b1..888d466 100644
--- a/lua/avante/providers/openai.lua
+++ b/lua/avante/providers/openai.lua
@@ -51,7 +51,6 @@ M.parse_message = function(opts)
   end

   return {
-    { role = "system", content = opts.system_prompt },
     { role = "user", content = user_content },
   }
 end
@@ -91,7 +90,6 @@ M.parse_curl_args = function(provider, code_opts)
     body = vim.tbl_deep_extend("force", {
       model = base.model,
       messages = M.parse_message(code_opts),
-      stream = true,
     }, body_opts),
   }
 end

I've gotten this far by trying to use the openai provider and seeing it fail and return an error message. This time, it's not returning anything. I see Generating response ... and it never changes.

I assume they'll open it up in the near future but currently to use the reasoning models via the API you need to hit tier 5 https://platform.openai.com/docs/guides/rate-limits/usage-tiers?context=tier-five meaning you have to have bought $1000 in tokens

@Alextibtab if you need to test anything I have this level of API access.

lmao if you have tier 5 then feel free to use it. For now we have to wait till GA for API usage.
btw the API costs would be pretty high for o1 from my testing.

@abatilo I have access and just tried to implement your diff-- experiencing the same freeze at Generating response... unfortunately. Presumably there's some deeper logic in Avante interfering with the o1 non-streamed response mechanic?

@aarnphm There might be a misunderstanding. There is an API but it's closed to certain tiers. It's mostly the same as the current chat completion endpoint but doesn't support streaming. According to OAI, they don't plan on returning the full CoT tokens. Unless maybe they've changed their mind.
I think we would mostly just need to add code paths for handling non-streaming results to make this work

I've added a pull request that adds O1 support, without interfering with other models, and also solving the response hanging issue people had experienced in this thread.
Waiting for a review and approval so that people with tier 5 API access could enjoy it too :)
Have an awesome day guys <3

Well done @LessComplexity ! Fantastic work

@aarnphm
This issue can be closed as my commit already adds o1 models support :)

thanks


Research about Angular JS

Tutorial: https://docs.angularjs.org/tutorial
Developer guide: https://docs.angularjs.org/guide
Angular Style Guide: https://github.com/johnpapa/angular-styleguide
Build structure for frontend: https://scotch.io/tutorials/angularjs-best-practices-directory-structure

First of all, let’s go over what not to do. Many AngularJS tutorials show an app structure that resembles the code below:

A Better Structure and Foundation

Note that the names of arguments are significant, because the injector uses these to look up the dependencies.


Event emitted in constructor doesn't appear in logs eth_logs

Event emitted in constructor doesn't appear in logs eth_logs
The test case to reproduce the issue
Contract:
// SPDX-License-Identifier: MIT
pragma solidity >=0.8.4;

contract Greeter {
    string public greeting;

    event Geeting(string _greeting);

    constructor(string memory _greeting) {
        greeting = _greeting;
        emit Geeting(_greeting);
    }

    function setGreeting(string memory _greeting) public {
        greeting = _greeting;
        emit Geeting(_greeting);
    }
}

Hardhad based test-case
import {ethers} from 'hardhat';
import {Greeter, Greeter__factory} from "../../typechain";

async function main(): Promise<void> {
  const greeterFactory: Greeter__factory = await ethers.getContractFactory("Greeter");
  const greeter: Greeter = <Greeter>await greeterFactory.deploy("Hello1");
  await greeter.deployed();
  console.log('Deployed to: ', greeter.address);

  await greeter.setGreeting("Hello2")

  await new Promise( resolve => setTimeout(resolve, 10_000) );

  let logInfo = {
    address: greeter.address,
    topics: [],
    fromBlock: "earliest",
    toBlock: "latest",
  }
  await ethers.provider.getLogs(logInfo).then((res) => {
    console.log(res)
  })
}

Rinkeby (Geth) output:
Deployed to:  0xeb71C517d7086369f14A8a4bBa610c49854D9c72
[
  {
    blockNumber: 9445539,
    blockHash: '0x0b47060fe21040c27dab60c1c25742ac7d128da6448f0548a22956023f85c9f9',
    transactionIndex: 30,
    removed: false,
    address: '0xeb71C517d7086369f14A8a4bBa610c49854D9c72',
    data: '0x0000000000000000000000000000000000000000000000000000000000000020000000000000000000000000000000000000000000000000000000000000000648656c6c6f310000000000000000000000000000000000000000000000000000',
    topics: [
      '0x5d9206fc543eb3056f9434b45fa582b8c2511de9f6e403f4432918e538d12161'
    ],
    transactionHash: '0x0d752fc049baa6adfd35c7ea58f6d4ae56858f409ac17bd959bc0fcb93ad1a35',
    logIndex: 73
  },
  {
    blockNumber: 9445540,
    blockHash: '0x172243a4f3e21a828331bb1c853b2fb371c9208263edd62a27071fcdbd6ca154',
    transactionIndex: 29,
    removed: false,
    address: '0xeb71C517d7086369f14A8a4bBa610c49854D9c72',
    data: '0x0000000000000000000000000000000000000000000000000000000000000020000000000000000000000000000000000000000000000000000000000000000648656c6c6f320000000000000000000000000000000000000000000000000000',
    topics: [
      '0x5d9206fc543eb3056f9434b45fa582b8c2511de9f6e403f4432918e538d12161'
    ],
    transactionHash: '0xc05f4b9e70e1cdfa918d0ba20608424981f408759fad793d82d684e5804c88a2',
    logIndex: 61
  }
]

Aurora  testnet (https://testnet.aurora.dev) output:
Deployed to:  0x4E693DcD7b30AFd40D8F04D17a5298257E8a2958
[
  {
    blockNumber: 66867153,
    blockHash: '0x24f2182ae11c0685ce101edeae915844c82a596dcec2f92d84cbb1aefcc198b0',
    transactionIndex: 0,
    removed: false,
    address: '0x4E693DcD7b30AFd40D8F04D17a5298257E8a2958',
    data: '0x0000000000000000000000000000000000000000000000000000000000000020000000000000000000000000000000000000000000000000000000000000000648656c6c6f320000000000000000000000000000000000000000000000000000',
    topics: [
      '0x5d9206fc543eb3056f9434b45fa582b8c2511de9f6e403f4432918e538d12161'
    ],
    transactionHash: '0xe699cffe626b66159d7b0e8a0cd8d064245ea03a6aace77527e8c927c79ff708',
    logIndex: 0
  }
]

Decoded data from the event is "Hello2"

A quick deployment using remix shows that Remix is able to parse the logs
Next Ethers troubleshooting.

How many events do you get in the response? Could you also share the address of the deployed contract?


deb rpm: fix local dependency gem

Before:

missing fluent-plugin was installed

After:

missing fluent-plugin was installed
missing dependency gem was also installed

NOTE:

if missing gem requires development packages to build it, it will
fail.
fluent-diagtool depends on highly systemd service, so it is simple
just to use fluent-gem detecting missing gems.

service restart may need some more tweaks in rpm CI, but it is out of this PR.

TODO: add missing test case

TODO: refactor v6 artifact path to reduce total diff.

Reduced modified test case (.sh) because of changed v6 artifact path.

When upgrade next major version package, looks like that it fails to install missing gem.
And the test was passed...
Is this expected result?
   ERROR:  Error installing readline-ext:
  	ERROR: Failed to build gem native extension.

      current directory: /opt/fluent/lib/ruby/gems/3.3.0/gems/readline-ext-0.2.0/ext/readline
  /opt/fluent/bin/ruby extconf.rb
  checking for rb_io_descriptor()... *** extconf.rb failed ***
  Could not create Makefile due to some reason, probably lack of necessary
  libraries and/or headers.  Check the mkmf.log file for more details.  You may
  need configuration options.

  Provided configuration options:
  	--with-opt-dir
  	--without-opt-dir
  	--with-opt-include=${opt-dir}/include
  	--without-opt-include
  	--with-opt-lib=${opt-dir}/lib
  	--without-opt-lib
  	--with-make-prog
  	--without-make-prog
  	--srcdir=.
  	--curdir
  	--ruby=/opt/fluent/bin/$(RUBY_BASE_NAME)
  /opt/fluent/lib/ruby/3.3.0/mkmf.rb:480:in `try_do': The compiler failed to generate an executable file. (RuntimeError)
  You have to install development tools first.

  	from /opt/fluent/lib/ruby/3.3.0/mkmf.rb:573:in `try_link0'
  	from /opt/fluent/lib/ruby/3.3.0/mkmf.rb:591:in `try_link'
  	from /opt/fluent/lib/ruby/3.3.0/mkmf.rb:802:in `try_func'
  	from /opt/fluent/lib/ruby/3.3.0/mkmf.rb:1110:in `block in have_func'
  	from /opt/fluent/lib/ruby/3.3.0/mkmf.rb:983:in `block in checking_for'
  	from /opt/fluent/lib/ruby/3.3.0/mkmf.rb:344:in `block (2 levels) in postpone'
  	from /opt/fluent/lib/ruby/3.3.0/mkmf.rb:314:in `open'
  	from /opt/fluent/lib/ruby/3.3.0/mkmf.rb:344:in `block in postpone'
  	from /opt/fluent/lib/ruby/3.3.0/mkmf.rb:314:in `open'
  	from /opt/fluent/lib/ruby/3.3.0/mkmf.rb:340:in `postpone'
  	from /opt/fluent/lib/ruby/3.3.0/mkmf.rb:982:in `checking_for'
  	from /opt/fluent/lib/ruby/3.3.0/mkmf.rb:1109:in `have_func'
  	from extconf.rb:4:in `<main>'

  To see why this extension failed to compile, please check the mkmf.log which can be found here:

    /opt/fluent/lib/ruby/gems/3.3.0/extensions/x86_64-linux/3.3.0/readline-ext-0.2.0/mkmf.log

  extconf failed, exit code 1

  Gem files will remain installed in /opt/fluent/lib/ruby/gems/3.3.0/gems/readline-ext-0.2.0 for inspection.
  Results logged to /opt/fluent/lib/ruby/gems/3.3.0/extensions/x86_64-linux/3.3.0/readline-ext-0.2.0/gem_make.out
  Can't install missing gem automatically: please install readline-ext manually.

https://github.com/fluent/fluent-package-builder/actions/runs/11475137886/job/31935272678?pr=688#step:7:471

When upgrade next major version package, looks like that it fails to install missing gem. And the test was passed...
Is this expected result?

This is the case in NOTE: which we can't salvage it:

if missing gem requires development packages to build it, it will
fail.

Thus, it is not desired, but the answer is yes.

marked readline-ext as skipped target by default.

rebased with feature branch

Looks like we should increase the job timeout.

workflow run timeout was extended, so rebased with latest feature-nodowntime branch.

Fixed typo in postinst

rebased with recent feature branch.

Let's cache v6 dummy
      - name: cache deb
        uses: actions/cache@v4
        id: cache-deb
        with:
          path: |
            fluent-package/apt/repositories
            fluent-apt-source/apt/repositories
            fluent-lts-apt-source/apt/repositories
+            v6-test/fluent-package/apt/repositories
          key: ${{ runner.os }}-cache-${{ matrix.rake-job }}-${{ hashFiles('**/config.rb', '**/Rakefile', '**/Gemfile*', 'fluent-package/templates/**', 'fluent-package/debian/**', 'fluent-package/apt/**/Dockerfile') }}
      - name: Build deb with Docker
        if: ${{ ! steps.cache-deb.outputs.cache-hit }}
        run: |
          rake apt:build APT_TARGETS=${{ matrix.rake-job }}
      - uses: actions/checkout@master
+        if: ${{ ! steps.cache-deb.outputs.cache-hit }}
        with:
          path: v6-test
      - name: Build v6 deb with Docker
+        if: ${{ ! steps.cache-deb.outputs.cache-hit }}
        run: |
          cd v6-test
          git config user.email<EMAIL_ADDRESS>          git config user.name "Fluentd developers"
          git am fluent-package/bump-version-v6.patch
          rake apt:build APT_TARGETS=${{ matrix.rake-job }}

Fixed not matched local_missing_plugins.

NOTE:

win: should be fixed build failure in another PR.
deb: service issue should be fixed in another PR.

(ボタン押し間違えました :bow: )


Login info correct, can't retrieve snaps or user information & gzdecode(): data error

Hello, the login info for the user is correct, can't retrieve snaps or send message to friends. Says the user is not added as a friend. This is the exampleFunctional.php. All of the required php plugins are installed. Recently changed from xampp to wamp because i was told xampp didnt work well with this api. I have tried with 3 different users still the same.
*username* is ofcourse the correct username
*password* is the correct password.

--
<?php

require_once("../src/snapchat.php");

//////////// CONFIG ////////////
$username = "*username*"; // Your snapchat username
$password = "*password*"; // Your snapchat password
$debug = true; // Set this to true if you want to see all outgoing requests and responses from server
////////////////////////////////

$imagePath = "http://i.imgur.com/LX9UrjP.jpg"; // URL or local path to a media file (image or video)
$sendTo = array();

$snapchat = new Snapchat($username, $debug);

//Login to Snapchat with your username and password
$snapchat->login($password);

// Get your friends in an array
$friends = $snapchat->getFriends();

echo "My friends: ";
print_r($friends);

// Send snap adding text to your image and 10 seconds
$snapchat->send($imagePath, $sendTo, "this is a test :D", 10);

// Set a story
// $snapchat->setStory($imagePath);

// Set a story adding text to the image and 5 seconds
$snapchat->setStory($imagePath, 5, "This is my story");

// Get snaps data (Without storing them)
//$snapchat->getSnaps();

// Automatically downloads Snaps and store it in 'Snaps' folder
$snapchat->getSnaps(true);

// Send chat message to "username"
$snapchat->sendMessage("nichotheman", "hello from Snap-API!");

?>

Debug:
REQUEST TO: https://feelinsonice-hrd.appspot.com/loq/device_id
DATA: timestamp=1429056434190&req_token=93030254891132186e13b6e5f9ace7442cef4d8719657a8309b42214d3c512bb
RESULT:

REQUEST TO: https://android.clients.google.com/auth
Array
(
    [device_country] => us
    [operatorCountry] => us
    [lang] => en_US
    [sdk_version] => 19
    [google_play_services_version] => 7097038
    [accountType] => HOSTED_OR_GOOGLE
    [system_partition] => 1
    [has_permission] => 1
    [add_account] => 1
    [service] => ac2dm
    [source] => android
    [androidId] => 356663c3e9de45ef
    [get_accountid] => 1
    [Email] =><EMAIL_ADDRESS>    [app] => com.google.android.gms
    [client_sig] => 38918a453d07199354f8b19af05ec6562ced5788
    [EncryptedPasswd] => AFcb4KQph7WUX3mq8yTD_rBCGrLHZr2yMrySlte4EFlQNEVdIlDUQz51QP5NvxxUIa_kUi9gR8wckw0IfS0l4oxohl_LsxBBO0NuyF6fMZNoO2DprJa3CAPfrYGhz9ALt1vZpR3KHWSV3SNs3pYEvRzye1qLM9p9_Ij6xgpZHzzFxlG9Sw==
)
DATA: 1
RESULT: SID=DQAAAMkAAAALhQDclk1dOwLQ7JqR5_yNAEy9S2LhfX_InGXmkClto2XUBFUgv5T5M5k3hpQ2zRSQqyfXqpcPUtJ3UCdJtI11CDeV4QHc2_qo2_VtWa5sklb3Xcc3QsUqmiLZgnhVb6YnY6iMdtOH-zYd32y_NRl8Ao7MrRe2Bd7CkX-szExbpPg3ZtcaM4H_25gA-Td5Hl4mFIk1J50IOTVAzcxeDdY_L01PaSXVAU06E20Sn4gaiwYBN6O-dacxONZgFFP-6EElCAlBW39ow7dQKCEhUsBk
LSID=DQAAAMwAAAASLbai602QqD4T2IAIpGabbeCM0Rw1IFZiBQUiUQ0tAb9jXszar-R6HHHAYMCbf6UQ6Yx9xkRCTfJMDnG4qglz8FHzyQNfb7NPXMz3Hb45lJuznj7cNq88eQ8Q5jmgeDBOUlPaLBfUrYTZF9xQB5GzsX_ayoHmuSWbbZyU_wRt5G9guYiCEPCT0N_neEzQlL38Zqx66BF-dO6hO4deUjN5FcQw-sDnG3bsj7Fi9edpUt0SeZdjl4NjO13d5e8nvBTAhQLCo4L7TMM8JrdX2KX6
Auth=DQAAAM0AAADna8CTW56LDBvpL1gJH3OEnclHR-8gPcsKhhCfVDJW7hujrFslsJdCtmtqQO4mST7Bf6B_SpZL6kOZkCm4DnkpBazz7S8OnEJx-qBtNqZWnVsAgZBMEIDXkaWkk5GiahAUbs1I16Q5Y8a5i1y6QSA3ut704IAF7ASHKj4v9D5ORt5c7uFCPyXZK61PTWgislVSachF9Q7BT_jUZnQaBT8rhJmwN8gq-aK10YgwyI21wIJaQSlgHmPBBIiCgUkQIZ4ANU9YwJW5pGS42q1_AGHX
issueAdvice=auto
services=hist,mail
<EMAIL_ADDRESS>Token=oauth2rt_1/lxvQ9ouzhu1okXCGQz6aE_NnUM7hSmuU0AehuUe1-lI
GooglePlusUpgrade=1
RopText=
RopRevision=1
firstName=john
lastName=thomas
accountId=105923436531085384857

REQUEST TO: https://android.clients.google.com/auth
Array
(
    [device_country] => us
    [operatorCountry] => us
    [lang] => en_US
    [sdk_version] => 19
    [google_play_services_version] => 7097038
    [accountType] => HOSTED_OR_GOOGLE
    [Email] =><EMAIL_ADDRESS>    [service] => audience:server:client_id:694893979329-l59f3phl42et9clpoo296d8raqoljl6p.apps.googleusercontent.com
    [source] => android
    [androidId] => 378c184c6070c26c
    [app] => com.snapchat.android
    [client_sig] => 49f6badb81d89a9e38d65de76f09355071bd67e7
    [callerPkg] => com.snapchat.android
    [callerSig] => 49f6badb81d89a9e38d65de76f09355071bd67e7
    [EncryptedPasswd] => oauth2rt_1/lxvQ9ouzhu1okXCGQz6aE_NnUM7hSmuU0AehuUe1-lI
)
DATA: 1
RESULT: issueAdvice=auto
Auth=eyJhbGciOiJSUzI1NiIsImtpZCI6IjJhNmQ4NWU2ZTA1M2I4MWIzOTMxNWYwMzNmNGE2ZmI0MGU5MGM1MzcifQ.eyJpc3MiOiJhY2NvdW50cy5nb29nbGUuY29tIiwic3ViIjoiMTA1OTIzNDM2NTMxMDg1Mzg0ODU3IiwiYXpwIjoiNjk0ODkzOTc5MzI5LXFnMGkwdTg4dDBobThrNmsxbWJyYm5zdWoxMDFoNzN2LmFwcHMuZ29vZ2xldXNlcmNvbnRlbnQuY29tIiwiZW1haWwiOiJha2xzZmhzeXVpZGkzaTMzQGdtYWlsLmNvbSIsImVtYWlsX3ZlcmlmaWVkIjp0cnVlLCJhdWQiOiI2OTQ4OTM5NzkzMjktbDU5ZjNwaGw0MmV0OWNscG9vMjk2ZDhyYXFvbGpsNnAuYXBwcy5nb29nbGV1c2VyY29udGVudC5jb20iLCJpYXQiOjE0MjkwNTY0MzAsImV4cCI6MTQyOTA2MDAzMH0.GK3bUpsu8DYWZlkamRZLcY9NEwPGXbWHH40dgLdPrQKfzdm1SfMmsBnWnPWaOroU-2jhEHo8CH7Ksap13h7MC4fZLGZJc5kxAIIz-_HFwwv4balXIa0NNXbZD0pGT5WYyEKcHVoEi0_c4H4gB6P9PwyWqDN0kBIV48RbK2vvVfs
Expiry=1429060030
storeConsentRemotely=1

( ! ) Warning: gzdecode(): data error in C:\wamp\www\Snap-API\src\snapchat_agent.php on line 389
Call Stack
#	Time	Memory	Function	Location
1	0.0004	245288	{main}( )	..\exampleFunctional.php:0
2	0.0068	793728	Snapchat->login( )	..\exampleFunctional.php:19
3	3.3339	799976	SnapchatAgent->post( )	..\snapchat.php:367
4	3.3572	805160	gzdecode ( )	..\snapchat_agent.php:389

REQUEST TO: https://feelinsonice-hrd.appspot.com/loq/login
DATA: username=*username*&password=*password*&height=1280&width=720&max_video_height=640&max_video_width=480&dsig=2b3ad02b3c229d95f4db&dtoken1i=00001%3Axer2xggsQFy9DFJPAMNrn%2BSWtAt6eMjr2hPwxlukeN64%2BVSHw7SqrxGK3GS6okTB&ptoken=ie×tamp=1429056434425&req_token=9305f4526351bd886e3489e6feab2b641bea4d861998fa85d9b48914d8c51acb
RESULT: 1
My friends:
REQUEST TO: https://feelinsonice-hrd.appspot.com/loq/conversation_auth_token
DATA: username=*username*×tamp=1429056437801&conversation_id=*username*%7Enichotheman&req_token=09687913acae643506afc215b88fbf8cca6b1bce526f37da6c1da15a23b9522f
RESULT:

REQUEST TO: https://feelinsonice-hrd.appspot.com/loq/conversation_auth_token
DATA: username=*username*×tamp=1429056438012&conversation_id=*username*%7Enichotheman&req_token=096cd414832e91550600791dbb8fc64ca9661bc55249d7d54c1d4a5a23b959cf
RESULT:

You must add nichotheman to your friends list first!

Well.. Are they your friend?

Yes

I ran into a lot of gzdecode issues when running under Windows myself, not sure why.

@NichoTheMan can you tell me which is your username ? and which user are you trying to post the message?

The username is isnap.com, and the other one i've tried is nichotheman.
I'm attempting to send the message from isnap.com to nichotheman. This is what i sent to you on mail aswell. Also, no friends are being listed when logging into either of the users.
// Get your friends in an array
$friends = $snapchat->getFriends();

echo "My friends: ";
print_r($friends);

@NichoTheMan ill send you an email, ill try to help you.

@NichoTheMan send me a mail with your data, debug, etc, im going to help you now.
Regards

The fix for this is by adding some lines of code in the top of snapchat.php located in the src folder:
function gzdecode($data)
{
   return gzinflate(substr($data,10,-8));
}


PoC on providing gRPC-based APIs for OMRS

There are currently two challenges we seem to face with our JSON-based REST APIs:

The CVEs that seem to continually arise with JSON serde libraries (like Jackson).
The performance overhead of translating from object-native form into a JSON-serialized string on each request send, and then the reverse (JSON-serialized string to object-native form) on receiving the request -- and then the same again in the response (object-native form to JSON-serialized string when sending, JSON-serialized string to object-native form when receiving).

As I like to say, performance optimization is all about "avoiding unnecessary work" -- and (2) suggests that we're doing something 4 times that should only really be necessary if there are external consumers that must make use of such a JSON-form of communication because they are unable to support some other more object-native option.
At least in the realm of the OMRS APIs, we have also in the past suggested that this is not a public API interface but one that should be accessed using the provided Java clients -- therefore implying that there should not be any such external consumers of at least these APIs.
From reading up on gRPC, it sounds like in such circumstances it can provide from 2x-10x performance improvement by avoiding the overheads of (2), and may also help us address (1) by defining and enforcing a strict schema for all objects being communicated over an API (something I suspect we are doing already via Jackson due to the strict set of annotations we define in the various beans, but for which gRPC may provide a simpler development process than relying on correctly configuring these various annotations).
Given these points, I would be interested to do something like a proof-of-concept of implementing gRPC-based APIs (possibly as just a configurable option) for the OMRS APIs, to see what kind of potential performance benefits we could gain.

This is a good idea.  One advantage of the way that members register in the cohort is that they pass the connection for their client-side REST connector on their registration request.  This means a repository can unilaterally use a different protocol for their REST protocols to the other members of the cohort.  The only caveat is that the connector JAR that they reference is present in the platform of all of the other members.
The client-side connector is configured in the server's configuration document as the localRepositoryRemoteConnection in the LocalReporsitoryConfig.

Yes! +1
I am supporting any performance improvements we can do in the core especially with the fact that OCF allows us to do it in a pluggable way not affect existing deployments. @mandy-chessell Kudos the design :)


Add transferwise variant of tap-oracle

Migrated from GitLab: https://gitlab.com/meltano/hub/-/issues/143
Originally created by @DouweM on 2020-11-05 23:14:14

https://github.com/transferwise/pipelinewise-tap-oracle
https://transferwise.github.io/pipelinewise/connectors/taps/oracle.html

View 6 previous comments from the original issue on GitLab


mlterm: build fails due to missing tar file (nixos-unstable)

Describe the bug
When building system containing mlterm, the github derived file name for the source package
does not exist.
Steps To Reproduce
Steps to reproduce the behavior:

nix-build -A mlterm

or include mlterm in nixos-rebuild switch --update
Gets error:
building '/nix/store/g90wwdp19hpd8s1v6mcz75g7zk937zyb-source.drv'...

trying https://github.com/arakiken/mlterm/archive/rel-3_9_2.tar.gz
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   126  100   126    0     0    461      0 --:--:-- --:--:-- --:--:--   463
  0    14    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
curl: (22) The requested URL returned error: 404
error: cannot download source from any mirror
error: builder for '/nix/store/g90wwdp19hpd8s1v6mcz75g7zk937zyb-source.drv' failed with exit code 1
error: 1 dependencies of derivation '/nix/store/0b899pv8xyrfq65w0cs2v017pcbw4nfk-mlterm-3.9.2.drv' failed to build

Expected behavior
A sucessfull build or install
Notify maintainers
@vrthra @ramkromberg @Atemu
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
 /nix/store/jaa5l5g36aj1wwlmk9lqhslclmdqms3x-bash-interactive-5.1-p12-dev
copying path '/nix/store/jaa5l5g36aj1wwlmk9lqhslclmdqms3x-bash-interactive-5.1-p12-dev' from 'https://cache.nixos.org'...
 - system: `"x86_64-linux"`
 - host os: `Linux 5.10.93, NixOS, 22.05 (Quokka)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.5.1`
 - channels(root): `"nixos-22.05pre348581.c07b471b52b"`
 - channels(barry): `""`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Fixed by https://github.com/NixOS/nixpkgs/pull/156860
In the meantime, run nix-build https://github.com/NixOS/nixpkgs/archive/fb2b2f59c83316e727dc679f1b8b65afd692cccb.tar.gz -A mlterm.src and then your system should build again.


armpolicy -- inconsistent & case-sensitive

consider a policy definition like
{
	"id": "/providers/Microsoft.Management/managementGroups/somemg/providers/Microsoft.Authorization/policyDefinitions/A4C6C44C-3CE3-4A50-A280-513EEDCB78F2",
	"name": "A4C6C44C-3CE3-4A50-A280-513EEDCB78F2",
	"type": "Microsoft.Authorization/policyDefinitions",
	"properties": {
		"displayName": "limit Data Bricks workspaces SKUs",
		"metadata": {
			"Id": "A4C6C44C"
		},
		"parameters": {
			"effect": {
				"type": "string",
				"metadata": {
					"displayName": "Effect",
					"description": "The effect for this policy to take. e.g. Audit"
				},
				"defaultValue": "Audit",
				"allowedValues": [
					"Deny",
					"Audit",
					"Disabled"
				]
			},
			"allowedDbSku": {
				"type": "Array",
				"metadata": {
					"displayName": "SKU",
					"description": "Allowed SKUs for databricks workspaces."
				},
				"defaultValue": [
					"trial"
				],
				"allowedValues": [
					"trial",
					"standard"
				]
			}
		},
		"policyrule": {
			"if": {
				"allOf": [
					{
						"field": "type",
						"equals": "Microsoft.Databricks/workspaces"
					},
					{
						"field": "Microsoft.DataBricks/workspaces/sku.name",
						"NotIn": "[parameters('allowedDbSku')]"
					}
				]
			},
			"then": {
				"effect": "[parameters('effect')]"
			}
		}
	}
}

This is an invalid policy, since the policyrule is outside of the properties block
when you un-marshall this JSON using following go code
func parseDefinitionJSON(defJson string) (armpolicy.Definition, error) {

	defb, err := os.ReadFile(defJson)
	if err != nil {
		return armpolicy.Definition{}, errors.Wrap(err, "error while reading from file "+defJson)
	}

	// now parse the string

	var defns armpolicy.Definition

	err = defns.UnmarshalJSON(defb)
	if err != nil {
		return armpolicy.Definition{}, errors.Wrap(err, errPolicyDefnInvalidJSON.Error())
	}

	return defns, nil

}

this does not throw any error while un-marshaling.
While the service (azure policy RET API) throws an error. (using something like azure github actions)
Is there a way to validate the JSON before submitting to the API to create the policy definition?
Additionally, it seems the field names are case-sensitive in go (while un-marshalling), but the service is case-in-sensitive
Example:
{
	"id": "/providers/Microsoft.Management/managementGroups/somemg/providers/Microsoft.Authorization/policyDefinitions/A4C6C44C-3CE3-4A50-A280-513EEDCB78F2",
	"name": "A4C6C44C-3CE3-4A50-A280-513EEDCB78F2",
	"type": "Microsoft.Authorization/policyDefinitions",
	"properties": {
		"displayName": "limit Data Bricks workspaces SKUs",
		"metadata": {
			"Id": "A4C6C44C"
		},
		"parameters": {
			"effect": {
				"type": "string",
				"metadata": {
					"displayName": "Effect",
					"description": "The effect for this policy to take. e.g. Audit"
				},
				"defaultValue": "Audit",
				"allowedValues": [
					"Deny",
					"Audit",
					"Disabled"
				]
			},
			"allowedDbSku": {
				"type": "Array",
				"metadata": {
					"displayName": "SKU",
					"description": "Allowed SKUs for databricks workspaces."
				},
				"defaultValue": [
					"trial"
				],
				"allowedValues": [
					"trial",
					"standard"
				]
			},
			"policyrule": {
				"if": {
					"allOf": [
						{
							"field": "type",
							"equals": "Microsoft.Databricks/workspaces"
						},
						{
							"field": "Microsoft.DataBricks/workspaces/sku.name",
							"NotIn": "[parameters('allowedDbSku')]"
						}
					]
				},
				"then": {
					"effect": "[parameters('effect')]"
				}
			}
		}
	}
}

here policyrule is all lower case instead of policyRule. When we parse the JSON, using the SDK, the field Definition.Properties.PolicyRule is nil, but if we go ahead and create this policy, using other tools, (like azure github policy actions), this policy gets created properly.
instead if this is set as policyRule, the JSON is parsed properly and Definition.Properties.PolicyRule has the proper field value in go SDK.
The azure REST service seems to be able to cope with case-in-sensitivity, but GO SDK seems to enforce case-sensitivity.

@raych1, could you help to take a look?

Hello @vijayrajah , it's by design the unknown property is dropped off when Go sdk un-marshall a JSON string.
Both swagger and Go Sdk are case sensitive, as per the swagger definition policyRule needs to be matched in case to succeed the sdk call.

Both swagger and Go Sdk are case sensitive, as per the swagger definition policyRule needs to be matched in case to succeed the sdk call.

Then, why isn't azure github actions (with I think uses typescript), doesn't throw an error.
The issue is that this behavior, is in-consistent between the SDKs

@vijayrajah , REST api talks to service directly and service might support case insensitive. Can you share the details about github action where uses azure JS sdk?

Hello @raych1  -- This is the github actions we are using
https://github.com/Azure/manage-azure-policy

@vijayrajah  what I'm seeing in your referenced repo uses the policyRule in correct case format, e.g. here.


How to disconnect a PHP client from the router?

Hello!
SITUATION:
I'm building a calendar application and I'm trying to notify connected clients (browsers) that the next event is about to start.
WHAT I ALREADY GOT WORKING:
1 - A PHP router running in the server.
2 - Client's Javascript code using Autobahn (running in browsers) that connects to the router, subscribes and receives the notification.
3 - A PHP client that runs on the server, query the database and if there are any event that needs to be notified to a user, it connects to the router and publish the notification to the specific user.
All of this are working fine. The router must be running as a service (will probably use a supervidord here. Suggestions?) The problem is that the PHP client keeps running forever. I can't find a way to make it stop after publishing the events. I'm thinking in run it in cron like every 5 minutes.
MY CODES
ROUTER (PHP) - Runs on server
$router = new Router();
$transportProvider = new RatchetTransportProvider("<IP_ADDRESS>", 9090);
$router->addTransportProvider($transportProvider);
$router->start();

JAVASCRIPT CLIENT - Runs on browsers
<script src="https://autobahn.s3.amazonaws.com/autobahnjs/latest/autobahn.min.jgz"></script>
<script type="text/javascript" language="JavaScript">
    var connection = new autobahn.Connection({
        url: 'ws://<?= $_SERVER['HTTP_HOST'] ?>:9090/',
        realm: 'diretor'
    });
    connection.onopen = function (session) {
        // session is an instance of autobahn.Session

        function nextEvent(data) {
            console.log(data) // for simplicity
        }

        session.subscribe('diretor.agenda.1', nextEvent);
    };
    connection.open();
</script>

PHP CLIENT - Runs on server, from cron, in 5 minutes interval
$connection = new Connection([
    "url" => 'ws://<IP_ADDRESS>:9090',
    "realm" => 'diretor'
]);

$connection->on('open', function (ClientSession $session) {
    $sql = "SELECT idFunc, title, DATE_FORMAT(start, '%H:%i') as start FROM calendario_evento WHERE start > now() GROUP BY idFunc ORDER BY start";
    $rs = Yii::$app->db->createCommand($sql)->queryAll();
    foreach ($rs as $evt) {
        $session->publish('diretor.agenda.1', [$evt], [], ["acknowledge" => false])
    }

    //$session->close(); // Tried this
    //$session->shutdown(); // this also
    //$session->getLoop()->stop(); // this also (breaks the notification, it's not delivered)
});

$connection->open();

Any ideas on how to make the PHP client run and after the notifications are sent, just quit?
Or maybe another approach for this cron job?
NOTES:
1 - For simplicity in the example I used a fixed topic but in the application every user will have their own topic for receiving notifications for their events.
2 - Yeah, I'm using Yii 2 so the router and the php client are console applications.

Modify your client code to (have not run this - but it is the general idea):
$connection = new Connection([
    "url" => 'ws://<IP_ADDRESS>:9090',
    "realm" => 'diretor'
]);

$connection->on('open', function (ClientSession $session) use ($connection) { // we are going to need this
    $sql = "SELECT idFunc, title, DATE_FORMAT(start, '%H:%i') as start FROM calendario_evento WHERE start > now() GROUP BY idFunc ORDER BY start";
    $rs = Yii::$app->db->createCommand($sql)->queryAll();
    foreach ($rs as $evt) {
        // set acknowledge to true so we know it sent before shutting down
        $session->publish('diretor.agenda.1', [$evt], [], ["acknowledge" => true])->then(function () use ($connection) {
            $connection->close(); // shutdown after successful publish
        }, function () use ($session) {
            $connection->close(); // shutdown after unsuccessful publish
        });
    }

    // running this stuff here would most likely kill your connection prior to it being able to publish
    //$session->close(); // Tried this
    //$session->shutdown(); // this also
    //$session->getLoop()->stop(); // this also (breaks the notification, it's not delivered)
});

$connection->open();

That worked! Thank you so much.
But I had to move the $connection->close() to another place since there can be more than one event beeing published inside the foreach loop.
$connection->on('open', function (ClientSession $session) use ($connection) { // we are going to need this
    $sql = "SELECT idFunc, title, DATE_FORMAT(start, '%H:%i') as start FROM calendario_evento WHERE start > now() GROUP BY idFunc ORDER BY start";
    $rs = Yii::$app->db->createCommand($sql)->queryAll();
    foreach ($rs as $evt) {
        // set acknowledge to true so we know it sent before shutting down
        $session->publish('diretor.agenda.1', [$evt], [], ["acknowledge" => true])->then(function () use ($connection) {
            //$connection->close(); // shutdown after successful publish - Do it outside foreach
        }, function () use ($session) {
            //$connection->close(); // shutdown after unsuccessful publish - Do it outside foreach
        });
    }

    $connection->close(); // <--- This works!

    // running this stuff here would most likely kill your connection prior to it being able to publish
    //$session->close(); // Tried this
    //$session->shutdown(); // this also
    //$session->getLoop()->stop(); // this also (breaks the notification, it's not delivered)
});

It is working now, but do you think the approach is a good one? What would you suggest to improve it?

Instead of using cron, you might want to look at using a periodic timer and keep the client running.
You can read more about timers here http://blog.wyrihaximus.net/2015/01/reactphp-timers/

Thanks @davidwdan.
And what do you guys like to use to keep PHP scripts running (if you need to)?
Besides keeping the router running, if I use a periodic timer I will need to keep the client running too.

Your best bet would be to use supervisor.

Thank you all for your help!


Remove CipherSuite

The CipherSuite trait creates some overhead for consumers, even if it is small, to me the trade-off seems to be between user-facing API and internal implementation. Personally I prefer the user-facing API.
This is very subjective and I actually just created this as an experiment, tell me what you think.
As an example: hpke does the same thing.

Interesting, I like it and agree! I think as long as for the consumer-facing API, the generics are always specified as <Group, Hash> together (as opposed to some functions being just  or ), then you are correct in that this is easier for the user-facing API.
What is the reason for removing the Hash trait defined in hash.rs and just doing BlockInput + Digest everywhere? I would think that it would be more convenient to use Hash -- is there a downside?

I think as long as for the consumer-facing API, the generics are always specified as <Group, Hash> together (as opposed to some functions being just or ), then you are correct in that this is easier for the user-facing API.

I do that everywhere except NonVerifiableClientFinalizeResult and VerifiableClientFinalizeResult, which contain nothing but a GenericArray. I can't hook up Group without introducing a PhantomData, which I'm afraid will be a bit awkward considering this structs field is pub. I think it's okay, considering these types could basically be removed in favor of just returning straight GenericArrays. I could actually do that if you approve.

What is the reason for removing the Hash trait defined in hash.rs and just doing BlockInput + Digest everywhere?

No particular reason, I prefer not to implement supertraits that don't do anything, one less API the user has to look up, I felt it was in the same spirit. Tell me if you want it back.

I think it's okay, considering these types could basically be removed in favor of just returning straight GenericArrays. I could actually do that if you approve.

I see, yes, good point, let's just return GenericArray and remove the type in that case. I'll await for this change before merging. Thanks!

No particular reason, I prefer not to implement supertraits that don't do anything, one less API the user has to look up, I felt it was in the same spirit. Tell me if you want it back.

Fair enough!


Update README.md

.NET lib section added with some items and Nodify properly relocated.

super cool projects 💯


OnBeforeGetSourceDocumentsRun Event in Codeunit 5751 "Get Source Doc. Inbound" has the wrong position

The OnBeforeGetSourceDocumentsRun Event in Codeunit 5751 "Get Source Doc. Inbound" has the wrong position in my opinion. It is run after the GetSourceDocuments.RunModal. Can you fix this?

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Automated cherry pick of #6020: fix: opslog FetchOwnerId fail

Cherry pick of #6020 on release/3.2.
#6020: fix: opslog FetchOwnerId fail

Codecov Report

Merging #6021 into release/3.2 will decrease coverage by 0.00%.
The diff coverage is 0.42%.

@@               Coverage Diff               @@
##           release/3.2    #6021      +/-   ##
===============================================
- Coverage         7.02%    7.02%   -0.01%
===============================================
  Files              643      644       +1
  Lines           110333   110429      +96
===============================================
  Hits              7756     7756
- Misses          101978   102074      +96
  Partials           599      599

Flag
Coverage Δ

#aFlag
7.02% <0.42%> (-0.01%)
:arrow_down:

Impacted Files
Coverage Δ

pkg/cloudcommon/db/enabledstatusdomainresource.go
0.00% <ø> (ø)

pkg/cloudcommon/db/enabledstatusinfrasresource.go
0.00% <ø> (ø)

pkg/cloudcommon/db/enabledstatusstandalone.go
0.00% <ø> (ø)

pkg/cloudcommon/db/fetch.go
0.00% <0.00%> (ø)

pkg/cloudcommon/db/infraresource.go
0.00% <0.00%> (ø)

pkg/cloudcommon/db/namevalidator.go
0.00% <ø> (ø)

pkg/cloudcommon/db/opslog.go
1.50% <0.00%> (-0.16%)
:arrow_down:

pkg/cloudcommon/db/owner.go
7.40% <0.00%> (-2.60%)
:arrow_down:

pkg/cloudcommon/db/sharablebase.go
7.33% <0.00%> (+0.02%)
:arrow_up:

pkg/cloudcommon/db/sharablevirtual.go
0.00% <0.00%> (ø)

... and 45 more

Continue to review full report at Codecov.

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


discovery/dns: add test case for SDConfig.UnmarshalYAML.

No test code coverage:
https://github.com/prometheus/prometheus/blob/4f47806a7d33507dc9ff89c96cc41c91d3fd1d77/discovery/dns/dns.go#L74-L94

Thanks! A few nits otherwise LGTM.

updated.

updated.

Thanks!


Physical interface hasn't been recovered back to original state

Description of problem
As far as I know, kata tries to pass through physical interface into vm for better performance by using vfio, in that case when the container teardown kata would bind the physical interface to previous driver except for the network configuration (route, ipaddr, namespace and so on).
In some cases, the network framework (cni/cnm) would tear down the interface no matter what condition they are under, however, for better compatibility with runc, I think it would be better if we recover the physical interface totally to original state.
This issue wouldn't be just physical interface related as well.
Expected result
Actual result

kata-runtime  : 1.1.0
commit   : 5aed178ee99298df9e3f17eeaa31a405a1da38d7-dirty
OCI specs: 1.0.1

cc @amshinde  @egernst  @mcastelino @bergwolf @sboeuf

@devimc in the case of virtual interfaces they are normally deleted. In the case of the physical interfaces you do have a point. We have cases here.

If the physical device was not reprogrammed, but just moved into the namespace. So its network configuration is inherited from the host.
If the physical device network configuration was generated by the IPAM and is namespace specific.

As the network interfaces are moved back into the hostname space on teardown, the recovery of the original information is required only in the first case. However we have not seen such a plugin yet.

@mcastelino thanks for replying. I didn't catch why recovery isn't required in the second case, which is exactly the situation our plugin has trouble with (We rely on detecting interfaces within network namespace). However, it sure can be worked around, I don't think it's a bad idea if we make kata more runc-compatible.

@linxiulei The recovery is needed to bind it back to the host. It is the network parameter recovery I was talking about.  So https://github.com/kata-containers/runtime/issues/612 is a bug in our implementation and we should fix it. We just assumed all devices that fit the second case are PCI devices which is not true.

@mcastelino I was talking  about network parameters too (namespace, ip addr, routes and so on). As I mentioned, physical interfaces were bind back to host namespace and the previous driver without any network configuration. What the recovery I want is also set those interfaces into the container network namespace


ASP.NET Core 2.1 - Identity Pages Override

While overwriting Identity Pages on Visual Studio via Add > New Scaffolded Items > Identity, if the "Overwrite all files" is ticked, all Identity Pages (Accounts, Manage) are overwritten. This is the intended behaviour.
In addition to that, the default CSS, JS and image files (site.css, site.js, banner*.svg) also gets overwritten. That surely doesn't look like the intended behaviour.

@javiercn / @vijayrkn  - any thoughts on this?

@seancpeters As he's the owner of the scaffolders. It sounds like a bug to me

The identity scaffolder is designed to check for a wwwroot directory, and only put content there if the directory does not exist or is empty. Overwriting the css, js, etc. in the wwwroot directory is not intended, so yes, this is a bug.

This issue was moved to aspnet/Scaffolding#844


Create a sample app that exposes a GRPC service

Sample app available on grpc-spanner-sample branch.
Putting this issue on hold for now.

Leaving the sample app in a branch and closing this for now.


THRIFT-3686 Java processor should report internal error on uncaught exception

This should be rebased against upstream/master since there are conflicts, and resubmitted; also see THRIFT-1805 as there is some discussion on what is okay and what is not in this context.

@nsuke This should be rebased against master since there are conflicts, and resubmitted; also see THRIFT-1805 as there is some discussion on what is okay and what is not in this context.


OSGI 5.5.1 project fails due to clash with Felix Framework.

During upgrade to the latest version of Dozer, we have some compatibility problems with one of the linked bundles.
It would seem that the package is referencing it's own OSGI framework version:

org.apache.felix
org.apache.felix.framework
4.0.3
provided

and here:
<import.packages>
org.dozer.*,
org.osgi.framework,
Although I can't determine if this is the problem, when we load a class via the bundle context it causes the following linkage error:
java.lang.LinkageError: loader constraint violation: loader (instance of org/apa
che/felix/framework/BundleWiringImpl$BundleClassLoaderJava5) previously initiate
d loading for a different type with name "org/osgi/framework/BundleContext"
at java.lang.ClassLoader.defineClass1(Native Method)
at java.lang.ClassLoader.defineClass(ClassLoader.java:792)
at org.apache.felix.framework.BundleWiringImpl$BundleClassLoader.findCla
ss(BundleWiringImpl.java:2228)
at org.apache.felix.framework.BundleWiringImpl.findClassOrResourceByDele
gation(BundleWiringImpl.java:1472)
at org.apache.felix.framework.BundleWiringImpl.access$400(BundleWiringIm
pl.java:75)
at org.apache.felix.framework.BundleWiringImpl$BundleClassLoader.loadCla
ss(BundleWiringImpl.java:1923)
This is running in Adobe CQ5.6.1.
The code is roughly as follows:
BundleContext bundleContext = componentContext.getBundleContext();
//Fails here:
ServiceReference ref = bundleContext.getServiceReference(MyServiceClass.class.getName());
I'd thought that a jar/bundle shouldn't really specify it's own framework version and should just "go with the flow", I may wrong here, however this just doesn't seem right to me.
Could you take a look at the problem for me, in the meantime we will have to downgrade back to 5.4.0
Many thanks
Pete

@inthegarage ; is this still a problem?

I will see if I can check this . May take a day or two.

This should be fixed in master now. #348 fixes the issue when OSGi API and Felix classes were embedded into dozer-osgi bundle. Now there is only one OSGi-ready artifact dozer, which contains only Dozer binaries.

@orange-buffalo Many thanks for that. unfortunately I'm not able to re-test this situation as I do not have the environment any-more.

@inthegarage thanks for the feedback. As there is no way to check the problem and its rootcause is fixed, don't you mind closing this issue?

Yes, Issue closed.

Thanks!


fix: Memory leak

🔗 Linked issue
https://github.com/nuxt-modules/i18n/issues/2629
This is just an example, no need to merge it. Because of this fix, everything works 10 times slower. We need a proper fix.
❓ Type of change

[ ] 📖 Documentation (updates to the documentation or readme)
[X] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📝 Checklist

[x] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

the existing approach where vueI18nOptions.messages is passed to loadInitialMessages and then both modified inside and returned feel unnecessarily confusing
if the problem is that loadInitialMessages sets something on messages that blocks it from being collected then maybe it should itself take care of not doing that (by assigning stringified/parsed data maybe).

the existing approach where vueI18nOptions.messages is passed to loadInitialMessages and then both modified inside and returned feels unnecessarily confusing
if the problem is that loadInitialMessages sets something on messages that blocks it from being collected then maybe it should itself take care of not doing that (by assigning stringified/parsed data maybe).

The situation is more complicated here. I tried the approach you described first, but the issue is something else, which I haven't identified yet. For now, this is the only working method to detect the problem. I created the pull request just as an example to help solve the leak issue.

Thanks for your continued investigation of this leak @s00d! First testing of this change seems promising, but I will have a more thorough look later.
I agree with @rchl, but as it is, this PR is helpful in proving that this may be the source of the memory leak. I'll test it further and see if I can refactor/simplify the relevant functions.

 seems the entire problem lies here: sometimes deepCopy malfunctions and returns true for objects and arrays. After that, the garbage collector can't clean up the data because it's being copied by reference into the objects. No wonder there were comments about deepCopy.


Bug in c# code

In c# code in AStarPathfinder.SortedAddToOpenList() field "inserted" is always "false".

cc/ @ScaryG

Second bug in C#.
In "GetSuccessors(...)" method instead
NodePosition parentPos = new NodePosition(0, 0);
we should use
NodePosition parentPos = new NodePosition(-1, -1);
Because (0, 0) is a real position on the map, and we can not use (0, 0) as position of nonexistent parent.

Thank you @gorodovoy
If you fix the other one we can close the issue :)

Moved cs to a new repo, see readme. Closing this issue


Ask for confirmation before auto-deleting an invalid polygon

The editor should warn you that if you unselect this polygon, it will be destroyed since it's invalid. But you will also be given the option to cancel to keep your invalid polygon selected so you can fix it.

Unselecting this polygon will remove it because it is invalid. Are you fine with that?
[Cancel] [OK]

More context: WMGX on Discord

Maybe I can help, where is the file responsible for this?

Gladly!
Here is the code that handles mouse events and updates the selection based on the selection box.
https://github.com/getkey/ble/blob/34ae3a3f8a58f13ee7eacbfbc1b5c680d1dd919d/src/epics/select.ts#L223
What I would do:

for each element that's in the old selection but not in the new one
if one of them is invalid (entity.params.isValid is false)
show a popup (alert() is fine)
if Ok -> continue normally
if Cancel, use store.undoManager.stopGroup(); store.undoManager.undo(); to revert to the old selection box


Support anchorage migration

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

Comparison is base (66039f7) 32.95% compared to head (f0100eb) 30.62%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##           develop      #98      +/-   ##
===========================================
- Coverage    32.95%   30.62%   -2.34%
===========================================
  Files          151       64      -87
  Lines         6891     3798    -3093
  Branches      1138      749     -389
===========================================
- Hits          2271     1163    -1108
+ Misses        4468     2560    -1908
+ Partials       152       75      -77

Flag
Coverage Δ

cannon-go-tests
?

chain-mon-tests
27.14% <ø> (ø)

common-ts-tests
26.74% <ø> (ø)

contracts-bedrock-tests
?

contracts-ts-tests
12.25% <ø> (ø)

core-utils-tests
44.03% <ø> (?)

sdk-next-tests
39.16% <ø> (ø)

sdk-tests
39.16% <ø> (ø)

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


Add troubleshooting entry to documentation about surprising Terraform behavior

As discussed in https://github.com/open-policy-agent/opa/issues/5619. Let me know if any changes to wording, formatting, or placement are necessary.

Thanks @nafarlee, could you do the DCO sign-off thing?  https://www.openpolicyagent.org/docs/latest/contrib-code/#developer-certificate-of-origin

Thanks @nafarlee, could you do the DCO sign-off thing? https://www.openpolicyagent.org/docs/latest/contrib-code/#developer-certificate-of-origin

Ah, my apologies. I will get that done right away.


feat: use tsup for bundling

This PR is just an example of how we can use tsup to take care of bundling. I personally love tsup and many projects make use of it. -> https://tsup.egoist.dev/
bundling takes less than 1 second on my machine:
CLI Building entry: index.ts
CLI Using tsconfig: tsconfig.json
CLI tsup v6.5.0
CLI Using tsup config: ./micro-btc-signer/tsup.config.ts
CLI Target: node16
CLI Cleaning output folder
CJS Build start
ESM Build start
DTS Build start
CJS dist/index.cjs 50.92 KB
CJS ⚡ Build success in 210ms
ESM dist/index.js 47.46 KB
ESM ⚡ Build success in 210ms
DTS ⚡ Build success in 662ms
DTS dist/index.d.ts 60.88 KB

Process finished with exit code 0

I see it's faster. What are the other benefits? We are bringing at least 89 packages with it.

yeah, true, there are dependencies of it we bring along. I'd say the benefits are you don't have to roll your own rollup/esbuild setup. there are also benefits around bundle size, minification, etc. you can recreate it by constructing a build step by hand, it all depends on the needs.

89 new deps is a no-go, since we're moving towards supply chain security


Colon instead of localhost

When I type npm start it shows no localhost but a colon:
Listening at http://:::6000

Hey what was the fix for this?