Fix broken authentication to FTL with flankAuth task

Describe the issue
Authentication to our cloud-based automation tests service Firebase Test Lab is broken. The flow of using ./gradlew flankAuth is not working now.
Can you reliably reproduce the issue?
Switch to the latest commit on main branch fill in necessary ZCASH_FIREBASE_TEST_LAB_PROJECT=android-firebase-b0cf1 gradle.properties value and run ./gradlew flankAuth.
Expected behavior
A caller is authenticated to FTL and can then do the local running of our tests on the FTL service, as described in https://github.com/zcash/secant-android-wallet/blob/main/docs/Setup.md#firebase-test-lab.
Actual behavior + errors
Actually the calling of authentication through the flank fails with:
> Task :app:flankAuth FAILED
Visit the following URL in your browser:
https://accounts.google.com/o/oauth2/auth?response_type=code&client_id=32555940559.apps.googleusercontent.com&redirect_uri=http://localhost:8085/oauth2callback&scope=https://www.googleapis.com/auth/cloud-platform&access_type=offline&approval_prompt=force&login_hint=flank&include_granted_scopes=true
java.net.BindException: Address already in use
        at java.base/sun.nio.ch.Net.bind0(Native Method)
        at java.base/sun.nio.ch.Net.bind(Net.java:555)
        at java.base/sun.nio.ch.ServerSocketChannelImpl.netBind(ServerSocketChannelImpl.java:337)
        at java.base/sun.nio.ch.ServerSocketChannelImpl.bind(ServerSocketChannelImpl.java:294)
        at io.netty.channel.socket.nio.NioServerSocketChannel.doBind(NioServerSocketChannel.java:134)
        at io.netty.channel.AbstractChannel$AbstractUnsafe.bind(AbstractChannel.java:562)
        at io.netty.channel.DefaultChannelPipeline$HeadContext.bind(DefaultChannelPipeline.java:1334)
        at io.netty.channel.AbstractChannelHandlerContext.invokeBind(AbstractChannelHandlerContext.java:506)
        at io.netty.channel.AbstractChannelHandlerContext.bind(AbstractChannelHandlerContext.java:491)
        at io.netty.channel.DefaultChannelPipeline.bind(DefaultChannelPipeline.java:973)
        at io.netty.channel.AbstractChannel.bind(AbstractChannel.java:260)
        at io.netty.bootstrap.AbstractBootstrap$2.run(AbstractBootstrap.java:356)
        at io.netty.util.concurrent.AbstractEventExecutor.safeExecute(AbstractEventExecutor.java:164)
        at io.netty.util.concurrent.SingleThreadEventExecutor.runAllTasks(SingleThreadEventExecutor.java:472)
        at io.netty.channel.nio.NioEventLoop.run(NioEventLoop.java:500)
        at io.netty.util.concurrent.SingleThreadEventExecutor$4.run(SingleThreadEventExecutor.java:989)
        at io.netty.util.internal.ThreadExecutorMap$2.run(ThreadExecutorMap.java:74)
        at io.ktor.server.netty.EventLoopGroupProxy$Companion.create$lambda-1$lambda-0(NettyApplicationEngine.kt:251)
        at io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30)
        at java.base/java.lang.Thread.run(Thread.java:833)

The link to the browser seems to be working, as well as the authentication through the browser itself, but the Gradle task is failing independently on the result from the browser.
It has been broken probably by changes in  the commit 94d259d5b0c0175c40dad8b89940bcb4b4e48691 [#190] Pull screenshots on CI from May 20, 2022.

Update: the flankAuth command seems to be working when run from AS Gradle tasks (still not working form cmd). But the following command runFlank fails for me with:
Flank encountered a 403 error when running on project android-sdk-14de8. Please verify this credential is authorized for the project and has the required permissions.
Consider authentication with a Service Account https://github.com/Flank/flank#authenticate-with-a-service-account
or with a Google account https://github.com/Flank/flank#authenticate-with-a-google-account

Caused by: com.google.api.client.googleapis.json.GoogleJsonResponseException: 403 Forbidden
POST https://toolresults.googleapis.com/toolresults/v1beta3/projects/android-sdk-14de8:initializeSettings
{
  "code": 403,
  "errors": [
    {
      "domain": "global",
      "message": "The caller does not have permission",
      "reason": "forbidden"
    }
  ],
  "message": "The caller does not have permission",
  "status": "PERMISSION_DENIED"
}


[rvdm] Failure to halt hart when debugging over JTAG

Description
Summary
Sometimes when trying to debug Tock, OpenOCD will fail to connect and report that it cannot halt the hart.
Here is the console log from OpenOCD with an error message (seems to match this issue):
[jw opentitan]$ ./bazelisk.sh run //third_party/openocd -- \
    -f "bazel-opentitan/$(./bazelisk.sh outquery //third_party/openocd:jtag_adapter_cfg)" \
    -c "adapter speed 500; transport select jtag; reset_config trst_only" \
    -f util/openocd/target/lowrisc-earlgrey.cfg
[...]
Open On-Chip Debugger 0.12.0
Licensed under GNU GPL v2
For bug reports, read
	http://openocd.org/doc/doxygen/bugs.html
trst_only separate trst_push_pull

Info : Hardware thread awareness created
force hard breakpoints
Info : Listening on port 6666 for tcl connections
Info : Listening on port 4444 for telnet connections
Info : clock speed 500 kHz
Info : JTAG tap: riscv.tap tap/device found: 0x10001cdf (mfg: 0x66f (lowRISC), part: 0x0001, ver: 0x1)
Info : datacount=2 progbufsize=8
Info : Examined RISC-V core; found 1 harts
Info :  hart 0: XLEN=32, misa=0x40901104
Info : starting gdb server for riscv.tap.0 on 3333
Info : Listening on port 3333 for gdb connections
Info : accepting 'gdb' connection on tcp/3333
Info : [riscv.tap.0] Found 4 triggers
Error: unable to halt hart 0
Error:   dmcontrol=0x80000001
Error:   dmstatus =0x00030c82
Info : dropped 'gdb' connection

Context
We can reliably connect while ibex is spinning in a wfi loop, but other times it will often fail. It's difficult to tell where the code is when it fails since we can't debug it.
We have also observed that setting a breakpoint and continuing to it (or using the next command in GDB) will sometimes take a very long time. Interrupting GDB while it's doing this triggers the "unable to halt hart 0" problem again, and then the debugger will not be able to connect again until reset.
Note that ibex is able to run through the code fine while breakpoints are not set. I'm using this OpenOCD config which forces hardware breakpoints.
Reproducing
I can reliably hit the problem when running the latest Tock on the CW310.
I will try stepping through until GDB hits a long delay to narrow down the problem. Maybe then we can have a small software reproduction for this, but I don't have one yet.

CC @a-will @andreaskurth @GregAC

@jwnrt can you check whether this problem still persists with the current setup? It would be great to root cause this and see whether it is a HW issue that we need to fix for PROD, or whether it is "just" a setup issue (similar to the reset sequencing problem we had experienced last year). Thanks!

With the usual setup, Tock is running with the watchdog timer enabled (and locked) with the "bite" at 1 second, isn't it? That's totally incompatible with debugging, since aon_timer doesn't stop counting when Ibex is halted.

Yes, I can reproduce with the Tock that's run with bazel test //sw/device/silicon_owner/tock/tests/basic:basic_test. If it's caused by Tock hitting the watchdog then I guess that explains it. If the watchdog can be turned off in Tock easily then it might be worth confirming that's the issue, but I would have to wait until next week to check that myself.
My reproduction steps are here if you want to try yourself:
sudo apt install openocd
./bazelisk.sh test //sw/device/silicon_owner/tock/tests/basic:basic_test
./bazelisk.sh run //sw/host/opentitantool -- --interface cw310 gpio write TAP_STRAP0 false && bazel run //sw/host/opentitantool -- --interface cw310 gpio write TAP_STRAP1 true
./bazelisk.sh run //third_party/openocd -- \
    -f /usr/share/openocd/scripts/interface/ftdi/olimex-arm-usb-tiny-h.cfg \
    -c "adapter speed 500; transport select jtag; reset_config trst_only" \
    -f util/openocd/target/lowrisc-earlgrey.cfg

Good point @a-will!
Confirming that'd be great @jwnrt.

When trying to patch the latest Tock, we found another problem that prevented the debugger connecting. @pamaury worked out that Tock was configuring the pinmux to connect the TAP straps as GPIO pins which breaks trying to use them in RMA mode where they're continuously sampled. We submitted this PR to fix that.
My attempts to disable the watchdog have so far not resolved this problem, but I'll keep trying.

https://github.com/tock/tock/pull/3809#issuecomment-1904152516
This suggestion resolved the Failed to halt hart problem for me: seems to be a problem with the ePMP configuration and not a bug with RVDM.


typos, inline outs, redundant usings

These "reformat the world" PRs are never great to review and cause conflicts with existing in flights PRs 87% of the time. That's not to say we won't take it but sometimes they do more harm than good. It's also pretty pointless to do this if there's no way to enforce this style in the future.

i dont think any of these are stylistic. it fixes spelling errors and makes some code more succinct. eg would you reject a PR that removes a few 100 redundant usings? that is not "style", it is "removing noise"?
perhaps the "inline outs" is stylistic. happy to revert those if u want

also i havnt actually applied any formatting tool on any of the files.

Agreed on the merge conflict issue. However, I checked with the other devs and we're all in between work items so this should be safe to merge at the moment.


Bug: LARAVEL_STATIC_TO_INJECTION Rule

This rule is converted in the wrong way that's laravel will not allow.
class SkipWithArguments
{
+    public function __construct(private \Illuminate\Routing\Redirector $redirector)
+    {
+    }
     public function run()
     {
-        return redirect('/destination')->back()->with('error', 'Incorrect credential.');
+        return $this->redirector->back('/destination')->back()->with('error', 'Incorrect credential.');
     }
}

As per the documentation of the laravel first parameter of the back method, it's an integer but it's passing a string.

@hirenkeraliya I guess the output should be like the following?
return $this->redirector->to('/destination')->back()->with('error', 'Incorrect credential.');

is changing to:
-new ArgumentFuncCallToMethodCall('redirect', 'Illuminate\Routing\Redirector', 'back'),
+new ArgumentFuncCallToMethodCall('redirect', 'Illuminate\Routing\Redirector', 'to'),

make it work? If yes, could you create PR for it? Thank you.


xtensa: cavs: support running test by west flash and twister with remote host

There are some changes in this PR:

Add a dedicate west runner "intel_adsp" for intel_adsp_cavs boards.
Add a client-server base firmware loader instead of original SSH-base one.
Add a support for running multiple intel_adsp_cavs boards at same time. This feature rely on the hardware map support for serial-pty. I has already split into another PR: https://github.com/zephyrproject-rtos/zephyr/pull/45197) and try to make this PR clean.

Now we can run the test on cavs boards by west flash command, ex:
west build -b intel_adsp_cavs25 samples/hello_world
west flash --remote-host [remote hostname or ip addr]

And if you want to specify the location of SOF tool instead of the default ones, such as rimage tool, config and the key,
run it by:
west flash --remote-host [remote hostname or ip addr]
                 --rimage-path [path to rimage tool] \
                 --config-path [path to .toml config file] \
                 --key [path to signing key]

#Run the twister:
twister -p intel_adsp_cavs18 --device-testing
--device-serial-pty="$ZEPHYR_BASE/soc/xtensa/intel_adsp/tools/cavstool_client.py,-s,192.168.x.x,-l" \
--west-flash="--remote-host=192.168.x.x, --run-twister"
-T samples/hello_world -vv"

Hi @mbolivar-nordic , thanks so much for your review and I am working on addressing you comment :)

