Mem-leak in test-registrable-domain.exe

When building for MSVC (x86) in _DEBUG mode; options -MDd -DBUILTIN_GENERATOR_LIBICU=1 -DWITH_LIBICU=1,
there is this memory-leak report which I suspect is from inside ICU (since no file-names or lines are shown):
have 9189 suffixes and 8 exceptions
Summary: All 95 tests passed
Dumping objects ->
{528} normal block at 0x0C138B78, 12252 bytes long.
 Data: <                > 01 00 00 80 00 00 00 00 00 00 00 00 01 00 00 80
{527} normal block at 0x0C149A20, 52 bytes long.
 Data: <x    @ g A g    > 78 8B 13 0C A0 40 82 67 E0 41 82 67 00 00 00 00
{526} normal block at 0x00DB3B60, 228 bytes long.
 Data: <         <    _h> E4 00 00 00 00 00 00 00 E0 3C 13 0C 00 9A 5F 68
{525} normal block at 0x0C133CE0, 28 bytes long.
 Data: <    p _h        > 00 00 00 00 70 99 5F 68 00 00 00 00 01 00 00 00
{524} normal block at 0x0C133FF8, 28 bytes long.
 Data: <    0  g        > 00 00 00 00 30 1D C8 67 00 00 00 00 01 00 00 00
{523} normal block at 0x0C133BC0, 28 bytes long.
 Data: <D  g   g   g    > 44 95 8E 67 00 85 8F 67 90 85 8F 67 01 00 00 00
Object dump complete.
0 bytes in 0 Free Blocks.
0 bytes in 0 Normal Blocks.
36459 bytes in 502 CRT Blocks.
0 bytes in 0 Ignore Blocks.
0 bytes in 0 Client Blocks.
Largest number used: 59979 bytes.
Total allocations: 67519 bytes.
Detected memory leaks!

I suspect the test_iso() function triggers this 12252 bytes long leak.
My ICU package was produced by vcpkg install icu:x86-windows-static. And yes, I do link with the ICU *d.lib  libraries.
No leaks in any other test-program.

Not sure what I can do. Some libraries allocate buffers for regular used data on first use and then never free it again 🤷🏻♂
If you amend the code to run test_iso() in a loop (e.g. 100 times) and the reported amount of memory doesn't increase, then we should not waste our time on this.

If you amend the code to run test_iso() in a loop (e.g. 100 times) and the reported amount of memory doesn't increase,

That's exactly what is happening.
OK, so I'm closing this.


Creating a Callback URL for Safaricom Daraja API - (Lipa na Mpesa) with Firebase Cloud Functions in Android

Article checklist
Pre-submission checks

Please ensure that you have only one open topic suggestion (issue) + in-review article (linked pull request) at a time. This will ensure that we complete the review process in a timely manner from inception to publishing

If you have not already, please go over our Resources Page for more tips and tricks on how to improve your overall technical writing so reviews are swifter, increase the chance of a payout and provide more value to our readers.

For first-time contributors and for more details on our submission guidelines, see our Contributing Guidelines.

Formatting and structure checks

[x] Have you used the correct folder and file structure? - see Contribution Guidelines

[x] Is your article properly formatted in Markdown? - see Github Markdown guide

