ztest: allow for user-defined main function

The goal is to be able to define and run test suites from anywhere and not obligatorily from test applications.
main() function isn't redefined.
User has to call test_main() to run the tests.

Should I add this feature only with the new API? 🤔

@yperess
I want to leverage ZTest in my main application without having to create new test applications so that tests are performed under my application context.
I could modify my code to not link my main function when ZTEST is enabled and use test_main instead but:

I don't want the report to be printed only when returning from test_main(), but right after the tests are run (asap)
I don't want to stop my application after tests are run

I could certainly find solutions to bypass the current behavior and make it work for my use case but I thought this was probably easier to remove definition of main() from ztest and make calling the tests easily accessible.
Let me know what you think

@yperess I want to leverage ZTest in my main application without having to create new test applications so that tests are performed under my application context. I could modify my code to not link my main function when ZTEST is enabled and use test_main instead but:

I don't want the report to be printed only when returning from test_main(), but right after the tests are run (asap)

test_main() should be running the tests, is the issue that you're trying to merge it with your application logic so this function has extra latency (or doesn't return?)

I don't want to stop my application after tests are run

I could certainly find solutions to bypass the current behavior and make it work for my use case but I thought this was probably easier to remove definition of main() from ztest and make calling the tests easily accessible.
Let me know what you think

I have 2 concerns currently that I'm trying to work around:

We do have initialization code in ztest's main() function, so as that changes you (or any other downstream user) would have to keep up with the changes.
ztest tightly binds to twister so similar to issue #1, you might have to keep up with changes to other parts of ztest (not just the main() function).

Would you be able to rename your application's main function instead based CONFIG_ZTEST? So what I'm suggesting is:
Rename your main entry point
#ifdef CONFIG_ZTEST
void app_main(void *arg0, void *arg1, void *arg2)
#else
void main(void)
#endif

Add a new thread that's only linked in when testing
/*
 * Include the following in your CMakeLists.txt:
 *     target_add_sources_ifdef(CONFIG_ZTEST app test_main.c)
 */

extern void app_main(void *, void *, void *);
K_THREAD_DEFINE(app_main_tid, CONFIG_MAIN_STACK_SIZE,
    app_main, NULL, NULL, NULL,
    CONFIG_MAIN_THREAD_PRIORITY, 0, 0);

void test_main()
{
  /*
   * Use a condvar here to wait for the tests to all run,
   * you'll need to trigger that from app_main or where
   * ever in your application you call:
   *     ztest_run_registered_test_suites()
   */
}

It's not as pretty as just replacing the main() function, but it should give you more flexibility without having to debug strange errors if we ever make changes to ztest. Alternatively (not trying to dissuade you from using ztest) if you don't want to use twister, you could link in gTest instead which would give you the additional flexibility you want, in theory it's possible to modify gTest to work with twister but I haven't had time to look into it.

@yperess I understand your concerns
I am closing so that we don't increase complexity over time 😉
Thanks!


Objective-C class method 'initialize', which is not guaranteed to be invoked by Swift and will be disallowed in future versions

Posted by @mrkd: https://github.com/Swinject/Swinject/issues/238
Objective-C class method 'initialize', which is not guaranteed to be invoked by Swift and will be disallowed in future versions
Xcode - Version 8.3 (8E162), Swift 3.1

Swinject (2.0.0)
SwinjectStoryboard (1.0.0):

Swinject (= 2.0.0)

/Pods/SwinjectStoryboard/Sources/Storyboard+Swizzling.swift:25:30: Method 'initialize()' defines Objective-C class method 'initialize', which is not guaranteed to be invoked by Swift and will be disallowed in future versions

I think this can be fixed by migrating this extension to Objective-C. Are there any objections to that? I can take a crack at it.

wouldn't that kinda defeat the purpose of this being an injection framework for Swift?

The project already uses an Objective-C class for the base storyboard subclass. Rewriting the swizzle extension in Objective-C shouldn't affect the public interface either.
That is assuming this can be done this way; I've looked into it a bit, and it seems like the swizzle extension references some static Swift methods, and I can't import SwinjectStoryboard-Swift.h for some reason.

I agree with @mpdifran's idea. Writing the extension in Objective-C sounds good. Originally I just tried write code in Swift as much as possible, but now I think writing the extension in Objective-C is simpler.

@yoichitgy I've got most of the work done, but I'm stuck with the import SwinjectStoryboard-Swift.h problem. Mind taking a look at my branch if I push it up?

@mpdifran, I answered in the pull request😃

@yoichitgy @mpdifran any progress?

Waiting on @yoichitgy for advice on how to proceed for the last initialize() warning. I'll update to remove the merge conflict though.

v1.1.2 fixed the issue. Thanks @mpdifran 👍👍👍


I need your help

Hi @Ganapati ,
I was given a public key and a prime.txt file. And I was challenged to decrypt a file.
I guess I need to regenerate the private key from the public key and the prime.txt file. But I do not know how to do it.
You seem to master the subject well. Can you help me ?

i can take a look on irc next week if you want :)
Le sam. 18 mai 2019 à 19:23, Guillaume Camus<EMAIL_ADDRESS>a
écrit :

Closed #73 https://github.com/Ganapati/RsaCtfTool/issues/73.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Ganapati/RsaCtfTool/issues/73?email_source=notifications&email_token=AAP47OYH763RZQ2OLBHZRZ3PWA3SJA5CNFSM4HNYEN6KYY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGORQRFXBY#event-2351061895,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAP47O73NKLXJI5GGKJEXMDPWA3SJANCNFSM4HNYEN6A
.


refactor: minor rewrites

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@bb109d6). Click here to learn what that means.
The diff coverage is 100.00%.

:exclamation: Current head 43fed14 differs from pull request most recent head dd77333. Consider uploading reports for the commit dd77333 to get more accurate results

@@           Coverage Diff           @@
##             main      #53   +/-   ##
=======================================
  Coverage        ?   99.49%
=======================================
  Files           ?        8
  Lines           ?      197
  Branches        ?       50
=======================================
  Hits            ?      196
  Misses          ?        1
  Partials        ?        0

Flag
Coverage Δ

unittests
99.49% <100.00%> (?)

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

Impacted Files
Coverage Δ

src/parser.ts
100.00% <100.00%> (ø)

src/utils.ts
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Cannot set property Transport of # which has only a getter

I used tone.js in react, but use
import Tone from "tone";

get error : Cannot set property Transport of #<Object> which has only a getter
demo: https://codesandbox.io/s/angry-tereshkova-q07wl?file=/src/index.js
tonejs: 14.7.39

Try import * as Tone from 'tone'

@tambien  get the same error

Seems related to #699. Could you describe the setup to reproduce the error?

If I don't use codesandbox, in my project use webpack compile, it will show error:
Uncaught (in promise) TypeError: Cannot assign to read only property 'Symbol(Symbol.hasInstance)' of function 'function AudioBuffer(options) {
      if (nativeOfflineAudioContextConstructor === null) {
        throw new Er...<omitted>... }'
    at audio-buffer-constructor.js:8
    at createAudioBufferConstructor (audio-buffer-constructor.js:8)
    at Module../node_modules/_standardized-audio-context<EMAIL_ADDRESS>(module.js:215)
    at __webpack_require__ (bootstrap:837)
    at fn (bootstrap:129)
    at Module../node_modules/_tone<EMAIL_ADDRESS>(AudioContext.js:1)
    at __webpack_require__ (bootstrap:837)
    at fn (bootstrap:129)
    at Module../node_modules/_tone<EMAIL_ADDRESS>(Global.js:1)
    at __webpack_require__ (bootstrap:837)

Saame issue both on local webpack or in codesandbox. Any news on this ?

Unable to recreate this error outside of the codesandbox example. These are my steps to try and reproduce it locally
npx create-react-app test-project
cd test-project
yarn add tone@next
yarn start

And added this import to index.js: import { Transport } from "tone";
It compiles and runs fine. could you walk me through the steps of reproducing this locally?

Going to close in favor of #699. Please continue discussion there. Still looking for a repro case, since i'm unable to reproduce this issue outside of codesandbox and unable to debug there.

I just encountered this same issue using tonejs 14.7.77 in a Gatsby v4 project:

TypeError: Cannot assign to read only property 'Symbol(Symbol.hasInstance)' of function 'function AudioBuffer(options){if(nativeOfflineAudioContextConstructor===null){throw new Error('Missing the nati......;}'

I’ll read through the AudioBuffer code and see if I can come up with anything.

I haven’t been able to find a quick solution, but I may have found some clues. I think this is actually an issue with the dependency standardized-audio-context.
An expanded stack trace referenced “createAudioBufferConstructor”. This and nativeOfflineAudioContextConstructor are both from the standardized-audio-context dependency. Looking closer at that library, it seems to directly assign Symbol.hasInstance in a few places, which explains the error.
I’ll log a bug on that repo as well, but I don’t have more time to look into this today. Hopefully this helps someone encountering the same issue!

Steps to reproduce:
This is far from a minimal demo, but the site I’m working on is open source at https://github.com/rileyjshaw/rileyjshaw.github.io. To see the build error locally:
npm i -g gatsby-cli
gatsby telemetry --disable  # optional
git clone --single-branch --branch dev https://github.com/rileyjshaw/rileyjshaw.github.io demo-site
cd demo-site
npm i
BABEL_ENV=test gatsby build


Modified Editor Input

Fixes https://github.com/Microsoft/HoloToolkit-Unity/issues/448

@StephenHodgson,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.
It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

@StephenHodgson,
Thanks for having already signed the Contribution License Agreement. Your agreement has not been validated yet. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@StephenHodgson,
Thanks for having already signed the Contribution License Agreement. Your agreement has not been validated yet. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@StephenHodgson,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@thebanjomatic @maxouellet what do you guys think?


Fixed trajectory planning memory leak.

This fixes a crucial memory leak when repeatedly calling the newLongTermTrajectory function.


💅: enchanted the colors

Fixed border error with the button, there was extra colon mark.
Also added the colors that were requested.

#0d2f41 -> for the variable "--background-main"
#1f2536 -> added as hover background value.

looks good, approved 👍


Add better logging

Currently the only logging we do is to print some of the graphics hardware info to the console, but a nicer approach would be to use the log crate to have proper logging levels/turn it off in release modes.
That said, if Tetra has no log output out of the box, that's going to make things a massive pain when it comes to reporting issues - people would have to install a logging library and hook it into their game to be able to figure out e.g. what drivers/OpenGL version is in use.
I think the approach I might want to take is to include fern (or something similar) in Tetra by default, but ensure it can be disabled via Cargo features if people want to use their own logger/a newer version.

A tricky thing here is that logging is very platform-dependent, so if we bundle a logger, we'd possibly have to provide two different ones for desktop and web. That then means we'd have to abstract over configuring those loggers, and uuuugh.
I think I might just leave this up to the end user to set up for now.

