Is it possible to increase MTU size?

Hi,
In file btlib.c the LEDATLEN is defined as 20. Does that mean the MTU size, right?
I want to send and receive data up to 128 byte. Is it possible? If yes how?
I changed LEDATLEN to 128, and in my nRF52840 firmware also, but it sends/receives still only 20 byte.
I think in btlib.c lots of arraysizes are hardcoded with 20.

Change LEDATLEN to 128, and set SIZE=128 in the devices file LECHAR entry. The write_ctic code will then send a 128-byte packet. The only extra step that may be necessary is to tell the nRF that the MTU has changed from the default by calling the following function. Put it in btlib.c and call it immediately after connection with the node number of the nRF.
int setmtu(int node)
  {
  int ndevice;

  ndevice = devnp(node);
  if(ndevice < 0)
    return(0);

  mtuset[14] = (unsigned char)((LEDATLEN+3) & 0xFF);
  mtuset[15] = (unsigned char)(((LEDATLEN+3) >> 8) & 0xFF);
  sendhci(mtuset,ndevice);
  readhci(0,0,0,200,0);
  return(1);
  }

If you turn on verbose mode, you should see an opcode=03 reply from the nRF. The last two bytes are the MTU that the nRF will accept, and if they are 17 00 then LEDATLEN cannot be increased beyond 20 - they should be equal to the MTU you set on the nRF. I tried this between two Pis set up as client/server and the larger packet is transmitted OK.

Thank you,
I try it now, but the connection is always closed.
Log after connection:
`
SEND LE connect to CO2 Sensor Module
Set [10].. board address reversed 05..E0
< HCI OGF=08 OCF=0D
0000  01 0D 20 19 60 00 60 00 - 00 01 05 C6 27 97 D0 E0
0010  00 18 00 28 00 00 00 11 - 01 00 00 00 00

Event 0F = 00 01 0D 20
0000  04 0F 04 00 01 0D 20
Event 3E = 01 00 40 00 00 01 05 C6 27 97...
0000  04 3E 13 01 00 40 00 00 - 01 05 C6 27 97 D0 E0 27
0010  00 00 00 11 01 00
Connect OK as LE client
Handle = 0040
Set [1][2] handle 40 00
< L2CAP 0004 Opcode = 02
0000  02 40 00 07 00 03 00 04 - 00 02 83 00
L2CAP 0004 Opcode = 03
0000  02 40 20 07 00 03 00 04 - 00 03 F7 00
Event 13 = 01 40 00 01 00
0000  04 13 05 01 40 00 01 00
Reading LE services from CO2 Sensor Module..
SEND read UUID (opcode 08)
Set [10][11] starting ctic handle 0001
Set [14][15] UUID 2803
Set [1][2] handle 40 00
< L2CAP 0004 Opcode = 08
0000  02 40 00 0B 00 07 00 04 - 00 08 01 00 FF FF 03 28
SEND read UUID (opcode 08)
Set [10][11] starting ctic handle 0002
Set [14][15] UUID 2803
Set [1][2] handle 40 00
< L2CAP 0004 Opcode = 08
0000  02 40 00 0B 00 07 00 04 - 00 08 02 00 FF FF 03 28
Event 13 = 01 40 00 02 00
0000  04 13 05 01 40 00 02 00
L2CAP 0004 Opcode = 09
0000  02 40 20 30 00 2C 00 04 - 00 09 07 02 00 20 03 00
0010  05 2A 05 00 0A 06 00 29 - 2B 07 00 02 08 00 2A 2B
0020  0A 00 02 0B 00 00 2A 0C - 00 02 0D 00 01 2A 0E 00
0030  02 0F 00 04 2A
SEND read UUID (opcode 08)
Set [10][11] starting ctic handle 0010
Set [14][15] UUID 2803
Set [1][2] handle 40 00
< L2CAP 0004 Opcode = 08
0000  02 40 00 0B 00 07 00 04 - 00 08 10 00 FF FF 03 28
L2CAP 0004 Opcode = 09
0000  02 40 20 45 00 41 00 04 - 00 09 15 11 00 12 12 00
0010  48 FC 85 75 90 C7 93 92 - 2D 48 9B E1 02 00 EA E9
0020  14 00 04 15 00 48 FC 85 - 75 90 C7 93 92 2D 48 9B
0030  E1 03 00 EA E9 16 00 04 - 17 00 48 FC 85 75 90 C7
0040  93 92 2D 48 9B E1 04 00 - EA E9
SEND read UUID (opcode 08)
Set [10][11] starting ctic handle 0018
Set [14][15] UUID 2803
Set [1][2] handle 40 00
< L2CAP 0004 Opcode = 08
0000  02 40 00 0B 00 07 00 04 - 00 08 18 00 FF FF 03 28
Event 13 = 01 40 00 02 00
0000  04 13 05 01 40 00 02 00
L2CAP 0004 Opcode = 01
Error 0A = Attribute Not Found
0000  02 40 20 09 00 05 00 04 - 00 01 08 18 00 0A

Characteristics saved to device info
ctic
index       LE Characteristics
0  Service Changed      ? byte Permit 20 i   Handle=0003 UUID=2A05
1  Client Supported Features          1 byte Permit 0A rwa Handle=0006 UUID=2B29
2  Database Hash        16 byte Permit 02 r   Handle=0008 UUID=2B2A
3  Name                 17 byte Permit 02 r   Handle=000B UUID=2A00
4  Appearance           2 byte Permit 02 r   Handle=000D UUID=2A01
5  Pref Connection Parameters         8 byte Permit 02 r   Handle=000F UUID=2A04
6  E9EA0002E19B482D9293C7907585FC48   128 byte Permit 12 rn  Handle=0012
UUID=E9EA0002E19B482D9293C7907585FC48
7  E9EA0003E19B482D9293C7907585FC48   ? byte Permit 04 w   Handle=0015
UUID=E9EA0003E19B482D9293C7907585FC48
8  E9EA0004E19B482D9293C7907585FC48   ? byte Permit 04 w   Handle=0017
UUID=E9EA0004E19B482D9293C7907585FC48
Write CO2 Sensor Module E9EA0003E19B482D9293C7907585FC48 = 00 00 48 44
SEND write LE characteristic
Set [3][4] [5][6] packet lengths
Set [9] opcode 52
Set [10][11] characteristic handle 0015
Set [12].. 04 data bytes
Set [1][2] handle 40 00
< L2CAP 0004 Opcode = 52
0000  02 40 00 0B 00 07 00 04 - 00 52 15 00 00 00 48 44
Write CO2 Sensor Module E9EA0004E19B482D9293C7907585FC48 = 00
SEND write LE characteristic
Set [3][4] [5][6] packet lengths
Set [9] opcode 52
Set [10][11] characteristic handle 0017
Set [12].. 01 data bytes
Set [1][2] handle 40 00
< L2CAP 0004 Opcode = 52
0000  02 40 00 08 00 04 00 04 - 00 52 17 00 00
read index: 6, write index: 7, ctrldata index: 8 data: 0
Read LE characteristic
Set [10][11] handle 0012
Set [1][2] handle 40 00
< L2CAP 0004 Opcode = 0A
0000  02 40 00 07 00 03 00 04 - 00 0A 12 00

Event 13 = 01 40 00 02 00
0000  04 13 05 01 40 00 02 00
L2CAP 0004 Opcode = 0B
0000  02 40 20 45 00 41 00 04 - 00 0B 42 42 42 42 42 42
0010  42 42 42 42 42 42 42 42 - 42 42 42 42 42 42 42 42
0020  42 42 42 42 42 42 42 42 - 42 42 42 42 42 42 42 42
0030  42 42 42 42 42 42 42 42 - 42 42 42 42 42 42 42 42
0040  42 42 42 42 42 42 42 42 - 42 42
BBBBBBBBBBBBBBBBBBBBBBBBBBBBBBBBBBBBBBBBBBBBBBBB

Disconnecting everything and exit..
< HCI OGF=08 OCF=0A
0000  01 0A 20 01 00

Event 13 = 01 40 00 01 00
0000  04 13 05 01 40 00 01 00
L2CAP 0005 ? =  12 02 08 00 18 00 28 00 00 00 2A...
0000  02 40 20 10 00 0C 00 05 - 00 12 02 08 00 18 00 28
0010  00 00 00 2A 00
Event 0E = 01 0A 20 0C
0000  04 0E 04 01 0A 20 0C
SEND Close connection
Set [4][5] handle 40 00
< HCI OGF=01 OCF=06
0000  01 06 04 03 40 00 13
Event 0F = 00 01 06 04
0000  04 0F 04 00 01 06 04
CO2 Sensor Module has disconnected
Event 05 = 00 40 00 16
0000  04 05 04 00 40 00 16
GOT Disconnected OK (Event 05)
HCI closed
`

Second question:
Is there any callback for client? My nRF works as bt-server and I connect to it with Raspberry Pi using btferret library.
When I reset the server (nRF), I want to be informed for disconnect event. Is it possible?
Thanks.

The characteristic scan is finding the 128-byte entry correctly, and is reading the 64 bytes that the nRF has chosen to send, so this seems to be working OK. It looks like the 128 length might be coming from the devices.txt file, in which case an extra step may be needed. With my nRF52840 I have found that a data length command must be sent for the Pi to see a 128-byte entry correctly, so the modified version of setmtu is now:
int setmtu(int node)
  {
  int ndevice;
  static unsigned char setdatlen[16] = {10,0,S2_HAND,0,1,0x22,0x20,6,0x40,0,0x1B,0,0,2};

  ndevice = devnp(node);
  if(ndevice < 0)
    return(0);

  mtuset[14] = (unsigned char)((LEDATLEN+3) & 0xFF);
  mtuset[15] = (unsigned char)(((LEDATLEN+3) >> 8) & 0xFF);
  sendhci(mtuset,ndevice);

  setdatlen[10] = (unsigned char)((LEDATLEN+3) & 0xFF);
  setdatlen[11] = (unsigned char)(((LEDATLEN+3) >> 8) & 0xFF);
  sendhci(setdatlen,ndevice);

  readhci(0,0,0,200,0);
  return(1);
  }

Test this by temporarily removing the characteristic from devices.txt. On my system, the old version of setmtu results in a reported size of 22 bytes when a services scan is run, but the new version sees and reads/writes 128 bytes.
In the original btferret.c code, the "Disconnecting everything.." message only appears because btlink() has terminated, so the cause of the disconnect is in btlink().
If the readhci() function is running, it will see a disconnect (line 5403 in btlib.c). But readhci does not run in the background, so a disconnect can only be seen by polling. The read_notify function can be used for this because it just runs readhci for the specified time. If a disconnect happens when readhci is not running, the info will be saved in a buffer and readhci will see it when it does run, so for example:
read_notify(50);    // run readhci for 50ms - it will see any past or current disconnects
if(device_connected(node) == NO_CONN)
  printf("Device disconnected\n");

OR just call readhci directly

readhci(0,0,0,50,0);    // 50ms time out

The ability to increase the characteristic size beyond 20 for the local and remote devices looks so useful that I will incorporate the MTU-setting functions into btlib so they will run automatically. Hope to release this new version within a week.