I appreciate the review and I got lots of knowledge of west runner!  Hi, @mbolivar-nordic , @nashif , @MaureenHelm , @stephanosio, I have address all the comments. Could you please help with reviewing it again when you got a chance, thank you!

Dear all, could you please help to review this PR again when you got a chance, and the major changes compare to previous PR are:

Try to split the twister hardware map support with the west runner itself.
Remove the signing image process in cmake I made previously to keep it simple. (If you suppose we can have it, then I can add it back anytime)
Add a rst document and also update the docs of cavs15 and cavs25.

Thanks for your review and all the comments!

Excuse me @nashif , @MaureenHelm, @mbolivar-nordic  and all , could you please help to review it again when you got a chance, thank you!

Rebase and correct alphabetical order only, thank for your review again.

Overall looks good to me. I have a few nitpicks about the runner implementation but I think it's a big improvement and it would be better to have it merged, so +1 from me. I guess this must wait for v3.2 though.

No nitpick at all, @mbolivar-nordic , I appreciate you and other reviewers spending your time to review this and I got lots of benefit of understanding the runner, thank you so much! :)

@enjiamai @aborisovich @marc-hb This unfortunately breaks the SOF CI currently. We've been relying on the cavstool.py and this PR now removes the old tool and replaces it with a client-server model. I'll talk with SOF CI folks how we can cope with this.

Excuse me, @kv2019i , yes, it will change the way we run the cavstool, please let me know if I can do anything to mitigate it. or you need any message to set the new thing up. Thank you!

Hi @kv2019i , could you please provide me some information about how you use the original cavstool.py in your CI ? Maybe I could help to speed up the migration, thanks!

@enjiamai Our CI has used "cavstool.py -l" to gather logs and run the tool on the DUTs. We also have some deployment scripts that have assumed cavstool.py to exist in Zephyr tree, like in here https://github.com/thesofproject/sof/blob/main/scripts/xtensa-build-zephyr.py
I'll let our CI folks to comment in, whether we should move to the client-server model, or should we try to restore the "local cavstool.py" variant of the tool.

Thanks , @kv2019i , maybe restore the "local cavstool.py"  is a quick short-term solution. Beside, I am trying to submit a fix for it in https://github.com/zephyrproject-rtos/zephyr/pull/46308 , please kindly to review and comment it. if there is anything also need to be added. Sorry for the inconvenience.

In the future can someone please make sure at least one SOF person is copied on backward incompatible changes like this before they're merged?
I'm not aware of any other, open-source ADSP user besides SOF, is there?

could you please provide me some information about how you use the original cavstool.py in your CI ?

A bit too late now but you can very simply submit any Zephyr PR to SOF CI with a one-line SOF PR: https://github.com/thesofproject/sof/blob/ebc222dab5686/scripts/xtensa-build-zephyr.py#L304

Thanks Robin for drawing my attention to the MAINTAINERS.yml file: it seems to cover production code but not test code... typical? I will try to fix it.

A bit too late now but you can very simply submit any Zephyr PR to SOF CI with a ~one~ two-lines SOF PR

Real-world example where the fix for this is getting tested in SOF CI:

https://github.com/thesofproject/sof/pull/5895

Anyone ever working on changes affecting SOF please have a look. Don't postpone, click now; it's literally 2 lines long and lets anyone run any unmerged Zephyr PR on SOF hardware. If you look at these 2 lines you will remember them, if not then you won't.

I think that means when there are changes related to SOF/CAVS, then we can get the SOF CI test result in advance. right?

Not automatically but manually yes, that's the whole point of this test "feature"/hack. Please use it!
Automatically would be great but there are a number of challenges, for a start SOF CI does not have the bandwidth to test every single Zephyr PR and that would not make sense anyway. Other challenges:

https://github.com/thesofproject/sof/issues/3517
https://github.com/zephyrproject-rtos/zephyr/discussions/34713

I think it's quite nice already if we can manually submit then test it like this! will do it next time. :)

Thanks Robin for drawing my attention to the MAINTAINERS.yml file: it seems to cover production code but not test code...

I was wrong, the MAINTAINERS.yml file should have added SOF people, see below.
@nashif, @cfriedt  there seems to some get_maintainer.py-related glitch somewhere. Where is the get_maintainer.pyinvocation that assigns reviewers and corresponding log?
./scripts/get_maintainer.py commits e4c171ed2255~5..e4c171ed2255
Intel Platforms (Xtensa)
	status: maintained
	maintainers: nashif
	collaborators: andyross, dcpleung, lyakh, lgirdwood, mengxianglinx, marc-hb, kv2019i, MaureenHelm
	inform:
	labels: platform: Intel CAVS
	description:

Twister
	status: maintained
	maintainers: nashif
	collaborators: chen-png, galak, PerMac, enjiamai, hakehuang, gopiotr
	inform:
	labels: area: Twister
	description:

West
	status: maintained
	maintainers: mbolivar-nordic
	collaborators: carlescufi, swinslow
	inform:
	labels: area: West
	description:


Match theme to specs

This is the image provided to use by @1Umair7:

We want to update the style to match as close as possible

We were able to find the actual source of this - thanks the @AftabSaraz.
https://ugobyte.wixsite.com/home/ui - this is the person who designed it and we also have a simulation accessible at https://biomeddeviceslms.com/tv-100-simulation/ (username and password are both biomedtv100).
We can play around with it and I feel most interactions will be quite similar to that interface so we can use it as a base.

This issue should be closed too, they settled for a different theme.

Yeah, that makes sense. I'll create an issue regarding the new task of consolidating themes.

Closing as we moved to use a separate theme.


build env error

Thanks for trying. I have updated the yaml file. Would you please pull and try again?

The previous error was resolved successfully, but there are still new error here.


Add homebrewed rcVector class and tests, and use it to implement rcIntArray.

This implements most of the std::vector interface, with some minor elisions of
infrequently used methods such as "insert", and some STL bookkeeping.
Performance is favorable compared to std::vector, and, in some cases, somewhat
better than naive code to manually manage a malloc'd dynamic array.
BM_FlatArray_Push:                     100 iterations in   72889721 nanos:  728897.21 nanos/it
BM_FlatArray_Fill:                     100 iterations in    2700746 nanos:   27007.46 nanos/it
BM_FlatArray_Memset:                   100 iterations in    2687810 nanos:   26878.10 nanos/it
BM_rcVector_Push:                      100 iterations in   51114316 nanos:  511143.16 nanos/it
BM_rcVector_PushPreallocated:          100 iterations in    9532106 nanos:   95321.06 nanos/it
BM_rcVector_Assign:                    100 iterations in    2371534 nanos:   23715.34 nanos/it
BM_rcVector_AssignIndices:             100 iterations in    2573052 nanos:   25730.52 nanos/it
BM_rcVector_Resize:                    100 iterations in    2455761 nanos:   24557.61 nanos/it
BM_stdvector_Push:                     100 iterations in   71739167 nanos:  717391.67 nanos/it
BM_stdvector_PushPreallocated:         100 iterations in   15723737 nanos:  157237.37 nanos/it
BM_stdvector_Assign:                   100 iterations in    2564985 nanos:   25649.85 nanos/it
BM_stdvector_AssignIndices:            100 iterations in    5174923 nanos:   51749.23 nanos/it
BM_stdvector_Resize:                   100 iterations in    2520054 nanos:   25200.54 nanos/it

I think it would be helpful to have a style guide somewhere in the documentation. C++ has had destructors for 39 years (since before it was called "C++"); I don't think eliding support for them clearly follows from avoiding "modern" C++.
FWIW:
* I didn't bother with support for moves since the test suite runs on VC2010, which does not support them anyway.
* The bulk of the complexity comes from supporting v.push_back(v[0]), while maintaining good performance, and the distinction between resize() and reserve(). IMO, both are worth it.
* The support for destruction is only a few lines (destroy_range and call sites).
* Likewise, supporting copying correctly adds a couple of lines at best.
* Even if no checked-in code uses rcVector, sprinkling printfs and asserts in constructors and destructors is valuable for debugging.
IMO, imitating std::vector makes sense if only because that it is the interface most likely to be familiar to a potential contributor, and decreases friction for finding bugs and sending patches.
Getting into some specifics about the interface:
* I think reserve is critical to clean, fast code.
* The least useful methods here (given that Recast doesn't support C++11) are operator=, at, begin/end, front, and assign.
* I think SmallVector is very useful, but there are much bigger gains to be made with algorithmic changes than with small-vector optimization.
If you'd like me to include a few changes to calling code (a la #316), I'd be happy to do that.

I took a closer look and you are right. My main concern was adding a lot of code that we won't use, but I see now after you mention it that most of the complexity is from making it robust, and that is probably fine. The alternative would be some asserts, but I think it is fine as it is.
I think we should just get rid of at. I also think we should delete operator=; if we need to copy a vector, it should be more explicit.
I think leaving begin/end is fine, as (some parts of) C++11 should eventually be allowed. I don't think people would have objections to this.
I will leave some other comments too in a review.

If you'd like me to include a few changes to calling code (a la #316), I'd be happy to do that.

It would be nice, but it can wait to future PRs. It was mainly if we wanted to develop it for our exact use cases more incrementally.


Error with minify CSS

Hello i have a problem, I installed the plugin, and it seems it works with JS, and me generate the files in cache.
But if I try to create a CSS file gives me this error:
Fatal error: Cannot redeclare class JSMin_UnterminatedStringException in C:\xampp\htdocs\beta\include\librerie\Smarty\plugins\minify\JSmin.php on line 435
This is my code:
`{combine input=array('/beta/_inc/public/plugins/search.js','/beta/_inc/public/plugins/pagination.js','/beta/_inc/public/plugins/load/jquery_load.js') output='/beta/cache/big.js' age='30' debug=false}
{combine input=array('/beta/_inc/public/plugins/owl-carousel/owl.carousel.css','/beta/_inc/public/plugins/font-awesome/css/font-awesome.min.css') output='/beta/cache/big.css' age='30' debug=false}`

Hi, @Ande1232!
Your error occurs because you have already found in the code JSMin_UnterminatedStringException class.
The easiest way to deal with this mistake is to rename JSMin_UnterminatedStringException class in C:\xampp\htdocs\beta\include\librerie\Smarty\plugins\minify\JSmin.php file
Lines 161, 203, 435
Calling for example, you can JSMin_UnterminatedString2Exception

Hi @dead23angel,
I replaced everything as you suggested, unfortunately continues to give me error. The error I've only if I also want to compress css, if I remove the string for the CSS, It created the minify JS files without error.
Fatal error: Cannot redeclare class JSMin_UnterminatedString2Exception in C:\xampp\htdocs\beta\include\librerie\Smarty\plugins\minify\JSmin.php on line 435

Hi @Ande1232!
Tell me which version you are using Smatry, so I tried to reproduce the bug

@Ande1232 I made changes to avoid similar mistakes. Try to update the plugin.

@dead23angel It's working fine now thanks.

Hello.
{combine input=array('/design/default/css/bootstrap.css','/design/default/css/tip.css') output='/cache/main.css' age='30' debug=false}
When using style compression, the site stops working. A file is created in the cache folder, but with 0 weight.


[POAE7-2773] Port ConstantQueryTest to nextgen

What changes were proposed in this pull request?
Port ConstantQueryTest to nextgen and bug fix
Why are the changes needed?
Port ConstantQueryTest to nextgen
Does this PR introduce any user-facing change?
No
How was this patch tested?
UT
Which label does this PR belong to?

@jikunshang @kd97 please help approve again. Thanks.


Melo TTS pronounces "US" as "ʌs", not "ˌjuː ˈes".

Here is the sample sentence: "US paused shipment of bombs to Israel".

Try: U-S paused shipment of bombs to Israel

Try: U-S paused shipment of bombs to Israel

Yeah, it supposes to work by adding "-" between U and S.
But it can't fix the problem if the input text comes from  other sources.

I am interested in this solution for clearly spelling out names using melo(TTS) . Specifically, I have a name, "Tomas," which I need to articulate slowly so that the recipient can easily understand it. While I've tried parsing the name as "T-o-m-a-s" or adding commas between the letters, the TTS engine still pronounces it too quickly. Instead of adjusting the playback speed, I am seeking a method to ensure the name "Tomas" is spelled out slowly and clearly.


Updated RestfulService to include PATCH

RestfulService now allows PATCH and also removed CURLOPT_POST as it seems redundant if implying POST via the $method

Thanks @daOutlawz. As this is a new feature, it will need to go against the 3 branch instead of 3.1.
Is CURLOPT_POST definitely redundant? What happens if you’re sending a POST request, but $data is null? I don’t see any harm in explicitly setting it like we currently are

@kinglozzer, thanks for that.
Sending null data without explicitly setting CURLOPT_POST to true shouldn't be an issue.  I actually thought there would be more issues setting that explicitly with a PATCH as CURLOPT_POST assumes your Content-Type is application/x-www-form-urlencoded.

@kinglozzer - is there any movement on this?  Or did you imply that you wanted it changed to re-include CURLOPT_POST

Hi,
If I do this against 3.0 will it be merged into 3.1 also?  As we need support for 3.1 at this stage.

@robpetreski new features can only go in to the 3 or master branch sorry. Good news is it shouldn't be hard to upgrade a site from 3.1 to 3.2 or 3.3 when it comes out... there shouldn't be a compelling reason to leave a site on 3.1 now that 3.2 is out.

If you want to use a custom implementation in 3.1 you'll need to use injector to set your custom class.
E.g.
Injector:
  RestfulService: CustomRestfulService

And implement your overridden method in your class. :)
Caveat: I'm not 100% sure that it will let you override it in this way.
In any case, this can't be merged into 3.1 so I'm closing this PR for now. If you'd like to merge this into the 3 branch, I suggest you create it soon because we're soon to release 3.3.0 beta, after which new features won't be available until 3.4.


