Bug in logging error

I noticed something odd when loading a case multiple times:
...

loqusdb.exceptions.case_exceptions.CaseError: Case 15026-miptest already exists in database. Can not add case twice.

During handling of the above exception, another exception occurred:

...

  File "/home/hiseq.clinical/miniconda/envs/apps/lib/python3.5/site-packages/loqusdb/commands/load.py", line 68, in load
    logger.warning(error.message)
AttributeError: 'CaseError' object has no attribute 'message'

seems like the error variable isn't the exception that's get's thrown

Hehe ok I'll fix this

Wow hard to foresee stuff like this with tests etc..
At least good that you could not load same case twice 😸

👍 🇿🇲 ⚡


Show other editors

As editor I want to know which other users are in the system

This information is available to all users from the profile page added in https://github.com/tactilenews/100eyes/pull/1549
Closing this, we can re-open if there is added more context that would make it clear what else, if anything, should be done.


Feat: Enhance sst console issues overview

The issue tracker of the sst console is great and I think with a small change it could be even greater.
The naming of aws lambda function of api routes is totally unreadable for us humans - thats why you tag it with a great lambda function description during deployment.
Now it would be great if that description would be also displayed in the item cell below the function name:

Oh interesting, is this an API Gateway route Lambda?

No no I was just asking. Are you setting a description right now?

No no sst / pulumi sets it automatically during deployment (I guess because the name is so generic that it is easier to see in the aws console what the function is actually about (see image)).
And exactly that auto generated description would be nice to display in the sst issue console.

Yeah that showing that would be useful.


Update s2n_aead_chacha20_poly1305_set_encryption_key to use correct EVP …

…init function
Please note that while we are transitioning from travis-ci to AWS CodeBuld, some tests are run on each platform. Non-AWS contributors will temporarily be unable to see CodeBuild results. We apologize for the inconvenience.
Issue # (if available):
N/A
Description of changes:
This fixes the weird looking (but harmless) code calling EVP_DecryptInit_ex in s2n_aead_chacha20_poly1305_set_encryption_key. This has no impact because the correct EVP_EncryptInit_ex is called before use in s2n_aead_chacha20_poly1305_encrypt.
The only difference between EVP_DecryptInit_ex and EVP_EncryptInit_ex is the enc direction in the context https://github.com/openssl/openssl/blob/OpenSSL_1_1_1-stable/crypto/evp/evp_enc.c#L234 which is only used to track the direction if using the EVP_CipherUpdate functions.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@720df26). Click here to learn what that means.
The diff coverage is n/a.


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/78085

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

TagBot fix

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/81709

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/84150

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/91275

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/92728

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/97207

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/109045

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/109450

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/118112


Enhance platform_error_log() to also report source code location and pid / tid values as part of the error message.

To make debugging easier, it will be useful to extend the message printed by platform_error_log() to also report:

Source code-location info: file / line / function name
OS-pid / thread-ID info. (This is helpful to understand thread's identity when it encounters an error.)

These changes are inline what has been done already for the messages printed by platform_assert(), initially added under SHA 3674df6 (issue #102) and was subsequently enhanced under issue #462 to report OS pid/tid info.
Prefixing source-code location was extensively used in an on-going dev prototype effort and was found to be useful to quickly spot where the error is coming from.
This issue is being registered to productize that prototype code and integrate it to /main.
A sample output with this kind of change will look like so, when platform_error_log() is called from a new unit test case:
TEST 4/5 misc:test_platform_error_log_w_args
Verbose mode on.  This test exercises error-reporting logic, so on success it will print a message that appears to be an error.
tests/unit/misc_test.c:198:ctest_misc_test_platform_error_log_w_args_run(): OS-pid=3405337, OS-tid=3405337, Thread-ID=0, This is a test message with arguments: sizeof(pid_t)=4, 'String arg to test msg.'

This bit tests/unit/misc_test.c:198:ctest_misc_test_platform_error_log_w_args_run(): OS-pid=3405337, OS-tid=3405337, Thread-ID=0 is extra, that will be prepended by this suggested change to the error message text.

We discussed this proposal with core dev team ... sometime in Jan 2023 (I think; could n't find the slack thread).
There was no agreement in adding this change.
People were not convinced about the usefulness of this extra info coming out, by default, for every error message.
People thought that it won't be very difficult to map the error text to the line of code where it's raised, so making this extra file / line # available by default (along with OS/tid/pid) may needlessly clutter the error message stream.
The basic concern was that if we do something in this area, we should look into building a proper error reporting machinery, with error-levels and so on. And may be put this extra-info as an optional opt-in, perhaps with a --verbose-error-info or --extended-error-info (my preference) cmdline argument.
No further work to be done on this issue.


fileroller is not found

I think the correct package is file-roller instead

Oh fixed, thank you!


Revert "Testing connection & storage on Supabase"

Reverts jlsanzbreton/flood-claim-helper#1

We undo the previous merging because of corrupted files in the Development branch


Handlebars raw tags not working

There seems to be an issue in raw og triple mustache tags fixed in issue #5957 ... it doesn't work in the demo either. All content after the triple tag is not colour coded:
https://codemirror.net/mode/handlebars/index.html

Support for triple braces seems to have been added in a6a9696aa61ffd918726 , without ever working. The mode multiplexer will cut off the inner mode at the first occurrence of its end token (in this case }}).

Thank you for at quick resolution. I can confirm the change has fixed the issue 👍


lat_0, lon_0 ignored

I am trying to set up Basemap to use a Cylindrical Equal Area projection and to place the origin at a specific lat/lon (this is to match some input x/y data that uses a false easting/northing), but it seems that setting the lat_0 and lon_0 parameters does nothing.
from mpl_toolkits.basemap import Basemap

# Oklahoma bounds
# ('o' = origin at corner of Kingfisher/Logan/Canadian/Oklahoma counties)
lats = {'n':  37.1, 's':  33.6,  'c':  35.5, 'o':  35.726}
lons = {'e': -94.4, 'w': -103.0, 'c': -97.5, 'o': -97.674}

buf = 0.05 # plot buffer size (in degrees)

res = 'c' # map shapes resolution

# projection parameters
proj = 'cea' # cylindrical equal area
lat_ts = lats['c'] # true scale at geographic center
lon_0 = lons['o']  # x=0 at counties corner
lat_0 = lats['o']  # y=0 at counties corner

m = Basemap(projection = proj, lat_ts = lat_ts, resolution = res,
            lon_0 = lon_0, lat_0 = lat_0,
            urcrnrlon = lons['e'] + buf, urcrnrlat = lats['n'] + buf,
            llcrnrlon = lons['w'] - buf, llcrnrlat = lats['s'] - buf*4)

Then checking the origin gives:
In [15]: m(0, 0, inverse=True)
Out[15]: (-103.05000000000305, 33.399999999963455)

...which is the lower-left corner of the map bounds, not the coordinates I gave for lat_0 and lon_0. I get the same output if I leave out lat_0 and lon_0 entirely.
At first, I thought maybe the CEA projection doesn't support these parameters and that I should just shift my data manually, but the docs claim that lat_0 and lon_0 are used by all projections to set the origin.
As a workaround, I can set an offset to get the intended results:
offset = m(lon_0, lat_0)
df['x'], df['y'] = (df.x + offset[0], df.y + offset[1]) # where df is a Pandas dataframe with my data

Note: Originally posted to StackOverflow, where I was encouraged to move the issue here. http://stackoverflow.com/questions/31302427/lat-0-lon-0-ignored-by-basemap

I also have the same issue, and would really like to get this fixed.

After some debugging found out this was intentional, in a clear contradiction with the documentation

To add to this issue, some psuedocyl projections ('moll' and 'hammer' checked so far) appear to respect the lon_0 value, but there is a strange xaxis switch at specific values. Notice how the second value switches to "celestial" orientation (longitude increasing to the left).
(Note: This was using basemap 1.0.7)
import mpl_toolkits.basemap as basemap
import numpy as np
import pylab as plt

lon_0,lat_0 = -80.5,0
parallels=np.arange(-90.,120.,30)
meridians=np.arange(0,420.,60.)

fig,ax = plt.subplots(3,1,figsize=(6,9))
for i,(lon_0,lat_0) in enumerate([(-80.5,0),(-80.58345277606,0),(-80.6,0)]):
    plt.sca(ax[i])
    m = basemap.Basemap(projection='moll',lon_0=lon_0,lat_0=lat_0)
    m.drawparallels(parallels)
    m.drawmeridians(meridians)
    for mer in meridians[:-1]:
        plt.annotate(r'$%i^{\circ}$'%mer,m(mer,5),ha='center')
plt.savefig('moll.png',bbox_inches='tight')


Please comment "connect wallet" button for now

connect wallet has no real use case at this point in time. we have decided to just remove it altogether for now until we have some web3 functionality needed

Still showing on mobile. Heads up.


[DOC] Replace phase with transit_midpoint

Problem description
#361 swapped out arg phase in favor of transit_midpoint for lc.fold but the docs still use phase
Example

Snippet from http://docs.lightkurve.org/tutorials/how-to-recover-the-first-tess-candidate.html gives a TypeError
folded_lc = clipped_lc.fold(period=6.27, phase=0.4)

TypeError: fold() got an unexpected keyword argument 'phase'

But this works just fine:
folded_lc = clipped_lc.fold(period=6.27, transit_midpoint=0.4)

Expected behavior

Docs use current args for lc.fold
Environment:

platform (e.g. Linux, OSX, Windows): Linux
lightkurve version (e.g. 1.0b6): master 21833db
installation method (e.g. pip, conda, source): pip install -e .

Proposed change
PR #363
PS Thank you! Great presentation last night! :+1:

#363 has been merged, and the online docs will be updated accordingly in the next 48 hours.
@brennv thank you for this contribution, and @christinahedges thank you for last night's presentation =)


[CORE] [THEMES] Select templates defined in the theme

Currently the user can write any text in the field template in the options of the new page. Would be good to create a folder inside the theme with templates and show to the user these templates.
Example

Active themeblogx
Folder for templates, /bl-themes/blogx/templates/
Template games, /bl-themes/blogx/templates/games.php
Template animals, /bl-themes/blogx/templates/animals.php
Show to the user in the field template = games and animals

i like to do this. i already have an idea. i could use $WHERE_AM_I, currently possible vaues were home ans page a possible solution were to use $WHERE_AM_I to determine wich template to use. maybe i move home.php and page.php to template folder to


Live chats + prospector fix

Description
Add link to discord server in contact sections of the documentation.
Change prospector version to enable static code analysis.
How Has This Been Tested?

actions executed

Process

I know the process and did my best to follow it

Pull Request Test Coverage Report for Build 481

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 470:
0.0%

Covered Lines:
350

Relevant Lines:
350

💛  - Coveralls


Added request.catch

Avoid uncaught exceptions, because request is a promise.

https://github.com/codemanki/cloudscraper/issues/194

Travis tests have failed
Hey @Revadike,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.
Node.js: 10
View build log

     npm test

><EMAIL_ADDRESS>test /home/travis/build/codemanki/cloudscraper
> npm run lint && nyc --reporter=html --reporter=text mocha

><EMAIL_ADDRESS>lint /home/travis/build/codemanki/cloudscraper
> eslint --ext .json --ext .js .

/home/travis/build/codemanki/cloudscraper/index.js
  143:17  error  Expected error to be handled  handle-callback-err
  144:5   error  Unnecessary return statement  no-useless-return

✖ 2 problems (2 errors, 0 warnings)
  1 error and 0 warnings potentially fixable with the `--fix` option.

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>lint: `eslint --ext .json --ext .js .`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>lint script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!  Test failed.  See above for more details.

Node.js: 8
View build log

     npm test

/home/travis/build/codemanki/cloudscraper/index.js
  143:17  error  Expected error to be handled  handle-callback-err
  144:5   error  Unnecessary return statement  no-useless-return

✖ 2 problems (2 errors, 0 warnings)
  1 error and 0 warnings potentially fixable with the `--fix` option.

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>lint: `eslint --ext .json --ext .js .`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>lint script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!  Test failed.  See above for more details.

Node.js: 6
View build log

     npm test

Done. Your build exited with 1.

TravisBuddy Request Identifier: a1a45dc0-5e02-11e9-9212-e1acf2d44b97

@codemanki I'd close this too unless you want to coach him on how to fix it. It's such a poor attempt and his tone isn't right. I'm done helping it.

I agree it's bad to dismiss the caught error as I did, but I'm not sure how to handle it in your environment. I've loved to be "coached" on how to do it properly.

If a callback is provided, I agree that the error makes little sense other than to encourage users to use request as the requester.
First, you'll want to consider how this change will affect the user base. If they're using request as the requester, it won't affect them at all. If they're using request-promise with a callback but still expecting a promise to either be resolved or rejected (We do this in almost every test), you'll be breaking their code.
Thus it's a breaking change and requires updating the documentation, a major version bump, etc. Not to mention rewriting all of the tests. Assuming none of that deters you from wanting to introducing this change, continue reading...
The first thing you're going to want to do is check to see whether the user supplied a callback. You can determine that by checking to see if options.callback is a function in the performRequest function. Next, determine whether or not the requester is request-promise. If it's not request-promise, the catch method won't exist and there is no need to do anything at all. I'm sure this was something else that was bugging me about this change but I can't recall what it was, maybe it was nothing... Let me know if you need anymore assistance.
Generally, the addition of a condition will cause the code to branch which means test code coverage will decrease if you don't add a test to cover that branch of code. You'll want to become familiar with the unit tests and ask yourself how that change might affect those tests. It's likely that some tests will need to be modified. If you miss something, that's what reviews are for so no worries.

@Revadike Did you decide whether or not you still want to working on this?

@Revadike if you ever decide to get back to this PR, please feel free to reopen it.

@codemanki That's a nice sentiment but he can't reopen this PR because you closed it. Had he closed then he'd be able to reopen it but not otherwise. @Revadike could always just open a new PR with the changes and reference this one if he wants to refer to the comment history. Simply asking this to be reopened is an option as well.

@pro-src @Revadike sorry guys, I wasn't aware of this fact. Tho I could reopen it if @Revadike pings me here

I completely agree with the decision to close it for now. I just wanted to let you know that Github isn't that liberal. :laughing:

Nah, in fact, I deleted the whole branch.


Disable StrictMode for quick reads/writes

Fixes #1624.
Note that it's not currently easily disable StrictMode warnings caused by Firebase Crashlytics initialization. Alternatives are currently being investigated by Firebase team: https://github.com/firebase/firebase-android-sdk/issues/3795
@shobhitagarwal1612 PTAL?

/gcbrun

Codecov Report

Merging #1636 (9cfc386) into master (89acd47) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1636      +/-   ##
============================================
- Coverage     29.28%   29.26%   -0.02%
  Complexity      863      863
============================================
  Files           352      353       +1
  Lines          7530     7548      +18
  Branches        539      539
============================================
+ Hits           2205     2209       +4
- Misses         5154     5168      +14
  Partials        171      171

see 2 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Add Wallix's JavaScript library for FIDO2

I used this small library to integrate FIDO2 authentication on the browser and a NodeJS API.
These libraries works in pair and make the integration of FIDO2 pretty quick!

Client library means: library that acts as client, like a browser. Please remove your library from the client and I will merge it.

@Kmaschta I think you've removed wrong thing. Keep the server and remove the client lib.

Damn! My bad... I haven't drink my coffee at the time

Thx @Kmaschta


Performance Issues of Maintain Database with h2 1.4+

When using h2 version 1.4.x I see huge performance decrease and disk space increase during the database maintenance task.
The task normally only takes up a couple of seconds with h2 1.3.x. With 1.4.190 it takes more than 30 minutes (I never let it run till it finishes and stopped after ~ 30 minutes) and database file size increases to more than 3 GB.
File size for 1.3 is ~250MB.
Connection string for 1.4 is jdbc:h2:file:%s;FILE_LOCK=FS;AUTOCOMMIT=ON;
Any idea whats causing this issue?
I'm trying to integrate DependecyCheck with a SBT plug-in to scan applications developed with play framework that come bundled with a h2 db v1.4+
I could force downgrade the version in the project to scan but that would be just a workaround that needs to be done for every project individually.

I have no idea why this is happening. While not ideal could you shade 1.3.x in the SBT plugin to get around the issue? Alternatively, we could look into upgrading the dependency-check-core.
--Jeremy

Hi Jeremy,
overriding the dependency version of h2 in build.sbt as a workaround is fine though a bit tedious and might not work for projects that require a specific h2 version for integration testing.
If I find the time I will dig a bit deeper into h2 what causes the issue. Just wanted to check if someone had similar issues with h2 1.4.x
Thanks,
Alexander


Empty titles break dragging behavior

First of all, thank you for this amazing library!
Steps to reproduce:
<script lang="ts">
  import { Pane } from 'svelte-tweakpane-ui';
</script>

<!-- I've done this to avoid the default title that comes up -->
<Pane title='' position='draggable'>
  ...
</Pane>

When this is done the pane is no longer draggable.

Good catch, thank you!
The div that acts as the drag handle was collapsing to zero height in the absence of the title. Fixed in 1.0.1.


Nordic CONFIG_GPIO_AS_PINRESET not working

I'm working on a nRF52805 with Visual Studio code.
Zephyr version: 3.2.99 build: v3.2.99-ncs2
I need to use the RESET pin as a GPIO and looks like the configuration is not working.
Writing CONFIG_GPIO_AS_PINRESET=n or CONFIG_GPIO_AS_PINRESET=y it doesn't change the result.
The PSELRESET register in the UICR  is always :15000000 that means RESET on GPIO 21
The only way to change the configuration is to erase the UICR register with nrfjprog --eraseuicr after every  programming.
Thank you!
Simone

The only way to change the configuration is to erase the UICR register with nrfjprog --eraseuicr after every programming.

If you use an application with CONFIG_GPIO_AS_PINRESET=n, you need to erase UICR before flashing it if the nRESET functionality was enabled earlier. It it was not enabled, there is no need to erase UICR.
When you flash an application with CONFIG_GPIO_AS_PINRESET=y (which is the default setting), UICR will be modified and the SoC will be reset automatically if needed. Take a look here for the relevant part of SystemInit.

Hello, thank you for the quick answer.
I have tried your suggestion using Visual studio

Set CONFIG_GPIO_AS_PINRESET=n in the .prj file
erase UICR
flash again
read back the UICR

But i still get:
:101200001500000015000000FFFFFFFFFFFFFFFFBC
means the GPIO is set as a RESET, not as a GPIO
Checking in the function void SystemInit(void)
i see the part of #if defined(CONFIG_GPIO_AS_PINRESET) is commented out
so the code is working but some how the register is still set when i flash the device.
Now i have made another test:
If i flash the .hex file with J-Flash and not with Visual Studio, it works in both cases:
With  CONFIG_GPIO_AS_PINRESET=n i see the UICR register all 0xFF
And with  CONFIG_GPIO_AS_PINRESET=y i see :101200001500000015000000FFFFFFFFFFFFFFFFBC
So looks like the issue is in Visual Studio when it flash the device

Closing since this seems to be related to VSCode and not to Zephyr itself.


Ddpb 1201 - fix data

Excellent work! I assume that you had to look at the prod data you extracted to diagnose this? Will this script need re running each time or has the underlying issue been fixed?


Instead for now going to allow the user to query the saved chat history, and edit the raw text.

Sweet. Done.
Editing is done through the chat management tab.
Can search for a conversation, see the JSON or HTML representation of it.
Can edit + Save + See the updated chat.


LIBUSB_ERROR_OVERFLOW when running in thread

I'm trying to run ADB command in thread (specifically, using Twisted deferToThread()) but everytime the returning result is longer than just handful of lines, such as from Shell('getprop'), I get ReadFailedError with this description on subsequent command:
ERROR - [Failure instance: Traceback: <class 'adb.usb_exceptions.ReadFailedError'>: Could not receive data from ddeec0a1 (timeout 10000ms): LIBUSB_ERROR_OVERFLOW [-8]

The AdbCommands object is created in reactor (main) thread and later is passed to threaded execution. Subsequent commands are executed sequentially.
Am I doing something illegal/stupid/wrong here?

Okay, PEBKAC. Found a bug in my own code, sorry for the noise.

And what was the bug ? I am also facing the same issue while suing Install

@ashwinipatankar I'm sorry, I can't recall. Something related to Twisted and deferreds.


Failed to load resource problem while loading fonts in head tag

What versions are you running?
$ go list -m github.com/chromedp/chromedp
github.com/chromedp/chromedp v0.9.5
$ google-chrome --version
127.0.6533.100
$ go version
go version go1.22.4 darwin/arm64

What did you do? Include clear steps.
I have created an repo with the source code which can be used to reproduce the problem
https://github.com/SergeyKrasnolobov/screenshot-golang/tree/main
What did you expect to see?
Local fonts are loaded and successfully applied
What did you see instead?
I've got error message in console.log
Failed to load resource: net::ERR_INVALID_ARGUMENT

@ZekeLu

So, I solved the problem by adding the correct headers to the static file server
// main.go

func staticFileServer() http.Handler {
	fs := http.FileServer(http.Dir("./public/assets/"))
	return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
		w.Header().Set("Access-Control-Allow-Origin", "*")
		w.Header().Set("Access-Control-Allow-Headers", "Access-Control-Allow-Origin, Authorization, Origin, X-Requested-With, Content-Type, Accept, ETag, Cache-Control, If-None-Match")
		w.Header().Set("Access-Control-Expose-Headers", "Access-Control-Allow-Origin, Authorization, Origin, X-Requested-With, Content-Type, Accept, Etag, Cache-Control, If-None-Match")
		w.Header().Set("Access-Control-Allow-Methods", "POST, GET")
		fs.ServeHTTP(w, r)
	})
}

