Removing module/packages inter-dependencies

Below there is a fast analysis of module/packages inter-dependencies:
root
    __init__ ->                 {}
    __main__ ->                 {app}
    app.py ->                   {three, qt, __init__}
    colorutils.py ->            {three}
    connector.py ->             {qt, three, utils, __init__}
    consts.py ->                {}
    io.py ->                    {utils}
    lexers ->                   {}
    pathutils ->                {}
    procutils ->                {three, qt}
    qt ->                       {__init__}
    reutils ->                  {}
    structs ->                  {}
    three ->                    {}
    utils ->                    {}

    widgets
        dialog.py ->            {widgets.helpers}
        droparea.py ->          {}
        editor.py ->            {three, connector, widgets.helpers, qt, __init__}
        eval_console.py ->      {three, app, utils, qt, widgets.helpers}
        filechooser.py ->       {three, structs, consts, widgets.helpers, helpers}
        helpers.py ->           {connector, qt}
        locationlist ->         {three, widgets.helpers, consts, connector, qt, helpers.intent}
        minibuffer ->           {three, app, connector, qt, widgets.helpers}
        misc ->                 {app, qt, widgets.helpers}
        search ->               {three, consts, widgets.helpers, helpers, reutils, qt, widgets.locationlist}
        splitter ->             {consts, widgets.helpers, qt}
        tabs ->                 {consts, three, qt, connector, widgets.droparea, widgets.helpers, helpers}
        window ->               {connector, three, qt, consts, widgets.helpers, widgets.editor, widgets.tab, widgets.splitter, widgets.droparea}

    helpers
        _lexercolorgroups ->    {three}
        actions ->              {three, connector, qt, __init__}
        bookmarks ->            {connector, helpers.actions}
        buffers ->              {connector, app, widgets.helpers}
        build ->                {connector, procutils, pathutils, qt}
        build_annotations ->    {connector, pathutils, helpers.buffers, helpers.styles}
        build_locationlist ->   {connector, pathutils, app, consts, widgets.locationlist}
        confcache ->            {qt, three, helpers.file_monitor}
        editor_search ->        {connector, widgets.editor, widgets, three, qt, __init__, helpers.buffer}
        file_monitor ->         {three, qt, connector}
        file_search ->          {connector, reutils, app, helpers.file_search_plugins, helpers.intent}
        focus_light ->          {connector, app, colorutils, widgets.helpers}
        folding ->              {connector, widgets.editor, qt}
        intent ->               {connector, structs}
        keys ->                 {three, helpers.actions, pathutils}
        lexer ->                {connector, lexers}
        lexercolor ->           {three, connector, colorutils, lexers, helpers._lexercolourgroups, helpers.styles}
        macros ->               {connector}
        margins ->              {connector}
        minimap ->              {connector, widgets.editor, widgets.window, widgets.helpers, three, qt}
        nav_history ->          {app, connector}
        projects ->             {qt, connector, reutils, __init__, helpers.widgets}
        qt_all ->               {}
        qt_doc ->               {}
        quote_surround ->       {connector}
        remote_control ->       {three, connector, app, qt, __init__, helpers.intent}
        script_reload ->        {three, app, connector, qt, helpers.file_monitor}
        session ->              {three, connector, pathutils, widgets.window, widgets.splitter, widgets.tabs, widgets.editor}
        styles ->               {}

Comments:

System packages or builtin python modules don't count like dependencies in the above analysis

Less dependencies per module/package is ideal

Modules with no dependencies (empty set) are perfectly testable in isolation

If one or more modules from one package share common code think twice whether you want to factor out that code to live in the same package or maybe merging those modules in one file

If one module from packageA is using code from packageB and it's only used by this module, maybe merge it

Relative imports are hell... relative imports to parent directories are a symptom of bad package structuring, ie:
 apps
     libs
         lib1.py
         lib2.py
     app1.py
     app2.py

Does it make sense lib1/lib2 using app1/app2 code?

what i exectly need to compile this on my win7 32bit os
i have portable py 3.8.2

You've got the requirements listed here https://github.com/hydrargyrum/eye/blob/master/requirements.in , after that it should run out of the box.
I've found a problem with this project (which has some really great ideas)though), it was the same problem i've faced couple of years ago when I was trying to create my own sublimetext clone and failed miserably :) ... it's based on QScintilla, and while this widget has some goodies it's pretty limiting as it doesn't support properly multiple/selections, ie: try to undo/redo when making multiple selections and you'll see. Other than that, good luck ;)


Skip/filter certain diffs

After running cd for a couple of weeks, I notice that most changes are uninteresting and belong to the following categories:

[x] Whitespace
[ ] Navigation pane changes
[ ] Stock tracker
[ ] News
[ ] Time/date

We should a.) check if we can filter for these given the current options, and b.) implement them ourselves.

Whitespace should actually be handled by https://github.com/dgtlmoon/changedetection.io/pull/345.


Issue with Google Maps tutorial

So issue is with the Google Maps tutorial (https://www.omkar.cloud/botasaurus/docs/google-maps-scraping-tutorial/). I don't think anything is wrong with the code (aside from the missing comma on line 57), but I do seem to have issues with Google Maps itself. I'm testing this from Hong Kong using a residential HK ip. My Internet speed is 1Gbps and Google has data centers right in HK so it is usually a very low latency experience not to mention good download speeds on anything hosted by Google. I also have access to a US residential proxy and know how to use botasaurus with it if I need to.
So anyway I fired up the bot built from the tutorial. It's scrolling...., scrolling....., and then after a few times it looks like it's stuck as it's not scrolling anymore. I thought that there was an issue with the bot at first so I terminated it and then opened up a chrome browser in incognito mode prepared to inspect some elements using chrome developer tools.
I go to https://www.google.com/maps/search/restaurants+in+delhi/ and I noticed that it loads after scrolling fine initially (just like with the bot), but after scroll loading several times, I hit a chokepoint where google takes forever to load (just like the bot too). It actually took like 5 to 10 minutes to get through the first chokepoint. The second chokepoint is taking even longer. I've been waiting 30 minutes now to load what is after "Smoke Trailer Grill" but the multi colored circle is still spinning.
Are you experiencing such a phenomena on your end for the "restaurants in delhi" query? I can't see the actual end of the page and am unable to see the element(s) that will indicate to bot that it had reached the end of the page when it's scraping.

Tutorial needs to be corrected, if you want to scrape google maps you can use https://github.com/omkarcloud/google-maps-scraper, It handles all these issues.

Thanks! I mainly did the Google Maps scraper to understand botasaurus framework. Happy new year!

Happy New Year


Remove download_dart_sdk.py and use CIPD instead.

The script tries to reproduce CIPD behavior when trying to download artifacts from GCS and has been a bit flaky. Instead of trying to reinvent CIPD, we should just use it instead. Except, those artifacts are not currently published to CIPD. The issue https://github.com/dart-lang/sdk/issues/40690 tracks uploading those artifacts. Once that is fixed, this issue tracks removing the script and updating DEPS to use CIPD to fetch the Dart SDK.

Bumping to P2 since this is affecting local developer velocity especially on Windows.

This is block on the Dart issue above.

Working on elevating the priority of the linked issue.

Updates in https://github.com/dart-lang/sdk/issues/40690#issuecomment-998921864.

@godofredoc Would you be able to help @athomas with the items from https://github.com/dart-lang/sdk/issues/40690#issuecomment-998921864?

Updated permissions to the cipd packages to allow dart service account to upload packages to flutter and flutter_internal namespaces.

Progress is being tracked in the linked issue in dart-lang/sdk.

Progress detailed in https://github.com/dart-lang/sdk/issues/40690#issuecomment-1016199663

Progress is being on the linked issue.

Updates

Refactoring work needed for this has landed in Dart main, this has to be rolled to Dev, Beta and Stable before we start the CIPD uploads. Dev, Beta and Stable will be migrated in “days”.
would like to understand why this is a P1 issue?

Does Flutter need any additional artifacts that we don’t already build in the Dart SDK? We’re only have Dart SDKs for some platforms, but not for all of Flutter’s target platforms.

This was marked P2 because the python script we currently use is prone to spiky flakyness. It can go a long time without flaking, but then flake a lot in a short time span, closing the tree for up to a day or more.
For platforms where Dart doesn't vend an SDK, Flutter can still build one out of the Engine repo, but I think since this is only needed host-side, all the platforms we support for that already have SDKs vended by Dart.

From @a-siva:

The work remaining is the changes he made to main (tip of tree) need to be rolled into the dev/beta and stable branches too so that things will work seamlessly once the change is turned on

This isn't expected to see any movement for about a month.

@sortie can you update the status here, looks like you landed some CLs, is the refactoring work for dealing with dev, beta and stable branches done ?

A change was landed that uploads Dart SDK to CIPD when we're building, you can see these packages here (note that there's only a dev "ref" because we haven't built beta or stable since this landed):
https://chrome-infra-packages.appspot.com/p/flutter/dart-sdk/linux-amd64/+/

With respect to next steps

should we cherry pick the change into the current 2.17 beta branch so that it is available in the beta branch
2.17 stable release would take care of the stable branch
do we need to wait for a good dev 'ref' before updating the Flutter builds with the 'ref'

Moving it to P3 as this is a process change and it is not blocking anything.

We shouldn't pick this to beta, it will flow to beta and stable with the final 2.17 branchout. The work on the Flutter side isn't blocked by this, because it can be done on Engine's main branch until the next branchout to beta.
The refs are created by the dart-sdk builders which the autoroller seems to already wait for. So if it waits for all the relevant SDK builders to succeed, there will be CIPD packages uploaded. Note that the autoroller won't use the CIPD ref, because it is a moving pointer not a pin.

@chinmaygarde @zanderso seems like this has been unblocked, is there anyone working on the changes to the DEPS file?

https://github.com/flutter/engine/pull/33345 isn't quite going to work. CIPD downloads the zip, extracts it, and then deletes the zip. But we need to retain the zip so that we can plumb it to the place where Flutter artifacts live.
@athomas @godofredoc is there any way to get CIPD to retain the zip file?

The unzipping steps of the cipd workflow happens automatically. We'll need to modify the archiving steps to generate a zip file out of the cipd content.

This is done in https://github.com/flutter/engine/pull/33345

https://github.com/flutter/engine/pull/33345 was reverted due to the signed Dart SDK mac binaries lacking the right entitlements for macOS.

@christopherfujino @athomas for signing issue.

The context on the revert:

https://github.com/flutter/engine/pull/33345 changed the way the engine gets the Dart SDK from an ad hoc Python script to DEPS and CIPD
The CIPD binaries sometimes(?) get codesigned without entitlements, which caused OOM errors in the Dart VM on some macOS versions https://github.com/flutter/flutter/issues/104387
I reverted this in https://github.com/flutter/engine/pull/33558 as a temporary workaround

We should re-enable CIPD downloads once we have determined this codesign operation adds the correct entitlements (or alternatively that it is not being codesigned at all, and all codesigning is handled by the Flutter process)

There's no signing issue on the Dart side, so I unassigned myself.

Opened a reland of the CIPD change which will re-fix this in https://github.com/flutter/engine/pull/33569

This change removed the use of the "download_dart_sdk" flag in engine DEPS, so a gclient checkout always tries to download a Dart SDK from CIPD.  This makes it impossible to use a gclient checkout for a tip-of-tree SDK, or a patched SDK, and broke our iOS tip-of-tree builders, and our new monorepo system.
The gclient sync fails with the error that it cannot find the cipd package for that hash.
Could we replace the conditions for these download steps, adding in the "download_dart_sdk" flag?

Patch sent, https://github.com/flutter/engine/pull/33345


Add cstat bkg to timeseries

This allows for the extraction of the selected background interval's counts rather than using the polynomial modeled counts. The choice comes at the point of creation of the plugin. All the identifiers  in the background spectrum are set automatically and the background exposure is properly scaled.

Something strange is happening with the OSX builds

I already fixed it, please rebase and let's see if that solves the build here

Shall we merge this as well?


[MapleStory.gg] Use WebP icon

Description
This pull request is similar to #13545. I changed my extension icon from PNG to WebP. This can save some disk spaces for users.
I also want to try if this can work properly. Which means we can convert all extension icons from PNG to WebP to get a better performance.
This CI fails. The linting part from Ray CLI seems not open-source. So I can't help to update it.
Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Hi! Your extension is currently not passing all the checks. I will mark it as a draft. Please, mark as "Ready for review" as soon as you fix it :)
Cheers!

