VSOCK socket client would almost always fail

Describe the bug
Function s_update_local_endpoint(struct aws_socket *socket) defined at
source/posix/socket.c#L273 has the following logic:
        /* VSOCK port is 32bit, but aws_socket_endpoint.port is only 16bit.
         * Hopefully this isn't an issue, since users can only pass in 16bit values.
         * But if it becomes an issue, we'll need to make aws_socket_endpoint more flexible */
        if (s->svm_port > UINT16_MAX) {
            AWS_LOGF_ERROR(
                AWS_LS_IO_SOCKET,
                "id=%p fd=%d: aws_socket_endpoint can't deal with VSOCK port > UINT16_MAX",
                (void *)socket,
                socket->io_handle.data.fd);
            return aws_raise_error(AWS_IO_SOCKET_INVALID_ADDRESS);
        }
        tmp_endpoint.port = (uint16_t)s->svm_port;

When it is used as a client, it would almost always fail, as the ephemeral port grabbed from the system is likely be large number. How did we test this code?
Expected Behavior
aws_socket_connect() should succeed
Current Behavior
Failure with aws_socket_endpoint can't deal with VSOCK port > UINT16_MAX
Reproduction Steps
Create a client and connect to any VSOCK address
Possible Solution
use 32 bit port number
Additional Information/Context
No response
aws-c-io version used
version containing https://github.com/awslabs/aws-c-io/commit/df07e42286aa67d15ebe86d1de2d68c22e6eadb2
Compiler and version used
clang14
Operating System and version
Linux

VSOCK support isn't 100% complete, as you've discovered. Someone submitted this partial support, and we integrated it to unblock them, but never bound it out to any other languages.
Finishing support would mean widening aws_socket_endpoint.port, add error-checking anywhere we cast it gets narrowed to 16bits, widen the value in our other language bindings, etc, etc, etc. It's not on our current roadmap.
I assume you're trying to use this directly from C? What's your use case?

I'm working on a fix...

It is done
port is 32bits now everywhere


Add 6 icons

Screenshots
Original Icons:

New Icons:

New icons - Orbot, Tutanota & Pixel Wallpapers:

Apparently, a QuickSwitch icon has been already added in #297. I will keep my version of it in this PR, since i like it more, but let me know if i should remove it.

Apparently, a QuickSwitch icon has been already added in #297. I will keep my version of it in this PR, since i like it more, but let me know if i should remove it.

Hey, I actually like your version! It looks even better! Hopefully this got merged soon! 😁

Thank you, @IAmRuMaks!


Upgrade Checkstyle 10.12.1 -> 10.12.2

This PR contains the following updates:

Package
Type
Update
Change

Checkstyle (source)
build
patch
10.12.1 -> 10.12.2

[ ] If you want to rebase/retry this PR, check this box

Warning
Renovate's suggested commit message is being replaced with improved initial commits to enable automerging. As a side effect, these suggested commit messages might have changed. Consider comparing the initial commit message and this message to determine the most suitable one. Please leave feedback in #sys-renovate.

Suggested commit message:
Upgrade Checkstyle 10.12.1 -> 10.12.2

See:
- https://checkstyle.sourceforge.io/releasenotes.html
- https://github.com/checkstyle/checkstyle/releases/tag/checkstyle-10.12.2
- https://github.com/checkstyle/checkstyle/compare/checkstyle-10.12.1...checkstyle-10.12.2


Fix tank health stage not updating

When using a custom tank model, pop extensions currently won't set the model of the tank if it gains health instead of losing it.

Found another issue while playing around with this. Tanks begin animating tracks in CTFTankBoss::Spawn but since we swap the model after that happens, custom tank tracks currently do not animate.
I've submitted a fix for this too, let me know if there are issues.


Support for OpenAI Base API Integration

This PR introduces a feature mentioned here
This feature introduces the capability for users to integrate their own OpenAI Base API. Currently, our system is reliant on the OpenAI's primary API Base. Given the associated costs and the availability of different models such as GPT 3.5 and GPT 4, it's imperative to provide an alternative. By allowing users to supply their own base API, we can potentially offer more cost-effective or even complimentary access to GPT models. This can be configured through a dedicated settings file, enabling users to utilize the GPT Engineer with their preferred version of GPT.
The command to Run GPT-Engineer using Custom base API.
python -m gpt_engineer.cli.main projects/example 'gpt-4' --openai-api-base https://heaven-gpt.haseebmir.repl.co

@ATheorell: I kindly request your feedback on the following proposed adjustments:
Current Configuration:

The API Base is configured via the command line option --openai-api-base.
At present, the streaming functionality is deactivated.
It's mandatory to set the API Key using the .env (Environment) file.

Proposed Enhancements:

Implement a mechanism to modify the API Base for both streaming and non-streaming responses.
Allow the API Key to default to Empty when a specific base API is chosen. While this is feasible with LiteLLM, it presents challenges with LangChain and OpenAI. I am exploring more solutions to address this.
I have created this issue on LangChain about this here LangChain OpenAI API Key Optional check it out.

This runs for me.
@pbharrin and @AntonOsika, this goes beyond my skill level, would be great if you could give some input.
I also suggest that, if we merge #787 , we should add this as an example of a server that can be run @krrishdholakia .
Comparing to #787 , maybe it would be enough to set OPENAI_API_BASE as an environment variable (no need for a new flag in that case)?


High availability for Seata

Why you need it?
Is your feature request related to a problem? Please describe in details
Running Seata without HA seems to be a serious bottleneck as it it will be a single point of failure
How it could be?
What do we need to do to enable HA in Seata ? There is go version of the service which is supposedly HA, however that version has zero documentation to set it up
To make the Java version HA what are the changes we need to do ?

@amit2103 https://seata.io/zh-cn/docs/ops/deploy-ha.html

@slievrly  Thanks , but most of it is Chinese, an example will help in English

@slievrly Thanks , but most of it is Chinese, an example will help in English

https://seata.io/en-us/docs/ops/deploy-ha.html

I will close this issue later. If you still have problem and you can keep report to us.


Feature: Support Ollama and/or setting custom OpenAI endpoint

This would be good to use with Ollama which has both a native API which provides things such as model listing etc... and an "Open"AI compatible API.

You can use it with Ollama already: https://github.com/j0rd1smit/obsidian-copilot-auto-completion/blob/master/docs/how-to/Ollama setup guide.md


Upgrade to version 1.16.0 generates error CS0426

After upgrading to version 1.16.0 I get build error on all my azure functions:
CS0426 - The type name 'ExampleFunction' does not exist in the type 'ExampleFunction' - GeneratedFunctionExecutor.g.cs
VS:2022
SDK:7.0.403
Works as expected if i downgrade to version 1.15.1
### Tasks

@hulken76 Do you happen to have the same value("ExampleFunction") for the namespace and your class name? If so, you are experiencing this known issue #1990 . We have a fix coming out soon. Until then, please add the below entry to your csproj file to explicitly specify a namespace to be used for the auto generated types.
<FunctionsGeneratedCodeNamespace >YourProjectNamespace</FunctionsGeneratedCodeNamespace>

Closing this as it should be resolved by #1990


CPK source headers not supported for Copy Blob

@alzimmermsft, you were thinking this is a swagger issue.

Should this be a service request as the Copy Blob documentation doesn't state that it takes CPK headers.
https://docs.microsoft.com/en-us/rest/api/storageservices/copy-blob

Closing this issue as the Copy Blob REST APIs don't take CPK and we internalize CPK through our builders so it isn't an issue that these APIs don't take it.


Cannot find chinese_wwm_pytorch

Hi, I encountered a problem when running the code in benchmark/bertret and want to seek your help. It seems that the 'chinese_wwm_pytorch' cannot be found, including all related files (/vocab.txt, /added_tokens.json etc.):

Another weird problem is when running ./train_film(music, travel) in other models except bert, like LM andseq2seq, I always encountered the segmentation fault and didnt figure out the reason:

Thanks so much!

You should prepare the pretrained model, such as from https://huggingface.co/hfl/chinese-bert-wwm-ext
I am not sure the causes. Maybe you can use another python version like 3.6. I guess that the different versions of libraries (such as pytorch and tensorflow) may also lead to the problem

Hi chujie, thanks so much for your reply.
The problem2 has been solved by changing the python version to 3.6.
For problem1, I tried to download the BERT-wwm-ext from the link and it only contains 3 files: bert_config.json, pytorch_model.bin and vocab.txt

and after downloading it, when I tried to run train_film.sh in bertret folder, it shows the following:

I want to ask

is it normal that added_tokens.json, special_tokens_map.json, tokenizer_config.json are not found?
there is no chinese_stop_words.txt (resources folder) in data, did I miss any step?

Thanks so much!

It is normal. The new version of transformers will try to load these files, but the earlier models do not have these files. So feel free to ignore the warnings.
You can download a stop word list like this: https://github.com/goto456/stopwords


Type Hinting on suffix rules enforcing boolean values

The generated componentClassSuffixRule.d.ts and directiveClassSuffixRule.d.ts files require a boolean value for validation, whereas either:
"component-class-suffix": true,
"directive-class-suffix": true,

or
"component-class-suffix": [true, "Component"],
"directive-class-suffix": [true, "Directive"],

appear to be supported

Would you give me more information for what you mean here?
I'm not sure I understand the problem.

Sure thing.
When adding component-class-suffix or directive-class-suffix rules to your tslint.json, the type hinter tells you that a boolean is expected.  This is a fairly low hanging fruit issue, either rule works, but if using the [true, "Component"] syntax the type hinter will complain.  Here's an example from Visual Studio Code:

I see, thanks. Will work on this tomorrow.

Opened PR here https://github.com/SchemaStore/schemastore/pull/306.


Fix same size padding

As mentioned in #174.
If the kernel_size is even, the same_size_padding requires an additional right column at the height and/or an additional bottom column so that the input_size is not changed.

However, I see the problem with the optional parameter input that it is not available at the time of initialisation.

Hm, yeah I didn't think of that.

For this reason I would suggest that this additional padding is done within the created ConvBlock

I think we could generalize this a bit more: we could implement a PaddedConv2d that also takes "valid" (default), "same", and "full" as the parameter padding. In forward we could call torch.nn.functional.pad and simply pass this output to Conv2d. Thoughts?

Yeah, I'm okay with that.

@jbueltemeier I've settled for same size padding only. Please review if you find obvious shortcomings.

This was more work than I expected, but I think its done now. @jbueltemeier please review and merge if you are happy.
I've implemented the _SameSizeConvNdMixin which should be used in double inheritance with any nn.Conv* module. With it SameSizeConv2d and SameSizeConvTranspose2d is just an empty class definition.
Furthermore, this removes the now obsolete functions same_size_padding, same_size_output_padding and adapts all places where they were used.


[SFT] add model_init_kwargs to training_args

Fix SFTTrainer warning by moving model_init_kwargs to the SFTConfig

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@vwxyzjn this is the only example with this warning


trivial: remove second definition of _COMMA

This was moved into Makefile in #82 !

/retest
/approve
/lgtm


Proposal: use inheritance instead of composition for the repository pattern

I think having chosen composition for the repository pattern is a problem here:
UserRepository.js
class UserRepository {
  constructor(repository) {
    this.repository = repository;
  }

  persist(userEntity) {
    return this.repository.persist(userEntity);
  }

  // ...

}

UserInMemoryRepository.js
// Nothing to see here, no explicit dependency :(

UserController.js
  // ...
  const userRepository = new UserRepository(new UserRepositoryInMemory());
  //...

It has at least three inconvenients:

having to explicitely delegate every method call to this.repository
breaking the plugin architecture (here a change in the outer circle code would require to change the inner circle code)
having no explicit dependency to the UserRepository class in the in memory & sqlite implementations

I propose to use inheritance instead:
UserRepository.js
class UserRepository {
  persist(userEntity) {
    // This optional generic implementation is obviously meant to be overriden.
    // See also Python's NotImplementedError and Smalltalk's subclassResponsibility.
    throw new Error('ERR_METHOD_NOT_IMPLEMENTED');
  }

  // ...

}

// Explicit dependency here :)
const UserRepository = require('../path/to/UserRepository')

class UserRepositoryInMemory extends UserRepository {
  persist(userEntity) {
    // Real implementation here
    // ...
  }
}

UserController.js
  // This initiation could be done in a composition root,
  // which would be the only place where things could get a little dirty ;)
  const userRepository = new UserRepositoryInMemory();

  // ...

About composition roots:

https://blog.ploeh.dk/2011/07/28/CompositionRoot/

Illustrated in PR #6.
Note: even better, make UserRepository a real interface (requires to use a JS static type system like TypeScript). But this is another topic.
Cheers!
Sébastien.

Change accepted and merged (thanks @sroccaserra ) 👌

Change accepted and merged (thanks @sroccaserra ) 👌


Auto-detect terminals and their fonts

List of supported terminals for auto generating a good fontconfig file.

If your terminal isn't supported yet, comment this issue to tell me where to lookup its font familly, PR are welcome (see scripts/detect_font_terminal/).
If your terminal is listed as supported but the script install-autodetect.sh fails, please comment this issue.

[ ] aterm  link
[ ] Cool Retro Term  link
[ ] Eterm  link
[ ] evilvte  link
[ ] Gate One  link
[ ] Germinal  link
[x] GNOME Terminal  link
[ ] Guake  link
[ ] Konsole  link
[ ] LilyTerm  link
[ ] LXTerminal  link
[ ] MATE terminal  link
[ ] mlterm  link
[ ] Pantheon Terminal  link
[ ] QTerminal  link
[ ] ROXTerm  link
[ ] rxvt  link
[ ] sakura  link
[ ] shellinabox  link
[ ] st  link
[ ] Terminal  link
[x] terminator  link
[ ] Terminology  link
[ ] Termite  link
[ ] Tilda  link
[ ] Tilix  link
[ ] tinyterm  link
[ ] urxvt  link
[ ] xterm  link
[ ] Yakuake  link

You can add fbterm to the list of supported terminals. I installed this using install-autodetect.sh (for gnome-terminal) and all the glyphs show up nicely in fbterm too.

Works in tilix too. Thanks!

Hi @BlitzKraft and @IvoLimmen, I didn't write any script to autodetect the fonts in tilix or fbterm.
There could be 2 reasons why it works with you:

You're using the same font in tilix/fbterm and gnome-terminal (or terminator, the only 2 supported)
You have no other fonts in your system with glyphes inside the private use areas, so freetype look for glyph in all the others fonts of your system until find one in icons-in-terminal.ttf

I recommend you to check the file .config/fontconfig/conf.d/30-icons.conf to see if the font you're using in tilix/fbterm is listed

I checked the config, it just says "No font found". Here is the full xml doc:
<?xml version="1.0"?>
<!DOCTYPE fontconfig SYSTEM "fonts.dtd">
<fontconfig>
  <alias><family>No font found</family><default><family>icons-in-terminal</family></default></alias>
</fontconfig>

So, I must be missing something. The font I am using in gnome-terminal is "Hack". I also tested with Iosevka (patched for powerline) and that doesn't work. Many symbols show up with "missing glyph squares".

@BlitzKraft I've open an issue about that #7
In the meantime, you can run install.sh, it will replace .config/fontconfig/conf.d/30-icons.conf. Edit this file to remove all the lines but the ones with 'Hack' and the font you're using in fbterm

Mine says:
<?xml version="1.0"?>
<!DOCTYPE fontconfig SYSTEM "fonts.dtd">
<fontconfig>
  <alias><family>Droid Sans Mono for Powerline</family><default><family>icons-in-terminal</family></default></alias>
  <alias><family>No font found</family><default><family>icons-in-terminal</family></default></alias>
</fontconfig>

I use the Hack font. Demo did just fine.

Mac support would be awesome, in particular iTerm2!

@amoriello I don't have a Mac so I can't help.
Can you try this method and tell me if it works ? I will adapt it to icons-in-terminal if you confirm it

Hey Seb,
I finally found cycles to look at this.
The tutorial you pointed out worked just fine (iTerm2 3.0.15), thanks.
Just replace the section about editing Plist that says:

add a new row and type in Pomodoro. Do this three more times for FontAwesome, Octicons, and Icomoon

to

add a new row and type in icons-in-terminal.

I've attached the screenshot to help you making this.

Regarding print_icon.sh
This stuff just doesn't work for me, it prints the unicode code point instead of the icon.

I did not took time to look in this shell madness of yours, so I made a quick python script to ensure the fallback mechanism is working properly:
#!/usr/bin/env python3

import os
from collections import namedtuple

_, columnsStr = os.popen('stty size', 'r').read().split()
columns = int(columnsStr)

IconInfo = namedtuple('IconInfo', ['name', 'code'])

def banner(text, width=80):
    title = ' '.join(text.upper()).center(width)
    print('#' * width)
    print('%s' % title)
    print('#' * width)

def getIcons(line):
    icons = line.split(';')
    for icon in icons[:-1]:
        yield IconInfo(*icon.split(':'))

def getFontsInfo(mapping):
    fontsInfo = {}
    currentFontName = ''
    for line in mapping:
        if line.startswith('#'):
            currentFontName = line[1:].split(':')[0]
            continue
        yield currentFontName, getIcons(line)

mapping = open('./build/mapping.txt', 'r')
for fontName, icons in getFontsInfo(mapping):
    banner(fontName, columns)
    for icon in icons:
        print('%s  ' % chr(int(icon.code, 16)), end='')
    print('\n' * 3)

Tutorial Screenshots
Font book

Plist

Finder

Cheers.

@amoriello Thank you very much for your help !
I will update the readme and include your nice pictures.
Good to know it works on Iterm

It works also with xfce4-terminal

I just see a bunch of boxes on Urxvt (however terminator works on the same system). 30-icons.conf below. Font family is in Xrerouces URxvt.font: xft:Noto Mono:size= 14
<!DOCTYPE fontconfig SYSTEM "fonts.dtd">
<fontconfig>
<alias><family>Noto Mono</family><prefer><family>icons-in-terminal</family></prefer></alias>
</fontconfig>

It don't work for me on xterm & uxterm. (But i don't hard try)
But it work on termite very well. https://github.com/thestinger/termite

WSLtty on Windows 10 doesn't play nice with auto or manual installation.
I'm guessing fontconfig can't update the terminal font via device control codes?

Konsole works nicely. It can be checked off. I tested on konsole version 19.08.3. Installed using the install.sh script.

Just want to say the walkthrough posted for MacOS no longer works in Catalina. The DefaultFontFallbacks.plist has either moved or been replaced with some other solution so people on that repo are moving back to Patched Fonts.

hello to all ,
i did everything i could and followed as well the post...im using kitty the test does work im using hack as my main font ...if i test it in the terminal i can see all the glyphs but whatever im doing ...theres no way to have the icons displayed in nnn ...did i miss something ?? thanks for your help

can i use it on windows 10 without WSL??

Can you please add support to kitty?
Website: sw.kovidgoyal.net/kitty

Kitty seems to work fine for me.

could you add alacritty to the list of terminals that need supporting

The solution mentioned doesn't work for me on MacOS 13.1 for iterm as there is no DefaultFontFallback.plist:

Foot terminal font successfully installed through autodetect, works perfectly

@sebastiencs Fancy to add Alacritty as suggested by @godalming123 previously?

Hey Seb,
I finally found cycles to look at this. The tutorial you pointed out worked just fine (iTerm2 3.0.15), thanks.
Just replace the section about editing Plist that says:

add a new row and type in Pomodoro. Do this three more times for FontAwesome, Octicons, and Icomoon

to

add a new row and type in icons-in-terminal.

I've attached the screenshot to help you making this.
Regarding print_icon.sh
This stuff just doesn't work for me, it prints the unicode code point instead of the icon.

I did not took time to look in this shell madness of yours 😜 , so I made a quick python script to ensure the fallback mechanism is working properly:
#!/usr/bin/env python3

'''
check_icons.py
'''

import os
from collections import namedtuple

IconInfo = namedtuple('IconInfo', ['name', 'code'])
FontInfo = namedtuple('FontInfo', ['name', 'num_entries'])

def banner(text, width=80):
    ''' Just a banner
    '''
    title = ' '.join(text.upper()).center(width)
    print('#' * width)
    print('%s' % title)
    print('#' * width)

def icons_info(line):
    ''' Generator over icons in line
    '''
    icons = line.split(';')
    for icon in icons[:-1]:
        yield IconInfo(*icon.split(':'))

def fonts_info(mapping):
    ''' Generator over fonts and their icons
    '''
    for line in mapping:
        if line.startswith('#'):
            font_info = FontInfo(*line[1:].split(':'))
            continue
        yield font_info, icons_info(line)

def main():
    ''' main
    '''
    mapping = open('./build/mapping.txt', 'r')
    columns = int(os.popen('stty size', 'r').read().split()[1])
    for font_info, icons in fonts_info(mapping):
        banner(font_info.name, width=columns)
        for icon in icons:
            print('%s  ' % chr(int(icon.code, 16)), end='')
        print('\n' * 3)

if __name__ == '__main__':
    main()

Tutorial Screenshots
Font book

Plist

Finder

Cheers.

Do you know any reason why print_icons.sh shows only the code points, but your Python script successfully shows the icons?
I'm using macos sonoma, Iterm2, and zsh shell.


ARM: alignment problems in libc/newlib

The following Kconfig option indicates that CONFIG_LIBC_ALIGNED_HEAP_SIZE only gets a non-zero value for ARMv7-M builds with MPU: Builds with ARMv8-M and NXP_MPU do not have the requirement for power-of-two & size alignment.
config NEWLIB_LIBC_ALIGNED_HEAP_SIZE
	int "Newlib aligned heap size"
	depends on MPU_REQUIRES_POWER_OF_TWO_ALIGNMENT
	depends on NEWLIB_LIBC
	depends on USERSPACE
	default 0

However, when building with CONFIG_USERSPACE, we still enclose the heap in an user-accessible memory partition:

static int malloc_prepare(struct device *unused)
{
	ARG_UNUSED(unused);

#if CONFIG_NEWLIB_LIBC_ALIGNED_HEAP_SIZE
	z_malloc_partition.start = (u32_t)heap_base;
#else
	z_malloc_partition.start = USED_RAM_END_ADDR;
#endif
	z_malloc_partition.size = MAX_HEAP_SIZE;
	z_malloc_partition.attr = K_MEM_PARTITION_P_RW_U_RW;
	return 0;
}

In the above snippet, USED_RAM_END_ADDR is taken directly from .end address in the linker, which might not be aligned with the minimum MPU granularity for ARM (32 bytes), triggering an ASSERT when attempting to program the region.

@andrewboie FYI

can we just align the _end symbol in the linker script? seems like a trivial fix

can we just align the _end symbol in the linker script? seems like a trivial fix

