Community level Beacon identifier's assignment

Currently, the Beacon identifier is just an ID, assigned by the developer of the Beacon.
Of course, it is everybody’s best interest to use a unique ID, and in the worth case it shouldn’t be a problem if the ID is not unique.  Anyway, it may be a good practise to set up a kind of central authority to assign Beacon unique identifiers.

Couldn't we use the reversed namespace approach?  i.e. org.ega-archive.beacon

+1, the reverse domain approach sounds like a good idea.

+1
Tony

From: Miro Cupak<EMAIL_ADDRESS>Sent: 28 June 2016 15:54
To: ga4gh/beacon-team
Subject: Re: [ga4gh/beacon-team] Community level Beacon identifier's assignment (#55)
+1, the reverse domain approach sounds like a good idea.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/ga4gh/beacon-team/issues/55#issuecomment-229074393, or mute the threadhttps://github.com/notifications/unsubscribe/AI_EVOYNsXBe8Mdkgk47TCN3wU5HSyg4ks5qQTWUgaJpZM4IqlgI.

+1


Received UART data but cannot show in trace

I need to receive the UART data and show up on the trace but i only got the length of the data. Here is what it shows up on the coolwatcher.

And here is my code which is a simple UART receive testing code. i have tested to use the UART_Read to get the data but with the same problem.
#include <api_os.h>
#include <api_hal_uart.h>
#include <api_debug.h>
#include <string.h>
#include <stdio.h>
#include "api_os.h"
#include "api_event.h"
#define RECEIVE_BUFFER_MAX_LENGTH 200
#define TIMEOUTMS 1000
#define MAIN_TASK_STACK_SIZE    (1024 * 2)
#define MAIN_TASK_PRIORITY      0
#define MAIN_TASK_NAME         "UART Test Task"
static HANDLE uartTaskHandle = NULL;
uint8_t data[RECEIVE_BUFFER_MAX_LENGTH];
uint32_t timeout = TIMEOUTMS;
static void EventDispatch(API_Event_t* pEvent)
{
switch(pEvent->id)
{
case API_EVENT_ID_UART_RECEIVED:
Trace(1,"uart received API_EVENT_ID_UART_RECEIVED:%d",pEvent->param1);
if(pEvent->param1 == UART2)
{
//int length;
//length = UART_Read(UART2,data,length, timeout);
memcpy(data, pEvent->pParam1, pEvent->param2+1);
Trace(1,"uart received data,length:%d,data:%s",pEvent->param2,data);
}
break;
default:
break;
}
}
static void uart_MainTask()
{
API_Event_t* event=NULL;
UART_Config_t config = {
.baudRate = UART_BAUD_RATE_2400,
.dataBits = UART_DATA_BITS_8,
.stopBits = UART_STOP_BITS_1,
.parity   = UART_PARITY_NONE,
.rxCallback = NULL,
.useEvent = true,
};
UART_Init(UART1,config);
UART_Init(UART2,config);
while(1)
{
UART_Write(UART1,"testing",strlen("testing"));
       if(OS_WaitEvent(uartTaskHandle, (void**)&event, OS_TIME_OUT_WAIT_FOREVER))
        {
            EventDispatch(event);
            OS_Free(event->pParam1);
            OS_Free(event->pParam2);
            OS_Free(event);
        }

}

}
void uart_Main(void)
{
uartTaskHandle = OS_CreateTask(uart_MainTask ,
NULL, NULL, MAIN_TASK_STACK_SIZE, MAIN_TASK_PRIORITY, 0, 0, MAIN_TASK_NAME);
OS_SetUserMainHandle(&uartTaskHandle);
}

From a quick glance it looks like it should work, however there is a bug with the UART buffers where it seems that both UART1 and 2 use the same buffer. Since you're sending data from UART1 to UART2 it could be messing things up. Try using a USB-serial converter connected to UART2 instead.
Also, memcpy(data, pEvent->pParam1, pEvent->param2+1); looks like you're copying an extra byte, hoping for a null to terminate the string, but that extra byte could be anything. You need to use something like data[pEvent->param2] = '\0'; after the memcpy instead.

Thank you for replying.
And now i can get some message from another device but the data will not show up until i power the sender device. Those data mixed up to one string and shown after powered off the sender.
Why would that happen? and How can i fix it?

when you use USE_EVENT,don't use rxCallback ,choose the true uart number,just only one, UART_Write(UART1,data,pEvent->param2) wiil send that you get.

@Johhnzero It might be usful to post your code for this, since @benqll hasn't used any callback stuff in any of the code they've provided so far.


Fix: Correct import to resolve TypeError in RefundItemCollection

Got this TypeError while unserializing a RefundProcessed event from a queue:
TypeError: Illuminate\Database\Eloquent\Relations\HasMany::match(): Argument #2 ($results) must be of type Illuminate\Database\Eloquent\Collection, Laravel\Cashier\Refunds\RefundItemCollection given, called in /srv/app/vendor/laravel/framework/src/Illuminate/Database/Eloquent/Builder.php on line 785 and defined in /srv/app/vendor/laravel/framework/src/Illuminate/Database/Eloquent/Relations/HasMany.php:60

The error occurred because RefundItemCollection extended Illuminate\Support\Collection instead of Illuminate\Database\Eloquent\Collection, leading to a type mismatch in Eloquent relationships that require the latter.
This change fixes the issue by extending from the correct Eloquent\Collection.

Patched and released, thanks @bobab12 !


1de Co-op map timer doesn't reset between rounds

Just had a pretty good run on a 1de map, ~1min, the timer at the end showed 4mins, I guess it wasn't reset for the new round.
Any easy fix for this?

Resolved in 8e1fcc2 with the help of 88c3df9
Timers start on Coop when the first player spawns, not when objectiveGameStart is seen.  But, I still needed to reset the first spawn trigger on objectiveGameStart, so the server would know what to do.  This methodology allows players to discuss the game plan after spawn is available without that discussion negatively impacting their record.  Unless, of course, they are playing with Leeroy Jenkins ;)
Thanks.


Fix non-coded iframe

By not using back-ticks to "codify" the iframe tag, it caused hugo to render a blank iframe and consume/remove a bunch of text that came after it.
Comparison of before and after:

Good catch. I should have built the documentation and tested it when #125 in the other day.

No problem, definitely an easy one to overlook.
I could see a tool like you have on the main repo to do visual diffs of the pages to make sure it looks right. However, I'm not sure how much of a pain it would be to set up and if it would be worth the effort to do a visual review of each doc change. 😆 We're all supposed to test changes locally first before opening a PR, right?

Thank you!


How to control-alt-f1 to TTY terminal on UTM Linux?

Hello
I only have a root user on arch linux.
I'm stuck on sddm
I'm trying to go to a tty terminal in order to create a user.
But I can't go to it with control-alt-f1 or any other combination I tried.
Could anyone please advise?

Control + Option + Fn + F2

How can you do this when ctrl + option is the key in UTM to capture input?

@bendavis78 Settings > "Use Command+Option for input capture/release"

I have this setting

fn+control+option+F1 didn't work for some reason, but fn+control+option+F2 worked using MacOS keyboard


wip: feat(blog): KRE with flux v2

Signed-off-by: Batuhan Apaydın<EMAIL_ADDRESS>
What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind content

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area blog

What this PR does / why we need it:
This PR will explain how we can use Flux v2 to set up KRE (Kubernetes Response Engine).
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: developer-guy
To complete the pull request process, please assign radhikapc after the PR has been reviewed.
You can assign the PR to them by writing /assign @radhikapc in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/assign @radhikapc

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: developer-guy
To complete the pull request process, please ask for approval from radhikapc after the PR has been reviewed.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

593c9d7 Merge branch 'master' into feature/kre-with-flux-v2

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

ec559f6 Merge branch 'master' into feature/kre-with-flux-v2

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

let's keep up the process from #477


Update installation instruction on Debian systems

Hi @zyedidia,
Now since we have micro available on apt, it'd make more sense to use that, instead of snap :)
Let me know if this needs further editing.

This is roughly the same as #1506. To make sure I understand correctly, micro is only available via apt on Ubuntu 20.04 which has not been released yet, so for most people this won't work yet. Please correct me if I'm wrong.
I think it's also best to keep the instructions for installing via snap, since that it still a useful method.

Looks like I created #1506 without looking at the existing PR's.
The only difference is that I added a badge there.
Should I close #1506 ?


Game Crashes when getting a game over

Describe your problem here.
So this modchart crashes my game when i get a game over and i cant seem to find the problem with it, someone please help.
The script is the cause of the problem
aaronmode.zip
.
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
Added a few songs, Win icons, New diff,etc

Fixed it!

Fixed it!

I'm Having The Same Issue, How Did You Fix It?


Crash on importing models (Unimplemented texture encoding ) .

thanks for sharing this great work.
I can load FBX format correctly but
when i am trying to test other format ( .obj .ply) , ue crashed and exit :

using lastest release [5.4.2 texture support]

can you provide the file you are testing with ?

here is the file i tested:
bunny.zip
and my environment is UE5.0


Missing conversion functions documentation

The documentation for the functions UInt32.{of,to}_int32 and UInt64.{of,to}_int64, is missing.
From reading the macro-heavy C code, I understand that the bit pattern is just copied as-is. Is this correct? Should this be documented more precisely?

It'd be good to have these documented.
The functions inherit their semantics from C, where the conversions are defined in terms of values, not bit patterns (even if the effect is the same on many systems):

When a value with integer type is converted to another integer type other than _Bool, if the value can be represented by the new type, it is unchanged.
Otherwise, if the new type is unsigned, the value is converted by repeatedly adding or subtracting one more than the maximum value that can be represented in the new type until the value is in the range of the new type.

(So, for example, Uint32.of_int32 (-1l) is equal to -1 + 0xffffffff + 1.)

I will make an MR to add documentation.

I was checking the rest of this spec. In the case of unsigned→signed conversions (e.g., UInt32.to_int32) the spec says

Otherwise, the new type is signed and the value cannot be represented in it; either the result is implementation-defined or an implementation-defined signal is raised.

Does this mean this function might act differently on binaries linked against different libraries? Can it raise exceptions in some of them?
And more importantly, is it something that matters to the library?

Thanks for looking into this.  It means that the function might act differently when compiled with different compilers (but the compilers have to document what the behaviour is).  We could either
(a) say that the result is implementation defined (and depends on the C compiler used)
(b) change the implementation so that it's not implementation defined
I think (a) should be easier, but I'm happy with either (a) or (b).

