Refactor output of OpenQASM 3 exporter to use fewer aliases

Summary
This removes spurious _loose_bit "registers" from the OpenQASM 3 output, and instead emits loose bits with individual bit and qubit declarations.  Non-overlapping registers are emitted using regular bit[n] and qubit[n] definitions when possible, and we only resort to aliasing if we must to describe the structure.
This avoids introducing structure to the definitions that does not exist in the original program, making round-trips and interactions with other OQ3 consumers more straightforwards.  It's better not to use advanced features that don't map to hardware particularly well when it's not necessary.
On the technical side, all bits are now properly tracked in the symbol table.  Previously, there was a lot of code duplication, internal state tracking and magic inferences that attempted to "guess" how a qubit/clbit should be referred to.  Instead, we just properly add them as variables to the symbol table, which also drastically reduces the number of objects that effectively reserve names that the user may not use.

Details and comments
As an example, given this code:
from qiskit import QuantumCircuit, QuantumRegister, ClassicalRegister, qasm3
from qiskit.circuit import Qubit, Clbit

qc = QuantumCircuit(
    QuantumRegister(3, "qmem"),
    [Qubit(), Qubit(), Qubit(), Clbit(), Clbit(), Clbit()],
    ClassicalRegister(3, "a"),
)
qc.h(0)
qc.cx(0, 1)
qc.cx(0, 2)
for i in range(3):
    qc.cx(i, i + 3)
qc.measure(range(6), range(6))

print(qasm3.dumps(qc))

the output before this PR was:
OPENQASM 3;
include "stdgates.inc";
bit[3] _loose_clbits;
bit[3] a;
qubit[6] _all_qubits;
let qmem = _all_qubits[0:2];
h qmem[0];
cx qmem[0], qmem[1];
cx qmem[0], qmem[2];
cx qmem[0], _all_qubits[3];
cx qmem[1], _all_qubits[4];
cx qmem[2], _all_qubits[5];
_loose_clbits[0] = measure qmem[0];
_loose_clbits[1] = measure qmem[1];
_loose_clbits[2] = measure qmem[2];
a[0] = measure _all_qubits[3];
a[1] = measure _all_qubits[4];
a[2] = measure _all_qubits[5];

and now it is the more natural
OPENQASM 3;
include "stdgates.inc";
bit _bit0;
bit _bit1;
bit _bit2;
bit[3] a;
qubit _qubit3;
qubit _qubit4;
qubit _qubit5;
qubit[3] qmem;
h qmem[0];
cx qmem[0], qmem[1];
cx qmem[0], qmem[2];
cx qmem[0], _qubit3;
cx qmem[1], _qubit4;
cx qmem[2], _qubit5;
_bit0 = measure qmem[0];
_bit1 = measure qmem[1];
_bit2 = measure qmem[2];
a[0] = measure _qubit3;
a[1] = measure _qubit4;
a[2] = measure _qubit5;
``

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

58 of 59 (98.31%)  changed or added relevant lines in 3 files are covered.
13 unchanged lines in 5 files lost coverage.
Overall coverage increased (+0.06%) to 85.904%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit/qasm3/exporter.py
49
50
98.0%

Files with Coverage Reduction
New Missed Lines
%

crates/accelerate/src/sabre_swap/mod.rs
1
99.77%

crates/qasm2/src/expr.rs
1
93.76%

qiskit/extensions/quantum_initializer/squ.py
2
80.0%

crates/qasm2/src/lex.rs
3
91.65%

crates/qasm2/src/parse.rs
6
96.65%

Totals

Change from base Build<PHONE_NUMBER>:
0.06%

Covered Lines:
71222

Relevant Lines:
82909

💛  - Coveralls

A minor point. Why is this line
https://github.com/Qiskit/qiskit-terra/blob/c365df87c62c58c378a242f57e4d563da42cea09/test/python/qasm3/test_export.py#L141
not written qubits = [Qubit() for _ in range(10)] ? Is it because the dummy variable _ is always set to None, suggesting that its value is irrelevant?

It's just a bit of an idiosyncrasy - there's no meaning.  It so happens that [None]*n is one of the fastest iterators in CPython, despite the allocation.  It's technically beaten by itertools.repeat(None, n), but for some reason I just tend to default to [None]*n.

I'm having difficulty understanding the main point(s) of this PR. What is the problem with aliasing and "trying to guess how a qubit/clbit should be referred to" ?
Putting the summary at the top:  I don't see that aliasing is essential if you want to collect loose bits as the existing code does (but I may be missing something). I do see that the PR may allow a more faithful round trip.
The example given in the opening comment doesn't make it clear (to me at least) that the change is better.
In this code
from qiskit import QuantumCircuit, QuantumRegister, ClassicalRegister, qasm3
from qiskit.circuit import Qubit, Clbit

n_qubits = 3
n_clbits = 3

use_register = False
if use_register:
    qubits = QuantumRegister(n_qubits, "qmem")
    clbits = ClassicalRegister(n_clbits, "a")
else:
    qubits = [Qubit() for _ in range(n_qubits)]
    clbits = [Clbit() for _ in range(n_clbits)]

qc = QuantumCircuit(qubits, clbits)

qc.h(0)
for i in range(1, n_qubits):
    qc.cx(0, i)

qc.measure(range(n_qubits), range(n_qubits)) # assume enough clbits
print(qasm3.dumps(qc))

what is the semantic difference when use_register is True vs False? Should it be reflected in the OQ3?
To me it looks cleaner in some cases, such as this example, to collect the bits in registers.
Is the following an instance of the unwanted aliasing?
qubit[3] _all_qubits;
let qmem = _all_qubits[0:2];

Can you avoid aliasing by outputting something like this ?
qubit[3] user_supplied_register_name_1;
qubit[3] _loose_qubits_1;
qubit[3] user_supplied_register_name_2;

One possible advantage that I can see of this PR is the following. If the OC3 representation doesn't collect "scalar" qubits into a register, then the round-trip representation in qiskit can be more faithful to the original. Maybe that's worth preserving.

It's mostly about the OQ3 reflecting the structure of the Qiskit program more closely, which allows better round-tripping. That's originally why Ian had requested I do this; qiskit.qasm3.loads(qiskit.qasm2.dumps(qc)) == qc would typically be False if the circuit had loose bits, because the OQ3 dump would insert spurious structure.
The other problem is that a bit can be in more than one register.  The original code always used aliasing so that there only needed to be a single path, but that quickly needed modifying when it became clear that hardware was never going to support the concept of aliasing classical registers, so the alias_classical_registers option was born.  This PR simplifies a bunch of the magic that was added along with that option, moving the decision about "how should this bit be referenced?" into a single place, with the same logic working for both classical and quantum bits - previously, every place that referenced a bit did some magic guesswork to determine what OQ3 node should be output for a given bit.
To speak to the edit: whatever names you give your bits in Python space are irrelevant, because they're not stored in the QuantumCircuit.  There's no possibility of round-tripping those in the current way Qiskit works.  This is mostly just about making the Qiskit <-> OQ3 translation higher fidelity where there are simple ways to do that, and as a side-effect, it's a long overdue refactor to remove a lot of the magic names and tightly coupled logic between "definition" and "use" in the OQ3 exporter.

whatever names you give your bits in Python space are irrelevant, because they're not stored in the QuantumCircuit

I tried to say the same thing. The identifiers used in Python are not preserved as identifiers in OC3 (I don't know how to do this.) But whether the bit is in a Qiskit register or not can be preserved. I think this part of the structure you are talking about.

An answer to my question above:

Is the following an instance of the unwanted aliasing?
qubit[3] _all_qubits;
let qmem = _all_qubits[0:2];

In fact you have simplified this for input QuantumRegister(3, "qmem"). I somehow missed this in my tests.


feat(translations): New translations zh-TW.ts

Hi,
This PR has added and completed the translation for zh-TW (100%).
I've run npm run format, but most files have undergone changes. Since the majority of files are not within the scope of this submission, I'm only submitting locales.ts and zh-TW.ts to avoid making too many changes to the entire project.
Feel free to raise any questions or suggestions.

Great thanks :)


Add text_area helper to form_for

cc @AlfonsoUceda, @jodosha

:+1: I like it

Can you add text_area helper to https://github.com/lotus/helpers/blob/master/lib/lotus/helpers/form_helper.rb#L29 ?
:+1:

@AlfonsoUceda, @joneslee85, I added some small changes, please see

@khaiql except the minor comment above, :+1:

@khaiql please squash all commits into 1

@joneslee85 done :D


Enabling minify_js can cause inf loop / hang for malformed script tags

I found that when you enable minify_js and have a script tag inside of a second script tag, minify_html will inf loop and peg the cpu at 100% forever.
Below is a proof of concept that hangs on my machine:
import minify_html

html = '''
<script><script>
</script></script>
'''

print("Starting minify...")
o = minify_html.minify(html, minify_js=True)
print("Finished minify...")
print(o)

Thanks for raising this, I've confirmed that the issue lies with JS parser.


[test] Add negative tests for service APIs

Add negative TC with DBUS unconnected.

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #271. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.

:octocat: cibot: @anyj0527, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nnstreamer-api/ci/repo-workers/pr-checker/271-202303241106040.91204905509949-61ff01fe4e61038619f5b4786a02f679f5cf6e34/.


chore: secret key path

Description
This is like #668, only without all the refactoring
To be rebased on main once #674 is merged
How has this been tested?
⬇
Checklist

[x] changelog was updated with notable changes
[x] documentation was updated

/e2e --tests sdk,frontend

End to end tests: :hourglass_flowing_sand: STARTING

/e2e --tests sdk,frontend

Closed in favour of #676


Validate the PWM Driver

Use Logic Analyzers or test against a servo motor to confirm that the PWM Driver (for Orbital Platform Rev2 in particular) works fine

Document any tests to verify functionality, including the test setup (hardware-wise), the code being tested, and the result. Mention if the test passed or failed

Hardware setup: 1-1 GNG 1-2 PWM AF15
Code:
`pwm_initTimer(20000);
while (1) {
	PWM_TIMER_ON();
	pwm_setDutyCycle(1);`

Period can be seen correctly, duty cycle seen correctly - Passed (this is the same period a servo)
PWM can be be turned off - Passed (Seperate code but it just turns PWM off after ten seconds and turns it backs on after 10 seconds)

In the future simply move the task to In-Progress
For the 2nd test, is the code stored anywhere?
In both cases, it might be good to save the code as a file in the Timers folder. Refer to https://docs.google.com/document/d/1hv3hMiiT1q_joIU-QT4tFjU30lH0IN8SvIerq2Uw3Ag/edit (assuming you were testing on the master branch itself as others might now have the right support in main.c for this)
Also I'm confident in the success of this one, but in the future, it would be good to add any images or screenshots of results. If videos, upload them on the drive and then link them.


Detected blocking call to open inside the event loop

Extract from log file:
WARNING (MainThread) [homeassistant.util.loop] Detected blocking call to open inside the event loop by custom integration 'watchman' at custom_components/watchman/utils.py, line 258: for i, line in enumerate(open(yaml_file, encoding="utf-8")): (offender: /config/custom_components/watchman/utils.py, line 258: for i, line in enumerate(open(yaml_file, encoding="utf-8")):), please create a bug report at https://github.com/dummylabs/thewatchman/issues

Not reproducible in 0.6.3


Bower warning about eve short-sha

I get the following warning each time I install raphael via bower:
bower eve#eef80ed
short-sha consider using longer commit sha to avoid conflicts

Would adding full eef80ed8d188423c2272746fb8ae5cc8dad84cb1 in place of eef80ed here: https://github.com/DmitryBaranovskiy/raphael/blob/db2b2f2/bower.json#L6 fix the warning?
Perhaps upgrading it to the latest 0.5.0 commit: e5743970ae26ec1e6ce099ca5e6492c727ed7147 (adobe-webplatform/eve@e5743970ae26ec1e6ce099ca5e6492c727ed7147 -> Jul 2014) instead of continue to use the existing one: eef80ed8d188423c2272746fb8ae5cc8dad84cb1 (adobe-webplatform/eve@eef80ed -> Jan 2013) would bring some goodness as well..