Yep, marking this as out of scope for now, in favour of providing the information/functionality needed so people can implement their own logging.
I am kinda tempted to write a facade crate for cross-platform logging at some point though...


ADO (devenv/pane '') An onscreen element must not have a null BoundingRectangle property.

The following accessibility issue needs investigation.
App: devenv
Element path: pane ''
Issue Details: An onscreen element must not have a null BoundingRectangle property. Section 508 502.3.1
How To Fix: If the element is offscreen, set its IsOffscreen property to TRUE.
If the element is onscreen, provide a BoundingRectangle property.


Banner Ads not aligned to center when screen is fullscreen and landscape.

The android app I am working on, has full screen mode enabled, and runs on landscape orientation.
When displaying banner ads, they tend to align too much to the left of the screen.
This happens because the plugin calculates left and right margins based on the display width, which ignores the navigation bar and status bar, even though they are hidden in full screen mode.
I have fixed the issue on my end, by getting the real width of the screen, detecting if we are in full screen mode, and applying the margin accordingly.
I would like to contribute the change.
Please have a look and accept PR if all is well.

PR #271

So this issue can be closed?


[Bug]: Internet speed is absolutely horrible

Describe the bug
After I installed Distrod, ubuntu focal, I tried installing some programs, but the internet speed is crazy slow.
I had normal WSL 2 installed previously, and the internet speed was fine, just wondering if this issue is well known or?
Kindly
Steps to reproduce
Install distrod, run a sudo apt-get install command
Expected behavior
No response
Windows version
21H2 19044.1348
Linux kernel version
Linux win <IP_ADDRESS>-microsoft-standard-WSL2 #1 SMP Wed Aug 25 23:20:18 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Distro
Ubuntu 20.04
Logs
.
additional comment
No response

Check what's slow is actually the Internet connection or just apt, please? Distrod doesn't have any ability to slow down the Internet connection. My guess is you're using the default config of Ubuntu apt that connects to default US servers, and your network location is far from them. Try updating the mirror server list.

I edited my /etc/apt/sources.list to use au.archive etc.. url's but it made no different in internet speed. I'm getting speeds of between 1kB/s and 50kB/s, average internet speed to America from Australia is about 400kB/s so, should still be more even if it was from America.
And I ran a speedtest-cli and the output was
lopu@win:~$ speedtest
Retrieving speedtest.net configuration...
Testing from iPrimus (<IP_ADDRESS>)...
Retrieving speedtest.net server list...
Selecting best server based on ping...
Hosted by Comvergence (Melbourne) [0.42 km]: 20.91 ms
Testing download speed................................................................................
Download: 4.37 Mbit/s
Testing upload speed......................................................................................................
Upload: 21.70 Mbit/s

One thing I noticed after installing was that a lot of programs like wget, curl, nano, were not installed already and I had to install them manually, is this normal with Distrod?

I don't think this is a distrod issue. I've tested installing vanilla WSL and it is still doing it. Weird that it only started after using distrod installer but hey. Who knows

this fixed it https://answers.microsoft.com/en-us/windows/forum/all/low-internet-speed-in-wsl-2/21524829-18be-4611-bb5f-cabccd2cae31


Return named_tuple from WebSocketResponse.can_prepare

What do these changes do?

Return named tuple from WebSocketResponse.can_prepare
Are there changes in behavior for the user?
No

Related issue number

https://github.com/KeepSafe/aiohttp/issues/1016
Checklist

[x] I think the code is well written
[x] Unit tests for the changes exist
[x] Documentation reflects the changes

Coverage decreased (-2.4%) to 95.564% when pulling 8236bd950839a21e6d0927ce52ced41af2dc5d28 on esaezgil:master into e2386a79e20db787d4e739e781ff0785693715b0 on KeepSafe:master.

Coverage decreased (-2.3%) to 95.595% when pulling 8236bd950839a21e6d0927ce52ced41af2dc5d28 on esaezgil:master into e2386a79e20db787d4e739e781ff0785693715b0 on KeepSafe:master.

Coverage decreased (-2.3%) to 95.655% when pulling 8236bd950839a21e6d0927ce52ced41af2dc5d28 on esaezgil:master into e2386a79e20db787d4e739e781ff0785693715b0 on KeepSafe:master.

Coverage decreased (-2.3%) to 95.595% when pulling 868457e80eea7ba50884e2b798b379e03b5b3f29 on esaezgil:master into e2386a79e20db787d4e739e781ff0785693715b0 on KeepSafe:master.

Coverage decreased (-2.4%) to 95.564% when pulling 30d82f2ce8a6f4a99704ce38f095c414717f2b65 on esaezgil:master into e2386a79e20db787d4e739e781ff0785693715b0 on KeepSafe:master.

Coverage decreased (-2.4%) to 95.564% when pulling 30d82f2ce8a6f4a99704ce38f095c414717f2b65 on esaezgil:master into e2386a79e20db787d4e739e781ff0785693715b0 on KeepSafe:master.

Coverage decreased (-2.3%) to 95.686% when pulling 9cceb5d5956266e3d1f60f53425b3858b3504f43 on esaezgil:master into e2386a79e20db787d4e739e781ff0785693715b0 on KeepSafe:master.

Implement __bool__ method.
Write tests

Coverage decreased (-2.3%) to 95.674% when pulling 187c85ae12d816e644ce6f57a83de8f1e7a3d161 on esaezgil:master into 1a61823974d3b635f9bc75125e36af3630906a1e on KeepSafe:master.

Coverage decreased (-2.3%) to 95.674% when pulling c154a76b8b55f67c870aa3061afab6aabae13a5b on esaezgil:master into 1a61823974d3b635f9bc75125e36af3630906a1e on KeepSafe:master.

Much better but tests and __bool__ are still required.

Current coverage is 97.59% (diff: 100%)

Merging #1020 into master will increase coverage by <.01%

@@             master      #1020   diff @@
==========================================
  Files            28         28
  Lines          6532       6535     +3
  Methods           0          0
  Messages          0          0
  Branches       1101       1102     +1
==========================================
+ Hits           6375       6378     +3
  Misses           83         83
  Partials         74         74

Powered by Codecov. Last update 1cc9462...11f8abc

Please add a test for __bool__.
I believe it is the last missing point.

Hi Andrew, thanks for reviewing the PR. I updated it after your last comment. Please let me know if you have any other comments.
Enrique

Thanks!


"workbench.action.openSettings" command with parameter doesn't filter settings anymore

Issue Type: Bug
Hello,
We're using the "workbench.action.openSettings" command to open the settings page with our setting name, to help our customers find it:
vscode.commands.executeCommand(`workbench.action.openSettings`, `Disconnect After Debugging`);

However, this doesn't work anymore: the settings page opens properly with the input text set, but the settings are not filtered properly anymore.
I'm not sure when it started reproducing, but it was definitely working fine a couple of months ago.
Could you please help with this?
VS Code version: Code - Insiders 1.58.0-insider (57930758db9e2c83f59b9ee1851400bd16192d32, 2021-06-30T05:14:00.370Z)
OS version: Windows_NT x64 10.0.19043
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Xeon(R) W-2133 CPU @ 3.60GHz (12 x 3600)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.73GB (14.71GB free)

Process Argv
--crash-reporter-id 5c157f73-8b5a-49bd-be58-51711bfc02c4

Screen Reader
no

VM
0%

Extensions (9)

Extension
Author (truncated)
Version

file-downloader
min
1.0.11

mindaro
min
1.0.120210615

csharp
ms-
1.21.18

vscode-aks-tools
ms-
0.0.4

vscode-kubernetes-tools
ms-
1.2.0

python
ms-
2020.8.103604

azure-account
ms-
0.8.11

java
red
0.62.0

vscode-yaml
red
0.8.0

A/B Experiments
vsins829:30139715
vsliv368cf:30146710
vsreu685:30147344
vspor879:30202332
vspor708:30202333
vspor363:30204092
vspre833:30321513
vsdfh931:30280409
vshan820:30294714
vscus158:30321503
vscod805:30301674
bridge0708:30335490
dockerwalkthru:30344881

Closing this issue: once I updated to the latest insider (1.59.0-insider), it doesn't reproduce anymore.


chore: update kzg module's abstraction

Pulls out the changes in #85

The main change here is adding a new OpeningKey that is only for kzgmulti -- this keeps the old OpeningKey "simple"


[FEATURE] Check if Table Exists

Summary:
Is there any easy way to check if a table exists before calling createTable()?
I don't see anything in the Docs: https://blazejkustra.github.io/dynamode/docs/guide/managers/tableManager

It works! Thanks!

@rafaelpolonio92 if you find some time please leave feedback on the library, what would you change about it or any missing features?
Also consider staring the repo ⭐


Feat store connection state and (own) facettype in redux-state onload #2823  (rebased)

blocked by #2840
related to #2828 (same changes just rebased with current master)
this pr stores the connection state and the (own) facet(type) of each connection directly after retrieving the connection container, this makes it easier to reload or prioritize loading by connectionstate and/or facet
fixes #2823
fixes #2840

was blocked by #2840 -> fixes/changes pushed directly into this branch


How can I listen to player data from $sope?

Let's say I have:
var source = video.multiSource();
source.addSource('mp4', 'videos/flux@flux.mp4');
source.addSource('mp4', 'videos/oscar@inception.mp4');
source.addSource('mp4', 'videos/react@iugh.mp4');
source.save();
$interval(()=>{
	console.log($scope.currentTime);
}, 750);

That would log undefined, how can I access complete player data and listen to events?

You'd have to use the feedback directive. Probably this will get a re-write with Angular 2 soon, and then we'd have an accessible API for such information.


Fix logging

This PR addresses a number of problems with the logging setup:
Java logging config
Due to Maven's previous configuration (or lack thereof) for the assembly jar, we had no control over the Java logging output (the application was reading a log4j config from one of the dependencies). This PR changes the log4j config to output dates in the ISO8601 format and makes sure that it gets installed in the assembly jar.
Stream redirection
The stdout / stderr redirection feature in the Python app was misleading, because only part of the streams were actually being redirected:
[simleo@simleo-U36SD:pydoop-features (logging)]$ pyfeatures --stderr err --stdout out serialize test/integration/37117_small.tif
[simleo@simleo-U36SD:pydoop-features (logging)]$ head out err
==> out <==
2016-08-31 08:49:55,384 INFO: [loci.formats.ImageReader] OMETiffReader initializing test/integration/37117_small.tif
2016-08-31 08:49:55,421 INFO: [loci.formats.in.MinimalTiffReader] Reading IFDs
2016-08-31 08:49:55,422 INFO: [loci.formats.in.MinimalTiffReader] Populating metadata
2016-08-31 08:49:55,431 INFO: [it.crs4.features.ImageToAvro] Reading from test/integration/37117_small.tif
2016-08-31 08:49:55,643 INFO: [it.crs4.features.ImageToAvro] Writing to 37117_small_0.avro
2016-08-31 08:49:55,644 INFO: [it.crs4.features.ImageToAvro] All done
==> err <==

