[NCLSUP-569] Individually log uploads/downloads

This commit individually logs each uploads and downloads rather than
printing one big string that contains both. This is done because we send
this log to Kafka, which has a max message size of 1 MB. Unfortunately
it is often the case that the one big string exceeds the 1 MB limit,
causing that message to be lost.
This commit instead sends many small log messages to be under the 1 MB
Kafka limit.

@matejonnet @pkocandr has also suggested that this log might belong to debug instead of info. Do you agree / disagree?

Maybe I would put the number of downloads/uploads being returned on info level and the list of artifacts on debug or trace level.
Regarding the context at uploads/downloads it is not really critical issue, but would be nice to have something like that to allow better usage of the logs.

@matejonnet Would you agree that the list of artifacts should be a debug or trace level?

Agree to lower the log level.

Updated the PR based on recommendations. Please have a last look to see if everything is ok. Thanks! :)


Diagramm still excists while the selected list is cleared

the default diagram should be cleared while the selected list gets cleared

fixed. While the list of selected diagrams is cleared the data of the default diagram gets cleared too

fixed. While the list of selected diagrams is cleared the data of the default diagram gets cleared too


update: focalboard instructions for prebuild for standalone desktop app dev setup

Summary
Updated the focalboard documentation to inform the developer to run the make prebuild step before the make linux-app or make win-wpf-app or make mac-app step.
Ticket Link
Related issue #1079
Signed-off by: imasdekar<EMAIL_ADDRESS>

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!

Looks good. @justinegeffen you can check this and approve/merge if its fine.

ping @justinegeffen.


feat(database): backup the db on unit stop

When performing an upgrade, any db records added since the last backup are lost. This becomes obvious when testing upgrades, but should occur anytime the database is stopped and started.
The thought here is that a backup be done proactively on stop to avoid it. Maybe theres a better place for it, but this does "work".

perhaps this is something that could be added to on_exit()? That way we can cover this with tests!

@bacongobbler makes sense, ill change this around.

This is related to #2819. I don't understand why this is happening, though, since we are continuously shipping WAL logs...

@iancoffey Let me know if I can help with this. This is definitely a good fix.

So the term signal doesnt get actually trapped on container stop - not sure why. Anyway, putting the backup stuff in the existing on_exit() wont do the job since it doesnt get called when database is stopped.

@iancoffey Are we sure the ExecStopPosts get called in order? As in, systemd waits for the first to complete before moving onto the second?
If so, code LGTM.

Kicked off a new CI build: https://ci.deis.io/job/test-ec2/2088/

@carmstrong yes, from what I am able to grok from the man page, they are executed sequentially.

Code LGTM.

code LGTM


Process parallel tf scalar summary hangs

When I use parallel='process', the tf.summary.scalar call in the summarize_episode function doesn't return. If I use 'thread' or 'none' the tf.summary.scalar function finishes normally. I have no idea why. Has anyone encountered this issue already and found a fix?

Hi, thanks for your message. The parallel data collection isn't supported. Feel free to debug it if you're interested, but it's not really needed as Dreamer is data-efficient and the computational bottleneck is training the model

Unfortunately, my environment (RLBench with vision sensor) is about 10x slower than mujoco envs. And Dreamer would spend the majority of the time on data collection.
I wasn't really able to solve the problem because there seems to be a CUDA issue when using multiprocessing in python. You could use 'spawn' as multiprocessing start method. However, this doesn't work for us because we have to pass objects which are not picklable.
In the end, I have only set the training envs to use multiprocessing and deactivatet tensorboard logging from these envs. This increases the speed of data generation and I can still log from the test env.

Why do you need to pass objects that cannot be pickled? In case you were talking about the environment instance itself, it would be better to instantiate it directly in the separate process. There is also an implementation of a async wrapper in gym3 that might be helpful. Sorry for not being able to help more with this.


Add the CLI Options page to the Sphinx documentation

Description
The CLI Options page has been created as a place holder (see here).
This page is for documenting the CLI options we have for Macaron. It contains the instructions on how to use the Macaron CLI.

Close because it is a duplicated of https://github.com/oracle-samples/macaron/issues/239


Text reflow not working on some websites

Text reflow doesn’t work on the messages of the forum https://onlinephilosophyclub.com. It works fine in Yandex browser.

I cannot see any issue, can you post screenshots of what you get on Kiwi & Yandex browsers ?

I have enabled on both browsers the text wrap option. The follofing is what I am able get on Kiwi browser, on my Android smartphone. If I swipe horizontally the screen, it just doesn’t move, so, I can’t even try to see the trimmed text by swiping the screen.

The following is what I am able get on Yandex browser. If I swipe the screen horizontally, the page moves, so that I can center it perfectly on the screen.

Alright, I can reproduce, although horizontal scrolling is not fully blocked for me : it just glitches a bit but works if you're persistent.
I don't really know this feature, what is it for ? I see not much difference whether it's enabled or disabled, whether I'm zooming or not.

It makes a huge difference in a smartphone, because the screen is small and narrow and many websites have small fonts. So, if I zoom in without text wrap, I need to continuously swipe horizontally to read the the text that, because of the zooming, goes outside the screen on both sides. With text wrap, the lines are made shorter according to the screen and the words that would be at tbe end of the line outside the screen are instead moved to the next line, so I can see all the text in the screen and I need only to scroll vertically to follow the text.

What you're describing doesn't seem to happen on your second screenshot featuring Yandex Browser. Am I wrong ?

The picture shows that text wrapping works fine on Yandex: you can read at the center of the second screenshot “We all have free will, the freedom...”: Yandex moved the word “freedom” to the next line, because the action of zooming left no room in the first line. This way no words are missing. In the first screenshot, instead, which is from Kiwi, you can see that the words “the freedom to think on...” are outside the screen, I need to swipe horizontally to read them. On Yandex I don’t need to swipe horizontally: all the words of the phrase are there and I need just to scroll down to follow comfortably the text.

I see.
It seems to work on some lines but not all, like the second one from the top of your screenshot.
Also I would have thought it would preserve margins and borders.

Yes, even Yandex doesn’t manage properly the quoted text boxes, those ones with a grey/brown background color, I guess just because they are text boxes added in the middle of the flow of the main text. But, apart from this difficulty when additional text boxes are in the conversation, at least in Yandex I can read the message just scrolling it up and down, without needing to swipe horizontally.


Modifying neural_style.lua's verbose output format into a more CSV friendly format?

I am attempting to modify this function into a more CSV friendly format for graphing:
local function maybe_print(t, loss)
    local verbose = (params.print_iter > 0 and t % params.print_iter == 0)
    if verbose then
      print(string.format('Iteration %d / %d', t, params.num_iterations))
      for i, loss_module in ipairs(content_losses) do
        print(string.format('  Content %d loss: %f', i, loss_module.loss))
      end
      for i, loss_module in ipairs(style_losses) do
        print(string.format('  Style %d loss: %f', i, loss_module.loss))
      end
      print(string.format('  Total loss: %f', loss))
    end
  end

The function by default creates this output:
Iteration 50 / 100
  Content 1 loss: 5779007.421875
  Style 1 loss: 16.728337
  Style 2 loss: 3375.864029
  Style 3 loss: 35580.190659
  Style 4 loss: 7511.037111
  Style 5 loss: 231.612459
  Total loss: 5825722.854470
Iteration 100 / 100
  Content 1 loss: 5779007.421875
  Style 1 loss: 16.728337
  Style 2 loss: 3375.864029
  Style 3 loss: 35580.190659
  Style 4 loss: 7511.037111
  Style 5 loss: 231.612459
  Total loss: 5825722.854470
Iteration 150 / 100
  Content 1 loss: 5779007.421875
  Style 1 loss: 16.728337
  Style 2 loss: 3375.864029
  Style 3 loss: 35580.190659
  Style 4 loss: 7511.037111
  Style 5 loss: 231.612459
  Total loss: 5825722.854470