I think (a) should be easier, but maybe not as useful for users.
In our project we would prefer (b) because it would avoid multiple machines disagreeing on what shared data means. (Even if that's unlikely in practice.)
There's also a third option: to make the conversions partial with option. I don't think it fits in the scope of this library (e.g., use in ctypes) or maybe just as a stricter variant. E.g., something along the lines of {of,to}_fitting_…. Or maybe some functions to test whether the value fits (currently requires a bit of a setup from the user) would be welcome?
My current plan (over the next week) is to

fix the semantic by making bit twiddling functions for the platform-dependent parts (that's not my area of expertise so I'll probably make somewhat inefficient code that might need some tweaking)
provide tests and documentation for those functions
In addition, if the following are of interest, I can contribute those too:
provide YY.fits_into_XX for each pair YY/XX that has conversion functions
provide hex pretty-printing (to mimic "%x") and string conversion

fix the semantic by making bit twiddling functions for the platform-dependent parts (that's not my area of expertise so I'll probably make somewhat inefficient code that might need some tweaking)

Thanks.  This sounds good.  Somewhat inefficient is fine; we can always improve the performance later, if necessary.

provide tests and documentation for those functions

This would be very welcome

provide YY.fits_into_XX for each pair YY/XX that has conversion functions

I'd prefer to leave this one out of the library for the moment.

provide hex pretty-printing (to mimic "%x") and string conversion

This one would be useful, though.

The MR is now ready, let's take all the discussion there.


[Triton] Implement Liger Kernels

Consider implementing the Liger Kernels which has shown to yield large memory savings.

RoPE: 3X speedup with ~3X peak memory reduction.
SwiGLU: 1.5X peak memory reduction
Cross Entropy: >4X memory reduction for 128k vocab size

Benchmark:

https://github.com/linkedin/Liger-Kernel

cc: @lessw2020 who is looking at this.


azure-cli: developer docs for extensions

Description of changes
Late followup to https://github.com/NixOS/nixpkgs/pull/317841, adding some documentation on how to add extensions.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

LGTM, but found a few ambiguities that could be addressed.

Thanks! Applied your suggestions. :)

Fixup was squashed into the wrong commit, fixed it.


refactor: deploy contracts with forge script

Adds script/TopTournament.s.sol to contracts
Adds script/ to Docker image
Edits deploy_anvil.sh to use script/TopTournament.s.sol

Looks great!

Rebased.


Registry UI fixes

Limit registry keys to a single line in the main list view with ellipsis when a key is truncated.
Allow double clicking anywhere on a registry key to edit, including the icon and text.
Improve polymer dialog style when resizing page so edit box does not go off screen.

@DanielSank, @adamcw

Restructured the event listeners to better delegate the responsibility and avoid duplication. The CSS and remaining changes LGTM.


From where did you get waveglow?

You trained it yourself or used the pretrained from nvidia repo?

We trained it ourselves


escape identifiers in SQL statements

Defines esc_id and esc_str (currently unused) for escaping identifiers in SQL queries. I've only updated the new interface so far.

Also I capitalised a bunch of SQL statements, to make it easier to understand what's going on.

Thanks for this @simonbyrne.
The only thing I wonder is whether esc_str is needed or if it's completely covered by parameter binding? I think you can use parameter binding everywhere that you could use a string.

Yes, I guess that is probably true.

I'll let @quinnj have the deciding vote on that one, but I'm happy for this to merge once he's had a look.

This looks great. Thanks @simonbyrne! Can you walk me through the usage of esc_str? I haven't thought totally thru it, but would it be worrisome if I do something like drop!(db, esc"my_table") and then have it "double" escaped because of the auto-call to esc_id?

esc_str is a function, not a macro (the _str suffix was unintentional), it is simply so that you can do things like:
x = "isn't"
query(db,"SELECT * FROM table WHERE foo = $(esc_str(x))")

but as @Sean1708 pointed out, most uses could probably be filled better by parameter binding.

Sorry for the hold up here. Good to merge @simonbyrne?

I've removed esc_str, as it seems to be unnecessary. I'm happy to merge it.


PIS transformation extraction: disease step HPO Phenotypes

Description
This is for the disease step, which downloads four ontologies and transforms them.
Transformations PIS was doing
This is a TSV, PIS was converting it into a JSONL and doing some other transformations: Reference here
Tasks

[ ] Figure out where to put this transformation
[x] Implement transformation
[x] Test output is equal to the one legacy PIS generates

This one is done too!


Ark os wont format and cant add any files

When i add files they wont stay but when i turn the system on they are not there it just the same thing for example i go and rename a file or delete it and i restart the system and it goes back to where it was before and i cannot format my SD card either.

Sounds like an issue with the SD card.  It’s as if it is preventing writing.

could you help

I can't fix a hardware issue like a possibly bad SD card.  What SD card is it and where did you get it from?

the one that comes with the rg351p

Those are known to fail quickly and cause issues.  That's why it's not recommended to use the default card that comes with the rg351p unit.  Get an authentic Sandisk or Samsung SD card and install the latest image on that.


Add support for yaml flow mapping and sequence

To test, copy package.json demo content to package.yaml editor
Before:

It does not mark missing name and format errors in .contributors[0]
After:

Since yaml is a superset of json, I've also make all json validation test cases to be run under yaml mode.

brilliant, thanks for this @xdavidwu !


chore: bump gitleaks version

Description of changes:
bump gitleaks version:

https://github.com/gitleaks/gitleaks/releases/tag/v8.18.1

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Why do we want to bump this version? Anything specific in v8.18.1?

@hugo-syn Can you respond to the question above, otherwise this PR will be closed.


Predictions of marketed drugs (application of knowledge graphs)

Deprecating, because merged with genome interoperability issues.


Error: Connection lost: The server closed the connection.

I'm trying to run the following code:
const mysql = require('mysql');

const dbPool = mysql.createPool({
  connectionLimit: 10,
  host: '<IP_ADDRESS>', // a different server
  user: 'user',
  password: 'pw',
  database: 'database',
});

dbPool.query('SELECT * FROM database.accounts;', (error, results, fields) => {
  console.log('Error:', error);
  console.log('Results: ', results?.[0]);
});

This works just fine on my laptop and prints the data I am expecting.
Node Version: v14.16.0
Windows 10 Version 20H2
But when I try to run the exact same code on a server, it fails instantly
Node Version: v14.16.0
SUSE Linux Enterprise Server 12 (x86_64)

Error: Error: Connection lost: The server closed the connection.
at Protocol.end (/root/testmysql/node_modules/mysql/lib/protocol/Protocol.js:112:13)
at Socket. (/root/testmysql/node_modules/mysql/lib/Connection.js:94:28)
at Socket. (/root/testmysql/node_modules/mysql/lib/Connection.js:526:10)
at Socket.emit (events.js:327:22)
at endReadableNT (internal/streams/readable.js:1327:12)
at processTicksAndRejections (internal/process/task_queues.js:80:21)
--------------------
at Protocol._enqueue (/root/testmysql/node_modules/mysql/lib/protocol/Protocol.js:144:48)
at Protocol.handshake (/root/testmysql/node_modules/mysql/lib/protocol/Protocol.js:51:23)
at PoolConnection.connect (/root/testmysql/node_modules/mysql/lib/Connection.js:116:18)
at Pool.getConnection (/root/testmysql/node_modules/mysql/lib/Pool.js:48:16)
at Pool.query (/root/testmysql/node_modules/mysql/lib/Pool.js:202:8)
at Object. (/root/testmysql/index.js:12:8)
at Module._compile (internal/modules/cjs/loader.js:1063:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:1092:10)
at Module.load (internal/modules/cjs/loader.js:928:32)
at Function.Module._load (internal/modules/cjs/loader.js:769:14) {
fatal: true,
code: 'PROTOCOL_CONNECTION_LOST'
}
Results:  undefined

What am I doing wrong?
It shouldn't be a firewall issue:

$ nc -zv <IP_ADDRESS> 3306
Connection to <IP_ADDRESS> 3306 port [tcp/mysql] succeeded!

Very strange. I can say from the error, that just like you demonstrated with nc, the TCP connection is established just fine. The error you are seeing from this library is that, after your TCP connectioj has been opened, a packet was received with either FIN or RST set, but you didn't make any API calls to end the connection. This us what it means by the server closed the connection.
Of course the question as to why the server closed your open connection on you is the mystery. Perhaps there is something in the mysql server logs to indicate why? Perhaps if you could include a packet capture of the transaction I can confirm that this module did indeed see the connection close, though that doesn't answer the why, only confirms the issue is outside of this module, somewhere in the environment.

We don't actually see anything in the MySQL logs, but we found the following in /var/log/messages:

mysqld[2960]: refused connect from <IP_ADDRESS>

But the connection is possible from other servers.
We set the bind_address to <IP_ADDRESS> and allowed connections for the user:

grant insert,select,delete,update on database.* to user@'%' identified by '....'

Its a MariaDB server, is that an issue? But then again, it works just fine from my laptop.
Do you have any idea if another MySQL setting could be the issue?

We fixed it by allowing the host in /etc/hosts.allow :)


resolver dependency contamination

Hello
I see a weird behavior on the version resolution on pipenv update, for dependencies where the latest version is to be avoided by another dependency.
Given a Pipfile with no special restriction:
[dev-packages]
"flake8" = "*"
pycodestyle = "*"
pyflakes = "*"
pylint = "*"
yapf = "*"
cryptography = "*"
isort = "*"
PyYAML = "*"
pytest = "*"
tox = "*"
"flake8-comprehensions" = "*"
asynctest = "*"
rope = "*"
pytest-sugar = "*"
pytest-mock = "*"
pytest-watch = "*"
mock = "*"
"autopep8" = "*"
pytest-cov = "*"
bandit = "*"

[packages]

For example, pycodestyle latest version is 2.4.0. Some dependencies such as flake8 adds the restriction "<2.4.0". Another one ask for "pycodestyle<1.7.0". But the resolver fails at the end with the message

Could not find a version that matches pyflakes<1.7.0,==2.0.0,>=1.5.0

While reasonably the "==2.0.0" have nothing to do here since no one ever restrict or freeze to this version. This looks like the latest found version on pypi in the previous resolver stage is taken are a hard restriction (with ==), so any later resolution stager should follow it, or fail.
This appears to me as a bug since later stages can only add new dependencies, not restrict the version of a package already resolved.
To me, each resolver stage should keep the resolution context (for pycodestyle, keep the "require=*" information instead of "require==2.4.0").
We can see this behavior in the logs:
$ pipenv update --clear --verbose
...
pycodestyle (from -r /tmp/pipenv-bvm1pyac-requirements/pipenv-jtt7r2p0-constraints.txt (line 4))
...

