Fix Haystack 2 tutorials test failing when using main branch Docker image

We recently updated the base Docker image used to create our Haystack Docker images from Python 3.10 to 3.12, see deepset-ai/haystack#7540.
This is now causing most nightly tests to fail when installing dependencies that are common to all tutorials, example failure.
   Installing build dependencies: started
  Installing build dependencies: finished with status 'done'
  Getting requirements to build wheel: started
  Getting requirements to build wheel: finished with status 'error'
  error: subprocess-exited-with-error

  × Getting requirements to build wheel did not run successfully.
  │ exit code: 1
  ╰─> [33 lines of output]
      Traceback (most recent call last):
        File "/opt/venv/lib/python3.12/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 353, in <module>
          main()
        File "/opt/venv/lib/python3.12/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 335, in main
          json_out['return_val'] = hook(**hook_input['kwargs'])
                                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "/opt/venv/lib/python3.12/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 112, in get_requires_for_build_wheel
          backend = _build_backend()
                    ^^^^^^^^^^^^^^^^
        File "/opt/venv/lib/python3.12/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 77, in _build_backend
          obj = import_module(mod_path)
                ^^^^^^^^^^^^^^^^^^^^^^^
        File "/usr/local/lib/python3.12/importlib/__init__.py", line 90, in import_module
          return _bootstrap._gcd_import(name[level:], package, level)
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "<frozen importlib._bootstrap>", line 1387, in _gcd_import
        File "<frozen importlib._bootstrap>", line 1360, in _find_and_load
        File "<frozen importlib._bootstrap>", line 1310, in _find_and_load_unlocked
        File "<frozen importlib._bootstrap>", line 488, in _call_with_frames_removed
        File "<frozen importlib._bootstrap>", line 1387, in _gcd_import
        File "<frozen importlib._bootstrap>", line 1360, in _find_and_load
        File "<frozen importlib._bootstrap>", line 1331, in _find_and_load_unlocked
        File "<frozen importlib._bootstrap>", line 935, in _load_unlocked
        File "<frozen importlib._bootstrap_external>", line 995, in exec_module
        File "<frozen importlib._bootstrap>", line 488, in _call_with_frames_removed
        File "/tmp/pip-build-env-duezb8qt/overlay/lib/python3.12/site-packages/setuptools/__init__.py", line 16, in <module>
          import setuptools.version
        File "/tmp/pip-build-env-duezb8qt/overlay/lib/python3.12/site-packages/setuptools/version.py", line 1, in <module>
          import pkg_resources
        File "/tmp/pip-build-env-duezb8qt/overlay/lib/python3.12/site-packages/pkg_resources/__init__.py", line 2172, in <module>
          register_finder(pkgutil.ImpImporter, find_on_path)
                          ^^^^^^^^^^^^^^^^^^^
      AttributeError: module 'pkgutil' has no attribute 'ImpImporter'. Did you mean: 'zipimporter'?
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error

× Getting requirements to build wheel did not run successfully.
│ exit code: 1
╰─> See above for output.

note: This error originates from a subprocess, and is likely not a problem with pip.

We need to update the installation of those dependencies so it doesn't break.

I believe this is fixed. Can you confirm @silvanocerza ?

Yup, closing.


Add to Prop to UTabs Component for Enhanced Navigation

Description
I'd like to propose an enhancement for the UTabs component. Currently, the UTabs component lacks a to prop, which I believe could significantly improve the navigation experience within our application. This addition would align the UTabs component with the functionality already present in the UAccordion component, which benefits from the to prop.
Proposed Change:
Feature: Implement a to prop in the UTabs component.
Benefit: This will allow users to navigate seamlessly through different parts of the application, similar to the navigation experience provided by the UAccordion component.
Rationale:
Consistency: Having similar navigation capabilities in both UTabs and UAccordion components would maintain a consistent user experience across the application.
User Experience: This enhancement is expected to make the navigation more intuitive and user-friendly, as it allows for direct navigation from the tabs.
I believe this feature would be a valuable addition to our project, enhancing both the developer experience and the end-user interface. I look forward to feedback and any further suggestions on this proposal.
Additional context
No response

I adopted the officially recommended method, which has a higher degree of freedom

Of course, I also recommend adding a to. In the simple mode where the router exists, the code will be simpler.

I think using the HorizontalNavigation component would be more appropriate.


make the apk building part of flutter run optional

add a --build (--no-build) option to flutter run, defaulting to true. This makes the building part of flutter run optional and will be used in cases where a separate build system is creating the built APK.

@johnmccutchan @cbracken


Validatableprotocol

Checking to see where conflicts with #111 are

Hi, I want to know if I can use protocol version of swiftvalidator? Are you going to merge on master?

@mapo80 I'm going to merge it to the master branch.  I'm just waiting for @dadalar to resolve the conflicts on its PR first.

Ok, thanks for tour answer. For now I'll use branch, later I'll switch to master.
Thanks.


How to use ShortestPathFollower ?

Hello there.
Long time no see.
While going through the sound-spaces files, I could not help but notice that there is a ShortestPathFollower class here.
Would it be possible to use this class to instantiate as a form of oracle to generate optimal trajectories across the scenes ?
Also, is it perhaps compatible with SAVi environments ?
Thank you for your time.

Hi @dosssman, yes, you can use this to generate the oracle trajectory for either AudioGoal or Semantic AudioGoal, although it uses information that is not accessible by a real navigation agent.

Hello again.
Thank you very much for taking the time to check on this issues.
Just leaving this here for other people that might stumble upon this issue:
When I was trying to use that class, it would fail instantiating as done in the ppo_trainer.py, it would throw and error along the lines of DummySimulator does not have '_path_finder' attribute.
Workaround to get it working somehow, was to not use pre-rendered observations, by having config.TASK_CONFIG.SIMULATOR.USE_RENDERED_OBSERVATIONS = False.
Another alternative I found to get optimal actions for an episode was to access envs.workers[0]._env._env._sim._oracle_actions, which returns a list of actions that lead the agent to the goal location upon execution.
This method could even be used with USE_RENDERED_OBSERVATIONS set to True.
This was tested with the SAVI environment, whcih uses MP3D dataset. Not sure about Replica.
Thanks again for your time.
I would consider this issue closed.


Mupdf inverse search only works when page number is in title string

See #89.

Yep, I meant those latexmkrc lines that change the Window title of mupdf according tobthe compile state. Didn't know they come with vim-latex because the quickfix window never opens over here when compilation fails.

See latexmk.vim:341,  it shows how the callbacks for opening the quickfix window works. Also note the quickfix options
  g:latex_quickfix_autojump
  g:latex_quickfix_ignored_warnings
  g:latex_quickfix_ignore_all_warnings
  g:latex_quickfix_mode

In particular, the last option allows to control when the error window should open.
If youo find that this does not work as expected, please open a new issue with a minimal working example.