[simleo@simleo-U36SD:pydoop-features (logging)]$ pyfeatures --stderr err --stdout out serialize README.md
Traceback (most recent call last):
  File "/home/simleo/.local/bin/pyfeatures", line 33, in <module>
    main(sys.argv[1:])
  File "/home/simleo/.local/lib/python2.7/site-packages/pyfeatures/app/main.py", line 77, in main
    args.func(args, extra_argv=extra_argv)
  File "/home/simleo/.local/lib/python2.7/site-packages/pyfeatures/app/serialize.py", line 45, in run
    sp.check_call(sp_argv, stdout=stdout, stderr=stderr)
  File "/usr/lib/python2.7/subprocess.py", line 540, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['java', '-cp', '/home/simleo/.local/lib/python2.7/site-packages/pyfeatures/pydoop-features-1.0-SNAPSHOT.jar', 'it.crs4.features.ImageToAvro', 'README.md']' returned non-zero exit status 1
[simleo@simleo-U36SD:pydoop-features (logging)]$ head err out
==> err <==
Exception in thread "main" loci.formats.FormatException: Invalid XML description
	at loci.formats.in.LIFReader.initFile(LIFReader.java:417)
	at loci.formats.FormatReader.setId(FormatReader.java:1426)
	at loci.formats.ImageReader.setId(ImageReader.java:835)
	at it.crs4.features.ImageToAvro.main(ImageToAvro.java:104)
==> out <==
2016-08-31 08:41:03,058 INFO: [loci.formats.ImageReader] LIFReader initializing README.md
2016-08-31 08:41:03,060 INFO: [loci.formats.FormatHandler] Reading header

[simleo@simleo-U36SD:pydoop-features (logging)]$ pyfeatures --stderr err --stdout out calc 37117_small_0.avro
[simleo@simleo-U36SD:pydoop-features (logging)]$ head err out
==> err <==
writing to /home/simleo/git/simleo/pydoop-features/37117_small_0_features.avro
processing [0, 0, 0]
All done
==> out <==

[simleo@simleo-U36SD:pydoop-features (logging)]$ pyfeatures --stderr err --stdout out calc README.md
Traceback (most recent call last):
  File "/home/simleo/.local/bin/pyfeatures", line 33, in <module>
    main(sys.argv[1:])
  File "/home/simleo/.local/lib/python2.7/site-packages/pyfeatures/app/main.py", line 77, in main
    args.func(args, extra_argv=extra_argv)
  File "/home/simleo/.local/lib/python2.7/site-packages/pyfeatures/app/calc.py", line 70, in run
    reader = AvroFileReader(fin)
IOError: Error opening file: Incorrect Avro container file magic number
[simleo@simleo-U36SD:pydoop-features (logging)]$ head err out
==> err <==
writing to /home/simleo/git/simleo/pydoop-features/README_features.md
==> out <==

The above examples show a number of issues:

Java logging to stdout and Python to stderr
Different formats in Java and Python (Python actually lost formatting when the redirection was on)
Actual error message "lost" to the console in pyfeatures calc

With this PR:

All logging is sent to stdout. Since all commands write their ouput to files, stdout can be used for the "normal" console output, while exception tracebacks and warnings go to stderr
Both the Java and the Python part log in the same format
Stream redirection is left to the calling process (i.e., the feature has been removed). See https://github.com/manics/celery-example/pull/1
Python logging has been centralized

TODO
Currently the logging level is hardwired (to INFO) in Java. This should be changed so thatpyfeatures serialize can pass the specified level down to the subprocess.

:+1: to getting the logging working, @simleo. However, what jar is being used here? bioformats_package should be using slf4j rather than log4j. (That's used by the outdated loci_tools).
cc: @sbesson

@joshmoore both Bio-Formats and pydoop-feature use slf4j, but that's just an abstraction layer that allows to plug in a "concrete" logging framework (log4j, logback, ...) at deployment time.


Experimental named field support in tsv-join

This PR is a follow-on to PRs #284, #285, and #286. It adds named field support to tsv-join. It works the same way as the support added to tsv-select, see PR #284 for more info.
Named field support is backward compatible with numeric fields. As with other tools, this support is not documented yet, even in the help text. Documentation will be added when named field support has been added to all tools and a release performed.
One notable behavior of named field support in tsv-join is that the --k|key-fields option applies to two different files, the "filter file" and the "data files". When a numeric field is specified, it refers the same field number in each file. However, when it's a named field, the columns may have different field numbers in each file. Previously, if the join key was in different columns, the distinction required using both the --k|key-fields and --d|data-fields options. Now only the --k|key-fields option is needed if the join keys has the same field names in both files.
This is a step towards enhancement request #25.

Codecov Report

Merging #288 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #288   +/-   ##
=======================================
  Coverage   99.32%   99.33%
=======================================
  Files          18       18
  Lines        6556     6575   +19
=======================================
+ Hits         6512     6531   +19
  Misses         44       44

Impacted Files
Coverage Δ

common/src/tsv_utils/common/fieldlist.d
99.87% <100.00%> (ø)

tsv-join/src/tsv_utils/tsv-join.d
100.00% <100.00%> (ø)


Enforce the usage of github.com/stretchr/testify/require

Signed-off-by: Julien Pivotto<EMAIL_ADDRESS>

I would like to know why.

All I can think right now is: Keep it consistent. I haven't dug the differences between require and assert, but I would like to know if you had some use for assert.

Inconsistent use of require and assert caused panics during tests. It is a way to make sure tests fail early, which is the best way to avoid those panics.
We can not expect all the contributors to learn the difference between both, especially because we want to lower the barrier to contribute. Choosing a single one (even if it is not the perfect technical solution for every situation) helps reaching that goal.

if you had some use for assert

Yes, the use for assert is to allow all incorrect results to be reported on a single run of a test. Also it is significantly faster, but that wasn’t a factor on the day I asked the question.
(Clearly you are free to make the rules on a project you maintain, but it’s nice to know why they are set up that way)


Removed submodules

This pull requests removed the (git) submodule from miniaudio, replacing it by plain files. This makes importing from go much easier when using GO MODULES.

Already done today :) No need for all files, just .h file is enough.

Great ! Thks
Xavier
Le ven. 17 avr. 2020 à 18:30, Milan Nikolic<EMAIL_ADDRESS>a
écrit :

Already done today :) No need for all files, just .h file is enough.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gen2brain/malgo/pull/17#issuecomment-615342876, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABX67VYING65ZYVIPP77ISLRNB7ZTANCNFSM4MK3T3GQ
.


Add Linux session to the future plan

If add a session about Linux, we will be able to learn more things .

will add @rumesh-athukorala please help on adding the learning plan for linux part as well

closing as @heshand updated


Clean up Sungduks copy of early ClimSim data

We have a copy of the lowres? Climsim Data from early ingestinon on leap-persistent-ro/sungdukyu. Now that we are reingesting stuff, we should delete this (but wait for the REU to be done?).

As a matter of fact, I am still struggling with this recipe, so I will use sungduks data for the catalog for now.


Starting point missing in AMBS

The starting point defined in an HpProblem should be added to the first batch of evaluations during AMBS.
See this line:
https://github.com/deephyper/deephyper/blob/master/deephyper/search/hps/ambs.py#L75

Now able to use multiple starting points in AMBS.


Next and Previous buttons have wrong tooltips

The tooltip for the 'next' and 'previous' buttons shows the title of the current page, instead of the title for the next page or previous pages, respectively.

@jvlasblom I'll look into this, I'll update you shortly.  Thanks for letting me know.

any updates on this issue yet?

The buttons themselves dont have tooltips, the tooltip is only for the current step.  See #79, we will be adding a feature to disable tooltip but probably not to add ones to the buttons


Ethereum-Pool with MPOS

Hello,
i try to install an Ethereum Pool with MPOS, i had some Pools (PPC, WDC, TAG, CSC) already running with MPOS. But with Ethereum i had a Problem. What Kind of Algorithm should i use?
The doc said:
Algorithm
Underlying coin algorithm that you are mining on. Set this to whatever your coin needs, sha256d, scrypt, scryptn or x11.
Default Values:
algorithm    =    'scrypt'
algorithm
•sha256d: Bitcoin and similar ones.
•scrypt: Litecoin and most of the all-coins.
•scryptn: Vertcoin and similar ones.
•x11: Darkcoin and similar ones.
Has anybody an idea?
Thank you very much.
Best regards
Zimbo

It's called Ethhash but isn't supported in MPOS yet. Maybe someone can share a proper config so it could be added to MPOS.

FYI Ehtereum RPC is completely different to any bitcoin based RPC system used in other coins. At least according to: https://github.com/ocminer/ethcore-pool-mpos/issues/3
Please re-consider if you wish to run a pool based on this coin since it won't be supported in MPOS unless someone finds a way to include it to the core code.

Ocminer from suprnova has done that but he is not willing to share his changes with anyone.
Meanwhile you can use Sammy's Pool, it is great.

The usual "I make it work and cash-in since I am the only pool owner" mentality. Thanks for the confirmation though, I wasn't 100% sure about this myself.


Error

Hi How its run?
I get error

This error happens because the field JWTKey in your BlszorServer appsettings.json is empty, you must fill this field with any random string between 15 and 40 characters, please note that a connection string to a database with the proper migration applied is also needed; with Visual Studio you can run ScriptDbContext in order to obtain a .sql that will create the default database structure for you.


Disable Monaco code actions highlighting

Describe the bug
The highlighting for code actions appears in the editor, but the user cannot interact with them.
To Reproduce
Steps to reproduce the behavior:

Go to https://www.freecodecamp.org/learn/javascript-algorithms-and-data-structures/basic-javascript/comment-your-javascript-code
Enter

function anyname(){
  this.property = 'is set';
}

see the three dots below the 'a' in 'anyname'

Expected behavior
Since the user cannot act on this information these dots should not appear.
StackOverflow suggests they can be disabled  in vscode, and hopefully the same is true of Monaco.
Screenshots

Thanks to @lasjorg for investigating this and olgamlop (who I'm not sure has a GitHub account, so I didn't want to ping the wrong person) for reporting it.
Originally discussed on the forum, here

@ojeytonwilliams Not sure if there is an editor option for it, but here is the CSS for removing the three dots.
client\src\templates\Challenges\classic\classic.css
.monaco-editor .squiggly-hint {
  background: none !important;
}

I did look a bit for an option but the API docs for Monaco are pretty dense and hard to comb through.
Just let me know if I should make a PR with the CSS change.

I made a PR just in case. If we want to use it its there.

Thank you for reporting this issue.
This is a standard message notifying you that the issue has become stale, so we are going to close it.
If you think we're wrong in closing this issue, please request for it to be reopened. Thank you and happy coding.