Finding the best candidates
...
  found candidate pycodestyle==2.4.0 (constraint was <any>)
...

Finding secondary dependencies:
..
  flake8-comprehensions==1.4.1 requires flake8!=3.2.0, flake8-comprehensions==1.4.1, mccabe<0.7.0,>=0.6.0, pycodestyle<2.4.0,>=2.0.0, pyflakes<1.7.0,>=1.5.0
...
pycodestyle==2.4.0        requires pycodestyle==2.4.0
  ^=== This is were the issue happen, the ==2.4.0 should not be, it should be something like pycodestyle==2.4.0 (restriction="*"), to keep the information
...
  flake8==3.5.0             requires flake8==3.5.0, mccabe<0.7.0,>=0.6.0, pycodestyle<2.4.0,>=2.0.0, pyflakes<1.7.0,>=1.5.0
...

New dependencies found in this round:
...
  adding ['pycodestyle', '<2.4.0,==2.4.0,>=2.0.0,>=2.3', '[]']

From now on, the resolution couldn't succeed since "<2.4.0" and =="2.4.0" are obviously incompatible.
Workaround: recopy the restriction "<2.4.0" in my Pipfile. From now, the resolver will take the latest version that respect <2.4.0 and everybody will be happy later on.
Happens with pycodestyle and pyflakes (restricted to "<1.7.0" by flake8 3.5.0)

Don't repeat dependencies in your Pipfile, pycodestyle and pyflakes are dependencies of flake8, please remove them.
This will solve most issues since Pipenv's resolver is not smart enough to handle cases like this.

I understand, but this is just an example on how to easily reproduce it. The first round of dependencies résolution kind of freeze the version of what has been found while it should not.

great news! looking forward to testing that ! thanks for the amazing work on pipenv :)

Saldy, I still have this issue.

Suddenly project started showing failed dependency of
Warning: Your dependencies could not be resolved. You likely have a mismatch in your sub-dependencies.   First try clearing your dependency cache with $ pipenv lock --clear, then try the original command again.  Alternatively, you can use $ pipenv install --skip-lock to bypass this mechanism, then run $ pipenv graph to inspect the situation.   Hint: try $ pipenv lock --pre if it is a pre-release dependency. Could not find a version that matches pycodestyle<2.4.0,>=2.0.0,>=2.4.0 Tried: 2.0.0, 2.0.0, 2.1.0, 2.1.0, 2.2.0, 2.2.0, 2.3.0, 2.3.0, 2.3.1, 2.3.1, 2.4.0, 2.4.0 Skipped pre-versions: 1.8.0.dev0, 1.8.0.dev0, 2.0.0a1, 2.0.0a1 There are incompatible versions in the resolved dependencies.
But I don't have this dependency inside Pipfile D:
Project was working month ago, not suddenly stopped. Could it be my upgrade to Python 3.7?

not, it is still happening. What I called "contamination" still happens, because autopep8 depends on a version of flake8 that requires pycodestyle<2.4.0. Your solution is to update flake8, or prevent update of autopep8 with autopep8<1.4.1.
The resolver algorithm is pretty hard to debug, probably needs to be completely rewritten. I actually do not understand why they do not delegate completely this to pip which is able to find the right version if you pip install all direct dependencies on a single line.

Because pip isn’t a resolver and doesn’t have one to resolve things. That’s why we don’t delegate it.  Pipenv is backed by dependency resolution which doesn’t just do randomly ordered installation of whatever it encounters.
We did rewrite the entire thing from scratch btw, https://github.com/sarugaku/passa — feel free to try it out


Tracking issue for sprint ending 8/30

This week Beau is focusing on the GoViral sprint for Rumi (which is due on Friday).

#212 exceeded the sprint and will be finished today; I'm moving it to this week's sprint to reflect that it was overdue.

#198 may change based on #212 and so is also remaining open and moving to this week's sprint.


SSL certificate check fails

I have a domain gourmendo.com and a subdomain api.gourmendo.com.
The check for the ssl certificate on gourmendo.com is ok but the one for api.gourmendo.com fails.
Error message: Request error occurred: hostname 'api.gourmendo.com' doesn't match 'gourmendo.com'
I have created one "Let's Encrypt" certificate for both of the domains. Is this maybe the problem?

This is to do with SNI support in Python < 2.7.9: see http://docs.python-requests.org/en/master/community/faq/#what-are-hostname-doesn-t-match-errors
Moving to a more predictable docker-based setup will help solve this issue but it's not really a bug in Cabot so much as a deployment challenge. Going to close accordingly.


Export Feature

The export feature is not behaving as expected.
The tool successful extracts the pdf->table, however when I hit 'export', it renders but doesn't appear to export to my downloads.

Thanks for the report @number-cruncher. Are you using Chrome? This is a known issue for Chrome users. Until the next bugfix release (soon!), the workaround is to use Firefox.

Chrome/Safari bug fixed in version 1.0.1 — https://github.com/tabulapdf/tabula/releases/tag/v1.0.1


The behavior of ec2.describe_instances is not right

Basically in describe instances paginator (not the regular one) API call, if instance id is given, you could not use Pagination Config because aws expect to get small list of object. It will raise botocore.exceptions.ClientError: An error occurred (InvalidParameterCombination) when calling the DescribeInstances operation: The parameter instancesSet cannot be used with the parameter maxResults error. But moto doesn't validate the parameter.
ec2_client.paginator("describe_instances").paginate(
     InstanceIds=instance_ids,
     PaginationConfig={
         "MaxItems": 9999,
         "PageSize": 100,
     },
)

@bblommers I can take a look at this


Feat:Make a forum for user discussion

Describe the feature
We should make a forum for users to discuss and post about books the forum can have threads based on genres present on https://swapreadsconnect.netlify.app/#genre
Add ScreenShots
we can shift the forum on subdomain forums.swapreadsconnect.netlify.app/
Record

[X] I agree to follow this project's Code of Conduct
[ ] I'm a GSSOC'24 contributor
[X] I want to work on this issue

ok @AnshhSingh , work on this issue . and create a pr.

I have started working on this already

@abhi03ruchi will create the pr once some pages are done within 2-4 days


Added test to update LOG_RETENTION_HOURS parameter

This PR adds a new test with the context:

To update the value of LOG_RETENTION_HOURS from default 168 to 200
To check if there are 3 nodes running after update
To check if statefulset is in ready state after parameter value update
To check if the value of LOG_RETENTION_HOURS is changed after updation successfully.

This PR also adds a new function for Kudo TestClient named UpdateInstanceParams which takes a map of parameter names and their values that needs to be updated in a given instance name.
Signed-off-by: Vibhu Jain<EMAIL_ADDRESS>

rebased from master, to have new deploy-kudo.sh step


Create Jk.developer

le release winner please pride settle manual slogan universe lock derive uniform antique canc

Muhammad Jamshaid
F23BSEEN1E02135


No dead code warnings in variable initializers

Given the following code:
Never f() { throw ''; }

class C {
  static final x = [1, 2, f(), 4];
}

var x = [1, 2, f(), 4];

I would expect to see the 4's marked as dead code, but the aren't. They do get marked if you place the last line inside a top-level function.

I believe the reason there's no dead code warning is that the analyzer doesn't create an instance of the FlowAnalysis class when evaluating top level initializers.  In retrospect, that was a mistake--it totally should (and it would improve null safety; there's a bunch of analyzer code that currently has to check whether flow analysis is null; that would go away).


Enhance tabsAreaTheme style settings, and set the tabs's horizontalAlignment

1.set the tabsArea's decoration of borderRadius etc by tabsAreaTheme.

2.Could support to make the tats center on the tabsArea and the tabsArea's width equals the tabs's width?
just like:

Hi!
I'll take a look ok?
About:

the width of the tabsArea is equal to the width of the tabs

The tab area is inside a parent container. So its width is the maximum size. To achieve the same visual effect as the image, it would be necessary to have an internal container in the tab area. However, this area may have extra buttons that must always be visible in the corner.
If you will always have a fixed number of tabs and if they are centered, they do not exceed the full width. So maybe Flutter's tab widgets (eg TabBar) are enough.

Why not provide a "tabsArea.decoration" instead of border and color, it will be more flexible.
Also, I see you already use decoration at multiple places, but not yet there. It will be helpful to improve the styling.
(Also integrate the last version in your docking component will be great)
Thx


DocumentViewer: does not load if primefaces.HIDE_RESOURCE_VERSION enabled

Describe the bug
When using the pe:documentViewer component and setting the context param primefaces.HIDE_RESOURCE_VERSION to true, e.g.

primefaces.HIDE_RESOURCE_VERSION
true

the widget does not work properly.
In our tests primefaces version is the same as primefaces-extensions, 12.0.7.
Reproducer
Attached is a sample project where you can verify that in one case the pdf is displayed correctly, while activating the primefaces.HIDE_RESOURCE_VERSION param, results in a blank pdf.
primefaces-test-master.zip
Expected behavior
No response
PrimeFaces Extensions version
12.0.7
JSF implementation
MyFaces
JSF version
2.2.15
Browser(s)
No response

@segaura you can read about why here: https://github.com/primefaces-extensions/primefaces-extensions/issues/1265
Can I ask why you are hiding the resource version?

Sure you can.
It was a requirement we've been asked to add by our company's security department.
Something like "The web application discloses specific technologies or software version numbers" meaning an attacker can gain valuable knowledge on your system (e.g. presence of a bug in a specific library version).
The requirement is related to this CWE-200 which web page provides various useful information.
Being PrimeFaces PRO users we asked PrimeTek to implement this HIDE_RESOURCE_VERSION toggle which improves overall PrimeFaces security, maybe by a little, but in the right direction.

the param just improves absolutly nothing

you can easy check the used technologies by includes JS sources or CSS
you can easily get the PF version via JS: PrimeFaces.VERSION

agree with @tandraschko it was a non-sense requirement from your security department.
See this StackOverflow on why: https://security.stackexchange.com/questions/215205/is-it-really-security-misconfiguration-to-show-a-version-number/215212#215212

I agree with everybody, the fact you are right does not necessary means my security department (and similarly OWASP) is wrong.
We had similar extensive discussion with PrimeTek support team before going this way ...now that I remember, they were really stimulating.
I understand that in this specific case the technology and its version can be obtained in other way but having the version in the querystring add another way to obtain them. In security
I understand also this is a generic requirements which applies everywhere, a best practice which in turn I personally do not discuss every time.
I propose you two additional non-strictly-security-related perspectives I have, the first one is the architectural one: is this version in querystring really necessary? Is it part of a good architectural design? Can I do the same without it.
Often a design that can do the same thing without some elements is more robust and generally better.
The latter one is the user experience one: I interact with my web application sending it requests; let's say I do the same thing now and a year later; why have the requests to be different in the two sessions only because a library has been updated?  If you think this is negligible by the user point of view, consider that probably this querystring parameter forces developers to update their Gatling and WebDriver scripts accordingly.

@segaura without the v= string then your browser cache won't be invalidated when you upgrade from PF 13.0.0 to PF 13.0.1 so now you don't get latest JS and CSS changes in that version which can break and annoy your customers.

This is an interesting example.
As I said: can this cache invalidation be obtained without this querystring parameter?
I think the answer is yes: for instance if each static resource includes a hash of the file content in its name
Is this solution better?
I think it is. Now a file which content is not changing is not refreshed in vain inside the browser cache between two primefaces[-extensions] versions. In addition this makes more happy some security departments.
Whenever possible we already moved our static resources to this content-hash-in-the-filename pattern, it works with primefaces resources, also. As far as I can remember it is done in a build step, possibly by a maven plugin, but I can be wrong.

Yes that is possible. But it all goes back to "why" and "is it worth it" and the answer is definitely NO. your app is not more secure all you have done is make someone in your security department happy who doesn't clearly know anything about security...

Also i believe this all will work if you make your hash ?v=<hash> the v= is the important part.

Also i believe this all will work if you make your hash ?v=<hash> the v= is the important part.

Correct, and if v stands somehow for version, it is now the "version" of the file which in turn changes only if content change, not the version of the library.
We are going exactly like this, we have a ?ourHashSomething=<hash>.
This has not to do with security, it is a matter of performance, efficiency and ultimately elegance.
But looking back at the issue, it can be a way to efficiently remove the library version from the querystring.
My security department only wants to make OWASP and MITRE guys happy, the ones who wrote A05:2021 and CWE-200.
I want to make my security department happy so I asked PrimeTek to add the HIDE_RESOURCE_VERSION toggle.
Now you have the opportunity to make all of us happy, supporting the ?v=something absence :-)

