Add wildcards to reduce IDEA warnings

JSON Class 代码中泛型增加通配符, 减少IDEA警告

Codecov Report

Merging #63 (db7dac1) into main (a27fb2d) will decrease coverage by 1.13%.
The diff coverage is 88.46%.

@@             Coverage Diff              @@
##               main      #63      +/-   ##
============================================
- Coverage     64.91%   63.77%   -1.14%
+ Complexity     8061     7930     -131
============================================
  Files           420      420
  Lines         45443    45308     -135
  Branches       9168     9137      -31
============================================
- Hits          29499    28897     -602
- Misses        12389    12891     +502
+ Partials       3555     3520      -35

Impacted Files
Coverage Δ

...rc/main/java/com/alibaba/fastjson2/JSONWriter.java
71.52% <ø> (-1.61%)
:arrow_down:

...alibaba/fastjson2/reader/ObjectReaderImplEnum.java
72.50% <ø> (-1.31%)
:arrow_down:

...ibaba/fastjson2/writer/ObjectWriterBaseModule.java
79.16% <0.00%> (-0.24%)
:arrow_down:

.../alibaba/fastjson2/writer/ObjectWriterCreator.java
76.65% <0.00%> (-2.17%)
:arrow_down:

...ibaba/fastjson2/writer/ObjectWriterCreatorASM.java
96.42% <0.00%> (-0.19%)
:arrow_down:

core/src/main/java/com/alibaba/fastjson2/JSON.java
85.11% <97.87%> (-0.86%)
:arrow_down:

...n/java/com/alibaba/fastjson2/util/UnsafeUtils.java
30.18% <0.00%> (-49.06%)
:arrow_down:

...main/java/com/alibaba/fastjson2/util/JDKUtils.java
40.25% <0.00%> (-30.07%)
:arrow_down:

...va/com/alibaba/fastjson2/reader/ValueConsumer.java
16.66% <0.00%> (-16.67%)
:arrow_down:

...in/java/com/alibaba/fastjson2/JSONReaderASCII.java
66.66% <0.00%> (-14.13%)
:arrow_down:

... and 49 more

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 8589ec0...db7dac1. Read the comment docs.


Add VK_NULL_HANDLE vertex buffer binding support

This change enables VK_EXT_robustness2 features so that VK_NULL_HANDLE may be bound to a vertex buffer binding in order to render with just an index buffer. See #41
The first commit of this change adds a draft function for testing, bind_null_vertex_buffer. This function will very likely be replaced with something nicer before merge.
TODO:

[x] Test using fuzzer.rs
[ ] Expand AnyBufferBinding or make a similar enum for these cases
[ ] Gracefully handle unavailable features (test on such hardware)

Although this is entirely possible to do, I don't know of any applicable use-case for drawing from a null buffer. Closing until there is a reason to do this.
Drawing with no buffer specified (such as quads driven by gl_VertexIndex) is possible - see examples/multipass.rs.


how to pass in the conference_max_minutes value?

@emrahcom is there a way to pass in the conference_max_minutes value as a JWT param?

Yes, you can put it in the context block in the token


Add ALTER TABLE statement

Altering a table is one of the features that Genji currently needs but depending on the system, this feature has not the same set of options.
For example, in sqlite you only have the possibility to rename a table or add a new column while in postgres you can do more operations like dropping a column, setting a new type for an existing column etc...
I suggest that for a first iteration, we could add only two operations (renaming a table and adding a new field) and iterate in the future for adding more.
Renaming a table
genji> CREATE TABLE foo;
genji> INSERT INTO foo VALUES {name: "John Doe"};
genji> ALTER TABLE foo RENAME TO bar;
genji> SELECT * FROM bar;
{
  "name": "John Doe"
}
genji> SELECT * FROM foo;
table not found

If the table bar already exists, we should return an error.
Adding a new field
genji> ALTER TABLE bar ADD FIELD age int;

If the field age already exists, we should return an error.
The only constraint supported in the first iteration would be NOT NULL but a default value must be specified for the existing documents.
genji> ALTER TABLE bar ADD FIELD age int NOT NULL DEFAULT 50;
genji> SELECT * FROM bar;
{
  "name": "John Doe",
  "age": 50
}

What do you think?

I think that this issue is quite big and it can be split into two sub-issues. Closing it.


add footer

Context
Link to Notion ticket
Additional information
It's a footer
Testing (tester)
The reviewer should check on this branch:

The code
The site in CloudCannon (link here)

Before merge (PR owner)

[ ] Delete the test site in CloudCannon

Made a couple changes:

Fix the overflow that @aleksandrgordeev mentioned (not ideal for word logos but better than nothing)
Switch to using the footer data file rather than the nav data file
Allow links/bold text/italic text in the copyright section


Use snapshot testing

WHY
WHAT
REF

Codecov Report

Merging #34 into master will increase coverage by 5.08%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #34      +/-   ##
==========================================
+ Coverage   21.69%   26.78%   +5.08%
==========================================
  Files          35       35
  Lines        1051     1053       +2
==========================================
+ Hits          228      282      +54
+ Misses        808      745      -63
- Partials       15       26      +11

Impacted Files
Coverage Δ

pkg/grapicmd/internal/module/generator/service.go
71.42% <100%> (-3.04%)
:arrow_down:

pkg/grapicmd/internal/module/generator/base.go
37.5% <0%> (+31.25%)
:arrow_up:

...picmd/internal/module/generator/template_string.go
55.55% <0%> (+55.55%)
:arrow_up:

pkg/grapicmd/internal/module/generator/status.go
63.63% <0%> (+63.63%)
:arrow_up:

pkg/grapicmd/internal/module/generator/command.go
75% <0%> (+75%)
:arrow_up:

pkg/grapicmd/internal/module/generator/project.go
84.61% <0%> (+84.61%)
:arrow_up:

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 3546f7c...fbdcb77. Read the comment docs.


Vibration/Rumble bug

I experience this bug when using DualShock 3 in bluetooth mode, in native PC games and in RPCS3 as well.
Basicly what is happeing is that when a vibration need to happen it does but the controller doesnt stop vibrating until i perform an action again which triggers a vibration.
I am using a USB Bluetooth dongle, and this "bug" doesnt happen when using a cable to play with DualShock 3

This is explained here, which is as good as it gets.

Thanks for the fast response! I'll check it out!


Event and background colors and shading problem

There is only one setting for event text shading but different color options for event background, past event background and future background.
This limits choices for background colours
Perhaps some sub-menus in settings and more colour options would solve this.
e.g.
Settings | Colors | Past

Day header text shading
Event text shading
Event background color

Settings | Colors | Today

as above

Settings | Colors | Future

as above

I agree there are already a lot of settings so perhaps there is an easier way to implement this....
Maybe day header text shading and event text shading could be just one option. ie. they are the same shading. This would create only 2 color options in the settings structure above, rather than 3.

@bushrang3r Thank you for suggestions.

Regarding settings structuring and ordering.
As you may have noticed, I changed an order of settings in each section (mostly) to correspond to the order of elements on the screen: starting from Widget header and down...
Similarly we do have "Past events" and "Current and future events" sections inside "Event filters". So I think that structuring Colors settings into "Past events", "Today's events" and "Future events" will be logical and will really help to find needed settings.

Regarding day header shading vs event shading.
I separated them only recently in #218. Previously Day header color was dependent on event color, but as some users noticed, is was always light and never black...
Now we can set all widget text black or white thus adjusting for different backgrounds.

And finally: Why do we have "Text shading" with only 4 values and not continuous "Text color" with the same color picker as for backgrounds?
Actually "shading" means changing a "style", i.e. a group of attributes, together, including switching between white and black icons. See the four "Shadings" here: https://github.com/andstatus/todoagenda/blob/master/app/src/main/res/values/themes.xml

OK, thanks. I understand.

The problem with this (as you can see in my screenshot) is that if my header text shading or event text shading is dark or black, I can't have any dark backgrounds for past, today, or future. They must all be light colours.

@bushrang3r Yes, I see. So we need 2 * 3 text shading settings instead of 2.
I will do this.

Done in v.3.0.5 Please check in #320
In order to minimize number of settings the app hides color settings that are not actual.  It hides color settings for Day headers if "Show Day headers" setting is turned off. "Past events" section is removed completely if no past events may be shown and no task sources selected.

@bushrang3r I like this extention so much that I even started to consider changing default today's colors. Previously all events where white text on dark background.
Now we can empathize today by inverting the colors. I found that such a change really makes today easily visible.
What do you think?

@yvolk
Yes! It's wonderful! I love it. It's great to be able to show past, today, and future with different colours. Very easy and quick to differentiate events and days. It's very customizable now. I'll keep testing this beta and keep using it everyday.
Thanks for all your hard work!


Fix plot and provenance of recipe_consecdrydays

Description

Closes #2242

Before you get started

[x] ☝ Create an issue to discuss what you are going to do

Checklist
It is the responsibility of the author to make sure the pull request is ready to review. The icons indicate whether the item will be subject to the 🛠 Technical or 🧪 Scientific review.

[x] 🛠 This pull request has a descriptive title
[x] 🛠 Code is written according to the code quality guidelines
[x] 🛠 Tests run successfully
[x] 🛠 The list of authors is up to date
[x] 🛠 All checks below this pull request were successful

New or updated recipe/diagnostic

[x] 🧪 Recipe runs successfully
[x] 🛠 Provenance information has been added

To help with the number of pull requests:

🙏 We kindly ask you to review two other open pull requests in this repository

@esmvalbot Please run recipe_consecdrydays.yml

Thanks for reviewing @Emmadd! Does the documentation look better now?


cannot use "cuda" with the python codes

It says that data could not be transferred to device. Let me now if there is a fix around this. Thanks!

Will try to create the same for the new pyTAGI as it seems to be running well on both cpu and CUDA.

@miquelflorensa and @jamesgoulet are you facing the same issue? I am off the grid for a couple of days. I will address the issue in a couple of days if it won't be resolved