[x] Have you used our preferred heading structure? - H3 (###) and up

[x] Does your article follow any of the suggested structure formats? - see example formats

[x] Is your article over 750 words? - Tool: Wordcounter.net

[x] Is the article you are submitting an in-depth and unique article? Does it go beyond what is in the official docs and what is covered in other blog sites. See these articles as examples.

[x] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.

[x] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.

[x] Have you included a hero image that is 600x400 pixels and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images

Grammar & plagiarism checks

[x] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid - Grammarly
[x] Please place your article through a 3rd party plagiarism checker. We suggest using Quetext, this tool is free and has a daily limit. We typically accept articles with 10% or less.
[x] Have you checked your article for readability? - Tool: Hemmingway
[x] Have you added sources for quotes and images that aren't yours?

Technical checks

[x] Are your code snippets properly formatted for syntax highlighting - see Syntax guide
[x] Have you checked your code runs correctly and you've highlighted all necessary dependencies for installation?
[x] Are the software programs and packages you're highlighting in your article up to date, using current versions and not deprecated?

Topic suggestion this closes

Remove the backticks and add the issue number below to link and close your Topic Suggestion (issue) when your article has been published (PR has been merged). See this video for more details.

This closes #4328

Plagiarism check done ✔

Hello @Ericgacoki
I have made the necessary changes.

You might have forgotten to hide the phone number in this Toast image. Kindly fix this as I proceed with the review. 😎

👏 Awesome tutorial @osirevaline. Kindly take a look at my changes!

@osirevaline
Thank you for your patience as our team has been working diligently to work through the recent increased volume of articles coming in.
We are going through are ready articles in the order they were received and working on improving our review time as well.
🚀

jumping on this one

👍 overall nice work - very detailed and straight forward tutorial
Please review and suggestions/changes made by your peer reviewer and/or myself during a final review.


Fix broken restore defaults in data table

Description
The Restore defaults action in the DataTable was not properly restoring the hidden columns when selected. This was because the TableContext which is used to pass to the reducer for restoring, was being modified when hiding the columns. Changing the code to not to modify fixes the issue.
Steps to reproduce:

Run a BLAST job
Select the results when it has finished running
Click on the genome in a sequence it was run against to open the results table (far right)
Click on the Actions select field (far left) in the table header
Select Show/hide columns option and select any fields in the popup panel to hide
Now select Restore defaults to see if the hidden columns have been restored

Related JIRA Issue(s)
ENSWBSITES-1870
Deployment URL(s)
http://fix-data-table.review.ensembl.org
Views affected
BLAST -> Results view -> Sequence -> Results table

I've just realised there are tons of problems with the Actions section of the data table:

The word "Actions" appears twice, and in fact "Actions" is a selectable option (this must already be in our main branch)

More related to "restoring defaults", if you click on "Restore defaults" when you have hidden some table rows, the rows selection will be reset, but the actions UI will remain in the "select rows" mode. I doubt that this is correct. Don't know if this is something you want to include in this PR or not:

https://user-images.githubusercontent.com/6834224/223675355-a0028156-c394-4da3-aecc-b554105bddee.mov

@azangru I've fixed both the issues in ca944d4


[REQUEST] Model serving via deepspeed's inference module

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
I am trying to run my model serving code in a model-parallel fashion. The tutorial shows how to run code on multi-GPU but the data is predefined, which cannot be used for serving. My original code is using fastapi to do the serving work. When using deepspeed --num_gpus n example.py the fastapi server will also be initiated n times, which cause port conflict.
Describe alternatives you've considered
Do I have to first start the model in parallel using deepspeed in one script and then start another script for fastapi, and finally connect them somehow?
Additional context
None.

Hi @callzhang
Could you please provide a test script so that I can try on my end and can understand the problem better?
Thanks,
Reza

Here is the minimum code I tried:
from fastapi import FastAPI, Request, Response, Query
from transformers import pipeline
import deepspeed, torch, os, uvicorn

app = FastAPI()

local_rank = int(os.getenv('LOCAL_RANK', '0'))
world_size = int(os.getenv('WORLD_SIZE', '1'))
generator = pipeline('text-generation', model='gpt2', device=local_rank)

@app.get("/gen")
def generate(text):
    return generator.generate(text, max_length=100)

if not torch.distributed.is_initialized() or torch.distributed.get_rank() == 0:
    print(f'initiating server on rank: {local_rank}')
    uvicorn.run(
        "min_example_deepspeed_mp:app",
        host="<IP_ADDRESS>", port=8500,
        log_level="info",
        workers=1
    )

Then I ran deepspeed --num_gpus 2 nlp/sentence_generation/min_example_deepspeed_mp.py and I got the following error:
[2021-11-03 01:33:39,359] [WARNING] [runner.py:122:fetch_hostfile] Unable to find hostfile, will proceed with training with local resources only.
[2021-11-03 01:33:39,373] [INFO] [runner.py:360:main] cmd = /home/stardust/anaconda3/bin/python -u -m deepspeed.launcher.launch --world_info=eyJsb2NhbGhvc3QiOiBbMCwgMV19 --master_addr=<IP_ADDRESS> --master_port=29500 nlp/sentence_generation/min_example_deepspeed_mp.py
[2021-11-03 01:33:39,993] [INFO] [launch.py:80:main] WORLD INFO DICT: {'localhost': [0, 1]}
[2021-11-03 01:33:39,994] [INFO] [launch.py:86:main] nnodes=1, num_local_procs=2, node_rank=0
[2021-11-03 01:33:39,994] [INFO] [launch.py:101:main] global_rank_mapping=defaultdict(<class 'list'>, {'localhost': [0, 1]})
[2021-11-03 01:33:39,994] [INFO] [launch.py:102:main] dist_world_size=2
[2021-11-03 01:33:39,994] [INFO] [launch.py:104:main] Setting CUDA_VISIBLE_DEVICES=0,1
initiating server on rank: 1
initiating server on rank: 0
initiating server on rank: 0
Traceback (most recent call last):
  File "nlp/sentence_generation/min_example_deepspeed_mp.py", line 17, in <module>
    uvicorn.run(
  File "/home/stardust/anaconda3/lib/python3.8/site-packages/uvicorn/main.py", line 447, in run
    server.run()
  File "/home/stardust/anaconda3/lib/python3.8/site-packages/uvicorn/server.py", line 68, in run
    return asyncio.run(self.serve(sockets=sockets))
  File "/home/stardust/anaconda3/lib/python3.8/asyncio/runners.py", line 44, in run
    return loop.run_until_complete(main)
  File "uvloop/loop.pyx", line 1501, in uvloop.loop.Loop.run_until_complete
  File "/home/stardust/anaconda3/lib/python3.8/site-packages/uvicorn/server.py", line 76, in serve
    config.load()
  File "/home/stardust/anaconda3/lib/python3.8/site-packages/uvicorn/config.py", line 448, in load
    self.loaded_app = import_from_string(self.app)
  File "/home/stardust/anaconda3/lib/python3.8/site-packages/uvicorn/importer.py", line 21, in import_from_string
    module = importlib.import_module(module_str)
  File "/home/stardust/anaconda3/lib/python3.8/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1014, in _gcd_import
  File "<frozen importlib._bootstrap>", line 991, in _find_and_load
  File "<frozen importlib._bootstrap>", line 975, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 671, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 843, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/home/stardust/algorithms-playground/nlp/sentence_generation/min_example_deepspeed_mp.py", line 17, in <module>
    uvicorn.run(
  File "/home/stardust/anaconda3/lib/python3.8/site-packages/uvicorn/main.py", line 447, in run
    server.run()
  File "/home/stardust/anaconda3/lib/python3.8/site-packages/uvicorn/server.py", line 68, in run
    return asyncio.run(self.serve(sockets=sockets))
  File "/home/stardust/anaconda3/lib/python3.8/asyncio/runners.py", line 33, in run
    raise RuntimeError(
RuntimeError: asyncio.run() cannot be called from a running event loop
initiating server on rank: 1
Traceback (most recent call last):
  File "nlp/sentence_generation/min_example_deepspeed_mp.py", line 17, in <module>
    uvicorn.run(
  File "/home/stardust/anaconda3/lib/python3.8/site-packages/uvicorn/main.py", line 447, in run
    server.run()
  File "/home/stardust/anaconda3/lib/python3.8/site-packages/uvicorn/server.py", line 68, in run
    return asyncio.run(self.serve(sockets=sockets))
  File "/home/stardust/anaconda3/lib/python3.8/asyncio/runners.py", line 44, in run
    return loop.run_until_complete(main)
  File "uvloop/loop.pyx", line 1501, in uvloop.loop.Loop.run_until_complete
  File "/home/stardust/anaconda3/lib/python3.8/site-packages/uvicorn/server.py", line 76, in serve
    config.load()
  File "/home/stardust/anaconda3/lib/python3.8/site-packages/uvicorn/config.py", line 448, in load
    self.loaded_app = import_from_string(self.app)
  File "/home/stardust/anaconda3/lib/python3.8/site-packages/uvicorn/importer.py", line 21, in import_from_string
    module = importlib.import_module(module_str)
  File "/home/stardust/anaconda3/lib/python3.8/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1014, in _gcd_import
  File "<frozen importlib._bootstrap>", line 991, in _find_and_load
  File "<frozen importlib._bootstrap>", line 975, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 671, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 843, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/home/stardust/algorithms-playground/nlp/sentence_generation/min_example_deepspeed_mp.py", line 17, in <module>
    uvicorn.run(
  File "/home/stardust/anaconda3/lib/python3.8/site-packages/uvicorn/main.py", line 447, in run
    server.run()
  File "/home/stardust/anaconda3/lib/python3.8/site-packages/uvicorn/server.py", line 68, in run
    return asyncio.run(self.serve(sockets=sockets))
  File "/home/stardust/anaconda3/lib/python3.8/asyncio/runners.py", line 33, in run
    raise RuntimeError(
RuntimeError: asyncio.run() cannot be called from a running event loop
sys:1: RuntimeWarning: coroutine 'Server.serve' was never awaited
sys:1: RuntimeWarning: coroutine 'Server.serve' was never awaited
Killing subprocess 8312
Killing subprocess 8313
Traceback (most recent call last):
  File "/home/stardust/anaconda3/lib/python3.8/runpy.py", line 194, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/home/stardust/anaconda3/lib/python3.8/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/home/stardust/anaconda3/lib/python3.8/site-packages/deepspeed/launcher/launch.py", line 171, in <module>
    main()
  File "/home/stardust/anaconda3/lib/python3.8/site-packages/deepspeed/launcher/launch.py", line 161, in main
    sigkill_handler(signal.SIGTERM, None)  # not coming back
  File "/home/stardust/anaconda3/lib/python3.8/site-packages/deepspeed/launcher/launch.py", line 139, in sigkill_handler
    raise subprocess.CalledProcessError(returncode=last_return_code, cmd=cmd)
subprocess.CalledProcessError: Command '['/home/stardust/anaconda3/bin/python', '-u', 'nlp/sentence_generation/min_example_deepspeed_mp.py', '--local_rank=1']' returned non-zero exit status 1.

Thanks I will try this on my end

Any update on this? Is there another recommended way to do this, e.g. without using the deepspeed launcher?

Can you resolved it by gunicorn?

any updates on this issue @RezaYazdaniAminabadi

Any update on this?

Any update or workaround?

Face same problem in my project.

from transformers import pipeline
import transformers
import deepspeed
import torch
import os
from transformers.models.t5.modeling_t5 import T5Block
import sys
import torch.distributed as dist

local_rank = int(os.getenv('LOCAL_RANK', '0'))
world_size = int(os.getenv('WORLD_SIZE', '2'))

pipe = pipeline("text2text-generation", model="t5-v1_1-small", device=local_rank)

pipe.model = deepspeed.init_inference(
    pipe.model,
    mp_size=world_size,
    dtype=torch.float
)

pipe.device = torch.device(f'cuda:{local_rank}')

if not dist.is_initialized() or dist.get_rank() == 0:
    from flask import Flask
    app = Flask(__name__)

    @app.route("/")
    def hello_world():
        object_list = ["Is this review positive or negative? Review: this is the best cast iron skillet you will ever buy"]
        dist.broadcast_object_list(object_list, src=0)
        output = pipe(object_list)
        print(output)
        return "
Hello, World!
%s"%output
    app.run()

else:
    while True:
        object_list = [None]
        dist.broadcast_object_list(object_list, src=0)
        output = pipe(object_list)

I use 'broadcast_object_list' method to solve this problem.


Fixed info in setup.py

Summary
This pr fixed the author, project urls, in setup.py.
Checklist

[ ] If code changes were made then they have been tested.

[ ] I have updated the documentation to reflect the changes.

[x ] This PR fixes an issue.
[ ] This PR adds something new (e.g. new method or parameters).
[ ] This PR is a breaking change (e.g. methods or parameters removed/renamed)
[ ] This PR is not a code change (e.g. documentation, README, ...)

PR gone stale for months, with changes requested, closing due to inactivity.


Rdsimaps 3346 sample map link copy link

Exposes "Copy Link" button on Sample Maps.

Updated to use template.  Please re-review.  Thanks.

Looks good.


Not a issue [Can someone help in describing algo and payload structure for sending message]

Hi Team,
I am contributing in Sigalor whatsapp-web-reveng and stuck in sent message function.
I have created payload and able to send message but it does not work sometimes. Can someone describes the payload structure and sample code for help.

What kind of payload have you created? Did you create a custom *proto.WebMessageInfo or did you use one of our message types?

What kind of payload have you created? Did you create a custom *proto.WebMessageInfo or did you use one of our message types?

I have used Sigalor whatsapp-web-reveng  structure to create custom payload in python.

Ahh sorry I misread that. Probably you are better of creating an issue over at Sigalor´s repository.
Media handling is described quite well in the README his repository. See
You can look at our implementation here.

Ahh sorry I misread that. Probably you are better of creating an issue over at Sigalor´s repository.
Media handling is described quite well in the README his repository. See
You can look at our implementation here.

Thanks @SchulteMK , I am able to upload media but issue i am facing in sending text/media to user.
Can you please reference in your code, as i am new to Go.

All protobuf messages get send via this method. wac.writeBinary writes the data onto the websocket connection. Source
metric and flag are important constants to be set, otherwise the message won´t send. Our metric is message (16) and our flag is ignore (128). We used the same values for every message.

Thanks @SchulteMK, I will try to convert this into python.


Upgrade to Node 16

Fixes #281

If this is the right way to do this then I would do the same on https://github.com/coursier/cache-action

@alexarchambault I guess this is done?
BTW, not sure if this change should imply an increase in the version.

BTW, not sure if this change should imply an increase in the version.

You mean, a major or minor version bump? I don't know, I guess I'll tag the next version as v1.3.0.

Thanks for this PR, it's merged!

You mean, a major or minor version bump? I don't know, I guess I'll tag the next version as v1.3.0.

@alexarchambault I mean, I don't know either. That is why I just leave the question since not sure what is the recommendation or standard followed by other actions.


Adds a test case that can be solved using hasOwnProperty to the High Score Board exercise

The README.md in the High Score Board exercise mentions hasOwnProperty, but it was never used in the rest of the exercise, so I added a test case that can be solved using hasOwnProperty.
This is my first PR to this repository, so if I'm missing something, let me know. I'm not clear about what may or may not need to be updated on the problem-specifications repository, or in the hidden directories (like .meta).

All right, thanks for letting me know. I'll close this now.


Add Monkey Business Inc. parts from SpaceDock

This pull request was automatically generated by SpaceDock on behalf of Blacksky33, to add Monkey Business Inc. parts to CKAN.
Mod details:
name = /mod/584/Monkey%20Business%20Inc.%20parts
author = Blacksky33
abstract = Parts pack. first circular(ring int/ext fueltank) landing solution.
license = cc 4.0
Homepage = http://forum.kerbalspaceprogram.com/index.php?/topic/100643-11-monkey-business-inc-classicpartsoctolanderretroparts/&page=1
description =
Please direct questions about this pull request to Blacksky33.

Closing in favour of ref'ed PR (PR list cleanup mode engaged).


Infer codegen types from schema

Motivation

To build the XataRecord objects properly we need the schema of the tables

Impossible to build links or advanced types like dates without the schema

Before this PR

We gathered the schema on a network call and memoized it in the constructor of the client

Objective

Remove the extra network call
Provide the schema with the codegen
Infer the types from the schema tables

Breaking changes

Changed signature of the client constructor (codegen needs to be executed after upgrading)
Type names are not homogenised (singular/plural and case) anymore (I think this is better to be honest)

Will do another PR and try to pass an analyzer that finds for other unused dependencies


The firmware was successfully flashed, but there are issues with the display.

I don't know why my LCD screen shows a white screen after successfully burning the firmware. I have no idea where to start troubleshooting this issue; please help me.

Kconfig.projbuild.txt
device:ESP32S3-WROOM-1-N16R8、ST7789 -2.4

Hi, Can you tell me from Main.c which App demo is it running? Second thing for ST7789 configuration i recommend you turn on "Enable Frame Buffer" for Smooth Rendering.
Replace everything in main.c With Following snippet, This should draw a Orange rectangle. Let me know it it works or not.
#include "main.h"
#include "app.h"

void app_main(){
    start_uCanvas_engine();
    uCanvas_Scene_t* scene_1 = New_uCanvas_Scene();
    uCanvas_set_active_scene(scene_1);
    uCanvas_universal_obj_t* rect = New_uCanvas_2DRectangle(20,20,50,50);
    rect->properties.fill = FILL;
    uCanvas_Set_Color(rect,255,150,0);

}

I noticed I hardcoded the display resolution in display port file. I have fixed it on main branch, so now it should take resolution from configuration file. @zhenchuanpeng let me know if you face any issues :)

Hey buddy, I can't find the 33rd pin of the ESP32S3. I'm not sure if this is the reason. Besides, after replacing the code in the “main\main.c” file under this path and burning it in, there's still no effect.

I'll give it another try, including updating your code locally. Thanks for your explanation.

I can't find the 33pin of the ESP32S3
Configure it to pin that is available on your board. you can use any GPIO for connection display.

Sure,I've already set it to 39pin :)

@zhenchuanpeng  whats the issue in previous image? have you managed to resolve it?

Hey, buddy! The problem has been solved. I've made some adjustments to the configuration. Let me show you the effect. Kudos!

I'll study carefully how to operate it:)

@zhenchuanpeng whats the issue in previous image? have you managed to resolve it?

Hey, buddy! The problem has been solved. I've made some adjustments to the configuration. Let me show you the effect. Kudos!
img_v3_02hj_39192992-4340-4a39-8ee4-b80bf9728edg
I'll study carefully how to operate it:)

That's great! I see the colors are inverted for some reason. It should have looked like this :

@zhenchuanpeng Since this project is in development stage, especially with ST7789 Support there will be some bugs, unexpected behaviors and limitations. Feel free to report issues you encounter. Thank you for your interest in this project :D

@zhenchuanpeng Hi, another thing to note,  I have recently added support for Fonts, initially there was fixed font type of 10px size. you can now change font type property in your code from available selection of fonts. I recommend you use fetch current version of code. There is demo for how to use fonts as well :)

@zhenchuanpeng Hi, another thing to note, I have recently added support for Fonts, initially there was fixed font type of 10px size. you can now change font type property in your code from available selection of fonts. I recommend you use fetch current version of code. There is demo for how to use fonts as well :)

Of course, buddy. I'll handle the repo management based on your last update. Meanwhile, I've noticed some issues while running the repo, and I will provide you with feedback and some suggestions for improvement. I'll get back to you once I'm home.

I've noticed some issues while running the repo,

Alright Sure! Feel free to create new issues so we can keep things on track. I'll close this issue as of now :)


[Request] Pull all tags

Usecase:

I want to keep my folder structure as per https://help.phraseapp.com/en/articles/2136270-structuring-translations-how-to-keep-the-original-files
I'm using tags on PhraseApp as equivalent to my Symfony domains
I have one Symfony domain per view (so the translation file is not too big)
So I have a lot of tags and I keep adding new ones
I want to use the PhraseApp client in my CI/CD process

Right now, I have to create a configuration file where I list all my tags:
phraseapp:
  access_token: "some token"
  project_id: "some project id"
  file_format: yml
  pull:
    targets:
      - file: ./translations/<tag>.<locale_name>.yml
        params:
          tags: tag1, tag2, ....

I would like to have a way to say "all tags", for instance:
phraseapp:
  access_token: "some token"
  project_id: "some project id"
  file_format: yml
  pull:
    targets:
      - file: ./translations/<tag>.<locale_name>.yml
        params:
          tags: "*"

Ive came across to the same problem. Then question is why there is parameter called 'tags' if it can have only one tag.
push:
    sources:
    - file: ./public/locales/<locale_name>/<tag>.json
      params:
        file_format: i18next
  pull:
    targets:
    - file: ./public/locales/<locale_name>/<tag>.json
      params:
        file_format: i18next
        tags: 'common'

And why the push command works fine and it can recognise multiple tags, but for pull you have to target it one by one.
Is there any solution to this ?

@jakubgereg or anyone interested in this:
After some experimenting, I figured phrase pull works when specifying tags comma separated without space after the comma, and either without quotes or quotes wrapping the entire string (not individual tags) in the  .phraseapp.yml. eg:

       tags: tag.a,tag.b works
       tags: tag.a, tag.b returns 400/bad request
       tags: 'tag.a,tag.b' works
       tags: 'tag.a','tag.b' returns error
       tags: "tag.a,tag.b" works
       tags: "tag.a","tag.b" returns error

Would be good to have proper specs of the .phraseapp.yml format though. :man_shrugging:

Is there any update on this issue? Is it possible to pull all tags without listing them explicitly?

@jakubgereg or anyone interested in this:
After some experimenting, I figured phrase pull works when specifying tags comma separated without space after the comma, and either without quotes or quotes wrapping the entire string (not individual tags) in the .phraseapp.yml. eg:

       tags: tag.a,tag.b works
       tags: tag.a, tag.b returns 400/bad request
       tags: 'tag.a,tag.b' works
       tags: 'tag.a','tag.b' returns error
       tags: "tag.a,tag.b" works
       tags: "tag.a","tag.b" returns error

Would be good to have proper specs of the .phraseapp.yml format though. 🤷♂

Thank you 👍  this works, the sample config files on the documentation portal  are misleading and won't work since there's spaces between tags

@hamz4k thanks a lot for pointing this out. The docs are indeed not correct for the tags param and we are adapting this


Add block break progress events

Adds two new events to the API to allow bots to observe block break progress animations. These are sent while players mine blocks, and when they stop.

:+1:

:+1:


Test Case
tests/unit/llms/LeptonAIModel_unit_test.py
Pytest Logs
FAILED tests/unit/llms/LeptonAIModel_unit_test.py::test_batch - openai.RateLimitError: Error code: 429
FAILED tests/unit/llms/LeptonAIModel_unit_test.py::test_abatch - openai.RateLimitError: Error code: 429
FAILED tests/unit/llms/LeptonAIModel_unit_test.py::test_batch_with_system_context - openai.RateLimitError: Error code: 429
FAILED tests/unit/llms/LeptonAIModel_unit_test.py::test_abatch_with_system_context - openai.RateLimitError: Error code: 429

Root Cause
openai.RateLimitError: Error code: 429
Potential Solutions
maybe adding a time.sleep() to the tests will solve it
Additional Context (optional)
No response
Type of Test Case
Unit

The PR #711 solves this
I have added a time.sleep(1) to the parametized test cases
Even after doing this, I am still getting  openai.RateLimitError: Error code: 429
Hopefully, if this is tested with the leptonAI API key in the workflow, it will pass
But if it doesn't pass, we might have to move the test cases to x.fail


Migration to Rice 4

This is an upgrade of the generator to the rice 4.0 interface.
It uses the latest rice and, by the way, properly detects the arch attribute https://github.com/metanorma/packed-mn/issues/140.

It works with all Rubies starting from 2.5. We tested it on the expressir library here. A generated by antlr4-native code is here (warning, it's a pretty large file).
Also added a dependency on a new version (4.0) of rice, so a new antlr4-native is used only when a project uses a new rice.

Ok awesome! Do you think I should release a major version bump then? Also, if you have any ideas around reducing generated code size, I'd love to hear them.

Since output changes are quite big, I think, it can be a major version bump. So the previous major release can be kept for Rice 3.0 and its bug fixes.
Ok :)
Thank you @camertron 🙏

Hey @alexeymorozov I tried re-generating the code for python3-parser-rb using your changes. Everything compiles just fine, but the tests fail with "Bad cast" exceptions. Do you have any idea what might be going on? I haven't been able to pinpoint the cause, but it appears to happen in the visitor code somewhere. You can reproduce by running rake generate, then rake compile, then bundle exec rspec.

Still haven't dig much further, but it fails the same way with Rice 2.0.
To compare with Rice 4.0 build.

Hey @alexeymorozov, I was able to dig in and I'm pretty sure I know what happened here. Several PRs from outside contributors were merged a while ago and I never regenerated the C++ code for python3-parser. One of the PRs introduced the std::bad_cast bug, which I believe is entirely unrelated to the upgrade to Rice v4.0. The issue should be fixed in master. Feel free to merge or rebase whenever you have the chance :)

I'm glad it's fixed now! Rebased the branch. Tests pass with python3-parser.

Released in v2.0.0 :)

Thank you @camertron 😄


Logging to a file

Is there a way to log to a file?
This would be perfect to couple with fail2ban to block spammers and bots.

No, not by mox itself. If you want to log to a file, you could redirect the output to a file yourself. Luckily I never had the need for fail2ban or
similar tools. Don't they support reading logs from the common logging destinations?
By the way, mox has some builtin options for ratelimiting. E.g. too many auth failures cause new connections to be rejected (smtp/imap), and so do a
high connection rate or too many concurrent connections (from ip or subnets).

if you use systemd (journald), you can set backend = systemd in your fail2bans jail config

Thanks belst, sounds like a good approach. I'll consider this resolved.


1.2 Add dataTables to HTML tables

adds dataTables functionality to sub-threshold, gene and exon stats tables
improves usability through adding searching, deferred rendering scrolling etc
simplified some of coverage_report_single.py through adding styling to the report instead of in the script
all table styling functions now output list of lists of data to datatables to render
example output report may be found here: https://cuhbioinformatics.atlassian.net/wiki/spaces/P/pages/2216230925/Development

Hello @jethror1! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file bin/coverage_report_single.py:

Line 1190:80: E501 line too long (88 > 79 characters)


Full AWS APIG stage configuration

Is there a way to set the settings Enable CloudWatch Logs and Enable Detailed CloudWatch Metrics for the APIG stage?
In Cloudformation that would be the params in  AWS::ApiGateway::Stage -> MethodSettings.

Currently not, but sounds like a good idea.
As the plugin creates a stage resource (and thus has the possibility of complete configuration) an addition and implementation of the feature should be straight forward.
How would you see a possible configuration? I would see it somewhere at function and service level.

Here is the documentation for the API Gateway Stage MethodSettings.
Is there a way we should default to values (where applicable) from the main serverless.yml file?
I agree that it makes sense to be able to set the MethodSettings at either the service or function level.

I inspected the available stage settings and came to the conclusion that the plugin should offer a possibility to configure all of the stage settings instead of only the CW log settings.
To limit possible conflicts with Serverless (in case someone wnats to add this as basic feature to the framework somewhen - I don't really believe in that 😈 ) I'd suggest that the stage setting would reside in an object prefixed with "alias". So the following definition would be possible at the service, function and http event level. Having these 3 locations would let us configure it globally, per function (affecting all endpoints in the function, or only for a specific endpoint). The defaults, if no option is set at all would be the configuration as it currently is in the plugin).
If an option is not explicitly specified in the stage configuration it will be omitted from the CF template. I think that's the expected behavior.
The order of application of multiple configurations would be SVC_LEVEL -> FUNC_LEVEL -> ENDPOINT_LEVEL
The configuration at all locations will then be specified as (YAML):
aliasStage:
  cacheDataEncrypted: Boolean
  cacheTtlInSeconds: Integer
  cachingEnabled: Boolean
  dataTraceEnabled: Boolean
  loggingLevel: String
  metricsEnabled: Boolean
  throttlingBurstLimit: Integer
  throttlingRateLimit: Number

If everyone agrees with this proposal I will implement it exactly that way.

There will be two additional settings that can only be set on the service level:
cacheClusterEnabled: Indicates whether cache clustering is enabled for the stage.
CacheClusterSize: The stage's cache cluster size.

I'll add these for completeness to have any possible stage configuration available

Excerpt from the new README. Explains the exact use of the new feature:
Stage configuration (NEW)
The alias plugin supports configuring the deployed API Gateway stages, exactly as
you can do it within the AWS APIG console, e.g. you can configure logging (with
or without data/request tracing), setup caching or throttling on your endpoints.
The configuration can be done on a service wide level, function level or method level
by adding an aliasStage object either to provider, any function or a http event
within a function in your serverless.yml. The configuration is applied hierarchically,
where the inner configurations overwrite the outer ones.
HTTP Event -> FUNCTION -> SERVICE
The aliasStage configuration object
All settings are optional, and if not specified will be set to the AWS stage defaults.
aliasStage:
  cacheDataEncrypted: (Boolean)
  cacheTtlInSeconds: (Integer)
  cachingEnabled: (Boolean)
  dataTraceEnabled: (Boolean) - Log full request/response bodies
  loggingLevel: ("OFF", "INFO" or "ERROR")
  metricsEnabled: (Boolean) - Enable detailed CW metrics
  throttlingBurstLimit: (Integer)
  throttlingRateLimit: (Number)

There are two further options that can only be specified on a service level and that
affect the whole stage:
aliasStage:
  cacheClusterEnabled: (Boolean)
	cacheClusterSize: (Integer)

For more information see the AWS::APIGateway::Stage or MethodSettings documentation
on the AWS website.
Sample serverless.yml (partial):
service: sls-test-project

provider:
  ...
	# Enable detailed error logging on all endpoints
	aliasStage:
		loggingLevel: "ERROR"
		dataTraceEnabled: true
  ...

functions:
  myFunc1:
	  ...
		# myFunc1 should generally not log anything
		aliasStage:
		  loggingLevel: "OFF"
		  dataTraceEnabled: false
		events:
		  - http:
					method: GET
					path: /func1
			- http:
					method: POST
					path: /func1/create
			- http:
					method: PATCH
					path: /func1/update
					# The update endpoint needs special settings
					aliasStage:
					  loggingLevel: "INFO"
					  dataTraceEnabled: true
						throttlingBurstLimit: 200
						throttlingRateLimit: 100

	myFunc2:
	  ...
		# Will inherit the global settings if nothing is set on function level

The feature is available in master now. If someone could give it a try before I release 1.3 it would be great.

Released!


"restangularizeElement" then got: Converting circular structure to JSON

I have meet a problem that to use "restangularizeElement".
Restangular.restangularizeElement("", user, "users")
user.put()

restangularizeElement is fine but when I update the user, browser show a error: Converting circular structure to JSON. The tricky thing is once I refresh the browser, everything goes well, update function works well.
I don't think it is a good idea to let user refresh their browser before using the function.
Is there any approach to solve this issue?

Can you provide a plunkr with sample response data from your server and Restangular use so we can reproduce?

@Perpherior Any update? Can you provide an example plunkr?


fix: updates to installation instructions for Kubernetes

Checklist
I have:

[x] run the content through Grammarly
[x] linked to sample apps when relevant
[x] added the meta description for each page in the PR
[x] minimized the callouts and added only when necessary
[x] added the queryString parameter to the Tabs (if used)
[x] masked PII in images. For example, login credentials, account details, and more
[x] added images only when necessary
[x] deleted the images that are no longer used for the updated pages in the PR
[x] followed the image file naming convention while renaming or adding new images. (Use lowercase letters, dashes between words, and be as descriptive as possible)
[x] used the <figure/> tag instead of a markdown representation for images
[x] added the <figcaption/> tag to add a caption to the image
[x] added the alt attribute in the <img/> tag
[x] verified and updated the cross-references or created redirect rules for the changed or removed content
[x] reviewed and applied the style changes for UI formatting. For example, Bold the UI elements(Buttons on screen) used in the doc.

Updates
Below are the changes for this PR:

moved prerequisites under each tab
removed Minikube from Business Edition tab
removed Minikube from Configure High Availability tab

This format looks confusing. Can we have pre-requisites outside as before. its just one tab of Minikube that is different and hidden on Business. Can we just mention that HA will not be suppored for this method and that if they want HA they should use one of the other three.

Based on our discussion, updates are done.


chore(): upgrade to<EMAIL_ADDRESS>

Description
Making covalent support all the breaking changes (and deprecating changes) coming from material following their changelog https://github.com/angular/material2/blob/master/CHANGELOG.md#200-beta10-découpage-panjandrum-2017-08-29
What's included?

TBA

Test Steps

[ ] yarn install
[ ] ng serve
[ ] Test everything!

General Tests for Every PR

[ ] ng serve --aot still works.
[ ] npm run lint passes.
[ ] npm test passes and code coverage is not lower.
[ ] npm run build still works.

Opened PR just as a placeholder.. i only solved the compilation errors, but there are lots of small issues around layouts, chips, etc etc

Is there any eta? 😄

We are trying to target early next week since we also have some data-table features and virtual scroll base implementation :P so stay tuned!

A visited link in a dark theme is extremely difficult to read. IMO.

Chip Events:
http://localhost:4200/#/components/chips
ERROR Error: ExpressionChangedAfterItHasBeenCheckedError

Loading Style/layout
http://localhost:4200/#/components/loading
Input label crashes into tab header underline.

@jerryorta-dev i think that already happens https://teradata.github.io/covalent/#/components/loading

Data Table with components
http://localhost:4200/#/components/data-table
ExpressionChangedAfterItHasBeenCheckedError:  when changing number or rows displayed.

I think the chips was introduced before, but great catch. Pushed a commit to fix it

Dynamic File Input Element
http://localhost:4200/#/components/dynamic-forms
Text overlays default text after selecting a file to upload.
Change events are not detected.

That seems to me like an mdInput issue, so we should probably open an issue to track this. Thanks @jerryorta-dev

@jotatoledo we are gonna release after we merge this and after i merge another PR with data table features and virtual scroll implementation (basic)

The paging module seems to effect the md-select. I see that several things are removed from the TdPagingComponent such as the pageSizes, etc... Since these are no longer used I think the css for this component should be removed  as well.

After switching to nightly build and material.beta.10, cdk.beta.10
"@angular/cdk": "^2.0.0-beta.10",
"@angular/material": "^2.0.0-beta.10",
"@covalent/core": "git+https://github.com/Teradata/covalent-nightly.git",

The paginator still works before upgrade but now there is error message
Template parse errors:
Can't bind to 'pageSizes' since it isn't a known property of 'td-paging-bar'

My paging code

  Row per page:
  {{pagingBar.range}} of {{pagingBar.total}}

Remove the [pageSizes]="[defaultPagesize, 20, 25]", then paginator works again.

@tuanngominh if you are using the nightly build, you need to be aware that there might be breaking changes like that one.


[suggestion] Running this locally?

I would actually really love to be able to run this as a script on my machine before submitting a PR as a gut check

Thanks for the suggestion, I like it

That would require submitting your patch file in the extension. Quite some work. Why not iterate and submit the PR, use the extension and iterate again? I'm not sure if it's useful enough to add support for it.

I'm going to close this as I'm not sure how this could work with a chrome extension. I think it would be better to develop a CLI based git hook or so that before you submit it, it sends the code to OpenAI. Be careful, as you might be sending private code to an API that did not sign an NDA ;-)


[Unit Test] Add test case for Joiner

Code classes that need to develop unit tests:

cn.hippo4j.common.toolkit.Joiner

Please refer to the manual for the first contribution: guide
Unit test development refers to the following two examples:

cn.hippo4j.core.executor.DynamicThreadPoolExecutorTest
cn.hippo4j.core.executor.ExtensibleThreadPoolExecutorTest

please assign it to me

@barret-yzh Welcome

Please assign me @magestacks ! I am eager to add the unit test.


Uncaught TypeError: $(...).toc is not a function

Hello, when I use

in the head part to import this package, and using the following to dynamically generate TOC
function generateTOC() {
    //get the
[toc]

    var ptoc = $("p").filter(function(index){
        return $(this).text() == '[toc]';
    })
    // if [toc] exist
    if (ptoc.length == 1) {
        // replace
[toc]
 to the following part
        ptoc.replaceWith($('

'));
        $("#tocjs").toc({content: "article", headings: "h1,h2,h3"});
    };
};

it sometimes works in a local testing environment, however, when pushed to github pages, it always raises this error.
It may due to my poor javascript skills rather than your package problem, and I am not quite sure if it is correct to use it in such a function.

The full error is
page.html?id=19:147 Uncaught TypeError: $(...).toc is not a function
    at generateTOC (page.html?id=19:147)
    at Object.success (page.html?id=19:233)
    at c (jquery.min.js:2)
    at Object.fireWith [as resolveWith] (jquery.min.js:2)
    at l (jquery.min.js:2)
    at XMLHttpRequest.<anonymous> (jquery.min.js:2)

This is likely caused by your code executing before the jquery.toc.js file has loaded. You can try using the jQuery load event to execute your code.
If that doesn't help, please provide a link to the page in question, and I can take a look.

PS: I previously use file/// to do the local test which is not a correct way. Then I use python to run a localhost and now the local environment performs the same as the online environment (raise .toc() is not a function error).
I tried it again, it may not the jquery.toc.js not loaded error because if I use that function directly:

[toc]

esdgsefd

it functioned all well.
However, the problem may be, my page contents were dynamically generated via markdown file:
var converter = new showdown.Converter();
$.get("md/000.md", function( md ) {
    html = converter.makeHtml(md);
    $('#page-placeholder').html(html);

    // use TOC function inside jQuery raise this error
    generateTOC();
},'text');

// use TOC function outside jQuery don't raise this error but make no sense.
//generateTOC();

It seems $.get() is the last step to run in javascript, if I move generateTOC() outside $.get() block, it isn't meaningful because the article hasn't been loaded yet and nothing will be generated.
I think the problem may be something like, how to use $().toc() as a global function inside a JQuery function?
I understand it now changed to another jQuery problem and not related to this package issue. I am very appreciated for your patience if you could help me solve this！

I was able to find the page in question from your GitHub profile (https://www.sigmameow.com/blog/page.html?id=19), and took a look at the source.
The problem is that you have added jQuery twice on the page: once in the HEAD, and then again near the bottom of the page: .
What's happening is that:

jQuery loads the first time.
jquery.toc.js loads and adds itself to the loaded jQuery instance.
jQuery.slim.min,js loads, and clobbers the loaded jQuery instance.
Your code runs, and this time, the jQuery plugins that were loaded earlier are no longer available.

The simple fix is to remove the SCRIPT tag at the bottom which is referencing jquery.slim.min.js.
Let me know if this works for you.

Yes, you are correct, I simply copied the bootstrap template and haven't realized the jquery.slim.min.js has already existed at the end of that page!
Thank you a lot for your patience to go through these "trash" code of my website!

No worries, always happy to help!


Fix/current_schema

Makes set current schema option functional and will update the server job.
SET SCHEMA and it's variations only work with SQL naming.

This PR is going to be re-written after a cleanup PR has been finished. Closing.


Thank you!

Hello,
I'd like to say thank you for reviving hastebin in your or rendition.
Couple things:
0. Do you think line numbers are needed for the horizontal scroll?
For example: http://tastebin.x-berg.de/#seyizahedekunigfanyey.dns

Some of the theme label names disappear when you select it. Try sunburst and you'll see the name is no longer visible.

"Distraction free"  makes the text disappear when switching from index. Is that expected behavior?

Thanks and keep up the great work!
Best,
jb

Sean,

what do you mean if line numbers are needed? Are you saying you'd like line numbers not to be fixed? You see them more "natural" to scroll with the content? I don't, but then again I also don't think the OSX natural scrolling is natural :)

disappearing label names - can't reproduce it here. Which browser? Screenshot?

not sure what is the problem with "distraction free", but please have a go with the latest version. maybe it will still be up at https://frozen-bayou-3399.herokuapp.com/

Cheers

Hi Andrei,
I like and really depend on line number for content; however, I don't think line numbered scroll bars are actually that necessary. Do you? Do we really need to know the line number 5 is for the scroll bar as my example shows?
I'll get screenshots in firefox and chrome, both on Windows.
Things look much better on your link with distraction free. If you commit, I'll update and test.
thanks,
jb/sean

Now I understand. So, for better or worse, on save, there's a new line that
gets added (among trimming trailing spaces, etc), so the 5 you see, it's
actually for a 5th line
For instance, if I run curl http://tastebin.x-berg.de/tastes/seyizahedekunigfanyey --trace - in the
end, I see 02f0: 09 09 09 09 09 09 09 09 09 09 09 09 30 0a ............0. --- 0a stands for new line character.
All is committed
On Wed, Jul 29, 2015 at 12:09 AM, jungle-boogie<EMAIL_ADDRESS>wrote:

Hi Andrei,
I like and really depend on line number for content; however, I don't
think line numbered scroll bars are actually that necessary. Do you? Do we
really need to know the line number 5 is for the scroll bar as my example
shows?
I'll get screenshots in firefox and chrome, both on Windows.
Things look much better on your link with distraction free. If you commit,
I'll update and test.
thanks,
jb/sean
—
Reply to this email directly or view it on GitHub
https://github.com/andreineculau/tastebin/issues/4#issuecomment-125767543
.

--
andreineculau.com http://www.andreineculau.com

Cannot restart tastebin either with forever or npm start:
https://gist.github.com/jungle-boogie/d48e94431d69fa80cca4

Fixed in 0.9.5
On Wed, Jul 29, 2015 at 1:26 AM, jungle-boogie<EMAIL_ADDRESS>wrote:

Cannot restart tastebin either with forever or npm start:
https://gist.github.com/jungle-boogie/d48e94431d69fa80cca4
—
Reply to this email directly or view it on GitHub
https://github.com/andreineculau/tastebin/issues/4#issuecomment-125780806
.

--
andreineculau.com http://www.andreineculau.com

Fixed in 0.9.5

looks great! Thanks.

Hello,
attached image is regarding issue 1 where the theme name is blank. Even if I mouse over the drop down menu, it remains blank.

Can you see the same here https://frozen-bayou-3399.herokuapp.com/ ?
I don't. Try to upgrade to the 0.9.7 I've just pushed and let me know if
things are the same, which browser, OS, etc
On Wed, Jul 29, 2015 at 3:48 PM, jungle-boogie<EMAIL_ADDRESS>wrote:

Hello,
attached image is regarding issue 1 where the theme name is blank. Even if
I mouse over the drop down menu, it remains blank.
[image: blank]
https://cloud.githubusercontent.com/assets/1111743/8959099/bcba63d2-35bd-11e5-935a-5bf05da08677.png
—
Reply to this email directly or view it on GitHub
https://github.com/andreineculau/tastebin/issues/4#issuecomment-125958333
.

--
andreineculau.com http://www.andreineculau.com

Can you see the same here https://frozen-bayou-3399.herokuapp.com/ ?

Don't see it there. I'll update my version.
I will say that some of the font selections seem to be white on white, making it only possible to see the text once you mouse over the selection.
For example, zenburn.
Firefox 39 on linux.

These these selections also exhibit the same behavior:
dark
darkula
hybrid
monokai
obsidian
pojoaque
rails cat
rainbow
tomowwow-night-eightes
tomowwow-night
Bar far the easiest text to read is if its black.

please have a go with 0.9.8
https://frozen-bayou-3399.herokuapp.com/
thanks for bringing it up
On Thu, Jul 30, 2015 at 1:28 AM, jungle-boogie<EMAIL_ADDRESS>wrote:

These these selections also exhibit the same behavior:
dark
darkula
hybrid
monokai
obsidian
pojoaque
rails cat
rainbow
tomowwow-night-eightes
tomowwow-night
Bar far the easiest text to read is if its black.
—
Reply to this email directly or view it on GitHub
https://github.com/andreineculau/tastebin/issues/4#issuecomment-126126523
.

--
andreineculau.com http://www.andreineculau.com

please have a go with 0.9.8

That's much better!
I think you've corrected all the problems I've found, thank you.


Customer portal UI 9

http://order-organic.com/
Customer Portal-->search options are displaying in wrong place

fix


Validation on unicity of multiple keys

For a specific CT, when finilising a draft , the system should verify if a combination of multiple fields is unique. And if not, should refuse it and show an error.

You can use a computed field to generate the field. After that you can, in the post process, query elasticsearch to see if another object has the same key;
NB: the cant_be_finalized function will come with core 1.6.76
{% set result = {'type': 'my_type', 'index': 'id_preview', 'body': {
'query' : {
'term' : {
'my_field': _source
}
}
} }|search %}
{% for item in result.hits.hits %}
{% if item._id != _id %}
{{ cant_be_finalized('My field must be unque') }}
{% endif %}
{% endfor %}
{{ _source.asset|json_encode|raw }}


html attributes are always lowercased when added through a rivets binder

Closing as not a bug or issue with rivets


support projected name for model

Description
Fix https://github.com/Azure/autorest.csharp/issues/3662

The model name will be retrieved in following orders: projectedName, friendlyName, NameWithTemplate, the origin Name

current getLibraryName of TCGC only considers projected Name. After getLibraryName adopt other factors, we will adopt getLibraryName
Checklist
To ensure a quick review and merge, please ensure:

[ ] The PR has a understandable title and description explaining the why and what.
[ ] The PR is opened in draft if not ready for review yet.

If opened in draft, please allocate sufficient time (24 hours) after moving out of draft for review

[ ] The branch is recent enough to not have merge conflicts upon creation.

Ready to Land?

[ ] Build is completely green

Submissions with test failures require tracking issue and approval of a CODEOWNER

[ ] At least one +1 review by a CODEOWNER
[ ] All -1 reviews are confirmed resolved by the reviewer

Override/Marking reviews stale must be discussed with CODEOWNERS first

As there are api change in the regen PR for OpenAI and ContentSafety,  @chunyu3, could you work with the owners to get the breakings addressed/approved?

As there are api change in the regen PR for OpenAI and ContentSafety, @chunyu3, could you work with the owners to get the breakings addressed/approved?

Yes, I have contacted owners to review. ContentSafety owner has approved, and waiting for OpenAI response. @lirenhe


Failed to Login

It shows the following error when I put down my number for login. I am using +44
ERROR: {"statusCode":400,"errorKey":null,"data":null,"fields":null,"timestamp":"2024-01-17T13:59:17.618Z","requestId":"c96ed64a4bd15bbe827784d3d76cc07c"}
Failed with Vonage login provider, re-trying to login with Firebase...
Can someone help? thanks

I have the same issue here in switzerland (+41). I really hope someone can fix this as I can't login.

Same issue for PH (+63)


pin old runtime version until we do the migration

Description
Pin old runtime
Checklist
Please ensure that each of these items has been addressed:

[X] I have tested these changes in my local environment
[ ] I have added/modified tests as applicable to cover these changes
[X] (Atlassian contributors only) I have removed any Atlassian-internal changes including internal modules, references to internal tickets, and internal wiki links

Runtime has been migrated


Updating all dependencies

Merges in https://github.com/facebookarchive/instagram-ruby-gem/pull/204

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla - and if you have received this in error or have any questions, please drop us a line at<EMAIL_ADDRESS>Thanks!


BDI: custom Contact and Account Unique ID's don't support fields from other packages

the optional Contact Custom Unique ID and Account Custom Unique ID settings allow the user to specify their own custom field for doing matching with.  These fields currently are assumed to always be in the org's non-namespaced code.  We don't support the fields living in another managed package, as well as a mixture of the Contact or Account field coming from another managed package, but the Data Import field being added by the customer without a namespace.
Note that no customer has yet asked for this support, but it is a limitation of the current code and settings.

**lurch: add

@davidhabib good one. I'm frankly surprised this hasn't come up already... in the meantime, should anyone review this, a workaround would be a workflow rule or process that copies the value to a custom field?

that might work, but having workflows run during data import is probably not a good practice.

I was thinking the workflow rule would fire when the external ID was added by the app, which is not necessarily in bulk.

Included in beta release 3.110 (Beta 7)

Included in production release rel/3.110

I can no longer see fields we were using for custom mapping - if there is another issue or article that could help that would be great.  I'm not sure how to get fields we were using for custom account and contact ID matching to show up again.  They were there and now they're not.  What could have changed to make them not available? Thanks!

https://github.com/SalesforceFoundation/Cumulus/releases/latest says "The Batch Data Import Configuration Options for Contact Custom Unique ID and Account Custom Unique ID now only display fields that are marked Unique or External ID. Additionally, those fields can now come from other managed packages."  That means the field needs to be marked as Unique in the custom field set up. This prevents errors for scenarios where two records could have the same ID (if not marked unique) and then data could be overwritten.

Hi - Thanks for the quick response.  We were using a formula field to match.  Do we need to create a new external ID field and a workflow to populate it with the IDs from our custom ID field? Do you have any recommendations?

Yes that would be the best way. It can be either an external Id field or simply marked as "unique"


DRAFT: Improvements based on recent experiences

Update implementations and document based on recent experiences using this template.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+3.7%) to 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
3.7%

Covered Lines:
0

Relevant Lines:
0

💛  - Coveralls

All changes reviewed and cleaned.  All tests are passing and the logs of all new and modified actions reviewed.


Automated cherry pick of #1376: ucloud create instance & huawei change instance config bugfix

Cherry pick of #1376 on release/2.10.0.
#1376: ucloud create instance & huawei change instance config bugfix

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.10.0@84ba38e). Click here to learn what that means.
The diff coverage is 0%.

@@               Coverage Diff                @@
##             release/2.10.0   #1377   +/-   ##
================================================
  Coverage                  ?   7.77%
================================================
  Files                     ?     372
  Lines                     ?   58284
  Branches                  ?       0
================================================
  Hits                      ?    4531
  Misses                    ?   53351
  Partials                  ?     402

Flag
Coverage Δ

#aFlag
7.77% <0%> (?)

Impacted Files
Coverage Δ

pkg/util/ucloud/host.go
0% <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 84ba38e...6fb3e0c. Read the comment docs.

/lgtm
/approve


Math inequality symbols in attributes are not properly encoded

The tag open < and > are not properly encoded inside attribute values.
They should become < and > - they makes the lib dangerous in projects dealing with user input/output.

For when running inside the web browser, a safe way to encode attribute values would be something like this:
  function htmlEncode(value) {
    const div = document.createElement('div');
    div.textContent = value;
    return div.innerHTML;
  }

Unfortunately, node env does not offer such a built-in, so you cannot rely on it,  you could nevertheless,  reference https://www.npmjs.com/package/js-htmlencode in your dependencies and use those functions to perform encoding. Encoding is not trivial task, use a library!
This person already tried disabling the encoding part of this project:
https://github.com/edkotkas/node-xml-writer/commit/f0bf5d95948e4f53427ff3d24f2dfa3debb05d59

I'm not sure to understand
XML specefiction  doesn't allow & < and > : https://www.w3.org/TR/xml/#NT-AttValue
So they should be encoded, but how ?
Probably like HTML spec ? https://www.w3.org/TR/html51/syntax.html#attribute-value-double-quoted-state
The package that  you cite replace special character by there corresponding literal entity
https://github.com/emn178/js-htmlencode/blob/master/src/htmlencode.js#L295-L298
Just like this package :
https://github.com/Inist-CNRS/node-xml-writer/blob/master/lib/xml-writer.js#L258-L265


m3u format didn't work for me

m3u didn't work for me and I implemented python2 support - although I don't need it ;)

Awesome, @schuellerf thank you very much! Cannot tell, whether the extm3u format is more accurate than the plain m3u format, but it works perfectly with my cmus. Also py2 support for free is nice. So thanks a lot!


Coverage with c8 and tsx reporting as 100% since version 4.3.0

Acknowledgements

[X] I searched existing issues before opening this one to avoid duplicates
[X] I understand this is not a place for seek help, but to report a bug
[X] I understand that the bug must be proven first with a minimal reproduction
[X] I will be polite, respectful, and considerate of people's time and effort

Minimal reproduction URL
https://github.com/OscarBarrett/tsx-coverage-issue
Version
v4.6.2
Node.js version
v20.10.0
Package manager
yarn
Operating system
Linux
Problem & Expected behavior
We use c8 and tsx together when running our test suites, and since tsx version 4.3.0 any file that is imported in our tests gets 100% coverage.
Locking to tsx 4.2.1, we get the expected coverage.
Assuming this is related to #405
Contributions

[ ] I plan to open a pull request for this issue
[ ] I plan to make a financial contribution to this project

I have the same issue. FWIW, you can work around this by running tests on the built js output. If you have TypeScript generate .map files, then the CC output should look the same as before (i.e. reported on ts files rather than js files). Here's an example of the changes necessary (besides building with tsc):
https://github.com/andreashuber69/verify-coldcard-dice-seed/commit/35b72b9bcc4a087790848db0afb3df9407b0d919

Ran into the same; just wanted to chip in and confirm that downgrading works.

I haven't investigated this but I'm thinking this is as simple as removing this check so sourcemaps are always inlined:
https://github.com/privatenumber/tsx/blob/8c778919496a8164ec63d0b488c805bce1aba839/src/source-map.ts#L23
There will need to be tests though. PR welcome if anyone wants to tackle.

Test coverage works in 4.7.0 if I run mocha with source-map-support/register.
My .mocharc file to run it with tsx/esm:
{
  "import": "tsx/esm",
  "require": "source-map-support/register"
}

This issue is related to the source maps conversion with v8-to-istanbul, and it is not related to tsx. see here
V8 provides the correct coverage data
[
  [ { startOffset: 0, endOffset: 592, count: 1 } ],
  [ { startOffset: 47, endOffset: 113, count: 2 } ],
  [ { startOffset: 114, endOffset: 143, count: 1 } ],
  [ { startOffset: 161, endOffset: 195, count: 0 } ]  <- uncovered
]

However, it has not been applied to the source file index.ts
[
  CovLine { line: 1, startCol: 0, endCol: 0, count: 1, ignore: false }
]

I built a tool that can be used to generate native v8 coverage reports, you might have a try
npm i monocart-coverage-reports -g
mcr npx tsx bin/test.ts -r v8,console-details --entryFilter **/*.ts

┌─────────────────┬──────────┬────────────┬──────────┬───────────┬──────────┬─────────────────┐
│ Name            │    Bytes │ Statements │ Branches │ Functions │    Lines │ Uncovered Lines │
├─────────────────┼──────────┼────────────┼──────────┼───────────┼──────────┼─────────────────┤
│ bin             │          │            │          │           │          │                 │
│ └ test.ts       │ 100.00 % │   100.00 % │          │           │ 100.00 % │                 │
│ tests           │          │            │          │           │          │                 │
│ └ maths.spec.ts │ 100.00 % │   100.00 % │ 100.00 % │  100.00 % │ 100.00 % │                 │
│ index.ts        │  55.47 % │    50.00 % │ 100.00 % │   50.00 % │  50.00 % │ 5-7             │
├─────────────────┼──────────┼────────────┼──────────┼───────────┼──────────┼─────────────────┤
│ Summary         │  89.90 % │    75.00 % │ 100.00 % │   66.67 % │  84.21 % │                 │
└─────────────────┴──────────┴────────────┴──────────┴───────────┴──────────┴─────────────────┘

Seems like both versions of tsx in the reproduction doesn't work in the latest version of Node, but the reproduction still works fine with Node v20.10.0.
I'm curious what happened between tsx v4.2.1 and v4.6.2 that broke c8 in Node v20.10.0.

@privatenumber the generated sourcemap content could be different

<= tsx 4.2.1 generating sourcemap with source-map-support

= tsx 4.3.0 generating sourcemap with native Node.js (removed source-map-support)

This has been completed in the development repo.

:tada: This issue has been resolved in v4.10.4
If you appreciate this project, please consider supporting this project by sponsoring :heart: :pray:

Hi @privatenumber 👋🏼 😊
I still have the issue. Basically, if I run Node tests with tsx and c8 as a coverage tool on the latest versions of both, I get the coverage at 100% for all tests. While if I downgrade tsx to an older version (f.e. 4.2.0), I get the right coverage. Do you know if this can be solved by passing a specific setting?

An observation that I find interesting to relate is that after v4.10.4, the coverage of a project dropped by almost 2%.
Looking at the differences between tsx versions, I noticed that it was mainly the imports of types.
To fix it, I just used c8's recommendation:
/* c8 ignore next */
import type { SomeType } from '../some-type.js';

I don't see this as an issue, so I preferred to add it to the context of this issue.

@privatenumber, thanks for this project and the fix 💙


Mechanics of when the weapon scope is opened and the scope is not opened.

Description
particles are generated is more detailedly divided into when the weapon scope is opened and when the scope is not opened, that can better create the effect of muzzle flash.
Configuration
Shoot:
  scope:
    Mechanics:
      - "Particle{particle=COMPOSTER, count=1} @Source{eye=true, offset=~0 -0.08 0.7}"

Payment (optional)

[x] I'm willing to pay for this feature

what are you asking for exactly?

now Mechanics just can set when you shot, but the weapon model has classified default and scope, so when I shoot in default,
the particle location is strange, it was same location of scope shoot
sorry my english is bad

Shoot:
  Mechanics:
    - "Particle{particle=COMPOSTER, count=1} @Source{eye=true, offset=~0.4 -0.1 0.7} ?IsScoping{inverted=true}"
    - "Particle{particle=COMPOSTER, count=1} @Source{eye=true, offset=~0 -0.08 0.7} ?IsScoping{}"


How to start?

SDL3 is near release, with the 3.1.3 preview version having been released earlier this week. As such, we can start getting ready to work on the SDL3 units.
So, my question is - how should we start? I see three options here:

Base our SDL3 units on the SDL2 units we already have. Use the SDL3 migration guide to help us identify which parts need to be changed.

Start anew. Use some tool like h2pas to automatically convert the C headers into Pascal units and then tweak the output until we arrive at something we're satisfied with.

Start from scratch, manually.

Option 1) would definitely be the easiest, though I am rather worried about the risk of us missing things here and there, leading to hard-to-find bugs where the units compile fine, but the definitions don't match what's in the C headers. As such, I'd be willing to devote some time to try out option 2) and see how that works out.

I totally agree with everything you said. If there was plenty of time I would prefer opt. 2, too. - But also I'd like to mention that just updating the sdl2 repo to version 2.0.14 for every unit took around 3 years. My experience with h2pas is that is still needs a lot of manual attention; I didn't find better tools though.
For personal reasons I had and have few spare time in the last and coming months. Hence, for opt. 2 probably you would have to do a larger part. If you are willing to do so, I will contribute as much as possible, too (as always :D).
The big benefits of opt. 2 are: We could still rely on many "rules" we established for the sdl2 repo. We could freely decide to change some bothering things (see below). And we would know about "every corner" of the code.
If we go for opt. 2:
The first steps would be to decide for

general folder/file/code structure
a code style (have a guideline)
a license

To my mind, actually we could heavily replicate the folder structure, file structure and code style from the sdl2 repo. We decided for these in many discussions already.
Some things to consider, that I would like to change compared to the sdl2 repo:

decide for uniform file headers for every file (similar to sdl3's file headers)
have the inc-file names being identical to the h-file names ("SDL_events.inc" instead of "sdlevents.inc")
for the license I would go JUST for the zlib license (like the sdl3 project itself)

What are your thoughts on this?
Best regards
Matthias

My experience with h2pas is that is still needs a lot of manual attention; I didn't find better tools though.

Well, yeah, I don't really expect it to produce something that'll be 100% production-ready; but anything that reduces the amount of manual work is a good idea.

decide for uniform file headers for every file

I'm not sure what do you mean by this. Uniform licence info headers?

have the inc-file names being identical to the h-file names ("SDL_events.inc" instead of "sdlevents.inc")

Minor change, but yeah, doesn't affect anything from the user's perspective and makes the repo more consistent with the original sources.

for the license I would go JUST for the zlib license (like the sdl3 project itself)

zlib licence is fine by me.

Well, yeah, I don't really expect it to produce something that'll be 100% production-ready; but anything that reduces the amount of manual work is a good idea.

Absolutely!

I'm not sure what do you mean by this. Uniform licence info headers?

Right, also project name, project url.
My suggestion:
SDL3 for Pascal
Copyright (c) 2024 Pascal Game Development Team (https://github.com/PascalGameDevelopment/SDL3-for-Pascal)

This software is provided 'as-is', without any express or implied warranty. In no event will the authors be held liable for any damages arising from the use of this software.

Permission is granted to anyone to use this software for any purpose, including commercial applications, and to alter it and redistribute it freely, subject to the following restrictions:

1. The origin of this software must not be misrepresented; you must not claim that you wrote the original software. If you use this software in a product, an acknowledgment in the product documentation would be appreciated but is not required.
2. Altered source versions must be plainly marked as such, and must not be misrepresented as being the original software.
3. This notice may not be removed or altered from any source distribution.

Another question, which arises: How do we do testing? For the sdl2 repo I suggested a testing framework (by the PR). We could also go for a simpler approach as we do now. What are your thoughts on this topic?

On one hand, I'm okay with the header. On the other hand, I think I'd prefer keeping the licence in a LICENSE file and just having the header point people to said file. Or we could just slap an SPDX license tag on it:
SPDX-License-Identifier: Zlib

How do we do testing? For the sdl2 repo I suggested a testing framework (by the PR)

Uh, yeah, sorry for not reviewing that. :sweat_smile:  I know that FPC ships with some testing framework, but it's largely undocumented; I'd prefer to look into this tool first, instead of dragging such a large 3-rd party dependency into the repo.

So, what about
SDL3 for Pascal
(https://github.com/PascalGameDevelopment/SDL3-for-Pascal)
SPDX-License-Identifier: Zlib

?

Uh, yeah, sorry for not reviewing that. 😅 I know that FPC ships with some testing framework, but it's largely undocumented; I'd prefer to look into this tool first, instead of dragging such a large 3-rd party dependency into the repo.

No harm done. - Well, according to my research this 3rd party repo should be the better choice compared to FP's own testing framework. That said, I also don't like the fact that the 3rd-party repo is so large though. I will reconsider this; maybe we go with the FP framework for convenience or use the simple tests like before. For the moment we should focus on other tasks anyway. :)
I will craft the initial commit soon, so we can start. :D

I just saw that you also want to get to grips with SDL3. I'm already working on it and it's almost finished, the header already runs with SDL 3.1.6
I did it from scratch.
https://github.com/sechshelme/Lazarus-SDL3.0-Packages_and_Examples

@sechshelme Thanks for letting us know. Impressive work! Well done! Maybe we could collab in a way, any suggestions?
@suve How should we proceed now? I'm kind of undecided right now knowing that there is repo having done a lot of effort already. What are your thoughts?

@sechshelme Thanks for letting us know. Impressive work! Well done! Maybe we could collab in a way, any suggestions?

My binding should be fully operational for SDL 3.1.6.
It's just the SDL3_xxx stuff that's causing problems, as things in the c-libs have been changed in the meantime.
As soon as these appear in the releases, I'll recompile them.

I just compiled the new SD3_images 3.1.0.

Thanks for letting us know. Impressive work!
@suve Please check your mails.

@sechshelme Although I really appreciate your impressive work on the sdl3 bindings, we decided to start a community driven project with at times different characteristics. We whole-heartedly invite you to join us here though :), that would be really great!
Best regards
Matthias


Update tours.json 'details' text

Updated details description and first PR test...

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@shant-hcl  can this be closed?  there are no changes (all reverted)

closing this one @shant-hcl @testradav


SwaggerDefaultValues: What is wrong with this class definition?

I have created class like below. I have similar classes in my project, non of them causing error except this one. I can't find any problem in my class but it's causing exception, In my description variable, ModelMetadata property is null which is cause error.
This is how description variable is defined in my SwaggerDefaultValues.cs
var description = context.ApiDescription.ParameterDescriptions.First(p => p.Name == parameter.Name);
And It throws exception here, ModelMetadata is null which is cause System.NullReferenceException:
parameter.Description = description.ModelMetadata.Description;
As i trace my code, this class definition causes issue,
CardPlusCNumberUpdateQueryParameter class:
using System;
using System.ComponentModel.DataAnnotations;

namespace WestCore.AppCore.Services.PCsbins.Customer
{
    public class CardPlusCNumberUpdateQueryParameter
    {
        /// <summary>
        /// Customer Number
        /// </summary>
        [Range(1, 999999)]
        [Required]
        public int CustomerNumber { get; set; }

        /// <summary>
        /// CardPlus Customer Number
        /// </summary>
        [Required]
        public string CardPlusCustomerNumber { get; set; }
    }
}

StackTrace:
at WestCore.Api.Swagger.SwaggerDefaultValues.Apply(Operation operation, OperationFilterContext context) in C:\Users\can.pahali\source\WestCoreApiSS\Main\WestCore.Api\Swagger\SwaggerDefaultValues.cs:line 38
       at Swashbuckle.AspNetCore.SwaggerGen.SwaggerGenerator.CreateOperation(ApiDescription apiDescription, ISchemaRegistry schemaRegistry)
       at Swashbuckle.AspNetCore.SwaggerGen.SwaggerGenerator.CreatePathItem(IEnumerable`1 apiDescriptions, ISchemaRegistry schemaRegistry)
       at Swashbuckle.AspNetCore.SwaggerGen.SwaggerGenerator.<>c__DisplayClass6_0.<CreatePathItems>b__2(IGrouping`2 group)
       at System.Linq.Enumerable.ToDictionary[TSource,TKey,TElement](IEnumerable`1 source, Func`2 keySelector, Func`2 elementSelector, IEqualityComparer`1 comparer)
       at System.Linq.Enumerable.ToDictionary[TSource,TKey,TElement](IEnumerable`1 source, Func`2 keySelector, Func`2 elementSelector)
       at Swashbuckle.AspNetCore.SwaggerGen.SwaggerGenerator.CreatePathItems(IEnumerable`1 apiDescriptions, ISchemaRegistry schemaRegistry)
       at Swashbuckle.AspNetCore.SwaggerGen.SwaggerGenerator.GetSwagger(String documentName, String host, String basePath, String[] schemes)
       at Swashbuckle.AspNetCore.Swagger.SwaggerMiddleware.Invoke(HttpContext httpContext, ISwaggerProvider swaggerProvider)

context.ApiDescription.ParameterDescriptions view: https://i.stack.imgur.com/WyqJI.png
Throwen exception: https://i.stack.imgur.com/P9nim.png

I don't think this is an issue with your model. According to the screenshot you provided, you have a route parameter called {CustomerNumber}, which does not have model metadata. Without seeing the route and action method, I can only speculate as to the issue. This usually means that the route parameter does match to an action method so the API Explorer doesn't know what the corresponding type is to build up its model metadata. There's two ways to fix this.
First, you can simply use the null propagation operator which will yield a null description, but won't throw an exception:
parameter.Description = description.ModelMetadata?.Description;

The second way is to obviously fix your action method or wherever this parameter is coming from. I hope that helps.

Did this help? Where you able to track down the issue?

Yes It helped, but I'm still don't understand why ModelMetadata is null. Thank your for your answer.


Modifier -> SwiftUI -> Drawing and graphics: brightness(_:)

Doc: https://developer.apple.com/documentation/swiftui/view/brightness(_:)

[ ] Swift implementation
[ ] Elixir implementation

Contribution needed


[🐞]useLocation() always returns "http" protocol, even though it is on https

Which component is affected?
Qwik City (routing)
Describe the bug
I have deployed my app to a live server, with https, and useLocation() always returns "http:" as protocol. This leads to the following error where I need to change URL parameteres for certain pages:
DOMException: Failed to execute 'replaceState' on 'History': A history state object with URL 'http://[...myURL]' cannot be created in a document with origin 'https://[...myURL]' and URL 'https://[...myURL]'.
At the moment, it looks like I'm gonna just to have use the standard document.location.
Reproduction
https://stackblitz.com/edit/qwik-starter-ddc5id?file=src%2Froutes%2Findex.tsx
Steps to reproduce
Deply any code to a live server with https and console.log(location.url).
System Info
NAME="Rocky Linux"
VERSION="9.3 (Blue Onyx)"
ID="rocky"
ID_LIKE="rhel centos fedora"
VERSION_ID="9.3"
PLATFORM_ID="platform:el9"
PRETTY_NAME="Rocky Linux 9.3 (Blue Onyx)"
ANSI_COLOR="0;32"
LOGO="fedora-logo-icon"
CPE_NAME="cpe:/o:rocky:rocky:9::baseos"
HOME_URL="https://rockylinux.org/"
BUG_REPORT_URL="https://bugs.rockylinux.org/"
SUPPORT_END="2032-05-31"
ROCKY_SUPPORT_PRODUCT="Rocky-Linux-9"
ROCKY_SUPPORT_PRODUCT_VERSION="9.3"
REDHAT_SUPPORT_PRODUCT="Rocky Linux"
REDHAT_SUPPORT_PRODUCT_VERSION="9.3"

Additional Information
No response

this is technically working as expected since useTask is running on the server and we pass the url from the server environment you're on. so in your adapter for your platform you can update the url to the correct url you want or it depends on which adapter you're deploying with

the fix would be to make sure your platform

ok I see what you're saying it's not really clear and only works correctly in certain adapters

Hi @PatrickJS - thanks for your reply. So this is the adapter config being used in the project:
export default extendConfig(baseConfig, () => {   return {     build: {       ssr: true,       rollupOptions: {         input: ["src/entry.express.tsx","@qwik-city-plan"],       },     },     plugins: [       nodeServerAdapter({ name: 'express'})     ],   }; });
Is there anything I can do in this config to remedy the situation?

Hi,
I can relate to that problem. In my setup, the qwik application is running within a cluster, and there are multiple proxies in front of it. It is necessary to always include the X-Forwarded-Proto Header within your proxy settings. Qwik uses the header to change the protocol of the origin.
It was confusing because I always set the ORIGIN env variable, and requests sent directly to qwik (without going through the proxies) always returned the correct URL.
I hope this is helpful.


No OSS license

There is no OSS license listed on github.
NPM has the License field as "MIT" but github has no matching information. Code posted on github cannot be assumed to be OSS without an explicit license, and NPM has no evidence of the license text or author's license intent.

Oh, I forgot that. Thanks for the reminder. It's MIT and a LICENSE file has been added.


MixinConfigPlugin class corrupted, likely by another mod

latest.log
This is the most concerning crash log I've ever acquired

looks like the mixin plugin had an error in a way that made mixin decide to dump the entire contents of the JVM stack in the log.

This is a raw stack dump produced by the JVM in response to an invalid or corrupted class file. Some other mod has corrupted Fabrication or one of the classes in ObjectWeb ASM; please try removing any mods that modify the Mixin subsystem, and ensure you are using the stable Java 11 release and not 15.
This crash is occurring before Fabrication even has the chance to load its mixins. It's pretty firmly not my fault.

More potentially useful background: Fabrication uses a somewhat obscure feature in Mixin that allows full code control over what mixins are loaded. I use it to make it so disabled features are never loaded at all to avoid incompatibilities being caused by disabled features.
The first part of doing this is the discovery process, where I scan my own classpath and figure out what mixins are available and under what conditions they can be enabled. This process uses ObjectWeb ASM to load my own classes and read their annotations without patching them at all. For some reason, in your copy of Minecraft, an essential part of the ASM API is missing, where ClassNode does not implement ClassVisitor.

I've just tested Fabrication under 1.16.4 and it works, so this isn't related to the Minecraft version.

I've looked through the mod list and, of the things I recognize, don't see anything terribly suspect. The things I thought might cause problems (MixinTrace, NotEnoughCrashes) seem to work fine on their own. I don't know what eer or loadcatcher are. I have good guesses for manyacharacter (NotEnoughCharacters fork?) and geyser-fabric (GeyserMC? The Bedrock↔Java translation layer?) but am not totally sure what they are.

Good assumption, yet this is a server-side crashlog.

Ah, I missed Mixin mentioning the environment. (This is an incredibly short log, a lot of the usual landmarks are totally missing.)
All I can recommend is removing mods one-by-one until you figure out which one is causing this. (Binary searching the mod list, if you know how to do it, is a lot faster, but I'm not quite sure of the best way to explain how to do it in sufficient detail.)
Once it's known why this is happening, I can look more into why and try and fix it.

After using a real JDK11 the crash log became this:
	at net.fabricmc.loader.EntrypointStorage.getEntrypointContainers(EntrypointStorage.java:193) ~[fabric.jar:?]
	at net.fabricmc.loader.FabricLoader.getEntrypointContainers(FabricLoader.java:272) ~[fabric.jar:?]
	at net.fabricmc.loader.entrypoint.minecraft.hooks.EntrypointUtils.invoke0(EntrypointUtils.java:44) ~[fabric.jar:?]
	at net.fabricmc.loader.entrypoint.minecraft.hooks.EntrypointUtils.invoke(EntrypointUtils.java:36) ~[fabric.jar:?]
	at net.fabricmc.loader.entrypoint.minecraft.hooks.EntrypointServer.start(EntrypointServer.java:32) ~[fabric.jar:?]
	at net.minecraft.server.Main.main(Main.java:92) [intermediary-server.jar:?]
	at jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:?]
	at jdk.internal.reflect.NativeMethodAccessorImpl.invoke(Unknown Source) ~[?:?]
	at jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source) ~[?:?]
	at java.lang.reflect.Method.invoke(Unknown Source) ~[?:?]
	at net.fabricmc.loader.game.MinecraftGameProvider.launch(MinecraftGameProvider.java:224) [fabric.jar:?]
	at net.fabricmc.loader.launch.knot.Knot.init(Knot.java:141) [fabric.jar:?]
	at net.fabricmc.loader.launch.knot.KnotServer.main(KnotServer.java:27) [fabric.jar:?]
	at jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:?]
	at jdk.internal.reflect.NativeMethodAccessorImpl.invoke(Unknown Source) ~[?:?]
	at jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source) ~[?:?]
	at java.lang.reflect.Method.invoke(Unknown Source) ~[?:?]
	at net.fabricmc.loader.launch.server.FabricServerLauncher.launch(FabricServerLauncher.java:62) [fabric.jar:?]
	at net.fabricmc.loader.launch.server.FabricServerLauncher.setup(FabricServerLauncher.java:106) [fabric.jar:?]
	at net.fabricmc.loader.launch.server.FabricServerLauncher.main(FabricServerLauncher.java:50) [fabric.jar:?]
