VGM playback & sound effects ?

Hello o/
I was wondering if it was possible with your sound driver to play a vgm & some sound effects at the same time?
Reserving maybe 1 or 2 channels to overlap the song.
I have already made a HuC tetris clone demo but no official sound driver is available so far for this SDK.
Would it be complex to switch from pure HuC to your SDK?
Btw, what soft could I use to produce the vgm, Deflemask is ok? ( + using your utility to modify the header )
Cheers
PS :
the actual demo I would like to port:
https://lunoka.itch.io/tetraminos-pce

The VGM replay is very simple. See it as a PSG register stream. It will not be that hard to add sound effects. Unfortunately it's not supported by the current replay routine.
About switching to HuC. I don't think it'll be that hard but it'll need some work.
If you only need vgm replay, I can try to make it HuC friendly so that you can use it without moving to HuDK.
You can use Deflemask or Furnace to produce VGM songs. I used it on some of my old demoscene stuffs.
Note that the vgm_strip tool is very simple. Moreover, VGM data can be pretty big. It may be possible to save some space with smarter encoding and some pattern matching.

thank you very much for your answer.
To give some context, I'm into retrodev just for months now. I may have missed the golden age where people were very active on making homebrews on PCE. I've started with the SGDK from Stephane D. and wanted to port some stuff on other plateforms like PCE or SNes.
The only thing I was able to implement in my demo is a very simple sound playback with 1 channel based on a 20 year old tutorial from JyCet, making the PCE sounds like a ... Atari 2600 haha
I agree with you on the VGM size, I've used a simple midi to vgm conversion to add a playback on my SGDK version and the vgm is weighting 256 ko... it's ok for a simple demo but quite problematic for a bigger project with 10+ songs.
I know about Squirrel but I have to switch to MML and gosh ... it's troubling ( + no SFX either and only for free projects ).
I wasn't able to build with the japanese Husic driver, maybe I missed something with embedding the song in asm through PCEAS but I'm very new to this, although this driver seems to be able to play music & sfx ( assuming the xpcm driver is for the sfx part ),
It's very kind of you to propose the HuC integration but I wouldn't want you to work just for me ^^; especially if I don't dig further on the PCE side. The PCE is quite underrated and It must be difficult to work on tools for this platform if no one or few ask for it. I haven't found any discord related server on PCE dev yet.
By the way, I don't really understand what is the HES format. Is it a VGM based format? I tried an export from Deflemask and it just doesn't play at all on my winamp plugin.
Cheers

I don't mind adding more features to the vgm replay. If that means more PCE homebrew, it's totally worth it.

Awesome, thank you. I think a public release of a working sound driver compatible with HuC would be great for everybody into PCE stuff.

I have plans to do the same kind of work for Furnace file format but I don't know when...

Very interesting, the Furnace file would be directly added to the rom image and played from scratch without any conversion? Software is very promising and the main dev doesn't seem to sleep considering his updates rate ^^;
Btw, actually my current demo was made with the Elmer's fork of HuC  :
https://github.com/jbrandwood/huc
Is it ok or would I have to switch to Turboxray's fork if you were to modify your sound driver?
Cheers

Some conversion tool will be required at some point. I'll have to take a look at Furnace file format someday :D
I think Elmer's fork is the most up to date.
The sound driver will be in assembly with the appropriate HuC interface. It's not a lot of work.
I think it'll move to its own project. This way people won't have to choose between HuDK or magic kit.

Awesome news , thank you!
Btw, I'm also looking forward to the mod/xm import into Furnace that seems to be on the roadmap, since I have some stuff available from my jurassic Amiga period \o/

Hello o/
I've noticed your vgm player was quite isolated from the rest of your sdk already, so I've tried to extract it and import it in a simple test with HuC env ( last version from Elmer 02/2022 ). I've built the vgm_stripper aside and succeeded in building a rom with a tune I made with Deflemask ( thanks to TOUKO with the assembly part, I wasn't aware of the double underscore notation change in HuC 3.99 that were failing PCEAS on the vgm.s ). Unfortunately, I have some issues in the playback. Some cuts in the tempo and the volume goes to zero after a moment, here is a link to the vgm I've made just in case :
https://www.lunoka.net/files/song.vgm
It's possible I've missed something on my side. I've seen you were linking the vgm_update to the vbl between 2  disable/enable interupts in your example whereas I just call vgm_update() right after vbsync() in a while(1) loop in HuC.
And btw, is it possible to make the vgm loop ?
Cheers

It's possible but I think the current converter has a bug.
I'm nearly done with the updated version. I'm currently working on the sound fx part.
I'll push it this week (fingers crossed).
main.pce.zip

Awesome news! I'm looking forward to it!

and great work on the fixed playback in the rom :D, I think I'm confident now in getting more involved with Deflemask compo now I know it will be playable on my pce (as soon as I receive my flashcard )!

Here's the updated vim driver.
The vgm_strip program now accepts a list of songs and subtracks. For example
vgm_strip -s song00.vgm -s song01.vgm -t 1 fx00.vgm -t 0 fx00.vgm -t 0 fx01.vgm output_directory

-s file.vgm adds a vgm song
-t ch file.vgm add channel ch of a vgm song as a subtract

The C API is like:
void vgm_play_song(char id);

Play song id.
void vgm_stop_song();

Stop playing song.
void vgm_play_subtrack(char ch, char id);

Play subtrack id on channel ch.
void vgm_stop_subtrack(char ch);

Stops subtrack on channel ch.
void vgm_update();

Update vgm driver. It must be called every vsync.
The pce/examplefolder contains sample programs in ASM and C.
vgm_driver.zip

Thank you very much MooZ! I will try to implement your driver into my current tetris demo this week-end.

I'm trying to compile the new vgm stripper. If I don't use -Wall -Werror flags, the exe is generated but I have this message when using the tool:
failed to expand track data
If I use -Wall -Werror, I can't compile and fail on that issue:

vgm_strip.c: In function 'output_wav':
vgm_strip.c:460:49: error: unknown conversion type character 'z' in format [-Werror=format=]
fprintf(state->stream, "\n    .dw wav%02zx", i);

Any idea on what I'm doing wrong?

You are not doing anything wrong. It looks like a compiler version issue. I used clang and it looks like yours doesn't support 'z' format. Replace 'zx' by 'lx' and you'll be good.

I'll replace the current HuDK vgm driver with this one later.
vgm_driver.zip


[CI | FIX] Improve and run Fisher-MC tests on main

I noticed that the Monte-Carlo tests were not running anymore because we switched from master to main.
This PR re-activates them, slightly refactors them to make them faster, and adapts their tolerances to account for cases we added but never tested.

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

0 of 4 (0.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.2%) to 88.786%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

curvlinops/examples/utils.py
0
4
0.0%

Files with Coverage Reduction
New Missed Lines
%

curvlinops/examples/utils.py
1
31.25%

Totals

Change from base Build<PHONE_NUMBER>2:
-0.2%

Covered Lines:
1346

Relevant Lines:
1516

💛  - Coveralls


Redesign blog

Now we have a awesome logo #2717
Detekt has maintained a blue blog for a long time, but I think we need to consider changing it to harmony with logo.
The blog is the origin of this issue, but the context is branding.

see https://github.com/detekt/detekt/issues/4235 for some updates 🎉

Thoughts on the new site design?
I think this one can now be closed, thanks to the excellent work @cortinico has done!

I think it is very beautiful. If anyone has a different opinion, please create a new issue.


driver: timer: power: npcx: Add system timer and power management support in npcx series.

This PR is separated into three CLs.
This first CL introduces a kernel device driver implemented by the internal
64/32-bit timers in Nuvoton NPCX series. Via these two kinds of timer,
the driver provides a standard "system clock driver" interface.
The second CL introduces power management driver that improves the efficiency
of ec operation by adjusting the chip’s power consumption to the level
of activity required by the application in npcx series.
And this CL implements one power state, PM_STATE_SUSPEND_TO_IDLE, with
two sub-states for Zephyr power management system.
Sub-state 0 - "Deep Sleep" mode with “Instant” wake-up if residency time
is greater or equal to 1 ms
Sub-state 1 - "Deep Sleep" mode with "Standard" wake-up if residency time
is greater or equal to 201 ms
The third CL introduces power management device support in uart device.
Athought the uart device clock will be stopped after ec entered sleep or
deep sleep state and restore its clock automatically (there is no need
to implement code for suspending and resuming devices manually.),
we still need to check the current status of uart device when it
wants to change its power state to LOW or SUSPEND power state. It is
crucial to forbid ec enters sleep or deep sleep state when uart device is
busy with transmitting data. Or we will observe broken characters on the
uart console.

