Adding normalizer check and accelerating distribution clustering

Closes #235 and speeds up clustering by a factor of ~20 by jumping peak values rather than doing comparisons for every value in the array

Agreed, just pushed different wording. Let me know how that looks


Unable to run example app

I have installed the plugin as specified on a new project on Android Studio and copied over the example app to my main.dart. Unfortunately I am getting the error below:
The name BluetoothState isn't a type so it cant be used as a type argument
at these lines:
  final StreamController<BluetoothState> streamController = StreamController();
 StreamSubscription<BluetoothState> _streamBluetooth;

Hi @santhoshraje
I'm using this library on my daily project, didn't found any issue regarding BluetoothState. Can you please send me your example project, so I can help you with it.
Thanks.

Same here!
I just created a new flutter project, added this dependency and If I try to run the example it, it shows the same error.

@guiassemany I think I did the same thing as you. I created a new flutter project, installed the dependencies and copied the code from the example main.dart to my own main.dart and got errors.
But when I download the zip file and run the provided example application, it works flawlessly. I am currently developing using the example application that was provided.
@alann-maulana unfortunately I deleted the example project, but it was basically just a new project with the dependencies and the code from main.dart.

Within master branch I do a lot of improvements and some breaking changes that doesn't publish it yet to the pub server. Can you change you dependencies if this is work for you?
dependencies:
    flutter_beacon:
        git: https://github.com/alann-maulana/flutter_beacon


Can the top layer be a triangle?

I believe you're looking for chart.bottomWidth = 0

Correct, provided you also invert the funnel.


Add webmock and add full integration specs

Includes #11 pls merge this first
First, this adds webmock to stub the WSDL lookup & speed up test.
Second, create_shipment_dd_spec.rb got updated to assert a full savon request, to ensure future updates wont break the soap request.
If you like this, I'll update all other specs too..

thx!


type_keys() does not properly sanitize string parameters

The type_keys() method does not properly sanitize strings passed to it.
For example, the following throws a KeySequenceError:
from pywinauto.application import Application

app = Application(backend='uia').start('notepad.exe')
app['Untitled - Notepad'].Edit.type_keys('(')

Traceback:
Traceback (most recent call last):
  File "C:/Users/Nick/PycharmProjects/HobbitFarmRedux/testing.py", line 30, in <module>
    app['Untitled - Notepad'].Edit.type_keys('(')
  File "C:\Users\Nick\PycharmProjects\HobbitFarmRedux\venv\lib\site-packages\pywinauto\base_wrapper.py", line 970, in type_keys
    turn_off_numlock)
  File "C:\Users\Nick\PycharmProjects\HobbitFarmRedux\venv\lib\site-packages\pywinauto\keyboard.py", line 672, in send_keys
    keys = parse_keys(keys, with_spaces, with_tabs, with_newlines)
  File "C:\Users\Nick\PycharmProjects\HobbitFarmRedux\venv\lib\site-packages\pywinauto\keyboard.py", line 581, in parse_keys
    raise KeySequenceError('`)` not found')
pywinauto.keyboard.KeySequenceError: `)` not found

I have confirmed the following substrings will cause this behavior, no matter what else is in the string:
(
)
{
}
{}
{example text}
Additionally, when passing a substring such as (example text) the program will exit successfully, however the output will be example text not (example text)
The same is true for both default and UIA backends.

Pywinauto version: 0.6.6
Python version and bitness: 3.7 (32-bit)
Platform and OS: Windows 10 (64-bit)

The reason of that: all brackets have their own meaning to press special keys like {ENTER}. To type a single special bracket as a bracket you have to wrap it this way: .type_keys("{(}"). It will type single bracket ( correctly. This is described in the docs for keyboard module: https://pywinauto.readthedocs.io/en/latest/code/pywinauto.keyboard.html
Of course the error messages can be improved (so marking the issue as enhancement, but low priority).

Well. I guess I should have read the docs a bit more closely. Thanks!

I wrote this little sanitization loop:
password = 'ss{)747&FH%4)}'
clean_pass = ''
for c in password:
    if c in ['(', ')', '{', '}', '%']:
        clean_pass += ('{' + c + '}')
    else:
        clean_pass += c
print(clean_pass)


refactor: sortProperty -> overProperty

Rename this function, add hasOwnProperty check, so we won't add extra keys to the object

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Can I include InternalFields and Prettify the JSON output like the other tool did?

The other (old/previous/deprecated) azure-documentdb-datamigrationtool had a couple of fields for prettifying the output and including the internal fields:
/t.Prettify:true
/s.InternalFields:true
I can't seem to find anything similar in the syntax or migrationsettings.json - am I missing something, or are these options not available in this tool?

I found the answer to one of my queries: you can use the IncludeMetadataFields flag for the internalFields:
"IncludeMetadataFields": true,

The JSON extension includes an "Indented" option for formatting as indented and multi-line. This should be included in the SinkSettings.
https://github.com/AzureCosmosDB/data-migration-desktop-tool/blob/main/Extensions/Json/README.md#sink


HeavyWeightWindow fixture

Could you please also add corresponding methods to the CommonContainerFixture?

Could you please also add corresponding methods to the CommonContainerFixture?

Done


Slash in the name

It looks like the validator in the uploader doesn't throw error when there is a slash in the model name.

any other symbols we shouldn't allow in the name?

any other symbols we shouldn't allow in the name?

Cannot think of any other symbols, let's start with that?

in tensor names we keep allowing slashes?

.. and author names...


Swagger $ref to other .yaml file in the same directory is not working.

I have gone through some discussion and still confused whether
$ref: 'Pet.yaml'
or
$ref: 'definitions.yaml#/Pet'
work or not. Because it is not working for me.

Not yet. Swagger-node currently relies on swagger-tools and support is not yet there. See: https://github.com/apigee-127/swagger-tools/issues/227


[Machine Learning] Understanding embeddings in Machine Learning

Proposed title of article
Understanding embeddings in Machine Learning
Proposed article introduction
Working with non-dimensional data can be tough, even for experienced data scientists. A typical machine learning model expects the data to be numbers and not words, emails, website pages, lists, graphs, or probability distributions. For the data to be useful, it has to be transformed into a vector space first.
Existing approaches are not that useful and applicable as they tend to have flaws when transforming the data. A better approach would be to use embedding which is a low dimensional representation of high dimensional data.
In the proposed article, we will look at embeddings in detail and how they can be used to solve real-world problems that involve complex data.
Key takeaways
At the end of the proposed article, the learner would be able to understand:

the concept of embedding in data transformation
how embeddings are created
most commonly used embeddings models
various embedding operations
how embeddings operate in today's world.

Article quality
Unlike most articles found online which provides scattered information about the topic, the proposed article covers comprehensively the whole concept of embeddings in Machine Learning, highlighting the real-world use of embeddings. The article will be essential for beginners and readers at an advanced level.
References
N/A

Sounds like a helpful topic - let's please be sure it adds value beyond what is in any official docs and/or what is covered in other blog sites. (the articles should go beyond a basic explanation - and it is always best to reference any EngEd article and build upon it). @ogoti-lilian
Please be attentive to grammar/readability and make sure that you put your article through a thorough editing review prior to submitting it for final approval. (There are some great free tools that we reference in EngEd resources.)
ANY ARTICLE SUBMITTED WITH GLARING ERRORS WILL BE IMMEDIATELY CLOSED.
Please be sure to double-check that it does not overlap with any existing EngEd articles, articles on other blog sites, or any incoming EngEd topic suggestions (if you haven't already) to avoid any potential article closure, please reference any relevant EngEd articles in yours. - Approved

@ahmadmardeni1 Kindly do not close this topic am currently finishing up on it and I will upload it within the week. Thank you
cc @hectorkambow


Reuse current session if no prompt is selected

As an end user I want to switch between applications and be logged in right away without choosing my account again, so I don't have to select the same user all the time.
Acceptance Criteria

[ ] If I have an open session in app 1 and switch to app 2 I want to be directly in the app without seeing the login
[ ] If the applications sends the "select_account" prompt, I want to see the select account page of the login, so I have to select my acount again before seeing the app 2

Additional Context
Discussed in https://github.com/zitadel/zitadel/discussions/4636

Originally posted by helmut72 October 29, 2022
Is there an option to enable auto redirect back to the app if an user is logged on in Zitadel?
For trying out Zitadel it's great to choose one my test users or the admin user. From work (Azure) and also from home with Keycloak or Authelia I'm used to be auto redirect back into the app when I'm logged on. IMHO it's unnecessary or do I oversee anything?
Thank you.

Also had a similar topic today. User has two applications, one from a third party with SAML and an own web app based on OIDC.
Given a user has an active session already,
when the user logs into the third party appication via SSO,
then zitadel should not use the account picker or login prompt, but reuse the existing session,
and then redirect to the app automatically

@hifabienne @livio-a is there a known workaround?
Received another question about this via support channel.

@mffap No currently there is no workaround.
The next sprint will be about bugfixes and smaller issue. But as we already have a lot, I am not sure if we can get this one in aswell

what's the status?

We have not yet planned when we will implement it.

This results in a breaking behavior change. I will sort out how we can inform our customers before the changes.

Again requested in a discovery call.


fetchLocalSettings is never called to load local project.cson

I have a pretty straightforward project.cson in the root of a project, with e.g.
{
  settings:
    "*":
      core:
        themes: [
          "atom-light"
          "atom-light-syntax"
        ]
}

but the settings (and variantions of it that should work) is never added to the project settings.
Looking through the code it appears that the project should be loaded from the fetchLocalSettings function in lib/models/Project.js, but when I added some debugging it was never called.
It's possible I'm quite confused on how to use it, but I never saw any reference to fetchLocalSettings anywhere else in the project-manager (or anywhere else in the Atom code base, that I saw), so I thought I'd raise the problem here.
:beers: Cheers

It currently only looks for local settings if the project is saved (found in the projects.csonfile), so is that the case?

Got it – the project is not in the projects.cson file.
For my scenario it would be ideal if projects.cson were not needed, but alternatively I think it'd be a worthwhile note this in the README.

... though even after I have added the project to projects.cson it appears fetchLocalSettings is never called.

How/when does fetchLocalSettings get called?
It's weird that it never seems to get called and I can't find any reference to it ... anywhere.

Ehmmm, neither can I. Will have a look at it.

@danielbrodin I think this is the line that got removed and caused this issue: https://github.com/danielbrodin/atom-project-manager/commit/5aa85309001524abab13e09fbd4a61cc9f0adb8d#diff-04f8f60b0079f6cd99f98cc30de69398L187

Has this issue been addressed? I just tried using a project.cson file for my project and Atom doesn't load any settings from it. If I put the settings in the projects.cson file, they are loaded.

Seeing this as well.


'method'  not in the props, cannot configure it

Cannot configure 'method',  i want to change the request method to 'put', but 'method' not in the props
Thank you.

Provide your component snippet how are you doing.
On Nov 10, 2017 8:37 AM, "Hardcore0331"<EMAIL_ADDRESS>wrote:

cannot configure 'method', i want to change the request method to 'put',
but 'method' not in the props
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/rowanwins/vue-dropzone/issues/251, or mute the thread
https://github.com/notifications/unsubscribe-auth/AME1nTlxP1LUhCKmGY9ZqKDowoIJZrZ-ks5s073mgaJpZM4QZAgB
.

It should go in the options prop if I understand you correctly

It seems that it is not in the options?  (\node_modules\vue2-dropzone\src\index.vue)

Ah you should update to the latest version of this component, version 3 is an entire rewrite and supports all props.

Thanks for your time.


Error occurred when executing DZ_Face_Detailer:  Could not run 'torchvision::nms' with arguments from the 'CUDA' backend.

Hello author, maybe the new version of comfyui has been updated, and this error has occurred, may I ask how to solve it?

same+1

still no fix?


[util] Set enableRtOutputNaNFixup for Stars End

I'm starting to think that it might be better to make NaN texture reads defined behaviour in Vulkan rather than piling up workarounds.


On insert mode, the cursor will be before the character

I only use a subset of vim navigation functionality so I'm really often in "insert" mode.
Then when I want to easy motion to a character, the cursor will be just before the character I want to, and in my case, I almost always want to go after this character.
Any idea how I can do it ?

How do I get into insert mode? I am also wanting the cursor to go after the character I chose. FYI, I am on ST3.


default installed version of tensorflow lite arduino library is pre-compiled, causing confusing error reports

hiya @petewarden ive been re-porting my demos (hooray) for Arduino boards
https://learn.adafruit.com/tensorflow-lite-for-edgebadge-kit-quickstart
right now when folks install the TensorFlow library it defaults to the pre-compiled version, which causes very obscure errors about register arguments if they are not using the exact same processor.
https://learn.adafruit.com/tensorflow-lite-for-edgebadge-kit-quickstart/troubleshooting#uses-vfp-register-arguments-and-libtensorflowlite-dot-a-does-not-error-10-2
please make the default non-pre-compiled...Arduino IDE has a huge collection of supported boards, and as is, will confuse a lot of people :)

Sorry, I missed this one originally! I will dig into this, since we are keen to have widespread support.

thank u 👍

I've contacted the Arduino IDE team for help on this, I hope to have more progress to report soon.

hihi checkin' in on this issue - i saw also SFE bumped into the same issue
https://learn.sparkfun.com/tutorials/programming-the-sparkfun-edge-with-arduino
NOTE: It is imperative that you install the non-precompiled version of the library. Installing the pre-compiled library will only lead to failure and sadness.

The Arduino team now have a pending PR which they believe should fix this problem:
https://github.com/arduino/arduino-cli/pull/512
Could you take a look and provide feedback? Thanks for your patience!

super rad! i will! :) i published a guide this weekend - https://learn.adafruit.com/tensorflow-lite-for-circuit-playground-bluefruit-quickstart