mux := http.NewServeMux()
// serve assets files
mux.Handle("/assets/", http.StripPrefix("/assets", staticFileServer()))

			<meta charset="UTF-8">
			<title>Title</title>
                         <link rel="preload" href="http://<IP_ADDRESS>:3001/assets/fonts/manrope/manrope-bold.woff2" as="font" type="font/woff2" crossorigin>

				Hellow world


Translations are not loaded from composer repository

Describe the bug
The documentation says that when using composer and requiring codeigniter4/translations, the translations should be automatically picked up and that there is no need to copy them to the app directory.
This does not currently work unfortunately.
CodeIgniter 4 version
4.0.0-beta4
Affected module(s)
language, autoloader
Expected behavior, and steps to reproduce if appropriate

Make new composer installation and require codeigniter4/translations
Set default locale to anything other than English
In the demo controller, try for example var_dump(lang('Validation.noRuleSets'))
It will display the English message.

The cause is that the Language module tries to look up the file Language/{$locale}/{$file}.php, but the translations composer repository registers the Translations namespace as the Language directory, which means that codeigniter will try to look for Translations/Language/{$locale}/{$file}.php in the package. The correct file would be Translations/{$locale}{$file}.php. Adjusting the path that is looked up to {$locale}/${file}.php does allow it to work, but it breaks the look-up inside the application and is of course not the real solution (no idea how to solve this cleanly).
Context

OS: Linux
Web server lighttpd 1.4.54
PHP version 7.3.6

oops! I cn fix this next week, when I am vack from vacation :-/

Corrected the composer autoload config in the translations repo.
You should be able to do a composer update in your project now, and have the translations properly detected.
Can you confirm? Thanks.

Unfortunately, composer version 4.0.0-beta.4 does not seem to contain the fix. I think you may have released the wrong commit?

Hmm - I see that. I re-released beta.4, but the binaries were not updated.
I have made a new beta.5 for the translations repo only, which will hopefully correct this :-/

Not hearing anything contrary in 4 days, I presume the fix works.

No, it does not seem to work. As far as I can tell, the namespace you put into the composer.json is irrelevant for the lookup actually. Did it work on your end?

Gah - I thought it worked for me. WIll recheck :-/

Finally nailed this, as far as I can tell.
Nothing to change inside the CodeIgniter4 repo, only the composer.json in codeigniter4/translations.
I moved the translations to their own namespace, rather than trying to map them inside the CodeIgniter one; CI's Autoloader "conveniently" strips the CodeIgniter namespace(s) from the composer autloaded set, since it is loaded earlier. Arghhh.
I tested this on a fresh download of the appstarter, then composer required codeigniter4/translations, and joy ensued :)


Can't resize marker list pane

When I edit the settings for a marker, the dialog box has a pane with a list of all the markers.  But it is quite narrow, and there is no way to make it wider.  Most markers only need that much space, but sometimes some need more.
It would be nice to be able to resize that pane.  Absent that, if there could at least be a tooltip when we hover over a marker, even that would help.
Attaching a video to show the steps, and what does and doesn't work for seeing the full marker.
https://github.com/sillsdev/solid/assets/4039852/990284d0-6069-4182-a8d3-b42024881c11

Attaching the project and .solid file from the video.  Remove .txt from both filenames.
AruStems-LongMarkers.sfm.txt
AruStems-LongMarkers.solid.txt

This looks really good!  Hooray!  I can now resize the pane, and the other parts kept their behavior also.
There is one tiny fiddly that I didn't mention.  At the top of the dialog it is supposed to say the marker "can occur..."   This appears for short markers, but it got truncated for the long ones (I'll attach screenshots).  This is really minor, and I didn't mention it before.  We can live with this, but I wanted to at least note it.

Thanks @bbryson for the feedback!  I've created a new issue #25 so let's discuss further over there.


DDL batching support

spanner-cli -e" {statements} " can execute multiple statements, but they are executed sequentially.
If statements are DDLs, it should be executed as a single large batch in practice.
https://cloud.google.com/spanner/docs/schema-updates?hl=en#large-updates
Consideration:

add a flag like --batch or not
check all statements before execution and batch them only if they are all DDL statements

or batch a subsequence of statements that are consecutive DDL statements

Thank you for filing a feature request.
It makes sense to run multiple DDLs in a batch for better performance, but I'm wondering how many users use spanner-cli -e "{statements}" to apply multiple DDLs although they can use gcloud spanner databases ddl update for that purpose.

I want to unity the operation between the actual Cloud Spanner instance and an emulator (such as a handy-spanner) but gcloud spanner databases ddl update doesn't seem to support SPANNER_EMULATOR_HOST.


Cannot run

Hi! I cloned the repo and built it using .Net Core SDK 2.2.106. I use Visual Studio 2017, 15.9.11 on Window Server 2012. The solution builds successfully and I see db populated with records. However when I debug I was directed to localhost:56543 with message "This localhost page can’t be found No webpage was found for the web address: http://localhost:56543/". The port 56543 was found on NHLStats.Api project / Properties / launchSettings.json. I tried to send request using Postman but no luck. I am new to GraphQL and suspect this is an IIS issue. What goes wrong here?

@lucasxyang
You need to switch to NLStats.API

This will run on port 5000.


[Bug] [dolphinscheduler-master] The handling of task dependency logic is problematic.

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
org.apache.dolphinscheduler.server.master.utils.DependentExecute#findLastProcessInterval

Considering tasks A and B, where task B is contingent upon the completion of task A, and this dependency is established within the operational period of the current month. It is known that task B initiated its first successful run on the 28th day and task A is configured to be scheduled 2 minutes subsequent to B's scheduling time. The present scenario involves an evaluation logic that has segmented the month into a period ranging from April 1st to April 30th. Upon assessing April 1st – a date for which no instance of task B exists – the logic promptly returns a 'waiting' status without proceeding to evaluate the remaining days of the month.
What you expected to happen
Given that the dependency timeframe is set for the current month, and there indeed exists a successful execution instance of task B within this month, the expected outcome of the dependency check should logically yield a 'success' status. This, in turn, would permit task A to proceed with its scheduled execution.
How to reproduce

Task A schedule is [0 30 10 28 4 ? *]
TaskB schedule is [0 32 10 28 4 ? *]
Task B relies on Task A, with the dependency anchored within the business cycle of the current month.

Anything else
No response
Version
dev
Are you willing to submit PR?

[x] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Hello, does anyone have any ideas? I'm experiencing the same problem. I also tried the 'this week' option, but it has the same issue.

I think the logic behind the dependent node design should be that for it to be considered successful, every time slice needs successful results. For periods like this month or this week, it means starting from the first day right up to today, there has to be a successful dependent task each day for it to count as a success. If even one day is missing a task run, or the tasks on that day weren't successful, then the whole task is marked as a failure. It will be checked again during the next scheduled run. The product team should probably think that if a day was missed, someone needs to manually run the tasks for that day to make up for it. Otherwise, they might just 'force success'.

I think the logic behind the dependent node design should be that for it to be considered successful, every time slice needs successful results. For periods like this month or this week, it means starting from the first day right up to today, there has to be a successful dependent task each day for it to count as a success. If even one day is missing a task run, or the tasks on that day weren't successful, then the whole task is marked as a failure. It will be checked again during the next scheduled run. The product team should probably think that if a day was missed, someone needs to manually run the tasks for that day to make up for it. Otherwise, they might just 'force success'.

Correct. The current thisMonth represents every day of the month, not any day of the month. But this is not described in the documentation. I'll add it.


Escape backslashes properly in BaseDescription

Before this commit, the following assertion:
assertThat("\n", equalTo("\\n"));

gives the output:
java.lang.AssertionError:
Expected: "\n"
     but: was "\n"

This is confusing because it seems like two identical strings have not
compared equal.
The problem is that in BaseDescription.toJavaSyntax(char) escapes
newline characters but doesn't escape backslash characters, so the two
strings being compared are transformed to the same output for display,
despite having different actual content.
I haven't written a test for this as I couldn't find a test for any of the existing escaping behaviour.

Done.  There is a BaseDescriptionTest that you could have extended.


include setup of dependencies in timeout

currently the asyncio.wait_for is only started after the dependencies are set up, in case this includes sockets there might be a significant delay before the timeout starts when the service is offline

https://github.com/ldruschk/enochecker3/blob/cb4fdc537275d4ceb3a185c6b40242b23079f49d/enochecker3/enochecker.py#L289


Flex volume is currently hardcoded to 100 MB

The size of flex volumes is currently hardcoded in a constant:
https://github.com/nerdalize/nerd/blob/v2/cmd/flex/main.go#L73
For the public launch it would be nice to have a different solution like specifying it as part of the pod resource limits or the dataset resource metadata. If this is not feasible given the time we have, then it should at least be hardcoded to a more sensible size for our customers.