Caused by: java.lang.NoSuchMethodError: 'sun.reflect.ConstructorAccessor sun.reflect.ReflectionFactory.newConstructorAccessor(java.lang.reflect.Constructor)'
	at com.unascribed.fabrication.FabricationMod.<clinit>(FabricationMod.java:55) ~[fabrication-1.2.3.jar:?]
	at java.lang.Class.forName0(Native Method) ~[?:?]
	at java.lang.Class.forName(Unknown Source) ~[?:?]
	at net.fabricmc.loader.util.DefaultLanguageAdapter.create(DefaultLanguageAdapter.java:45) ~[fabric.jar:?]
	at net.fabricmc.loader.EntrypointStorage$NewEntry.create(EntrypointStorage.java:114) ~[fabric.jar:?]
	at net.fabricmc.loader.EntrypointStorage$NewEntry.getOrCreate(EntrypointStorage.java:101) ~[fabric.jar:?]
	at net.fabricmc.loader.EntrypointStorage.getEntrypointContainers(EntrypointStorage.java:186) ~[fabric.jar:?]
	... 19 more

Is this still conflict?

"A real JDK 11"? What were you using before?
This crash is new, caused by a feature in 1.2.3 I neglected to test on a restrictive JRE. You're making it past the point that the other crash occurred, though.