@lhnguyen102 Yes, I have tried and I run in the same issue. I can run GPU for both the C++ code and the pyTAGI_v1. In case you will deprecate the old version once pyTAGI_V1 will be finalize, we can wait for the new one...

I will take a look at it for a quick fix. In general, you don't really need the CUDA for two fully-connected layer of 50 hidden units.

👌🙏. I think that @bhargobdeka idea was to create config files for the large's UCI regression dataset so that we can post a repeatable baseline.

Hi Ha,
Hoping I am disturbing you in the middle of your vacation :-)
It will be nice to have all the benchmark results available in the Pytagi package with both cpu and CUDA. I will try to put the other methods, e..g., deep ensemble, MC dropout, PBP, Laplace etc in the python package also so that we have a holistic comparison for the regression datasets, both small and large. The large ones need to run with CUDA as it will take forever otherwise.
I will also try to see how to fix the CUDA problem together with Miguel.
Regards,
Bhargob Deka
Postdoctoral Researcher
Department of Civil, Geologic, and Mining
Polytechnique Montréal, QC
@.***
LinkedIn: https://www.linkedin.com/in/bhargob-deka-ph-d-87aba7117/

On Feb 12, 2024, at 6:10 AM, James-A. Goulet @.***> wrote:
👌🙏. I think that @bhargobdeka https://github.com/bhargobdeka idea was to create config files for the large's UCI regression dataset so that we can post a repeatable baseline.
—
Reply to this email directly, view it on GitHub https://github.com/lhnguyen102/cuTAGI/issues/46#issuecomment-1938472174, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQS77FTGUBYTGCLQERZAFKTYTH2D7AVCNFSM6AAAAABC6HNPMKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMZYGQ3TEMJXGQ.
You are receiving this because you were mentioned.

Nevermind! I figured it out

@lhnguyen102 Yes, I have tried and I run in the same issue. I can run GPU for both the C++ code and the pyTAGI_v1. In case you will deprecate the old version once pyTAGI_V1 will be finalize, we can wait for the new one...

@jamesgoulet, I couldn't even run the newest version using GPU on Ubuntu server using the main branch. It gave some weird error... If I recalled correctly, everything works just fine on my desktop. Could you please test it again? Thanks.

I might have a hint. Here is the error
Program hit cudaErrorUnsupportedPtxVersion (error 222) due to "the provided PTX was compiled with an unsupported toolchain." on CUDA API call to cudaGetLastError.

There might be a mismatch between current cuda version and the driver version... my guest is that the current cuda version is 12.3 but the current server is most likely compatible with cuda 12.2. If you test on GPU works, my assumption mentioned-above is wrong

@lhnguyen102 Yes, I have tried and I run in the same issue. I can run GPU for both the C++ code and the pyTAGI_v1. In case you will deprecate the old version once pyTAGI_V1 will be finalize, we can wait for the new one...

@jamesgoulet, I couldn't even run the newest version using GPU on Ubuntu server from the main branch. It gave some weird error... If I recall correctly, everything works just fine on my desktop (I couldn't had access to it now for testing). Could you please test it again? Thanks. Here is my command on terminal
build/main test_fc_mnist

Please make sure uncomment this line  in order to run the test on GPU

It works for me both through the Python API or c++

No worries and understood! @jamesgoulet how CUDA is installed in Ubuntu server? It seems to me that CUDA has not been installed at least for me.
pytagi) lhn@MLCIVS1:~/lhn_tagi/cuTAGI$ nvcc --version
Command 'nvcc' not found, but can be installed with:
sudo apt install nvidia-cuda-toolkit

I could install it if needed but a good practice is to install a version for all users because the size of CUDA installation is large. In addition, I found these instructions are the best to install CUDA on Ubuntu 22.04. I have done that before for @miquelflorensa on my desktop. Note that CUDA version should be 12.2 because of its current Nvidia driver
nvidia-smi
Tue Feb 13 04:01:12 2024
+---------------------------------------------------------------------------------------+
| NVIDIA-SMI 535.154.05             Driver Version: 535.154.05   CUDA Version: 12.2     |

Initially, @miquelflorensa installed it but it seems that it is only installed locally for each user... I had to reinstall it as well following the same instructions you posted.

In theory CUDA toolkit should be installed globally in the server but it is just that in the binding you need to specify that the version is 12.1. And as you mentioned, there could be a mismatch between CUDA driver version that is 12.2 and CUDA toolkit version that is 12.1.