@mil3na As I mentioned in decription, I can't fix this since the error is from Ray CLI. I just want to let your team notice the WebP stuff so I created this PR.
I am looking forward to the WebP support for icons and metadata screenshots.

To be fair , I don't think this change would be necessary. Icons are already super small and you wouldn't see much change on only one icon. @pernielsentikaer what do you think?

Are you having performance issues on loading icon images?

I noticed that your web store display page has processed the images into WebP before displaying them. If we use WebP directly in the source files, it can also save you the work in this regard.

It will not make the repo smaller tho, because Git stores history. So it would be even worse, having those 2 image formats.
This concerns more the app than the website. I understand when optimising for the website, but for the extension which will be bundled and so on, I don't think that a few hundred KB will make any difference.

Now you have the opportunity to slow down its growth, isn't it good?
Whether to use WebP depends on you. For me, hundreds of KB is not a small number, and it has a lot of space for optimization.
Thank you for your reply. Let's see what other members think about this topic.

I agree that it might be something we'll look into in the future, but for now, we're not ready for it since I'm pretty sure that AppKit on macOS 12 Mojave doesn't support .webp out of the box, which would be the preferred way without the need for custom code. As soon as it's fully supported, it makes sense to allow that.
As stated earlier, it doesn't make the repository smaller if you clone it directly since it's in the history, but shallow cloning does, of course.
Does it make sense, @LitoMore? So, it's more of a "not right now" than a decline

@pernielsentikaer Agree!

Thanks for your understanding, let's circle back to this as soon as possible 🙂


Setup issue - OSError connecting to X-TOUCH MINI

Hi,
I got a problem with Behringer x-touch mini and using the X-Touch-Mini-FS2020.
The X-TOUCH MINI is setup via the Behringer Editor app, and rotary encoders are set to "Relative 2" mode.
When I start MSFS 2020 and then run the x-touch mini-fs2020.exe, the python script gives this error:
OSError: unknown port ‘X-TOUCH MINI 1’
[1228] Failed to execute script main
I also tried changing the Device ID to different values in Behringer Editor, as well as Global CH to other channels, but that didn't solve the issue.
The thing is, the x-touch mini works perfectly fine with my ableton software (with Mackie Control mode). So there is no issue with the detection, drivers etc.
In Device Manager, under "Software devices" I can see "X-TOUCH MINI [0]" and "X-TOUCH MINI [1]" as well as under "Sound, video and game controllers" I can see "X-TOUCH MINI".
I also have an audio interface (EDIROL FA-66) with physical MIDI in/outs, I think that could be the reason why the script can’t connect to my behringer using python’s “mido”.
Edit: If I disable my audio interface under "Sound, video and game controllers", the script can connect without any problems. So it seems the audio interface's midi ports somehow conflict with this script.
Any idea how to fix it?

Ok, I think I know why it doesn't work.
If i import mido and rtmidi in python myself, and then run the code below, I get the list of MIDI devices.
>>> mido.get_output_names() ['FA-66 MIDI OUT 1', 'Microsoft GS Wavetable Synth 0', 'X-TOUCH MINI 2']
However this code in main.py runs this:
outport = mido.open_output('X-TOUCH MINI 1')
So I think I should just change it in main.py from ('X-TOUCH MINI 2') to ('X-TOUCH MINI 1').
But I must say, I don't have any idea how to change this code and build the exe again.. Never did such a thing with python and my coding skills have been diminished very bad. How can I build it? Or can anyone build it and send it to me?

Ok, I think I know why it doesn't work.
If i import mido and rtmidi in python myself, and then run the code below, I get the list of MIDI devices.
>>> mido.get_output_names() ['FA-66 MIDI OUT 1', 'Microsoft GS Wavetable Synth 0', 'X-TOUCH MINI 2']
However this code in main.py runs this:
outport = mido.open_output('X-TOUCH MINI 1')
So I think I should just change it in main.py from ('X-TOUCH MINI 2') to ('X-TOUCH MINI 1').
But I must say, I don't have any idea how to change this code and build the exe again.. Never did such a thing with python and my coding skills have been diminished very bad. How can I build it? Or can anyone build it and send it to me?

This definitely needs to become a feature to select the midi device in the config file.
I just created a hotfix for you which you that uses 2 as the output device and 1 as the input:
https://github.com/maartentamboer/X-Touch-Mini-FS2020/pull/20
https://github.com/maartentamboer/X-Touch-Mini-FS2020/actions/runs/523210791
I've attached the result of that build here
X-Touch-Mini-FS2020 (2).zip
I will add the selection of midi device to the config file pretty soon

This definitely needs to become a feature to select the midi device in the config file.
I just created a hotfix for you which you that uses 2 as the output device and 1 as the input:
https://github.com/maartentamboer/X-Touch-Mini-FS2020/pull/20
https://github.com/maartentamboer/X-Touch-Mini-FS2020/actions/runs/523210791
I've attached the result of that build here
X-Touch-Mini-FS2020 (2).zip
I will add the selection of midi device to the config file pretty soon

Hi Maarten,
Thank you very much, and also the effort for building it again for me :) I can confirm that it works now!
Actually, I also changed it myself and was running it by executing the main.py for the past two days. It would be great to change the midi device, since I think the people buying x-touch not only for sim purposes would've also probably have another midi-equipped audio interfaces as well.
However, occasionally the rotary dials only change the values by 10s (for heading/speed) and by 1000s (for alt), but sometimes it works smooth by changing the values as it should (100s for altitude and 1 for spd/hdg). Somehow when I reset the route and start again, it fixes itself. But I think that's another problem unrelated to this.

Hi Maarten,
Thank you very much, and also the effort for building it again for me :) I can confirm that it works now!
Actually, I also changed it myself and was running it by executing the main.py for the past two days. It would be great to change the midi device, since I think the people buying x-touch not only for sim purposes would've also probably have another midi-equipped audio interfaces as well.
However, occasionally the rotary dials only change the values by 10s (for heading/speed) and by 1000s (for alt), but sometimes it works smooth by changing the values as it should (100s for altitude and 1 for spd/hdg). Somehow when I reset the route and start again, it fixes itself. But I think that's another problem unrelated to this.

https://github.com/maartentamboer/X-Touch-Mini-FS2020/pull/22

Fixed in the latest release https://github.com/maartentamboer/X-Touch-Mini-FS2020/releases/tag/v1.1.0


Item disappearing when dragging

I am trying to use where my sortable list is in a modal. I tried react-bootstrap modal and react-modal-dialog libraries and with both my item disappears when dragging. You mention in your readme about potential of item disappearing. Is there an easy solution if this occurs?

Hey @jeffbonasso, this is a duplicate of https://github.com/clauderic/react-sortable-hoc/issues/87.
Short answer: It's a z-index issue

Awesome. Set a higher z-index on SortableElement and worked like a charm. Thanks!


Update WebExtension packageBrowserLog funx to resolve missing data values in logs

looks like some fields are set to null that don't need to be (e.g., toolversion, toolname, useraleversion). Should pull in info from globals.