Thank you.
Can you please share also the configuration of nRF board? Somehow I couldn't set the mtu, I have always 20 Byte.
If I want to have 128 byte MTU, I should write a number less then 128. I don't rememeber exactly.
CONFIG_BT_L2CAP_TX_MTU=119

The MTU needs to be data length + 3. The new version 7 sets the MTU to whatever value the nRF requests, and can now read/write 128 byte characteristics set up by nRF Connect Desktop/Low Energy/Server Setup option which seems to always request the maximum MTU of 251. I have not written nRF code to set the MTU.

// ATT_MTU data cannot be more than 244  //line#43
#define LEDATLEN 244
while the spec BLUETOOTH SPECIFICATION Version 5.0 | Vol 3, Part F specifies (on page 2178).

The maximum length of an attribute value shall be 512 octets.

Can I set LEDATLEN to 512? Or did I misunderstood something?


Reproducibility for H2O4GPU Factorization

Can we add a seed to ensure the Factorization is reproducible?
(I will do some tests and share results)

@sh1ng numpy's RandomState gives exact same result across all versions (>0.8 or something, which is quite old) and OS. I tested a few versions across Mac, Linux and Windows and they give same results.
So this should work.


Add a metadata field for when source data was produced

Is your feature request related to a problem? Please describe.
The current problem is that information about when source data was produced is not easily visible. Though there are a variety of metadata fields available in the dataset viewer, time period information is not included. This feature request suggests making metadata relating to the time that the underlying source data was produced more prominent and outlines why this specific information is of particular importance, both in domain-specific historic research and more broadly.
Describe the solution you'd like
There are a variety of metadata fields exposed in the dataset viewer (license, task categories, etc.) These fields make this metadata more prominent both for human users and as potentially machine-actionable information (for example, through the API). I would propose to add a metadata field that says when some underlying data was produced. For example, a dataset would be labelled as being produced between 1800-1900.
Describe alternatives you've considered
This information is sometimes available in the Datacard or a paper describing the dataset. However, it's often not that easy to identify or extract this information, particularly if you want to use this field as a filter to identify relevant datasets.
Additional context
I believe this feature is  relevant for a number of reasons:

Increasingly, there is an interest in using historical data for training language models (for example, https://huggingface.co/dbmdz/bert-base-historic-dutch-cased), and datasets to support this task (for example, https://huggingface.co/datasets/bnl_newspapers). For these datasets, indicating the time periods covered is particularly relevant.
More broadly, time is likely a common source of domain drift. Datasets of movie reviews from the 90s may not work well for recent movie reviews. As the documentation and long-term management of ML data become more of a priority, quickly understanding the time when the underlying text (or other data types) is arguably more important.
time-series data: datasets are adding more support for time series data. Again, the periods covered might be particularly relevant here.

open questions

I think some of my points above apply not only to the underlying data but also to annotations. As a result, there could also be an argument for encoding this information somewhere. However, I would argue (but could be persuaded otherwise) that this is probably less important for filtering. This type of context is already addressed in the datasheets template and often requires more narrative to discuss.
what level of granularity would make sense for this? e.g. assigning a decade, century or year?
how to encode this information? What formatting makes sense
what specific time to encode; a data range? (mean, modal, min, max value?)

This is a slightly amorphous feature request - I would be happy to discuss further/try and propose a more concrete solution if this seems like something that could be worth considering. I realise this might also touch on other parts of the 🤗 hubs ecosystem.

A question to the datasets maintainers: is there a policy about how the set of allowed metadata fields is maintained and expanded?
Metadata are very important, but defining the standard is always a struggle between allowing exhaustivity without being too complex. Archivists have Dublin Core, open data has https://frictionlessdata.io/, geo has ISO 19139 and INSPIRE, etc. and it's always a mess! I'm not sure we want to dig too much into it, but I'm curious to know if there has been some work on the metadata standard.

Metadata are very important, but defining the standard is always a struggle between allowing exhaustivity without being too complex. Archivists have Dublin Core, open data has frictionlessdata.io, geo has ISO 19139 and INSPIRE, etc. and it's always a mess! I'm not sure we want to dig too much into it, but I'm curious to know if there has been some work on the metadata standard.

I thought this is a potential issue with adding this field since it might be hard to define what is general enough to be useful for most data vs what becomes very domain-specific. Potentially adding one extra field leads to more and more fields in the future.
Another issue is that there are some metadata standards around data i.e. datacite, but not many aimed explicitly at ML data afaik. Some of the discussions around metadata for ML are also more focused on versioning/managing data in production environments. My thinking is that here, some reference to the time of production would also often be tracked/relevant, i.e. for triggering model training, so having this information available in the hub would also help address this use case.

Related: https://github.com/huggingface/datasets/issues/3877

Also related: the Data Catalog Vocabulary - DCAT standard will be discussed in a new Working Group at the W3C: https://www.w3.org/2022/06/dx-wg-charter.html


Added the bisection method

great! :clap:

signbit must be written as std::signbitto compile in gcc

Fixed :) Now this pull request passes all checks.

Also some docs have been clarified

Merge?

by the way some clue why the problem C fails ? :disappointed:

There are rounding problems. I am working on that.

There are serious rounding problems, f(1000) should return 0 in a third test case.
When points are scaled by 2^20 using double returns 12816.7 and using long double returns 8.74445. Some techniques can be used to prevent these errors like this one https://en.wikipedia.org/wiki/Kahan_summation_algorithm but it still seems not working.


BATS and APIv2: more tests and tweaks

(minor): apiv2 tests: check for full ID
Observation made while reviewing #6461: tests were checking
only for a 12-character container/image ID in return value.
It's actually 64, and we should test for that. This should
also minimize confusion in a future maintainer.

podman pause/unpause: new test
Runs a 'date/sleep' loop, pauses container, sleeps 3s,
restarts, then confirms that there's a 3- to 6-second
gap in the logs for the container.

podman healthcheck: new test
run a container with healthcheck, test both healthy
and unhealthy conditions

podman pod: check '{{.Pod}}' field in podman ps
Hey, as long as we have a pod with two running
containers, might as well confirm that 'podman ps'
returns the expected pod ID.

Signed-off-by: Ed Santiago<EMAIL_ADDRESS>

/lgtm

/approve


Fix bug in src/enUS.json

Fixed fail in the translation of large image description and rest description.

Thank you for your contribution!


Uncaught TypeError: window.parse_fasta is not a function

Loading a fasta file doesn't work on the latest code from the main branch. I see these errors in Chrome developers view:
home.js:111 Uncaught TypeError: window.parse_fasta is not a function
    at Object.load (home.js:111)
    at FileReader.reader.(:5000/anonymous function) (http://<IP_ADDRESS>:5000/static/js/filereader.js:347:31)

Ignore, browser had cached out-of-date js files.


Abstract out resource pruning code to reuse it in kpt fn render and kpt fn eval

As part of implementing resource pruning for kpt fn render, a concern was raised about having common resource pruning code between kpt fn eval and kpt fn render. There isn't a straight forward to share this code in the current implementations. I am filing this issue to record it as a technical debt for the project.

This is no longer needed.


Group number

What does the group number mean, described in task 1?

Are you talking about the boundaries?

In the boundaries, I think it is referring to the homework group number, as explained here https://github.com/ubsuny/23-Homework6G1/issues/11

@WildJimmy Thank you in advance.

It is your homework group number

Why do use "Homework6g1 should have boundaries [0,10] for the first function, [0. pi] for the second, and [-1,1] for the third. Homework6g1 should have boundaries [0,20] for the first function, [0. 2pi] for the second, and [-1,1] for the third, and so on"

This is correct.


v3.7.1: remove anchor in images

Issue: #110

Beta released<EMAIL_ADDRESS>


new -bk command does not save inside the newly created backup directory

In raising this issue, I confirm the following:
{please fill the checkboxes, e.g: [X]}

[X] I have read and understood the contributors guide.
[X] The issue I am reporting can be replicated.
[] The issue I am reporting can be is directly related to the pivpn installer script.
[] The issue I am reporting isn't a duplicate (see FAQs, closed issues, and open issues).

Issue
the new command -bk creates a new directory "/home/pi/pivpnbackup/" but the created .tgz file is not saved inside this directory but also in "/home/pi/"
Have you searched for similar issues and solutions?
(yes/no / which issues?)

Console output of      curl -L install.pivpn.io | bash
  Output Here

Console output of      pivpn add or pivpn add nopass
  Output Here

Console output of      pivpn debug
  Output Here

Have you taken any steps towards solving your issue?
  which?

will take a look at that as soon as i have a bit of time. thanks for reporting.

Hi,
I was probably veeeery drunk when i did the backup script.. i don't even know how it worked in first place so i could push it...
For that i have to apologize for the really bad code i've pushed to a stable branch.
I have now pushed a fix as well as support for the new wire-guard setup to test branch.
Commit: dd6bb069f05559fb626ee9b1bb398df07ff45206
I won't be pushing it to master as it has a bunch of changes and tbh its not worth the effort at this point.
Test branch will become the new master quite soon, just a bit more patience please.

@4s3ti  happy new year started a little early for you ;-)

@TinCanTech that's the only logical reason! xD


Promote some Util functions to be extensions

Turns out if you use the Convert Parameter to Receiver refactoring you get some this. lint to clean up!


docs: fix MAX_CONTENT_LENGTH calc to match description

description below says the value is in megabytes, so it should be * 1024 instead of * 1000?

Checklist:

[ ] Add tests that demonstrate the correct behavior of the change. Tests should fail without the change.
[x] Add or update relevant docs, in the docs folder and in code.
[ ] Add an entry in CHANGES.rst summarizing the change and linking to the issue.
[ ] Add .. versionchanged:: entries in any relevant code docs.
[ ] Run pre-commit hooks and fix any issues.
[ ] Run pytest and tox, no tests failed.

This change isn't needed. The 1000 vs 1024 issue is not worth it.

The next line in docs says "The code above will limit the maximum allowed payload to 16 megabytes.", which is not this number. Hence this PR to fix the discrepancy.

You're thinking of mebi (1024) vs mega (1000), it's correct as-is. But personally I don't care about the distinction.


Avoid send empty body to ext_proc server if decodeData() never called

If decodeData() is not called, currently ext_proc filter is still possible to send an empty body to ext_proc server with
https://github.com/envoyproxy/envoy/blob/b8853dfe3c7a4248eefa0e6b0dcb667a1b6ac748/source/extensions/filters/http/ext_proc/ext_proc.cc#L480
and
https://github.com/envoyproxy/envoy/blob/b8853dfe3c7a4248eefa0e6b0dcb667a1b6ac748/source/extensions/filters/http/ext_proc/ext_proc.cc#L565
After body mutation from ext_proc server is received by ext_proc filter, it will modify the decoding buffer, which leads to an ASSERT() in filter manager
common/http/filter_manager.cc:426] assert failure: parent_.state_.latest_data_decoding_filter_ == this
The condition to trigger this ASSERT is ext_proc filter is configured to send body, but client only sends headers and trailers.

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

The solution is to add a bufferedData() check before sending body to ext_proc server.  bufferedData() will only become valid pointer if decodeData() is every called.
If no data received, and decodeData() is never called, it will be nullptr.