Flyway MSSQL connection problem

Which version and edition of Flyway are you using?
5.2.1
If this is not the latest version, can you reproduce the issue with the latest one as well?
(Many bugs are fixed in newer releases and upgrading will often resolve the issue)
I want to use firefly with MSSQL 2017 edition (SQLEXPRESS)
I set up conf file like:
flyway.url= jdbc:sqlserver:////SQLEXPRESS:1433;databaseName=TestDb
flyway.user=admin
flyway.password=admin
All other parameters are not touched.
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin)
Command-line
Which database are you using (type & version)?
SQL Expression Server MSSQL 2017
Which operating system are you using?
Windows 10
What did you do?
(Please include the content causing the issue, any relevant configuration settings, the SQL statement that failed (if relevant) and the command you ran.)
flyway.url= jdbc:sqlserver:////SQLEXPRESS:1433;databaseName=TestDb
flyway.user=admin
flyway.password=admin
All other parameters are not touched.
Than when I run comman flyway info, and it prompts me for password. I insert password admin&admin
and then I get the error:
WARNING: Discarding INCOMPLETE dataSource configuration! flyway.url must be set.
ERROR: Unable to connect to the database. Configure the url, user and password!
What did you expect to see?
Successfully connected to SQL...
What did you see instead?
WARNING: Discarding INCOMPLETE dataSource configuration! flyway.url must be set.
ERROR: Unable to connect to the database. Configure the url, user and password!

Where did those 4 slashes come from? Or better said, what made you think 4 were needed? (serious question, trying to understand how you arrived at this)
Replace with 2 slashes and things should work as expected. See https://flywaydb.org/documentation/database/sqlserver#driver

Any news?

Closing due to lack of response.


Add ls command

$ ./js-performance list
  profileA: Do not try this at home
  profileB: This is a super cool thing
...

PR open https://github.com/haensl/js-performance/pull/44

Done in v1.7.0


missing implementations on SpatialMaxPooling_updateGradInput

I'm porting some cunn code over to clnn and stumbled over the following error:
/Users/brunoro/dev/torch/install/bin/luajit: ...dev/torch/install/share/lua/5.1/nn/SpatialMaxPooling.lua:41: Not implemented at /Users/brunoro/dev/clnn/SpatialMaxPooling.cpp:166
stack traceback:
	[C]: in function 'SpatialMaxPooling_updateGradInput'
	...dev/torch/install/share/lua/5.1/nn/SpatialMaxPooling.lua:41: in function 'updateGradInput'
	...rs/brunoro/dev/torch/install/share/lua/5.1/nn/Module.lua:30: in function 'backward'
	...runoro/dev/torch/install/share/lua/5.1/nn/Sequential.lua:84: in function 'backward'
	neural_style.lua:259: in function 'opfunc'
	.../brunoro/dev/torch/install/share/lua/5.1/optim/lbfgs.lua:66: in function 'lbfgs'
	neural_style.lua:278: in function 'main'
	neural_style.lua:439: in main chunk
	[C]: in function 'dofile'
	.../dev/torch/install/lib/luarocks/rocks/trepl/scm-1/bin/th:131: in main chunk
	[C]: at 0x010a4d1190

Browsing through SpatialMaxPooling.cpp I found out that there's actually some commented code just above the line that throws that exception. Is there any plans on implementing those cases?
Also, some CNN noob questions: what those cases stand for? I'd be happy to implement those if anyone points out some reference to what actually this method is doing.

I'm porting some cunn code over to clnn

Awesome!

Is there any plans on implementing those cases?

I sort of implement stuff as and when it becomes necessary.  If you have a moment to implement one of the commented out cases that would be great! :-)

Also, some CNN noob questions: what those cases stand for?

In max-pooling, we take a square of pooling width (kW) by pooling height (kH) pixels, and find the maximum value in that grid.  The output from that grid is this max value.  Then, we take the next square of pixels, and do the same thing.  Useful web pages: http://ufldl.stanford.edu/wiki/index.php/Pooling and http://andrew.gibiansky.com/blog/machine-learning/convolutional-neural-networks/
When we back-propagate, we take the gradient from the gradOutput pixel, and send it back to the one that produced the max value on the way forwards.  So we have to store which pixel that was, when we do the forwards propagation.
The pooling areas can be exactly contiguous, which is the easy situation.  In this case each gradOutput value maps to exactly one gradInput pixel.  Well, that's always true.  But the point is, each gradInput pixel will take input from exactly 0 or 1 gradOutput pixels.  But rather than making the pooling areas contiguous, we can make them overlap.  And then some gradInput pixels need to be updated from the sum of several gradOutput pixels.  This is a bit tricky to do, if done from parallel threads, so we consider this case separately.  The stride, dW and dH decides the horizontal and vertical distance between each pooling area (kW by kH).  When dW == kW and dH == kH, this is the contiguous case.  When dW is less than kW, or dH is less than kH, then we have overlapping pools.
Actually, you might not need to handle this, because:
if (input->nDimension == 3) {

Spatial max pooling is over a 2d image, so that is 2 dimensions, W ('width') and H (height).  But typically, there will be more than one image plane for each incoming example, so we have nInputPlane image planes per example, three dimensions.  Finally, we can provide a mini-batch of multiple examples, so that would be 4 dimensions.
Currently, implementation is done for mini-batches, but not for single non-batched examples.

if you can shoe-horn your current code into mini-batches, eg by reshapeing to have an extra dimension of length 1, then you dont need to change the SpatialMaxPooling code at all
alternatively, perhaps we can simply modify the clnn_SpatialMaxPooling_updateGradInput method to reshape the incoming tensor to have an extra dimension?  In which case, we can reuse the rest of the updateGradInput code as-is

As far as this bit:
} else if((kW>>1) <= dW && (kH>>1) <= dH) {
To handle overlapping pools, I noticed that mostly we only have 3 x 3 pools, with 2 x 2 stride.  So, as a first simplification, I only consider this case.  Actually, let's consider an even simpler case for now, which is an input image with one single row, with 1 x 3 pools, and 1 x 2 stride.  In this case, the pools overlap horizontally:
|pool 1|
     |pool 2|
          |pool 3|
              ... etc

But no more than 2 pools overlap for any input/gradInput pixel.  So, we can do the pooling in two batches.  In the first batch, we do the odd pools:
|pool 1|
          |pool 3|

These dont overlap :-)
Then we do the even pools:
     |pool 2|
               |pool 4|

... dont overlap either.  And we add their results to the gradInput results from the first batch of pools.
In 2 dimensions, we'll need 4 such batches, like we will have these pools first:
X . X .
X . X . X .
 X . X .  X . X .

X . X . X .

X . X . X .

 X . X .  X . X . X

So, 4 times.
This will actually generalize to any case where the pooling size is no smaller than half the stride, or something like this, hence that weird looking if condition earlier.
It would be easy enough to handle the fully general case though.  eg, if stride is 1, and pooling size is 3, then we'd simply need to do the backpropagation 9 times, adding the results to the output of the earlier updateGradInput batches.

@hughperkins nn and cunn SpatialMaxPooling (and soon SpatialAveragePooling) were updated to support arbitrary padding pad_w and pad_h. Also, the CUDA kernels were changed (borrowed from Caffe), and now there's no more need for atomic operations on the backward case. I have absolutely no knowledge in opencl, but I think that adapting those kernels would be easier to generalise SpatialMaxPooling to all the cases.

@fmassa Ah, good info. Thanks!

Hi Gustavo,
Several people in the neural-style project encountered the same issue, so I've copy/pasted an implementation for contiguous non-batched pools, and if you update your clnn, luarocks install clnn, it might do what you need.  If you're using non-contiguous pools, you'll need to either do some extra copy/pasting (or even, ideally, some factorization :-) ), or else follow Francisco's heads-up that the cunn implementation might be worth re-porting over now.  I reckon the path of least resistance for now would be to just add an extra copy-paste block :-)  Or perhaps factorize a bit.
In terms of testing clnn, I basically run the following tests currently, on a single OS (ubuntu 14.04), and a single GPU (NVIDIA 940M).  This is quite quick to do, so please feel free to change things in whatever way you think is beautiful :-)  Ideally not diverging too much from cunn, but I've changed tons of stuff, so fairly flexible.  The tests I do are:
th -l clnn -e 'clnn.test()'
-- all tests should pass
git clone
cd ~/git/char-rnn
th -opencl 1 train.lua
-- training loss should decrease from 3-4 ish to about 2-3 ish, shouldnt become NaN, shouldnt crash.  About 5 iterations are sufficient