Values not populated from application.properties files

Note that all other settings are working just fine.
I only recently adderd Spring Cloud GCP - it's the only piece that does not work as expected currently.
I am running a particular profile which points locally to a credentials file as follows:
spring.cloud.gcp.project-id=<project-id>
spring.cloud.gcp.credentials.location=file:/path/to/file/credentials.json

If I run the same thing with GOOGLE_APPLICATION_CREDENTIALS=/path/to/file/credentials.json it's working.
What am I missing?
Caused by: java.io.IOException: The Application Default Credentials are not available. They are available if running in Google Compute Engine. Otherwise, the environment variable GOOGLE_APPLICATION_CREDENTIALS must be defined pointing to a file defining the credentials. See https://developers.google.com/accounts/docs/application-default-credentials for more information.
	at com.google.auth.oauth2.DefaultCredentialsProvider.getDefaultCredentials(DefaultCredentialsProvider.java:134) ~[google-auth-library-oauth2-http-0.22.0.jar:na]
	at com.google.auth.oauth2.GoogleCredentials.getApplicationDefault(GoogleCredentials.java:119) ~[google-auth-library-oauth2-http-0.22.0.jar:na]
	at com.google.auth.oauth2.GoogleCredentials.getApplicationDefault(GoogleCredentials.java:91) ~[google-auth-library-oauth2-http-0.22.0.jar:na]
	at com.google.api.gax.core.GoogleCredentialsProvider.getCredentials(GoogleCredentialsProvider.java:67) ~[gax-1.60.0.jar:1.60.0]
	at org.springframework.cloud.gcp.core.DefaultCredentialsProvider.getCredentials(DefaultCredentialsProvider.java:67) ~[spring-cloud-gcp-core-1.2.6.RELEASE.jar:1.2.6.RELEASE]

I am using spring-cloud-gcp-starter (gradle):
// Spring Cloud Google Secret Manager
implementation 'org.springframework.cloud:spring-cloud-gcp-starter:1.2.6.RELEASE'
implementation 'org.springframework.cloud:spring-cloud-gcp-starter-secretmanager:1.2.6.RELEASE'

Update
It appears to be the same for other properties e.g. database name:
spring.cloud.gcp.sql.database-name=<db-name>

Debugging the code confirms that the properties are not getting set:

It looks like you're using the secretmanager module -- those settings have to be put in a bootstrap.properties instead of application.properties.
This is because the secrets are loaded by the "bootstrap phase" of Spring. The bootstrap.properties file is placed in the resources/ directory same as the application.properties file.
Let me know if this fixes the problem for you. I will also look into ways of documenting this better -- were you looking at sample apps or docs, etc.?

It looks like you're using the secretmanager module -- those settings have to be put in a bootstrap.properties instead of application.properties.
This is because the secrets are loaded by the "bootstrap phase" of Spring. The bootstrap.properties file is placed in the resources/ directory same as the application.properties file.
Let me know if this fixes the problem for you. I will also look into ways of documenting this better -- were you looking at sample apps or docs, etc.?

Hi @falk-stefan perhaps you can share the full credentials file and maybe the bootstrap file (if there), so we can get a full picture.

Hi @falk-stefan perhaps you can share the full credentials file and maybe the bootstrap file (if there), so we can get a full picture.

Please do not share the full credentials file. That's supposed to stay a secret ;-)

Please do not share the full credentials file. That's supposed to stay a secret ;-)

Oh..my bad!...I should have said, the applications.properties file and the bootstrap file ;-)

Oh..my bad!...I should have said, the applications.properties file and the bootstrap file ;-)

Sorry for the late response. I was somehow able to make this work but I cannot tell what the problem really was. But it was definitely something on my side so I guess we can close this issue. Thank you!

Sorry for the late response. I was somehow able to make this work but I cannot tell what the problem really was. But it was definitely something on my side so I guess we can close this issue. Thank you!


Ensure backward() idempotence

As per issue 53 -- just makes backward() do a quick first pass over the tree to reset grads, ensuring that backward() is idempotent so that users don't trip over accidentally calling it twice without redefining the values.

Closed per discussion in issue


Add to Microsoft To Do via PowerToys Run

Description of the new feature / enhancement
A command like "/TODO" must trigger a dialogue that allows one to rapidly add tasks to their Microsoft ToDo List
Scenario when this would be used?
This would be useful for 'Quick Capture', basically capturing spontaneous thoughts, tasks and more
Supporting information
No response

/dup #4233


Support for zero-sized types

At some point, MiniVec should really support ZSTs

Proper ZST support will require following:

Specialization - https://github.com/rust-lang/rust/issues/31844
Ability to trait bound on ZST types in generic way - Not clear if it is currently possible with nightly features. Needs investigation


Session migration file conflicting

Session migration file conflicting and there is no way to rename the sessions table without forking it

We will namespace the tables... Update incoming


First edit doesn't repopulate channel

The first time you try to edit something in a session, the channel is not re-populated in the dropdown. You'll see anonymous. You can select it from the list though. Subsequent edits work.

Fixed.


How to import excel data in js grid?

How to import excel data in jsgrid?

You have to read data from the excel file, checkout https://stackoverflow.com/questions/8238407/how-to-parse-excel-file-in-javascript-html5 or any other explanation.
Then it's easy to feed json to the grid.


Module not found error in dll polyfill during run command of "build:dll"

face these errors during script run of "build:dll"
ERROR in dll polyfill
Module not found: Error: Can't resolve 'sockjs-client' in '/home/crucible/Programming/angular2webpack2-starter-master'
@ dll polyfill
ERROR in dll polyfill
Module not found: Error: Can't resolve 'querystring-es3' in '/home/crucible/Programming/angular2webpack2-starter-master'
@ dll polyfill
ERROR in dll polyfill
Module not found: Error: Can't resolve 'strip-ansi' in '/home/crucible/Programming/angular2webpack2-starter-master'
@ dll polyfill

it seems you're missing dependencies, did you run npm install on a fresh clone?

I can't reproduce this error...if you do a fresh clone and install and still see this issue then let me know.

Yeah i did a fresh clone and installed again. Worked. Thanks. Awesome repo.


Don't output inodes created to run a container

There is a group of inodes that get created when running a container
if they do not exist.
containerMounts = map[string]bool{
	"/dev":               true,
	"/etc/hostname":      true,
	"/etc/hosts":         true,
	"/etc/resolv.conf":   true,
	"/proc":              true,
	"/run":               true,
	"/run/.containerenv": true,
	"/run/secrets":       true,
	"/sys":               true,
}

If the destination inode does not exist, libpod/runc will create the inode.
This can cause programs like podman diff to see the image as having changed,
when actually it has not.  This patch ignores changes in these inodes.
Signed-off-by: Daniel J Walsh<EMAIL_ADDRESS>

LGTM

bot, retest this please

@rh-atomic-bot r=giuseppe


Using client.say with emote msg triggers message with null emotes.

Actual behaviour:
If I send a msg in a chat with client.say When i get the msg I just sent the message event userstate data has no emote data.
client.say(currentChannel, "Kappa Kappa Kappa");
``

Expected behaviour:
When sending "Kappa Kappa Kappa" from the normal Twitch client and listening for a message event I get the correct userstate info.

Am I missing something here? Is there a way to add the emote data to the say function? Any help would be awesome.
Thanks,
Curtis

tmi.js version: 1.2.1

Sadly Twitch doesn't send back IRC tags for sent messages, the library would only be able to guess. You will have to parse you're own messages for emotes using the identity's available emote sets and the Twitch API. I could point you in the right direction if you wish

tmi.js will auto-populate the emotesets property with the "https://api.twitch.tv/kraken/chat/emoticon_images" endpoint for the available emote sets sent by Twitch and will keep it relatively up-to-date. client.emotesets is an object with each property being a set ID. Each of these points to an array of emote objects with an ID and a string.
ex:
{
	"0": [
		{ "id": 973, "code": "DAESuppy" },
		{ "id": 15, "code": "JKanStyle" },
		{ "id": 16, "code": "OptimizePrime" },
		{ "id": 17, "code": "StoneLightning" }
	]
}


Create first time setup endpoint

New features

If no users with the authority role exist in the database, the server allows the use of a setup REST endpoint to create the first authority user and register the CLI client. A token, saved to the keys directory, is required to prevent the endpoint from being used by an opportunist
All providers now support a hidden boolean property which hides the provider off the sign-in screen
The password provider can be limited to only authenticate users with a certain role using the allowRoles array property on the provider configuration

Reviewed.


SSL package cannot be installed

I'm using Python 3.8 on Ubuntu-WSL. I'm trying to install package SSL but it is not possible:
SyntaxError: Missing parentheses in call to 'print'. Did you mean print('looking for', f)?
----------------------------------------
ERROR: Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.

This is 'cause SSL package is not updated to work with Python 3.* however in your tutorial, you are working with it, how is it possible?
Could you update your tutorial? or tell us, how did you get to install SSL package with Python 3.*? and what requirements  are needed (OS, etc.) for this packages can be installed?
Thanks in advance.

What command gave you that output? Could you give more context?
As for the rest of your comment, websockets only works on python3 (3.6.1 and up), so I'm guessing you're confusing things.

The SSL package is part of Python's standard library. You don't need to install it. Just import it.


[Dropdown] Doesn't dropdown have a searchDelay property?

I'm trying to set a searchDelay for a dropdown that gets data via AJAX but it's not working. A request is being sent for every keyup event no matter what. The Dropdown component should behave as the Search component in cases like this right?
Please advice.

I found the right property: delay.search
Some sort of consistency would be interesting, nonetheless, thank you for the good work!


Can I export historical data with this lib

Hi
are you intending to be able to export historical data with this library using authentication?
Best
Zeno

Not yet. For that, though, I'd probably recommend a different API service
specializing in historical crypto data.
On Mar 5, 2017 5:38 AM, "Zeno R.R. Davatz"<EMAIL_ADDRESS>wrote:

Hi
are you intending to be able to export historical data with this library
using authentication?
Best
Zeno
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Barabbas0/bitfinex-api-python/issues/1, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AIZ3Y7kAzAdYrY53RUXKixEJtu_Ii95rks5ripCogaJpZM4MTYA7
.

Can you recommend a library that works? I need the historical data of my accounts not of publicly available data.

I could probably make this work. I basically have added functionality as I've needed it. What are you looking for, past trades?

Yes, correct. What one needs for Tax reasons is all currencies and all wallets on a monthly basis. See: https://github.com/zdavatz/rapflag/blob/master/rapflag.rb


Default CPT classmap for resolving

Feature Request:
A filter, where i can set a default classmap for the $PostClass for all get_posts would be awesome.
Furthermore, when acf relationship fields are fetched the classmap is used too.
As it is now, i have for any cpt a custom Class wich extends the timber post class. I have to manually build up the posts for relationship fields etc. all the time, because i cant set a classmap - then all posts are instances of timber post instead of my own.