@miquelflorensa Cuda 12.1 works just fine (see @jamesgoulet 's comment). The issue is that the default CUDA version in server is 12.3 which is not compatible with the current driver. Rule of thumbs is that it must either match 12.2 or a bit lower.  Version 12.3 appears in my case because I added this to ~/.bashrc. I haven't installed any new CUDA version to the server.
export PATH=/usr/local/cuda/bin:$PATH
export LD_LIBRARY_PATH=/usr/local/cuda/lib64:$LD_LIBRARY_PATH

Will find a workaround to fix it but the long-term solution is to have a unique working CUDA version for all users. For this case, it could be 12.1 or 12.2 (I preferred 12.2). The rest should be removed in order to avoid any other issues

@lhnguyen102 I see. I removed every CUDA version installed except from 12.1 for now. I also added the export in /etc/environment.

@miquelflorensa Thank you! It all works now

@bhargobdeka I found a quick workaround to you problem. Please replace this line by the following code
ud_idx_batch = np.zeros((batch_size, 1), dtype=np.int32)

The reason is that the original code will create ud_idx_batch as an empty array which will be handled in the upcoming version. I will fix this issue in my upcoming PR. Let me know if it works

@bhargobdeka it is fine if you want to stick to the old version (I will transfer your code to new version later). It would be great if you can use the newest version which is well organized and pytorch-like API but the catch is that there might be bugs here and there

I will give it a try tomorrow. Thanks! I created the regression benchmark with the new pytagi version. It seems to work for both cpu and cuda without error but not there yet in terms of results. I will try to see if I did any mistake otherwise I will discuss with you on what all I did.
Bhargob

On Feb 13, 2024, at 7:59 PM, Luong-Ha Nguyen @.***> wrote:
@bhargobdeka https://github.com/bhargobdeka I found a quick workaround to you problem. Please replace this line https://github.com/lhnguyen102/cuTAGI/blob/main/python_examples/regression.py#L237 by the following code
ud_idx_batch = np.zeros((batch_size, 1), dtype=np.int32)
The reason is that the original code will create ud_idx_batch as an empty array which will be handled in the upcoming version. I will fix this issue in my upcoming PR. Let me know if it works
—
Reply to this email directly, view it on GitHub https://github.com/lhnguyen102/cuTAGI/issues/46#issuecomment-1942926181, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQS77FQNEGIAKT46YLE6DSTYTQD7VAVCNFSM6AAAAABC6HNPMKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNBSHEZDMMJYGE.
You are receiving this because you were mentioned.

Yes and I am not good in C++ at all :/
If the old code works, then I can generate all the benchmarks really quick with cuda.

On Feb 13, 2024, at 8:09 PM, Luong-Ha Nguyen @.***> wrote:
@bhargobdeka https://github.com/bhargobdeka it is fine if you want to stick to the old version (I will transfer your code to new version later). It would be great if you can use the newest version https://github.com/lhnguyen102/cuTAGI/blob/main/pytagi/pytagi_v1/test.py which is well organized and pytorch-like API but the catch is that there might be bugs here and there
—
Reply to this email directly, view it on GitHub https://github.com/lhnguyen102/cuTAGI/issues/46#issuecomment-1942933918, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQS77FQPO6EX3F7JEOSLCHDYTQFEZAVCNFSM6AAAAABC6HNPMKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNBSHEZTGOJRHA.
You are receiving this because you were mentioned.

Both the old code and the new one has the python API. For the benchmark, you wont need to touch the C++/CUDA part (see the link)

yes, I got that part. I will let you know tomorrow morning if your suggestion worked.

On Feb 13, 2024, at 8:23 PM, Luong-Ha Nguyen @.***> wrote:
Both the old code and the new one has the python API. For the benchmark, you wont need to touch the C++/CUDA part (see the link)
—
Reply to this email directly, view it on GitHub https://github.com/lhnguyen102/cuTAGI/issues/46#issuecomment-1942944460, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQS77FT44L5QMUKLYSXAIMDYTQGZLAVCNFSM6AAAAABC6HNPMKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNBSHE2DINBWGA.
You are receiving this because you were mentioned.

However, it is much faster with cuda for larger networks. Here I tried with 2 layers and 1000 hidden units and can clearly see the difference it makes.

On Feb 14, 2024, at 1:48 PM, Bhargob Deka @.***> wrote:
Hi Ha,
The previous pytagi version works with both cpu and cuda without errors. So, first of all, kudos to that.
However, I do not see the advantage in training time with cuda. See the screenshots below. The one with cpu (1.04 sec) is faster than cuda (1.23 sec).
Am I doing something wrong?
Let me know if there are additional things that I missed.
Thanks,
Bhargob
<Screen Shot 2024-02-14 at 1.39.45 PM.png>
<Screen Shot 2024-02-14 at 1.40.59 PM.png>

On Feb 13, 2024, at 8:29 PM, Bhargob Deka @.*** @.***>> wrote:
yes, I got that part. I will let you know tomorrow morning if your suggestion worked.

On Feb 13, 2024, at 8:23 PM, Luong-Ha Nguyen @.*** @.***>> wrote:
Both the old code and the new one has the python API. For the benchmark, you wont need to touch the C++/CUDA part (see the link)
—
Reply to this email directly, view it on GitHub https://github.com/lhnguyen102/cuTAGI/issues/46#issuecomment-1942944460, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQS77FT44L5QMUKLYSXAIMDYTQGZLAVCNFSM6AAAAABC6HNPMKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNBSHE2DINBWGA.
You are receiving this because you were mentioned.


Feature: Show menu when right clicking back/forward buttons to show history

Is your feature request related to a problem? Please describe.
It's can be annoying clicking back (or pressing Alt+left arrow) countless times to go back to a folder I was using previously, it would be nice to have a convenient way to view the history for a tab.
Describe the solution you'd like
I think right clicking the back or forward buttons should show a popup of the tabs history.
Describe alternatives you've considered
Navigate manually.
Additional context
Web browsers have this feature, this is how it is in Edge.

@winston-de this feature will be included in the next release.


[Impeller] Clipping bug in OpenGL backend on wonderous.

Maybe state error with scissor? Tested on Pixel 7 Pro.
https://github.com/user-attachments/assets/83fc1a81-1c90-4989-a482-bea7712ccbeb

This might be related to https://github.com/flutter/flutter/issues/158392.  This is clipping things that shouldn't be clipped, but that is failing to clip things that should be clipped.

This is working correctly after https://github.com/flutter/engine/pull/56494


Server entries look weird without connection info

Testing #14201

Without detail the servers look weird compared to the first one, I wonder if we can have some placeholder like "Never connected" or something similar to indicate that the entry is valid but it's not connected yet. Right now it feels like they are partially broken.

Fixed


ISPN-13444 Json to Protostream conversion does not show real error

https://issues.redhat.com/browse/ISPN-13444

fixed and rebased @karesti


Update api-management-howto-deploy-self-hosted-gateway-to-docker.md

Tried APIM gateway on Ubuntu 19.10 running on Raspi-4 but container error out as "exec format error". Raspi-4 is based on ARM64 arch and not x86_64.

@suhasaraos : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


[Fix] Process halts when body is undefined.

The change will do a null check on the body variable before calling body.forEach()

Could you elaborate on what you meant by that? Did you mean that you want to see the result of all test cases? I surfed online and all mentioned that I have to test the changes against all test cases
Disclaimer: I am new to open source contributions.

@senputra sure! What i mean is, what is this fixing? Without a new test - one that would be failing without your fix - there's no way to verify that the fix is working, and the day after merging this, there's nothing to stop anyone from removing the truthiness check, since no tests would be failing.
Bug fixes in any software project, open source or otherwise, typically include regression tests like this to make sure the fix is verified and the problem can't occur in the future.


sn范围在(0<=n<=7)

https://github.com/Wiznet/ioLibrary_Driver/blob/c5be3a85b9799959c87d2e8897682e43c061bbb1/Ethernet/socket.c#L85

Hi~ @ZhiFengJia
The value of WIZCHIP_SOCK_NUM  is '8' or '4' in wizchip_conf.h
Therefore, it is correct code.


Return error, and consistency in naming

Couple of suggestions as I've been working through the API

I see you also included the name -> id fix, which was missed when those were recently renamed.


changefeed for rethinkdb

Signed-off-by: David Lawrence<EMAIL_ADDRESS>(github: endophage)

@cyli filed an issue to track that https://github.com/docker/notary/issues/1216


Problem with the layers "IBGE - Mapa de setores urbanos" and "IBGE - Mapa de setores rurais"

Hello everyone!
When I try to edit a map in Open Street Map and put the layers "IBGE - Mapa de setores urbanos" and "IBGE - Mapa de setores rurais", the maps for that layer don't appear and the screen for editing goes black.
Do you know what may be happening? What would be the solution to this?
Use the link https://user-images.githubusercontent.com/76920627/103488749-30ba4100-4dee-11eb-94c5-9638d83d85ed.png to see how my editing screen of OSM is going.
Any help will be appreciated!
Thanks.

The map is zoom out quite far in that screenshot. Perhaps the layers don’t support that zoom level – does zooming in further allow the layers to appear?

The map is zoom out quite far in that screenshot. Perhaps the layers don’t support that zoom level – does zooming in further allow the layers to appear?

@1ec5 I zoomed in, but the edit screen, which should have the map, remains black.

@1ec5 I zoomed in, but the edit screen, which should have the map, remains black.

Someone can help me?

Someone can help me?

@GabrielMO-2020 could you add a link to the location and layer using this editor version https://ideditor-release.netlify.app/#background=EsriWorldImagery&disable_features=boundaries&map=2.00/10.7/33.4 which will store location and layer in the URL so we can more easily look at it. – I looked it up, eg here https://ideditor-release.netlify.app/#background=IBGE_Setores_Rurais&disable_features=boundaries&map=14.57/-13.8527/-46.2409
In general, all Layers in iD are pulled from https://github.com/osmlab/editor-layer-index ("ELI"), which is a separate project.
So solving this will most likely have to happen there.
The layer you name, is here https://github.com/osmlab/editor-layer-index/blob/d5bab4f6f6c1a15b9e83d683313570f17dc7fcf7/sources/south-america/br/IBGE_Setores_Urbanos.geojson?short_path=bf8d23f
Some debugging:
When you look at your developer console (https://developers.google.com/web/tools/chrome-devtools) while loading the images, the error code ist 410 "Gone" (https://developer.mozilla.org/de/docs/Web/HTTP/Status/410).
This sounds to me, that this error is not just a temporary hiccup.
Btw, the same is true for the Layers in the JOSM-layer index at https://josm.openstreetmap.de/wiki/Maps, they also throw a 310 for me.
You can look at the history of the files in ELI at https://github.com/osmlab/editor-layer-index/commits/gh-pages/sources/south-america/br/IBGE_Setores_Urbanos.geojson
Maybe the guys from https://github.com/osmlab/editor-layer-index/pull/672 which edited data in Brasil know more (they did not edit those two files directly though).
And in https://github.com/osmlab/editor-layer-index/issues/555 the was an issue with the token that is now part of the URL. That could also be the issue here.
Also, https://wiki.openstreetmap.org/wiki/IBGE_Tile_Layer says "Mapbox kindly hosts this layer" – maybe this changed?
My recommendation is: Open this issue in ELI and see if someone there can help solving it.

@GabrielMO-2020 could you add a link to the location and layer using this editor version https://ideditor-release.netlify.app/#background=EsriWorldImagery&disable_features=boundaries&map=2.00/10.7/33.4 which will store location and layer in the URL so we can more easily look at it. – I looked it up, eg here https://ideditor-release.netlify.app/#background=IBGE_Setores_Rurais&disable_features=boundaries&map=14.57/-13.8527/-46.2409
In general, all Layers in iD are pulled from https://github.com/osmlab/editor-layer-index ("ELI"), which is a separate project.
So solving this will most likely have to happen there.
The layer you name, is here https://github.com/osmlab/editor-layer-index/blob/d5bab4f6f6c1a15b9e83d683313570f17dc7fcf7/sources/south-america/br/IBGE_Setores_Urbanos.geojson?short_path=bf8d23f
Some debugging:
When you look at your developer console (https://developers.google.com/web/tools/chrome-devtools) while loading the images, the error code ist 410 "Gone" (https://developer.mozilla.org/de/docs/Web/HTTP/Status/410).
This sounds to me, that this error is not just a temporary hiccup.
Btw, the same is true for the Layers in the JOSM-layer index at https://josm.openstreetmap.de/wiki/Maps, they also throw a 310 for me.
You can look at the history of the files in ELI at https://github.com/osmlab/editor-layer-index/commits/gh-pages/sources/south-america/br/IBGE_Setores_Urbanos.geojson
Maybe the guys from https://github.com/osmlab/editor-layer-index/pull/672 which edited data in Brasil know more (they did not edit those two files directly though).
And in https://github.com/osmlab/editor-layer-index/issues/555 the was an issue with the token that is now part of the URL. That could also be the issue here.
Also, https://wiki.openstreetmap.org/wiki/IBGE_Tile_Layer says "Mapbox kindly hosts this layer" – maybe this changed?
My recommendation is: Open this issue in ELI and see if someone there can help solving it.

@tordans Thanks for looking into this. It does indeed look like something that needs to be handled in ELI.

@tordans Thanks for looking into this. It does indeed look like something that needs to be handled in ELI.


Possible to localize payment method name

It is possible to config the payment method name (in adyen-checkout__payment-method__header) based on the locale configuration.
eg: Instead of "credit card", payment method name shows "carte bancaire" if locale is "fr_FR".
Thanks!

Hi @lpan42,
Thanks for reaching out. This can be achieved with the name property on any component (through the paymentMethodsConfiguration object). This doesn't depend on a specific locale, but could be a variable you set dynamically with a condition on your side.
const checkout = await AdyenCheckout({
  paymentMethodsConfiguration: {
    card: {
      name: 'Custom card name'
    }
  }
});

Hope this helps!


Use autorest for UI server requests

When merged, this PR will:

Modify server routes to use a design compatible with autorest (no uri query parameters, application/json for all requests/responses)
Use autorest-generated classes for all server queries

The idea behind this is that if/when a route needs to change, swag and autorest should do a lot of the heavy lifting of ensuring that the server and client stay in accord. The endpoints should be fully specified by their annotations, which swag uses to generate an OpenAPIv2.0 spec. That spec file is then used by autorest to generate typescript code used for making client requests.
This allows a lot of things to change about the API routes without having to manually modify any code (such as changing the verbs used for an endpoint, or the static portions of the url). Other changes (such as the number of url parameters or new/different required body parameters) would require writing code, but the amount that would have to be changed should be reduced.

Build currently failing on warning because of lint error in generated code, which cannot be ignored with .eslintrc or .eslintignore because of https://github.com/facebook/create-react-app/issues/2339 (see also: https://github.com/Azure/autorest.typescript/issues/669)

Lint errors solved by creating a script that injects /* eslint-disable */ into the top of generated code files as needed


namespace for Qt port?

I've written Qt4 port before, and now testing to use the source for Qt5 port.
Should I use different namespaces for Qt4 port and Qt5 port, like SVGNative::Qt4SvgRenderer and SVGNative::Qt5SvgRenderer? In my understanding, linking both of Qt4 and Qt5 to single library or binary is not recommended usage (I think it's almost impossible). So, even if I separate Qt4 and Qt5 into different namespaces, libSVGNativeViewer cannot be built well. There might be some plugin technique to link Qt4 only or Qt5 only in run-time. But I'm afraid it's overkilling.
I would propose SVGNative::QtSvgRenderer style naming, but if anybody prefers the versioned namespaces (even if the single version is available), please let me know.

@mpsuzuki Are the rendering APIs in Qt4 and Qt5 incompatible? Kind of hard to imagine that they reinvented drawing :P If possible, absackt the code to be as independent of Qt versions as possible. If needed we could either create 2 separate ports or compiler flags for the differences.

There is Qt's own QtSvg module (that only does SVG Tiny):
https://github.com/qt/qtsvg/tree/dev/src/svg
I wonder if you could create a QtNativeSvg renderer is such a way that it would be easy to adapt to those who have been been using QtSvg)

Ps. Qt5 was released 2012 and a lot of code has been adapted. Qt4 is old. I think it would be best to target Qt5, and treat Qt4 as "legacy".

Sorry for long delayed response!

@mpsuzuki Are the rendering APIs in Qt4 and Qt5 incompatible? Kind of hard to imagine that they reinvented drawing :P If possible, absackt the code to be as independent of Qt versions as possible. If needed we could either create 2 separate ports or compiler flags for the differences.

I did small experiment to reuse my codebase for Qt4 to Qt5, and I found no significant troubles. I think there is no big change in the side of the drawing API in Qt.
However, in the development of the Qt library, the application linked to QtN shared library cannot run with Qt(N+1) shared library. The developer have to relink their applications with Qt(N+1) libraries. And, it is almost impossible to have an application linking both of QtN and Qt(N+1) libraries.
For example, Poppler PDF rendering library has always one or two Qt frontends for each Qt major versions: Qt3 frontend (deprecated, removed from latest source tree), Qt4 frontend (deprecated, removed from latest source tree) and Qt5 frontend. Can I take such strategy?

@twardoch, Thank you for comment!

Ps. Qt5 was released 2012 and a lot of code has been adapted. Qt4 is old. I think it would be best to target Qt5, and treat Qt4 as "legacy".

Yes, I understand. Could you expect when Qt6 would come? It would come soon? If Qt5 would be deprecated soon, I want to postpone the incorporation of Qt port into official tree.

I wonder if you could create a QtNativeSvg renderer is such a way that it would be easy to adapt to those who have been been using QtSvg)

Sorry, my main interest in SVG Native Viewer is the provision of the unified interface with multiple backends, so writing Qt-specific frontend is not in my high priority. If we can amuse QtSvg users, what would happen? I think SVG Native is yet another subset of SVG spec, so, having QtSvgNative is not so much exciting news of QtSvg users (e.g. SVG Native removes animation feature, although SVG Tiny supports it). How do you think?

According to the Qt roadmap for 2020 ( https://www.qt.io/blog/2019/08/07/technical-vision-qt-6 ), Qt does plan to release Qt 6 at the end of this year or early 2021. However, they don't plan major API breaks with Qt 5. They want to remove deprecated functionality, and they focus on new hardware-backed rendering via Vulkan, and much 3D stuff. It’s most likely that they'll keep the QWidgets (C++-driven GUI app development) intact, and also the current Qt 5 QML environment (though that's a bit more in flux, it's kind of like their WPF/XAML ;) ).
But either way it goes, I think that the adoption rate of Qt6 vs. Qt5 will be small.
I understand the unified API, and you're right.
I'm personally rather interested in the Adobe SVG Native viewer that has Qt 5 support. In 2013, I pitched the idea of bringing SVG to OpenType, and was part of the initial "SVG in OpenType" W3C working group, along with Sairus, Jonathan Kew and others. I’ve been also part of the standardization efforts of both SVG and variable fonts in the OpenType spec.
My day work is at FontLab, and our FontLab 7 (previously VI) is the first GUI font editor that supports the creation of all color font formats supported by OpenType. And FontLab 7 is a QWidget, Qt5 app that ships on macOS and Windows.
Currently, we use the QPainter drawing provided by QtSvg to render the SVGs that users import into the app, and, well, the results are terrible. The SVG Tiny support in QtSvg is just very limited on top of the very limited scope of SVG Tiny.
I know that Sairus has worked on SVG Native following his work on SVG in OpenType. My understanding is that the plan is to ultimately align the custom-specified subset of SVG that is currently defined in the OpenType SVG  table with SVG Native, so SVG Native actually becomes the normative profile of SVG used in OpenType+SVG fonts.
So I’d love to adopt the Adobe SVG Native viewer as the default SVG renderer for SVG glyphs inside FontLab. This would be logical, because ultimately, it would be WYSIWYG — the SVG glyphs rendered via Adobe SVG Native viewer inside FontLab would be a close approximation of what the end-users of the fonts would get in apps that use SVG Native and OpenType+SVG fonts.
So the sooner SVGNV gets a Qt5-conpatible QPainter API, the sooner people can make OpenType+SVG fonts that will actually work, and that cause more rather than less headache for all of us who have worked on bringing sensible color glyphs to typography these past six years.
Also, when you deploy your Qt5 solution, this might actually positively influence the Qt people in their decision-making regarding SVG. Perhaps they might incorporate SVGNV-based rendering of OpenType+SVG fonts, or even work on incorporating SVGNV as a replacement or alternative renderer for their QtSvg module. I think this might be a win for everyone, and I think this is more likely to happen before they actually set their Qt6 plans in stone.
(Btw, I also know that there has been some interest in using the SVG Native Viewer in FreeType for rendering SVG glyphs.)
At FontLab, we’d happily take a look at options to integrate SVGNV into our QPainter-based glyph rendering as early as possible. I can be reached at<EMAIL_ADDRESS>Many thanks for your efforts!
Best,
Adam

Ps. I don't know enough to address your specific namespace question.


642 separating built from tested backends

This solves the issue with the internal CI by by separating backends being tested (i.e., run tests) and the backends being built, which depend on the user's selction. E.g.:

the user selects the hybrid backend and excludes all other backends
both the hybrid and the reference_omp backends are built because of the dependency between backends
only the tests for the hybrid backend are built and run

This is in line with the backends "composability".
Thanks a lot to @byjtew for providing the code for this fix.

From the description I don't totally get what's currently going wrong on the internal CI -- could you (@byjtew or @alberto-scolari) please clarify? Many thanks!!

From the description I don't totally get what's currently going wrong on the internal CI -- could you (@byjtew or @alberto-scolari) please clarify? Many thanks!!

There two problems currently:

some tests fail (this you know)
currently, when we run unit tests for LPF (bsp1d and hybrid), we also run unit tests for reference and reference_omp, because of the dependencies among backends; in other words: the dependencies among backends also reflect onto tests

Point 2 makes the LPF unit tests (job tests_unit_lpf) much longer (with the risk of timeouts) and is useless, because unit tests for reference and reference_omp already run within job tests_unit (which runs for any pipeline). Therefore, this PR introduces a separation between "which backends are built" and "which backends are tested": the user chooses the latter in the CMake configuration (or the bootstrap.sh script), the internal logic configures the former. This feature is also much needed for the upcomig PR on coverage reports, where each backend is profiled in a separate job to avoid timeouts (due to profiling, compilers optimizations must be disabled, which greatly increases runtime). Hence, we decided to anticipate this separation as part of this PR, instead of hacking now a fix and then do the separation (and remove the hack).
More thoughts
As a further thought on point 2., the fundamental problem is that unit tests run for all backends and the user (or the CI) has no way to select subsets (e.g., category==unit && backend==reference) when launching. This is a limitation of our current infrastructure, which will be solved once and for all in the next months with the introduction of CTests. With this infrastructure, a user will be able to enable potentially all backends and run only a specific subset of tests (down to a single one).

Hi @alberto-scolari , sorry, I don't get how the two items you mention cause a linker error in the internal CI? I see things like
/usr/bin/ld: ../../src/graphblas/bsp1d/hybrid/libgraphblas.a(init.cpp.o): in function `grb::RC grb::finalize<(grb::Backend)7>()':
/builds/zrc-von-neumann-lab/spatial-computing/graphblas/src/graphblas/bsp1d/init.cpp:87: undefined reference to `grb::RC grb::finalize<(grb::Backend)1>()'

Is the cause to this series of issues resolved by this PR? (If so, how?)

Problem 2. causes this error (and the other ones about compilation). In the CI, we excluded the reference_omp backend for LPF-related tests (because we don't want to run those tests both in tests_unit and in tests_unit_lpf, and possibly timeout), but in doing so we excluded not only the tests for the reference_omp backend, but also the reference_omp backend itself ((grb::Backend)1), on which the hybrid backend ((grb::Backend)7) depends; that's why it doesn't link. This is why this PR separates the backends it builds from the backends it tests: it builds all backends needed for the backends under test. This PR solves problem 2., hence these issues.

Also I'm not aware of any test failing for bsp1d or hybrid -- which ones are those? (Or do you mean the ones that require some golden output?)

Indeed, those fail because the test producing the golden output (reference backend) is not built, and hence not run (you can see in the CI logs that the file with the golden output is missing). This PR:

fixes problem 2. by allowing the reference backend to be built, hence the reference test to be built
forces the build of "golden tests" via the force_grb_executable() you noticed
forces their run even if the reference backend is not tested (example at https://github.com/Algebraic-Programming/ALP/blob/5b972d4890323e1b92237b8928a8cb4c468ac038/tests/unit/unittests.sh#L667)

Got it, many thanks!!
Did some small code style fix, now waiting for CIs to OK before merge. Many thanks both


Added /NORUN parameter to ShareX manifest

Microsoft Reviewers: Open in CodeFlow

https://github.com/microsoft/winget-cli/blob/e997f3e394e66f2389f4f33f36c58b1c2768e975/src/AppInstallerRepositoryCore/Manifest/ManifestInstaller.cpp#L137-L140 @Jaex /VERYSILENT and /SILENT are both included by default, but I don't think there are any ramifications to providing the flag twice.

https://github.com/microsoft/winget-cli/blob/e997f3e394e66f2389f4f33f36c58b1c2768e975/src/AppInstallerRepositoryCore/Manifest/ManifestInstaller.cpp#L137-L140 @Jaex /VERYSILENT and /SILENT are both included by default for the inno installertype, but I don't think there are any ramifications to providing the flag twice.

Those default flags are not included when you override switch to add new flag.
But still I tested it now without add silent flag and indeed I was right, installer not run as silent.

@Jaex is the Custom switch field what you need then? seen in the spec here: https://github.com/microsoft/winget-cli/blob/master/doc/ManifestSpecv0.1.md


Remove plotting

Following on from #231

Coverage remained the same at 91.918% when pulling 159b6a9b3a595d4d35fb87ea289c285e392d9161 on gidden:remove-plotting into 205001aaaa70fe37f9d47c3a4db320ee7557c569 on IAMconsortium:master.

Coverage remained the same at 91.918% when pulling 159b6a9b3a595d4d35fb87ea289c285e392d9161 on gidden:remove-plotting into 205001aaaa70fe37f9d47c3a4db320ee7557c569 on IAMconsortium:master.

Coverage remained the same at 91.918% when pulling 159b6a9b3a595d4d35fb87ea289c285e392d9161 on gidden:remove-plotting into 205001aaaa70fe37f9d47c3a4db320ee7557c569 on IAMconsortium:master.

Thanks again @KatharinaGruber !


Flutter and Python Integration in Windows?

Is it possible to use python and flutter in Windows together?
Creating button in Flutter and executing Python function on click (in Flutter Windows App)?

Flutter is Dart-based; it doesn't support Python.


Subscription modal does not show all field types if project has more than 50 fields

When we fetch the "create issue metadata" data from Jira, it limits the number of fields expanded to a maximum of 50 fields. For a given project & issue type that has more than 50 configured fields in Jira, some fields will not be shown if it happens to be omitted from the 50 that are returned from Jira.
The task here is to make sure we include the field data for all fields on a given project & issue type. We may need to change something about the existing request that expands the fields, or perform additional requests to fill in the missing data.

Here is an example that is reproducing this issue. Notice the fields are in alphabetical order, and that fixVersions is the last listed field. Since Jira only returned 50 fields, all fields that come after fixVersions were omitted. In the case of the project & issue type in question, there should be other fields such as priority.
create-issue-metadata-response.json

Hi @wiggin77,
I spent some time investigating the issue and tried to reproduce it, but I was unable to do so. I created more than 50 fields in a Jira issue and successfully received all of them in the Jira subscription modal. Here is the response from the createIssueMetadata.json API.
Here is the list of custom field types we support in the subscription modal:

Checkbox
Labels (but gets no option in the last dropdown)
Radio
Select list (single choice)
Select list (multiple choice)

Below is the list of custom field types we don't support in the subscription modal:

Date picker
Date time picker
Number
Paragraph
Select list (cascading)
Short text (plain text only)
URL field
User picker (single user)

Possible reasons for confusion may be:

The custom fields are listed in the order of number, uppercase character, lowercase character. The fields you were expecting to see after customField might be above them because they start with an uppercase character.
The fields you were expecting in the subscription modal are not supported by us yet.

@Kshitij-Katiyar I agree we can close this.


"tab -w inexistent" should warn or error

current behaviour, when attempting to close unexistent tab: silent
desired behaviour: display a warning or an error

Yeah, this makes sense!  I'll add this to v0.4.0

Has this been fixed in commit a0d726e588122cbcb6e61be61ffa83f84ee2f31d?
It now displays "Tab not running: inex/".

Yes, in #176.  I'm preparing to release it in 0.4.0 today.

Closing, v0.4.0 is released!


Recommended way to secure Application Hosted Media bot

I have an application hosted media bot that only places calls into Teams - it doesn't listen for incoming Teams calls.
Given the application's HTTPS endpoint and media socket are open to the internet, what is the recommended way to ensure they call only be accessed by Teams? I  have tried using "AzureBotService" service tag on the inbound firewall rules, but didn't receive any Teams traffic. Is there another service tag I should be using?
Also, how does this advice change for bots that are not hosted in Azure?

Is there any update on this please?

My customer is getting close to going live with our service, and understandably is nervous about having their web service completely open to the world. Is it possible to get an answer on this please?

The Authorization Handler for Communications Client that Microsoft uses in this Samples work with a OAuth flow and only allow traffic from Teams but if you want to secure all API Endpoints you should register your API and use a proper Auth flow with ClientId and Secret for Asp.Net Core

The Authorization Handler for Communications Client that Microsoft uses in this Samples work with a OAuth flow and only allow traffic from Teams but if you want to secure all API Endpoints you should register your API and use a proper Auth flow with ClientId and Secret for Asp.Net Core

Thanks, @1fabi0 - you're right in that the auth provider will validate the auth token coming from the calls from Microsoft, which at least ensures that our traffic comes from the right place. However, this does not do anything to protect us from e.g. DDOS attacks, and adds a small amount of load to the bot. I'd prefer to make sure that all traffic that actually gets as far as the bots is already verified. I could build a service that sits closer to the edge to handle this, but given that everything is Microsoft (Teams, Azure), I'd expect there to be a service tag we could configure in an inbound rule

Thanks, @1fabi0 - you're right in that the auth provider will validate the auth token coming from the calls from Microsoft, which at least ensures that our traffic comes from the right place. However, this does not do anything to protect us from e.g. DDOS attacks, and adds a small amount of load to the bot. I'd prefer to make sure that all traffic that actually gets as far as the bots is already verified. I could build a service that sits closer to the edge to handle this, but given that everything is Microsoft (Teams, Azure), I'd expect there to be a service tag we could configure in an inbound rule

Yes but I guess this is then not a Bot related thing because then you are searching for a NAT rule that only allows Azure traffic, and I think that is not as easy as you expect because Azure is not just a private Network it's a big WAN that's everywhere interacting with the internet, so you should write your own reverse proxy that does this stuff or you just accept the risk of DDoS and scale in case of DDoS of a singel instance of your Bot

Yes but I guess this is then not a Bot related thing because then you are searching for a NAT rule that only allows Azure traffic, and I think that is not as easy as you expect because Azure is not just a private Network it's a big WAN that's everywhere interacting with the internet, so you should write your own reverse proxy that does this stuff or you just accept the risk of DDoS and scale in case of DDoS of a singel instance of your Bot

Well, Service Tags are Microsoft's answer to "a NAT rule that only allows Azure traffic", it just doesn't seem that there's one that identifies traffic from the Teams/Bot Framework infrastructure to a bot, which is surprising. Would still appreciate an official answer on this if anyone from Microsoft is listening

Well, Service Tags are Microsoft's answer to "a NAT rule that only allows Azure traffic", it just doesn't seem that there's one that identifies traffic from the Teams/Bot Framework infrastructure to a bot, which is surprising. Would still appreciate an official answer on this if anyone from Microsoft is listening


Fix for html5 notification tag problem. Fixes #3774

Description:
Fixes problem where some html5 push notification configuration options were discarded after first use.
Related issue (if applicable): fixes #3774
Checklist:
If the code does not interact with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

Thanks 🐬


Arch Linux, VS Code Latest Update Broken, Misses A Package, libre2.so.0, re2, electron7

Version Information

VSCode Version:

1.43.2
0ba0ca52957102ca3527cf479571617f0de6ed50
x64

OS Version:

Linux archlinux 5.5.3-arch1-1 #1 SMP PREEMPT Tue, 11 Feb 2020 15:35:41 +0000 x86_64 GNU/Linux

Steps to Reproduce:
Simply update the code package [ see here ] from 1.42 to 1.43 using the command,
sudo pacman -S code

It causes VSC to break and not even start using the GUI. Using the CLI to open it, I received the error,
electron7: error while loading shared libraries: libre2.so.6: cannot open shared object file: No such file or directory

The Fix
I updated electron to make sure it wasn't because of mismatching versions.
Searching a bit online, I found this forum for help, thought it is related to chromium instead.
The fix was to install a package name re2, using the command,
sudo pacman -S re2

Does this issue occur when all extensions are disabled?: Yes, since Code does not even open, it's independent of any extension related issues.
End
I'm closing this issue, just wanted to make an issue that people can look the fix up if they encounter the same problem.
Thank you!

Closing the issue.

re2 will also break chromium - please update chromium for the fix.


Tree view command to open terminal or cmd

Thinking Shift + O keyboard shortcut

[x] Figure out command line for terminal + cmd
[x] Add help to tree view h mode :rose:

Tested on mac. Will test on windows later. Nonethelss released with 2.21.0 :rose:

Tested on windows and it works just file :heart:


rawConfig and config / different properties after initialization/transformation

The API is a little bit confusing, we should consider to make it easier or how exactly we want to document it, see #233
Currently we have some kind of config transformation.
I use some placeholder like X, X' and a,b,c,d, c', d' to make it easier to understand.
You can pass an object to the deepstream constructor or load a file in the structure X with properties a,b,c,d.
After transformation some properties will be transformed and saved with a different property name, so the structure X become structure X' with the properties: a,b,c', d'.
Transformed properties can be set on the deepstream instance via deepstream.set(type, value), but you need to set the properties of X' and not X and you need to know that it's not just c and d anymore instead it's  c' and d' now. The value differs as well, because c is some kind of a string and c' is an instance which was required and initialized as a javascript class from the path c.
Some properties which are affected: connectors within the plugins property (raw) vs. connector properties in the root (transformed):

messageConnector
cache
storage

We keep it, but will add an alias: message which points to messageConnector.


Port override for srv

TBD

Codecov Report

:exclamation: No coverage uploaded for pull request base (route-on-hostport@bc2a8bd). Click here to learn what that means.
The diff coverage is 63.63%.

@@                 Coverage Diff                  @@
##             route-on-hostport      #36   +/-   ##
====================================================
  Coverage                     ?   49.25%
====================================================
  Files                        ?       24
  Lines                        ?     1080
  Branches                     ?        0
====================================================
  Hits                         ?      532
  Misses                       ?      507
  Partials                     ?       41

Impacted Files
Coverage Δ

http/backendpool/backend.go
7.14% <ø> (ø)

lib/resolvers/srv.go
60.86% <63.63%> (ø)

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 bc2a8bd...60fcd5c. Read the comment docs.

Moving to https://github.com/mwitkow/kedge/pull/37


feat(impl):[TRI-1461] lower cpu requests

lower cpu requests

Is there anything we can improve for the other components? DTR etc.


Unsuccessful TensorSliceReader constructor: Failed to find any matching files for          [[Node: save/RestoreV2 = RestoreV2[dtypes=[DT_FLOAT], _device="/job:localhost/replica:0/task:0/device:CPU:0"](_arg_save/Const_0_0, save/RestoreV2/tensor_names, save/RestoreV2/shape_and_slices)]]

I am working on a TensorFlow Speech Recognition challenge and following https://www.tensorflow.org/tutorials/audio_recognition tutorial. The model training is completed, but I'm not able to Freeze the model.
This is what I get after typing the required command:
`C:\Users\VinithaNair\AppData\Local\Programs\Python\Python36\Lib\site-packages\tensorflow\examples\speech_commands>python freeze.py \
2017-12-17 01:43:38.739183: I C:\tf_jenkins\home\workspace\rel-win\M\windows-gpu\PY\36\tensorflow\core\platform\cpu_feature_guard.cc:137] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX AVX2
2017-12-17 01:43:39.455821: I C:\tf_jenkins\home\workspace\rel-win\M\windows-gpu\PY\36\tensorflow\core\common_runtime\gpu\gpu_device.cc:1030] Found device 0 with properties:
name: GeForce 940MX major: 5 minor: 0 memoryClockRate(GHz): 1.2415
pciBusID: 0000:03:00.0
totalMemory: 4.00GiB freeMemory: 3.36GiB
2017-12-17 01:43:39.455957: I C:\tf_jenkins\home\workspace\rel-win\M\windows-gpu\PY\36\tensorflow\core\common_runtime\gpu\gpu_device.cc:1120] Creating TensorFlow device (/device:GPU:0) -> (device: 0, name: GeForce 940MX, pci bus id: 0000:03:00.0, compute capability: 5.0)
2017-12-17 01:43:39.580220: W C:\tf_jenkins\home\workspace\rel-win\M\windows-gpu\PY\36\tensorflow\core\framework\op_kernel.cc:1192] Not found: Unsuccessful TensorSliceReader constructor: Failed to find any matching files for
2017-12-17 01:43:39.588122: W C:\tf_jenkins\home\workspace\rel-win\M\windows-gpu\PY\36\tensorflow\core\framework\op_kernel.cc:1192] Not found: Unsuccessful TensorSliceReader constructor: Failed to find any matching files for
2017-12-17 01:43:39.596502: W C:\tf_jenkins\home\workspace\rel-win\M\windows-gpu\PY\36\tensorflow\core\framework\op_kernel.cc:1192] Not found: Unsuccessful TensorSliceReader constructor: Failed to find any matching files for
2017-12-17 01:43:39.601314: W C:\tf_jenkins\home\workspace\rel-win\M\windows-gpu\PY\36\tensorflow\core\framework\op_kernel.cc:1192] Not found: Unsuccessful TensorSliceReader constructor: Failed to find any matching files for
2017-12-17 01:43:39.607839: W C:\tf_jenkins\home\workspace\rel-win\M\windows-gpu\PY\36\tensorflow\core\framework\op_kernel.cc:1192] Not found: Unsuccessful TensorSliceReader constructor: Failed to find any matching files for
2017-12-17 01:43:39.613370: W C:\tf_jenkins\home\workspace\rel-win\M\windows-gpu\PY\36\tensorflow\core\framework\op_kernel.cc:1192] Not found: Unsuccessful TensorSliceReader constructor: Failed to find any matching files for
Traceback (most recent call last):
  File "C:\Users\VinithaNair\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\client\session.py", line 1323, in _do_call
    return fn(*args)
  File "C:\Users\VinithaNair\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\client\session.py", line 1302, in _run_fn
    status, run_metadata)
  File "C:\Users\VinithaNair\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\framework\errors_impl.py", line 473, in __exit__
    c_api.TF_GetCode(self.status.status))
tensorflow.python.framework.errors_impl.NotFoundError: Unsuccessful TensorSliceReader constructor: Failed to find any matching files for
         [[Node: save/RestoreV2_1 = RestoreV2[dtypes=[DT_FLOAT], _device="/job:localhost/replica:0/task:0/device:CPU:0"](_arg_save/Const_0_0, save/RestoreV2_1/tensor_names, save/RestoreV2_1/shape_and_slices)]]
         [[Node: save/RestoreV2_4/_3 = _Recv[client_terminated=false, recv_device="/job:localhost/replica:0/task:0/device:GPU:0", send_device="/job:localhost/replica:0/task:0/device:CPU:0", send_device_incarnation=1, tensor_name="edge_18_save/RestoreV2_4", tensor_type=DT_FLOAT, _device="/job:localhost/replica:0/task:0/device:GPU:0"]()]]

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "freeze.py", line 180, in <module>
    tf.app.run(main=main, argv=[sys.argv[0]] + unparsed)
  File "C:\Users\VinithaNair\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\platform\app.py", line 48, in run
    _sys.exit(main(_sys.argv[:1] + flags_passthrough))
  File "freeze.py", line 117, in main
    models.load_variables_from_checkpoint(sess, FLAGS.start_checkpoint)
  File "C:\Users\VinithaNair\AppData\Local\Programs\Python\Python36\Lib\site-packages\tensorflow\examples\speech_commands\models.py", line 123, in load_variables_from_checkpoint
    saver.restore(sess, start_checkpoint)
  File "C:\Users\VinithaNair\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\training\saver.py", line 1666, in restore
    {self.saver_def.filename_tensor_name: save_path})
  File "C:\Users\VinithaNair\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\client\session.py", line 889, in run
    run_metadata_ptr)
  File "C:\Users\VinithaNair\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\client\session.py", line 1120, in _run
    feed_dict_tensor, options, run_metadata)
  File "C:\Users\VinithaNair\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\client\session.py", line 1317, in _do_run
    options, run_metadata)
  File "C:\Users\VinithaNair\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\client\session.py", line 1336, in _do_call
    raise type(e)(node_def, op, message)
tensorflow.python.framework.errors_impl.NotFoundError: Unsuccessful TensorSliceReader constructor: Failed to find any matching files for
         [[Node: save/RestoreV2_1 = RestoreV2[dtypes=[DT_FLOAT], _device="/job:localhost/replica:0/task:0/device:CPU:0"](_arg_save/Const_0_0, save/RestoreV2_1/tensor_names, save/RestoreV2_1/shape_and_slices)]]
         [[Node: save/RestoreV2_4/_3 = _Recv[client_terminated=false, recv_device="/job:localhost/replica:0/task:0/device:GPU:0", send_device="/job:localhost/replica:0/task:0/device:CPU:0", send_device_incarnation=1, tensor_name="edge_18_save/RestoreV2_4", tensor_type=DT_FLOAT, _device="/job:localhost/replica:0/task:0/device:GPU:0"]()]]

Caused by op 'save/RestoreV2_1', defined at:
  File "freeze.py", line 180, in <module>
    tf.app.run(main=main, argv=[sys.argv[0]] + unparsed)
  File "C:\Users\VinithaNair\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\platform\app.py", line 48, in run
    _sys.exit(main(_sys.argv[:1] + flags_passthrough))
  File "freeze.py", line 117, in main
    models.load_variables_from_checkpoint(sess, FLAGS.start_checkpoint)
  File "C:\Users\VinithaNair\AppData\Local\Programs\Python\Python36\Lib\site-packages\tensorflow\examples\speech_commands\models.py", line 122, in load_variables_from_checkpoint
    saver = tf.train.Saver(tf.global_variables())
  File "C:\Users\VinithaNair\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\training\saver.py", line 1218, in __init__
    self.build()
  File "C:\Users\VinithaNair\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\training\saver.py", line 1227, in build
    self._build(self._filename, build_save=True, build_restore=True)
  File "C:\Users\VinithaNair\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\training\saver.py", line 1263, in _build
    build_save=build_save, build_restore=build_restore)
  File "C:\Users\VinithaNair\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\training\saver.py", line 751, in _build_internal
    restore_sequentially, reshape)
  File "C:\Users\VinithaNair\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\training\saver.py", line 427, in _AddRestoreOps
    tensors = self.restore_op(filename_tensor, saveable, preferred_shard)
  File "C:\Users\VinithaNair\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\training\saver.py", line 267, in restore_op
    [spec.tensor.dtype])[0])
  File "C:\Users\VinithaNair\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\ops\gen_io_ops.py", line 1020, in restore_v2
    shape_and_slices=shape_and_slices, dtypes=dtypes, name=name)
  File "C:\Users\VinithaNair\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\framework\op_def_library.py", line 787, in _apply_op_helper
    op_def=op_def)
  File "C:\Users\VinithaNair\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\framework\ops.py", line 2956, in create_op
    op_def=op_def)
  File "C:\Users\VinithaNair\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\framework\ops.py", line 1470, in __init__
    self._traceback = self._graph._extract_stack()  # pylint: disable=protected-access

NotFoundError (see above for traceback): Unsuccessful TensorSliceReader constructor: Failed to find any matching files for
         [[Node: save/RestoreV2_1 = RestoreV2[dtypes=[DT_FLOAT], _device="/job:localhost/replica:0/task:0/device:CPU:0"](_arg_save/Const_0_0, save/RestoreV2_1/tensor_names, save/RestoreV2_1/shape_and_slices)]]
         [[Node: save/RestoreV2_4/_3 = _Recv[client_terminated=false, recv_device="/job:localhost/replica:0/task:0/device:GPU:0", send_device="/job:localhost/replica:0/task:0/device:CPU:0", send_device_incarnation=1, tensor_name="edge_18_save/RestoreV2_4", tensor_type=DT_FLOAT, _device="/job:localhost/replica:0/task:0/device:GPU:0"]()]]`

OS - Windows 10
TensorFlow version - 1.4
Python version - 3.6.3
GPU - CUDA V 8 and cuDNN V 6.0
I came across [https://github.com/tensorflow/tensorflow/issues/6082] and https://github.com/tensorflow/tensorflow/issues/7547 where the suggested fix was to add "./" to the model name. But, in this case I'm not able find the list of codes where I'm supposed to make the change. How do I find the code that needs the fix? Or is there another issue that I'm unaware of?
Please help.

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow installed from
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce

Nagging Awaiting Response: It has been 14 days with no activityand the awaiting response label was assigned. Is this still an issue?

Hey, the problem has been sorted. Thank you!

Hey there,
I seem to the exact same problem. From what I have read, I can see that I need to add a ./ somehwhere in the code. Not sure where though.
I am using a Windows 10 x64
Tensorflow was installed using the command pip3 install --upgrade tensorflow after installing python.
The training of the model has been done. And I have the following issue while trying to freeze the model:
The command I typed was:
_python  "C:\Program Files\python35\Lib\site-packages\tensorflow\examples\speech_commands/freeze.py"_
The result I got was:
2018-06-22 16:45:52.269435: I T:\src\github\tensorflow\tensorflow\core\platform\cpu_feature_guard.cc:140] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2
2018-06-22 16:45:54.710014: W T:\src\github\tensorflow\tensorflow\core\framework\op_kernel.cc:1318] OP_REQUIRES failed at save_restore_tensor.cc:170 : Not found: Unsuccessful TensorSliceReader constructor: Failed to find any matching files for
Traceback (most recent call last):
File "C:\Program Files\Python35\lib\site-packages\tensorflow\python\client\session.py", line 1322, in _do_call
return fn(*args)
File "C:\Program Files\Python35\lib\site-packages\tensorflow\python\client\session.py", line 1307, in _run_fn
options, feed_dict, fetch_list, target_list, run_metadata)
File "C:\Program Files\Python35\lib\site-packages\tensorflow\python\client\session.py", line 1409, in _call_tf_sessionrun
run_metadata)
tensorflow.python.framework.errors_impl.NotFoundError: Unsuccessful TensorSliceReader constructor: Failed to find any matching files for
[[Node: save/RestoreV2 = RestoreV2[dtypes=[DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT], _device="/job:localhost/replica:0/task:0/device:CPU:0"](_arg_save/Const_0_0, save/RestoreV2/tensor_names, save/RestoreV2/shape_and_slices)]]
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Program Files\python35\Lib\site-packages\tensorflow\examples\speech_commands/freeze.py", line 180, in
tf.app.run(main=main, argv=[sys.argv[0]] + unparsed)
File "C:\Program Files\Python35\lib\site-packages\tensorflow\python\platform\app.py", line 126, in run
_sys.exit(main(argv))
File "C:\Program Files\python35\Lib\site-packages\tensorflow\examples\speech_commands/freeze.py", line 117, in main
models.load_variables_from_checkpoint(sess, FLAGS.start_checkpoint)
File "C:\Program Files\python35\Lib\site-packages\tensorflow\examples\speech_commands\models.py", line 123, in load_variables_from_checkpoint
saver.restore(sess, start_checkpoint)
File "C:\Program Files\Python35\lib\site-packages\tensorflow\python\training\saver.py", line 1802, in restore
{self.saver_def.filename_tensor_name: save_path})
File "C:\Program Files\Python35\lib\site-packages\tensorflow\python\client\session.py", line 900, in run
run_metadata_ptr)
File "C:\Program Files\Python35\lib\site-packages\tensorflow\python\client\session.py", line 1135, in _run
feed_dict_tensor, options, run_metadata)
File "C:\Program Files\Python35\lib\site-packages\tensorflow\python\client\session.py", line 1316, in _do_run
run_metadata)
File "C:\Program Files\Python35\lib\site-packages\tensorflow\python\client\session.py", line 1335, in _do_call
raise type(e)(node_def, op, message)
tensorflow.python.framework.errors_impl.NotFoundError: Unsuccessful TensorSliceReader constructor: Failed to find any matching files for
[[Node: save/RestoreV2 = RestoreV2[dtypes=[DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT], _device="/job:localhost/replica:0/task:0/device:CPU:0"](_arg_save/Const_0_0, save/RestoreV2/tensor_names, save/RestoreV2/shape_and_slices)]]
Caused by op 'save/RestoreV2', defined at:
File "C:\Program Files\python35\Lib\site-packages\tensorflow\examples\speech_commands/freeze.py", line 180, in
tf.app.run(main=main, argv=[sys.argv[0]] + unparsed)
File "C:\Program Files\Python35\lib\site-packages\tensorflow\python\platform\app.py", line 126, in run
_sys.exit(main(argv))
File "C:\Program Files\python35\Lib\site-packages\tensorflow\examples\speech_commands/freeze.py", line 117, in main
models.load_variables_from_checkpoint(sess, FLAGS.start_checkpoint)
File "C:\Program Files\python35\Lib\site-packages\tensorflow\examples\speech_commands\models.py", line 122, in load_variables_from_checkpoint
saver = tf.train.Saver(tf.global_variables())
File "C:\Program Files\Python35\lib\site-packages\tensorflow\python\training\saver.py", line 1338, in init
self.build()
File "C:\Program Files\Python35\lib\site-packages\tensorflow\python\training\saver.py", line 1347, in build
self._build(self._filename, build_save=True, build_restore=True)
File "C:\Program Files\Python35\lib\site-packages\tensorflow\python\training\saver.py", line 1384, in _build
build_save=build_save, build_restore=build_restore)
File "C:\Program Files\Python35\lib\site-packages\tensorflow\python\training\saver.py", line 835, in _build_internal
restore_sequentially, reshape)
File "C:\Program Files\Python35\lib\site-packages\tensorflow\python\training\saver.py", line 472, in _AddRestoreOps
restore_sequentially)
File "C:\Program Files\Python35\lib\site-packages\tensorflow\python\training\saver.py", line 886, in bulk_restore
return io_ops.restore_v2(filename_tensor, names, slices, dtypes)
File "C:\Program Files\Python35\lib\site-packages\tensorflow\python\ops\gen_io_ops.py", line 1546, in restore_v2
shape_and_slices=shape_and_slices, dtypes=dtypes, name=name)
File "C:\Program Files\Python35\lib\site-packages\tensorflow\python\framework\op_def_library.py", line 787, in _apply_op_helper
op_def=op_def)
File "C:\Program Files\Python35\lib\site-packages\tensorflow\python\framework\ops.py", line 3392, in create_op
op_def=op_def)
File "C:\Program Files\Python35\lib\site-packages\tensorflow\python\framework\ops.py", line 1718, in init
self._traceback = self._graph._extract_stack()  # pylint: disable=protected-access
NotFoundError (see above for traceback): Unsuccessful TensorSliceReader constructor: Failed to find any matching files for
[[Node: save/RestoreV2 = RestoreV2[dtypes=[DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT], _device="/job:localhost/replica:0/task:0/device:CPU:0"](_arg_save/Const_0_0, save/RestoreV2/tensor_names, save/RestoreV2/shape_and_slices)]]
C:\Program Files\Python35\Lib\site-packages\tensorflow>python "C:\Program Files\python35\Lib\site-packages\tensorflow\examples\speech_commands/freeze.py"
Traceback (most recent call last):
File "C:\Program Files\python35\Lib\site-packages\tensorflow\examples\speech_commands/freeze.py", line 42, in
import os.path.abspath
ImportError: No module named 'os.path.abspath'; 'os.path' is not a package
C:\Program Files\Python35\Lib\site-packages\tensorflow>python "C:\Program Files\python35\Lib\site-packages\tensorflow\examples\speech_commands/freeze.py"
Traceback (most recent call last):
File "C:\Program Files\python35\Lib\site-packages\tensorflow\examples\speech_commands/freeze.py", line 43, in
import os.path.abspath
ImportError: No module named 'os.path.abspath'; 'os.path' is not a package
Would be great if you could let us know how it worked out for you.
Thanks a lot,