Ok. The problem was that by default Terminal vim under Linux is started without a servername (see https://groups.google.com/forum/m/#!topic/vim_use/Jsf4vsmoeXM) . That's why it didn't work out of the box. Now that I supply a servername it works fine.
However, the commands at latexmk.vim:341 do not change the Window title. The proposed settings
$compiling_cmd = "xdotool search --name %D set_window --name \"%D compiling...\"";
$success_cmd   = "xdotool search --name %D set_window --name \"%D OK\";";
$failure_cmd   = "xdotool search --name %D set_window --name \"%D FAILURE\";";

also rely on different main files for different projects.

I'll look into this later. I think we should be able to use the window ID to set the window title, which means it should no longer depend on the main file names.

I just pushed some commits; I've begun to prepare for trying to fix some of these issues. As far as I can see, there are actually several issues:

Imlement the window title hacks as part of vim-latex and use mupdf_id for robustness
Improve robustness of getting and setting mupdf_ids
Get page number from mupdf window without reading from title string

Alright. Regarding 2., over here it works fine for the moment. Cannot explain why. Do you see issues?

Not really, no. It seems to work fine. However, I'd like to try to identify the mupdf window that is opened by latexmk, so that we don't need to open a new window in addition to the first one.
Regarding the main topic of this issue: It seems I were wrong; the original hack that inspired the functionality here also relied on the window title to parse the page number. Thus I don't really know how to reliably combine the proposed settings to set window titles with the reverse search. That is, I am not able to solve my above number 3.

However, I've tried to improve the callback functionality. I'm pushing some changes soon that makes the callback function echo a colorized status to indicate success or fail. This should remove the necessity of the commands proposed in the docs.

Ah, that's a fine idea to use colors. More expressive than text.

Thanks! I'm pushing some more changes soon, in particular one that I am not entirely sure is good: After starting latexmk you added a sleep and then some code to get the window ID. I've refactored this so that the window ID getter is only implemented once, and I've moved the window ID getter to the first callback. The only drawback that I can see is that this only works when callbacks are used.

I've just pushed some relatively large changes:

After latexmk has been started, the first callback will call latex#view#poststart(). This function saves the mupdf window ID and returns the focus to the vim window. It also sends keys to MuPDF g:latex_view_mupdf_send_keys (default empty string).
The same function is also called when MuPDF was opened with <localleader>lv.

I consider this issue solved. That is, the documentation might require an update, since the trick mentioned is not really that interesting any longer. However, it seems to me that this is not an important update, and I'll leave it as it is for now.

Regarding callbacks, perhaps a note in the documentation that terminal vim does not use them by default is in place?
The send keys could be used to open the pdf document where the tex document is at?
I think the issue about the Window id occurs when you open two vim instances and in each a pdf and close the fistly opened pdf.
The id of the still open mupdf window is reassigned to that of the first now closed one. May that be?

Yes, I think the documentation needs a thorough update. I've opened a new issue for this.
I'm using the send keys to set the zoom level in MuPDF to show the entire page. That is, I send the H key. I don't see how the send keys option could be used to open the pdf where the tex document is at, but the callback could definately do that in a similar fashion as the forward search. I'm pushing an update for this soon.
For the last part: I've tested and it seems you are right. It seems that for some reason the order is not always as assumed, that the last reported window ID is the last window opened MuPDF window. However, this seems like a very rare issue, and a simple workaround is to close all pdf windows and open them again.

One thing: It seems the window IDs are the same for the same pdf file every time it is opened again. Do you agree? If that is the case, then it seems we shouldn't need to set the window ID if it is already set.

At least my last comment seems correct on my end. I pushed a change that utilizes the assumption that a pdf file window ID does not change, so that one only needs to set the mupdf_id once for each session.

Regarding

It seems that for some reason the order is not always as assumed, that the last reported window ID is the last window opened MuPDF window.

I am not sure what the last window opened MuPDF window means.
First off, the window id in the last line of the output of xdotool search --class MuPDF corresponds to the window last opened. However, if a window that was not opened last is closed, say in the n-th place, then every windows opened afterwards, say at the m-th place, take over the window id of its predecessor, the m-1-st window.
That was my idea. But just checked again and it seems false.
Now you say that the window ID is assigned to a PDF file instead? Well, this is false as well.
Open two Vim instances and two different PDFs. Run xdotool search --class MuPDF. Then close both and open the second PDF. Then xdotool search --class MuPDF show that the ID is that of the first PDF.

Interesting. It seems slightly difficult to understand this rule. In any case, the latest version seems to work flawlessly here. Do you manage to produce an ID conflict?

Well, happened that :VimLatexView does not open a window when it was closed, but it's hard to reproduce so far. Before, sometimes two vim instances would forward search to the same mupdf window.
However, sinc then threre have been some changes to vim-latex. So perhaps best to forget about it and come back to it if one can distill a reproducable ID conflict.

Yes, I've pushed some updates and changes that should fix the issues you mention. I think we should leave this discussion and issue for now as solved, and reopen (or open a new issue) if we find a problem with the current version.


ungoogled-chromium left a "burning" mark in the screen - ArchLinux

Describe the bug
I install the ungoogled-chromium  version: 75.0.3770.100-1 on Archlinux, all works perfect except for a burning-mark the browser left on my screen, the mark have the form of the address box, a white line rounded the address -box burn my screen and its more visible if i use the browser for a long time.
After go here i test anything, i left my test list:

update and downgrade xorg and video drivers (reboot after any try)
mod my screen options (change render, swappiness, buffer and other options)
change to windows 10 after the burn (i confirm the burn appears here too, after change of OS)
disable any window animation or option that force screen to render
if i shutdown the laptop and wait more than 10min and power up again, the burning disapears but if i use the laptop (using the browser) so much time (more than 4 hours), the mark appears and stay after the shutdown or change of OS.
if the mark  disapears and i dont use chromium the mark dont apears again, for that reason i know its problem of the browser and not my screen or drivers.

After that i try one more thing, put the browser in "normal" window mode (not maximize the window), and Voila! the browser dont left burning-mark and i can work for 12 hours without any sign of the issue... if i use the full screen works perfect too, but if i use the maximize window option again the error apears.
To Reproduce
Steps to reproduce the behavior:

Open chromium
Put the window on maximize
Browse for 10min o more
Close or minimize the browser and see a mark on the screen with the form of the address box

Expected behavior
A mark (silouette) of the address box its in the screen (like a burning) and its persistence after reboot (and its visible if you change your OS)
Screenshots

i solve the problem not maximize the window, i dont like put to run the error again because  dont like f#$%.. my screen,  in this moment its working perfect, sorry if i dont put screenshots

Environment (please complete the following information):

OS/Platform and version:  ArchLinux 5.2.1-arch1-1-ARCH and my DE its Deepin Rolling
ungoogled-chromium version: 75.0.3770.100-1 from AUR

Additional context
If you have the same problem dont maximize your browser, put it in "normal" mode, rezise your window and enjoy.

If it is persisting after a reboot I'd say it is a problem with your display and not with chromium.

its back... i think its the display but its weird... the line only appears with the siluette of the address box, i put anothers windows and video players and the burn dont appears... but its the display i can confirm that.


User losing input data after changing tab on the application page

I filled out everything on "Home" and then thought I can fill out "Team", and I lost all my written data
I thought it was a single page application

(user report)
I'll send a PR if i get the chance to work on it, doesn't seem too important multiple seem to be running into this issue so I'll have to do this soon.
a potential solution would be to show an alert() onclose if input was made

Is this sufficient enough? https://github.com/compsoc-edinburgh/hackassistant-registration/commit/a11a2ead2bc17a5ebf54613f45bf092b8e9fbcf7
I feel like the setTimeout is a bit hacky but it seems like the only option since there's no easy way to programmatically check if all the fields in a form validates (clientside)
(setting it back is necessary to prevent attempted submissions of incomplete forms from removing the check)
if the commit there is good enough, i can send a pr over

Not good enough.
https://stackoverflow.com/questions/6903457/is-the-onbeforeunload-event-not-supported-on-iphone
We should catch a href click ourselves, watching out to make sure they aren't ctrl+clicking anywhere

Quick fix for this: Show team tab only when they finished application. What do you think @qaisjp ?
I think it will avoid having hackish stuff on js and will also avoid confusion. If this works I can submit a PR myself tomorrow, should be simple.

That solution sounds good to me.


inaccurate tflite model

1. System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 20.04
TensorFlow installation (pip package or built from source): 2.7.0
TensorFlow library (version, if pip package or github SHA, if built from source):

2. Code
# h5 model (correct)
import tensorflow as tf
import numpy as np

data = np.ones((1, 320, 160, 6), dtype=np.float32)
model = tf.keras.models.load_model("model_float32.h5")
model.predict(data)

# tflite full integer quantization model (incorrect)
import tensorflow as tf
import numpy as np

interpreter = tf.lite.Interpreter(model_path="model_quant_uint8.tflite")
interpreter.allocate_tensors()

# Get input and output tensors.
input_details = interpreter.get_input_details()
output_details = interpreter.get_output_details()

interpreter.set_tensor(input_details[0]['index'], np.ones((1, 320, 160, 6), dtype=np.uint8))
interpreter.invoke()

interpreter.get_tensor(output_details[0]['index'])

3. Failure after conversion
If the conversion is successful, but the generated model is wrong, then state what is wrong:

Model produces wrong results and/or has lesser accuracy.

4. (optional) RNN conversion support
If converting TF RNN to TFLite fused RNN ops, please prefix [RNN] in the title.
5. (optional) Any other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.
links to models:
https://www.dropbox.com/s/pssd04thvgndo74/model_float32.h5?dl=0
https://www.dropbox.com/s/fpititzzwcsdmi2/model_float32.tflite?dl=0
https://www.dropbox.com/s/k88obxfdb3uowov/model_quant_uint8.tflite?dl=0

@dillon3344 ,
Can you please provide the results of the model to confirm the issue.It helps to analyse and debug the issue.Thanks!

sure, here you go
model_float32.h5:
array([[-7.0198089e-02,  3.5972762e-01, -2.1505730e-01,  3.5191351e-01,
8.9106321e-02,  7.9619581e-01, -1.8258725e+00, -1.8856851e+00,
-2.0833955e+00, -3.4985578e+00, -4.3255777e+00, -3.1656742e+00,
9.5930970e-01,  2.2216064e-01,  1.6680446e-02,  1.6571397e-01,
4.5229942e-02, -3.8629866e+00, -5.0174055e+00, -4.7954006e+00,
-6.6687365e+00,  9.8105502e-01,  4.9424553e-01, -1.2304291e-02,
1.4696333e-01,  4.4353329e-02, -3.7876964e+00, -4.8331466e+00,
-4.8142977e+00, -6.5505095e+00,  9.6800566e-01,  3.4754962e-02,
4.0744931e-02,  8.8596672e-02,  2.5172849e-05,  7.3022842e-03,
6.9045961e-02,  5.5118501e-03,  9.9129879e-01]], dtype=float32)
model_quant_uint8.tflite:
array([[204, 223, 217, 214, 223, 243, 202, 201, 197, 144, 130, 161, 239,
208, 221, 225, 223, 168, 115, 115,  21, 249, 214, 222, 225, 223,
169, 113, 113,  26, 248, 234, 232, 236, 223, 239, 253, 231, 224]],
dtype=uint8)

@sanatmpa1 ,
I was able to reproduce the issue in tf v2.5, v2.7 and nightly.Please find the gist of it here.

@sanatmpa1 any update?

@sachinprasadhs any update?

Can you try with the post training quantization by referring this documentation, also quantization is subjected to loss of accuracy, you can check if the difference in accuracy is in acceptable range here.

I've had this issue on all models. Even Resnet 50.
Is there a version where this quantization works?

@sachinprasadhs i have applied post training quantization. Is there a way to compare the outputs of each layer with the known good model to find where the problem is?

You can compare the model with the final outputs using details mentioned here, not at the layer level though.

I used the quantize debugger to get the op-by-op accuracy you see above. It takes a bit longer. There's tutorials for how to set it up.

Issue is definitely not stale.
Any update?

Thanks for reopening this.
I tested yet another model that exhibits this same issue.
I have the Netron graph for before (Keras) and after (Tensorflow Lite int8):

I'm seriously confused how everyone else has this working because I'm yet to find a working model.

Any updates on this? How it not be working for so long?
I just tried v2.9 and quantisation still does not work.

@dillon3344 @xsacha !
I was able to replicate this issue in 2.11 and nightly version(2.12dev). Just quick FYI - Suspected layers can be disabled using tfl.quantization_debugger now.
Thank you!

Hi @dillon3344
The model provides the output as int8_value which needs to be converted into real_value using quantization parameters, scale and zero point.
$$ real value = (int8 value - zero point) \times scale $$
Please find the gist for the same.
Thanks.


[Feature Request] Dutch parler-tts.

Hi, It would be nice to add dutch support to this lanaguage maybe in few months I could help with that with some grant for collecting the audio data, just would need to know how to retrain the model then and there is still the issue with voice consistency.

Hey there, you can already use this checkpoint: https://huggingface.co/parler-tts/parler-tts-mini-multilingual-v1.1
It does work with Dutch although quality might disappoint, the training dataset isn't that great!

@ylacombe Thanks I didn't know it existed tested Dutch with the default speaker and asked multiple people but the voice is like 5/10. But maybe we manage to collect some data. And if the voice cloning start working properly I could copy voice with better accent.


Add hidden columns and change export function to suit CRUD architecture

This issue still may be up for discussion.
What ive done here is made the export functionality suit the Backpack CRUD architecture rather than its native Datatables architecture.
The export will now look for columns with a given class rather than those that are visible.
The CRUD setup would be:
$this->crud->addColumn([
    'name' => "Name",
    'type' => 'text',
    'exportable' => true
]);

I have also added a hidden column template, so columns may be included in the export but not visible in the listing.
$this->crud->addColumn([
    'name' => "Name",
    'type' => 'hidden',
    'exportable' => true
]);

I think this looks really good and is a really simple and nice approach. @tabacitu think this will make it into master ? Not really a hard to review PR.. :)

Thanks @niladam, one thing i did notice this arvo is the column filter button is rendered pretty useless.  Ill have a look at it tomorrow, but it may be worth just ditching it as part of this PR?  What do you guys think?

Yeah, but the column filter should stay there.
Personally i think providing people with the option to CHOOSE what they want to see is a really strong point in datatables.
I think it makes sense to use a different approach, and not use hidden for the export => false, as this could also allow for integration with the column filters ?

Updated it now to just use styles to hide hidden cols. Column filter works and 'Actions' are not included in export.