I do something of a class map in my own project. We have a Class that is basically a wrapper for Timber. Basically we call Lumber::getSingle(); or Lumber::getArchive(); which does a bunch of setup for us (like pulling theme options among other things) and then checks the post type to map the post object to one created which extends Timber\Post.
These child classes do all of the object extension for us. It works very well and our Lumber class is very small. It's basically just syntactic sugar. I have wondered if it made sense for Timber to offer a map for Class -> Post_Type sort of interface.

@corradomatt sounds neat!
Can you make a gist out of the class? or show the code somehow?
I understand that helper functions, but (if you work with acf) how are you handling the relationship fields etc. There you get the default TimberPost. For that a filter would be awesome. Timber already has a classmap functionality.

ahh...didn't realize I could pass a class map instead of a class name as a string @dseidl -- that will help me streamline my class.
There's really not much to my class. We're not using ACF much and haven't used relationship fields or anything more advanced than a standard text field...but I would assume that if there is login built into TimberPost for ACF support, it should still work for us.
The key is that our custom post objects are extending TimberPost. In our case, we have a class called PostContract that is an abstract class. This class extends TimberPost and provides some logic that all of our custom Post Objects should share. Then each custom Post object extends our PostContract like so....
<?php

namespace hji\blvd\abstracts;

use Timber\Post;

abstract class PostContract extends Post
{
    public $schema;

    public function __construct($pid = null)
    {
        parent::__construct($pid);
    }

    // some custom stuff....
}

class Something extends PostContract
{
    public $PostClass = 'our\namespace\Something';

    public function __construct($pid = null)
    {
        parent::__construct($pid);
    }
}

The whole thing works very well. We are able to add custom methods and properties to our data models in this way. The one tricky part is that we sometimes have data we don't want exposed on archive pages...only for single post pages. Since we want to use the same data model for both archive pages and single pages, we have some conditionals to exclude certain properties from the archive page version of the model.

just to recap: the filter i was searching for already exists.
it is named Timber\PostClassMap.
it would be awesome to have all filters documentated. maybe if i have some time i could start writing that down.
this ticket can be closed


User should be redirected to login page when he clicks on 'comment' on a post he accessed through a shared  link

Summary:

a)User should be redirected to login page when he clicks on 'comment' on a post, he accessed through a shared    link and
b) should land on post details page ,after he logs in
Comment button should Not be highlighted when there are no comments for a post which is accessed via shared
link as in figure below:

Steps:
1.Click on 'Share' button of a post  and choose email option to share the post  .
2.As an unlogged in user , click on the shared link for the post.

Post is displayed  and click on comment button

Expected:The user should be redirected to login page (same behavior as in Help Board ). The comment button should not be highlighted when there is no comment

To clarify, this isn't really related to the share button, sharing is just a way a non logged in user can get a specific post's url. A non logged in user can view a post if they have the url.


Could not find the version that satisfies the the requirement tensorflow-directml

Hey, I have been trying to install it on :
Windows System Windows 10 Pro
x64 Based
Python 3.11.5
pip version 23.3
Tensorflow : 2.14.0
Here is the screenshot of the specifications

@ag2s20150909
i have the same problem about it.
here is my information on cmd:
PS C:> pip install tensorflow-directml
ERROR: Could not find a version that satisfies the requirement tensorflow-directml (from versions: none)
ERROR: No matching distribution found for tensorflow-directml
PS C:> python --version
Python 3.10.6
my pc:
Windows 11 22H2 22621.2428
x64 Based
Python 3.10.6
pip version 23.3.1
Tensorflow : 2.14.0

@ag2s20150909 i have the same problem about it.
here is my information on cmd: PS C:> pip install tensorflow-directml ERROR: Could not find a version that satisfies the requirement tensorflow-directml (from versions: none) ERROR: No matching distribution found for tensorflow-directml PS C:> python --version Python 3.10.6
my pc: Windows 11 22H2 22621.2428 x64 Based Python 3.10.6 pip version 23.3.1 Tensorflow : 2.14.0

Same here, tried on 3.10.6 and 3.11.7 versions
i512400f
rx6600
Win10 x64 21H2

AFAIK, tensorflow-directml has only 1.15.x pip releases.
for tf 2.x, try the dml plugin?
On Thu, 4 Jan 2024 at 23:25, RestleSSOtaKU @.***> wrote:

@ag2s20150909 https://github.com/ag2s20150909 i have the same problem
about it.
here is my information on cmd: PS C:> pip install tensorflow-directml
ERROR: Could not find a version that satisfies the requirement
tensorflow-directml (from versions: none) ERROR: No matching distribution
found for tensorflow-directml PS C:> python --version Python 3.10.6
my pc: Windows 11 22H2 22621.2428 x64 Based Python 3.10.6 pip version
23.3.1 Tensorflow : 2.14.0
Same here, tried on 3.10.6 and 3.11.7 versions
i512400f
rx6600
Win10 x64 21H2
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/DirectML/issues/518#issuecomment-1877282167,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACORGAM2TYYWD4H227PI32LYM3CW7AVCNFSM6AAAAAA6I2FDGOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZXGI4DEMJWG4
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
Regards,
JnJn


How do I use the SDK with Bearer access tokens?

I'm not finding anything that allows me to use the SDK using a bearer access token from an OAuth2 authintication.
Surely the package isn't missing this kind of support?

OAuth isn't currently support but will be coming shortly.

It is however possible to bypass the api key and just pass your header in as a query paramerter
KlaviyoAPI.configure do |config|
# don't put api-key here like normal
end

key = "Bearer #{$YOUR_KEY_KEY}"
response = KlaviyoAPI::Accounts.get_accounts({
  header_params: {
    'Authorization': key,
  },
})


Adam maley

Adam Maley
Self Study Questions

also importance to computer
good
good
good
good

MVP Objectives
Home Page

[x] Build the HTML and CSS to create the missing navigation and header.
[x] Link the About navigation item to the about.html page
[x] Color The Boxes Correctly

About Page

[x] Copy and paste your home page navigation and header into the about page
[x] Update the header image with the about page image
[x] Link the Home navigation item back to the index.html page.
[x] Build the rest of the about page layout to match the design

Additional Comments