Digging the build error log further, I found pm_policy_low_power_devices() is not implemented in tests/kernel/profiling/profiling_api test suite. (This test suite enabled CONFIG_PM_POLICY_APP but has no default pm_policy_low_power_devices() function. I thought this function should be added in the test suite, not the pm driver itself.

Digging the build error log further, I found pm_policy_low_power_devices() is not implemented in tests/kernel/profiling/profiling_api test suite. (This test suite enabled CONFIG_PM_POLICY_APP but has no default pm_policy_low_power_devices() function. I thought this function should be added in the test suite, not the pm driver itself.

again I prefer lower-case hex

Done.

Hi @yperess and @keith-zephyr :
I have finished most of the requests and left few open items for further discussion. Thanks for the effort of reviewing this.

Digging the build error log further, I found pm_policy_low_power_devices() is not implemented in tests/kernel/profiling/profiling_api test suite. (This test suite enabled CONFIG_PM_POLICY_APP but has no default pm_policy_low_power_devices() function. I thought this function should be added in the test suite, not the pm driver itself.

A new CL  is added in this PR for the build issue of tests/kernel/profiling/profiling_api test suite in case buildkite/zephyr blocks this PR.

Rebasing since PR31578


Add truncate function

Merged into 3617530.

Coverage remained the same at 100.0% when pulling 1354e2fa88c3d880ba9c7a1151385fb3f238db4f on fonkadelic:truncate into ebe046707a7e5325d13eaa627baef290171409a5 on kylef:master.


how to use more controller with GetBuilder

**Reproduction code
GetPage(
        name: 'IndexPage',
        page: () => IndexPage(),
        transition: Transition.rightToLeft,
        binding: BindingsBuilder(() {
          Get.put(AController());
          Get.put(BController());
          ...more...
        })),

 GetBuilder<AController>(
                init: AController(),

how to use BController() and more

Above GetBuilder (at the beginning of the page), you will have to find the other controller instances:
final controllerB = Get.find<BController>();
final controllerC = Get.find<CController>();

Above GetBuilder (at the beginning of the page), you will have to find the other controller instances:
final controllerB = Get.find<BController>();
final controllerC = Get.find<CController>();

i hope use GetBuilder<controllerB ,controllerC,...> ^^

in controllerA
BController get b => Get.find<BController>();

controller.b....

in controllerA
BController get b => Get.find<BController>();

controller.b....

thanks


Lighting issue with Enchantment Library

Minecraft v1.16.5
Forge v36.2.4
Apotheosis v4.8.1
When the enchantment library is facing east/west, it produces an odd shadow on its side, which is more easily visible if multiple are placed together. It doesn't have this problem when facing north/south.

what in the world

¯_(ツ)_/¯
Forgot to mention this was on a server, but probably unrelated.


New norway

Hi I took the liberty of adding two more shape files for norway, the borders for both municipalaties and counties has changed

Hei, The map for Norway is missing a number of small municipalates such as Utsira, Kvitsøy, Fedje, Giska, Træna, Røst and Værøy. Information about these municipalaties communes is present, but there are no lines and, accordingly, the type of line is not correct: "type:null". Can you fix the map so that these municipalities will be also available?


impl From<NestedCondition> for Condition + validation + doc

Noticed that the new NestedConditions haven't had the same convenience treatment yet.
I've also added a recursive check for the unsupported has_id conditions in the filters; not sure how effective it will be, given that the initialiser is public, but if one uses only the convenience methods, it should work.
Because it is recursive, and thus potentially costly, it's only a debug_assert (although it's hard to imagine how massive the filters should become for it to be a performance toll). An alternative could be to impl TryFrom instead, but this wouldn't be semantically sound, as incorrect NestedConditions shouldn't themselves be constructible in the first place, not Conditions on top of those.

Looks good to me. The final validation will be handled on the server side anyway, so doing client-side validation on a best-effort basis should be acceptable here.

The reason to introduce the validation on the client side was that (at least due to my quick observations) there's no specific error message that the server sends to tell the user that the query is incorrect (and which condition is incorrect), it just returns an empty SearchResponse, which I thought would make it hard to debug: it behaves like any other unsatisfiable query without being obviously wrong. A panic here would much more likely point the user to look at the specific docs.


server: add version monitoring

@yichengq @philips
First step:
Members are periodically checking their internal store for changes to the underlying key (/_etcd/internal-version). The master waits 10sec to wait for all members to append the entry and then exit itself.
changes: internal-version -> next-internal-version

lgtm


fix(popover): reduced space between/around elements

Part of https://github.com/patternfly/patternfly-next/issues/2653. This is a breaking change so shouldn't be merged to master.

Preview: https://patternfly-next-pr-2762.surge.sh


Remix documentation severely lacking

Incorporating Clerk into Remix has been super frustrating, as far as I can tell, the docs and this repo is completely outdated and not caught up to all the breaking changes in the latest @clerk/remix package. Please let me know if I'm mistaken, but honestly have been pretty disappointed at our experience implementing Clerk when it's supposed to be designed for developers
Here are all the things I see that are wrong:

This section in the docs makes it seem like you can retrieve user from getAuth() as long as you turn on loadUser = true, and the return type of getAuth also suggests it should be possible to do so, but as far as I can tell, this is totally wrong. The only hint I had of this after hours of debugging was checking this link, which was hidden away in that page. It seems from that page, the only three valid things that can be returned from getAuth are: userId, sessionId, getToken

Data fetching is importing from experimental. Additional source of confusion – am I supposed to be using this? Is it unstable? Why is it the only way to perform an action in a major stable release?

In the breaking changes page, it says you can't do import { users } from<EMAIL_ADDRESS>anymore in 2.x, but this repo, which is using 2.x, does use that import? https://github.com/clerkinc/remix-auth-starter/blob/main/app/routes/ssr-user-demo.tsx#L6

Am I the only one running into these issues?

Am I the only one running into these issues?

No you are not - I completely agree!
I just spent the last several hours trying to get Clerk to work with supabase RLS + SSR for an MVP project and my only conclusion is that I'm strongly reconsidering using Clerk. Which is sad because I was 100% sold on the promise that Clerk is making.
If the idea is to cater to developers, then I strongly suggest the clerk team either updates their example repos or archives them. Nothing frustrates us more than a SaaS product that promises to make things easier but actually ends up wasting time.

Thank you both for bringing this to our attention. Clerk is operating, by definition, in a fast-changing environment. Keeping up to date with all framework changes and applying them to our starter repos is challenging. For now, I advise using our official starter guides in our docs. If you face further integration issues while using Clerk, please talk to us.


issue with button

I want you to build 2 buttons on my website, first button will be "Connect Wallet" to connect Phantom wallet or sollet.io wallet to the website, after connection, a button will appear at bottom of the page named "Send transaction" that will send 0.05 sol to our address after user's confirmation. but there is an issue with the code I found on your account it's not working.
please reply to me ASAP. I am looking forward to your positive response. Thankx

You're posting in the wrong place, my friend.

sorry
[image: GeoTrack]
https://geotrack.email/?utm_source=gmail&utm_medium=signature&utm_campaign=viralSignature
Sender
notified with GeoTrack
https://geotrack.email/?utm_source=gmail&utm_medium=signature&utm_campaign=viralSignature
[image: 44]
On Mon, Sep 27, 2021 at 12:10 PM Jordan Sexton @.***>
wrote:

Closed #98 https://github.com/solana-labs/wallet-adapter/issues/98.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/solana-labs/wallet-adapter/issues/98#event-5363399553,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVYTSHNZQVSXWST4WUHOH5LUEAKGLANCNFSM5EZ2QUNA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


@alloy => warning for short passwords, invalid emails on create account

Adds a warning banner if either the password is too short (<6 chars) or the email fails a very simple validation check (no "@").
I've modified the ARWarningView class so that the warning icon is tinted to match the view's textColor, this way we can change the banner color and the icon and text will always match.
I've also replaced the AttentionIcon resource with a version that is opaque, should we need semi-transparency (we don't currently), we can set it in code.
Closes #238.

What @orta said, otherwise this looks good, I think that test failure is a known flaky one.

Looks good, let me know when you get in tomorrow and I'll jam with you on some tests :+1:

I think this is good to go @alloy

Looks great :clap:
Have you tested this extensively and on a device? If so, add the :iphone: emoji to the title.

All set @alloy

Perfect, thanks!


How to share my printer from host to docker image?

I deployed my GUI app on docker image, How to share my printer from host to docker image? Thanks very much

Hi, @BinhuaLiu, the GitHub issue tracker is intended for tracking bugs, feature requests and pull requests, and not really for questions.
Could you ask this question on https://forums.docker.com/, https://groups.google.com/forum/#!forum/docker-user or the #docker IRC channel on Freenode?
I'm going to close this issue, because it's not a bug. (You'll still be able to comment on this issue after it's closed), thanks!


Spaces in title cause incorrect redirect

https://github.com/github/gollum/blob/master/lib/gollum/frontend/app.rb line 80 does not redirect to the cname of the wiki page, So, for example, if I create a page titled Foo Bar, the page is created as the filename Foo-Bar.ext but the redirection is to '/Foo+Bar' which results in another page create form.
I would imagine that a fix would be trivial, but was wondering if I was doing anything wrong before I submit a patch for this?

Hitting this as well. You can see the problem if you try and edit a page that has spaces in the name.

Looks like a bug to me.

Yep. It is. Ran into the same problem.

It seems, this issue is a duplicate of issue 220: https://github.com/github/gollum/issues/220

@paschelino Thanks for pointing that out. Since that bug repors more detailed I'll close this one and watch the other.


Google Collab error

Hello,
I have a hard time trying to make the Google Colab works.
encoder_model = torch.load(vggface_modelpath)
Give this error:
FileNotFoundError                         Traceback (most recent call last)
[<ipython-input-6-f7e8409fb9ee>](https://localhost:8080/#) in <cell line: 1>()
----> 1 encoder_model = torch.load(vggface_modelpath)
      2 ann_index = AnnoyIndex(2048, 'angular')
      3 _ = ann_index.load(celeb_index_annpath)

2 frames
[/usr/local/lib/python3.10/dist-packages/torch/serialization.py](https://localhost:8080/#) in __init__(self, name, mode)
    424 class _open_file(_opener):
    425     def __init__(self, name, mode):
--> 426         super().__init__(open(name, mode))
    427
    428     def __exit__(self, *args):

FileNotFoundError: [Errno 2] No such file or directory: 'celeb_detector/models/vggface_resnet50.pt'

I can't find this file anywhere on the the repo. I need to download it somewhere?
Thanks!

Thanks for trying out the repo.
I have uploaded the model files to my huggingface repo: https://huggingface.co/resnet151/celeb_detector.
You can install everything using the following commands:
git lfs install
git clone https://huggingface.co/resnet151/celeb_detector
cd celeb_detector
pip install -e .

You should be able to use the colab notebook after that by pointing to the correct model path.
Best way to run this would be in python by directly importing the package:
import cv2
import celeb_detector
out = celeb_detector.get_celebrity(cv2.imread("/path/to/image"))

I will be updating the readme soon.

There are 117 celebs, I just used a scraping script to download the images, you can easily scale this to many more celebs. The json file has names of all the celebrities.
For training your own model, you can try to follow the readme, might give you some issues here and there, as that script has not been tested for different scenarios, but the code is pretty straightforward, you should be able to debug that.
Regarding the huggingface space, if the model does not detect any celebrity, it just returns back the same image. Check the json file to see the list of supported celebrities.
If you still faces any issue, feel free to ping here.

The confidence value is not scaled to 1.0 actually, a good threshold would be 0.2.
For training, anything more than 20 pictures works well. On CPU it would be very slow, you can use a colab GPU itself to train this.
Just fyi, the model is not getting trained in the traditional sense. The script creates embeddings for the celeb faces and saves those in a vector space (using annoy library), and mappings are stored in a json file.
During inference, we find the embedding for the celeb to be detected, and compare it with the vector space to find the closest match.

I just found a bug in the create_celeb_model.py file, can you take  the latest pull and try again? Or you can just copy the latest code from the file and try.
Regarding the face_recognition issue, did you run this command?
pip install git+https://github.com/ageitgey/face_recognition_models
If not, please run this once and check.

hey thanks for the PR, I have merged it, feel free to use or modify the code for your application.


Wrong output when the head is a link

In the following example,  it shows  [tile](link) not just the title
## [tile](link)

Thanks.

Formatting in nodes is not officially supported, but I think we could handle this case.


Test and fix debug mode

Some tests of various bits of functionality with debug_mode switched on. Tests are structured so that we can check that issues specifically arise only when debug_mode is on - if the test fails when it is off, a different error will be flagged, to help isolate things. These can be expanded to help diagnose issues, but just as an initial starting point.
This will also include fixes for the failing tests, which will be opened as separate PRs into this branch.

Oddly the tests that fail are passing in sqlite

@RobinL - there's nothing particularly major here, but might be worth a glance over.
All of the 'real' changes are in separate PRs with some explanation should you feel the need. Beyond that it might also be useful to look at the test decorator for checking issues with functionality in debug mode, to see if you think that seems a sensible way to go about it.


Move TrustPush and related classes to new trust module

Not all trust related classes moved as part of this PR, only main interfaces
to the rest of the code.

Why a new project? Please keep it in the core project (because it's strictly related with L0 nodes) but in the trust module.
I told you that I see TrustPush program that is in the cluster module, but you created a separate trust module and didn't move TrustPush there.

We don't need a separate project just for trust 🤔

@marcinwadon my mistake, will open a new PR, misunderstood.


Fix World/Screen/Tile/Pixel/Thing positioning.

Video demonstration - https://www.youtube.com/watch?v=JCK1xbpxQPE&feature=youtu.be


Fix build

(trivial) Maybe it would be worth fetching a few more (like somewhat in range 10-100) to cover other edge cases as well. 10-100 should't hurt at all IMHO.

Why do we change the pom version again?

Normally the commits from the release on the master are merged back to develop after the release. This failed here https://github.com/swisspost/apikana-java/actions/runs/8006164501 Therefore I'm merging these commits manually in this PR

(trivial) Maybe it would be worth fetching a few more (like somewhat in range 10-100) to cover other edge cases as well. 10-100 should't hurt at all IMHO.

I was thinking about this as well. But we actually only need enough so that develop and master have a common ancestor and so git knows what to merge.


Cisco AppDynamic

Plugin contact emails
choonhoson<EMAIL_ADDRESS>Project Summary
Cisco AppDynamics Inventory Plugin
Project Description
Cisco AppDynamics provides observability of Application.
Integration components

Application information

Product Introduction URL
https://www.appdynamics.com/
Product API URL
https://docs.appdynamics.com/appd/22.x/22.3/en/extend-appdynamics/appdynamics-apis
Additional repos
No response
Contributing or Sponsoring Org
Cisco
IP Policy

[X] If the project is accepted, I agree the project will follow the Cloudforet IP Policy

Why Cloudforet?
An information of application is hard to maintain.
There are lots of application monitoring platform. AppDynamics is one of world-best Application observability tool.
This information will guide best knowledge to infra engineers.
Benefit to the Landscape
Inventory
Project Advisor
Jihyung Song
Additional information
No response

There are possible plugins

monitoring.Webhook
inventory.CloudService

monitoring.Webhook
https://github.com/cloudforet-io/plugin-cisco-appdynamics-mon-webhook

SAP integration
https://docs.appdynamics.com/sap/en/architecture-overview

Plugin Integration Guide
https://docs.google.com/presentation/d/1yPApVltTbLXccaMWuQ_5WxKcYT2HQEHSIzi3V1b7_sg/edit?usp=sharing

Move development issue to cloudforet repository issue


feat(project-focus): add header module for project focus module (DEV-820)

resolves DEV-820

I'm doing everything for this task in #49


Adding missing environment variable DJANGO_DB_HOST

"docker-compose up" command had the below mistake:
api_1  | Traceback (most recent call last):
api_1  |   File "manage.py", line 11, in
api_1  |     execute_from_command_line(sys.argv)
api_1  |   File "/root/.local/share/virtualenvs/src-NCIGkioc/lib/python3.7/site-packages/django/core/management/init.py", line 381, in execute_from_command_line
api_1  |     utility.execute()
api_1  |   File "/root/.local/share/virtualenvs/src-NCIGkioc/lib/python3.7/site-packages/django/core/management/init.py", line 325, in execute
api_1  |     settings.INSTALLED_APPS
api_1  |   File "/root/.local/share/virtualenvs/src-NCIGkioc/lib/python3.7/site-packages/django/conf/init.py", line 79, in getattr
api_1  |     self._setup(name)
api_1  |   File "/root/.local/share/virtualenvs/src-NCIGkioc/lib/python3.7/site-packages/django/conf/init.py", line 66, in _setup
api_1  |     self._wrapped = Settings(settings_module)
api_1  |   File "/root/.local/share/virtualenvs/src-NCIGkioc/lib/python3.7/site-packages/django/conf/init.py", line 157, in init
api_1  |     mod = importlib.import_module(self.SETTINGS_MODULE)
api_1  |   File "/usr/local/lib/python3.7/importlib/init.py", line 127, in import_module
api_1  |     return _bootstrap._gcd_import(name[level:], package, level)
api_1  |   File "", line 1006, in _gcd_import
api_1  |   File "", line 983, in _find_and_load
api_1  |   File "", line 967, in _find_and_load_unlocked
api_1  |   File "", line 677, in _load_unlocked
api_1  |   File "", line 728, in exec_module
api_1  |   File "", line 219, in _call_with_frames_removed
api_1  |   File "/usr/src/app/settings/master-docker.py", line 12, in
api_1  |     'HOST': os.environ['DJANGO_DB_HOST'],
api_1  |   File "/usr/local/lib/python3.7/os.py", line 681, in getitem
api_1  |     raise KeyError(key) from None
api_1  | KeyError: 'DJANGO_DB_HOST'

Thanks!

No worries 👯  I'm having some issues with the docker-compose on the frontend as well, hope I can find a way to make it work 😃

Is there anything I can do about this one? It is still not merged yet...

Sorry for the delay!

@dabeeeenster Great, thank you! 👍


feat: BasePostSimple type adds isInProcess field

see https://github.com/halo-dev/halo-admin/issues/438
Signed-off-by: Ryan Wang<EMAIL_ADDRESS>

HOLD. Please merge this PR after halo-dev/halo#1617 merged.

Yup

/retitle WIP: feat: BasePostSimple type adds isInProcess field


[0.10.0] Include tag values in response of last()

Hi community,
sience the last updates of influxdb, I don't know how to solve this problem:
I want to create a query which repeats the last value of a series and its tag values. I don't want to have the tag values as seperated columns. I need them in the header to use them in Grafana as alias.
SELECT last("value"), max, min FROM airtemp
--> returns the tag values as columns.

SELECT last("values") FROM airtemp GROUP BY max, min
--> returns multiple series if there are more than one max/min-value in the whole airtemp-serie.

Can you give me a hint, whether this is possible with a influx query or whether I have to change Grafana code?
If you need more examples, please ask me.
Thanks,
Niklas

+1. I've been struggling with this too.

@The-Nik can you give me an example of the output you want or expect?

I'm going to close this since more information hasn't been given about what is needed. Please reopen if you have more information. Thanks.


Smarter Docker + Vagrantfile caching

@amfarrell - after digging around on the documentation for Vagrant, it looks like caching Docker images will be a non-trivial endeavor.  The best solution I found is this:
https://github.com/fgrehm/vagrant-cachier/issues/131
Basically, we can't map the Docker cache to a synchronized directory on the host directory.  The best solution is to manually save and load Docker images manually.  I think I can automate this in the Vagrant script, but I may not get to it tonight.

How frequently are users going to be spinning up new vagrant boxes? If they are only installing the system locally and running salt highstate once, how impactful is this issue?

This is a developer, not a user issue. It's not priority.
On Friday, June 5, 2015, Andrew Farrell<EMAIL_ADDRESS>wrote:

How frequently are users going to be spinning up new vagrant boxes? If
they are only installing the system locally and running salt highstate
once, how impactful is this issue?
—
Reply to this email directly or view it on GitHub
https://github.com/memex-explorer/memex-explorer/issues/503#issuecomment-109436015
.

This isn't really an issue anymore as Docker is no longer part of the development build.


update README.md

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@a1d1aac). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             master   #187   +/-   ##
=======================================
  Coverage          ?   100%
=======================================
  Files             ?      4
  Lines             ?     45
  Branches          ?      5
=======================================
  Hits              ?     45
  Misses            ?      0
  Partials          ?      0

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 a1d1aac...ffe5f69. Read the comment docs.


Provide better return indicating stale scheduler

This should not return a 204.  Instead it should return something more indicative of the scheduler being stale.
Version and Environment Details

Scale API: 7.0.1rc4

Can be found at /api/v6/scheduler/status

Scale UI: 0.8.4
Cluster details below are very helpful for problem resolution, but external Scale users aren't expected to provide.
DC/OS: kraken

Additional context
https://github.com/ngageoint/scale/blob/330b6e6d15501faf52e069cf2a7f68619d959077/scale/scheduler/views.py#L145-L146

Return the status but with a field warning that it's stale?  Return only the stale warning with no status?  I assume the problem was you tore your hair out debugging the scheduler only to figure out it was slow so something more obvious would be nice.

That bug was also complicated by some sort of hang up in the node deprecate code that confused the scheduler.

On Sep 6, 2019, at 5:51 PM, John Tobe<EMAIL_ADDRESS>wrote:
Return the status but with a field warning that it's stale? Return only the stale warning with no status? I assume the problem was you tore your hair out debugging the scheduler only to figure out it was slow so something more obvious would be nice.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


vscode: PERM: operation not permitted, copyfile

Type: Bug
This issue is detailed here:  https://stackoverflow.com/questions/77010682/vscode-perm-operation-not-permitted-copyfile

Install Google Drive. this should install a G: drive on your windows 11 system
-When setting this up, select 'streaming' instead of mirrored
Create project folder somewhere on the G drive.
Open project in vscode and create a dev container.
Im thinking now we have to wait for the files to be 'unmirrored'
Re-open the project and then the devcontainer
Try to cut and paste a file or folder from one location to another folder.

VS Code version: Code 1.81.1 (6c3e3dba23e8fadc360aed75ce363ba185c49794, 2023-08-09T22:22:42.175Z)
OS version: Windows_NT x64 10.0.22621
Modes:
Connection to 'dev-container+7b22686f737450617468223a222f686f6d652f6d6168656e6472612f70726f6a656374732f72616173222c226c6f63616c446f636b6572223a66616c73652c2273657474696e6773223a7b22686f7374223a227373683a2f2f6e727a646f636b657274657374227d2c22636f6e66696746696c65223a7b22246d6964223a312c2270617468223a222f686f6d652f6d6168656e6472612f70726f6a656374732f726161732f2e646576636f6e7461696e65722f646576636f6e7461696e65722e6a736f6e222c22736368656d65223a227673636f64652d66696c65486f7374227d7d' could not be established
Remote OS version: Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2

System Info

Item
Value

CPUs
AMD Ryzen 9 4900HS with Radeon Graphics          (16 x 2994)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
39.42GB (21.81GB free)

Process Argv
--crash-reporter-id a94c5544-2680-458f-803c-0aecde90a1fc

Screen Reader
no

VM
0%

Connection to 'dev-container+7b22686f737450617468223a222f686f6d652f6d6168656e6472612f70726f6a656374732f72616173222c226c6f63616c446f636b6572223a66616c73652c2273657474696e6773223a7b22686f7374223a227373683a2f2f6e727a646f636b657274657374227d2c22636f6e66696746696c65223a7b22246d6964223a312c2270617468223a222f686f6d652f6d6168656e6472612f70726f6a656374732f726161732f2e646576636f6e7461696e65722f646576636f6e7461696e65722e6a736f6e222c22736368656d65223a227673636f64652d66696c65486f7374227d7d' could not be established

Item
Value

Remote
Dev Container: redacted

OS
Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2

CPUs
AMD Ryzen 9 4900HS with Radeon Graphics (16 x 2994)

Memory (System)
19.25GB (16.93GB free)

VM
0%

Extensions (35)

Extension
Author (truncated)
Version

terraform
4op
0.2.5

vscode-azurevirtualmachines
ms-
0.6.5

remote-containers
ms-
0.304.0

remote-ssh
ms-
0.102.0

remote-ssh-edit
ms-
0.86.0

remote-wsl
ms-
0.81.0

vscode-remote-extensionpack
ms-
0.24.0

remote-explorer
ms-
0.4.1

remote-server
ms-
1.4.3

terraform
4op
0.2.5

vscode-postgres
cko
1.4.3

vscode-eslint
dba
2.4.2

gitlens
eam
14.2.1

git-graph
mhu
1.30.0

azure-dev
ms-
0.7.0

vscode-azureappservice
ms-
0.25.0

vscode-azurecontainerapps
ms-
0.5.1

vscode-azurefunctions
ms-
1.12.4

vscode-azureresourcegroups
ms-
0.7.5

vscode-azurestaticwebapps
ms-
0.12.2

vscode-azurestorage
ms-
0.15.3

vscode-azurevirtualmachines
ms-
0.6.5

vscode-cosmosdb
ms-
0.19.4

python
ms-
2023.14.0

vscode-pylance
ms-
2023.8.40

jupyter
ms-
2023.7.1002162226

jupyter-keymap
ms-
1.1.2

jupyter-renderers
ms-
1.0.17

vscode-jupyter-cell-tags
ms-
0.1.8

vscode-jupyter-slideshow
ms-
0.1.5

azure-account
ms-
0.11.5

azurecli
ms-
0.5.0

vscode-node-azure-pack
ms-
1.2.0

sqltools
mtx
0.28.0

sqltools-driver-pg
mtx
0.5.1

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vstes627:30244334
vslsvsres303:30308271
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscoreces:30445986
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593cf:30376535
pythonvs932:30410667
vsclangdc:30486549
c4g48928:30535728
dsvsc012cf:30540253
pynewext54:30695312
azure-dev_surveyone:30548225
vsccc:30803844
282f8724:30602487
89544117:30613380
showlangstatbar:30737416
962ge761:30823813
a2ce3375:30757347
pythonfmttext:30731395
fixshowwlkth:30771522
showindicator:30805244
pythongtdpath:30769146
i26e3531:30792625
gsofb:30804716
pythonnosmt12:30797651
pythonidxptcf:30805731
pythonnoceb:30805159
asynctok:30821568
dsvsc013:30795093
dsvsc014:30804076
diffeditorv1:30821571

We closed this issue because it is a question about using VS Code rather than an issue or feature request. Please search for help on StackOverflow, where the community has already answered thousands of similar questions. You may find their guide on asking a new question helpful if your question has not already been asked. See also our issue reporting guidelines.
Happy Coding!

This issue is not about using vscode.  It is I believe a bug with vscode + dev containers + Google drive mounts from a windows machine.
That said it may also be a docker + ask issue with Google drive. I don't know .. but the user experience issue is evident with trying to copy files and folders in vscode in a devcontainer


The status of Argo was misrepresented in CI/CD Technical 101

In Technical 101 of CI/CD says

Flux is the CNCF gratuated project in this space, Argo and Keptn are the two CNCF incubating projects

But in fact, Argo has already graduated.

can you send an updated PR to fix this given the change of graduation
status?
On Fri, Dec 30, 2022 at 12:25 PM HaoTian @.***> wrote:

In Technical 101 of CI/CD says
Flux is the CNCF gratuated project in this space, Argo and Keptn are the
two CNCF incubating projects
But in fact, Argo has already graduated.
—
Reply to this email directly, view it on GitHub
https://github.com/cncf/landscape/issues/2948, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAPSIOXMGIK5CWCDNHNFLLWP4SJNANCNFSM6AAAAAATNCDASY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
Cheers,
Chris Aniszczyk
https://aniszczyk.org

fixed by #2950


add get_android_api_usage method in Analysis class

Hi @reox,
I would like to know whether it is helpful to provide this get_android_api_usage function. In many malware analysis cases, we hope to be able to get all Android APIs usage, and some APIs may not require permission to use, so providing this method allows users to get all Android APIs.
For example:
a, d, dx = AnalyzeAPK("14d9f1a92dd984d6040cc41ed06e273e.apk")

for meth_analysis in dx.get_android_api_usage():
    print(meth_analysis.full_name)

The output could be:
Ljava/lang/Object; <init> ()V
Ljava/lang/Object; equals (Ljava/lang/Object;)Z
Ljava/lang/Object; toString ()Ljava/lang/String;
Ljava/lang/Object; notifyAll ()V
Ljava/lang/Object; wait (J)V
Ljava/lang/Object; notify ()V
Ljava/lang/Object; wait ()V
Landroid/os/Parcel; obtain ()Landroid/os/Parcel;
Landroid/os/Parcel; writeInterfaceToken (Ljava/lang/String;)V
Landroid/os/Parcel; readException ()V
Landroid/os/Parcel; recycle ()V

Thanks you!

Codecov Report

Merging #795 into master will decrease coverage by 0.01%.
The diff coverage is 20.00%.

@@            Coverage Diff             @@
##           master     #795      +/-   ##
==========================================
- Coverage   76.84%   76.83%   -0.02%
==========================================
  Files          51       51
  Lines       16288    16293       +5
==========================================
+ Hits        12517    12518       +1
- Misses       3771     3775       +4

Impacted Files
Coverage Δ

androguard/core/analysis/analysis.py
78.09% <20.00%> (-0.38%)
: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 5ddede4...5fdb670. Read the comment docs.

@reox Thank you for your review! , I will write some tests and documentation later


Mention Mantle in the AllowanceHolder addresses section

#209 includes these same changes. Is it important to update the README.md ASAP?

@duncancmt
nope! I wasn't aware of the pending PR. -- I will close this PR.


Rename step connection to datatbase

Close #52

@rultor merge

@rultor merge

@smallcreep OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@smallcreep Done! FYI, the full log is here (took me 22min)


FluidObject: Add Breaking note, and update remaining non-synthesize cases

Adds a note to breaking about the deprecation of IFluidObject and updates remaining internal usages which are not synthesize which will come with #8074
fixes #8076

@wes-carlson i'd like to get this in before the release is cut today


MessagePack Protocol

From @huaxiangyang on https://github.com/aspnet/Docs/issues/10763 ...
When to add the MessagePack protocol to Java Clients?

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

ID: e807e138-f84c-e63e-ab37-eec0d554d3bb
Version Independent ID: 3ac41883-3bc2-30ba-a47f-462196850c12
Content: Use MessagePack Hub Protocol in SignalR for ASP.NET Core
Content Source: aspnetcore/signalr/messagepackhubprotocol.md
Product: aspnet-core
GitHub Login: @bradygaster
Microsoft Alias: bradyg

Also note that the latest version of signalR (3.0.0-preview5-19227-01) is NOT compatible with the matching version of MassagePack.
"<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>https://github.com/aspnet/AspNetCore/issues/10307

@Rick-Anderson @bradygaster Any chance this will be available in the nearest future? Or maybe you can suggest how to deal with the problem when large json payloads are causing SignalR requests to time out. Our field metrics show ~16 mb payloads are "too much" for Azure SignalR :(.

We've added the MessagePack protocol in rc1 - Using this issue to track adding documentation for it

I see rc1 here<EMAIL_ADDRESS>thx

Resolved by https://github.com/dotnet/AspNetCore.Docs/pull/21090

Resolved by https://github.com/dotnet/AspNetCore.Docs/pull/21090


Fix infinite loop & Avoid UIAlertController

I noticed a change in the history of topViewController where presentedViewController was changed to presentingViewController. I'm not sure what this was intended to fix but it easily sends it into an infinite loop where it goes like this:

NavigationController
Modal (via visibleViewController)
back to NavigationController (via presentingViewController)
etc

The proposed changes are as follows:

Don't set the presentationViewController explicitely in the demo project and rely on the topViewController implementation instead - this way it gets tested as part of the UITests
Change presentingViewController back to presentedViewController
Stop using visibleViewController the nav controller (since the presented part is covered in the previous case) and just use topViewController
Additionally, make an exception for UIAlertControllers when going up the presented stack - it looks really strange when a message appears in an alert modal

Number 4 is probably not foolproof but does handle the most obvious cases. Let me know what you think. Thanks!
Hope this helps 😄

Hey thank you for the detailed PR. Will review as soon as I can :) but may be a bit delayed due to being sick and work responsibilities...

Ok. So to start with can you provide more detail as to how to reproduce this infinite loop?
In the demo app bundled with the project, the DemoViewController is a navigation controller and presents an RMessage. From there I present modals -> present RMessage -> dismiss modals, present RMessage etc etc and I dont see an infinite loop.
Not saying its not there, but the first step in addressing this issue is reproducing the inf. loop to understand the reason it is caused.

I have managed to fix the issue. Thanks so much for the report, though I unfortunately won't be using your PR exactly as is because there is something a bit more insidious as a result of being a bit new to swift when I made the conversion... Please see PR 58.

If you get the chance I'd appreciate some details as to what scenario I failed to cover - always good to learn something new.


temp prompt reduction

We are experiencing significant rate limiting issues with the name suggestion endpoint, primarily due to the size of the prompt. This omits the wordlist (which is technically unnecessary) while retaining the guidance to avoid inappropriate output.

🟡 Heimdall Review Status

Requirement
Status
More Info

Reviews

🟡
0/1

Denominator calculation

Show calculation

1 if user is bot
0

1 if user is external
0

From .codeflow.yml

1

Additional review requirements

Show calculation

Max
0

0

From CODEOWNERS

0

Global minimum
0

Max

1

1

1 if commit is unverified
0

Sum
1


Delaying reporter start

Currently, this is just a test to POC the approach of reporting events before starting the disruptor without losing events.
@felixbarny this is what we discussed - it can be used in ApmServerReporter#start that would asynchronously invoke a blocking ReportingEventHandler#init and start the disruptor only after it returns. It can then block on metadata discovery as well as APM Server client and health check, so that we can start accumulating events in the ring buffer before we are ready to send them.
One drawback of this approach is that it won't cover for intermittent connection failures with APM server, so maybe the best approach is to simply resubmit events to the buffer when send fails.
I currently open this as draft so that we can further look into it later on.
Applying better coverage to these scenarios may also resolve the inconsistent integration tests failures reported in #2176

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2021-11-01T12:50:33.467+0000

Duration: 69 min 9 sec

Commit: e1469718f150078d960ad385c7120e35d1da6a7a

Test stats :test_tube:

Test
Results

Failed
0

Passed
2491

Skipped
19

Total
2510

:green_heart: Flaky test report
Tests succeeded.
:robot: GitHub comments
To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.

run benchmark tests : Run the benchmark test.

run compatibility tests : Run the JDK Compatibility test.

run integration tests : Run the APM-ITs.

Won't merge this way. We may consider this or other approach in the future.


don't send disconnect message in response to a cancel control request

Cancel is a hard terminate. It doesn't need an ack, and replying to it is a loop risk.

I've been debating this one in my head, might as well mention it here:
This is clearly a good idea in isolation, (my first attempt at a mongrel2 handler caused such a loop), but if there are backends out there that rely on the disconnect message to cleanup resources, even when they specifically request a cancel, then this will silently cause resource leaks.
Silently causing a problem that will slowly leak resources is one of the worst side-effects of changing things.

The original intended contract for the cancel control request is that it shouldn't have an ack, so I'd say the question is whether anyone might have used it without knowing this. I don't think the extended requests I added were ever documented, right? It could be Pushpin is the only user.
Another thing to consider is that proper handler code needs to be resilient in the absence of disconnect messages anyway. For example, if Mongrel2 crashes & restarts. If there is handler code out there relying on 100% delivery of disconnect messages then they are going to get bit at some point or another.
We could introduce a setting to determine the behavior, but I think this is overkill. It's up to you.

Oh, this is an extended request; that's totally fine.


Allow client to add object to class when the class exist in the schema

Hi,
When setting allowClientClassCreation to false, the client is forbidden to add object to non-existing class.
The method to check if the class exist, is by checking to the collection exists in the database. (RestWrite.js line 120, RestQuery.js line 177)
That method to check is not good enough when the class exist in the schema.
When creating class only by adding at to the schema, the client is unable to create object.
I think that when the class exist in the schema, the user should be able to add object to it (and create implicitly new collection at the DB)
Thank you

The method to check if the collection exist is very fast. and after the first object is created, it's olso


'Line separator' setting in PHPstorm potentially causes issues.

This setting can be changed at the bottom right corner of the screen.
the file package.json often contains changes after running npm install while nothing is changed. This is due to the fact that PhpStorm uses the wrong line-endings for this file. The result is that a git pull will not work since the file contains open 'changes'.

This is solved in the .gitattributes file of the projects by adding the line package.json eol=lf.
The easiest way to enforce this change from being applied on your computer is to delete the repository and clone it again.


fix the link

the old link can't access.

Hi @Ricardio609,
I can access it without any problem. Could you give me more information?


ci: publishing to gh packages from develop

fixes KILTProtocol/ticket#599
In order to use develop versions of the sdk in our dependencies (at least on the devnet) we also need to publish a dev version of Portablegabi on the GitHub Packages repo. This is due to name-spaced repository settings (yarn will expect any package from the @KILTprotocol namespace to be on GitHub Packages) and assures that we also use a dev version of Portablegabi.
How to test:
I could trigger a test release by adding this branch to the on section in the workflow file.
Checklist:

[ ] I have verified that the code works
[ ] I have verified that the code is easy to understand

[ ] If not, I have left a well-balanced amount of inline comments

[ ] I have left the code in a better state
[ ] I have documented the changes (where applicable)

¯_(ツ)_/¯


Communication between downstream devices after reboot

Hi

Third, it enables communication between downstream devices that normally would go through IoT Hub.
Question:
Is communication between downstream devices also enabled and possible after a reboot of the system even if internet connection after reboot is still not available?

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

ID: 6f8e1616-f3c9-ba1c-2ebd-9d7f74d79005
Version Independent ID: 3158bd35-4d7d-63ec-1df7-520f00c053c0
Content: Operate devices offline - Azure IoT Edge
Content Source: articles/iot-edge/offline-capabilities.md
Service: iot-edge
GitHub Login: @PatAltimore
Microsoft Alias: patricka

@kdotdk Thanks for the feedback! We are currently investigating and will update you shortly.

@kdotdk As mentioned in the doc, While disconnected from IoT Hub, the IoT Edge device, its deployed modules, and any downstream devices can operate indefinitely. Modules and downstream devices can start and restart by authenticating with the IoT Edge hub while offline. Telemetry bound upstream to IoT Hub is stored locally. Communication between modules or between downstream devices is maintained through direct methods or messages.
Does that answer your query? Do let us know if you have any further queries.


Cannot extract keys when Content-type header is not set

When Content-type header is not set, mmock cannot extract keys.
Example log:
{     "scheme": "http",     "host": "1",     "port": "80",     "method": "POST",     "path": "/jsonrpc/v1/payments.create",     "queryStringParameters": {},     "fragment": "",     "headers": {         "Accept-Encoding": [             "gzip"         ],         "Authorization": [             "Basic cWE6cWE="         ],         "Cache-Control": [             "no-cache"         ],         "Content-Length": [             "33"         ],         "Cookies": [             "{}"         ],         "User-Agent": [             "Go-http-client/1.1"         ]     },     "cookies": {},     "body": "{\"id\":\"e2a69502\",\"jsonrpc\":\"2.0\"}" }
Mapping:
{"id": "{{request.body.id}}", "jsonrpc": "{{request.body.jsonrpc}}"}
What mmock returns:
{"id": "{{request.body.id}}", "jsonrpc": "{{request.body.jsonrpc}}"}
P.S. Sorry, i got it wrong for the first time, because of escaped JSON in log

Maybe create additional parameter to set default content, based on matched mapping?


Fix decoding of publish command, publishing type is optional

I have noticed that some tools are not providing the publishing type in the publish command. It is not explicitly optional in the specification but in practice it seems to be. And so by default I use the live value

Hello,
Unfortunately I don't know exactly what is pushing that... from the information I have it is coming from an extron electronics (hardware equipment) through the restream SaaS platform. But I don't know exactly who is not providing the publishingType

I see, thank you very much. It's intriguing, so I think I'd like to investigate based on the information you've provided.


[3.05] Fix #786 OpenCL linkage on macOS Sierra

This includes and builds on @stweil 's patch to fix linkage on macOS.
@zdenop This includes PR #809, so if you accept this you can close PR #809.

@jbarlow83:  You could also make this PR's merger depend on that of #809 instead of including it within this PR outright by stating so in either your commit message and/or this PR's initial description post and removing your copy of said other issue's resolution from your PR branch here — as a matter of fact, you most likely should, as @stweil has explicitly stated in a comment on #809 that he's going to add more commits to his branch there until he's backported every OpenCL improvement he feels is reasonable to include from 4.0 into 3.05.0x that he sees fit to, so it would probably be worth a try to save yourself the effort, little that it might be, of keeping up with and duplicating his commits in your branch.

@jbarlow83:  I should also note that, in resolving #786, this will fix OpenCL linkage problems not just on Sierra, but also on El Capitan, due said issue's nature.

@RandomDSdevel I agree, I should have done it that way. I didn't realize there was a way to create a PR dependent on another PR.
@stweil If you prefer to cherry-pick my commit into your PR that is just fine, since you are planning other changes.

@jbarlow83:  Well, GitHub doesn't exactly recognize these kind of PR relationships as such, but you can do it informally, at least…; some repositories (some I can name are those owned by the Homebrew organization) even have a 'do not merge' label (I looked, but this one doesn't) used to indicate that a PR has an unresolved dependency of some sort, but the tooling doesn't extend any further beyond that.  As for this PR's interaction/relationship with @stweil's, what you could have him do after discussing it with him is just to have him include #813, the version of this PR for Tesseract 4.0, in it since what he's doing will end up being a mass backport like I think I mentioned.


RezoScans: Fix CDN URL

Checklist:

[ ] Updated extVersionCode value in build.gradle for individual extensions
[x] Updated overrideVersionCode or baseVersionCode as needed for all multisrc extensions
[ ] Referenced all related issues in the PR body (e.g. "Closes #xyz")
[ ] Added the isNsfw = true flag in build.gradle when appropriate
[x] Have not changed source names
[ ] Have explicitly kept the id if a source's name or language were changed
[x] Have tested the modifications by compiling and running the extension through Android Studio
[ ] Have removed web_hi_res_512.png when adding a new extension

move this to multisrc, the other sources has the same change
also regex is already broke, try with this:
realUrl\s*=\s*`[^`]+//(?<host>[^/]+)

and then this to replace all the tags just in case:
\$\{[^}]*}


.NET Core 2.1 DirectoryInfo.EnumerateFiles() Performance is Slower than .NET Framework

Issue Title
.NET Core 2.1 DirectoryInfo.EnumerateFiles() Performance is Slower than .NET Framework
General
I have a .NET Core 2.1 Web API that is being built as a replacement for an older .NET Framework MVC Web Application (.NET Framework 4.5).
One of the responsibilities of this application is to take in a directory and search template and return the top 10 results, ordered by most recent. This is being accomplished using System.IO.DirectoryInfo and the EnumerateFiles() method:

List<FileInfo> fileInfos = new DirectoryInfo(directory).EnumerateFiles(fileTemplate)
.OrderByDescending(f => f.LastWriteTime).Take(10).ToList();

However, I am noticing that this search is much, much slower in .NET Core than in .NET Framework. I created two console apps with the same code, one .NET Framework and one .NET core, to compare run times. Below are the results in .NET Framework:

And then the results in .NET Core:

Can anyone help me understand why there is such a dramatic decrease in performance?

better move to https://github.com/dotnet/corefx
/cc @JeremyKuhne, @pjanotti

Issue moved to dotnet/corefx #32098 via ZenHub


[Coverity CID: 185273] Resource leaks in /tests/net/socket/getaddrinfo/src/main.c

Static code scan issues seen in File: /tests/net/socket/getaddrinfo/src/main.c
Category: Resource leaks
Function: test_getaddrinfo_ok
Component: Tests
CID: 185273
Please fix or provide comments to square it off in coverity in the link: https://scan9.coverity.com/reports.htm#v32951/p12996

@stig-bjorlykke: As the author of this test, would you like to have a look/try to do something? If there're any issues, just let me know and I'll cover it. Thanks.

I suspect this is because coverity expects the addrinfo returned from getaddrinfo() to be freed with freeaddrinfo(). In Zephyr this is not strictly needed, but I will provide a fix for this test to be coverity compliant.

That would also be my solution, glad we're on the same line here, and thanks for sharing the maintenance load. Please keep in mind that there's always a choice to mark a Coverity warning as "false positive", and in tests, being formally correct is indeed not that important. But if a better fix is simple and doesn't affect the flow of test, indeed, why not fix it, instead of silencing on Coverity side?


DeriveActiveEnum requires sea-query

At first, thank you for all your work on sea-orm! It is a fantastic addition to the Rust ecosystem ❤
Upgrading to version 0.10.0.
That version, being a release candidate, I think should have been named 0.10.0-rc1 or 0.10.0-beta1, when following semver standards, so that people only looking at versions and not release notes don't get confused.
This change now requires that sea-query is added as a dependency (as DeriveActiveEnum seem to rely on that now) where it was previously possible to achieve this with just sea-orm as a dependency. Was this intentional?

Related: #1143
You can import sea_query via use sea_orm::sea_query to resolve without adding sea_query crate as a dependency.

Hey @inf0rm4tik3r, thanks for the report!! @siketyan and I are working on fixing the issue.

https://github.com/SeaQL/sea-orm/pull/1154

A new patch will be released shortly :)