The original output format is good for following the process visually, but it does not well for any CSV related tasks like graphing.
I am trying to make the terminal output look like this (Markdown table format does not need to be used in the actual code, I'm just using it here to show the look of the format I am aiming for):

Iteration
Content 1 loss
Style 1 loss
Style 2 loss
Style 3 loss
Style 4 loss
Style 5 loss
Total loss

50
5779007.421875
16.728337
3375.864029
35580.190659
7511.037111
231.612459
5825722.854470

100
5779007.421875
16.728337
3375.864029
35580.190659
7511.037111
231.612459
5825722.854470

150
5779007.421875
16.728337
3375.864029
35580.190659
7511.037111
231.612459
5825722.854470

200
5779007.421875
16.728337
3375.864029
35580.190659
7511.037111
231.612459
5825722.854470

I have tried using various combinations of local variables, but I always get this error: Attempt to concatenate global 'variable' (a nil value). I don't really have much experience with Lua at the moment, but I understand that concatenating each of the variables into a single string and then printing the string, is the simplest method of changing the terminal output format?
In order to capture the terminal output to a text file, I add this code: 2>&1 | tee ~/neural-style/mylog.log to the very end of the Neural-Style command parameters I am using.

Looks like a scope problem... you are accessing a local variable which does not exist in the scope where you access it, then Lua is looking for a global with the same name, which does not exist either. The error then comes because you are concatenating into nothing.
For collecting data from multiple iterations you need a variable declared in such a place that it will "live" throughout the iterations.

My code was looking like this:
      ```
      local a = string.format('Iteration %d / %d', t, params.num_iterations)
  local b = string.format('  Content %d loss: %f', i, loss_module.loss)
  local c = string.format('  Style %d loss: %f', i, loss_module.loss)
  local d = string.format('  Total loss: %f', loss)

      local e = a .. b  -- "concatenate"
      local f = e .. c
      local g = f .. d

Then I was going to print "g" to the terminal.

I was following the information I found here: https://stackoverflow.com/questions/21782240/how-to-concatenate-strings-into-one-using-loop

I tried putting the code I wrote above, into the function like this with the intent to remove the old "print(string.format" lines after I got my code working:

local function maybe_print(t, loss)
local verbose = (params.print_iter > 0 and t % params.print_iter == 0)
if verbose then
print(string.format('Iteration %d / %d', t, params.num_iterations))
local a = string.format('Iteration %d / %d', t, params.num_iterations)
for i, loss_module in ipairs(content_losses) do
print(string.format('  Content %d loss: %f', i, loss_module.loss))
local b = string.format('  Content %d loss: %f', i, loss_module.loss)
end
for i, loss_module in ipairs(style_losses) do
print(string.format('  Style %d loss: %f', i, loss_module.loss))
local c = string.format('  Style %d loss: %f', i, loss_module.loss)
end
print(string.format('  Total loss: %f', loss))
local d = string.format('  Total loss: %f', loss)
local e = a .. b  -- "concatenate"
local f = e .. c
local g = f .. d
-- Somehow print "g" to the terminal.
end
end

>For collecting data from multiple iterations you need a variable declared in such a place that it will "live" throughout the iterations.

I was/am planning to just output the data to terminal so that I can easily capture it using tee. Then a conversion to CSV would be easy to do, like what I did with Caffe's training outputs before I realized there was a built in logging script.

It is not a problem with concatenation but scope. You are defining local variables inside a if-then block, which means that outside this if-then block, the variable ceases to exist. Same thing with for-do block
To solve this, you have to declare the variables at the beginning of the function.  That way they will survive throughout the function (and cease to exist when the when the function exits).
local function maybe_print(t, loss)
    local a = ""
    local b = ""
    local c = ""
    local verbose = (params.print_iter > 0 and t % params.print_iter == 0)
    if verbose then
      print(string.format('Iteration %d / %d', t, params.num_iterations))
      a = string.format('Iteration %d / %d', t, params.num_iterations)
      for i, loss_module in ipairs(content_losses) do
        print(string.format('  Content %d loss: %f', i, loss_module.loss))
        b = string.format('  Content %d loss: %f', i, loss_module.loss)
      end
      for i, loss_module in ipairs(style_losses) do
        print(string.format('  Style %d loss: %f', i, loss_module.loss))
        c = string.format('  Style %d loss: %f', i, loss_module.loss)
      end
      print(string.format('  Total loss: %f', loss))
      local d = string.format('  Total loss: %f', loss)
       local e = a .. b  -- "concatenate"
       local f = e .. c
       local g = f .. d
       -- Somehow print "g" to the terminal.
    end
  end

Then, instead of using so many variables, you could do
      a = string.format('Iteration %d / %d', t, params.num_iterations)
      for i, loss_module in ipairs(content_losses) do
        print(string.format('  Content %d loss: %f', i, loss_module.loss))
        a = a .. string.format('  Content %d loss: %f', i, loss_module.loss)

Actually, single variable is usually enough, as one can also concatenate line feeds etc into it.

So this code appears to work without error:
local function maybe_print(t, loss)
    local a = ""
    local b = ""
    local c = ""
    local verbose = (params.print_iter > 0 and t % params.print_iter == 0)
    if verbose then
        a = string.format('Iteration %d / %d', t, params.num_iterations)
      for i, loss_module in ipairs(content_losses) do
        a = a .. string.format('  Content %d loss: %f', i, loss_module.loss)
      end
      for i, loss_module in ipairs(style_losses) do
        a = a .. string.format('  Style %d loss: %f', i, loss_module.loss)
      end
	a = a .. string.format('  Total loss: %f', loss)
	print(a)
       -- Somehow print "g" to the terminal.
      end
  end

Which results in:
Iteration 50 / 1000  Content 1 loss: 7515056.250000  Style 1 loss: 2958737.54882                                                                                8  Style 2 loss: 59383289.062500  Style 3 loss: 36834720.703125  Style 4 loss: 7                                                                                79240.936279  Style 5 loss: 19465.633392  Total loss: 107490510.134125
Now I remove the text:
local function maybe_print(t, loss)
    local a = ""
    local b = ""
    local c = ""
    local verbose = (params.print_iter > 0 and t % params.print_iter == 0)
    if verbose then
        a = string.format('%d ', t, params.num_iterations)
      for i, loss_module in ipairs(content_losses) do
        a = a .. string.format(' %f', i, loss_module.loss)
      end
      for i, loss_module in ipairs(style_losses) do
        a = a .. string.format(' %f', i, loss_module.loss)
      end
	a = a .. string.format(' %f', loss)
	print(a)
      end
  end

And that results in:
50  1.000000 1.000000 2.000000 3.000000 4.000000 5.000000 107547864.963627
100  1.000000 1.000000 2.000000 3.000000 4.000000 5.000000 19892982.002258
150  1.000000 1.000000 2.000000 3.000000 4.000000 5.000000 14015005.483246
200  1.000000 1.000000 2.000000 3.000000 4.000000 5.000000 11786042.230940
250  1.000000 1.000000 2.000000 3.000000 4.000000 5.000000 10424066.450262
300  1.000000 1.000000 2.000000 3.000000 4.000000 5.000000 9689034.508419
350  1.000000 1.000000 2.000000 3.000000 4.000000 5.000000 9298543.469238
400  1.000000 1.000000 2.000000 3.000000 4.000000 5.000000 9169725.155449

The style/content layer value is overriding the associated loss values?

It seems that I need to keep the %d along with the %f for the style and content loss values. Which is fine as I can either use or omit those rows in a CSV file:
  local function maybe_print(t, loss)
    local a = ""
    local b = ""
    local c = ""
    local verbose = (params.print_iter > 0 and t % params.print_iter == 0)
    if verbose then
        a = string.format('%d ', t, params.num_iterations)
      for i, loss_module in ipairs(content_losses) do
        a = a .. string.format(' %d %f', i, loss_module.loss)
      end
      for i, loss_module in ipairs(style_losses) do
        a = a .. string.format(' %d %f', i, loss_module.loss)
      end
	a = a .. string.format(' %f', loss)
	print(a)
      end
   end

The above code results in this output:
50  1 7522738.281250 1 2960488.037109 2 59444056.640625 3 36916854.492188 4<PHONE_NUMBER>93 5 19509.374142 107645894.902706
100  1 8126899.218750 1 1157280.029297 2 7763930.419922 3 2619203.796387 4<PHONE_NUMBER>06 5 11579.954624 19932185.182285
150  1 8272956.250000 1<PHONE_NUMBER>19 2 3874417.236328 3<PHONE_NUMBER>70 4<PHONE_NUMBER>70 5 9400.653362 13921646.248150
200  1 8359832.031250 1<PHONE_NUMBER>56 2 2105716.552734 3<PHONE_NUMBER>48 4<PHONE_NUMBER>24 5 8412.817240 11587818.655252
250  1 8372617.187500 1<PHONE_NUMBER>18 2 1146672.546387 3<PHONE_NUMBER>12 4<PHONE_NUMBER>14 5 7760.557652 10274508.593082

The problem here
      for i, loss_module in ipairs(content_losses) do
        a = a .. string.format(' %f', i, loss_module.loss)
      end
      for i, loss_module in ipairs(style_losses) do
        a = a .. string.format(' %f', i, loss_module.loss)
      end

is that you specify format for one number only, so i gets printed instead of loss.  Note that string format gives first a template for formatting the output and then the variables to be filled into that template. In your code above, the template asks for only one float, but you give both i and loss, and loss is not printed.
Your solution to insert %d prints out both i and loss, but I wonder if you really need to print the numbers from 1 to 5 on every line.
If not, remove i as follows.
      for i, loss_module in ipairs(content_losses) do
        a = a .. string.format(' %f',  loss_module.loss)
      end
      for i, loss_module in ipairs(style_losses) do
        a = a .. string.format(' %f', loss_module.loss)
      end

Thanks for the help, the suggested edits resolve the issue of printing the unneeded numbers:
local function maybe_print(t, loss)
    local a = ""
    local b = ""
    local c = ""
    local verbose = (params.print_iter > 0 and t % params.print_iter == 0)
    if verbose then
        a = string.format('%d ', t, params.num_iterations)
      for i, loss_module in ipairs(content_losses) do
        a = a .. string.format(' %f',  loss_module.loss)
      end
      for i, loss_module in ipairs(style_losses) do
        a = a .. string.format(' %f', loss_module.loss)
      end
	a = a .. string.format(' %f', loss)
	print(a)
      end
   end

Results in:
50  7527201.562500 2959383.544922 59384783.203125 36886895.507812<PHONE_NUMBER>07 19818.699360 107561894.682026
100  8069984.375000 1155963.317871 7789196.777344 2680919.128418<PHONE_NUMBER>74 12078.706741 19980599.031448
150  8239482.031250<PHONE_NUMBER>32 3884134.643555<PHONE_NUMBER>88<PHONE_NUMBER>65 10329.926491 13931224.006081
200  8380553.906250<PHONE_NUMBER>88 2126519.531250<PHONE_NUMBER>84<PHONE_NUMBER>52 8890.881300 11633606.351423
250  8417350.000000<PHONE_NUMBER>28 1159272.583008<PHONE_NUMBER>45<PHONE_NUMBER>46 7739.640713 10359111.605740

Good. Yoy could now also remove the redundant lines
    local b = ""
    local c = ""

as you no longer use those variables.

Have you seen texture-net, it uses torch.display to show both intermediate images and a loss graph on a browser. That would be one option (real time!).

I was not aware that DmitryUlyanov's Texture-net? graphed the loss values in real time. I'll have to look into that. Though I have had trouble using the browser in browser supported implementations with AWS.

It shows a single (total?) loss together with images. I have added torch.display to fast-neural-style also, although images only, no graph so far. I like torch.display with the browser as it allows me to monitor training remotely from a laptop or ipad instead of sitting by the Linux box (requires running torch-display so that it accepts remote clients).

I recorded the loss values of an output I made recently that uses 2 style images:
Content Image: https://i.imgur.com/B7fyiS1.jpg
Style Images: https://i.imgur.com/xAZ2M7O.jpg, https://i.imgur.com/qXyz1cm.jpg
The styled image: https://i.imgur.com/xjviyXi.png
I used 7 multires steps as per this multires template.
The loss graphs https://i.imgur.com/sQGDz4S.png
I created this Github repository for my graphing project: https://github.com/ProGamerGov/Neural-Graph

Using this: https://github.com/ProGamerGov/Neural-Graph and some ImageMagick stuff, I created this comparison of 4 different output's using 7 multires steps: https://i.imgur.com/BBI6S2v.png
It seems that content loss always rises? And the graph gets more jagged in the later steps with each subsequent multires step.

@htoyryla Using this: https://github.com/ProGamerGov/Neural-Graph and some ImageMagick stuff, I created this comparison of 4 different output's using 7 multires steps and -print_iter 1: https://i.imgur.com/BBI6S2v.png
It seems that content loss always rises? And the graph gets more jagged in the later steps with each subsequent multires step. The lines follow similar paths on step 6 and 7. The style 2 and style 3 values cross over each other momentarily, except for on the run that was using 2 style images at once.
This was the multires pattern I was using: https://github.com/ProGamerGov/Neural-Tile/blob/master/multires.sh

So it looks like -normalize_gradients actually causes the content loss to increase. -optimizer adam and -optimizer lbfgs create distinct loss graph patterns.  Changing the -style_weight and -content_weight all affect the graph as you would expect.
Big 4x4 Graph Comparison Chart:
https://i.imgur.com/absiOwx.png

If Texture-net only shows the total loss value, than you miss out on the different loss amounts between the content and style image, as well as between different layers.

I accidentally closed the issue, my mistake!

"If Texture-net only shows the total loss value, than you miss out on the different loss amounts between the content and style image, as well as between different layers."
Texture-net is like fast-neural-style, it trains a feed-forward network using a VGG model to measure and minimize losses for style transfer. I guess one could, if one really wants, add more graphs to show the losses from the different layers inside the VGG model for both content and style. I am not at all sure if that would be helpful. Usually we are training with a set of content images, so the losses are either from different images or should we perhaps calculate an average. Frankly, I don't see a point.
For me, it was a fresh experience to work with texture-nets, thinking more about which images to use in the training and how to scale them, then running training for a while looking at how the images develop, interrupting if it does not look good.
It could be that just because neural-style outputs these losses, the losses have received so much attention. People look at them and interpret them, but do they really understand what the losses are and how they work?

I am not at all sure if that would be helpful. Usually we are training with a set of content images, so the losses are either from different images or should we perhaps calculate an average. Frankly, I don't see a point.

I think it's helpful for seeing how each parameters affects each loss value, and thus the resulting effect on the total loss value.

People look at them and interpret them, but do they really understand what the losses are and how they work?

Looking around, I haven't been able to find anyone really analyzing the losses and trying to figure out what they mean. I have just seen the really obvious stuff.

So looking back to my early Neural-Style days, I noticed at the time that there appeared to be a cycle of the output image moving between "noisy" and "clean" as the iterations went on. I now suspect that cycle could have been shown as spikes on a loss graph.
Looking into these graph spikes:

Direct link to graph: https://i.imgur.com/PPuunSp.png
Iteration 1420-1450: https://i.imgur.com/TidQDrZ.gifv
Iteration 1390-1420: https://i.imgur.com/bzf1D9g.gifv
The changes that the image undergoes during the graph spike is interesting. These graph spikes don't seem to correspond to anything I can think of, but I do know that the graphs of more aesthetically pleasing outputs seem to have more "smooth" loss graphs with far fewer and even no spikes at all.
Neural-Style uses your parameters to try and find the most efficient loss function. So I think that the spikes on the loss graph indicate inefficiency in the loss function.
This seems to be in-line with how some of the graphs are basically a single giant spike like this:

Direct link to graph: https://i.imgur.com/sVWAZEY.png

What you are looking at is the behaviour of the optimizer (l-bfgs or adam) trying to look for a minimum of total loss in a mindbogglingly multidimensional space. Each possible image corresponds to one point in that space. The space has a terrain, with hills and valleys corresponding to the total loss at each point. When researchers speak of loss function, they mean loss as the function in this multidimensional space.
Neural-style inputs an image into the model, and measures the losses from the selected layers, then the optimizer changes the image a bit in an attempt to decrease losses. Your curves show how the optimizer is performing.
That the spikes correspond to poor images is simply natural, high loss by definition means the image is far different from what we expect it to be. That a spike occurs means that the optimizer has missed and gone astray, but that it then recovers is a good sign.
If you would use -init random, the search would always start from a different point in the multidimensional space and the loss curves would look different, as the loss curve corresponds to the terrain crossed on the path chosen by the optimizer.
Using -init image is different as one always starts from the same point (corresponding to the content image).  Different from -init random, where both content and style losses start from a high value, we now have content and style as if competing. The style losses can only decrease if we go away from the content image, so that the content losses will necessarily rise. But anyway, the optimizer will still be looking for a minimum for total losses.
When you change the layer settings, then, the whole playground changes, meaning that the minima and maxima will be places differently in the multidimensional loss terrain.

_Me: I am not at all sure if that would be helpful. Usually we are training with a set of content images, so the losses are either from different images or should we perhaps calculate an average. Frankly, I don't see a point.
You: I think it's helpful for seeing how each parameters affects each loss value, and thus the resulting effect on the total loss value._
I was commenting specifically concerning texture_nets and other feed-forward solutions like fast-neural-style. If you believe that examining how losses behave during optimization helps you to better select parameters, then you should be able to transfer your knowhow to a feed-forward solution which is also using a VGG model to measure losses.
Which still leaves everything I wrote in my previous comment. In my view, losses are simply a tool to find an optimum, not an absolute quality measure. When we change parameters and images, the whole terrain changes together with the starting point and the optimum.


[master] Upgrade to latest dependencies

Produced via:
./hack/update-deps.sh --upgrade
./hack/update-codegen.sh

/cc slinkydeveloper
/assign slinkydeveloper

Please don't /hold, just close them  🙏


Support for project-specific ycm extra config

YCM supports project-specific extra configuration files, that can be loaded automatically from current folder (or one of the parent folders). At the moment these file are not loaded because with default settings YCM requires load confirmation. Please, provide an option to disable extra config loading confirmation, because the only way to load extra config now is to apply this patch to YCM:
@@ -13,7 +13,7 @@
   "max_num_identifier_candidates": 10,
   "extra_conf_globlist": [],
   "global_ycm_extra_conf": "",
-  "confirm_extra_conf": 1,
+  "confirm_extra_conf": 0,
   "complete_in_comments": 0,
   "complete_in_strings": 1,
   "max_diagnostics_to_display": 30,

Sorry, missed the one that already existed.

which file can change the ycm settings?


JSON API support?

Hi, I'm investigating the feasibility of implementing the JSON API specification with Flama.
It's probably easy to add a JSON API codec to Flama using marshmallow-jsonapi.
The other parts, like which status codes to return and what query parameters should look like, are possibly harder to achieve without messing with Flama internals.
Would you agree with this quick assessment or do you think it would be easier than I'm currently thinking?

Looking at this some more, it probably already matches quite closely and is more about extending flama to handle the "filter", " paginate" and "side-loading" features for the List endpoint according to the spec.

Looking at this some more, it probably already matches quite closely and is more about extending flama to handle the "filter", " paginate" and "side-loading" features for the List endpoint according to the spec.

I agree on that, but I see it more like an optional feature over Resources. The bad side about using marshmallow-jsonapi is that isn't compatible with apispec so the schema and docs generation must be disabled for using it.
I would say that a side library extending Flama's Resources could be a good starting point.

@perdy, I'm closing this issue. I'm afraid the current state of the project is no longer compatible with these changes. Indeed, this might've been resolved indirectly via other changes that have come up over the last few months for the release of versions >=1.0.0. Thus, we're closing the issue, and will re-consider whether this is still needed in the framework over a discussion prior to any work. This way we can ensure we are note duplicating efforts. And, for sure, thanks a lot to @Korijn for this and any future suggestions :)


src_files not being watched?

I have a variety of src_files declared, but when I modify or touch them, the tests don't rerun.  I have to go hit enter in the terminal window where testem is running.  shouldnt this work out of the box?
its for an open source project here https://github.com/brewster1134/tiler where you can see my testem file and related directory structure.

I've got the same problem.


fr: faut

/fo/
https://en.wiktionary.org/wiki/faut

This is already in the database. Did you check the most recent version?


Ignore CSSImportRule with no href

Fixes #83

Thank you for this PR!
Can you please add applicable test(s)?

@smhg sorry, I had a quick look at the existing tests, and I didn't find a similar test to mimick -- so that sadly makes it a too time-consuming task for me at the moment. I may revisit this in the future if no-one else steps up to the plate.

I understand. I'm trying to taking care of this project, but some parts (including tests) need more attention.
I've added a basic test for the Reloader class to master hoping you can pick up and implement another basic test for your change. Any test is a win!
I'm happy to release it immediately after.


Update developers.list

add atzlinux

Is there one Debian sid Distro?
在 2022/6/15 23:55, Wei Wu 写道:

@.**** approved this pull request.
Welcome!
The default env is an Unmatched board w/ Ubuntu Linux Distro. Feel
free to reply this comment if you need any different or specific
software support.

@yuzibo may provide help on this.


fixed noscript break

Deployed to: http://staging.quran.com:33018

Deployed to: http://staging.quran.com:33019


Add !pp command

This should be an easy add, I made it on my local branch once but it was a bit messy because im not that good at node.
Basically what it does is it prints an ascii string that starts with 8, then has a random amount of = symbols between 2 and 9 and then ends with a D.

i'll make sure to add that

done in branch v2; hopefully will be merged into master soon.


Add new duplicate grouping logic

In addition to the current duplicate grouping logic (updates to which are described in #530), add another way duplicates are grouped, based on DOI.
If a publication has the same DOI as another publication, put them in a duplicate group together. If either or both of the publications are already in duplicate groups, combine the duplicate groups.
Note that we may need to adjust the timing of when the current AI-Pure automerger runs to ensure that this additional grouping doesn't interfere with that automerge process (which will only merge if the duplicate group contains two pubs with one import each). That process probably needs to run after each AI import and before this new grouping process. When we roll out these changes, we should run the revised duplicate grouping process (from #530 ), then run the AI-Pure automerger, then run this new process.
Blocked by #529 and #534
Background discussion is in #523

The grouping process is done during the import of publications (for every publication individually) during the Pure, Activity Insight, and OAI (PSU Law) importers.  We can have the AI-Pure automerge do the same thing for just the Pure and AI imports.  The DOI grouping process can then follow in the same way.  We can also then follow up with the DOI automerge and any other automerge.
We should probably still run the automerge cron tasks weekly, just to clean up anything that may not have been merged during import.
Somewhat unrelated question and something to consider: what about the Web of Science publications?  I guess we can hope they get grouped with Pure or AI imports as those come in, but what if there are duplicates within Web of Science that don't match with AI or Pure imports?  There may not be many.  I'm not too familiar with that dataset.  We do have the ability to run a task the goes through every publication in the database and group them.  I think this takes a very long time to complete.  I've never run it to completion before.

Looking at publications from 2015 to present, I see publications with Web of Science imports from before 2019 only. I suspect it's unlikely (but not impossible) that new imports from Pure or AI will have matches for them. Of 2235 publications that have a Web of Science import, 1519 have only a Web of Science import (i.e., haven't been merged with anything else).
tl;dr -- Once we have finished all of the updates to the grouping and merging processes, I think we should run the task that goes through all publications and tries to group them. And ideally the new duplicate grouping logic in this issue would get included in that task as well.


Fix match between uppercase and lowercase of extensions

Everything looks good, but I just merged #46, so you'll need to resolve the conflicts. Try to avoid a merge commit if you can, it would probably be easiest to undo this commit, pull the changes from master, and just manually edit these 6 lines again.

Great. Let's finish it.


Hue hang up error

Still with version 2.0.4 on several bulbs i have this issue.
Cant be solved by 2nd API user, only grp node, but in my case this is only a temp work around.

maybe here the screenshot after changing it to a grp

This is really strange as HueMagic does a batch request anyway to get all lights and their current states in background.
So you should face these errors anyway if I don’t overthink something. Unfortunately the Hue Bridge only offers a REST API at the moment and not a websocket based one, which leads to this kind of performance issues – especially with many lights connected. This might change in near future (as far as I know) with a firmware upgrade for the bridge.
You could try to increase the poll interval to a higher number until then. Maybe this will help to avoid this errors... 😶?

tnx for your fast answer. well i will try increasing the poll interval and let you know.
What else is wired, the mobile app and win app make change super fast. but i experience issue with Alexa since 2-3 week to: with the statements:

hub not responsive, - even telling me having double names on devices/ rooms.
I believe HUE is not ready for semi-pro zigbee bridge ;)

Huejay is no longer used in the latest update.
It should be fixed now


Node sample 24.bot-authentication-msgraph updates

Fixes #1538
Proposed Changes

pin graph client version (1.4.0 works as expected)

remove call to getManager (dotnet version has this call removed already)

fix console.error logging

Testing
Manually tested against aadv1

1.7 is the latest, but does not work.  I do not know if it is our code, or the GraphClient ... but I do know 1.4 works as expected.  Someone better with javascript might be able to get 1.7 functioning.


Unable to download optimized image, JavaScript, and CSS resources

Original report: https://talk.plesk.com/threads/extension-google-pagespeed-insights-issue.345160/
using this extension in the domain page we got the link:
Download optimized image, JavaScript, and CSS resources for this page.
which points to: https://developers.google.com/speed/pagespeed/insights/optimizeContents?url=http://DOMAIN.TLD&strategy=desktop
but clicking it we got error from Google API:
The referrer https://OURPLESKSERVERNAME:8443/modules/pagespeed-insights/index.php/index/result?dom_id=5&site_id=5 does not match the referrer restrictions configured on your API key. Please use the API Console to update your key restrictions.
I swapped urls for privacy reasons.

This bug was fixed in the latest release (2.0.0): https://ext.plesk.com/packages/3d2639e6-64a9-43fe-a990-c873b6b3ec66-pagespeed-insights


Add code tags to Get JSON challenge plus linting

Pre-Submission Checklist

[x] Your pull request targets the staging branch of FreeCodeCamp.
[x] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[x] You have only one commit (if not, squash them into one commit).
[x] All new and existing tests pass the command npm run test-challenges. Use git commit --amend to amend any fixes.

Type of Change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Add new translation (feature adding new translations)

Checklist:

[x] Tested changes locally.
[ ] Closes currently open issue (Closes #XXXX): Closes

Description

Removes unnecessary and unbalanced </code> tags
Remove trailing whitespace
Change </br> to <br>
Add <code> tags to appropriate text

:sparkles: :sparkles: LGTM. :sparkles: :sparkles:
Thanks @erictleung .


Add https support

If you Google "Hex Clock" the top result returns an https address.

Browsing with an https results in a blank white screen and several console errors.

Hey @bryanbraun, thanks for reporting this! Fixed it! :)


Error no banana update

banana update
Internet  [OK]
Cloning into 'bananapkg'...
remote: Enumerating objects: 64, done.
remote: Counting objects: 100% (64/64), done.
remote: Compressing objects: 100% (64/64), done.
remote: Total 412 (delta 37), reused 0 (delta 0), pack-reused 348
Receiving objects: 100% (412/412), 232.41 KiB | 672.00 KiB/s, done.
Resolving deltas: 100% (234/234), done.
 Obrigado pelo report!


so run_snafu.py can work in CI

This makes the 2 CRs conform to the snafu README.md documentation for how to point to elastic search.

@bengland2 is it possible to instead update the templates to accept elasticsearch.server etc like the other benchmarks instead? That would also make it consistent with what the metadata collection expects as well.

@dry923 ok, but then we should update the snafu documentation to reflect that.   @acalhounRH @aakarshg any opinions?  Also, what happens to es_index?   Does it make sense to have something like this example:
elasticsearch:
  host: marquez
  port: 9200
  index: ripsaw-fs-drift-results
  test_user: homerSimpson
  clustername: springfield

instead of what's documented now:
es_host: marquez
es_port: 9200
es_index: ripsaw-fs-drift-results
test_user: homerSimpson
clustername: springfield

The advantage of the elasticsearch: format is that if someone doesn't want elasticsearch they then know what to leave out.

This PR depends on snafu PR 99, merge this when you merge that one or things will break.  I tested both of them together for smallfile and fs-drift, and it does run successfully with tests/test_{fs_drift,smallfile}.sh but not yet tested for fio_distributed.   Please tell me if this is what you want, while I try to test fio_distributed this way.


Test failure with pathspec >= 0.10.0

The test_ignore_hidden_files test is broken when used with pathspec >= 0.10.0.
I'm not sure if this was a regression in that package or a bugfix.
self = <test_dirhash.TestGetIncludedPaths object at 0x7fad218e2c10>

    def test_ignore_hidden_files(self):
        self.mkdirs('root/d1')
        self.mkdirs('root/.d2')

        self.mkfile('root/f1')
        self.mkfile('root/.f2')
        self.mkfile('root/d1/f1')
        self.mkfile('root/d1/.f2')
        self.mkfile('root/.d2/f1')

        # no ignore
        filepaths = included_paths(self.path_to('root'))
        assert filepaths == ['.d2/f1', '.f2', 'd1/.f2', 'd1/f1', 'f1']

        # with ignore
        filepaths = included_paths(
            self.path_to('root'),
            match=['*', '!.*']
        )
>       assert filepaths == ['.d2/f1', 'd1/f1', 'f1']
E       AssertionError: assert ['d1/f1', 'f1'] == ['.d2/f1', 'd1/f1', 'f1']
E         At index 0 diff: 'd1/f1' != '.d2/f1'
E         Right contains one more item: 'f1'
E         Use -v to get more diff

Thanks! Sorry for the huge delay -> https://github.com/andhus/dirhash-python/issues/9
Seems related to https://github.com/cpburnz/python-pathspec/issues/57 (?)
Now also exposed here: https://github.com/andhus/dirhash-python/pull/17 (WIP)

See also: https://github.com/cpburnz/python-pathspec/issues/62#issuecomment-1236501291

proposed fix #25

Fixed as per ⬆ https://github.com/andhus/dirhash-python/pull/25


boost: find_dependency -> find_package

Local jenkins test results in SUCCESS https://github.com/ruslo/hunter/pull/825#issuecomment-306501789


Migration to Sequelize and added Mod Check to index.js

Nice work, this looks like a substantial update!


Adding institut gili i gaya

The High School website
The proof @institutgiligaya.cat is an actual domain

Oh, sorry the website is all in catalan '^~^

@willinternet Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Column permission settings

I would like to be able to set column permissions as there are some fields that are sensitive data and not open to all to view, and this feature is very much needed.

help me.plz

@zck573693104 Now you can create a view which not contain the sensitive column. Then you can grant this view to people who want to read.

good idea!!!


Works fine but how if it use only listener not widget

everythings works fine but i want to use only listener without putting widget and how do you listen if usb is connected?

You can use SizedBox.shrink() as a child if you don't need to display anthing.
USB or no USB, doesn't matter.  If it triggers keypress event this library will handle it correctly.


Add precommit configurations & basic GitHub actions

Resolves #153

The PR cannot be merged at this point yet since it has a lot of expected formatting and linting changes. The details are available at this job.
I will update the install scripts for normal setup and dev setup soon.

Hey @franklinselva, please let me know when you want this to be merged, the PR looks okay to me so far! Thanks! 🙌

@mufeedvh I think waiting until #24 is merged is better since this PR will add more conflicts. But I would like you to look over the current changes before adding any formatting and linting-related commits.

@mufeedvh I think we can go ahead with merging this PR. Can you go through the changes? This commit is strictly formatting related changes. The linting part is being improved as a part of refactoring


WebApp Error

Selecting target as Extension throws the error as below @ obfuscator.io
ReferenceError: Validation failed. errors:
`target` errors:
    - target must be one of the following values: browser,browser-no-eval,node

Oh, thank you. Will fix it later today

Should be fixed now


Disable route planning indicator when car is parked

I’ve had a few people now when I show them the comma 3 when we first get into the car and it’s parked they ask why the green line is waggling all over the place. And I just tell them it’s trying to figure out where to drive. And then they inevitably make a joke along the lines of “well hopefully it doesn’t drive into the garage door”.
Figure it might be a good idea to just not have it trying to route a path while in park. May make the system seem more polished or confident in its abilities. Maybe even a message similar to when the car is in reverse gear. Just have it say “parked” greyed out or something.

I don't think we want to hide the model outputs in these scenarios, though we are working on communicating model confidence in the UI. Closing unless @HaraldSchafer has thoughts on this.


Fix typo for box 5b - Change AD FA RA to AD FS RA

Just fixing a typo

@nathanmcnulty thanks for the correction!


Fix incorrect argument name and update description in RNN documentation

The documentation for the base class for recurrent layers (class RNN exported as keras.layers.RNN) incorrectly mentioned the call argument sequences as inputs. This PR corrects the argument's name in the documentation and updates the argument's description to align more closely with similar arguments in similar layers.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 60.02%. Comparing base (4895458) to head (4415772).

:exclamation:  There is a different number of reports uploaded between BASE (4895458) and HEAD (4415772). Click for more details.
HEAD has 6 uploads less than BASE

Flag
BASE (4895458)
HEAD (4415772)

keras
4
1

keras-tensorflow
1
0

keras-torch
1
0

keras-jax
1
0

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #20525       +/-   ##
===========================================
- Coverage   82.11%   60.02%   -22.09%
===========================================
  Files         515      515
  Lines       47790    47790
  Branches     7486     7486
===========================================
- Hits        39242    28686    -10556
- Misses       6736    17349    +10613
+ Partials     1812     1755       -57

Flag
Coverage Δ

keras
60.02% <ø> (-21.94%)
:arrow_down:

keras-jax
?

keras-numpy
60.02% <ø> (ø)

keras-tensorflow
?

keras-torch
?

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests


Remove extra form-selects in a results file

Fixes 2 mistakes in the test_prepended_appended_select.html result file.
Remove extraneous form-select values in two class attributes in the
results file.
Causes the previous failing test_select_prepend to pass.

Thanks for kicking off CI! The Python 3.6 series received its last security update about six months ago and is no longer supported. Does it make sense to just run CI against Python 3.7+, especially since Python 3.6 and older are causing problems? It looks like 3.6 would still work, but would just have a couple of repeated class values.

Yes we should drop 3.6 and add 3.10.


Fix snapshot docs in README

There's always one tarball now that we switched to emptyDir for the
example
Link to the snapshot docs

Signed-off-by: Ken Simon<EMAIL_ADDRESS>

LGTM


Does this project support for Unity WebGL build?

Does this project support for Unity WebGL build?

Unfortunately, the Firebase Unity SDK (on which this asset is built) is only available for Android and iOS. While you can use Firebase's REST API to interact with the Database, this asset would have to be modified to use that instead of the .NET implementation, and you would separately have to solve the auth issue, which is that anyone can see the JSON (including auth info) being sent to and from the server in a web application.


Extra Example

Would you be able to put together a simpler example to follow?

I added an example script at examples/example.py. Is this helpful?


[feature request] Support infiniband

All nodes installed infiniband, support infiniband device in job container.
Not all nodes installed infiniband, support those who installed.

code complete.


STYLE: removed duplicated messages

Removed itkWarningMacro just before itkExceptionMacro with exactly the same messages.
Looks rather unusual and highly likely not useful.

Indeed, the itkWarningMacro calls don't seem very useful to me either.
For the record, these combined itkWarningMacro-itkExceptionMacro calls appear introduced by commit b5d2dbba8fff02a7febf84f76a47e435a3b108a8 "BUG: fixing bug 4560; add exceptions when indices are out of range...", Brad Davis, 2008-05-26.


PR & issue templates

That's duplicate of #484 and #485
Closing this one and we will figure out it in mentioned issues.


Tried to send variable from threadGroup to elastic

Hello !
I have few questions about the Backend Listener like a discussion it is why I open an issue for it.
Indeed, I need to call differents url and retrieve the environment linked with them In the backend listener to exploit it in Kibana.
I used a file which contains lines like : "env url apikey" and I make a loop on this file which call a request for each lines.
The ${url} in my threadGroup is good and the request works for each environment called but when I try to get it with custom parameter in the backendListener like : name:"environment", value:"${url}" and in Kibana I see the value as ${url} and not deployment for exemple.
Can you tell me if I make something wrong ?

Hey @cprevot04 ,
So this is related to #27. Basically, the way that JMeter and its backend listener are build makes it so that the backend listener doesn't have access to JMeter variables that are declared anywhere else but in the JMeter's "Test Plan" element.
I would really love to work around this, but I'm afraid I have no way of accessing these variables unless the way it works is changed directly into the JMeter's source code. 😢
So to answer your question; no you didn't do anything wrong - it's simply a limitation of the software :( !

Hello,
An option would be to submit a patch to jmeter implementing this .
I guess having variables declared in sample_variables available  in SampleResult would do what you need.
See for contributing :

http://jmeter.apache.org/building.html

Very true, this could very well solve the issue. I personally don't really have the time to implement it right now, but I'll try to work it in my schedule in the following weeks/months.
Thanks!

Hello,
An other solution for us will be to exploit the RequestHeaders.
Would it be possible that your plugin parse and forward the request&response headers as "real" key/values in ElasticSearch ?
Hence, those headers would be much easier to exploit in Kibana.
Thanks !

@cprevot04 Wow that's actually a great idea! I could sure as hell do that 😄 ! I'm on vacation right now, but I'll try to work on it ASAP!
Thanks for the feedback!

@cprevot04 Just to let you know that I'm working on the headers method as I'm writing this. Should be out sometime this week 😄 !

This was added with version 2.5.0! It is currently being pushed to the central repository and will be merged to JMeter-Plugins ASAP. For the meantime, you can manually uninstall your old version and install the new with the JAR available on GitHub! 😄
Cheers all,

Hello,
Maybe this would require some documentation.
AFAIU, plugin would use request headers starting with "es-"  ?
If my understanding is correct, this might break some applications. But maybe I misunderstand something here.
Regards

Hi @pmouawad !
I documented the whole thing here https://github.com/delirius325/jmeter-elasticsearch-backend-listener/wiki/Sending-JMeter-variables .
I don't think it would break any applications per say, you might just have some unwanted key/pair values in your ElasticSearch index. Otherwise we can always change the startsWith("es-") to something more unique like startsWith("es-backend-").

Thanks for your rapid answer.
1/ Reading documentation, I don't understand where Header Manager would be put, and my notes depends on where it is located.
Maybe you could add a screenshot with a little test plan showing where it would be located.
2/ For custom fields, maybe the screenshot should show the whole config element with a red rectangle around the required values.
As you know, Variables are related to Thread , Property are global, I am a bit confused for custom fields, do you allow customization per thread or do you send properties?

@pmouawad
I'll rework the documentation, but to answer your question:

The Header Manager can either be located globally or locally, attached to a specific request.

The custom fields include the variables loaded through the the jp@gc CSV config element, the Test Plan variables, and properties that would be specified as ${__P(myProp)} or ```${__groovy(props.get("myProp"))}.

Maybe it isn't clear in the documentation but Custom Fields are one of the two ways to include custom values to results sent to ElasticSearch. The second way being with the headers.
Thanks,

Thanks again.

In this case, it means es-XXX headers would be sent in every HTTP Request. Some network component or application might refuse such headers and return 401 or 400 status codes. Usually such headers should be prefixed with 'X-' although this seems to have been deprecated (https://stackoverflow.com/questions/3561381/custom-http-headers-naming-conventions).
IMO using Headers is too invasive.

So maybe indicate that those "variables" should not vary per Thread, so they are king of Global Variables / Properties. Because currently users might think they can send value A for thread 1, and B for thread 2.... AFAIU, it's not expected to work this way

Hello!
I thought the idea behind headers was just to insert into ElasticSearch ALL the headers which are part of the actual request and responses as keys and values (instead of a single string we can not use in ElasticSearch/Kibana). Not to add some more "es-XXX" headers to the actual requests.
At least that's how I understood the suggestion... :)

Hey there!
That could also be another functionality, I'd just add a boolean to do the work! I'll add this in a minor version tomorrow :smile
Cheers,

Hi @ncasaux ,
So I just pre-released version 2.5.1. It includes the following modifications:

Renamed the method to parseHeadersAsDocumentProps(boolean).
The boolean in the method indicates whether or not the user wants to parse all headers or simply the X-es-backendones
The headers used to pass variables as key/value in ES must now be written as X-es-backendinstead of es-

If some of you guys can try it and give me feedback/suggestions, I'd greatly appreciate it!
Thanks,
https://github.com/delirius325/jmeter-elasticsearch-backend-listener/releases/tag/2.5.1-SNAPSHOT

Hi @delirius325 ,
I tried the "simple" headers parsing and it works! :)
One remark though: I think that request headers and responses headers should be in 2 different arrays in the json structure.
I hope it makes sense! :)
Thanks!