"A real JDK 11"? What were you using before?
This crash is new, caused by a feature in 1.2.3 I neglected to test on a restrictive JRE. You're making it past the point that the other crash occurred, though.

It was a JDK 8, didn't pay much attention

JDK 8 should work... whatever, as long as it works now.

JDK 8 should work... whatever, as long as it works now.

Tested, it works now on Java 11

And JRE 8, thanks!


fix: small updates to dataproc sample, test and pom

Slight change to pom, changed region tags to include 'dataproc', added endpoint back to test (would not successfully delete the cluster without this)

@bradmiro Can you rebase this to the current version of master?


React + Vite SSR with Hono

What is the feature you are proposing?
I'm trying to perform SSR using React + Vite in Hono instead of Express, as explained in all online examples. The problem is that Hono middlewares are not compatible with Connect instances, so it's not possible to achieve the same goal.
Vite server provides a vite.middlewares which is a Connect instance, good for Express but not for Hono 'cause it expects as req an IncomingMessage that Hono context doesn't provide.
Is there any plan to make Hono middlewares fully compatible with Connect or is there any walkaround to get compatibility or any example to follow?
Thanks

Hi @redbaron76
Have you ever tried @hono/vite-dev-server? https://github.com/honojs/vite-plugins/tree/main/packages/dev-server?