Did you test it with ajax tables as well?

Just a note this won't work with the ajax code, also having a type hidden doesn't allow you to use the different column types for extracting the data.
The way I implemented it was to set a hidden = true attribute on the column, then in the data tables code I get the array index of all columns that are set hidden = true, and define
@if (count($hidden) > 0)
        "columnDefs" : [
        	{
                "targets" : [{{ implode(',', $hidden) }}],
                "visible" : false
        	}
        ],
@endif

This then hides the columns, allowing the visibility options on data tables to work for export.

Totally agree with both @b8ne and @Spodnet here:

actions should not be exportable (nobody needs that)
we should have a way to hide columns from the table by default
we should be able to hide columns other than text (so it’s better to have an option on all column types instead of a hidden column type)

I’m glad to say I’ve just finished this PR https://github.com/Laravel-Backpack/CRUD/pull/1703 , which fixes all of the above :-)
So I’m going to close this PR - thank you for submitting it though @b8ne - it helped developing a more general solution. The syntax is a little changed, but you’ll notice it’s all there :-)
Cheers!


Cannot use Bearer Token in new asp.net core identity API

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
Following the sample published here:
https://github.com/dotnet/blazor-samples/tree/main/8.0/BlazorWebAssemblyStandaloneWithIdentity
And the documentation published here:
https://learn.microsoft.com/en-us/aspnet/core/blazor/security/webassembly/standalone-with-identity?view=aspnetcore-8.0
I tried to setup the backend and the front end to use Bearer Token instead cookie but the WASM client doesn't login (but the Register endpoint works)
cc: @guardrex https://github.com/dotnet/AspNetCore.Docs/issues/31194
Expected Behavior
Change the Sample and /or the Documentation indicating

if the new asp.net core Identity API are suitable to be used eg. from a mobile client (that cannot use cookies)
the risk of using bearer token
if a WASM standalone client could use the token or only the cookies

Steps To Reproduce
Change line 9 of backend program.cs adding .AddBearerToken() instead to .AddIdentityCookies()
Change line 123 of frontend CookieAuthenticationStateProvider.cs putting false instead true
Exceptions (if any)
No response
.NET Version
8.0.0
Anything else?
I was able to call from Postman the new API with bearer token NOT including the
.AddAuthentication() but only the .AddAuthorization()
basically I copied what you obtain from the new Blazor 8 web app template when you choose to include identity... see attach
Program.zip

My current understanding is that we aren't going to carry all of the code to implement this scenario per ...

you then need to parse the AccessTokenResponse on the client and manage the access and refresh tokens yourself.

We've left language in the doc to that effect at ...
https://learn.microsoft.com/en-us/aspnet/core/blazor/security/webassembly/standalone-with-identity?view=aspnetcore-8.0#token-authentication
@halter73 ... You can re-open this if you (and/or Jeremy) plan to provide additional guidance on this.