This is still in the works, I pushed out a minor version yesterday but I realize now that it comes with its set of challenges and problems. I'll rethink the whole thing soon and push a patch.

Hello,
Having issues with this functionality.  I think i'm missing something simple but I'm not sure.
I'm using JMeter 5.1.1 with ES Plugin version 2.6.7.  ElasticSearch backend is version 5.6.
What I'm trying to do is within my JMeter Thread Group I add an HTTP Header Manager with the following Names and Values
X-es-backend-test-variable hola1
es-test-variable hola2
When I run JMeter I see these headers being added to all my requests.  However, I do not see them appearing in ElasticSearch.  The rest of my JMeter data is appearing fine in ElasticSearch.


Unable to save url after recent Wayback changes

I already updated the library to version to 1.0.1 , still I am unable to save urls . Here is the codes :
def save_wayback(main_url):
    try:
        wayback = savepagenow.capture_or_cache(main_url)
        wayback_url = wayback[0]
        archived = True
        error = None
    except Exception as e:
        wayback_url = ''
        archived = False
        error = str(e)
    return wayback_url, archived, error

error message:
{'Server': 'nginx/1.15.8', 'Date': 'Fri, 17 Jul 2020 12:24:19 GMT', 'Content-Type': 'text/html; charset=utf-8', 'Content-Length': '178', 'Connection': 'keep-alive', 'X-App-Server': 'wwwb-app103', 'X-ts': '520'}
and sample url : Link