I think I would promote a more clean solution @andrewboie. I mean, in the libc hooks, we are still creating a mem partition with nPRIV rights outside the #ifdef NEWLIB_LIBC_ALIGNED_HEAP_SIZE, which obviously needs MPU. So IMHO it's the responsibility of the modulte to make sure it uses the right address for the heap. It's not the linker script that is responsible for that...

You can also round up z_malloc_partition.start = USED_RAM_END_ADDR; to whatever it needs to be.

You can also round up z_malloc_partition.start = USED_RAM_END_ADDR; to whatever it needs to be.

Yes @andrewboie we could do that. We also need to round up the _end inside the libc cause it is used directly as is.


Fix error observed on admin page when running with the mockAPI.

What is the context of this PR?
Fixes an error when navigating to the admin page when running with the mockAPI.
The fix involved specifying a reducer for the scalar GraphQL Date type and ensuring that time values do not make it into the localStorage data store.
How to review
All tests and checks should pass.
Should no longer see the error on the admin page after creating a questionnaire when running with the mockAPI.

@WickyNilliams be good to test the third merging option with this PR?


The generated BOM is not in a format that works with suppliers

Can we get an export of the BOM that works with octopart and/or digikey?

The BOM was never intended to be in a supplier-compatible format. It consists of 34 different components.
I believe that typing them in manually shouldn't stop anyone from building their own card. And, as a courtesy, I took the time to add DigiKey part numbers to make it even simpler. That's as much effort as I'm willing to spend on this matter.

All you need to do is add a data format like CSV.  It is probably one of the default export options.

@peon374 If you like to create such a CSV file then feel free to contribute the file back to the repo with a pull request. I don't think it makes sense that you insist that someone else should do that for you.

Awesome, will do shortly.


Add license file

added apache license

@yanivmo could you please merge this? I don't seem to have write access to this repo


Swapping nodes of a linked list

Hi I am samar. I am a GSSOC 2021 participant. I want to implement code for swapping kth node from beginning and end of linkedlist.  I will provide proper documentation and comments to make it understandable for a newbie in coding. I will also follow proper code structure to make it easily understandable.
Thanks and regards
A.M.Samara simha reddy

Hello, I am a participant of GSSoC'21. I can contribute to this issue.
Please assign me this task
Thank you!

Hello, I'm Upasana Singh. I want to contribute to this issue in c++. Please assign this to me.


Add pull consumer interface and polished Admin interface

(1) Polish existed ons api.
(2) Add pull consumer.

Some important design points hope to be covered:

Does need to support consumerRebalanceListener in subscribe method? It is very important when we using async commit or hope to store the offset in the external storage.
Manually control the queue assignment, adding a similar method such as assign?
CommitSync and commitAsync variants and control parameters, such as commit interval, enable auto commit
Interceptor for consumers, such as beforeCosume and AfterCommit.
Time windows based thread-safe consumer logic.
Replace long type time with Duration object.


Text Filters

Motivation:

https://github.com/smpallen99/ex_admin/issues/49

Changes:

Use the new string filters available in ex_queb
https://github.com/E-MetroTel/ex_queb/pull/5
Add the new filter options to the select options
Setup tests and styles.

So, I am using webpack in my local setup. For some reason the .css.css activeadmin file does not work with this setup. So I couldn't properly test the activeadmin theme's styling, please take a look.

@gwincr11 Default does not work. Default select box says "Contains", but the filter applied is _eq.
I'll also release a new ex_queb to hex so this will pickup the correct version of ex_queb.

@smpallen99 should the default be equals or contains?

@smpallen99 ok, I have it working with the default now.


chore: add examples for phyrexian hybrid mana

closes #230

Closing this for now. We'll add something like this if/when it becomes relevant for a combo.


Bugfix for Content-Encoding: deflate

Refer to https://stackoverflow.com/a/22311297
zlib library supports:
RFC 1950 (zlib compressed format)
RFC 1951 (deflate compressed format)
RFC 1952 (gzip compressed format)
choosing windowBits
to (de-)compress deflate format, use wbits = -zlib.MAX_WBITS
to (de-)compress zlib format, use wbits = zlib.MAX_WBITS
to (de-)compress gzip format, use wbits = zlib.MAX_WBITS | 16
See documentation in http://www.zlib.net/manual.html#Advanced (section inflateInit2)
Signed-off-by: Phus Lu<EMAIL_ADDRESS>

Move PR to https://github.com/httplib2/httplib2/pull/54


misspelling in markdown file

renamed 02 Building_DSC_Artefacts.md → 02 Building_DSC_Artifacts.md

It's an UK Vs US spelling difference ;)


fix: ui spacing issues, alignments & fonts exlm-1053

Please always provide the Jira Issue your PR is for, as well as test URLs where your change can be observed (before and after):
Jira ID: https://jira.corp.adobe.com/browse/EXLM-1053
Test URLs:

Before: https://main--exlm--adobe-experience-league.hlx.live
After: https://bugfix-exlm-1053--exlm--adobe-experience-league.hlx.live

@nitin-rachabathuni Please make sure it is not impacting the existing behaviors.


[0006] Take a stance on function references