Update sentry sdk for python

Using the productino Sentry SDK for Python, because it now supports OTel.

wrong.


feat: add r2 oc region to accepted location hints

This adds the new R2 Oceania region to the list of supported location hints in the provider

i'm unable to push changes for you here so you'll need to run make docs and push up the result.

i'm unable to push changes for you here so you'll need to run make docs and push up the result.

Have run that and pushed those changes, apologies for missing that.
As a side note there was an unrelated docs change in the notification_policy file, I didn't push that here but presumably it was missed in another PR so it's out of sync.


Ai routinely loads editor tagging into code

Describe the bug
While having bolt edit the web page it'll routinely paste code in that still has editor tagging on it causing the code to fail.  It will have a tag describing the code as typescript with three tildes as is common practice for copy pasting into editors for formatting.  In order to get it to run properly again after this I have to go in and manually remove the tagging because the AI isn't able to detect the issue.  In chrome developer tools I find these files because the exports from them show as syntax errors:
Uncaught SyntaxError: The requested module '/src/components/admin/taskComponents/TagSelector.tsx?t=1734472963065' does not provide an export named 'TagSelector' (at ComponentBasicInfo.tsx:4:10)
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-xxcvzj
Validations

[X] Please make your project public or accessible by URL. This will allow anyone trying to help you to easily reproduce the issue and provide assistance.