Same.

Try again. Whatever issue appears to have been transient.

Currently this solution solves my problem.


Provide inner (miri) backtraces

When miri generates an error, it should be possible to get a backtrace of where the error originated.
I kind of thought we'd need error_chain for that, but @oli-obk suggested we could just use the backtrace crate.

fun fact: error_chain uses the backtrace crate

I've used error_chain + backtraces in a project and now I want in the compiler. cc @nikomatsakis


fix issue 12064 - std.typecons.wrap doesn't handle NVI

Adressed the review comment but it seems that now there's a random error due to win32 sockets, see auto-tester.


Implement automated change of names for the staff main channel

The same way our off topic channels change their name every day, the staff one<PHONE_NUMBER>61755392, whatever its name is now) could do the same.
This should have the same interface of commands as the off topic channels, giving the possibility to add names, delete them, search them, and everything else, so most of the code should be shared (beware #658). But the name pools for each category, off topic and staff, should not. This requires a bit of work with the site.
The staff channel should probably have a prefix for its name, so it can be easily identifiable (maybe stf-?)

Hello @manusaurio, have you made any progress on this?

I haven't, and I should have said something earlier, my apologies. I'm also afraid I won't be able to do so for now and the following months, so this could be assigned to anyone else.

In consensus, we have decided that we don't want automatic renaming of the lounge name.


Lost reactstrap, cannot get it back

I think this is more my ignorance with using both NPM and GIT.
I was using reactstrap just fine.  I initially installed it using npm install reactstrap then I installed the dependencies.  I had been using it to great success till I tried to use a component and realized I didn't have it.  I assumed it was because I was on an older version.
NPM wouldn't install the newer version so I deleted reactstrap from my node_modules directory.
Then I tried to reinstall reactstrap but now it just installs some txt files but none of the components.  tried npm remove and then reinstall and the same thing.
Even worse react now gives me this error:

Module not found: Can't resolve 'reactstrap' in '/home/jeff/nodejs/react-app/front-end/src'

which is an error on importing reactstrap.  Not sure why it's looking for it in there, should be looking in my node_modules (not that it's installed there properly anyway)
I'm not sure how to get reactstrap back!  Can someone please help?
I saw an issue where someone suggested git clone and then build.  But I have no idea how to do that!