@segaura here is a MonkeyPatch for you to get DocumentViewer working with your code just replace v= in the code below to ourHashSomething= and the code will work.
PrimeFaces.resources.getFacesResource = function(name, library, version) {
  // just get sure - name shoudln't start with a slash
  if (name.indexOf('/') === 0) {
    name = name.substring(1, name.length);
  }

  // find any JS served JSF resource
  var scriptURI = PrimeFaces.resources.getResourceScriptURI();
  var scriptName = PrimeFaces.resources.getResourceScriptName(scriptURI);

  // replace core.js with our custom name
  scriptURI = scriptURI.replace(scriptName, name);

  // find the library like ln=primefaces
  var libraryRegex = new RegExp('[?&]([^&=]*)ln=(.*?)(&|$)');

  // find library to replace e.g. 'ln=primefaces'
  var currentLibraryName = 'ln=' + libraryRegex.exec(scriptURI)[2];

  // In a portlet environment, url parameters may be namespaced.
  var namespace = '';
  var urlParametersAreNamespaced = !(scriptURI.indexOf('?' + currentLibraryName) > -1 ||
    scriptURI.indexOf('&' + currentLibraryName) > -1);

  if (urlParametersAreNamespaced) {
    namespace = new RegExp('[?&]([^&=]+)' + currentLibraryName + '($|&)').exec(scriptURI)[1];
  }

  // If the parameters are namespaced, the namespace must be included
  // when replacing parameters.
  scriptURI = scriptURI.replace(namespace + currentLibraryName, namespace + 'ln=' + library);

  if (version) {
    var extractedVersion = new RegExp('[?&]' + namespace + 'v=([^&]*)').exec(scriptURI)[1];
    scriptURI = scriptURI.replace(namespace + 'v=' + extractedVersion, namespace + 'v=' + version);
  }

  var prefix = window.location.protocol + '//' + window.location.host;
  return scriptURI.indexOf(prefix) >= 0 ? scriptURI : prefix + scriptURI;
};

/**
 * Gets the resource URI of the first Javascript JS file served as a JSF resource.
 *
 * @return {string} The first JavasScript resource URI.
 */
PrimeFaces.resources.getResourceScriptURI = function() {
  if (!PrimeFaces.resources.SCRIPT_URI) {
    function findScriptWithVersionParam(scripts) {
      scripts.each(function() {
        var src = $(this).attr('src');
        if (src && src.indexOf('v=') !== -1) {
          PrimeFaces.resources.SCRIPT_URI = src;
          return false; // Exit the loop early
        }
      });
    }

    // normal '/showcase/javax.faces.resource/jquery/jquery.js.xhtml?ln=primefaces&v=13.0.5'
    findScriptWithVersionParam($('script[src*="/' + PrimeFaces.RESOURCE_IDENTIFIER + '/"]'));

    // portlet 'javax.faces.resource=jquery/jquery.js.xhtml?ln=primefaces&v=13.0.5'
    if (!PrimeFaces.resources.SCRIPT_URI) {
      findScriptWithVersionParam($('script[src*="' + PrimeFaces.RESOURCE_IDENTIFIER + '="]'));
    }
  }
  return PrimeFaces.resources.SCRIPT_URI;
};


EnableReliableConferenceDeletion available or not

Hi,
I have heard different kind of information that EnableReliableConferenceDeletion should be available for on-premises Skype as well, and not only for Skype online. Could you clairfy how it is? And if that is available for on-premises also, could you then write how to use it.

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

ID: 2eb630b9-b94d-a647-ca21-821e1d520b21
Version Independent ID: ede8297e-aa49-fc5e-23bb-2d678bc23e6b
Content: Set-CsConferencingPolicy (skype)
Content Source: skype/skype-ps/skype/Set-CsConferencingPolicy.md
Service: skypeforbusiness-powershell
GitHub Login: @tomtau
Microsoft Alias: tomkau

@officedocsbot assign @yogkumgit

Hello @tomkau
Can you please confirm us if EnableReliableConferenceDeletion is available in the Online version? Tried to use it in the Online version but couldn't.
Thank you

Hi @get-itips, the parameter does not apply to Online.  The documentation is not consistent.  It has the correct information early in the doc:
The following parameters are not applicable to Skype for Business Online: ApplicationSharingMode, AsJob, AudioBitRateKb, Description, EnableMultiViewJoin, EnableOnlineMeetingPromptForLyncResources, EnableReliableConferenceDeletion, FileTransferBitRateKb, Force, Identity, Instance, MaxMeetingSize, MaxVideoConferenceResolution, PipelineVariable, Tenant, and TotalReceiveVideoBitRateKb_
But the parameter itself has the wrong information and needs an update

Hello @tomkau thanks for the info, created a pull request to correct the article.

Big thanks @get-itips and @tomkau, but is there possible to get the clarification for that also? How to use it and what are the expected results? I understood it should impact how the meetings are stored after the expired time has been reached, but at least by my own experience it seems to have no impact to that.

@tomkau could you possibly fix your GitHub issue reporting https://github.com/MicrosoftDocs/office-docs-powershell/issues/3864#issue-483395047 so that your login appears in the issue details instead of mine? I've been pinged about many issues in this repository even though I have nothing to do with them (I think! :))

Btw, will you fix the document for New-CsConferencingPolicy around this, or does it really requires a separate bug report ? As there is a same problem.

@tomkau could you possibly fix your GitHub issue reporting #3864 (comment) so that your login appears in the issue details instead of mine? I've been pinged about many issues in this repository even though I have nothing to do with them (I think! :))

I will fix that, sorry

Btw, will you fix the document for New-CsConferencingPolicy around this, or does it really requires a separate bug report ? As there is a same problem.

I'll review it myself. Thanks

@get-itips  Thank you very much for the contribution and updating the documentation with PR. @Petri-X Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.

Hi @yogkumgit and @get-itips,
Great helps to fix the New- and Set-  :D
But there was another hope than only fixing the "Applies to" section. It is also missing the clarification/description. How to use it and what are the expected results? I understood it should impact how the meetings are stored after the expired time has been reached, but at least by my own experience it seems to have no impact to that.

Hello @tomkau can you please give us more information about this parameter? so we can add a description to the article. Thank you

Here's the comment from the code regarding this parameter (EnableReliableConferenceDeletion).
          If true, we provide instantaneous consistency of distributed conference state, when user deletes a conference. We ensure that the conference state is removed from all replicas when the user deletes the conference.

          If false, we only provide eventual and not instantaneous consistency for conference state deletion.

Thanks @tomkau created a PR to improve the article based on the information you provided.

@get-itips  Thank you very much for the contribution and updating the documentation with PR. @Petri-X Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


NPM support?

maybe when it's finished. we're on a break but want to get back together this fall.


Fix potential vulnerability that may allow unrestricted write access

While researching the DIDs and the way it is integrated to OrbitDB I found that the DID (did:key:1234) that is stored as an ID of the Identity object is not validated in any way in the verifyIdentity function of a provider, which makes it possible to create a fake identity that is pretending as an admin identity with write access (with the same DID in the ID field of the identity), however generated with entirely new randomly generated set of private keys.
The fix is simple – validate that that ID from the identity is equal to the id from the JWS header in the verifyIdentity function.

Thanks for the updates. The tests are failing but it looks like a simple linting will solve the issue.
If you want to run npm run lint:fix and then commit and push a lint fix and I'll merge your changes.

oops, indeed :) fixed

@c58 Thanks for your contribution. It is always appreciated.


bootstrap-saas on webjar does not have boootstrap.min.js

bootstrap-saas on webjar does not have boootstrap.min.js. How we can add one?

The webjar isn't official and we have no control over it. Try asking whatever project controls the webjar. Good luck!


Input URLs have FUZZ

Hi, I see -w wordlist:FUZZ does not work if you input URL list from Stdin input which already has the FUZZ keyword.
Ex:

Stdin (input.txt):

https://domain.com/in/goods/84?id=FUZZ&utm_medium=FUZZ&utm_source=FUZZ
https://domain.com/in/items/84?id=FUZZ&utm_medium=FUZZ&utm_source=FUZZ

My Command:

cat input.txt | ffuf  -w -:URL -w ~/Fuzz.metacharacters.txt:FUZZ -u URL

Error:

Encountered error(s): 1 errors occured.
        * Keyword FUZZ defined, but not found in headers, method, URL or POST data.