Hono does actually provide the IncomingMessage using the node-server: https://hono.dev/docs/getting-started/nodejs#access-the-raw-node-js-apis
import { serve, type HttpBindings } from '@hono/node-server'

const app = new Hono<{ Bindings: HttpBindings }>()
app.all(
  '/*',
  createMiddleware<{ Bindings: HttpBindings }>(
    (ctx, next) =>
      new Promise((resolve) =>
        viteServer.middlewares(ctx.env.incoming, ctx.env.outgoing, () =>
          resolve(next())
        )
      )
  )
)

serve({ fetch: app.fetch, port: 3000 }, () => {
  console.log('Listening on http://localhost:3000')
})

Got something working to get Hono & Vite in both Bun & Node. Not sure if it extends to SSR, only supports streaming in Node.js
// Usage
import { serve, type HttpBindings } from '@hono/node-server'

const app = new Hono<{ Bindings: HttpBindings }>()
app.all('/*', viteMiddleware())

serve({ fetch: app.fetch, port: 3000 }, () => {
  console.log('Listening on http://localhost:3000')
})

// Library
import { createServer, type ViteDevServer } from 'vite'
import { IncomingMessage, ServerResponse } from 'http'
import react from '@vitejs/plugin-react'
import { HttpBindings } from '@hono/node-server'
import { createMiddleware } from 'hono/factory'