It might be worth to note that there is currently a limit of 5GB (#298) for the maximum size a anonymous user can upload. There is also an issue that covers giving feedback to the user when its upload is to large #265


Allow list s3 objects with special chars in name

Codecov Report

Merging #48 into master will increase coverage by 0.38%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #48      +/-   ##
==========================================
+ Coverage   85.93%   86.31%   +0.38%
==========================================
  Files          39       39
  Lines         718      738      +20
  Branches       63       62       -1
==========================================
+ Hits          617      637      +20
  Misses        101      101

Impacted Files
Coverage Δ

.../airlock/proxy/provider/aws/SignatureHelpers.scala
88.97% <100%> (+2.06%)
:arrow_up:

Continue to review full report at Codecov.

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


Can't run on localhost(gunicorn & google oAuth)

Hi jobguy!
There is probably an issue with the repo running on localhost.
First of all, the repo is on production setting even in develop branch. but turning it to settings.development still doesn't solve the issue.
there are  two ERRORs that I face when i try to run the project with this command : docker-compose up --build.
the dockerfile gets built successfully but here are the errors (related to gunicorn & google oAuth):
gunicorn.errors.HaltServer: <HaltServer 'Worker failed to boot.' 3>, 'Add GOOGLE_OAUTH_ID env variable, example: export GOOGLE_OAUTH_ID=123456'

hello @hossshakiba
i think this is not a big problem.you can temporarily set some test environment variables for your container
this link can help you i think:
https://docs.docker.com/compose/environment-variables/


Change ProcessDelegate.processHadOutput(output:) argument from String to [UInt8]

There's no guarantee that a read from a child process' stdout will yield valid UTF-8, even if the entirety of the output would be a valid UTF-8 string. Instead of attempting to decode process output in the Swift bindings, forward the raw bytes so clients can handle it appropriately for their use case. This is consistent with the similar commandProcessHadOutput delegate method.
rdar://97019909

@swift-ci please smoke test


Unable to decode an event from the watch stream: stream error: stream ID 3; INTERNAL_ERROR

Here is my question:https://stackoverflow.com/questions/54760830/unable-to-decode-an-event-from-the-watch-stream-stream-error-stream-id-3-inte
Please help me.Thanks

should know http timeout


Error window going crazy.

Please see the gif. When, the error message is too long, it keeps jumping up and down what is really distracting.

I have this code for the error sample :
var foo = {somethingReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyReallyLong:123, c1:'123',c2:'123',c3:'123',c4:'123',c5:'123',c6:'123',c7:'123',c8:'123',c9:'123',c10:'123',c11:'123',c12:'123',c13:'123',c14:'123'};
foo  = 123 /**/

It doesn't reproduce this ... however we do see the error message trimming and not trimming (because of the pending error request screen).
I'll make is to that the pending requests icon always takes space even if not visible. Thanks for the report :rose:

Fore ref, Code https://github.com/TypeStrong/atom-typescript/blob/master/lib/main/atom/views/mainPanelView.ts

https://github.com/TypeStrong/atom-typescript/releases/tag/v7.1.5 :rose:


Add support for MongoDB Atlas vector database

🚀 The feature
Requested by a user on discord forum:

Discord link: https://discord.com/channels/1121119078191480945/1125758905310519327/1195378054063726713
Motivation, pitch
We want to support as many vector databases as possible to meet user needs.

@deshraj Hi, could you assign this issue to me? I am a student and having OSS course, would like to give a try.

Hey @danqixu, thanks for your interest. Please feel free to work on this issue.
Let us know if you have questions. You can ask me on our:

Slack: https://embedchain.ai/slack
Discord: https://embedchain.ai/discord

I am interested to work on this

Hey @AshishKingdom, please feel free to work on this.

Great! Please assign this to me. I'll start working on it

@Dev-Khant I can see there are a bunch of vectorbases that need to be added and I am thinking of writing scripts for all of them. But I need a little help from you. I understand I need to add a new script containing class and methods to interact with vectorbases in the folder embedchain/vectordb. I will also add test cases for these new vectorbases. But I can see there are many configuration files and other scripts that contain some information about these vectorbases. Can you list all the files I will need to modify to make sure new users can use the vectordb i add properly.

@Dev-Khant I can see there are a bunch of vectorbases that need to be added and I am thinking of writing scripts for all of them. But I need a little help from you. I understand I need to add a new script containing class and methods to interact with vectorbases in the folder embedchain/vectordb. I will also add test cases for these new vectorbases. But I can see there are many configuration files and other scripts that contain some information about these vectorbases. Can you list all the files I will need to modify to make sure new users can use the vectordb i add properly.

@Dev-Khant whenever you can, please spare a couple minutes to answer this query so I can work on this over the weekend. Thank you for your continuous guidance and making contributing easier for me

@Dev-Khant I can see there are a bunch of vectorbases that need to be added and I am thinking of writing scripts for all of them. But I need a little help from you. I understand I need to add a new script containing class and methods to interact with vectorbases in the folder embedchain/vectordb. I will also add test cases for these new vectorbases. But I can see there are many configuration files and other scripts that contain some information about these vectorbases. Can you list all the files I will need to modify to make sure new users can use the vectordb i add properly.

@MoizKhuzema Thanks for your willingness to work on separate vector databases. Here I would suggest you that at a time pick one vectordb to add and look at other vectordbs that are already added as a reference. So for every new vector database you will have to add new files for it: config file, Vectordb file',  it's test fileanddocs`. Also you can check at what other places the reference of existing dbs are used and also make a change there for new vectordb.

I would like to work on this issue.
Thankyou,
Adithya Hegde Kota

@hegdeadithyak Please feel free to work on this.

Is there any update on this? We need this for our app Hana -> hana.hanabitech.com

Hey @todaysolve, we haven't pushed this feature out yet. We will prioritize it soon.
You can use Mem0 Platform in the meantime which offers managed service.


EMI experienced a critical exception while reloading, check log

I have had no issues with EMI but it seemingly quite working and I'm not sure what the cause is. I get that error when viewing the inventory screen and it produces this log: https://mclo.gs/Zgx9QVV
I'm going through disabling mods and datapacks, but it is so strange to me.

There seems to be a lot of broken recipes in this log, but I don't see anything about EMI. I will note you have both create and farmer's delight, the latter of which has been breaking create for months. However I don't think that's the only problem here.

I have no idea what happened. I restored an older backup with the exact same modpack and datapack used in the save. So a config setting or file somewhere was what caused it. Which is extremely confusing.

Closing since resolved.


Remove dependency on tchannel-go

Resolves #2129
This PR removes /tchannel package and dependency on tchannel-go from Jaeger core.
Organizations that are using tchannel can copy tchannel package to their codebase and initialize tchannel collector and reporter in their mains.
@yurishkuro could you please review this one?

Added one more commit to remove tchannel comments from the codebase and remove tchannel reporter metrics.


Support anonymous registry access

ACR team recently introduced support for anonymous pull access for registries, which we need to also support in bicep. Presumably, as long as the SDK supports this, we should get this for free.
Related to #5030

This will require us to slightly change how auth config works because the anonymous access is only possible for ACRs. In addition, only some registries in bicepconfig.json would support it but not necessarily all of them.

Hitting this issue when using bicep module registries. Bicep is unable to pull modules as part of bicep build.
As a workaround I setup a service principal with acr-pull on the registry, and run az login in our GH Actions workflow before running bicep build

Related to https://github.com/project-radius/radius/issues/1699

This will require us to slightly change how auth config works because the anonymous access is only possible for ACRs. In addition, only some registries in bicepconfig.json would support it but not necessarily all of them.

Maybe an isPublic boolean property on the br object in bicepconfig to determine an unauthenticated call to the acr?

Hi @majastrz
Thanks for the feedback on this at the last community call.
I am really eager to start using this and was wondering, if there is a working branch for this feature I could test out.

Unfortunately, we don't have a branch to try this out yet ☹


Update MS-PIM reference

Required items, please complete
Change(s):

Updated the entity that references MS-PIM

Reason for Change(s):

the current  entity "InitiatedByDisplayName" does not exist

Version Updated:

No

Testing Completed:

Yes

Checked that the validations are passing and have addressed any issues that are present:

No

Hi @RafaelRuales,
have you completed your changes or still you are working on this? As I see you have updated one line in query but still that line is commented.

Thanks

Good morning @v-rusraut , I left it commented out as I figured it would be up to the individual to determine whether or not they want to see PIM activity. With that being said, the entity "InitiatedByDisplayName" was giving me an error in the query, I figured that the right entity if one was to uncomment that line was "InitiatingAppName" which is referenced earlier in the query, as opposed to "InitiatedByDisplayName" which doesn't seem to be. Does that make sense?


Documentation Improvements.

Description
There are some areas of cdk-nag which cold benefit for some more documentation / tutorials.
(1) Lack of Python examples:  Python is probalby the second most used Cdk Language.  Mirror the how to use documents in Python.
Use Case
Becuase i use CDK in Python as well as Ts
Proposed Solution
Write examples for Python / Mirror the typescript documents.
Other information
I will write this and submit a PR
Acknowledge

[X] I may be able to implement this feature request
[ ] This feature might incur a breaking change

I'm worried about the README becoming too long and hard to read (if the examples were to be mirrored there). Some thoughts related to this.

The ConstructHub page for cdk-nag does a best effort translation of the README examples to Python (and I believe that those translations work). Instead of mirroring examples in the README, perhaps we make it a point to point users to there
Perhaps a dedicated examples folder or cdk-nag specific examples in the cdk-examples repo, and we point users to there

Is there somewhere to see python examples?
There really needs to be better python documentation.  The auto-generated code refers to callbacks, which aren't really a python concept, and it doesn't seem to accept a function passed in as an argument.


Compatibility with chart.js 3.0.0

Is there a new version planned for the latest chart.js?

Hey 👋 thanks for raising an issue! I haven't got the time to update this yet, but definitely will do once I got some free time. PR are very welcome though 😊

Alright, just released v2 of this plugin, which is compatible to Chart.js v3 🎉
Beware that the API of the plugin changed (in order to be more consistent with other plugins).
I'll close this issue, please re-open another one if you experience any issues with the new release!


GitHub Copilot might steal this code - Let's move away!

I've got VERY serious news for you. The Software Freedom Conservancy (SFC) and the Free Software Foundation (FSF) are both urging open source developers to ditch GitHub in favour of something else! Why? Because GitHub Copilot, a new AI-powered coding assistant made by Microsoft's subsidiary, is potentially stealing open source code, including code that is licensed with a copyleft like GPL which requires you to make your work available under the same license as the initial work. I can't support an AI which steals Copyleft code.
Alternatives might be:

GitLab
CodeBerg
SourceHut (Alpha stage)

If you want to read more, get into here: https://sfconservancy.org/GiveUpGitHub/

Apparently, Codeberg doesn't really like seperate webpages as we can do it here. Help?


Default template argument for Scalar

I added a default value of double for LDA, LDABuilder and all the Parameters. @paschalidoud do you think this is enough or we need to add it to more places?
We can now create an LDA as simple as
LDA lda = LDABuilder().initialize_topics_random(100, 10);

Do you think it is bad that we hide the ability to use another Scalar type?


Nav problem with page-progression-direction=rtl(cloud reader 2.28.0-alpha)

This issue is a Bug
Expected Behaviour
From Navigation, link to valid fragment identifier.
Observed behaviour
Instead, Readium does not link to valid fragment identifier, when EPUB's page-progression-direction="rtl"
Steps to reproduce

First, open the test file by readium-js-viewer : cloud-reader
Then open navigation
Click, Chapter2, Chapter3, then jump far backward from valid  fragment.

Test file(s)
https://github.com/toshiakikoike/epub_file/blob/master/holms-hound-of-the-baskervilles.epub
https://github.com/toshiakikoike/epub_file/tree/master/holms-hound-of-the-baskervilles
Product

Other
cloud-reader from readium-js-viewer(develop), in the dist folder contents (generated by npm run dist):
hypothes.is plugin active
2.28.0-alpha

Chrome extension(from Chrome Web Store) has no problem.
Additional information
For example, "book_0003.xhtml" has 3 fragment identifiers;

Chapter 1
 -- this fragment is in the top of this html file.

Chapter 2
 -- this fragment is in the middle of this html file.

Chapter 3
 --  this fragment is in the middle of this html file.
In toc.xhtm;

Chapter 1

Chapter 2

Chapter 3

Thank you for reporting this RTL vertical-writing-mode bug!
Test link:
https://readium.firebaseapp.com/?epub=https%3A%2F%2Frawgit.com%2Ftoshiakikoike%2Fepub_file%2Fmaster%2Fholms-hound-of-the-baskervilles

cc @JCCR


Improvements to work with table relations, added support to associations such as oneToMany and manyToOne

I've added support to associations such as manyToOne and oneToMany, changing the classes FilterExpressionParser and JpaSpecificationExpressionVisitor to use in my own projects.
The modifications also include :

Added support to is null or is not null filters in JpaSpecificationExpressionVisitor
Commented the method convertIfDate to allow the use of date types other than Date, such as OffsetDateTime,  LocalDateTime or LocalDate.

I'd like to contribute to this project with these improvements.
This is an example project I've made using these improvements.
graphql-dynamicquery.zip

Hello,
Thank you for your improvements; I found them very useful. Is there a good way to map the model variableName (linked table) with the name of the input object in GraphQL? I would prefer not to name my linked table the same as the input object :)


Create convex-hull.md

This PR adds an implementation of the Convex Hull problem using the Divide and Conquer approach
📥 Pull Request
Description
This PR adds an implementation of the Convex Hull problem using the Divide and Conquer approach. The Convex Hull algorithm finds the smallest convex boundary that can enclose a set of points in a 2D plane. The implementation provides an efficient solution to the problem with a time complexity of O(n log n).
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Documentation update

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

Thank You.


[WIP] Reduce retained memory by re-deriving TcEnv objects when needed

This is a slightly more speculative memory reduction technique discussed in https://github.com/Microsoft/visualfsharp/issues/2202#issuecomment-274334172
It basically aims to reduce the memory retained in the caches due to the the active foreground checking of documents. Only three documents are held in the cache so we don't expect vast wins here. There are also tradeoffs in reconstituting the information when the UI demands it.
For .\Release\net40\bin\LanguageServiceProfiling.exe ..\FSharp.Compiler.Service CGSFGS (which measures the difference in retained memory between the two "S" actions, i.e. the amount added after checking 6 foreground files then running a GC)
this-branch-after-also-applying-2377-and-2378:     TimeDelta: 8.59     MemDelta:  119     G0:  262     G1:  211     G2:    5
master-after-also-applying-2377-and-2378:     TimeDelta: 8.80     MemDelta:  155     G0:  260     G1:  195     G2:    5

So we get a 36MB (24%) reduction in retained foreground information.
Unlike the reductions from #2377 and #2378, this won't particularly increase for larger projects/files since the files being foreground-checked by the test program are pretty huge, bigger than normal, but still it looks like a useful reduction.  However we need to check that the reconstitution of environments doesn't hurt usability.

Note I'm less convinced this is needed (or at least not urgent) given https://github.com/Microsoft/visualfsharp/pull/2388.

One failure - seems unrelated to this PR
CompilerOptions\fsc\out (--out:out1.exe) -- failed

I'll close this, though we may resurrect it if we find that TcEnv costs are killing us in some specific scenario


support for "kebab case"

not sure this is the right term. i'm talking about this (kebab case).
it's very useful for some of the web programmers

I only encountered that term recently myself. Since this library predates me learning that, it happens to be called param-case.

I came looking for kebab case too. Could we have a kebab-case alias?
Lodash uses the term kebab case.
I've never heard of param-case and I don't think it makes much sense. Parameters (function(foo-bar) {}) is JS cannot be param-case.
http://stackoverflow.com/questions/11273282/whats-the-name-for-hyphen-separated-case/12273101

Also never heard about param-case :) everyone here called it kebab-case... As far as i got there are no plans to add alias?
P.S. Even wiki article has info about kebab-case, not a param-case :) if it is an argument :)

You're welcome to submit a PR. The kebab-case name on NPM is already taken, however, so renaming it properly won't happen.


docs: add more content to cloud integration

What's Changed in this PR
Adds more content to cloud integration docs:

Prometheus: http api intro
Vector: logs
Kafka: new content
Alloy: new content
MindsDB: use our extention
InfludDB: add telegraf
OpenTelemetry: minor update

Checklist

[x] Please confirm that all corresponding versions of the documents have been revised.
[x] Please ensure that the content in sidebars.ts matches the current document structure when you changed the document structure.
[ ] This change requires follow-up update in localized docs.

Updated @nicecui PTAL


Generate unique variable names to deal with name shadowing and Erlang bindings escaping expression blocks

From @erszcz 's comment
Describe the bug
We're now translating Caramel's expression blocks (begin ... end or ( ... )) and some of let bindings (nested ones intentionally, and other ones unintentionally but that's not causing major issues other than reduced readability), to Erlang's begin/end blocks. Unfortunately, because of the way how expression sequences in the Ocaml's Typedtree are represented (unnecessary blocks are getting "erased"), and because of Erlang scoping rules (Erlang has different than "traditional FP" rules about variable scoping where Erlang bindings from within expression blocks are available in the rest of the function as well), sometimes when not shadowing variable names in Caramel code, we're introducing clashes in the generated Erlang code so it fails at runtime.

Create a file main.ml with

let print thing = Io.format "~0tp~n" [ thing ]

let main _ =
  begin
    let a = 1 in print a
  end;
  begin
    let a = 5 in print a
  end

The blocks are there to trick Caramel into thinking there is no shadowing. This is just one example of such situation, there are more e.g. when a binding escapes erlang's begin/end block or a case expressions etc.

Run command caramel compile main.ml && escript main.erl
See result:

$ caramel compile main.ml && escript main.erl
Compiling main.erl      OK
1
escript: exception error: no match of right hand side value 5
  in function  erl_eval:expr/5 (erl_eval.erl, line 453)
  in call from escript:eval_exprs/5 (escript.erl, line 872)
  in call from erl_eval:local_func/6 (erl_eval.erl, line 567)
  in call from escript:interpret/4 (escript.erl, line 788)
  in call from escript:start/1 (escript.erl, line 277)
  in call from init:start_em/1
  in call from init:do_boot/3

The generated Erlang code right now is:
% Source code generated with Caramel.
-module(main).

-export([main/1]).
-export([print/1]).

-spec print(_) -> ok.
print(Thing) -> io:format(<<"~0tp~n">>, [Thing | []]).

-spec main(_) -> ok.
main(_) ->
  A = 1,
  print(A),
  A = 5,
  print(A).

(I think the first begin/end we typed into the .ml file is actually not present in the typedtree, which for OCaml is fine because it doesn't change anything, and because an expression sequence is similar to a "linked list" we are flattening out the second one instead of printing begin/end everywhere)
Expected behavior
The result of running the code should be:
1
5

similar to when running this ocaml file:
(* let print thing = Io.format "~0tp~n" [ thing ] *)

let main _ =
  begin
    let a = 1 in print_int a
  end;
  begin
    let a = 5 in print_int a
  end ;;

main ()

$ ocaml main.ml
15

I think the correct Erlang to generate here could be:
% Source code generated with Caramel.
-module(main).

-export([main/1]).
-export([print/1]).

-spec print(_) -> ok.
print(Thing) -> io:format(<<"~0tp~n">>, [Thing | []]).

-spec main(_) -> ok.
main(_) ->
  A = 1,
  print(A),
  A@1 = 5,
  print(A@1).

similarily to what Gleam generates in such situations.
If we keep track of variable names (or append some unique reference from typedtree to them), solving this should allow shadowing regular variable names in funcitons in general 🚀
Environment (please complete the following information):

OS: Ubuntu Linux 18.04
Caramel version: 0.1.0+git-5f77e1d
Erlang version: 23.2.2

                            pattern (main.ml[5,76+14]..main.ml[5,76+15])
                              Tpat_var "a/23"
                            expression (main.ml[5,76+18]..main.ml[5,76+25])
                              Texp_apply
                              expression (main.ml[5,76+18]..main.ml[5,76+23])
                                Texp_ident "print/15"
                              [
                                <arg>
                                  Nolabel
                                  expression (main.ml[5,76+24]..main.ml[5,76+25])
                                    Texp_ident "a/23"
                              ]

I'm looking at those a/23 names, which suggests there's some number assigned by OCaml to names in the typedtree already that we may be able to just re-use...

Those are stamps and sometimes scope numbers of an Ident.t source - I think we can use them!


redirecting to env variable that doesn't exist causes minishell to exit

[rboudwin@AR-Shell /Users/rboudwin/Documents/repos/minishell]$ echo hello > $NOTREAL
missing file for redirection
<EMAIL_ADDRESS>~/Documents/repos/minishell]$

<EMAIL_ADDRESS>~/Documents/repos/minishell]$ ./minishell
[rboudwin@AR-Shell /Users/rboudwin/Documents/repos/minishell]$ echo hello >> $NOTREAL
missing file for redirection

same issue with append.


Add yaml templates - remove dependency on toml gem

hi @zoojar , thanks for the PR! Do you have some links to the telegraf docs that mention the yaml format? I didn't know that's supported and I didn't find it in the docs.

no worries! I can't see anywhere where telegraf supports yaml. BUT, this is to remove the dependency on the toml gem - the templates are only used as hashes to pass to the telegraf input defined type!

Iv resolved some issues with the CI if anyone would like to try this one again

Iv resolved some issues with the CI if anyone would like to try this one again

thank you.


Request for ARM64v8/aarch64 build of the SDK

OS and version used: Linux tegra-ubuntu 4.4.38-tegra aarch64

Python runtime used: 2.7 and 3.6

SDK version used: 1.4.4

Description of the issue:
Trying to build a docker module based of the arm64v8/debian:stretch docker image, To run on a Nvidia Jetson Tegra TX2 device. When building the image and running the pip install azure-iothub-device-client i get the pip install error "Could not find a version that satisfies the requirement azure-iothub-device-client==1.4.4"
I suppose this is due to the fact that there hasn't been built any release of the Python SDK for ARM64/aarch64 yet. So I guess this is more a request for it to be built and available through the regular PIP channel if possible.
Code sample exhibiting the issue:
pip install azure-iothub-device-client==1.4.4
Console log of the issue:
Collecting azure-iothub-device-client==1.4.4 (from -r requirements.txt (line 1))
  Could not find a version that satisfies the requirement azure-iothub-device-client==1.4.4 (from -r requirements.txt (line 1)) (from versions: )
No matching distribution found for azure-iothub-device-client==1.4.4 (from -r requirements.txt (line 1))
The command '/bin/sh -c pip install -r requirements.txt' returned a non-zero code: 1

@FluxReality That's highly unlikely to happen with the current version of the SDK. We're focusing on rewriting a new version that will be a pure-python implementation instead of a wrapper over the C SDK and should support arm64. The current, wrapper-based implementation will only see bugfixes but not enhancements or new platforms support.
We do have a build guide so if someone wants to try and setup a build for arm64 architectures, we'd be happy to share the dockerfile in this repository, as part of our dockerfiles folder.
We do hope to have a preview of the v2 SDK very soon. it will support only MQTT at first, and we will progressively roll out features (starting with telemetry, cloud-to-device messages & input/output channels for Azure IoT Edge). If you have suggestions about how to prioritize features for the new version, we're all ears.

@FluxReality, thank you for your contribution to our open-sourced project! Please help us improve by filling out this 2-minute customer satisfaction survey


[BUG]EventHubChannelResourceManagerProvisioner->validateOrCreateForConsumer-ProvisioningException in Azure Spring Cloud Stream Binder

Describe the bug
spring.cloud.stream.bindings.output.destination is getting created but not the input destination when used below configuration
spring.cloud.azure.auto-create-resources=true
Exception or Stack Trace
2021-01-17 22:05:42.632  INFO 1839 --- [           main] c.a.s.c.context.core.impl.AzureManager   : Fetching EventHubNamespace with name 'event-hub-spring-cloud-test' ...
2021-01-17 22:05:42.673  INFO 1839 --- [           main] c.a.s.c.context.core.impl.AzureManager   : Fetching EventHubNamespace with name 'event-hub-spring-cloud-test' finished in 0 seconds
2021-01-17 22:05:42.674  INFO 1839 --- [           main] c.a.s.c.context.core.impl.AzureManager   : Fetching EventHub with name 'toUpperCase-out-0' ...
2021-01-17 22:05:43.603  INFO 1839 --- [           main] c.a.s.c.context.core.impl.AzureManager   : Fetching EventHub with name 'toUpperCase-out-0' finished in 0 seconds
2021-01-17 22:05:43.603  INFO 1839 --- [           main] c.a.s.c.context.core.impl.AzureManager   : Creating EventHub with name 'toUpperCase-out-0' ...
2021-01-17 22:05:46.063  INFO 1839 --- [           main] c.a.s.c.context.core.impl.AzureManager   : Creating EventHub with name 'toUpperCase-out-0' finished in 2 seconds
2021-01-17 22:05:46.066  INFO 1839 --- [           main] c.a.s.i.c.a.r.DefaultMessageHandler      : DefaultMessageHandler sync becomes: false
2021-01-17 22:05:46.066  INFO 1839 --- [           main] c.a.s.i.c.a.r.DefaultMessageHandler      : DefaultMessageHandler syncTimeout becomes: ValueExpression [value=10000]
2021-01-17 22:05:46.067  INFO 1839 --- [           main] c.a.s.i.c.a.r.DefaultMessageHandler      : Started DefaultMessageHandler with properties: {sendTimeout=ValueExpression [value=10000], destination=toUpperCase-out-0, sync=false}
2021-01-17 22:05:46.067  INFO 1839 --- [           main] o.s.c.s.m.DirectWithAttributesChannel    : Channel 'application.toUpperCase-out-0' has 1 subscriber(s).
2021-01-17 22:05:46.068  INFO 1839 --- [           main] o.s.c.s.binder.DefaultBinderFactory      : Retrieving cached binder: eventhub
2021-01-17 22:05:46.083  INFO 1839 --- [           main] c.a.s.c.context.core.impl.AzureManager   : Fetching EventHubNamespace with name 'event-hub-spring-cloud-test' ...
2021-01-17 22:05:46.150  INFO 1839 --- [           main] c.a.s.c.context.core.impl.AzureManager   : Fetching EventHubNamespace with name 'event-hub-spring-cloud-test' finished in 0 seconds
2021-01-17 22:05:46.150  INFO 1839 --- [           main] c.a.s.c.context.core.impl.AzureManager   : Fetching EventHub with name 'toUpperCase-in-0' ...
2021-01-17 22:05:47.081  INFO 1839 --- [           main] c.a.s.c.context.core.impl.AzureManager   : Fetching EventHub with name 'toUpperCase-in-0' finished in 0 seconds
2021-01-17 22:05:47.086 ERROR 1839 --- [           main] o.s.cloud.stream.binding.BindingService  : Failed to create consumer binding; retrying in 30 seconds
org.springframework.cloud.stream.provisioning.ProvisioningException: Event hub with name 'toUpperCase-in-0' in namespace 'event-hub-spring-cloud-test' not existed
at com.azure.spring.eventhub.stream.binder.provisioning.EventHubChannelResourceManagerProvisioner.validateOrCreateForConsumer(EventHubChannelResourceManagerProvisioner.java:35) ~[azure-spring-cloud-stream-binder-eventhubs-2.0.0-beta.1.jar:na]
at com.azure.spring.eventhub.stream.binder.provisioning.EventHubChannelProvisioner.provisionConsumerDestination(EventHubChannelProvisioner.java:32) ~[azure-spring-cloud-stream-binder-eventhubs-2.0.0-beta.1.jar:na]
at com.azure.spring.eventhub.stream.binder.provisioning.EventHubChannelProvisioner.provisionConsumerDestination(EventHubChannelProvisioner.java:18) ~[azure-spring-cloud-stream-binder-eventhubs-2.0.0-beta.1.jar:na]
at org.springframework.cloud.stream.binder.AbstractMessageChannelBinder.doBindConsumer(AbstractMessageChannelBinder.java:402) ~[spring-cloud-stream-3.0.8.RELEASE.jar:3.0.8.RELEASE]
at org.springframework.cloud.stream.binder.AbstractMessageChannelBinder.doBindConsumer(AbstractMessageChannelBinder.java:90) ~[spring-cloud-stream-3.0.8.RELEASE.jar:3.0.8.RELEASE]
at org.springframework.cloud.stream.binder.AbstractBinder.bindConsumer(AbstractBinder.java:143) ~[spring-cloud-stream-3.0.8.RELEASE.jar:3.0.8.RELEASE]
at org.springframework.cloud.stream.binding.BindingService.doBindConsumer(BindingService.java:176) [spring-cloud-stream-3.0.8.RELEASE.jar:3.0.8.RELEASE]
at org.springframework.cloud.stream.binding.BindingService.bindConsumer(BindingService.java:133) [spring-cloud-stream-3.0.8.RELEASE.jar:3.0.8.RELEASE]
at org.springframework.cloud.stream.binding.AbstractBindableProxyFactory.createAndBindInputs(AbstractBindableProxyFactory.java:112) [spring-cloud-stream-3.0.8.RELEASE.jar:3.0.8.RELEASE]
at org.springframework.cloud.stream.binding.InputBindingLifecycle.doStartWithBindable(InputBindingLifecycle.java:58) [spring-cloud-stream-3.0.8.RELEASE.jar:3.0.8.RELEASE]
at java.util.LinkedHashMap$LinkedValues.forEach(LinkedHashMap.java:608) ~[na:1.8.0_152]
at org.springframework.cloud.stream.binding.AbstractBindingLifecycle.start(AbstractBindingLifecycle.java:57) ~[spring-cloud-stream-3.0.8.RELEASE.jar:3.0.8.RELEASE]
at org.springframework.cloud.stream.binding.InputBindingLifecycle.start(InputBindingLifecycle.java:34) [spring-cloud-stream-3.0.8.RELEASE.jar:3.0.8.RELEASE]
at org.springframework.context.support.DefaultLifecycleProcessor.doStart(DefaultLifecycleProcessor.java:182) ~[spring-context-5.2.12.RELEASE.jar:5.2.12.RELEASE]
at org.springframework.context.support.DefaultLifecycleProcessor.access$200(DefaultLifecycleProcessor.java:53) ~[spring-context-5.2.12.RELEASE.jar:5.2.12.RELEASE]
at org.springframework.context.support.DefaultLifecycleProcessor$LifecycleGroup.start(DefaultLifecycleProcessor.java:360) ~[spring-context-5.2.12.RELEASE.jar:5.2.12.RELEASE]
at org.springframework.context.support.DefaultLifecycleProcessor.startBeans(DefaultLifecycleProcessor.java:158) ~[spring-context-5.2.12.RELEASE.jar:5.2.12.RELEASE]
at org.springframework.context.support.DefaultLifecycleProcessor.onRefresh(DefaultLifecycleProcessor.java:122) ~[spring-context-5.2.12.RELEASE.jar:5.2.12.RELEASE]
at org.springframework.context.support.AbstractApplicationContext.finishRefresh(AbstractApplicationContext.java:895) ~[spring-context-5.2.12.RELEASE.jar:5.2.12.RELEASE]
at org.springframework.context.support.AbstractApplicationContext.refresh(AbstractApplicationContext.java:554) ~[spring-context-5.2.12.RELEASE.jar:5.2.12.RELEASE]
at org.springframework.boot.SpringApplication.refresh(SpringApplication.java:758) ~[spring-boot-2.3.7.RELEASE.jar:2.3.7.RELEASE]
at org.springframework.boot.SpringApplication.refresh(SpringApplication.java:750) ~[spring-boot-2.3.7.RELEASE.jar:2.3.7.RELEASE]
at org.springframework.boot.SpringApplication.refreshContext(SpringApplication.java:405) ~[spring-boot-2.3.7.RELEASE.jar:2.3.7.RELEASE]
at org.springframework.boot.SpringApplication.run(SpringApplication.java:315) ~[spring-boot-2.3.7.RELEASE.jar:2.3.7.RELEASE]
at org.springframework.boot.SpringApplication.run(SpringApplication.java:1237) ~[spring-boot-2.3.7.RELEASE.jar:2.3.7.RELEASE]
at org.springframework.boot.SpringApplication.run(SpringApplication.java:1226) ~[spring-boot-2.3.7.RELEASE.jar:2.3.7.RELEASE]
at com.spring.cloudstream.CloudStreamApplication.main(CloudStreamApplication.java:13) ~[classes/:na]
To Reproduce
Steps to reproduce the behavior:
Use below sample project with Azure event hub (From Spring One conference 2019)
https://github.com/schacko-samples/s1p-2019-scst-kafka-talk/tree/master/hello-streams-application
I have used below Microsoft document to create the event hub example and worked fine
https://docs.microsoft.com/en-us/azure/developer/java/spring-framework/configure-spring-cloud-stream-binder-java-app-azure-event-hub
But not working with latest spring cloud stream code, I'm using latest version from azure which has latest spring cloud stream dependency

com.microsoft.azure
spring-cloud-azure-eventhubs-stream-binder
2.0.0-beta.1

Code Snippet
application.properties used:
spring.cloud.azure.credential-file-path=my.azureauth
spring.cloud.azure.resource-group=spring-cloud-test
spring.cloud.azure.AzureEnvironment=Azure
spring.cloud.azure.region=southcentralus
spring.cloud.azure.auto-create-resources=true
spring.cloud.azure.eventhub.namespace=event-hub-spring-cloud-test
spring.cloud.azure.eventhub.checkpoint-storage-account=springcloudteststorage
spring.cloud.stream.bindings.input.group=$Default
spring.cloud.stream.eventhub.bindings.input.consumer.checkpoint-mode=MANUAL
Expected behavior
I should be able to test the event hub with latest spring cloud stream dependency

org.springframework.cloud
spring-cloud-stream
3.0.8.RELEASE

Screenshots
If applicable, add screenshots to help explain your problem.
Setup (please complete the following information):

OS: iOS
IDE : IntelliJ
Version of the Library used
spring-cloud-stream (3.0.8.RELEASE)
spring-cloud-azure-eventhubs-stream-binder (2.0.0-beta.1)

Additional context
Try to avoid boiler plate code and want to see if I can avoid source and sink configuration
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[ ] Bug Description Added
[ ] Repro Steps Added
[ ] Setup information Added

@jialindai could you please help route this issue?

@jialindai could you please help route this issue?

Hi @pdp5210, sorry for the late reply. are you using the functional programming model? From this 'toUpperCase-in-0, it's the default binding name for an event hub consumer. You could follow this sample, we've just applied the functional programming model for it. https://github.com/Azure/azure-sdk-for-java/tree/master/sdk/spring/azure-spring-boot-samples/azure-spring-cloud-sample-eventhubs-binder

Thanks Dai for the response, will try out the sample and will update you.
Thanks,
Pradeep
On Mon, Feb 1, 2021 at 2:41 AM Xiaolu Dai<EMAIL_ADDRESS>wrote:

Hi @pdp5210 https://github.com/pdp5210, sorry for the late reply. are
you using the functional programming model? From this 'toUpperCase-in-0,
it's the default binding name for an event hub consumer. You could follow
this sample, we've just applied the functional programming model for it.
https://github.com/Azure/azure-sdk-for-java/tree/master/sdk/spring/azure-spring-boot-samples/azure-spring-cloud-sample-eventhubs-binder
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Azure/azure-sdk-for-java/issues/18658#issuecomment-770679577,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEIEDSDXLQFBWPWMKYNM23LS4ZSL5ANCNFSM4WGW2YVA
.

Good afternoon,
Is there an option to download the specific Azure maven dependency when
running the sample? My organization has an artifact repository which allows
to download the maven central if not found and azure sdk repositories are
not listed. It worked out fine in my personal laptop as I built the
complete azure java sdk before running the sample. Please suggest
Sample: azure-spring-cloud-sample-eventhubs-binder, it's failing to
download below dependencies also attached the error in below email

com.azure.spring
azure-spring-cloud-stream-binder-eventhubs
2.2.0-beta.1

com.azure.spring
azure-spring-cloud-storage
2.2.0-beta.1

[ERROR] Failed to execute goal on project
azure-spring-cloud-sample-eventhubs-binder: Could not resolve dependencies
for project com.azure.spring:azure-spring-cloud-sample-eventhubs-bi
nder:jar:1.0.0: The following artifacts could not be resolved:
com.azure.spring:azure-spring-cloud-stream-binder-eventhubs:jar:2.2.0-beta.1,
com.azure.spring:azure-spring-cloud-storage
:jar:2.2.0-beta.1: Could not find artifact
com.azure.spring:azure-spring-cloud-stream-binder-eventhubs:jar:2.2.0-beta.1
in XXXXXXX (organization artifact repo)
Thanks,
Pradeep
On Wed, Feb 3, 2021 at 4:30 PM Pradeep N<EMAIL_ADDRESS>wrote:

Thanks Dai for the response, will try out the sample and will update you.
Thanks,
Pradeep
On Mon, Feb 1, 2021 at 2:41 AM Xiaolu Dai<EMAIL_ADDRESS>wrote:

Hi @pdp5210 https://github.com/pdp5210, sorry for the late reply. are
you using the functional programming model? From this 'toUpperCase-in-0,
it's the default binding name for an event hub consumer. You could follow
this sample, we've just applied the functional programming model for it.
https://github.com/Azure/azure-sdk-for-java/tree/master/sdk/spring/azure-spring-boot-samples/azure-spring-cloud-sample-eventhubs-binder
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Azure/azure-sdk-for-java/issues/18658#issuecomment-770679577,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEIEDSDXLQFBWPWMKYNM23LS4ZSL5ANCNFSM4WGW2YVA
.

Hi @pdp5210, you could use this version instead https://github.com/Azure/azure-sdk-for-java/blob/azure-spring-cloud-stream-binder-servicebus-queue_2.1.0/sdk/spring/azure-spring-boot-samples/azure-spring-cloud-sample-eventhubs-binder/pom.xml#L32-L42.

just realized the example shared was azure-spring-cloud sample, do you have an example where i can use azure event hub or azure event hub with kafka under azure spring boot samples. My example runs on AKS on a container and i dont need azure spring cloud rather a spring boot sample with eventhub.
https://github.com/Azure/azure-sdk-for-java/tree/azure-spring-cloud-stream-binder-servicebus-queue_2.1.0/sdk/spring/azure-spring-boot-samples
Appreciate your help.
Thanks,
Pradeep

I'm assuming below examples should work out fine with latest functional code similar to Spring cloud stream example
https://docs.microsoft.com/en-us/azure/developer/java/spring-framework/configure-spring-cloud-stream-binder-java-app-azure-event-hub#add-sample-code-to-implement-basic-event-hub-functionality
https://docs.microsoft.com/en-us/azure/developer/java/spring-framework/configure-spring-cloud-stream-binder-java-app-kafka-azure-event-hub

Closing this issure because no response from the customer.


Migrate to Github Actions

Resolves #484, #486
Migrates CI to GitHub actions.
TODO:

[ ] Update .scripts/update-gh-pages to work with github actions

Codecov Report

Merging #486 (b02f681) into master (95b81b7) will decrease coverage by 98.87%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #486       +/-   ##
==========================================
- Coverage   98.87%   0.00%   -98.88%
==========================================
  Files           6       6
  Lines         177     175        -2
==========================================
- Hits          175       0      -175
- Misses          2     175      +173

Impacted Files
Coverage Δ

ReSwift/CoreTypes/Store.swift
0.00% <0.00%> (-100.00%)
:arrow_down:

ReSwift/Utils/TypeHelper.swift
0.00% <0.00%> (-100.00%)
:arrow_down:

ReSwift/Utils/Synchronized.swift
0.00% <0.00%> (-100.00%)
:arrow_down:

ReSwift/CoreTypes/Subscription.swift
0.00% <0.00%> (-100.00%)
:arrow_down:

ReSwift/CoreTypes/StoreSubscriber.swift
0.00% <0.00%> (-100.00%)
:arrow_down:

ReSwift/Utils/Assertions.swift
0.00% <0.00%> (-71.43%)
:arrow_down:

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 3825d44...b02f681. Read the comment docs.

@DivineDominion The update docs script appears to be heavily dependent on travis-ci environment variables. Haven't looked closely but I'm assuming it will need modifications. Will take a look today.

Codecov Report

Merging #486 (87cf01d) into master (95b81b7) will increase coverage by 0.09%.
The diff coverage is n/a.

:exclamation: Current head 87cf01d differs from pull request most recent head 5a17f3c. Consider uploading reports for the commit 5a17f3c to get more accurate results

@@            Coverage Diff             @@
##           master     #486      +/-   ##
==========================================
+ Coverage   98.87%   98.96%   +0.09%
==========================================
  Files           6        6
  Lines         177      193      +16
==========================================
+ Hits          175      191      +16
  Misses          2        2

Impacted Files
Coverage Δ

ReSwift/CoreTypes/Store.swift
100.00% <0.00%> (ø)

ReSwift/Utils/TypeHelper.swift
100.00% <0.00%> (ø)

ReSwift/CoreTypes/Subscription.swift
100.00% <0.00%> (ø)

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 3825d44...5a17f3c. Read the comment docs.


Support the format preamble format option in swc.minify

Describe the feature
See https://swc.rs/docs/configuration/minification#jscminifyformat

preamble, Currently noop

It should be possible to pass preamble. SWC should be able to insert it to the minified file without breaking source map associations.
Babel plugin or link to the feature description
No response
Additional context
FWIW, jQuery is currently switching from UglifyJS to Terser in https://github.com/jquery/jquery/pull/5258. We considered switching to SWC instead as it's way faster and optimizes better than Terser but the lack of the preamble option is a blocker for us at the moment.

@kdy1 Is this already in progress? If it's not urgent, maybe I can submit a PR next week?

Not yet, and it would be so great. Thank you in advance!


Create pre configure Vault with CLI

Is your feature request related to a problem? Please describe.
Nope
Describe the solution you'd like
Vaults can be created and, like vscode, a customized list of plugins can be installed, as well as the default configuration can be modified
Or execute some additional shell commands to complete the auxiliary
This can all be done via configuration files。

Thanks for the feedback but I an not sure I understand it. This sounds like a feature request to add a command to create a vault, install plugins etc. But unfortunately that is not possible with this CLI as its built on top of Obsidian URIs that only have actions for interacting with notes


Incorrect time range when choose "last 1hour" in grafana

Incorrect time range
Local browser TIMEZONE is GMT+3 (2020-11-13 02:45:41)
When time range picked from calendar in grafana using "from day to day", pickups correct date/timestamp
Grafana v7.1.3 (5723d951af)

SELECT toDateTime(1604177999) AS first_date

Query id: aba7fda3-8bd4-44b3-b3d4-019ff2ecfb77

┌──────────first_date─┐
│ 2020-10-31 20:59:59 │
└─────────────────────┘

1 rows in set. Elapsed: 0.002 sec.

SELECT toDateTime(1604350800) AS second_date

Query id: f514acaa-62e7-4d76-9b99-4b5b602c149b

┌─────────second_date─┐
│ 2020-11-02 21:00:00 │
└─────────────────────┘

1 rows in set. Elapsed: 0.002 sec.

@DefauIt could you answer, which time zone you have on your clickhouse server?
also, please disable the Extrapolate checkbox in your query editor to skip default extrapolation behavior, maybe it will solve your issue, cause plugin always pass time range as unix timestamp uint32 value

Disabling the Extrapolate checkbox in query editor didn't solved.
clickhouse-server timezone:
$ ls -l /etc/localtime
lrwxrwxrwx 1 root root 33 Jan 12  2020 /etc/localtime -> /usr/share/zoneinfo/Europe/Moscow

@DefauIt I can't reproduce this behavior on my local environment
let's try to describe steps to reproduce?
could you share CREATE TABLE statement for your table and smile dashboard JSON from your environment?

with interpolation disabled, the query works correctly, thanks.


Phase 6: Testing & Merging Code

For Testing
We would want to use pytest to cover as much functionality and code as possible and include the documentation about how to run the tests.

Tests code creation

[x] Add a suite of unit tests to repo under tests folder/directory using the pytest framework, for all functions.

Edit the top-level README.md file to include documentation for end users
Include for end users:

[x] setup/installation instructions, including information about the expected input file formats
[x] examples

Edit the top-level README.md file to include documentation for contributors:

[x] local testing instructions

For Merging Code

Merge different functions into one or two class objects

Can be based on recommender type or just one class with different recommenders as attributes

Code Review

When we have done with one function or feature, create a PR and assign the other two as code reviewers. After both approvals, we merge the specific branch to Main

We have created, updated, and made test files for parsing data and popularity recommender.


Export Ecto.Adapters.SQL.query/query! directly in SQL repositories

Should this be a general additional callback for adapters?
I would also consider adding something like Repo.query("...", load_into: MyStruct). Today executing the query is not that hard, what is hard is processing the result.

Hey @josevalim, I've got some extra time at work to help out with OS. Any pointers for getting started on this?

So instead of calling
Ecto.Adapters.SQL.query(repo, sql, params, opts \\ [])
The use Ecto.Repo, otp_app: :my_app macro will add the query function directly to the repo module?

I think I can work on this

Yes, please!


DELETE POSITION failed with 400 Status Code

Very Nice Job, and clean code - This is the first time, i saw Katlin code.
However, i have a Problem with Closing Position - DELETE Request:

request: https://www.etoro.com/sapi/trade-demo/positions/1683586719?PositionID=1683586719&client_request_id=74dbbd5f-9550-4da8-9889-88e08107ce1d
i am getting always a repsone code - 400
i tried to print the response.body() - but apparently, no body is specified.

thanks for pointing this out,
it seems they made request body required for this request, I will update my code

just made a hotfix, should work now

Thanks alot, it works great


Add task ref snippet when editing pipelines

Aftring generating a YAML skeleton for a Pipeline, it would be useful if the extension assist the user to generate task reference snippets to add to the tasks attribute of the pipeline, based on the Tasks that are installed in the namespace and the ClusterTasks that are available on the cluster.
Similar capability is available in the web console based on an extension CR:
https://github.com/openshift/tektoncd-pipeline-operator/pull/127/files

Fixed with https://github.com/redhat-developer/vscode-tekton/pull/138


linux: enable SND_OSSEMUL

Motivation for this change
Fixes #54074
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)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Tested simply using:
import ./make-test.nix ({ pkgs, ...} :
{
  name = "alsa-oss";
  nodes.server.sound.enable = true;
  testScript = ''
    $server->waitForUnit("multi-user.target");
    $server->fail("journalctl -b | grep \"Failed to find module 'snd_pcm_oss'\"");
  '';
})

Although, should this go to staging?

No, we merge kernel-only changes directly to master.  It's not that much to rebuild in practice.

I've checked that a number of kernels: linux linux_latest linux-libre linux_4_14 linux_4_19 linux_4_4 linux_4_9 linux_hardened still build with this change, so I'll go ahead and merge.


 However, the pre-loaded library constructor is never called. We can interpose, but we aren't emitting any detail.
The root cause for this behavior is a subtle difference between the ld-musl.so and ld-linux.so (glibc) dynamic linker behavior. glibc seems to call shared object constructors in dynamic linker context (_dl_start_user), while musl libc executes constructors in __libc_start_main called by crt1.c. The musl libc dynamic linker loads the application along with preloaded and dependent shared objects. Then it jumps to the application entry point. Typically, the application entry point is code defined in crt1.c, when linked with a musl compatible tool chain. But, in the case when the application has been linked with the internal Go linker,  __libc_start_main is never invoked. The internal Go linker does not link against crt1.o and just calls the main function directly from the startup code in the executable.

Go Dynamic Execs
Given that the library is loaded and preload is possible, an approach could be to interpose a function that the Go runtime calls during it's init and call the library constructor directly.
With experimentation, it can be seen that the Go runtime calls pthread_create() as part of init. Interposing pthread_create() and calling the constructor, ensuring that it's only called once, appears to work. We see Go functions interposed and data is emitted.
As part of library init we get the process name. Both glibc and libmusl use the global variable program_invocation_short_name. However, since we are early in the init procedure or because lib constructors weren't called, the variable isn't initialized when we need it. So, we drop back to getting the exe path and basename() the string to get the process name.

Go Static Execs on Musl
Go static execs segfault when started with ldscope or preloaded.
It turns out that init of the function pointer object g_fn fails. The use of dlsym() with the handle RTLD_NEXT has been the default for all cases. However, the libmusl dlsym() doesn't resolve symbols using RTLD_NEXT when the lib is loaded from dlopen(); the case for Ge static execs. It sort of makes sense in that the handle indicates the use of the next objects symbol table. It's actually not clear in the man page how this is supposed to work. It appears that glibc interprets RTLD_NEXT behavior more broadly than libmusl.
The use of RTLD_DEFAULT works. The symbols for the function pointer object are resolved in the Go static case on libmusl. Added a get handle function to determine which handle to use when populating the functions. Changed several places where we were using internal function pointers to use the common function pointer object.

Libmusl Doesn't Support Direct Execution
A glibc shared lib can be executed from the command line. It's a bit of hackery to get this to work. It's a nice option. It's used in libscope to emit help and config messages. Not all loaders support this model. ld-musl.so does not.
Might need to move the help and config output to ldscope.

No libresolv in a Musl Distro
We use libresolve to parse DNS answer packets. Looks like this will need to be modified without the use of libresolve.

Console I/O and libmusl
On glibc we had to hook an internal function in order to ensure we emitted all console I/O. The same function doesn't exist in libmusl. Depending on the public interfaces is better in libmusl, but it appears that there is missing console detail. Will need to look into libmusl code.

single lib built on ubuntu runs on Alpine
ldscope extracts ldscopedyn, ldscopedyn extracts libscope.so
inspect .interp for loader strings to determine glibc vs libmusl
update the loader string in .interp for glibc exec on libmusl
create a soft link from glibc loader to musl loader
create a dir for extraction & sym link using a libscope version string
set ld lib path env var in the proper case/scenario
go dynamic, static and cgo exec work and emit expected events
removal of dependency on libresolv
DNS events are emitted as expected
no change to interposed funcs remove for glibc __write
all help text is handled in ldscopedyn, not in os.c
no lib main (help text moved)
dependency on dl_argv removed
shm_open issues resolved
all console writes, stdout, stderr, stream and direct I/O, captured
command line switch added to ldscope to optionally specify a sym link dir
removed dependency on linking with crt0/crt1
added interposition of pthread_create to enable Go apps to init libscope.so
remove dependency on program_invocation_shortname being init'd at constructor time
ensure that multiple g_fn objects exist in multiple execs
unit tests all run
/proc/pid/maps format is different using ld-musl. when running with Go dynamic execs the extraction of the base address needs to change.
creation of auxv is different when run with ld-musl. the use of an environ global variable is updated in glibc, but not in libmnusl. need to use a calculated address for aux location.
go_switch segfaults on cgo static execs. turns out that the Go tools or runtime on Alpine do not update a TLS value in an 'm' object. added code to validate each object and fall back to getting the 'fs' value from the kernel if an object can not be validated.
6+ additional symbols are included in libscope from the addition of libssl and libcrypto. need to interpose these and call the appropriate code when run with libmusl.
elf header details change between a dynamic exec and a dynamic shared object. this turns up when updating the loader string on older distros. found this on an Debian 8 distro. we need to reference the .interp section from ph_offset and not ph_v addr.