Hi @hughperkins, thanks for the detailed explanation. Cool, one of the pieces of code I was trying to get running with clnn was actually neural-style, so thanks for pointing out the issue on their repo.
I'll try this week to get it running with some copy/pasting (and or factorization). Otherwise, re-porting the cunn implementation might be a nice way to brush up my openCL :D

Cool :-)  By the way, if you report cunn code, you might want to try the following:
git clone git@github.com:torch/cunn.git
git clone git@github.com:hughperkins/clnn.git
cd clnn
python util/port.py
meld port . &

This will show you the diff between automatically ported cunn files (in port directory), and the current clnn files (in . directory).  It's far from perfect, but it can provide useful first draft to work from.

Hi Gustavo, please note that cunn spatialmaxpooling has been ported across now, since :ceil() needed by neural-styling.  You can luarocks install clnn to pull down latest version.

Oh, sweet. I was in the middle of the process porting the kernel from cunn, so I guess I don't need to finish that.

Ok.  I can close this issue, right?

Yep, thanks a lot!


[incubator-kie-issues-1468] Starting a project in development mode in Windows will fail if it has nested resources

Many thanks for submitting your Pull Request :heart:!

Closes/Fixes/Resolves https://github.com/apache/incubator-kie-issues/issues/1468

Description:
In a Windows machine, a nested resource will have a Windows path with backslash (\). The getResourceAsStream only supports Posix paths (/). This PR adds a toPosixPath method which transform Windows paths to Posix paths, and it keeps Posix paths unchanged.
Manually tested in Windows, WSL and macOS.

Please make sure that your PR meets the following requirements:

[ ] You have read the contributors guide
[ ] Your code is properly formatted according to this configuration
[ ] Pull Request title is properly formatted: Issue-XYZ Subject
[ ] Pull Request title contains the target branch if not targeting main: [0.9.x] Issue-XYZ Subject
[ ] Pull Request contains link to the JIRA issue
[ ] Pull Request contains link to any dependent or related Pull Request
[ ] Pull Request contains description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket

How to replicate CI configuration locally?

Build Chain tool does "simple" maven build(s), the builds are just Maven commands, but because the repositories relates and depends on each other and any change in API or class method could affect several of those repositories there is a need to use build-chain tool to handle cross repository builds and be sure that we always use latest version of the code for each repository.
build-chain tool is a build tool which can be used on command line locally or in Github Actions workflow(s), in case you need to change multiple repositories and send multiple dependent pull requests related with a change you can easily reproduce the same build by executing it on Github hosted environment or locally in your development environment. See local execution details to get more information about it.

@gitgabrio Yeah, but in another area I guess. And most likely this will be no the last case :V

guys give me a min to take a look before merging!

Just ping me when ready and I can merge @pefernan.

@baldimir +1


packaging of project

Hey, since for example many teams within organizations does not have access to public github, will u package this repo in order to initialize project structure and everything

Hi @goldstein0101 this project can now be initialized with the Databricks CLI, so this should get around the public Github aspect!


Unify tab bar props

Standardize tab bar props for various tab bar button types (simple,
avatar, icon)
Use one set of props for mobile and desktop
Ensure that all tab bar use cases are providing properly typed props

:mag_right: The commits 7e6bebf43b64...5bd97ad1513d introduced some visual changes:

Changed: Username_Email_Login-Name (before) (after)

Changed: Invite_Code-Error (before) (after)

@keybase/react-hackers

Bump @keybase/react-hackers

I just added compliance with the new trailing-comma style

👍 after tweaking the typedefs

[ ] Fix typedefs to use new $Exact type
[ ] Use IconType instead of IconProps

:mag_right: The commits 18a26e794259...aaa164bcb5c2 introduced some visual changes:

Changed: Menubar-Public_Truncated (before) (after)

Changed: Menubar-Public_Normal (before) (after)

Changed: Menubar-Private_Truncated (before) (after)

Changed: Menubar-Private_Normal (before) (after)

Changed: Menubar-Private_Menu_Showing (before) (after)

Changed: Folders_TLF-Normal_Public (before) (after)

Changed: Folders_TLF-Normal_Private (before) (after)


Syntax error when after correct by Performance/Detect

Before correct.
[1, 2, 3].find_all { |i| i == 3 }.first(1)

After correct.
[1, 2, 3].find { |i| i == 3 }(1)

But, It's syntax error.

Just pushed a fix to my open PR.


docs: update readme with docker-compose info

Signed-off-by: Igogrek<EMAIL_ADDRESS>Some basic information on how to run tests and docker-compose for this repo

I see that CLA should be signed for this and there's a list of people who already did https://github.com/trustbloc/clabot-config but I could not find any place to sign it. Is it possible?

@igogrek Thanks very much for your contribution. We are working to resolve this issue.

@Igogrek Sorry for the delay. We have updated the contribution guidelines with CLA instructions.

@cla-bot check again please?

@cla-bot check again please?

@Igogrek you are all set!


how to use internet gateway (e.g. nowsms)?

Seems like nowsms does NOT support sending messages in PDU format. You would have to find gateway that has API that accepts PDU formatted messages.
http://support.nowsms.com/discus/messages/1/19757.html

yes, but they have a modem app that generated the pdu and uses htc sendRawPdu api, and sends this pdu via the htc device modem, see http://www.nowsms.com/send-wap-push-and-binary-sms-with-android-phone-as-modem

I never used this new feature of SMSnow, but as long as they support PDU - you are good to go with SharpSMS


Add Support for 1.20/1.20.1

Yo man, here, have some code to allow 1.20 Fabric Servers to use your mod. I know the Merge says 1.19, but there is no 1.20 Branch that I can commit to, lmk if I should make another pull Request or something, have fun!

Thank you! i will make a new branch and switch it to that

You're welcome!

it took me a bit to test sorry about that. i'll merge this tomorrow.
btw do you mind if i add you to the authors instead of your description changes?

I don't mind you changing it, don't worry! Feel free to modify it at your will.


Feature/fix input

What's new?

Refactor input to input manager on scripts
Refactor controller handling
Fix critical animation bugs

And with this we need the <= or not?

@yalania We dont need it, changing the +1 in the AnimationController solved the issue


Added some ports

This PR adds the following ports

gmp
mpfr
mpc

And fixes the following port

gcc

Thank you for the PR!


allow configuration of MacVim's tab width

I have a tendency to keep lots of page tabs open and after upgrading to MacVim 8.0 I noticed after I got to about 7-8 tabs open, the "add tab" button would change to a double more arrow and all new tabs would be placed in there, this means I can't easily jump to tabs using Ngt because I don't know the tab number of the off screen tabs, or what's open.
Digging into the documentation I saw that I could set MMTabMinWidth which seemed exactly like what I wanted, but setting it had no effect, so I dug into the source code and it looks like a change made in MMVimView (specifically, this commit) might be the culprit:
#if MAC_OS_X_VERSION_MIN_REQUIRED >= MAC_OS_X_VERSION_10_10
    CGFloat screenWidth = [[NSScreen mainScreen] frame].size.width;
    [tabBarControl setStyleNamed:@"Yosemite"];
    [tabBarControl setCellMinWidth:120];
    [tabBarControl setCellMaxWidth:screenWidth];
    [tabBarControl setCellOptimumWidth:screenWidth];
#else
    [tabBarControl setCellMinWidth:[ud integerForKey:MMTabMinWidthKey]];
    [tabBarControl setCellMaxWidth:[ud integerForKey:MMTabMaxWidthKey]];
    [tabBarControl setCellOptimumWidth:
                                     [ud integerForKey:MMTabOptimumWidthKey]];
#endif

If you are on a newer version of the OS, the minimum tab width looks to be pegged at 120 instead of being set to MMTabMinWidth, is there a reason for this?
I would much rather have control over this value or have it set back to the previous default of 64.
Thanks :)

Could you test #357?

👍  Awesome! Seems to be working as expected.
Thank you :)


Problem in the profiles tab

When you go to the profiles tab, the profile itself looks like in the picture.

I think this is a duplicated issue: https://github.com/DiscordStyles/Fluent/issues/91


api how to use?? help!!! 😫

Can I use https://creepjs-api.web.app/fp api in my development environment other than https://abrahamjuliot.github.io/creepjs/ url?

Negative, it's locked down. However, you can swap out the URLs and send requests to your server environment.

Could you please explain in more detail?

Can you elaborate on the process of creating an FP ID?
I want to make a fingerprint without looking at the useragent. While specifying which device

Sure, Fingerprint ID is a hash of browser, engine, platform, and hardware features reported by the browser. Fingerprints can be generated in a raw form or hash identifier, or both.
Raw values are great for server-side analysis. Hash identifiers are useful for snapshot history and grouping patterns.
There are many techniques, but I recommend starting out simple and fast. Give this lite fingerprint script a try. It runs in under 5ms.
function getHash(x: unknown) {
    const json = JSON.stringify(x)
	const hash = json
        .split('')
        .reduce((hash, _, i) => Math.imul(31, hash) + json.charCodeAt(i) | 0, 0x811c9dc5)
	return ('0000000' + (hash >>> 0).toString(16)).substr(-8)
}

function getLiteFingerprint() {
    const { width, height, availWidth, availHeight } = screen // OS/Engine Screen
    const {
        deviceMemory, // Memory
        hardwareConcurrency, // Cores
        maxTouchPoints, // Touch
        platform, // OS
        language, // Lang Preference
        languages, // Lang Most Preferred
    } = navigator
    const { timeZone, locale } = Intl.DateTimeFormat().resolvedOptions() // OS/Engine Timezone and Language
    const browserAndClientFeatures = Object.getOwnPropertyNames(window); // Browser Release and Client Features

    return [
        width,
        height,
        availWidth,
        availHeight,
        devicePixelRatio,
        deviceMemory,
        hardwareConcurrency,
        maxTouchPoints,
        platform,
        matchMedia('(prefers-color-scheme:light)').matches, // Color Preference
        matchMedia('(prefers-reduced-motion:reduce)').matches, // Animation Preference
        timeZone,
        locale,
        language,
        languages,
        browserAndClientFeatures,
    ]
}
const start = performance.now()
const result = getLiteFingerprint()
console.clear()
console.log(`complete: ${(performance.now() - start).toFixed(2)}ms`)
console.log(`fingerprint: ${getHash(result)}`)

Thank you  😁 very much for your detailed answer.
However, it is not the result I want.
Every time I run the code you have guided me to
Strings are changing.
ex)
"fingerprint: afe25423"
"fingerprint: 6b722854"
The conditions I need to meet are as follows.

The only key that can be specified by any browser

Do not allow duplication

=============
So I created sophisticated creepjs with typescript.
After building with yarn, converting to javascript, removing useragent and api, and checking in the local environment. I am having difficulty removing useragent items during this process.
We're analyzing well-written code after the build.

Strings are changing.

Try commenting out the browserAndClientFeatures value or hash it separately. If you have extensions installed on your browser, such as DuckDuckGo, some store random properties on the window object.