Closing this issue with this PR https://github.com/apache/flagon-useralejs/pull/407


Tests contributorsService

deleted unnecesary rule to ignore in PHPstan cause is not happening anymore.

Does this PR have a ticket? Can you add the ticket reference to the PR title?


[3/6] Liability - Merchant Clawback Charges & Refunds (Add LC fee to entity)

Added

Added field to MerchantFeesEntity to allow clawback_subsidy_refund_percentage to pass through (152104370)

Delivers #152104370

👍  Tests not needed will test with GI


Download posts with specific mentions

Hi! I just started using instaloader and I can't find a way to download posts that mention a specific username (either mentioned in the post or comments).
Is it possible to do that with instaloader?

Just to make it clearer. For example if you search for an account without @, on the mobile app, it will show a list of Top posts that mention that account (accounts with many followers show other top accounts, idk why).
I wonder if it's possible to retrieve that list of posts, which mention that username either on the caption or comments

This is not yet immediately possible with Instaloader's current features, but I could imagine that with some research and development effort it would be possible to execute this query with Instaloader and further process the result in Instaloader structures.


Failing to upload via Arduino IDE in Windows 10

Description
I followed the instructions here:
https://coral.ai/docs/dev-board-micro/arduino/#2-install-the-coral-package
Installed the additional boards and tried uploading the BlinkLed Basics Example. And I keep getting this No module named 'progress'.
I tried reinstalling via pip the progress package and it still isn't working.
I tried in a second computer (Windows 10) and I keep getting the same error.
Windows 10 isn't mentioned anywhere in the documentation, So my question is:
is there any way of uploading code using the Arduino IDE in Windows 10? Or am I forced to switch into a Linux partition?
Click to expand!
Issue Type
Build/Install
Operating System
Windows 10
Coral Device
Dev Board
Other Devices
No response
Programming Language
Other
Relevant Log Output
[5620] Failed to execute script 'flashtool' due to unhandled exception!
Traceback (most recent call last):
  File "flashtool.py", line 16, in <module>
ModuleNotFoundError: No module named 'progress'
Failed uploading: uploading error: exit status 1

Hi @Hanqaqa we don't have any support ardunio IDE on windows. However please try set up the environment and build it locally on GIT bash using the command at: https://github.com/google-coral/coralmicro/blob/main/.github/workflows/ci.yml#L156.. Thanks!!

Thanks hjonnala.
Unfortunately in Windows 10 I run into errors after I have cloned coralmicro and all submodules with the git bash. When I bash build with the following arguments as you said
bash build.sh -n -a -g -s
I get the following errors in the git console
$ bash build.sh -n -a -g -s
+ cmake -B /c/Users/MarcoVM/Desktop/coralmicro/build -G Ninja -S /c/Users/MarcoVM/Desktop/coralmicro
-- Fetching https://developer.arm.com/-/media/Files/downloads/gnu-rm/9-2020q2/gcc-arm-none-eabi-9-2020-q2-update-win32.zip
-- Toolchain prefix: C:/Users/MarcoVM/Desktop/coralmicro/third_party/toolchain-win/gcc-arm-none-eabi-9-2020-q2-update/lib/gcc/arm-none-eabi
-- Toolchain prefix: C:/Users/MarcoVM/Desktop/coralmicro/third_party/toolchain-win/gcc-arm-none-eabi-9-2020-q2-update/lib/gcc/arm-none-eabi
-- The C compiler identification is GNU 9.3.1
-- The CXX compiler identification is GNU 9.3.1
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: C:/Users/MarcoVM/Desktop/coralmicro/third_party/toolchain-win/gcc-arm-none-eabi-9-2020-q2-update/bin/arm-none-eabi-gcc.exe - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: C:/Users/MarcoVM/Desktop/coralmicro/third_party/toolchain-win/gcc-arm-none-eabi-9-2020-q2-update/bin/arm-none-eabi-g++.exe - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- The ASM compiler identification is GNU
-- Found assembler: C:/Users/MarcoVM/Desktop/coralmicro/third_party/toolchain-win/gcc-arm-none-eabi-9-2020-q2-update/bin/arm-none-eabi-gcc.exe
-- Configuring done (26.0s)
CMake Error at cmake/toolchain-arm-none-eabi-gcc.cmake:230 (add_library):
  Cannot find source file:

    C:/Users/MarcoVM/Desktop/coralmicro/third_party/nxp/rt1176-sdk/components/osa/fsl_os_abstraction_free_rtos.c

  Tried extensions .c .C .c++ .cc .cpp .cxx .cu .mpp .m .M .mm .ixx .cppm .h
  .hh .h++ .hm .hpp .hxx .in .txx .f .F .for .f77 .f90 .f95 .f03 .hip .ispc
Call Stack (most recent call first):
  libs/nxp/rt1176-sdk/CMakeLists.txt:231 (add_library_m7)

Then, the same CMake errors keeps getting repeated for 30 or 40 times with different toolchains
I have tried in two Windows 10 computers, making sure I have installed chocolatey in both of them, as well as cmake and ninja and installed the Python requirements.txt. But I still get the same errors
After getting tired I decided to just boot up my external SSD with a Ubuntu, build it from source and everything worked fine.
If you see anything I might have missed in Windows 10 please tell me. If not this issue can be closed

I am able to falsh, blibk_led example, with Arduino IDE on windows 11 with some changes ( Haven't tested for other examples though). Please try the below steps and let me know if it works for you as well. Thanks!!

FInd the flashtool.exe and replace with this flashtool. On my system it is located at: C:\Users\heman\AppData\Local\Arduino15\packages\coral\tools\flashtool\1.0.0
git clone --recurse-submodules -j8 https://github.com/google-coral/coralmicro
copy coralmicro/third_party to root_dir. On my system root_dir is C:\Users\heman\AppData\Local\Temp

@hjonnala Thank you so much.
Copying your flashtool.exe (Windows 10 alerted me it was a virus, I had to manually allow that file to exist) and copying the third party folder worked great in my Windows 10.
I have flashed a few examples in the Arduino 2 IDE, incluiding some Tensorflow examples and they have all worked perfectly.
My question is, would it be hard to change the source files here so it was easier to install on Arduino IDE 2 on Windows 10?
Now it has just been copying and pasting two files. Would it be too hard to make it Windows 10 compatible? I have already seen some other people complaining they couldn't get the Coral Micro running on Windows.

@Hanqaqa Regarding your issue building via build.sh -- It's likely that Git didn't check out all the files due to path length issues. In our CI runners, we enable longpaths support: https://github.com/google-coral/coralmicro/blob/main/.github/workflows/ci.yml#L136
Alternately, you could clone the repository to something with a shorter prefix (e.g. C:\coralmicro).
I've reproduced the issue with flashtool not working in Arduino -- seems that PyInstaller didn't bring in all the dependencies -- will take a look into a patch and new release to fix this, but hopefully for the time being @hjonnala's solution has you unblocked.

Closing as the issue got fixed with this new release: https://github.com/google-coral/coralmicro/releases/tag/arduino-1.0.1


KAFKA-17422: Adding copySegmentLatch countdown after expiration task is over

The given test took 5 seconds as the logic was waiting completely for 5 seconds for the expiration task to be completed.
Fix

After the expiration task complete , manually called copySegmentLatch.countDown().

@chia7712 @showuon  Stream test case are failing , I don't think they are related with the above change ?

Failed tests are unrelated.

Thanks for the contribution!


[BUG] terabox

Terabox
Bug Description

Steps to Reproduce

Expected Behavior

Actual Behavior

Screenshots

Device Information

Device Model:
Android Version:
App Version:

Additional Context

Error Log

Paste your logs here

Checklist

[ ] I have checked for similar issues
[ ] I am using the latest version of the app
[ ] I can consistently reproduce this issue
[ ] I have included all the required information above

Related Permissions

[ ] Internet
[ ] Access Network State
[ ] Install Packages
[ ] Delete Packages
[ ] Query Packages
[ ] Notifications
[ ] Boot Completed

Paste your logs here


Mk

PR Fixes:

1 bug: Reply option in comments feature is not working

Resolves #750
Checklist before requesting a review

[ ] I have performed a self-review of my code
[ ] I assure there is no similar/duplicate pull request regarding same issue

The bug is fixed through Mk #751. Attached the video recording for reference.
https://github.com/code100x/cms/assets/99182229/45bca94f-be36-4beb-b8af-2ffa8fff9700

maybe solved in PR #602

closing, #602 fixes it


Dont request policies when the token dosent have access to them

When a non-root token is used for the salt-master, the token creation for minions is going to fail because the master is trying to create a token for policies that doesn't exists (or that he doesn't have).
This fix the issue, by requesting only policies of the current token if the option is set.

Hey, thanks for this, and sorry that I missed it for so long!
One potential way around this if you are using a non-root token (good!) would be to investigate the roles feature. This can be configured to let the salt master token create tokens that has certain other policies than it has itself. It does not support giving policy patterns, though, so it might give some administrative overhead in managing the role.
Over to the patch itself, it looks quite nice! I'll leave some comments inline that I'd like to discuss before merging.


Add support for genome build 38

Chose which build to use in cli, or in config (defaults to 37)
remove all hard coding related to build 37 (par regions)
Add 'profiling'-variants based on what genome build is specified
Always build variant id excluding possible 'chr' contig notation

Hi,
@henrikstranneheim and I talked about making loqusdb compatible with the latest genome build. This PR tries to fix this, removing all hard coding to build 37. The profiling-variants now exists as resources in this repo, both for the 37, and 38 build. I also changed so that in case a vcf-file uses 'chr' in the contig names, this prefix is ignored when creating the variant IDs. Let me know if you have any comments or suggestions @moonso