Oh, I see. This is kind of intended. Iterative keyword placing (ie. placing & replacing a keyword from another input from wordlist or -input-cmd) is not currently supported. I can see some use cases for this feature, but the implementation would be really hard and prone to errors on different levels of execution.
I understand that your use case in the description is just for demo purposes, but for that exact workflow it would work to have only the changing part of the URL in the wordlist resulting into command like:
 cat input.txt | ffuf  -w -:URL -w ~/Fuzz.metacharacters.txt:FUZZ -u URL?id=FUZZ&utm_medium=FUZZ&utm_source=FUZZ

With the wordlist input of:
https://domain.com/in/goods/84
https://domain.com/in/items/84

That all said. If you like to live dangerously, there's a very hacky way to omit the restrictions and go full YOLO. This only works if the -w -:URL is on the command line BEFORE the other one, as that will be the order of iteration.
CONTINUE AT YOUR OWN RISK
cat input.txt | ffuf  -w -:URL -w ~/Fuzz.metacharacters.txt:FUZZ -u 'URL#FUZZ'

This works as ffuf detects that the FUZZ keyword is present in the request, but the # and everything after that do not get sent to the server because of the clientside nature of it.

But It's just an example. input.txt file can contain a list of URLs with different places of FUZZ keyword. I know this maybe raise many problems but this feature will increase the flexibility of fuff. This just a feature suggestion bro xD.


'apiKey' to use posthog-docusaurus is missing

Due to .env being added on .ignore #6490 , apiKey of posthog is also missing

Hi @sirNugg3ts this is intended. the posthog API key is private and can't be shared.
If you're having issues building/running the repo, could you open a new issue?
Thank you!


Initial Elements API Check-in

This PR adds the first batch of API functions around the objects autogenerated from the proto.
All of these functions are not yet in use in the current reader/writer flows we are developing. The goal of this API is to make it easier to work with these objects from applications that use protobom as a library. In other words, this PR should not affect the translation logic we are building, it only supercharges the autogenerated objects to make them easier to use.
I wrote simple tests for most functions, I will add more test cases as the API moves forward.
API Functions
The initial API functions include:

Object
Method
Description

Edge
Copy()
Deep copies an Edge object

Edge
PointsTo()
Returns a bool if an edge points to an element

Node
Copy()
Deep copies a Node object

Node
Augment()
Agregates data to a node's fields from another node

Node
Update()
Updates a node's fields from another node

NodeList
Add()
Adds nodes from a second NodeList

NodeList
RemoveNodes()
Removes nodes from a NodeList

NodeList
GetEdgeByType()
Returns a pointer to an edge that matches From + Type

NodeList
Intersect()
Returns a new NodeList combining the elements common to two NodeLists

NodeList
Union()
Returns a new NodeList combining all elements of two NodeLists

Additionally the internal API adds the following functions to the NodeList object:

Method
Description

indexNodes()
returns an index of all nodes, keyed by ID

indexEdges
returns and index of all edges, keyed by from and type

indexRootElements
returns an index of all root elements

cleanEdges
Cleans the edges of an element removing broken and empty edges, deduping them , etc

OK, merging this, I'll start factoring out some of the logic in the reader/writer objects to use these new functions :rocket:


Base stats are all 0

Testing out the web version, I opened my Pokemon Platinum save file. When I hit Pokemon Box > Action > View > Stats, the 'Base' column is all zero.

This appears to be the case for all my Pokemon in the Pokemon Box. Weirdly, Pokemon in my party do not have the same issue.

This is not a bug, it is the nature some of the games store the data in the save files.
For most games, the base stats of Pokemon in your box are not stored, only the ones in your party.
One game I could see is an exception to this rule is for instance Let's Go Pikachu, where I could see those values in the box as well

Interesting; thanks for the quick response. I believe the PKHeX windows build shows base stats for Pokemon in the box. I presume it's just reading from a large table of base stats?

Thanks for the input on this @john-champagne
Just pushed c9c30de, which should always calculate a base stats when one is not available

💯 thanks Arley!


WslRegisterDistribution failed with error: 0xc03a0014

Windows Version
Microsoft Windows [Version 10.0.22621.1702]
WSL Version
<IP_ADDRESS>
Are you using WSL 1 or WSL 2?

[X] WSL 2
[ ] WSL 1

Kernel Version
Error code: Wsl/WSL_E_DEFAULT_DISTRO_NOT_FOUND
Distro Version
Ubuntu
Other Software
No response
Repro Steps
Every time I open my Ubuntu shell I get
Installing, this may take a few minutes...
WslRegisterDistribution failed with error: 0xc03a0014
Error: 0xc03a0014 A virtual disk support provider for the specified file was not found.

Press any key to continue...

[process exited with code 1 (0x00000001)]

Expected Behavior
My ubuntu shell
Actual Behavior
It does not load at all
Diagnostic Logs
WslLogs-2023-05-13_16-55-38.zip
Also, I noticed while filling out the above:

wsl.exe --status
Default Version: 2
WSL1 is not supported with your current machine configuration.
Please enable the "Windows Subsystem for Linux" optional component to use WSL1.

Same issue here i remove all instances and create a new one but the instalation fails with this error.

same

Did any of the virtualization features get disabled?
In powershell: What's the output of Get-WindowsOptionalFeature -online | Where-Object { $_.FeatureName -match "hyper" -or $_.FeatureName -match "linux"} ?
I rather dislike suggesting this: Disable-WindowsOptionalFeature -Online -FeatureName VirtualMachinePlatform, followed by Enable-WindowsOptionalFeature -Online -All -FeatureName VirtualMachinePlatform
The underlying api is failing to create the virtual disk.

> Get-WindowsOptionalFeature -online | Where-Object { $_.FeatureName -match "hyper" -or $_.FeatureName -match "linux"}

FeatureName : HypervisorPlatform
State       : Disabled

FeatureName : Microsoft-Windows-Subsystem-Linux
State       : Disabled

FeatureName : Microsoft-Hyper-V-All
State       : Disabled

FeatureName : Microsoft-Hyper-V
State       : Disabled

FeatureName : Microsoft-Hyper-V-Tools-All
State       : Disabled

FeatureName : Microsoft-Hyper-V-Management-PowerShell
State       : Disabled

FeatureName : Microsoft-Hyper-V-Hypervisor
State       : Disabled

FeatureName : Microsoft-Hyper-V-Services
State       : Disabled

FeatureName : Microsoft-Hyper-V-Management-Clients
State       : Disabled

You're gonna need to have something. Was that ouput after running the disable command?
You need Enable-WindowsOptionalFeature -Online -All -FeatureName VirtualMachinePlatform

I'm going to close this out seeing that virtualization features are disabled.

after executing this (Get-WindowsOptionalFeature -online | Where-Object { $.FeatureName -match "hyper" -or $.FeatureName -match "linux"}), I got same output as @earlbellinger,
I tried disable and enable VirtualMachinePlatform, with restart after every step and still getting same result.

any update on this?

Open the "Device Manager."

Look for "System devices."

Locate and enable the following devices related to virtualization:
Microsoft Hyper-V Virtualization Infrastructure Driver
Composite Bus Enumerator
Microsoft Virtual Disk Enumerator
UMBus Root Bus Enumerator
Microsoft Virtual Network Adapter Enumerator NDIS
Device Redirector Bus Enumerator (if applicable)

Please note that the actual names might slightly vary depending on the language localization of your Windows system (This names was translated from russian). Ensure you enable the corresponding devices related to virtualization.

same result
Uploading WslLogs-2024-07-24_16-16-43.zip…


Timestamp for this request was 1000ms ahead of the server's time

Hey when I make a request it always tells me that the server is ahead of my time, how can I fix it? Btw I'm a completely beginner so don't flame me haha

Your machine's local time is out of sync,  this site may be helpful to check.
https://time.is/

Thanks a alot :)


feat(RHTAPWATCH-1204): use custom cert in push-snapshot

Adjusting the push-snapshot task so it can push to a repository that uses a self-signed certificate. It does that by mounting a config map containing custom CA certificate/bundle to a path which the OS uses for loading certificates.

/retest-required

Can you gpg sign the commit? https://github.com/konflux-ci/release-service/blob/main/CONTRIBUTING.md#signing-commits

Can you gpg sign the commit? https://github.com/konflux-ci/release-service/blob/main/CONTRIBUTING.md#signing-commits

@johnbieren done. I think...

Needs rebase then I think we can merge

Needs rebase then I think we can merge

done


Fix integer overflow in BlobVec::push for ZST

reserve_exact is no-op for ZST because self.item_layout.size() > 0 is always false.
https://github.com/bevyengine/bevy/blob/daa8bf20dfb0d5e3e2c8dbd032d532bde133f221/crates/bevy_ecs/src/storage/blob_vec.rs#L112-L120
Then in push we just increase .len ignoring integer overflow.
https://github.com/bevyengine/bevy/blob/daa8bf20dfb0d5e3e2c8dbd032d532bde133f221/crates/bevy_ecs/src/storage/blob_vec.rs#L232-L237

I'm not sure how this resolves the bug mentioned in the PR description.

Attempt to call resolve_exact(usize::MAX) with vec of len 20 will panic now.

The Rust standard library also has an unchecked addition in Vec::push

Because reserve calls above in std::vec::Vec guarantees that after call to reserve Vec has enough spare capacity. BlobVec::reserve_exact does not guarantee that. Which this PR fixes.

The corresponding grow method for RawVec also makes explicit checks to see if it's handling a ZST, which this PR is changing to be an unconditional assertion instead of a soft check.

This "soft check" is converted to panic in handle_reserve in the same file. Which is exactly the same this commit is doing: panicking on ZST in grow.

The removal of the check in reserve_exact means we do not guard against this panic.

I'm sorry, I don't understand.

This may cause panics on BlobVecs holding ZSTs

Correct. We should panic when attempting to add more than usize::MAX elements to BlobVec. It is the same behavior as Vec.

something that is not part of reserve_exact's defined interface, and something we must support.

The interface of reserve_exact is: if reserve_exact is successful (i.e. does not panic), push will be successful.

We should not allow BlobVec's len to overflow, but I don't think the changes here are appropriate for addressing it.

Is it possible you misunderstood this PR?
Otherwise, what do you suggest?

Added a test.

Rephrased comments.


Save/load fulfilled requests info

Err... I somehow missed that part in #1040 /facepalm/
:)

I wonder if it's really necessary to save this information.  Why not just start with a clean slate on each startup ?

That's the other part of solving issues I mentioned in #1040 - fulfilled requests should not only expire but they should also be preserved while they are not expire. We need to do that because:
a) on wallet restart you shouldn't ask same nodes for a heavy request to avoid being banned for doing so;
b) you were able to increase dos score for some nodes that are trying to violate same rule from their side (rather than start counting from 0) and eventually ban them even after your wallet restarted.
I agree that in most cases this won't matter much (most users don't restart their wallet every few minutes or so) but I think it worth fixing to have consistent solution.