/assign @tyxia @yanavlasov

/assign @stevenzzzz

test_ext_proc copy.odt

The root cause of the issue is this:
If ext_proc filter is configured to send body, and if only headers and trailers are received, no body is received by the ext_proc filter, it will send an empty boy to the ext_proc server.
The ext_proc server will mutate the body, and send back a body response. The body response will modify the decoding buffer, which trigger the ASSERT there:
https://github.com/envoyproxy/envoy/blob/522f991095db1985ddaec656de53f3931fbf5d27/source/common/http/filter_manager.cc#L426
So, filter_manager carefully maintain state_.latest_data_decoding_filter_ during decodeData():
https://github.com/envoyproxy/envoy/blob/522f991095db1985ddaec656de53f3931fbf5d27/source/common/http/filter_manager.cc#L658
In this case, decodeData() is never called due to client is not sending data, and data is constructed by ext_proc filter itself. That leads to latest_data_decoding_filter_ is an empty pointer not pointing the ext_proc filter
Thus the ASSERT failed,  and crashed there.
Overall, the ext_proc filter should not send data based on the ext_proc filter configuration. It should only send data if the data presents.  Concept wise, this issue is similar to sending trailer when trailer is not present case : https://github.com/envoyproxy/envoy/pull/28592

/assign @mattklein123

Kind Ping

The "skipping body" approach in this PR feels like a behavior change that we can avoid by calling de/encodeData(dataMutations, false); on receiving the data mutation response from ext_server.
But no strong feelings.

Thanks for the comments!
The ext_proc behavior is now changed into "if configured as such, deliver body/trailer to the ext_proc server if they are present".  I am not sure what's the original intent behavior, but "if configured as such, deliver body/trailer even though they are missing" does not sound right, also is not supported as we can see here.

@envoyproxy/api-shepherds PTAL


Add a preset setting to forbid specific ItemSources from holding progression

Currently, can_contain_progression is a setting in the ItemSource JSON and cannot be configured outside of modifying the code.
It would be very good to have a setting to make this configurable inside the preset instead.

Added in c9671e7


Unable to run in windows terminal

The error when run in windows terminal is the following...
C:\Users\Knowi\Desktop\Flock\main.py:2: DeprecationWarning: The distutils package is deprecated and slated for removal in Python 3.12. Use setuptools or check PEP 632 for potential alternatives   from distutils.errors import LibError Traceback (most recent call last):   File "C:\Users\Knowi\Desktop\Flock\main.py", line 4, in <module>     from cryptography.fernet import Fernet ModuleNotFoundError: No module named 'cryptography'
Planned to fix in ALPHA Build #2 (v0.1.0)

This is very simple to solve. Do the following to insure functionality.
py -m pip install cryptography       py -m pip install colorama


f32 and f64 additive features

Another option. I wanted to see what it would take to have both f32 and f64 function at the same time.
This makes each implementation available under bevy_xpbd_3d::single::XpbdPlugin and bevy_xpbd_3d::double::XpbdPlugin. If only one of them is defined, adds re-exports at the crate level, i.e. bevy_xpbd_3d::XpbdPlugin.
Not sure what I think of it though, but just thought I'd share.

Interesting idea, haven't seen it before. This does maybe feel a bit hacky though, and seems to basically accomplish the same thing as making separate crates for f32 and f64, except with features and modules

Yeah, definitely feels hacky.
Another option would be to use generics, so XpbdPlugin<f32> for instance, and then use associated types for everything (XpbdPlugin<f32>::Vector would be Vec3)... if it's even doable it would probably be a lot less readable, and everything would probably have to go into impl blocks.


A Calendar Feature to sync w/ Events

A calendar on OC's homepage where we post our monthly forum dates/our staff's speaking engagements
From slack convo:

@kaylarep
this could even be scaled to each collective/org's profile where their Events could be viewed on a calendar
@alanna
Just like the awesome idea to have a big feed of all Updates across the platform, how about a place that shows events? i know this is much bigger than just showing our events, but it jumped to mind.
I also think a calendar with our own events is a good idea
@Betree
As for showing all updates across the platform, an important problem when doing that is SPAM. SPAM would be everywhere for such pages.
Maybe a good first step on this topic would be to show all updates/events for a given host?
@alanna
couldn’t we solve that with filters based on only showing updates from collectives that meet certain thresholds of activity?
@Betree
We could, yes
@kaylarep
if we really want to expand this, we could do a lot with it... I personally love using events calendars, and having filter options (e.g. in-person/virtual, location, free, tags)

from Caroline: --> make a good calendar that shows ALL events across ALL hosts and collectives so as a user I can see what events are happening (especially those tagged with education) and can attend.


Crash caused by Unwhistling with a Goat Horn with "Use particle effects instead of status messages" enabled

Consistent serverside crash when trying to spawn particle effects when unwhistling with a goat horn. Version 1.21.4

Oh, looks like there was a breaking change in 1.21.4 that I didn't catch.
Really simple fix,1.4.4 is out!
P.S.
Nice to know people are actually using the goat horn feature!


New "process start date" and "process end date" data items

Currenty OBI has a "date process started" [http://purl.obolibrary.org/obo/OBI_0002471] measurement datum, which carries semantics of being about a process that has started in the past.  A few open issues: #898 and #899 have suggested "has start time" "has end time" as data properties, but this approach, though it allows for future time points to be expressed, depends on expanding the list of OBI data properties, which many are averse to. (Is EuPath using these?). This would also help close #981 .
Can we have a more abstract "process start date" data item, which may reference a past or future (intended) date?  Similarly, a "process end date" to capture real or intended completion date.  Process duration can then be calculated from these, or two of the three can be used to calculate the remaining dependency. (Specific values can be attached to instances of these via 'has representation' or 'has value specification' as they arise).  I'd like the definitions to embrace past observations but also simulations and predictions.
label: 'process start date'
definition: 'A data item that is the date when a process was initiated or is forecast to be initiated'
proposed parent: 'data item'
Likewise,
label: 'process end date'
definition: 'A data item that is the date when a process was ended or is forecast to end'
(Later we could introduce a 'time datum' class to put these, and age, and other classes of time datums in.)

Discussed on OBI call July 15, 2019. Would like to see logical definition started. How does this relate to date of birth?
If the data item is to apply to something to be initiated then need to relabel. Process start date. Process end date.
Should look at RO and how this is addressed. Annotation property: 'has start time value' and 'has end time value'

We could include the temporal examples at http://oupr.inference.me:8080/ in our analysis.

Here's a diagram of the axiomization that seems to fit with BFO ISO.

And here's a diagram of birth day and age datum.  "postnatal age datum" could be called "age since birth datum".  There are all sorts of OBI (new) terms shown there to support a full axiomatization of "postnatal age datum".  I put them in there just to show where this could go in the context of significant and anticipated prenatal and natal dates too.

Discussed on OBI call 9/9/2019. The next step is to determine which terms from these diagrams that would be useful to add to OBI.

A follow-up diagram based on today's discussion. Name changed to "age since birth datum"; addition of "aging process", revising "postnatal history" to "postnatal process".

I think we can close this.  Process start/end times can be handled by time ontology.


Update Library to use android.browser-1.2.0

The version of the androidx.browser that android-browser-helper depends has reached stable, with some minor API changes.
We need to update android-browser-helper with those changes, as a path for version 1.0.

Closed via #48


Fixes up to the latest VSF

@kwreczycki thanks for taking the time to look into this. :100:
I'd like to work on this a bit more before a merge if you don't mind, for one thing I need to figure out what's wrong with the nested definitions in ES (removing the empty annotations fixes it, but they don't end up in the index).
As we're planning to use this for our VSF integration, I'll likely work extensively on this repo, hopefully you or some other owner will be around to review in the future too. :beers:

The issue with nested definitions is they would need to be nested (which makes sense, since categories are nested), which ONGR doesn't support, and neither does ES really.
If we leave it as is, it works, but the data for children_data is not ingested into Elasticsearch. I'm not sure if that's an issue?

This works for me currently as is, the Coreshop demo data is showing up in VSF and it's browseable.


Add a warning bar to show warning messages

Codecov Report
Base: 86.52% // Head: 85.50% // Decreases project coverage by -1.02% :warning:

Coverage data is based on head (8e1992f) compared to base (5924c64).
Patch coverage: 68.51% of modified lines in pull request are covered.

:exclamation: Current head 8e1992f differs from pull request most recent head d01a1f4. Consider uploading reports for the commit d01a1f4 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #104      +/-   ##
==========================================
- Coverage   86.52%   85.50%   -1.03%
==========================================
  Files          79       80       +1
  Lines        2925     3021      +96
  Branches      449      309     -140
==========================================
+ Hits         2531     2583      +52
- Misses        303      338      +35
- Partials       91      100       +9

Flag
Coverage Δ