[x] Follows Lambda Git Flow
[x] Frequent, clear, and descriptive commits
[x] Code is organized and easy to read
[x] Code follows consistent naming conventions, language idioms, and style.
[x] code is DRY (Don't Repeat Yourself)

Stretch

[ ] Build a page of your choosing from the navigation items. Come up with content and images that fit the theme.
[ ] Introduce CSS animations to your site.
[x] Build a contact page and create a form with several inputs of your choosing
[ ] Add responsive breakpoints to your code by using media queries

Score: 3

Header is a bit off on both pages, but not bad.
Careful with the white space.  It would be nice to clean up your code a bit.
Your use of classnames is great.  The naming convention is easy to understand and it is amazing that you resorted to nesting over making a new classname.
Contact Page looks great!  Great job with the inputs!


Fails to start for GCM

I am getting the following error while deploying to a kubernetes cluster running in google cloud.
kubectl create -f deploy/kube-config/google/
I am on the master ... should I try a release?
Error from kubectl logs
flag provided but not defined: -poll_duration
Usage of /heapster:
-allowed_users="": comma-separated list of allowed users
-alsologtostderr=false: log to standard error as well as files
-cache_duration=4m0s: The total duration of the historical data that will be cached by heapster.
-listen_ip="": IP to listen on, defaults to all IPs
-log_backtrace_at=:0: when logging hits line file:N, emit a stack trace
-log_dir="": If non-empty, write log files in this directory
-logtostderr=false: log to standard error instead of files
-max_procs=0: max number of CPUs that can be used simultaneously. Less than 1 for default (number of cores)
-model_frequency=2m25s: Frequency at which model will be updated. Applies only if use_model is true
-model_resolution=1m0s: The resolution of the timeseries stored in the model. Applies only if use_model is true
-port=8082: port to listen to
-sink=[]: external sink(s) that receive data
-sink_frequency=10s: Frequency at which data will be pushed to sinks
-source=[]: source(s) to watch
-stats_resolution=5s: The resolution at which heapster will retain stats.
-stderrthreshold=0: logs at or above this threshold go to stderr
-tls_cert="": file containing TLS certificate
-tls_client_ca="": file containing TLS client CA for client cert validation
-tls_key="": file containing TLS key
-use_model=true: When true, the internal model representation will be used
-v=0: log level for V logs
-vmodule=: comma-separated list of pattern=N settings for file-filtered logging

cc @piosz


Change Nomenclature for ParameterBounds

For constraining the infinite domains over which constraints are set, we should change the terminology ParameterBounds to a more intuitive one. SubDomain might be a better term for such conditions or perhaps DomainConditions. We could also discontinue the use of parameter bounds for hold variables since the same behavior can be achieved via putting the needed conditions directly on the constraints.

Upon further discussion, the plan is to use the term DomainRestrictions.


Actually focus selected messages, make message controls more accessible

Testing Plan:
GIFs or Screenshots:

The focused elements can be activated with enter.

This seems to work reasonably, but the Puppeteer failure seems to be reproducible on this branch locally.

Yes, the puppeteer test fails consistently. Since I struggled debugging the test failure, I opened #15957 to facilitate debugging puppeteer tests.

I merged one of the commits here as 4cd23103db2172aed1594a3e958a1a8eefe3631f, and am deploying the other to chat.zulip.org, after changing some comments.

Pushed back here with the update comment.

This got pulled off again due to the bug described here: https://chat.zulip.org/#narrow/stream/9-issues/topic/double.20send.20bug/near/967405.  I imagine @chrisbobbe has a reproducer and thus can help test the next revision of this.

Yeah—I may be especially affected, because I use a trackpad for very fast scrolling, so I can help test. But here are the steps:

In a narrow with many screenfuls of messages, focus a message that is many screenfuls above the bottom
Scroll down very quickly, all the way to the bottom
It should stay at the bottom; instead, it "jumps" the scroll position back to the focused message.

@chrisbobbe I cannot reproduce this, scrolling fast with a regular mouse. What browser do you use? Can you reproduce it without a trackpad?

Hmm, I'll try with my regular mouse once it's charged (if I forget to do so by tomorrow, please remind me). I'm on Chrome.
Can you confirm whether the focused message is on-screen at the moment your scrolling reaches the bottom? If it is, then you've scrolled too slowly and it won't be observed. 🙂

I start scrolling, the focused message outline vanishes, I stop scrolling the focused message outline reappears. I still cannot reproduce what you are describing. I tried scrolling with the mouse wheel and scrolling by dragging the scroll bar button. With Firefox I even set my scroll speed  (mousewheel.default.delta_multiplier_y) to over 9000, with still everything working as intended.

Hmm. I just tried on Firefox Developer Edition (this time at 012f8bfec on this branch, instead of 6146224a6 when it was on chat.zulip.org), and I still see the bug when I use the trackpad; I'm on macOS.
I just tried it by dragging the scrollbar and by pressing the spacebar to scroll down (these on Firefox), and it didn't happen either way.
I did hook up a regular mouse and use the scroll wheel, and I did see the bug with that.

I stop scrolling the focused message outline reappears.

I'm curious at what point you decided to stop scrolling with the mouse wheel. What makes this bug particularly annoying, at least for me, is that I'll start scrolling without a precise idea of when to stop; I keep scrolling and scrolling, watching for the screen to stop moving, as a signal that I've reached the bottom—and with this bug, that signal doesn't arrive when it should.
So, to be clear: you don't see the bug if you keep scrolling and scrolling for quite a while? You don't see any "bouncing" of the scrollbar?
The bug looks like this; I start scrolling soon after the GIF starts, and keep scrolling until the "bouncing" stops (being scrolled to the bottom coincides with the selected message being onscreen):

@chrisbobbe Thanks for the detailed report, I managed to reproduce it! Fix is pushed. @timabbott please redeploy on CZO on occasion.

Redeployed this on chat.zulip.org for testing.

Cool, and I don’t see the bug currently on CZO. Thanks, both of you!

I noticed a small bug with this deployed on chat.zulip.org: The ctrl+. keyboard shortcut loses focus on the compose box.  I think the fix is likely to be additive -- i.e. making that keyboard shortcut explicitly focus the compose box now that it has an effect that can adjust focus.

Because the source repository for this pull request was deleted, this pull request has been automatically closed by GitHub as an unintended side effect of renaming our default Git branch. If this change is still desired, please open a new pull request, referencing this one in the description.


Bug report: Your title here

Describe the bug

A clear and concise description of what the bug is.

Jslint edition:

Code snippet (if applicable):

/*jslint devel*/
console.log("hello world");

Expected behavior

closing issue as poorly defined


Remove Lora specific checks from warmup and profile

These changes are irrelevant with the new changes in warmup

@hlahkar  please close if not needed anymore


Would love a tool to "tray" any app

Description of the new feature / enhancement
Some applications do not include a "minimize to tray" or "close to tray" solution.
This is case for example of WhatsApp, but also of any webapp installed (or simply opened) that runs in a browser window.
so addons exists on the web, but I would feel it valuable to be included as a PowerToys capability, as this is very aligned with the purpose and similar to "always on top" capability.
I would also feel much more secure getting this kinda utility from the PowerToys package rather than from various sources over the web.
Scenario when this would be used?
Would permit, for a single time or to save for next session, to remove the program icon from the taskbar and transform it to a minimized icon in tray. Ideally it would manage the notifications activity (bubble on tray), and enable to 'capture' the program's 'close' action to not close but minimize to tray. (and provide the close(quit) action from the tray rmb menu).
Supporting information
https://www.maketecheasier.com/send-windows-programs-to-system-tray/

grat idea!

/dup https://github.com/microsoft/PowerToys/issues/47


Fixed various .rst issues with FAQ

Fixes https://github.com/pyviz/holoviews/issues/3470 and various other formatting issues with the FAQ.

Tested usingpandoc FAQ.rst > out.html, but haven't tried with building the full website.

Looks good thanks.


gaps cant find2020-12-20 13:13:51.720 WARN 7 --- [nio-8484-exec-7] c.j.gaps.service.PlexQueryImpl : Can't find ID to match tvdb://6115

2020-12-20 13:13:51.720 WARN 7 --- [nio-8484-exec-7] c.j.gaps.service.PlexQueryImpl : Can't find ID to match tvdb://6115
not sure if its from using the new plex movie agent?

yup I just chnaged the agent to tmdb and it worked no problem.. Basically it tries to use the tvdb search built into the new movie agent if that is selected for some reason


Response stuck on (Loading...) on long messages

cloudflare cpu time limit exceeded

maybe ws is broken

Can the new conversation continue? If it can continue, perhaps it has been truncated due to exceeding the token length limit.

I'm sorry what does ws stands for? Is there a workaround for that? 😅

Can the new conversation continue? If it can continue, perhaps it has been truncated due to exceeding the token length limit.

Yes the conversation continues as normally, in the WebApp the bot shows the full message normally when asked a similar prompt.

I'm sorry what does ws stands for? Is there a workaround for that? 😅

web-socket, If you cannot continue the conversation, it is caused by the disconnection of the web-socket.

Then it may have been truncated because it was too long. You can check the same history on the POE web of the same account to see if it is also truncated.

Then it may have been truncated because it was too long. You can check the same history on the POE web of the same account to see if it is also truncated.

On the web interface the message is shown complete

There are two possibilities, one is that the length exceeds the limit of the telegram message length. The second is that it is so long that the number of refreshes exceeds the limit of telegram. You can try if this complete message can be sent at once in telegram.

It seems that is not length limit as in the example I shown the length is 2741 and the Telegram limit is 4K characters.

also experience this, what i did is to temporarily disable the message streaming, so the bot will reply the complete response just the same on the web page

How did you modified the code to accomplish that?

How did you modified the code to accomplish that?

try to replace this as main.py https://dropden.com/fbf07ff2fc49

try to replace this as main.py https://dropden.com/fbf07ff2fc49 take note that non-streaming text make take 5-10 secs to reply depending on how long the response but it should give you a complete response.

Yup! Worked for me. Thanks! Closing issue i guess 💯

But there is no stream mode after making these modifications. You can try adjusting the refresh rate from 25 to a larger number and see if it works.
https://github.com/TBXark/poe-telegram-bot/blob/7c26a44d1ac87943736b586ffee2dfc9c9cf4cbb/main.py#L107

I made a slight change to make the refresh rate slower and reduce the number of times long texts are refreshed.

I made a slight change to make the refresh rate slower and reduce the number of times long texts are refreshed.

thanks, it works fine now.. :)


Ability to filter generated variants

Hello! I’ve used Tailwindcss on a couple of different projects now, and one thing I’ve wondered each time if there’s a method for filtering out rules that will be generated by a variant context.
For example, I’d love to be able to generate responsive variants for some of the defined colours, but not for all of them. At the moment I don’t think there’s a way to do that easily.
Is there an existing way to do that? Or is it a feature that would be possible to implement?
Thanks!

No, there is no way for that atm, can only toggle variants on a per-plugin level.
And I don't think it would be possible without a bigger rewrite.
Currently, the usual workflow is to use purgecss to remove the unused css instead.

Yeah unfortunately no plans to support any finer level of granularity than we currently do.


Catalog problems

http://a.pomf.se/iloydb.webm
I guess after the update of how the catalog works it kind of fucks up with the 8chanX. If you start loading a catalog things will become unclickable until it loads the whole catalog, which has been taking a while in the past few weeks.
I am using firefox 38.0.1 and the pure version of 8chanX.

Huh, strange. I was trying already a few days ago to find what option from 8chanX could be causing this and I am pretty sure I disabled everything and it was still happening, but it seems like the page indicator was actually the thing doing the problem. Right now loading without it doesn't have that problem. So have that in might, if you are still alive.

Did you try with 8chanX disabled completely?

Yes, it works fine without 8chanX. As I said the problem seems to be the page indicator. Also as a bonus info I found it is caused only when using https.
Had to disable the https anyway, because it seems it was the reason the site was also loading slower for me. Sadly HW hasn't said anything since I asked about the https problems concerned the connections, but this by itself is probably not exactly related to the 8chanX problem. But it is not like I would know.


Search: Concatenate multiple tags via OR

Steps to reproduce

Create two tags: tagA and tagB
Add tagA to a resource
Search for tagA and tagB via a concatenated search pattern Tags:"tagA,tagB"

Expected behavior
I expect the resource to show up in the results because the filter should have an OR behaviour.
Actual behavior
The resource does not show up in the search results. I know that I could use Tags:"tagA" Tags:"tagB", but according to https://github.com/owncloud/ocis/issues/6305, this is a bug and should concatenate via AND in the future - so that doesn't help.

Alright, let me try to explain how the search syntax works. I already talked to @kulmann about this in person and it looks like the current implementation supports everthing that's needed for web in the end.
First of all it has to be noted that there are two different 'modes' you can use with the search term, a simple and a complex one.
The simple syntax is used when you just submit a phrase (which can consist of one or more words). This is mapped to a substring query on the Name field by the search service, essentially transforming it to a Name:*<phrase>* query internally. Please note that this substring query searches for an exact match inside the Name field, so if you have a document named The 100 Best Targets for Astrophotography searching for 100 Best Targets will find the document while 100 Targets will not find it.
As soon as a field is specified in the query however (e.g. Content:foo) the search switches to the complex low-level mode. That means that the search term is passed to the search engine (almost) unchanged, effectively making this a bleve syntax query with the following effects:

All search "items" are considered optional by default
Required search items can be added by prefixing them with +, e.g. +Tags:foo
Exclude items can be specified using a - instead, e.g. -Tags:bar
Any item not prefixed with a field will be considered a match clause (not a substring one) for the composite _all field, which will be just the Name field once https://github.com/owncloud/ocis/pull/6356 has been merged

Some examples:
Content of the space:

foo.txt (tagged: foo)
bar.txt (tagged: bar)
foobar.txt (tagged: foo, bar)
empty file.txt

Simple search: 'file' (substring search on the name)
Result: empty file.txt

Complex search: 'Tags:foo' (all tagged foo)
Result foo.txt, foobar.txt

Complex search: 'Tags:foo Tags:bar' (all tagged foo OR tagged bar)
Result foo.txt, bar.txt, foobar.txt

Complex search: 'Tags:foo +Tags:bar' (all tagged foo that are tagged bar)
Result bar.txt, foobar.txt

Complex search: '+Tags:foo +Tags:bar' (all tagged foo AND tagged bar)
Result foobar.txt

Complex search: 'Tags:foo -Tags:bar' (all tagged foo that are not tagged bar)
Result foo.txt

Complex search: 'Tags:bar file' (all tagged bar OR that match 'file')
Result bar.txt

Complex search 'Tags:bar Name:*file*' (all tagged bar OR that substring-match 'file')
Result bar.txt, empty file.txt

Complex search 'Tags:bar "empty file.txt"' (all tagged bar OR that match 'empty file.txt')
Result bar.txt, empty file.txt

I hope that makes sense and allows for everything you want to do.

@aduffeck Thanks for the clarification, I got most of the cases working!
However, there is still one edge case: Let's say I want to search all files named file.txt AND have either tag foo, OR tag bar. So something like Cond1 && (Cond2 || Cond3). Is this also possible?

@JammingBen no, the boolean algebra kind of query you describe isn't possible with our current search interface.
It's possible on the index at a lower lower but would require us to design a new, more complex way of sending queries to the search service.