let server: ViteDevServer

export async function getViteServer() {
  if (!server) {
    server = await createServer({
      ...
      appType: 'spa',
      plugins: [react()],
    })
  }
  return server
}

export function viteMiddleware() {
  return createMiddleware<{ Bindings: HttpBindings }>(async (ctx, next) => {
    const vite = await getViteServer()
    return new Promise((resolve) => {
      // Node.js
      // @ts-expect-error
      if (typeof Bun === 'undefined') {
        vite.middlewares(ctx.env.incoming, ctx.env.outgoing, () =>
          resolve(next())
        )
        return
      }

      // Bun
      let sent = false
      const headers = new Headers()
      vite.middlewares(
        {
          url: new URL(ctx.req.raw.url, 'http://localhost').pathname,
          method: ctx.req.raw.method,
          headers: Object.fromEntries(
            // @ts-expect-error
            ctx.req.raw.headers
          ),
        } as IncomingMessage,
        {
          setHeader(name, value: any) {
            headers.set(name, value)
            return this
          },
          end(body) {
            sent = true
            resolve(
              ctx.body(body, {
                status: this.statusCode,
                statusText: this.statusMessage,
                headers,
              })
            )
          },
        } as ServerResponse,
        () => sent || resolve(next())
      )
    })
  })
}