This fixed it.  I think it was the @next that I wasn't using.
npm install --save reactstrap@next <EMAIL_ADDRESS><EMAIL_ADDRESS>


typo: Compatability -> Compatibility

@ManikaDhiman @lomayor @AndreaBarr - PR is ready for final review. This is just a typo issue. Thanks.


fetchGit requires git as runtime dependency

Issue description
nixos/nix#2419
nix already requires git to build, but it fails to evaluate anything fetchGit without git installed. Maybe git should be a runtime dependency?
Steps to reproduce
Install nix but not git, use fetchGit

This is intentional. git is not a hard dependency since it's only needed if you use fetchGit. Since Git is a sizeable dependency, we don't want to pull it in unconditionally. (Likewise for Mercurial).

If we define correctness as "will the program work in more cases", then the existing situation is worse than the new situation w.r.t. correctness. As such, this can be classified as an optimization.
Performance optimizations (especially if they do not come with data and are breaking correctness) should be options that organizations can adopt on a case by case basis.

I think we should then at least ship the NixOS installer with git installed.

We are not using fetchGit anywhere in nixpkgs. What builds in the installer should fail?

Users might use fetchGit in their configuration, for example in order to obtain packages from a nixpkgs fork.

This is intentional. git is not a hard dependency since it's only needed if you use fetchGit. Since Git is a sizeable dependency, we don't want to pull it in unconditionally. (Likewise for Mercurial).