Hi @adrosenbaum , thanks for this!
 To me they are mostly confusing. Why not use '37' and '38' everywhere. We know that will not change in many years. I give some suggestions in the code and then you can reason about your approach?

Hi, I can change this. I set the genome build as a global, to avoid passing this as an argument to all functions, and embedded functions, that needs this information. Agree that it adds some complexity to the code.

Tested on stage, works fine.


OCPBUGS-11967: Set OVN-k gateway next hop to an internal IP address

Closes OCPBUGS-11967

/jira refresh

We decided to implement this conditionally on MicroShift startup when the default gateway is not set.
Closing this PR now
/close


256dark supports less extensions

dircolors.256dark is not in sync with the dircolors.ansi* (has less file-types).
Just wondering if it's intentional?
PS - I use the ansi-light colors though.

no, its not intentional ... someone just forgot it I assume. PR pls


Do not read web.config

HTTPClient must not read the old web.config any more.

Solved by version 7.3.0


[storage-preview] Bump azure_mgmt_storage to 2022_09_01 and clear out-of-date commands

Description

Bump azure_mgmt_storage to 2022_09_01
Synchronize the behavior of az storage account create/update with the main repo.
Remove the command az storage account network-rule add/list/remove, az storage account local-user create/update/delete/list/show/list-keys/regenerate-password, az storage file upload/upload-batch, az storage fs service-properties show/update, and az storage fs list-deleted-path/undelete-path.

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

storage-preview

Don't forget to upgrade the version in setup.py and history.rst to trigger auto release


gparted: 0.33.0 -> 1.0.0

Motivation for this change
Things done

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

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

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

The only thing causing the major version change is the transition from gtk2 to gtk3.

Built and tested locally :+1:

@adisbladis thanks!


Create a Smart equivalent to React Flow's Floating Edge

There is a case that two adjacent nodes are not showing a straight edge. Instead it just move around and connect behind the node.

Hey @AnhHuy02 , could you expand more on the problem you're experiencing? From the screenshot you shared it looks like everything is correct: for the edge at the top of "Strategist" node to connect to the bottom of "generating" node it needs to take this path to avoid intersections

Yes.
But can you add a Floating Edge from React Flow documentation with Smart Edge feature?  It will be cool to see the diagram.
https://reactflow.dev/examples/floating-edges

I see, so you'd like to see an "smart" equivalent of of React Flow's floating edge. I'm not sure if it would be possible, but I'll add to the backlog and leave this issue open

any luck on this @tisoap @AnhHuy02

@vaibhav-systango Didn't started working on this yet

it'll be a great help if we can do this, can you guide me the direction incase you're keeping occupied I can try some approach for this. It'll be the coolest to have smart floating edges feature @tisoap @AnhHuy02

@vaibhav-systango I've just launched version 2.0.0 of this library that exposes a getSmartEdge function instead of factories, check the README.
Looking at how the Floating Edges example was created, I think it's just a case of feeding the output of the example's getEdgeParams function into the getSmartEdge input, but I haven't tested it.

This doesn't seem to help @tisoap

@vaibhav-systango I've just launched version 2.0.0 of this library that exposes a getSmartEdge function instead of factories, check the README.
Looking at how the Floating Edges example was created, I think it's just a case of feeding the output of the example's getEdgeParams function into the getSmartEdge input, but I haven't tested it.

Feeding the output of getEdgeParams into getSmartEdge worked perfect, thanks for the note!
Attached is my working solution below:
import React, { useCallback } from "react";
import { useNodes, BezierEdge, useStore } from "reactflow";
import { getSmartEdge } from "@tisoap/react-flow-smart-edge";
import { getEdgeParams } from "./utils";

const myOptions = {
    // your configuration goes here
    nodePadding: 20,
    gridRatio: 15,
};

export default function SmartEdgeWithButtonLabel(props: any) {
    const { style, markerStart, markerEnd, source, target } = props;

    const sourceNode = useStore(
        useCallback((store) => store.nodeInternals.get(source), [source])
    );
    const targetNode = useStore(
        useCallback((store) => store.nodeInternals.get(target), [target])
    );

    const nodes = useNodes();

    const { sx, sy, tx, ty, sourcePos, targetPos } = getEdgeParams(
        sourceNode,
        targetNode
    );

    const getSmartEdgeResponse = getSmartEdge({
        sourcePosition: sourcePos,
        targetPosition: targetPos,
        sourceX: sx,
        sourceY: sy,
        targetX: tx,
        targetY: ty,
        nodes,
        // Pass down options in the getSmartEdge object
        options: myOptions,
    });

    // If the value returned is null, it means "getSmartEdge" was unable to find
    // a valid path, and you should do something else instead
    if (getSmartEdgeResponse === null) {
        return <BezierEdge {...props} />;
    }

    const { svgPathString } = getSmartEdgeResponse;

    return (
        <>
            <path
                style={style}
                className="react-flow__edge-path"
                d={svgPathString}
                markerEnd={markerEnd}
                markerStart={markerStart}
            />
        </>
    );
}


Splink4 examples notebooks

Update all the examples notebook to Splink 4 syntax.
Add a Colab link at the top of each one
At the moment a pip install splink doesn't install enough in Colab because the dbapi imports pyspark, so for the moment you need to pip install pyspark in colab as well

Gonna go ahead and merge this without a review - it's just updating the examples to reflect splink4 syntax, and they'll need further changes.  It doesn't make any changes to the actual core library


Change filenames for IndexMetadata and Manifest

Description
Change file names of Index Metadata and Manifest files to include the codec version as well.
File structure:
base folder/
    |
    |--> index/
    |     | --> index_UUID/
    |              | --> metadata__<inverted_index_metadata_version>__<inverted__timestamp>__<codec_version>.dat
    |              | --> metadata__<inverted_index_metadata_version>__<inverted__timestamp>__<codec_version>.dat
    |
    |--> global-metadata/
    |       | --> metadata__<inverted_metadata_version>__<inverted__timestamp>__<codec_version>.dat
    |
    |
    |--> manifest/
    |       | --> manifest__<inverted_term>__<inverted_version>__C/P__<inverted__timestamp>__<codec_version>
    |       | --> manifest__<inverted_term>__<inverted_version>__C/P__<inverted__timestamp>__<codec_version>

Related Issues
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff
[ ] Commit changes are listed out in CHANGELOG.md file (See: Changelog)
[ ] Public documentation issue/PR created

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

LGTM!

Gradle check is failing due to
1.org.opensearch.action.admin.cluster.node.tasks.ResourceAwareTasksTests.testTaskResourceTrackingDuringTaskCancellation - https://github.com/opensearch-project/OpenSearch/issues/5329
2. org.opensearch.action.search.SearchRequestTests.testEqualsAndHashcode was fixed here https://github.com/opensearch-project/OpenSearch/issues/10643


Exposing NHibernate configuration for database autogeneration

As instructed by Chris, I'm opening this issue based on this question I asked on SO:
http://stackoverflow.com/questions/21023951/how-to-configure-masstransit-runtimeservices-to-autogenerate-the-subscriptions-d
We would like to configure NHibernate to generate the database for the RuntimeService automatically.
Since it is possible to set this up in a pure NHibernate application, it should be possible to expose it through the NHibernate integration layer when setting up the SagaRepository.

It's unlikely this will be addressed.


Feature: support max-len config for better readability

Hey, me again :) Thanks for fixing #21, now this works great on our entire code base without breaking the code. However, ...
Here's an excerpt from one of the file's transformations:
-import React, {
-  Component,
-  PropTypes
-} from 'react';
-
-import {
-  View,
-  ScrollView,
-  StyleSheet,
-  LayoutAnimation,
-  Animated,
-  TouchableOpacity
-} from 'react-native';
+import React, { Component, PropTypes } from 'react';
+import { Animated, LayoutAnimation, ScrollView, StyleSheet, TouchableOpacity, View } from 'react-native';

I agree with the re-formatting of the react import being readable one way or another (and yes, honestly I don't care as long as it's consistent).
On the other hand, I don't think the readability of the react-native import improved much, quite the opposite.
Would it be possible to support a maximum length (idk, 80, 120, something like that) configuration option for, if the resulting import line would exceed that, the transformer would choose the style that is the original one in the above mentioned example ?
See also: eslint's max-len rule.

This is something I really want to add in, (its actually keeping whats keeping me from using it in my larger projects).
unfortunately recast (on which this is built on top), doesn't fully support max-len yet.
I've recently seen Prettier which maybe I could integrate with.
For the time being I've just be running eslint --fix, its not ideal, but its slightly better.


✨ IMPROVE: Spacing and width defaults for better accessibility

This PR gives the main content a maximum width so that there aren't too many characters per line. Web Accessibility Standards suggest using a width so that only ~80 characters are displayed per line. This width is slightly larger than that but hopefully not in a problematic way.
If you want to play around with it, try the RTD demo and resize the width to wide/narrow. You should see two margins pop up at some wax width.
related to #165
A few references

Best practices for line width and accessibility and this criterion page from W3C as well
Visual design guide from accessibility.digital.gov
wikipedia page discussing their intent to cap the line width
the Tufte CSS page

a max-width of 1400px for all content, and then sets 55% for the main text and 45% for the sidebar text.
Main content: 55% of full width. Roughly 85 characters per line. Font size 1.4rem
Margin content: 50% of width of the main content. Roughly 60 characters per line. Font size 1.1rem

webtypography recommends about 66 characters per line

Note that the max width in this PR is a bit wider than most of the recommendations above. This is partially arbitrary, but partially because of a similar argument to the Wikipedia page above, I could imagine that Jupyter Books are often more information-dense with scannable material, and this might help people scan more easily.
Demos
New behavior:

Old behavior:

Eurgh, I'm very much against this; you buy a nice wide screen, and then a site decides that you are not allowed to use it 🤷
Literally those same web-standards you link have full page text!
Also wikipedia doesn't limit the width, which is probably the biggest purveyor of web articles I can think of.
If you really want it, it should be configurable, and off by default

