Update the package yaml file in kubefed-operator

Update package yaml to 4.2.0
Update the image-references file to include ose-kubefed as well.
Let operator version percolate down through the olm-e2e script.

nit: You need to also make changes in this file (https://github.com/openshift/kubefed-operator/blob/master/hack/catalog.sh#L17) to ensure that we're testing our changes against the version listed in package.yaml file.

/test olm-e2e

/lgtm

Overall, LGTM

/lgtm

/test olm-e2e

/test olm-e2e

/lgtm


config: override actor_runtime_worker_threads_num with --parallelism

Recently we found some users are deploying 2 compute nodes on an 8-core machine with each configured with --parallelism 4, and then expect each compute node to take no more than 400% CPU usage.
This is not true as the parallelism is a per-fragment concept, so the CPU usage is actually bounded by the number of threads. If actor_runtime_worker_threads_num is not set, tokio will determine this value based on the number of CPU cores (acquired with num_cpus crate), which is 8 in this case. So there could be resource contention between two compute node processes.
This also reminds me that we've managed to be aware of the limitation set by the container when deciding the default parallelism. As talked above, it seems that we also have to apply this to the worker thread number.
cc @fuyufjh @lmatz @hzxa21

Makes sense to me. Since --parallelism is the CN's command-line argument, I cannot image why users set it manually but expect higher CPU utilization.


Changed ToDoApp to show the actual remaining todos

Fixes #521

LGTM although it might be clearer to write:
return this.todos.length - this.todos.reduce((count, todo: Todo) => count + todo.done, 0);

@petebacondarwin That will show the done count and not the remaining count.

I agree with @ntaoo; the method should do what it says its doing. I'd keep the reduce.
  get remaining() : number {
    return this.todos.reduce((count, todo: Todo) => count + !todo.done, 0);
  }

No need for the explicit return type either; TS knows it's a number.

Actually I really like the filter version!

@petebacondarwin why prefer filter to reduce? It's not a huge deal in practice but reduce is more to the point.
It is also more efficient because it does not create a new array, only to discard it after calculating its length (I repeat: not measurable in practice).

@wardbell well it just matches my brain's understanding of what we are trying to do. But on second look I am happy with your suggestion too.

THE MOST IMPORTANT thing is that we fix the darned method soon ... because it presently returns the number of completed Todos rather than the number remaining.
With everyone's indulgence, I'm just going to git-er-done here.

@wardbell : You just wrote my thoughts during this entire conversation.

Indulge yourself, with post-yule fervour!


VSimporter does not work when it's located on a network drive

I am using Parallels on my Mac and the WinObjC project is located on my Mac's partition so that it is mounted as a network drive when using Windows via Parallels.
As long as the VSimporter lies on the network drive, it fails with [E] Invalid WINOBJC_SDK_ROOT specified: "\Mac\Slimport\Source\Windows-Bridges\WinObjC\winobjc". The SDK directory does not exist.. As soon as I copy it to a local path like C:/Temp it works without problems.

@robinmanuelthiel vsimporter should now work much better with network paths.


divide by zero in part wear

escape possible divide by zero that happens in some situations

 _wear /= partCount; line 393 in FormsMethodTrailer

closed because pull request #20


Add metacontroller for kubeflow 1.4

This is an MVP of the charm. A fully implemented charm should have better status handling, teardown, etc.

This is an MVP of the charm.  A fully implemented charm should have better status handling, teardown, etc.

Unit tests work locally but fail in GH because I forgot to mock away a call to lightkube.Client() and it needs k8s config
integration tests fail I think because the charm is trying to create the service account multiple times.  Will debug tomorrow.

Unit tests work locally but fail in GH because I forgot to mock away a call to lightkube.Client() and it needs k8s config
integration tests fail I think because the charm is trying to create the service account multiple times.  Will debug tomorrow.

You can stop the behaviour where lightkube is looking for a kubeconfig with something like:
@patch("lightkube.core.client.GenericSyncClient", Mock)


Can this erank be used to measure the denoising performance of each decoder in LLM for the prompt during inference?

Hello, Thank you for your excellent work! Can this erank be used to measure the denoising performance of each decoder in LLM for the prompt during inference?
For example:
pre_erank = compute(hidden_states)
hidden_states = Decoder(hidden_states)
erank = compute(hidden_states)
diff_erank = erank - pre_erank

My output on vicuna-7B:
input erank: 42.17709568549561
Layer-0 erank: 22.278847713844357
Layer-1 erank: 25.993923685907696
Layer-2 erank: 1.4035135446476772
Layer-3 erank: 1.70448817561117
Layer-4 erank: 2.1620211017106823
Layer-5 erank: 2.746670162487955
Layer-6 erank: 3.5887634940906863
Layer-7 erank: 4.865407429924545
Layer-8 erank: 5.946523446662109
Layer-9 erank: 7.201062758878346
Layer-10 erank: 8.996432653324947
Layer-11 erank: 10.834089326797553
Layer-12 erank: 12.656450419761345
Layer-13 erank: 14.85684317424668
Layer-14 erank: 17.62239728006985
Layer-15 erank: 20.693067609921357
Layer-16 erank: 24.21716760094568
Layer-17 erank: 27.88568061557354
Layer-18 erank: 30.811372641972376
Layer-19 erank: 33.095506446060455
Layer-20 erank: 34.60140509509908
Layer-21 erank: 35.78787189742235
Layer-22 erank: 36.743443398364526
Layer-23 erank: 37.19281398112061
Layer-24 erank: 37.87511391452051
Layer-25 erank: 38.53895323976143
Layer-26 erank: 39.11498239993629
Layer-27 erank: 39.59134616840688
Layer-28 erank: 40.08329612333418
Layer-29 erank: 40.41086106326149
Layer-30 erank: 39.91761263915652
Layer-31 erank: 33.17868285609608

Thanks for your question. Actually how to analyze the mechanism of previous decoder layers is still a challenge, and we are not sure what will happen when using Diff-eRank to measure the denoising performance of different layers. Maybe you can refer to [1], where you can find similar trends between Diff-eRank (in your experiments) and Token Cosine similarity.
[1] Xue, Fuzhao, et al. A Study on Transformer Configuration and Training Objective. In ICML 2023.


typo?

Vivek,
I was trying to run your example and had some trouble doing so:
https://github.com/vivek-bala/radical.entk/blob/0bf075dbdddaaa5c7a23934c71d0140ad5bc1d8a/src/radical/entk/execman/resource_manager.py#L252
Shouldn't database_url be dburl ?

Hey @mpbl , yes I changed it to dburl now. I was testing against different versions of RP and I think I missed pushing this change. Thanks, please test and feel free to close the ticket if it is resolved.


Add data generation to terraform config?

Question
should we add the new data generation server to the default terraform config?
could be done without adding the cloud scheduler component.
pro: would be useful in getting people up and running with test data
con: should be disabled before going to production.
-- Low risk of harm if left on, as we also generate padded fake data on downloads.

/assign


restart command and background image random from the folder

sorry for the behavior priority code.
keybinding ctrl + shift + f7  refresh command
add option randomImageFolder
Allow the following path pattern
・ "c:\Users\hot3\Pictures\vscode\sao01\"
・ "c:\Users\hot3\Pictures\vscode\sao02"
・ "c:/Users/hot3/Pictures/vscode/nisekoi01"
・ "c:/Users/hot3/Pictures/vscode/nisekoi02/"
・ "/Users/hot3/Pictures/vscode/konosuba01"
・ "/Users/hot3/Pictures/vscode/konosuba02/"

Thank you, I like this feature very much, I will post it to the market later.


replace Range with kotlin Range

check if the kotlin Range serializes well with jackson

in kotlin 0.10.4 the Range types are not serializable in Kotlin. Bug [filed in youtrack]
(https://youtrack.jetbrains.com/issue/KT-6637)
Hoping to get a fix for the next milestone.

not an easy fix, closing this for now...


Support thumbnails

SeaweedFS supports returning .png, .jpg and .gif files with custom height and width. We can use this feature to add a ?thumbnail parameter for these types and have a smaller image returned.
Reference: https://github.com/chrislusf/seaweedfs/blob/b3f714612d289588c0772cf5fb1b8d2b9c874061/weed/server/volume_server_handlers_read.go#L138

 Thumbnail support in cdn-origin

Is their anyway to output a higher resolution thumbnail?

Thumbnails are supposed to be smaller versions of the normal image so they can be shown in lists and stuff where only a smaller sized (pixel and filesize) image can be displayed. SeaweedFS probably could output an image with a bigger resolution, but I don't see a valid use for cdn-origin to include image embiggening functionality.

Fair enough. I know we talked about this in Discord, so yh...


Swap json with toml format

Now, Meta data are in stuff/titles.json
Task: Let move to toml format with tomland library.

I would like to tackle this issue.

Resolved by #78


fix: miniapp app.json pages field need be filtered by targets

Codecov Report

Merging #982 (bdbaa31) into release-next (506ccfa) will increase coverage by 0.10%.
The diff coverage is 100.00%.

@@               Coverage Diff                @@
##           release-next     #982      +/-   ##
================================================
+ Coverage         51.48%   51.59%   +0.10%
================================================
  Files                47       47
  Lines              1280     1287       +7
  Branches            297      302       +5
================================================
+ Hits                659      664       +5
- Misses              621      623       +2

Impacted Files
Coverage Δ

packages/plugin-rax-miniapp/src/index.js
58.88% <100.00%> (+1.05%)
:arrow_up:


Dont show rules if PARK-CENTER is false

Rules are displayed on every treks in Geotrek-mobile. While it should be displayed only on treks that have value PARK_CENTER = True as it is in Geotrek-rando !

fixed by bb70fd1e35e67beb0e45b554a301ebccbb438133


Add line number to JUnitFormatter

Feature Request
I am working on adding the ability to run specs to the Crystal VSCode extension and I am using the junit file in order to find/pass/fail testcases. I would like to add a line number parameter to the JUnit XML output. This would make things easier for me so that I don't need to parse the spec files manually to figure out where they are.
An example of what the output would look like now (with added line parameter):
<?xml version="1.0"?>
<testsuite tests="1" skipped="0" errors="0" failures="1" time="2.003540435" timestamp="2023-05-13T19:25:33Z" hostname="my-hostname">
  <testcase file="/path/to/project/directory/mlcsim/spec/mlcsim/transcoder_spec.cr" classname="spec.mlcsim.transcoder_spec"
    name="MLCSim::Transcoder when transcoding encodes a value into cells" time="2.3956e-5" line="5"/>
</testsuite>

On another note, I am unsure if two testcases with the same name are allowed. If they are, this would help differentiate them:
  <testcase file="/path/to/project/directory/mlcsim/spec/mlcsim/distribution_spec.cr" classname="spec.mlcsim.distribution_spec"
    name="MLCSim::Distribution #normal_midpoint successfully finds the midpoint between 0.4 and 0.8 for different stdevs" time="1.005175956" line="10"/>
  <testcase file="/path/to/project/directory/mlcsim/spec/mlcsim/distribution_spec.cr" classname="spec.mlcsim.distribution_spec"
    name="MLCSim::Distribution #normal_midpoint successfully finds the midpoint between 0.4 and 0.8 for different stdevs" time="1.003250633" line="20"/>

All alternate solutions (parsing the files manually, adding a custom formatter, etc.) were much more complex than adding this parameter. As far as I know, this does not break backwards compatibility.

Sounds very reasonable. This was probably just forgotten when JUnit formatter was initially implemented.


Implement Vreman SGS closure

We should implement the Vreman subgrid closure described in
http://www.vremanresearch.nl/Vreman-PF2004-subgridmodel.pdf
cc @tapios

It's implemented here: https://github.com/climate-machine/CLIMA/blob/master/src/Atmos/Model/turbulence.jl
We should move this out of the 'Atmos' tree into some space for shared numerics for atmosphere, ocean, etc. @asraero @simonbyrne ?

At the moment it is somewhat coupled to the AtmodModel, but we could try to unify them once the Ocean part is in place.

I'm closing this issue because I'm judging that it's not of current, timely relevance to Oceananigans development. If you would like to make it a higher priority or if you think the issue was closed in error please feel free to re-open.


Investigate why InitialSessionState.CreateDefault2() is slower than CreateDefault()

CreateDefault2() should only be loading the Core cmdlets module while CreateDefault() loads more, yet CreateDefault2() is 3-8 times slower.
'CreateDefault','CreateDefault2' | %{  (measure-command {
$iss = [InitialSessionState]::$_();
1..100 | % { $ps = [powershell]::Create($iss).AddCommand('echo').AddArgument(1); $ps.Invoke(); $ps.Dispose() } }).TotalMilliseconds }

1190.377
3268.448

Formally 'CreateDefault2' is faster 'CreateDefault':
'CreateDefault','CreateDefault2' | %{  (measure-command {
$iss = [InitialSessionState]::$_();
1..100 | % {  } }).TotalMilliseconds }

5.1233
2.0122

Just creating an InitialSessionState is not interesting.

But running a command in that runspace - that's a typical scenario, and in theory, in that scenario, CreateDefault2 should do less work, so it should be measurably slower.

What seems to be happening is that during Invoke() Microsoft.PowerShell.Utility\Write-Output (aka. echo) triggers CommandDiscovery.TryModuleAutoDiscovery. While CreateDefault() returns with that module already loaded.
I've tried to explicitly import "C:\WINDOWS\system32\WindowsPowerShell\v1.0\Modules\Microsoft.PowerShell.Utility\Microsoft.PowerShell.Utility.psd1" before Write-Output but it didn't yield the expected results - AddCommand('Write-Output').AddArgument(1).Invoke() is now fast, but most of the time is being spent before that in Import-Module. Seems like it's loading more slowly than in CreateDefault, I must be doing something wrong.


Update free-courses-en.md

a more beautiful way to write. so we add - in between free courses that look professional
What does this PR do?
Add resource(s) | Remove resource(s) | Add info | Improve repo
For resources
Description
Why is this valuable (or not)?
How do we know it's really free?
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Don't invent titles! Use the title given by the resource.


⚠ Bofrost.at has degraded performance

In 406a824, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1571 ms

Resolved: Bofrost.at performance has improved in 3ad2141 after 20 minutes.


Perl alternate comments using pod directives

As described here:
http://perldoc.perl.org/perlfaq7.html#How-can-I-comment-out-a-large-block-of-Perl-code%3F
The idea is to add on a line previous to the selection '=begin comment', and add on a line after the selection '=end comment'

forgot the closing =cut, so it should look like:
=begin comment

$this->is($code);
...

=end comment
=cut


chore(main): release 0.13.0

:robot: I have created a release beep boop
0.13.0 (2024-08-31)
Features

figure: add evaluation of the Pearson coeff for figure 4 (3b71c2d)

Styles

format files and fix permissions (c896258)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.13.0
:sunflower:


252 recurring booking

IMPORTANT: Please do not create a Pull Request without creating an issue first.
Please provide enough information so that others can review your pull request:
Complete rework of booking requests to now use reservation as a pivot table for rooms.

Explain the details for making this change. What existing problem does the pull request solve?
Allows for multiple reservations to be part of a booking request

can add and remove dates from a creation.
Test plan (required)
Demonstrate the code is solid. Example: The exact commands you ran and their output, screenshots / videos if the pull request changes UI.

Closing issues
List all issues this PR resolves.

closes #252

Pre-Merge checklist
Complete these items before requesting reviews

[x] Static Analysis Rules Validated
[x] All commits follow naming conventions

Sub-tasks Checklist

[x] UI Implementation
[x] UI Test
[x] Backend Implementation
[x] Backend Tests
[x] Integration Tests
[x] System Tests
[ ] Documentation

Currently needs some documentation updates but feel free to review.

Currently needs some documentation updates but feel free to review.

I'll start adding checks for conflicts to this PR. I also need to make sure the maximum is not set.

I'll start adding checks for conflicts to this PR. I also need to make sure the maximum is not set.


New pinax-starter app requirements

New travis and tox config (drop 1.7 support)

Coverage increased (+0.04%) to 71.855% when pulling 20eef386db7b1abc3ee187216876cdaac740a684 on arthur-wsw:starter_app into 6abe60679b382f746e4d38487ed54520f913c8fb on pinax:master.

I had to delete old migrations and regenerate new one with makemigration.py to make build succeed because of new AppConfig


refactor open AEAD selection

No functional change, just cosmetic. It still looks kind of gross, but
IMO better than before.

Fixed clippy warning.

Well, that broke qlog builds...


CIV-6925 Refactor solicitor notification event

Before creating a pull request make sure that:

[ ] commit messages are meaningful and follow good commit message guidelines
[ ] README and other documentation has been updated / added (if needed)
[ ] tests have been updated / new tests has been added (if needed)

Please remove this line and everything above and fill the following sections:
JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/CIV-6925
Change description
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ ] No


Fix #2214: Use Utils.isDarkTheme consistently through app

Description (required)
Fixes #2214 Inconsistent usage of Utils.isDarkTheme
Tests performed (required)
Tested 2.9.0-debug-fix-2214~03c36bb11 on Galaxy Nexus (emulator) with API level 28.

Codecov Report

Merging #2268 into master will decrease coverage by 0.02%.
The diff coverage is 25%.

@@            Coverage Diff            @@
##           master   #2268      +/-   ##
=========================================
- Coverage    6.33%   6.31%   -0.03%
=========================================
  Files         233     233
  Lines       11599   11589      -10
  Branches     1066    1065       -1
=========================================
- Hits          735     732       -3
+ Misses      10802   10796       -6
+ Partials       62      61       -1

Impacted Files
Coverage Δ

...fr/free/nrw/commons/settings/SettingsActivity.java
66.66% <ø> (+5.55%)
:arrow_up:

...n/java/fr/free/nrw/commons/theme/BaseActivity.java
66.66% <25%> (+10.41%)
:arrow_up:

app/src/main/java/fr/free/nrw/commons/Utils.java
25.6% <0%> (+1.21%)
:arrow_up:

Continue to review full report at Codecov.

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


Restoring degit

it seems like @Rich-Harris is back (last commit is 5 days ago on the original repo). Any plans to move the changes to Rich-Harris/degit ?

If Rich is actively merging PRs, this fork might not be needed. I'll wait and see for now.


Would a clojure-style thread macro (->) be possible?

This guy: https://clojure.org/guides/threading_macros
Which allows rewriting
(string (first (range 0 10)))

as
(-> (range 0 10)
  first
  string)

(both psudeocode)

I suppose, it's trivially done with function since functions in Finkel are lazy:
;;; File: cljthr.fnk

(defmodule Main
  (import (Data.Function (&))))

(defn (:: main (IO ()))
  (do
    ;; Fully parenthesized
    (print (show (head (enumFromTo 0 10))))

    ;; Same as above, but with operator expansion with `$'
    ($ print show head (enumFromTo 0 10))

    ;; Same again, but with operator expansion with `&'.
    (& (enumFromTo 0 10) head show print)

    ;; Making partially aplied function in Haskell and Finkel is easy,
    ;; mimicing following clojure expression which uses `->>' macro:
    ;;
    ;;   (->> (range 10)
    ;;        (filter odd?)
    ;;        (map #(* % %))
    ;;        (reduce +))
    ;;
    (print (& [0 .. 10]
              (filter odd)
              (map (flip ^ 2))
              (foldr1 +)))))

Running above:
$ finkel make cljthr.fnk
[1 of 1] Compiling Main             ( cljthr.fnk, cljthr.o )
Linking cljthr ...
$ ./cljthr
"0"
"0"
"0"
165

Perhaps operator application section in the documentation may help how $ and & are treated.
@0atman Does above show what you were looking for?

 The & form is EXACTLY what I need. Thank you so much!
I wanted to get this working in haskell, but I see Finkel has what I need! (https://stackoverflow.com/q/60464090/333294)


Update poetry installation in Dockerfile

the poetry installation URL in dockerfile had deprecated. (ref: https://github.com/python-poetry/poetry/issues/6377)

@simonprickett could you check this PR? ☺


Error reading Temp2 Exception: bytearray index out of range

Did you read the instructions?

[X] I have read the README.

The problem
I have a Kamstrup 403 flow and heat meter connected to control unit for district heating, which I would like to integrate into Home Assistant.
I bought a USB IR read/write module via Ebay. Installing the kamstrup component via HACS worked out of the box. I was able to connect to the USB IR read/write module and the meter is detected, however, no values are displayed. In the Home Assistant log I find two recurring error messages (see below).
What version of this integration has the issue?
1.2.1
What version of Home Assistant Core has the issue?
2022.9.0
Home Assistant log
Error message 1:

This error originated from a custom integration.

Logger: homeassistant
Source: custom_components/kamstrup_403/sensor.py:68
Integration: Kamstrup 403 (documentation, issues)
First occurred: 7:57:46 PM (3 occurrences)
Last logged: 7:59:46 PM

Error doing job: Task exception was never retrieved
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 151, in _handle_refresh_interval
    await self._async_refresh(log_failures=True, scheduled=True)
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 283, in _async_refresh
    self.async_update_listeners()
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 110, in async_update_listeners
    update_callback()
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 348, in _handle_coordinator_update
    self.async_write_ha_state()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 533, in async_write_ha_state
    self._async_write_ha_state()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 571, in _async_write_ha_state
    state = self._stringify_state(available)
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 539, in _stringify_state
    if (state := self.state) is None:
  File "/usr/src/homeassistant/homeassistant/components/sensor/__init__.py", line 386, in state
    native_unit_of_measurement = self.native_unit_of_measurement
  File "/config/custom_components/kamstrup_403/sensor.py", line 68, in native_unit_of_measurement
    return self.coordinator.data[self._command].get("unit")
KeyError: 87

Error message: 2
This error originated from a custom integration.

Logger: custom_components.kamstrup_403
Source: custom_components/kamstrup_403/__init__.py:100
Integration: Kamstrup 403 (documentation, issues)
First occurred: 7:57:46 PM (6 occurrences)
Last logged: 7:59:46 PM

Error reading Temp2 Exception: bytearray index out of range
Error reading Flow Exception: bytearray index out of range

Additional information
This is my Kamstrup meter:

Hi @kalbfuss
I think this might be due to the reader not being perfectly placed on the meter. In my experience this positioning is pretty exact.
Could you try moving the reader a bit and see after at least a minute of its able to read?
Also, can you post the logs from the logfile directly? In my experience the logs in the frontend do not always give all details, thanks.

Thanks for your hint, @golles. It brought me onto the right track. I got it to work by

Inverting the IR sensor head
Inserting a piece of paper between the sensor head and the meter

Looking at the sensor head I noticed that it contains a transparent and a dark diode. Same is true for the sensor port on the kamstrup meter. Matching colors inversely required to turn the sensor head upside down.
The trick with the piece of paper I learnt from the following thread. It appears that the IR LED on some sensor heads is too bright and needs to be dampened.
I am very happy now. This is an awesome piece of software. Thank you so much for the great work.

Thank you so much for explaining how you fixed this, I wasn't aware of such a thing.
I think I also have my reader upside down (cable goes up). Someone in my neighborhood 3d printed a holder to snap nicely onto the meter.
Maybe I should make a simple FAQ on the readme page with these tips.
Thank you so much for your kind words


Updated manuscript: Add MATEUSZ as author

What section of the manuscript should be updated?
Add @mkoptyra as author!!!
Proposed changes
What changes need to be made and/or what details are missing?
Explain your rationale for the update.
Please include a level of detail that will allow another participant to make the update.
Relevant literature
If there is relevant scientific literature, put links to those items here.

closed with #1


pull request for e2e test at 01/18/2017 22:26:14

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


[synthetix] Feat/extend synthetix strategy

Changes proposed in this pull request:

Add OVM support for synthetix strategy

@andytcf getting this error
TypeError: Cannot read property '0' of undefined

      2145 |                 case 1:
      2146 |                     data = _a.sent();
    > 2147 |                     return [2 /*return*/, Number(data.blocks[0].number)];
           |                                                  ^
      2148 |             }
      2149 |         });
      2150 |     });

      at call (dist/snapshot.cjs.js:2147:50)
      at step (dist/snapshot.cjs.js:93:23)
      at Object.next (dist/snapshot.cjs.js:74:53)
      at fulfilled (dist/snapshot.cjs.js:64:58)

Can you check plz :)


Wait() never unblocks when no goroutine is running

Hi,
I am using tomb to monitor rather short-lived goroutines. It is possible that no goroutine is running when I do Kill(nil) and Wait(), or even no goroutines has ever been started. I find it strange and not very intuitive that in this case Wait() blocks forever. Wouldn't it make more sense to close the dead channel immediately when no registered goroutine is running?

It would be stranger to allow Wait to be called and to unblock it immediately because the logic did not yet reach the point of starting to goroutine. If the background activity won't be started, return an error rather than allowing concurrent code to manipulate the tomb that will never have background activity related to it.


neon: implementation for mul_n and mull_n

Not much to say about this one either.
(CC @tlively @ngzhian)

TBH I don't think I knew these existed.  NEON is really nice… I don't even want to think about how many times I've written _mm_mul_ps(a, _mm_set1_ps(b)).
Great job on the patch, I'll merge in tonight, thanks!


Release and publish

The library looks promising. Are you planning to publish it on the Maven Central?

At this very moment it's not production ready. I need some time to complete
the work and then to publish to the repo.
чт, 17 сент. 2020 г., 00:25 PawelJ-PL<EMAIL_ADDRESS>
The library looks promising. Are you planning to publish it on the Maven
Central?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dobrynya/zio-jms/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFCKS4J5MBGRY6O26BEIXDSGEULFANCNFSM4RPN4G3Q
.

@PawelJ-PL Hi Pawel
I've published this library to Bintray repo. Please take a look at README.


Module reference to manifest

Added possibility to specify terraform modules version to be cloned for a deployment in a manifest:

manifest:
version: cf1ac826168e546453f0b5a45f57f5294b2b5cc1
modules:
- s3
- vpc
- eks_cluster

Closing stale PR


Crud operations

thank you dear @OLIPLICHE ! unfortunatelly, this is due to Linters. it was working correctly but when formatting code and follow eslint rules ; some blocks of the code were no longer working correctly.
I fixed them. thanks for checking.
everything is working ✅ 💯


NullPointerException in SerializerPojo when serializing POJOs on JRockit

Several JRockit implementations of Java 6 have a caching bug where ObjectStreamClass.lookup(...) may return a non-null value when invoked with a non-Serializable class. This makes SerializerPojo throw a NPE when looping in makeFieldsForClass(...), as (according to the specification) the loop expects ObjectStreamClass.lookup(Object.class) == null. Changing the two invocations to ObjectStreamClass.lookup(...) in that method to invocations to the following:
    private static ObjectStreamClass fixForLookup(Class<?> c) {
        ObjectStreamClass result = null;
        if (Serializable.class.isAssignableFrom(c)) {
            result = ObjectStreamClass.lookup(c);
        }
        return result;
    }

seems to "sort-of solve" the issue (though I suspect the stream format will not be spec-compliant). I have tested this with JRockit R28.2.7, which is the latest publicly available JRockit release.
Ideally this should be corrected in JRockit itself and not worked around in MapDB. The problem is that JRockit has been discontinued after Java 6, and getting patches for it is not straightforward at all. Using MapDB in Jrockit is very uncomfortable without the "fix": either you restrict yourself to types with built-in serializer implementations in MapDB, or you provide custom serializers for all your POJOs. You cannot rely on SerializerPojo because of this problem with ObjectStreamClass.lookup(...).

A link describing the JRockit issue:
https://community.oracle.com/thread/2298350

I will look into it, thanks for raising it.

Not going to fix it, sorry.
I feel restricting lookup to Serializable classes might be too dangerous and affect other cases.

Thank you anyway, I see now a warning is logged, that bug is a bit nasty to
find out...
2015-09-30 12:13 GMT+02:00 Jan Kotek<EMAIL_ADDRESS>
Closed #572 https://github.com/jankotek/mapdb/issues/572.
—
Reply to this email directly or view it on GitHub
https://github.com/jankotek/mapdb/issues/572#event-422895684.


State color change is disabled.

Hi,
I am trying to change the color of "Resolved" state in all work items but Azure DevOps is not allowing to change the color. It seems to be disabled or locked.
Thanks
Sachin

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

ID: 715eca62-8a31-4163-c27b-f5e71d781982
Version Independent ID: 06dc711f-1946-29e9-aece-75af903da09b
Content: Customize the workflow of an inherited process - Azure DevOps Services
Content Source: docs/organizations/settings/work/customize-process-workflow.md
Product: devops
Technology: devops-agile
GitHub Login: @KathrynEE
Microsoft Alias: kaelli

@sachinzen  Thank you for the feedback.
Unfortunately, default states are non-editable and you will need to use a custom state instead.
For product team suggestions, please use Suggest a Feature in the Azure DevOps community.  You can use this post for reference.

Closing the issue now, please comment below if needed.


Info about visualization tool

Hi! really nice work, I particularly like the idea of driving manipulation with AI generated images

As depicted in the DinoV2 repo https://github.com/facebookresearch/dinov2/issues/23#issuecomment-1540475277 when we have a bunch of images of the same class to visualize similar parts with the same color it's needed to put them in the same "batch" and perform pca on the flattenized tensor (N x f), with f feature size
So to visualize the Goal image with the same color pattern of the point cloud you needed to perform PCA of the goal together with the other images or did u manage to achieve this directly (and how?)
Thanks in advance for your time!

@WangYixuan12 thanks for your reply! And on how many instances have you used to compute the pca for one class?

I just use 4 images. I could upload PCA precomputation code later today.

Ah that's great thank you!


psize clarification

It is not clear if the psize argument will corresponds to the reserved size for both pipe direction or only one direction.

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

ID: 2b68d264-1456-94ff-2898-557567cde992
Version Independent ID: 4778fdc8-20b7-4f84-52ff-27201c4deb0c
Content: _pipe
Content Source: docs/c-runtime-library/reference/pipe.md
Product: visual-cpp
Technology: cpp-ucrt
GitHub Login: @corob-msft
Microsoft Alias: corob

@plangevin
Thanks for asking. The psize parameter is passed directly to the operating system, as a suggested buffer size for the pipe. There's only one memory buffer for a pipe, whether you're reading from it or writing to it. It wouldn't make sense to have more space to write to than you can read from, or vice-versa. I hope that clarifies things for you.


Document how to run the server privately

Also, possibly look into ways of limiting traffic on a server to only the site "allowed" to use it possibly with referrer headers).


Support initialization of an arbitrary-backend Tensor from a numpy array or nested list

Numpy array should always work given you use backend.convert_to_tensor. Nested lists behave differently for each backend so that might be harder.

Fair enough. Probably just from an array is fine.

Hey,
Is this issue still still open?

Yep, all yours. I think we aren't going to support nested lists, but it would be good if initialization in an arbitrary backend could be done from a numpy array:
A = np.zeros(shape, dtype) At = tn.Tensor(A, backend="numpy") # produces a tn.Tensor with the numpy backend At_tf = tn.Tensor(A, backend="tensorflow") # produces a tn.Tensor with the TensorFlow backend

@alewis Hey, I tried running the snippet in the last comment and it seems to work fine, did someone already write this?


Traefik ingress not accessible from host IP

Describe the bug
A fresh install of k3s on a single node on a private network with node addresses on the <IP_ADDRESS>/24 range. Dialing the node address <IP_ADDRESS> on port 80 fails to connect to Traefik.
To Reproduce

install k3s
try to connect to Traefik

Expected behavior
curl http://192.168.5.x should work
Screenshots
$ curl -sv http://<IP_ADDRESS>
*   Trying <IP_ADDRESS>...
* TCP_NODELAY set
* connect to <IP_ADDRESS> port 80 failed: No route to host
* Failed to connect to <IP_ADDRESS> port 80: No route to host
* Closing connection 0

Additional context
I could make it work on another testing host where a public routable IP address was directly assigned to the host. But on this new install, I can't make it work, and I don't understand why.
svclb-traefik-* pod seems to work:
[root@elyas elyas]# kubectl logs -n kube-system svclb-traefik-rfd9q lb-port-80
+ trap exit TERM INT
+ cat /proc/sys/net/ipv4/ip_forward
+ '[' 1 '!=' 1 ]
+ iptables -t nat -I PREROUTING '!' -s <IP_ADDRESS>/32 -p TCP --dport 80 -j DNAT --to <IP_ADDRESS>:80
+ iptables -t nat -I POSTROUTING -d <IP_ADDRESS>/32 -p TCP -j MASQUERADE
+ '[' '!' -e /pause ]
+ mkfifo /pause

[root@elyas elyas]# iptables -n -t nat -L | grep <IP_ADDRESS>
KUBE-MARK-MASQ  tcp  -- !<IP_ADDRESS>/16         <IP_ADDRESS>          /* kube-system/traefik:http cluster IP */ tcp dpt:80
KUBE-SVC-X3WUOHPTYIG7AA3Q  tcp  --  <IP_ADDRESS>/0            <IP_ADDRESS>          /* kube-system/traefik:http cluster IP */ tcp dpt:80
KUBE-MARK-MASQ  tcp  -- !<IP_ADDRESS>/16         <IP_ADDRESS>          /* kube-system/traefik:https cluster IP */ tcp dpt:443
KUBE-SVC-IKNZCF5XJQBTG3KZ  tcp  --  <IP_ADDRESS>/0            <IP_ADDRESS>          /* kube-system/traefik:https cluster IP */ tcp dpt:443

traefik itself is working:
[root@elyas elyas]# curl -sv http://<IP_ADDRESS>
*   Trying <IP_ADDRESS>...
* TCP_NODELAY set
* Connected to <IP_ADDRESS> (<IP_ADDRESS>) port 80 (#0)
> GET / HTTP/1.1
> Host: <IP_ADDRESS>
> User-Agent: curl/7.64.0
> Accept: */*
>
< HTTP/1.1 404 Not Found
< Content-Type: text/plain; charset=utf-8
< Vary: Accept-Encoding
< X-Content-Type-Options: nosniff
< Date: Tue, 02 Jul 2019 21:09:58 GMT
< Content-Length: 19
<
404 page not found
* Connection #0 to host <IP_ADDRESS> left intact

Can the fact that the node is not assigned a public IP address be the reason?

It does not work any better when connecting to localhost:
[root@elyas elyas]# curl -sv http://<IP_ADDRESS>
*   Trying <IP_ADDRESS>...
* TCP_NODELAY set
* connect to <IP_ADDRESS> port 80 failed: No route to host
* Failed to connect to <IP_ADDRESS> port 80: No route to host
* Closing connection 0

While it should normally work

No route to host sounds like some sort of networking setup issue.
Would you be able to provide the output of ip a and ip route?

There was a system firewall enabled on this host. Sorry about the noise.
To debug it, I followed the packet generated using curl <IP_ADDRESS> using iptables -v -n -t nat -L PREROUTING to all the chains, until it arrives via DNAT to a container, then done the same within the container network namespace (nsenter -t PID --net) until I found a DNAT rule outside the container. Ten executed curl  <IP_ADDRESS> (the DNAT target) within the container and followed the packet back outside the container. PREROUTING seemed to be fine but I found in iptables -v -n -t filter -L FORWARD a DROP target that was followed. And found chain names relative to the system firewall. That triggered the clue.


New build scripts

About the PR
Better build scripts, batch ones have been tested
Shell ones need testing (@CharcoalGungan) but should be mostly fine

it be funny as all hell i approve my own changes : )

I am going to implement the "latest one to push cannot approve changes" rule just because of this

Awaiting


Unable to pass string parameter on toast.tvwindow.show

Hi folks,
There is a type verification that only support integers when calling toast.tvwindow.show.
So if I have this code would not work on TOAST:
var rect =   ['50%', '50%', '50%', '540px']
tizen.tvwindow.show(function() {}, function() {}, rect, 'MAIN');

toast.tvwindow.show should be able to allow string to indicate units or even accept an integer on a string ('0').

Hi
thanks for your comment and suggestion.
TOAST API spec is a bit defferent from tizen.
Unfortunately, toast.tvwinodw.show do not currently support string.
It only support number units parameter and display with pixel(px) in screen.
We will consider support it.
Thank you for reading.


crypto/internal/boring: move boring.Cache to a separate package

Cl 395882 added a GC-friendly runtime-backed concurrent map to the boring package which is being used in several crypto packages.
We (Microsoft) are patching Go to add support for OpenSSL as crypto backend at microsoft/go (github.com), and it would be convenient for us, and also for other companies which forked Go for the same purpose, to be able to use crypto/internal/boring.Cache without importing the whole boring package.
It worth nothing that we could also implement boring.Cache in our fork, but that would also mean patching runtime/mgc.go, which I would like to avoid in order to reduce conflict likeliness.
Would it be possible to move crypto/internal/boring/cache.go to a new package, i.e., crypto/internal/boring/cache.
@rsc @FiloSottile

Would this be satisfied by https://go.dev/cl/407135 ?  That change is approved but not yet submitted.

Would this be satisfied by https://go.dev/cl/407135 ? That change is approved but not yet submitted.

This is exactly what I was asking for, thanks!!

Updated the milestone, although we don't usually try to maintain milestone for closed issues.


Boot directly into last opened project or into a new project

Would need to have a persistent setting that stores the currently open project.
Also because of potentially long time it takes to load larger samples from sdcard into ram and the fact that the sdcard may even have been removed or swapped out the project needs to be saved into flash and the previously loaded samples can then just be kept in flash and used on boot (great idea for this from @democloid ).

done as of #219


Generate incorrect backlink URLs for GitHub subdomain user

Thank you for sharing this! I enjoy it a lot, only one issue of backlink URLs came with me.
I am using the GitHub subdomain, so the backlink URLs should be https://<user name>.github.io/quartz/<note>, but https://<user name>.github.io/<note> is generated, missing /quartz/ part. I believe this issue is caused by layouts/partials/backlinks.html because URLs in other partials are all correct.
To reproduce this issue, I forked this repository and left everything unchanged, except the baseURL in config.toml and the cname in .github/workflows/deploy.yaml. Here are the page and repository links:
https://qilong-kirov-liu.github.io/quartz/
Qilong-Kirov-LIU/quartz
Would you be so kind to follow up on this? Thanks indeed!

Hi there, thanks for filing an issue! This should be fixed in this commit: 5234fae080f1d3dca4f105438e3e0151f12ca61c


make error when use docker

Please use this template for reporting suspected bugs or requests for help.
Issue Description
Scanning dependencies of target OpenrCtrl-cpp2-target
[ 32%] Generating OpenrCtrl files. Output: /home/openr/build/thrift-out/openr/if
[ 32%] Built target OpenrCtrl-cpp2-target
Scanning dependencies of target OpenrCtrl-cpp2-obj
[ 32%] Building CXX object CMakeFiles/OpenrCtrl-cpp2-obj.dir/thrift-out/openr/if/gen-cpp2/OpenrCtrl_constants.cpp.o
In file included from /home/openr/build/thrift-out/openr/if/gen-cpp2/OpenrCtrl_constants.h:11:0,
from /home/openr/build/thrift-out/openr/if/gen-cpp2/OpenrCtrl_constants.cpp:8:
/home/openr/build/thrift-out/openr/if/gen-cpp2/OpenrCtrl_types.h:16:46: fatal error: openr/if/gen-cpp2/Decision_types.h: No such file or directory
compilation terminated.
make[2]: *** [CMakeFiles/OpenrCtrl-cpp2-obj.dir/thrift-out/openr/if/gen-cpp2/OpenrCtrl_constants.cpp.o] Error 1
Environment
ubuntu 18.04 gcc 7.4
Minimal test code / Steps to reproduce the issue
What's the actual result?
What's the expected result?
build success

This error also happens when I make without docker.
[ 23%] Building CXX object CMakeFiles/OpenrCtrl-cpp2-obj.dir/thrift-out/openr/if/gen-cpp2/OpenrCtrl_constants.cpp.o
In file included from /home/yy/openr/build/thrift-out/openr/if/gen-cpp2/OpenrCtrl_constants.h:11:0,
from /home/yy/openr/build/thrift-out/openr/if/gen-cpp2/OpenrCtrl_constants.cpp:8:
/home/yy/openr/build/thrift-out/openr/if/gen-cpp2/OpenrCtrl_types.h:17:10: fatal error: openr/if/gen-cpp2/Dual_types.h: No such file or directory
#include "openr/if/gen-cpp2/Dual_types.h"
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
CMakeFiles/OpenrCtrl-cpp2-obj.dir/build.make:102: recipe for target 'CMakeFiles/OpenrCtrl-cpp2-obj.dir/thrift-out/openr/if/gen-cpp2/OpenrCtrl_constants.cpp.o' failed

Hello @silveric10 - can you try build again with latest code, we had some changes and fixes in cmake dependencies that will resolve your issue.

Hello @silveric10 - can you try build again with latest code, we had some changes and fixes in cmake dependencies that will resolve your issue.

Hi @saifhhasan , your update still doesn't work, and I want to tell you that the build script "build_openr.sh" also has a bug.
install_openr() {
pushd .
cd "$BUILD_DIR"
cmake
-DBUILD_SHARED_LIBS=OFF
-DBUILD_TESTS=ON
-DADD_ROOT_TESTS=ON
-DCMAKE_CXX_FLAGS="-Wno-unused-parameter -fPIC"
../openr/  //There is no "CMakeLists.txt" in this directory, I think it should be “..” not "../openr"
By the way, the compile error is the same.
[ 13%] Built target OpenrCtrl-cpp2-target
Scanning dependencies of target OpenrCtrl-cpp2-obj
[ 13%] Building CXX object CMakeFiles/OpenrCtrl-cpp2-obj.dir/thrift-out/openr/if/gen-cpp2/OpenrCtrl_constants.cpp.o
In file included from /home/yy/openr/thrift-out/openr/if/gen-cpp2/OpenrCtrl_constants.h:11:0,
from /home/yy/openr/thrift-out/openr/if/gen-cpp2/OpenrCtrl_constants.cpp:8:
/home/yy/openr/thrift-out/openr/if/gen-cpp2/OpenrCtrl_types.h:17:10: fatal error: openr/if/gen-cpp2/Decision_types.h: No such file or directory
#include "openr/if/gen-cpp2/Decision_types.h"
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.

Hi, once I close #56 I'll try make the docker builds all work again and will update when I feel they are working.

Please try the new Dockerfile


Prepend X-Original-To with destination data

Outlook seems to omit the BCC recipient's emails from the To: header, therefore it is not possible to easily get the SES email that the message was sent to.
The solution I found was to extract the recipient from the SNS payload - https://docs.aws.amazon.com/ses/latest/DeveloperGuide/notification-contents.html#mail-object
This solution was inspired by a PR for the ActionMailbox Mailgun integration, which had a similar issue - https://github.com/rails/rails/pull/38738

Hi, @dnamsons , thanks a lot for the PR. The concept looks sound to me but I would like to a) make a couple of tiny tweaks (just minor refactoring, no change to functionality), b) leave this conversation open for a couple of days in case anybody else has some input here.
 I would also like to have a regression test added for this specific case - if you are able to add that then that would be perfect, otherwise I can try to put something together (in which case an example JSON document would be helpful so I could use it as a fixture).
Either way, I see no problem getting this functionality added to the gem and I can do a release shortly after merging the final PR.
Thanks again for taking the time to look at this and for finding a solution.

Hey @bobf! Thank you for the response!
I will write a test for this feature and I am also open to improvements to the code.

Hi, @dnamsons , sorry it's taken me so long to get round to this, I've been unwell the last week but I am doing much better now so I will reserve some time this weekend to give you some feedback. The changes to the code I was looking at are really minor so I think the easiest way to do it is for me to add a commit to this PR with my proposed changes and you can let me know if you disagree with them. We'll get this PR merged soon for sure.

@dnamsons I've pushed a branch to the main repository which just does a quick refactor of the changes you made:
https://github.com/bobf/action_mailbox_amazon_ingress/commit/d66a0b2c5796f2d313a170752c8366d67bc065d4
I believe it to be functionally equivalent to your code and I did not change any of the logic you implemented, I just moved it into a few smaller methods.
Could you please add this code to your PR (feel free to suggest any improvements if you see any) and, if possible, write a test to cover the new functionality ?
If so, I am happy to merge your changes into master and push a new release of the gem. If the test is too awkward to write then I guess we can live without it but i would always prefer a test if it is possible to add one.
Thanks again for your contribution - we should be able to get this wrapped up pretty soon now. I'm happy to support you if you have any questions.

@dnamsons Is this one ready to go now ? Thanks for the recent updates to the PR.

@bobf Yes, this is done. I added a test case to ensure that an email will be extracted from the SNS payload.
I didn't add a new email fixture file because there is no real visible difference - Outlook sends the email directly to each BCC recipient, omitting the BCC data.

@dnamsons Thanks a lot for these changes. I've merged the PR and will do a release in the next few days if you don't make any more changes. :+1:


[NNBD] No static warning for overriding parameter default value

According to the Dart specification (Version 2.8.0-dev, April 22, 2020) 11.2.2 Correct Member Overrides

• If m and m0 are both methods, p is an optional parameter of m, p0 is the parameter of m0
corresponding to p, p has default value d and p0 has default value d0, then d and d0 must be identical, or a static warning occurs

It's true for non-NNBD, but not so when analyzer is run with --enable-experiment=non-nullable option. In this case there is no warning at all
class A {
  foo([x = 1]) {}
}

class C extends A {
  foo([x]); /// static type warning
}

main() {
  new C().foo(1);
}

Analyzer with --enable-experiment=non-nullable finds no issues in the code above
Please see also https://github.com/dart-lang/sdk/issues/34437
dartanalyzer version 2.9.0-13.0.dev

I believe this WAI, because C.foo is abstract.
It was updated in 85c197a848a3083f2870e0e448d3603ec37cf073, as clarified by @eernstg here.

Right, an abstract method declaration with an optional parameter that does not specify a default value does not have a default value, so there is no conflict.


Cap java 2.0 for tutorial

Both services are created with Java version set to Java 17
The manifest.yaml is changed so it is now specifies Java 17, disables Spring Boot Auto Configuration (buildpack is quite loud about it being deprecated) and explicitly sets the cloud profile for Spring Boot.
xs-security.json is updated with "oauth2-configuration". In default data center offered for trial accounts it is mandatory.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 3 committers have signed the CLA.:white_check_mark: renejeglinsky:x: vmikhailenko:x: rjayasingheYou have signed the CLA already but the status is still pending? Let us recheck it.


Blockchain tip not set in REST context

Context
Unless I'm missing some configuration changes, it appears that the latest two cardano-wallet versions don't work paired with jormungandr. Either with existing or wiped database (chain folder).

Information
-

Version
v2020.5.6 (git revision: cf3d9c91e895d1a01f90e4ab3d10b81e260b3510) with jormungandr v0.8.18 AND v2020-04-28 with jormungandr v0.8.15 or v0.8.18

Platform
Linux the least

Installation
From Github Release

Steps to Reproduce

$ cardano-wallet launch --node-port 3100 --genesis-block-hash $(cat genesis-hash.txt) -- --config ./stakepool-config.yaml
Above referenced files exist and are valid, as previous versions of cardano-wallet and jormungandr v0.8.15 and v0.8.18 on their own run well with them.
Expected behavior
Jormungandr bootstraps successfully and cardano-wallet starts communicating with it.

Actual behavior
Fails as follows:
[cardano-wallet.main:Info:4] [2020-05-08 23:01:04.58 UTC] Running as v2020.5.6 (git revision: cf3d9c91e895d1a01f90e4ab3d10b81e260b3510)
[cardano-wallet.main:Info:4] [2020-05-08 23:01:04.58 UTC] Command line: /home/thevictor/.local/bin/cardano-wallet launch --node-port 3100 --genesis-block-hash 8e4d2a343f3dcf9330ad9035b3e8d168e6728904262f2c434a4f8f934ec7b676 -- --config ./stakepool-config.yaml
[cardano-wallet.main:Info:4] [2020-05-08 23:01:04.58 UTC] LaunchArgs {_hostPreference = Host "<IP_ADDRESS>", _listen = ListenOnPort 8090, _nodePort = Just 3100, _stateDir = Nothing, _syncTolerance = SyncTolerance 300s, _logging = LoggingOptions {loggingMinSeverity = Debug, loggingTracers = Tracers {applicationTracer = Const (Just Info), apiServerTracer = Const (Just Info), walletEngineTracer = Const (Just Info), walletDbTracer = Const (Just Info), stakePoolEngineTracer = Const (Just Info), stakePoolDbTracer = Const (Just Info), networkTracer = Const (Just Info), ntpClientTracer = Const (Just Info)}, loggingTracersDoc = Nothing}, _jormungandrArgs = JormungandrArgs {genesisBlock = Left (Hash {getHash = "\142M*4?=\207\147\&0\173\144\&5\179\232\209h\230r\137\EOT&/,CJO\143\147N\199\182v"}), extraJormungandrArgs = ["--config","./stakepool-config.yaml"]}}
[cardano-wallet.main:Info:4] [2020-05-08 23:01:04.58 UTC] Wallet state: Using directory: /home/thevictor/.local/share/cardano-wallet/jormungandr/testnet
[cardano-wallet.main:Info:4] [2020-05-08 23:01:04.58 UTC] Wallet databases: Using directory: /home/thevictor/.local/share/cardano-wallet/jormungandr/testnet/wallets
[cardano-wallet.application:Info:4] [2020-05-08 23:01:04.58 UTC] Wallet backend server starting. Launching Jörmungandr...
[cardano-wallet.application:Info:4] [2020-05-08 23:01:04.58 UTC] Node is Jörmungandr on testnet (0)
[cardano-wallet.network:Notice:4] [2020-05-08 23:01:04.58 UTC] Starting process jormungandr
     --genesis-block-hash 8e4d2a343f3dcf9330ad9035b3e8d168e6728904262f2c434a4f8f934ec7b676
     --rest-listen <IP_ADDRESS>:3100
     --storage /home/thevictor/.local/share/cardano-wallet/jormungandr/testnet/chain
     --config ./stakepool-config.yaml

[cardano-wallet.network:Info:4] [2020-05-08 23:01:04.58 UTC] [jormungandr.8657] Process started
May 08 23:01:04.594 INFO Starting jormungandr 0.8.18 (HEAD-690eb547, release, linux [x86_64]) - [rustc 1.42.0 (b8cedc004 2020-03-09)], task: init
May 08 23:01:04.594 WARN Node started without path to the stored secret keys (not a stake pool or a BFT leader), task: init
May 08 23:01:04.595 INFO storing blockchain in '"/home/thevictor/.local/share/cardano-wallet/jormungandr/testnet/chain/blocks.sqlite"', task: init
May 08 23:01:05.259 INFO loading from storage, currently at d79f2082-00000001-0.530 processing=0ns (0ns per block) ..., task: bootstrap
[cardano-wallet.network:Info:12] [2020-05-08 23:01:05.59 UTC] Waiting for Jörmungandr to be ready on tcp/3100
[cardano-wallet.network:Notice:12] [2020-05-08 23:01:05.59 UTC] [jormungandr.8657] Terminating child process
[cardano-wallet.main:Debug:4] [2020-05-08 23:01:05.59 UTC] Logging shutdown.
cardano-wallet: ErrUnexpectedNetworkFailure (Link {_segments = ["api","v0","tip"], _queryParams = []}) (FailureResponse (Request {requestPath = (BaseUrl {baseUrlScheme = Http, baseUrlHost = "<IP_ADDRESS>", baseUrlPort = 3100, baseUrlPath = ""},"/api/v0/tip"), requestQueryString = fromList [], requestBody = Nothing, requestAccept = fromList [text/plain;charset=utf-8], requestHeaders = fromList []), requestHttpVersion = HTTP/1.1, requestMethod = "GET"} (Response {responseStatusCode = Status {statusCode = 500, statusMessage = "Internal Server Error"}, responseHeaders = fromList [("content-length","38"),("content-type","text/plain; charset=utf-8"),("date","Fri, 08 May 2020 23:01:05 GMT")], responseHttpVersion = HTTP/1.1, responseBody = "Blockchain tip not set in REST context"}))

Resolution

QA

Thanks for the report.
I can reproduce this error locally with the itn_rewards_v1 testnet but not the jormungandr self node. So that would explain why the jormungandr integration tests are still passing.

I recall a configuration change in Jörmungandr  regarding the API prefix. It used to be included in the configuration but was later removed and assumed to always be "/api" and implicitly required. Could it be linked?

If I am not mistaken the issue is that currently during Jormungandr bootstrap when you ask it for a network tip the response is 500.
Start Jorm:
jormungandr \
     --genesis-block-hash 8e4d2a343f3dcf9330ad9035b3e8d168e6728904262f2c434a4f8f934ec7b676 \
     --rest-listen <IP_ADDRESS>:8080 \
     --storage ./data_incentivized/chain \
     --config node_config_incentivized.yaml

Check status:
GET http://<IP_ADDRESS>:8080/api/v0/node/stats

v

Response body:
{"version":"jormungandr 0.8.19-776347cf","state":"Bootstrapping"}

While it is boostrapping, try to get the tip:
GET http://<IP_ADDRESS>:8080/api/v0/tip"

v

failed to make a REST request
  |-> node internal error
  |-> HTTP status server error (500 Internal Server Error) for url (http://<IP_ADDRESS>:8080/api/v0/tip)

And I suppose this is what is happening, we are asking for tip while Jormungandr is still bootsrapping and it returns 500 then.
Probably here -> https://github.com/input-output-hk/cardano-wallet/blob/master/lib/jormungandr/src/Cardano/Wallet/Jormungandr/Network.hs#L277, instead of asking for tip make sure that Jormungandr is no longer in Boostrapping state.

Doh ... it keeps changing. The endpoint used to return HTTP status 503 during bootstrap. But before that I think it did return 500 - however now we consider 500 to be a fatal error.
I don't think it's right to check the /api/v0/node/stats endpoint. The thing we need to succeed is /api/v0/tip. And there shouldn't be a reason why this endpoint can't work during bootstrap.

/api/v0/node/stats is a bit misleading name I guess... But it actually shows the state of Jorm and, as far as I see, currently on  state:"Boostrappting" pretty much all other endpoints return 500 (except for /api/v0/node/stats)... so, currently it seems to be the only valid endpoint to query for the state.
Well, the alternative is to raise error in Jormungandr and desrcibe what would we expect. :man_shrugging:


Open content editor to a specific entry by path

This is a feature request.
I use jekyll to generate my pages and would like to have buttons with the link to edit the site.
For github, I use:
via Github

This works nice, because I can easily access the page.path
I would like to use my-domain.com/admin/#/edit/{{page.path }} for Netlify. This is currently not supported.

The end result would be great, but entries don't map 1:1 with the resulting file path. We'd need to consider how best to do this.

The problem is that netlify cms does not accept a path, but needs the name of a collection defined in the config and the slug defined in the file.

Hmm we could transform the url you're proposing, come to think of it. Given {cmsUrl}/edit/{filePath}/{fileName}:

Use a catch-all route for the edit url segment
Find a file entry in the config with a path that matches {filePath}/{fileName}
If no file matches, find a folder collection in the config whose directory matches {filePath}
If a folder collection matches, search the entries in the backend for a file name that matches { fileName }
If no matches found, 404

Sent with GitHawk

This would be nice. I'll be very busy until the end of the year, but maybe I can contribute later.

Hey, it's good to be busy! Most of the time, anyway lol. Sounds great :)

Hello, I am new to the Netlify-CMS community and am interested in taking on this ticket. Is it currently available? Thanks!

They @RoObsessedCode, cool you find your way to this ticket. Please take it!

@RoObsessedCode welcome! It's all you my friend :)

My first question for you @rriemann @erquhart (Hope it is okay if I post my queries here).
Right now, the way it is, collection name is needed in the file path so suppose you have these 2 urls(for this example lets assume pages and blogs are two collections)

{cmsUrl}/edit/blog/2

{cmsUrl}/edit/page/2

If we refactored to eliminate the collection name we would end up with the same path but they need to be different to represent two different views
Both would end up as: {cmsUrl}/edit/2
Do you recognize the conflict I am proposing? How do you suggest to work around this issue?

In my setup, every resource has only one view. We could just define that if a person has access to more than one view/collection to edit the same file, the person is either presented a choice dialogue or the link must contain the collection in the URL fragment.
I see another problem. I think the CMS uses not the filename (or part of it) like github, but instead a token (can someone confirm?). In some cases, this may be equivalent, but this is not true in all cases.

@RoObsessedCode we wouldn't change the URL structure - we'd just parse the edit path and redirect to the real one. @rriemann it won't actually matter what identifier Netlify CMS uses, we'll use the filename to query the backend for folder collections. Might be good to do that for both file and folder collections just for consistency.

Thanks @erquhart. I am in the process of seeing how everything works together and how exactly to implement the steps you listed above. I will keep the team updated. Thanks!

Feel free to reach out for help in Gitter, great community there.

@RoObsessedCode Let us know if you need any help on this!

Considering this no longer in process.

Closed in #2556


Release on crates.io?

Is there a reason why this crate is not released on crates.io?

On one hand, we're still doing some fundamental modifications to the code, so the interface is far from stable. I know that 0.x versions don't guarantee any sort of stability, but we would be releasing new versions way too often. There would've been no advantage to labeling releases with semver.
Another issue is the name of the package. A lot of the good names have been already taken.

Thanks for the clarification!
About the name: The uefi crate wasn't updated since mid 2017, maybe the author is willing to transfer the name.

+1 for publishing unstable versions

On one hand, we're still doing some fundamental modifications to the code, so the interface is far from stable.

https://github.com/rust-osdev/uefi-rs/graphs/code-frequency
Is this still the case after one year of development?
Also, releasing this crate will automatically make documentation available on docs.rs which is really handy for development, as currently you have to build it manually on each development machine.

@edigaryev I agree that the core API is more-or-less stable and I'd love to have easily accessible docs too.
But these are just excuses, it's more my laziness that's to blame :/
I've contacted the person who owns the uefi name on crates.io, and hopefully they will give us the name. Otherwise, I will start thinking about new names for the crate and work towards publishing it.

Great news! The author of the other crate replied and provided us the name. We're now published on crates.io! And the docs.rs page works too!
As part of this, I've eliminated the uefi_alloc and uefi_logger crate and instead made them into optional features of the core crate. This will simplify publishing / dependencies and I see no potential downsides. Instead of uefi_alloc:: the users now have to use uefi::alloc:: (and enable the option in Cargo.toml). It gives me the idea to keep on simplifying and possibly remove the exts and services crates too.


⚠ Bofrost.fr has degraded performance

In c1177e6, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1606 ms

Resolved: Bofrost.fr performance has improved in 3fefd21 after 6 minutes.


const_fn has been removed in latest nightly

In this PR const_fn is removed.
Replace const_fn with const_fn_trait_bound works:
#![cfg_attr(feature = "nightly", feature(const_fn_trait_bound))]

Sorry, did not notice the fix commit.


MQTT-SN IPv6: wrong family in zsock_socket

Describe the bug
The File
https://github.com/zephyrproject-rtos/zephyr/blob/main/subsys/net/lib/mqtt_sn/mqtt_sn_transport_udp.c#L51
contains the line:
udp->sock = zsock_socket(PF_INET, SOCK_DGRAM, 0);

in order to support IPv6 PF_INET needs to replaced by PF_INET6. Assuming AF_INET == PF_INET && AF_INET6 == PF_INET6 a solution could be:
udp->sock = zsock_socket(udp->gwaddr.sa_family, SOCK_DGRAM, 0);

Would that be a good solution according to your coding style? Than I could try to prepare a pull request for this...

udp->sock = zsock_socket(udp->gwaddr.sa_family, SOCK_DGRAM, 0);

Would that be a good solution according to your coding style? Than I could try to prepare a pull request for this...

Yes, this was my first thought as well, the AF for socket should be set according to the destination address provided. Please send a PR.

Hi, from what I understand similar bug is introduced here:
https://github.com/zephyrproject-rtos/zephyr/blob/main/subsys/net/lib/mqtt_sn/mqtt_sn_transport_udp.c#L64

Hi, from what I understand similar bug is introduced here:
https://github.com/zephyrproject-rtos/zephyr/blob/main/subsys/net/lib/mqtt_sn/mqtt_sn_transport_udp.c#L64

Well, technically sockaddr_in/sockaddr_in6 structures have the same layout when it comes to family/port, so I'd rather call this one an oversimplification rather than a bug, it still should work fine. However, I'm thinking it'd be cleaner if port was also provided by the get_ip_str() helper.

Also there is a bug in the example project:
https://github.com/zephyrproject-rtos/zephyr/blob/main/samples/net/mqtt_sn_publisher/src/udp.c#L119

This indeed looks wrong.

Also there is a bug in the example project:
https://github.com/zephyrproject-rtos/zephyr/blob/main/samples/net/mqtt_sn_publisher/src/udp.c#L119

This indeed looks wrong.

How would you fix this one? Adding an additional CONFIG flag or using zsock_getaddrinfo() with AI_NUMERICHOST hint?

Also there is a bug in the example project:
https://github.com/zephyrproject-rtos/zephyr/blob/main/samples/net/mqtt_sn_publisher/src/udp.c#L119

This indeed looks wrong.

How would you fix this one? Adding an additional CONFIG flag or using zsock_getaddrinfo() with AI_NUMERICHOST hint?

It's just a matter of wrong error code check. As @napent already mentioned: inet_pton() returns 1 on success - hence, the check should compare to 1, not 0.


Possible to get API from Stravistix?

Hi, I was wondering if it's possible to get an API from Stravistix for the Fitness Trend graph? I'd like to build a combined view of this against my weight loss from Fitbit.

It's local to your browser. You can get data with:
chrome.storage.local.get((d) => {
     console.log(d);
});

Hmm. This is doesn't seem to work in Chrome console. The only things I see in a local storage are a couple of parameters for the fitness trend graph like fitnessTrend_EBikeRidesEnabled.

Strange. Is your history synced?


General Cleanup

Is this pull request associated with an issue(s)?
No.
Description
This PR is intended to generally clean up the repository to prepare it for publication. This includes proofreading documentation, creating examples, and completing TODOs remaining in the code.
TODOs

[x] Proofread existing documentation.
[x] Create end-to-end examples for users and developers to get started with.
[ ] Complete outstanding TODOs.

@ryanmrichard This PR is the current progress of cleaning everything up for publication. There are a few TODOs still left in the documentation for me to complete, but I think this is close enough to start being reviewed. I think the parts that need the most scrutiny right now are docs/source/getting_started/example_project.rst and docs/source/getting_started/initial_setup.rst.

@zachcran I won't have a chance to look at this until tomorrow.

@ryanmrichard This PR should be r2g after the CMakeTest issues are resolved in CMakeTest PR #65.


Can't debug main process on ubuntu

hello
After yarn watch，I Want to debug the main process，Operation as shown in the picture.
It was successful in MacOS, but failed in Ubuntu， In Ubuntu, breakpoints are not entered.
How to debug in Ubuntu? thanks

How are you debugging? Which launch configuration (in launch.json) are you using?

How are you debugging? Which launch configuration (in launch.json) are you using?
this one

"name": "Main Process",

/jsDebugLogs

It looks like the main process doesn't start for you, the "call stack" view is empty. Make sure that VS Code is correctly set up and that you can launch it outside of the debugger.

Please collect trace logs using the following instructions:

If you're able to, add "trace": true to your launch.json and reproduce the issue. The location of the log file on your disk will be written to the Debug Console. Share that with us.
warning This log file will not contain source code, but will contain file paths. You can drop it into https://microsoft.github.io/vscode-pwa-analyzer/index.html to see what it contains. If you'd rather not share the log publicly, you can email it to<EMAIL_ADDRESS>

Hello, This is the logs
vscode-debugadapter-4a308ed7.json.gz

It looks like the main process doesn't start for you, the "call stack" view is empty. Make sure that VS Code is correctly set up and that you can launch it outside of the debugger.

It can be started successfully, but the main process cannot be debugger (it can be on Mac OS, but not on Ubuntu)

/jsDebugLogs

this file?

Verbose logs are written to: /tmp/vscode-debugadapter-4a308ed7.json.gz

vscode-debugadapter-4a308ed7.json.gz

Our team had a similar sounding issue this week, and eventually found this workaround:

The built in Node Debug extension ms-vscode.node-debug2 is default using a new version of the js debugging extension as of a new update. Even rolling back vscode seems to use the new Node Debug extension. For whatever reason, that's broken the breakpoints in my services. The solution was to revert to the old behaviour by adding this to my vscode/settings.json
"debug.javascript.usePreview": false

@peter-dolkens as posted elsewhere your issue is not related

I can repro on Windows. @deepak1556 it seems like this might be something with the Electron that we grab, it works with https://github.com/abartho/electron-typescript-vscode on Electron 13 with the same launch config. But it seems our electron doesn't get the NODE_OPTIONS/bootloader wired in like vanilla Electron does.

Electron will filter NODE_OPTIONS when packaged https://github.com/electron/electron/blob/main/docs/api/environment-variables.md#node_options, before Electron 12 this filter was not working so all options would be allowed. But it has been addressed via https://github.com/electron/electron/pull/29419
Currently a application is determined to be packaged if the executable name is different than electron.exe, which is the case on windows and linux. On macOS we haven't renamed the executable that is why the user is able to debug it.
What NODE_OPTION is needed for use case ? I will see if it can be allowed in the filter.

Huh, interesting. The debugger uses --require to inject the debugger bootloader. While there is a workaround that we use to debug Deno for example, it's a little less reliable and feature-ful than the bootloader.

--require will remain filtered in upstream electron as a security measure. Some untrusted content that launches application via external uri should not be able to inject scripts. Is it the only way to inject bootloader ? Can it be bundled into core and run when debugger kicks in ?


Write up how to guide for HPHI

This could potentially be a protocols.io guide.
Apply for an account

Apply for an HPHI account.
⏲ wait for account to be created

Download and setup the required software

Download X2Go client & Putty
Set up a connection to the Cambridge VPN service
Choose the operating system you use (eg Windows 10, OS X Sierra etc) from the links on the left side of that page to find the specific instructions for your device.

Connect

Connect (either via X2Go, or Putty) to wbic-gate.vss.cloud.private.cam.ac.uk
Check out the pictures at this help page to make sure you get all the right checkboxes filled in!

Need to create python enviroment: https://uoa-eresearch.github.io/eresearch-cookbook/recipe/2014/11/20/conda/
My command: conda create -n kwPython361 python=3.6.1 anaconda

If you want to use git you need to load the git version 2.13.2 module: module load git/2.13.2
This is in my bashrc file.
If you have 2 factor authentication turned on then you can't use your password to push to github. You need to use your personal access token: https://help.github.com/articles/creating-a-personal-access-token-for-the-command-line/
I've saved this in my home directory in a folder called API_TOKENS and a file called GitHub. Note that the default on the HPHI is to give read access to all members so you need to change this setting!! You can type chmod 700 API_TOKENS and then chmod 600 -R API_TOKENS/* from your home directory.

Change the default settings for the prompt by adding the following to your .bashrc file
# Hide a bunch of the directories in the terminal
# This guide prints out the user, host and
# just the basename of the working directory
# (not the full path)
# eg<EMAIL_ADDRESS>SCRIPTS]$
export PS1='[\u@\h \W]$ '

Then also create a .inputrc file that contains this line so you don't have to double tab to complete symbolic links:
set mark-symlinked-directories on

Here's a great guide for SLURM http://www.cism.ucl.ac.be/Services/Formations/slurm/2016/slurm.pdf

From HPHI set up a ssh key so you can easily link to the HPC by following the instructions here: http://www.thegeekstuff.com/2011/07/rsync-over-ssh-without-password/


php artisan migrate:refresh error 1064 on sql syntax

when Im trying run : php artisan migrate:refresh, I cant run migrate. Help me pls
http://prnt.sc/aayhhs

It seems like you have an issue with your mysql version accepting json fields.. Try updating it or moving to homestead and hit the command back again

5.6.21 is my sql version, I think it support json type. :(

sorry, when I trying change type json to string, it can run, thank you gocanto :D

Np

just found this https://mattstauffer.co/blog/strict-mode-and-other-mysql-customizations-in-laravel-5-2
u can try this out as well

ok, I will try, thank you gocanto :)


Fix lint error & shorten failure output

Travis is whining in https://github.com/pachyderm/pachyderm/pull/5331, but that's a bigger PR so I factored this out

Just noticed @actgardner is out for the day, so I'm reassigning


'Describing decision' in wrong section

Currently, the question "Please describe the decision(s) that will be automated" is under Impact Assessment, only when "yes" is answered to "Will the system be replacing human decisions that require judgement or discretion?". This is wrong. That question to describe the decision would be much better situated under "About the decision", and always relevant.

Moved the question to the section "About the decision", and removed the visibleIf condition.


Seem to be failing to connect to character.ai

Hi. I have triple checked that my user token is correct but am still somehow having this issue. Would be helpful if the error message from character.ai was printed as it might help me figure out the problem. Is this still working for anyone else? I worry the char ai devs might have broken something.
I am on Windows. I have tried running as administrator and making exceptions in the firewall for the program, still nothing.

Nevermind, I accidentally copied the access token instead of the key... Of course I didn't notice until after opening the issue. Thank you, this is kickass!

But you're right, gotta add error description to a failure logs


Review and update CUD APIs for migrations

As part of the data seeding work now tracked in https://github.com/aspnet/EntityFramework/issues/629, we created CUD APIs that can be used inside migrations.
The data seeding work is currently not scheduled to make it in 2.0, but the CUD APIs are already in.
Before we release 2.0.0 we should make sure the CUD APIs are in the state we want them to be, e.g.

Are compatible with the usage in the current PR https://github.com/aspnet/EntityFramework/pull/7824
Are very usable and we are confident we won't want to make breaking changes on them after we ship.

Otherwise we should consider either fixing them or pulling them from the release.
cc @ajcvickers

We should also SET IDENTITY_INSERT for you on SQL Server when inserting.

It needs a bit of cleanup, but it'll fit into Preview 2. I believe we can ship without making breaking changes to enable seed data later.


Refactor of Unity Sample for Scene Understanding

This is a pretty large update on the sample, i will try to list all the changes.
-DisplayManager.cs, DataProvider.cs, TransformUtils.cs, SaveDataUtils and SceneUnderstanding Utils have all been refactored into
1 component called SceneUnderstandingManager that handles all logic for loading a SU scene into Unity

Added  Custom Editor for the SceneUnderstandingManager component

A serialized Scene can now be loaded out of playmode and generate unity colliders automatically

Added two more examples scenes for NavMesh and Hologram Placement

-SceneUnderstandingManager allows for callbacks when it loads to allow devs to run their own custom code when SU loads
-Improved WireFrame mode and labels, now using TextMeshPro

Scenes can now be splitted into multiple fragments and can still be loaded in Unity

Added Home Scene that allows very simple navigation to the rest of the example Scenes, also improved visual quality on the Menu for the rest of the examples scenes.

-Added an Invisible Occlusion mode for Scene Objects called Ghost Mode, where Scene Objects are invisible but still occlude other objects like holograms, allowing realistic feel in AR

@SzymonSPS Ready for review

Tested the latest build on device once more, it all seems good!


[Bug]: Gsheets: Fetch many query is fetching deleted spreadsheets instead of existing ones.

Is there an existing issue for this?

[X] I have searched the existing issues

Description
On creating Fetch Many query from Google Sheet, in the spreadsheets dropdown it is showing the deleted spreadsheets, rather then the existing spreadsheets, or in case you dont have deleted spreadsheets it shows  No results found. Try to search a different keyword
https://user-images.githubusercontent.com/80334441/222361569-e39595b2-440a-4d10-a208-6224b3aef750.mov
Steps To Reproduce

Create a Google Sheets Fetch Many query, set the entity as Sheet Rows
Click on Spreadsheet dropdown, observe it shows deleted spreadsheet and in case you dont have deleted spreadsheet it shows No results found. Try to search a different keyword

Public Sample App
No response
Environment
Release
Issue video log
No response
Version
Release

This is a known issue and a feature that is behind feature flag. This feature would be merged into Release as and when the job stories are finished. Please use accounts apart from appsmith and moolya to use the regular flow


Turn servers credentials required

Hi! I've faced a problem, that username and password fields are required when defining turnServers object in sip.js. But in native RTCIceServer object credentials are optional. So is there a way i can use turn server without authorization?

Hi @Klimashin,
Can you see if the commit referenced above fixes this issue?
Thanks

It's been 3 weeks since the fix, so I am going to close this issue. If the commit in question did not fix the issue, please re-open this issue or open another issue.
Thanks,
James

Merged #367


Creating archive tar.gz file including submodules in the create-archive action

Description
This ticket is related to https://github.com/packit/packit/issues/1648#issuecomment-1585161153.
I tried ways to create tar.gz archive including submodules.
I referred to the https://stackoverflow.com/questions/5482544.
I see 2 possible ways. However, both commands fail in the CI.
1. git ls-files + tar
actions:
  create-archive:
    - git ls-files --recurse-submodules | tar caf .packit/simde-all.tar.gz -T-

The RPM spec file .packit/simde.spec is in the same directory with the .packit in my case.
I succeeded the command git ls-files --recurse-submodules | tar caf .packit/simde-all.tar.gz -T- in my local
https://dashboard.packit.dev/results/copr-builds/817758
https://download.copr.fedorainfracloud.org/results/packit/junaruga-simde-wip-ci-packit/srpm-builds/06058329/builder-live.log.gz
2023-06-09 23:04:01.720 utils.py          ERROR  Preparation of the repository for creation of an SRPM failed: Command 'git ls-files --recurse-submodules | tar caf .packit/simde-full.tar.gz -T-' failed.

2. git-archive-all PyPI package
actions:
  create-archive:
    # https://github.com/Kentzo/git-archive-all
    - pip install --user git-archive-all
    - git-archive-all --prefix simde-all/ .packit/simde-all.tar.gz

https://dashboard.packit.dev/results/copr-builds/817776
https://download.copr.fedorainfracloud.org/results/packit/junaruga-simde-wip-ci-packit/srpm-builds/06058364/builder-live.log.gz
2023-06-09 23:35:07.749 utils.py          ERROR  Preparation of the repository for creation of an SRPM failed: [Errno 2] No such file or directory: 'pip'

It seems there is no way to install python3-pip RPM in the phase.
I hope an executable example to create an tar.gz archive file including submodules is documented.
Benefit
Users can save the time to investigate it by themselves.
Importance
Some of the open source project's repositories have submodules.
Workaround

[ ] There is an existing workaround that can be used until this feature is implemented.

Participation

[ ] I am willing to submit a pull request for this issue. (Packit team is happy to help!)

https://packit.dev/docs/actions/
The first thing, try this:
actions:
  create-archive:
    - bash -c 'git ls-files --recurse-submodules | tar caf .packit/simde-all.tar.gz -T-'

Thanks for the info. I tried your way. However, I still got an error on it.
https://dashboard.packit.dev/results/copr-builds/818044
https://download.copr.fedorainfracloud.org/results/packit/junaruga-simde-wip-ci-packit-submodule-archive/srpm-builds/06059468/builder-live.log.gz
2023-06-10 17:14:55.665 commands.py       DEBUG  Command: bash -c git ls-files --recurse-submodules | tar caf .packit/simde-all.tar.gz -T-
2023-06-10 17:14:56.665 utils.py          ERROR  Preparation of the repository for creation of an SRPM failed: The create-archive action did not output a path to the generated archive. Please make sure that you have valid path in the single line of the output.
Traceback (most recent call last):
  File "/usr/lib/python3.11/site-packages/packit/api.py", line 1293, in prepare_sources
    self.up.prepare_upstream_for_srpm_creation(
  File "/usr/lib/python3.11/site-packages/packit/upstream.py", line 545, in prepare_upstream_for_srpm_creation
    SRPMBuilder(upstream=self, ref=upstream_ref).prepare(
  File "/usr/lib/python3.11/site-packages/packit/upstream.py", line 1067, in prepare
    created_archive = self.upstream.create_archive(
                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/packit/upstream.py", line 306, in create_archive
    return Archive(self, version).create(create_symlink=create_symlink)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/packit/upstream.py", line 1147, in create
    raise PackitException(
packit.exceptions.PackitException: The create-archive action did not output a path to the generated archive. Please make sure that you have valid path in the single line of the output.

You can check the working branch here.
https://github.com/junaruga/simde/tree/wip/ci-packit-submodule-archive

Oh, I forgot about the path to the archive on output, maybe:
actions:
  create-archive:
    - bash -c 'git ls-files --recurse-submodules | tar caf .packit/simde-all.tar.gz -T-'
    - ls .packit/simde-all.tar.gz

?

@junaruga I am curious about the overall submodule workflow and CI tools that can be used

What utilities do you use to create reproducible archives? fabacab/git-archive-all.sh is the closest I could find because it actually calls git archive (needed for export-subst), but it would be nice to have a more modern tool. git-archive-all PyPI fails in that point, but maybe the ideas from the badh script can be reimplemented and modernized there
Are there CI tools that create the git archive properly for github release? How about PyPI sdist and python builder that can do that?

Oh, I forgot about the path to the archive on output, maybe:

Ah, I needed to the article path to the stdout. I will try it later. Thanks.

What utilities do you use to create reproducible archives?

I am not using any utilities yet. What does the "reproducible archives" mean? If you can create the same binary from the same files, it can be reproducible? The commands below create an identical archive file from the same files.
$ git ls-files --recurse-submodules | tar caf .packit/simde-all.tar.gz -T-

$ sha256sum .packit/simde-all.tar.gz
fa2623bba65b5ab60ed58670f9e532dfbf295ee441451c28ceaf8f998da65957  .packit/simde-all.tar.gz

$ git ls-files --recurse-submodules | tar caf .packit/simde-all_2.tar.gz -T-

$ sha256sum .packit/simde-all_2.tar.gz
fa2623bba65b5ab60ed58670f9e532dfbf295ee441451c28ceaf8f998da65957  .packit/simde-all_2.tar.gz

What utilities do you use to create reproducible archives?

I am not using any utilities yet. What does the "reproducible archives" mean? If you can create the same binary from the same files, it can be reproducible? The commands below create an identical archive file from the same files.

Indeed that. See the issue here. Although git-archive-all and to that extenct git ls-files | tar do create reproducible archives to some extenct, they do not read and expand git attributes, which is a more crucial issue. E.g. setuptools_scm uses a file called .git_archive.txt with contents in vcs is like:
node: $Format:%H$
node-date: $Format:%cI$
describe-name: $Format:%(describe:tags=true,match=*[0-9]*)$
ref-names: $Format:%D$

but after git archive is called it becomes like:
node: 35da674aae9bf5c0cd770e5aabb54a00bea77fe9
node-date: 2023-06-13T09:20:22-04:00
describe-name: v0.4.5-2-g35da674a
ref-names: HEAD -> main

Using the tar will not expand those variables and %pybuild_wheel will fail because there is no information of what version/tag/commit has been used in the git archive.
fabacab/git-archive-all.sh doe recursive git archive in order to preserve that information, but it is not maintained, unit-tested or packaged.

Hi, did you have any chance to try out the suggestions? Or is there anything else I could help with? :)

Ah, I needed to the article path to the stdout. I will try it later. Thanks.

I was finally able to see that the archive file with the submodules were created successfully with the commands below. I needed to run the git submodule update --init. I also ran the tar --transform="s|^|simde-all/|" to create the archive file with the top prefix directory "simde-all" in my case.
diff --git a/.packit.yml b/.packit.yml
index 3d31fca1..2f4c834d 100644
--- a/.packit.yml
+++ b/.packit.yml
@@ -4,6 +4,13 @@
 # A minimal RPM spec file to run the tests.
 specfile_path: .packit/simde.spec

+actions:
+  create-archive:
+    - git submodule update --init
+    # Create an archive file with the simde-all directory prefix.
+    - bash -c 'git ls-files --recurse-submodules | tar --transform="s|^|simde-all/|" -caf .packit/simde-all.tar.gz -T-'
+    - echo '.packit/simde-all.tar.gz'
+
 jobs:
   - &copr
     # Use Fedora COPR in the CI.

The pipeline: https://copr.fedorainfracloud.org/coprs/packit/junaruga-simde-wip-ci-packit-submodule-archive/build/6169513/
The branch that I tested: https://github.com/junaruga/simde/tree/wip/ci-packit-submodule-archive

Glad to hear that, I'm going to close for now :) if you need anything related, feel free to reopen, otherwise don't hesitate to open new issue or reach out on Matrix/Slack o7

@mfocko Do you think we should add this to our example section?

@mfocko Do you think we should add this to our example section?

I guess, I'll adjust it a bit.


Weaviate Backend: Create objects in batch instead of one at a time

The current setup creates objects sequentially by calling the method _setitem, which also checks if objects exists. This code flow is good when the creation requests to the backend are asynchronous, and are not waiting for each object individually to be created. Since the current version of the python client (v3.3.0) does not allow asynchronous calls, we can achieve it by creatinf objects in batches.
The draw back of creating objects sequentially is that every time we make a request to create the object we wait until Weaviate Server creates the objects and sends back a response. While we wait for the response we could actually send more requests to weaviate to create objects, since objects do not depends on each over (I assume the don't). This can be achieved by collecting objects in a batch and then send them to Weaviate server in one go. This workflow will reduce the waiting time of the main python thread.
Another thing to notice is that creating objects in batch will replace the object/s if the UUID already exists, this means that this  _setitem can be reduced to one line and one batch request.

Hi @StefanBogdan , thank you for the suggestions. Currently we did apply batch update with the extend method, but did not realize the batch's add_data_object API also does replace/update. As for _setitem, it was done this way because the ID field of a document may change. This behavior however is now superseded by _change_doc_id.  That said, when performing batch set/update, I think it may still be possible that users update the IDs of some documents. One way I can think of to maybe address this is a two-pass approach (one pass to batch add/update, one pass to batch delete docs with old IDs)? Is this something feasible/something you have in mind?Also we wonder if Weaviate is planning to add async support to the python client? Thanks.

Also we wonder if Weaviate is planning to add async support to the python client? Thanks.
Yes. This is the main feature for the next release.


StoageFake should implement Illuminate\Contracts\Filesystem\Factory

I am using this package for some Dusk tests that use Spatie MediaLibrary
It throws this error...
[2020-06-22 15:14:54] testing.ERROR: Argument 1 passed to Spatie\MediaLibrary\Filesystem\Filesystem::__construct() must implement interface Illuminate\Contracts\Filesystem\Factory, instance of NoelDeMartin\LaravelDusk\Fakes\StorageFake given {"userId":1,"exception":"[object] (TypeError(code: 0): Argument 1 passed to Spatie\\MediaLibrary\\Filesystem\\Filesystem::__construct() must implement interface Illuminate\\Contracts\\Filesystem\\Factory, instance of NoelDeMartin\\LaravelDusk\\Fakes\\StorageFake given at /xxx/xxx/xxx/vendor/spatie/laravel-medialibrary/src/Filesystem/Filesystem.php:24)

The StorageFake currently doesn't work with the media library package as the StorageFake needs to implement the Filesystem Factory.
https://github.com/spatie/laravel-medialibrary/blob/2f8af49d6e67769baf19b269f275284554a7caa6/src/MediaCollections/Filesystem.php#L17-L26
I have implemented this locally with success, although now I am unable to generate the paths to fake files as it throws the following error...
[2020-06-22 16:32:15] testing.ERROR: Argument 2 passed to Spatie\MediaLibrary\UrlGenerator\S3UrlGenerator::__construct() must be an instance of Illuminate\Filesystem\FilesystemManager, instance of NoelDeMartin\LaravelDusk\Fakes\StorageFake given (View: /xxx/xxx/xxx/app/Console/Employer/AdditionalPages/Views/form.blade.php) {"userId":1,"exception":"[object] (ErrorException(code: 0): Argument 2 passed to Spatie\\MediaLibrary\\UrlGenerator\\S3UrlGenerator::__construct() must be an instance of Illuminate\\Filesystem\\FilesystemManager, instance of NoelDeMartin\\LaravelDusk\\Fakes\\StorageFake given (View: /xxx/xxx/xxx/app/Console/Employer/AdditionalPages/Views/form.blade.php) at /xxx/xxx/xxx/vendor/spatie/laravel-medialibrary/src/UrlGenerator/S3UrlGenerator.php:14)

This is with v7 of the Spatie Media Library
Thanks

Hey, I am so sorry about not merging this until now 🙇♂. I must have missed the notification or something.
For future reference, if I haven't replied to an issue/PR in a month in any of my repos feel free to ping me again.
Thanks for contributing!


[Request] Improve error message when DID cannot be resolved

Bug description
Try to resolve an Identity such as did:iota:dev:DfdEgjf1MQbh23QHAZwRpUtrTknf6BFk4MrXQMv7hQbG on the mainnet
An error such as the one below is raised, not informative nor helpful
Error: Error [ChainError]: Chain Error: Invalid Root Document
    at module.exports.__wbg_new_342a24ca698edd87 (/Users/jmcanterafonseca/work/IOTA/develop/tangle.js/tools/tangle-cli/node_modules/@iota/identity-wasm/node/identity_wasm.js:2884:15)
    at <anonymous>:wasm-function[4054]:0x1ade22
    at <anonymous>:wasm-function[3455]:0x19d68e
    at <anonymous>:wasm-function[1543]:0x13da21
    at <anonymous>:wasm-function[1160]:0x11c421
    at <anonymous>:wasm-function[3096]:0x190fac
    at <anonymous>:wasm-function[2592]:0x17cd4e
    at _dyn_core__ops__function__FnMut__A____Output___R_as_wasm_bindgen__closure__WasmClosure___describe__invoke__hca1c949258eb7ac8 (<anonymous>:wasm-function[5764]:0x1c886b)
    at __wbg_adapter_32 (/Users/jmcanterafonseca/work/IOTA/develop/tangle.js/tools/tangle-cli/node_modules/@iota/identity-wasm/node/identity_wasm.js:214:10)
    at real (/Users/jmcanterafonseca/work/IOTA/develop/tangle.js/tools/tangle-cli/node_modules/@iota/identity-wasm/node/identity_wasm.js:199:20)

Language version
WASM bindings 0.4.0
Expected behaviour
A controlled error informing that the Identity cannot be resolved
Actual behaviour
A not controlled error that reveals internal working details of the library

The error messages should be more descriptive with the changes in #506 and #512. This should be included in the next release.
Examples:
The common cases are as follows:

From your example, trying to resolve a DID using a Client configured for a different network will now throw the following error:Err > DIDNotFound: DID network 'dev' does not match client network 'main'

Trying to resolve a non-existent DID or one that has been pruned from the node will throw:Err > DIDNotFound: DID not found or pruned

Trying to resolve a DID whose root document is invalid, e.g. was not signed correctly or otherwise violates the method specification:Err > DIDNotFound: no valid root document found


Size limitation documentation is absent

Please document what is the size limitation for this OneDrive format conversion API. Does the limitation apply to the input content size, output content size, or both?

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

ID: 46acaeca-c7f2-1013-239e-93853818f3d7
Version Independent ID: 8b42fae5-53c9-f8c3-b88f-f2a087b23f8c
Content: Convert to other formats - OneDrive API - OneDrive dev center
Content Source: docs/rest-api/api/driveitem_get_content_format.md
Product: onedrive
GitHub Login: @JeremyKelley
Microsoft Alias: JeremyKe

@daviburg
Thank you for your feedback, we will take a look at what content can be provided.
Unfortunately, it is not just Content-size that is a factor for conversion limitation but a number of other factors too


Changing the for (auto i = 0; ...) in #822 to for (size_t i = 0; .. to silence warnings during building

Context:
New warning when you make all #822 here: https://github.com/PennyLaneAI/catalyst/blob/main/mlir/lib/Gradient/Transforms/ConversionPatterns.cpp#L908 (and a bunch of similar places)
[53/172] Building CXX object lib/Gradi...ransforms.dir/ConversionPatterns.cpp.o
/home/paul.wang/catalyst/mlir/lib/Gradient/Transforms/ConversionPatterns.cpp:910:28: warning: comparison of integers of different signs: 'int' and 'unsigned long' [-Wsign-compare]
        for (auto i = 0; i < argc * 2; i++) {

Changing auto to uint64_t or size_t will silence this. (argc is i64 https://github.com/PennyLaneAI/catalyst/blob/main/mlir/include/Gradient/IR/GradientOps.td#L318)

auto strikes again 🙈


Cannot install @microsoft/kiota-bundle under Yarn

yarn add @microsoft/kiota-bundle
➤ YN0000: · Yarn 4.3.1
➤ YN0000: ┌ Resolution step
➤ YN0082: │ @microsoft/kiota-abstractions@npm:*: No candidates found
➤ YN0000: └ Completed in 0s 380ms
➤ YN0000: · Failed with errors in 0s 404ms

I can install @microsoft/kiota-abstractions directly without an issue:
 yarn add @microsoft/kiota-abstractions
➤ YN0000: · Yarn 4.3.1
➤ YN0000: ┌ Resolution step
➤ YN0085: │ + @microsoft/kiota-abstractions@npm:1.0.0-preview.66, @opentelemetry/api@npm:1.9.0, @std-uritemplate/std-uritemplate@npm:1.0.6, guid-typescript@npm:1.0.9, tinyduration@npm:3.3.1, uuid@npm:10.0.0
➤ YN0000: └ Completed in 1s 480ms
➤ YN0000: ┌ Post-resolution validation
[...]
➤ YN0000: └ Completed
➤ YN0000: ┌ Fetch step
➤ YN0013: │ A package was added to the project (+ 441.71 KiB).
➤ YN0000: └ Completed in 0s 956ms
➤ YN0000: ┌ Link step
➤ YN0008: │ nuxt-app@workspace:. must be rebuilt because its dependency tree changed
➤ YN0000: └ Completed in 7s 790ms
➤ YN0000: · Done with warnings in 10s 414ms

Same issue for me , I could not install my kiota client project in CI/CD
yarn install v1.22.22
warning package-lock.json found. Your project contains lock files generated by tools other than Yarn. It is advised not to mix package managers in order to avoid resolution inconsistencies caused by unsynchronized lock files. To clear this warning, remove package-lock.json.
[1/4] Resolving packages...
warning Lockfile has incorrect entry for "@microsoft/kiota-abstractions@". Ignoring it.
Couldn't find any versions for "@microsoft/kiota-abstractions" that matches ""
? Please choose a version of "@microsoft/kiota-abstractions" from this list: (Use arrow keys)

1.0.0-preview.66
1.0.0-preview.65
1.0.0-preview.64
1.0.0-preview.63
1.0.0-preview.62
1.0.0-preview.61
...

@hognevevle @medevod @zZHorizonZz
Any chance you can confirm if this issue is still present with the latest release of the bundle package?
<EMAIL_ADDRESS>

Hi @andrueastman,
It works for me with this latest version of kiota-bundle.
I'am using yarn v1.22.19
Thank you


Javadoc Improvements

Some opportunities spotted while reviewing #96 -- javadoc in the following files could be expanded

HttpWebRequest
StringExtensions
OAuth2Strategy
CertificateCredential

Closing. Will create work item for separate effort when prioritized