Ok, how to parse the AccessTokenResponse.AccessToken?  In the .net 7.0 blazor wasm, I used JWT tokens, and they are easy to parse. But the Bearer token is not. First is encrypted, second it is not only base64 encoded but as i see in code it is in binary format. (I saw the code in BearerTokenOptions.Unprotect() method.
So I have the problem of using BearerTokens in my wasm blazor app where i need to create claims from it to be able to uze cleint side authorization on pages with the [Authorize] attribute.


add __cuda as run dependency for GPU build

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Setting cude requirement in order to prevent GPU build from behind selected by conda in a non GPU environment.
https://github.com/conda-forge/fairscale-feedstock/issues/17

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Thanks!


It's already partially possible.
In Vim mode, you can select text > type C to show the context menu, and here you can navigate through the menu items using hjkl. In this context menu, you can change the color, and also you can customize the content of the menu from the plugin settings.


Tweak the Loogle search implementation slightly in a few ways.

Fix Loogle sometimes returning responses with newlines, breaking nvim_buf_set_lines which wants none -- e.g. Complex.cos_add_mul_I returns a type with newlines in it. What we do now is split, then indent any newlines so the preview file looks "reasonableish"
Make the result be put charwise, so that e.g. #check  will never insert extra lines
Make it usable outside of telescope via require'lean.loogle'.search(), which simply returns results
Add some more typing for Loogle API responses
Use Lua errors (exceptions) in a place or two to avoid some manual error checking

There's probably a few small other things that can be tidied.

These are all quite minor I think, and mostly was a way to just poke at your code. Will give it a fresh look in the morning before merging, but @hargoniX if you have any comments feel free to lemme know!

I meant to check if Loogle ever returns names with newlines for any absurdly long one. Hopefully not but leaving a just in case note to check.

Thanks both!


One NPE in PropertiesConfiguration.java

Hello,
Our static analyzer found a following potential NPE. We have checked the feasibility of this execution trace. It is necessary to defend this vulnerability to improve the code quality.

Return null to caller (Trace staring point)
https://github.com/apache/dubbo/blob/f26ba91b67f642148a10d3b197502e29928b77bf/dubbo-common/src/main/java/org/apache/dubbo/common/extension/ExtensionLoader.java#L466

Function getDefaultExtension executes, stores the return value to defaultextension (defaultextension can be null) and return defaultextension to caller, which can be null
https://github.com/apache/dubbo/blob/f26ba91b67f642148a10d3b197502e29928b77bf/dubbo-common/src/main/java/org/apache/dubbo/common/extension/ExtensionLoader.java#L434

Return the return value of function getExtension to caller
https://github.com/apache/dubbo/blob/f26ba91b67f642148a10d3b197502e29928b77bf/dubbo-common/src/main/java/org/apache/dubbo/common/extension/ExtensionLoader.java#L426

Function add executes and one of the elements in orderedPropertiesProviders can be null
https://github.com/apache/dubbo/blob/f26ba91b67f642148a10d3b197502e29928b77bf/dubbo-common/src/main/java/org/apache/dubbo/common/config/PropertiesConfiguration.java#L40

Function next executes and stores the return value to orderedPropertiesProvider (orderedPropertiesProvider can be null)
https://github.com/apache/dubbo/blob/f26ba91b67f642148a10d3b197502e29928b77bf/dubbo-common/src/main/java/org/apache/dubbo/common/config/PropertiesConfiguration.java#L53

orderedPropertiesProvider is passed as the this pointer to function initProperties (orderedPropertiesProvider can be null), which will leak to null pointer dereference
https://github.com/apache/dubbo/blob/f26ba91b67f642148a10d3b197502e29928b77bf/dubbo-common/src/main/java/org/apache/dubbo/common/config/PropertiesConfiguration.java#L54

Commit: f26ba91b67f642148a10d3b197502e29928b77bf
ContainerAnalyzer

I will fix it.

I will fix it

fix by #8215


[Mobile] When tapping on filter, screen zooms in, dropdown does not opened and text selector is displayed when tapping on an option

Description
When tapping on filter, screen zooms in, dropdown does not opened and text selector is displayed when tapping on an option
Steps/Code to Reproduce

Tap on Filter
Tap on any Filter to open dropdown
Pay attention to screen
Tap on any option of a filter

Expected Results
Screen should maintain its correct size and text selector should not be displayed when selecting a filer type
Actual Results
Screen zooms in
https://user-images.githubusercontent.com/63799148/146270169-a54edd63-f4ac-41d4-a4f1-9446169a1a90.mov

https://github.com/chanzuckerberg/sci-components/pull/98 should fix this issue. I'll make a PR to upgrade czifui to 1.5.4


Continuous Delivery Integration

Issue

With #58 and #24 in progress allowing for continuous integration of our features, we need a way to prepare to deploy this in some way. For now, it would be nice to have something that can be manually deployed.

Solution

Find out how we can set up some kind of Continuous Delivery that follows all unit testing, NodeJs testing, and Docker build testing that can be found in .github/
Have Pipelines that are only manually run to deploy the latest of the discord-ollama container.

Figure out a proper name to the container to publish to a registry.

With #64 complete, this should help prevent any build failures

Pushing a docker image has documentation of how it is done. Find it below

docker image push

This also mentions looking at docker image tag for proper tagging of the image which will be important to look at.

Ensure to specify the platform when trying to push an image.

This talks about an example pushing a docker image to a registry for the first time. This will be important for keep a correct version for the app.

With #97, it is still Continuous Delivery as Deployments are manually triggered via tag release.


There are undesired server requests to /all route

Hi @Ngoakor12 !
STR:

open root page
open dev tools
go to network tab
observe extra requests to /all route

Hey, thanks for finding this and letting me know. I wasn't aware and I don't think I would have found it. I'll definitely investigate further. Feel free to open a PR if you can.


question: why np.concatenate is not called in solve_foreground_background.py ?

I see this code line is called when --solve-fg is setted

But in solve_foreground_background.py there is not.

Just confused why is that?

I think there is no any great idea behind this.
In the first case we concatenate foreground with alpha to output RGBA image that will be nicely displayed in most of image viewers and can be easily composed on the new background with almost any image editor.
Since solving for foreground with known alpha is very specific usecase, I made an assumption that user knows what he/she wants to do with foreground and background so we can just output raw data. Probably, it was bad design decision.
I think PR adding --rgba output option to solve_foreground_background.py and keeping --foreground and --background outputs intact is warmly welcome. I hope @MarcoForte will agree with me.

@merofeev Thank you very much.
I am working on developing a tool (for windows) with the "intelligent image cut out" functionality, and the cut out calculation should take place in the client side.
I use this code and found it a bit slow without gpu acceleration, even if I zoom in the image to about 200 pixels.
Can you provide some suggestion?

Can you provide some suggestion?

In your case I would try two things:

Don't use solve_foreground in interactive mode and just use it for final pass. In interactive mode use original image instead of estimated foreground image. In many cases solving for foreground gives just a minor barely noticeable improvement, so it can be omitted.
Apply solve_foreground to downscaled image to save some computational time.

@merofeev Thanks a lot !


database for challenge storage optional in AR projects

allows for optional usage of filesystem even if the project should be ActiveRecord enabled
I did not want to utilize my database for the challenge in a project of mine.
Thanks for the plugin!

Merged, thanks!


Create per-plant fuel summary for FERC

The FERC fuel and steam plants table are intimately related. Without the plants table, the fuel table is... just not that useful. Additionally, some of the columns in the fuel table are junk -- e.g. the reported fuel units, or the per-unit net generation values (heat rate, fuel cost per kWh) are not meaningful unless applied to the sum of all fuels. Thus, it's possible and helpful to create a simplified summary of the core data in the fuel table for use directly with the plants table.
The fundamental information of interest that comes from the fuel_ferc1 table is:

fuel costs
fuel quantity in terms of heat content
number of units of fuel consumed

From those quantities we can also derive:

fuel cost per unit
fuel cost per mmBTU
fuel heat content per unit

However, in practice, the units field is a mess. For the most part we are usually discarding the "units" string and assuming standard units based on the different fuel types (tons for coal, mcf for gas, barrels for petroleum) and "correcting" the other quantities based on those assumptions (which isn't always correct).
Having fuel information merged in with the plants table would also provide some very valuable features for inferring which records belong to the same plant (see issue #144) and differentiating between different sub-plant level generation units, since gas and coal units are often reported as separate "plants" to FERC.
It is also very helpful to be able to filter plants based on the primary fuel type of a plant, and having the relative quantities of each fuel available in the plants table would make assigning that "primary" fuel type (or the lack thereof) very easy.
Having the primary fuel types for each plant should also allow us to fill in missing fuel type values sometimes. See #101.

See the pudl.transform.ferc.fuel_by_plant_ferc1() function for how this issue was resolved.


fix: ignore errors produced by /usr/bin/file

/usr/bin/file can return errors on MacOS. Honest! I ran into this with a .wasm file in my project:
% /usr/bin/file emglken/glulxe-core.wasm
emglken/glulxe-core.wasm: ERROR: Bad magic format `version %#x (MVP)' (bad format char: #)

file exits with a nonzero error code, and therefore the makeUniversalApp() call fails.
This patch catches that case and swallows the error, effectively treating those files as unrecognized types -- thus PLAIN.

This is due to https://bugs.astron.com/view.php?id=170
Build and install file from https://github.com/file/file master, the issue is fixed there.

I'm interested in this fix too. @MarshallOfSound sorry for a random ping, but it looks like you're a big contributor here – do you think it would be possible to merge this fix (or mine in #16) and push a new release?


[SPARK-44549][SQL] Support window functions in correlated scalar subqueries

What changes were proposed in this pull request?

Support window functions in correlated scalar subqueries.
Support in IN/EXISTS subqueries will come after we migrate them into DecorrelateInnerQuery framework. In addition, correlation is not yet supported inside the window function itself.
Why are the changes needed?

Supports more subqueries.
Does this PR introduce any user-facing change?

Yes, users can run more subqueries now.
How was this patch tested?

Unit and query tests. The results of query tests were checked against Postgresql.

Can we also add test cases for window functions with:

in EXISTS
non-equality correlation cond
aggregate

added more tests

thanks, merging to master!


Doc: Create Groups required fields

The Create Groups documentation (http://docs.cloudfoundry.com/uaa/#create60) spells out the requirements for each of the members fields individually, but doesn't specify whether the members object as a whole is required or optional.

Strange -- at appears to document this now:
members			Array	Optional			Members to be included in the group
members[].value	String	Required ...

So can we close this?

Thanks for the update!


Suppress update dialog until the album ends

Never show the update dialog while music is currently playing, wait until the end of the album ✨ ✨ ✨ ✨

I had the mini player open when the update message poped up. I had to kill the app as there was no way to clsoe the notification. Minor issue with an easy work around.

@jimmyt86 Looks like you were back on 3.5.0 when you got the update (I can tell from the transparent bar at the top of the mini player).  This issue was fixed in an update between 3.4.0 and 4.0.1 (can't remember which).  Please be wary of hijacking issues 👍
For instance #1817 (quite a recent issue) is exactly what you just posted in this unrelated issues 😄

Thanks @MarshallOfSound. I checked, but I didn't realise there was a closed ticket filtered out. This seemed related, as the popup appeared mid song.  Glad to see it's already fixed :)

I totally forgot this was here.  I just pushed a change to the UpdateModal that will prevent the modal from being displayed again for an entire day after clicking "Not Now".  I don't feel as though the application should update without giving any warning or feedback so the modal has to appear at least once IMO.
This change should stop the modal being annoying 👍


Parse magic failed

Hello, I try to use your model. I've installed nuget packages and loaded these bin and param files (RFB-320.bin, RFB-320.param).
But it failed with this error: System.AccessViolationException at the string var faceInfos = ultraFace.Detect(inMat).ToArray();.
And it writes in the console this message

My code looks like code from your demo example.

Any ideas to fix this?

@Feodoros
I guess your RFB320.param is corrupted.
I deployed the correct RFB-320.bin and corrupted RFB-320.param (param is renamed RFB-320.bin).
This is result of running program.
dotnet run -c Release -- RFB-320.bin RFB-320.param test.jpg
parse magic failed
network graph not ready
Processing test.jpg
find_blob_index_by_name input.1 failed
find_blob_index_by_name 537 failed
find_blob_index_by_name 538 failed
find_blob_index_by_name 539 failed
find_blob_index_by_name 540 failed
heatMap is nullptr,please check!

At least, parse magic failed means that parameter file has something wrong.
https://github.com/Tencent/ncnn/blob/e841ae73c6f1282c365be84f226bfff56c37c367/src/net.cpp#L517

Hello, thank you for your fast reply. Yes, it's really works! One more question: Does this nuget package work on Windows 7 x64?

And I got memory leak too. I cloned your last develop branch, but still have this memory leak, when I work with loop of images.
How I can fix this?
https://github.com/takuya-takeuchi/UltraFaceDotNet/issues/3

OK. I fixed this with this simple Dispose of all objects. Sorry for my concern :)

@Feodoros

Hello, thank you for your fast reply. Yes, it's really works! One more question: Does NcnnDotNet nuget package work on Windows 7 x64?

I think so. But I have not ever tried.

OK. I fixed this with this simple Dispose of all objects. Sorry for my concern :)

No problem :)


Calendar pop-up window does not close if using Macbook touchpad [fix #628]

https://github.com/Hacker0x01/react-datepicker/issues/628

Current coverage is 91.71% (diff: 100%)

Merging #629 into master will decrease coverage by 0.12%

@@             master       #629   diff @@
==========================================
  Files            12         12
  Lines           539        543     +4
  Methods          16         16
  Messages          0          0
  Branches         98         99     +1
==========================================
+ Hits            495        498     +3
  Misses           15         15
- Partials         29         30     +1

Powered by Codecov. Last update 9c8dfe6...106dc37

Looking good to me! Is there anything else to be done / can I help somehow?

@Kadrian
Well, I guess the solution i've provided isn't most accurate one.
It looks more like a crutch.
If you can figure out how to fix it more "pretty" - you are welcome :)


fix(Button): add href="#" when no href provided for anchors

Fixes #44
Also converted AnchorSpec to TS

I think we removed this behavior for a reason...but I don't remember what it was. Do know we why this situation occurs? Is it that users just forget to add a href?

This was missed in RB since the nav dropdown toggle doesn't explicitly supply a href and user isn't able to.
Since we are applying button semantics to an anchor, I think it makes sense to put it here?  If not, I can explicitly declare it on the RB side.

IDK honestly, lets add it and see if it causes issues.

Might be a dumb question - but is this available as a release on the npm registry or do you have a schedule/idea of when it will be? :)

@laneparton I'll try and cut a release on Sunday night.


Add support for core-lightning

This adds support for core-lightning as a lightning node backend in addition to LND.
The high level changes:

Create a new trait called LightningNodeClient with abstract methods for interacting with a lightning node.
Move the existing LND code into a struct LndClient that implements the trait.
Create a new struct CoreLightningClient that implements the trait.

Note: core-lightning does not allow multiple invoices with the same memo/label. I updated the files.rs module to append a random UUID to the end of the memo/label when adding invoice to avoid conflicts.

@casey Slow tests should be passing now
I tested with the following:
cargo test

cargo test --features slow-tests slow_tests

cargo test --all-features --all

@casey Slow tests should be passing now
I tested with the following:
cargo test

cargo test --features slow-tests slow_tests

cargo test --all-features --all

Check out the github actions workflow for the tests that run on CI.

A few updates:

the CI build for ubuntu-latest is no longer failing, but it's getting cancelled by github actions after a few minutes for some reason.
the CI build for windows-latest is failing because the cln-rpc library has a dependency on unix: https://github.com/ElementsProject/lightning/blob/aae5e3d070ee0ab30a865cabe6ba40fc52e72a18/cln-rpc/src/lib.rs#L11-L12
I think that the cln-rpc library breaks portability with Windows OS.
the Docker multi-platform build is failing with exit code 137, meaning that it is probably getting killed for using too much memory. This appears to be an open issue with docker multi-platform builds: https://github.com/docker/build-push-action/issues/621

The ubuntu-latest build was failing was getting canceled because it was configured to "fail-fast" as soon as the Windows build failed. I fixed this by adding the fail-fast: false config.
Now the ubuntu-latest build is passing all of the tests but failing on Clippy lint of the generated LND protobuf code.

@casey  Another update:

Docker github action is failing because of the OOM issue.
macos-latest is passing.
ubuntu-latest is passing.
windows-latest is failing, because the cln-rpc library is not compatible with Windows.

I think it's safe to disable the tests on windows. It's not the end of the world if Core Lightning isn't supported there.

All github actions are now passing!

Very nice! I'm prepping for a BTC++ workshop, so I probably won't be able to look at this until I get back on the 13th of June, but I'll review it then.

I'm slammed with other, much more frivolous projects. (My Bitcoin NFT scheme and a generative art engine.) So no real time to give this a proper review. One thing I was thinking was that, in liu of review, we could merge this into a non-master branch, e.g., a core-lightning branch, and point to it from the readme. You could point people to the branch and they could build it from source, you could also make changes to the core-lightning branch without much review (I would just rubber-stamp PRs as long as the tests passed). What do you think?

@casey I have already merged these changes into the master branch of my fork. So maybe just use that instead of the core-lightning branch in this repo?
I am going to open a new PR to point to my fork of Agora in umbrel-apps, now that they support core-lightning. EmbassyOS also uses core-lightning, so I will also point to my fork in the EmbassyOS wrapper.
I guess it doesn't hurt to have a core-lightning branch in this repo, unless it confuse people.

It's up to you! It might have better visibility in the main repo, but either way is fine. Definitely open a PR adding a link to your branch from the readme.

Sure. I'll do that!
For the umbrel app, the way they organize apps is that each has a list of other apps as dependencies. So for any lightning app, they package it twice (once for LND and once for core-lightning). For example, RideTheLightning:

https://github.com/getumbrel/umbrel-apps/blob/master/ride-the-lightning/umbrel-app.yml
https://github.com/getumbrel/umbrel-apps/blob/master/core-lightning-rtl/umbrel-app.yml

So I can leave the existing package the same, pointing to this repo. And for the core-lightning version, point to my repo.

So I can leave the existing package the same, pointing to this repo. And for the core-lightning version, point to my fork.

Perfect, sounds good.


routerWillLeaveHook being deleted

Hey guys,
I am using react-router for our universal app at work. I just bumped up to react-router 2.0.0 and am very interested in the navigation confirmation feature passed through the context:
https://github.com/reactjs/react-router/blob/latest/docs/guides/ConfirmingNavigation.md
The thing is that we are internationalising our app and I need to dynamically append the iso of the country every time the location changes, at the beginning of the URL.
For instance, someone clicks on the '/login' button, which, if the user is based in the US, will redirect to '/us/login'.
It works great most of the times but at some point my routerWillLeave hook is being deleted within the RouteHooks (I've been debugging and seeing that happen but it gets a little bit too complex for me to exactly understand why).
I guess that's it has to do something with the way I am using it/what I am doing with it. Here is my code in my routerWillLeave hook:
routerWillLeave(nextLocation) {
    const hasCountryPath = /^(\/au|\/gb|\/intl|\/us)([\/]|$)/
                                .test(nextLocation.pathname); //regex to figure if location already starts by the iso or not

    if (!hasCountryPath) {
      const url = //building new URL to push instead of the previous one. for example. /login would become /us/login

      this.context.router.push(url); //pushing the new URL
      return false; // returning false so that the prior transition would not happen
    }
  }

The hook is declared at my top component level.
As I said before, doing that works perfectly in most of the cases, until at some point my routerWillLeave hook is being detached from the RouteHooks array and therefore never triggered again.
If someone could tell me if that's just not possible, or if there is something wrong with the library somewhere that'd be great!
Cheers

Thanks for your question!
We want to make sure that the GitHub issue tracker remains the best place to track bug reports and feature requests that affect the development of React Router.
Questions like yours deserve a purpose-built Q&A forum. Could you post this question to Stack Overflow with the tag #react-router? https://stackoverflow.com/questions/ask?tags=react-router
We also have an active and helpful React Router community on Reactiflux, which is a great place to get fast help with React Router and with the rest of the React ecosystem. You can join at https://discord.gg/0ZcbPKXt5bYaNQ46


Provide content-length in the HTTPFields' literal when creating a response

This allows the HTTPFields to allocate sufficient room for Content-Length, Server and Date, preventing reallocs

The main issue was that contentl-length wasn't set yet, so the HTTPFields had a size of 1 due to reserveCapacity being called with 1 (from the literal). Then we were immediately adding our content-length on top of the literal, finishing it off with server and date headers.

This is likely a great example of something that could use optimisation in HTTPFields

This was already covered in PR#454


Express integration returns 500 instead of 401 with useJWT() plugin

Describe the bug
Hi all. First of all, thank you for creating this awesome, awesome library. I've been digging into the useJWT(), and it looks like the plugin itself actually returns a 401: https://github.com/dotansimha/graphql-yoga/blob/main/packages/plugins/jwt/src/index.ts#L117C21-L117C21
Is there a standard best practice using Express to bubble up errors properly? I'm all for error masking but we want to be able to display an Unauthorized error on the front-end and act accordingly instead of a 500 Internal Error code which is inappropriate.
(This was initially a discussion question but we opted to move it to an issue.)
Your Example Website or App
https://codesandbox.io/p/devbox/loving-dijkstra-mfgj67
Steps to Reproduce the Bug or Issue
Here is my example loosely based on the Express Integration docs:
import express, { Application, Request, Response } from "express";
import dotenv from "dotenv";
import { createYoga, createGraphQLError } from "graphql-yoga";
import { useJWT } from "@graphql-yoga/plugin-jwt";
import helmet from "helmet";

import { pgPool } from "./lib/db/postgres";
import { createSchema } from "./lib/graphql/schema";

const PORT: number = Number(process.env.NODE_PORT) ?? 15000;
const app: Application = express();
const yogaRouter = express.Router();

yogaRouter.use(
  helmet({
    contentSecurityPolicy: {
      directives: {
        "style-src": ["'self'", "unpkg.com"],
        "script-src": ["'self'", "unpkg.com", "'unsafe-inline'"],
        "img-src": ["'self'", "raw.githubusercontent.com"],
      },
    },
  }),
);

export async function init() {
  try {
    const schema = await createSchema();

    const yoga = createYoga({
      schema,
      cors: {
        origin: ["http://localhost:3000"],
        methods: ["GET", "POST"],
        allowedHeaders: ["Content-Type", "Authorization"],
        credentials: true,
      },
      plugins: [
        useJWT({
          issuer: process.env.AUTH_ISSUER ?? "",
          jwksUri: `${process.env.AUTH_ISSUER}/oauth/v2/keys`,
        }),
      ],
    });

    yogaRouter.use(yoga);
    app.use(yoga.graphqlEndpoint, yogaRouter);
    app.use(helmet());

    app.listen(PORT, "<IP_ADDRESS>", () => {
      console.log(`Server started on http://<IP_ADDRESS>:${PORT}/`);
    });

    return app;
  } catch (err) {
    console.error(`Failed to start the server`, err);
  }
}