Steps to reproduce
It doesn't happen on every request so I do not know how to reproduce it, but it happens most frequently when I ask it to fix an issue that it is having.
Expected behavior
I expect the code to be saved to the file without the code block tagging.
I expect:
console.log('hi')
instead of:
```Typescript
console.log('hi')
```
Screen Recording / Screenshot

Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 131
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 134698613,
  "usedJSHeapSize": 123190277,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = NoHitCrit
Chat ID = b3c988bc86b0

Additional context
No response

I was hoping I could just tell the ai about the issue, but it doesn't fix it:

Seems like this happens more often as I continue editing in the same session.  It's now to the point that It happens on every code change request.

Current workaround is to refresh the browser each time it begins to happen.

i have an issue with it coding in the chat and not updating the actual file. more and more and yes refreshing helps. what a waste of money

I have the same problem, except refreshing won't help.


refactor: files{} -> challengeFiles[], and key -> fileKey

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Related to #39719
As usual, my PR went a bit further than hoped 😅

[x] Renames cases of files -> challengeFiles
[x] Renames cases of key (challengeFile.key) -> fileKey
[x] Miscellaneous fixes thrown in, as needed and pointed out by TS and in my escapade
[x] Converts challengeFiles to an array, to match the current database schema
[x] Cleans up (kind-of) prop-types - eventually, this file needs to be renamed, and probably moved (once client no longer uses PropTypes package)
[x] Removes a lot of unnecessary TS-ignoring comments (my OCD kicked in)