I think this is now fixed and checked in, so closing! Let me know if I'm wrong and I'll reopen, thanks for your work on this.


Documentation missing on runtime object creation from container

Hi,
Assuming a typical createSharedApp() app, is there a way at runtime for a class to get (or create a new object) from the "main container" for the app?  Not sure if the following code is helpful, but typically if I do new Foo() on an object that uses @state/@action, I get errors even if I tried using @autobind:
Uncaught Error: 'this' in method 'AddFoos' of class 'FooList' decorated by '@action' must be bound to the current class instance.
I am guessing I need to get the object from the container and not do New on it so the @action function is correctly bound to the class.  Here is a simple example, although I understand the code could be changed to return a simple Array or Map instead of a FooList but trying to come up with a simple example of wanting to do New myClassName() at runtime instead of constructor injection; This could be good to also add to the documentation.
class Foo{
  @state
  Name: string = "";

  @state
   classifier: string = "";
}

class FooList
{
  @state
  Foos: Array<Foo> = [];

  @action
  UpdateFoos(foos: Array<Foo>){.....}
}

class FooView{
   constructor(inject("AllFoos") allFooList: FooList) {}

  get SpecialFoo(): FooList
  {
      const foolist = new FooList(); //This line would be replaced with getting a new FooList from the container...
     foolist.UpdateFoos(this.allFooList.filter((foo)=>......);
     return foolist;
  }
}

Thank you

@DaveGUI , you need the dynamic module injection feature. Here are docs of relevant dynamic modules.
In your example, you can get a new instance of FooList this way.
class FooView{
  constructor(inject("AllFoos") allFooList: FooList) {}

  async getSpecialFoo(): FooList {
     const container = await load(this, [{ provide: 'newFooList', useClass: FooList }]);
     const foolist = container.get('newFooList');
     foolist.UpdateFoos(this.allFooList.filter((foo)=>......);
     return foolist;
  }
}

Thank you!  I didn't think to look under dynamic modules, I had been checking all the dependency injection pages.


js-ng in container setup , 3sdk shell does not start

Following this guide for in-container setup link
After building the sdk binary and going to the dist repo , did ./3sdk and got the following :
(js-ng-Yg2_m37q-py3.6) root@sdk:~/sandbox/code/threefoldtech/js-ng/dist# ./3sdk
Traceback (most recent call last):
  File "jumpscale/entry_points/usershell.py", line 19, in <module>
  File "<frozen importlib._bootstrap>", line 971, in _find_and_load
  File "<frozen importlib._bootstrap>", line 955, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 665, in _load_unlocked
  File "/root/.cache/pypoetry/virtualenvs/js-ng-Yg2_m37q-py3.6/lib/python3.6/site-packages/PyInstaller/loader/pyimod03_importers.py", line 623, in exec_module
    exec(bytecode, module.__dict__)
  File "jumpscale/threesdk/__init__.py", line 1, in <module>
  File "<frozen importlib._bootstrap>", line 971, in _find_and_load
  File "<frozen importlib._bootstrap>", line 955, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 665, in _load_unlocked
  File "/root/.cache/pypoetry/virtualenvs/js-ng-Yg2_m37q-py3.6/lib/python3.6/site-packages/PyInstaller/loader/pyimod03_importers.py", line 623, in exec_module
    exec(bytecode, module.__dict__)
  File "jumpscale/threesdk/threebot.py", line 10, in <module>
  File "<frozen importlib._bootstrap>", line 971, in _find_and_load
  File "<frozen importlib._bootstrap>", line 955, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 665, in _load_unlocked
  File "/root/.cache/pypoetry/virtualenvs/js-ng-Yg2_m37q-py3.6/lib/python3.6/site-packages/PyInstaller/loader/pyimod03_importers.py", line 623, in exec_module
    exec(bytecode, module.__dict__)
  File "jumpscale/threesdk/identitymanager.py", line 8, in <module>
  File "<frozen importlib._bootstrap>", line 971, in _find_and_load
  File "<frozen importlib._bootstrap>", line 955, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 665, in _load_unlocked
  File "/root/.cache/pypoetry/virtualenvs/js-ng-Yg2_m37q-py3.6/lib/python3.6/site-packages/PyInstaller/loader/pyimod03_importers.py", line 623, in exec_module
    exec(bytecode, module.__dict__)
  File "jumpscale/god.py", line 228, in <module>
  File "jumpscale/god.py", line 217, in __getattr__
  File "jumpscale/god.py", line 117, in __getattr__
AttributeError: Can't find attr tools
[2666] Failed to execute script usershell
(js-ng-Yg2_m37q-py3.6) root@sdk:~/sandbox/code/threefoldtech/js-ng/dist#

OS : Ubuntu 18.04

verified.


handicap=int(sgf_prop(props.get('HA', [0])))    ValueError: invalid literal for int() with base 10: '吴受先'

when i run "python3 main.py preprocess data/other/tmp/wuqingyuan/"  get this error info:
366 sgfs found.
Estimated number of chunks: 17
Traceback (most recent call last):
File "main.py", line 94, in
argh.dispatch(parser)
File "/usr/local/lib/python3.5/dist-packages/argh/dispatching.py", line 174, in dispatch
for line in lines:
File "/usr/local/lib/python3.5/dist-packages/argh/dispatching.py", line 277, in _execute_command
for line in result:
File "/usr/local/lib/python3.5/dist-packages/argh/dispatching.py", line 260, in _call
result = function(*positional, **keywords)
File "main.py", line 49, in preprocess
test_chunk, training_chunks = parse_data_sets(*data_sets)
File "/mnt/ken-volume/MuGo/load_data_sets.py", line 140, in parse_data_sets
test_chunk, training_chunks = split_test_training(positions_w_context, est_num_positions)
File "/mnt/ken-volume/MuGo/load_data_sets.py", line 60, in split_test_training
positions_w_context = list(positions_w_context)
File "/mnt/ken-volume/MuGo/load_data_sets.py", line 52, in get_positions_from_sgf
for position_w_context in replay_sgf(f.read()):
File "/mnt/ken-volume/MuGo/sgf_wrapper.py", line 124, in replay_sgf
handicap=int(sgf_prop(props.get('HA', [0]))),
ValueError: invalid literal for int() with base 10: '吴受先'
it's look same sgf file props.ge('HA',[0]) get a string ,not a int.

Can you give me an example of the sgf file that it's running into issues on?
I suspect it's an sgf file that violates the standards, so having the file itself would be useful to be able to reproduce and verify the fix.

most sgff file  use gb18030 codec  in china ,so  i changed  load_data_sets.py  :
line    48
wqy00.zip
#with open(file) as f:
 with open(file,'rt',encoding='gb18030',errors='iqnore') as f:

to fix bug 👍 :
366 sgfs found.
Estimated number of chunks: 17
Traceback (most recent call last):
File "main.py", line 94, in
argh.dispatch(parser)
File "/usr/lib/python3.5/site-packages/argh/dispatching.py", line 174, in dispatch
for line in lines:
File "/usr/lib/python3.5/site-packages/argh/dispatching.py", line 277, in _execute_command
for line in result:
File "/usr/lib/python3.5/site-packages/argh/dispatching.py", line 260, in _call
result = function(*positional, **keywords)
File "main.py", line 49, in preprocess
test_chunk, training_chunks = parse_data_sets(*data_sets)
File "/home/ken/ai/go/MuGo/load_data_sets.py", line 140, in parse_data_sets
test_chunk, training_chunks = split_test_training(positions_w_context, est_num_positions)
File "/home/ken/ai/go/MuGo/load_data_sets.py", line 60, in split_test_training
positions_w_context = list(positions_w_context)
File "/home/ken/ai/go/MuGo/load_data_sets.py", line 52, in get_positions_from_sgf
for position_w_context in replay_sgf(f.read()):
File "/usr/lib64/python3.5/codecs.py", line 321, in decode
(result, consumed) = self._buffer_decode(data, self.errors, final)
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xce in position 5: invalid continuation byte

Oh.. ugh, this makes me sad.
So, the SGF file should declare that its encoding is GB18030; I can't just assume it. To be fair, most western-generated SGFs assume UTF-8, so putting in this new assumption would just break the other half of SGFs.
The other issue is that the HA property should be a number http://www.red-bean.com/sgf/go.html#types , not "Wu played first", even though that was the convention back then. I can't really ask you to go fix whatever SGF editor created these files, though, so I think the best I could do is just have a try-except to try different encodings.

Yes,I fix this bug  changed sgf_wrapper.py to 👍
try:
metadata = GameMetadata(
result=sgf_prop(props.get('RE')),
handicap=int(sgf_prop(props.get('HA', [0]))),
board_size=19)
except:
metadata = GameMetadata(
result=sgf_prop(props.get('RE')),
handicap=0,
board_size=19)
f=open("./error.txt",'a')
traceback.print_exc(file=f)
f.flush()
f.close()

Hi brilee:
encodeing bug fixed , testied ok both utf-8 and GB18030 sgf files.
need rum "pip3 install cchardet"  to install cchardet modulle first
change load_data_sets.py line 48 to:
import cchardet as chardet
def get_positions_from_sgf(file):
with open(file,'rb') as f:
result = chardet.detect(f.read())['encoding']
f.close
with open(file,'rt',encoding=result,errors='iqnore') as f:


Migrate from react-scripts-rewired to something else

Looks like react-scripts-rewired is not maintained anymore.
The problem it has now that every time you run/build the app, it updates tsconfig.json with an invalid (after typescript v4 release) configuration

In safe-react we migrated to Rescritps to avoid issues:
https://github.com/harrysolovay/rescripts

In safe-react we migrated to Rescritps to avoid issues:
https://github.com/harrysolovay/rescripts


setIconSize warning (CLANG)

Hi,
You should use @property (nonatomic, assign) NSSize iconSize;
if you want to prevent xCode 4 with Clang compiler 2.0 to display a warning.
You should also look on other warning as there are few when compiling with clang ;)

This was fixed in commit 51784809bf70


fix: node_modules created on current dir

Description
It was creating a node_modules directory on the directory the command was executed. It's fixed now.
Related issue(s)
Fixes #289

FYI @Mrc0113


wine{Unstable,Staging}: 6.10 -> 6.11

Motivation for this change
Upstream update.
Making this pr while compiling them.

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

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

[x] Fits CONTRIBUTING.md.

Seeing a failure on my end:
$ NIXPKGS_ALLOW_UNFREE=1 nixpkgs-review rev wine-6.11
[...]
error: hash mismatch in fixed-output derivation '/nix/store/4zli964kl3gs01rhfj5ah2l65x11jjwd-source.drv':
         specified: sha256-XGHssX7Qy2icaOIPLYgdFjrfP7MyiCQC2LV2KvpG57A=
            got:    sha256-dWV2eoI2SZj2ZELZggmIIb6MaxFwEenObyvQLR2KbD0=
error: 1 dependencies of derivation '/nix/store/9m6g7l41gs811dsssmr7mh2f5vq0jfzm-wine-6.11-staging.drv' failed to build
error: 1 dependencies of derivation '/nix/store/rhkn4yrpzis0k7ncsc1vcag54bglglwm-wine-6.11-staging.drv' failed to build
[...]

nixpkgs-review passes for me if I change the hash, but staging seems to break 2 out of 3 applications that I use.
Maybe @avnik knows about patchsets that are not stable at the moment?

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
Result of nixpkgs-review pr 127627 run on x86_64-linux 1

11 packages built:

lutris
lutris-free
lutris-unwrapped
pipelight
wine-staging
winePackages.staging
winePackages.unstable
wineWowPackages.staging
wineWowPackages.unstable
yabridge
yabridgectl

The following issues got detected with the above build packages.
Please fix at least the ones listed with your changed packages:

lutris:

An empty (vendor) directory got detected.
If this is a go package try replacing vendorSha256 = "0sjjj9z1dhilhpc8pq4154czrb79z9cm044jvn75kxcjv6v5l2m5";
with vendorSha256 = null;


[Errno 111] Connect call failed

So , i got this error
"response": "\n    502 Bad Gateway\n\n\n    502 Bad Gateway\n    Multiple exceptions: [Errno 111] Connect call failed ('::1', 8090, 0, 0), [Errno 111] Connect call failed ('<IP_ADDRESS>', 8090)\n\n"

This error code indicates that there was an error in the upstream server.
Which means that you had an error with flaresolverr. Are you sure that what you are trying to reach exists? Have you tested flaresolverr on its how to see if it worked?
I would need to see the configuration you are using and what you are trying to access/how you are trying to do it so I could potentially reproduce it and look around to see what the problem is. But from the 502 error description. It seems to come from upstream which is normally flaresolverr in this case

Closing this issue since it's been a month now with no response


Opposite of  --crawlers-only?

Since there's a  --crawlers-only  option to only show crawlers, is there an opposite of it to ignore crawlers?

Nevermind.  I just found ignore-crawlers!!  :-)