I'll close this issue.
As mentioned above, the Node.js adapter can handle the IncomingMessage. Then, we have the @hono/vite-dev-server https://github.com/honojs/vite-plugins/tree/main/packages/dev-serve.
You can use the @hono/vite-dev-server. Or if you want to implement another one yourself, you can refer to it.
Regarding the Connect compatible API, we can discuss it here: https://github.com/honojs/hono/issues/3293


Out of FOV arrows - change Distance From Center based on distance from enemy

Add a little toggle to make Distance from center dynamically change depending on distance from enemy.
Depending on how distance from center is written, this may end up requiring a new method of distancing per-arrow..

No need to add your input if all you're going to do is tell everybody else reasons why they shouldn't waste their time on it. If someone likes the look of it, they'll work on it.


fastly: 8.2.1 -> 8.2.4

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/fastly/cli/releases.
meta.description for fastly is: Command line tool for interacting with the Fastly API
meta.homepage for fastly is: https://github.com/fastly/cli
meta.changelog for fastly is: https://github.com/fastly/cli/blob/v8.2.4/CHANGELOG.md
Updates performed

Golang update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 8.2.4 with grep in /nix/store/871fp582611qjy9lw4l3s7w0nb2qa92m-fastly-8.2.4
found 8.2.4 in filename of file in /nix/store/871fp582611qjy9lw4l3s7w0nb2qa92m-fastly-8.2.4

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
fastly

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/871fp582611qjy9lw4l3s7w0nb2qa92m-fastly-8.2.4 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A fastly https://github.com/r-ryantm/nixpkgs/archive/76fc06d8da14a1d187281e60a41e74cba0c55400.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/871fp582611qjy9lw4l3s7w0nb2qa92m-fastly-8.2.4
ls -la /nix/store/871fp582611qjy9lw4l3s7w0nb2qa92m-fastly-8.2.4/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