This line of argument seems to throw the entire purpose and practicality of nix into question... Personally, I don't buy it, though. In my opinion, if nix doesn't want to support a pure dependency on git out of the box, it seems like fetchGit support should correspondingly be configured out by default.

This wasn intentional so closing.

I ran into this issue several times when executing my nix expressions in different environments. Strange feeling since the whole purpose of using nix was to get rid of dependency issues. Now nix brings one in by itself. If fetchGit cannot be assumed working, why do the docs recommend it for pinning nixpkgs? Wouldn't it be better to avoid using it as much as possible?

Sorry for the naive question, but what is the workaround? At the moment I'm installing git first and then I add the package that uses fetchGit.

Installing git first seems to be the workaround. You could also install it globally on your system, in the nixos config.

The other alternative would be fetchTarball, which works at least with github or flakes in future.

Tanks for the solutions. Unfortunately fetchTarball isn't an option for now. I'm fetching private repo from gitlab. When I install git, then switch and then add the derivation that uses fetchGit, it's working. I've hoped for only one switch and no global git.

Depending on what you are doing you can run this build inside a nix-shell that makes git available, it doesn't have to be globally installed first.

I'm building a nodejs server as a systemd service. This is what I import in my configuration.nix
{ config, pkgs, git, ... }:
let
  yarn2nix = pkgs.yarn2nix-moretea.override {
    nodejs = pkgs.nodejs-12_x;
    yarn = (pkgs.yarn.override { nodejs = pkgs.nodejs-12_x; });
  };
  my-nodejs-server = yarn2nix.mkYarnPackage {
    name = "my-nodejs-server";
    src = builtins.fetchGit {
      url = "https://gitlab.com/my-namespace/my-project.git";
      rev = "my-rev";
    };
  };
in
{
  systemd.services.my-nodejs-server = {
    serviceConfig = {
      ExecStart = "${pkgs.nodejs-12_x}/bin/node ${my-nodejs-server.outPath}/bin/my-bin";
      Restart = "always";
      RestartSec = "10s";
    };
    after = [ "network.target" ];
    wantedBy = [ "multi-user.target" ];
    environment.PORT = "4001";
  };

  environment.systemPackages = [ my-nodejs-server ];
}

@vidyu You could use fetchgit from nixpkgs instead of builtins.fetchGit. If you need ssh key you could override the derivation returned by fetchgit and add your other credentials: https://github.com/NixOS/nixpkgs/blob/master/pkgs/build-support/fetchgit/default.nix

Thank you @Mic92 ! I had some problems finding the right sha256, but it's working like expected and I don't need to make two switches (one for git and one for the nodejs package).

Well then... you have a bultins fetchGit that's not work?? great...


ngComponentRouter not calling $routerOnActivate in angular 1.5 when using a template URL

I know it is not released yet, but I am testing out the new router  with Angular 1.5 RC1. The router file I am using is pulled from Here: https://github.com/brandonroberts/angularjs-component-router/blob/master/lib/angular_1_router.js
The $routerONActivate hook is never called if my router directive uses a templateUrl. When I switch it to just be a template (using an ng-include pointing to the url) it gets called as expected.

@petebacondarwin could you take a look at this?

I will add it to my list of router related tasks for this week and next

We should cross post this to angular/angular since that is where the code is and also where other issues with ng1 component router are being managed

@petebacondarwin Yes, duplicate of angular/angular#4633


Fix AP code config , updating code:native to code:XLM

What
[TODO: Short statement about what is changing.]
Why
[TODO: Why this change is being made. Include any context required to understand the why.]
Known limitations
[TODO or N/A]
Checklist
PR Structure

[ ] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[ ] This PR does not mix refactoring changes with feature changes (split into two PRs otherwise).
[ ] This PR's title starts with the name of the package, area, or subject affected by the change.

Thoroughness

[ ] This PR adds tests for the new functionality or fixes.
[ ] This PR contains the link to the Jira ticket it addresses.

Configs and Secrets

[ ] No new CONFIG variables are required -OR- the new required ones were added to the helmchart's values.yaml file.
[ ] No new CONFIG variables are required -OR- the new required ones were added to the deployments (pr-preview, dev, demo, prd).
[ ] No new SECRETS variables are required -OR- the new required ones were mentioned in the helmchart's values.yaml file.
[ ] No new SECRETS variables are required -OR- the new required ones were added to the deployments (pr-preview secrets, dev secrets, demo secrets, prd secrets).

Release

[ ] This is not a breaking change.
[ ] This is ready for production.. If your PR is not ready for production, please consider opening additional complementary PRs using this one as the base. Only merge this into develop or main after it's ready for production!

Deployment

[ ] Does the deployment work after merging?

stellar-disbursement-platform-backend-preview is available here:SDP:      https://sdp-backend-pr45.previews.kube001.services.stellar-ops.com/healthAP:       https://sdp-ap-pr45.previews.kube001.services.stellar-ops.com/healthFrontend: https://sdp-backend-dashboard-pr45.previews.kube001.services.stellar-ops.com

Closing this in favor of #45


Clarify browser version support

Polyfills:

IE >=9 <12
Chrome >=5 <33

Underlying implementation also usable for:

IE <9
Chrome <5

Could add support for Chrome <5 using __defineGetter__


Add BDF+ writer

Adds backend support for writing to a BDF file. For science!

Still need to add:
[ ] 16 channel support (different frequency)
[ ] Aux/Accel data


React:  ?component name? ?epic template?

This epic contains functional and design links that define the scope of the deliverable.  This epic also includes links to all of the work required to deliver the component.
This work is a follow-on to the development of the Web Components version.  Design and functional spec documentation is included in the Web Component epic.
Additional Information