Which proposal does this relate to?
https://github.com/microsoft/hlsl-specs/blob/main/proposals/0006-reference-types.md
Describe the issue or outstanding question.
A stance on function references (somewhat affects variables too #99 ) needs to be taken, esp w.r.t. possibility of leaking, i.e. storing a bunch of function references in a local array and dynamically indexing that or using ? operator to pick between two references on assignment.
Essentially the question is if the functions have to be strictly inlineable by the compiler at compiler time or not, if the answer is no then you're likely to run into a wall:
https://github.com/KhronosGroup/Vulkan-Docs/issues/2232
However I would not ban function references outright because then you can't specialize a template on a function.
Additional context
Here's an example of a specialization that fails because I can't declare a void(&)(void), note that I actually never need to call it through the reference, or even create a variable of this type!
https://godbolt.org/z/M7xTqK5T3
Aside from unholy things I'm trying to polyfill around, this is important because its the reason we're having to rework every single simple stateless function I have into a struct functor with an operator() (its also annoying that operator() can't be static but thats something fixed in C++23)
https://github.com/kpentaris/Nabla/blob/hlsl/include/nbl/builtin/hlsl/binops.hlsl
because I can pass a struct add to workgroup::exclusiveScan<uint32_t,add,Accessor> but not when its a function.
https://github.com/kpentaris/Nabla/blob/hlsl/include/nbl/builtin/hlsl/workgroup/arithmetic.hlsl

Actually if I use the compiler builtin __decltype at my own perif I can coax it into giving me a void(&)(void) and specialize a template on a function
https://godbolt.org/z/q7dc9EoP1

I guess something like only allowing function references which look like constexpr Ret(&)(Args..) would do the job
https://godbolt.org/z/EaKaa5KGv

Implementing shuffles (think subgroup shuffles) at the workgroup level and it would be nice to specify code once and just pass a bijective function for indices as a template parameter instead of having to rely on macros


feat: separate value from text content when filtering

I have a use case where I want my value to be different than the label presented to the end user. When typing in the search field today, the user would have to know the underlying value to be able to see something return.... typing what is visible text content wouldn't yield the expected filtered results.
My ask is that we change the behavior so that the search field only looks at the text content rendered (we have to pass this anyway for something to show up). The only way an average user would know what the value is is if they opened their dev tools, which is not expected.

You should be able to pass a value prop to Command.Item and use children to display the label.

Yes I understand this, but as I’ve pointed out, doing so affects the filtering capability. It will only filter on the value, not the children’s text content

I think this is a valid use case that is a little difficult in the current API.
@Kinbaum Can you share where you want to use this internal-only value? Inside of onSelect?

As a workaround, you could include both values in the value string:
<Command.Item value={`${internal}:${searchLabel}`}>{searchLabel}</Command.Item>

I think so. And I’ve been stringifying a JSON object as sort of a hack (still has issues - see below)
A use case would be pushing the user to a new route on select of an item.
But this also poses challenges when setting the current value as that cannot be derived from the cleaned URL if you go with the hacked approach.

@pacocoursey Is there any update on this feature request

I have unique IDs for each of the items in my list, but because filtering happens on values and not labels, I'm having to use the labels as values, which is not ideal.
I wonder why filtering was done on values, to begin with. Isn't it always preferred to filter on labels?

I think I was very confused in my last comment on here. I really wanted value to be a unique identifier, but I don't know why.
My requirements have changed a bit since. I now need to be able to find items with additional hidden search terms. I was able to get things working perfectly by appending additional search terms to the value property and using a custom filter function.

One way could be to pass the label to the Command.Item keyword prop. This might help, but it has one drawback: the filter will also show the items with matching values.


✨ [Feature] Remove framework GPL dependencies

Before feedback (在反馈之前)

[X] There are no features that I want to submit in Issues (当前 Issues 中没有我将要提交的新功能)

Describe Feature (描述功能)
According to the dependency analysis results provided by FOSSA, we may need to remove the GPL part of the framework, which is mainly used in docker testing and sqlite driver.

Remove these packages below:

github.com/golang-migrate/migrate
github.com/glebarez/go-sqlite

We can study the implementation of https://github.com/go-gorm/playground. They did not import the docker package for testing.

I think if we remove the Docker testing, we will lack confidence in whether the function of Orm is normal ~ Or we can find another way to confirm this, go-gorm/playground requires active feedback from User, which is a bit passive ~

I think if we remove the Docker testing, we will lack confidence in whether the function of Orm is normal ~ Or we can find another way to confirm this, go-gorm/playground requires active feedback from User, which is a bit passive ~

go-gorm/playground also use CI to run, I think it can also be done to run tests.

Oh, you mean this: https://github.com/go-gorm/playground/blob/master/.github/workflows/tests.yml, yes, we can check it.

Oh, you mean this: https://github.com/go-gorm/playground/blob/master/.github/workflows/tests.yml, yes, we can check it.

Yes, they run a fully functional GORM including a variety of different databases, interesting.

We need to add an explanation to show how to run github/action locally.

We need to add an explanation to show how to run github/action locally.

Running GitHub Action locally? It seems difficult. I wonder if GitHub provides a private deployment solution for Action.

This: https://github.com/nektos/act


Q: Mobile Safari >10.* buggy calc()

Great library, being using it for a while now and it's been working great!
Now to my question: there is a known issue starting in Mobile Safari 10.* (e.g. iPads) that calc(% px) does not work correctly. In the code I can see a fallback for IE8, I am wondering how to trigger this for iOS such that Split.js wont use calc but assign static sizes (and obviously disable dragging)?
Thanks in advance

Do you have minimum code to replicate this issue in Mobile Safari?


[5.0] Fix runtime pack RID list: omit source-built RID

Fixes https://github.com/dotnet/source-build/issues/1746
Running a local build now to verify manually.

I confirmed locally that this fixes it: dotnet restore -r centos.7-x64 in my CentOS 7 build downloads the linux-x64 runtime packs.


chore(nr1-learn-nrql): add project

Thank you @jospdeleon for the PR. Merging. Expect to see this in prod tomorrow.


Update high-availability-guide-suse-nfs.md

Adding that we should be using at least SLES 12 SP3 and also , links referenced for installation were broken, so removed.
These links that were removed:
https://portal.azure.com/#create/suse-byos.sles-for-sap-byos12-sp1

@mabicca : Thanks for your contribution! The author, @mssedusch, has been notified to review your proposed change.

looks good
#sign-off


MACS Differential Binding Output And Peak Call Numbers Doesn't Match

Hi taoliu,
I ran MACS2 callpeak like this
macs2 callpeak -B -t chip11.bed chip21.bed chip31.bed -c input1.bed -n set1 --nomodel --extsize 195 -f BED -g mm
macs2 callpeak -B -t chip12.bed chip22.bed chip32.bed -c input2.bed -n set2 --nomodel --extsize 195 -f BED -g mm
and did bdgdiff to get the differential peaks.
However, the three sets of files that bdgdiff generated had the following numbers
enriched_in_condition1=1428
enriched_in_condition2=29
common=1155
But the MACS2 callpeak command outputs 5349 peaks in the excel sheet for condition1 and 2610 peaks for condition2. This number doesn't match to the numbers from bdgdiff output. Why is it so? Can you please share any pointers. Thanks in advance.

@gk7279 Because the differential caller uses a different type of threshold and parameters (the likelihood ratios) to call the regions. I would suggest you run callpeak for each condition and do an overlap.

Thanks @taoliu . However, each condition has 3 replicates and its own input. But if we run them separately by asking for three bdgs, how do you recommend running bdgdiff to compare more than 1 bdg file? Is that possible? Kindly advice.

Currently, the bdgdiff can ONLY consider 1 to 1 comparison. When I said 'for each condition', it means your pooled condition1 (set1) and condition2 (set2).

Sorry. But I am missing something. I quite didn't get you. What about the overlap? Please share some pointers. Thanks.

@gk7279 About the overlap. What I suggested is to overlap your 5349 cond1 peaks or/and 2610 cond1 peaks with the bdgdiff calls to make a detail comparison. For example, if a peak in the 1428 enriched_in_condition1 list can be found in 5349 cond1 peaks, not in 2610 cond1, then it may be a gain-of-function in condition 1; if a peak in the 1428 enriched_in_condition1 list can be found in both 5349 cond1 peaks and in 2610 cond1, then it may be an increase of enrichment in condition 1. And etc.

Thanks @taoliu . So you are saying to identify the loss of regions during individual calls, I can overlap with the bdgdiff outcomes? And whatever calls are released by bdgdiff are different from the individual calls and are based on likelihood ratios? Any more info? Thanks again!

@gk7279 Don't understand what is the definition of your 'loss of regions'. About your second question, if any called region from bdgdiff does not exist in individual "callpeak" outcomes, that may be caused by cutoff differences -- first the likelihood ratios (LLR) used in bdgdiff vs the Poisson p/q-value used in callpeak; secondly the actual value of the cutoff, e.g. LLR 100 vs 10, or q-value 0.05 vs q-value 0.01 and etc. More information on bdgdiff calls: enriched_in_condition1 calls should be the regions showing stronger (in terms of LLR) cond1 IP enrichment against cond2 IP, and stronger (in terms of LLR) cond1 enrichment against cond1 control; enriched_in_condition2 calls in a similar way; common calls should be the regions showing stronger enrichment comparing cond1 IP to cond1 control, and comparing cond2 IP to cond2 control, and insignificant difference (LLR) between cond1 IP and cond2 IP.

Thank you. This was the explanation I was looking for.


Unable to render templates for job 'kubo-dns-aliases'

When I used to deploy kubo on softlayer, bosh returned error like below:
  - Unable to render jobs for instance group 'master'. Errors are:
    - Unable to render templates for job 'kubo-dns-aliases'. Errors are:
      - Error filling in template 'aliases.json.erb' (line 8: undefined method `address' for #<Bosh::Template::EvaluationLink:0x0055ae65fbb5f8>)
    - Unable to render templates for job 'kubernetes-api'. Errors are:
      - Error filling in template 'kubernetes.pem.erb' (line 1: Can't find property '["tls.kubernetes.certificate"]')
      - Error filling in template 'kubernetes-key.pem.erb' (line 1: Can't find property '["tls.kubernetes.private_key"]')
    - Unable to render templates for job 'kubeconfig'. Errors are:
      - Error filling in template 'ca.pem.erb' (line 1: Can't find property '["tls.kubernetes.ca"]')
  - Unable to render jobs for instance group 'worker'. Errors are:
    - Unable to render templates for job 'kubo-dns-aliases'. Errors are:
      - Error filling in template 'aliases.json.erb' (line 8: undefined method `address' for #<Bosh::Template::EvaluationLink:0x0055ae67da7508>)
    - Unable to render templates for job 'kubeconfig'. Errors are:
      - Error filling in template 'ca.pem.erb' (line 1: Can't find property '["tls.kubernetes.ca"]')

I know it was related with bosh-dns. but I deploy director with powerdns. So is bosh-dns must set in kubo-deployment? And how can I access manual docs about kubo-deployment on any IaaS?
Thank you.

Hi @EdwardStudy,
Bosh DNS is now a requirement for the deployment, and PowerDNS has been discontinued.
The custom installation docs can be found at https://docs-kubo.cfapps.io/installing/customizing-cfcr/

@jhvhs According to what you said, Should I replace PowerDNS by bosh-dns?
And that doc is very helpful. Thank you.

@EdwardStudy Yes, you should replace PowerDNS with Bosh-dns. Please see the bosh documentation for migration. Another option is to redeploy bosh from scratch using the instructions on CNCF user guide


Migrate write query builders to Document API v2

What this PR does:
See title.
Which issue(s) this PR fixes:
Fixes #1728 (partially)
Checklist

[x] Changes manually tested
[x] Automated Tests added/updated
[ ] Documentation added/updated
[x] CLA Signed: DataStax CLA

This is in a good state if we want to merge now.
There are a couple of things that could still be improved:

insert queries are always used in CQL batches, and delete queries all but in one place, so the builders could produce BatchQuery instances, in order to avoid converting later.
buildQuery should also produce a proto builder (e.g. BatchQuery.Builder), it's not necessary to build the proto object since we're going to rebuild anyway in bind(). But that will require changes in QueryBuilderImpl.

We can address both when migrating the service.

Rebased on latest v2.0.0, addressed comments and removed the "Prepare other classes..." commit (I've kept it on a local branch).

Guess this can be merged :rocket:


add shared config state to session

What I did
By adding this flag to the session, we force the AWS Go SDK to read the ~/.aws/config file. By default, the Go SDK doesn't read this file which is often not what we or customers expect. Many customers store their assume role based prfoiles in the .aws/config file rather than the .aws/credentials file.
This new behavior is what the AWS CLI does, by default - but that's because this parameter is enabled by default in the python SDK.
Here's a little example of the difference and how it'll fail.
(not mandatory) A picture of a cute animal, if possible in relation with what you did

This cute doggy took a nap after doing hard work. I will take a nap after this PR, as well.

This unfortunately breaks integration with aws-okta we use by Docker :'(

This change had a side effect on internal AWS setup we use by Docker (based on Okta).
Our setup relies on source profile:
[okta]
aws_saml_url = home/amazon_aws/abc/123

[profile identity.devtools.developer]
role_arn = arn:aws:iam::12345:role/devtools.developer

[profile sandbox.devtools.developer]
source_profile = identity.devtools.developer
role_arn = arn:aws:iam::6789:role/okta.sandbox

I have .aws/credentials file configured with AWS access tokens both for identity.devtools.developer and sandbox.devtools.developer profiles
With this change, I can't use the ecs command anymore:
User<EMAIL_ADDRESS>is not authorized to perform: sts:AssumeRole on resource: arn:aws:iam::12345:role/devtools.developer

@kohidave  Any thoughts ?

Something weird I noticed during a debug sessions:
AssumeRoleWithContext's input is set as expected with `RoleARN=arn:aws:iam::6789:role/okta.sandbox"
But HTTP request when logged includes:

So I think I figured out what I had done. I think its the way aws-okta does what it does. I can reproduce this error in aws-okta by itself if I set:
export AWS_SDK_LOAD_CONFIG=1
Enter passphrase to unlock /home/metcalfc/.aws-okta/:
panic: SharedConfigAssumeRoleError: failed to load assume role for arn:aws:iam::175142243308:role/okta.sandbox, source profile identity.devtools.developer has no shared credentials

goroutine 1 [running]:
github.com/aws/aws-sdk-go/aws/session.Must(...)
        /go/src/github.com/segmentio/aws-okta/vendor/github.com/aws/aws-sdk-go/aws/session/session.go:326
github.com/segmentio/aws-okta/lib.(*OktaClient).AuthenticateProfile3(0xc00014a000, 0xc00025814b, 0x31, 0x34630b8a000, 0xc0000cf769, 0x9, 0x0, 0x0, 0x0, 0x0, ...)
        /go/src/github.com/segmentio/aws-okta/lib/okta.go:269 +0xd61
github.com/segmentio/aws-okta/lib.(*OktaProvider).Retrieve(0xc0002b7718, 0x0, 0x0, 0x0, 0x0, 0xc000320001, 0x273, 0xc000270030, 0x27)
        /go/src/github.com/segmentio/aws-okta/lib/okta.go:672 +0x4c2
github.com/segmentio/aws-okta/lib.(*Provider).getSamlSessionCreds(0xc00014ac30, 0x0, 0x0, 0x0, 0x0, 0xc000320000, 0x412188)
        /go/src/github.com/segmentio/aws-okta/lib/provider.go:261 +0x389
github.com/segmentio/aws-okta/lib.(*Provider).Retrieve(0xc00014ac30, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, ...)
        /go/src/github.com/segmentio/aws-okta/lib/provider.go:133 +0x993
github.com/segmentio/aws-okta/cmd.writeToCredentialsRun(0xfb8ba0, 0xc0000c1880, 0x2, 0x2, 0x0, 0x0)
        /go/src/github.com/segmentio/aws-okta/cmd/write-to-credentials.go:114 +0x451
github.com/spf13/cobra.(*Command).execute(0xfb8ba0, 0xc0000c17e0, 0x2, 0x2, 0xfb8ba0, 0xc0000c17e0)
        /go/src/github.com/segmentio/aws-okta/vendor/github.com/spf13/cobra/command.go:647 +0x3eb
github.com/spf13/cobra.(*Command).ExecuteC(0xfb8320, 0xf, 0xa89d88, 0x1)
        /go/src/github.com/segmentio/aws-okta/vendor/github.com/spf13/cobra/command.go:726 +0x2bc
github.com/spf13/cobra.(*Command).Execute(...)

I think I was playing with my own creds tool project and I set:
export AWS_CONFIG_FILE=/dev/null

To test something. So it all worked for me. So @ndeloof if you do:
aws-okta env
export AWS_CONFIG_FILE=/dev/null

Then our tool flow works. So I think there is something funky about how our aws-okta roles / config is setup.

Howdy ya'll! So sorry for the hickups, hopefully we can get this all sorted out. I have a couple of questions that may be you can help me out with.
What happens if you try to use the AWS CLI with one of these profiles? Do you see the same behavior?
The role_arn field in the profile has a meaning in the config file - does Okta have a different interpretation of this field?
I was briefly looking at okta's SSO integration - is that what ya'll are using? (I'm guessing not, but maybe you can share some reference for how the okta tooling uses the config/credential files?
Thanks a bunch!

I get mixed behavior.
If I use the aws-okta tool to set the env variables only, the aws cli (I used aws ec2 describe-instances) works. If I use a profile with a credential helper, it works. If I get a fresh environment and just use the aws-okta tool to write ~/.aws/credentials it does not work. The error is:
Unable to locate credentials. You can configure credentials by running "aws configure".

My ~/.aws/config looks like:
[okta]
aws_saml_url = home/amazon_aws/abcdefghijklmnopqrst/123

[profile identity.devtools.developer]
role_arn = arn:aws:iam::111111111111:role/devtools.developer
region = us-west-2

[profile sandbox.devtools.developer]
source_profile = identity.devtools.developer
role_arn = arn:aws:iam::222222222222:role/okta.sandbox
region = us-west-2

[profile sandbox]
credential_process = aws-okta cred-process sandbox.devtools.developer
region = us-west-2

The credential_process profile is a hassle because its going to need a password every time and the prompt gets swallowed by the tool.

we're not using AWS SSO, but https://github.com/segmentio/aws-okta with 2FA token to generate credentials.
The AWS CLI works without issue for me with this exact same okta-generated config and credentials as long as I pass --profile sandbox.devtools.developer to select the adequate credentials. Makes me thing there's some subtle difference with the aws-go-sdk or the way we configure it to parse config files.

Running aws CLI (I have no AWS* environment variable set):
➜ aws-okta write-to-credentials identity.devtools.developer ~/.aws/credentials
➜ aws-okta write-to-credentials sandbox.devtools.developer ~/.aws/credentials
➜ AWS_PROFILE=sandbox.devtools.developer aws ecs list-account-settings
{
    "settings": [
        {
            "principalArn": "arn:aws:iam::...:role/okta.sandbox",
            "name": "containerInstanceLongArnFormat",
            "value": "enabled"
        },
...

we try to run the same from ecs-plugin using go SDK, and the Sign phase fails with reported error (User: ... is not authorized to perform: sts:AssumeRole on resource: ...)

That's ... super weird. They should behave the same.
Could you run AWS_PROFILE=sandbox.devtools.developer aws sts get-caller-identity? Where is it getting the identity from? The credentials in the credentials file, or the assumed role? (and if so, which one?)
Thanks!

Update part one million - I spoke with the folks who own the Go AWS SDK - and they were super helpful in confirming my understanding. Unfortunately, changing the the behavior  in v1 of the SDK would be a backwards incompatible change, and with v2 just around the corner (in development), adding a new flag to the SDK to match the CLIs behavior might not be super worth it.


Pull latest EBS snapshot from tag instead of failing.

In many use cases it makes sense to have multiple snapshots under a tag. Rather than failing, Rancher EBS should pull the latest snapshot from the tag.
Tested this on Rancher v1.6.24

Closing this due to staleness. Feel free to reopen or open a new PR if there's still a relevant change to be made. Thanks!


SNYK: github.com/docker/distribution Denial of Service (DoS) #3

Summary of issue

Severity: medium
Affected versions of this package are vulnerable to Denial of Service (DoS) due to improper validation of the value passed to the n parameter in the /v2/_catalog endpoint. Exploiting this vulnerability is possible by sending a crafted malicious request to the /v2/_catalog API endpoint, which results in an allocation of a massive string array and excessive use of memory.

Link to detailed description

Fixed in https://github.com/equinor/terraform-azurerm-service-bus/pull/26


Remove staked well market from app

Confirm this is not in the registry, if so we need to rebuild appconfig to remove it from the UI.

Closed with #1626


[BUG] Very long Youtube stream's live chat stopped being recorded prematurely

Basic information

Program version: 0.1.2
Python version: 3.9.2
Operating system: Microsoft Windows 10 Pro: Version 10.0.19041 Build 19041

Describe the bug
A bit of warning beforehand, this will be difficult to reproduce because it was a bug with downloading the chat of a then live stream. Downloading the chat of the stream after it finished and was archived seems to work just fine, but this is undesirable to me because, for reasons beyond my comprehension, Youtube only chooses to archive some chat messages and not all of them in the chat replay. The only way to ensure you get every message sent on Youtube with any tool or method is to get them live.
With that out of the way, I attempted to archive the chat for Inugami Korone's recent Final Fantasy IX playthough (link: https://www.youtube.com/watch?v=Zs4E3g0rwoU). I started running chat-downloader sometime between 4 and 4:30 AM my time (I believe about 3 hours before she went live) and then went to bed. I watched some of the ongoing stream when I woke up later, and the chat was still being archived properly up until I left for work. At about 3:11 and 59 seconds PM my time (8:11:50 in the vod) chat abruptly stopped being archived with the message from chat-downloader saying "[INFO] Finished retrieving chat."
The issue is that chat was decidedly not finished being retrieved, the video continues for a little over an hour more for a total run time of 9:14:34. The point at which chat stopped being archived doesn't correspond to any technical difficulties I can see in the vod. I also have reason to believe my connection to the stream wasn't interrupted, and if in fact it was I would assume it would result in a more descriptive error from chat-downloader.
Command/Code used
If running from the command line, provide the following:

The command used (including the verbose tag, -v):
Both of these were running simultaneously in different instances of cmd

chat_downloader https://www.youtube.com/watch?v=Zs4E3g0rwoU --output plaintext\Inugami_Korone_7-10-21-Final_Fantasy_IX_Part_1.txt
chat_downloader https://www.youtube.com/watch?v=Zs4E3g0rwoU --output json\Inugami_Korone_7-10-21-Final_Fantasy_IX_Part_1.json

Output from the above command:
Files are extremely long. The are included in the attached Youtube chat archives.zip file

Inugami_Korone_7-10-21-Final_Fantasy_IX_Part_1.txt is the results of the first command
Inugami_Korone_7-10-21-Final_Fantasy_IX_Part_1.json is the results of the second command
Inugami_Korone_7-10-21-Final_Fantasy_IX_Part_1_afterstream.txt is the results of the first command when it is run again now that the stream is over. It is the only file included that should be reproducible now.
Expected behavior
The chat would continue to be archived for at least another hour until the stream ended and chat was closed afterwards.
Additional context/information
Again, to reiterate, this is an issue that occurred over the course of 8+ hours of a very long live stream on Youtube., The issue doesn't exist downloading the chat replay now after the fact. Because of this, I can't give much advice on how to reproduce it except to find another streamer on Youtube with a tendency to stream for an extremely long time. Korone does so fairly frequently if you don't have anyone else in mind, but I don't know if the issue will manifest again in another stream anyways.
Youtube chat archives.zip

Thanks for the explanation! Fortunately, there seems to be an upcoming live stream: https://www.youtube.com/watch?v=2rs5PDzkUzU, so I will run it on that. I have also been running the program on other streams, so hopefully we can find the cause of the problem.

At about 3:11 and 59 seconds PM my time (8:11:50 in the vod) chat abruptly stopped being archived with the message from chat-downloader saying "[INFO] Finished retrieving chat."

This is very useful info for debugging as this usually occurs when YouTube does not return so-called "continuation information". In general, the way to debug this is by running the command with the "verbose" flag (-v).
chat_downloader -v https://www.youtube.com/watch?v=Zs4E3g0rwoU --output plaintext\Inugami_Korone_7-10-21-Final_Fantasy_IX_Part_1.txt

This will not affect the output at all, it will just print debugging information to the console.

Program version: 0.1.2

While this is most likely not the issue, the latest version is 0.1.3, so you can update using
pip install chat-downloader --upgrade

I've noticed this issue with "chat rooms" after they've been downloading for several days, so you don't have to hunt for long-running streams. Coco's chat room is highly active and should work for a good while: https://www.youtube.com/watch?v=hFh7ww-F624
As an aside, you can query Youtube for whether a video is live/upcoming and use that to inform whether to restart the downloader. I don't know to what extent chat_downloader queries Youtube's internal API, but it may be worthwhile to add this check to avoid a premature exit.

@krichbanana Thanks for the info 👍 I'll test on that chat room.
If possible, could you run the command with the -v flag and show what happens when the program says "Finished retrieving chat"? As mentioned above, this usually means that YouTube itself does not provide any continuation information. Which is why chat downloader thinks the stream has ended.

As an aside, you can query Youtube for whether a video is live/upcoming and use that to inform whether to restart the downloader.

Depending on what the actual issue is (whether it is a problem with chat-downloader, or a problem with YouTube itself), this sounds like a pretty good idea. 👍 I do use the youtube internal player API to get the initial information, then poll the live_chat endpoint to get the chat. I have seen youtube_dl, for example, query the get_video_info endpoint, so this could also be used.

Good news! I was running the program for around 20 hours and I finally got the error:
[DEBUG] https://www.youtube.com:443 "POST /youtubei/v1/live_chat/get_live_chat?key=AIzaSyAO_FJ2SlqU8Q4STEHLGCilw_Y9_11qcW8 HTTP/1.1" 200 None
[DEBUG] Continuation parameters: {'click_tracking': {'clickTrackingParams': 'CAEQl98BIhMIguWakKfd8QIVhEHGCh10HgO3'}, 'continuation': '0ofMyAOnARpYQ2lrcUp3b1lWVU5UT1hWUlNTMXFRek5FUlRCTU5FbHdXSGwyY2paM0VndG9SbWczZDNjdFJqWXlOQm9UNnFqZHVRRU5DZ3RvUm1nM2QzY3RSall5TkNBQijgrLHykN7xAjAAQAJKGggAGAAgAEoCCAFQs-qakKfd8QJYA3gAogEAUIqw1fKQ3vECWLvB1fa91vECggECCAGIAQCgAdLnibOR3vEC'}
[DEBUG] Session headers: User-Agent, Accept-Language, origin, x-youtube-client-name, x-youtube-client-version, x-origin, x-goog-authuser, x-goog-visitor-id, content-type, referer
[DEBUG] Logged-in info: {'key': 'logged_in',
'value': '0'}
[DEBUG] Total number of messages: 7627
[DEBUG] Continuation info: {'invalidationId': {'objectSource': 1056, 'objectId': 'Y2hhdH5oRmg3d3ctRjYyNH41NDIwMzc4', 'topic': 'chat~hFh7ww-F624~5420378', 'subscribeToGcmTopics': True, 'protoCreationTimestampMs': '1626113656942'}, 'timeoutMs': 10000, 'continuation': '0ofMyAOnARpYQ2lrcUp3b1lWVU5UT1hWUlNTMXFRek5FUlRCTU5FbHdXSGwyY2paM0VndG9SbWczZDNjdFJqWXlOQm9UNnFqZHVRRU5DZ3RvUm1nM2QzY3RSall5TkNBQijgrLHykN7xAjAAQAJKGggAGAAgAEoCCAFQs-qakKfd8QJYA3gAogEAUIqw1fKQ3vECWLvB1fa91vECggECCAGIAQCgAdXgi7eR3vEC'}
[DEBUG] Sleeping for 8000ms.
[DEBUG] https://www.youtube.com:443 "POST /youtubei/v1/live_chat/get_live_chat?key=AIzaSyAO_FJ2SlqU8Q4STEHLGCilw_Y9_11qcW8 HTTP/1.1" 200 None
[DEBUG] Continuation parameters: {'click_tracking': {'clickTrackingParams': 'CAEQl98BIhMIguWakKfd8QIVhEHGCh10HgO3'}, 'continuation': '0ofMyAOnARpYQ2lrcUp3b1lWVU5UT1hWUlNTMXFRek5FUlRCTU5FbHdXSGwyY2paM0VndG9SbWczZDNjdFJqWXlOQm9UNnFqZHVRRU5DZ3RvUm1nM2QzY3RSall5TkNBQijgrLHykN7xAjAAQAJKGggAGAAgAEoCCAFQs-qakKfd8QJYA3gAogEAUIqw1fKQ3vECWLvB1fa91vECggECCAGIAQCgAdXgi7eR3vEC'}
[DEBUG] Session headers: User-Agent, Accept-Language, origin, x-youtube-client-name, x-youtube-client-version, x-origin, x-goog-authuser, x-goog-visitor-id, content-type, referer
[DEBUG] Logged-in info: {'key': 'logged_in',
'value': '0'}
2021-07-12 20:14:17 | Quadra Shishiron: yo
[DEBUG] Total number of messages: 7628
[DEBUG] Continuation info: {'invalidationId': {'objectSource': 1056, 'objectId': 'Y2hhdH5oRmg3d3ctRjYyNH41NDIwMzc5', 'topic': 'chat~hFh7ww-F624~5420379', 'subscribeToGcmTopics': True, 'protoCreationTimestampMs': '1626113665382'}, 'timeoutMs': 10000, 'continuation': '0ofMyAOnARpYQ2lrcUp3b1lWVU5UT1hWUlNTMXFRek5FUlRCTU5FbHdXSGwyY2paM0VndG9SbWczZDNjdFJqWXlOQm9UNnFqZHVRRU5DZ3RvUm1nM2QzY3RSall5TkNBQijqlb-3kd7xAjAAQAJKGggAGAAgAEoCCAFQs-qakKfd8QJYA3gAogEAUNSc9reR3vECWLvB1fa91vECggECCAGIAQCgAY_wjruR3vEC'}
[DEBUG] Sleeping for 8000ms.
[DEBUG] https://www.youtube.com:443 "POST /youtubei/v1/live_chat/get_live_chat?key=AIzaSyAO_FJ2SlqU8Q4STEHLGCilw_Y9_11qcW8 HTTP/1.1" 503 None
[DEBUG] Continuation parameters: {'click_tracking': {'clickTrackingParams': 'CAEQl98BIhMIguWakKfd8QIVhEHGCh10HgO3'}, 'continuation': '0ofMyAOnARpYQ2lrcUp3b1lWVU5UT1hWUlNTMXFRek5FUlRCTU5FbHdXSGwyY2paM0VndG9SbWczZDNjdFJqWXlOQm9UNnFqZHVRRU5DZ3RvUm1nM2QzY3RSall5TkNBQijqlb-3kd7xAjAAQAJKGggAGAAgAEoCCAFQs-qakKfd8QJYA3gAogEAUNSc9reR3vECWLvB1fa91vECggECCAGIAQCgAY_wjruR3vEC'}
[DEBUG] Session headers: User-Agent, Accept-Language, origin, x-youtube-client-name, x-youtube-client-version, x-origin, x-goog-authuser, x-goog-visitor-id, content-type, referer
[DEBUG] Logged-in info: None
[DEBUG] No continuation information found: {'error': {'code': 503, 'message': 'The service is currently unavailable.', 'errors': [{'message': 'The service is currently unavailable.', 'domain': 'global', 'reason': 'backendError'}], 'status': 'UNAVAILABLE'}}
[INFO] Finished retrieving chat.
[DEBUG] Session closed.

The important line is:
[DEBUG] No continuation information found: {'error': {'code': 503, 'message': 'The service is currently unavailable.', 'errors': [{'message': 'The service is currently unavailable.', 'domain': 'global', 'reason': 'backendError'}], 'status': 'UNAVAILABLE'}}

503 is the HTTP code for "Service Unavailable", which basically meant YouTube had a backend error while trying to handle the request. Fortunately, this looks like a really simple thing to fix: just try again if YouTube fails.
I will implement this in the next update!
Strangely enough, I was running the program for around 10 streams simultaneously, and the first program to fail (https://www.youtube.com/watch?v=hFh7ww-F624) only happened after around 20 hours. So, the probability of it happening seems to be very low. That being said, it of course is better to handle these situations.

Fixed in the latest version👍 (v.0.1.4)
Let me know if you have any other questions


Confused about the logic in edit_model.py

Hi, thank you for the great work! I've been using your library and it has been really helpful. While working with it, I came across something I wasn't sure about.
Are the obtained new_emb and old_emb corresponding to the [pad] part?
https://github.com/rohitgandikota/unified-concept-editing/blob/b66412e9c89f0f1b5aceed626aa25bf7c0bd5631/train-scripts/train_erase.py#L253C1-L260C104

They correspond to the final token and the some of the padding tokens (how much ever to match both the lengths). For example: in the figure below, if the top embeddings representing old_emb (concept embeddings) has 4 tokens and the bottom one representing new_emb (or target embeddings) has only 2, we take only enough paddings for new so that the length matches with old. Hope that made sense!

Thank you so much for the detailed explanation! I appreciate your prompt response and help on this matter.
I'm not deeply familiar with NLP. I have another question though: Why not use the first 4 tokens?

Then you might be replacing the last 2 tokens with the padding - this will be a problem when u use these tokens in a long sentence. Next time the model sees a prompt with last 2 tokens, it will discard all the other tokens that follows them as it things that could be the ending token.

Hi, could you explain a bit about the intuition for keeping the original  embedding unchanged?

We are altering the weights such that the layer outputs differently for the embeddings that we wish to edit and same for the embeddings we dont

Thanks for your reply! Sorry that I didn't make my question clear.
I realize that your method involves altering the weights, but I'm a bit confused about why the mapping starts from final_token_idx and final_token_idx_new here. It seems that this generally disregards the embedding of the start-of-text token. Based on the figure you provided, it appears that only the final embeddings of the text and padding elements are being considered.
Would you mind clarifying this a bit? Thanks!

When the concept has multiple tokens, if we consider the final embedding, then we would be changing the concept not the individual tokens. Consider "Van Gogh" for example, if we change the embedding from the last token, we will not be altering "Van" (which can mean other things). Also since text embedders are sequential processors, the "Gogh" token embedding has information of "van". We are not really changing "Gogh", but we are in fact changing the "Van Gogh" together. Hope that made sense

Quite clear! Thanks :)

When the concept has multiple tokens, if we consider the final embedding, then we would be changing the concept not the individual tokens. Consider "Van Gogh" for example, if we change the embedding from the last token, we will not be altering "Van" (which can mean other things). Also since text embedders are sequential processors, the "Gogh" token embedding has information of "van". We are not really changing "Gogh", but we are in fact changing the "Van Gogh" together. Hope that made sense


I do not believe that skydive and the system can safely converge in only 200ms.


VoxelEmbeddedMultiMesh

VoxelMultiMesh with the addition of a embedded "VoxelSet", self managed and updated, so as to reduce storage.

rework is making VoxelSets mandatory, so in a way all VoxelObjects will have a "embedded" VoxelSet


chore: converted new-components to typescript

Added ts-loader, babel-loader as dev dependencies to grapiql project to allow storybook to compile and use typescript files.
Added typings for theme-ui
Converted new components to typescript.

LGTM! thanks


Support Scala3

Remove Scanamo module as it is blocking Scala3 support
Made some deprecated classes private

The logs appear to have expired, so I'm unable to see the error anymore, but here's the error I'm getting with commit 1c2cdb652bd9b5dbcedd71c9917bce8dd13bc11d:
meteor-pr301-fail.txt

Scala v3.3.1 was released on 2023-09-08.
Any chance of someone looking at this issue again?
Please also check out https://github.com/d2a4u/meteor/pull/343.

Thanks for getting back @d2a4u ! 🙇
I can try to review, but I'm not familiar with the codebase from before, and as a result may end up asking lots of questions.
Based on a quick look, many of the changes appear to be Scala 2 compatible e.g. trivial syntax changes or Meteor internal API refactoring. Would it be possible to extract these changes to a PR separate from Scala 2 incompatible changes needed to make the leap to Scala 3? 🤔 Another way would be to split into Scala 2 compatible internal API refactorings and the rest. This would make it easier to reason about the review the changes.
Would the resulting Scala 3 libraries be compatible with Scala versions >= 3.0 and < 3.3?

@d2a4u Sorry, for the delay!
Finally got around doing the review.
The PR looks good to me, so I approved, but left some minor comments.
Thanks again for your work on Meteor in general as well as adding Scala 3 support! 🥇 🙇

Hi again! 👋 How are things looking with finalising this PR @d2a4u ? 😄 Are there some impediments?

Hi again! 👋 How are things looking with finalising this PR @d2a4u ? 😄 Are there some impediments?

Sorry for the delay, I had some issues with sonatype which now have been resolved.

Great to see a new version of meteor released with Scala 3 support! 🥳 Thanks @d2a4u ! 🙇


How to edit and delete any post from Google My Business

Kindly provide examples of edit and delete the post from Google My Business

what's wrong in below code
`
$name_post = "accounts/xxxx/locations/xxxx/localPosts/xxxx";
$post_body = new \Google_Service_MyBusiness_LocalPost;
$post_body->setLanguageCode('en');
$post_body->setSummary('test_new'); // post text
$call = new \Google_Service_MyBusiness_CallToAction;
$call->setActionType('LEARN_MORE'); // call to action type, see https://developers.google.com/my-business/content/posts-data#call_to_action_posts
$call->setUrl('https://www.xyz.net/'); // URL to link to
$post_body->setCallToAction($call);
$media = new \Google_Service_MyBusiness_MediaItem;
$media->setMediaFormat('PHOTO');
$media->setSourceUrl('https://www.xyz.net/modules/images/home.jpg');
$post_body->setMedia($media);
try {
$resp = $mybusinessService->accounts_locations_localPosts->patch($name_post, $post_body); // create post for the location
var_dump($resp);
die;

}
catch (\Google_Service_Exception $e) {
print_r($e);
}
`
getting error message
"error": {     "code": 400,     "message": "Request contains an invalid argument.",     "errors": [       {         "message": "Request contains an invalid argument.",         "domain": "global",         "reason": "badRequest"       }     ],     "status": "INVALID_ARGUMENT"   }

Has anyone addressed this?  I can't for the life of me figure out how to update summaries and store hours.


fix: made the in range condition work with a property again

Description
In range condition did not use the property for completing, which is fixed by this PR.

It should actually take of the reference but still be broken due to missing property, which I could autofix, if wanted. But as far as I have tested it, the validation system will also warn here.

It should actually take of the reference but still be broken due to missing property, which I could autofix, if wanted. But as far as I have tested it, the validation system will also warn here.

That is exactly what I meant, old training courses will require extra work (at least to use the "fix" button) to continue working.
Also, you are not considering that all users might not have creator pro with the validation system, there could be the case where an innocent designer updates their creator, and boom!
Now the course does not work!

due to this line:
property = Data.DistanceDetector.Value.GameObject.GetOrAddComponent();
it should actually still work 🤔

due to this line:
property = Data.DistanceDetector.Value.GameObject.GetOrAddComponent<TransformInRangeDetectorProperty>();
it should actually still work 🤔

I don't mean in the code, I mean that with this update, old courses need manual rework, look:

The reference object will be lost and courses will be broken.
Btw, I created a course that only has a transition with an Object Nerbay condition, then I updated it to this branch and try to run the course without doing any "upgrade" to the course.

@Gusinuhe Reworked it to properly update without errors.

/azp run

:tada: This PR is included in version 1.6.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


fzf binary for directories

Category

[x] fzf binary
[ ] fzf-tmux script
[ ] Key bindings
[ ] Completion
[ ] Vim
[ ] Neovim
[ ] Etc.

OS

[x] Linux
[ ] Mac OS X
[ ] Windows
[ ] Windows Subsystem for Linux
[ ] Etc.

Shell

[ ] bash
[x] zsh
[ ] fish

Is there a way to call the fzf binary with an option to search for directories only?
I use fzf in my emacs setup and I would like to have a command to open directories with dired.

fzf is just an interactive filter, it's up to you to feed a list of directories to fzf. Please read usage section of the documentation: https://github.com/junegunn/fzf#usage


For discussion - Add config class

Something like this could work regarding ssh_connection in #25.
Could also ship a SimpleYaml config plugin similar to the SimpleInventory one.
Currently one would access the variables with task.brigade.config.ssh_config_file which seems a bit long, so perhaps some shortcut is needed.

Could also ship a SimpleYaml config plugin similar to the SimpleInventory one.

I think it should be provided by default tbh by the config class itself, it's simple enough that using "plugins" for that seems a bit too much. You can always provide your own configuration class if you have special needs.

Makes sense to me.
Regarding order I think I prefer environment variables before config file.
If we add yaml support to the default config class we need to send in the file as a parameter. Based on that I would change it so that the Brigade init looks like this instead:
    def __init__(self, inventory, dry_run, config=None):
        self.config = Config(config)

In that case I don't think there's an immediate need to use a custom config class. If needed it could perhaps be changed to something like:
    def __init__(self, inventory, dry_run, config=None):
        if isinstance(config, str):
            self.config = Config(config)
        else:
            self.config = config or Config()

The alternative would be to have both config and config_file as arguments. Or to always load brigade.yaml from the current directory or something like /etc/brigade/config.yaml, which I don't really like. Thoughts?
Thinking of storing defaults in a dictionary or a yaml file we include with Brigade. Have to figure out a good way to document the defaults. Probably just have Sphinx read that dict or yaml file and create some table.

I don't think there's an immediate need to use a custom config class

I agree, I am happy providing load_json and load_yaml and load_toml and load_whatever functions there. As long as the data structure in those file is consistent it should be pretty straightforward.

The alternative would be to have both config and config_file as arguments

I personally prefer this one.

I was thinking of something like this. After _assign_properties() is run we can trigger something to set all of the ssh parameters.
Considering of the CONF variable should be moved to an external .py or perhaps .yaml file.

I think this is ready for review now. Not really happy about the documentation of the parameters of the Config class, it will work from the website but not as much from the built in help of Python, perhaps we can just provide a few examples there.
The next step would be to have the Config load things such as ssh_config files, don't know if that should be done by default though or if it should be done once the first ssh connection is done.
Also we can thing of what other parameters should be added like ssh_system_config_file.

One thing I realised with this is that the Host object wouldn't currently have access to Brigade.config from the current ssh_connection(). That would also have to be addressed in some way.

Yeah, I think we will have to add a brigade reference to the host so the host can access it. Right now tasks can but it's probably a good idea to do the same in the host as well.

Fixed the assignments and added a BASEPATH to the Sphinx conf since the tests failed when it couldn't find the config jinja template.


VSCode changing Mode from Insert to Normal

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

VS Code Version: 1.84.0
OS Version: Darwin x64 22.6.0

Steps to Reproduce:

Pressing CMD + C on Mac Ventura 13.6.1 changes the mode from --INSERT-- to --NORMAL-- on VSCode Version: 1.84.0 (Universal)
When you press letter I, the mode changes from --NORMAL-- back to --INSERT--

Additional Context:

VIM came with the MacOS version and cannot be uninstalled.
Research showed that if VIM was installed manually, it could be removed and the issue wont persist again.
This issue causes the user of VSCode not to be able to type as he/she wishes
How can this be fixed as quickly as possible?

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.84.2. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

VIM came with the MacOS version and cannot be uninstalled.

I don't think we ever pre-install the Vim extension for VS Code. Where are you getting VS Code from that's it comes with VIm?

Hi, I downloaded VS Code from the official VSCode store.
using this link - https://code.visualstudio.com
What I am saying seems not be clear to you.
I am saying that VIM came with the Operating System - MacOS.
I have made research on how to uninstall it, however, I am yet to get a way to do that.
Is there any way around this issue at the moment?

This sounds like an issue with VIM or with MacOS, not with VS Code.


Cannot install Spyder with pixi (but with miniforge). Constrained to old (1.4) version of python.app?

Checks

[X] I have checked that this issue has not already been reported.

[X] I have confirmed this bug exists on the latest version of pixi, using pixi --version.

Reproducible example
Please check out https://github.com/spyder-ide/spyder/issues/22386 for a detailed description.
I wasn't sure if this is a pixi, spyder or conda-forge bug, so I filed it with spyder-ide first.
However, it turns out I can install spyder fine using miniforge. As pixi and miniforge should use the same conda-forge packages as far as I understand, I thought it might be a pixi problem and thus should be reported here. Or if not, perhaps somebody from the pixi team has the experience to quickly point out what is going on.
Issue description
Executive summary:

install spyder via pixi results in spyder not being executable . The error reads:

❯ pixi run spyder
/Users/fangohr/spyder-pixi-issue/.pixi/envs/default/bin/pythonw: line 3: /Users/fangohr/spyder-pixi-issue/.pixi/envs/default/python.app/Contents/MacOS/python: No such file or directory

A possible cause of the problem is the version of the python.app package that is installed: pixi uses version 1.4 (which probably has a bug) and miniforge uses version 3. Please see https://github.com/spyder-ide/spyder/issues/22386#issuecomment-2352215111
pixi refuses to allow version 3: see https://github.com/spyder-ide/spyder/issues/22386#issuecomment-2352220129

Expected behavior
Spyder should start.

Is there a conflict if you try to force the newer python.app version using constraints? E.g. pixi add "python.app>=3"? It would be interesting to find out if there is another package in your environment that depends on an older version of python.app (and hence is the reason why it's being held back).
Maybe you could also study the lock file to find all packages that depend on python.app.

Thanks for getting back to this. While I didn't say it, the above command is nearly all to reproduce. (It seems I am not getting to the pixi.lock file.) Full example:
/tmp via 🐍 v3.12.4
❯ mkdir issue2053

/tmp via 🐍 v3.12.4
❯ cd issue2053

/tmp/issue2053
❯ pixi init
✔ Created /private/tmp/issue2053/pixi.toml

/tmp/issue2053
❯ pixi add "python.app>=3"
  × failed to solve the conda requirements of 'default' 'osx-arm64'
  ╰─▶ Cannot solve the request because of: No candidates were found for
      python.app >=3.

/tmp/issue2053
❯ ls -l
total 8
-rw-r--r--  1 fangohr  wheel  232 Sep 16 10:13 pixi.toml

/tmp/issue2053
❯ cat pixi.toml
[project]
authors = ["Hans Fangohr<EMAIL_ADDRESS>channels = ["conda-forge"]
description = "Add a short description here"
name = "issue2053"
platforms = ["osx-arm64"]
version = "0.1.0"

[tasks]

[dependencies]

Actually the version 3 is only available from defaults. So if that is really the culprit, then it cannot work.

Closing as this issue is not part of pixi itself.

Thank you @wolfv - that was really helpful input. (miniforge tricked me into using the default channel somehow ..)


Usage question

Excited to try out this webpack loader!

How does this loader compare to workerize-loader?
Does this loader require a JS class export or can any other function be exported and used?

I've tried both and here's my learnings:
workerize-loader and comlink-loader achieve the same thing - putting code into a worker at compile time. A small difference is that workerize uses individual exported functions in __webpack_exports__, while comlink uses a class as a whole.
I chose workerize-loader over comlink-loader because:

Workerize is smaller and simpler.
Workerize is ES3, while comlink requires ES6 (Proxy, Map, WeakSet, generator) and ES7 (async functions). This means less build config and smaller bundle size (doesn't require all the polyfill) if you must support some legacy devices (Android 4.4, iOS 9)

Hi @fnlctrl, thanks for your reply. I had similar findings and also ended up going with workerize-loader.


Invalid authorization when using SelfHosting

Before you post
Make sure to search through the issue list to check if others have reported the same bug already. If you find an existing issue, make your voice heard there. Your case may provide valueable extra clues towards a solution and will increase its priority.
Describe the bug
A clear and concise description of what the bug is.
I have a wordpress site hosted with 2 bindings (pcrecommends.com, www.pcrecommends.com, both 80 and 443)
lately i noticed that one of my websites failed to get a new certificate. Turns out that the SelfHosting was failing to validate the website. 5 other subdomains work just fine.
When i finally gave in trying to fix it, i tried to serve the file from the root of the webpage (FileSystem), and it finally worked.
I really don't know what happened, since it worked the first time I created the certificate.
This could also be wordpress just being wordpress, but I tried manually creating the folders and I was able to fetch them just fine.
To Reproduce

Run the tool
go to renewals
try to renew a certificate.

Expected behavior
A certificate was renewed successfully.
Log
2021-05-11 18:12:14.319 +02:00 [INF] Please report issues at https://github.com/win-acme/win-acme
2021-05-11 18:12:18.123 +02:00 [INF] Running in mode: "Interactive, Simple"
2021-05-11 18:12:24.593 +02:00 [INF] Target generated using plugin IIS: pcrecommends.com and 1 alternatives
2021-05-11 18:12:26.219 +02:00 [INF] [pcrecommends.com] Authorizing...
2021-05-11 18:12:26.225 +02:00 [INF] [pcrecommends.com] Authorizing using http-01 validation (SelfHosting)
2021-05-11 18:12:31.569 +02:00 [ERR] [pcrecommends.com] Authorization result: invalid
2021-05-11 18:12:31.592 +02:00 [ERR] [pcrecommends.com] {
"type": "urn:ietf:params:acme:error:unauthorized",
"detail": "Invalid response from https://pcrecommends.com/.well-known/acme-challenge/sd9PL6z8zGhl9yQCr4WMQwQyBQSDkGBFd00vaPfUrZU [2606:4700:3031::ac43:c804]: "\n<html lang=\"en\">\n\n<meta charset=\"utf-8\">\n<meta http-equiv=\"X-UA-Compatible\" content=\"IE=edge\">\n<meta name="",
"status": 403
}
2021-05-11 18:12:37.034 +02:00 [ERR] Create certificate failed: [pcrecommends.com] Validation failed
2021-05-11 18:13:14.964 +02:00 [INF] No command line arguments provided
2021-05-11 18:13:15.147 +02:00 [INF] Software version <IP_ADDRESS>5 (release, trimmed, standalone, 64-bit) started
2021-05-11 18:13:15.150 +02:00 [INF] Connecting to "https://acme-v02.api.letsencrypt.org/"...
2021-05-11 18:13:16.284 +02:00 [INF] Scheduled task looks healthy
Platform:

OS: Windows Server 2019
Version: <IP_ADDRESS>5 (release, trimmed, standalone, 64-bit)

Self Hosted validation works via http port 80 by default. You are using Cloudflare to proxy your site and enforce https so http requests get redirected before they get to port 80 on your actual server.
So if you want to use http validation with cloudflare you need to also allow http (not just https) requests. I think the Cloudflare SSL option might be called 'Full' (vs 'Full (strict)').


tasks.json input does not prompt when default is set

I have a task in .vscode/tasks.json that uses an input variable. When the input variable does not have a default value set, it works as expected and prompts for a value. However, if I add a default value to the input variable, it does not prompt at all but instead runs the task using the default value.
I would expect that it should prompt you to override the default input before running the task instead of immediately executing with the default value.
{
  "version": "0.0.1",
  "tasks": [
    {
      "label": "Task",
      "type": "shell",
      "command": "echo \"${myVariable}\""
    }
  ],
  "inputs": [
    {
      "id": "myVariable",
      "type": "promptString",
      "description": "A value to echo",
      // Without this, it prompts. With this, it skips the prompt and immediately uses "default_value"
      "default": "default_value",
    }
  ]
}

Nevermind, I managed to figure it out. I did not realize that default_template_prompt would prevent it from showing unless it was set to always.

Honestly, I haven't quite been happy with the configurability of this behavior. One of the issues is that it treats parameters that set optional = true and parameters with a default value equivalently. While you did find a workaround, I think it's obvious that this is not good default behavior.
I just pushed a commit that changes how default_template_prompt = "allow" works. It should now prompt any time a non-optional parameter needs a value, which I believe is what you're looking for. The old behavior can be restored by setting default_template_prompt = "avoid".
Let me know if you have any additional feedback!


Safari can't set right image size on retina display

You can see at: http://mastershop.dsht.in picturefill.js gets actual image size. I try to set size="200" but it doesn't help.

Hey @abozhinov, couple of questions, and some things I'm noticing.

I'm assuming that the image you're referring to is the main image with the macbook?
The sizes attribute is meant to represent how large the image will be displayed on your page. Your sizes attribute doesn't specify a unit of measurement. 200 doesn't mean a lot to the browser. If you mean px, it would be apt to change it to 200px.
Just to be sure, your media queries (min-width: 400px, and min-width: 100px) are the size of the viewport, correct? The media attribute works just like CSS media queries.

Unfortunately I don't have a retina screen anymore to test on, but in the meantime, answering those questions or providing a little more context will help.
Thanks!

Additionally to everything mike-engel already said. sizes only makes sense if you use width descriptors (w) you use density descriptors (x). So you don't need / can use sizes at all. You must use the media attribute to control everything.

I set size attr in PX, but it doesn't help.

I use media attribute to control which image to show but the problem is in the size at Safari.

To me this looks like a styling issue. Have you tried:
img {
    width: 100%;
    height: auto;
    /*max-width: 100%;*
}

No, the problem is that safari ready img tag  and in this image tag picturefill add src @2x image. The size of this image is twice of the normal one and for that reasone safari print it with actual size.
I should write css rules to fix that but it's not so good solution.

It is actually a good solution to write this with CSS. But you can try the following plugin:
https://github.com/scottjehl/picturefill/tree/master/dist/plugins/intrinsic-dimension

Find solution without picturefill

You should rewrite your sizes to something like this: sizes="(min-width: 770px) 200px, 100px"

This won't work cross browser:
You are testing this in Safari 9 right?

Any browser that doesn't support sizes but runs on a retina will have your initial error. So your problem still happens in Safari 8, IE10/IE11 and older Firefox.
As I said: Using CSS is a good solution to style things in general and it is also a good solution to style the width of a responsive image.
But if you really want to have the same responsive sizing feature responsive images have, use the intrinsic ratio plugin. It's 1.6kb minified and below 0.9kb compressed.

I had the same issue on Safari 7 and using that intrinsic plugin did the trick. It seems like something that should work out of the box, though.

@stewx
You are the third dev who wants this feature. (see also: https://github.com/scottjehl/picturefill/issues/594)
About one year ago we had a lot of issues, where people saw this as a bug in native implementations (see a recent issue here: https://github.com/scottjehl/picturefill/issues/575).
My problem with having this in core is pretty simple, that I want to have picturefill lightweight. We already have two modules included which should be optional in my eyes (webp detection and FF respimg workaround), but we have them for backwards compatibility and it's hard to change on this.
At the same time, there are two other "plugins" (image type detection and mutation observer), which also absolutely make sense to include to make picturefill a "full polyfill". Obviously this would go against picturefill being as lightweight as possible and to only include what 80% of all developers need.  Maybe a better possibility would be to  create a download builder, that makes it easy to add/remove plugins. This way also the current plugins are more upfront and people realize that there is more to it than just one core file? And maybe we can add an option to deselect the two plugins I spoke about above.

I like the download builder idea--similar to what jQuery UI does.
If it's a bug in native implementations, how are you supposed to specify what size the rendered image should be on the page? Just with CSS?

If it's a bug in native implementations, how are you supposed to specify what size the rendered image should be on the page? Just with CSS?

It's not a bug in native implementations. Only a lot people thought it is. And yes, currently we assume that 80% of all developers specify at least one dimension with css (mostly width). If not you need to use the intrinsic sizing plugin.


yum update changed mode of /root ?

I don't know which update and can't even prove that this was done by yum update
but that's my only candidate so far.  Unfortunately this was my first update in a long time
(close to a year) so there were a lot of packages to upgrade and I have no idea which one
did it.
I did see some warnings but nothing about changing /root.
I hope there's no NEED to change the mode of root and of course that made certain
things not work for me until I figured out the problem and fixed it.
I now worry what else might have been "fixed" for me that will cause things to break.

Oracle Linux customers, please file your issue at https://support.oracle.com
Thanks for filing an issue with Oracle Linux.
GitHub Issues is not an official support channel and we don't offer
product support here. If you're not yet an Oracle Linux customer,
consider signing up at https://linux.oracle.com.
Even if you're not a customer, if we can confirm that an issue is a
bug we will do our best to fix it and to update this issue
once it has been fixed. We don't guarantee a fix or feedback and
for now, we will close this issue. If you have Oracle Linux support,
please use support.oracle.com to report issues.

Unfortunately, I can't open a bug as there is not enough to go on.  I don't know what OS version you are running, what versions you updated to, why you suspect yum, or even what the changes were to root.

Jeffery Yoder writes:

Unfortunately, I can't open a bug as there is not enough to go on.  I don't know what OS version you are running, what versions you updated to, why you suspect yum, or even what the changes were to root.

oracle linux 9.  I've now managed (with difficulty) to login to support.oracle.com
and I don't see how to submit a bug report.
Give me a hint how to do that.

--
Reply to this email directly or view it on GitHub:
https://github.com/oracle/oracle-linux/issues/164#issuecomment-2495536802
You are receiving this because you authored the thread.
Message ID: @.***>

If you're not an Oracle Customer you won't be able to log a Service Request.  Here is the procedure just in case: https://www.oracle.com/assets/support-process-guide-cloud-4428288.pdf.
I will open a bug for this and wait to see if anyone else reports this.

I've opened SR=4-0000139788
I guess that's how you're supposed to report bugs?
(Seems strange to me...)
Jeffery Yoder writes:

If you're not an Oracle Customer you won't be able to log a Service Request.  Here is the procedure just in case: https://www.oracle.com/assets/support-process-guide-cloud-4428288.pdf.
I will open a bug for this and wait to see if anyone else reports this.
--
Reply to this email directly or view it on GitHub:
https://github.com/oracle/oracle-linux/issues/164#issuecomment-2498221457
You are receiving this because you authored the thread.
Message ID: @.***>


Minor figure 2 tweaks

This PR builds upon #23 and addresses the following additional reviewer comments:

Figure 2C and Supplementary Figures 3, 4: Unlike suggested by the figure legends (“ratio of the count over the true class label”), the red color gradient does not seem to be “row-wise” (i.e., the ratios of each row/true class sum up to 1), but “column-wise” (i.e., the ratios of each column/predicted class sum up to 1). Please clarify the figure legends and/or adapt the figures accordingly. Further, please add a similar statement to the legend of Figure 2C.

Main text, fifth paragraph: “When holding out individual treatments, we observed high performance for predicting genotoxin and redox injuries, but poor performance for saponin and kinase” - this does not seem to match the confusion matrix in Supplementary Figure 4, according to which, for example, Redox samples are predicted to belong to the “Redox” class equally as often as to the “Mitochondria” class.

Figure 2 updates

Updated panel D axis labels and title to more clearly describe what we're plotting
Updated the legend title in Panel C to more clearly describe what the color represents
Updated the color to represent rowwise ratios
Minor tweaks to sizing to accommodate new labels

Supplementary Figure 3 updates

Updated the legend title to more clearly describe what the color represents
Updated the color to represent rowwise ratios

Supplementary Figure 4 updates

Updated the legend title to more clearly describe what the color represents
Updated the color to represent rowwise ratios
Added purple boxes to clearly show diagonal predictions
Added zeros to more clearly align confusion matrix expectations

thanks for the review @axiomcura - squash-merging now


frequent MaxListenersExceededWarnings when running test suite

TypeScript Version:  master

Search Terms:  MaxListenersExceededWarning, EventEmitter leak, leak
Code n/a
Expected behavior:
Running gulp runtests-parallel does not result in leak warnings
Actual behavior:  Many leaks, e.g.
[Worker 0] (node:98823) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 unhandledRejection listeners added to [process]. Use emitter.setMaxListeners() to increase limit

[Worker 1] (node:98824) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 uncaughtException listeners added to [process]. Use emitter.setMaxListeners() to increase limit

Playground Link: n/a
Related Issues: mochajs/mocha#4329, PR #41403
This may be a duplicate, but I have created it for #41403.

I get this all the time on Codespaces containers...


[CLI] 1.6 rancher cli will still pull from cli.json even if environmental variables are set, breaking some calls

Rancher versions:
rancher/server or rancher/rancher: 1.6.17
rancher/agent or rancher/rancher-agent: 0.6.9
Steps to Reproduce:
use rancher config to generate cli.json file, use environmental variables for other environment. rancher catalog ls <using environment credentials for a different deployment> will fail with error forbidden.

I have been unable to reproduce.
With the release of Rancher 2.0, development on v1.6 is only limited to critical bug fixes and security patches.


predefining SERVER_NAME doesn't allow for flask serving multiple domains

Hi there,
When defining SERVER_NAME in the app config, a limitation is introduced if your same flask app serves multiple domains.
Would it be possible to use the request.host element in the generator instead to generate the URLs when the sitemap is called?
I tried to get it done myself, but I have to admit I'm getting lost here.

Hi, good idea, I will look into this

It seems this limitation is removed by using with self.app.test_request_context(): instead of with self.app.app_context(): in line 65 of sitemapper.py
I will do some tests and create a new release with the changes. Thank you!

Hi, thanks for the feedback and code proposal.
Have you found that the current version using self.app.test_request_context() to generate the URLs is still not properly supporting apps with multiple domains?
I will look into this and work on your suggestion over the next few days

Thanks,
I have created a branch for this issue and have made changes based on your suggestion in that branch. I have used a slightly different approach than your proposal in order to keep the lastmod, changefreq, and priority arguments working and include recent commits.
The URLs are now stores as URL objects rather than dicts, with methods that generate their URL loc and XML during sitemap generation. The changes are in sitemapper.py and templates.py
Does the version on this branch work in your use case now?

These changes will be merged into the main branch and included in the next release. Thank you!


fix: expose OutOfBandEvents

Signed-off-by: Moriarty<EMAIL_ADDRESS>closes #1150

Codecov Report

Merging #1151 (f0166f0) into main (9f10da8) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #1151   +/-   ##
=======================================
  Coverage   88.32%   88.32%
=======================================
  Files         706      706
  Lines       16430    16431    +1
  Branches     2664     2664
=======================================
+ Hits        14512    14513    +1
  Misses       1911     1911
  Partials        7        7

Impacted Files
Coverage Δ

packages/core/src/modules/oob/domain/index.ts
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Add support to set imagePullPolicy attribute on runtime images

Describe the issue
KFP has introduced the ability to set imagePullPolicy attribute for a given pipeline and/or node. By default, the Image pull policy can be one of Always, Never, IfNotPresent. Defaults to Always if :latest tag is specified, or IfNotPresent otherwise. Cannot be updated. More info: https://kubernetes.io/docs/concepts/containers/images#updating-images
This is useful due to the recent rate limitations on the docker.io repository.

In addition to being able to set the image pull policy, we will need to have the ability to add image registry credentials to the metadata. If the user is creating or debugging a runtime image in the pipeline, they will need to have the image pull policy set to always to get the latest and trigger rate limitations (dockerhub).

see #1234 as related