Alright, we've adjusted the functionally in Web to work around this.
I think we can close here for now and create a new issue if the scenario becomes relevant in the future. Thanks again for the time to dig into it!


DEV 10348.3 for u14, update the aws cli version to 1.11.91 for fresh installations

https://github.com/Shippable/pm/issues/10348

this is done, closing


bad copyright year

Steps to Reproduce

load https://socalpython.com/
expect "copyright 2018", got 2017

Fixed in 9456b2049daf81e3ccb0f83c43011a39648bc224.
Thanks for reporting!


A parameter cannot be found that matches parameter name 'TargetVMName'

[Enter feedback here]
When attempting to restore to a new VM:
A parameter cannot be found that matches parameter name 'TargetVMName'.

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

ID: e50dfe50-c177-f976-753b-9fa689d20f92
Version Independent ID: 7f787259-8ce5-9ba6-8df7-58ab0b89dd5c
Content: Back up and recover Azure VMs with PowerShell - Azure Backup
Content Source: articles/backup/backup-azure-vms-automation.md
Service: backup
GitHub Login: @jyothisuri
Microsoft Alias: jsuri

@onkeys
Glad to know you have self-resolved the issue.


Wrong url on documentation

Wrong url references on "Likelihood" documentation. (FaceAnnotation and SafeSearchDetection)
Here is correct url : https://googlecloudplatform.github.io/google-cloud-node/#/docs/vision/0.12.0/vision/v1/data_types?method=Likelihood
Wrong url : https://googlecloudplatform.github.io/google-cloud-node/#/docs/vision/0.12.0/Data Types%2Fv%2Fdata_types?method=Likelihood

Duplicate: #2546
Thanks for reporting. We are moving all of our docs off from GitHub pages and onto cloud.google.com. It is unlikely we will resolve this specific problem, as the transfer is happening quite quickly. Sorry for the inconvenience.


Replace flux transfer in HPMR

The MRAD case currently transfers heat flux from Bison to Sockeye using a nearest node transfer.
This transfer is super slow, and could easily be replaced by a UO transfer. The existing files (i) compute heat flux as a UO and then (ii) convert it into an auxvariable, finally to pass (iii) that auxvariable. The conversion and the very slow nodal-based transfer can be replaced.
Furthermore, by transferring the UO directly, we don't need to manually normalize the flux - the value sent to Sockeye is simply used as-is. The current input files are  brittle if one is to change the underlying meshes, since the user would need to be careful to modify the preamble lines at the top of the Bison file.
I also believe there was a minor correction to make in the Sockeye mesh settings - the length of the evaporator is 180, which when divided by 3 equals 60 (not 50). This necessitated a re-gold of the Sockeye standalone test, but which had only very minor differences due to the slightly finer mesh (~1e-5 relative).

Job VTB Documentation on f5a8de3 wanted to post the following:
View the site here
This comment will be updated on new commits.

Thanks for the patch!
@miaoyinb for awareness.
I ll do the rebase to consolidate with the other PR.
With these changes I think we can consider doing CI on the full multiphysics model in the regular suite. I'll re-assess that.

@aprilnovak Thanks for the patch. I was wondering why normalization (polygonization correction) is not needed for the UO transfer approach. Thanks.

I think a normalization is not required because the Sockeye meshes are 1-D. The "area" with which they multiply the incoming heat flux (to get power) is exact.

The Sockeye mesh is 2D-RZ in this case. But you are right that the related calculation in Sockeye is exact.
But the BISON mesh is 3D and the heat pipe is a polygon instead of a perfect circle. That polygon size is also "modified" to preserve volume. The original normalization was meant to correct this difference between the polygonized circles in the 3D mesh and the "idealized" circles in the 2D-RZ mesh.

Thanks, yes I meant 2D-RZ.
Are you saying that the surface area of the cylinders in the BISON mesh does not converge to pi * diameter * H if you were to refine the mesh? (due to some modifications from volume adjustments?)
If that is the case, I'd suggest we add a conservative transfer (to preserve power in BISON and power received in Sockeye) rather than this by-hand modification.

I don't think you need the correction factor. You could add this to the flux transfer in the BISON input files instead, and it would re-normalize the heat flux such that the power in the sub-app is identical once accounting for different cylinder areas.
    from_postprocessors_to_be_preserved = 'hp_heat_integral'
    to_postprocessors_to_be_preserved = 'A_int_master_flux'

That power (flux * polygonized cylinder area) may not be exactly identical to the power in the Griffin model, but it never would be with finite elements, because flux conservation is not guaranteed.

I don't think you need the correction factor. You could add this to the flux transfer in the BISON input files instead, and it would re-normalize the heat flux such that the power in the sub-app is identical once accounting for different cylinder areas.
    from_postprocessors_to_be_preserved = 'hp_heat_integral'
    to_postprocessors_to_be_preserved = 'A_int_master_flux'

That power (flux * polygonized cylinder area) may not be exactly identical to the power in the Griffin model, but it never would be with finite elements, because flux conservation is not guaranteed.

I think this will work, thanks.

I'll test this out locally just to be sure, and then I'll update this PR with those changes. Thanks for the discussion.

I actually do not think I can accomplish what I was proposing. For this to work, I need to have a vector postprocessor which computes 192 heat flux values (one on each heat pipe surface sideset). But the only viable option in MOOSE is the NearestPointIntegralVariablePostprocessor, which does not have the correct side restriction I need for heat flux. If I were to use that vector pp, it'd be averaging a bunch of zero values into the quantities sent to my 192 sub-apps (right?)
@zachmprince can you confirm if my understanding is correct? git blame shows that you wrote most of the MultiAppConservativeTransfer, so hoping you can double-check my reasoning :)

We ll want to add our conclusions on this discussion here after
https://github.com/idaholab/moose/issues/23334

I think we can do this now, with an arbitrary MeshDivision object. Then use the VPP-based conservative transfer option with the MeshDivisionFunctorReductionVPP


fix: remove many of the perf trackers from amplitude, fixes #5338

The Issue

#5338

Amplitude has so many util.TimeTrack() you can't see anything else
How This PR Solves The Issue
Remove most
Manual Testing Instructions
DDEV_VERBOSE=true ddev list
Automated Testing Overview

Related Issue Link(s)
Release/Deployment Notes

I took this for a spin and don't see anything wrong with it. There's still way too much information when DDEV_VERBOSE=true


WEB-961 Update Automate.SDK to match new AutomationRunData shape

Description & motivation
The shape of the Automate API is changing wrt status reporting and "triggers" for automate runs. The SDK should reflect these changes.
Changes:

AutomationRunData struct updated to match new shape
References to AutomationRunData.VersionId and AutomationRunData.ModelId replaced with references in trigger(s)
Use switch statements to handle "union" of types of possible triggers (only 1 currently available)
Update gql operation in ReportRunStatus

Integration tests probably need temp disabling.


5.0 Release Branch

Description
We want to merge  #2995  and #2928 which are both backwards breaking changes
#2995 removes the _log() lpdf functions, which the compiler already errors on when users try to use them
#2928 allows reverse mode matrix functions to return arena_matrix types, which has implications for memory safety we did not previously have
Are there more things we want to merge for a major release? I made a branch here we could merge everything for 5.0 into before we do a final large merge before release. But if it is just these two then I think we should just merge the directly into develop
Current Version:
v4.8.1

I think #3027 also fits in this

I think that's it!


Location Change being fired when cloning location using React.cloneElement

Hi @timdorr,
I ran into an obscure bug, while trying to implement modals similar to React-Router's Pinterest example. I made one tweak to the pattern there - trying to pass the new location to previous children which are being rendered in the background.
const childrenWithUpdatedModalProps = (offsprings) => React.cloneElement(
  offsprings,
  { location: location }
);

When that code is hit, lets say on a push action, the location change seems to be fired twice, corrupting the new location that was being set. I have the list below of the three states of the location. Without cloning, the second location state would be final (which is desired).
I am curious if this is expected behavior, and if yes, I would love to understand why?
Thank you!
Initial Location:
{
  action: "POP",
  hash: "",
  key: "1mqj8o",
  pathname: "/messages",
  query: {
    id: "1"
  },
  search: "?id=1",
  state: undefined
}

Desired Location:
{
  action: "PUSH",
  hash: "",
  key: "ngu2ml",
  pathname: "/home",
  query: {},
  search: "",
  state: {
   lastLoc: "messages"
  }
}

Unfortunate actual Location
{
  action: "PUSH",
  hash: "",
  key: "in0z12",
  pathname: "/home",
  query: {
    id: "1"
  },
  search: "?id=1",
  state: undefined
}

This is a bug tracker, not a support system. For usage questions, please use Stack Overflow or Reactiflux. Thanks!


Remove dependency to go-git

Just use raw git commands instead. go-git introduces a lot of dependencies we don't otherwise use. Our usage of Git functions is also very shallow, nothing that would justify import such a powerful lib.

hello @nscuro, I can take care of this one, would you mind assigning it to me? I'm willing to work on it 🙋🏻♂

@developer-guy Sure, thank you!

Is this necessary to support go mod download in cases where the license should be detected? Or is it needed for anything else?
If not, it might be worth moving license detection code entirely into a separate module in this repo, so that users who want to depend on the library don't have to take a dependency on go-git, or any license detection code.
I'd be happy to help work out how that package should be arranged, I'm working on consuming this in ko in https://github.com/google/ko/pull/573/, and it ballooned our binary size quite a lot 🎈
Opting out of these dependencies could also be toggled behind a build tag, which would reduce binary size. But Go modules would still download the code, and vulnerability scanners etc would be needlessly triggered for unused deps.

Is this necessary to support go mod download in cases where the license should be detected? Or is it needed for anything else?

go-git is currently only used for version detection.

I'm working on consuming this in ko in google/ko#573, and it ballooned our binary size quite a lot 🎈

Yeah... While go-git definitely plays a role in this, the majority of the size comes from go-license-detector, which includes more than 4mb of binary license data. The easiest way to drop this would be to find a lighter substitute for go-license-detector, but I'm open to other tweaks as well.

One idea that might be worth investigating: instead of having WithLicenses(bool), take WithLicenses(Licenser), with a minimal Licenser interface and the default being some no-op licenser. Then, you can provider another Licenser implementation in a separate Go module in the cyclonedx-gomod repo that actually does license detection, go-git, etc.
Users who don't want to detect licenses don't have to import your heavyweight licenser implementation in its separate module, and users who do, can. It also opens up the possibility of other licenser implementations, inside or outside your repos.

@imjasonh Interesting idea for sure! I like the modularity aspect of it, but managing multiple modules could become a little cumbersome. Especially since license detection an essential feature for the app and mod modes of cdx-gomod.
Maybe I can combine this suggestion with one of your earlier ideas:

Opting out of these dependencies could also be toggled behind a build tag