Also wikipedia doesn't limit the width, which is probably the biggest purveyor of web articles I can think of

Wikimedia is doing a redesign right now of their interface, that fixing this is one of their goals: https://diff.wikimedia.org/2020/09/23/wikipedia-is-getting-a-new-look-for-the-first-time-in-10-years-heres-why/

Our second change introduces a maximum line width to our content on pages where reading is the focus, such as article pages and discussion pages. Research has shown that limiting the width can lead to better retention of the content itself, as well as a decrease in eye strain.

Good for them, but I still want ours to be configurable 😉

Thanks @pradyunsg for those extra references, I'll add to the top PR to aid discussion as well.
I found the wikipedia discussion you noted to be a reasonable argument.
I'm +1 on having the max-width be configurable, though I think we should discuss and decide on the default. Our default themes should make good out-of-the-box choices for people based on best practices, so I want to make sure that we have a good rationale for whatever direction we take. This PR was mostly about having something to look at and get feedback. I'm curious what others think.

I'm +1 on having the max-width be configurable, though I think we should discuss and decide on the default. Our default themes should make good out-of-the-box choices for people based on best practices

I agree with that point of view.
A sensible default is super important and configurability is always welcome 🤗  .

First off, I’ll say: I’m completely fine with whether content-width is configurable in SBT or not. I don’t really feel that this theme needs to account for my preferences/biases. :)
TLDR: More configuration options != more good. IMO the only users who would want to change content width, would be the ones who wanna change it to something bad.
I’ve grown to strongly dislike providing configurable content-width to documentation authors in documentation themes. If the theme has picked a reasonable choice, most users won’t want to change it, either because they don’t care or because they understand the importance of content width being limited sanely by default was a factor in picking the theme.
To me, the authors most likely to use this knob would be doing things that just a plain degradation of reading experience, for folks who aren’t like the author or on a completely different setup/workflow. If this knob is being used, it’s not being used to slightly tweak the width; it’s to drastically tweak the width. An example of this, is what virtualenv.pypa.io is doing — which is a realllly bad reading experience in favour of “use all the pixels”.
Conversations with the author, around these design choices are exceedingly difficult to navigate usually since most authors who are doing this need to be convinced that what they’re doing is hurting their documentation readers, and chances are that isn’t particularly a priority if they’re making such a broad change without looking up the TON of literature (and literally every website design that was done with a team of designers) that very clearly communicate “don’t do that”. And that’s assuming the author hasn’t done it because “I like it”. Because now you need to approach it from a different angle. :)
All this is to say, this knob when used, will likely be used to do more harm than good to the reader’s experience. The damage is, of course, soft-of limited to just that documentation set. I personally don’t think that’s acceptable even then, which is why I don’t like providing these knobs.

FWIW, An open question would be how should margin content work with this change?

I really like these passionate replies :wink:
You have some really good points about why more configurabilty is sometimes detrimental.
I am pretty sure I could find some examples or biases on the other side (not providing enough config options being detrimental).
Most likely, the key is to find the subtle point where you provide enough configurability to let people build on top of your stuff without the risk of damaging themselves. When that subtle point is not easy to achieve I am probably biased towards providing config options even when it is risky... but in the end, it is my bias 😉 .

@pradyunsg a couple thoughts below!
re: configurability, one fairly easy way around this might be to document what CSS rule to apply to change this value. That's a little bit harder than a config option, and also doesn't commit ourselves to more testing etc infra.
re: the margin, I think one benefit to provide a max width to the main content is that it could provide more room for margin content. Ours is fairly small right now (I believe about 25% of main content), and if we could get this to 33% without crowding things too much, I think that'd be a more useful use of that space (and would make "full width" content pop out more effectively). This is closer to what the tufte theme does, which is a major source of inspiration for this theme's structure.

closing this as we can follow up in later discussion and iteration, we've also got some conflicts so it'll probably be easier to start from scratch


[No QA] Fix broken onyx logger. Update react-native-onyx.

Details

Adding some logs in https://github.com/Expensify/react-native-onyx/pull/105 to detect if/when we are calling Onyx.set() directly after Onyx.merge() and this PR is holding on that change.
Noticed that the Onyx logger broke after being moved here.

Fixed Issues
N/A
Tests
QA Steps
No QA
Tested On

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

🚀 Deployed to production by @roryabraham in version: 1.1.11-1 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


[Feat][Java] Fill two incompatible gaps between C++ and Java

Proposed changes

[x] Use inherit to implement EdgesCollection
[x] Add internal id column to vertex payload file

Types of changes
What types of changes does your code introduce to GraphAr?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have read the CONTRIBUTING doc
[x] I have signed the CLA
[x] Lint and unit tests pass locally with my changes
[x] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Further comments
Related issue: #272

@Thespica Great Work!


Problem on reproducing the effect of coloring on the web with this repo

I used cgi-bin/cgi_exe.py and feed my example image to it. While I expect a similar performance, I got totally unrecognizable results. Here is my example image, the corresponding result and the ref image respectively. I set the alpha channel of ref image to be all 1. Actually I don't quite understand how to generate a ref image, so I assume that it should be the same image because it has to be named with the exact same name with input image in the code.
So can you provide your ref image or simply describe how to produce one?

I have the same problem, any comments would be helpful! 🙏


Little refactor

shorthand
arrow function

Hi Gabriel. Thanks for making this change!
I think I didn't set up Travis correctly for Pull Requests, which is why the build failed (your code looks fine). You can send me another one if you like.

I am making changes to the code on my computer. I'll leave it to send another day.


MSID: 90333 Version: 2 DOI: 2<IP_ADDRESS>1185

[PLACE MANUSCRIPT AND EDITOR DETAILS HERE WHEN AVAILABLE] See step 3
[PLACE PDF URL HERE WHEN AVAILABLE] See step 7
Step 1. Inform bioRxiv
Who can help: @QueenKraken, @nlisgo, @scottaubrey

[x] Manuscript is in data hub index (https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=90333)

Step 2. Create preview of manuscript
Who can help: @fred-atherden, @nlisgo, @scottaubrey