the branch name has nothing to do with the changes, because my brain is acting up

Closing, because I still do not know how to use Git :/


@ViewedStore missing from UIKIt?

I've been using a version with @ViewedStore for UIKit compatibility but this is not part of the release. Did I miss something where it was replaced with another mechanism?

That was one of many renditions riffs on the view store. After landing on our current SwiftUI solution we decided to consolidate concepts and encourage folks to instantiate view stores directly. A few examples can be seen in the UIKit Case Studies and Tic Tac Toe apps. For example:
https://github.com/pointfreeco/swift-composable-architecture/blob/d2240d0e76c1a758dbadbf737ceefc888b2e807c/Examples/CaseStudies/UIKitCaseStudies/CounterViewController.swift#L33

Cool to close this one out @matadan? Or do you have any other questions?

Got it.


There was no sound in insomniac

for some reason i couldnt hear the music for insomniac once i played it.

for some reason i couldnt hear the music for insomniac once i played it.

i also couldn't hear the songs for the other songs


fix(subscribe): Deprecate null starting parameter signatures for subscribe

Description:
Per discussion on #4159, this deprecates the signatures for .subscribe and .tap that are (null, null, complete), (null, error, complete), and (next, null, complete).
Related issue (if exists):
#4159

Coverage decreased (-0.2%) to 96.806% when pulling 4d20b0825c3468e3a24b720ee8583d24e638760e on ajcrites:deprecate-subscribe-fn into 6a2b524bc62cb1039123ecc45c7ccc65358d9eb1 on ReactiveX:master.

