React Native KeyboardAvoidingView not working with ImageBackground

Description
KeyboardAvoidingView stopped working once I added ImageBackground. Actually it's working but really bad working because when i open the keyboard, picture is broken
React Native Version
0.68.0
Output of npx react-native info
System:
OS: Windows 10 10.0.19045
CPU: (8) x64 Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz
Memory: 3.27 GB / 15.65 GB
Binaries:
Node: 14.17.3 - C:\Program Files\nodejs\node.EXE
Yarn: 1.22.19 - ~\AppData\Roaming\npm\yarn.CMD
npm: 8.6.0 - ~\AppData\Roaming\npm\npm.CMD
Watchman: Not Found
SDKs:
Android SDK: Not Found
Windows SDK: Not Found
IDEs:
Android Studio: Not Found
Visual Studio: Not Found
Languages:
Java: 12.0.2 - C:\Program Files\Common Files\Oracle\Java\javapath\javac.EXE
npmPackages:
@react-native-community/cli: Not Found
react: 17.0.2 => 17.0.2
react-native: 0.68.0 => 0.68.0
react-native-windows: Not Found
npmGlobalPackages:
react-native: Not Found
Steps to reproduce
first pic is my layout with imagebackground and second pic is children that i sent props to  my BGImage layout.
Snack, code example, screenshot, or link to a repository

any fix ? i have the same issus


I2C all of a sudden stops working

I bought two i2c-moisture sensors a long time ago in 2015 with version v2.7.1. I never really tried them but now wanted to use them using micropython and an ESP32. It worked like a charm in the beginning, but then all of a sudden the first one stopped working, the micropython exception says:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/lib/chirp.py", line 49, in reset
  File "/lib/chirp.py", line 44, in _register_8
OSError: [Errno 19] ENODEV

which means: there is not device at this address. Also a i2c scan doesn't return anything back.
So i tried the second old sensor, but also there, after some hours, all of sudden, same problem, sensor seems to become dead.
My setup:
VCC is 3.3V, 2k2 pullup resistors on both SCL and SDA
The micropython code is:
>>> from machine import Pin, SoftI2C
>>> import chirp
>>> i2c = SoftI2C(scl=Pin(22), sda=Pin(21))
>>> chirp_sensor = chirp.Chirp(i2c)
>>> chirp_sensor.reset()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/lib/chirp.py", line 49, in reset
  File "/lib/chirp.py", line 44, in _register_8
OSError: [Errno 19] ENODEV

What I tried:

I toggeled SCL low and high several times, same with SDA, no effect
I pulled "RESET" line low and high several times, the led flashes when pulled high, but no effect

Is there any way to bring them back, or are they dead? Any idea what could have killed them both?
Thanks a lot!

Hi,
2015 was long time ago...
commit ba48c0ac055df5c0bccb623ff6fc83b1be8b0bf2
Author: miceuz<EMAIL_ADDRESS>Date:   Wed Feb 17 00:17:30 2016 +0200

    Enable BOD finally!

Most probably flash is being corrupted due brown out. You should re-flash the sensor with the latest firmware.

Sure, that is a long time ago, but why throwing them away if they are still good (if the are...). I am totally happy to buy a new rugged one, just I somehow would want to be sure that the new one doesn't get bricked again after few hours of runtime because I am doing something wrong.
Any hints or documentation how to flash new firmware?

Sure, no need to throw them away. It's a standard AVR flashing. Are you familiar with that? Are your sensors rugged or plain?

They are both plain, a reason I would anyhow buy a rugged one. Sadly no clue about AVR flashing. Would be awesome if you could throw some pointers my way, e.g. do I need special hardware, where would I connect the sensor, which firmware to use, etc..

Good that you have plain sensors, this means you have access to the programming port.
Here is the pinout for the programming header:

It's Sunday and I don't have hardware in my hands to double-check it myself, but these links can help:
This one probably is the most relevant for you:
https://learn.adafruit.com/stand-alone-programming-avrs-using-circuitpython
These are C++ based:
https://github.com/lbernstone/ESP_AVRISP
https://github.com/ESP32-Musings/OTA_update_AVR_using_ESP32
In order for you not to compile the sensor firmware from scratch, you can down download it here: https://gist.github.com/Miceuz/9762f93cad23a95064086fe6c73c575a
I will look into this more next week if you still need it, let me know how it goes.

Wohoo, this is just awesome!  Just it seems, the light measurement always shows 65535, is it the right firmware for the 2.7.1 version? Thank you so much!


Adding Cloud marketplace integration section

Added cloud marketplace integration section under platform
Added Google cloud marketplace documentation.

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the CODEOWNERS.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screenshot.

 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.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6605c94d0a8a0225becbf110--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is:

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6605cae1b18ed82a81608adb--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is:

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6605d23b54c2bc31c0598c26--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6605f4fc773f5250e15ea30a--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6605f9439a3fbb0fd01d4b2b--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6605fb9566b23356e1c1d08e--harness-developer.netlify.app


fix: Reduced aggregated bundle size

Description
References

Codecov Report

Merging #485 (60a8e07) into main (21978a1) will decrease coverage by 69.73%.
Report is 170 commits behind head on main.
The diff coverage is 16.08%.

@@             Coverage Diff             @@
##             main     #485       +/-   ##
===========================================
- Coverage   83.07%   13.35%   -69.73%
===========================================
  Files         456      418       -38
  Lines       10418     8161     -2257
===========================================
- Hits         8655     1090     -7565
- Misses       1438     7052     +5614
+ Partials      325       19      -306

Files Changed
Coverage Δ

...ivedMessages/ArchivedMessageGetDocumentListener.cs
0.00% <ø> (ø)

.../ArchivedMessages/ArchivedMessageSearchListener.cs
0.00% <0.00%> (ø)

source/Api/Common/HttpHeadersExtensions.cs
0.00% <ø> (ø)

source/Api/Configuration/InboxMessageMetadata.cs
0.00% <0.00%> (ø)

...iddleware/Authentication/ActorForAuthentication.cs
0.00% <ø> (ø)

...ntication/Bearer/BearerAuthenticationMiddleware.cs
0.00% <ø> (ø)

...MarketActors/MarketActorAuthenticatorMiddleware.cs
0.00% <ø> (ø)

...ation/Middleware/Correlation/CorrelationContext.cs
66.66% <ø> (ø)

.../Middleware/Correlation/CorrelationIdMiddleware.cs
0.00% <ø> (ø)

...iguration/Middleware/Correlation/UserProperties.cs
0.00% <ø> (ø)

... and 152 more

... and 402 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Revert "Fix: Sides did not apply the layout position correctly."

Reverts emilk/egui#5232
I should have tested it first. cursor can contain infinites. There is a better fix to be found @zhatuokun

That's true, so just use available_rect_before_wrap.


[BUG] Python API background value different from what's expected

Environment
Operating System: Linux
Version / Commit SHA: master
Describe the bug
Something it's wrong with the python wrappers for FloatGrid. I'm trying to debug this issue myself but so far I haven't reach anywhere. Looks like the precision of floating point numbers is messing around, or sometihng similar.
I manage to narrow down the error to a minimalistic example.
NOTE This problem only appears in the Python wrappers in pyopenvdb.so, not a C++ API issue.
To Reproduce
C++ Example
Code:
#include <openvdb/openvdb.h>

#include <iostream>

using std::cout;
using std::endl;

int main() {
    openvdb::initialize();
    const float background_value = 0.3;
    openvdb::FloatGrid::Ptr grid = openvdb::FloatGrid::create(background_value);

    grid->print();
    cout << "grid->background() = " << grid->background() << endl;
    cout << std::boolalpha << (grid->background() == background_value) << endl;
    return 0;
}

Output:
Information about Tree:
  Type: Tree_float_5_4_3
  Configuration:
    Root(0), Internal(32^3), Internal(16^3), Leaf(8^3)
  Background value: 0.3
Transform:
  voxel size: 1
  index to world:
     [1, 0, 0, 0]
     [0, 1, 0, 0]
     [0, 0, 1, 0]
     [0, 0, 0, 1]

grid->background() = 0.3
true

Python Example
Code:
import pyopenvdb as vdb

background_value = float(0.3)
grid = vdb.FloatGrid(background_value)
print(grid.info())
print("grid.background =", grid.background)
print(grid.background == background_value)

Output:
Information about Tree:
  Type: Tree_float_5_4_3
  Configuration:
    Root(0), Internal(32^3), Internal(16^3), Leaf(8^3)
  Background value: 0.3
Transform:
  voxel size: 1
  index to world:
     [1, 0, 0, 0]
     [0, 1, 0, 0]
     [0, 0, 1, 0]
     [0, 0, 0, 1]

grid.background = 0.30000001192092896
False

Expected behavior
As shown in the example, the value 0.3 gets interpreted as 0.30000001192092896 when using the Python API. Which seems strange...
Moreover, the comparission grid.background == background_value return False, only on the Python example, which confirms that something is "wrong".
Additional context
Any ideas on where to look for this?
The problem with this error is that when working with python vdb grids, the background value doesn't get propperly interpreted, leading to a denser vdb representation(since 0.3 is not a background value anymore, it will be considered an active value on the grid)

What you're observing is different floating point precision interpretations from python to C++ i.e. floats in python are 64bits where as "float" in C++ is 32bits (usually and definitely in your case).
In your C++ example, both the value of "background_value" and the grid type are the same. That is 32bit floats. In your python example, the value of "background_value" is a 64bit float, as the "float" type in python is 64bits.
To illustrate this further see this example:
https://wandbox.org/permlink/8RNWGPCZBOcVrHvK
and consider running the following in python:
print("{:.30f}".format(0.3));
When you call grid = vdb.FloatGrid(background_value), the C++ API will truncate this to 32bits. But when you call print(grid.background == background_value) this is then taking the truncated value, extending to 64bits and comparing to the original 64bit float - but at this point, the original precision has been lost so the values differ.

@Idclip Thanks for the clear and precise reply. I didn't reply before because I'm still investigating other issues. So far, I've cast all my floats to np.float32 but I'm still struggling with this. So, I say we leave this issue open and once I found the solution I will post it here and close it.

So, I made a simple example to showcase my problem so far... It's not the original code I'm using but the behavior matches 100% of what I'm seeing.
To showcase my problem I made a simple VDB -> np -> VDB pipeline, without changing any value or anything...
Simple Example
import numpy as np
import pyopenvdb as vdb