composer doesn't support short sha. Update it, please ))

Updated in 55ee142

Is it merged to master?


feat(typing): type App classes

This is a follow up of ##2271

sorry for the rebase + squash, but it was starting to become a mess

Thanks, LGTM now bar these triple ''' docstrings, I regret even starting that discussion 🤣

I actually think it's a lot nicer for reading the source code. Having to escape the """ is quite ugly


Add note about configuration of travis and use of quay robot account

Adds notes about how travis and quay settings are configured to push images to the quay repo.

/lgtm
Thanks !


Csharp autocompletion

Observed behavior
After writing few characters and invoking <C-Space> I get No completions found, then after writing something like Console. i get a different error, saying Wrong type argument: hash-table-p, nil. Invoking M-x company-omnisharp returns Cannot complete at point. My first thought was that it's server problem because omnisharp-roslyn doesn' support Arch Linux, however, omnisharp functions work perfectly fine and running M-x omnisharp-check-alive-status returns Server is alive and well. Happy coding! and M-x omnisharp-add-dot-and-auto-complete returns a autocomplete pop-up with a prompt

Expected behavior
Normal company autocompletion :)
Steps to reproduce

Download and extract omnisharp-roslyn
Point to server executable with (after! omnisharp (setq omnisharp-server-executable-path "~/omnisharp/run"))
Open .cs file and start omnisharp-server

System information

Click to expand
DOOM Doctor
Running Emacs v25.3.1, commit 5dacbb7cb1c6ac246a9ccd15e6c4290def67757c

shell: /usr/bin/zsh
Compiled with:
  XPM JPEG TIFF GIF PNG RSVG IMAGEMAGICK SOUND GPM DBUS GCONF GSETTINGS
  NOTIFY ACL GNUTLS LIBXML2 FREETYPE M17N_FLT LIBOTF XFT ZLIB
  TOOLKIT_SCROLL_BARS GTK3 X11 MODULES
uname -a:
  Linux shittywok 4.15.3-1-ARCH #1 SMP PREEMPT Mon Feb 12 23:01:17 UTC
  2018 x86_64 GNU/Linux

Attempt to load DOOM: success! Loaded v2.0.9
Revision 5dacbb7cb1c6ac246a9ccd15e6c4290def67757c

----

test-emacs
test-windows
test-fonts
Found font material-design-icons.ttf
Found font weathericons.ttf
Found font octicons.ttf
Found font fontawesome.ttf
Found font file-icons.ttf
Found font all-the-icons.ttf
test-gnutls
test-tls
Validated https://elpa.gnu.org
Validated https://melpa.org
Successfully rejected https://self-signed.badssl.com
Successfully rejected https://wrong.host.badssl.com/
test-tar
Everything seems fine, happy Emacs’ing!

because omnisharp-roslyn doesn't support Arch Linux

It's odd you say that. I use roslyn on Arch Linux without problem.

Would you mind trying out develop? I've made a number of changes to the lang/csharp module. Unfortunately, complications in the upcoming release have delayed the merge into master, so we're in an odd limbo where develop is more stable than master.

Oh, I should mention, if you try develop, you should re-copy init.example.el -- there have been a couple changes.

Okej, I switched to develop branch and re-copied init.example.el like you suggested. Company autocompletion now works perfectly fine, however, I had few minor issues.
Mainly if I set omnisharp-server-executable-path to my custom folder, I get Omnisharp server isn't installed, completion won't work even tho I'm able to start the server and completion does work. And this is because it checks omnisharp--server-installation-path.
So I did try to use omnisharp-install-server witch froze my laptop, and forced me to reboot, but this is probably a bug with omnisharp package.
Ultimately I manually extracted omnisharp-roslyn to ~/.emacs.d/.local/cache/omnisharp/server/v1.26.3/ witch fixed the issue.
I do have a question tho, where do you recommend we keep private config files, ~/.doom.d/, ~/.config/doom, ~/.emacs.d/modules/private/ or ~/.emacs.d/modules/config/private, also does this project have slack, discord, irc, matrix or gitter channel?

does this project have slack, discord, irc, matrix or gitter channel?

We have a discord server: https://discord.gg/bcZ6P3y

Mainly if I set omnisharp-server-executable-path to my custom folder, I get Omnisharp server isn't installed ... this is because it checks omnisharp--server-installation-path.

I've discovered why this happens; 8af46aa should fix this issue.


Streaming from the server

I want to stream a webcam at /dev/video0 from the server without a browser. Can I stream over WebRTC a local device with NodeJS? I am on terminal-only Debian 7.8 (wheezy).

You probably want the node-webetc project. It's a webrtc client plugin for node. https://github.com/js-platform/node-webrtc


Insert the http://<IP_ADDRESS>:5000

Hi,
in the version of the popcorntime.sh , where a insert the http:<IP_ADDRESS> for fetching the content ?
King Regards

Here: https://github.com/popcorn-official/butter-settings-popcorn


Add tests for moment.d.ts

as discussed in https://github.com/moment/moment/pull/3280 with @ichernev
These tests are triggered after the grunt test task. The basis of the tests were the DefinitelyTyped moment tests found here.  Plugging this in revealed a few missing elements of the moment.d.ts Typing file.  I have added these in based on the definition in DT.
Hope this helps.

Took me a little while to realise how to get Travis to trigger the TypeScript test.  It does now.  There may well be a way to plug this testing into your grunt build; I'm afraid I don't know grunt though and so I can't help there.

@johnnyreilly you need to add an exec that runs the tsc and then add that to https://github.com/moment/moment/blob/develop/Gruntfile.js#L198
Also I was hoping that it will run all our test cases under tsc and print warnings? Or this is too hard (or there are too many issues -- I guess from negative tests)?

Hi @ichernev,
Thanks for the exec pointers; I'll give it a try. I'm afraid.

Also I was hoping that it will run all our test cases under tsc and print warnings?

I don't think I entirely follow your intention. So I'll hedge my bets and say "yeah, too hard" 😉

I meant -- maybe we can run the actual moment tests (there are many of them) against this thing, so it will actually get updated. I'm worried that now we have to keep adding stuff to the typing tests, that is independent from all other tests, and also independent of the documentation excerpts. So it will get out of sync, and become useless one day.
Well not too useless because now we check for the syntax of d.ts but that we can do with an empty test file.

Ah I see. I think the answer is no. Reason being that tsc is just a compiler. It's not a JavaScript runtime and so has no way to run tests. Remember the TypeScript "tests" are really just examples of API usage and their only way to fail is to fail to compile . So it would need to be independent I think.

An alternative approach would be to port your tests to TypeScript and plug into the existing runner. Then you could remove the TypeScript tests supplied as part of this pr in favour of them.
Is that something you'd be open to?

This would probably be the nicer approach but not having looked at your tests I'm not sure how much work it would involve.

@johnnyreilly well, typing-tests/moment-tests.ts is no more typescript than the tests. You can have a look at src/test/*/*.js. I agree the file ending is different, but we can just trick it to "compile" it and not execute (because as you said it can't execute).

I'll take a look when I get a moment and report back.

Pun not actually intended

Hey @ichernev,
I've just moved to use the Gruntfile driven suggestion of yours.  I can't look at the running all tests together approach right now although I'll try to get time at some point.  Would you be interested in taking this PR in the interim?

@johnnyreilly I can not get tsc command to work. I installed 1.5.3 and it tells me error TS5023: Unknown compiler option 'moduleResolution'. and according to https://github.com/axmor/typecs/issues/93 it should work with 1.5.0-beta

What version of TypeScript are you running? I'm on 1.8. I think that was added in 1.6.... http://www.typescriptlang.org/docs/handbook/compiler-options.html

Either way; it's not necessary and so I've removed it.  Want to try now?

Merged in 6f3d87ae71ee938874e7d6fde8ccab4eb07efe42

The main problem was that your implementation depended on a globally installed typescript, and I happened to install the wrong one (which worked half way but was old version so displayed some weird errors), so I just made sure to use the locally installed one -- https://github.com/moment/moment/commit/2c23c881070ffa5131e26372c3badc6dbf461092. Its now also run as part of the tests -- https://github.com/moment/moment/commit/12b4fc7f3a3076de60d8a5c0d9b3ccd05ca3bab7

Awesome - thanks!

Was it not run as part of the tests already?  Did the following code not include this in the test run?
grunt.registerTask('test', ['test:node', 'test:typescript']);

But the default task did only test:node, now it does test.

Ah - I see.  Cheers.


add GH Actions badges

The Markdown Links Validation GitHub Action won't succeed until we merge the modification to the writing documentation page since we move the documentation repository from public to private. In other words, it's normal for now.


Docker container

I wrapped up CLIFF into a Docker container that should help ease large-scale deployments. https://github.com/johnb30/cliff-docker

I have this deployed to https://cliff-docker-bhrycyuowx.now.sh if anyone needs to use it for testing.
Example:
https://cliff-docker-bhrycyuowx.now.sh/CLIFF-2.1.1/parse/text?q=Some%20clever%20text%20mentioning%20places%20like%20New%20Delhi,%20and%20people%20like%20Einstein.%20Perhaps%20also%20we%20want%20mention%20an%20organization%20like%20the%20United%20Nations?

Results:
{
  "results": {
    "organizations": [
      {
        "count": 1,
        "name": "United Nations"
      }
    ],
    "places": {
      "focus": {},
      "mentions": []
    },
    "people": [
      {
        "count": 1,
        "name": "Einstein"
      }
    ]
  },
  "status": "ok",
  "milliseconds": 80,
  "version": "2.1.1"
}

@purplecones It's no more deployed there.


Parallel Solves

I have a JuMP (mixed-integer) model where I'm trying to determine the upper and lower bounds on many variables. Each individual solve is very quick, but since I have many variables (~1,000), the overall solve time is slow (~5s).
One thing that I observed is that julia often seems to be using only one core at a time, and I was hoping that parallelization might be able to reduce solve times significantly. However, I'm running into an issue that seems to have to do with there being only a single Gurobi environment created.
Here's an example of a snippet of code that doesn't work. (map runs fine, but pmap does not).
addprocs(2)

using JuMP
using Gurobi

@everywhere begin
env = Gurobi.Env()
m = JuMP.Model(solver = Gurobi.GurobiSolver(env, OutputFlag=0))

@JuMP.variable(m, 0 <= x[i=1:100] <= i)

function upperbound_mip(x)
    @JuMP.objective(x.m, Max, x)
    JuMP.solve(x.m)
    return JuMP.getobjectivevalue(x.m)
end
end

@time map(upperbound_mip, x)
@time pmap(upperbound_mip, x)

Output
Academic license - for non-commercial use only
	From worker 3:	Academic license - for non-commercial use only
	From worker 2:	Academic license - for non-commercial use only
 2.317813 seconds (1.72 M allocations: 90.407 MiB, 1.04% gc time)
ERROR: LoadError: On worker 2:
AssertionError: env.ptr_env != C_NULL
get_error_msg at /home/vtjeng/.julia/v0.6/Gurobi/src/grb_env.jl:38
Type at /home/vtjeng/.julia/v0.6/Gurobi/src/grb_env.jl:50 [inlined]
get_intattr at /home/vtjeng/.julia/v0.6/Gurobi/src/grb_attrs.jl:16
setvarLB! at /home/vtjeng/.julia/v0.6/Gurobi/src/GurobiSolverInterface.jl:187
#build#119 at /home/vtjeng/.julia/v0.6/JuMP/src/solvers.jl:338
#build at ./<missing>:0
#solve#116 at /home/vtjeng/.julia/v0.6/JuMP/src/solvers.jl:168
upperbound_mip at /home/vtjeng/Dropbox/Documents/MIT/UROP/adversarial_examples/pset/6_using_MIPVerify/debug/feature-parallel-solves.jl:14
#106 at ./distributed/process_messages.jl:268 [inlined]
run_work_thunk at ./distributed/process_messages.jl:56
macro expansion at ./distributed/process_messages.jl:268 [inlined]
#105 at ./event.jl:73
Stacktrace:
 [1] #571 at ./asyncmap.jl:178 [inlined]
 [2] foreach(::Base.##571#573, ::Array{Any,1}) at ./abstractarray.jl:1733
 [3] maptwice(::Function, ::Channel{Any}, ::Array{Any,1}, ::Array{JuMP.Variable,1}, ::Vararg{Array{JuMP.Variable,1},N} where N) at ./asyncmap.jl:178
 [4] wrap_n_exec_twice(::Channel{Any}, ::Array{Any,1}, ::Base.Distributed.##204#207{WorkerPool}, ::Function, ::Array{JuMP.Variable,1}, ::Vararg{Array{JuMP.Variable,1},N} where N) at ./asyncmap.jl:154
 [5] #async_usemap#556(::Function, ::Void, ::Function, ::Base.Distributed.##188#190, ::Array{JuMP.Variable,1}, ::Vararg{Array{JuMP.Variable,1},N} where N) at ./asyncmap.jl:103
 [6] (::Base.#kw##async_usemap)(::Array{Any,1}, ::Base.#async_usemap, ::Function, ::Array{JuMP.Variable,1}, ::Vararg{Array{JuMP.Variable,1},N} where N) at ./<missing>:0
 [7] (::Base.#kw##asyncmap)(::Array{Any,1}, ::Base.#asyncmap, ::Function, ::Array{JuMP.Variable,1}) at ./<missing>:0
 [8] #pmap#203(::Bool, ::Int64, ::Void, ::Array{Any,1}, ::Void, ::Function, ::WorkerPool, ::Function, ::Array{JuMP.Variable,1}) at ./distributed/pmap.jl:126
 [9] pmap(::WorkerPool, ::Function, ::Array{JuMP.Variable,1}) at ./distributed/pmap.jl:101
 [10] #pmap#213(::Array{Any,1}, ::Function, ::Function, ::Array{JuMP.Variable,1}) at ./distributed/pmap.jl:156
 [11] pmap(::Function, ::Array{JuMP.Variable,1}) at ./distributed/pmap.jl:156
 [12] include_from_node1(::String) at ./loading.jl:576
 [13] include(::String) at ./sysimg.jl:14
 [14] process_options(::Base.JLOptions) at ./client.jl:305
 [15] _start() at ./client.jl:371

One thing I've thought about trying to do is to generate one environment per worker thread, but 1) I don't know whether that would help and 2) I don't know how to do that.

One option is to use a caching pool https://docs.julialang.org/en/stable/stdlib/parallel/#Base.Distributed.CachingPool
julia> addprocs(2)

julia> @everywhere using JuMP, Gurobi

julia> @everywhere function upperbound_mip(x)
           @JuMP.objective(x.m, Max, x)
           JuMP.solve(x.m)
           return JuMP.getobjectivevalue(x.m)
       end

julia> m = JuMP.Model(solver = Gurobi.GurobiSolver(OutputFlag=0));

julia> @JuMP.variable(m, 0 <= x[i=1:100] <= i);

julia> let x=x
       wp = CachingPool(workers())
       pmap(wp, (i)->upperbound_mip(x[i]), 1:100)
       end
        From worker 2:  Academic license - for non-commercial use only
        From worker 3:  Academic license - for non-commercial use only
100-element Array{Float64,1}:
   1.0
   2.0
   3.0
   4.0
   5.0
   ?
  95.0
  96.0
  97.0
  98.0
  99.0
 100.0

Closing because this is not an issue with Gurobi.jl. In the future, you can ask usage questions like this on Discourse.


tici: 12 bit camera readout only

Before #23887 is ready, this makes the camera use 12 bit mode instead of 10 bit mode. Required for IMX390 bringup. Copied relevant parts from #23887
Tested on desk to work with streamer, not sure what else I should do.

Is the output still limited to 10 bits? Otherwise, the scaling value should also change on selfdrive/camerad/cameras/real_debayer.cl:52.
Check that changing OP_WORD_CLK_DIV does not change the readout time past 50ms. Not sure what it's at now: https://github.com/commaai/thecalbi/pull/192/files#diff-5131e21485d33d4526d452fcc8d9c8c2b6587309fa4e788086a9fdda936f5e43R140

I just divide it super early, I don't think this affects AE.

Looks like changing OP_WORD_CLK_DIV does not change the readout timing, so this should be fine. It's currently at 49.9933875 ms, which is why we might be seeing artefacts at long exposure times.


🐛 [Bug] Advanced Indexing/GatherND compilation causes error: isObject() INTERNAL ASSERT FAILED at "libtorch/_virtual_includes/ATen/ATen/core/ivalue_inl.h":123, please report a bug to PyTorch. Expected Object but got None

Bug Description
PyTorch uses advanced indexing to implement GatherND. For example:
torch[[0,1], :, None, torch.tensor((0,1))]

is considered a valid indexing operation.
However, when applying TensorRT compilation, we see the following error:
isObject() INTERNAL ASSERT FAILED at "bazel-out/k8-opt/bin/external/libtorch/_virtual_includes/ATen/ATen/core/ivalue_inl.h":123, please report a bug to PyTorch. Expected Object but got None

To Reproduce
Run with nvcr.io/nvidia/pytorch:22.07-py3 container (gathernd.py):
  import tensorrt
  import torch
  import torch_tensorrt

  class GatherNDListModel(torch.nn.Module):
      def __init__(self):
          super().__init__()

      def forward(self, x):
          return x[[1,0], :, [1,0]]

  class GatherNDTensorModel(torch.nn.Module):
      def __init__(self):
          super().__init__()
          self.index = torch.tensor((1,0))

      def forward(self, x):
          return x[self.index, :, self.index]

  if __name__ == '__main__':
      torch.manual_seed(0)
      t = torch.randn(3, 4, 5).cuda()

      try:
          m = GatherNDListModel()
          o = m(t)
          print(f"PyTorch: {o.shape}")

          mt = torch_tensorrt.compile(m, inputs=[t], truncate_long_and_double=True)
          o = mt(t)
          print(f"TRT: {o.shape}")
      except Exception as err:
          print(f"GatherND list indexing failed: {err}")

      try:
          m = GatherNDTensorModel()
          o = m(t)
          print(f"PyTorch: {o.shape}")

          mt = torch_tensorrt.compile(m, inputs=[t], truncate_long_and_double=True)
          o = mt(t)
          print(f"TRT: {o.shape}")
      except Exception as err:
          print(f"GatherND tensor indexing failed: {err}")

Outputs the following:
PyTorch: torch.Size([2, 4])
WARNING: [Torch-TensorRT] - Truncating weight (constant in the graph) from Int64 to Int32
GatherND list indexing failed: isObject() INTERNAL ASSERT FAILED at "bazel-out/k8-opt/bin/external/libtorch/_virtual_includes/ATen/ATen/core/ivalue_inl.h":123, please report a bug to PyTorch. Expected Object but got None
PyTorch: torch.Size([2, 4])
WARNING: [Torch-TensorRT] - Truncating weight (constant in the graph) from Int64 to Int32
GatherND tensor indexing failed: isObject() INTERNAL ASSERT FAILED at "bazel-out/k8-opt/bin/external/libtorch/_virtual_includes/ATen/ATen/core/ivalue_inl.h":123, please report a bug to PyTorch. Expected Object but got None

Note that the pytorch version (ie. no-TRT) model outputs the expected shapes, but the TRT conversion fails.
Expected behavior
The TRT compilation should succeed and output the same output as the non-TRT version.
Environment

Build information about Torch-TensorRT can be found by turning on debug messages
nvcr.io/nvidia/pytorch:22.07-py3

Additional context
https://partners.nvidia.com/Bug/ViewBug/3735309

I have a fix for this error as well as the general None case but we hit the current limitation where the converter does not support more than one tensor used to index. https://github.com/pytorch/TensorRT/blob/c63a5a5717e28d1a195740356fffbb9afcff36a8/core/conversion/converters/impl/select.cpp#L287
@ruoqianguo Do you have more information on this limitation? Is this something we can address but have not had time or is it a fundamental tensorrt limitation?

@narendasan The complete function looks like that https://github.com/pytorch/TensorRT/pull/921#discussion_r823279468. I think we can address this problem and i will try to support several indices input.

Just tested this on master d1768aa3d2c7d7d91d9f061e3e5dc5f976124dfe built in NGC pytorch:22.08-py3, but I'm still seeing the same errors running the above script:
root@2e1a9b9e2880:/opt/TensorRT# python /scripts/gathernd.py
PyTorch: torch.Size([2, 4])
WARNING: [Torch-TensorRT] - Truncating weight (constant in the graph) from Int64 to Int32
GatherND list indexing failed: isObject() INTERNAL ASSERT FAILED at "bazel-out/k8-opt/bin/external/libtorch/_virtual_includes/ATen/ATen/core/ivalue_inl.h":123, please report a bug to PyTorch. Expected Obj
ect but got None
PyTorch: torch.Size([2, 4])
WARNING: [Torch-TensorRT] - Truncating weight (constant in the graph) from Int64 to Int32
GatherND tensor indexing failed: isObject() INTERNAL ASSERT FAILED at "bazel-out/k8-opt/bin/external/libtorch/_virtual_includes/ATen/ATen/core/ivalue_inl.h":123, please report a bug to PyTorch. Expected O
bject but got None

@narendasan ^

In container pytorch/pytorch:1.12.0-cuda11.3-cudnn8-devel, I just tested this script on master d1768aa3d2c7d7d91d9f061e3e5dc5f976124dfe. And the results look like correct. Whether this results are related to container?
PyTorch: torch.Size([2, 4])
WARNING: [Torch-TensorRT] - For input x.1, found user specified input dtype as Float32. The compiler is going to use the user setting Float32
WARNING: [Torch-TensorRT] - If indices include negative values, the exported graph will produce incorrect results.
WARNING: [Torch-TensorRT TorchScript Conversion Context] - TensorRT was linked against cuDNN 8.4.1 but loaded cuDNN 8.3.2
WARNING: [Torch-TensorRT TorchScript Conversion Context] - The getMaxBatchSize() function should not be used with an engine built from a network created with
NetworkDefinitionCreationFlag::kEXPLICIT_BATCH flag. This function will always return 1.
WARNING: [Torch-TensorRT TorchScript Conversion Context] - The getMaxBatchSize() function should not be used with an engine built from a network created with
NetworkDefinitionCreationFlag::kEXPLICIT_BATCH flag. This function will always return 1.
TRT: torch.Size([2, 4])
PyTorch: torch.Size([2, 4])
WARNING: [Torch-TensorRT] - For input x.1, found user specified input dtype as Float32. The compiler is going to use the user setting Float32
WARNING: [Torch-TensorRT] - If indices include negative values, the exported graph will produce incorrect results.
WARNING: [Torch-TensorRT TorchScript Conversion Context] - TensorRT was linked against cuDNN 8.4.1 but loaded cuDNN 8.3.2
WARNING: [Torch-TensorRT TorchScript Conversion Context] - The getMaxBatchSize() function should not be used with an engine built from a network created with
NetworkDefinitionCreationFlag::kEXPLICIT_BATCH flag. This function will always return 1.
WARNING: [Torch-TensorRT TorchScript Conversion Context] - The getMaxBatchSize() function should not be used with an engine built from a network created with
NetworkDefinitionCreationFlag::kEXPLICIT_BATCH flag. This function will always return 1.
TRT: torch.Size([2, 4])

Verified on v1.2 again. Closing. @chaoz-dev please comment if you're still facing this issue.

Keeping this open since we are seeing some intermittent issues.

Tested the latest release 1.2.0 using NGC nvcr.io/nvidia/pytorch:22.09-py3. Running the above script, I'm still seeing the same errors:
PyTorch: torch.Size([2, 4])
WARNING: [Torch-TensorRT] - For input x.1, found user specified input dtype as Float32. The compiler is going to use the user setting Float32
WARNING: [Torch-TensorRT] - Truncating weight (constant in the graph) from Int64 to Int32
**GatherND list indexing failed: isObject() INTERNAL ASSERT FAILED at "bazel-out/k8-opt/bin/external/libtorch/_virtual_includes/ATen/ATen/core/ivalue_inl.h":123, please report a bug to PyTorch. Expected Object but got None**
PyTorch: torch.Size([2, 4])
WARNING: [Torch-TensorRT] - For input x.1, found user specified input dtype as Float32. The compiler is going to use the user setting Float32
WARNING: [Torch-TensorRT] - Truncating weight (constant in the graph) from Int64 to Int32
**GatherND tensor indexing failed: isObject() INTERNAL ASSERT FAILED at "bazel-out/k8-opt/bin/external/libtorch/_virtual_includes/ATen/ATen/core/ivalue_inl.h":123, please report a bug to PyTorch. Expected Object but got None**

@ruoqianguo for viz.
Your output above looks correct though, so it looks like this should work... checking the 1.2 release, looks like it should contain d1768aa3d2c7d7d91d9f061e3e5dc5f976124dfe, so we should be ahead of the working commit here...

@chaoz-dev is this ok to close?

This is good to close on my side


Fix [suggest]

Ref https://github.com/badges/shields/issues/2420#issuecomment-450400904

Is it possible to have a smoke test for this functionality?

It gets tested as part of the service test run. The test was failing, but hadn’t been picked up on yet.

I don't see any failures in https://github.com/badges/shields/commits/master - is service test run is a manual process?

We can’t run all the tests on every commit

Why? It is only 4 minutes in the daily log.

Our PR build runs in about 20 seconds; 4 minutes is much too slow. The other issue is that the service tests rely on external services. #927 is the discussion about that. We used to run the full suite on every commit to master, but stopped because they often fail for reasons outside our control. The thing we could use more help with is keeping up with fixing the failures identified tests (i.e. #1359).


[Bug]: Table data is not updated when page size is modified

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Table data is not updated when page size is modified
Steps To Reproduce

Drop a table and connect to data
Reduce page size on the property pane and observe the table height setting shorter
Now increase page size and observe the table height increasing but not filling up with data

Public Sample App
No response
Environment
Deploy Preview
Severity
High (Blocker to building or releasing)
Issue video log
No response
Version
cloud

@ramsaptami can you check if this is still happening. I was not able to repro it.

@ramsaptami could you check it? @jsartisan said that this should be solved.

@ramsaptami could you check it? @jsartisan said that this should be solved.

This is working now
A note though: this has seen multiple regressions @jsartisan @KelvinOm


WIP Use IPv6 loopback instead of IPv4

Use IPv6 loopback address instead of IPv4 one
Change migration proxy sockets to listen
on [::1] instead of <IP_ADDRESS>, unless IPv6 is disabled.
According to standard, listening on [::1] will also allow
incoming IPv4 connections, but not vice versa.
It does work without it because Golang net library use implicitly
an IPv6 socket, but its not readable and not maintainable.
This PR still considers IPv4 mapping is enabled.
Depends on: #3392
Depends on: #3386
None

hey, can we get some more context on what we're aiming to solve with this PR?

hey, can we get some more context on what we're aiming to solve with this PR?

Sure,
https://github.com/kubevirt/kubevirt/pull/3392#issuecomment-626559676
The reason IPv6 can connect to a socket that is listening to <IP_ADDRESS> / <IP_ADDRESS>
is that golang net library, has an abstraction (details in the link).
The team asked to use IPv6 so the code will be explicit, and handle the case,
where IPv4 Mapping is disabled.
This PR is for focusing on the <IP_ADDRESS> parts, which runs locally  between the virt-launcher and the virt-handler, in order to create the migration-proxy and update /etc/hosts, as a WA
for libvirt hostname resolving.
See please the whole discussion if needed, or i am here if needed.
Thanks

/hold
Added here a revert of a commit that cause IPv6 master to be broken,
i need the lane to work for my changes, but dont want this revert to be pushed.

/hold cancel

/hold
adding fc32 revert again, to check the lanes

/test pull-kubevirt-e2e-k8s-1.16

/test pull-kubevirt-e2e-k8s-1.17

/test  pull-kubevirt-e2e-k8s-1.17

/lgtm - please remove the non-relevant commits.

/lgtm - please remove the non-relevant commits.

thanks, just please keep in mind that after i do it , i will need #3429 merged , or cherry-picked

@AlonaKaplan
can you please lgtm again so i will ask a maintainer to review?

/hold cancel
removed the revert, tests might fail as long as we dont have #3429

Using #3429
And the guest-agent migration 5 tests pass
https://prow.apps.ovirt.org/view/gcs/kubevirt-prow/pr-logs/pull/kubevirt_kubevirt/3396/pull-kubevirt-e2e-kind-k8s-1.17.0-ipv6/1263340949879132160

/retest

Thanks Edy.
@vladikr
Can you review and approve please ?

/retest

Prow has a generic problem now, so the failing lane wont even start.
as can see on
https://prow.apps.ovirt.org/view/gcs/kubevirt-prow/pr-logs/pull/kubevirt_kubevirt/3396/pull-kubevirt-e2e-k8s-1.17/1265713623297167360

/retest

/retest

/test pull-kubevirt-e2e-k8s-1.17

/retest

/retest

/test pull-kubevirt-e2e-kind-k8s-1.17.0-ipv6


Min/Max Pool size modified on every Puppet run adding "" around the values

I have a datasource resource that I am creating
wildfly::standalone::datasources::datasource { 'jpnDS':
    config => {
      'driver-name'    => 'mysql',
      'min-pool-size'  => '10',
      'max-pool-size'  => '60',
      'connection-url' => 'jdbc:mysql://localhost:3306/DB',
      'jndi-name'      => 'java:jboss/datasources/jpnDS',
      'user-name'      => 'my_app',
      'password'       => 'my_app'
    }
  }

It looks to correctly create the datasource but then on every run it tries to modify the resource to append quotes " around the values.
$ sudo puppet apply --modulepath=/vagrant/modules/ /vagrant/manifests/jboss_wildfly9.pp

Warning: Config file /etc/puppet/hiera.yaml not found, using Hiera defaults
Faraday: you may want to install system_timer for reliable timeouts
Notice: Compiled catalog for wildfly.example.dev in environment production in 1.22 seconds
Notice: /Stage[main]/Whois_wildfly/Wildfly::Standalone::Datasources::Datasource[jpnDS]/Wildfly::Util::Resource[/subsystem=datasources/data-source=jpnDS]/Wildfly_resource[/subsystem=datasources/data-source=jpnDS]/state: state changed '{"max-pool-size"=>60, "min-pool-size"=>10}' to '{"max-pool-size"=>"60", "min-pool-size"=>"10"}'
Notice: Finished catalog run in 1.13 seconds

If you use 10 and 60 instead of '10' and '60', you'll probably won't have this problem.
This line is problem the source of this issue: https://github.com/biemond/biemond-wildfly/blob/master/lib/puppet/type/wildfly_resource.rb#L51
I'll change this line to should.map(&:to_s).sort == is.map(&:to_s).sort in order to force string to string comparison, do some tests and submit a PR.

I just tested and using 10 and 60 instead of '10' and '60' and I still see the issue.

I wasn't able to reproduce. Are you using wildfly 9?

Yes 9.0.0 final.
Sent from my iPhone

On Jul 22, 2015, at 11:43 AM, Jairo Junior<EMAIL_ADDRESS>wrote:
I wasn't able to reproduce. Are you using wildfly 9?
—
Reply to this email directly or view it on GitHub.

I can look at sanitizing and publishing my vagrant environment if it would help

No need. I tested with Wildfly 9 and was able to reproduce the issue.
As a matter of fact, I did a bad reading of your puppet apply output. Wildfly HTTP API is the one who is returning integers instead of strings. =/
I tested with the change proposed in my first comment and it's working on both versions. I'll submit a PR, but be aware that version 0.4.0 introduces some breaking changes (check CHANGELOG.md for more info).

pulled down the 3.7 tag and I can double confirm that its working for me also...thanks for the quick fix!


Remove Microsoft.PIX 2203.30

Newer versions available, removing for AppsAndFeaturesEntries in newer versions
Microsoft Reviewers: Open in CodeFlow

URL: https://query.prod.cms.rt.microsoft.com/cms/api/am/binary/RE4TrLd
Response Error: The remote server returned an error: (502) Bad Gateway.
(Automated message - build 593)

URL: https://query.prod.cms.rt.microsoft.com/cms/api/am/binary/RE4TrLd
Response Error: The remote server returned an error: (502) Bad Gateway.
(Automated message - build 593)


Feature/password reset with check fix

In response to bug filled @sebfie in https://github.com/lynndylanhurley/devise_token_auth/pull/317#issuecomment-128759414

Can we merge this PR plz?

LGTM!

Thx
Le 14 oct. 2015 20:19, "Brent"<EMAIL_ADDRESS>a écrit :

Merged #374.
—
Reply to this email directly or view it on GitHub.


[FLINK-22311] [connector/jdbc] Validate maxRetries for XA Sink

What is the purpose of the change
Validate maxRetries for XA Sink
Brief change log

Validate maxRetries for XA Sink
Add documentation

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: ( no )
The runtime per-record code paths (performance sensitive): ( no )
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (no
The S3 file system connector: ( no)

Documentation

Does this pull request introduce a new feature? ( no)
If yes, how is the feature documented? (docs )

@maver1ck Thanks for reporting and fixing this!

Would you mind changing the commit-msg from "Fix for FLINK-22311" to something more informative, like
"[FLINK-22311] [connector/jdbc] ....",  same as your PR title

I think precheck is good, but what do you think we set JdbcExecutionOptions.maxRetries = 0 directly and document it?

@curcur
Ad 1 and checkstyle - fixed.
Ad 2.
I checked JdbcExecutionOptions class and there is no method for setting maxRetries so I prefer to leave it with Preconditions

@flinkbot run azure

doc checks seem to fail:

@curcur
Should I merge your commit to this branch ?
Sorry I was absent last week.

@curcur
I left only code changes in this PR.

Hi @maver1ck I would like to facilitate merging your PR. Could you please rebase on the current master and run mvn spotless:apply?

@afedulov This is done

@maver1ck We've now moved the code from the JDBC connector to its own dedicated repository; please re-route this PR to https://github.com/apache/flink-connector-jdbc


Linked server in babelfish

Hi Support,
Can you please please help me on set up of linked server from one Babelfish to another Babelfish server.

Hi, something like this should work (reference):
exec sp_execute_postgresql 'create extension tds_fdw'
exec sp_addlinkedserver @server='babelfish_remote', @provider='SQLNCLI', @datasrc='<IP_ADDRESS>:1433'
exec sp_addlinkedsrvlogin @rmtsrvname='babelfish_remote', @useself='False', @rmtuser='jdbc_user', @rmtpassword='12345678'
select * from openquery(babelfish_remote, 'select pg_backend_pid()')

@staticlibs thanks for the query i configured linked server but getting this error when tried to query using linked server can you please suggest any. please find the error in the attachments

@kranthi1385
It looks like your build of Babelfish does not include the support for Linked Servers. You need to build it with FreeTDS lib (sybdb) and tds_fdw extension included, see details in the install guide.

@staticlibs
I have built FreeTDS lib and followed the same steps that are mentioned in install guide.

@kranthi1385
The flag ENABLE_TDS_LIB did not come through in your case, you can add it (and sybdb lib) in a Makefile instead:
diff --git a/contrib/babelfishpg_tsql/Makefile b/contrib/babelfishpg_tsql/Makefile
index f89bdbcf..b03e79c0 100644
--- a/contrib/babelfishpg_tsql/Makefile
+++ b/contrib/babelfishpg_tsql/Makefile
@@ -96,10 +96,10 @@ PG_CFLAGS += -g -Werror -Wfloat-conversion
 PG_CFLAGS += -fstack-protector-strong
 PG_CFLAGS += -fcommon -Wno-error=format-security -Wno-error=maybe-uninitialized
 PG_CFLAGS += -ggdb -O0 -fno-omit-frame-pointer -Wno-error=infinite-recursion
-PG_CFLAGS += -Wno-error=deprecated-declarations
+PG_CFLAGS += -Wno-error=deprecated-declarations -DENABLE_TDS_LIB -Wno-error=address
 PG_CPPFLAGS += -I$(TSQLSRC) -I$(PG_SRC) -DFAULT_INJECTOR -Wfloat-conversion

-SHLIB_LINK += -L$(ANTLR4_RUNTIME_LIB_DIR) $(ANTLR4_RUNTIME_LIB) -lcrypto $(ICU_LIBS)
+SHLIB_LINK += -L$(ANTLR4_RUNTIME_LIB_DIR) $(ANTLR4_RUNTIME_LIB) -lsybdb -lcrypto $(ICU_LIBS)

 UPGRADES = $(patsubst sql/upgrades/%.sql,sql/%.sql,$(wildcard sql/upgrades/*.sql))

@staticlibs
i am not able to resolve the issue still getting same error
Msg 33557097, Level 16, State 1, Line 1
Could not establish connection with remote server as use of TDS client library has been disabled. Please recompile source with 'ENABLE_TDS_LIB' flag to enable client library.
Can you please share step by step commands to setup

@kranthi1385
Please show your compilation output for linked_servers.c file, it should look like this, note that -DENABLE_TDS_LIB flag must be present there:
gcc -Wall -Wmissing-prototypes -Wpointer-arith -Wdeclaration-after-statement -Werror=vla -Wendif-labels -Wmissing-format-attribute -Wimplicit-fallthrough=3 -Wcast-function-type -Wshadow=compatible-local -Wformat-security -fno-strict-aliasing -fwrapv -fexcess-precision=standard -Wno-format-truncation -Wno-stringop-truncation -g -ggdb3 -O0 -DENABLE_TDS_LIB -Wno-error=address -g -Werror -Wfloat-conversion -fstack-protector-strong -fcommon -Wno-error=format-security -Wno-error=maybe-uninitialized -ggdb -O0 -fno-omit-frame-pointer -Wno-error=infinite-recursion -Wno-error=deprecated-declarations -fPIC -fvisibility=hidden -DENABLE_TDS_LIB -Wno-error=address -g -Werror -Wfloat-conversion -fstack-protector-strong -fcommon -Wno-error=format-security -Wno-error=maybe-uninitialized -ggdb -O0 -fno-omit-frame-pointer -Wno-error=infinite-recursion -Wno-error=deprecated-declarations -fPIC -fvisibility=hidden -I. -I/home/alex/projects/postgres/dev/postgresql_modified_for_babelfish/ -DFAULT_INJECTOR -Wfloat-conversion -I. -I./ -I/home/alex/projects/postgres/dist/include/server -I/home/alex/projects/postgres/dist/include/internal  -I -I/home/alex/projects/postgres/dev/postgresql_modified_for_babelfish/ -DFAULT_INJECTOR -Wfloat-conversion -I. -I./ -I/home/alex/projects/postgres/dist/include/server -I/home/alex/projects/postgres/dist/include/internal  -D_GNU_SOURCE -I/usr/include/libxml2   -c -o src/linked_servers.o src/linked_servers.c
src/linked_servers.c: In function ‘getOpenqueryTupdescFromMetadata’:
src/linked_servers.c:1026:59: warning: the comparison will always evaluate as ‘false’ for the address of ‘bind_typename’ will never be NULL [-Waddress]
src/linked_servers.c:967:49: note: ‘bind_typename’ declared here
src/linked_servers.c:1045:59: warning: the comparison will always evaluate as ‘false’ for the address of ‘bind_colname’ will never be NULL [-Waddress]
src/linked_servers.c:966:49: note: ‘bind_colname’ declared here

@staticlibs
please find the output.
gcc -Wall -Wmissing-prototypes -Wpointer-arith -Wdeclaration-after-statement -Werror=vla -Wendif-labels -Wmissing-format-attribute -Wimplicit-fallthrough=3 -Wcast-function-type -Wshadow=compatible-local -Wformat-security -fno-strict-aliasing -fwrapv -fexcess-precision=standard -Wno-format-truncation -Wno-stringop-truncation -g -g -O2 -fstack-protector-strong -Wformat -Werror=format-security -fno-omit-frame-pointer -fPIC -fvisibility=hidden -I./include/ -fvisibility=hidden -I/lib/x86_64-linux-gnu -DENABLE_TDS_LIB -Wall -I. -I./ -I/usr/include/postgresql/16/server -I/usr/include/postgresql/internal  -Wdate-time -D_FORTIFY_SOURCE=2 -D_GNU_SOURCE -I/usr/include/libxml2   -c -o src/deparse.o src/deparse.c
gcc -Wall -Wmissing-prototypes -Wpointer-arith -Wdeclaration-after-statement -Werror=vla -Wendif-labels -Wmissing-format-attribute -Wimplicit-fallthrough=3 -Wcast-function-type -Wshadow=compatible-local -Wformat-security -fno-strict-aliasing -fwrapv -fexcess-precision=standard -Wno-format-truncation -Wno-stringop-truncation -g -g -O2 -fstack-protector-strong -Wformat -Werror=format-security -fno-omit-frame-pointer -fPIC -fvisibility=hidden -shared -o tds_fdw.so src/tds_fdw.o src/options.o src/deparse.o -L/usr/lib/x86_64-linux-gnu  -Wl,-Bsymbolic-functions -Wl,-z,relro -Wl,-z,now -L/usr/lib/llvm-10/lib  -Wl,--as-needed  -fvisibility=hidden -L/lib/x86_64-linux-gnu -lsybdb
cp sql/tds_fdw.sql sql/tds_fdw--2.0.4.sql
cp README.md README.tds_fdw.md
/usr/bin/clang-10 -Wno-ignored-attributes -fno-strict-aliasing -fwrapv -Wno-unused-command-line-argument -O2  -I./include/ -fvisibility=hidden -I/lib/x86_64-linux-gnu -DENABLE_TDS_LIB -Wall -I. -I./ -I/usr/include/postgresql/16/server -I/usr/include/postgresql/internal  -Wdate-time -D_FORTIFY_SOURCE=2 -D_GNU_SOURCE -I/usr/include/libxml2  -flto=thin -emit-llvm -c -o src/tds_fdw.bc src/tds_fdw.c
src/tds_fdw.c:3953:8: warning: unused variable 'is_sql_server' [-Wunused-variable]
bool            is_sql_server = true;
^
src/tds_fdw.c:3954:17: warning: unused variable 'buf' [-Wunused-variable]
StringInfoData buf;
^
2 warnings generated.
/usr/bin/clang-10 -Wno-ignored-attributes -fno-strict-aliasing -fwrapv -Wno-unused-command-line-argument -O2  -I./include/ -fvisibility=hidden -I/lib/x86_64-linux-gnu -DENABLE_TDS_LIB -Wall -I. -I./ -I/usr/include/postgresql/16/server -I/usr/include/postgresql/internal  -Wdate-time -D_FORTIFY_SOURCE=2 -D_GNU_SOURCE -I/usr/include/libxml2  -flto=thin -emit-llvm -c -o src/options.bc src/options.c
/usr/bin/clang-10 -Wno-ignored-attributes -fno-strict-aliasing -fwrapv -Wno-unused-command-line-argument -O2  -I./include/ -fvisibility=hidden -I/lib/x86_64-linux-gnu -DENABLE_TDS_LIB -Wall -I. -I./ -I/usr/include/postgresql/16/server -I/usr/include/postgresql/internal  -Wdate-time -D_FORTIFY_SOURCE=2 -D_GNU_SOURCE -I/usr/include/libxml2  -flto=thin -emit-llvm -c -o src/deparse.bc src/deparse.c

@kranthi1385
This is the output from compilation of the tds_fdw extension. And I was pointing out the linked_servers.c that is a part of babelfishpg_tsql extension. Please check that -DENABLE_TDS_LIB is passed to linked_servers.c.

@kranthi1385
When building babelfishpg_tsql extension you need to make sure that -DENABLE_TDS_LIB compiler flag is actually passed to gcc. The definition ENABLE_TDS_LIB is used in linked_servers.c (and in the corresponding header) to decide whether to compile in the support for linked servers or not. The Babelfish binaries you are using were apparently built without this flag. One of the ways to enable this flag is to apply this patch to Babelfish sources before running the build. Your build of tds_fdw is likely okay (though the mix of gcc and clang in your log is unusual), but the tds_fdw extension cannot be used from Babelfish unless Babelfish itself was compiled with the support for linked servers.


Include more details in the release notes

I think it would be a great idea if you included more details in the release notes when you release a new version, as you guys just published 0.141 around 10 hours ago, but the release notes for the release are incredibly basic, and I noticed you guys made a lot of commits for the 0.141 release before you guys actually released it, so you guys might want to make the release notes more detailed, as in detailing all the changes instead of just covering the "Notable changes", as it would prove really useful to the people who want to know all the changes.

@kevinsawicki You forgot to close this issue. :smiley:


The difference between XFG and PDG?

The corresponding paper claim that a novel representation of IR, called conteXtual Flow Graph (XFG), was propsed. However, I realize that this representation is similar with traditional Program Dependence Graph.

Good question. In the paper, there is a direct comparison between XFGs and PDGs. In particular, the XFG can be generated in a linear time with the number of statements, where PDGs are generated in squared time w.r.t. the number of statements. In large codebases, such as TensorFlow, this makes a huge difference.

@tbennun Thanks for your explanation. I have looked into the creation process of XFG and realize the differences between PDG:

The data dependence only resides in the statements belongs to the same basic block. The exceptions are the edges between identifiers and their root nodes.
The control dependence is more like the edge in CFG, rather than the edge the Control Dependence Graph. The latter is build on top of dominator tree, which is time-consuming.

Yes, both statements are correct.
Since this is resolved, I'll close the issue now.


Exception during a webclient request | Google drive

Hi, recently i started to use keeanywhere and when i am trying to open a database in google drive, this happens

(Spanish)
help?

@Mataisplays That's great! If you don't mind my asking, what was the problem and solution? If someone ever comes with the same issue, maybe they can find the answer here.


feat(idp):ui changes

Description
IDP UI Changes
Why
UI Enhancement
Issue
n/a
Checklist
Please delete options that are not relevant.

[X] I have followed the contributing guidelines
[X] I have performed a self-review of my own code
[X] I have successfully tested my changes locally

IP issue created: https://gitlab.eclipse.org/eclipsefdn/emo-team/iplab/-/issues/10500

@nidhigarg-bmw you can commit the dependencies file now

Waiting for IP team to review/approve the 21.1.0 version of shared component

please update the dependencies files which should now be created from the check dependencies action
direct download link: https://github.com/eclipse-tractusx/portal-frontend/suites/16285154537/artifacts/929758347


[release/v1.3] Prepare OSM v1.3.4

What this PR does / why we need it:
Following the release process, bumping OSP versions to v1.3.4 to be in line with the upcoming OSM patch release to address Flatcar issues.
Which issue(s) this PR fixes:

Fixes #
What type of PR is this?
/kind chore
Special notes for your reviewer:
Does this PR introduce a user-facing change? Then add your Release Note here:

NONE

Documentation:

NONE

/cc @ahmedwaleedmalik


model functions need to know which chunk of fits they are processing

this information may be needed in order to know where to access the user info.  If the user info consists of all of the X values of the fit, for example, the model function needs to know which chunk of fits it is processing, so that it can properly index the user info.

related to issue #39, what about letting the model decide which part of the user_info and data and weights it will need. otherwise we will make assumptions about the user_info (like it has to fit into the GPU) limiting ourselves (data and weights does not have to fit into GPU memory)

The model function does decide what parts it needs, but the model
function is not called until everything is already loaded onto the GPU.
Hence, we need to make sure that all of user info is already there.
On 23/01/2017 11:16, jkfindeisen wrote:

related to issue #39 https://github.com/gpufit/GPUfit/issues/39, what
about letting the model decide which part of the user_info and data and
weights it will need. otherwise we will make assumptions about the
user_info (like it has to fit into the GPU) limiting ourselves (data and
weights does not have to fit into GPU memory)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gpufit/GPUfit/issues/40#issuecomment-274448522, or
mute the thread
https://github.com/notifications/unsubscribe-auth/ADb8MUUD4i70hXjKlh34aEa7Fx51C_vTks5rVH4GgaJpZM4Lq05O.

Or we need to invoke the model function earlier.  For now it's fine to load the user data.

fixed


Added AutoSaveMiddleWare to 18.bot-authentication to simplify sample

Fixes  #1338
Proposed Changes
Added AutoSaveMiddleWare to this sample so it is easier to use and doesn't confuse people learning OAuth flows.
Without saving state the proposed WaterfallDialog will never reach all of its steps and confuse learners without any benefit.

Closing the PR as this is part of the new samples coming soon. See #1338 for details.


Sample given in this page will result into error.

It seems that XDT transforms have changed so current sample will give the following error
No element in the source document matches '/configuration/system.webServer/modules'
To solve this, a new xdt attribute must be added.
<system.webServer xdt:Transform="InsertIfMissing">
<modules xdt:Transform="InsertIfMissing">
<add name="ContosoUtilities" type="Contoso.Utilities" xdt:Transform="InsertIfMissing"/>
<add name="MyNuModule" type="Sample.MyNuModule"  xdt:Transform="InsertIfMissing"/>
</modules>
</system.webServer>
xdt:Transform="InsertIfMissing"

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

ID: 40bb9dee-46d6-4064-83e8-ba2e22ac6afe
Version Independent ID: aac0bbd8-645b-ccf8-bec9-e0d970c8b089
Content: Source and config file transformations for NuGet packages
Content Source: docs/create-packages/Source-and-Config-File-Transformations.md
Product: nuget
GitHub Login: @karann-msft
Microsoft Alias: karann

Think this one is similar/related https://github.com/NuGet/docs.microsoft.com-nuget/issues/1907


[REEF-599]:Move DriverConfigurationProviders from Org.Apache.REEF.Cli…

…ent.Local to Org.Apache.REEF.Client.API
This addressed the issue by moving DriverConfigurationProviders from Org.Apache.REEF.Client.Local to Org.Apache.REEF.Client.API
JIRA: REEF-599
This closes #

This looks good, I will merge.


-d:rc Vs -f

Bug Short Description:
-d:rc is not the same as vanilla -f :grey_question:
Nim help says -f, --forceBuild    Force rebuilding of all modules.
Should we pass -f instead of our custom -d:rc :grey_question:
Operating System:
Choose 1 or more. Mark the checkbox it with an X.

[x] Other or Unknown or Unsupported.

Bug Severity:
Choose only 1. Mark the checkbox it with an X.

[X] Low, Minor, Aesthetic, Small, Detail, Trivial.
[ ] Medium, Normal, Standard.
[ ] High, Important, Major.
[ ] Critical, Catastrophic, Epic Fail.
[ ] Other or Unknown.

Version
Semantic Version of this software the Bug Reproduces into. Type in the Version Number
    5.0.0

I have no opinion about the syntax: rc or f. But -f will not do anything different than rc? We still need to manually catch it and recompile, right?

Both do the same.
But if we use a custom -d:rc we have to document it, maintain it, etc.
We should pass -f to the nimwc_main and it should do the same as -d:rc theoretically.

I'm not totally sure, that I understand.
-d:rc does only tell nimwc.nim's proc startupCheck() to compile nimwc_main.nim. I can't see how passing -f will solve that since nimwc_main is not a module, but an individual file we specify when to compile.
https://github.com/ThomasTJdev/nim_websitecreator/blob/37c933c7f65ca451e956b8728a6c7d084f3ac0f8/nimwc.nim#L224-L228

Oh, maybe you are right.


:ambulance: pos_product_available_negative

Popup alert-message for exceding available quantity of product by orderlines

will be tested and merged in #1154


gitlab-runner: 10.5.0 -> 10.6.0

Motivation for this change
Update to latest version
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build gitlab-runner

Success on x86_64-linux (full log)
Attempted: gitlab-runner
Partial log (click to expand)
/nix/store/7xyjb7cr8b92bqhs6vgnwxcj050jbyil-gitlab-runner-10.6.0/share/go/src/gitlab.com/gitlab-org/gitlab-runner/dockerfiles/alpine/entrypoint: interpreter directive changed from "/bin/bash" to "/nix/store/q1g0rl8zfmz7r371fp5p42p4acmv297d-bash-4.4-p19/bin/bash"
/nix/store/7xyjb7cr8b92bqhs6vgnwxcj050jbyil-gitlab-runner-10.6.0/share/go/src/gitlab.com/gitlab-org/gitlab-runner/dockerfiles/build/gitlab-runner-cache: interpreter directive changed from "/bin/sh" to "/nix/store/q1g0rl8zfmz7r371fp5p42p4acmv297d-bash-4.4-p19/bin/sh"
/nix/store/7xyjb7cr8b92bqhs6vgnwxcj050jbyil-gitlab-runner-10.6.0/share/go/src/gitlab.com/gitlab-org/gitlab-runner/dockerfiles/build/gitlab-runner-service: interpreter directive changed from "/bin/sh" to "/nix/store/q1g0rl8zfmz7r371fp5p42p4acmv297d-bash-4.4-p19/bin/sh"
/nix/store/7xyjb7cr8b92bqhs6vgnwxcj050jbyil-gitlab-runner-10.6.0/share/go/src/gitlab.com/gitlab-org/gitlab-runner/dockerfiles/build/gitlab-runner-build: interpreter directive changed from "/bin/bash" to "/nix/store/q1g0rl8zfmz7r371fp5p42p4acmv297d-bash-4.4-p19/bin/bash"
/nix/store/7xyjb7cr8b92bqhs6vgnwxcj050jbyil-gitlab-runner-10.6.0/share/go/src/gitlab.com/gitlab-org/gitlab-runner/dockerfiles/ubuntu/entrypoint: interpreter directive changed from "/bin/bash" to "/nix/store/q1g0rl8zfmz7r371fp5p42p4acmv297d-bash-4.4-p19/bin/bash"
/nix/store/7xyjb7cr8b92bqhs6vgnwxcj050jbyil-gitlab-runner-10.6.0/share/go/src/gitlab.com/gitlab-org/gitlab-runner/packaging/root/usr/share/gitlab-runner/post-install: interpreter directive changed from "/bin/sh" to "/nix/store/q1g0rl8zfmz7r371fp5p42p4acmv297d-bash-4.4-p19/bin/sh"
/nix/store/7xyjb7cr8b92bqhs6vgnwxcj050jbyil-gitlab-runner-10.6.0/share/go/src/gitlab.com/gitlab-org/gitlab-runner/packaging/root/usr/share/gitlab-runner/clear-docker-cache: interpreter directive changed from "/bin/sh" to "/nix/store/q1g0rl8zfmz7r371fp5p42p4acmv297d-bash-4.4-p19/bin/sh"
/nix/store/7xyjb7cr8b92bqhs6vgnwxcj050jbyil-gitlab-runner-10.6.0/share/go/src/gitlab.com/gitlab-org/gitlab-runner/packaging/root/usr/share/gitlab-runner/pre-remove: interpreter directive changed from "/bin/sh" to "/nix/store/q1g0rl8zfmz7r371fp5p42p4acmv297d-bash-4.4-p19/bin/sh"
checking for references to /tmp/nix-build-gitlab-runner-10.6.0.drv-0 in /nix/store/7xyjb7cr8b92bqhs6vgnwxcj050jbyil-gitlab-runner-10.6.0...
/nix/store/iz466grn1zm7gsp0nvw6jhs8gmfdpq24-gitlab-runner-10.6.0-bin

Success on aarch64-linux (full log)
Attempted: gitlab-runner
Partial log (click to expand)
/nix/store/ais361h5c3ykqyzw10xj1wf9g4qrdmkx-gitlab-runner-10.6.0/share/go/src/gitlab.com/gitlab-org/gitlab-runner/packaging/root/usr/share/gitlab-runner/clear-docker-cache: interpreter directive changed from "/bin/sh" to "/nix/store/3gg2p6n4kv2f0lsxd41f5iz1ivkbzyzr-bash-4.4-p19/bin/sh"
/nix/store/ais361h5c3ykqyzw10xj1wf9g4qrdmkx-gitlab-runner-10.6.0/share/go/src/gitlab.com/gitlab-org/gitlab-runner/packaging/root/usr/share/gitlab-runner/post-install: interpreter directive changed from "/bin/sh" to "/nix/store/3gg2p6n4kv2f0lsxd41f5iz1ivkbzyzr-bash-4.4-p19/bin/sh"
/nix/store/ais361h5c3ykqyzw10xj1wf9g4qrdmkx-gitlab-runner-10.6.0/share/go/src/gitlab.com/gitlab-org/gitlab-runner/packaging/root/usr/share/gitlab-runner/pre-remove: interpreter directive changed from "/bin/sh" to "/nix/store/3gg2p6n4kv2f0lsxd41f5iz1ivkbzyzr-bash-4.4-p19/bin/sh"
/nix/store/ais361h5c3ykqyzw10xj1wf9g4qrdmkx-gitlab-runner-10.6.0/share/go/src/gitlab.com/gitlab-org/gitlab-runner/scripts/changelog2releasepost: interpreter directive changed from "/bin/sh" to "/nix/store/3gg2p6n4kv2f0lsxd41f5iz1ivkbzyzr-bash-4.4-p19/bin/sh"
/nix/store/ais361h5c3ykqyzw10xj1wf9g4qrdmkx-gitlab-runner-10.6.0/share/go/src/gitlab.com/gitlab-org/gitlab-runner/scripts/check_race_conditions: interpreter directive changed from "/bin/sh" to "/nix/store/3gg2p6n4kv2f0lsxd41f5iz1ivkbzyzr-bash-4.4-p19/bin/sh"
/nix/store/ais361h5c3ykqyzw10xj1wf9g4qrdmkx-gitlab-runner-10.6.0/share/go/src/gitlab.com/gitlab-org/gitlab-runner/scripts/codequality: interpreter directive changed from "/bin/sh" to "/nix/store/3gg2p6n4kv2f0lsxd41f5iz1ivkbzyzr-bash-4.4-p19/bin/sh"
/nix/store/ais361h5c3ykqyzw10xj1wf9g4qrdmkx-gitlab-runner-10.6.0/share/go/src/gitlab.com/gitlab-org/gitlab-runner/tests/test_installation.sh: interpreter directive changed from "/bin/bash" to "/nix/store/3gg2p6n4kv2f0lsxd41f5iz1ivkbzyzr-bash-4.4-p19/bin/bash"
/nix/store/ais361h5c3ykqyzw10xj1wf9g4qrdmkx-gitlab-runner-10.6.0/share/go/src/gitlab.com/gitlab-org/gitlab-runner/vendor/github.com/docker/docker/hack/generate-authors.sh: interpreter directive changed from "/bin/bash" to "/nix/store/3gg2p6n4kv2f0lsxd41f5iz1ivkbzyzr-bash-4.4-p19/bin/bash"
checking for references to /build in /nix/store/ais361h5c3ykqyzw10xj1wf9g4qrdmkx-gitlab-runner-10.6.0...
/nix/store/nq792lvalpmm6y1adhqm7idgbayvx87n-gitlab-runner-10.6.0-bin

Success on x86_64-darwin (full log)
Attempted: gitlab-runner
Partial log (click to expand)
/nix/store/q4sixas1d6lq84xj8q5lfp103kc33vwv-gitlab-runner-10.6.0/share/go/src/gitlab.com/gitlab-org/gitlab-runner/scripts/check_race_conditions: interpreter directive changed from "/bin/sh" to "/nix/store/8hdxjv3m14jgdnawi5l0mz2h59qkk2g9-bash-4.4-p19/bin/sh"
/nix/store/q4sixas1d6lq84xj8q5lfp103kc33vwv-gitlab-runner-10.6.0/share/go/src/gitlab.com/gitlab-org/gitlab-runner/scripts/changelog2releasepost: interpreter directive changed from "/bin/sh" to "/nix/store/8hdxjv3m14jgdnawi5l0mz2h59qkk2g9-bash-4.4-p19/bin/sh"
/nix/store/q4sixas1d6lq84xj8q5lfp103kc33vwv-gitlab-runner-10.6.0/share/go/src/gitlab.com/gitlab-org/gitlab-runner/scripts/codequality: interpreter directive changed from "/bin/sh" to "/nix/store/8hdxjv3m14jgdnawi5l0mz2h59qkk2g9-bash-4.4-p19/bin/sh"
/nix/store/q4sixas1d6lq84xj8q5lfp103kc33vwv-gitlab-runner-10.6.0/share/go/src/gitlab.com/gitlab-org/gitlab-runner/dockerfiles/alpine/entrypoint: interpreter directive changed from "/bin/bash" to "/nix/store/8hdxjv3m14jgdnawi5l0mz2h59qkk2g9-bash-4.4-p19/bin/bash"
/nix/store/q4sixas1d6lq84xj8q5lfp103kc33vwv-gitlab-runner-10.6.0/share/go/src/gitlab.com/gitlab-org/gitlab-runner/dockerfiles/build/gitlab-runner-cache: interpreter directive changed from "/bin/sh" to "/nix/store/8hdxjv3m14jgdnawi5l0mz2h59qkk2g9-bash-4.4-p19/bin/sh"
/nix/store/q4sixas1d6lq84xj8q5lfp103kc33vwv-gitlab-runner-10.6.0/share/go/src/gitlab.com/gitlab-org/gitlab-runner/dockerfiles/build/gitlab-runner-service: interpreter directive changed from "/bin/sh" to "/nix/store/8hdxjv3m14jgdnawi5l0mz2h59qkk2g9-bash-4.4-p19/bin/sh"
/nix/store/q4sixas1d6lq84xj8q5lfp103kc33vwv-gitlab-runner-10.6.0/share/go/src/gitlab.com/gitlab-org/gitlab-runner/dockerfiles/build/gitlab-runner-build: interpreter directive changed from "/bin/bash" to "/nix/store/8hdxjv3m14jgdnawi5l0mz2h59qkk2g9-bash-4.4-p19/bin/bash"
/nix/store/q4sixas1d6lq84xj8q5lfp103kc33vwv-gitlab-runner-10.6.0/share/go/src/gitlab.com/gitlab-org/gitlab-runner/dockerfiles/ubuntu/entrypoint: interpreter directive changed from "/bin/bash" to "/nix/store/8hdxjv3m14jgdnawi5l0mz2h59qkk2g9-bash-4.4-p19/bin/bash"
/nix/store/q4sixas1d6lq84xj8q5lfp103kc33vwv-gitlab-runner-10.6.0/share/go/src/gitlab.com/gitlab-org/gitlab-runner/vendor/github.com/docker/docker/hack/generate-authors.sh: interpreter directive changed from "/bin/bash" to "/nix/store/8hdxjv3m14jgdnawi5l0mz2h59qkk2g9-bash-4.4-p19/bin/bash"
/nix/store/5sh7a8ayz8cg54vm2h8jabcr61gp5phm-gitlab-runner-10.6.0-bin


Add minimal support for ssl module using Mbed TLS and sockets

Details of changes:

Add support for ssl client in binary and passive modes, with no certificate verification
Add APIs to otp_socket so it can be called from ssl bio callbacks
Fix a bug in lwIP otp_socket's recv revealed by ssl tests
Fix a bug in BSD otp_socket's recvfrom revealed by refactoring
Fix a bug in esp32 tests where main context and its resources were not properly destroyed
Update documentation and workflows to reflect the requirement on Mbed TLS
Fix exported types of inet module

This code was tested on:

Pico-W
ESP32 using ESP-IDF 5.1 release branch
Unix (macOS)

using atomvm_netbench associated test.
The test takes 0.5s with Erlang/OTP or AtomVM on macOS.
It takes 7.3s then 1.0s on ESP32
It takes 12.0s then from 2.2s to 2.6s on Pico-W
These changes are made under both the "Apache 2.0" and the "GNU Lesser General
Public License 2.1 or later" license terms (dual license).
SPDX-License-Identifier: Apache-2.0 OR LGPL-2.1-or-later

Mbed TLS from armv5 CI build is too old. I need to either detect the version and disable SSL or handle the older API

Looks all good. Maybe this might be the first component implementing this structure: https://github.com/atomvm/AtomVM/issues/930
Otherwise we can just merge it, do this on a second step.

Is it your feeling that this is not mature enough to have a place in the Programmer's Guide?  Do we want to implement PXIX certificate chain validation first?  (As it is, I believe the current implementation would be vulnerable to MitM attacks, without authentication).

Looks all good. Maybe this might be the first component implementing this structure: #930 Otherwise we can just merge it, and do this on a second step.

There are still a lot of open points in #930 proposal, so let's just focus in merging this PR, and let's take care of reorganizing code as soon all open points are closed.

Is it your feeling that this is not mature enough to have a place in the Programmer's Guide? Do we want to implement PXIX certificate chain validation first? (As it is, I believe the current implementation would be vulnerable to MitM attacks, without authentication).

Proper documentation for an ssl module would both require a stable API and a reasonable amount of work, in order to explain caveats and how it differs from OTP one. This kind of work deserves a standalone PR, so let's merge this PR and document ssl module properly as soon as we are ready.


Handle auth errors

Currently on signing up if an account with the same email exists, the error is'nt handled
Similarly on logging in if account does'nt exist, it simple shows axios error
Find code for the same in /Users/shivanshgupta/Desktop/projects/zapier/primary-backend/src/router/user.ts

Hey @Shivg2901 . I would like to solve this issue, kindly assign it to me

@swymbnsl
Feel free to be creative
Either improve the existing code or you can also integrate authjs or some external auth provider.
We are flexible with that


Optional inclusion of ACS Commons Clientlibs

Not sure if the ship has sailed on this, but it might be nice to start optionally including clientlibs that effect (and potentially break) major UIs.
For @HitmanInWis's we adjusted gave each clientlibs a discrete name and for someone to use it, they must create a proxy Clientlib that includes our feature clientlib into cq.widgets or cq.authoring.editor based on their needs.
We could even manage our own "acs-commons.widgets.all" if someone always wanted everything? I dont like that we have a slew of stuff already added to the OOTB categories, but dont htink we can do much about that.
WDYT? good, bad, overkill?
https://adobe-consulting-services.github.io/acs-aem-commons/features/shared-component-properties.html > How to Use > 2. Adding the Client Libraries for Authoring
//cc @justinedelson, @schoudry, @bstopp

@davidjgonzalez i think it's a good idea; though additional configuration needed and some bugs might be missed (as the logic executes only after adding respective proxy clientlib) upside is - it doesn't break UI on hotfix or FP installs (which ACS commons almost never tests with) or upgrades

I like the idea. Auto-including the ACS widgets is akin to auto-loading services. We require configs to be defined to enable any feature. Same should be for custom widgets; yes, maybe it requires an extra development step but creating aggregate clientlibs is almost standard ops now.


Add Annual Technology Baseline Calculators

Insert URLs to the project here: https://github.com/NREL/ATB-calc

[x] The projects is active, documented, open source licensed, shows usage from external parties and is directly targeting environmental sustainability. Find more details in the Contribution Guide.

All issues labeled as Good First Issue of the project listed on OpenSustain.tech will be visible on ClimateTriage.com. This is a great way to welcome new community members to your project.
All new projects listed will be posted on our Mastodon channel.

@brtietz @mikebannis


Change workflow interactions to use queue

Acceptance Criteria

[ ] Change remaining calls to workflow/camunda to use queue

This is Duplicated


Make All Generated Classes internal

Description of Change
This PR changes generated classes, i.e., code generated by TextAlignmentExtensionsGenerator from public to internal.
The bug in #158 is caused by two matching public classes being generated in both the .NET MAUI library (eg net6.0-ios) and the referenced class library (e.g. net6.0).
📦MyMauiSolution
 ┣ 📂MyMauiApp
    - <TargetFrameworks>net7.0-android;net7.0-ios;net7.0-maccatalyst</TargetFrameworks>
 ┗ 📂MyLibrary
    - <TargetFramework>net6.0</TargetFramework>

Linked Issues

Fixes #158

PR Checklist

[x] Has a linked Issue, and the Issue has been approved(bug) or Championed (feature/proposal)
[x] Has tests (if omitted, state reason in description)
[x] Has samples (if omitted, state reason in description)
[ ] Rebased on top of main at time of PR
[x] Changes adhere to coding standard

Additional information
This aligns with a similar discussion on the CommunityToolkit.Maui repo: https://github.com/CommunityToolkit/Maui/discussions/756#discussioncomment-4125731

NuGet Package Generated by this PR: https://dev.azure.com/dotnet/696bc9fd-f160-4e97-a1bd-7cbbb3b58f66/_apis/build/builds/83337/artifacts?artifactName=nuget&api-version=7.0&%24format=zip


Add multi-host support for Group

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
❌

Fixed issues
#166

Need changelog

Done


Typescriptify whip.js and whep.js ?

@murillo128 how do you feel about having whip.js and whep.js inTypescript?
I don't feel strongly one way or the other, but as I am writing type files for both WHIPClient and
WHEPClient, in order to use them in Typescript, I am wondering if they should be in Typescript.
Do you have any thoughts?

I don't think I would rewrite the lib in typescript, but I am not against adding typescript inline annotations as we have done in https://github.com/medooze/semantic-sdp-js

There is now a PR for adding typescript type definition files for both whip and whep.
It looks like in semantic-sdp-js typescript type information was added using the JSDoc commenting method.
I didn't do that here, since there isn't even a build script etc yet, I just dropped in the typescript xxx.d.ts files which get searched for and used if found by tsc.
This seemed to solve my typescript errors when using either whip.js or whep.js in a typescript project.

done in https://github.com/medooze/whip-whep-js/commit/fd5508e503252cea1879a6894cb4c12f26ed91a6


support configuration of antenna_delay property

Summary: We need to be able to set antenna_delay therefore changing the API and config format.
Reviewed By: abulimov
Differential Revision: D42228455

Codecov Report
Base: 58.82% // Head: 58.80% // Decreases project coverage by -0.01% :warning:

Coverage data is based on head (2bf34fb) compared to base (12f938f).
Patch coverage: 85.18% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #243      +/-   ##
==========================================
- Coverage   58.82%   58.80%   -0.02%
==========================================
  Files          99       99
  Lines        9376     9380       +4
==========================================
+ Hits         5515     5516       +1
- Misses       3391     3393       +2
- Partials      470      471       +1

Impacted Files
Coverage Δ

calnex/cmd/config.go
17.64% <0.00%> (ø)

calnex/config/config.go
87.82% <100.00%> (+0.32%)
:arrow_up:

timestamp/timestamp_linux.go
58.75% <0.00%> (-1.88%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Command Token missing a case

You defined COMMAND_TOKEN='regexp:\\([a-zA-Z]+|.)' but this will trigger an error message if the command \ is followed by a new line character although this is possible.
You can use the file pdftex-t.tex to see what I mean.

Thanks for the heads-up, this is indeed the case. According to the JFlex manual, the . matches [^\r\n\u2028\u2029\u000B\u000C\u0085]. We will make sure to fix this soon.
Also, thanks for your interest in this plugin, we really appreciate it. If you have any other suggestions, make sure to let us know!


(Nontrivial) use-cases for --take-rand?

Just wondering what a (non-trivial) use of the --take-rand option is, especially now that we have a well-thought-out --print-rand option, which allows users a more "accurate" handle on how long the resulting list will be (since it makes its cuts later in the process).
If we can't come up with a non-trivial use for it, I'm not necessarily in favor of removing the option -- I think the symmetry with --take-first and --print-rand is nice to have. But that's a natural follow-up...

Found one: Let's say you have an alphabetically sorted 100,000-word list (or multiple lists that total 100,00 words) as your input. You want to make a 7,776-word list. And let's say you want these 7,776 words to be Schlinkert pruned, a time-intense process.
tidy -m 3 -K --print-rand 7776 100k.txt will take hours and hours to run, because Tidy is running the Schlinkert prune on almost all 100,000 words (mercifully, words under 3 characters won't be passed on to the Schlinkert prune process).
Running tidy --take-rand 12000 -m 3 -K --print-rand 7776 100k.txt is much faster, since take-rand cuts the inputted 100k words down to 12k before further processing. On my laptop, it finished in under 5 minutes.


Update @peculiar/react-components

Updated @peculiar/react-components to v.1.1.1

@OlhaPeculiar Could you check here https://fortify-tools.vercel.app/ please

All work correct on  https://fortify-tools.vercel.app/


windows client connect failed because connect.go:171: wgtypes: failed to parse base64-encoded key: illegal base64 data at input byte 8

Describe the problem
I created a Setup Key (value: DFA5BE66-8C79-4019-9CE6-720B09F0E632), configured it on the Windows client (version 0.24.2), and then tried to connect. However, the connection failed.
I checked the logs and it prompted: ERRO client/internal/connect.go:171: wgtypes: failed to parse base64-encoded key: illegal base64 data at input byte 8.
However, the above issue did not appear on the Linux client. (Command: netbird up --setup-key DFA5BE66-8C79-4019-9CE6-720B09F0E632).
I checked the source code and found that there was an error in the logic, which caused the "illegal base64 data at input byte 8" error when running on both Windows and Linux.
The exception code is as follows:
import "encoding/base64"
import log "github.com/sirupsen/logrus"
func main() {
var s = "DFA5BE66-8C79-4019-9CE6-720B09F0E632"
decodeString, err := base64.StdEncoding.DecodeString(s)
if err != nil {
log.Error(err)
}
println(decodeString)
}
To Reproduce
Steps to reproduce the behavior:

created a Setup Key
configured it on the Windows client
tried to connect
See log

Expected behavior
A clear and concise description of what you expected to happen.
NetBird status -d output:
If applicable, add the output of the netbird status -d command
Screenshots

Additional context
Add any other context about the problem here.

Hi @fage520, indeed the SetupKeys are not base64 encoded by design.
The error you see
ERRO client/internal/connect.go:171: wgtypes: failed to parse base64-encoded key: illegal base64 data at input byte 8.

is more likely related to your PreSharedKey from the config(C:\ProgramData\Netbird\config.json on Windows).
See the output of netbird --help:
  -p, --preshared-key string    Sets Wireguard PreSharedKey property. If set, then only peers that have the same key can communicate.

Could it be that by any chance to set that value on the Windows machine? You can check that in your config.json.


[Documentation] Demo working with backbone DO NOT MERGE

What was done here

[x] Setting the navigate, back and start functions (Router initialization)
[x] Setting the global link for a good backbone navigation on the application (Global link initialization)
[x] Using of the updated components on the good way (button-back, menu)

Where to look, what to do then

Initialize the router : https://github.com/KleeGroup/focus-demo-app/blob/e6f6177825893e5191e4a65d483919b882d62091/app/initializer/scripts/router-initializer.js
The initialization should look like this. You now have to import your Backbone dependency and a function : setNavigationFunctions which set the navigate, back  and start functions

Before, in the code, we used to do this import history from 'focus-core/history', then that history.navigate('#search/advanced', true)
Now do this import {navigate} from 'focus-core/history', then that navigate('#search/advanced', true);. It will be the same for back and start
(For start, please look here : https://github.com/KleeGroup/focus-demo-app/blob/d726e8d20cb6e77fdafb0f2842fef0fb346828e4/app/application.js)

The main goal of this PR is to be used as documentation for all projects to upgrade their version of Focus core.

In focus core, we no longer depend on Backbone, but we don't remove all our code inside the Router to not break compatibility with the projects.


Support install target in qmake project

Signed-off-by: Maxime Gervais<EMAIL_ADDRESS>

Thanks.


Application (web api) shuts down without giving any reason

Dot net core 2.2
When hosted in iis (in process/out of process), the application shuts down randomly.
I tried 'out of process' first. When application shuts down, the next http request doesn't restart the application. Restarting the website and recycling app pool manually doesn't bring up the application either.
Then I changed it to 'in-process'. Now if the application shuts down I can restart it by restarting the website or recycling the App pool within iis. However the application still shuts down randomly.
The only entry in the log in
Application started. Press Ctrl+C to shut down.
Application is shutting down...
The shutdown doesn't happen at any specific time but roughly in every 24 hours.
My main method:
public static void Main(string[] args)
{
BuildWebHost(args).Run();
}
    public static IWebHost BuildWebHost(string[] args) =>
        WebHost.CreateDefaultBuilder(args)
        .UseIISIntegration()
            .UseStartup<Startup>()
        .CaptureStartupErrors(true)
       .UseSetting("detailedErrors", "true")
            .Build();

Preload is enabled in iis. Startmode Always Running and Idle timeout is 0.
I added this line to web.config this morning.
applicationInitialization doAppInitAfterRestart="true"
After saving the web.config, the application stopped. I had to then run iisrest command this restart the application. But in dot net standard the application get restarted automatically if we modify web.config.
It doesn't look like there is any error in the code. It looks more like the application is not restarting if it stops for whatever reason.

Thanks for contacting us, @a6datta.
This can happen if your app doesn't get traffic for some period of time. There should be a setting on the AppPool for your app in IIS controlling the period of time after the last request to keep the app alive.

@Tratcher I said this, but just thought whether this is correct or not for ASP.NET Core apps, hosted under IIS. Do the same AppPool rules apply to these?

@mkArtakMSFT the idle timeout is set to zero in iis. Even if it wasn't and the site shuts down, shouldn't it start again whenever the next request comes in. That's iis default behaviour.

Yes, it should:

shouldn't it start again whenever the next request comes in

I'm out of ideas here. Not sure what can cause this behavior. Given the nature of this question (it's not really directly indicating an issue in ASP.NET Core) I would recommend trying to post a question in StackOverflow, as you may get a faster response there.

I think I figured out the issue. IIS doesn't not restart the site if it's in development mode. I changed the hosting environment in web.config. It seems to be working now.

Sounds like this can be closed then? @a6datta.

@mkArtakMSFT The issue is not resolved. The log doesn't show "Application is shutting down..." but it stops working. I manually restarted the website and application pool. It started working after that. I previously came across similar issues in dot net framework but in those cases I was able to see the the application pool stopped in iis management console. But now it just hangs and by looking at the console I cannot say if the pool has stopped or running. It could be an issue with the iis integration. Do you know the correct forum where I can raise this. I posted this question on stack overflow with no luck. We will have to make a decision today whether to go back to old web api as it has become very critical now.

@shirhatti can you please look into this?

My best guess from reading this issue is that your application "shutting down" randomly is actually your application crashing.
I would look to see if there are events in the EventLog when your application crashed/shut down.
If so, I highly recommend enabling Windows Error Reporting (WER) to capture a dump when your process crashes- https://docs.microsoft.com/en-us/aspnet/core/host-and-deploy/iis/troubleshoot?view=aspnetcore-2.2#create-a-dump
If analyzing the dumps shows a crash in ANCM/IIS Integration as opposed to your code, then please share the dump with me to further analyze what's going on.

@shirhatti I understand and I will do as you suggested. However rapid fail protection is disabled within iis. Shouldn't that mean, in case of crash, a new worker process will fire up on next request. No matter how good you code sometimes the application may crash. I guess that's the reason why iis starts a new worker process. Correct me if I am wrong please.

Shouldn't that mean, in case of crash, a new worker process will fire up on next request

Yes. A new worker should be started on next request. There isn't enough information here for me to ascertain why that isn't happening. Check Event Logs for more information
cc @jkotalik

I have the same issue. It seems everyone is so busy with .net 3-preview that they have no time to workaround this issue. My main problem is that users get logged out while working with app. And that is super annoying. Unfortunately I don't have access to Windows Error Reporting (WER) as I have this issue with a remote host while my app works fine on local host. Any new suggestions?
Similar issue someone else posted on Stackoverflow: https://stackoverflow.com/questions/55025485/net-core-kestrel-application-is-shutting-down-without-reason

My main problem is that users get logged out while working with app.

Even an app restart shouldn't cause logouts unless your DataProtection is misconfigured to only use in-memory keys. Do you get warning logs at app startup about your DataProtection?

My main problem is that users get logged out while working with app.

Even an app restart shouldn't cause logouts unless your DataProtection is misconfigured to only use in-memory keys. Do you get warning logs at app startup about your DataProtection?

Yes:
warn: Microsoft.AspNetCore.DataProtection.Repositories.EphemeralXmlRepository[50]       Using an in-memory repository. Keys will not be persisted to storage. warn: Microsoft.AspNetCore.DataProtection.KeyManagement.XmlKeyManager[59]       Neither user profile nor HKLM registry available. Using an ephemeral key repository. Protected data will be unavailable when application exits.

@mobinseven When hosting in IIS, make sure to set up the registry hive used by data protection as described here- https://docs.microsoft.com/en-us/aspnet/core/host-and-deploy/iis/?view=aspnetcore-2.2#data-protection
As for your original concern about about your app shutting down crashing, please file a new issue and provides us relevant Event Logs and possibly a process dump to diagnose the issue.

@shirhatti The application stopped twice today. However no dump was created. I am re checking if there is any issue on my side. It could also mean that the application did not crash as we do have a global exception handler to catch uncaught exceptions. I will get back with more information.

@shirhatti I have been monitoring the application for all these days. It was stopping randomly. However no dump was created. To understand if there was any issue on my side, I added a global exception handler to catch all uncaught exceptions. There were a few exceptions which were not handled in code. After I handled those in code, in last 2 weeks no more uncaught exceptions were recorded in log. However the application kept on stopping.
Finally I decided to run the application as widows service and since then the application hasn't stopped once.
So clearly the issue is in iis integration. Our servers are all windows based so I didn't get a chance try on any other web server.
We have decided no to do any more development in dot net core until this issue has been resolved.

In my case, lack of Garbage Collecting was making the problem. I traced the memory usage and it got increased by any page load. I just added this to my .csproj file:
 <PropertyGroup>     <ServerGarbageCollection>true</ServerGarbageCollection>     <ConcurrentGarbageCollection>false</ConcurrentGarbageCollection>   </PropertyGroup>
and the problem got solved.


[Enhancement]: Improve the country guesser game

Is your enhancement request related to a problem? Please describe.➕
No, It's an enhancement in the existing design of the game.
Describe the solution you'd like
I wish to improve the overall design of the game, add a new feature and improve user experience for the game.
Select program in which you are contributing
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @AnkitaBarbora !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign

Hey @AnkitaBarbora ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