toChecksumAddress function would be useful

Other implementations of web3 all have a version of toChecksumAddress (EIP-55), notably:

https://web3js.readthedocs.io/en/1.0/web3-utils.html#tochecksumaddress
https://github.com/ethereum/eth-utils/blob/master/eth_utils/address.py#L105

I tried implementing this but I'm finding that it's less obvious than I'd like.

@Swader Sounds good, will do it!


Cleanup

There's a lot of changes shown here, but a lot of stuff is just moving around.
Here's a reference sheet: https://docs.google.com/spreadsheets/d/1l2xQPH6REr7fhuvItOri1HmsbNbWGpiby06ZzWdkddQ/edit?usp=sharing

Looks like me updating the roscore files after going through a round of docker building created an issue.  Should be simple enough to just copy those new versions of the files to the correct place? Once that's done, feel free to squash and merge.


Implement access restrictions to restricted portions of the site

Certain portions in the site should be restricted to internal audience only.

Being tracked here.


Setting unknow property issue

Setting unknown property: yii\mongodb\rbac\MongoDbManager::parents

Q
A

Is bugfix?
yes

New feature?
no

Breaks BC?
no

Tests pass?
yes

Fixed issues
fix issue of Setting unknown property: yii\mongodb\rbac\MongoDbManager::parents

Resolved by commit 87dd4072db6506e9906eed0db49d41eb51c5c50d


Is prometheus Support Remote storage in Warp 10 Platform  database

Dear Sir/Madam, i have been using Prometheus for past 4 months and using influx-db as remote storage. But the problem in the influx-db opensource version doesn't support clustering so we planned to store our metrics in Warp 10 Platform  database remotely. This opensource db will support clustering is there any way to send Prometheus metrics data to Warp 10 Platform  database (http://www.warp10.io/introduction/platform/). Thanks in advance.
Regrads
Abhilash S

You can use remote write to send data to 3rd party systems, See https://www.robustperception.io/using-the-remote-write-path/


The test suite exits half-way though when it calls Test.Spec.Runner.run

The offending code is here: https://github.com/owickstrom/purescript-spec-reporter-xunit/blob/master/test/Test/Spec/Reporter/XunitSpec.purs#L38
Test.Spec.Runner.run runs a test spec, and then exits the process, so the test suite never completes!

Oh, that's not very good. I think it's https://github.com/owickstrom/purescript-spec/blob/master/src/Test/Spec/Runner.purs#L166 that exits when run in NodeJS. The Config of the runner shoulds probably include a boolean field, whether to exit on success or not.


Add inspector preview to Repeater Deformer

The Repeater Deformer should have an inner editor for it's deformer like the one on the Deformable.

Added this functionality. At the time of posting this it is just on the develop branch


Linker Issues

Hello,
I built BGFX in vs2017 with the Release configuration. I now have lib files for bgfx, bimg, bimg_decode, and bx. I started a new project in vs2019 that is using these lib files. I have also included all of the include folders. However, when I build my project, I receive the following linker errors

The usual procedure for this is to search Microsoft docs for the function names without mangling (e.g. EnumProcessModules) and scroll down to requirements/library to see which library needs to be linked. In this case it's kernel32.lib and psapi.lib. Or copy all the linker options from a bgfx example.

Thanks! That did the trick. Ill close this issue out


Add bulk action for multilingual XLIFF export

Short description

This PR adds a bulk action and a pop-up window for XLIFF export in multiple languages.
Proposed changes

Add bulk action option
Add a language selection and confirmation pop-up

Resolved issues

Fixes: #1549

Thanks for your feedback! I changed the behaviour based on your suggestions and I think you're right, it feels a bit more natural this way. Does this match your expectations as well? 🙂


Linking issue

At cmake toolchain, users are have to link ws2_32 or uring.

[ ] https://github.com/microsoft/vcpkg/pull/40566

TODO: Add linux dependency and add the way to setup pkg-config at README.md


back when show error message edit age profile

When I try to change my age in my profile, I get an error and after the error is displayed, I can't dismiss it!!!
Uploading edit age.webm…

Thanks for letting me know, it's fixed now


Very low throughput when using L7NetworkPolicies with an external host

Describe the bug
A user reported low throughout when using the following L7 NP:
apiVersion: crd.antrea.io/v1beta1
kind: NetworkPolicy
metadata:
  name: testl7
spec:
  egress:
  - action: Allow
    enableLogging: true
    appliedTo:
    - podSelector: {}
    ports:
    - protocol: TCP
      port: 443
    l7Protocols:
    - tls:
        sni: 'ash-speed.hetzner.com'
  priority: 1
  tier: baseline