Corresponding Web Components epic  (#TBD)

Will be moving away from support of Pure React components.  Closing this epic and all linked issues.


Subdivisions

Hi,
what do you think about adding subdivision (ISO 3166-2) information ?
We need that for some project so im currently implementing it (europe + usa for now). Its basically an ISubdivisionInfo with informations from wikipedia and iso obp and an subdivision list on country info.
If you are interested, i would prepare an PR with structure and some reference definitions.
here is our structure so far:
public interface ISubdivisionInfo
{
	/// <summary>
	/// CommonName
	/// </summary>
	string CommonName { get; }
	/// <summary>
	/// NativeName
	/// </summary>
	string OfficialName { get; }
	/// <summary>
	/// ISO-3166-2 code
	/// </summary>
	string TextCode { get; }
	/// <summary>
	/// Subdivision code
	/// </summary>
	SubdivisionCode Code { get; }
	/// <summary>
	/// Subdivison category
	/// </summary>
	SubdivisionCategory Category { get; }
}

CommonName is the reference name (from wikipedia, mostly english)
OfficialName is the iso name (mostly local)
SubdivisionCode is an enum with all subdivison codes (without the - ) from here
was thinking about adding administration language but it seems that you dont have this on the CountryInfo and the LanguageCode enum is part of the translation project

Yes, I could imagine that. My suggestion would be to implement it like Nager.Country.Translation simply as an additional module to Nager.Country. This would not increase the size of the main package.

@puschie286 is there any news here yet?

@tinohager looks good so far, we have a working solution in use for a few weeks now. It may have some unusual (related to the your base project) things like fixed enum values or extension methods.
Created a PR for this state #33 with all relevant changes.
If you like to have the generators for the subdivision files and initializer classes, i can add them too or send them - didnt see them in the country project so i left them out


Suggesting alternative to ejecting

When ejecting you can't benefit from upstream create-react-app changes. Suggesting something like grommet-css as an alternative.

sounds a good option. I believe folks that need theme capabilities under create-react-app could fork your grommet-css project and add their variables. Thanks for working on this.
I still would like to still keep this project in place with the ejected version so that folks that still SASS support could benefit from it. Hopefully it makes sense.
For this reason I'm closing this issue but feel free to re-open this if you think otherwise.
I think it would a great a idea for you to create a grommet-react-app that uses your grommet-css, I would happy to promote.

Great! Yes I wouldn't want to replace this repo with grommet-css, it could
potentially be added to the readme as an alternative?
On Mon, Mar 27, 2017 at 2:38 PM Alan Souza<EMAIL_ADDRESS>wrote:

Closed #2 https://github.com/grommet/grommet-react-app/issues/2.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/grommet/grommet-react-app/issues/2#event-1017516829,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABo6B2IXgF5dz_Q43gF32IUl1B06f4Daks5rqCxPgaJpZM4Mo--P
.

definitely. when you have that send it over to me. I will make sure to add that as an alternative 👍


Made the docs prettier for the eventual docs parser.

Merging my own PR, only doc changes on this branch.


Multiple detection for the same tag

I have realized that even in a frame that has only a single tag, there are multiple detections of it (3-4) at the same location. Is this an artifact of the wrapper?

Yes, I met this before too. For me , I just use a  NMS to alleviate it.


Reduce starting creeps for SW and IF

SA starting gold is quite high, mainly SW and IF.
Remove a couple creeps from each camp. (Hogger camp is particularly punishing to new players and TH has a fairly large concentration of creeps also.

Closed by #3073


Add Authorization header when testing using request

What version of Hono are you using?
4.5.11
What runtime/platform is your app running on?
cloudflare worker + bun
What steps can reproduce the bug?
when I want to create unit test using app.request, I can't add authorization header to check JWT is valid or not

What is the expected behavior?
It should not error and unit test can run successfully
What do you see instead?
unit test error and can't run it
Additional information
No response

Hi @ikrom
I can't reproduce it. Please provide a minimal project to reproduce it.

Hi @yusukebe
Actually I want to create unit testing, I have app with route /home
import { Hono } from "hono";
const app = new Hono<{ Bindings: Bindings }>().basePath("/v1");
app.get("/home", async (c) => {
  return c.json({ env: c.env.ENVIRONMENT, message: "Hello World!" });
});

Then I create file home.test.ts to test the route /home
import { describe, test, expect } from "bun:test";
import { Bindings, MOCK_ENV } from "@type";

describe("Index Test", () => {
  test("GET /home", async () => {
    const res = await app.request(
      "/home",
      {
        headers: {
          authorization: `Bearer token`,
        },
      },
      MOCK_ENV
    );
    expect(res.status).toBe(200);
  });
});

But the result is like picture below

Could you help how to create unit test with authorization header jwt?

@ikrom
You should access /v1/home:
const res = await app.request(
  '/v1/home',
  {
    headers: {
      authorization: 'Bearer token',
    },
  },//...
)

thank you for the correction @yusukebe

but there is another error, the res is undefined

@ikrom
I don't know, but the following code is working well:
const app = new Hono().basePath('/v1')

app.get('/home', async (c) => {
  return c.json({})
})

const res = await app.request(
  '/v1/home',
  {
    headers: {
      authorization: 'Bearer token',
    },
  } //...
)

console.log(res.status) // 2000

should the const app = new Hono() in the same file with unit test? because the unit test can't run if the app in different file with unit test

should the const app = new Hono() in the same file with unit test?

No.

Not sure if this helps, but I had to do a fallback in middleware.ts for env to work in production as well in tests.
app.ts
import { Hono } from "hono";
import { configureAppMiddleware } from "./middleware";
import { configureAppRoutes } from "./router";

export function getHonoApp() {
    const app = new Hono();
    configureAppMiddleware(app);
    configureAppRoutes(app);
    return app;
}

middleware.ts
import { Hono } from "hono";
import { env } from "hono/adapter";
import { bearerAuth } from "hono/bearer-auth";

export const configureAppMiddleware = (app: Hono) => {
app.use('/api/*', bearerAuth({
        verifyToken: async (token, c) => {
            // this works in runtime/production with the hono/adapter as per hono docs
            let { API_KEY } = env<{ API_KEY: string }>(c);

            // in testing when MOCK_ENV is passed in, we need to do this as a fallback
            if (!API_KEY) {
                API_KEY = c.env.API_KEY;
            }

            return token === API_KEY
        }
    }))
}

the test looks like this:
app.authed.test.ts
import { beforeAll, describe, expect, test } from "vitest";
import { getHonoApp } from "./app";

const MOCK_ENV = {
    API_KEY: 'test_api_key',
}

describe('App - authed routes', () => {
    let app = getHonoApp();
    let headers: Record<string, string>;

    beforeAll(() => {
        headers = {
            "Content-Type": "application/json",
            "Authorization": `Bearer ${MOCK_ENV.API_KEY}`
        }
    });

    test('should return 200 and You are authorized on /api/page', async () => {
        // there is a /api/page route that returns some json
        const res = await app.request('/api/page', { headers }, MOCK_ENV);
        // this succeed because of the fallback (hack?) in middleware.ts
        expect(res.status).toBe(200);
        expect(await res.json()).toEqual({ message: 'You are authorized' })
    })
})

@gerhardcit thanks for this example. It's what finally clarified the approach I should take for swapping middleware during tests.
I now have a createApp(db?) function that takes an optional db client. If it's present, override the normal database middleware before returning the app instance.
export function createApp(db?) {
  const app = new Hono()
  if (db) {
    app.use((c, next) => {
      c.set('db', db)
      return next()
    })
  } else {
    app.use(databaseMiddleware)
  }
  return app
}

const appPostgres = createApp() // this one for live
const appPGlite = createApp(drizzle(new PGlite())) // this one for tests


We have both Accessibility View and Accessibility Help commands

Testing #186214
Write now we have both "Accessibility View" and "Accessibility Help" commands. Could we unify them in once command, why do we need both?
If we really need to keep the commands separate, could we at least use the same keyboard shortcut? So that when users learn to use Ctrl + F1 they can use the same all across the workbench

They do different things and there may be a case where we want each to apply separately - for example, with focus in the notebook output, we will have an accessibility help menu and also an accessible view.

Thanks for clarifying. But is that really a use case? In what scenario would a user want to have two separate experiences? Why not combine them into one for the notebook example you provided?

In my opinion, mixing help menu content with the content of a particular chat response, hover, or notebook output would not be expected or helpful.

I say that because then the user has to navigate through all of the content or help menu to reach what they want. Instead, with this approach, they just go to it directly using the particular keybinding.

Ok. Feel free to close this if we do not hear user feedback that aligns with my suggestion.


CVE-2015-6644

Security issue found in latest 1.59 version: https://nvd.nist.gov/vuln/detail/CVE-2015-6644

We actually picked this up as part of the initial FIPS project. It was fixed in BC 1.55 and is mentioned in the defect list in release notes - we didn't use the same approach as Android which would explain why the code appears to be missing - GCMBlockCipher.getNextCTRBlock() has the check in it.
Looking at the Android patch it actually appears have a bug in it (non-security related).

Thanks!


Platform asking for a one-time code during log-in

We received the following screenshot from an institution trying to log into the platform

This was fixed in prod via the the keycloak Admin console.  This will be fixed in all dev envs via #220.


Added missing trailing slash

Coverage decreased (-1.3%) to 94.406% when pulling c8da2348d5ef546cfa973387e3acda1180bf8e6b on MarkusH:feature/trailing-slash into 9ec5b1de6e9e2fa12bbea0ad2d5ddb248a39a0e3 on Bouke:master.


Could you share the code that only using RGB as inputs?

Hello,
Thank you for sharing the code. Could you share the code that only using RGB as inputs? (Ours (RGB) in Table.1).
Thanks a lot.

have you got the code? i want it too

I am also confused about how to implement RGB only in this methods. Is there any public code？


Transaction domain types

Is your feature request related to a problem? Please describe.
The clients currently deserialise the polymorphic response from the /safe/{address}/all-transactions into Multisig, Module and Ethereum transactions. Then map each of those types into Transfer, Settings or Custom transaction types.
Describe the solution you'd like
Deserialization is already largely implemented. The domain types (Transfer, Settings and Custom) should be defined according to the needs of the apps.
Additional context
For reference, the domain transaction data layouts expected by Android are as follow:
sealed class Transaction {
    abstract val status: TransactionStatus

    // If status is Successful, Failed or Canceled, the confirmations can be null
    abstract val confirmations: Int?

    data class Custom(
        override val status: TransactionStatus,
        override val confirmations: Int?,
        val nonce: BigInteger?,
        val address: Solidity.Address,
        val dataSize: Long,
        val date: String?,
        val value: BigInteger
    ) : Transaction()

    data class SettingsChange(
        override val status: TransactionStatus,
        override val confirmations: Int?,
        val dataDecoded: DataDecodedDto,
        val date: String?,
        val nonce: BigInteger
    ) : Transaction()

    data class Transfer(
        override val status: TransactionStatus,
        override val confirmations: Int?,
        val recipient: Solidity.Address,
        val sender: Solidity.Address,
        val value: BigInteger,
        val date: String?,
        val tokenInfo: ServiceTokenInfo?,
        val nonce: BigInteger?
    ) : Transaction()
}

enum class TransactionStatus() {
    AwaitingConfirmations,
    AwaitingExecution,
    Cancelled,
    Failed,
    Success,
    Pending // Not supported yet as these correspond to the ones issued from the device
}

I would propose the following:
pub struct Transaction {
    pub id: String,
    pub timestamp: u64,
    pub tx_status: TransactionStatus,
    pub tx_info: TransactionInfo,
    pub execution_info: Optional<ExecutionInfo>
}

pub enum TransactionStatus {
    AwaitingConfirmations,
    AwaitingExecution,
    Cancelled,
    Failed,
    Success,
    Pending
}

pub enum TransactionInfo {
    Transfer(Transfer),
    SettingsChange(SettingsChange),
    Custom(Custom)
}

pub struct ExecutionInfo {
    pub nonce: u64,
    pub confirmations_required: u64,
    pub confirmations_submitted: u64
}

TBD

Properties of the TransactionInfo objects
Should there be a Option for transaction hash
How to calculate the id

Should our timestamp already be parsed into DateTime<Utc> ? We already have that type used in the deserialization. We'd need to define which field goes into it.

I prefer a number, as this is easier to handle locally and avoid string parsing on client side ... but depends on what is better for the clients.

For the id I would propose a format that allows us to distinguish between the different low level tx types (e.g. by using the type as a prefix: multisig_, ethereum_, module_) in combination with information to uniquely identify the information.

I am not sure why id is necessary. Is this for the web interface?

If you want to query the details you need a way to reference what details to load ;)

Then it would make sense to encode what you mentioned here: https://github.com/gnosis/safe-client-gateway/issues/24#issuecomment-664424029 so that we can make a decision of which endpoint we use as there are different ones as I understand it:
/safe/{address}/incoming-transfers, /safe/{address}/module-transactions, etc

I would not add the transaction hash, since it is not required by the mobile interface (web might need it, but in that case we can add it later.)

pub struct Custom{
    pub to: Address,
    pub data_size: String,
    pub value: String,
}

Do we need pending status in gateway? Afaik, this status is only reachable from a device when a transaction is proposed, right?

A transaction can be failed or cancelled (only ethereum tx are always successful)


Truncated Endpoint Column flashes whenever resource status changes

On the resource dashboard, if the endpoints column for a resource is truncated, whenever the resoruce status changes for a resoruce, then the endpoint value in the resource's row briefly flashes.
Whenver the resource updates, the endpoint column will briefly flash without the +N icon for truncation

Before settling back to displaying +N

Below is a simple app that can be used to reproduce this.
This is definitely not a major issue, but it is an irritating one.
var builder = DistributedApplication
    .CreateBuilder(args);

builder.AddExecutable("pwsh", "pwsh.exe", ".")
    .WithArgs("-Command", "Start-Sleep -Seconds 9999")
    .WithEndpoint(name: "One", port: 5551, isProxied: false)
    .WithEndpoint(name: "Two", port: 5553, isProxied: false)
    .WithEndpoint(name: "Three", port: 5553, isProxied: false);

builder.Services.AddLifecycleHook<TestHook>();
await builder.Build().RunAsync();

public class TestHook(ResourceNotificationService resourceNotificationService) : IDistributedApplicationLifecycleHook
{
    public Task AfterResourcesCreatedAsync(DistributedApplicationModel appModel, CancellationToken cancellationToken = default)
    {
        var resource = appModel.Resources.Single(x => x.Name == "pwsh");
        _ = RotateStatus();
        return Task.CompletedTask;

        async Task RotateStatus()
        {
            while(true)
            {
                await Task.Delay(1000);
                await resourceNotificationService.PublishUpdateAsync(resource, x => x);
            }
        }
    }
}
```

@adamint PTAL.


Possible to call Azure FHIR API using MSAL authentication process

I am using MSAL daemon server authentication mechanism to call FHIR API using example specified in below link,
https://github.com/Azure-Samples/active-directory-dotnetcore-daemon-v2
I tried with sample application, able to retrieve access token using MSAL library but its giving forbidden error while calling FHIR API from daemon process, Is it possible to access FHIR API using daemon application?

Its mentioned in MSAL sample application that we need application permission in order to access API, let me know if it is possible to provide application permission for FHIR API from app registration, currently application permission for Azure Healthcare API is disabled and not able to add as shown in below image,

I am assuming that by "daemon server authentication" you mean service to service authorization and so you are using "client credentials flow" (in Azure AD parlance).
It is not exactly clear from this whether you are getting a 401 or 403 error code, but it sounds like 403. So if that is the case, the reason is probably that the service principal that you are getting a token for does not have access to the FHIR API. Did you give that service principal access? And is this specifically related to this fhir-server-samples repo or is this a general question related?

@hansenms
Yeah, you are right about below points,

Error Code : 403, Forbidden
Using Client Credential Flow, sample flow as,

But I am trying to access FHIR API from daemon process, and authentication mechanism is MSAL latest library, I will check details of service principals....
MSAL team is looking for someone from Azure Healthcare API's, please have a look into below git hub issue,
https://github.com/Azure-Samples/active-directory-dotnetcore-daemon-v2/issues/89

If you are getting a 403, it means that the service principal (the app registration) does not have access to the FHIR API. You need to grant it access. How you do that depends on whether you are using Azure RBAC or Local RBAC.
You need to take a look at your token (e.g. with https://jtw.ms) and figure out what the oid claim is. That is the object id of the identity that you need to grant access. It is slightly confusing because there is both and app id and an object id, but the object id is what matters. If you are using local RBAC, you simply add that oid to the list of allowed object ids per instructions above. If you are using Azure RBAC, follow the instructions there.
@ginalee-dotcom is on the Healthcare APIs team and can assist.

Issue resolved by following steps for Azure RBAC, I need to assign FHIR roles to desired app to access FHIR API from daemon or web api, I assigned role to my user name instead of App...
https://docs.microsoft.com/en-us/azure/healthcare-apis/configure-azure-rbac


Add support to configure labels and annotations of the pods created by kanister functions

Change Overview
This PR introduces two new args podLabels and podAnnotations to the the Kanister functions that create new pods when they are executed.
Pull request type
Please check the type of change your PR introduces:

[ ] :construction: Work in Progress
[ ] :rainbow: Refactoring (no functional changes, no api changes)
[ ] :hamster: Trivial/Minor
[ ] :bug: Bugfix
[x] :sunflower: Feature
[ ] :world_map: Documentation
[ ] :robot: Test
[ ] :building_construction: Build

Issues

fixes #issue-number

Test Plan

[x] :muscle: Manual
[ ] :zap: Unit test
[ ] :green_heart: E2E

//ToDo: Add tests unit or e2e.

I see there are other PRs related to this change. Can we have them in the same PR? There shouldn't be too much code to keep it reviewable and it's actually easier to review them all as one.

@viveksinghggits can you PR link to update docs here?

@viveksinghggits can you PR link to update docs here?

I have added the link in PR desc.


support git submodules

if repo has submodules, the commits are not found

https://github.com/randomize/git-standup/commit/5b3677fb4c0474c74d062db7088e17de60987988
Simplest possible solution to add 'git submodule foreach', but perhaps a more sophisticated one would be updating .git directory search algorithm so it doesn't ignore submodule dirs (they have .git as file instead of dir).

Really excited to see this tool exists, bummed to learn it ignores submodules

I found that commenting all these lines will show the commits from submodules:
For a strange reason, this condition evaluates to TRUE and skips the submodule commits.
You need to pass -F though.
git standup -F -d 3 -m 2
continue if not a git directory
if [[ ! -d ".git" || -f ".git" ]] ; then
cd ${BASE_DIR}
continue
fi

Issue still exists in master. Applying the patch in #116 seems to fix it for me. Were there some other issues with the PR that made you close it @mihaicris ?


Remove post references in the articles

There are a bunch of references linking series articles for the first (series begin).
After #100, is not needed anymore to add texts like

"if you don't know what's is this about, here's the link for the first article of this series"

Closed by #108


Export to csv - App breaks down

Steps to reproduce the behaviour

Pick Menu Export,
export to file/owncloud/mail (same behaviour for all)
format csv
time from 01.01.2019
delete transactions after export: no
repeat: none

Tip "Exportieren"
Expected behaviour
A file is created in csv-format
Actual behaviour
gnu-cash breaks down, Application halted
no file created.
whenn i pick export to "speichern unter" there is a file created, but empty.
Software specifications

GnuCash Android version: 2.4.0
System Android version: 6.0.1
Device type: FP 2 (FairPhone)

I am working on this issue.

I resolved the issue


Using TempDirectory in packaging tasks

updating packaging tasks to require min agent version 2.115.0, and updating temp directory to use Agent.TempDirectory
updating packaging tasks to contain the current sprint as the minor version number.

Linux CI failing, looks like it's not related to this change.


New game Arkanoid

Description
I want to add Arkanoid game .
assign me this under GSSOC'2023
reference google image of game given below its not build yet
Screenshots

Additional information
No response

@raj03kumar My pull request got merged but didnt got label as gssoc23 please can you do it as it counts


Invalid POST transaction using Bigchaindb JS client

BigchainDB version: 1.3.0

Operating System: Ubuntu 16.04

Deployment Type: Host

Documentation followed:

https://docs.bigchaindb.com/projects/server/en/latest/quickstart.html
https://github.com/bigchaindb/js-bigchaindb-driver

BigchainDB driver: yes

If using a driver please specify, driver type: js<EMAIL_ADDRESS>

Description
I was trying to run the js client as mentioned in https://github.com/bigchaindb/js-bigchaindb-driver.
But when I run the node client, I get the following error:
{ message: 'HTTP Error: Requested page not reachable',
  status: '400 BAD REQUEST',
  requestURI: 'http://localhost:9984/api/v1/transactions' }

In the bigchaindb console, logs generated:
[2018-02-02 09:23:30] [ERROR] (bigchaindb.web.views.base) HTTP API error: 400 - POST:/api/v1/transactions - Invalid transaction (InvalidHash): The transaction's id 'd0b649dbd0b76e254be72425f2cdd525a6b03ebfd593f418c4d68bf03a762e50' isn't equal to the hash of its body, i.e. it's not valid. (webapi     - pid: 2175)

Steps to Reproduce

https://docs.bigchaindb.com/projects/server/en/latest/quickstart.html
https://github.com/bigchaindb/js-bigchaindb-driver

What I Did
Installed mongodb, bigchaindb, installed bigchaindb-driver for node and ran the node client using the command "node <filename.js>"

This seems like a JS Driver issue so I created an issue for it on the JS Driver repository:  https://github.com/bigchaindb/js-bigchaindb-driver/issues/148

Having the same problem a node express app, running ubuntu 16.04

i am facing the same issue


docs: add geometry preparation for Fluent simulation

Description
Follow up of current NACA airfoil example
Issue linked
None
Checklist

[X] I have tested my changes locally.
[X] I have added necessary documentation or updated existing documentation.
[X] I have followed the coding style guidelines of this project.
[X] I have added appropriate unit tests.
[X] I have reviewed my changes before submitting this pull request.
[X] I have linked the issue or issues that are solved to the PR if any.
[X] I have assigned this PR to myself.
[X] I have added the minimum version decorator to any new backend method implemented.
[X] I have made sure that the title of my PR follows Conventional commits style (e.g. feat: extrude circle to cylinder)

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 90.59%. Comparing base (c807809) to head (2ca253c).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1183   +/-   ##
=======================================
  Coverage   90.59%   90.59%
=======================================
  Files          93       93
  Lines        7015     7015
=======================================
  Hits         6355     6355
  Misses        660      660

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Update MXParallaxHeader Dependency

I wanted to use a newer Version of MXParallelHeader that includes the fix for https://github.com/maxep/MXParallaxHeader/issues/24 but for that, the dependency on MXParallaxHeader needs to be increase to 0.5.0

I've jumped to 0.5.1


Replace Travis CI with GH Actions

This is for replacing Travis CI with GH Actions

TravisCI is no longer used, Scrutinizer has been sufficient so far. GH Actions are great too, but Scrutinizer with its integrated code quality reports are handy.


README instructions don't work

yarn add formik-Fluent-UI-react
results in error An unexpected error occurred: "https://registry.yarnpkg.com/formik-Fluent-UI-react: Not found"..
Also after installing yarn add formik-fluent-ui-react (case changed), when I yarn start, I get Module not found: Error: Can't resolve 'formik-fluent-ui-react'... not sure if it's related.

same problem here.... and if you clone the repo and you try to build it doesn't work....


Shifting the response of all packages to a map from a list.

@YIDING-CODER Can you check this over quick and then get it merged? We need it for testing the vscode plugin integration. Thanks!
CC @nirlipo

Hi @haz ,
It look ok to me. I have merged it. Thank you.
Thank you.
Regards,
Yi