difficulty removing useragent

User agent items are mostly in worker and navigator files.

Thank you so much for your help. I will proceed as you said on March 30th. Thank you!! I will continue to analyze the code :)

Closed issue as it didn't solve the problem.
What I was trying to do was a way to specify using fingerprints on mobile devices even if the browser is different.
Fingerprints seem to have limitations.
Thanks for your help. 😋

For Chrome, you can get the number of bytes allocated to the site for storage. That's over 128 Billion possible fingerprints on a 120 GB device. Not many limitations there.

Chrome on Android provides a significant amount of entropy. You can get the exact device model and network condition (Wi-Fi, Ethernet, etc.). Moreover, on mobile, the device pixel ratio and screen resolution combined can hint to the device model for all browsers.

The other factor to consider is the context fingerprinting occurs. A fingerprint on site A can be more unique to a user than it is on site B.

me and my friend
Assuming that the device model is the same and all settings are the same
Is it possible to distinguish me from my friends using fingerprints?
ex)
me               friend
phonemodel              phone1        phone1
fingerfirnt(chrome)         aaaa   bbbb
fingerfrint(edge)             aaaa   bbbb
fingerfrint(safari)            aaaa   bbbb
When I analyzed the creep.js source, it seems impossible..

If you purchase 2 matching devices, use the same SD card, same time zone/language, same OS preferences, same browser and release, same settings/permission, same network, and don't modify storage, the fingerprints can match.
CreepJS does not showcase or even attempt more advanced techniques. There is also code that is used for analysis and does not get factored into the fingerprint, and there are also behavioral patterns not implemented here. Combine that with site-specific events, and it becomes more possible to distinguish matching browsers.

thanks for the answer 😊


Theme Switch Improvement

improve code of category page - Ensure that the dark/light mode persists after page reloads by using localStorage or cookies in JavaScript.

As I see the dark, light theme toggle is still not ready, can I take on this task? @ANSHIKA-26

Hey @ANSHIKA-26, Can i Also work on this issue as it is more aligned with my skills

@ananyag309 are u still working on this ?

@ANSHIKA-26 will be done by today


OSASINFRA-3538: openstack: small fixes for CLI

What this PR does / why we need it:
Small fixes for OpenStack CLI.

/cc MaysaMacedo bryan-cox

/approve

/test e2e-azure

/test e2e-azure

the azure CI seems unstable but this isn't related to that PR. This can be safely merged I think.

/lgtm


alpacafarm reward-model-human as gold reward

Hello, I have found your work and code exetremely helpful! Thank you for your code and work.
However, during my usage, I found several point to be confusing and not sure if I am doing right, so I hope you can generously help me.
In particular, I am not sure about some point regarding using Alpacafarm/reward-model-human as gold reward model, as in the paper:

After ppo training, when using Alpacafarm/reward-model-human to assign the gold reward for generated responses, I found i have to set is_alpaca_rm: true instead of false which is originially set in configs/config_rl.json.
In that case, when using alpaca rms, I believe there's a typo in function _parse_entry of src/data_utils/rm_dataset_formatter.py. When using this function, the prompt would only contain the first line "Below is an instruction that describes a task, paired with an ", instead of the whole prompt used in alpaca farm. I fixed it by adding a \ at the end of each line, but I amnot sure if it is the right way.

It is deeply appreciated if you can help me with my problems. Thank you!

@georgao35 did you ever get clarity on these two questions?

Also make sure to add the f for string fromatting. For me it looks like the following, but it would be great to hear from Thomas
def _parse_entry(entry: dict, output_alpaca: bool = False):
    instruction = entry["instruction"]
    input = entry.get("input", "")
    answers = entry["answers"]
    if output_alpaca:
        input_ = f"### Input:\n{input}\n\n" if len(input) > 0 else ""
        start_prompt = "Below is an instruction that describes a task, paired with an " \
        "input that provides further context. Write a response that appropriately " \
        f"completes the request.\n\n### Instruction:\n{instruction}\n\n{input_}### " \
        "Response:\n"
        end_token = ""
    else:
        input_ = f"\n{input}" if len(input) > 0 else ""
        start_prompt = f"<|prompter|>{instruction}{input_}<|endoftext|><|assistant|>"
        end_token = "<|endoftext|>"
    return start_prompt, answers, end_token

@georgao35 did you ever get clarity on these two questions?

not yet

Apologies, I don't monitor this that often, will try to monitor more frequently moving forward. I was doing a lot of refactoring towards the end so there seem to be a couple typos, you are correct.

This seems correct and I should change it in the config. When using the alpaca_farm reward model you will want to set that flag to true.
You want this all to print out as a single string, so you will want to either add the \ at the end of each line or wrap the whole string in brackets ().  I used to have it all on one line and didn't double-check this, my bad. And indeed you also need to add the "f" for the f-string as @JohannesAck suggested.

I will try to push a fix for these changes soon. In the meantime I hope my answer helps clarify things.


Add support for temporary credentials

Describe the solution you'd like
Amplify CLI should support temporary credentials set as environment variables in the terminal.  Every other tool and SDK under the sun supports these.
Describe alternatives you've considered
Using profiles defined in ~/.aws/credentials with the temporary credentials that I have to update every 12 hours.
Additional context
It seems Amplify CLI only supports a profile OR pasting your IAM credentials from an IAM user into the configure prompt.
This is especially cumbersome when one is used to a workflow that uses a tool like aws-runas.

If you configure the Amplify CLI to use 'general' config level on your project, then the CLI will not configure credentials for the aws sdk it internally uses. This allows the aws sdk to resolve credentials by itself, and environmental variables will be used. ref
This can be done in two ways:

headless mode:
Run amplify init or amplify configure project in headless mode, and set the configLevel  in AWSCLOUDFORMATIONCONFIG as general. check this example.

Manually executeamplify configure project on a project already init'ed by the Amplify CLI,
and when prompted Do you want to update or remove the project level configuration
select remove

Thanks for the tips.  Is this documented somewhere?

It is explanation in this doc.

The links in this issue appear to be a bit out of date.  I think I found the relevant docs at https://docs.amplify.aws/cli/usage/headless#sample-script .
That said, I thought the whole point of amplify was to simplify the building process. Where I work, we have fully embraced Control Tower and the whole AWS multi-account strategy.  We don't allow our solutions teams to create IAM users except in very specific circumstances-- mainly we use the temporary credentials we get from the account portal (ala AWS SSO) if we need to run something via the CLI.
The fact that I have to cobble together a shell script to say I want to use the access key ID, secret access key, and session token is really kind of unwieldy.  I tried running this from the CLI, and amplify was able to create some resources in my target account, but then the script failed with "Failed to get profile: Profile configuration is missing for: undefined".  As a newcomer, this doesn't fill me with a lot of confidence that the process is set up correctly.  Trying to run amplify push just kind of sits there and spins after that.  Doesn't look like there is an actual network connection if I look for one in another terminal window.
Am I missing something?  Is there some more straightforward way I am supposed to run this using temporary credentials?


Property 'commuter' of type 'BenefitFeaturesAndOperations | null | undefined' is not assignable to 'string' index type 'BenefitFeaturesAndOperations | null'

Hi there!
I am getting these errors when i install finch. Why this is happening?
node_modules/@tryfinch/finch-api/resources/hris/benefits/benefits.d.ts:94:5 - error TS2411: Property 'commuter' of type 'BenefitFeaturesAndOperations | null | undefined' is not assignable to 'string' index type 'BenefitFeaturesAndOperations | null'.

94     commuter?: BenefitFeaturesAndOperations | null;
       ~~~~~~~~

node_modules/@tryfinch/finch-api/resources/hris/benefits/benefits.d.ts:95:5 - error TS2411: Property 'custom_post_tax' of type 'BenefitFeaturesAndOperations | null | undefined' is not assignable to 'string' index type 'BenefitFeaturesAndOperations | null'.

95     custom_post_tax?: BenefitFeaturesAndOperations | null;
       ~~~~~~~~~~~~~~~

node_modules/@tryfinch/finch-api/resources/hris/benefits/benefits.d.ts:96:5 - error TS2411: Property 'custom_pre_tax' of type 'BenefitFeaturesAndOperations | null | undefined' is not assignable to 'string' index type 'BenefitFeaturesAndOperations | null'.

96     custom_pre_tax?: BenefitFeaturesAndOperations | null;
       ~~~~~~~~~~~~~~

node_modules/@tryfinch/finch-api/resources/hris/benefits/benefits.d.ts:97:5 - error TS2411: Property 'fsa_dependent_care' of type 'BenefitFeaturesAndOperations | null | undefined' is not assignable to 'string' index type 'BenefitFeaturesAndOperations | null'.

97     fsa_dependent_care?: BenefitFeaturesAndOperations | null;
       ~~~~~~~~~~~~~~~~~~

node_modules/@tryfinch/finch-api/resources/hris/benefits/benefits.d.ts:98:5 - error TS2411: Property 'fsa_medical' of type 'BenefitFeaturesAndOperations | null | undefined' is not assignable to 'string' index type 'BenefitFeaturesAndOperations | null'.

98     fsa_medical?: BenefitFeaturesAndOperations | null;
       ~~~~~~~~~~~

node_modules/@tryfinch/finch-api/resources/hris/benefits/benefits.d.ts:99:5 - error TS2411: Property 'hsa_post' of type 'BenefitFeaturesAndOperations | null | undefined' is not assignable to 'string' index type 'BenefitFeaturesAndOperations | null'.

99     hsa_post?: BenefitFeaturesAndOperations | null;
       ~~~~~~~~

node_modules/@tryfinch/finch-api/resources/hris/benefits/benefits.d.ts:100:5 - error TS2411: Property 'hsa_pre' of type 'BenefitFeaturesAndOperations | null | undefined' is not assignable to 'string' index type 'BenefitFeaturesAndOperations | null'.

100     hsa_pre?: BenefitFeaturesAndOperations | null;
        ~~~~~~~

node_modules/@tryfinch/finch-api/resources/hris/benefits/benefits.d.ts:101:5 - error TS2411: Property 's125_dental' of type 'BenefitFeaturesAndOperations | null | undefined' is not assignable to 'string' index type 'BenefitFeaturesAndOperations | null'.

101     s125_dental?: BenefitFeaturesAndOperations | null;
        ~~~~~~~~~~~

node_modules/@tryfinch/finch-api/resources/hris/benefits/benefits.d.ts:102:5 - error TS2411: Property 's125_medical' of type 'BenefitFeaturesAndOperations | null | undefined' is not assignable to 'string' index type 'BenefitFeaturesAndOperations | null'.

102     s125_medical?: BenefitFeaturesAndOperations | null;
        ~~~~~~~~~~~~

node_modules/@tryfinch/finch-api/resources/hris/benefits/benefits.d.ts:103:5 - error TS2411: Property 's125_vision' of type 'BenefitFeaturesAndOperations | null | undefined' is not assignable to 'string' index type 'BenefitFeaturesAndOperations | null'.

103     s125_vision?: BenefitFeaturesAndOperations | null;
        ~~~~~~~~~~~