While running curl https://ash-speed.hetzner.com will work fine, running curl https://ash-speed.hetzner.com/100MB.bin -o /dev/null (essentially a speed test) will show very low throughput, less than 10Kbps.
When removing the policy, the speed is much better (a few 10s Mbps in my case).
The same issue can be observed with no-TLS HTTP traffic (using http://ash-speed.hetzner.com) as the host.
When capturing the traffic on antrea-gw0, I observed some large packets (larger than the 1500 MTU) with an incorrect checksum:
root@kind-worker:/# tcpdump -vvvv -n -i antrea-gw0 port 443
tcpdump: listening on antrea-gw0, link-type EN10MB (Ethernet), snapshot length 262144 bytes
00:05:56.295410 IP (tos 0x0, ttl 64, id 32016, offset 0, flags [DF], proto TCP (6), length 60)
    <IP_ADDRESS>.54872 > <IP_ADDRESS>.443: Flags [S], cksum 0x1f53 (correct), seq 767025568, win 64860, options [mss 1410,sackOK,TS val<PHONE_NUMBER> ecr 0,nop,wscale 7], length 0
00:05:56.380776 IP (tos 0x0, ttl 62, id 63701, offset 0, flags [none], proto TCP (6), length 60)
    <IP_ADDRESS>.443 > <IP_ADDRESS>.54872: Flags [S.], cksum 0x41f6 (correct), seq<PHONE_NUMBER>, ack 767025569, win 29184, options [mss 1460,nop,nop,TS val<PHONE_NUMBER> ecr<PHONE_NUMBER>,nop,wscale 7], length 0
00:05:56.384369 IP (tos 0x0, ttl 64, id 32017, offset 0, flags [DF], proto TCP (6), length 52)
    <IP_ADDRESS>.54872 > <IP_ADDRESS>.443: Flags [.], cksum 0xdd6a (correct), seq 1, ack 1, win 507, options [nop,nop,TS val<PHONE_NUMBER> ecr<PHONE_NUMBER>], length 0
00:05:56.389085 IP (tos 0x0, ttl 64, id 32018, offset 0, flags [DF], proto TCP (6), length 569)
    <IP_ADDRESS>.54872 > <IP_ADDRESS>.443: Flags [P.], cksum 0xacb3 (correct), seq 1:518, ack 1, win 507, options [nop,nop,TS val<PHONE_NUMBER> ecr<PHONE_NUMBER>], length 517
00:05:56.390122 IP (tos 0x0, ttl 62, id 63702, offset 0, flags [none], proto TCP (6), length 52)
    <IP_ADDRESS>.443 > <IP_ADDRESS>.54872: Flags [.], cksum 0xdc73 (correct), seq 1, ack 518, win 223, options [nop,nop,TS val<PHONE_NUMBER> ecr<PHONE_NUMBER>], length 0
00:05:56.470800 IP (tos 0x0, ttl 62, id 63703, offset 0, flags [none], proto TCP (6), length 2979)
    <IP_ADDRESS>.443 > <IP_ADDRESS>.54872: Flags [P.], cksum 0x2405 (incorrect -> 0xa8f2), seq 1:2928, ack 518, win 4096, options [nop,nop,TS val<PHONE_NUMBER> ecr<PHONE_NUMBER>], length 2927
00:05:56.672110 IP (tos 0x0, ttl 62, id 63706, offset 0, flags [none], proto TCP (6), length 1450)
    <IP_ADDRESS>.443 > <IP_ADDRESS>.54872: Flags [.], cksum 0xbec2 (correct), seq 1:1399, ack 518, win 4096, options [nop,nop,TS val<PHONE_NUMBER> ecr<PHONE_NUMBER>], length 1398
00:05:56.672803 IP (tos 0x0, ttl 64, id 32019, offset 0, flags [DF], proto TCP (6), length 52)
    <IP_ADDRESS>.54872 > <IP_ADDRESS>.443: Flags [.], cksum 0xd3b2 (correct), seq 518, ack 1399, win 502, options [nop,nop,TS val<PHONE_NUMBER> ecr<PHONE_NUMBER>], length 0
00:05:56.673378 IP (tos 0x0, ttl 62, id 63707, offset 0, flags [none], proto TCP (6), length 1581)
    <IP_ADDRESS>.443 > <IP_ADDRESS>.54872: Flags [P.], cksum 0x1e8f (incorrect -> 0xaf0d), seq 1399:2928, ack 518, win 4096, options [nop,nop,TS val<PHONE_NUMBER> ecr<PHONE_NUMBER>], length 1529
00:05:56.873772 IP (tos 0x0, ttl 62, id 63709, offset 0, flags [none], proto TCP (6), length 1450)

I was able to resolve the issue by disabling transmit checksum offload on antrea-gw0 (ethtool -K antrea-gw0 tx-checksumming off).
Versions:
Antrea v2.2.0
Additional information
Surprisingly, the packets captured on eth0 on the receive path are also larger than the MTU, which is a bit surprising to me, as GRO is disabled on eth0. But maybe I am misunderstanding something and GRO doesn't apply here, for this traffic which is forwarded by the Linux kernel from eth0 to antrea-gw0.

@tnqn @hongliangl it seems that this PR was aimed at solving this issue: https://github.com/antrea-io/antrea/pull/3957. Any idea why it was abandonned.
On a side note, we should also find a way to catch this condition with our L7NP e2e tests.

@antoninbas  IIRC, we don't get a good idea how to deal with the default behavior of checksum for antrea-gw0 and the option disableTXChecksumOffload.

The default behavior is on, disableTXChecksumOffload is false (by default), do nothing.
The default behavior is on, disableTXChecksumOffload is true, set the checksum to off.
The default behavior is off, disableTXChecksumOffload is false (by default), se the checksum to on (I am not sure if this is a good idea).
The default behavior is off, disableTXChecksumOffload is true,  do nothing.

cc @tnqn

@jsalatiel
This is not expected:
08:53:02.559098 IP (tos 0x0, ttl 64, id 48263, offset 0, flags [DF], proto TCP (6), length 52)
    <IP_ADDRESS>.59318 > <IP_ADDRESS>.443: Flags [.], cksum 0x5aa0 (incorrect -> 0xe51d), seq 1, ack 1, win 507, options [nop,nop,TS val<PHONE_NUMBER> ecr<PHONE_NUMBER>], length 0

This is traffic from the Pod to the external server. If disableTXChecksumOffload is set to true in the Antrea configuration, this should not happen. The kernel will compute the checksum when transmitting the packet on the eth0 interface of the Pod (no offload), and when you capture it on antrea-gw0, the checksum should still be correct.
This is why the request is failing for you (instead of just very low throughput). The first data packet (HTTP request) is never received by the server, and you can see that the server keeps trying to re-transmit the SYN-ACK packet (which has a correct checksum):
08:53:05.590637 IP (tos 0x0, ttl 49, id 0, offset 0, flags [DF], proto TCP (6), length 60)
    <IP_ADDRESS>.443 > <IP_ADDRESS>.59318: Flags [S.], cksum 0xae42 (correct), seq<PHONE_NUMBER>, ack 283223421, win 65160, options [mss 1452,sackOK,TS val<PHONE_NUMBER> ecr<PHONE_NUMBER>,nop,wscale 13], length 0

The only explanation I can think of (assuming disableTXChecksumOffload is set to true) is that the client Pod (with IP <IP_ADDRESS>) was created before setting disableTXChecksumOffload to true in the Antrea config, and not recreated thereafter. Setting disableTXChecksumOffload to true and then restarting Antrea Agents will not affect existing workload Pods. This is consistent with how we handle MTU settings as well. Such datapath changes require a restart of existing workload (non-hostNetwork) Pods.

I investigated the root cause of the low throughput and identified that Suricata is unable to send oversized packets back to OVS through antrea-l7-tap1.
For a connection between a Pod and an external network governed by an L7 NetworkPolicy, reply packets traverse the following network adapters:

Name
Type
TX Checksum Offload
TSO
GSO

ens224
Physical network adapter on Node
Enabled
Enabled
Enabled

antrea-gw0
OVS internal on Node
Enabled
Enabled
Enabled

antrea-l7-tap0
OVS internal connected to Suricata
Enabled
Disabled (set by Suricata)
Disabled (set by Suricata)

antrea-l7-tap1
OVS internal connected to Suricata
Enabled
Disabled (set by Suricata)
Disabled (set by Suricata)

eth0
Veth pair in Pod
Disabled (set by Antrea)
Disabled (set by Antrea)
Enabled (set by Antrea)

Packet Flow Analysis:

ens224:
Oversized reply packets (larger than the MTU) are successfully received and forwarded to antrea-gw0 without fragmentation because antrea-gw0 supports TSO.
Reference: 1-ens224.pcap in l7np-antrea-gw0-tx-checksum-on.tar.gz

antrea-gw0:
Similar to ens224, oversized packets are forwarded without fragmentation.
Reference: 2-antrea-gw0.pcap in l7np-antrea-gw0-tx-checksum-on.tar.gz

antrea-l7-tap0:
Oversized packets are successfully received by Suricata for processing.
Reference: 3-antrea-l7-tap0.pcap in l7np-antrea-gw0-tx-checksum-on.tar.gz

antrea-l7-tap1:
When Suricata attempts to send permitted packets back to OVS through antrea-l7-tap1, the transmission fails with the following error:
[350 - W#01-antr..tap0] 2024-11-29 10:06:05 Warning: af-packet: antrea-l7-tap1: sending packet failed on socket 17: Message too long

This happens because both TSO and GSO are disabled on antrea-l7-tap1.
Reference: 4-antrea-l7-tap1.pcap in l7np-antrea-gw0-tx-checksum-on.tar.gz

eth0:
The oversized packets never reach this adapter. Instead, TCP retransmissions are triggered until the remote server resends normal-sized packets. This is the primary cause of the low throughput.
Reference: 5-eth0.pcap in l7np-antrea-gw0-tx-checksum-on.tar.gz

Solution:
To resolve the issue, the following changes must be applied to antrea-gw0:

Disable TSO:
This ensures that oversized packets are split into fragments that fit within the MTU before being forwarded to antrea-l7-tap1.
Disable TX Checksum Offload:
After processing by Suricata, checksum offload metadata is lost. If TX checksum offloading remains enabled, the Pod will reject packets due to checksum errors. See capture 6-eth0.pcap in l7np-antrea-gw0-tx-checksum-on-tso-off.tar.gz.

After disabling TX checksum of antrea-gw0, throughput is restoredd. See files in  l7np-antrea-gw0-tx-checksum-off.tar.gz.
l7np-antrea-gw0-tx-checksum-on.tar.gz
l7np-antrea-gw0-tx-checksum-on-tso-off.tar.gz
l7np-antrea-gw0-tx-checksum-off.tar.gz

@hongliangl I don't have a great idea. Maybe we could change disableTXChecksumOffload to a pointer (*bool), which should not be a breaking change as we are just making the field optional. By default the fields would be nil / omitted, which gives us 3 possible values:

nil: do not apply any setting
false: try to force checksum offloading on
true: try to force checksum offloading off

We do need to address this however, as the L7NP feature is currently broken because of this issue.

@antoninbas, I discussed this with @tnqn , and we agreed it’s better to keep the current bool type. This ensures compatibility with released versions while allowing continued use of the existing option. Proper documentation is essential to clearly inform users about how the option works (if we disable TX checksum with this option), what they need to be aware of, and how to restore the default behavior if necessary.

@hongliangl I don't have a great idea. Maybe we could change disableTXChecksumOffload to a pointer (*bool), which should not be a breaking change as we are just making the field optional. By default the fields would be nil / omitted, which gives us 3 possible values:

nil: do not apply any setting
false: try to force checksum offloading on
true: try to force checksum offloading off

@antoninbas the problem is that the option is already set to false in the config file of existing releases, which means doing nothig. Changing false to force checksum offloading on would change the system setting for an upgraded cluster.


1.3.10 / master on Ubuntu 15.04: getrandom_wrapper() fails with Function not implemented (38)

When using mbed TLS 1.3.10 or the current mbedtls-1.3 branch on Ubuntu 15.04, mbed TLS attempts to use the getrandom syscall to retrieve random. That fails with errno 38: Function not implemented. There is currently no way to control this behaviour through cmake, config.h, or from the application.

Thanks for your report.
The issue is that, at compile-time, it looks like the getrandom() syscall is available, since its syscall number is defined in unistd.h, but at runtime a kernel older than 3.17 is used. We'll probably move to runtime detection of the kernel version using uname(2) to avoid this issue.

This should be fixed by this commit: https://github.com/ARMmbed/mbedtls/commit/d97828e7afbc7c3bc5824083f79a2a6a73bc2420
If you can get test it on your machine and confirm if it works, that would be greatly appreciated.

I just tested the fix on an Arch machine with a recent unistd.h but an older (3.14) kernel, and it works for me.

That was quick. Thanks! This indeed fixes the problem for me.

Great! Thanks for testing it.


Added Fibonacci in Javascript

Hi! I have added a Javascript function for displaying the Fibonacci series. Please review the same.
Regards,
Aftab

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 11

Complexity increasing per file
==============================
- javascript_fibonacci.js  2

See the complete overview on Codacy


Pre-release model RuntimeError: Error(s) in loading state_dict for CascadedNet

The baseline.pth file included in the release https://github.com/tsurumeso/vocal-remover/releases/tag/v6.0.0b4 fails with https://github.com/tsurumeso/vocal-remover/commit/99f92fe4b6bfe37bf4ff5bf4110ce224007312e5.
https://github.com/tsurumeso/vocal-remover/releases/tag/v5.1.1 seems to work.

# OSError: cannot load library 'libsndfile.so': libsndfile.so: cannot open shared object file: No such file or directory
# ImportError: libGL.so.1: cannot open shared object file: No such file or directory
# ImportError: libgthread-2.0.so.0: cannot open shared object file: No such file or directory
docker exec -it vc bash -c "apt-get update && apt-get install --yes libsndfile1 libgl1 libglib2.0-0"

docker exec -it vc bash -ci "curl -sLo input.wav https://upload.wikimedia.org/wikipedia/commons/f/fd/Ryan_Merkley_20190817.wav"

Run failed v6.0.0b4
docker exec -it vc bash -ci "curl -sLO https://github.com/tsurumeso/vocal-remover/releases/download/v6.0.0b4/vocal-remover-v6.0.0b4.zip"
docker exec -it vc bash -ci "unzip vocal-remover-v6.0.0b4.zip vocal-remover/models/baseline.pth"
docker exec -it vc bash -ci "rm -f models/baseline.pth && mv vocal-remover/models/baseline.pth models/baseline.pth"
docker exec -it vc bash -ci "python inference.py --input input.wav"

/opt/vc/inference.py:131: FutureWarning: You are using `torch.load` with `weights_only=False` (the current default value), which uses the default pickle module implicitly. It is possible to construct malicious pickle data which will execute arbitrary code during unpickling (See https://github.com/pytorch/pytorch/blob/main/SECURITY.md#untrusted-models for more details). In a future release, the default value for `weights_only` will be flipped to `True`. This limits the functions that could be executed during unpickling. Arbitrary objects will no longer be allowed to be loaded via this mode unless they are explicitly allowlisted by the user via `torch.serialization.add_safe_globals`. We recommend you start setting `weights_only=True` for any use case where you don't have full control of the loaded file. Please open an issue on GitHub for any issues related to this experimental feature.
  model.load_state_dict(torch.load(args.pretrained_model, map_location='cpu'))
loading model... Traceback (most recent call last):
  File "/opt/vc/inference.py", line 189, in <module>
    main()
  File "/opt/vc/inference.py", line 131, in main
    model.load_state_dict(torch.load(args.pretrained_model, map_location='cpu'))
  File "/root/venv/lib/python3.11/site-packages/torch/nn/modules/module.py", line 2584, in load_state_dict
    raise RuntimeError(
RuntimeError: Error(s) in loading state_dict for CascadedNet:
        Missing key(s) in state_dict: "aux_out.weight".
        size mismatch for out.weight: copying a param with shape torch.Size([4, 32, 1, 1]) from checkpoint, the shape in current model is torch.Size([2, 32, 1, 1]).

Run successful v5.1.1
docker exec -it vc bash -ci "curl -sLO https://github.com/tsurumeso/vocal-remover/releases/download/v5.1.1/vocal-remover-v5.1.1.zip"
docker exec -it vc bash -ci "unzip vocal-remover-v5.1.1.zip vocal-remover/models/baseline.pth"
docker exec -it vc bash -ci "rm -f models/baseline.pth && mv vocal-remover/models/baseline.pth models/baseline.pth"
docker exec -it vc bash -ci "python inference.py --input input.wav"

The problem is due to not using the branch https://github.com/tsurumeso/vocal-remover/tree/v6 which corresponds to the baseline.pth file included in the release https://github.com/tsurumeso/vocal-remover/releases/tag/v6.0.0b4.
See https://github.com/tsurumeso/vocal-remover/issues/195


Fix hhvm

Fix HHVM (for real this time 😉)
Upon merge, I shall open a bug/PR in this repo to remove the line dist: trusty, which should be removed once Travis updates their stable container to trusty.

I've also tacked on hhvm-nightly as a build stage, since that is supported in trusty (I think?) – which we'll now be using.
cc: @PhrozenByte

I'll also add another bug to remove that special case I just added (this special case is following advice from Travis – to update phpunit manually until they include a working version in hhvm trusty).
https://github.com/travis-ci/travis-ci/issues/7712#issuecomment-300509886

[...] If it is faulty, I suggest updating it yourself until the image is updated.

IMO the appropriate version of phpunit should be added to require-dev in Parsedown's composer.json and Parsedown should always use the self-provided phpunit cli, not just for HHVM. However, doing so is controversial, although they officially mention this option.

IMO the appropriate version of phpunit should be added to require-dev in Parsedown's composer.json and Parsedown should always use the self-provided phpunit cli, not just for HHVM. However, doing so is controversial, although they officially mention this option.

I also hold this opinion, though perhaps this belongs in a separate PR? 😉
I figure having it downloaded as part of the travis HHVM installation most honours Parsedown's current setup (though we could of course have this change, as it would be unneeded if Parsedown later changes to put phpunit in require-dev)

I also hold this opinion, though perhaps this belongs in a separate PR? 😉

I think it's quite reasonable to fix both problems in one PR, especially since you'll have to address the issue anyway ("all or nothing" 😆)

I'll defer this to @erusev in that case. @erusev would you like me to add
phpunit into require-dev?
On Wed, 14 Jun 2017 at 20:48, Daniel Rudolf<EMAIL_ADDRESS>wrote:

I also hold this opinion, though perhaps this belongs in a separate PR? 😉
I think it's quite reasonable to fix both problems in one PR, especially
since you'll have to address the issue anyway ("all or nothing" 😆)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/erusev/parsedown/pull/512#issuecomment-308538746, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADIvOOAycV9MaZVldVKbkBybH9EvTHTcks5sEDkfgaJpZM4N599c
.

@erusev any word on your preference here?
If you don't want to add phpunit into require-dev then this PR is good to go as is, as far as fixing the tests is concerned.
If you do, then I can add it in and slim down some of the installation instructions in the tests.

@aidantwoods Travis CI was already rewritten in #539. You can close this PR


Removed whitespace from scriv templates

Changed the whitespace after bullets in scriv templates from 3 to 1 space.
Changed the documentation and outstanding scrivlets

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.04%

Covered Lines:
178770

Relevant Lines:
195656

💛  - Coveralls


[Question] Which license is project distributed under?

https://github.com/qa-guru/allure-notifications/blob/master/LICENSE
ok?

Perfect, thank you!


New suggestion

Make a button to unload all items from me super chest like the crafting buffer has

https://github.com/reobf/Programmable-Hatches-Mod/releases/tag/v0.1.1p1-beta


Diagram

Consider adding a diagram that goes from the event, triggering a pipeline run, through to a job being executed on an Agent.

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

ID: 7f8c8539-89a7-a426-868b-96e961c39e29
Version Independent ID: de260a68-aab7-7c6b-6727-90c92c65b225
Content: Azure Pipelines New User Guide - Key concepts - Azure Pipelines
Content Source: docs/pipelines/get-started/key-pipelines-concepts.md
Product: devops
Technology: devops-cicd
GitHub Login: @juliakm
Microsoft Alias: jukullam

For example:

@DaveJarvis Thank you for this amazing diagram!!!! I'm working on getting it updated to meet our style guidelines and incorporated into the docs.


Convert to JSON?

Currently, it's a .TSV, with space-delimited arrays. This is quite non-standard formatting. If it instead explicitly mapped each language family to a standard array - ["eng", "fra"], etc. - it'd be a damn lot more convenient to parse.

If you refer to the files in data then those files are compiled from various sources to feed into the modules. Some of them come as tsv files and others are converted into tsv as some kind of intermediate format to be processed when creating the module data structures. Feel free to reformat into json if you like. Personally, I find tsv often quite convenient to be processed by line-based unix command-line tools and pipes but it's a matter of taste.

Personally, I find tsv often quite convenient to be processed by line-based unix command-line tools and pipes but it's a matter of taste.

@jorgtied, if you're used to having to process text with RegEx like most UNIX binaries' outputs necessitate, that makes more sense. I find that a little fragile, and prefer to use object-oriented approaches. Thanks for the response.


Update Doxx

As far as version number, I'm not sure what this update would be considered, but the following changes have been made:

[x] The lib is now written in ES6 and should be developed there and not in src
[x] Temporarily fix #70
[x] Updated commander.js so that the options are shorter. i.e. target_extension is just extension and -j represents --template
[x] Replaced grunt with gulp and made the build process much easier to understand
[x] Updated Prism.js to support ES6 syntax
[x] Updated template to support basic ES6 syntax... just the extends part.
[x] Updated tests by switching to mocha as it makes the test process easier with gulp.
[x] Updated REAMDE
[x] Fixed License extension

Updated commander.js so that the options are shorter. i.e. target_extension is just extension and -j represents --template

At least this statement will make it a major version :)

:+1:

@iwatakeshi released in v2.0.0! \o/

@FGRibreau :+1: I'm glad I was able to help! :)


[TIR] Fix rewrite_simplify tir::builtin::shift_left

@tqchen Please take a look, thanks

Thanks @lixiaoquan !


[WIP] Bugfix: No change reported when reloading buffers

Fixed this in the reason-libvim layer: https://github.com/onivim/reason-libvim/pull/24


Improve Javadoc and consistently use SourceVersion.latest()

Thanks. Could you please add your name the the AUTHORS file, so we are legally covered?

Thanks. Could you please add your name the the AUTHORS file, so we are legally covered?

Done.

Done.


Fix validation when using keyvaultSecretRef

Fix a regression introduced recently.
When using keyvaultSecretRef in apimodel, currently getting:
must either set useManagedIdentity in the kubernetes config or set --client-id and --client-secret

@cpuguy83 FYI, does this check out (I think your change to accommodate no sub id introduced this change)

/lgtm


Update Arch Linux installation instructions

Update arch linux installation instructions to reflect the new powershell-bin package, and the differences in the various ways of installing/building through the AUR.

@kylesferrazza   I added a docker file to build based on the powershell-bin package and referenced it as an example in the instructions.  I suspect we need to add more packages to the image for things like SSL support, but this should be a good start.  Please review

The dockerfile looks great! Just built the image and it worked perfectly.

I just merged the fix for the macOS image.

I restarted Travis-ci which should pick up the macOS change.

Looks good, all the tests passed.


Binarizing output for each audio label in AudioSet(527 classes)

Hi Yuan  ,
First of all , I would like to say huge thanks for your great work !
It would be great if you can share more details about the output values in the Readme.md.
I run demo.py and I got the linear output values (positive and negative). I would like to know what is the best way to binarize those output values  (0: audio label is absent , 1: audio label is present) ?
Anar Sultani

Hi Anar,
Thanks for your interest.
Regarding your question, the output of AST is logits, you can use the sigmoid function to convert it to value in (0, 1). For a single-label classification problem, you can use argmax to get the binary label (i.e., the class with the largest logit is 1, otherwise 0); for a multi-label classification problem, you can first apply sigmoid to logits and pick a threshold (e.g., 0.5, or the mean value of the audio samples of the class in the validation set) to binarize the output.
-Yuan

Btw I have updated the readme file to clarify the input and output of the AST model.

Hi Yuan ,
Thanks for quick and detailed reply and updating readme file.
I have looked at AudioSet  validation set to calculate mean value of the audio samples of the class but there is not any probability value in the validation set , only positive labels defined segment-wise in the validation set. I think static threshold  will not work for my case because each audio labels must have different threshold values to binarize them.
I would be happy to hear your opinions about this issue.
Anar

Hi Anar,
Sorry I didn't make myself clear - I didn't mean using the pre-defined threshold provided by the dataset as the threshold should be model-dependent. What I mean is passing all validation samples to the model and get the outputs, and use the mean of the outputs to decide the threshold.
Let's say you have a validation set of 100 samples, and you are interested in the 'speech' class, in your validation set, there are 20 samples that are labeled as 'speech'. You can feed the 100 validation samples to the AST model, and the output of AST model would be [100, 527], since you are interested in the 'speech' class, you can get the score of 'speech' at index 0 (because the index of the speech class is 0). Then you have 100 scores, you can get the average score of the 20 samples that are labeled as speech, and the average score of the other 80 samples that are not labeled as speech. Let's say the first is 0.8, and the second is 0.1, then you probably can set 0.45 as the threshold.
The above is just an example, the threshold really depends on the false positive and false negative tradeoff. And also calculating standard mAP / AUC metrics doesn't require a binarization process. Setting the threshold is not a problem specific to audio tagging / AST, but of many machine learning problems.
-Yuan

Hi Yuan,
Thanks for the very detailed answer. I really appreciated it.
Best Regards
Anar


Add AmBX SDK

SteamDB app page links to a few games using this
https://steamdb.info/app/217750/
Brief explanation of the change
Find amBX_bn files.

There's not a lot of games with this, and seeing that the company seems to have changed its focus away from gaming i doubt there will be more games with this


Fix closeTimeout being ignored

Coverage decreased (-27.5%) to 72.527% when pulling 00e4476563ad7939f799bdf993a6c9ef0d61c1a6 on helmi77:master into b1a2e3e74c2131d1c826853c0f0f49a28e7a7426 on zewish:master.


Custom edge rendering examples

Thank you for the package! It's been great to use. However, customizing rendering with WebGL is very challenging for a beginner. Your custom node rendering example is helpful to get started, but if possible could you add more documentation or examples for customizing rendering with both nodes and edges?
Also, related to this, can someone point me in the right direction for giving the edges "padding" (i.e. a bigger hover and click area than their size)?

Thank you for your response @jacomyal. That makes a lot of sense, very excited for v3! I will try out upgrading to the v3 alpha in the meantime and mess around with it.

Also! And forgive my webgl ignorance, but could some of this edge hovering functionality be covered by a setting like edgeHoverSize? And this could be impossible because of the early exit you were referencing, but then edge detection could use that size in it's calculations rather than the visible size? It might be simpler than a picking layer (although that sounds way more extensible and useful). Was trying it out with a forked version of the main branch but maybe I'll experiment with the v3 branch.


Refactoring: Split UI content into several files for maintainability (and pack them into one using e.g. webpack)

QA passed


Type incompatibility for Apollo v3

Greetings folks,
I have a project using NestJS v9, with @nestjs.apollo v10.2.0 and Apollo Server v3. The package @escape.tech/graphql-armor v1.8.2 has some type incompatibilities(*), but v1.7.2 works fine (before implementing support to Apollo Server v4).
So, the question that I have is: do we have any information relating the versions of graphql-armor and which version of the server packages they support? Also, is it the idea of this package to be compatible with multiple versions, or is it supposed to support only the latest version of the server packages?
Thanks in advance.
Cheers
(*): it seems that the incompatibility is that in Apollo v3 we don't have the includeStacktraceInErrorResponses option, and the plugins are not compatible but I haven't yet investigated why.
ps: migrating to Apollo v4 is planned, but I cannot do it right now because some refacto is needed. I just wanted to start using graphql-armor before that :)

Hey @g-belmonte,
Currently, it should be safe for you to use version <1.8.
Our goal is to support the most used version over the GraphQL ecosystem.
I will attempt to migrate our NestJS example to the versions that you provided in order to collect additional feedback. Please let  me know if you notice anything else.

Hey @c3b5aw,
Thanks for the quick response! I'll keep using the v1.7 then until our refacto to use Apollo v4 is done (propably a pair of weeks). I'll let you know if anything happens :)
Also, if I can help with any info on using graphql-armor on NestJS+Apollo, let me know!

Hey there,
I am backporting the commit I mentionned above.
Next task will be to update the documentation and propose folks to use <1.8 if they are using Apollo V3 without the new Apollo types v4.

Thanks @g-belmonte.
We will try to keep the branch 1.7 up to date with Apollo v3 until we reach EOL.

@c3b5aw Thanks for the support!


Fix file to run with powershell or bash

It works for me in mintty with ./bootstrap-vcpkg.bat. Which terminal are you using that requires running ./bootstrap-vcpkg.sh?

Ubuntu bash on windows
davide@Desktop-Davide:/mnt/c/Users/Davide/Downloads/td/vcpkg$ ./bootstrap-vcpkg.bat
./bootstrap-vcpkg.bat: line 1: @echo: command not found
& : Impossibile caricare il modulo '%~dp0scripts'. Per ulteriori informazioni, eseguire 'Import-Module %~dp0scripts'.
In riga:1 car:6
+ & {& '%~dp0scripts\bootstrap.ps1' %*}
+      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : ObjectNotFound: (%~dp0scripts\bootstrap.ps1:String) [], CommandNotFoundException
    + FullyQualifiedErrorId : CouldNotAutoLoadModule```

To receive build instructions for Windows Subsytem for Linux (WSL) you need to choose Linux as operating system and then choose your distro. Instructions for Windows will be anyway wrong there, because WSL doesn't have Windows environment.

Oh sorry, thank you.


Fixed markdown in ngModal doc

Not much to say

Landed in 8297244aba1f00bc5ade125f2aaa4c024a09ab84
Sorry I didn't merge this in -- the hassle of merging and rebasing was more work than just make this quick ajustment.

Doesn't really matter. Fixing the doc does :smile:
But TBH I disagree, merging a PR is easier than doing the commit by yourself :confused:

@posva You obviously havent seen our PR-merging policy ;-)
https://github.com/angular-ui/bootstrap/wiki/Merging-a-Pull-Request

haha Thanks! I did not. You are absolutely right, that is tedious :stuck_out_tongue:

Tedious for super small changes like this one, that is :wink:
For all other PR handling, this way of merging PR's works perfectly.


Figure out testing story for cql-engine-fhir

There are some providers in the cql-engine-fhir project that are REST based. We need a story to provide unit and/org integration tests for those components.
A few options:

Set up dummy FHIR server
Mock the IGenericClient and refactor such that that's the only thing we're using.

@csandersdev has provided example tests using WireMock in his recent contributions, such as #463 . We'll plan on following that pattern going forward.


New Paypal subscription support?

Does the library support new Paypal subscription api ?

Yes:

https://github.com/plutov/paypal/blob/master/subscription.go
https://github.com/plutov/paypal/blob/master/subscription_plan.go


doc(es) Jul 21

Jul 21
Update reactivity-fundamentals.md (#2439)
docs: transition hook add comment (#2441)
Update reactivity-fundamentals.md (#2442)

shall I add the reviewers?

shall I add the reviewers?

yes @pmart25 you can request a review any time 😄

@pmart25 there is something missing here?


ArtifactoryClient.create(...) is deprecated

So use an ArtifactoryClientBuilder as advised in ArtifactoryClient.groovy

Thank you for this needed doc update (even if my IDE was suggesting me this way of doing already)

Thank you for this @ajeans!
Can you please join our contributors community by signing JFrog's CLA so that we can merge your code?


Some problem with the sieve example

Thank you for your work on providing a fast light weight jit! As I was playing with some c examples, I notice that there are some problem with the sieve example provided in the readme. It is not the exact algorithm used to count the number of primes less or equal to SieveSize. For example, if we change the SieveSize to 100, function will return 45, while it should be 25. This doesn't affect its role as a benchmarking tool but may cause a little confusion to the new comer. I'd like to submit a pr to fix it if you like :).
FYI, the correct algorithm should be:
int sieve (int n) {
  int i, k, prime, count, iter;
  char flags[SieveSize];

  for (iter = 0; iter < n; iter++) {
    count = 0;
    for (i = 0; i < SieveSize; i++) flags[i] = 1;
    for (i = 1; i < SieveSize; i++)
      if (flags[i]) {
        prime = i + 1;
        for (k = i + prime; k < SieveSize; k += prime) flags[k] = 0;
        count++;
      }
  }
  return count;
}

FYI, the correct algorithm should be:

Thank you for pointing this out.  I took it from somewhere w/o checking the algorithm but I can not remember the source.
I've fixed all wrong sieve programs in the repository to use the right algorithm.


play carousel on mouseup

(mouseleave) can't be used on a mobile devise, so a (mouseup) should be added.

@felixkubli can you please rebase on latest dev?

Current coverage is 84.57% (diff: 100%)

Merging #1433 into development will not change coverage

@@           development      #1433   diff @@
=============================================
  Files               76         76
  Lines             2146       2146
  Methods             20         20
  Messages             0          0
  Branches           287        287
=============================================
  Hits              1815       1815
  Misses             231        231
  Partials           100        100

Powered by Codecov. Last update 8c59839...2aacca8

@valorkin rebase is done.

awesome, thanks
seems I will have to revisit it again, (https://github.com/twbs/bootstrap/blob/v4-dev/js/src/carousel.js#L232-L237)
but should work for now


POST data isn't send at localhost?

I have the latest version of FFx64 (62.0) and RESTClient (3.0.7)
Where i try to send data via method POST to php script, it return me empty $_POST array. The script code is very simple:
<?php var_dump($_REQUEST);
The result is
array(0) { }
Body param string is something like dddd=1
The remote test is ok, for example http://httpbin.org/post
I use OpenServer, php 7.0.26.
Other rest test app work fine, for example postman return me
array(1) {   ["ddd"]=>   string(3) "111" }

I also tested the same problem as you. This is a bug. I used other tools and curl commands generated by him are correct.

POST and PUT will send data from the Body panel but only when Header Content-Type: application/x-www-form-urlencoded. GET does not send the Body - in all cases the curl statement generated will send the body content.


Align WatchGuard parser to Azure Sentinel CIM

I really appreciate the work being done to align log sources to the OSSEM CDM 👍
This PR updates the WatchGuard parser to align with the Azure Sentinel common information model.
There are no publicly deployed analytics currently using this parser, however I can understand if there's hesitancy to change an existing parser. Happy to discuss the canonical approach and create a secondary "normalisation" parser to avoid breaking changes. On that topic:

Should all vendor parsers use vernacular associated to that log source and then project-rename in a dedicated normalisation parser? E.g. Like the parsers in Azure-Sentinel/Parsers/Normalized Schema - Networking (v1.0.0)
Or, for all new log sources should we look to write parsers that align to the CIM rather than having a 2nd layer of abstraction?

Field changes
Added

EventVendor
EventProduct
EventResult
EventSchemaVersion
EventTimeIngested
EventType
DvcAction
EventCount

Renamed

FireboxVersion -> EventProductVersion
LoginUser -> SrcUserName
Interface -> DvcInboundInterface
GeoDst -> DstGeoCountry
GeoSrc -> SrcGeoCountry
SentBytes -> SrcBytes
RcvdBytes -> DstBytes

@shainw : Could you please review the query


Tabby Cat (combined) (#1118)

prepare tabby_cat for PR
prepare tabby_cat for PR
done to fix federation_square from showing up here

Merge branch 'instructlab:main' into main

resolve issues with Tabby_Cat qna.yaml

make changes as specified by BJ

Apply suggestions from code review

submit cleaned up federation_square knowledge files

Merge branch 'instructlab:main' into main

Revert "submit cleaned up federation_square knowledge files"

This reverts commit dfd2ee97b1ffb7ec86ee200423711b4363a859c8.
If your PR is related to a contribution to the taxonomy, please, fill
out the following questionnaire. If not, replace this whole text and the
following questionnaire with whatever information is applicable to your PR.
Describe the contribution to the taxonomy

...
...
...

Input given at the prompt

   ...

Response from the original model

  ...

Response from the fine-tuned model

  ...

Contribution checklist

[ ] The contribution was tested with ilab generate
[ ] No errors or warnings were produced by ilab generate
[ ] All commits are signed off (DCO)
[ ] The qna.yaml file contains at least 5 seed_examples
[ ] The qna.yaml file was linted and prettified (yaml-validator can do both)
[ ] An attribution.txt file in the same folder as the qna.yaml file
[ ] Content does not include PII or otherwise sensitive or confidential information
[ ] Content does not include anything documented in the project's Avoid these Topics guidelines

@instructlab-bot precheck

@instructlab-bot precheck

Hi! :wave:
With the new update and release of 0.18.0, our base knowledge submissions have drastically changed. This has given us an opportunity to significantly improve the accuracy of our fine-tuned models, which is a major win for our community!
But alas, that has caused some changes that this PR isn't able to continue forward. We greatly appreciate the work you've put into this, and wish we could accept it but the amount of work to get it to be version: 3 is unrealistic for us to ask.
If you do want to do this, please don't hesitate to take the qna.yaml update it, and build it out. Otherwise, we hope you come back to our community with a new submission and help build out our Open Source community-driven LLM.
If you have any questions directly, please don't hesitate to comment on the PR or email me at awesomeATinstructlab.ai (yes, really), and I'll do my best to answer your questions.


Error while trying to list sp

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az ad sp list --all
Errors:
The command failed with an unexpected error. Here is the traceback:
[WinError -2146893813] : ''
Traceback (most recent call last):
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\knack/cli.py", line 231, in invoke
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 658, in execute
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 721, in _run_jobs_serially
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 713, in _run_job
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/command_modules/role/commands.py", line 69, in graph_err_handler
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 692, in _run_job
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 328, in __call__
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/command_operation.py", line 112, in handler
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/command_modules/role/commands.py", line 81, in get_graph_client_service_principals
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/command_modules/role/_client_factory.py", line 24, in _graph_client_factory
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/_profile.py", line 335, in get_login_credentials
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/_profile.py", line 593, in _create_credential
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/auth/identity.py", line 209, in get_user_credential
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/auth/msal_authentication.py", line 44, in __init__
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\msal/application.py", line 930, in get_accounts
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\msal/application.py", line 968, in _find_msal_accounts
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\msal_extensions/token_cache.py", line 54, in find
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\msal_extensions/token_cache.py", line 35, in _reload_if_necessary
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\msal_extensions/persistence.py", line 172, in load
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\msal_extensions/windows.py", line 111, in unprotect
OSError: [WinError -2146893813] : ''

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az ad sp list --all

Expected Behavior
Environment Summary
Windows-10-10.0.22000-SP0
Python 3.8.9
Installer: MSI

azure-cli 2.31.0 *

Extensions:
account 0.2.3

Additional Context

Duplicate of https://github.com/Azure/azure-cli/issues/20231


Flaky LSP HoverRequest response

While working on the LSP testing suite (#351), I had several problems with the textDocument/hover request. I initially believed the problem to be in my LSP client implementation (and I was correct, to some extent) but now I think the problem also lies in Effekt/Kiama.
A reproducible communication flow that sometimes returns the wrong result is as follows:

InitializeRequest -> ...
InitializedNotification
DidOpenTextDocumentNotification effect Exc(msg: String): Unit
HoverRequest (e.g. line 0, char 8) -> null / { ... }

Where the last HoverRequest returns null instead of the correct result around 1/2 of the times.
You can try it yourself using this JS script.
I spent a lot of time trying to debug this but ultimately did not get much further than that this function call sometimes returns None instead of Some(...):
https://github.com/effekt-lang/effekt/blob/e0d28ef099d3004ddfdde63d1e07826c7983b265/effekt/jvm/src/main/scala/effekt/Server.scala#L91
I was not able to reproduce this issue with other similar request types.

Maybe it is some kind of timing issue? It looks like process (which in turn will compile the file) will only be called onsave
https://github.com/effekt-lang/kiama/blob/3401470a4cbbe294d2df0c6923e919b8a45ca1f5/jvm/src/main/scala/kiama/util/Server.scala#L498
so maybe this call is not fully processed, when the onhover is triggered?

I don't know what's the LSP best practices here. Should there be some form of synchronization? Should we block until the first process is completed? Or is this completely against how LSP usually works?

Don't we already process on textDocument/didOpen here and, in the reproduction, here?
I assumed process to be blocking already, otherwise it should probably return a future or something, right? I think this is probably the best practice, I have not come across synchronization yet.
It's also interesting that if you comment out this line, i.e. always open and close the same file, the following hover results will always stay on the first result (null/{...}).

I just thought there might be a race condition with save, open, or another action that will indeed perform the processing but is not finished yet.

I tried the fix we talked about again; doesn't change anything. I believe that was the idea, right:
def getSymbolAt(position: Position)(implicit C: Context): Option[(Tree, Symbol)] =
  for {
    id <- getIdTreeAt(position)
    _ = C.compiler.Frontend.run(position.source) // hotfix
    sym <- C.symbolOption(id)
  } yield (id, resolveCallTarget(sym))

Yes, exactly. This is really strange. Does adding sleep in some places help to diagnose?

Added many sleeps, doesn't change anything (except execution time)... So weird


updateMonitorParticipantState(connect.MonitoringMode.SILENT_MONITOR) how to use in practice?

we have this API but how can we use it for a contact we may not have? Assuming the agent performing this API call is a supervisor/manager, how will they call this on a given contact ...if they don't have a contact yet. From what I found, there was no way for a supervisor/manager agent to retrieve ongoing contacts currently in the same instance.

This API is intended to be used after silent_monitoring has been initiated from real time metrics page in admin console. After silent_monitoring has been initiated - updateMonitorParticipantState(connect.MonitoringMode.BARGE) can be used, if that feature is enabled, and after that if supervisor wants to return back to monitoring - updateMonitorParticipantState(connect.MonitoringMode.SILENT_MONITOR) can be used in that case.
I hope that helps. Feel free to reopen the issue, if you still have remaining questions.

Just to be clear @vpogudina - you are saying that the only way to initiate silent monitoring is from the real time metrics page? I'm having a little trouble what use case is envisioned for this API if it requires using your interface to initiate it. We're building a fully custom control panel and were hoping to integrate the monitor/barge functions in our existing product. Requiring our customers to go to the real time metrics page and then come back to our product seems like a less than ideal user experience.

There is a public API to initiate monitoring that you can use to initiate silent monitoring. https://docs.aws.amazon.com/connect/latest/APIReference/API_MonitorContact.html
updateMonitorParticipantState API is intended to be use to update current monitoring state, when you have more than one AllowedMonitorCapabilities

And possibly this https://docs.aws.amazon.com/connect/latest/APIReference/API_GetCurrentUserData.html API can help getting contacts, but I haven't tried myself.

Just to clarify - https://docs.aws.amazon.com/connect/latest/APIReference/API_MonitorContact.html this can be used to initiate monitoring, and after that updateMonitorParticipantState(connect.MonitoringMode.BARGE) / updateMonitorParticipantState(connect.MonitoringMode. SILENT_MONITOR) should be used to initiate barge, return back to monitoring same contact.

Just to clarify - https://docs.aws.amazon.com/connect/latest/APIReference/API_MonitorContact.html this can be used to initiate monitoring, and after that updateMonitorParticipantState(connect.MonitoringMode.BARGE) / updateMonitorParticipantState(connect.MonitoringMode. SILENT_MONITOR) should be used to initiate barge, return back to monitoring same contact.

right understood. that's what I'm trying now, however I'm experiencing an error when I initiate monitoring via API_MonitorContact and then the subsequent updateMonitorParticipantState(connect.MonitoringMode.BARGE) via streams API call fails. I can open up a new issue for this


Readme

Coverage decreased (-0.6%) to 50.313% when pulling bea4f47980cd0bd3de2b865fe80569d641033e34 on readme into bb505e4880d964d050d1f29c6d2e60ef02624de3 on master.


Using name and version from json

Instead of hardcoding name and version, better to take that info from json that has that info.

It will solve your problem.

https://github.com/sirinoks/SSGNode/pull/6
Solved the problem and did pull request for you :)


fix failing style checks

Fixed some failing checks when running npm test.

Oh I didn't see this!


Document special ISOBMFF boxes

I am the author of Kim, the Kotlin Image Manipulation library, available on https://github.com/Ashampoo/kim
Recently I added general support for ISOBMFF and now I want to parse JPEG XL, too. The information about image size and rotation for example are not covered by the ISOBMFF standard and file formats need to specify their own.
Please document all special JPEG XL BMFF boxes related to metadata, so I can fully support them.

@kmilos Thank you, I will look into that JUMBF box.
Can you tell me where I can find some sample JPEG XL files based on BMFF or even how to create or convert JPEG to JXL?
Most tools I found are always creating the naked byte stream, I want to analyze the container.

Can you tell me where I can find some sample JPEG XL files based on BMFF or even how to create or convert JPEG to JXL?

cjxl will write those boxes AFAIK:
 --container=0|1
    0 = Do not encode using container format (strip Exif/XMP/JPEG bitstream reconstruction data).1 = Force using container format
(default: use only if needed).

 --compress_boxes=0|1
    Disable/enable Brotli compression for metadata boxes (not provided = default, 0 = disable, 1 = enable).

Other than that, you can e.g. export one w/ (uncompressed) Exif and XMP from darktable  (assuming it is built w/ libjxl support; Windows and macOS installers are, Linux packages may vary...).
You can also try a sample used in exiv2 unit tests: https://github.com/Exiv2/exiv2/blob/main/test/data/Reagan.jxl (also uncompressed)

Thank you both for your help! :)
I see my question fully answered.


[FLINK-6076] Refactor HeartbeatManager to extend HeartbeatTarget

Let the HeartbeatManager extend the HeartbeatTarget interface, because all HeartbeatManager implementations implement the HeartbeatTarget as well. Furthermore, this PR removes the HeartbeatManager#start method and passes an HeartbeatListener instance directly to the HeartbeatManagerImpl and HeartbeatManagerSenderImpl.

@tillrohrmann , it is really good to pass the HeartbeatListener directly when construct the HeartbeatManagerImpl. When I realized the heartbeat logic between different components like TM and JM, it is a little difficult to control the process between threads in order to verify the timeout expectation in tests. Now the HeartbeatListener is exposed out of the world, the test can extend the notifyHeartbeatTimeout method to control the process more easily.
And I also notice that you change the sendHeartbeat to receiveHeartbeat, have you considered to use responseHeartbeat instead which is constract with requestHeartbeat.
BTW, what is your suggestions of my current pull requests related with this modification? Should I resubmit that after this improvement merging into master?  Thank you!

Thanks for the review @zhijiangW. Yes I think it is better to initialize as much as possible in the constructor instead of calling methods afterwards.
I'm not sure whether responseHeartbeat is the best name since I like to start methods with a verb which describe what this method does. But I'm also not super excited about the current names. If you come up with better names, then let's change it then.
I've already adapted your PR and will merge it later today. I think for the remaining PRs, it's best if you take a look at the changes and adapt the other PRs accordingly.


Use LoadLibraryEx to be able to find dependencies in the DLL:s directory

If the DLL you point to has dependencies in the same directory it won't load with LoadLibrary, as that directory is not on the library path. LoadLibraryEx with the LOAD_WITH_ALTERED_SEARCH_PATH flags searches that directory, so it works as one would expect.

Yes, our own ykcs11, which has a couple of dependencies that get installed
in the same directory as the pkcs11 dll itself.
The workaround I use today is to put that installation directory on the
PATH. With my proposed change I wouldn't have to do that.
I suppose I could also copy the pkcs11 dll and it's dependencies to the
dotnet core application's directory, but in my case those are not related
so isn't a good solution.
You can find the pkcs11 implementation I'm referring to here:
https://developers.yubico.com/yubico-piv-tool/Releases/
However, I don't think this is specific to our PKCS11 implementation.
LoadLibrary only seems to include the calling application's path (in
addition to system-wide library path) in the directories it searches which
doesn't make a lot of sense when used in a .net application. I tested this
on dotnet core 2.0 and 3.0 (i haven't tested with classic .NET)
Best regards
Per
On Fri, Jan 31, 2020 at 9:43 PM Jaroslav Imrich<EMAIL_ADDRESS>wrote:

Hello @qpernil https://github.com/qpernil and thanks for the PR!
I'll need to take a closer look at the documentation of LoadLibraryEx
function and carefully consider both compatibility and security impact of
this change. Did you use Pkcs11Interop with any specific PKCS#11
implementation that needed this change?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Pkcs11Interop/Pkcs11Interop/pull/140?email_source=notifications&email_token=ANHMTEQS6APKVEQNW7YNGRTRASEOJA5CNFSM4KOGUGC2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKP6GKY#issuecomment-580903723,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANHMTESGXPCPN535W4MDMBLRASEOJANCNFSM4KOGUGCQ
.

--
--
Per Nilsson
Firmware Engineer | Yubico http://www.yubico.com/
Phone: +46 720 77 8240

Have you had a chance to look at this yet ?
Best regards
Per

Sorry not yet. I'll try this weekend.

I'll just leave these relevant docs here:

LoadLibraryExA function
Dynamic-Link Library Search Order / Alternate Search Order for Desktop Applications
Dynamic-Link Library Security

Re-reading the docs it seems one should use Path.GetFullPath(fileName) before callin LoadLibraryEx

I hope you're OK with that I slightly modified your implementation in https://github.com/Pkcs11Interop/Pkcs11Interop/pull/143 because I'm regularly using relative paths in Pkcs11Admin application. I'm going to test that code now with a few modules to confirm it does not break anything.
Could you please also test the code in LoadLibraryEx branch and confirm that it works as expected with ykcs11?

Wouldn't it work as I implemented it anyway ? I.e. resolving the path to absolute will give the same outcome, just with less code ? Just thinking, I'm fine with your way as well.
An unrelated thing you may have seen already: They now support platform-independent dynamic loading of libraries directly in dotnet core. I guess you might not be able to use it because you need to support older versions, including the 'classic' dotnet.

I will test your version later today, thx for your feedback !

Closing this PR as there is a jariq has a better solution. Thx !

Wouldn't it work as I implemented it anyway ? I.e. resolving the path to absolute will give the same outcome (LoadLibraryEx will resolve the relative path anyway), just with less code ?

No, it wouldn't because LoadLibrary* functions use different resolution algorithm than System.IO.Path.GetFullPath method.
In my previous comment I wrote that "I'm regularly using relative paths in Pkcs11Admin application" but now I see that relative is not correct term here. I'll try to explain with more details:
Let's assume there's c:\SoftHSM2\lib\softhsm2.dll file available on your disk and c:\SoftHSM2\lib\ is part of the PATH environment variable. With this setup you can call LoadLibrary("softhsm2") without any path and even without .dll prefix and LoadLibrary still loads  c:\SoftHSM2\lib\softhsm2.dll file. This is the feature I'm using regularly with Pkcs11Admin. When library dialog pops up I just write softhsm or opensc-pkcs11 then I hit enter and correct library is loaded. I know it's not the safest method and many things can go wrong but it's very convenient and well documented behavior of LoadLibrary* functions.
I hope this better explains my reasons for an alternative implementation.


Linkify text in chat

Fixes #386
Fixes #466

Maybe we can reuse the hydrogen-web linkify function? https://github.com/vector-im/hydrogen-web/blob/master/src/domain/session/room/timeline/linkify/linkify.ts
Or at least the regex? https://github.com/vector-im/hydrogen-web/blob/master/src/domain/session/room/timeline/linkify/regex.ts

https://github.com/vector-im/hydrogen-web/blob/master/src/domain/session/room/timeline/linkify/regex.ts

This regex is only for link and does not pick matrix id pattern but issue #386 talk about converting matrix id to matrix.to links.
Maybe we can go with our current regex and later when we add proper html formatting for all messages we can remove it. Sdk give as parsed markdown as message body with https://github.com/vector-im/hydrogen-web/blob/master/src/domain/session/room/timeline/MessageBody.js

Ok, sounds good. We'll stick with our own for now 👍


Fix workflow.

I think the issue is the fetch-depth: 0 setting wasn't in the right place.  I also renamed the workflow file to fix a typo. 😊

@bengtlofgren I think this should fix the failed actions.


Difference between NeqOperator "~=" and (the inexistent) "!=" leads to confusion

Starting note: I know there is no "!=" operator.
I noticed some nunit tests pass ok, but with "wrong" expressions.
For example, the following (Engine) Test passes with flying colors!
        [Test]
        public void ParseForStatement()
        {
            Test("for(k = 0; k != 2; k++) { }", false);
        }

But, has one may notice, this particular test "should" have be written as:
        [Test]
        public void ParseForStatement()
        {
            Test("for(k = 0; k ~= 2; k++) { }", false);
        }

In these (YAMP.Core.Tests) Engine nunit tests, the expression is only Parsed, it's not Evaluated.
Actually, the first one (...k != 2...) will fail during Evaluation with:
   An exception during runtime occured:
   The left side of an assignment must be symbol.

which lead me to... what the heck is the "k != 2" being parsed into ?!
I would expect this to throw a parse error, but it seems not.
It seems that "k!" is k-factorial, and whatever comes after is... "=2". what?!
So...
Although "any" test is a "good" test, I would replace the nunit test for the k~=2 version, or (better) make sure that "k != 2" throws an error (unless there is a good reason for not doing so).
I'm a C,C++,C# guy... and new to YAMP, so it's rather confusing... but what does this expression mean to the parser: "a!=0" ?
The expression "a=2; a~=1" is fine for the parser and evaluation...
The "a=2; a!=1" is ok to the parser, but blows up during evaluation.
So, is there a reason to this behavior?
If one simply implements the operator "!=" equal to the "~=" operator, would that break anything?
Many thanks for this excellent library!
Nuno

As you have noticed YAMP introduces the factorial operator !. Therefore we either have to take != as a fixed expression (possible) or we introduce a new negation operator. I decided to go for the latter, as this avoids an ambiguity (I can't use ! for the classical negation as this is taken already, so I took the bitwise negation operator ~ which does not make much sense in YAMP).
To answer your questions:

I think one could implement the operator, but there are some subtle issues (and as YAMP tries to stay close to MATLAB I've picked the same operator ~=)
As YAMP does not know during compile-time what kind of element is on the left-side the capability of assignment is evaluated during runtime - hence parsing is fine, but evaluating is not

Hope this helps!

Thanks for your explanation.
I'm happy with the "~=" choice. As I would be with "!=", or even "<>".
It's just a pitty that this particular symbol combination "!=", which is so common in current programming languages, is confused with factorial and, particularly on this case, it's not caught by the parser...
Fortunately, at least it's caught during runtime. There could be nothing worse than evaluating the "wrong" / misleading expression...
If there would be some kind of "alias" to the operators implemented on yamp, i would probably use it to get both "~=" and "!=" do the same, and actually avoiding this factorial-thing. Like I said - unless this would break any other expression or feature already valid on yamp.
It seems there is no "alias", so in this case, I would just "fix" the ParseForStatement() nunit test. It's a misleading test case.
I stumbled upon it because I was trying to find examples of Keyword expressions, and found that one...
> for(k = 0; k != 2; k++) { }
An exception during runtime occured:
The left side of an assignment must be symbol.

My fault, but It took some experiments to even understand where the error was...
And this is a small expression :-)
Which leads me to another thing...
Is there a way to "output" the full expression tree in some readable text format?
Thanks.
Nuno

Yes, makes sense. I'll insert the alias and adjust the unit test (just to have the primary operator in place; even though the current unit test would then also make sense).


feat: adds UnstructuredURLLoader for loading data from urls

Summary
Adds a UnstructuredURLLoader that supports loading data from a list of URLs.
Testing
from langchain.document_loaders import UnstructuredURLLoader

urls = [
    "https://www.understandingwar.org/backgrounder/russian-offensive-campaign-assessment-february-8-2023",
    "https://www.understandingwar.org/backgrounder/russian-offensive-campaign-assessment-february-9-2023"
]
loader = UnstructuredURLLoader(urls=urls)
raw_documents = loader.load()

@MthwRobinson Can you please adjust UnstructuredURLLoader to allow loading text/plain files from URLs? (Ideally also other mime types like text/markdown.)
Currently I'm getting Error fetching or processing https://raw.githubusercontent.com/[...], exeption: Expected content type text/html. Got text/plain; charset=utf-8.

@Boscop - Thanks for flagging, we added an issue in the unstructured library to support other MIME types and will pick it up as soon as we can.

@MthwRobinson  Got error for pdf while fetching from s3 bucket,
raise ValueError(f"Expected content type text/html. Got {content_type}.")
ValueError: Expected content type text/html. Got application/pdf.

#2793 will allow for processing non HTML resources with the URL loader. @Boscop - you can pass the content_type kwarg into the loader to force it to use a specific MIME type.