[ ] Manuscript is available for preview (https://prod--epp.elifesciences.org/preview/90333/v2)

[ ] Trigger import for staging: https://prod--epp.elifesciences.org/import
[ ] Sync data folder with s3://prod-elife-epp-data/data
[ ] Trigger import for staging: https://staging--epp.elifesciences.org/import
[ ] Sync data folder with s3://staging-elife-epp-data/data
[ ] Pull request created to enhanced-preprint-data
[ ] Post link to pull request below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-manuscript
Step 3: Awaiting public reviews and QC
Who can help: Production team

[ ] QC passed or QC not needed

Who can help: Editorial team

[x] Public reviews available on sciety (https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>1185)
[ ] Manuscript and editor details at the top of this issue (Supplied by Editorial team)
[x] Reviews returned by EPP - https://prod--epp.elifesciences.org/api/reviewed-preprints/90333/v2/reviews

Example
"msas": "Genetics and Genomics", "Neuroscience"
"msid": "84628"
"version": "1"
"preprintDoi": "10.1101/2<IP_ADDRESS>4241"
"articleType": "Reviewed Preprint"
"status": "Published from the original preprint after peer review and assessment by eLife."

"Reviewed Preprint posted": "2023-01-02"
"Sent for peer review": "2022-10-28"
"Posted to bioRxiv": "2022-11-21" (link: "Go to bioRxiv": "https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>4241v1")

Editors:
Reviewing Editor
Michael B Eisen
University of California, Berkeley, United States
Senior Editor
Michael B Eisen
University of California, Berkeley, United States

Step 4: Modify manuscripts.json (no PDF)

[ ] Reviewed preprint is published to EPP (https://elifesciences.org/reviewed-preprints/90333)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[ ] Request that a doi be registered

[ ] Post link to #enhanced-preprint comment below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]

Instructions to modify manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/publish-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/334/files
Once the pull request is merged in it should be available a few minutes later.

Request that a doi
Post the following in #enhanced-preprint:
@Fred can you register a doi for https://elifesciences.org/reviewed-preprints/90333

Step 5: Awaiting search reindex

[ ] Reviewed preprint is avaliable on journal homepage (https://elifesciences.org)

The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.

Additional info
If needed, the jenkins pipeline to reindex search can be triggered sooner.
https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 6: Published! Request PDF generation

[ ] Post the link to the PDF url at the top of the issue

[ ] Post link to #enhanced-preprint comment below

#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]
Post the following to the #enhanced-preprint on slack:
@Ryan Dix-Peek please can you generate a PDF for https://elifesciences.org/reviewed-preprints/90333

Step 7: Introduce PDF to data folder and git repo

[ ] PDF is avaliable at https://github.com/elifesciences/enhanced-preprints-data/raw/master/data/90333/v2/90333-v2.pdf

Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-pdf
Step 8: Add PDF url to manuscripts.json

[ ] Reviewed preprint PDF is available for download (https://prod--epp.elifesciences.org/reviewed-preprints/90333)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[PLACE LINK TO PULL REQUEST HERE]

Instructions to add PDF url to manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/add-pdf-url-to-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/397/files
Once the pull request is merged in it should be available a few minutes later.

Step 9: Done!

[ ] Kettle is on!

The figure supplement images are missing from this version of the preprint. I'm not sure if we should hold and get the authors to update this in bioRxiv?
@fred-atherden What do you think? I assume because they included the legends, this is something we'd want to sort out?

Ah, it's because they've been moved to supplementary material. In that case, should I ask the authors to move them or shall we proceed?

Nice spot - thanks @JGilbert-eLife. I think we should ask the authors to include the supplementary files in their main PDF file so that they are included in the article proper. Are you OK to do so?

I've emailed the authors.

Thanks James. v3 MECA is available at s3://transfers-elife/biorxiv_Current_Content/October_2023/25_Oct_23_Batch_1775/c1791a0d-71aa-1014-b93a-c75d767dcd6f.meca

@JGilbert-eLife v3 is now loaded. Could you check this is displaying as you would expect now?

Yep, that looks good -- thanks!

@fred-atherden Just copying this issue over to here to record it: the first reference is messed up; the author spotted this following publication.

Thanks for flagging. The cause is a missing <article-title> element:
<ref id="c1"><mixed-citation publication-type="journal"><string-name><surname>Abad</surname> <given-names>C</given-names></string-name>, <string-name><surname>Cook</surname> <given-names>MM</given-names></string-name>, <string-name><surname>Cao</surname> <given-names>L</given-names></string-name>, <string-name><surname>Jones</surname> <given-names>JR</given-names></string-name>, <string-name><surname>Rao</surname> <given-names>NR</given-names></string-name>, <string-name><surname>Dukes-Rimsky</surname> <given-names>L</given-names></string-name>, <string-name><surname>Pauly</surname> <given-names>R</given-names></string-name>, <string-name><surname>Skinner</surname> <given-names>C</given-names></string-name>, <string-name><surname>Wang</surname> <given-names>Y</given-names></string-name>, <string-name><surname>Luo</surname> <given-names>F</given-names></string-name> <string-name><surname>et al</surname></string-name> (<year>2018</year>) A Rare De Novo RAI1 Gene Mutation Affecting BDNF-Enhancer-Driven Transcription Activity Associated with Autism and Atypical Smith-Magenis Syndrome Presentation. <source>Biology (Basel</source><italic>)</italic> <elocation-id>7</elocation-id></mixed-citation></ref>

Will fix via xsl

@JGilbert-eLife should be fixed now.

Aces, thanks!


[desktop_drop] Not support sub window on MacOS

I use desktop_drop in a sub window created by desktop_multi_window plugin, not work at all.

Looking at the MacOS implementation in DesktopDropPlugin.swift it seems that the plugin is not designed for multiple windows, because the DropTarget is created only once for the mainFlutterWindow at plugin registration time. Instead it should be created per window.

Has there been any further update on this issue, everyone?


Add bicep "deploy" functionality

As I understand it, the idea is to use "bicep build" to generate an ARM template. Then use your favorite method to deploy an ARM template. This is a great start, but to me it feels like we are switching "context". I think it would be a great add to add a "bicep deploy" command which would still build an ARM template, but then call some basic deploy command with some "defaults".
I am not asking to duplicate ALL the functionality of powershell/cli deploy commands. But at least the basic one?
Something like bicep deploy

We've gone back and forth on it. Our current plan is to build bicep support into the deployment commands - basically go the other direction. So I should be able to do:
az deployment group create -f ./main.bicep ...

That will compile and deploy all in one command, so you don't need to see or deal with the compiled JSON if you don't want to. What do you think about that?
Going to close and mark this as a dupe of #33


[USMO] Fix dynamic counter table TLS leak

What does this PR do?
Fix HTTP/2 TLS leaks in the http2_dynamic_counter table.

Motivation
The cleanup of the map happens either in TCP termination or during TLS shutdown event. The TCP termination is manages by the socket filter, thus it cannot clean TLS entries, as it does not have the access to the PID and NETNS. The actual server is not shutting down the TLS connection before closing the socket, so we never clean the entries.
This PR uses tls_finish for the cleaning of http2_dynamic_counter entris when we see tcp_close.

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

The code is good for POC, but not for the mainline Please redo, and try to make it more generic or to work with the current implementation that we have.

I have created a simpler implementation that currently does the work. However, we need to revisit our implementation to improve the design. I have created a Jira ticket for this task.

/merge


Add geocoding for ML

Imported from Google Code issue #148 created by cfkoch on 2012-06-16T02:33:59.000Z:

Data does not exist.
Numbering Plan as of 19.V.2010 contains it.
http://www.itu.int/oth/T0202000083/en

We've accepted this issue and will fix it in an upcoming release.

Fixed in release 7.3.2:  https://libphonenumber.appspot.com/phonenumberparser?number=%2B22320200000


Fix godoc badge to godev badge

I made it at https://pkg.go.dev/badge/

Codecov Report

Merging #131 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #131   +/-   ##
=======================================
  Coverage   79.35%   79.35%
=======================================
  Files          29       29
  Lines        1511     1511
=======================================
  Hits         1199     1199
  Misses        219      219
  Partials       93       93

Continue to review full report at Codecov.

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

Thank you


java.util.zip.ZipException

Whenever I try to import a world it shows this message and cannot import my world.
It asks for a zip file

That would likely be a problem with Minecraft as a product (Minecraft Java Edition?) and not quite in the scope here to address. You may want to look at https://help.minecraft.net/, perhaps?


Correct what might be a possible grammar error

Changed ... scraping metrics HTTP endpoints.. to ...scraping metrics from HTTP endpoints... as the latter might be more correct.

Thanks! Hmm, I see where you're coming from. Both variants are fine, but I'm not sure which is better actually. It was indeed intended to say "metrics endpoints", as the scraping cannot happen from arbitrary endpoints, so that nuance could get lost with the change. And there's already another "from" just before that in the sentence.

@juliusv Thanks for clarifying this. I think I can close this PR now


Adds "converting a local time to UTC time and vice versa" recipe

fixes #360 and should incorperate all the changes made

Thanks for sticking with it @Lapz !


Install jdk7 failed on OSX

~/yottafrontendnew (ttl_conf_key_more_desc ✔) ᐅ sdk install java 7u79
tr: Illegal byte sequence
Oracle requires that you agree with the Oracle Binary Code License Agreement
prior to installation. The license agreement can be found at:
http://www.oracle.com/technetwork/java/javase/terms/license/index.html
Do you agree to the terms of this agreement? (Y/n): Y
Downloading: java 7u79
In progress...
######################################################################## 100.0%
Binary validation passed...
We will be needing super powers...
Repackaging java 7u79...
Attaching to the DMG...
Mounting DMG as Volume...
One or more volume(s) failed to mount
Installing PKG inside DMG...
installer: Error the package path specified was invalid: '/Volumes/JDK 7 Update 79/JDK 7 Update 79.pkg'.
Copy JDK Home to temp folder...
cp: /Library/Java/JavaVirtualMachines/jdk1.7.0_79.jdk/Contents/Home: No such file or directory
Preparing archive...
zip error: Nothing to do! (try: zip -qr /Users/wrb/.sdkman/tmp/.zip . -i .)
Unmounting volume...
hdiutil: unmount: "/Volumes/JDK 7 Update 79" failed to unmount due to error 2.
hdiutil: unmount failed - No such file or directory
Done repackaging...
Cleaning up cookie...
mv: rename /Users/wrb/.sdkman/tmp/.zip to /Users/wrb/.sdkman/archives/java-7u79.zip: No such file or directory
unzip:  cannot find or open /Users/wrb/.sdkman/archives/java-7u79.zip, /Users/wrb/.sdkman/archives/java-7u79.zip.zip or /Users/wrb/.sdkman/archives/java-7u79.zip.ZIP.
rm: /Users/wrb/.sdkman/archives/java-7u79.zip: No such file or directory
Stop! The archive was corrupt and has been removed! Please try installing again.

The clue lies in the following line:
One or more volume(s) failed to mount
I suggest unmounting all volumes on your machine and trying again.

@wuranbo please confirm that this solves the problem before I close this issue.

Please close, it's OK when I following your instruction.
@marc0der   sorry, in my vacation. Thanks your sdkman.


MSID: 89367 Version: 2 DOI: 10.1101/2<IP_ADDRESS>5574

MSID: 89367
Version: 2
Preprint DOI: https://doi.org/10.1101/2<IP_ADDRESS>5574
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=89367
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>5574

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/89367v2
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.1101/2<IP_ADDRESS>5574
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/89367/v2/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!

warn: [mixed-citation-other-publication-flag] mixed-citation in reference (id=c47) has a publication-type='other'. Is that correct?

Reloading to fix above, but reviews not present; article still running in Temporal


Git log missing changes that exist in the release notes for v3.0.8

Describe your question
Comparing the diff from v3.0.7 to v3.0.8 I can't really see all the changes that were mentioned in the release notes. I do see them on the develop branch though, did you create the release from the develop branch so that all the changes are included?
Which project is your question related to?

Generic

@chkr1011 This might be also related: https://github.com/chkr1011/MQTTnet/issues/859 (The binaries for the server were available for 3.0.7 and prior, but not for 3.0.8).

Yes there was a mistake in that version. So I decided to remove the develop branch at all to make sure that this will not happen again. All these changes should be included. I will release 3.0.9 soon so that you can make sure that you have the latest changes.

Ok, nice 👍 I guess we can close this because 3.0.9 will be available soon and the issue was explained.


FreeBSD socketfile creation issue

Hey,
coming from prosody on FreeBSD 13.1-RELEASE, I think we caught an error. Upon creating a socket file for admin_socket, the created socket file lacks the last character. As in, trying to use a filename /socket.sock will result in a creation of a /socket.soc file.
Code that tests/reproduces this on there:
lua54 -e 'print(require("socket.unix").stream():bind("./socket.sock"))'
The FreeBSD package version is lua54-luasocket-3.0.0,1, but it reports as LuaSocket:      3.0-rc1.

Thanks for the report. I'd be happy to push through a fix if we had one, so if you find where this is originating from please go ahead with a PR and I'll get out a patch release.
The issue with luasocket reporting the wrong version internally was fixed just after release, sorry about that.
I tried reproducing this on Linux and was unable to, your MWE works fine.
What locale are you running under? I wonder if we are running into some locale specific string handling issue in Lua. I can't think why FeeBSD would behave differently than Linux in this particular case.

Hey,
I don't have much time, much less am I a Lua expert. @Zash on<EMAIL_ADDRESS>helped me with the command line originally, and pointed out that the error might be here: https://github.com/lunarmodules/luasocket/blob/master/src/unixstream.c#L192-L200
I know C so with this knowledge, I can try and cobble something together (given I'll have time), and come up with a PR if that suffices.
I can't tell when this is gonna happen though, as I'm pretty busy with life and other work. But I guess I'm no exception with that.

That bit of code does look suspicious.
When you look into this also consider perhaps that the Lua make_plat() function in the rockspec might be defining UNIX_HAS_SUN_LEN incorrectly for BSD. I don't actually know in this case if it is fair do lump it in with unix or if it would be more like macosx or if we need to split that platform setting further.
Given that I'm testing on Linux only, it might also be useful if somebody with macOS can test your reproduction above and confirm whether it does or does not work correctly there. It's possible Darwin is as broken as BSD and one side of that ifdef has an off-by-one bug.

I have a macbook pro as well (very rarely used, mostly for testing purposes), will check what gives there.

It would also be helpful to know if our existing test suite catches this. I just looked at CI and we are building and testing on macOS, but looking at the bind("path") examples we have I'm not 100% confident we are testing anything that would catch your problem. If not we should fix that.
Sorry the testing isn't cleaned up yet into a decent runner, but this script or series of commands run from the project root after build (luarocks build --no-install):
#!/usr/bin/env sh
set -e
cd test
lua hello.lua
lua testsrvr.lua > /dev/null &
lua testclnt.lua
lua stufftest.lua
lua excepttest.lua
lua test_bind.lua
lua test_getaddrinfo.lua
lua ltn12test.lua
lua mimetest.lua
lua urltest.lua
lua test_socket_error.lua
kill %1

Hey,
slowly coming back to this, I have somewhat bad news: I sold my macbook pro.
Is there an OSX environment by any chance I can access for testing?

this was fixed in the port https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=268968

Nice! One less problem for me.

@bqv Obviously we wouldn't do it exactly how that port is patched, but is the fix in general something we can upstream?

https://cgit.freebsd.org/ports/commit/?id=7b22ba2f41006419913adfab9cb4e47f27e96df1
It's a packaging issue really, though it might be nice if the build system autodetected freebsd?

If we need some sort of detection for luarocks install to work, then we should probably do that ever if the platform has packages available (so projects that use vendored libraries can run right).


ci: don't run subsequent build steps if a step fails

I think it would be good if we then reordered ASAN to run first. When ASAN fails, it produces a more useful stack trace than a debug fail.

E.g. see https://travis-ci.org/lyft/envoy/builds/201284741

@htuch not sure what you mean by run first? The five different builds are ran concurrently.


Kafka - KafkaContainer not working with confluentinc/cp-kafka:7.1.3

Hello,
When using
with KafkaContainer(image='confluentinc/cp-kafka:7.1.3') as container:
    ...

in a pytest fixture (but I do not think pytest is the cause, as it was working with the same setup and confluentinc/cp-kafka:5.4.9 docker image)
the container waits for 2 minutes before outputting the following :
INFO     testcontainers.core.waiting_utils:waiting_utils.py:46 Waiting to be ready...
...
INFO     testcontainers.core.waiting_utils:waiting_utils.py:46 Waiting to be ready...
wrapped = <bound method KafkaContainer._connect of <tests.gcn_kafka.conftest.kafka_server.<locals>.KafkaContainer object at 0x7f94b579a430>>
instance = <tests.gcn_kafka.conftest.kafka_server.<locals>.KafkaContainer object at 0x7f94b579a430>, args = (), kwargs = {}

    @wrapt.decorator
    def wrapper(wrapped, instance, args, kwargs):
        exception = None
        logger.info("Waiting to be ready...")
        for attempt_no in range(config.MAX_TRIES):
            try:
                return wrapped(*args, **kwargs)
            except transient_exceptions as e:
                logger.debug(f"Connection attempt '{attempt_no + 1}' of '{config.MAX_TRIES + 1}' "
                             f"failed: {traceback.format_exc()}")
                time.sleep(config.SLEEP_TIME)
                exception = e
>       raise TimeoutException(
            f'Wait time ({config.MAX_TRIES * config.SLEEP_TIME}s) exceeded for {wrapped.__name__}'
            f'(args: {args}, kwargs {kwargs}). Exception: {exception}'
        )
E       testcontainers.core.exceptions.TimeoutException: Wait time (120s) exceeded for _connect(args: (), kwargs {}). Exception: KafkaError: Unable to connect with kafka container!

It looks like the _connect method of the KafkaContainer class does not work with this version.
I tried to replace it by the following :
@wait_container_is_ready(UnrecognizedBrokerVersion, NoBrokersAvailable, KafkaError, ValueError)
def _connect(self):
    bootstrap_server = self.get_bootstrap_server()
    consumer = KafkaConsumer(group_id='test', bootstrap_servers=[bootstrap_server])
    if not consumer.bootstrap_connected():
        raise KafkaError("Unable to connect with kafka container!")

and it works.
I do not know if I can open a PR for this, or if someone can just replace the current code by the one I suggest.
My setup uses python 3.9.5 (from Ubuntu 20.04) with the following libraries :
pytest==7.1.2
pytest-cov==3.0.0
pytest-env==0.6.2
pytest-mock==3.8.2
testcontainers[kafka]==3.6.1
kafka-python==2.0.2

I wonder if the root cause is not that consumer.topics() returns an empty list with a newer version of confluentinc/cp-kafka, and makes the if test fail.

Thanks for reporting, @Panikabor. Yes, a PR would be great! It would be great to have a test showing that the updated code works with both older and newer versions of Kafka.

I'll try to do it. Which test framework are you using, so I can write the test accordingly ?

We're using pytest. You can have a look at existing Kafka tests here: https://github.com/testcontainers/testcontainers-python/blob/master/tests/test_kafka.py

Here is the PR : #243

Hello, Any idea of when it will be delivered on pypi ? Beacause I still have my monkey patch in my project, and I'd like to get rid of it. Thank you.


Order of method overloading declaration matters when one parameter type extends the other

Playing around with the type definitions for React 0.13 I ran into this problem:
declare module Test {
	interface Parent {
		y: number;
	}
	interface Child extends Parent {
		z: string;
	}

	interface ParentDeriv {
		yderiv: number;
	}
	interface ChildDeriv extends ParentDeriv {
		zderiv: string;
	}

	interface Api {
		makeParent(): Parent;
		makeChild(): Child;
		meth(element: Parent): ParentDeriv;
		meth(element: Child): ChildDeriv;
	}

	var api: Api;
}

var x = Test.api.meth(Test.api.makeParent());
x.zderiv; // Should be: error. Is: error.
var y = Test.api.meth(Test.api.makeChild()); // takes wrong method.
y.zderiv; // Should be: ok. Is: error. y is inferred ParentDeriv.

But if you flip the method declarations around, it works fine:
declare module Test {
	interface Parent {
		y: number;
	}
	interface Child extends Parent {
		z: string;
	}

	interface ParentDeriv {
		yderiv: number;
	}
	interface ChildDeriv extends ParentDeriv {
		zderiv: string;
	}

	interface Api {
		makeParent(): Parent;
		makeChild(): Child;
		// Note order: child class first
		meth(element: Child): ChildDeriv;
		meth(element: Parent): ParentDeriv;
	}

	var api: Api;
}

var x = Test.api.meth(Test.api.makeParent());
x.zderiv; // Should be: error. Is: error.
var y = Test.api.meth(Test.api.makeChild());
y.zderiv; // Should be: ok. Is: ok.

This is a bug, right? With TS I'm never sure..
Tested with 1.4 (Playground and local)

Overload resolution absolutely matters. The rule is that the first applicable overload is chosen. We tried a bunch of other rules and this was the only one anyone could ever make sense of in practice.
This does mean that it's possible to write overloads which are "unreachable" by putting the general signature before the more specific overload; care needs to be taken to put the more-specific overloads first.

Alright thanks for getting back so quickly, sorry for the false alarm.


Updated Guidance around SharePoint Web Service Identification and Throttling (https://aka.ms/spserviceidentity)

Category

[x] Question
[ ] Typo
[ ] Bug
[ ] Additional article idea

The guidance contains this statement about AppID/AppTitle:

If you have created an application, recommendation is to register and use AppID and AppTitle – This will ensure the best overall experience and best path for any future issue resolution. Include also the User Agent string information as defined in following step.

What is meant by 'use' the appID and appTitle? Is this referring to accessing SharePoint using SharePoint/Azure AD app registration (rather than using .NET CSOM with credentials or JavaScript running on a SharePoint page with no app registration)?
What about SPFx web parts that access SharePoint using REST (including pnp-js-core) or JSOM?

When is it appropriate to use ISV vs NONISV?
Should 3rd party developers extending SharePoint on behalf of a client use NONISV?

The guidance contains this statement about JavaScript libraries:

If you are building your own JavaScript libraries, which are used to call SharePoint Online APIs, make sure that you include the User Agent information to your http request and potentially register your web application also as an Application, where suitable.

It's not possible to set the user agent for http requests sent from a browser.
Is this referring to node.js build tooling/server applications/Azure Functions?

Are there any plans to update CSOM to make setting the UserAgent simpler?
Perhaps a userAgent string parameter added to a new ClientContext constructor?

Thanks!
CC: @VesaJuvonen

Yes, it is not clear how this affects JS application especially those running in the context of a SharePoint page.

When running an SPFX solution the requests are run as the user.  There is no appID or appTitle that you can decorate your requests with.  That documentation about throttling is targeting users who are using 2013 apps.

I'm trying to understand this guidance in respect to a standalone application (UWP, WinForms, console etc) accessing SharePoint via .Net CSOM, there is no registered App in Azure. I'm assuming this needs to be unique so that some throttling rules can identify the traffic and treat it differently? how do we go about creating this unique code and letting Microsoft know about it? CC: @VesaJuvonen


Cannot build factory demo (AEGHB-706)

Using component placed at C:\git\esp-box\examples\factory_demo\components\espressif__esp_schedule for dependency espressif/esp_schedule(~1.0.0)(introduced by component espressif/esp_rainmaker), specified in C:/git/esp-box/examples/factory_demo/main\idf_component.yml
....................................HINT: Please check manifest file of the following component(s): main
CMake Error at C:/Espressif/frameworks/esp-idf-v5.2.1/tools/cmake/build.cmake:544 (message):
ERROR: Because espressif/esp_rainmaker (1.0.0) depends on
espressif__esp_schedule (1.1.0)
and espressif__esp_schedule (1.1.0) depends on espressif/rmaker_common (~1.4.2), espressif/esp_rainmaker (1.0.0) requires espressif/rmaker_common (~1.4.2).
So, because no versions of espressif/rmaker_common match ~1.4.2
and project depends on espressif/esp_rainmaker (1.0.0), version solving failed.
Call Stack (most recent call first):
C:/Espressif/frameworks/esp-idf-v5.2.1/tools/cmake/project.cmake:605 (idf_build_process)
CMakeLists.txt:16 (project)
-- Configuring incomplete, errors occurred!
See also "C:/git/esp-box/examples/factory_demo/build/CMakeFiles/CMakeOutput.log".
cmake failed with exit code 1, output of the command is in the C:\git\esp-box\examples\factory_demo\build\log\idf_py_stderr_output_24636 and C:\git\esp-box\examples\factory_demo\build\log\idf_py_stdout_output_24636

Hello you can use this command pip install idf-component-manager==2.0.0.dev0 to update your idf-component-manage version.

Temporarily closed the issue. Should you have additional questions or concerns, don't hesitate to reopen the issue.


Crash when removing parks from library.json

at mcjty.lostcities.dimensions.world.LostCitiesTerrainGenerator.generatePart(LostCitiesTerrainGenerator.java:1821)
at mcjty.lostcities.dimensions.world.LostCitiesTerrainGenerator.generateStreet(LostCitiesTerrainGenerator.java:1512)
at mcjty.lostcities.dimensions.world.LostCitiesTerrainGenerator.doCityChunk(LostCitiesTerrainGenerator.java:792)
at mcjty.lostcities.dimensions.world.LostCitiesTerrainGenerator.generate(LostCitiesTerrainGenerator.java:223)
at mcjty.lostcities.dimensions.world.LostCityChunkGenerator.func_185932_a(LostCityChunkGenerator.java:205)

I am having a similar prob however i can get MC to load its just when i try to generate a world it hangs on the building terrain screen and its not just the parks any thing i try to remove causes this.
As well as removing all the parks iv even tried to remove the entire park section.
"parks": [
],
or
"fountains": [
],


New package: PGLib v0.1.0

Registering package: PGLib
Repository: https://github.com/noahrhodes/PGLib.jl
Created by: @noahrhodes
Version: v0.1.0
Commit: 67ad9addf15f2a0919421d5cb80ea0f0977897c9
Reviewed by: @noahrhodes
Reference: https://github.com/noahrhodes/PGLib.jl/issues/4#issuecomment-1030445488

The package name is based on the data library is provides, which is abbreviated to PGLib.  https://github.com/power-grid-lib
Please let me know if the name is too similar  for the package to be merged.


Player not start when host index.min.js on my server

Why I can not host the index.min.js on my CDN website?
Regards

Hi! Webtor monetizes itself by serving ads. Any modifications to embed sdk script can lead to revenue loss. To prevent such modifications webtor checks source of embed script to be original.

The problem is the type of ads, it is popup full of sexual content, if you
make the ads like YouTube, when the vedio starts and user can escape the
ads it will be more elegant.
But popup is the worst option to go with.
Any way, is there any one time fee that we can pay to remove the ads.?
Regard
On Sun, Mar 6, 2022, 1:57 PM Pavel Tatarskiy @.***>
wrote:

Hi! Webtor monetizes itself by serving ads. Any modifications to embed sdk
script can lead to revenue loss. To prevent such modifications webtor
checks source of embed script to be original.
—
Reply to this email directly, view it on GitHub
https://github.com/webtor-io/embed-sdk-js/issues/63#issuecomment-1059948964,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDLT26KFZJMO3TBPWVW2L3U6SMT5ANCNFSM5P76HZBA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

I know that it's not the best option for user experience, but I haven't found any better way so far. If you know any ad network that could provide better variants with good CPM just let me know.
You may start to support project (5$ every month or more) and I will remove ads for your domain.
https://www.patreon.com/bePatron?u=24145874 - donation link

Why dont use adsense.
Seconds, am looking for one time payment.
And unlimited useg.
Regard
On Sun, Mar 6, 2022, 2:19 PM Pavel Tatarskiy @.***>
wrote:

I know that it's not the best option for user experience, but I haven't
found any better way so far. If you know any ad network that could provide
better variants with good CPM just let me know.
You may start to support project (5$ every month or more) and I will
remove ads for your domain.
https://www.patreon.com/bePatron?u=24145874 - donation link
—
Reply to this email directly, view it on GitHub
https://github.com/webtor-io/embed-sdk-js/issues/63#issuecomment-1059952626,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDLT26NMH7ZM7SOB77FQWLU6SPFVANCNFSM5P76HZBA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

Why dont use adsense.

AdSense can't work in iframe (in embed) and I use it on main site already.

Seconds, am looking for one time payment.
And unlimited useg.

Oh, sorry, I can't help you here. But you can setup and host webtor api by yourself. Check this docs for more information.

I know that it's not the best option for user experience, but I haven't found any better way so far. If you know any ad network that could provide better variants with good CPM just let me know.
You may start to support project (5$ every month or more) and I will remove ads for your domain.
https://www.patreon.com/bePatron?u=24145874 - donation link

i started donate you 5$ montly but the ads where removed only for my gmail account, but not for the entire domain. Can you please help me on this ?
@vintikzzz

@gocejov please send me direct message through Patreon with yout site domain name.

@vintikzzz i just did


Tasks not working - ROS workspace is probably not sourced.

Hi,
I am using this workspace template with a different Dockerfile based on Nvidia's ISAAC ROS. I added commands to the Dockerfile to automatically source the ROS environment for the "ros" user I create in the Dockerfile. ~/.bashrc for the ros user looks like this:
source /opt/ros/humble/setup.bash
if [ -f /workspaces/header-control_workspace/install/setup.bash ];
    then source /workspaces/header-control_workspace/install/setup.bash;
fi

When I try to run a task like cppcheck, I get an error: /bin/bash: ament_cppcheck: command not found.
If i run ament_cppcheck in a normal shell inside the container, it works normally.
What do I have to add to tasks.json or the Dockerfile to make this work?

Assuming /workspaces/header-control_workspace is the right name it should work?
Might want to check this setting

I'd need a pointer to your dockerfile/workspace to see if I  can be reproduce, otherwise I'm not sure I can help much.

This issue is marked as stale because of no activity


Landing page spec entry error display needs polish

Right now errors are not visually distinguished from normal text.

When fixing this the STYLE.md guide should also be updated.

Colored red, will deal with STYLE.md later


Feature/doc 530

PR for fix https://github.com/apollographql/graphql-tools/issues/530
Sorry for the delay. :'(

@stubailo: hey, please check-out, Is it ready to merge?

Thanks!


azcore.JoinPaths strips query params

azcore.JoinPaths(rootURLWithQueryParams, segment1, segment2)

The query params in rootURLWithQueryParams will be removed.

Fixed


feat: 🎸 Add new events and extrinsics for 6.1 chain support

Description
Add new extrinsics and events for 6.1 chain support.
Following are the new extrinsics added -
instantiate_with_code_as_primary_key
instantiate_with_hash_as_primary_key
allow_identity_to_create_portfolios
revoke_create_portfolios_permission
create_custody_portfolio.

Following are the new events added -
RemoveAssetAffirmationExemption
AssetAffirmationExemption
PreApprovedAsset
RemovePreApprovedAsset
PreApprovedPortfolio
RevokePreApprovedPortfolio
InstructionAutomaticallyAffirmed.

Breaking Changes
NA
JIRA Link
DA-940
Checklist

[ ] Updated the Readme.md (if required) ?

Once a chain image is available, we can add in support for InstructionAutomaticallyAffirmed

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


[QUESTION] Acquiring Traefik Logs in K8s

I have the bouncer up and running on my K8s cluster and am able to ban and unban my own IP manually. I also see a bunch of decisions from the CAPI so I assume known malicious IPs are being blocked as well.
However, it doesn't look like any local attacks are being blocked (HTTP probing, XSS, etc.). When I use the nikto tool to simulate an attack, the bouncer doesn't ban anything. For comparison, my Docker compose instance of Traefik/Crowdsec immediately issues a ban within a few seconds of using the tool (both the K8s and Docker setups are very similar and on the same network).
I've used the example in this repo as a template so am confused about where I went wrong. Also confirmed that my Traefik access logs are enabled and that they show the real IP of the client.
I did notice that running cscli metrics does not show an Acquisition Metrics table on K8s (it does in my previous Docker setup). Maybe that's a clue? My agent.acquisition looks identical to the example.
Not sure if this is an issue with the bouncer or not, hopefully this is the right place to post. Any help is appreciated!

Looking more into it, it's obvious this is a Crowdsec issue, not the bouncer's fault.