fastly

Maintainer pings
cc @ereslibre @shyim for testing.

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

1 package built:

fastly

Result of nixpkgs-review pr 225862 run on x86_64-darwin 1

1 package built:

fastly

I have requested a new snapshot to wayback machine but they are overloaded and is currently scheduled. When there is a permalink I will open a new PR that supersedes this one. I will keep this PR open in the meantime.


Page content not instantly shown, instead blank page with spinner

Attach (recommended) or Link to PDF file here: https://lasers.llnl.gov/content/assets/docs/news/age_of_ignition_book.pdf
Configuration:

Web browser and its version: Firefox Nightly 123.0a1,<PHONE_NUMBER>2522
Operating system and its version: Debian sid/unstable all packages up to date (GNOME Shell)
PDF.js version: Whatever shipped
Is a browser extension:

Steps to reproduce the problem:

Open the URL
Scroll through the PDF pages

What is the expected behavior? (add screenshot)

When scrolling the pages should be visible right away.

What went wrong? (add screenshot)
A lot of white pages/blank pages with a spinner. The content was only shown after several seconds. Even going to already viewed pages, these were blank.
Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):

@Snuffleupagus, wdyt about caching some "large" images on the disk in using indexDB ?
In order to minimize the size, we could store them as jpeg images.

Isn't this (mostly) another case where we'd like to slightly delay rendering on scrolling, to improve overall performance?

wdyt about caching some "large" images on the disk in using indexDB ?

That might require a fair amount of care to avoid caching too much data overall since a long PDF document (e.g. with hundreds/thousands of pages) with large images on every page could still affect (another type of) memory usage negatively.
Also, you'd need to ensure that the data is properly evicted from such a cache e.g. when unloading the PDF document.


Added tags page with tag cloud

@biomadeira This is a simple tags page with an old fashioned tag cloud of sorts. The more posts with a certain tag, makes the tag appear larger. I hope you find this a useful addition to jasper2.

Look at tags cloud at the bottom of this page.


chore(main): release 1.13.4

:robot: I have created a release beep boop
1.13.4 (2023-11-10)
Performance Improvements

♻ Optimization updater (61ccd4a)
♻ Optimize mirror group (81c007f)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/viarotel-org/escrcpy/releases/tag/v1.13.4 :sunflower:


[T5A1][T09-B3]Clarence Chee

Ready for review.

Some comments added. Feel free to comment if you have any questions. Once done, you may the PR after reading comments.


New Maintainer Onboarding: Whit Waldo

Whit Waldo (@WhitWaldo) has been added as a Dotnet SDK maintainer. This issue is to capture the different activities needed to onboard him.

If I could be added to the Maintainers list as "Whit Waldo (Innovian)", I'd appreciate it!

I'll take care of:

[ ] Add maintainer to MAINTAINERS.md.
[ ] Add maintainer from CNCF project-maintainers.csv.
[ ] Add maintainer add Dapr Maintainers & Community Managers doc on Google Drive.
[X] Add maintainer to maintainers role on Discord. (already done by some-one else)