node_modules/@tryfinch/finch-api/resources/hris/benefits/benefits.d.ts:104:5 - error TS2411: Property 'simple' of type 'BenefitFeaturesAndOperations | null | undefined' is not assignable to 'string' index type 'BenefitFeaturesAndOperations | null'.

104     simple?: BenefitFeaturesAndOperations | null;
        ~~~~~~

node_modules/@tryfinch/finch-api/resources/hris/benefits/benefits.d.ts:105:5 - error TS2411: Property 'simple_ira' of type 'BenefitFeaturesAndOperations | null | undefined' is not assignable to 'string' index type 'BenefitFeaturesAndOperations | null'.

105     simple_ira?: BenefitFeaturesAndOperations | null;
        ~~~~~~~~~~

Found 12 errors in the same file, starting at: node_modules/@tryfinch/finch-api/resources/hris/benefits/benefits.d.ts:94

error Command failed with exit code 2.

Hey @kajan746, where exactly are these errors happening? When you run tsc?
I suspect you might have an import that looks like this
import { thing } from '@tryfinch/finch-api/src';

i.e. any import that includes src. These files are only included for better IDE experience and shouldn't be actually used at runtime.

Hi @RobertCraigie! Sorry for the late reply.

I am getting the above error when i commit my changes. And following is my tsconfig content
{   "compilerOptions": {     "target": "ES2020", /* Specify ECMAScript target version: 'ES3' (default), 'ES5', 'ES2015', 'ES2016', 'ES2017','ES2018' or 'ESNEXT'. */     "module": "commonjs", /* Specify module code generation: 'none', 'commonjs', 'amd', 'system', 'umd', 'es2015', or 'ESNext'. */     "outDir": "dist", /* Redirect output structure to the directory. */     "rootDir": "src", /* Specify the root directory of input files. Use to control the output directory structure with --outDir. */     "strict": true, /* Enable all strict type-checking options. */     "alwaysStrict": true, /* Parse in strict mode and emit "use strict" for each source file. */     "moduleResolution": "node", /* Specify module resolution strategy: 'node' (Node.js) or 'classic' (TypeScript pre-1.6). */     "esModuleInterop": true, /* Enables emit interoperability between CommonJS and ES Modules via creation of namespace objects for all imports. Implies 'allowSyntheticDefaultImports'. */     "experimentalDecorators": true,     "baseUrl": "src"   },   "exclude": [     "node_modules",   ],   "include": [     "src/**/*",     "src/test/**/*",     "tests/**/*",   ] }
And No, I just imported like this.
import Finch from '@tryfinch/finch-api';

Thanks @kajan746, if you manually modify the BenefitsSupport type in the benefits.d.ts file to use this index signature instead, does the type error go away?
[k: string]: BenefitFeaturesAndOperations | null | undefined;

Yes, but we cannot do that right since its in the node modules?

yes sorry @kajan746 we are working on fixing this on our end & I wanted to verify if the solution we were planning on using would work for your case.

Oh okay.. it worked after replacing like this
export interface BenefitsSupport {     commuter?: BenefitFeaturesAndOperations | null | undefined;     custom_post_tax?: BenefitFeaturesAndOperations | null | undefined;     custom_pre_tax?: BenefitFeaturesAndOperations | null | undefined;     fsa_dependent_care?: BenefitFeaturesAndOperations | null | undefined;     fsa_medical?: BenefitFeaturesAndOperations | null | undefined;     hsa_post?: BenefitFeaturesAndOperations | null | undefined;     hsa_pre?: BenefitFeaturesAndOperations | null | undefined;     s125_dental?: BenefitFeaturesAndOperations | null | undefined;     s125_medical?: BenefitFeaturesAndOperations | null | undefined;     s125_vision?: BenefitFeaturesAndOperations | null | undefined;     simple?: BenefitFeaturesAndOperations | null | undefined;     simple_ira?: BenefitFeaturesAndOperations | null | undefined;     [k: string]: BenefitFeaturesAndOperations | null | undefined; }

This should be fixed in v5.22.9, please try it out and confirm :)

Latest NPM package is not available yet. i'll let you know once its ready

Hey @kajan746, the new version of the finch sdk has been released!

Great! thank you :)

And it works!


How to use buck2 modes ?

Hello!
I have a small example here (buck2_example) where I try to use buck2 mode during build.
I run it using buck2 run @mode/opt //:main, but I'm getting this error:
thread 'main' has overflowed its stack
fatal runtime error: stack overflow
[1]    10198 abort      buck2 run @mode/opt //:main

Could you help ?
For more context what I would like to achieve is to be able to invoke buck2 build with different compilers and compiler flags (and probably also different cpu architectures). Could you advice on the best practices for it ? From the official documentation is not quite described or at least I could not found it.
Thanks!

This is just a wild guess, but maybe this line shouldn't be here? https://github.com/MarcoBonino/buck2_example/blob/main/mode/opt#L2 isn't that like, recursively including itself? that would explain the stack overflow.


TrackFeature futureStyle does not override style

I would expect the opposite to be true.
feature.style('strokeOpacity', 1);
feature.futureStyle('strokeOpacity, 0); // This line will be ignored.
feature.draw();

I wanted to provide a repro branch, but because of #1163 I was unable to do so.

I realized I could just type it into https://opengeoscience.github.io/geojs/tutorials/tracks/


[11.x] Add column name customisation options for Auth used with database provider

This pull request will add:
This pr will add the option to customise column names, in the users table when using the Auth login functions in combination with the database driver. Column name customisation is also possible when eloquent is used (by overriding the getter functions in the users model, but there is no way to do it when the database provider is used.
How to use:
Users can customise the column names by editing the providers config. My first thought was to add a new 'database' key, that is commented out by default, but since 'table' is already located in providers, it seems best to place it there.
By default, the eloquent provider is used, but when the database provider is used, you can easily replace the column names in the config.
    'providers' => [
        // 'users' => [
        //     'driver' => 'eloquent',
        //     'model' => env('AUTH_MODEL', App\Models\User::class),
        // ],

        'users' => [
            'driver' => 'database',
            'table' => 'users',
            // The columns config is optional, to be backwards compatible.
            // When not set, default column names will be used.
            'columns' => [
                'primary_key' => 'id',
                'password' => 'password',
                'remember_token' => 'remember_token',
            ],
        ],
    ],

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.

@devfrey Can you elaborate on your feedback? I feel like the database auth provider is lacking customisation, even if it isn't used as much as the eloquent method.
Is there a better way to implement it, or do you think column name customisation just doesn't add enough value?

@Daniel-H123 The user providers can already be customized or extended. Simply implement UserProvider or extend the DatabaseUserProvider and make changes as necessary. There's no need to further complicate the existing implementation just for your edge case. See https://laravel.com/docs/11.x/authentication#adding-custom-user-providers for further guidance.

I see. I agree that it makes the implementation more complicated. Thanks for explaining.


chore: increase the TTL for GitOps jobs

The 1 day TTL seems a tad excessive. Could 7 days be a default?

Some context here: https://kubernetes.slack.com/archives/C9MBGQJRH/p1675377751318029

/assign @msvticket @ankitm123

Related to #3431

/ok-to-test

Hey @gazal-k, just got versionstream chugging again. Can you update your branch to latest and we can get this merged?

Hey @gazal-k, just got versionstream chugging again. Can you update your branch to latest and we can get this merged?

Could you please let me know what is failing in the pipeline? I don't have the access to check.

/test gsm
@gazal-k  I think it's an ephemeral error


[$500] Assign task -After creating a task user is navigated back to the 1:1 conversation

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: v1.4.12-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by:
Slack conversation: @
Action Performed:
Preconditions: User is logged in.

Open https://staging.new.expensify.com/
Navigate to a 1:1 conversation with a user
Click on the + in the compose box
Select "Assign task"
Enter a Title and a Description
Press Next
Assign the task to yourself
VShare the task (Share somewhere)
Click "Create the task"

Expected Result:
The task is created and the user is navigated to the task "conversation"/Report
Actual Result:
The task is created and the user is navigated to the 1:1 conversation
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/Expensify/App/assets/115492554/99872d1b-ed28-4bd1-935b-95d0c495397c

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01261dbcad1b761a40
Upwork Job ID:<PHONE_NUMBER>368058368
Last Price Increase: 2023-12-13

Proposal
Please re-state the problem that we are trying to solve in this issue.
After creating a task user is navigated back to the 1:1
What is the root cause of that problem?
Here we navigate the user to the parent report not the created one
https://github.com/Expensify/App/blob/63a53805e65c17767c4328eab817415147a56cfa/src/libs/actions/Task.js#L224
What changes do you think we should make in order to solve the problem?
change it to:
Navigation.dismissModal(taskReportID );

I actually think this is the preferred outcome - we just end up in the same chat where we created the task (@kadiealexander if you agree, can we get the testrail updated for this?)

Ooh, I agree! Yes, I'll raise a GH to get testrail updated.

https://github.com/Expensify/Expensify/issues/349515


Update references to sample study

Rationale
There was a study in the root sampledata/study used by numerous tests. They often had the study's location hard-coded instead of using TestFileUtils.getSampleData
Related Pull Requests

https://github.com/LabKey/testAutomation/pull/376

Changes

Update reference to sample study
Upload study instead of setting pipeline root

Running private build to verify updated tests: https://teamcity.labkey.org/viewQueued.html?itemId=1037325


Correct a -> an in og:description

Sorry, this was bothering the grammar Nazi inside me :-)

Haha sure! Still a good PR to send over, thanks! 🎉


Replace process.env.REACT_APP_CHAIN_ID with CHAIN_ID from config

Now that we've introduced a CHAIN_ID constant in the config, we can reference it from everywhere in the app without using the environment variable directly.

Looks like a race condition with the storybook is showing up here for the vault page


Api inconsistency in resolve/relay_get

closes #37

Thanks! Looks good

@dvc94ch Do you care about versioning purity at this point? don't want to whip a v2.0.0 just for this.


The change event is triggered even if no actual change

The consensus of algolia/algoliasearch-helper-js#305 is "no change event if no changes". This is therefore a bug to fix.

The impact of extra processing due to change events doesn't match up against the logic needed to check all parameters IMO. As nobody else has asked this, let's close for now.


Splash screen in v5 seems to be broken

Describe the bug
After updating to 5.0.1 the splash screen is nearly blank. But after taking a closer look the splash.png is in the upper left corner, but super tiny.
To Reproduce
Steps to reproduce the behavior:

update to v5
config a splash screen
yarn cap open @capacitor-community/electron

Expected behavior
Show the spash.png filling the view.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Ubuntu
Version 23.10

Additional context
I guess this change is leading to this behaviour: https://github.com/capacitor-community/electron/blame/f7ce86454469e967b41c1db8f51ddb942d278ca9/src/electron-platform/ElectronSplashScreen.ts#L48
I also played around with the before and after html snippets an ran in the same issue.

Its already fixed and in the main branch:
Fix: Put the image size into the splashscreen #272
but we need a new release for that at npm :)

Ok. But that is just a PR, so it is not on the main branch. There is even another PR for this issue https://github.com/capacitor-community/electron/pull/201

@Frooger  any plan on merging this PR?


Cannot read property set of undefined