We can still introduce WithLicenses(Licenser), but instead of having Licenser implementations in separate modules, we could have them in a separate package. As long as that package is not importet by you, it won't end up in your binary. Build tags are kinda hard to keep track of, especially in libraries.
That being said, this will still hold true:

Go modules would still download the code

However, this not so much:

and vulnerability scanners etc would be needlessly triggered for unused deps.

If you do the vuln scanning based on what your app actually uses (e.g. eventually go audit, or if you do the scanning based on SBOMs 😎), this is a non-issue.
Sounds acceptable to you? I'll try to do some experiments this weekend.
I think a solution to this should definitely be included in the upcoming release.

Yeah I completely understand the headaches of managing multiple modules inside the same repo. We jump through quite a lot of hoops in go-containerregistry to isolate our main packages and CLI from heavy dependencies like k8s.io/client-go, cred helpers, etc., and there's always more to do. It's a pain.
If the build tag approach bears fruit that'd be great to see. I've been hacking a bit this morning on the multiple module approach just to get an idea for it, and it's pretty gnarly. You'd either have to have:

three modules, library, cli, licenses, where cli -> licenses, or
two modules, library and cli+licenses

In either case it means a pretty drastic overhaul of the package layout, and I don't know exactly which packages you have users for today that might be broken by changes. Of the two, option 2 sounds easier since the library is already new code since the last release, so there shouldn't be any breakage-sensitive users to worry about.
(btw, thank you so much for using internal packages for stuff, that makes everything at least a little easier 🙏 )

However, this not so much:
...
If you do the vuln scanning based on what your app actually uses...

Yeah, ideally scanners would be smart enough, but I've seen some pretty noisy false positives coming from dependabot especially, which is mostly what I'm worried about. 😕

Another thing to consider here is that github.com/go-enry/go-license-detector/v4 depends on go-git as well. This means that even if we remove the direct dependency to go-git, it will still be pulled in by all projects that use our local license detector.
I've also been reflecting on @samj1912's comment (https://github.com/CycloneDX/cyclonedx-gomod/pull/112#issuecomment-1022625812) about limiting external dependencies being a good thing (or even requirement) in some cases. His argument also feeds into https://github.com/anchore/syft/issues/761#issuecomment-1025951603.
In addition, there are still interesting CDX uses cases like pedigree that I'd like to experiment with. For example, when using replace with forks of modules, it may be possible to compare and record the differences between original and fork using git. Having a library like go-git for this instead of having to call the git command for everything would be desirable here.
In short, I'm thinking about removing this from the v1.2.0 milestone, and either including it in a later version or dropping this altogether. However, as always, I'm open to y'all's input.

Closing. We're going to keep using the library over shelling out to git for now. Thanks @developer-guy though for offering help!


[Suggestion] It would be better if the template file reopens and the dropdown list pops up to select a parameter file after clicking "Select" button

OS: All
Build Version: 20211111.17
Repro Steps:

Create a template file and two or more parameter files with similar names, e.g. template1.json, template1.params.json and template1.params.dev.json
Open user settings and reset the "Parameter Files" setting for the ARM extension
F1 -> Azure Resource Manager Tools: Reset Global State -> Yes -> Reload Now
Open the template file -> Get a notification asking to select one of the parameter files.
Close the template file -> Click "Select..." button in the notification.

Check whether the template file reopens and the dropdown list pops up to select a parameter file.

Actual:
Pop up a warning.

Suggestion:
It would be better if the template file reopens and the dropdown list pops up to select a parameter file after clicking "Select" button.

I can't repro this.  I get:


FSharp.Core, Version=<IP_ADDRESS>

I currently have a deployment with the following .NET SDKs available:
dotnet --list-sdks
3.1.404
5.0.403
6.0.400

When running a Fli test with version 1.0.0 and 0.11.0, I get the following error:
Unhandled exception. System.IO.FileLoadException: Could not load file or assembly 'FSharp.Core, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'.
The located assembly's manifest definition does not match the assembly reference. (0x80131040)
File name: 'FSharp.Core, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'

Can the FSharp.Core version be set to greater than or equal to 6 in the paket.dependencies file so .NET 6 can be used?
nuget FSharp.Core >= 6

Thanks

Hey @rastreus,
thank you for opening an issue to that.
This has been fixed in v1.0.1. It's already available at nuget.


Weave integration has false negatives

The scope script checks that the weave script is in the user's path (via command_exists). But this won't be the case in general; and in particular, if weave is being run remotely.
The result is that scope doesn't associate weave IPs with containers, and thereby doesn't detect short-lived connections over the weave network.

We should move the logic for enabling the weave integration into the entrypoint script (or even into the go code)


How to cite REANA?

CERN-IT-Note-2018-004 has a good description, but @hjhsalo pointed out that the REANA software itself should have a DOI.
Alternatives (of which one can do all):

Zenodo GitHub integration - Automatic DOI for each GitHub release of the code base
Journal of Open Source Software - quite light-weight ~1-pager software paper, fast-tracked peer-review using GitHub issues. DOI for "The Software (tm)". Useful as place-holder DOI without counting as "previous publication".
A traditional journal or conference paper - this could describe more on how REANA is used, architected etc. and compare with other. (takes the most time)

Ping :-)


Switch to using "reusable workflows"

Hooray! Works in https://github.com/ploperations/ploperations-unicorn/pull/19

Worked in temporary commit run at https://github.com/ploperations/ploperations-unicorn/actions/runs/5598128124


Buy limit order of a sell order getting executed at market price

Hi,
I am placing the  see order of bankniffty call at market order which is getting placed properly. But when i am trying to place order buy order for this call with limit price(basically placing a stop loss order of the sell order) it is getting executed at the market price instead of the limit price. Please see below image, the order got executed at 306 and my trigger price is 364 and limit 366.

Got the issue, the order type should be StopLossLimit  instead of Limit. Closing the issue


Added ability to disable conversion warning

@ryuoryuo looks good! Can you please add a test for that? Thanks

@ryuoryuo looks good! Can you please add a test for that? Thanks

Done. Not sure if this is the correct way to update the config stub, but couldn't find a better way.

Thanks for the PR @ryuoryuo !

:tada: This PR is included in version 4.16.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Added some more comments to the example

More comments in the example as installFunction is quite a crowded method

Hi,
instead to add comment on example probably IMHO is better to add javaDoc on installFunction method wdyt ?

I hear what you are saying and I think the JavaDoc should exist and I'll add that separately, but for the purposes of the example I think it helps people new to the API.


Fix for stretched images when you capture

Fixes https://github.com/VinceG/vue-web-cam/issues/9 and incorporates iOS fix https://github.com/VinceG/vue-web-cam/pull/10

@jonathankeebler Thank You.


A way to bypass browser cache upon requesting remoteEntry.json

I've been experiencing some caching issues after deploying a new version of a remote: even though exported modules get random file names, the old ones are still being served from the browser cache.
I am suspecting the old remoteEntry.json is still being served with the old package names, and all being served from browser cache.
I am wondering if there could be an option (maybe an options param added to initFederation()) for remoteEntry.json files to be requested with some random param to force it to always load a fresh version...
i.e: http://localhost:4201/remoteEntry.json?_=1709751014494  <-- this is just +(new Date())
That way, the remoteEntry.json file is always fresh, and as long as it's using same version packages, they will be retrieved from cache. Otherwise, it will get the newer packages with different names.

I think, in general, you should configure your web server to provide HTTP headers preventing caching of the remoteEntry.json and the manifest file.


ref in FocusableItem is null

React/React-dom version:18.0.0
React-Menu version:3.5.2
Describe the bug
I've created a menu where I have wrapped a react-router Link inside FocusableItem.
FocusableItem passes down ref which I further pass to my Link menu item, so that it becomes focusable
ref is coming as {current: null}
On focus of  menuitem, the app crashes giving the error "focusRef.current.focus is not a function"
I have also tried a button instead of Link, but it didn't work either.
Expected behavior
I think if ref has a valid value, then component inside FocusableItem will become focusable.
Code snippets/CodeSandbox examples/Screenshots
Here's my code
<FocusableItem>
            {({ ref, closeMenu }) =>
            <Link to={option.href || '/0/myhome'}>text </Link>
            }
</FocusableItem>

Hi @priyankalad , please help me check a few things:

Has the ref been assigned to the <Link/> component?

<FocusableItem>
  {({ ref, closeMenu }) => (
    <Link ref={ref} />
      text
    </Link>
  )}
</FocusableItem>;

Does the <Link/>  component actually forward ref to the DOM element which is supposed to receive focus?

Here is a working example with react-router.

@szhsin  Thank you for checking out the issue :)

Here the Link is not any custom component, its from react-router-dom
I do not understand by "Does the  component actually forward ref to the DOM element" as Link is the component of react-router-dom.

I will create a sandbox example and try to reproduce this issue.

I do not understand by "Does the component actually forward ref to the DOM element" as Link is the component of react-router-dom.

I mean the Link component needs to use React forward ref API to attach ref to the <a/> tag. The Link component from react-router already does it so it should be fine.
Here is another example integrating with the latest react-router version (v6).

It is working now.
The issue was the version of react-router-dom
Earlier it was   "react-router-dom": "^4.1.2",
And now upgraded I've upgraded it  to "^5.0.0"
Thank you so much @szhsin for your help! :)


GeckoDriver "Not in GZip Format"

I get this error every time when it's attempting to use the file geckodriver-v0.19.1-macos.tar.gz:
`initialize': not in gzip format (Zlib::GzipFile::Error)

I have confirmed that the above file is downloaded and placed in the ./webdrivers directory. This is just using the gem "out of the box", no special setup or weird configurations.
The error is happening on this line:
https://github.com/titusfortner/webdrivers/blob/master/lib/webdrivers/common.rb#L141
Update:
I've confirmed that what gets downloaded is 0 bytes. So this may be related to this issue: https://github.com/titusfortner/webdrivers/issues/3. However, when I change my local copy to use that code, the issue still appears because it's occurring on that first line, so it never gets to the subsequent lines that are presumably providing the fix.
This file (geckodriver-v0.19.1-macos.tar.gz) is apparently being downloaded as a zero byte file in the first place.

I can't reproduce this issue. If you ever had webdrivers 2.x installed, make sure you bundle uninstall webdrivers and make sure there is no 2.x. Then delete your ~/.webdrivers directory and it should work. It should override a bad copy of geckodriver assuming the remote server is working and you are connected to the internet.


Relocate to JuliaMath?

It seems this package would be a good fit for the JuliaMath organization, and @jverzani, the top contributor here, is also an active contributor in JuliaMath. Keno admittedly doesn't watch this package all that closely these days, and has expressed willingness to make the move should John deem it a good idea. In that case I can give Keno the necessary JuliaMath permissions to make the move.

Great idea if @keno thinks so.

The deed is done.