I'm actually thinking about that right now, as I'm writing a TSLint rule to do the same. I'll let you know what I come up with.

Regarding the coveralls. It seems to +/-0.2% changes in a seemingly non-deterministic way. I was speaking to someone on Monday, at a meetup, and he was of the opinion that it was due to the PR's branch not being completely up-to-date with master.
As an experiment, could you ensure your master is up-to-date and then rebase your PR and push it again? Like this:
git checkout master
git fetch upstream
git rebase upstream/master
git checkout deprecate-subscribe-fn
git rebase master
git push -f

I'd be really interested in whether or not coveralls passes after that. This +/-0.2% variation has been an annoyance for a long time.

regarding above, I'm curious if generated coverage have problem or the way coveralls translate it have some problems. ..brutal way, maybe we can setup codecov also and 1:1 compare for a while per each PRs?

@kwonoj I don't really know enough about Coveralls or what it's doing. If we can establish what the problem is, we might be able to get a decent solution.
ATM, if you cannot reliably see whether the coverage goes up or down, it's not especially useful.
TBH, though, I'm not super keen on getting into the coveralls nitty gritty. :grimacing:

I am usually just ignoring those minor deets, so on same boat. Only reason I care is most new contributor got highly confused like 'I only changed doc and coverage dropped, maybe I did something wrong'.