Cannot read property 'set' of undefined","stack":"TypeError: Cannot read property 'set' of undefined\n    at OrbitDB._addManifestToCache (/home/james/test/node_modules/orbit-db/src/OrbitDB.js:373:17

Hey @voxsoftware are you using a custom cache? Can you provide the code which throws the above error?

Ping @voxsoftware

Closing for now as more info needed. Pleas do reopen if this is still an issue :)


(git) Update to Version 2.39.1 from 18th of january 2023 to address 3 critical/high CVEs

Checklist

[X] I have checked the moderation queue that no new version has been submitted (only visible when logged in), or the submitted package version has failed the automated checks.

Please navigate to the following link to view the moderation queue.

[X] I have verified that the new version is a stable release.
[X] I have looked to see if there are any Bug reports that prevents a new version being submitted and that no existing Outdated Reports have been created.
[X] I have verified the new version has a Windows binary (EXE or MSI installer, Zip or other archive, etc).

New Software Version
v2.39.1.windows.1
Download location
https://github.com/git-for-windows/git/releases/tag/v2.39.1.windows.1
Package Page
https://community.chocolatey.org/packages/git

@ickk this package version has already been pushed, and is working its way through the moderation process.


Extra tags cause Loki to internal error

I tried the following
import logging
import logging_loki

handler = logging_loki.LokiHandler(
    url="https://$MYSERVER/api/prom/push",
    tags={"application": "bruno-test"},
)

logger = logging.getLogger("bruno-logger")
logger.addHandler(handler)
logger.error(
    "Something happened",
)

and that works:

but I do:
logger.error(
    "Something happened",
    extra={"tags": {
        "bruno-tag": "xxx"
    }},
)

I get
--- Logging error ---
Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/logging_loki/handlers.py", line 72, in emit
    raise ValueError("Unexpected Loki API response status code: %s" % resp.status_code)
ValueError: Unexpected Loki API response status code: 500
Call stack:
  File "a.py", line 14, in <module>
    "bruno-tag": "xxx"
  File "/usr/local/lib/python3.7/logging/__init__.py", line 1407, in error
    self._log(ERROR, msg, args, **kwargs)
  File "/usr/local/lib/python3.7/logging/__init__.py", line 1514, in _log
    self.handle(record)
  File "/usr/local/lib/python3.7/logging/__init__.py", line 1524, in handle
    self.callHandlers(record)
  File "/usr/local/lib/python3.7/logging/__init__.py", line 1586, in callHandlers
    hdlr.handle(record)
  File "/usr/local/lib/python3.7/logging/__init__.py", line 894, in handle
    self.emit(record)
  File "/usr/local/lib/python3.7/site-packages/logging_loki/handlers.py", line 74, in emit
    self.handleError(record)

on Loki server side:
level=warn ts=2019-09-25T05:34:17.971104105Z caller=logging.go:49 traceID=5ff3ee51249a93 msg="POST /api/prom/push (500) 242.786µs Response: \"parse error at line 1, col 71: syntax error: unexpected IDENTIFIER, expecting = or != or =~ or !~\\n\" ws: false; Accept: */*; Accept-Encoding: gzip, deflate; Content-Length: 205; Content-Type: application/json; User-Agent: python-requests/2.22.0; X-Forwarded-For: <IP_ADDRESS>; X-Forwarded-Host: $MYSERVER; X-Forwarded-Port: 443; X-Forwarded-Proto: https; X-Forwarded-Server: traefik-7856fd8df5-n4t8d; X-Real-Ip: <IP_ADDRESS>; "

i'm running loki official build from docker image grafana/loki:v0.3.0

@bclermont
I believe this is due to the hyphen in the "bruno-tag". I'm not sure that loki supports tags that have a hyphen. It may be worthwhile to replace the hyphen with an underscore when sending a request to loki.


Improvements in BindingBuilderTests to check binding parameters.

BindingBuilderTests didn't check parameters of binding, e.g. DestinationType, ExchangeName. Also added test for checking HeadersExchange binding.

Merged as https://github.com/spring-projects/spring-amqp/commit/c294fe7c0d49e744ecc5b1e0c1e570702a5b4114.
@neetkee , thank you for contribution. Looking forward for more!
@panchenko , thanks to you, too, for review!


docs: ADR 019 strict inflation schedule

ADR for https://github.com/celestiaorg/celestia-app/issues/1570

Pinging @evan-forbes @cmwaters @liamsi for review


add padding-mode choice to RandomCrop

In torch, RandomCrop can set padding mode but in PyTorch we cannot. So I think this feature may help many researchers including me!

@fmassa I think the PR #489 already handled this problem too and fixed some other problems

Indeed, looks like this patch is a subset of #489 that I have missed.
I'm reverting this and will merge #489 then


Support for @ngrx/store 16

I get a warning that ngrx-slice depends on @ngrx/store smaller than 16.

Release v7 to support angular 16


add sw[op] file to gitignore & make start script be executed without …

add sw[op] file to gitignore & make start script be executed without errors under mac os platform

thanks @kakiezhang, Done!!!!


security hole bug that staff can cheat in pos

Hello @yelizariev
I have found security hole bug that staff can cheat in pos.
The situation is:
1 water going to order 3 pizza and submit order to the kitchen (It work fine for normal workflow)
and the restaurant serves the client as normal.
and guest going to pay for the bill (Water ask for payment total 3 pizza) it is ok, guest leave.
but the problems are: the water deleted 2 of the pizza billing and check out the table.
the pos system has no notified or any cancel in the posted in the system
so the way to fixed that is:
the water can cancel (deleted order) but when he/she try to take payment (check out table) the system pop up notifies that order has been changed if they say yes, order cancel will be printed in the kitchen and posted to the system.
if they say no so they have to correct the order number and take payment.
Can you help for that?

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Check this: https://github.com/it-projects-llc/pos-addons/pull/389
it saves all removing at backend, but it doesn't print cancelation automatically.
We consider it as interesting option and could add it option if someone sponsor it.

Hello! that fine but the currently it does not save in my system what can I do?
I have updated the latest code. do I need to do any extra step?

@W360S
I was not merged to this repo until today. Try again.

do i have to install any extra module? @yelizariev

You need to install pos_order_cancel_restaurant

i try to install that module but a lot of error.
whole system fails not sign with multi section

I forgot to say, that you also need to install this module: pos_multi_session_restaurant_cancel_order

it breaks my system now, show error when im trying to login pos section.
here is the error: #381


Could not find module 'SQLite' for architecture 'x86_64'; found: arm64, arm, armv7

Issues are used to track bugs and feature requests.
Need  help or have a general question? Ask on Stack Overflow (tag sqlite.swift).

Build Information

Include the SQLite.swift version, commit or branch experiencing the issue.
pod 'SQLite.swift', '~> 0.12.0'
Mention Xcode and OS X versions affected.
Xcode - 10.2, MacOS Mojave 10.14.4
How do do you integrate SQLite.swift in your project?

CocoaPods with version 1.6.1

General guidelines

Be as descriptive as possible.
Provide as much information needed to reliably reproduce the issue.
Attach screenshots if possible.
Better yet: attach GIFs or link to video.
Even better: link to a sample project exhibiting the issue.

Duplicate of #959


Make ImageView zoom scale dynamic with window size

Details
Following up on https://github.com/Expensify/App/pull/6535, cc @railway17
Fixed Issues
$ none
Tests / QA Steps

Prepare a very long image.
Send the image to someone.
Click the image thumbnail in the chatbox.
Verify modal opens to display the image.
Verify image is center aligned.
Check the cursor while hovering around the image.
Check the zoom in and zoom out functionality.
Resize the window and make sure the image zoom region adjusts accordingly
Prepare a very wide image and send it to another.
Repeat steps 2-8 with the wide image.

Tested On

[ ] Web
[ ] Mobile Web
[ ] Desktop
[ ] iOS
[ ] Android

Screenshots

Web

Mobile Web

Desktop

iOS

Android

Don't remember the context here, going to just close this.


unable to bind listening socket for address '/run/php/php7.0-fpm.sock'

[SYSLOG] php-fpm[96]: [ERROR] unable to bind listening socket for address '/run/php/php7.0-fpm.sock': No such file or directory (2)
[SYSLOG] php-fpm[96]: [ERROR] FPM initialization failed
[ERROR] unable to bind listening socket for address '/run/php/php7.0-fpm.sock': No such file or directory (2)
[ERROR] FPM initialization failed
2017-02-16 20:04:08,569 INFO exited: php-fpmd (exit status 78; not expected)
2017-02-16 20:04:09,572 INFO success: syslogd entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
2017-02-16 20:04:09,575 INFO spawned: 'php-fpmd' with pid 136
2017-02-16 20:04:09,692 INFO success: php-fpmd entered RUNNING state, process has stayed up for > than 0 seconds (startsecs)

I'll try to sort out ASAP.

Fixed in next stable release. Thank you


Content Editable with empty trigger character not working.

It works fine, if a non empty trigger character is set. I have created a fiddle to explain this.
https://fiddle.jshell.net/t5wveetw/5/show/

not support empty trigger character by default.


11/18 PM Publish

Docs Build status updates of commit 2bf9105:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-conceptual/azps-5.0.0/index.yml
:white_check_mark:Succeeded
View (azps-5.0.0)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


SAT solver set parameters num_search_workers

Hi,
I try to run the example on https://github.com/google/or-tools/blob/master/examples/python/scheduling_with_transitions_sat.py
Set the num_search_workers greater than 1, but the status of solver become unknown.
I can only set this parameter to 0 or 1 and it can work normally. Thanks~

Platform: Win10 64 bit
Python version:  I test it in two python version. 3.6.5&3.5.3. Both version have the same problem.

ortools-6.9.5824

Is there any method to enable multi-treads solve for python? Thanks~

Fixed in the source. Should be out in 6.10.

Dear Laurent,
Thanks for help. I install from source and it work perfectly now.

Great. Thanks for the report.
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
Le lun. 8 oct. 2018 à 07:30, JohnsonLCS<EMAIL_ADDRESS>a écrit :

Dear Laurent,
Thanks for help. I install from source and it work perfectly now.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/870#issuecomment-427725175,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKj17QUSSLybmRce-TrPUabSpkzK9_Umks5uiuMKgaJpZM4XHMJK
.


Fix error in NODE_RESOURCE_GROUP:

NOTE:  I tried to submit a PR for this issue, but Github kept giving 500 errors.
Proposed change:
NODE_RESOURCE_GROUP=`az aks show --name $cluster_name  --resource-group $resource_group -o tsv --query 'nodeResourceGroup' | base64`

On a Mac, the command provided to get NODE_RESOURCE_GROUP doesn't work as expected.  This replacement uses the Azure CLI's native --query option to parse out the data needed, thus producing a more reliable reliable output:
# echo $CLUSTER_INFO | sed -e 's/^.*"nodeResourceGroup"[ ]*:[ ]*"//' -e 's/".*//'
{

      },

    },

      },

    }
  },

    {

    }
  ],

  },
MC_sandbox-aks-110-centralus_sandbox-aks-110-centralus_centralus

  },

}
# az aks show --name $CLUSTER_NAME  --resource-group $RESOURCE_GROUP -o tsv --query 'nodeResourceGroup'
MC_sandbox-aks-110-centralus_sandbox-aks-110-centralus_centralus

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