Docs for tts/

Is there a doc for tts tasks? or could you give me some hint about how to run it. Thanks.

Since tts is deprecated in this fork, please visit the upstream repository.
https://github.com/MoonInTheRiver/DiffSinger

This fork is focused on SVS and TTS functionality in this fork is entirely unmaintained and untested. No documentation is planned and you probably should use MoonInTheRiver's original repo for TTS instead too.


Create QuickSort.java

Can you keep label as hacktoberFest

As i will be counted as one of 4 prs for hactoberFest


Doesn't work at last Xcode 6.3.2 (6D2105)

Hi, thank you for this nice plugin.
Unfortunately after installing 6.3.2 plugin generates nothing. I tried installing with Plugin Manager and also build from last commit in master f6b13a6b88182b43af76a8f43b9a111a0e065881. All have the same effects.

Is your project name the same as the folder name which contains your Image.xcassets ?

Just rename it to match and all works well! Thanks! :+1:


web null safe tech preview (ddc, dart2js, web libraries)

@sigmundch @franklinyow what's missing here?

Thanks for checking. This is now complete.


[SR-15059] JSON5 support missing from open-source Swift

Previous ID
SR-15059

Radar
rdar://problem/81871548

Original Reporter
ryanw (JIRA User)

Type
Bug

Environment
Swift 5.5-dev, FC34

Additional Detail from JIRA

Votes
1

Component/s
Foundation

Labels
Bug

Assignee
None

Priority
Medium

md5: 3417224c6821cdb0321376f444c23656

Issue Description:
I see JSON5 support has been added as an opt-in option to JSONDecoder for iOS 15/macOS 12, however this does not yet seem to have been implemented in the open-source stdlib.
https://developer.apple.com/documentation/foundation/jsondecoder/3766916-allowsjson5

This is implemented in swift-foundation's JSONDecoder. See also #5001


Helm charts - externalize more configuration for consumers

Our egeria-base chart is a good starting point for a simple egeria deployment (with persistence)
However it's become clear when trying to add a new postgres chart (one example of many) that we may wish to
easily add

additional configuration variables
additional scripts (for example to configure integration service)
additional containers (for example postgres)

These are common practice when developing helm charts and some useful tips are documented at https://dzone.com/articles/the-art-of-the-helm-chart-patterns-from-the-offici

Though an egeria operator will change some of the configuration around egeria platforms and servers, the general approach is likely to remain valid with any charts we do provide - helm still adds value in integrating a broader range of components for demos, tutorials and real-world deployments


[9.x] Add support for native enum to enum column : $table->enum('status', OrderStatus::cases())

See many previous attempts why this is a bad idea.

@driesvints thank you !

Is there any link to those previous attempts ?

@MarienMupenda https://github.com/laravel/framework/pull/39854 https://github.com/laravel/framework/pull/40662 https://github.com/laravel/framework/pull/41026 https://github.com/laravel/framework/pull/41827 https://github.com/laravel/framework/pull/43483


Dropdown values are larger than dropdown label

On click on a dropdown, the values inside the dropdown are larger than the dropdown label.
Your Environment

Browser Name and version: Firefox 48.0 and Chrome 53.0.2785.116

Fixed recently.


Question about the attention

Hello
I have a question about your attention code.
In the decoder code, you concatenate the embedded and the context.
Is this the way how the original paper did?
and if not, could you explain why you concatenated them?
    rnn_input = torch.cat((embedded, context), dim=2)

    #rnn_input = [1, batch size, (enc hid dim * 2) + emb dim]

    output, hidden = self.rnn(rnn_input, hidden.unsqueeze(0))

Thanks a lot! :)


new version bump?

any planning about new version with GET rpc?

Yeah, we were planning to get the open pull requests taken care of and then release a new version.


Napari error : does not install plugin in GUI bundle app

🐛 Bug
I am using the bundle app without python in MAC OSX mojave.
I want to install plugins to use napari but it does not work..
Every time that I tried to install a plugin I received this error:
2021-07-28 22:57:36.775 napari[2163:247395] error: unrecognized arguments: --upgrade
To Reproduce
Downloadind the app and install in mac.
Try to install plugin

2021-07-28 22:57:36.775 napari[2163:247395] error: unrecognized arguments: --upgrade
Expected behavior
I want to be able to use plugins for segmentation and 3D
Environment

this is a duplicate of https://github.com/napari/napari/issues/2965,
it should have been fixed in https://github.com/napari/napari/pull/2980... but I just tested the recent and indeed it didn't work.  I'm going to close this issue, since we already have a dedicated (and open) issue for this.  please track progress at #2965


Publish to NPM

Is it possible to publish this library to NPM?

@greenSnot: Is it possible to change that back since this repo is now up to date?

@aemkei done!

@greenSnot Thanks!
But can you do it one more time? I forgot to update the package.json. See https://github.com/ubilabs/kd-tree-javascript/commit/165adc0b87968df14edb0f6418bb6565ba18aeb0

@aemkei Done and what's your npm account? I would like to transfer its ownership to you.

Thanks! It is aemkei, see https://www.npmjs.com/~aemkei

🎉

p.s. Thanks @aemkei for this awesome implementation. I'm searching point clouds in excess of 50k points at blazing speeds.


Ionic 2: NavController.onPageDidEnter() Is Called On All Pages On App Start

On app start it appears like all pages have their onPageDidEnter() event called.
The pages are setup like this:
import {Page, ViewController, NavController, Events} from 'ionic-framework/ionic';

@Page({
  templateUrl: 'build/pages/dashboard/dashboard.html',
})
export class DashboardPage {
  nav: NavController;
  events: Events;

  constructor(nav: NavController, events: Events) {
    this.nav = nav;
    this.events = events;
  }

  onPageDidEnter() {
    console.log("Dashboard page load event fired.");
    this.events.publish('pageLoad', "DashboardPage");
  }
}

Cordova CLI: 5.3.3
Gulp version:  CLI version 3.9.0
Ionic Version: 2.0.0-beta.0
Ionic CLI Version: 2.0.0-beta.17
Ionic App Lib Version: 2.0.0-beta.8
ios-deploy version: 1.5.0
ios-sim version: 4.1.1
OS: Mac OS X El Capitan
Node Version: v0.12.2
Xcode version: Xcode 7.2.1 Build version 7C1002

Disregard... In the MyApp class I had:
rootPage: Type = DashboardPage;
Then later in my initializeApp() function I was manually setting the root page with:
this.nav.setRoot(SettingsPage);
That made the onPageDidEnter() events fire for both pages and made me think all pages were getting their onPageDidEnter() event called on start.


Superb Tool; One cometic request

Thank you for the superb tool!  Is there a donation path?
One cosmetic change request is that the right edge of all network activity level graphs windows does not have a yellow border.  Since yellow is the same color as the download graph line, it looks like the download graph is continuously spiking to maximum level,  At the same time, you may wish to remove the bottom yellow border.  The frameless gray window looks good on the black surrounding area.
This change can wait until the next release, which may include other updates.   I am having a hard time distinguishing this maintenance effort from similarly named tools in the open source world.  There may be other changes from their work that you wish to integrate also.
Keep up the good work, and thanks

I'll look into it next week to see if it's an easy fix. Seems to be like a 1px white shadow.
If not, it will be replaced anyway since I do plan to switch to a different UI library.

Fixed in 2<IP_ADDRESS>8


error in running reconstruct_contig.py

Hi:
It reported ValueError when I ran reconstruct_contig.py with the following messages:
Traceback (most recent call last):
File "/share/nas1/liyingying/tools/soft/cogent/Anaconda/anaconda3/envs/anaCogent/bin/reconstruct_contig.py", line 4, in
import('pkg_resources').run_script('Cogent==2.1', 'reconstruct_contig.py')
File "/cogent/Anaconda/anaconda3/envs/anaCogent/lib/python2.7/site-packages/pkg_resources/init.py", line 750, in run_script
self.require(requires)[0].run_script(script_name, ns)
File "/cogent/Anaconda/anaconda3/envs/anaCogent/lib/python2.7/site-packages/pkg_resources/init.py", line 1527, in run_script
exec(code, namespace, namespace)
File "/cogent/Anaconda/anaconda3/envs/anaCogent/lib/python2.7/site-packages/Cogent-2.1-py2.7.egg/EGG-INFO/scripts/reconstruct_contig.py", line 352, in
main()
File "/cogent/Anaconda/anaconda3/envs/anaCogent/lib/python2.7/site-packages/Cogent-2.1-py2.7.egg/EGG-INFO/scripts/reconstruct_contig.py", line 268, in main
run_Cogent_on_split_files(dirs, depth=0)
File "/cogent/Anaconda/anaconda3/envs/anaCogent/lib/python2.7/site-packages/Cogent-2.1-py2.7.egg/EGG-INFO/scripts/reconstruct_contig.py", line 71, in run_Cogent_on_split_files
run_Cogent_on_input()
File "/cogent/Anaconda/anaconda3/envs/anaCogent/lib/python2.7/site-packages/Cogent-2.1-py2.7.egg/EGG-INFO/scripts/reconstruct_contig.py", line 157, in run_Cogent_on_input
seqid, weight = line.strip().split('\t')
ValueError: need more than 1 value to unpack
I have no idea to for this.Could you help me ?
Thanks!
LIYY

Hi @tengdiyouying608 ,
It looks like in the split directories, the file in.weights is not being properly propagated.
The in.weights file should look like below, where one column is the sequence ID and the second column is the weight (default is 1). The two columns should be separated by a tab.
i2_HQ_samplee5d47c|c63386/f3p36/2608    1
i2_HQ_samplee5d47c|c7469/f2p32/2292 1
i2_HQ_samplee46251|c31503/f2p29/2286    1

Can you check that you have in.weights in the run directory and the split directories?
--Liz