def level_set_to_numpy(grid: vdb.FloatGrid) -> np.ndarray:
    shape = grid.evalActiveVoxelDim()
    start = grid.evalActiveVoxelBoundingBox()[0]
    sdf_volume = np.zeros(shape, dtype=np.float32)
    grid.copyToArray(sdf_volume, ijk=start)
    return sdf_volume

# Input VDB, a simple sphere
vdb_grid_in = vdb.createLevelSetSphere(2.0, voxelSize=0.1)
vdb_grid_in.name = "vdb_grid in"

# Covert to dense np array
np_grid = level_set_to_numpy(vdb_grid_in)

# Covert back to a VDB grid
vdb_grid_out = vdb.FloatGrid()
vdb_grid_out.transform = vdb_grid_in.transform
vdb_grid_out.background = vdb_grid_in.background
vdb_grid_out.gridClass = vdb_grid_in.gridClass
vdb_grid_out.name = "vdb_grid out"
vdb_grid_out.copyFromArray(
    np_grid,
    ijk=vdb_grid_in.evalActiveVoxelBoundingBox()[0],
    tolerance=0.0,
)
vdb_grid_out.prune()
vdb_grid_out.pruneInactive()
vdb_grid_out.signedFloodFill()

print("Stats:")
print("Same background?", vdb_grid_in.background == vdb_grid_out.background)
print("Same min value ?", vdb_grid_in.evalMinMax()[0] == vdb_grid_out.evalMinMax()[0])
print("Same max value ?", vdb_grid_in.evalMinMax()[1] == vdb_grid_out.evalMinMax()[1])
print("max[vdb  in] = {:.30}".format(vdb_grid_in.evalMinMax()[1]))
print("max[vdb out] = {:.30}".format(vdb_grid_out.evalMinMax()[1]))
print("min[vdb  in] = {:.30}".format(vdb_grid_in.evalMinMax()[0]))
print("min[vdb out] = {:.30}".format(vdb_grid_out.evalMinMax()[0]))

vdb.write("in.vdb", vdb_grid_in)
vdb.write("out.vdb", vdb_grid_out)

Output
Stats:
Same background? True
Same min value ? False
Same max value ? True
max[vdb  in] = 0.2978250682353973388671875
max[vdb out] = 0.2978250682353973388671875
min[vdb  in] = -0.2970613539218902587890625
min[vdb out] = -0.300000011920928955078125

Figures
Input

Output

Cross-section Input

Cross-section Output

Analysis
While the volumes represent exactly the same geometry, due to an unknown error to me in the copyFromArray method (I suspect), some voxel values don't match the target background value, meaning that they won't get "pruned" and marked as inactive. In consequence, we end up with a "less sparse volume" since we need to also densely store all those values.
This is the exact same behavior I'm facing on my application, the background value is the exact same 32 bits representation value but there are more active voxels in the output grid.
Here is another comparison between the output grids, to showcase that the "copy error" gets reflected on the min value of the output grid...

Conclusion
Something feels "wrong" to me, on the C++ side of things... I'm currently debugging this issue, but I'd love to hear some comments from the OpenVDB people if possible.
Thanks and sorry for the long explanation!

I managed to hunt the error, and it's on the C++ API...
I will open a Merge Request soon, but this will need some debate(in my humble opinion). The problem is simply the following:

On openvdb::FloatGrid, some voxel values are -background and thus, marked as `inactive. Which makes total sense to me.
When copying the data to a dense array, the mask for the voxels is lost, which is also not a problem to my eyes.
When copying back from a dense array to a openvdb::FloatGrid, it's checked which values are within a certain mTolerance close to the background value, and thus, marked as inactive.

Problem
The voxel values that are close to -background are getting marked as active values:
https://github.com/AcademySoftwareFoundation/openvdb/blob/4bdc9959ea6e91548dc819d0c9b9e467f0bd87d5/openvdb/openvdb/tree/LeafNode.h#L1268
Solution
I've simply put an Abs call on the last check, and then use the sign of the value to decide between +background or -background.
diff --git a/openvdb/openvdb/tree/LeafNode.h b/openvdb/openvdb/tree/LeafNode.h
index 00da465c..c95bb258 100644
--- a/openvdb/openvdb/tree/LeafNode.h
+++ b/openvdb/openvdb/tree/LeafNode.h
@@ -1265,9 +1265,10 @@ LeafNode<T, Log2Dim>::copyFromDense(const CoordBBox& bbox, const DenseT& dense,
             const DenseValueType* s2 = s1 + yStride * (y - min[1]);
             Int32 n2 = n1 + ((y & (DIM-1u)) << LOG2DIM);
             for (Int32 z = bbox.min()[2], ez = bbox.max()[2]+1; z < ez; ++z, ++n2, s2 += zStride) {
-                if (math::isApproxEqual(background, ValueType(*s2), tolerance)) {
+                if (math::isApproxEqual(background, math::Abs(ValueType(*s2)), tolerance)) {
                     mValueMask.setOff(n2);
-                    mBuffer[n2] = background;
+                    auto sign = ValueType(*s2) / math::Abs(ValueType(*s2));
+                    mBuffer[n2] = sign * background;
                 } else {
                     mValueMask.setOn(n2);
                     mBuffer[n2] = ValueType(*s2);

Test Case
I'm running the example above and adding this check to show that the voxel values are exactly the same and are not getting lost in between. The problem is with the active/inActive state
def compare_2_grids(grid1, grid2):
    """Extracts a 1d array representation of ALL values of a given VDB grid and
    checks for equality on a per-value base comparison."""

    def level_set_to_1d(grid):
        sdf = []
        for f in grid.iterAllValues():
            sdf.append(f.value)
        return np.asarray(sdf, dtype=np.float32)

    return (level_set_to_1d(grid1) == level_set_to_1d(grid2)).all()

Output Without the patch
in == out (1D representation)? True
vdb_grid_in.activeLeafVoxelCount() 7674
vdb_grid_out.activeLeafVoxelCount() 9093
Same background? True
Same min value ? False
Same max value ? True

Output WITH the patch
in == out (1D representation)? True
vdb_grid_in.activeLeafVoxelCount() 7674
vdb_grid_out.activeLeafVoxelCount() 7674
Same background? True
Same min value ? True
Same max value ? True

Conclusion
The provided patch does solve all my issues, and this is not related to the python API. Looks like a bug, that's impacting the copyfromDense calls. I will do some more testing before opening a PR to make sure this doesn't break anything.
NOTE
Most of the time this doesn't present an issue, on my sphere example, if you don't cut the sphere to manually inspect the values that are inside, you will never notice that there are some errors.


Problem about interval.remove

couldn't use remove to delete interval whose mark is None.

Could you put a reproducible example into a Gist or similar pastebin site? I need the TextGrid file (or a file like it) and the code.
From this example I can't see what happens when you try this: does it simply fail to remove anything or does it throw an error. And it'd take me a while to type this in by hand since you only provided an image (and only part of the textgrid).


Remove gdal

Overview
This PR removes the gdal dependency and replaces it with:

geotiff
fiona

Related Issue / Discussion
Fix #398
Additional Information
See also PR #636
Contribution Agreement
(as per https://github.com/geopython/pywps/blob/master/CONTRIBUTING.rst#contributions-and-licensing)

[ ] I'd like to contribute [feature X|bugfix Y|docs|something else] to PyWPS. I confirm that my contributions to PyWPS will be compatible with the PyWPS license guidelines at the time of contribution.
[x] I have already previously agreed to the PyWPS Contributions and Licensing Guidelines

Coverage remained the same at 0.0% when pulling 957298aefc268302856922eabd66ac5919999668 on cehbrecht:fix-gdal-ci into ced1484299761a834cc1cc854f4d9e9c2856e8b2 on geopython:main.

I think we can just remove the section you commented out. I don't think there's any added value to the feature processing going on in the test.


Connection write() not send data as separated TCP Packet

Hi All,
I have Gate Controller that could be programmed/configured with TCP connection.
I've been trying to send command to the Gate Controller with reactphp/socket.
I've an issue when I try to send multiple command script through tcp connection,
In order for the Gate Controller to read the command, it need to be sent with 1 tcp packet per command.
But with the write() method of the Connection class, all the command got joined as 1 tcp packet eventhough I was sending the command with multiple write()
                    foreach($send_data as $sd){                         $conn->write(hex2bin($sd));                     }
My current workaround is  close the TCP Connection and reopen the TCP Connection for every command I need to send to the GateController.
But it seems the GateController could not handling many TCP Connection at once,
even after I tried to put timer following after every command.
So the solution is not good.
Is there any method to send data per tcp packet with Connection class without closing the connection, instead of joined all data and send it within 1 tcp packet?
I use Wireshark to monitor the tcp packet.
Regards,
Hendrik

@Mindhk92 That's an excellent question!
TCP/IP uses the concept of "streams", i.e. it does not know the concept of a "TCP packet". This means that if you write "hello" and "world" to the stream, the receiving side may or may not receive this as individual bytes or bigger chunks.
It sounds like what you're looking for is "message framing". This is a concept where the receiving side has a way to know when a message is completed and the next message begins.
You can use some arbitrary delimiter, such as a newline or NULL character between each message, so that the receiver can know this marks the end of a message. However, this implies it may not be included in the message payload anymore (or you have to escape this). This is commonly used for text based protocols and rarely for protocols transporting binary data.
As an alternative, you can also include the message length in the data. For example, many protocols prepend the message payload with a UINT32 that contains the length of the payload. This way, the receiving side knows how many bytes it needs to read before the next message begins. This is commonly used for binary protocols.
I hope this helps :+1:
I believe this has been answered, so I'm closing this for now. Please come back with more details if this problem persists and we can reopen this :+1:


Allow SVGWriter to avoid having the background element

With this PR I modified SVGWriter to be able to avoid having the background element on SVG images
If you set the "background" to None or (255,255,255,0) (i.e. transparent) in the writer_options parameter when calling render then the background will not be written in the SVG (instead of having a rectangle).
This is because transparent background does not seem to work in the current library (it is black)

Thanks!


GitHub hertogjanduevel Hometown profile

Tell the world fun things to do in your hometown!
Versoix, Switzerland

[x] Create a branch
[x] Create a file
[ ] Commit changes
[ ] Open a Pull Request
[ ]  Have discussion 👍
[ ]  Merge Pull Request 🚢

checking boxes


consolidate completions to single file

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Hangs when adding "jsx" to extensions

I'm attempting to support React .jsx files within my Ember app. Based on ember-react-components, I tried adding these settings to my ember-cli-build.js:
    babel: {
      plugins: [require.resolve("@babel/plugin-transform-react-jsx")],
    },
    "ember-cli-babel": {
      extensions: ["js", "ts", "jsx"],
    },

However, adding "jsx" to extensions (regardless of whether I add the plugin) causes ember serve to hang indefinitely. It gets stuck at ⠇ building... [Babel: my-app-name > applyPatches], without using significant CPU or memory. The command also becomes hard to kill; Ctrl+C transitions to applyPatches > Builder Cleanup, which also hangs indefinitely.
If I just add the plugin without the extensions, jsx files don't work (any component that imports them fails to render).
Any ideas for how to debug this, or for a different way to support jsx/tsx? I've got React components working if I write them in pure js (using React.createElement), but jsx would be much nicer.

Ember 3.28
Ember CLI 3.28.6
ember-cli-babel 7.26.11

It looks like extensions was a red herring; the real issue is the contents of the jsx file that becomes included. The applyPatches hang persists if, instead of setting extensions, I create a file hello-react.js (note: not .jsx) with contents
export default function HelloReact({ lessonId }) {
  return (

      Hello, React! {lessonId}

  )
}

This happens with or without @babel/plugin-transform-react-jsx installed. The issue persists if I simplify the function to return

, but goes away if I return null.


Savibg image in other formats

I have noticed that we can save the images in only png format. Can other formats be supported?

It could be possible. Any specific formats that needs to be supported?

I need the images to be saved in jpg format.


Python-dotnet is an invalid package

In API-Samples/Python/requirements.txt you mention python-dotnet as a requirement, but this package does not exist.
Should we instead be using this https://pypi.org/project/pythonnet/ ?

https://github.com/Azure-Samples/azure-video-indexer-samples/pull/168


Provide a copy/link function for segment id in measurement heading

Description
It would be great to:

Hover on the header, it becomes a link which opens https://www.openstreetmap.org/{way,node,relation};
Add a button/icon next to the header copying the link above.

This feature would significantly increase the efficiency when trying to get a history of a segment because when there are too many segments in the map, the query feature will often timeout.
Screenshots

Hover on the header, it becomes a link which opens https://www.openstreetmap.org/{way,node,relation};

The “View on openstreetmap.org” link at the bottom of the left sidebar opens the element’s page.

closing because this functionality is already present and the measurement box would not be the right place to put such a link.


IoTHub high availability zones

[Enter feedback here]
Hello, the high availability zones in the documentation are correct. for the iot hub East US2 does not have high availability?
https://docs.microsoft.com/en-us/azure/iot-hub/iot-hub-ha-dr
https://azure.microsoft.com/en-us/global-infrastructure/services/?products=iot-hub

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

ID: a6874376-9df5-65b7-6908-eaaba67992d0
Version Independent ID: c76d243b-a9a0-d37d-6cc5-53eb1bf589d2
Content: Azure IoT Hub high availability and disaster recovery
Content Source: articles/iot-hub/iot-hub-ha-dr.md
Service: iot-hub
GitHub Login: @kgremban
Microsoft Alias: kgremban

@jerfcero Thanks for the feedback! Adding author and MSE @kgremban to evaluate and update the doc as appropriate.

@jerfcero There are several layers of high availability discussed in the document that you linked to. While intra-region high availability is supported in all regions, I've confirmed that we currently do not support Availability Zones for IoT Hub in the East US 2 region. The current list in https://docs.microsoft.com/en-us/azure/iot-hub/iot-hub-ha-dr#availability-zones is correct. We'll update the doc as soon as new regions are supported. Thanks for checking.


Correction for raw affiliation INRA/IATE, Montpellier, France

Correction needed for raw affiliation INRA/IATE, Montpellier, France
raw_affiliation_name: INRA/IATE, Montpellier, France
new_rors: 05y503v71;0000n5x09
previous_rors: 05y503v71
works_examples: W1580720870
contact: d36338d46e776daf49daba3449e037d6:e7bdff74f446b4b2e21128b88f898aa5e1a5 @ inrae.fr

This issue was accepted and ingested by the OpenAlex team on 2024-11-18. The new affiliations should be visible within the next 7 days.


Interface Simplification

Hey People i want to simplify interface for Email (Or simply say any made using Website) agent i.e. i want to make interface such that user only  enter "TO" email address,rest should either be hard coded inside agent code or value of this TO text field should be concatenated inside OPTION field
like
To_address:...............................................
option
To: To_address
<EMAIL_ADDRESS>body: bla bla bla
Dont have idea how to do that

Hi @nineralpha,
to simplify the interface of the EmailAgent have a look at the form_configurable concern, as an example check out the HipchatAgent. I think what you need to to is remove the default options of the EmailAgent, just expose one form configurable field and replace the interpolated values with your hard coded ones.

I'm going to close this issue. Please reopen if you have further questions.


Add turbowarp.sh extension

turbowarp.sh is an extension that allows for the use of SSH over WebSockets! This requires some setup on the server side to allow for SSH over WebSockets (via websocat or websockify), but otherwise it runs entirely independently in the browser. It uses a Go SSH client compiled to WebAssembly to have a fully-fledged client.
The full git repository is at https://sr.ht/~reesericci/turbowarp.sh!

Also see #1392

This is an amazing extension, but I would like to request changes

Go ahead, you can just tell me them here.

I would like to share this video of a full terminal I made with this extension lol

Have you seen my requested changes yet?

https://github.com/TurboWarp/extensions/pull/1396#discussion_r1555644672

I can hover over it, GitHub will show me a preview but I can't click it or see it in the main panel for some reason

That's odd, another way you can see it is by viewing the files manually on this pull request and look through the js file for the suggestion.

Weird, I might as well just reiterate here. Your documentaripn must be made and hosted here. You can reference other extensions that made documentation using our system to make your own. You can reference my extensions, TurboHook, Window Controls or Key Simulation to learn how to make documentation here and where to put it.

Alright - updated the docs reference to point to here!

honestly if you're gonna host the docs somewhere else then sourcehut is a pretty great place to do that, but yeah ideally they are in-repository
will look at this more during an upcoming weekend

I'm actually working on some possible breaking changes to use a login shell instead of independent sessions, so I'm converting this to a draft while I complete those as to get them finished up before projects start depending on this.

I'm going to re-open this since it seems getting a login shell working is going to take a lot longer than I expected and the extension is still usable now, and I figured out how to implement it later without making breaking changes.


Evaluate TinyLlama/TinyLlama-1.1B-Chat-v1.0

@

completed, results are not good.

completed, results are not good


More instructions to reproduce the baseline model results?

Hi! Thank you for your amazing efforts.
I wonder do you plan to update the repo to include more reproducibility codes/instructions? I am (and I guess many others are) interested in reproducing the baseline results for research purposes.
Thank you very much.

@shi-kejian what issues are you facing in reproducing the results?


ptls error on macos

After installing on macOS and adding to ZSH path I receive the following error when running ptlsL
/Users/username/.local/bin/ptls: line 91: -1: substring expression < 0

Describe a situation, when bug occurs

Does it happen all the time?

This issue occurs because macOS ships with an outdated version of bash. Run brew install bash to get the latest version, then you won't get that error.
But you will still get problems running ls and touch, the other commands included with ptls will run fine.
touch doesn't work because macOS ships with the BSD version of readlink and not the GNU version. You can install the coreutils package using brew install coreutils and then replace readlink with greadlink in the file pttouch, which you can get to via $HOME/.local/bin/pttouch. Replace readlink with greadlink on line 29.
For the ls command it only displays the file icon for some reason. I haven't figured out how to fix it yet.

ls probably writes something to stderr, but stderr is disabled.
Remove 2>/dev/null from line 14 and 43 in ptls and show the output.

I think it's problem with --group-directories-first in line 43

I just checked and its simply because macOS doesn't use GNU core utils, so there are some commands that don't work. The problem with ls was because its using the BSD version of ls which doesn't take in the argument -Q.
So the easiest way to fix this issue is this:

Install latest version of bash using brew install bash
Install core utils using brew install coreutils
Core utils provides the same commands prefixed with a 'g', so to use the GNU variants instead of the BSD variants, you can add the gnubin directory to your path by putting export PATH="/usr/local/opt/coreutils/libexec/gnubin:$PATH", into your bashrc or zshrc.

Everything should work fine after that. No need to edit any of the ptsh files.


Fix Database Manager Not Checking Existing Data on Migration

Currently the only work done is to make the tests run. However, this PR should address the following concerns:

[x] Should the in-memory database use a shared cache? (I believe it should, revert this change)
[ ] The Database Manager should check for existing data before attempting a migration.
[ ] Regression tests should be added to ensure that the database only copies data when there is old data to copy.

I expect to revert the current change, but as I had already made and pushed it before realising the bigger issue here, I will leave this as a draft. Future changes will be made swiftly.

The more I review this code, the worse I feel like it is. Every single table houses the schema, then has to run checks to see if there's an old table, grab the old data, then insert it into the new table. This is horribly inefficient. So instead of simply adding the check, I think I'm going to break out the logic into more functions.

Honestly, the more I work on this the more I realise that this entire system is so flimsy! The database manager shouldn't be directly handling migrations. DB migrations should be in their own module and allow incremental updates to the database. So instead of resolving the issue how I intended, I'm thinking the better answer is to have the DBManager class determine if a migration needs to occur, call out to a separate module to run the migration scripts (to be written in the future when a migration occurs), and then resume. This does mean one major change: the in-memory database should not be rebuilding every time it connects.


Review Zod types in client library public exports

Fixed in https://github.com/sodazone/ocelloids-services/commit/2f26213b67c573eed5246d583658cd2e6881ce94


Add support for separate keyup and keydown listeners per key. (with examples in README and docs)

Hi!
I discovered this project today and it looks great! I was planning to use v-hotkey for one of my projects, but one of my requirements was to set a value when a key is being held. That is to say, set the value on keydown, and unset it on keyup.
While v-hotkey didn't support it out of the box, I found that it wasn't difficult to extend it to support the syntax:
{
  keymap: {

    //original way
    'key1': this.keyMethod

    //object extension
    'key2': {
       keydown: this.keydownMethod
       keyup: this.keyupMethod
    }
  }
}

I've tried to minimize the performance penalty by conditionally adding the listener to the keyup event only if a keyup method is registered in the keymap. However, I think some optimizations still may be possible.
I've also taken the liberty to add examples of the new syntax to the Readme and the docs (I had to renumber the steps to place it somewhere i thought was appropriate, sorry 😞  ) Please feel free to revert those changes as you see fit.
In addition to the above feature added, I added stylus and stylus-loader as dev dependencies, because npm run dev  would fail without it. I've also changed npm run build to make use of the webpack configuration present in the root folder (running npm run build without the changes resulted in a drastically different index.js than the one present). Once again, please feel free to revert the change as needed.
Thank you for taking the time to consider this!

Thank you for your contribution!
It seems wonderful and I just happened to be thinking about this feature. Give me some more time to review your commits.

Well, the example of key combination has been removed in your commits, it's not so good. I think the better way is add one page for the new feature.

By the way, there's no need to build the docs and libs, I would do it before release.

Hmmm, the indentation of source code should be 2 instead of 4. I think eslint is nessasery, I will add it to dependencies soon.

https://github.com/Dafrok/v-hotkey/commit/14a52caabcfbf17e90269378dc7e2d0fe61cc214

https://github.com/Dafrok/v-hotkey/commit/14a52caabcfbf17e90269378dc7e2d0fe61cc214 added eslint for this project, it also causes such a lot of conflicts. If trouble to you, I will resolve them in time.

Hi @Dafrok ! Thank you for taking your time to review my commits!
Regarding the documentation, I did add a page for the Keyup and Keydown listeners 😄 . In https://github.com/JZumun/v-hotkey/commit/15a81c6d9bacac8d3b597794cc1c4881edfb4e4d what i did was move Key Combinations to step 4, Private Hot Keys to step 5, and the final page to step 6, making the Keyup and Keydown Listeners the new step 3.
Good call on the eslint, as well! Don't worry, I'll handle the resolving!
Thank you!

Alright! Thank you! Keep up the great work 😄


Problem: extensions are provisioned altogether

This pollutes the environment of extensions that potentially creates issues during tests or experimentation.
Solution: isolate every extension in its own share dir
As of now, this does not handle dependencies (REQUIRES) or ad-hoc combinations of extensions for manual testing.
The reason why omni_ext tests work is because we hardcoded this dependency in add_postgresql_extension. This whole thing needs to be improved.

Stale


Caddy-WebDav has a reference to the current directory in its listing when used via WinSCP

Hello,
When Caddy-WebDav is used with WinSCP as the client, every directory, except for the root one has a bogus directory with the same name as the current directory when listing its contents.
For example, domain.com/dav/audio has a non-existing audio directory, and when downloading the directory contents, WinSCP tries to access domain.com/dav/audio/audio, as well.
Total Commander's WebDav plugin works, while CarrotDav (does not honour /path/ and tries domain.com/resourceName instead), Cadaver (Did not find a collection resource.) and LFTP (405 Method Not Allowed (HEAD)) are unable to access any WebDav resource, so unfortunately I was not able to test this further.

To add to this, the bogus directory issue only seems to happen when Caddy-Webdav is active for something other than the root, e.g.,  webdav /dav {

I'll check this out as soon as I can. Thanks for reporting!


NLP Generator Upgrade

Enabled multiple models run support and more customizability
Added prompts datasets with standard schema
Enabled reporting
Updated demo and documentations

We should remove all append uses. It is deprecated:

And is there a faster way to run perspective api? You mention run with fewer iterations - why don't we set that by default? Is there a benefit for multiple iterations that's important for the demo? What would be a smaller prompt dataset?

Ok, standardized everything.

Now run always creates a self.results objects and returns self
prerpare_results potentially transforms self.results and returns. If self.results doesn't exist a NotRunError is raised
get_results returns self.results. If self.results is None a NotRunError is raised.

@amrasekh , I have brought the NLP Generator into Lens. I had to make some decisions about the default prompt set to use and set up a default generator function (for gpt2) to be the comparison class. Also had to define the default assessment function. Most of that is included in credoai.assessments.nlp_utils.py
Added a create_reports function to the assessment as you can see. This allows lens to run no problemo.
Overall, very easy to make an assessment (though some decisions had to be made) once the module was made.
Corrected a bug with 'perspective_config' (you checked for the string instead of self.perspective_config) and moved over to validation errors.

We should remove all append uses. It is deprecated:

And is there a faster way to run perspective api? You mention run with fewer iterations - why don't we set that by default? Is there a benefit for multiple iterations that's important for the demo? What would be a smaller prompt dataset?

The append s have been replaced with concat.
I changed the number of iterations to 1. The bold_religious_ideology is the smallest dataset with protected attributes that we have builtin now, so that one cannot be made changed. I'll create and add a small counterfactual dataset based on this later.


Always on bottom

closes #113

What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Docs
[ ] New Binding issue #___
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change?

[ ] Yes, and the changes were approved in issue #___
[x] No

Checklist

[x] When resolving issues, they are referenced in the PR's title (e.g fix: remove a typo, closes #___, #___)
[x] A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.
[ ] I have added a convincing reason for adding this feature, if necessary

Other information

Linux I'd like to implement is_always_below_bottom and is_always_on_top as well but there is a long-living upstream bug in GTK https://gitlab.gnome.org/GNOME/gtk/-/issues/682
Windows might help for implementation https://stackoverflow.com/a/65052538

@henry40408  This is great. Although I'm noticing on Windows when clicking the 'Show Desktop' button, the window still gets hidden annoyingly. Any ideas?

hi @jabza I just tested and confirmed that window is always hidden whichever I set it "always on top" or "always on bottom", so it might not relate to "always on bottom" or "always on top". Would you kindly open another issue?

@jabza this is a normal behavior of Windows. Nothing we can control. I think what you want is to make your tauri window behave like a desktop wallpaper? in this case check out my comment here https://github.com/tauri-apps/tauri/issues/4261#issuecomment-1146767667

Windows 11 is re-introducing the concept of Window 'widgets' which will be interesting - but for now I think the wallpaper child approach seems like the only way to remain visible upon 'Show Desktop'.
I tried OR'ing in the NOZORDER flag and pretty much everything else I could find on the topic. It seems like perhaps this used to be possible but not any more.

@jabza Windows has no concept of "always on bottom", but you can change the z-order of the window to be the bottom most but that can be changed later due to many factors. The only way you can be truly on bottom is by using wallpaper/progman approach. Also the "Show Desktop" button hides all windows to show the destkop and since an "always on bottom" window is still a window, it makes sense for it to be hidden.

@amrbashir understood, but it is possible for windows to 'survive' "Show Desktop", assuming they're "always on top". I guess that's the one exception Windows makes.. Thanks for the alternative solution, i'll go ahead and attempt a plugin!

even "Always on top" can't survive "Show desktop" because "Always on top" guarantees it is on top of everything else but it doesn't guarantee that it can't be hidden or minimized.

I've tested it locally with Always on Top set and it does survive the event for me

All good, your alternative solution looks to be the ticket :) Hopefully come Windows 11 a cleaner way will present itself!

What you want to do is probably similar to Rainmeter, so you could try looking into their source code and see if you can find anything related there.

Wow this is interesting... https://github.com/rainmeter/rainmeter/blob/78a24859aa2aafd69e819619eb9bbf44b77d2bd3/Library/System.cpp they seem to be using a 'dummy window' to detect the 'Show Desktop' event... there's some gems in here I'm sure...

yeah Rainmeter has been around for a while and I bet they have a lot of experience/knowledge in this area.


fix periodically failed tests

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

Visual Studio Marketplace
GitHub release

Your semantic-release bot :package::rocket:


Allow HIP_LIB_PATH to be overriden at the command line. Use HIP_LIB_P…

…ATH for rpath instead of HIP_PATH/lib

I'm running into error:
Use of uninitialized value $HIP_LIB_PATH in concatenation (.) or string at /opt/rocm/hip/bin/hipcc line 770.
In regular HIP/HCC use-case. Need to set a default value after VDI doesn't set one.


fix relative module was not found error when having nuxt srcDir set

fixes #55

I will test it this week, sorry for delay! Btw, I think Nuxt now prefers @ as alias

np, thanks.
I didn't find a source saying nuxt prefers @ as alias, both seem correct.
Feel free to edit this if you prefer.


Repokitteh(ci) evaluation failure error: signaled

There are currently a lot of failure logs for repokitteh with the message:
:scream_cat: Error while processing event:

evaluation error
error: signaled

sometimes it outputs the error to a pull request comment (eg https://github.com/envoyproxy/envoy/pull/23084#issuecomment-1332258608)
the error happens very frequently - afaict it happens on synchronize events - which seems to be merging changes from the source branch (https://github.com/orgs/community/discussions/24567)

one thing worth mentioning is that grepping envoy code and searching github event docs i dont see a mention of signaled so it looks like an issue in the rk code


[C++] CI job ubuntu-cpp-emscripten is failing

Describe the bug, including details regarding any error messages, version, and platform.
The test introduced in #40237 failed on CI job ubuntu-cpp-emscripten. This is a failed run: https://github.com/ursacomputing/crossbow/actions/runs/8871520273/job/24354620994
Component(s)
C++

Issue resolved by pull request 41461
https://github.com/apache/arrow/pull/41461


@cfinsness: Removed TOS so we can make it in Marketo

Closes https://github.com/artsy/force/issues/1406
As we discussed, it would be better to allow the terms of use to be implemented in Marketo.

@craigspaeth can we actually add back the TOS as it was originally on here https://www.artsy.net/gallery-partnerships/support ?
I figured out how to add the TOS on marketo templates, so we don't actually need to add the TOS to the forms themselves like this http://pages.artsy.net/gallery-partnerships.html which is what was causing us issues earlier.
Thanks!

Sure I can take a look at that, but it won't be deployed until next week in case you need to put in a quick fix on Marketo's side.

Great. I'll be out next week but can jump on to fix quickly on marketo's side if needed, or can get @nicholassewitz  to help out.


ImplGenerics ToTokens ignores defaults of const Generics

let mut generics = syn::Generics::default();
generics.params.push(parse_quote!(const N: usize = 1));
println!("{:?}", generics.split_for_impl().0.to_token_stream().to_string());

prints:  "< const N : usize >"
When I would expect "< const N : usize = 1 >"
Looking at your source code this seems to be done on purpose, so I guess there are situations where it is preferred to leave out the defaults.
If this is the case, could it be possible to have an option to toggle this?

This is the point of ImplGenerics.
impl<const N: usize = 1> T {} is not valid.

Fair point, I should have read the docs for split_for_impl() again :facepalm:
For future reference calling to_token_stream() on Generics gives the result I was looking for.


Fix text in German overlay for project import

Description
Fix German text.
Related issue: none (do you really want the overhead of a new issue for a trivial spelling fix?)
Motivation and Context
It fixes a text which is shown to users.
How has this been tested?
It was only manually reviewed.
Types of Changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[x] Documentation content changes
[ ] Other (please describe):

Checklist

[x] I have read the contributor guide.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Hi @stweil , thanks for the fix. And yes those tiny PR don't need and issue, we have enough anyway.


"Received:" header with ESMPTS is bogus

Received: from mail-yb1-x.x.xx. ([xx.xx.xx.xx] helo=[xxx])
        by unknown with ESMTPS (unknow unknow)   <-- here
        (cocosmail 0.2.0)
        (envelope-from<EMAIL_ADDRESS>        id ....
        for ...; Wed, 09 Jun 2021 06:27:02 +0200

This seems to affect messages received with STARTTLS only.

fixed in 766ff1a04d4148de94fcff7a2866c021dd9048fd


feat: Adding support for poetry 1.8.0 - 1.8.2

Summary
Fixes #371 This PR adds support for poetry versions 1.8.0, 1.8.1, and 1.8.2.
Use Cases
See summary.
I do not believe an integration test is necessary. Please feel free to guide me if this is incorrect.
Checklist

[x] I have viewed, signed, and submitted the Contributor License Agreement.
[x] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[x] I have added an integration test, if necessary.
[x] I have reviewed the styleguide for guidance on my code quality.
[x] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

Please note: Integration tests are failing locally, both before and after the change. I'm happy to get some guidance on fixing these, but they currently end up timing out.
Unit tests are fine locally.

can we get attention to this PR and get it approved/merged?

@arjun024 Would you be able to help us get traction on this PR?
I wouldn't normally @ someone, but I saw you were recently reviewing PRs for this repo, and thought you might be able to help out with getting traction on this version update. Thanks!

https://github.com/paketo-buildpacks/python/releases/tag/v2.19.0


How to remove bounce up animation?

When transitioned to player I can swipe up and the view will go up a little bit
How can I prevent that? could not find anything related in code
Thank you

@yarodevuci You can set the usingSpringWithDamping to 1.0 on the ARNTransitionAnimator instance.

It is already with that value


[WIP] Detox Helper

This is going to be full featured Detox helper which provides an alternative to Appium for mobile testing.
Original detox tests:
    await expect(element(by.id('createdAndVisibleText'))).toNotExist();
    await element(by.id('GoButton')).tap();
    await waitFor(element(by.id('createdAndVisibleText'))).toExist().withTimeout(20000);
    await expect(element(by.id('createdAndVisibleText'))).toExist();

Same CodeceptJS detox test:
I.dontSeeElement('#createdAndVisibleText');
I.click('#GoButton');
I.waitForElement('#createdAndVisibleText', 20);
I.seeElement('#createdAndVisibleText');

Any volunteers to test this?

Moved to standalone helper https://github.com/Codeception/detox-helper


:sparkles: Add permission handler

Changes
This adds a permission handler which can convert integers in the correct permission and vice versa.
Check off the following

[x] I have tested my changes with the current requirements
[x] My Code follows the pep8 code style.

#197

Looking over this, I think an implementation similar to this would be better https://github.com/Pincer-org/Pincer/blob/main/pincer/objects/events/presence.py#L134

~Looking over this, I think an implementation similar to this would be better https://github.com/Pincer-org/Pincer/blob/main/pincer/objects/events/presence.py#L134.~ edit: Actually that doesn't work with None so that has to be ruled out

I feel like the whole purpose of this class is to ease the use of permissions, wouldn't a similar implementation to  this kind of defeat the purpose of this class

Ignore what I said about |=


Raise an error if changing data on an account without mut

#[account(mut)]
I recently forget to put mut above an associated account whose data was being changed. This didn't raise an error, it just failed silently. Ideally this would raise an error.
Other functions, like in the SPL token program, raise errors when trying to change account data without the mut flag

How do other programs do that?
Anchor could inject a check for fields with mut on them and return an error code if it's not mut.

I'm not sure how it's done in other programs exactly but usually I get an error when trying to do CPI with non mutable accounts. Maybe it's because they assume that there will be account changes, whereas in anchor an associated account might not be changed in a given transaction. Maybe if, like you suggested, the compiler detected a mutable reference to an account but didn't also detect a mutable account attribute that it should raise an error.
All I can say is it was a confusing error to debug because it was silently failing to update the value on the account permanently.

Anchor could inject a check for fields with mut on them and return an error code if it's not mut.

Here is the relevant code that gets executed only if mut is on the account.
pub fn generate_constraint_mut(f: &Field, c: &ConstraintMut) -> proc_macro2::TokenStream {
    let ident = &f.ident;
    let error = generate_custom_error(ident, &c.error, quote! { ConstraintMut }, &None);
    quote! {
        if !#ident.to_account_info().is_writable {
            return #error;
        }
    }
}

In other words, anchor ensures that is_writable == true if mut is present. However, it does NOT ensure that is_writable == false if mut is absent, because such a code is NOT generated to begin with. So all it needs is.. generate this immut code if mut is absent:
pub fn generate_constraint_immut(f: &Field, c: &ConstraintImmut) -> proc_macro2::TokenStream {
    let ident = &f.ident;
    let error = generate_custom_error(ident, &c.error, quote! { ConstraintImmut }, &None);
    quote! {
        if #ident.to_account_info().is_writable {
            return #error;
        }
    }
}

If anchor generates this immut code by default .. and only if mut is present, generate the previous code.
Is this doable? If yes, I can try creating a PR.

@armaniferrante

Anchor could inject a check for fields with mut on them and return an error code if it's not mut.

Or it can implement AccountsExit ensuring exit() writes data to AccountInfo for all fields, irrespective of whether mut is present or not (and then Solana will catch the error if is_writable == false for an account). That seems to be the only simple way because ctx.accounts  is a mutable reference and users can write any accounts. Unless individual members of a mutable struct can be made read-only and I don't see any easy way. Or DerefMut needs to assert is_writable == true, else panic!().

Alright. I think we have a better solution than what I suggested in the previous comment. Not only we can optimally do it, we can also get compilation error if trying to mutate an account which is not marked as mut.
The problem I see is with Account<> type which implements DerefMut which means it allows mutation irrespective of the absence/presence of mut attribute. What we need is.. another type here.. say Immutable<> which is just like Account<> except that it implements only Deref, not DerefMut. Then either we allow users to use this type directly.. OR when can change Account<> field to Immutable<> during code generation if mut is not present on the field.
Please let me know if this sounds good?
cc: @acheroncrypto

OK. I implemented this in this PR: https://github.com/coral-xyz/anchor/pull/2513


Change inspect icon

Hi Guys,
a small thing, as a new user to the shipyard container ui, the inspect icon is sort of misleading because it appears to indicate a search which distracts you from seeing the search box at the top of the screen.
Would it be possible to change it to something like a view / eye icon?
Thanks,
Damien

:+1:


Release v0.2.1

Trigger release v0.2.1. If everything goes well, this time the extension should be published to the Visual Studio Markerplace, in addition to public OpenVSX (open-vsx.org).

The following 1 GitHub releases will be created once this pull request got merged.
Release v0.2.1 with changes since v0.2.0

Release v0.2.1 3cc544b
[ci] add a job to publish the extension to the Visual Studio Marketplace 27b4f7c
perform yarn upgrade 37fd531

The following 1 GitHub releases will be created once this pull request got merged.
Release v0.2.1 with changes since v0.2.0

Release v0.2.1 b2c5f58
update to node 18 5913235
Add CODE_OF_CONDUCT.md to the repository and update reference. 10e5398
[ci] add a job to publish the extension to the Visual Studio Marketplace 27b4f7c
perform yarn upgrade 37fd531


ng-click in editable-text

Please see below. I want the editable-text to use 'ng-click' to show a map so that user can select the GPS point and it sends the data back to editable text field. But the ng-click does not work with the editable-text. Is there a way that user can click on the editable-text and fire the event.
 24.98, 121.90

i have same issue with ng-change.
  {{ (std.payment_date| date:"dd/MM/yyyy")||today}}

i am trying to get date in controller when one is selected.....
@itskashi have u got it working???

hey got it working by using   "onaftersave="p_date(std)" instead of ng-change or ng-click.

I believe you have to use e-ng-click in order for it to be put on the text field that is created.

@eugef question is answered, please close.


[postgres] Fix NPE in internedTypeFor()

attributeType() can return null. Then NPE is triggered when executing
rails52:test/cases/bind_parameter_test.rb:71
Check for null to prevent the exception. This turns the test above
from an error to a failure (missing deprecation warning).

Ignore this...mixed it up...


Color coded tasks

making each task a different task color

Are these colors automatically assigned?  Or are the colors the user sets up individually for each task?


added file_permission in load_dataset

As discussed in #2065 I've added file_permission argument in load_dataset.
Added mainly 2 things here:

Permission of downloaded datasets when converted to .arrow files  can be changed with argument file_permission argument in load_dataset (default is 0o644 only)
Incase the user uses map later on to generate another cache file of dataset, it ensures the permissions of newly generated file are similar to that of *-train.arrow file inside cache_dir for that dataset.

From offline discussions: we want to make the permissions handling consistent with transformers. However from discussion  in https://github.com/huggingface/transformers/pull/11119 it looks like it might not be a good solution to provide this argument. Users should use umask for now, and we'll see how things evolve.
@bhavitvyamalik I'm closing the PR for now if you don't mind


Improve way to import efflux

efflux is currently imported by copying the source code into the project.
Evaluate if there is a better way to include and/or what are the licensing requirements to comply with them properly.

I've created a separate Android library and also published it in maven central.


Context to Run Examples

Hi @kirshin, I have a lot of interest for this SDK.
I'm trying to run the examples, but unsuccessfully.
I tried executing enroll_user.rb (with or without ssl) using Ruby 2.6.5 and 2.6.0.
I have my environment already configured to run this hyperledger project and it is working.
I was hoping to have some more instructions on what I'm missing to start using Ruby with HyperLedger.
Thank you for you attention.

Hi Élisson Michael
Thank you for your interest in this project
In the attachment files for working with your example.
In the init.rb you need to specify the path to the crypto materials

On 31 Jan 2020, at 01:25, Élisson Michael<EMAIL_ADDRESS>wrote:
Hi @kirshin https://github.com/kirshin, I have a lot of interest for this SDK.
I'm trying to run the examples, but unsuccessfully.
I tried executing enroll_user.rb (with or without ssl) using Ruby 2.6.5 and 2.6.0.
I have my environment already configured to run this hyperledger project and it is working https://github.com/IBM/raft-fabric-sample. I was hoping to have some more instructions on what I'm missing.
Thank you for you attention.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/kirshin/hyperledger-fabric-sdk/issues/5?email_source=notifications&email_token=ACGZCVTNZ257YAEE7VTGUSLRANAT3A5CNFSM4KN5G3O2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IJ6Y57Q, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACGZCVR5D6IRDBVLROXXHNDRANAT3ANCNFSM4KN5G3OQ.

@kirshin thank you for your feedback.
I've managed to make it work with a rails, but I'm only able to do a query method.
Everytime I try to use invoke I got an GRPC::Unavailable (14:DNS resolution failed).
Do you have any tips to where should I check first?
This is the repository where I'll document this working sample.

@kirshin thank you for your feedback.
I've managed to make it work with a rails application.
This is the repository where I'll document this working sample.


Querying Data with Numerical and String Filters

When filtering data such as pat_ids, or any key with a numerical value, it only returns results that begin with that numerical input, instead of matching it. Instead, it needs to match it completely. Doug suggested "equals" instead of "like" when querying.

Remove LIKE_ and it will match string/numerical data completely.


Pin doc requirements to avoid CI breakages

Follow-up to https://github.com/jupyterhub/repo2docker/pull/1050
This pins the versions of all doc dependencies, so we can rely on dependabot to check for problems with updates.
There's no dependabot.yml file in https://github.com/jupyterhub/repo2docker/tree/master/.github but it looks like dependabot has previously opened PRs in this repo: https://github.com/jupyterhub/repo2docker/pulls?q=is%3Apr+author%3Aapp%2Fdependabot+is%3Aclosed
so presumably it's been configured elsewhere?

Oops, just seen https://github.com/jupyterhub/repo2docker/pull/1050

Re-opened with just the version pinning.

What's the difference between the circleci and readthedocs builds- do we need both?

@manics don't think we need both, so I removed CircleCI in https://github.com/jupyterhub/repo2docker/pull/1053


SideBearings of empty glyphs

What values for SideBearings should be used for empty glyphs (like space)?
It seems to be zero?
And how to calculate the minLeftSideBearing in hhea table. Are those zero values taken into account (makeOTF seems to ignore them).
And the same issue is with the vertical tables (vmtx and vhea).

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

ID: b569bf16-eea2-0953-3345-024bbd9bd8d4
Version Independent ID: 5e1b6957-a7c9-fef1-5928-dca56d848565
Content: hmtx - Horizontal metrix table specification - Typography
Content Source: typographydocs/opentype/spec/hmtx.md
Product: typography
GitHub Login: @PeterCon
Microsoft Alias: PeterCon

Hi, Here is the response from the engineering team. "Because empty glyphs don’t have glyph data, and solely rely on the hmtx (or proxies) for their spacing we are not able to define a right sidebearing (RSB) value. (Because there is no bounding box available). If you can’t have a RSB value, I don’t think an LSB value makes sense. Therefore, I’d recommend that the LSB value be zero—as we find with most fonts.
As far as minLeftSideBearing is concerned—I’m not confident about who actually uses that value and why. I’d argue that both minLeftSideBearing and minRightSideBearing values of zero are useless unless the whole font has zero respective side bearings. "

The issue is not solved. The documentation needs to clarify this.
So I should ignore the sideBearing values for emtpy glyphs?
And there is a similar issue in the vmtx table. makeOTF uses the ascender value for the TSB for empty glyphs. I would expect zero.

Clarify sidebearings for glyphs without outline data.

Proposed changes for next version:
'hmtx'
Add at the end of the chapter:

If a glyph has no contours, xMax/xMin are not defined. The left side bearing indicated in the 'hmtx' table for such glyphs should be zero.

'hhea'

FWORD
minLeftSideBearing
Minimum left sidebearing value in 'hmtx' table for glyphs with contours (empty glyphs should be ignored).

FWORD
minRightSideBearing
Minimum right sidebearing value; calculated as Min(aw - lsb - (xMax - xMin))min(aw - (lsb + xMax - xMin)) for glyphs with contours (empty glyphs should be ignored).

This also is relevant to x/y min/max bounding box data in the head table. Ie, empty glyphs should be ignored while computing those as well.

Adding changes in 'head':

int16xMinFor all glyph bounding boxes. (Glyphs without contours are ignored.)
int16yMinFor all glyph bounding boxes. (Glyphs without contours are ignored.)
int16xMaxFor all glyph bounding boxes. (Glyphs without contours are ignored.)
int16yMaxFor all glyph bounding boxes. (Glyphs without contours are ignored.)


Problem Analyzing Certain Variables

Doesn't detect i here (can't tell you the type or rename it):
package main

main :: proc() {
    i := int(5)
}

same with rune_ here, although it works just fine if you iterate over a []int instead:
package main

main :: proc() {
    for rune_ in "test" {}
}

Fixed both problems.


UI HTML could be improved

I was promised a search box & some data presentation layer ;-)

Where was that promise?  lol - but, good point and will take to heart here.

you set the expectations high :+1:
No big deal at all though, it works. People get it.
I'll be writing a blog post like that with some React front end probably though so don't worry - be happy.

The server part of this repo has been moved to its own repo, and this ticket migrated to https://github.com/mongodb-developer/atlas-search-java-server/issues/1


Twig default filter highlighting

When the default filter has no parenthesis (defaults to null) then the syntax highlighting doesn't work.
# Nice syntax highlighting
blah|default()

# No syntax highlighting
blah|default

Hey @ben-rogerson, I have just pushed an update to Twig Language 2 with a brand new formatter using Prettydiff 3 which brings about some major bug fixes. I'm not sure how to recreate this bug as seems to working correctly on my system with blah being coloured, however, I will look into it further for you. Sorry about the snail's pace development speeds but hopefully these updates have fixed some of your other problems.

Thanks for the reply @mblode, I'll be trying out the new version shortly 👍

Hey Matt, I updated and I don't think this was changed.
Version 0.8.10
You can see here that the same highlighting issue is present:
{{ blah|default }}

{{ blah|default() }}


base URI fixup for XInclude in the shell wrappers

How to do base URI fixup for XInclude on the Saxon command line interface and the shell wrappers of this project.
See the http://apache.org/xml/features/xinclude/fixup-base-uris feature of the Xerces parser in the documentation: https://xerces.apache.org/xerces2-j/features.html

added 15m of time spent

Commit 2ed1c4f tried to add this, but no success.

Switching back to Saxon 10 didn't solve this.

It was an issue in the TEI data, which had an xml:base attribute on the xincluded element, for what reason ever.


OCPBUGS-42434: Authenticate CAPZ with cert authentication on ARO HCP

What this PR does / why we need it:
This PR authenticates CAPZ with certificate authentication in order to communicate with Azure Cloud API. The certificate is stored in an Azure key vault and mounted into the capi-provider pod through a Secrets Store CSI driver SecretProviderClass.
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #
Checklist

[ ] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/close
in favor of https://github.com/openshift/hypershift/pull/5160


Uppercase letters stripped when copy&paste text to Windows Powershell tab

Environment
Windows build number: 10.0.19042
Windows Terminal version (if applicable): 1.4.3243.0

Relevant part of settings.json
...
"profiles":
    {
        "defaults":
        {
            // Put settings here that you want to apply to all profiles.
            "fontSize": 14,
            "fontWeight": "normal",
        },
        "list":
        [
            {
                // Make changes here to the powershell.exe profile.
                "guid": "{61c54bbd-c2c6-5271-96e7-009a87ff44bf}",
                "name": "Windows PowerShell",
                "commandline": "powershell.exe",
                "startingDirectory": "z:\\",
                "hidden": false
            },
...
    "actions":
    [
        // Copy and paste are bound to Ctrl+Shift+C and Ctrl+Shift+V in your defaults.json.
        // These two lines additionally bind them to Ctrl+C and Ctrl+V.
        // To learn more about selection, visit https://aka.ms/terminal-selection
        { "command": {"action": "copy", "singleLine": false }, "keys": "ctrl+c" },
        { "command": "paste", "keys": "ctrl+v" },
        { "command": "unbind", "keys": "ctrl+shift+v" },

        // Press Ctrl+Shift+F to open the search box
        { "command": "find", "keys": "ctrl+shift+f" },

        // Press Alt+Shift+D to open a new pane.
        // - "split": "auto" makes this pane open in the direction that provides the most surface area.
        // - "splitMode": "duplicate" makes the new pane use the focused pane's profile.
        // To learn more about panes, visit https://aka.ms/terminal-panes
        { "command": { "action": "splitPane", "split": "auto", "splitMode": "duplicate" }, "keys": "alt+shift+d" }
    ]
}

Steps to reproduce

Open Windows Powershell tab & Copy with Ctrl+C and paste with Ctrl+V (or with right mouse button) english text with some uppercase letters
Expected behavior

Uppercase letters preserved
Actual behavior

Uppercase letters stripped

I can't repro this. Are you using PSReadLine?

I can't repro this. Are you using PSReadLine?

Nevermind. The issue has been resolved after applying pending Windows update and rebooting PC.

Duplicate of  #6503, I can repro both of them.


Prune nodes more rapidly when there are many

The gateway was having a problem with a large node list, where many of the nodes
on that list were unconnectable, but the pruning was not happening fast enough
to keep the node list reasonably useful. Now, the gateway will prune nodes at a
more rapid rate when there are a lot of nodes.
This is an incomplete fix, we also need to extend the dial bit to do the full
handshake, merely seeing that a node is online is not enough if that node is not
running a recent version.

This PR needs two additional things, first it needs an extension of the g.dial bit in the purge loop to verify that the other node is in fact a node running Sia, and on a recent enough version. Secondly, it needs testing.

comments addressed.


License fee equity waiver page

Create a page that allows people to make a preliminary determination if they are eligible for having their license fee waived through the SB 160 cannabis equity program.

@wrendo Since we're halfway through December and we'd discussed a December 31 due date, I wanted to check in on the priority of this work. Current status is that we sent two drafts to Christina and Eugene.

Version that links to a zip code checker site
Version that lists the residence requirements, but does not offer people ways to check some of them

I haven't seen any feedback from them. How much do we want to follow up with them, especially with end of the year time off?

I'll follow up on this with Christina. I do know she's taking a couple of weeks off for the holidays, so hopefully I can catch her before she leaves.

The page has launched, so closing this issue as done.


Feature request: upload file once

wandb --version && python --version && uname
wandb, version 0.8.13
Python 3.7.4
Linux
Description
The file save function only supports "live" and "end" policies, but I was wondering if "once" policy could be added. Where the when wandb.save("test.txt", policy="once") is called, "test.txt" is uploaded once and it happens immediately with no future tracking or updates on this file.
Use case: sometimes I modify the files in a way that is not easily characterizable by hyper-parameters. It is desirable if the particular version of this file is uploaded. However, I lately realized that, because of the default "live" policy, the file is uploaded and overwritten again as I modify the file, which is not desirable in my use case.
In a way, it almost sounds like sync is a more proper name to this function than save because of the update policy. save should perhaps be more low-level that it saves the file as what it sees at that moment when it was called.
I also suspect that the policy "end" is perhaps not necessary. If the user desire to save the files in the end, why couldn't they just call wandb.save("test.txt") just before the script finishes?
In any case, I think it would perhaps be desirable to have two separate functions
# upload the file as it changes, overwriting the previous version
wandb.sync("test.txt")
# upload the file once and immediately
wandb.save("test.txt")

Sorry for the delay, this is a great feature request.  I think the easiest implementation is to add a "once" policy.  I like the sync api idea, we'll target that for 1.0.  As an immediate fix for your usecase you can copy the file to wandb.run.dir then call wandb.save("filename") to apply the "live" policy to it.  Then just don't copy the file when you modify the original.

Hey @vwxyzjn
In the past year we've majorly reworked the CLI and UI for Weights & Biases. We're closing issues older than 6 months. Please comment to reopen.


$scope.selected apply on children

I have this $scope.selected = array[2]; to select a Node. When I choose $scope.selected = array[10] for example it doesn't select the Node which is a child.

can you provide more information? What you are doing should work, without a running example (or at least the code) I cannot help

Hi yoavaa, thanks for your response. I have another solution for this issue. Can be closed :).


Extending IdentityManager RESTful API

Is possible to extend, for example, the UserController by adding new methods?
I would like to implement new actions that will expose some BrockAllen.MembershipReboot.UserAccountService class methods.
Or is there another better approach to do it?
Thanks

Well, the UI wouldn't know about any of these. The way I'd suggest doing this is via custom metadata properties.

My idea is to implement the BrockAllen.MembershipReboot features into my IdMgr(or somewhere on the server taking advantage of its settings like database connection and another stuff) to be consumed in another application. At this moment i´m not going to create any UI in the IdMgr, just only RESTful APIs.

Well, you're extending IdMgr's APIs then to do so. The only place IdMgr allows for custom stuff is via the metadata config.

Thanks Brock


Auto-select the Expensify Card tab when first creating a workspace

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Join the freePlan beta
Create a New Workspace via the + icon in NewDot
Name the Workspace and arrive at the workspace menu with no tab auto-selected

Expected Result:
The Expensify Card tab of the Workspace should be Auto-selected when a user has just created one for the first time.
Actual Result:
No tab is selected when a user first sees the Workspace, and they arrive at the menu.
Platform:
Where is this issue occurring?

[X] Web
[X] iOS
[X] Android
[x] Desktop App
[X] Mobile Web

Notes/Photos/Videos:
Notice no tab is selected when first creating the workspace.

Expensify/Expensify Issue URL: https://github.com/Expensify/Expensify/issues/171670#
View all open jobs on Upwork

Triggered auto assignment to @sonialiap (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

Proposal

This can be easily fixed by using constants strings inside menuItems
https://github.com/Expensify/App/blob/b08f80f26c0574e54f96677afda54b82a694e363/src/pages/workspace/WorkspaceSidebar.js#L54
We can change this line with    isActive: Navigation.isActive(CONST.WORKSPACE.CARD)

And a new isWorkspaceActive function can be use inside Navigation.js

@aman-atg That seems like a hack. Because Navigation.isActive() isn't working as intended.

@rushatgabhane  IMO selecting sidebar items after parsing and comparing is just an overkill for a simple task.

I mean, with this proposal you'll have to create a new constant and function every time we want to check if a route is active.
Does that make sense?
Anyway, check this out, getCurrentRoute().path is undefined the first time we visit the url.
(But works properly on refresh/ clicking Expensify Card)
https://github.com/Expensify/App/blob/b08f80f26c0574e54f96677afda54b82a694e363/src/libs/Navigation/Navigation.js#L133

new constant and function every time we want to check if a route is active

I don't know if you're listening to yourself ..

Anyway, check this out, getCurrentRoute().path is undefined the first time we visit the url.
(But gets defined on refresh/ clicking Expensify Card)

And yeah I know that. What are ya trying to say here?
gn

While I feel this issue is an issue with react-navigation at the first glance.
** Proposal **
We can modify the isActive function that'll look something like this
function isActive(routePath) {
	let path = '';

	if (navigationRef.current && navigationRef.current.getCurrentRoute().path) {
		path = navigationRef.current.getCurrentRoute().path.substring(1)
	} else {
		let pathFromState = parsePathFromPathString(getPathFromState(navigationRef.current.getState()));
		if(pathFromState) {
			path = pathFromState;
		}
	}

	return path === routePath;
}

function parsePathFromPathString(pathString) {

	// Parse path value from the following path string returned state
	// /WorkspaceSettings?initial=true&screen=WorkspacePeople&params=%5Bobject%20Object%5D&path=%2Fworkspace%2F51594D80CA931BBB%2Fpeopl
}

@mananjadhav We maintain currentURL as an ONYX key.
isActive() function can be a one-liner. What do you think?

That’s the first thing that I checked and it works too.
I thought we would be using isActive later for internal screen navigation too. It it is going to be route based then Onyx works.

Approach with Onyx:
Navigation.js
function isActive(routePath) {
  // We remove First forward slash from the URL before matching

  let currentUrl = getCurrentURL();
  const path =
    navigationRef.current && navigationRef.current.getCurrentRoute().path
      ? navigationRef.current.getCurrentRoute().path.substring(1)
      : currentUrl
      ? currentUrl.substring(1)
      : '';
  return path === routePath;
}

actions/App.js
let currentUrl;
Onyx.connect({
  key: ONYXKEYS.CURRENT_URL,
  callback: (val) => (currentUrl = val),
  initWithStoredValues: false,
});

function getCurrentURL() {
  return currentUrl;
}

While I tested and it worked fine. I am just concerned about one thing. Onyx uses a persistence layer with localStorage on the web. This is generally shared by the domain/application. So any updates on one open tab shouldn't affect the other.

Triggered auto assignment to @iwiznia (Exported), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

. Because Navigation.isActive() not working is the root cause.

I agree with this, given that when we create the workspace, we do:
Navigation.navigate(ROUTES.getWorkspaceCardRoute(response.policyID));

And then in the workspace card route, we do:
Navigation.isActive(ROUTES.getWorkspaceCardRoute(policy.id)),

So, that should return true... what I am not sure about is why this is happening... like why after changing the to the correct route/page the navigationRef is not properly updated?

That's right and I did check that when it gets redirected first, the path is undefined in getCurrentRoute.
I checked until the linkTo where we dispatch the action. The payload matches exactly when we redirect from Create Workspace and also when I reload the page with the WorkspaceCardRoute path.
I gave some workaround solutions, but I feel this issue might require a little more digging.

@iwiznia

So, that should return true... what I am not sure about is why this is happening... like why after changing the to the correct route/page the navigationRef is not properly updated?

It might be happening because we're passing an empty object here
https://github.com/Expensify/App/blob/69d740aac6104126a210763c3fbd0dda88f72772/src/libs/Navigation/AppNavigator/WorkspaceSettingsDrawerNavigator.js#L17

So, still the solution will same, i.e. , passing a constant string so that it can be used later.
One example is here,
https://github.com/Expensify/App/blob/cd663c70a34a4fd1c5d2d77b41d568fea825cc23/src/libs/Navigation/AppNavigator/AuthScreens.js#L391

@aman-atg passing a param sounds like a hack, this seems like something relating to our navigation is not totally correct. After you do Navigation.navigate(ROUTES.getWorkspaceCardRoute(response.policyID)); and right before the route is rendered, any call to Navigation.isActive(ROUTES.getWorkspaceCardRoute(policy.id)) should succeed.
So as @mananjadhav said, we need a bit more digging to see what exactly is going on.

This seems an issue with React navigation. The path is not set on the first navigation.

At the first attempt, I feel this line is what must be causing.
https://github.com/react-navigation/react-navigation/blob/6d518a46b89496f4a3bfd2da24245fe344f97290/packages/routers/src/CommonActions.tsx#L68

Can you expand on that @mananjadhav?

I am not 100% sure of it. I only traced visually without executing. In the navigate action for this line, it only returns the name and the param, not the path. In the other case, it returns the complete payload.
Hence when I compared the payload we dispatch it is exactly the same. Hence I didn't comment my findings earlier in the thread.

@iwiznia @parasharrajat While I couldn't find the root cause yet, I did find a way to get the correct path which is not a hack.
Based on the discussion in this thread #https://github.com/react-navigation/react-navigation/issues/9312
import { useLinkBuilder } from '@react-navigation/native';

function isActive(routePath) {
	const buildLink = useLinkBuilder();
    // We remove First forward slash from the URL before matching
    const path = navigationRef.current && navigationRef.current.getCurrentRoute().name
        ? buildLink(navigationRef.current.getCurrentRoute().name, navigationRef.current.getCurrentRoute().params).substring(1)
        : '';
    return path === routePath;
}

The getPathFromState seems to be a problem. When I logged the path returned in NavigationRoot and in the Navigation.isActive it gives me different results.
NavigationRoot.parseAndStoreRoute
const path = getPathFromState(state, linkingConfig.config);
// Outputs:
// /workspace/51594D80CA931BBB/card

and Navigation.isActive
const path = getPathFromState(navigationRef.current.getState(), linkingConfig.config); // or even useNavigationState
// Outputs:
// /workspace?initial=true&screen=WorkspaceCard&params=%5Bobject%20Object%5D&path=%2Fworkspace%2F51594D80CA931BBB%2Fcard

We can safely use useLinkBuilder and close this.

I would say that missing path is not a issue as this is optional property. So we can't blame React navigation for that.
@mananjadhav  Your solution seems a good to me. Building the path based out of active route is fine.
But useLinkBuilder is a hook. you have unfold it and then use it which is no issue. Internally it uses the same getPathFromState.

I agree. path is an optional property. Yeah, I'll rename the method along with the comments.  Yeah, I am aware internally it uses getPathFromState. But it'll be better if we use the hook rather than parsing the path. If you see the previous comment and the link, getPathFromState is giving different results.  useLinkBuilder would be a safe choice

@iwiznia Any feedback on the last approach ?

So does that solve the problem? Seems like no, since you say it's generating different results?

When I logged the path returned in NavigationRoot and in the Navigation.isActive it gives me different results.

Using getPathFromState directly gives me an issue. Using useLinkBuilder from react-navigation resolves it. There are few additional lines on fetching the root navigation state in useLinkBuilder.

Oh I see... I am not sure I 100% understand the fix, but seems you do and that it works, so I'll check it out and test it when you submit it.
@kevinksullivan can you hire @mananjadhav in upworks please?

Sure. Its after a lot of digging this one works properly rather than going through the getCurrentRoute approach.

While @kevinksullivan posts a job and gives a shoutout, I've raised the PR to you @iwiznia. I've also attached relevant screencasts to show the outcome.

here you are @mananjadhav
https://www.upwork.com/jobs/~0193613b388ad040c2

Hired!

@iwiznia Eep! 4 days overdue now. Issues have feelings too...

PR has been merged Melvin, leave me alone.

Paid @mananjadhav . Thanks!


[experiment]

experimenting with shared mempool outbound RPC

:umbrella: The latest upstream changes (presumably 621ac3ea8904692fd53058b1a15c4797f785ef49) made this pull request unmergeable. Please resolve the merge conflicts.

:umbrella: The latest upstream changes (presumably 630bd498e77e39941ea933d13198d4e7a01043d3) made this pull request unmergeable. Please resolve the merge conflicts.

:umbrella: The latest upstream changes (presumably 88f5b5bbb0dab0d435cdd7210e30d0e563c8491b) made this pull request unmergeable. Please resolve the merge conflicts.

Closing this out as its been sitting here for a looong time. Feel free to re-open.


console.clear() doesn’t work in the Playground (on Chrome)

https://6to5.org/repl/#?experimental=true&playground=true&evaluate=true&loose=false&code=console.clear()
I’m getting:
Uncaught TypeError: Illegal invocation

This is most likely due to the 'capturingConsole' used to grab output and place it into the 'console' div in the REPL.  We'll need to beef up that decorator to support more console methods (it doesn't support anything outside of log right now...i.e. console.error('string') fails with the same error).

mm, we should add some more methods and alias the rest of the console methods on the capturing console.
_(console).functions().each(function(name) {
  capturingConsole[name] = function() {
    console[name].apply(console, arguments);
  };
});

capturingConsole.log = ...;
capturingConsole.info = ...;
capturingConsole.assert = ...;
capturingConsole.clear = ...;

@thejameskyle Probably best to just alias error and info to log. console.clear can be trivially implemented with just buffer = [];.

yeah that sounds good


Added Myntra as a User

Myntra heavily uses Debezium for ingesting CDC Data.
Added Myntra as a User -- Update community/users.asciidoc

@dyaneshvarun Nice, thanks a lot!


message.guild.channels not returning array properly.

running console.log(message.guild.channels.array.length); inside of the client.on("message",function(message) function just returns 0 / null. Not sure what is happening.

array is a function not a property.

Or you could just use message.guild.channels.size

Collection#array is a method, not a property. You don't need to use that, though. Instead, use message.guild.channels.size.


Fix VirtualizedTable columns widths

Fixes: https://issues.redhat.com/browse/HAC-1818

Fixed broken column widths across VirtualizedTable.
Removed info option from table columns - it is broking this fix. We should rather pass the info button within the title element.

@vidyanambiar
@florkbr

Codecov Report

Merging #130 (3e14479) into main (b7d1136) will decrease coverage by 0.11%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #130      +/-   ##
==========================================
- Coverage   17.79%   17.68%   -0.12%
==========================================
  Files          40       40
  Lines        1388     1397       +9
  Branches      358      361       +3
==========================================
  Hits          247      247
- Misses       1141     1150       +9

Impacted Files
Coverage Δ

...tils/src/components/list-view/ListView.stories.tsx
0.00% <0.00%> (ø)

...ib-utils/src/components/table/VirtualizedTable.tsx
0.00% <0.00%> (ø)

...tils/src/components/table/VirtualizedTableBody.tsx
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b7d1136...3e14479. Read the comment docs.

@fhlavac Sure, you can refer to the hac-infra code in this branch - https://github.com/openshift/hac-infra/compare/main...vidyanambiar:hac-infra:ws-list-mockdata?expand=1
specifically src/components/WorkspaceList/WorkspaceList.tsx and src/components/WorkspaceList/WorkspaceListConfig.tsx
I applied the sdk utils package built from this PR on both HAC Core and HAC Infra's @openshift/dynamic-plugin-sdk-utils node_module. Then started HAC Core with ENVIRONMENT=prod yarn dev and hac-infra with npm run start:federated and opened https://prod.foo.redhat.com:1337/beta/hac/workspaces in the browser.

@vidyanambiar ah I probably forgot to mention. For the fix to work, you need to use Td element exposed from the SDK instead of the PF one. We need to add some classes there to fix the styling. I know it's not the best solution but temporarily there is probably no better way. Please try it if you have time.

@vidyanambiar it seems to be working with that change

Thanks @fhlavac. It looks much better with the Td from the sdk 👍 .
However, Td is not exposed via index.ts. To test this I had to add an index.ts file under packages/lib-utils/src/components/table to export Td and also update packages/lib-utils/src/index.ts accordingly. Could you please include that change in the PR?
Also, I do still see a partially hidden row action selector at the end of the row though. Could you take a look at that as well please?

I think it would help to include a Readme with guidance on special instructions such as this one (about using the Td from the SDK) as more folks are starting to use the ListView. In a future PR perhaps? WDYT? @fhlavac
cc: @florkbr

@vidyanambiar I added the exports after your first comment, it should be already exposed. Yeah, the row actions overflow is the bug I mentioned, preparing a fix. I will also update README. Thank you :slightly_smiling_face:

oops, Sorry I missed it!

/retest

@fhlavac there is a linter error :(
/go/src/github.com/openshift/dynamic-plugin-sdk/packages/lib-utils/src/components/table/VirtualizedTable.tsx
  3:10  warning  'Td' is defined but never used  @typescript-eslint/no-unused-vars

✖ 1 problem (0 errors, 1 warning)

@florkbr fixed the error, thank you. I'll take another look to try fixing the action cell width

FWIW the table column header misalignment issue only occurs with the virtualized table (disconnect between the table body and the header). For now this LGTM

/lgtm

/approve


Product feedback

Product feedback link point to a non-existing page on uservoice

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

ID: c18085b0-ca97-5623-fc42-5395acf55f1f
Version Independent ID: 942d682c-a471-a8c9-e12f-cc7d4aafbad3
Content: Support and troubleshooting information for Cloud App Security
Content Source: CloudAppSecurityDocs/support-and-ts.md
Service: cloud-app-security
Product: ****
Technology: ****
GitHub Login: @shsagir
Microsoft Alias: shsagir

@kennethvs Thanks for contacting Microsoft with feedback about our product documentation. Depending on the complexity, it could take from a few business days to a few weeks to look further into your question, issue, or suggestion. We'll contact you if we need further information, and you'll receive an email message each time we "@mention" you in a comment.
Thanks again for submitting your feedback. Your suggestions help improve our documentation.

@kennethvs Thanks for your feedback. We've submitted a documentation change based on your suggestion, and the updated article should be live by tomorrow. Please let us know if there's anything more we can do for you. To follow up on this issue, leave us a comment with an @mention.


GROOVY-9945: findParameterizedType: fill type parameters of super types

https://issues.apache.org/jira/browse/GROOVY-9945
There is quite a history to findParameterizedType and related methods.  I did not fully understand the reason for some of the looser checks.  I think they were to account for incomplete generics, which are now more complete.  Rather then building the whole type hierarchy at once, I took an approach of build as needed.  This should reduce some of the cost of searching.  I would have reduced the method count further, but it's all public api now.

Merged. Thanks!


Windows 10 Always-VPN clients - must be Active Directory joined?

In the "Windows 10 VPN Clients" section on this page it states that: "The Windows 10 VPN clients must be domain-joined to your Active Directory domain."
However, I am working with Win10 1909 devices that are only Azure AD joined and Always-On VPN is working just fine.  Can you update the page if Always-On VPN is officially supported with AAD Joined devices - Thanks!

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

ID: 6ecf5e50-aeae-4c29-dc40-e1222d17a4ed
Version Independent ID: bb4d5a7f-7805-e8b9-7c27-a161f59f8061
Content: Always On VPN Technology Overview
Content Source: WindowsServerDocs/remote/remote-access/vpn/always-on-vpn/always-on-vpn-technology-overview.md
Product: windows-server
Technology: networking-ras
GitHub Login: @Teresa-Motiv
Microsoft Alias: v-tea

@eross-msft : Even though the sentence could be interpreted as meaning that any Active Directory domain is a valid AD to be joined to, would it be practical to get this verified for Azure AD and update only that paragraph, or should we check for a planned general update for the whole document?

@illfated  We currently have no way to address GitHub issues (though I'm trying to figure out a plan), so if you want to create a PR to address this singular issue, I can certainly get that updated. Unfortunately, there is no one on point to review this content, so an overall review would be a tough promise.

Fair enough, thank you for the insight. I will dwell on these facts and consider the implications before I start writing something which might be useful enough for this specific case.

On Jun 1, 2020, 9:23 PM GMT+2, @tfurstms wrote:
In the "Windows 10 VPN Clients" section on this page it states that: "The Windows 10 VPN clients must be domain-joined to your Active Directory domain."
However, I am working with Win10 1909 devices that are only Azure AD joined and Always-On VPN is working just fine. Can you update the page if Always-On VPN is officially supported with AAD Joined devices - Thanks!

Could we have this confirmed, somehow, maybe from the dev team, that this document could benefit from an added statement that "only Azure AD joined an Always-On VPN" is a valid supported solution and useful part of the referenced document?

@tfurstms Please ask your question at Microsoft Q&A. https://docs.microsoft.com/en-us/answers/products/