init().catch((err) => console.error(err));

Expected behavior
I would expect to see a 401 Unauthorized error response to bubble up and be returned, instead of a 500 Internal Server error.
Screenshots or Videos
No response
Platform

OS: [macOS, Linux]
NodeJS: [20.9.0]
@graphql-yoga/* version(s): [5.0.0, 5.0.2]

Additional context
No response

Hi @peterklingelhofer and thank you for the report!
Sorry but I'm not adding a lot here but just labeling it according to our new Contribution Guide and issue flow.
It seems already got into stage 1 thanks to your reproduction! Thank you for that!
Now in order to advance to stage 2 we'll need a failing test, would be great if someone could help progress the issues through the stages.
Thank you and sorry that this comment is not a complete solution (yet).


Cacli Train Command Error -macOS

I am brand new to Cloud Annotations but I have done the Image Classification and training as per the documentation here: https://cloud.annotations.ai/workshops/classification/7.html
On the command
$ cacli train
I am getting this error
Starting training run... error Unable to get /v3/wml_instances/eb1fa779-607c-4d6b-ab0f-61c11bfb16aa [400]: {"trace":"4e89c1475002a38642e17f52720f4588","errors":[{"code":"invalid_request","message":"Instance in v2 plan should use v4 API."}]}
What should I be doing different?

Hi there, your watson machine learning instance is v2 which isn't compatible with cloud annotations at the moment. The best option is to use the train in colab button in the UI

is there any way to change the learning instance or update something

No sorry :(
all new instances are v2 as of September 1st. I’m doing my best to migrate, but there have been a lot of issues.
We are tracking the migration in this issue: https://github.com/cloud-annotations/training/issues/222


Refactor clinical services

Move to common module
Remove poc-common module
Simplify module structure
Move error messages template to application module

Coverage decreased (-0.006%) to 65.753% when pulling 35771a21f04f53777e514e91298d0d1dd0dbc1be on ynurmahomed:refactor-clinical-services into fb84b6cc534c525aedf03aec072fa4f61d7c66c5 on esaude:master.


[utils] refactor profiler

refactor profiler. Our profiler is a subclass of pytorch's profiler now, which can use other utils of pytorch's profiler (such as tensorboard_trace_handler and schedule).
add stateful tensor memory profiler extention.
move old profiler to colossalai.utils.profiler.legacy.

Example:
from colossalai.utils.profiler import profile
from torch.profiler import ProfilerActivity, tensorboard_trace_handler, schedule

engine, *_ = colossalai.initialize(model, optimizer, criterion)

with profile(activities=[ProfilerActivity.CPU, ProfilerActivity.CUDA],
             with_stack=False,
             record_shapes=True,
             profile_memory=True,
             on_trace_ready=tensorboard_trace_handler('events/st'),
             schedule=schedule(skip_first=0, wait=0, warmup=0, active=NUM_EPOCHS * NUM_STEPS),
             profile_stateful_tensor_memory=True,
             engine=engine) as prof:
    for epoch in range(NUM_EPOCHS):
        for i in range(NUM_STEPS):
            run_step()
            prof.step()

All tests passed.


drivers: sensor: tmp116: allow setting the configuration register

The eeprom is unlocked prior to setting the configuration so the settings are persistent.

I made it as a draft until I get the hardware ready to test.
Comments are welcome in the meantime.


Tutorial instructions and code contain errors

I am currently working through the Power BI visual development tutorial. On the 'Add formatting options' page, there are a number of issues that I have come across:

Under 'Adding formatting options', point (2) suggests that I should see a message that reads 'Formatting options are unavailable for this visual.' This is not the case - I can alter the title, background, and more for this visual.
Under 'Adding custom formatting options', point (2) provides code to add into the objects object. However, pasting this between the {} provided previously throws an error, as there are extra curly braces. Perhaps this instruction should be updated to simply replace "objects": {}, with:

"objects": {         "circle": {             "properties": {                 "circleColor": {                     "type": {                         "fill": {                             "solid": {                                 "color": true                             }                         }                     }                 },                 "circleThickness": {                     "type": {                         "numeric": true                     }                 }             }         }     },

Under 'Adding custom formatting options', point (8) provides code which imports VisualSettings then references it within the Visual class. Point (5) actually creates VisualSettingsModel, which means that this code throws an error. I updated the point (8) code to import/reference VisualSettingsModel.
Point (10) seems to break the Power BI visual. The code up to this step works correctly, but after this point, no visual is displayed. Any combination of those three lines of code causes the visual to disappear.
I was unable to get past point (10) so that may be the cause, but point (15) shows that new settings should be available in the Power BI Formatting pane. These do not appear for me.

I would be more than happy to provide any other documentation or details that you might find useful to support this issue. Thank you!

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

ID: 4bc4f82a-1cec-444c-8561-c02aa38caa96
Version Independent ID: c05253d9-f12e-b2ee-763f-96e4a76fec54
Content: Add formatting options to a Power BI custom visual - Power BI
Content Source: powerbi-docs/developer/visuals/custom-visual-develop-tutorial-format-options.md
Service: powerbi
Sub-service: powerbi-custom-visuals
GitHub Login: @mberdugo
Microsoft Alias: monaberdugo

Thanks for your question, @domdtyler. This tutorial is in the process of being updated. Hopefully, it should be done by next week.

Hi @domdtyler . The new tutorial is up. Good luck!


117 fix cluster installation guide

It is very doubtful that this link  to the file "docker-compose-local.yml.md " it will open reliably.
Given that the text of the file does not take up very much space, I would prefer to place this text at the end of the file and open it via internal link:
file
. . .
The section name
Program text


Status flags are not correct

The status flags for the position message are described in the PDF that can be found here: http://swift-nav.github.io/libsbp/
The status byte should be bitmasked before it can be interpreted as an integer value. Are you interested in a PR?

Thanks @rokusottervanger the bitmasked solution would result a more straightforward code. PR is welcome of course.
https://github.com/szenergy/duro_gps_driver/blob/fa11077a223896142b0a0be7e27d90eca3f59801/src/duro.cpp#L143

It's not only more straightforward. The current implementation leads to the wrong output. This is not an enhancement, it's a bug.

This issue made me curious, so i added the bitmasked version of the code, is this how you imagined it?

thanks @rokusottervanger


TS Migration

Original source files are dirty JS, that are requires to rewrite into TS to be release ready.
index.js - main execution chain is here.

madge should be isolated
all possible "hardcoded" params should be replace with barista.config.json

fs.js - helper functions to work with file system
neo.js - helper functions to work with Neo4j

Done


docs: remove extra space

move | counter: Counter| {...} -> move |counter: Counter| {...}

Yes, I'm sure that's correct and probably the result of me replacing cx, before replacing cx, .


What is wrong with me?

Dear Author:
Recently, I git the mav_dji_ros_interface project in my computer, and I follow the wiki page you list. But I found it dosen't work, for example, I find the package mav_dji_ros_interface is none, it didn't contain any launch file, while the command in the wiki package shows that it dose exist, I don't know while, Could you please tell me how to deal with that. Thanks sincerely!

Dear LZMHIT,
The folder "mav_dji_ros_interface" only contains package.xml file that describes dependencies to other packages such as dji_sdk_lib, dji_comm, dji_interface.
Please have a look ROS tutorial (http://wiki.ros.org/ROS/Tutorials/CreatingPackage) section 5.
BTW, launch files you are looking for are located at https://github.com/ethz-asl/mav_dji_ros_interface/tree/master/dji_interface/launch
Best.


initial

Pre-submission checklist:
Please check each of these after submitting your pull request:

[ ] Are you only including a repo_url if your project is 100% open source? If so, you need to pick the single best GitHub repository for your project, not a GitHub organization.
[ ] Is your project closed source or, if it is open source, does your project have at least 300 GitHub stars?
[ ] Have you picked the single best (existing) category for your project?
[ ] Does it follow the other guidelines from the new entries section?
[ ] Have you included a URL for your SVG or added it to hosted_logos and referenced it there?
[ ] Does your logo clearly state the name of the project/product and follow the other logo guidelines?
[ ] Does your project/product name match the text on the logo?
[ ] Have you verified that the Crunchbase data for your organization is correct (including headquarters and LinkedIn)?
[ ] ~5 minutes after opening the pull request, the CNCF-Bot will post the URL for your staging server. Have you confirmed that it looks good to you and then added a comment to the PR saying "LGTM"?

Build failed because of:
Membership category "CNCF Members" does not have corresponding category in landscape.yml

Deploy preview for landscape failed.
Built with commit 7f0ea47160d81f2aa10e521f68f3d13ddad8b7e2
https://app.netlify.com/sites/landscape/deploys/5e37653e0940ed000ad31b4a


fix(gas-oracle): handle Optimism Ecotone blob fees

Overview
Fixes gas price overestimation issue in the Optimism gas oracle following the Ecotone upgrade which introduced blob fees.
Changes

Add OptimismGasComponents struct to track L1/L2 fees
Implement proper L1 data posting cost calculation
Consider both base fee and blob fee with scalars
Fix gas price overestimation issue

Testing

Added unit tests covering:

Total gas calculation including L2 execution and L1 data fees
L2 data cost calculation with mock data
Both tests pass and verify correct fee calculations

Impact
This fix ensures accurate gas estimations for Optimism transactions by properly accounting for:

Base fee with scalar
Blob fee with scalar (new in Ecotone)
L2 execution costs

Fixes #336

Thanks for this. In the code comments where the gas price calculations are implemented, please add relevant links to the OpStack repo.

Closing because the current implementation of the gas estimate is identical to what's available in the optimism-sdk
Tesseract estimates are approximately correct as shown from the samples in the doc
https://docs.google.com/document/d/1-1Eo-5viezojO3EjPlYMVpgTx3zMVFxDWitG9aZnoDc/edit?tab=t.0
The issue was fixed by this PR https://github.com/polytope-labs/hyperbridge/pull/337


[WIP] Migrate from Stripe legacy checkout to the Stripe hosted checkout page

Testing

Run the migrations
You need to be on Stripe API version 2020-08-27(Current version). Change at https://dashboard.stripe.com/test/developers.
Run the stripe cli tool for recieving webhook events
https://stripe.com/docs/webhooks/test
stripe listen --forward-to localhost:9991/stripe/webhook/
Run dev server

Useful card numbers

4242 4242 4242 4242 - Normal card. Gets added to account and charge succeeds
4000000000000341 - Card is added to account but charge fails
4000 0027 6000 3184 - Requires authentication. .

Prep commits

Add a prep commit for upgrading the stripe API
Get the prep commits merged

https://github.com/hackerkid/zulip/tree/stripe-checkout-migration-cleanups-branch has CI only for that

V1
Billing page

[ ]  Fix the position of loading indicator in add a new card.
[ ] Ask to wait or something after clicking update card, since there is a delay between clicking on the button and redirecting to checkout page.

Use the create ajax request helper

[ ] Add authentication for webhook views

General (Handle while writing tests)

[ ] Handle all the errors in upgrade view properly
[ ] Handle errors during upgrade process and show it in status page.

Further investigation

[ ]  Consider add a field for PaymentIntent in Session model.
[ ]  If a customer has default_source and default_payment_method, which one will get the preference?
[ ] Figure out way to handle errors during invoice creation? Is this still applicable?
[ ] Check whether duplicate webhook events will cause issues
[ ] Changes statement descriptor behavior for card payments
https://stripe.com/docs/upgrades#2019-02-19

Tests

[ ] Write frontend tests
[ ] Write backend tests

Production checklist

Set business name in Stripe
Set webhook URL
Upgrade the Stripe API to 2020-08-27
Enable Send emails when card payments fail
Enable Send emails about expiring cards

V2
The features here are not necessary to make stripe hosted checkout version work. They are good to have. We can work on them after v1 is merged.
PaymentIntent

Handle states other than success and error in PaymentIntentStatus page
- https://stripe.com/docs/payments/intents
Send email asking users to upgrade again if PaymentIntent fails
If you open /upgrade page and there is an existing PaymentIntent that has not succeeded or failed, show it.

Cards which requires authentication does not seems to be working in dev environment. Try with 4000 0027 6000 3184 for 3D secure 2
If a payment intent fails because of card error, the user click on retry payment. If this errors include details on how the user can retry.

Billing page

Updating cards should also check whether the new card is chargable. We don't do this in the old version as well.
Hide other input fields when license is updated. Same for downgrade as well.

General

Make use of https://www.npmjs.com/package/@stripe/stripe-js
To best leverage Stripe’s advanced fraud functionality, include Stripe script on every page on your site, not just the checkout page.

Production

Enable Send a Stripe-hosted link for cardholders to authenticate when required
Enable Request 3D Secure for Billing payments that match Radar rules

Docs

https://stripe.com/docs/payments/save-and-reuse
https://stripe.com/docs/api/payment_intents/confirm
https://stripe.com/docs/api/payment_methods/attach
https://stripe.com/docs/webhooks/test
https://stripe.com/docs/webhooks/best-practices*

https://drive.google.com/file/d/1q_9CrLMYR7Jr2NvNfZFBisDOP73xiPEK/view should help visualize the flow better.

I have got a basic version working which collects the payment details using the new Stripe hosted checkout pages,  create charge using PaymentIntent and uses webhook to handle the upgrade.
The work remaining now is to handle all the PaymentIntent webhook events and errors properly, take care of onboarding flow, tests etc,
The tests part is going to be tricky since now we rely on webhooks for initiating various actions.

@timabbott Can you merge the first two commits in this to minimize merge conflicts (mostly the view splitting commit.)? They are prep refactoring commits. More details in the commit message. I have pushed a seperate branch with only those two commits to make sure that the CI passes. https://github.com/hackerkid/zulip/tree/stripe-checkout-migration-cleanups-branch

I merged those first two commits as the series ending with 55a9a019a0ffd9827e0898567b53f0c91aa4b40b; thanks @hackerkid!

Will it be possible to merge the "Upgrade Stripe library" commit without breaking anything, or does it need to be merged atomically with the workflow change?

Will it be possible to merge the "Upgrade Stripe library" commit without breaking anything, or does it need to be merged atomically with the workflow change?

@timabbott You can merge the API upgrade commit. It's not coupled with the big migration. To be clear this upgrades the Stripe API to the latest version. Not the library. So we need to switch the API version to 2020-08-27 from the Stripe Dashboard the same time this gets deployed in production.
The tests are passing for the commit at https://github.com/zulip/zulip/compare/master...hackerkid:stripe-checkout-migration-cleanups-branch-2
Our Stripe webhook integration seems  to have last updated in 2020-03-02 which is one version before the current version.
➜  gg api_version zerver/webhooks/stripe | grep 2020
zerver/webhooks/stripe/fixtures/invoice_created.json:    "api_version": "2020-03-02",
zerver/webhooks/stripe/fixtures/refund_event.json:    "api_version": "2020-03-02",

I don't know whether this will break the webhook integration(have not tested it) but we should not be the only folks using it right? So someone else must have been on the latest version of Stripe and we should have known if it breaks via Sentry.  Or do you recommend testing generating various events manually and check that the Stripe ingeration doesn't break when we switch the API to latest version?

OK, merged that prep commit and it seems to be working well in production.

I have made all the existing backend tests pass which I think was the hard part when it came to testing this. Now the work remaining is to

Add coverage for the newly added backend code.
Make the existing frontend tests pass.
Add coverage for the newly added frontend code.

The backend coverage is now 100% for corporate/.

Awesome!  Let me and @mateuszmandera know when this is ready for a review / manual testing.

@timabbott I think this is ready for manual testing. I am working making the frontend tests work. So pretty much all the work for V1 has been completed.

Added 1 commit each at the begining and end to make sure that the API version of the Stripe account and the one billing system supports are the same.

I rebased this.  An initial note:

We should put STRIPE_API_VERSION in zerver/lib/stripe.py, or similar; there's no reason for it to be in the project's central version file.  Would it make sense to just pull the first and third commits out to a separate PR that just does that?  @eeshangarg thoughts?  The main downside is that the middle commit definitely merge conflicts with the third; I haven't investigated deeply.

@timabbott @alya This is now ready for a review! :)

Is there something I should be reviewing from a UI/UX perspective, and if so, what is it? :)

@alya: Not really, it is mostly the same workflow except the checkout page is a separate page and is the default theme for a Stripe checkout page designed by Stripe.

This is enough of a rewrite that it's difficult to review for details.  I don't see anything obviously wrong, and there's probably a few strings we'll end up wanting to tweak after going through the payment flow in production, but I expect we'll need to be very much on the lookout to promptly fix bugs seen when running this in production.
I don't see a way to move forward on this part of the codebase other than merging this, so I'll do so now.  Merged, thanks @hackerkid and @eeshangarg for all the work on this!
I wish that Stripe didn't force this sort of total rewrite of large parts of integrating with them, but unfortunately we just have to live with them doing so.


Refactor Source Operators to use RelationManager

Previously, all source operators directly interacts with DataWriter and DataReader by providing the index directory and schema.
Now, after the RelationManager is finished, the upper-level source operators should interact with RelationManager instead. Lower-level dataWriter and dataReader should be hidden from these operators.

After PR #378 , all source operators are changed to use the "tableName" instead of "dataStore" (which includes the directory of the table). This issue can be closed.

Per our previous discussion, is it possible to hide the Lucene layer completely inside RelationManager so that other packages cannot see it?


Onchange event for  is not propagated in react when not used with class "browser-default"

I use materialize with react and had an issue on <select>s onChange event propagation.
This way the event is not fired, this.handleSelectChange is not called.
<select value="B" onChange={this.handleSelectChange}>
    <option value="A">Apple</option>
    <option value="B">Banana</option>
    <option value="C">Cranberry</option>
</select>

When i add the class browser-default to select it works pretty well.
<select className="browser-default" value="B" onChange={this.handleSelectChange}>
    <option value="A">Apple</option>
    <option value="B">Banana</option>
    <option value="C">Cranberry</option>
</select>

Also created an issue on the react repo: https://github.com/facebook/react/issues/3667

Hi, Have you solved this? If so please share details...

Hi, I just used the 'native' select form element with class browser-default.

ok fine... I was able to do that with browser-default.

Based on comments in the other thread, I worked around this with the code below:
handleSelectChange(event) {
     setState({value: event.target.value});
   },

componentDidMount() {
     $(React.findDOMNode(this.refs.mySelectBox)).on('change',this.handleSelectChange);
   },

render() {
   return (
    ...
      <select ref="mySelectBox" value={this.state.value}>
    ...
   )
}

This is probably caused by the fact that selects often ignore user input, as described in this bug report I filed:
https://github.com/Dogfalo/materialize/issues/4098
Please re-open that issue, and mark this as a duplicate. I described and fixed an input event tracking problem that causes select menus to ignore the item you select, if you don't click on it very quickly, or if you press down and drag.
To demonstrate this, try popping up the menu by clicking (because pressing and dragging does not work, and starts a text selection gesture, which is another bug), then point at an item different than the currently selected item, then click very quickly. If you click quickly enough, it works. Then try clicking on another item than the one you just selected, but more slowly so you're holding the button or finger down for about half a second to a second. (Not a press-and-hold gesture, which is slower, but just a slow click, which is very common for people to do.) Notice that the menu pops down at the instant you press down on the button or touch the screen, but the onclick handler that reports the selected item is never called. That is the bug I reported and fixed, but it was closed unfortunately because the developers don't see it as a bug. But it certainly is.

As materialize was not made to work specifically with React, I can not be sure of the exact problem here. You can try this library however, it may help in this scenario: https://github.com/react-materialize/react-materialize

onchange doesn't seem to work with just materialize either when it comes to selects


Index out of bounds

When using this code:
import trading_calendars as tc
exchange = tc.get_calendar('XSWX')
exchange.next_open(pd.Timestamp('2021-12-27')

an index 8032 is out of bounds error for axis 0 with size 8032 is thrown.
Traceback (most recent call last):
  File "<input>", line 1, in <module>
  File "C:\Users\raphael\anaconda3\envs\levine\lib\site-packages\trading_calendars\trading_calendar.py", line 462, in next_open
    idx = next_divider_idx(self.market_opens_nanos, dt.value)
  File "C:\Users\raphael\anaconda3\envs\levine\lib\site-packages\trading_calendars\calendar_helpers.py", line 12, in next_divider_idx
    target = dividers[divider_idx]
IndexError: index 8032 is out of bounds for axis 0 with size 8032

Seems to be a pandas Problem?

This is an artifact of https://github.com/quantopian/trading_calendars/blob/master/trading_calendars/trading_calendar.py#L47.
You can get around this via https://github.com/quantopian/trading_calendars/issues/147.
There was some discussion about making this process smoother via https://github.com/quantopian/trading_calendars/pull/176.
I think in the next year I'll work on some backwards incompatible features and put this enhancement in under a new version.


move alert message to bottom of the row

Are you requesting a feature, reporting a bug or ask a question?
Ask a question
What is the current behavior?
Alert message is under the question, is there any chance that if i could move alert message under form-inline class, please?

Provide the test code and the tested page URL (if applicable)
Tested page URL:
http://surveyjs.org/examples/react/trigger-visible/
f12 on it, do not select anything, click submit button

Hi!
Thank you for the interest in our product!
We've not such a functionality yet. I've added this ticket to our todo list.
We're going to introduce an option questionErrorLocation with values top and bottom in the Survey object.
I'll update this ticket when it will be implemented.

hi @tsv2013
do you think its good idea for alert message to be disappear once user has answered question?

one more request, is there any chance that If alert message appears, a question highlighted?
I would like to add red border wrap the question, if alert message appear.
Example as following:
|question: test question?               answer1 answer 2

* Please answer the question

@xiuzhen1103 We have added a new property survey.questionErrorLocation. Set it to bottom.
The functionality will be available in the next minor update. We will release it on this week.
Thank you,
Andrew

Thnaks @andrewtelnov, Is there any chance that if I could add border red on the question is not answer, please?


[Network Issue] Got 504 response and chat keep "thinking" for a long time

VS Code Version: latest
OS Version: Windows 11
Logs:
2024-03-13 00:53:58.503 [info] [auth] Got Copilot token for LikeDreamwalker 2024-03-13 00:54:07.566 [info] [chat fetch] url https://api.githubcopilot.com/chat/completions 2024-03-13 00:54:07.566 [info] [chat fetch] modelMaxPromptTokens 3072 2024-03-13 00:54:07.566 [info] [chat fetch] modelMaxResponseTokens 2201 2024-03-13 00:54:07.566 [info] [chat fetch] chat model gpt-4 2024-03-13 00:56:09.896 [info] [chat fetch] request.response: [https://api.githubcopilot.com/chat/completions], took 122328 ms 2024-03-13 00:56:09.900 [info] [chat fetch] Request ID for failed request: ,d51aafaa-3ef0-4ed1-b259-df8df1e02a53 2024-03-13 01:00:58.258 [info] [chat fetch] url https://api.githubcopilot.com/chat/completions 2024-03-13 01:00:58.258 [info] [chat fetch] modelMaxPromptTokens 3072 2024-03-13 01:00:58.258 [info] [chat fetch] modelMaxResponseTokens 2201 2024-03-13 01:00:58.258 [info] [chat fetch] chat model gpt-4 2024-03-13 01:03:01.777 [info] [chat fetch] request.response: [https://api.githubcopilot.com/chat/completions], took 123519 ms 2024-03-13 01:03:01.779 [info] [chat fetch] Request ID for failed request: ,aea37b9c-2873-44b8-ad5c-7689767d1243

Also, I tried to ping api.githubcopilot.com, but the response is pretty bad, and even if I use a proxy seems like it won't help

I didn't encounter this issue until months ago. I tried many ways, like presetting a specific proxy in vscode, and sometimes it worked. Now, this issue can be met very often, and even I am using codespaces, which I guess is not connected with my client directly but should be the codespaces server.

Are you still experiencing this? Does it happen often? Wondering if this is a service incident or something else

Are you still experiencing this? Does it happen often? Wondering if this is a service incident or something else

I tried two of my products, and now it seems like everything is fine, but I think this issue might happen again.
I can't tell if this issue is based on a server issue cause it seems like the connection between client and server goes wrong, and I don't know why. This might help that I meet this issue more often when I am using local vscode rather than codespaces. And not all the time I can get a 504 back, mostly a long time "thinking" which I cannot wait anymore.
I also considered this might be a network issue, but when this issue happened, I was able to use GitHub, until upper I tried ping the host directly, and it really responded badly. And now the server responds steadily enough.

And I can reply with requestID on this issue when I meet this issue again, maybe that can help you guys figure out what's wrong.
Thanks for your reply.

Yeah please do. Will close for now, since there was a service issue earlier this week that may have caused this. I also take it you're in China where the service is hosted in the U.S. which may explain the high pings you have.


Streamline console data transfer process

This also fixes a bug where console escape codes were causing premature disconnections on the client side.

LGTM. All tests passed on all database container types


Update smithy4s-aws-http4s, ... to 0.18.13

About this PR
📦 Updates

com.disneystreaming.smithy4s:smithy4s-aws-http4s
com.disneystreaming.smithy4s:smithy4s-sbt-codegen

from 0.17.19 to 0.18.13
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.disneystreaming.smithy4s" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.disneystreaming.smithy4s" }
}]

labels: library-update, early-semver-major, semver-spec-minor, version-scheme:early-semver, commit-count:1

Superseded by #123.


NRF52840 hangs in z_add_timeout function

Describe the bug
Nrf52840 (on Thingy91) randomly hangs in z_add_timeout() function. Most probably it happens only when Bluetooth is enabled and scanning is running on nrf52840.
What have you tried to diagnose or workaround this issue?
I noticed that NRF52840 is facing occasional, very random watchdog resets. In my application watchdog is feed from main loop (reset happen after 1 minute). I started investigation what is reason for these resets, I disabled watchdog and connected debugger. I have following observations:

with watchdog disabled hangs still happens, it last around 9 minutes and after this everything went to normal state.
During that 9 minutes I don’t see any logs, which means that interrupts are not working
Based on timestamps in logs I can say, that hangs always last the same amount of time – in my case break in logs is always 9 minutes and 2 seconds
Hangs happen very randomly – sometimes it is not seen for several hours, sometimes it happens in 10 minutes after reset.
When program execution is stopped with debugger it is always pointing to z_add_timeout() where infinite for loop is executed:

0x0003d9da in z_add_timeout (to=0x200039f0 <z_idle_threads+24>, fn=fn@entry=0x48011 <z_thread_timeout>, timeout=...) at /home/kazik/nordic/sdk/ncs/zephyr/include/sys/dlist.h:341
341		return (node != NULL) ? sys_dlist_peek_next_no_check(list, node) : NULL;
(gdb) next
108				if (t->dticks > to->dticks) {
(gdb) next
113				to->dticks -= t->dticks;
(gdb) next
107			for (t = first(); t != NULL; t = next(t)) {
(gdb) next
108				if (t->dticks > to->dticks) {
(gdb) next
113				to->dticks -= t->dticks;
(gdb) next
107			for (t = first(); t != NULL; t = next(t)) {
(gdb) next
108				if (t->dticks > to->dticks) {
(gdb) next
113				to->dticks -= t->dticks;

(gdb) print *to
$2 = {node = {{head = 0x200039f0 <z_idle_threads+24>, next = 0x200039f0 <z_idle_threads+24>}, {tail = 0x200039f0 <z_idle_threads+24>, prev = 0x200039f0 <z_idle_threads+24>}},
  fn = 0x48011 <z_thread_timeout>, dticks = 0}
(gdb) print *t
$3 = {node = {{head = 0x200039f0 <z_idle_threads+24>, next = 0x200039f0 <z_idle_threads+24>}, {tail = 0x200039f0 <z_idle_threads+24>, prev = 0x200039f0 <z_idle_threads+24>}},
  fn = 0x48011 <z_thread_timeout>, dticks = 0}

It looks that node is single element in timeout_list and have dticks = 0 and also the same element is being passed to z_add_timeout() function and for loop can’t be break. I don't understand how this situation happened and how it normally ends. I suspect that different timeout needs to be added to break this loop. When program execution is stopped with debugger break will never happen (even I let program to continue execution) – I also don’t know why this happen (probably Bluetooth module stop working correctly)
To Reproduce
Enable Bluetooth, start scanning and wait
Expected behavior
Hangs in z_add_timeout does not happened
Impact
showstopper
Logs and console output
Logs from gdb above
Environment (please complete the following information):

OS: Linux
Toolchain nrfConnect SDK (NCS version 1.4.2)
Commit SHA  (e34b2f477e1adc26b66690171508bfa102eb8538, tag: v2.4.0-ncs2-rc1, tag: v2.4.0-ncs2)

Additional context
With new NCS (version 1.6.1) it looks that problem  is gone, however switching to other version is not preferred option.

You are probably using Softdevice Link Layer. Can you check if issue happens also when zephyr LL is used (CONFIG_BT_LL_SW_SPLIT=y)

Yes, I'musing Softdevice Link Layer, I have switched to zephyr LL few hours ago and I'm testing it right now, so far no issue seen. I will update modified firmware on multiple devices so I should have results in 2 days (or earlier if problem appear).
Is it known issue with Softdevice LL ?

FYI @rugeGerritsen @seko-nordic

there might be two things caused by softdevice LL using ZLI (non-maskable interrupts)

from ZLI interrupt context kernel api must not be called but is called (e.g. inside ISR_DIRECT_PM`)
timer driver is somehow failing because it is not ready to be interrupted by ZLI while being in lock.

Regarding first option, those are places where this happens:
https://github.com/nrfconnect/sdk-nrf/blob/41202f304f0e67423448afc5368fd4d640cb73e6/subsys/mpsl/mpsl_init.c#L82
https://github.com/nrfconnect/sdk-nrf/blob/41202f304f0e67423448afc5368fd4d640cb73e6/subsys/mpsl/mpsl_init.c#L95
https://github.com/nrfconnect/sdk-nrf/blob/41202f304f0e67423448afc5368fd4d640cb73e6/subsys/mpsl/mpsl_init.c#L107
Can you try to remove calls to ISR_DIRECT_PM calls from those irq handlers and switch back to Softdevice LL?

Sure, I will check this today and leave one device for test during weekend.
Regarding Zephyr Link Layer (CONFIG_BT_LL_SW_SPLIT=y) - it looks that this completely resolved problem with hanging (and watchdog resets) on my devices. It is running on couple of devices for 2 days without single reset. I noticed only one problem when I switched to Zephyr LL - bt_scan_blocklist_clear function doesn’t clear blocked devices until bt_scan_stop/bt_scan_start is called – with Softdevice LL this worked, but I didn’t investigate this problem, just add stop/start scanning.

I can say now that removing calls to ISR_DIRECT_PM did't resolve problem with resets.

@nordic-krch how should we proceed here you think?

@gpuzio could you try enabling ZLI in the open source controller to see if that reproduces the issue as well? You can enable it with this config option, and you need to enable this option and this one

This issue seems to have appeared on Laird's MG100 when attempting to run the http_get example.
Looking for guidance on how to resolve this.
Note: As a shot in the dark, I've tried setting CONFIG_BT_LL_SW_SPLIT=y in prj.conf - this causes a bus fault:

[00:00:04.514,434] <err> os: ***** BUS FAULT *****
[00:00:04.514,434] <err> os:   Instruction bus error
[00:00:04.514,434] <err> os: r0/a1:  0x00000000  r1/a2:  0x2002665c  r2/a3:  0x00000004
[00:00:04.514,465] <err> os: r3/a4:  0x94f7c0b5 r12/ip:  0x00000000 r14/lr:  0x0003146d
[00:00:04.514,465] <err> os:  xpsr:  0x21000000
[00:00:04.514,465] <err> os: Faulting instruction address (r15/pc): 0x94f7c0b4
[00:00:04.514,465] <err> os: >>> ZEPHYR FATAL ERROR 0: CPU exception on CPU 0
[00:00:04.514,495] <err> os: Current thread: 0x2000f3e4 (main)
[00:00:04.665,557] <err> fatal_error: Resetting system

Please let me know what more diagnostic information I can provide.
OS: Win10
Board: nrf52840 on Laird MG100 (Note: I'm using the actual device, not the MG100 DVK)
Modem: hl8700
Zephyr Version: 2.3.0 (note, this is the version included with Laird's development firmware - I have not yet upgraded this, and am unsure if I should).
JLink: nrf52840 DK
Sample Package: Laird MG100 LTE-M/AWS, with code from Zephyr http_get sample project appended and altered as appropriate.
Expected Result:

A connection between the MG100 and remote host over HTTP(S) should be established,
A GET request should be sent by the MG100 to the remote host using demonstration code.
A properly formed response should be received by the MG100 from the remote host with the proper payload.

Actual Result:

The connection and send requests execute (using connect() and send() methods demonstrated in http_get/src/main.c : lines 109-110.
The methods called by these factory calls do complete, but execution is never returned to main.

NOTE: The connection is never actually established - I see no record of any connection between the MG100 and my server).

When debugging via GDB, it is observed that the execution path enters z_add_timeout() and never leaves.

Specifically, execution enters the for (t = first(); t != NULL; t = next(t)) block of code between lines 111-120 in timeout.c and never leaves.

The MG100 continues to execute other threads. As mentioned in point 2, no connection is ever established and no send is ever performed.

Other sources consulted:
I have looked at the issue regarding connect not sending a new SYN request in the case where a first connection attempt fails. The workaround did not help.

@scath999 You comment may have been overlooked... is this still an issue, if so, could you create a new bug report?


make rd suit large

About the PR
so you can steal it easily again
Why / Balance
regression with slots thing
Technical details
no
Media

[ ] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes
no
Changelog
no cl no fun

wait its normal already :trollface:


feat(tutorial): complete step 4 - Rochester

Closes #
{{short description}}
Changelog
New

{{new thing}}

Changed

{{change thing}}

Removed

{{removed thing}}

Looks good. Marking step 4 complete


Default filesystem is ext4 for some reason

This is likely related to #33. The default filesystem on Fedora is ext4, while on CentOS it is XFS. We need to get back to the XFS filesystem, which allows us to use quotas for things like pcap retention.

Added explicit declaration of xfs. Doesn't address root cause though.
https://github.com/rocknsm/rock-createiso/commit/1ff74b2af05fe855818503534784120d12420557

That worked. #33 still needs solved, but this bandaid works for now.


add affiliations

Thanks @jstac @Smit-create and the team.
I'd love to be added and please find my name and affiliation here:
Shu Hu
The Australian National University

Hey all! This is great - You can put my affiliation as New York University.

From Quentin Batista:
I would be happy to be a co-author. My current affiliation is MIT.

I think this is good to merge now. We have everyone's affiliation. Thank you all!

No problem @albop , I'll take care of it.


docs: fix send rate usage in api.md

fix issue #2014

Could you update it and rename the commit title to docs: fix send rate usage in api.md?

@trainchou This patch looks good to me. But can you please squash the 3 commits to one, so I can merge it? Thanks!

@xiang90 Yes, of course! Thanks for your suggestion!

lgtm


Modify message content

I have scheduled the app to pair up people every two weeks. However, the message says it's sent every week.
Can you help me to know if I can modify it and how I can modify it ?

Hi @issacfranklin1529 ,
Thanks for raising the query. Modification can be done but it needs to be done at the code level. And if done it will be considered as customization.

@gsv022
can you help me what I need to do / modify and how I can achieve the requirement

Hi @issacfranklin1529 ,
Please follow these steps mentioned below.

Fork the Icebreaker repository. Create the new branch on top of that.
Update the  App Service -resource file.
Push the changes to newly created branch.
Go to App service of existing resource group. Disconnect the present repository and connect to the forked repository and branch and check it out.

Thank you
Please check it out


Latest Insiders build cannot read property map of undefined

Upon updating to this morning's release of the Insider's build, most, if not all views are completely inaccessible, resulting in a snackbar notification that reads Cannot read property 'map' of undefined.
This occurs when attempting to open the integrated Terminal, viewing the Extensions, attempting to open the workspace settings, clicking the gear icon in the bottom right, and opening the command palette. Possibly a ton of other places.
VSCode is otherwise non-functional

VSCode Version: 1.36.0-insider
OS Version: MacOS Mojave (10.14.5)

Steps to Reproduce:

Update to latest Insiders release
Navigate anywhere

Does this issue occur when all extensions are disabled?: Yes

Hotkeys themselves also otherwise do not work. Upon running into this issue, I reopened Code using code-insiders --disable-extensions, and have not reloaded the window since

Duplicate of #75343

same problem - on a fresh ubuntu 19.04 machine

Anyone have a .deb of the previous version. I can't find a release page with changelog for code-insiders. Snap doesn't provide previous versions either: https://snapcraft.io/code-insiders

Temporary fix:
$ sudo apt install code-insiders=1.36.0-1559798873

@OmriSama I'm on MacOS. Don't have access to apt

shaneholloman@ops:~$ sudo apt install code-insiders=1.36.0-1559798873
Reading package lists... Done
Building dependency tree
Reading state information... Done

No apt package "code-insiders", but there is a snap with that name.
 Done
Building dependency tree
Reading state information... Done

No apt package "code-insiders", but there is a snap with that name.
Try "snap install code-insiders"

E: Unable to locate package code-insiders
shaneholloman@ops:~$

Make sure you add https://packages.microsoft.com/repos/vscode stable InRelease to your apt sources.
https://code.visualstudio.com/docs/setup/linux#_debian-and-ubuntu-based-distributions

@OmriSama I just updated my comment
--classic is failing version --edge --betaand--classic` are all the same failing version

@shaneholloman ah, you're right. Hopefully this can be resolved in the next couple hours

Closing as duplicate of #75343