Traceback (most recent call last):
File "/share/work2/liuying/software/anaconda2/envs/anaCogent/bin/reconstruct_contig.py", line 4, in
import('pkg_resources').run_script('Cogent==3.1', 'reconstruct_contig.py')
File "/share/public/software/WDLIB/lib/FALCON_unzip_0619.2017/lib/python2.7/site-packages/pkg_resources/init.py", line 739, in run_script
self.require(requires)[0].run_script(script_name, ns)
File "/share/public/software/WDLIB/lib/FALCON_unzip_0619.2017/lib/python2.7/site-packages/pkg_resources/init.py", line 1494, in run_script
exec(code, namespace, namespace)
File "/share/work2/liuying/software/anaconda2/envs/anaCogent/lib/python2.7/site-packages/Cogent-3.1-py2.7.egg/EGG-INFO/scripts/reconstruct_contig.py", line 337, in
main()
File "/share/work2/liuying/software/anaconda2/envs/anaCogent/lib/python2.7/site-packages/Cogent-3.1-py2.7.egg/EGG-INFO/scripts/reconstruct_contig.py", line 259, in main
run_Cogent_on_input()
File "/share/work2/liuying/software/anaconda2/envs/anaCogent/lib/python2.7/site-packages/Cogent-3.1-py2.7.egg/EGG-INFO/scripts/reconstruct_contig.py", line 218, in run_Cogent_on_input
sp.contract_sinks(G, path_d, mermap)
File "/share/work2/liuying/software/anaconda2/envs/anaCogent/lib/python2.7/site-packages/Cogent-3.1-py2.7.egg/Cogent/splice_graph.py", line 170, in contract_sinks
assert path_d[k][-1] == sink and path_d[k][-2] == pred
AssertionError

Hi @angeladarling ,
It looks like you are posting about a completely different issue. Please file through a new "issue" instead of using an existing unrelated issue.
Also, some of the bugs you see in 3.1 may be fixed in v3.3. Please update the v3.3 and try again. If the error remains, I can look at the input data for you.
Thanks,
--Liz


JSON Path support in dropdown rest

Currently we can use only a primitive syntax for accessing the Json elements in web-based calls. It would be good to support the syntax of JSONPath (for example: $.leaderboard[*].Team_tag).
Watch here: http://goessner.net/articles/JsonPath/.
Sincerely, OldCat, diligent user ;)

I support this idea

Thank you for the good suggestion. But existing syntax allows to solve most of real-life tasks (IMHO).


crutech

official url: https://crutech.edu.ng
IT related course: https://crutech.edu.ng/index.php/computer-science
Proof of email domain: https://crutech.edu.ng/images/Outlook_Results/outlook1.png

@zealnetworks Pull request merged. Thank you!


Added test project and some tests

I noticed you mentioned unit tests in https://github.com/radu-matei/websocket-manager/issues/13.
This contains the following changes:

Moved WebSocketManager.sln to the root directory, it contains the projects in the src and test folders.
Added test project for the WebSocketManager project
Added dotnet-watch tool to the test project, so tests may be run continously using dotnet watch test
Added some tests for the WebSocketConnectionManager class
Updated .travis.yml so it builds all projects and runs the tests

If you don't like how the tests are structured or something else, let me know 😄

Merged #20.
Thanks for your contribution, it is very much appreciated.
If you want to contribute further to this, please do so :D
Thanks!

Sure, just didn't want to add too much stuff before i knew whether it was acceptable or not 😄


walker.next() callback promise doesn't resolve

See below code example by @tbranyen:
var path = require('path');
var nodegit = require('./');

nodegit.Repository.open(path.resolve("./.git")).then(function(repo) {
 var walker = repo.createRevWalk();
 var commits = [];
 walker.pushHead();

 return walker.next(function(err, oid) {
   return repo.getCommit(oid).then(function(commit) {
      console.log('this is hit', commit);
      return commit;
   });
 }).then(function(commit) {
    console.log('this is never hit', commit);
 });
});

Walker.next doesn't currently generate a promise at all does it? It's based on an event emitter. I've been meaning to put in a helper function. Let me see if I have record of where I built that structure before.

@maxkorp, think the below code is what you had in mind (you wrote a few weeks back on Gitter).
I've been running into the issue where .then is never called – which basically is caused by this issue. @tbranyen and @orderedlist are currently being of great help on Gitter.
// in promise chain
.then(function(repo) {
  var walker = repo.createRevWalk();
  var innerPromises = [];
  return walker.next(walkCommit)
    .then(function() {
      return nodegit.Promise.all(innerPromises);
    });

  function walkCommit(commitOid) {
    if (commitOid) {
      innerPromises.push(repo.getCommit(commitOid));
      return walker.next(walkCommit);
    }
  }
})
.then(function(allOfTheCommits) {
  // do some stuff here
})
// continue the promise chain

Yeah, I had to rework that. Didn't see the gitter chat notifications though, I'll hop in.

The problem is that we currently do not allow mixing of callback-style and Promises.  You gotta go either one or the other.  In the case of this example, next was invoked with CPS (continuation-passing style) instead of as Promise, the fix is to use walker.next().then(function(oid).
I'm unsure if it makes sense to intermingle them by default.

I'm going to chalk this up as "not-a-bug", but definitely needs-documentation.

Yeah, intermixing doesn't really make sense.

Using walker.next().then(function(oid) (as demonstrated in the example https://github.com/nodegit/nodegit/blob/master/examples/general.js#L302-L316) fails with an error in case there are no more commits to walk. The method is never invoked with oid set to null or undefined, so i cannot "clean" terminate the walk.
Am I missing something?
(nodegit 0.6.3, node v4.1.2 on Windows, babel-register 6.3.13)

That may very well have changed :/ I'll look into it it later today and see if I cant figure out whats going on.

Is this truly an active issue? Is so, it'd be fun to contribute

So I was with NonCreature0714 yesterday when he commented. I've got the site generated (after updating dependencies), but I was hoping I could get some clarification on which docs need to be updated. The 'Revwalk#next' section updated with your `revwalk.next().then(function(oid) {


"helmfile -f helmfile.d/ diff --detailed-exitcode" output incomplete

Hello,
I have this annoying behaviour with helmfile diff --detailed-exitcode which might be a bug:
helmfile -f helmfile.d/ diff shows the full diff of all changes in different Helmfiles within the Helmfile-directory as expected.
helmfile -f helmfile.d/ diff --detailed-exitcode exits early with
identified at least one change, exiting with non-zero exit code (detailed-exitcode parameter enabled)

in helmfile.d/0-common.yaml: failed processing release cluster-autoscaler: helm exited with status 2:
  Error: identified at least one change, exiting with non-zero exit code (detailed-exitcode parameter enabled)
  Error: plugin "diff" exited with error

stopping after the diff of the first Helmfile in the directory with changes.
Any diffs in further Helmfiles are then not processed.
I would guess the problem might be how the Helmfile-directory is treated? The exit should probably be deferred until all Helmfiles have been processed?
It would be great if you could clarify the expected behaviour here.
helmfile version v0.85.2
Thanks for all your work on Helmfile 👍

@Pharb Hey! Thanks for reporting.

stopping after the diff of the first Helmfile in the directory with changes.

Good catch! I think this isn't an expected behavior. It should just match how it behaves without --detailed-exitcode
Labeling this as a bug.


Add a trainer abstraction to provide a default training loop.

Because most supervised models have similar training loops, to help users get
started more easily, this commit adds a sketch for what a pre-built training
loop might look like. This will be used to help flesh out the API surface
based on real usage. The API should not be assumed to be frozen!

Note: the implementation of the training loop should be completely re-written
to be callback driven. It also should have reasonable defaults for outputing
information to the console, and should have direct support for high
performance input pipelines. These are considered out of scope for the
initial check-in, in order to facilitate parallel development.

Note: about to add a test.

@saeta I'd like to continue work on this, Could I?

@Shashi456 Thanks a lot for offering to help! Please hold off on this because the library abstractions we rely on are not ready yet. We’ll file a GitHub issue with a more concrete and future-proof design later when the underlying library is more complete later this year.

@rxwei Alright, I'll continue with #54 and #127 then.


Add compile-address-map command

An address map is a file of the form
IPV4 {
psid_length=foo
shift=bar
}
...
Either of psid_length or shift can be given.  If neither is given, they
default to 0 and 16, respectively.
The idea is that we'll specify address maps apart from the binding
table.

Wouldn't be simpler to specify a Lua syntax compatible file that could be processed directly with loadstring or dofile? Something like:
"IPV4_1" = {
  psid_length=foo,
  shift=bar
},
"IPV4_2" = {
  psid_length=foo,
  shift=bar
},

parse opens the file, reads its content as a Lua table and converts it to the new format:
local function parse(file)
   local fd = io.open(file)
   local content = fd:read("*all")
   local t = loadstring(("return {%s}"):format(content))
   fd:close()

   local ret = {}
   for ipv4,psid_info in pairs(t) do
      table.insert(ret, convert_entry(ipv4, psid_info))
   end
   return ret
end

convert_entry would turn ipv4 and psid_info params into:
{host_endian_ipv4=uint32, psid_length=value, shift=value}

I don't know, it's just an idea. Other configuration files in Snabb tend to be Lua tables too (ports configuration files in SnabbNFV come to my head [1]). Is there a powerful reason (performance maybe? but the file is processed only once, right?) to have a customized parser for a file with a custom syntax? That's my concern.
[1] https://github.com/Igalia/snabbswitch/blob/lwaftr_olive/src/program/snabbnfv/test_fixtures/nfvconfig/test_functions/same_vlan.ports

Yeah I get what you're saying.  I wouldn't mind re-using Lua's parser if Lua had a way to parse a file without executing it.  However, with things as they are, I think using Lua's parser has a few down-sides:

Lua's parser can't give us good source locations for domain-specific parse errors.  Like if what you type is bad Lua syntax, you get an OK error (though probably with a domain-specific parser we could make a better error).  But if you type good Lua syntax but bad syntax for the conf file, you can't later produce a source location for the domain-specific parse error.  With a custom parser we can do this.
We are going to be having three configuration files, it seems (conf, binding table, address map), and it's easy to mix them up on the command line. Again, good parse errors can help users know what the error was.
It could be that Lua syntax isn't actually what we want.  For example in your example, it's easy to forget the comma between the entries.  It's also easy to forget the quotes around the IP address.  Neither of these syntactic bits are necessary but using Lua's parse requires them.
Of course, it is possible when using Lua's parser that loading a file produces some other error besides a parse error -- i.e. a runtime error, when evaluating the code.  This really bothers me, if you can't tell :)

Dunno.  I think going with our own grammar makes sense, if we can minimize the costs in terms of parser maintenance.  In that regard, I have another patch to clean up this one.  Let's see :)

I was kinda thinking that we should be able to specify the address map using CIDR notation.  Like:
<IP_ADDRESS>/24 {psid_length=4}

To do this we probably need to tweak the RangeMap builder to accept ranges.  Will work on that.

Alternately:
<IP_ADDRESS>-<IP_ADDRESS> { psid_length=4 }

Or even:
<IP_ADDRESS>-<IP_ADDRESS>, <IP_ADDRESS> { psid_length=4 }

On a practical level when we start simulating millions of entries in the binding table, it could be nice to have a readable address map at least.

@wingo: Agree with your comments. I think defining our own configuration syntax brings advantage. However, I was a bit worried about code maintenance of a new parser, specially if that happens for every configuration file we define. If the parser can be done with the less minimum code that would be the best solution for me.
I like the idea of ranges. If I have to pick one syntax over other I prefer the latter. Using CIDR makes me think there's a network address involved. The range makes explicit all the addresses within the range will have the same psid_length and shift values.

Newest patch adds support for address ranges; see the README diff.  Tx for comments @dpino, would you mind taking another look please?  Thanks :)

OK... thinking about this a little more, the compilation phase is pretty great but it's complicated to have to update a compiled address map.  The same will go for the binding table in the future.  My proposal is that:

We add on a magic header to the on-disk format of compiled address maps and binding tables.  This lets us know if we are loading a compiled file or a textual file.
The binding tables and address maps that the user specifies in the .conf file can be compiled or not.
If the lwAFTR goes to load a compiled address map or binding table, that's great -- you have a real low-latency restart.
If the lwAFTR realizes it's loading a textual address map or binding table, it will automatically try to look for a corresponding compiled version, by appending a ".o" extension to the input file name.
If it finds such a file, it checks its header to make sure it's a compiled file of the right type.  The magic header also embeds the mtime of the textual file, with nanosecond resolution.  If the header has the right magic value and the mtime embedded in it matches the mtime of the textual file, sweet, we use the compiled file.  Otherwise we automatically compile the file, and attempt to replace the contents of the .o file.  In this case we print a message saying what it's doing.  If we fail to replace the .o file, we issue a warning.  We use the newly compiled file in any case.

This is similar to what Guile does with the Scheme files it runs.  The advantage is, the user keeps their textual mental model, while the lwAFTR retains the ability to start up fast, and we minimize the possibility that a compiled file does not correspond to a textual file.
This probably isn't an amazing win for address maps, but I think it's probably important for binding tables, given the perf numbers we see in PodHashMap's self-test.

@andywingo +1 on the proposed method for config file compilation and loading.

LGTM, except for comments.

Fixed nits, going to see what travis does with this.  Thank you for the detailed review!

Buf, I managed to merge this into wip-psid.  I think that was because I initially created the PR before wip-psid was merged, and wanted only the relevant diffs.  Bummer!  Will create a new one and merge it.

Seems wip-psid isn't merged yet even, so I don't have to make a branch :) see #185.