bot: Encapsulate output logs in details tag

What kind of change does this PR introduce?
Readability change for @webpack-bot errors logs.
Did you add tests for your changes?
No
Summary
@webpack-bot is posting large tests errors logs in PRs creating large webpages with long scrolls, this make it harder to have meaningful conversations in PR.
This PR encapsulates those logs in a <details> tag (https://developer.mozilla.org/en-US/docs/Web/HTML/Element/details) which is support by GitHub markdown.
With this change large logs will be collapsed by default unless you click on it.
Before

After
Collapsed by default

After clicking on Details

I swapped Regex after testing them https://repl.it/@montogeek/LightyellowUprightMotion, probably I am wrong and thats how open-bot works

I swapped Regex after testing them https://repl.it/@montogeek/LightyellowUprightMotion, probably I am wrong and thats how open-bot works

The filter removes parts of the string by regexp. You can try it with str.replace(regexp, "")

https://repl.it/repls/WideProfitableAddon

Thanks


fixAdd log to investigate nereids_p0/stats/partition_key_minmax case failure.

What problem does this PR solve?
Add log to investigate nereids_p0/stats/partition_key_minmax case failure.
Issue Number: close #xxx
Related PR: #xxx
Problem Summary:
Release note
None
Check List (For Author)

Test

[x] Regression test
[ ] Unit Test
[ ] Manual test (add detailed scripts or steps below)
[ ] No need to test or manual test. Explain why:

[ ] This is a refactor/code format and no logic has been changed.
[ ] Previous test can cover this change.
[ ] No code files have been changed.
[ ] Other reason

Behavior changed:

[x] No.
[ ] Yes.

Does this need documentation?

[x] No.
[ ] Yes.

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[ ] Add branch pick label

run buildall

run p0


GCP: Google Project IAM Resources

I would like to import existing IAM settings as followings:

google_project_iam_policy
google_project_iam_binding
google_project_iam_member
google_project_iam_custom_role

Terraform

https://www.terraform.io/docs/providers/google/r/google_project_iam.html
https://www.terraform.io/docs/providers/google/r/google_project_iam_custom_role.html

MEMO: We already have the file to implement it. But, everying is in TODO.
https://github.com/GoogleCloudPlatform/terraformer/blob/master/providers/gcp/iam.go#L61

Apparently @reinoudk (partially?) implemented this, however there seems to be no info on this available in the documentation. What's the way to import IAM config?

@mrusme sorry, I forgot to update the README, I'll create a PR sometime today to fix that. You can import google_project_iam_member and google_project_iam_custom_role resources by running terraformer import google --resources=iam.
The google_project_iam_binding and google_project_iam_policy resources conflict with google_project_iam_member and have therefore not been implemented.

Hi @reinoudk - Does google_project_iam_binding and google_project_iam_policy resources available now? It would be super helpful if you could advise about it.

Hi @ponkumarm, I've just glanced at the iam code again and I don't think it has changed. So I think there's only support for google_project_iam_member as per the reasons mentioned above.


Localization Change Causes Publishing Problem (also not working on Emulator)

Describe the bug
My bot is working properly on the composer web chat whether the default language is English or Turkish. But on Emulator, it is only working properly when the default language is English. When I change the language to Turkish, it is not working on the Emulator.
It also causes problems in the Publishing phase, if the bot is not working on the Emulator, it is not working after publishing too.
Version
Version:  2.1.0-nightly.266986.c355633
Electron: 8.2.4
Chrome: 80.0.3987.165
NodeJS: 12.13.0
V8: 8.0.426.27-electron.0
(experienced the same issue on the stable version too)
Emulator: 4.13.0
Browser

Edge

OS

Windows

To Reproduce
Steps to reproduce the behavior:

Change your localization settings, make Turkish default language
Try on webchat, it will work properly
Try on Emulator, it will possibly fail (no answers)
Try to publish, if it is successfully published, try from Azure Portal, it will not work properly (no answers)

Screenshots

When localization is English: it works on emulator

Turkish case: it works on webchat but not on Emulator

Additional context
We spend so much time trying to understand why it fails after publishing and not responds, thats how we discovered that, when it stays on English as default, it works after publishing too, and emulator of course.

@humak This is an emulator setting that needs to be changed to the locale that you want the bot to respond to. The current locale set on Composer does not get set on Emulator when we use the operation "Open in Emulator" from Composer.

We tried your solution and it worked on Emulator. Thank you so much!
We still couldn't reach the published bot from Azure Portal WebApp Chat.
Do you have any suggestion about how we can add this language parameter during/after publishing bot to Azure WebApp? Since these problems are connected and similar, I think we are also missing some point during publishing phase.

We still couldn't reach the published bot from Azure Portal WebApp Chat.
Do you have any suggestion about how we can add this language parameter during/after publishing bot to Azure WebApp? Since these problems are connected and similar, I think we are also missing some point during publishing phase.

Hi @humak Are you referring to Test in WebChat or full WebChat?
The locale of the browser should be the thing that is telling WebChat and the bot what locale should be used. For example, in Emulator, you can see what locale is being sent:
{
  "type": "conversationUpdate",
  "membersAdded": [
    {
      "id": "______________________________",
      "name": "Bot"
    },
    {
      "id": "______________________________",
      "name": "User"
    }
  ],
  "from": {
    "id": "______________________________",
    "name": "User",
    "role": "user"
  },
  "locale": "de-de",
  "serviceUrl": "http://localhost:64085"
}

Are you able to test with a browser using both English and Turkish locales?

Related: https://github.com/microsoft/BotFramework-Emulator/issues/2268

Hi @dmvtech , maybe I can explain better with this example.

I have a bot build in Bot Composer , the localization is Turkish. It works in local webchat.

It works in the Emulator when we changed the locale as you suggested:

Then we publish bot to Azure. And In the Azure Bot APP Test Webchat , the bot is not running.

We got this error when we publish the bot to Azure as locale tr-TR (Turkish.)

When the bot’s locale is en-us (english), it works.
The question is : Should we add a property to publishing profile when we are publishing in Turkish? We can send you the publishing profile as json.

@humak
When you publish, it will publish all locales that you have created/configured. Just to confirm; you did not remove en-US locale from the bot, correct?.
So, my assumption is that there is something wrong with the published bot, regardless of locale. The Test in WebChat and Azure portal are obviously running in English (I'm assuming en-US).
One thing you can do is use Ngrok and test your local running bot through the Test in WebChat tool. This will confirm that everything but the bot running in Azure (App Service) is working correctly.
Otherwise:

Have you tried re-publishing?
Gateway timeout usually means the bot was running long. Does it do anything at the beginning (of the conversation start), that might take a long time (API call, etc)?

Hi @humak. Have you tried any of the above? Any answers to the questions above?

Closing due to lack of response. Please comment with answers to above questions if you still need assistance.


[Config] rename "n_bases" and "n_heads" to "num_bases" and "num_heads"

Follow the naming convention, we need to replace all n_ to num_ for those use the "n" for the "number".
So, rename "n_bases" and "n_heads" to "num_bases" and "num_heads"

Closed by #138


testing, ignore

omg, there's so many working dirs, which one should I use?

"playbook_dir": "/var/lib/zuul/builds/a1932656c7d84c6bab71fb1e6d4746a8/untrusted/project_0/github.com/packit-service/deployment/playbooks/zuul"
"ansible_user_dir": "/home/zuul-worker"
"zuul.project.src_dir": "src/github.com/packit-service/deployment"
"zuul_workspace_root": "VARIABLE IS NOT DEFINED!: 'zuul_workspace_root' is undefined"
"zuul.executor.src_root": "/var/lib/zuul/builds/a1932656c7d84c6bab71fb1e6d4746a8/work/src"
"zuul_work_dir": "VARIABLE IS NOT DEFINED!: 'zuul_work_dir' is undefined"


Hijack self.backend instead of trying to find a cache

The only requirement for a cache backend that will work with Jobtastic is that it has an atomic increment operation. Redis (and probably others) have this, so there's no reason we should limit the options to Memcached only. Ideally, #8 will be completed first so that we don't have to do a bunch of implicit configuration parsing, but at worst, we can look for Redis the same way we look for memcached.
Maybe there's a way to make this a bit more flexible though. It seems that all cache backends for both Django and Werkzeug must provide an incr method, regardless of whether it's atomic. There seem to be two options then:

Make it a documentation thing and call out the fact that non-atomic incr backends will not always protect you from thundering herd issues. We can let people use non-atomic incr backends this way, and they'll probably still improve things a majority of the time.
Use some kind of backend whitelist (which is the way things are currently implemented) that knows that Redis and Memcached are atomic, while the others aren't.

Of these two options, it seems like providing the freedom to use non-atomic backends is preferable.

In the light of #37 I think that this issue should be Closed. I don't think the Celery Result Backend is suitable as-is because we can't control the timeout, and so the only sensible way to use it would be to wrap the backend client in a caching library, e.g. Werkzeug, which means us choosing one caching library over another. I think pluggable caches as proposed in #37 is a better approach.

Agreed. #37 is a superior approach.


add modifier to EntitlementChecker to only allow registered operator to be checkers

At a v high level, this looks fine to me - but I don't know what's going on in a lot of / most of this PR, so I'm not sure I'm the best person to approve it. If you can't get someone else, let me know if you want to jump on a call.
request for changes for a nit.

@clemire The main component that's either missing or I'm not able to asses is whether or not we're registering operators prior to calling registerNode on the entitlement checker contract. If not, any calls to registerNode will fail on tests.
cc. @brianathere

I have added 2 steps that first register a node operator and that operator than registeres the xchain nodes. Not sure if this is the best place for it because running the script a second time will now give an error because it tries to register an operator/node for the second time. For CI that isn't an issue, for development it can be annoying.

yes, can you move these calls to the create_multi.sh script and fix the hard coded local_single (change to ${RIVER_ENV})?

I have added 2 steps that first register a node operator and that operator than registeres the xchain nodes. Not sure if this is the best place for it because running the script a second time will now give an error because it tries to register an operator/node for the second time. For CI that isn't an issue, for development it can be annoying.

yes, can you move these calls to the create_multi.sh script and fix the hard coded local_single (change to ${RIVER_ENV})?

Actually there are several things that need to happen for this to work, i will update this pr


Ensure alternative installations of Redis can be used

Currently, our README uses a cluster local install of Redis, but this is likely not our typical use case.

Currently blocked by this issue:
https://github.com/knative-sandbox/eventing-redis/issues/65

Currently blocked by this issue:
https://github.com/knative-sandbox/eventing-redis/issues/65

/kind enhancement

/kind enhancement

knative-sandbox/eventing-redis#65 has been merged and closed


init

[Performance] Avoid lock contention for getting count of inner dictionary size

[x] You've read the Contributor Guide and Code of Conduct.
[x] You've included unit or integration tests for your change, where applicable.
[x] You've included inline docs for your change, where applicable.
[ ] If any gains or losses in performance are possible, you've included benchmarks for your changes. More info
[ ] There's an open issue for the PR that you are making. If you'd like to propose a new feature or change, please open an issue to discuss the change or find an existing issue.

Summary of the changes (Less than 80 chars)
Avoid lock contention for getting count of inner dictionary size in EventBasedLRUCache.
Description
We have seen bottlenecks in some of internal cloud services where EventBasedLRUCache<TKey, TValue> calls Count() for its inner _map. This property calculation brings with heavy lock contention, espeicial in SetValue() and Compact()
This change adds a new private field int this._mapSize to be updated during entries in _map being updated. _mapSize will be updated as atomic and only used to new cache size calculation so, there should not be beheavior change from observation outside.

@microsoft-github-policy-service agree company="Microsoft"

@ms-ruochenyu Can you please open and link an issue to this?

@ms-ruochenyu what is the performance gain?

@ms-ruochenyu what is the performance gain?

@brentschmaltz, below is a sample we collected in 1 of internal workload. The CPU cost was clustered to taking bucket write lock inside ConcurrentDictionary.Count(). So does Compact() call.

@ms-ruochenyu i am unable to understand your CPU diagram.


Use at least 8 threads in Jetty thread pool

Motivation
Use at least 8 threads to avoid having Jetty go into threads starving and
having the possibility of getting into a deadlock where a Jetty thread is
waiting for another HTTP call to complete in same thread.
This solve the issues of requests timing out when the broker is making REST
calls to itself. Such examples are when running in standalone mode and
creating a function.

rerun java8 tests

rerun java8 tests

rerun java8 tests

Merged in 2.3.1 at
49baf5d

Can we make the number of jetty threads configurable? Looks like it's impacting a bunch of other issues that involves admin tasks: #5417, #3877, #3878...

@vicaya That is already configurable. The only question was always what was the most conservative default value.


Place Collection Upload Issue

I am attempting to make a place collection entitled "Decolonization of French West Africa," but it gives me a server error when I try to save it. Attached is a screenshot of what I have submitted in the fields

It now works and I successfully published it. Thank you!


Update UI when view more pressed

When user clikcs on the president, user will see more details. Will show bio if available.

Kindly check here to see the issue https://github.com/unclebay143/nigeria-presidents/issues/39#issue-1013932406


fix-contact-picker-compatibility && update phone_numbers_parser

flutter_native_contact_picker: ^0.0.10
phone_numbers_parser: ^9.0.3


add kxbmap-configs (and move scalaz from 7.1 to 7.2)

this also permits upgrading scalaprops (master requires scalaz 7.2)
the only other project we had with a direct scalaz dependence was
specs2, and specs2 actually prefers to be on 7.2 these days
(though they also publish a 7.1-based artifact)

test run: https://scala-ci.typesafe.com/view/scala-2.12.x/job/scala-2.12.x-integrate-community-build/734/consoleFull

fyi @xuwei-k @kxbmap

failing as expected because of https://issues.scala-lang.org/browse/SI-9946, but let's see if everything else passes

I'll make a separate PR with just the scalaz/scalaprops bump, and we'll return to this one once SI-9946 is resolved (or worked around)

(the branch name is now inappropriate, but I don't want to have to make a new PR)

rebased. new test run, against newer Scala SHA with the SI-9946 fix: https://scala-ci.typesafe.com/view/scala-2.12.x/job/scala-2.12.x-integrate-community-build/784/consoleFull

new test run: https://scala-ci.typesafe.com/view/scala-2.12.x/job/scala-2.12.x-integrate-community-build/785/consoleFull

and again: https://scala-ci.typesafe.com/view/scala-2.12.x/job/scala-2.12.x-integrate-community-build/786/consoleFull


Problem with Installing Amplify Libraries with React 18

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

How did you install the Amplify CLI?
npm install aws-amplify aws-amplify-react-native amazon-cognito-identity-js @react-native-community/netinfo @react-native-async-storage/async-storage @react-native-picker/picker
If applicable, what version of Node.js are you using?
18.4.0
Amplify CLI Version
9.0.0
What operating system are you using?
Mac
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
No manual changes made
Amplify Categories
auth
Amplify Commands
Not applicable
Describe the bug
When I go to install npm install aws-amplify aws-amplify-react-native amazon-cognito-identity-js @react-native-community/netinfo @react-native-async-storage/async-storage @react-native-picker/picker
I get an error that says:

Expected behavior
I expected to have installed the package. But it seems that it doesn't like to work with React 18.
Reproduction steps

Install Amplify Libraries (npm install aws-amplify aws-amplify-react-native amazon-cognito-identity-js @react-native-community/netinfo @react-native-async-storage/async-storage @react-native-picker/picker)

GraphQL schema(s)

# Put schemas below this line

Project Identifier
52be70da1fcebf94b086507ff6fe323e
Log output

# Put your logs below this line

Additional information
No response

Hey @ezekielharvey :wave: thanks for raising this! I'm transferring this over to our js repo for visibility


NIFI-8642 Select the default Old Gen Memory Pool for Memory Reporting…

Select the default Old Gen Memory Pool for Memory Reporting Task
https://issues.apache.org/jira/browse/NIFI-8642
Description of PR
Select the default Old Gen Memory Pool for Memory Reporting Task
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically main)?

[x] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[x] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[x] Have you verified that the full build is successful on JDK 8?
[x] Have you verified that the full build is successful on JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI for build issues and submit an update to your PR as soon as possible.

LGTM+1
Merged into main.


chore(osio): remove blank booster and dependencies

Ike Plugins (test-keeper)
Thank you @gastaldi for this contribution!

It appears that no tests have been added or updated in this PR.
Automated tests give us confidence in shipping reliable software. Please add some as part of this change.
If you are an admin or the reviewer of this PR and you are sure that no test is needed then you can use the command /ok-without-tests as a comment to make the status green.
For more information please head over to official documentation. You can find there how to configure the plugin.

/ok-without-tests

Hey @gastaldi! It seems you tried to trigger /ok-without-tests command, but this will not have any effect due to insufficient permission. You have to be admin or requested reviewer or pull request approver, but not pull request creator for this command to take an effect.

/ok-without-tests

(it seems hooks are dead)

I suggest to put it on hold until we remove all the client code.

fabric8-ui currently has the Blank in it.
Why are you removing it?
Are you removing it from the launcher-frontend?

Why are you removing it?
Because it was decided as per @bartoszmajsak's email

Are you removing it from the launcher-frontend?
Yes

Doesn't fabric8-ui still get it's UI for the launcher from your launcher-frontend? If so, then when you remove it there, it will remove it for us too.

@joshuawilson you're talking about ngx-launcher, which yes, will be removed too. But I believe there are some changes needed in fabric8-ui where this component is included.

I've created a story for the fabric8-ui work at https://openshift.io/openshiftio/Openshift_io/plan/detail/903
Please let me know when the launcher-ngx change is done and then we can follow up.

Depends on https://github.com/fabric8-ui/fabric8-ui/pull/3393


Request: Add section for testing crates

Would it make sense to have a section on testing crates? I was trying to find something for http mocking but it would be nice to know what other tools there might be. Below is what I found after a quick search:
https://crates.io/crates/mockito
https://crates.io/crates/httpmock
https://crates.io/crates/wiremock

Yes, this is very important. I'll try to work on it this week.

Thanks! Looking forward to learning about new crates 😄

Hey, just a friendly ping on this issue 😃

@UebelAndre thanks for the reminder. I'll try to have this done tomorrow 👌


HDX-9732 reading patches from github

the logic assumes that patches are structured in folders with the pattern: YEAR/MONTH/hapi_patch_SEQNUMBER_SUBCATEGORY.json as in 2023/11/hapi_patch_01_abc.json

first thing it finds the latest commit on the branch

then looks through the folders of the repo at that specific commit

new env var has been introduced for specifying the token for reading the Github repo HWA_PATCH_TOKEN. This one should be specified in .env

there 2 more env vars HWA_PATCH_REPO_URL, HWA_PATCH_BRANCH_NAME but we probably can live with the defaults for these ones

the tests are using this patch repo: https://github.com/alexandru-m-g/test-hdx-hapi-write-app-patches. I've created a special branch for the tests in this PR (test-patch-discovery)

there were changes to the docker-compose.yml -> so containers should be recreated

Closing PR as the requirements have changed


Update OpenLiberty tag

Before excluding OpenLiberty external test, try updating tags to see if it resolves failures.

Running Grinder at: https://ci.adoptium.net/job/Grinder/7284

Thanks for the reviews, I will merge it as it appears to fix the issue with JDK11 failures.  Once merged, I will also try against JDK8 and re-include if fixes it there too.


Magic ANetResponseType::getTransactionResponse method?

I'm having a hard time following the sample SDK code for parsing responses. All the sample code uses this line after constructing a controller and executing a response:
<?php

$response  = $controller->executeWithApiResponse(
  \net\authorize\api\constants\ANetEnvironment::SANDBOX
);
$tresponse = $response->getTransactionResponse();

The getTransactionResponse method does not exist in the method. I was expecting to see a __call() method that reflects the method name and accesses the corresponding property, or a class being extended where this method was present, but I didn't find anything.
Is there some php magic here that I'm missing? I would really love to know how this ticks. It's hard to use an SDK with methods that aren't enumerated in the code!

getTransactionResponse() is part of the Authorize.Net PHP SDK. It is defined  here in the php sdk.


Added darwinbox logo for thanos webpage

[ ] I added CHANGELOG entry for this change.
[x] Change is not relevant to the end user.

Changes

Verification

Added a new block in website/data/adapters.yaml for including Darwinbox company on thanos.io webpage.
Added a logo of the company under website/static/logos


Jan 22 08:51:45 plex sh[824]: ssh: /usr/lib/plexmediaserver/libcrypto.so.1.0.0: no version information available (required by ssh)

Hello,
I found this project after suffering problems with my virtual plex machine not being powerful enough to transcode movies. I have a couple processing beasts that I would like to load balance with. Throughout my setup, I run debian 8.6.
I have set up everything according to the debian wiki. My plex version is: Version <IP_ADDRESS>1
Everything mounts and seems to be working, but as soon as I try to watch a movie I get the following error before plex reports it was unable to play the media.
Jan 22 08:51:45 plex sh[824]: ssh: /usr/lib/plexmediaserver/libcrypto.so.1.0.0: no version information available (required by ssh)
Jan 22 08:51:45 plex sh[824]: ssh: /usr/lib/plexmediaserver/libcrypto.so.1.0.0: no version information available (required by ssh)
Jan 22 08:51:45 plex sh[824]: ssh: /usr/lib/plexmediaserver/libcrypto.so.1.0.0: no version information available (required by ssh)
Jan 22 08:51:45 plex sh[824]: ssh: /usr/lib/plexmediaserver/libcrypto.so.1.0.0: no version information available (required by ssh)
Jan 22 08:51:46 plex sh[824]: Traceback (most recent call last):
Jan 22 08:51:46 plex sh[824]: File "/usr/local/bin/prt_local", line 9, in
Jan 22 08:51:46 plex sh[824]: load_entry_point('prt==0.4.2', 'console_scripts', 'prt_local')()
Jan 22 08:51:46 plex sh[824]: File "build/bdist.linux-x86_64/egg/prt.py", line 297, in transcode_local
Jan 22 08:51:46 plex sh[824]: File "/usr/lib/python2.7/subprocess.py", line 710, in init
Jan 22 08:51:46 plex sh[824]: errread, errwrite)
Jan 22 08:51:46 plex sh[824]: File "/usr/lib/python2.7/subprocess.py", line 1335, in _execute_child
Jan 22 08:51:46 plex sh[824]: raise child_exception
Jan 22 08:51:46 plex sh[824]: OSError: [Errno 2] No such file or directory
I have verified that all mentioned files in the error messages here actually exist on both master and slave.
Hope there is a fix for this - it would really benefit me!
Thanks,
Kristoffer

It appears that it is this line that is failing, which would imply that either get_transcoder_path is returning the wrong path to the transcoder or that the transcoder is not installed.  On Linux, this path should be /usr/lib/plexmediaserver/plex_transcoder.

Ok i see you used the Wiki Article from me ^^
Please dont use the latest release just git clone the actual master because there the path is fixed.
@wnielson please push a new release from the actual master... so its working on all linux versions. Thanks

@wnielson ty for the push.
@milligan just use the 0.4.2 release from now it should be working flawless.

@ChoosenEye Released version 0.4.5

Thank you both, that seems to have corrected the problem and it's now working. I'm amazed at how CPU intensive the converter is though.
Has it been benchmarked to give an indication on how hard it will hit my processing machines? i.e if I set up a slave on my gaming computer - will I notice transcoding while playing my games?

no problem.
Yes its CPU intensive but you are transcoding videos ^^ The Plex Remote transcoder doesn't minimize CPU Usage its just load balance between your transcoding hosts.
For notice 2k Passmark Score you will need for around 1x 1080p 6-8mbps stream.
@milligan if all is working now please close the issue.


Add PlanetScale service tokens

Adds support for PlanetScale service tokens.

The docs are autogenerated but missing a bunch of stuff and I'd like to fix that before this is merged.

This should be good to go @s1ntaxe770r

Thanks!!, this was a big one on the roadmap


Expose detected-link color to themes

Extracted from discussion at #18357

@alexandrudima With respect to detected-link, this was not covered before becuse, I think this is foreground color and AFIK we cannot contribute foreground colors from themes other than tokens. Hence this was not handled then.

linkForeground it is.

How exactly do I set this in my themes? I've tried many scope combinations.
<dict>
        <key>name</key>
        <string>String URLs</string>
        <key>scope</key>
        <string>string.detected-link, detected-link, string.linkForeground, linkForeground</string>
        <key>settings</key>
        <dict>
          <key>foreground</key>
          <string>#C678DD</string>
        </dict>
      </dict>

But it's still not accepting my tmTheme color for it. I'm trying this on the latest VSC Insider's Build.

@joshpeng In the same way caret, foreground, invisibles, selection, etc. can be specified: in a block without any scope.
https://github.com/Microsoft/vscode/blame/master/extensions/theme-monokai/themes/Monokai.tmTheme#L14

The way VSCode is handling links is kind of weird for theming at the moment (at least compared to Atom).
Since there is no scope controlling the link coloring, this happens when I put a linkForeground rule:

Comparted to Atom:

How can we avoid this?

@akamud there are two links in your first screenshot. Both links get the purple color (presumably you picked a purple color, right?)
I'm not sure what is weird about it, would you like to be able to differentiate between links that are relative and links that are absolute?

I'm sorry for not explaining myself.
Yes, I used linkForeground to get a purple link, but I think it is weird for links to be purple even when they are commented code or a string inside HTML.
Atom exposes TM Scopes so I can color links inside strings, JSON and commented code differently. VSCode does not provide a way of doing this, since the only scope applied is the global scope linkForeground.
Before the TM Scopes changes I could achieve this in VSCode using the detected-link scope, the same scope Atom used.
Was this intended? Do you plan on providing a way to achieve this again?

Thank you, I understand now. Yes, unfortunately, this is intended behavior.
Links in VS Code are not detected from TM grammars, but rather from document link providers. So links come in "from the side", on top of TM scopes (from the extension host or from our built-in http:// , https:// or file:// link detectors).
By the time the view needs to paint the links, it now only has access to the token color and no longer to the underlying TM scopes that produced the token color. So, at this time, it is not possible to distinguish links inside strings vs links inside comments and color them differently.

I see, thanks for taking the time to explain this :)


Update and reintegrate Danger

Changes in this pull request
Issue fixed: #
Checklist

[ ] All tests pass. Demo project builds and runs.
[ ] I added tests, an experiment, or detailed why my change isn't tested.
[ ] I added an entry to the CHANGELOG.md for any breaking changes, enhancements, or bug fixes.
[ ] I have reviewed the contributing guide

Oops. I did it again! Sorry! 😅


Add warning about loss of data when page is being refreshed

We used to have this way back near the very beginning, so if it doesn't come to us straight away, we can check out the files from the earliest commits.

Would it be better to save the data between sessions?

@wacii: We're not going to be saving any data, partially because this is sensitive information and we'd have to jump through a whole lot of hoops to be able to store that kind of information.

Even if its just stored on the device?

It was enough of a question that it was decided it wasn't worth going into.


Initial manual

This need to be under doc/manual/ and it could likely be based on pandoc and markdown files to compile into .pdf and .html

Is this manual about setting up the environment?
What is the structure of the directory?

We could look at doing something similar to https://github.com/EnterpriseDB/barman/tree/master/doc/manual just with cmake.
Better ideas are welcomed !

Should we integrate CMake directly into our project, or should we keep it separate in the manual directory?
This way, if a user wants to generate it, they can do so; otherwise, we won't generate it by default.

I don't understand - it should be part of the standard cd build/ && cmake .. && make setup

Okay, thanks. I'll work on it.


chore: add ensure-npm-valid-dependencies.sh to sanity-checks make target

We forgot to do so on the PR that introduces this script.
This commit also moves all the CI code to separate scripts.
See: https://github.com/resin-io/etcher/pull/1371
Signed-off-by: Juan Cruz Viotti<EMAIL_ADDRESS>

Works on Travis but fails on Appveyor - weird...

Hm, must be the npm version running on Appveyor. I'll make sure they are synced.

A shame there's so much redundancy (repeated info) between all the scripts, but given the platform-specific and CI-service-specific nature to all this, I guess there's not much that can be done about that.

Yeah, I want to at least try moving everything to bash scripts we can call with MinGW. What do you think? I might do it in another PR soon :)