unittests
85.50% <68.51%> (-0.97%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

src/pyfx/view/common/frame.py
56.66% <46.55%> (-3.50%)
:arrow_down:

src/pyfx/view/view_mediator.py
56.52% <53.33%> (-43.48%)
:arrow_down:

src/pyfx/view/components/warning_bar.py
63.63% <63.63%> (ø)

src/pyfx/view/components/query_bar/query_bar.py
91.35% <85.71%> (-3.89%)
:arrow_down:

...omponents/autocomplete_popup/autocomplete_popup.py
94.00% <88.88%> (-3.92%)
:arrow_down:

src/pyfx/app.py
96.77% <100.00%> (+0.10%)
:arrow_up:

src/pyfx/cli.py
75.75% <100.00%> (+2.42%)
:arrow_up:

src/pyfx/view/components/help/help_popup.py
94.64% <100.00%> (-3.51%)
:arrow_down:

.../pyfx/view/components/json_browser/json_browser.py
100.00% <100.00%> (ø)

src/pyfx/view/view_frame.py
100.00% <100.00%> (ø)

... and 6 more

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Git6304

Description

Related Issue(s)

Testing

Release Notes & Documentation

Additional notes

The gradle build is failing. First indication is The error message was I/O error on GET request for "https://localhost:10448/servers/serverinmem/open-metadata/access-services/subject-area/users/garygeeke/categories": Remote host terminated the handshake; nested exception is javax.net.ssl.SSLHandshakeException: Remote host terminated the handshake', which suggests that the server chassis has exited. Subsequently there is an indication the gradle background daemon has disappeared, but this could be consequential.
Unfortunately the logs are not kept (see #5290). It could have been due to a server exception in egeria, or possibly a heap issue (github workers are limited to 7GB max, and so default for running the jar is probably 3.5GB). If the issue only occurs on the build infra the latter is more likely.

See #6450


[FLINK-25465][state] Correct the logic of materizating wrapped changelog state

What is the purpose of the change
ChangelogKeyedStateBackend#keyValueStatesByName would store wrapped state, such as TTL state or latency tracking state. It will throw exception on initMaterialization if wrapped:
for (InternalKvState<K, ?, ?> changelogState : keyValueStatesByName.values()) {
           checkState(changelogState instanceof ChangelogState);
           ((ChangelogState) changelogState).resetWritingMetaFlag();
}

Brief change log

Correct the logic in changelog keyed state backend.

Verifying this change
This change added tests and can be verified as follows:

Added test testMaterializedRestoreWithWrappedState

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

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

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

accepted with small comments.
Thanks for fixing this!


Enhances Installation Instructions And Documentation For Latest Oqtane Framework Version 5 With .NET 8 - Fixes #20

Fixes #20
This PR Updates installation instructions for Oqtane Framework latest version 5 and .NET 8
It also splits up the directions more to help focus on steps like building the application prior to running it along with links to the latest or last release of each version currently.
Modify things as desired left open to maintainers.

looks great IMHO.


What were your hyperparameter?

Hello.
I want know abot your setting for emotion recognition with CREMA-D.
I have tested your code for my research.
In your paper, your accuracy applied OGM-GE was 62.2 when dataset is CREMA-D and fusion method is sum.
I have tested with the same setting. But,  my best acc was 61.
What were your hyperparameters for emotion recognition? (alpha, batchsize and learning rate)

Your parameter can be found on this github page.
I did not read closely.
thank you


feat: add with_npm_publish_target to disable npm_publish generation

Type of change
New feature or functionality (change which adds functionality):
Arg with_npm_publish_target to npm_package to have possibility to prevent generation of {name}.publish target.
For changes visible to end-users
Suggested release notes are provided below:
Added arg with_npm_publish_target to npm_package to have possibility to prevent generation of {name}.publish target.
Test plan
Manual testing:

Provide with_npm_publish_target = False to npm_package
bazel query "..." | grep publish.

@gregmagolan Is it ok for you?

@SinimaWath I think we should rename this to publishable. WDYT?
I think we should also add a TODO(2.0): switch to false and in v2 we can make the breaking change of not creating this target by default. I think most of the time packages are only used internally and this target is not even necessary. Worst case we can drop the TODO if we decide we want to keep it as true, best case we switch the default in 2.0.

@jbedard
Looks good to me, changed that and added TODO

I proposed this back when we added the .publish target, but decided it was unnecessary. I don't think much changed since then. What would be justification for adding this other than making the query result smaller?

@thesayyn
Yes, you're right, there are 2 reason for me:
1 - Make query result smaller
2 - Replace name.publish with our internal implementation, but keeping DX the same as using npm_package, so developers don't need to learn a new target of publishing packages. I think it's small changes, but essential.


USB transport drops remaining data after PSM FINISHED or ERROR state

Bug Report
Proxy can't decode some RPC Messages
Reason

The USB driver of HU divides RPC Messages into many separate packages and then send them to Proxy.
In our case, when the data package at one time is less than 512B, it will be combined with the leading portion of the next RPC to be sent.

※　Part①～Part⑤ are data packages after division by the USB driver

Part①～Part③：data packages of the first time

RPCMessage① can be decoded
Since RPCMessage② has only the header portion but no trailing data, Proxy can't decode it and will discard it.

Part④～Part⑤：data packages of the second time

Since RPCMessage② has no header, Proxy can't decode it and will discard it.

Solution
Data of the header portion should be not discarded and decoded with the second data package together.
OS & Version Information

Android Version: Android 9
SDL Android Version: SDL Android Version: 4.10.1
Testing Against: SDL Core 6.0.1 + HMI running on our development environment.

The library should be handling this correctly. There is specific logic to build out packets in this way. The SdlPsm is moved through states byte by byte. So the data received through the USB transport will be gone over 1 byte at a time. When the SdlPsm reaches the finished state it will build the packet and push it out. After this, it resets the SdlPSM and continues to parse through the received byte array.
See this loop in the MultiplexUSBTransport
Can you confirm that the bytes being wrote out by Core/your USB driver are the same as they are being received in this loop?

Hi @zhouxin627, could you please confirm if this fix #1281 solves the issue?

Hi @bilal-alsharifi, the issue has been fixed. Thank you.

Awesome. Thanks for confirming.

Fix has been merged


Suggetion for rollback the deployment to the last successful deployment

Please provide us with the following information:

Suggestion required for rollback the deployment to the last successful deployment in case of any issue
This issue is for a: (mark with an x)
- [ ] bug report -> please search issues before submitting
- [ ] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

Any log messages given by the failure

Expected/desired behavior

OS and Version?

Windows 7, 8 or 10. Linux (which distribution). macOS (Yosemite? El Capitan? Sierra?)

azd version?

run azd version and copy paste here.

Versions

Mention any other details that might be useful

Thanks! We'll be in touch soon.

If you're using azd, then our hope is for azd to add support for this via deployment slots in the future. See discussion in this issue: https://github.com/Azure/azure-dev/issues/383
For now, the easiest approach is probably to deploy to a separate staging azd environment before deploying to prod environment.
I think I remember that you're not using azd however. In that case, you could start using deployment slots today. Deploy to your staging slot, run tests, and if it looks good, swap it to prod. If not, don't swap it. That's an over-simplified explanation, there are more details here: https://learn.microsoft.com/en-us/azure/app-service/deploy-staging-slots?tabs=portal

Hello @pamelafox, will be enough to change the BACKEND_URI env variable to the Slot one in the .env file?
Thank you

@AlexParton Sorry, can you clarify what you're hoping to achieve by changing that variable?

Sorry about that, @pamelafox.
I have been struggling in creating a slot deployment for staging but since the AZD deploy does not support slots I have tried GitHub actions, local git and docker, but no success.
Ive read your message about changing the env so azd deploy can be used, as workaround for slot deployment, and I was wondering if changing that variable would be enough for azd deploy to deploy the app on the slot 🤔.
Thank you very much for this, I am really in a dead end with this.


Errors after installing 0.97.1

After installing the package in a new Meteor app (1.2.1, no other packages), I get the following error in the browser:
materialize_materialize.js:10 Uncaught TypeError: Cannot set property 'guid' of undefined
(anonymous function) @ materialize_materialize.js:10
(anonymous function) @ materialize_materialize.js:15
(anonymous function) @ materialize_materialize.js?852eb8ab99e3508edd310071537cb33698a58a2b:47
(anonymous function) @ materialize_materialize.js?852eb8ab99e3508edd310071537cb33698a58a2b:56
global-imports.js?6fd341f2229c43c0fc084197a7405fcb1b6a7d64:12
Uncaught TypeError: Cannot read property 'Materialize' of undefined
(anonymous function) @ global-imports.js?6fd341f2229c43c0fc084197a7405fcb1b6a7d64:12
template.myapp.js:2 Uncaught ReferenceError: Template is not defined
(anonymous function) @ template.myapp.js:2
(anonymous function) @ template.myapp.js:14
myapp.js:1 Uncaught ReferenceError: Meteor is not defined

The error is not in v0.97.0.

Appears to be a duplicate of #2037, which was fixed. When will the fixed version be released?

Same here.
Current workaround is using<EMAIL_ADDRESS>in packages

@hitchcott why 0.96.1?  Surely you meant 0.97.0.

@msolters works with 0.97.0. I can confirm it

@msolters @ajinkyapisal Works with 0.97.0 but not with 0.97.1.
Temporarily switching back to 0.97.0 seems like a solution.
meteor add<EMAIL_ADDRESS>

@msolters I stand corrected :yellow_heart:


inconsistency in serialization of URIs

While in person, work and place URIs are rendered: <idno type="URL">, org has it rendered as <note type="IDNO">. It would be reasonable to have <idno> here as well

Ah, I'm wrong. No idno in location


add alignment for left-aligned rtl

Currently align-items is specified for:

left-aligned ltr, i.e. :host
right-aligned ltr, i.e. :host([right-aligned])
right-aligned rtl, i.e. :host([right-aligned][dir="rtl"])
But it's missing a specification for
left-aligned rtl, i.e. :host([dir="rtl"])
This PR adds this spec, which fixes mis-aligned overlays in the missing case.

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

Thanks for the PR. We didn't have time to review it yet because RTL improvements are postponed (due to other priorities). But we will make sure to review it in the next sprints.

This was fixed by #711, and will be available in 4.1 release. Closing.


fix: allow empty string as column id

Empty string can be a valid property name in row data objects, it should be allowed as column id. The existing check of !id still makes sense for other falsy values that are invalid.

Why should an empty string be allowed for a column id?

For examples, I could be displaying an CSV file with a column of empty header, or I could be displaying query result of SELECT "";. These result in an empty string column id and row data like:
{
  "": "some value",
  "anotherCol": "anotherVal"
}

AccessorKeys and IDs can be different. I suggest keeping the accessorKey as "", but providing a valid id column option with a value.

@KevinVandy We did so as a workaround for now. Just curious what's the concern of allowing empty string as column id?


replace rotation with an usable i32 implementation

rotation is basically a i32, so just replace it with one.

closed in #55


⚠ DesiTorrents has degraded performance

In 417ff71, DesiTorrents ($DST) experienced degraded performance:

HTTP code: 200
Response time: 1039 ms

Resolved: DesiTorrents performance has improved in 0889206.


ASF GRFN products and GDAL3's  vsis3

Hi,
Don't know if this is the right place to report this. Most of us end up using jupyter via conda and looks like gdal3 build with conda might not be recognizing netcdf_mem.h correctly. If this is not an issue with custom build GDAL; then this should be reported to conda-forge/gdal-feedstock.
I remember this being addressed by https://github.com/OSGeo/gdal/issues/1328
Here is a test script that shows that GRFN products are interpreted as HDF5:
#This is from https://media.asf.alaska.edu/uploads/InSAR/temporary_security_credentials.py
from json import loads
from requests import get

credential_url = 'https://grfn.asf.alaska.edu/door/credentials'
response = get(credential_url)
response.raise_for_status()

credentials = loads(response.text)['Credentials']

## This is how you use ASF s3 credentials with GDAL
from osgeo import gdal
gdal.SetConfigOption('AWS_REGION', 'us-east-1')
gdal.SetConfigOption('AWS_SECRET_ACCESS_KEY', credentials['SecretAccessKey'])
gdal.SetConfigOption('AWS_ACCESS_KEY_ID', credentials['AccessKeyId'])
gdal.SetConfigOption('AWS_SESSION_TOKEN', credentials['SessionToken'])

results = gdal.Info('/vsis3/grfn-content-prod/S1-GUNW-D-R-160-tops-20190710_20190628-162436-20935N_18926N-PP-6a53-v2_0_2.nc')
print(results)

Piyush

Turns out this is related to them building conda packages on old  kernels:
userfaultfd support: no
https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=55722
This is unlikely to change anytime soon.
Piyush

@piyushrpt We don't have a formal way to announce possible bugs that aren't directly related to to notebook servers but are worth mentioning. So I guess that issuing a issue is as good as anything else.
The current notebook servers are built using GDAL 2.4. For ISCE, It's install on Ubuntu 18.04 via (https://github.com/asfadmin/asf-jupyter-container/blob/master/dockerfile):
apt install gdal-bin
pip install 'gdal==2.4.0'

I ran the test script you mentioned in the production cluster and got

Driver: HDF5/Hierarchical Data Format Release 5
Files: /vsis3/grfn-content-prod/S1-GUNW-D-R-160-tops-20190710_20190628-162436-20935N_18926N-PP-6a53-v2_0_2.nc
Size is 512, 512
...

So it would appear that GRFN products are interpreted as HDF5.

Tracking this here: https://github.com/conda-forge/gdal-feedstock/issues/323

I reran the script against production again and got

Driver: netCDF/Network Common Data Format
Files: /vsis3/grfn-content-prod/S1-GUNW-D-R-160-tops-20190710_20190628-162436-20935N_18926N-PP-6a53-v2_0_2.nc
Size is 512, 512
...

Since the tracked conda-forge link has closed the issue and the products are reported as netCDF, this issue appears to be resolved.


Reverse Proxy configuration / UseBasePathMiddleware

Hi!
We have our application hosted at https://internal/app/
We have a reverse proxy in front of our application which takes requests at https://public/rp/app/ and rewrites them to https://internal/app/ using a rewrite rule.
Now we need the UseBasePathMiddleware to set the BasePath to /rp/app in order that our link generation works. However, if I set the ASPNETCORE_APPL_PATH environment variable to /rp/app the BasePath is not set (because the Request.Path is /app only - after the rewrite rule - and the StartsWith check in UseBasePathMiddleware doesn't work anymore).
What are we doing wrong? How to properly configure our proxy scenario?

UseBasePathMiddleware is designed to move a path segment from the path to the path base field. You don't want to move something, you want to introduce it. Try this:
app.Use((ctx,next) => {
 ctx.Request.PathBase = "/rp";
 return next();
});

Okay, thanks. I assumed such a middleware is already built-in.


Try to support Python 3.6.7

It looks like add_subparser did not support required in 3.6:
Traceback (most recent call last):
  File "/users/joverton/IMPACC2ImmPort/_venv/bin/cogs", line 11, in <module>
    sys.exit(main())
  File "/users/joverton/IMPACC2ImmPort/_venv/lib/python3.6/site-packages/cogs/cli.py", line 73, in main
    subparsers = parser.add_subparsers(required=True, dest="cmd")
  File "/programs/x86_64-linux/python/3.6.5/lib/python3.6/argparse.py", line 1707, in add_subparsers
    action = parsers_class(option_strings=[], **kwargs)

Once I remove that, most things seem to work.

It would also be good to test multiple Python versions with GitHub actions, like we are now doing for VALVE.


Master comment non existing branches

depends on #41 ,  removes dead merges and puts them in  a comment at the end of the yaml.
Could not do comment in place because of PyYaml limitations.
would have to do indepth mod of the DUmper, and may not even be possible.
rualmel seems to make it possible if this solution is not okay
https://copyprogramming.com/howto/how-to-insert-a-comment-line-to-yaml-in-python-using-ruamel-yaml
currently given input:
odoo:
  defaults:
    depth: ${WAFT_DEPTH_DEFAULT}
  remotes:
    - origin: https://github.com/odoo/odoo.git
    - origin: felafel

  target: origin ${ODOO_VERSION}
  merges:
    - origin ${ODOO_VERSION}

OCA/product-attribute:
  defaults:
    depth: ${WAFT_DEPTH_DEFAULT}
  remotes:
    origin: https://github.com/OCA/product-attribute.git
  target: origin ${ODOO_VERSION}
  merges:
    - origin ${ODOO_VERSION}

Where "origin felafel" is a  branch that does not exist:
odoo:
  defaults:
    depth: ${WAFT_DEPTH_DEFAULT}
  remotes:
    origin: https://github.com/odoo/odoo.git
  target: origin ${ODOO_VERSION}
  merges:
    - origin ${ODOO_VERSION}

OCA/product-attribute:
  defaults:
    depth: ${WAFT_DEPTH_DEFAULT}
  remotes:
    origin: https://github.com/OCA/product-attribute.git
  target: origin ${ODOO_VERSION}
  merges:
    - origin ${ODOO_VERSION}

# dumped in comment section non-existing merge:  - origin felafel    from repo  odoo , remote: origin:https://github.com/odoo/odoo.git

Don't know if this addition is useful , but i had an issue so i did it.
Possible problems:

         Resiliance: Not set up to consider connection problems. may remove branches if connection check fails

         Wrong  Remote:  in case of mispelled or non-existing remote name, it has not been tested.

         Duplicates:   HAs been tested for duplicate merges (works) , but not for duplicate remotes.

@thomaspaulb @Hussam-Suleiman let me know if :

this function is a worthwile one, if i should spend another hour or two hardening the three problem points i found in description.
if i should spend another hour implementing in-place comments instead of "end-of-file dump" by re-implementing with  ruamel and dropping YAML
If i should extend this by not only identifying  branches that do not exist. But also by identifying branches that have no extra commits if compared to target.
if i should rename script from update_depths  to  a more generic name, now that it does multiple things.

In my opinion, if this script can't support all Odoo installation versions we can freeze it until the next waft version.

In my opinion, if this script can't support all Odoo installation versions we can freeze it until the next waft version.

this message refers to https://github.com/sunflowerit/waftlib/pull/41 ?  this is just an addition to that.
I respond in that MR.

I think:

This functionality is very nice to have
The functionality in #41 is also very nice to have
So, by all means, let's keep pursuing this
But, let's make it part of Waft 2.0, which Hussam has been working on for a while now
@ddejong-therp Could you make a priority out of reviewing @Hussam-Suleiman 's work in Waft 2.0?

Meanwhile, @gfcapalbo I would let this rest, and #41 as well, until we make a decision on Waft 2.0

@thomaspaulb Added a connection check before looking to remove non-existing branches.
if the repo cannot be reached, nothing will be done to change the yaml.


Week of 11.02.15

Community

[ ] 4 paired calls with 8 club leaders (@hellyeah, hackedu/meta#263)
[x] Grow number of messages sent by non-core Slack members by 10% by next
Monday (@hellyeah, hackedu/meta#264)

Metrics

[ ] Get attendance count from all Labs clubs (@MaxWofford)

[ ] @hellyeah (hackedu/meta#265)

[ ] Armijo High School
[ ] Cherry Hill High School
[ ] Project Cipher
[ ] Lowell High School
[ ] Ravenwood High School
[ ] Elizabeth High School

[ ] @MaxWofford (hackedu/meta#268)

[x] USC Hybrid High School
[x] San Leandro High School
[ ] College Park High School
[x] South Brunswick High School
[x] Columbia Secondary School
[x] Glenaeon Rudolf Steiner School

[x] Put together system to aggregate attendance data and put the data from
last week into it (@MaxWofford, hackedu/meta#266)

Curriculum

[ ] Ship Maze workshop (@jonleung, hackedu/hackedu#339)
[x] MVP of playbook portal (just a Markdown file) (@jonleung,
hackedu/hackedu#421)
[ ] Talk with club leaders about their planned curriculum and add it to
https://clubs.hackedu.us in the Meetings sheet (@jonleung@maxwofford, hackedu/meta#262)

@zachlatta I motion that @jonleung's first issue is to get well.

Assigning this issue to you @zachlatta because I know that you said you needed to do some work on this before we can close it.

Closing because the week is over. Next week's issue is in #293.


Sort Submissions

User Story
As a challenge manager, in order to manage the progress of evaluations efficiently, I would like to be able to sort submissions based on submission and evaluation progress data.
Acceptance criteria:

[ ] A challenge manager can sort and filter the submissions (on the submission list view) by:

[ ] Filter by:

[ ] Prescreening status: completed, in progress, not started, recused
[ ] Eligible for Evaluations (Prescreen status)
[ ] Advancement status

[ ] Sort by:

[ ] Number of assigned evaluators: low to high, high to low
[ ] Average score (for completed evaluations only): low to high, high to low

Dev tasks:

[ ] Task 1
[ ] Task 2

Definition of Done
Doing (dev team)

[ ] Code complete
[ ] Code is organized appropriately
[ ] Any known trade offs are documented in the associated GH issue
[ ] Code is documented, modules, shared functions, etc.
[ ] Automated testing has been added or updated in response to changes in this PR
[ ] The feature is smoke tested to confirm it meets requirements
[ ] Database changes have been peer reviewed for index changes and performance bottlenecks
[ ] PR that changes or adds UI

[ ] include a screenshot of the WAVE report for the altered pages
[ ] Confirm changes were validated for mobile responsiveness

[ ] PR approved / Peer reviewed
[ ] Security scans passed
[ ] Automate accessibility tests passed
[ ] Build process and deployment is automated and repeatable
[ ] Feature toggles if appropriate
[ ] Deploy to staging
[ ] Move card to testing column in the board

Staging

[ ] Accessibility tested (Marni)

[ ]  Keyboard navigation
[ ]  Focus confirmed
[ ] Color contrast compliance
[ ] Screen reader testing

[ ] Usability testing: mobile and desktop (Tracy or Marni)
[ ] Cross browser testing - UI rendering is performant on below listed devices/browsers (Tracy or Marni)

[ ] Windows/Chrome
[ ] Windows/Edge
[ ] Mac/Chrome
[ ] Mac/Safari
[ ] iOS/Safari

[ ] AC review (Renata)
[ ] Deploy to production (production-like environment for eval capability) (dev team)
[ ] Move to production column in the board

Production

[ ] User and security documentation has been reviewed for necessary updates (Renata/Tracy/Dev team)
[ ] PO / PM approved (Jarah or Renata)
[ ] AC is met and it works as expected (Jarah or Renata)
[ ] Move to done column in the board (Jarah or Renata)

@r-bartlett-gsa for the Evaluation Progress filter, does Completed mean all Evaluators have completed, Not Started means none of the Evaluators have started, and In Progress means it's neither of those?
The recused filter is not in Figma, is that no longer needed? if needed, does that show submissions with any recused status?

@r-bartlett-gsa for the Evaluation Progress filter, does Completed mean all Evaluators have completed, Not Started means none of the Evaluators have started, and In Progress means it's neither of those?

@stepchud Yes

The recused filter is not in Figma, is that no longer needed? if needed, does that show submissions with any recused status?

@stepchud I had recused status filter only on one screen, but now all screens that show expanded filter are updated in Figma. Recused filter is needed and, yes, it shows submissions with a any recused status.


ROX-8277 Fix malformed operator k8s client user-agent

Description
In certain cases, the UserAgent string sent by the operator to the kubernetes API is malformed. e.g.
"userAgent": "stackrox-operator/v0.0.0 (linux/amd64) kubernetes/$Format/leader-election"

^ see the $Format/... suffix
It seems that the operator controller-runtime k8s client config is not used for all clients. For example, it seems that there is another k8s client instantiated for the leader election, which does not use the specified userAgent.

Closing as it is not currently possible to configure the leader-election user agent. https://github.com/kubernetes-sigs/controller-runtime/issues/2243

How about fixing it upstream?

@porridge yeah I can look at that. I see two issues

The client-go AddUserAgent not "adding an user agent", but ignoring a pre-existing user agent and using DefaultKubernetesUserAgent() + "/" + <whatever>
The controller-runtime ignoring the LeaderElectionConfig: rest.Config.UserAgent configuration.

k8s-client PR: https://github.com/kubernetes/kubernetes/pull/116801


Added EPSHL

Schoolwebsite: (https://www.epshl.de/)

@systemlukas Pull request merged. Thank you!


webhook reply

Hello!
I have found out that webhook can reply with message. See link
I have test it, and it is true. Here is a primitive webhook WORKING example
#[post("/telegram/test-dev", format = "application/json", data = "<telegram_update>")]
pub fn telegram_webhook(telegram_update: Json<tgbot::types::Update>) -> Result<Json<JsonValue>, String> {
    let Json(update) = telegram_update;

    if let UpdateKind::Message(message) = update.kind {
        if let Some(text) = message.get_text() {
            let chat_id: i64 = message.get_chat_id();

            let val = json!({
                "method": "sendMessage",
                "chat_id": chat_id,
                "text" : "My Webhook answer..."
            });

            return Ok(Json(val));
        }
    }

    Err("".into())
}

So, the question is, how can I build an answer using SendMessage struct?
something like this:
SendMessage::new(chat_id, "My Webhook answer...")

instead of hand-write json!({ }).
I have noticed that SendMessage struct have no "method" field and RequestBuilder seems like something other.
If there will be a working example how to handle if - will be ideal
Forward thank you for your answer

Hi!  You can write your own wrapper for methods:
https://play.rust-lang.org/?version=stable&mode=debug&edition=2018&gist=dffd072e8950417f3fafe10c415c69ad

Оh, thank you!!
That makes sense. Didn't know about #[serde(flatten)] macro
Maybe once you will have want add MethodWrapper that will work something like
let method_wrapper: MethodWrapper<SendMessage> = SendMessage::new(chat_id, "My Webhook answer...").as_webhook_answer();

But for now flatten macro solution suits me great!)
Again thank you


Fix temporary table syntax for postgres.

Adding the fix for mysql/mariadb to use a temporary table
broke postgres.
Postgres does not support creating a temp
table using a select statement, unless it is done
through a select into, which maridb does not support.
This fix just wraps the original code (for postgres)
in a postgres engine check, and leaves the mariadb/mysql
code in that env.
Closes-Bug: 1800466

Looks good.


Element disappearing after animate (flipInX) – Safari 9 OS X

Element is performing correct animation but disappears afterwards

Normally, is flipOutX or flipOutY which can give that result. Can you give a look of your code ?

If you wish you can also give a try to this javaScript side development that support animate.css animations. Here is an example of usage.
 //Select the elements to animate and enjoy!
 var elt = document.querySelector("#notification") ;
 iJS.animate(elt, "shake") ;
 //it return an AnimationPlayer object
 //animation iteration and duration can also be indicated.
 var vivifyElt = iJS.animate(elt, "bounce", 3, 500) ;
 vivifyElt.onfinish = function(e) {
     //doSomething ...;
 }
 // less than 1500ms later...changed mind!
 vivifyElt.cancel();

Take a look here

I've moved on, but I remember it was rather a few of the effects giving that result. It was definitely flipInX, though.

More animations that have "out" with its name are made to hidden elements when ended; and other with "in" are made to make visible the elements. So normally flipInX according to it implementation not have to make disappearing element.

However, can give a test to that on safari cause not have OS X.

It was definitely doing the 'flip in' animation but then immediately fades out.

I've hit this problem, both in Safari 9 on OS X and iOS.
Here's a test case showing the problem, along with a workaround:

https://jsfiddle.net/aaejuq67/2/

The problem occurs when the animated element is a descendant of another element with a transform applied. I have been able to work around this by adding an extra wrapper, which sets a zero transform.
It looks like Safari is creating some kind of shared 3D context between the various nested transforms, which causes flip animations to flip behind the plane of the container and thus eventually disappear.

Good tips! You can perhaps fix it directly to the source by adding your wrapper code to considered animations which occure the issue. For example, at the end of flipInX:
replace:
to {
    transform: perspective(400px);
  }

by
95% {
    transform: perspective(400px);
  }
to {
     transform: translate(0, 0);
  }

and submit for a Pull Request

Thank you so much @tnga ,your answer fixed my problem!

@tnga However , it appeared to be a problem:while the item animate, part of it will disappear, but after the animate finish, it will show correctly.
this problem only shows in IOS, Android is OK.

@frasern his solution is the simplest way to solve this problem.

I have been able to work around this by adding an extra wrapper, which sets a zero transform.

this is the solution.

@lugic1993 your welcome and thanks for the feedback 😁


Add "Confluentinc" repo and charts

All the Conluent inc helm charts are missing from the hub. They can be found here: https://github.com/confluentinc/cp-helm-charts

This will have to made into a helm repository and added using the instructions here - https://github.com/helm/hub/blob/master/Repositories.md

This can also be added to the Artifact Hub, as well.
I filed https://github.com/confluentinc/cp-helm-charts/issues/455 to ask them to add it. With that in mind, I'm closing this issue.


It would be nice to have support of Maps

It would be nice to have support of populating maps as well. I am not sure what API should be for this.
For lists "having" is perfect, may be something like
aPerson.havingDetails("age", 25)

where person has property
private Map<String, Integer> details;

Yes that could be possible. Have you tried to give this a shot and submit a pull request?

I didn't have a chance to create pull request. Let me work out this.

After giving this further thoughts, you'd be better off using a Map builder for this. You could create your own implementation of a Map builder quickly.
My latest commit (https://github.com/davidmarquis/fluent-interface-proxy/commit/b033243cbcc9e0f331a954f347d1a26e05574040) introduces a MapBuilder and it will be available in the next release. If you need it, let me know!


SILGen: Fix linkage for stored property initializers of private properties

Always give stored property initializers the linkage of the type
that contains them, even if the property is less visible than the
type.
This fixes the case where a type defines a private property, and an
extension in a different file from the same module defines a
constructor.
Also, if the type is public, don't give stored property initializers
public linkage. For now, we don't have a resilience or cross-module
story here -- we don't serialize the fact that a property has an
initializer, so there's no way to call it from outside the module
anyway.
Fixes rdar://problem/32743703.

@swift-ci Please smoke test

@swift-ci Please smoke test

@swift-ci Please smoke test Linux

@swift-ci Please smoke test


Gem cannot find mini_racer when depolying files on github

Following this youtube tutorial, I am trying to deploy the files of my website on github using (on Windows). Once I run bin/deploy --user, I face the following error:
$ bin/deploy --user
Deploying...
Source branch: source
Deploy branch: master
Do you want to proceed? [y/N] y
Switched to branch 'source'
Deleted branch master (was ddfbef8).
Switched to a new branch 'master'
C:/Ruby31-x64/lib/ruby/site_ruby/3.1.0/bundler/resolver.rb:235:in `block in verify_gemfile_dependencies_are_found!': Could not find gem 'mini_racer' in locally installed gems. (Bundler::GemNotFound)
        from C:/Ruby31-x64/lib/ruby/site_ruby/3.1.0/bundler/resolver.rb:228:in `map!'
        from C:/Ruby31-x64/lib/ruby/site_ruby/3.1.0/bundler/resolver.rb:228:in `verify_gemfile_dependencies_are_found!'
        from C:/Ruby31-x64/lib/ruby/site_ruby/3.1.0/bundler/resolver.rb:31:in `start'
        from C:/Ruby31-x64/lib/ruby/site_ruby/3.1.0/bundler/definition.rb:279:in `resolve'
        from C:/Ruby31-x64/lib/ruby/site_ruby/3.1.0/bundler/definition.rb:490:in `materialize'
        from C:/Ruby31-x64/lib/ruby/site_ruby/3.1.0/bundler/definition.rb:187:in `specs'
        from C:/Ruby31-x64/lib/ruby/site_ruby/3.1.0/bundler/definition.rb:244:in `specs_for'
        from C:/Ruby31-x64/lib/ruby/site_ruby/3.1.0/bundler/runtime.rb:18:in `setup'
        from C:/Ruby31-x64/lib/ruby/site_ruby/3.1.0/bundler.rb:161:in `setup'
        from C:/Ruby31-x64/lib/ruby/site_ruby/3.1.0/bundler/setup.rb:20:in `block in <top (required)>'
        from C:/Ruby31-x64/lib/ruby/site_ruby/3.1.0/bundler/ui/shell.rb:136:in `with_level'
        from C:/Ruby31-x64/lib/ruby/site_ruby/3.1.0/bundler/ui/shell.rb:88:in `silence'
        from C:/Ruby31-x64/lib/ruby/site_ruby/3.1.0/bundler/setup.rb:20:in `<top (required)>'
        from <internal:C:/Ruby31-x64/lib/ruby/site_ruby/3.1.0/rubygems/core_ext/kernel_require.rb>:85:in `require'
        from <internal:C:/Ruby31-x64/lib/ruby/site_ruby/3.1.0/rubygems/core_ext/kernel_require.rb>:85:in `require'

Then, I try to install mini_racer using gem install mini_racer -v 0.6.2, but it does not work as well:
$ gem install mini_racer -v 0.6.2
ERROR:  Error installing mini_racer:
        ERROR: Failed to build gem native extension.

    current directory: C:/Ruby31-x64/lib/ruby/gems/3.1.0/gems/libv8-node-<IP_ADDRESS>/ext/libv8-node
C:/Ruby31-x64/bin/ruby.exe -I C:/Ruby31-x64/lib/ruby/site_ruby/3.1.0 extconf.rb
creating Makefile
C:/Ruby31-x64/lib/ruby/gems/3.1.0/gems/libv8-node-<IP_ADDRESS>/ext/libv8-node/builder.rb:12:in `build_libv8!': failed to download node 16.10.0 (Libv8::Node::BuilderError)
        from C:/Ruby31-x64/lib/ruby/gems/3.1.0/gems/libv8-node-<IP_ADDRESS>/ext/libv8-node/location.rb:30:in `install!'
        from extconf.rb:9:in `<main>'
==== in C:/Ruby31-x64/lib/ruby/gems/3.1.0/gems/libv8-node-<IP_ADDRESS>/ext/libv8-node
==== running C:/Ruby31-x64/lib/ruby/gems/3.1.0/gems/libv8-node-<IP_ADDRESS>/libexec/download-node

extconf failed, exit code 1

Gem files will remain installed in C:/Ruby31-x64/lib/ruby/gems/3.1.0/gems/libv8-node-<IP_ADDRESS> for inspection.
Results logged to C:/Ruby31-x64/lib/ruby/gems/3.1.0/extensions/x64-mingw-ucrt/3.1.0/libv8-node-<IP_ADDRESS>/gem_make.out
Temporarily enhancing PATH for MSYS/MINGW...
Building native extensions. This could take a while...

I also tried gem install libv8-node and faced a very similar error ("ERROR: Failed to build gem native extension.") Note that I have already installed ruby_dev. Please note that I am a beginner (thanks for your patience!).

See if you can find a solution in the thread #691. Otherwise, leave a comment there.


Need to document developer logging

Need to document developer logging options, system properties,
logging.properties, etc.

I could find no information on this on your page.  If there, please provide a
link...

Thanks,
Jeff

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 2 Nov 2012 at 12:07

There is a way to get more logging out of this library:

enable debug logging mailer.setDebug(true)
increase log level of the native JavaMail framework using log4j config


Hi 👋!

Hey there - maintainer of Buildalyzer here. This project was pointed out by a Buildalyzer user and I thought it was really cool. Glad to see the tool being put to use. I'm also pretty curious about where Codelyzer (cool name :smile:) is being used (if you can say)?
In any case, don't be a stranger - happy to fix and issues upstream if you find any.

Hi Dave,
Thank you for reaching out, and for writing a great tool! We're using codelyzer to generate a code analysis to be used in other open source projects. Below are these projects. Please do check them out if you get a chance:
Porting assistant client that helps users analyze their .net dependencies and check compatibility with .net core:
https://github.com/aws/porting-assistant-dotnet-client
And
Code translation feature that allows users to automatically port some code from .net framework to .net core:
https://github.com/aws/cta
If you have any questions. comments, suggestions, or if you find anything that needs to be improved please don't hesitate to reach out.

Hi Dave,
Thank you for reaching out, and for writing a great tool! We're using codelyzer to generate a code analysis to be used in other open source projects. Below are these projects. Please do check them out if you get a chance:
Porting assistant client that helps users analyze their .net dependencies and check compatibility with .net core:
https://github.com/aws/porting-assistant-dotnet-client
And
Code translation feature that allows users to automatically port some code from .net framework to .net core:
https://github.com/aws/cta
If you have any questions. comments, suggestions, or if you find anything that needs to be improved please don't hesitate to reach out.

Cool, thanks! Always fun to learn where your projects are being used.

Cool, thanks! Always fun to learn where your projects are being used.


Suggestion: make is-a.dev be the main url instead of www.is-a.dev

Hey,
So I thought, why not make www.is-a.dev redirect to is-a.dev instead and have is-a.dev be the main page instead? It would make more sense in my eyes as it's a shorter domain, also the service is called is-a.dev

I think we will keep it on www.is-a.dev as root domains do not support CNAME records.

I'm pretty sure they do? I use one for my own website

They don't in regular DNS, I'm assuming you're using Cloudflare which does support it whereas we use Namecheap.


Volume adjust

Suggestion: ^^title
Maybe a slider or even a small pop up menu?
Thoughts?

A volume slider plus a mute option would be very useful when I want to mute the music but not everything else.  The option to replace the progress bar with volume would be good but might not suit everyone.

I would also like this feature added. I would like to be able to change the volume of just the spotify app, and not my entire system.

@Nalyd217 @Cynan @ingdanko
I have a fork which has a basic volume button implementation.
Im currently working on making it a bit more customizable, though it is already functional if you're still interested
https://github.com/AudioBand/AudioBand/releases/tag/v1.0.3


population of total 416 sectors

For making the prediction model purpose, I created population of total 416 sectors file (csv file) from  website : https://www.citypopulation.de/en/rwanda/sector/admin/
Description
Clearly documents the reason for the pull request and what has changed.

[ ] This is a bugfix.
[x] This is a new feature.
[x] This is part of a feature.

Organization

[x] Have you linked this pull request to a Trello card?

Review

[ ] Have you added two reviewers to this pull request?
[x] Is this a work in process?
[ ] Is this ready to be reviewed?

The population of 416 sectors.


bcm2836sdhc unload

Both sdport.sys and bcm2836sdhc.sys have bugs that prevent them from
successfully unloading. Fix the bug in bcm2836sdhc.sys so that it will
be able to unload correctly once sdport.sys gets fixed.
bcm2836sdhc:

Add a Cleanup handler. This is not optional -- sdport.sys does not
check for NULL before trying to call it, and it will be called every
time sdport.sys tries to unload the driver. In addition, if it
doesn't clean up WPP, we'll crash shortly after unloading.
Add a PowerControlCallback handler. This is not optional -- sdport.sys
does not check for NULL before trying to call it. This is less of an
issue because I've never actually seen sdport try to call this.
Clean up WPP if SdPortInitialize fails.

rpisdhc:

Add a PowerControlCallback handler. This is not optional -- sdport.sys
does not check for NULL before trying to call it. This is less of an
issue because I've never actually seen sdport try to call this.
Clean up WPP if SdPortInitialize fails.

Thanks!


Could Redis connect support redis.Sentinel ?

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Could we extend RedisConnect to support redis.Sentinel ?
Describe the solution you'd like
A clear and concise description of what you want to happen.
Our Redis uses sentinel mode, the master/backup nodes ip may change dynamically. Could LMCache support

RedisConnect uses sentinel to discovery the master/slaves>>> from redis import Sentinel
>>> sentinel = Sentinel([('localhost', 26379)], socket_timeout=0.1)

master is from connection pool for write>>> master = sentinel.master_for('mymaster', socket_timeout=0.1)
>>> master.set('foo', 'bar')

backup is from connection pool for read>>> slave = sentinel.slave_for('mymaster', socket_timeout=0.1)
>>> slave.get('foo')

Could we support sentinel hosts list passed as environment variable ?
like REDIS_SENTINEL_HOSTS="<IP_ADDRESS>:6222,<IP_ADDRESS>,6223"
Could we support Redis service name as environment variable ?
like REDIS_SERVICE_NAME=myredis
Describe alternatives you've considered

A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

@wengyao04 Hey, just wanted to keep you posted, we already created a new PR to support redis.Sentinel at #93
This functionality will also be updated into the docker image once merged

Passed end-to-end test.
Setup
Mistral 7B model on A40
Test result (TTFT in seconds):

Context length
8K
16K
24K

Without LMCache
1.35
2.45
4.10

LMCache CacheGen + Redis-sentinel
0.76
1.18
1.54


Problems rending programming language R in source code

When trying to render an R code black, I get the following spacing error. Note that doing the same thing in Python has normal spacing.

Here's what my markdown post looks like:

When I change to the default minima template, the error goes away.

I've been able to reproduce it locally... very strange.
Anyway this theme doesn't provide syntax highlighting for specific languages. So if you want a quick fix for your problem, just use triple backticks and do not specify R as a language and it renders fine.


Numeric parameter 5 digits long not handled properly

When I use a numeric value >=10000, it is not processed properly. This was discovered when using the QSYS2.NETSTAT_JOB_INFO service.
The following query works:
db2util -p 9999 'SELECT JOB_NAME,SLIC_TASK_NAME,LOCAL_PORT from QSYS2.NETSTAT_JOB_INFO WHERE LOCAL_PORT > ? ORDER BY LOCAL_PORT DESC'
Whereas this one fails:
db2util -p 10000 'SELECT JOB_NAME,SLIC_TASK_NAME,LOCAL_PORT from QSYS2.NETSTAT_JOB_INFO WHERE LOCAL_PORT > ? ORDER BY LOCAL_PORT DESC'
Seems some kind of data overflow/underflow issue.

On cursory inspection, it looks like the value is getting translated at four bytes. For instance:
 db2util -p 99999 'SELECT JOB_NAME,SLIC_TASK_NAME,LOCAL_PORT from QSYS2.NETSTAT_JOB_INFO WHERE LOCAL_PORT > ? ORDER BY LOCAL_PORT DESC'
seems to process the query as if the parameter was 9999
I'm guessing SQLDescribeCol is returning 4, meaning 4 bytes, when we really want character size. That's just a guess, however.

SQLDescribeCol wouldn't affect this, since it's the parameter that's the problem. We bind all parameters as SQL_CHAR, but maybe it's having issues with the precision/scale values we're passing back? Or maybe like most things, CLI is just completely garbage when it comes to type conversions.

Yes, looks like SQLDescribeParam is giving back 4 for the precision and we pass that back to SQLBindParam. (I believe this is still what the original code from Tony did). Of course, the docs say that shouldn't matter in this case:

Precision of the corresponding parameter marker.

If ParameterType denotes a binary or single-byte character string (for example, SQL_CHAR), this is the maximum length in bytes for this parameter marker.
If ParameterType denotes a double-byte character string (for example, SQL_GRAPHIC), this is the maximum length in double-byte characters for this parameter.
If ParameterType denotes SQL_DECIMAL or SQL_NUMERIC, this is the maximum decimal precision.
Otherwise, this argument is ignored.

In this case the ParameterType (the SQL type retrieved from SQLDescribeParam) is SQL_INTEGER, so according to the above docs this value should be completely ignored. Of course, according to the code, that's not what happens. Since 5.4, the code delegates conversions to SQ Runtime code and it uses this value to tell Runtime how much data there is.
Since we're always going to bind as SQL_C_CHAR, I think we can actually get rid of the whole SQLDescribeParam anyway. We just bind as SQL_C_CHAR with whatever length the data is as the precision.

Looks like the docs were fixed in 7.2: https://www.ibm.com/docs/en/i/7.2?topic=functions-sqlbindparameter-bind-parameter-marker-buffer as it now mentions ValueType, not ParameterType.


pathlib.Path support

System information

TensorFlow version (you are using):
Are you willing to contribute it (Yes/No):
Yes

Describe the feature and the current behavior/state.
In 3.4 python added the pathlib module for handling of file system paths. Currently, the corresponding pathlib.Path objects cannot be passed to tensorflow functions directly as they expect strings.
Will this change the current api? How?
Functions that expect a file or directory path would accept both string and Path parameters.
Who will benefit with this feature?
Anyone using both pathlib and tensorflow.
Any Other info.
The (probably incomplete) list of functions that should accept PathLike parameters:
tf.io.read_file
tf.io.TFRecordWriter
tf.io.write_file
tf.io.write_graph

tf.data.TextLineDataset
tf.data.TFRecordDataset
tf.data.FixedLengthRecordDataset

tf.saved_model.load
tf.saved_model.save
tf.train.Checkpoint
tf.train.CheckpointManager
tf.train.checkpoints_iterator
tf.train.get_checkpoint_state
tf.train.latest_checkpoint
tf.train.list_variables
tf.train.load_checkpoint
tf.train.load_variable

tf.estimator.Estimator

tf.feature_column.categorical_column_with_vocabulary_file
tf.feature_column.embedding_column
tf.feature_column.sequence_categorical_column_with_vocabulary_file
tf.feature_column.shared_embeddings

tf.lookup.TextFileInitializer

tf.summary.create_file_writer

keras.Model.save
keras.Model.save_weights
keras.Model.load_weights
keras.models.load_model
keras.models.save_model
keras.callbacks.CSVLogger
keras.callbacks.ModelCheckpoint
keras.callbacks.TensorBoard

keras.utils.get_file
keras.utils.plot_model

@ngc92 Sorry. It is not clear. Where do you what to implement the feature? in pathlib or tensorflow? Thanks!

this is a change to tensorflow. I think it would also be best not to specialize to pathlib.Path (though that is probably the main use case), but to make everything based on the os.PathLike interface for generality.

I'll be able to work on this later in the year

I have submitted a PR that adds pathlib support for the keras functions mentioned above. It introduces a generic helper function that converts a pathlib.Path to str and is a no-op for python versions that do not support pathlib.
The same approach would also work for all other tensorflow functions that do not take Tensors as arguments. I can submit a PR for that, but I am unsure of where to place the helper function.

I think we should mostly do the conversion just before calling the filesystem layer (e.g., in tf.io.gfile.... methods).
For the cases where those methods are not used and developers used the natural Python functions, I think we should first attempt a conversion to the wrapped ones. Only if this conversion does not make sense should we try to convert path-like objects to string in the other cases.
The utility can reside in TF proper, might need a public tf. name (via tf_export decorator).

I wasn't aware of this discussion but I ran into the same problem when moving some of our code to tf.io.gfile in order to support GCS natively.
I opend #41173 which implements @mihaimaruseac's suggestions and adds pathlib support to tf.io.gfile.

I'm not sure this issue should have been closed. It's still not possible to use pathlib.Path in TensorFlow 2.6:
import tensorflow as tf
from pathlib import Path

tf.io.read_file(Path("test.txt"))
# ValueError: Attempt to convert a value (PosixPath('test.txt')) with
# an unsupported type (<class 'pathlib.PosixPath'>) to a Tensor.

Stacktrace:

---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-9-5b1376127ba4> in <module>
      2 from pathlib import Path
      3
----> 4 tf.io.read_file(Path("test.txt"))

[...]/lib/python3.7/site-packages/tensorflow/python/ops/io_ops.py in read_file(filename, name)
    137     A tensor of dtype "string", with the file contents.
    138   """
--> 139   return gen_io_ops.read_file(filename, name)
    140
    141

[...]/lib/python3.7/site-packages/tensorflow/python/ops/gen_io_ops.py in read_file(filename, name)
    554     try:
    555       return read_file_eager_fallback(
--> 556           filename, name=name, ctx=_ctx)
    557     except _core._SymbolicException:
    558       pass  # Add nodes to the TensorFlow graph.

[...]/lib/python3.7/site-packages/tensorflow/python/ops/gen_io_ops.py in read_file_eager_fallback(filename, name, ctx)
    573
    574 def read_file_eager_fallback(filename, name, ctx):
--> 575   filename = _ops.convert_to_tensor(filename, _dtypes.string)
    576   _inputs_flat = [filename]
    577   _attrs = None

[...]/lib/python3.7/site-packages/tensorflow/python/profiler/trace.py in wrapped(*args, **kwargs)
    161         with Trace(trace_name, **trace_kwargs):
    162           return func(*args, **kwargs)
--> 163       return func(*args, **kwargs)
    164
    165     return wrapped

[...]/lib/python3.7/site-packages/tensorflow/python/framework/ops.py in convert_to_tensor(value, dtype, name, as_ref, preferred_dtype, dtype_hint, ctx, accepted_result_types)
   1564
   1565     if ret is None:
-> 1566       ret = conversion_func(value, dtype=dtype, name=name, as_ref=as_ref)
   1567
   1568     if ret is NotImplemented:

[...]/lib/python3.7/site-packages/tensorflow/python/framework/constant_op.py in _constant_tensor_conversion_function(v, dtype, name, as_ref)
    344                                          as_ref=False):
    345   _ = as_ref
--> 346   return constant(v, dtype=dtype, name=name)
    347
    348

[...]/lib/python3.7/site-packages/tensorflow/python/framework/constant_op.py in constant(value, dtype, shape, name)
    270   """
    271   return _constant_impl(value, dtype, shape, name, verify_shape=False,
--> 272                         allow_broadcast=True)
    273
    274

[...]/lib/python3.7/site-packages/tensorflow/python/framework/constant_op.py in _constant_impl(value, dtype, shape, name, verify_shape, allow_broadcast)
    281       with trace.Trace("tf.constant"):
    282         return _constant_eager_impl(ctx, value, dtype, shape, verify_shape)
--> 283     return _constant_eager_impl(ctx, value, dtype, shape, verify_shape)
    284
    285   g = ops.get_default_graph()

[...]/lib/python3.7/site-packages/tensorflow/python/framework/constant_op.py in _constant_eager_impl(ctx, value, dtype, shape, verify_shape)
    306 def _constant_eager_impl(ctx, value, dtype, shape, verify_shape):
    307   """Creates a constant on the current device."""
--> 308   t = convert_to_eager_tensor(value, ctx, dtype)
    309   if shape is None:
    310     return t

[...]/lib/python3.7/site-packages/tensorflow/python/framework/constant_op.py in convert_to_eager_tensor(value, ctx, dtype)
    104       dtype = dtypes.as_dtype(dtype).as_datatype_enum
    105   ctx.ensure_initialized()
--> 106   return ops.EagerTensor(value, ctx.device_name, dtype)
    107
    108

ValueError: Attempt to convert a value (PosixPath('test.txt')) with an unsupported type (<class 'pathlib.PosixPath'>) to a Tensor.

Same for tf.summary.create_file_writer, which was listed by the OP but not affected by #41173, it seems. This is as of TF 2.8.0.


Removing adaptators from Nanopore Dataset

Do you have an example or use case for removing adaptors from a raw dataset from Nanopore LongRead ?
here

@LucoLab you probably want to use https://github.com/rrwick/Porechop instead. It also looks for internal adaptors / chimeras etc.


fix #70 Name only parameters are ignored by parseParams

Ooh. Thanks a lot! Hadn't really considered anything but the base level use of parseParams, as it was only added as a kind of ad-hoc helper.
This, and the good tests does demonstrate the case perfectly though. Will merge and make a patch release :+1:


Update metrics-windows-network.rb

To resolved ...
https://github.com/sensu-plugins/sensu-plugins-windows/issues/9

Thanks for the PR! Can you take a look at the travis output and fix the rubocop issue.

There ya go.


Incremental Static Generation on the horizon?

A way to be able to generate pages statically but also update them live without a rebuild would be amazing. Some similar to the Next.js solution in 9.4. You can use methods such as fetch but this requires data to be grabbed every time the page is loaded, slowing it down.

I would also like to request this

Would also like to request this

This is now implemented in Nitro via routeRules: https://nitro.unjs.io/config#routerules.


Derive index url from context variables

My fault for not mimicking the JABBA_INDEX in the first place.
https://github.com/typelevel/jdk-index/blob/d0b36dfa69134cb38bc608ccd6c717152248ad53/.github/workflows/ci.yml#L10

Hmm, I thought we fixed the redirect thing.


Add opentelemetry as a submodule for latest xDS API

Prerequisite for https://github.com/grpc/grpc/pull/26848.
OpenTelemetry is depended by the latest xDS proto API.

/easycla

Known failures:
python.exe: No module named twine

Fri Jul 30 16:17:11 PDT 2021 - Test Case '-[InteropTestsLocalCleartext testUnaryResponseHandler]' passed (0.071 seconds).
Fri Jul 30 16:17:11 PDT 2021 - Test Suite 'InteropTestsLocalCleartext' failed at 2021-07-30 16:17:11.943.
Fri Jul 30 16:17:11 PDT 2021 - 	 Executed 32 tests, with 1 failure (0 unexpected) in 7.180 (7.254) seconds


Include Microsoft.AspNetCore.Razor.Common.dll in VS Mac bits

Fixes an error in the logs I'm seeing, trying to validate some stuff.

hmm.. maybe. I suppose as soon as someone uses Index or Range, anyway.


fix sam2_configs root path

fix sam2_configs root path like checkpoints

I don't believe that's desirable. Everything works as expected when you run pip install -e . in the repo.

Hi @triple-Mu, this is not required after installation as sam2_configs should be added already to the paths.


Print sources details of snapshots and published repositories

Description of the Change
For aptly wrapper tools to work such as pyaptly it needs to know sources of a snapshots and published repositories.
Such details are currently part of the description. However such description is hard to parse (or also to read for humans compare aptly publish list) and not always applicable (e.g. when a snapshot has been renamed, merged etc.).
Hence it would be good to have a Sources printed in the snapshot show command and have an additional command publish show to show details of a published repository as well.
Such details may also be very helpful for command line users to get a clue what has been done.
Checklist

[x] functional test added/updated (if change is functional)
[x] man page updated (if applicable)
[ ] documentation updated (this is docu on aptly.info right? how can this be updated?)
[x] author name in AUTHORS

@sliverc on docs: https://github.com/aptly-dev/aptly-dev.github.io

@smira
I recreated the man page with the patched version of ronn now and also created a PR for documentation changes at https://github.com/aptly-dev/aptly-dev.github.io/pull/29

Pushed as nightly aptly_0.9.8~dev+51+g207ebff


Add template for mooncard.co

Can't you merge stuff yourself as admin? For new templates there is no need to have a review or anything I think. :-)

If any setting is blocking you from merging, we should change it.


Error: Cannot find module 'rimraf'

Installing latest MCB throws this error about the rimraf dependency. I saw this was added later, not sure why it might not being installed as it appears as a dependency. I tried to install it manually but is still failing
Any ideas?

Could you update meteor-client-bundler to 0.5.8

the error is gone but the bundle still not working. I created a new react app using create-react-app and then a new meteor app with the --bare option. The first error happens when running the bundler. It says While processing files with static-html (for target web.browser.legacy): client/main.html:12: Expected <head> or <body> tag. I solved that by removing the static-html package from the meteor app.
Then I run the bundler again and start the app but it fails to compile. It throws Module not found: Can't resolve 'meteor/meteor'
This is how my index.js looks like
import "meteor-client";
import React from "react";
import ReactDOM from "react-dom";
import { Meteor } from "meteor/meteor";

import "./index.css";
import App from "./App";
import registerServiceWorker from "./registerServiceWorker";

Meteor.startup(() => {
  ReactDOM.render(<App />, document.getElementById("root"));
  registerServiceWorker();
});

Just remove static-html from .meteor/packages.

Yes, as I put on my comment I had to do that in order to start the meteor app. But then the react app is the one failing now

Then, you have to use custom bundler configuration instead of using -s parameter.

this is how I run the bundler meteor-client bundle -s api --url http://localhost:8000 and the meteor-client.js is being genereated under node_modules

Ok I created a config file and now it seems to be working. I copy the default config from the bundler. Not sure what is the difference tho.
Thanks

There is actually one more thing. I had to add the react-meteor-data on the imports section of the config file and also react & react-dom as an external npm package but I'm still getting a warning that react-meteor-data requires react as a peerDependency.
Also when trying to use Meteor.user() it throws this error Uncaught TypeError: __WEBPACK_IMPORTED_MODULE_1_meteor_meteor__.Meteor.user is not a function
Do you want me to open a new issue?

Yes I did that, but I still get the warning

I'm also trying to add any of this packages blaze-html-templates gadicc:blaze-react-component so I can load Blaze templates from react but the first one is totally being ignored even when generating the tmp meteor project before bundling it and the other one it does seems to get it bundled but when I try to import it it says module not found

Please do not add any html package to meteor client configuration.

so how do I load the Blaze template?
I'm not able to load the accounts-ui package templates nor the accounts-password
I just want to run the todo app demo that uses the accounts-ui package


Grids only apply on reboot

The program works but I have to reboot for them to show up. is this intended behavior?
Im on linux mint 21.2 using proton

This is the case on some systems and especially Linux.


HIVE-27175: Backport HIVE-20897 to branch-3

What changes were proposed in this pull request?
This PR backports HIVE-20897 to branch-3 to fix TestJdbcDriver2#testSelectExecAsync2
Why are the changes needed?
Test fix
Does this PR introduce any user-facing change?
No
How was this patch tested?
CI Job

Hi @amanraj2520 Can you please review this. We need this to fix TestJdbcDriver2

@vihangk1 This looks like a clean backport and this backport was targeted to fix this particular test in master as well. +1 from my side, LGTM.