Finance: Outlook Extra Info

Module: Finance
Lab/Demo: Draft an email to your insurance company using Copilot in Outlook
Task:
Step: 13 and 14
Description of issue
It is not clear what text to introduce. This text seems not to make a lot of sense. We need also to review ther next steps
Repro steps:

This was caused by a bad indentation messing up the instruction order. I have resolved.


[FLINK-33083] Properly apply ReadingMetadataSpec for a TableSourceScan

What is the purpose of the change
The PR fixes creating a TableSourceScan to also include a proper ReadingMetadataSpec which has been applied on the source.
Brief change log

revert the solution added in #22894 because it works around the outcome of the bug rather than fixes the bug
create a ReadingMetadataSpec when applying it on the TableSource in a TableSourceScan

Verifying this change

tests added in #22894 should still pass
added a dedicated test TableSourceJsonPlanITCase#testReadingMetadataWithProjectionPushDownDisabled

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

fyi @lincoln-lil @swuferhong I think we should be a bit more careful when fixing those kind of bugs so that we don't introduce hacks and workarounds


Hiding Base Reports

As we cannot extend base reports, we have to copy, customize and publish as our own report.  How can we then hide the base report, so that the end user is not confused as to which report to choose?
It would be great if we could minimally extend the base report to change the UsageCategory, Caption or something so that the end-user is not confused.

We have the same problem with pages which we do not want to show in Tell Me...

You can use this https://docs.microsoft.com/en-us/dynamics365/business-central/dev-itpro/developer/devenv-substituting-reports


[feat] add modal history feature

I find it will be much more convenient if the deck list can remember which entry the user have clicked before exiting the window. So I implemented this feature and have tested for several monthes. It works well so I do PR to the upstream.

Oh this is hella awesome! Nice work!


⚠ ClamAV has degraded performance

In a0d00d6, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 73 ms

Resolved: ClamAV performance has improved in 0f3b01e.


[SMALLFIX] Update timestamp of lost worker before becoming active

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/Tachyon-Pull-Request-Builder/6251/
Test PASSed.

I updated PR with a unit test. Thanks!

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/Tachyon-Pull-Request-Builder/6359/
Test PASSed.

@gpang Thanks for the PR. Left you some comments.

Thanks for the comments. I added your suggestions.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/Tachyon-Pull-Request-Builder/6364/
Test FAILed.

Jenkins, test this please.

LGTM

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/Tachyon-Pull-Request-Builder/6366/
Test PASSed.

LGTM


Craft a new UI window/layout for Tasks viewing

Implemented in #102  and can be closed

It looks good! Thanks for the work Jon!


pkg_resources.DistributionNotFound: boto3>=1.1.1

Seems like this is telling me I don't have the correct version of boto3 library, but:
version = '1.4.4'
What am I doing wrong?
rlauer@centos6 ~]$ sudo credstash setup
Traceback (most recent call last):
File "/usr/bin/credstash", line 5, in
from pkg_resources import load_entry_point
File "/usr/lib/python2.6/site-packages/pkg_resources.py", line 2655, in
working_set.require(requires)
File "/usr/lib/python2.6/site-packages/pkg_resources.py", line 648, in require
needed = self.resolve(parse_requirements(requirements))
File "/usr/lib/python2.6/site-packages/pkg_resources.py", line 546, in resolve
raise DistributionNotFound(req)
pkg_resources.DistributionNotFound: boto3>=1.1.1

@rlauer6, I just encountered the same issue. What I found, the issue wasn't on boto3 but on cryptography. When you ran pip install cryptography, you most likely pulled in >2.x. Currently, credstash requires cryptography<2.0,>=1.5.

pkg_resources.DistributionNotFound: The 'cryptography<2.0,>=1.5' distribution was not found and is required by credstash

Try downgrading cryptography to 1.5.x. It worked for me.


MM-3190 show user limit modal when cloud limit exceed

Summary
Shows the limit user modal to admin when they click the invite users in the welcome text channel and there are no more free seats.
Ticket Link
https://mattermost.atlassian.net/browse/MM-31905
Screenshots

Given that we may be adding entry points to the Invite Users screens in other places in the near future, does it make sense to move the logic of checking for the limit and showing the upgrade modal within the actual invite modal? This way we avoid duplicating this logic whenever we need to open the Invite People screen

+1

Given that we may be adding entry points to the Invite Users screens in other places in the near future, does it make sense to move the logic of checking for the limit and showing the upgrade modal within the actual invite modal? This way we avoid duplicating this logic whenever we need to open the Invite People screen

+1

yes, that was the latest update I did to the PR. @nickmisasi , @marianunez could you please take a look again?

Tested and Passed
The link to invite users at the bottom of the Town Square channel opens the 'You've reached the user limit' modal as expected. The modal also worked and is dismissed in all fashions as you would expect.
I did notice that on first use the share links appear for a flash before going away and the modal showing up. It only happens on the first click after opening the Spinwick but after speaking with @pablovelezvidal about it he's aware and he assured me that it won't be a problem once merged so this one is good to go!

To clarify what @stevemudie mentions, the PR that contains the necessary code so the flash don't happen is this one https://github.com/mattermost/mattermost-webapp/pull/7468, there is the code where we load the subscriptionStats on app initial boot.

/update-branch

/cherry-pick cloud-2021-02-16

Cherry pick is scheduled.


Can't set negative Axis value for DualShock 4 controller

Describe the bug
If I use controller.SetAxisValue, can't set negative axis value for DualShock4Axis.RightThumbX, just positive. How can I move the thumb sticks to the opposite direction? The SetAxis value's second parameter is byte.
To Reproduce
Use controller.SetAxisValue(DualShock4Axis.RightThumbX, -10);
Expected behavior
Can set positive and negative values as well for Axis

-10 is an invalid value for the DS4 stick axis. The range of the value (as reported by the hardware) is a full byte [0-255] with RS Left represented as [0-127], the neutral value represented as 128, and RS Right as [129-255]. It looks like you would actually want to pass value 118 as the second parameter to set the value 10 units to the left of neutral. The API is working as intended.

-10 is an invalid value for the DS4 stick axis. The range of the value (as reported by the hardware) is a full byte [0-255] with RS Left represented as [0-127], the neutral value represented as 128, and RS Right as [129-255]. It looks like you would actually want to pass value 118 as the second parameter to set the value 10 units to the left of neutral. The API is working as intended.

-10 is an invalid value for the DS4 stick axis. The range of the value (as reported by the hardware) is a full byte [0-255] with RS Left represented as [0-127], the neutral value represented as 128, and RS Right as [129-255]. It looks like you would actually want to pass value 118 as the second parameter to set the value 10 units to the left of neutral. The API is working as intended.

Oh, thank you very much for response. It's working now. I couldn't find anywhere the right values :)
Awesome library by the way, thank you!

-10 is an invalid value for the DS4 stick axis. The range of the value (as reported by the hardware) is a full byte [0-255] with RS Left represented as [0-127], the neutral value represented as 128, and RS Right as [129-255]. It looks like you would actually want to pass value 118 as the second parameter to set the value 10 units to the left of neutral. The API is working as intended.

Oh, thank you very much for response. It's working now. I couldn't find anywhere the right values :)
Awesome library by the way, thank you!

I take the liberty to close this one 😉


Docs site is not working

https://react.semantic-ui.com/ site is not working due to such errors in console
Refused to execute script from 'https://raw.github.com/Semantic-Org/Semantic-UI-React/gh-pages/app.0de29eaca1610b4cf71c.js' because its MIME type ('text/plain') is not executable, and strict MIME type checking is enabled.

Fix coming, hang tight.

Fixed, thanks for the quick report.


Create tserver.group, add group to location information

This change includes:

Adding TSERV_GROUP_NAME property to add resource groups to TabletServers like has been done for Compactors and ScanServers
Modified TServerInstance to include the group in the location information.
Modified TabletMetadata to remove the session from the location columns colq and put all of the location information in the column value.
Modified MetricsUtil to add the servers resource group as a tag to all emitted metrics
Modified CachedTablet to use the TServerInstance instead of interned strings for the server and session.

Closing in favor of the implementation in #3496