@ajcrites I'd suggest something like:
/** @deprecated Use an observer instead of an error callback */

We're in the midst of updating the terminology "observer" to "subscriber," at least in some documentation. Should I stick with "observer" for this?

Sorry about the closure.
My understanding is that the documentation is changing re: new Observable(subscriber => {}) - where the parameter really is a Subscriber - and that's not the case here. The argument is not a subscriber. I could be wrong, but that's my take. In fact, the type that corresponds to the argument, here, is actually PartialObserver, as not all methods are required.

@ajcrites ... subscribe accepts a PartialObserver, which is any subset of Observer... Subscriber is a superset of Observer in that it has a closed property on it (as well as other things in RxJS 5 and 6).


Not able to export "Geo Maps Leaflet - SAP Lumira visualization extension" in pdf format

Hi,
I tried to use leaflet extension in Lumira and I notices its not visible in exported pdf.
Below are the screensots for more details-
Able to see Leaflet map in Lumira

Not Visible in exported pdf.

Final output:
Leaflet sample .pdf
Any Hint or input why its happening?
Please let me know if any further details required.

@Vikash2402 PDF export of story pages with chart extensions is not working right now as of Lumira 1.31 https://ideas.sap.com/D38022

I'll close this issue here as this is a product bug. Please feel free to add your comments here, https://ideas.sap.com/D38022


Warning from react-responsive

I just got this warning  after installing livekit-react.
WARNING in ./node_modules/react-responsive/dist/react-responsive.js
Module Warning (from ./node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map: 'webpack:///dist/react-responsive.js' URL is not supported
<EMAIL_ADDRESS>9:0-49 1131:17-30

Hi @suryakumara
this is an issue with react-responsive https://github.com/yocontra/react-responsive/issues/286
Once the 9.0 version has been released we can upgrade to that version


⚠ Bofrost.nl has degraded performance

In eab9600, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 1843 ms

Resolved: Bofrost.nl performance has improved in 7153232.


FLORIS warning message

I'm getting an error like:
floris.floris_model.FlorisModel WARNING turbine_type has been changed without specifying a new reference_wind_height. reference_wind_height remains 110.00 m. Consider calling `FlorisModel.assign_hub_height_to_ref_height` to update the reference wind height to the turbine hub height.

after an update to FLORIS 4.2. Not sure exactly what's up with that.

@misi9170 has identified the issue and will fix it soon


Update Toscana attribution name

Change attribution text to attribution name to match the typical standard

@iandees I think the only issue here is the mapbox tiles are failing again while running the check.

We shouldn't be using any Mapbox tiles anymore.


[SMALLFIX] Add utility for fatal errors

I don't think so, using String.format() with %s already calls toString() if the object isn't a string

Merged build finished. Test PASSed.

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


xcode 6.3 uuid

Me again! Love this tool, happy to help keep it up to date.

Thanks :) The plugin doesn't actually work with 6.3 yet but I'll pull this once I get it fixed.

All sleepygarden's doing (Michael Cornell), but updates in place to get OFPlugin working again in 6.3. Haven't tried this in previous versions of XCode, so this might be better folded into a branch other than master.

Looks good, thanks @sleepygarden !
There's a bit of junk that got pulled in on the commits though (.DS_Store, .xcuserstate etc). I could rebase them out when I have a moment but can you submit a PR that just includes the necessary modifications?

Removed what I could, added a .gitignore but i'm sure it could use some additional items in it down the line.

woo thanks!


Changing the height and the width of the circle

Signed-off-by: Krishna K -<EMAIL_ADDRESS>

Thanks!


Generate JWT token in .env file instead of jwt-auth's default in config/jwt.php

Check if it was fixed in tymondesigns/jwt-auth, if not implement the fix here

jwt auth 1.0 is still in alpha
Will upgrade and fix this as soon as it hits a stable release


actionLabel does not seem to work

Hello,
Just realize I can't force actionLabel text with react-native 0.21.
It stays at "DISMISS" whatever I do.

It does work, you need an actionLabel AND a actionCallback.

@Ehesp Note the issue is from march 😁

Oh yeah haha... Oh well :D


chore(attributes): added owner path key

Part of #201
Changelog
New

added analyzed.owner.path key to the customResourcesAttributes object

Testing / reviewing
N/A

I have read the DCO document and I hereby sign the DCO.


feat(Storybook): Initial setup of how storybook can be used

Basic concept of what storybooks are. I've not added a makefile yet so to run go to frontend/ and run yarn storybook
If you are in the root repository, you can do make storybook. I might change this to dev-storybook or something.
TODO:

[ ] Flesh out the stories a bit more
[ ] Put test data outside the stories components

Fixes #228
Related to
Merge after
#246

RELEASE NOTES BEGIN
￼
RELEASE NOTES END

@lachmanfrantisek I updated the CONTRIBUTING.md file. WDYT?

Current issue, preview.ts seems to be parsed as a .js file as it complains about const preview: Preview =, which is fixed if Preview is removed

@jpopelka should be working now. Want to give it a go? Need to redo the installation of node modules

I'd say we're ready to merge this, can improve later
There is one thing I want answered so asked over in a plugin repo, but good for now
https://github.com/JesusTheHun/storybook-addon-react-router-v6/discussions/37


remove static path to yarn.cjs

Hello Manu,
Maybe there was a reason for this, but seems to be working fine without this, so it's best to remove this pinned-committed dependency.
Also, add .gitignore to .dockerignore.

I will fix this

Thanks @NicolasCARPi for having a look and help improving the repository, very appreciated!


Adds playback controls to student group one up view

Adds playback to the student one up view in Student Group teacher nav tab


Unclear Instructions for Onboarding Python Script

Step 3 of the section titled "Install a Linux Hybrid Runbook Worker" describes running an onboarding python script with some parameters specified. One of the parameters (-e) has a description of "automationEndpoint." It is unclear what value is supposed to be passed to this parameter as no other information is provided. It would be helpful to have some additional information explaining what this means and where to obtain the value.

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

ID: e38be5b8-d76d-a4f1-c014-7bf9248be2de
Version Independent ID: 976e5e90-b28c-d7ba-0495-69d92e62ea46
Content: Deploy a Linux Hybrid Runbook Worker in Azure Automation
Content Source: articles/automation/automation-linux-hrw-install.md
Service: automation
Sub-service: process-automation
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

@fib0nacci - To identify the value for that argument, you go to the Manage Keys page and copy the value for URL on that page.  For more information, you can see similar information under this section of How to install Windows HRW.
I had a bug already filed for this and am planning to address it shortly.
Hope that helps.

Thank you, this got me what I needed. Copying the value from the URL field of the Account Settings > Keys page of our Automation Account produced the results we were expecting. Adding similar information to the Linux page will be very helpful.