Looks like we have to prefix any external command we call in a .bat file with call, otherwise the script exits successfully, but the commands don't actually run :/

All tests finally green :D

Yeah, I want to at least try moving everything to bash scripts we can call with MinGW. What do you think?

You could, but there'll still be differences between the setup needed on Appveyor and Travis (and/or differences between OSes), and I was thinking also about the duplication of the "make" commands in the Travis scripts for the Docker and non-Docker versions.


Simpler syntax for git tag

https://github.com/conda-forge/iverilog-feedstock/blob/6713c41a19f8e1769ebc2e1ee3eb4f19a8d6077a/recipe/meta.yaml#L2
Not a real issue, but could probably be soemthing like
{{ version | replace(".", "_") }}

instead of duplicating the version.

Yup, nice one @hmaarrfk, thanks!

Very much so. The tools for FPGAs are far inferior than those for essentially any other programming language in existence.

Indeed...! 🤦♂


Should I run this test in QEMU?

tests/run.sh
#!/bin/bash
set -ex
gcc -o testgen -Wall -O1 -I../cproofs testgen.cc
./testgen > tests.c
/opt/riscv64b/bin/riscv64-unknown-elf-gcc -Wall  -march=rv64gb -Os -o tests tests.c
/opt/riscv64b/bin/spike --isa=RV64GCB pk tests

I want to modify the last command to use qemu, But it crashed during runtime.
./qemu-riscv64 -cpu rv64,x-b=true tests
[1]    22631 illegal hardware instruction (core dumped)  ~/Downloads/qemu-riscv64 -cpu rv64,x-b=true tests

Should I run this test in QEMU https://github.com/sifive/qemu/tree/rvb-upstream-v6?

IIRC SiFive's qemu only implement zba, zbb and zbs, so...that should be expected behavior I think...