ID: 301b44e3-4be7-6364-01b4-c2a55ff9b14a
Version Independent ID: 62569967-65c6-ece6-2d1e-689a73dd59ca
Content: Kubernetes on Azure - Cluster Autoscaler
Content Source: articles/aks/autoscaler.md
Service: container-service
GitHub Login: @sakthivetrivel
Microsoft Alias: sakthivetrivel

Thanks for the feedback! We are currently investigating and will update you shortly.

Thanks! I'll be sure to update the document.

@lastcoolnameleft

@lastcoolnameleft Thank you for the feedback!

There are upcoming changes around cluster autoscaler which are due within the next 4 weeks or so, with doc updates aligned with that. There are known issues with the current implementation, and we apologize for any issues you've encountered here.
For now, #please-close with the understanding this doc is being updated to reflect the upcoming autoscaler experience.
CC: @sauryadas


Provide API to handle situation when creating a vocabulary with IRI that already exists

Relates to https://github.com/opendata-mvcr/sgov-assembly-line/issues/36 and https://github.com/opendata-mvcr/mission-control/pull/226.
We need to handle situations when the user wants to create a vocabulary with IRI that already exists. We need to consider case when the existing vocabulary :

is already published
is created in a workspace but not published

Possible solution:

does not allow it if it is loaded or published
only when it is not published, provide the possibility to warn the user

this could be done by providing API to return all loaded vocabularies or to test for one vocabulary if it is loaded

I suggest going with option 1. But even for case 1, it makes sense to create an API that checks if vocabulary exists, so possible UI could show a warning upfront.

Implement solution 2

Temporary solution: deny creation on same IRI in SSP, otherwise, create.


Added metadata field which could be used to add custom attributes to the model

Metadata field is used to annotate elements and relationships with custom attributes which later could be used in other artifacts built from architecture model.

Nice job @pavelpykhtin, I like it!
Can you please add some smoke tests, like any from here

Can you please add some smoke tests, like any from here

Sure


Broken link to : FrameContext interface

Broken link in the below paragraph:
In the meeting as well as in all scenarios, the app will be rendered in an in-meeting tab that is 320px in width. Your tab must be optimized for this. See, FrameContext interface
broken link:
https://docs.microsoft.com/en-us/javascript/api/@microsoft/teams-js/microsoftteams.framecontext?view=msteams-client-js-latest&preserve-view=true
I assume this is the intended target ?
https://docs.microsoft.com/en-us/javascript/api/@microsoft/teams-js/framecontext?view=msteams-client-js-latest

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

ID: b60eeacc-a71e-9644-b869-3e0a4f1506d8
Version Independent ID: 313bf83f-5325-f9f4-5780-9bc81e315c1e
Content: Create apps for teams meetings - Teams
Content Source: msteams-platform/apps-in-teams-meetings/create-apps-for-teams-meetings.md
Product: msteams
GitHub Login: @laujan
Microsoft Alias: lajanuar

@laujan , Could you please help to update the broken link for Frame Context Interface in following documentation
:
Updated Link: https://docs.microsoft.com/en-us/javascript/api/@microsoft/teams-js/framecontext?view=msteams-client-js-latest


Get-AutomationConnection deprecated or not?

Get-AutomationConnection link leads to https://docs.microsoft.com/en-us/powershell/module/servicemanagement/azure/get-azureautomationconnection?view=azuresmps-4.0.0&viewFallbackFrom=azuresmps-3.7.0 which states that "This PowerShell command for Azure Automation will no longer be supported on 01/23/2020" - is that the case for Get-AutomationConnection too? It's not very clear from documentation here. Is it the same commandlet and will runbooks that use Get-AutomationConnection start failing after 01/23/2020? Thanks.

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

ID: 81c284ea-3656-836f-7eab-388773e0e382
Version Independent ID: 71329bef-2d4f-4ff6-5a03-83b99b2269e9
Content: Connection assets in Azure Automation
Content Source: articles/automation/automation-connections.md
Service: automation
Sub-service: shared-capabilities
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

Thanks for your comment. We are actively investigating and will get back to you shortly. Thanks for your patience.

@guntaled I believe no changes to Get-AutomationConnection cmdlet. Now the note in the link https://docs.microsoft.com/en-us/powershell/module/servicemanagement/azure/get-azureautomationconnection?view=azuresmps-4.0.0&viewFallbackFrom=azuresmps-3.7.0  states "The requested page is not available for azuresmps-3.7.0. You have been redirected to the newest product version this page is available for". Hope it clarifies now.

Hi @KrishnaG-MSFT , @guntaled
The link wrongly points to the Get-AzureAutomationConnection doc. The activity listed in the table is Get-AutomationConnection and is different from Get-AzureAutomationConnection that is being linked. The activity is not deprecated it will continue to work.
Activities are PS cmdlets that only work inside  an Azure Automation runbook and are different from the ARM/ASM PS cmdlets which can be used anywhere.
@MGoedtel we should remove this link as well as any other link from activities to PS cmdlets.

Corrected and updated article should go live tomorrow AM.  #please-close


Sending Messages to an IoT Device(C2D) by Using Message Feedback

Hi All,
I created a dotnet core(C#-7.3) consol app and wrote below code to Message Feedback at C2D situation
static async Task Main(string[] args)
        {
            var serviceConnectionString = "CON_STR";
            var serviceClient = ServiceClient.CreateFromConnectionString(serviceConnectionString);
           //msg delivery feedback for monitoring which msgs r not delivered
           var feedbackTask = ReceiveFeedback(serviceClient);
            while (true)
            {
                Console.WriteLine("Which device do you wish to send a message to? ");
                Console.Write("> ");
                var deviceId = Console.ReadLine();
                await SendCloudToDeviceMessage(serviceClient, deviceId); //send msg C2D

            }
        }
 private static async Task SendCloudToDeviceMessage(
            ServiceClient serviceClient,
            string deviceId)
        {
            Console.WriteLine("What message payload do you want to send? ");
            Console.Write("> ");

            var payload = Console.ReadLine();

            var commandMessage = new Message(Encoding.ASCII.GetBytes(payload));
            commandMessage.MessageId = Guid.NewGuid().ToString();
            commandMessage.Ack = DeliveryAcknowledgement.Full;
            commandMessage.ExpiryTimeUtc = DateTime.UtcNow.AddSeconds(10);

            await serviceClient.SendAsync(deviceId, commandMessage);
        }
 private static async Task ReceiveFeedback(ServiceClient serviceClient)
        {
            var feedbackReceiver = serviceClient.GetFeedbackReceiver();

            while (true)
            {
                var feedbackBatch = await feedbackReceiver.ReceiveAsync();

                if (feedbackBatch == null)
                {
                    continue;
                }

                foreach (var record in feedbackBatch.Records)
                {
                    var messageId = record.OriginalMessageId;
                    var statusCode = record.StatusCode;

                    Console.WriteLine($"Feedback for message '{messageId}', status code: {statusCode}.");

                }

                await feedbackReceiver.CompleteAsync(feedbackBatch);
            }
        }

It is giving an error at 'ReceiveFeedback' method
Pls guide, what things i am missing
Thanks,

@tutul2010 Thank you for the detailed feedback. We are actively investigating and will get back to you soon.

ANy update , Pls

Hello @tutul2010 please let us know the document you are following? If your question is not related with a specific doc, please leverage MSDN or Stack Overflow.
Thank you!

Thanks,
Pls #close


Some Flow bugs

Within declaration of modules:
declare module 'mymodule' {
     declare export class Server mixins UWS {}
}

I have a few more that I may post - have to confirm they are actual issues first :-P

Here is another one:


wifi.getVersion() and TCPServer example not working...

I'm facing with two different issues on this library:

I've tried to run the TCPServer example provided qith the library but after attaching to my Wi-Fi home network I didn't have any "signs of life" from the Arduino module via the Serial Monitor.
Through the browser I cannot see any received message from the Serial Monitor when I try to call th two IP address provided with the wifi.getLocalIP() function.
I've tried also to ping both the ip address assigned by the router to the "AccessPoint" and to the "Station" (es. 192.168.1.xx and the 192.168.4.x) without any success.

In addition I've got not result when invoking the wifi.getVersion() function.
May someone out there help me?
Thanks in advanced
p.s.: I add no code because I reter to the library example sketches (the function wifi.getVersion() is on every example sketch)

My environment: Arduino UNO + ESP-01 boards.

I found the communication between my arduino and ESP-01 board is working, somehow the text did not show as expected such as getVersion() . After do a research quickly I found the problem looks like is causing by "parsing function error". If you take a look at the implementation of this project can find easily, the library will parse the return string after issue a AT command. I believe the string sent from ESP-01 firmware is changed without notice so that this library is malfunction now.

Since my purpose is to confirm this ESP-01 works, so I did not fix the problem here.

I had to change this part of the code in ESP8266.cpp,
bool ESP8266::eATGMR(String &version)
{
    rx_empty();
    m_puart->println("AT+GMR");
    //return recvFindAndFilter("OK", "\r\r\n", "\r\n\r\nOK", version);
    return recvFindAndFilter("OK", "+GMR", "\r\nOK", version);
}

I changed all the recFindAndFilter response of each function


Update with license?

Thanks for sharing this repo Hector, was wondering if you could license it for open source?

You're under no obligation to choose a license. However, without a license, the default copyright laws apply, meaning that you retain all rights to your source code and no one may reproduce, distribute, or create derivative works from your work. If you're creating an open source project, we strongly encourage you to include an open source license. The Open Source Guide provides additional guidance on choosing the correct license for your project.

Sure, the repo is under the MIT license now.

Thank you!


long image low quality with blur

hi, I'm working on an application for reading manga, when I try to load a large image of 760 x 29095 pixels, the image loses a lot of quality and gets blurry on mobile devices, on PC and web everything is fine, but on mobile devices the quality becomes unreadable, apparently there is a problem with the fit images into a texture, but I couldn’t find parameters for controlling the size of the texture somewhere in the source code

Can you convert the same image to jpeg and test it with the regular Image widget? I think this might be related to https://github.com/flutter/flutter/issues/74172

unfortunately, yes, the problem persists with jpeg, tell me, is it possible to increase the texture size in flutter itself at a lower level, or will you still have to write your own module for the native one?

It is not possible because this plugin only decodes the image, rendering is still done via Flutter's RawImage widget. So the limitations of the Image widget also applies to here.

understood thanks


Release 0.11.0

This will be a feature release.

Coverage increased (+0.02%) to 83.612% when pulling 4ae56e9b8cb61f63babcb6d2c53519ea097cfab9 on develop into 564817e5c18f9ac9a092c26c13abac40892e9c27 on master.

Coverage increased (+0.02%) to 83.612% when pulling b16a551cebc84ee42b4589318d67fa3c39f09579 on develop into 564817e5c18f9ac9a092c26c13abac40892e9c27 on master.

Coverage increased (+0.02%) to 83.612% when pulling b16a551cebc84ee42b4589318d67fa3c39f09579 on develop into 564817e5c18f9ac9a092c26c13abac40892e9c27 on master.


Docs are not up to date with API

Recently working on a project that had this included but the mixin name as well as its params do not match up with the current documentation.

@guylyons Thanks for the feedback. Are you able to see what version of the mixin the project is using?

@josephfusco My apologies, there is another package with the exact same name by a different author. Your project looks great, though. Maybe we'll switch! haha. Thanks for your quick response.