print_tables and print_bars throws errors on Windows

I'm actively looking for a solution to this, but I wanted to log it in here so you were aware: On Windows, when users using Jupyter Notebooks try to print_table or print_bars with Agate, they get an error message that traces back to Babel. From a little scanning around, it seems that Babel is pretty insistent upon unicode and Windows command prompt is not unicode by default, so that may be the source of the issue. Here's the stack trace on the error message that is output by Jupyter Notebooks if anyone has any ideas. I'll report back on what switching to UTF-8 does or if something like Windows Powershell fixes the problem.

AttributeError                            Traceback (most recent call last)
 in ()
----> 1 big10.print_table()
C:\Users\DroneLab\Anaconda3\lib\site-packages\agate\table.py in print_table(self, max_rows, max_columns, output)
1190             A file-like object to print to. Defaults to :code:sys.stdout.
1191         """
-> 1192         print_table(self, max_rows, max_columns, output)
1193
1194     def print_html(self, max_rows=None, max_columns=None, output=sys.stdout):
C:\Users\DroneLab\Anaconda3\lib\site-packages\agate\preview.py in print_table(table, max_rows, max_columns, output)
84                 v = ''
85             elif number_formatters[j] is not None:
---> 86                 v = format_decimal(v, format=number_formatters[j])
87             else:
88                 v = six.text_type(v)
C:\Users\DroneLab\Anaconda3\lib\site-packages\babel\numbers.py in format_decimal(number, format, locale)
253         format = locale.decimal_formats.get(format)
254     pattern = parse_pattern(format)
--> 255     return pattern.apply(number, locale)
256
257
C:\Users\DroneLab\Anaconda3\lib\site-packages\babel\numbers.py in apply(self, value, locale, currency)
654             b = b or '0'
655             a = self._format_int(a, self.int_prec[0],
--> 656                                  self.int_prec[1], locale)
657             b = self._format_frac(b, locale)
658             number = a + b
C:\Users\DroneLab\Anaconda3\lib\site-packages\babel\numbers.py in _format_int(self, value, min, max, locale)
698         gsize = self.grouping[0]
699         ret = ''
--> 700         symbol = get_group_symbol(locale)
701         while len(value) > gsize:
702             ret = symbol + value[-gsize:] + ret
C:\Users\DroneLab\Anaconda3\lib\site-packages\babel\numbers.py in get_group_symbol(locale)
206     :param locale: the Locale object or locale identifier
207     """
--> 208     return Locale.parse(locale).number_symbols.get('group', u',')
209
210
AttributeError: 'NoneType' object has no attribute 'number_symbols'

Matt, I have a suspicion that your having the same issue tracked:
http://babel.edgewall.org/ticket/98
(Also here.)
Can you try setting that environment variable and see if that fixes things? (It would also be good to know if this var is set by default in Powershell, et al.)

I think I've found a temporary solution:
By using the bash shell included with the standard Git installation, I can make Matt's command run in Jupyter (or directly via the python interpreter).
The whole thing is running through Python 3.5.1 :: Anaconda 2.4.1, without activating a conda environment.
I'm unable to activate/source activate into any environment other than root. I'll have to tinker around with my ~/.bashrc profile to activate those properly.
Because python isn't used by any system-critical Windows scripts, it could be considered "safe" to run without switching to an environment.
This isn't ideal, but it works.

Can confirm Tony's workaround. I've tried setting the system locale, I've set it explicitly it in the basic command prompt and in Powershell, but when running it in the Anaconda environment, it breaks. So somewhere along the way, the environment variable is getting stripped (by Anaconda, by Jupyter, by Windows, by the shell I'm using? Don't know.) That Git Bash works makes me think Cygwin or any other Unix like terminal emulator will too. This will get me through today's class. I'll keep working on a solution and report back. This doesn't seem to be an Agate issue, so I'll close this.

@tonypapousek @mattwaite Thanks to both of you for reporting. I'm going to reopen this to say if there is way I can opt into a safer default. (utf8 if not specified)

I can report that today I saw this same error trip up one of our Windows users when they tried a print_table. Sad to say it's blocking their introduction to agate entirely. Is there a way to turn off babel somehow or pass in a default manually?

After a little study, I suspect an experimentation on a Windows shell with the bug, I suspect that passing in the locale keywoard argument explicitly here with the Number objects setting can fix the problem.
Within babel, that keyword argument is set to a global default that is sniffed off the system. On the Windows machine I was using, the sniffer returned a null value and I got the same error as Matt. When I substituted en_US as the override everything worked fine.
So I hope if we can get the Number to pass in its self.locale attribute the linked line above we'll be back in business.

Yeah, I think using the locale specified on the Number is correct. There might be a way to shoehorn this into make_number_formatter, but it's not obvious to me. I think instead I'd just add...
locale = table.column_types[j].locale

...on the line before. And then pass that into format_decimal. Seems like it should work to me.
We also need to fix it in print_bars and grep the code for any other occurrences of format_decimal that might have the same problem.

Ah ha, I see now you had already beat me to this. Woops.

Fixed for print_table. Still need to fix for print_bars.

The print_bars call in example.py does now work for me with the fix in #488 but I can't speak for others.

Ah my bad, I glanced at the patch and didn't see edits to print_bars, but I was just scanning too quickly. Thanks!

No problem. The key change we made was that new global call at the top that falls back to default when Windows shits out.


Fixes #1139 - Add ElementHandle.select method.

@austinkelleher Thanks for the PR.
I'd rather have Frame.select then ElementHandle.select.
Also, there's in-flight #1099; let's land that first so that there are no conflicts.

Isn't redundant to have selector argument in elementHandle.select method?
page.$('select#colors').select('select#colors', 'blue');

Closing in favor of  #1243


Default console area seem too large

See like an excessive amount of space for the console area requiring scroll bars without anything displaying.

Wrong area; sorry.


Release Test Updates

Updating the readme to reflect the changes in the test app and general system behavior
Removing run levels from the test application
Updating the test app to use Rust 2018

 Review our release integration test procedures


Will Nuxt release an app router similar to Next in the future?

Now the page and the components it needs to use are separated into multiple directories, as well as APIs. This is difficult to manage
If the Vue component is written into pages, it will be treated as a page router.

If the Vue component is written into pages, it will be treated as a page router.  already like that please check i guest. this https://nuxt.com/docs/guide/directory-structure/pages  will help you

It's possible to use layers to separate bits of your app into self contained Nuxt projects (with components/pages directories of their own).
Or you can use an ignore prefix to ignore directories within pages/
See https://www.youtube.com/watch?v=fr5yo3aVkfA and https://deltener.com/blog/nuxt-enterprise-patterns-component-management/

It's possible to use layers to separate bits of your app into self contained Nuxt projects (with components/pages directories of their own).
Or you can use an ignore prefix to ignore directories within pages/
See https://www.youtube.com/watch?v=fr5yo3aVkfA and https://deltener.com/blog/nuxt-enterprise-patterns-component-management/
Beyond that, there doesn't currently seem a need for more, but please let me know if you have a need Nuxt isn't meeting for you 🙏

I want to write the components that only need to be used once near page. vue for easy code organization. It seems that layer cannot meet this requirement, right? Is layer used as a template to inherit the project?

If the Vue component is written into pages, it will be treated as a page router. 已经这样了，请查看文档 https://nuxt.com/docs/guide/directory-structure/pages 将帮助你

Are you referring to layers? Is that convenient for organizing multiple projects? But it is not convenient to organize the components used for individual pages.

It's possible to use layers to separate bits of your app into self contained Nuxt projects (with components/pages directories of their own).
Or you can use an ignore prefix to ignore directories within pages/
See https://www.youtube.com/watch?v=fr5yo3aVkfA and https://deltener.com/blog/nuxt-enterprise-patterns-component-management/
Beyond that, there doesn't currently seem a need for more, but please let me know if you have a need Nuxt isn't meeting for you 🙏

I can automatically import components, but I cannot delete this page.

Please read the second link I provided. I think it is exactly what you are asking for.

Please read the second link I provided. I think it is exactly what you are asking for.

See, will this increase the loss of operational performance? It would be even better if nuxt could provide a built-in feature.


Schedule window should contain hint to latitude and longitude formats.

It is pretty hard to guess the latitude and longitude format in the Schedule input window if no other entries are visible. Add format hints in input field.

Angular does not have a x-ng-placeholder directive supporting conditional rendering of text. This is not the moment to introduce a 3rd party Angular directive to support this functionality. Solution to improvement postponed.


Fastnlight: Blacklist +1336-441-4118

Fastnlight requests the blacklist of the number +1336-441-4118. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by Jeff Schaller in Charcoal HQ


Added Canvas Viewport

Canvas is 100 x 100, but fits workspace now.

I messed something up with my branches, and I need to fix that first.


Draft orders - Go to order button has wrong link

Draft Order -> mark as paid -> Go to order (top)
This button has a "}" appended to the end of the link

ie "/a/orders/order_01FXC37KQENJK8E3CC9E9B47HW}"

Solved in #603


Add HTML autocomplete to backticks

Following up on https://github.com/TypeStrong/atom-typescript/issues/221 (shout out to @jbarket for implementing this!), it would be great to also add support for HTML element autocompletion. Perhaps this is only relevant for use with Angular, or there could be an option to toggle this feature.
Currently, it has autocomplete options but pretends to be in the global typescript context. This can lead to confusing autocomplete suggestions such as:

(Note how the term "directives" is used as part of Typescript in line 27 and therefore suggested in line 33 even though we're in HTML land at that point.)
Ideally the autocompletion would also auto-close tags, tab through attributes, etc, just like in any other .html file.

Emmet support with inline templates would be even more great

@Emnalyeriar, I actually have this setup in mine. One of the cool things about the Emmet package is that you can easily tell it other grammars to expand in. I have this in my keymap.cson:
'atom-text-editor[data-grammar="source ts"]:not([mini])':
'tab': 'emmet:expand-abbreviation-with-tab'
It'd be nice to find a way to only make Emmet work inside of the template tags, but I'm not sure if that's possible without actually adding something specific to the Emmet package itself.

@jbarket Thanks I'll try this!

Hi, for me it doesn't work :/
keymap :
`
{
"48": {
"alted": 64
},
"51": {
"accent": true,
"alted": 35
},
"52": {
"alted": 123
},
"53": {
"alted": 91
},
"54": {
"alted": 124
},
"56": {
"accent": true,
"alted": 92
},
"57": {
"alted": 94
},
"219": {
"alted": 93
},
'atom-text-editor[data-grammar="source ts"]:not([mini])':
'tab': 'emmet:expand-abbreviation-with-tab'
}
`

there is greate extension for that https://marketplace.visualstudio.com/items?itemName=jakethashi.vscode-angular2-emmet


Default question fails

Describe the bug
The default "What does this repo do?" question never gives a correct answer. Following questions work as expected.
Expected behavior
An answer that's not a "Sorry". :)
To Reproduce

Click on any repo.
Wait for the answer to the default "What does this repo do?" question.

Screenshots or output

Additional context

Using latest macOS bloop build available (v0.3.1)
Using both local (2x) and GitHub (1x) repositories.

Hey @ramnes, do the repos in question contain any markdown files? The initial search here is constrained to markdown files, and if none are found, we emit this message. If they do contain markdown files can you reproduce the issue on v0.3.2?

Yep, it does: https://github.com/ramnes/mongo-thingy
Yes, happens on v0.3.2.
That being said, it might be linked to issue #311, because this is the repository I tried to remove (and that still appears in Bloop).
I may have concluded too early that it happens all the time because I had the same message on another repository that indeed doesn't have any Markdown file. Maybe another question should be asked if no Markdown file exist in the repository. :)

The issue can be reproduced in v0.3.3

Using latest macOS bloop build available (v0.3.3)

Reproduction steps:

Clone https://github.com/centrifugal/centrifuge to local
Import
Ask this question: What does this repo do? repo:centrifuge lang:markdown

@ramnes @hanxuanliang I managed to reproduce this, but unfortunately not reliably. Deleting my repositories and re-indexing them from the user interface seems to have fixed the issue, though.
We're planning a new release from main soon which will fix issues with deleting repositories, so will be easier to pin down what's wrong.

This default question has been removed as of v0.4.0: https://github.com/BloopAI/bloop/releases/tag/v0.4.3


Use Appveyor to test on Windows

Let's see if we can make it happen :-)

Ok, this is ready! Another huge milestone for the project (although it takes a lot of time in Appveyor: ~20 min)
Testing with PyQt4 for now but I plan to switch to PyQt5 soon :-)

Sweet, well done Carlos!


Escape from Tarkov (DX11, Unity 5) crash on map load.

Hello, this is an issue not only I have, but pretty much everyone who plays this game on linux. I and many other people tried a lot of different wine and dxvk configurations (discussed here (reddit) or here (official forums), there is also a small discord server with a few people who tried a lot of things). We have a few (crash) logs, I only have 4 (I had to reinstall my whole system and lost all the other ones, sadly, but the errors are the same as in the logs from other people). I am not really experienced with all that yet, sorry. I know there are other people who posted about that issue here that know a lot more than me, so maybe they could provide some more information.
The game often crashed when the map loads or a few seconds after that, but not always. It crashes more often on maps with a lot of objects (like a mall with a lot of trash and goods laying around everywhere or just a really huge map). What I recognized is that it often crashed for me near areas where a lot of occlusion culling is going on and where there are a lot of occlusion culling issues even on windows. The fps is really bad at the beginning of a round, but that is also an issue on windows (maybe worth mentioning though?) and it usually doesn't crash after it worked fine for a few seconds (maybe up to a minute).
I ran an apitrace on windows because I was told this is what I should do. It took two whole days to complete (started the game, loaded into the map where it often crashed on linux and left shortly after that. No alt+f4, I exited via ingame buttons etc.). I used the dlls found here. I pasted the files d3d10.dll, d3d10_1.dll, d3d10core.dll, d3d11.dll and dxgi.dll into the directory where the exe is located and ran apitrace.
Escape from Tarkov, highest settings used.
Linux distros where the issue occured: Arch linux, debian 9/10, fedora and many more used by other people who experienced crashes.
Tested with and without lutris. Tested pretty much all runners available in lutris from 3.something up to ge 4.10 protonified. Also tested with custom wine versions (4.12 tested by me, for example. 4.something by someone else). Tested with DXVK 0.9.something up to 1.3. Other people tried even older versions, but I don't know which ones exactly.
The system I used for my linux testing (debian 10, cinnamon desktop):

CPU: Intel Pentium G4560
GPU: GTX 1050 Ti
Driver: Closed-source 430.something
Ram: 2x4GB DDR4

The system I used for the apitrace (windows 10 enterprise ltsc):

CPU: AMD Ryzen 5 2600
GPU: Radeon VII
Driver: Radeon Software 19.6.1
RAM: 4x8GB DDR4 3000 CL15

Apitrace files:
http://<IP_ADDRESS>/eft/EscapeFromTarkov.trace
http://<IP_ADDRESS>/eft/EscapeFromTarkov.1.trace
Logs/Files I still have:
http://<IP_ADDRESS>/eft/crash.dmp (generated by the game)
http://<IP_ADDRESS>/eft/error.log (generated by the game)
http://<IP_ADDRESS>/eft/output_log.txt (generated by the game)
http://<IP_ADDRESS>/eft/EscapeFromTarkov_d3d11.log

I'm aware that this game has issues (see #873), but it's not really obvious why.
Unfortunately, the crash dump and error logs generated by the game aren't particularly useful as they only point to kernel32 (pretty much anything could call into that) with no useful stack trace, and the DXVK log doesn't point to any issues either.
Replaying both of your apitraces works fine on my end as well, so I cannot reproduce the crash.

I will try to get an apitrace with the crash happening and send them here when they are done

I attached the last two logs I got while in-game running it on DXVK
CPU: i7 8750h
GPU: GTX 1070
Wine: 4.12.1 TKG Patched
DXVK: 1.3
As Vinjul1704 said, I have tried testing wine from 3.19 through 4.12.1 and DXVK .90 through 1.3
The last error I am getting is:
warn:  D3D11Device: Vertex input 'SV_POSITION0' not defined by input layout
I am not sure if this is relevant or not.
Originally, I was getting the DxvkMemoryAllocator error but that's been mitigated with THP set to never (currently). I have noticed there are a few things that do tend to set it off. On first load, if memory usage is high it would cause the crash (MemoryAllocator?).  This is substantially worse on the bigger maps.
Second, I have noticed that crashes happen around player interaction. While taking/receiving fire or being in their relative area, the game would crash. This seemed to get much worse around the last hotfix that they pushed and it could happen at any time during the raid. Currently, I am unable to play online due to this. Crashes are almost guaranteed.
Trying to run an apitrace in game is almost impossible because the raid will be over by the time we load into the game. Are we able to run a different apitrace on Windows (AMD GPUPerf which is much faster and also playable)? On Linux, the game will not run without DXVK or some wonky wine configurations. If we HAVE to do it on wined3d I am spend some time trying to get it working again.
https://pastebin.com/embed_js/59Xebahy
https://pastebin.com/embed_js/kHLvZy1K

I got an apitrace on linux with DXVK.
Setup:

Debian 10 Buster

Driver: Nvidia 430.something

TKG 4.5 Unity

DXVK 1.3

CPU: Intel Pentium G4560

GPU: GTX 1050 Ti

Ram: 2x4GB
It took me 3 or 4 attempts to get the crash, which is why the apitrace is so big.
<IP_ADDRESS>/eft/EscapeFromTarkov.trace.tar.gz

After further testing an analysis, I believe this issue is due to the anti-cheat that BSG is using currently. I would consider closing this. This is a not a Dxvk related issue.

We fixed the issue on different setups, but don't have a solid solution/guide yet (we believe it is because of missing registry entries that causes issues with their anticheat). It is not a DXVK issue though.
We will most likely update the lutris script once we figured out what exactly is going on.
I will close this now.


rawnode: expose per-follower MsgApp message stream

Part of #130

Superseded by https://github.com/cockroachdb/cockroach/pull/125002 in CRDB.


fix MigratePod

Signed-off-by: xulinfei.xlf<EMAIL_ADDRESS>Ⅰ. Describe what this PR does
When deleting QuotaGroup, the pod in the QuotaGroup will be moved to defaultQuota, while the background loop will move the defaultQuota to another quotaGroup. The two processes only have readLock constraints, it will result in some errors. So this PR will change the readLock to writeLock to fix the problem.
As the migrate operation is low-frequency, the writeLock will not influence the performance.

Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews
V. Checklist

[ ] I have written necessary docs and comments
[ ] I have added necessary unit tests and integration tests
[ ] All checks passed in make test

11_Run Go test.txt

/approve


testkit-backend: Skipping test SessionRunDisconnected.test_fail_on_reset

@bigmontz which testkit PR was this one for?


fix: Fix stdout/stderr logs not existing.

It's weird that a command would not produce these files, but let's check for this anyways.

!benchmark


Method not found

Good day,
I'm getting the following error when trying to use the AuthenticateAdmin("username", "password"); function:
Method not found: 'Void RestSharp.RestClient.set_Authenticator(RestSharp.|Authenticator)'
Please help, I've been trying to get it to work but fail to find a solution.
Using Magento ver. <IP_ADDRESS>

Are you using the library from the nuget package (1.0.1 = latest) or have you build it from source?
Make sure you have the correct corresponding RestSharp version. For the version on NuGet, the restsharp version should be: ≥ 105.1.0
I think a different version of restsharp is being used that doesn't have the property, or property setter for Authenticator.

I'm using the library from the Nuget Package 1.0.1 and RestSharp 105.2.2 is installed. Here is the list of packages installed:

Ok. Just reproduced it. It's the RestSharp version bump. Works with 105.1.0, but not with 105.2.2.
If you can downgrade to 105.1.0, you will be fine for now. I'll figure out what has changed in this minor upgrade that breaks oauth authentication in RestSharp.

Apparently the contrib namespace in RestSharp was renamed in between those versions.
I updated to latest version of RestSharp and pushed a new nuget package (1.0.2)

Awesome. Problem fixed. Thanks so much.


errors when installing socratex

I tried to install socratex using modern method and classic method. I got the following errors in both ways.

node:events:491
throw er; // Unhandled 'error' event
^
Error: bind EADDRINUSE null:443
at listenOnPrimaryHandle (node:net:1842:18)
at rr (node:internal/cluster/child:163:12)
at Worker. (node:internal/cluster/child:113:7)
at process.onInternalMessage (node:internal/cluster/utils:49:5)
at process.emit (node:events:525:35)
at emit (node:internal/child_process:944:14)
at process.processTicksAndRejections (node:internal/process/task_queues:83:21)
Emitted 'error' event on Server instance at:
at listenOnPrimaryHandle (node:net:1843:21)
at rr (node:internal/cluster/child:163:12)
[... lines matching original stack trace ...]
at process.processTicksAndRejections (node:internal/process/task_queues:83:21) {
errno: -98,
code: 'EADDRINUSE',
syscall: 'bind',
address: null,
port: 443
}
Node.js v19.9.0
[2@49972] Worker exited with error code: 1.
It just repeated the above errors.

I should have figured out what the cause is. This error message suggests that the port 443 is already in use. I killed the processes that used port 443. I was able to proceed in the installation then.

Cool! So everything works fine now?


Follow up on WorkerExecutor.submit() deprecation with third-party plugins

The deprecation of WorkerExecutor.submit() broke the following smoke tests:

AndroidPluginsSmokeTest.android library and application APK assembly (agp=#agpVersion, ide=#ide)
AndroidSantaTrackerCachingSmokeTest.can cache Santa Tracker Android application (agp=#agpVersion)
AndroidSantaTrackerSmokeTest.can lint Santa-Tracker (agp=#agpVersion)
AndroidSantaTrackerSmokeTest.check deprecation warnings produced by building Santa Tracker (agp=#agpVersion)
AndroidSantaTrackerSmokeTest.incremental Java compilation works for Santa Tracker (agp=#agpVersion)
KotlinPluginSmokeTest.kotlin javascript (kotlin=#version, workers=#workers)
KotlinPluginSmokeTest.kotlin jvm (kotlin=#version, workers=#workers)
ThirdPartyGradleModuleMetadataSmokeTest.publish

We should follow up with the respective plugin authors as necessary, and add a link to the issue where fixing the deprecation is tracked:

[ ] Android Gradle plugin
[ ] Kotlin Gradle plugin
[ ] Gradle publish plugin (is that what broke ThirdPartyGradleModuleMetadataSmokeTest?)

See https://github.com/gradle/gradle/pull/17044 for details on why the deprecations were not detected earlier.

This is an issue in JetBrains tracker for 7.1 deprecations in the Kotlin plugin: https://youtrack.jetbrains.com/issue/KT-46019

@rieske Great, thanks. Does this Kotlin issue cover all the occurrences listed above?

I will double-check and make sure it does.

I double-checked the deprecations coming from Android plugin - those I have followed up on in Slack when I was adding the deprecations - they will be fixed in AGP 7.0.

Fix for GMM test: https://github.com/gradle/gradle/pull/17064

I double-checked the deprecations coming from Android plugin - those I have followed up on in Slack when I was adding the deprecations - they will be fixed in AGP 7.0.

Do we have a link to an issue on issuetracker.google.com?


Parser throws Array Index Out Of Range on query "."

parser.Parse(".") throws an Out Of Range exception. Of course "." is not a valid function and should not parse, but the exception is thrown on Parse() and not Run(). This crops up when typing in decimal values without a leading zero.

I'll fix it.


Generate route index on the fly.

Generate route index on the fly in development mode.
Build route index in gulp task and leverage the pre-built index in production mode.
All routes are be order independent now.

Current coverage is 84.18%

Merging #42 into master will increase coverage by +0.71% as of 95f4f26

@@            master     #42   diff @@
======================================
  Files           27      28     +1
  Stmts          242     253    +11
  Branches         0       0
  Methods          0       0
======================================
+ Hit            202     213    +11
  Partial          0       0
  Missed          40      40

Review entire Coverage Diff as of 95f4f26

Powered by Codecov. Updated on successful CI builds.


Enable sending headers with HTTP requests

Is your suggestion related to an experience ? Please describe.
Sending additional headers with HTTP requests in adaptive scripts is not supported currently. We need to improve HTTP methods to support this capability.
There's an already existing PR [1] which provides this capability. We need to finalize the implementation and get the PR merged.
[1] https://github.com/wso2-extensions/identity-conditional-auth-functions/pull/107

This issue is fixed.
Feature -  PR
Docs - PR


correct typo in BLOCK.md

small typo correction in BLOCK.md where RedisModule_IsBlockedTimeoutRequest needs to be called like a function (with ctx as argument) inside the if test.

Yo @jaromil
The docs here are copied (and outdated) from upstream - in this case the redis-doc repo.

hi @itamarhaber! ACK, I see, you mean this file here is the upstream https://github.com/antirez/redis-doc/blob/f2590e156d329cfe801399dc194d3baedfde550b/topics/modules-blocking-ops.md the typo is there too, maybe will refile and in any case refer to @antirez' redis-doc in future

BTW, since I have your attention :^) still a topic about redis modules: would love to have more feedback about this https://github.com/antirez/redis/pull/6125 which is vital to the module I'm working on.

WRT redis-doc - yep.
WRT dlopen - I really don't know, but I'll ask the gang to opine.


Add flags to pip install

Stolen from the pixi docs, so I assume that's the recommended way

We are going to start using pixi's uv integration for this later on?

Yes, indeed


MySQLDump: skip dumpTable if mode set to NONE

@dg could you please merge this PR or is there sth. wrong with it?

Thank you!


Remove modules infrastructure from the CLI, Client and Core

So far, we've worked from GraphQL's concept of Query and Mutation modules. Supporting n named-modules would be similar to having a monorepo structure for wrappers by default. Therefore, we're removing the concept of modules and associated infrastructure from the toolchain.
This issue entails removing the infrastructure mentioned above, from the parts of the toolchain outside of the schema pipeline. Some identified subtasks (more may appear along the way) are:

Remove the modules key
Changes to compiler
Changes to query and build commands, maybe codegen?
Remove module argument from invoke and query
Updating tests and testing related utilities that use the client (like buildAndDeploy, or the e2e tests across the monorepo)

Would it makes sense to say that the new manifest; which will be something like:
format: 0.0.1-prealpha.9
name: NumberTypes
build: ./web3api.build.yaml
language: wasm/assemblyscript
schema: ./schema.graphql
main: ./src/index.ts

is non-compatible with the previous versions? The reason for this is that currently we have two entry points (query & module) and now we have one. So I see two paths here:

Take one of both, which I don't think makes sense because we would be ignoring one module (or query or mutation) and I don't think this is the right approach
Just make this an explicit breaking change. I think this is the way to go, we are at the perfect moment to introduce this kind of change. Making it an explicit breaking change would mean that, if you try to build your wrapper with an older version than this, the CLI will explicitly tell you that you need to update.

I will start implementing the changes with these assumptions but would love to hear your thoughts :) @Niraj-Kamdar @namesty @dOrgJelli

@cbrzn I think this makes sense!

Makes sense @cbrzn, but can we keep the module: property name, and not rename that to main:?

Makes sense @cbrzn, but can we keep the module: property name, and not rename that to main:?

Any particular reason for that, since from what I have seen in other languages, most of the languages call the entry-point file main (Ex: node, rust, etc)?

It's just because that's what we call it in the rest of our codebase, "module". Here are the names we use:
Wrapper ->
  Module ->
    Methods


[release/1.7] deps: Update otelgrpc

backport https://github.com/containerd/containerd/pull/9581

Issue:
Description of changes:
This change cherry-picks 78421616e0a6ba76ac25c94f0e7b5930d9b27d1a to release/1.7.
Reproduction of changes:
git cherry-pick -xsS 78421616e0a6ba76ac25c94f0e7b5930d9b27d1a
<manually resolve conflicts in go.mod>
go mod tidy
go mod vendor
pushd integration/client
go mod tidy
popd

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
(cherry picked from commit 78421616e0a6ba76ac25c94f0e7b5930d9b27d1a)

I was curious where this part of the commit went:

replace deprecated StreamServerInterceptor/UnaryServerInterceptor with NewServerHandler

But it looks like that was included in a somewhat unrelated cherry-pick that only update golang.org/x/net; https://github.com/containerd/containerd/commit/4e6335ebdf3ba54bf89d652c326b2127dd88639f (https://github.com/containerd/containerd/pull/10211)
On that matter, is it intentional to not include the second commit (https://github.com/containerd/containerd/commit/a1e0601e03d74bbbdfb3990064b8ac4679533fc7) from  https://github.com/containerd/containerd/pull/9581 ? It seems to indicate it was needed to satisfy compatibility, but I don't know the details.

@thaJeztah, oh good catch. That might be a miss by me. Let me take another look.

I don't know for sure it it's needed, but I know that OTEL can be a lot of "fun" w.r.t. subtle compatibility issues 😞 - so it won't hurt to have enough eyes on this PR 🤗

Okay I understand better now. The story is a1e0601e03d74bbbdfb3990064b8ac4679533fc7 is included in my backport. It resolves OTEL package build issues and 78421616e0a6ba76ac25c94f0e7b5930d9b27d1a alone is a broken commit. (I thought these failures were a result of my changes when I tested locally)
Unless there is any objections, we can update the description to state both commits are backported as a singular commit to avoid having a broken commit in history.


filled it in!

@WaylonWalker are you inclined to merge this? no pressure, but it might be good to remove the post otherwise. (i'd appreciate it, since your blank post is somehow my no.2 google search result)

oh dang, lets do it.


blog post for match_only_text field

Description
Blog post for match_only_text field introduced in OpenSearch 2.12. It explains how we achieved storage optimization and how phrase queries are run without positional data. It also hints on estimation of storage reduction to expect.
Issues Resolved
https://github.com/opensearch-project/project-website/issues/3056
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the BSD-3-Clause License.

@natebower these were great suggestions. Thanks for the review.
@pajuric please take it forward, thank you.

I hope you don't mind, but I've taken the liberty of pulling your changes into my own branch so I could edit the date and the meta description/keywords.  For some reason I couldn't push to your fork/branch.
#3092

@nateynateynate  thank you.


Omnisharp server crashses with no error message when solution_file config is invalid

When the config "solution_file" points to an invalid path (ex. when you make a typo), Omnisharp server crashes with no error message and it can take a while to figure what is wrong.
Maybe you could output something in the console when that happens?

Closing due to inactivity.


Update settings.yml

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #XXXXX

Hi @ahmadabdolsaheb - please avoid using GitHub to edit these files. You can use GitPod if you are not feeling like cloning a repo.
It's easy to miss formatting issues. The last time we merged something on the CDN, it broke everything on the main repo.
cc @ojeytonwilliams

Thanks for the heads up.


Clap changed to configure_me in wired

This implements basic changes to use configure_me instead of clap
for managing the configuration. This instantly allows us to pull the
configuration from files, directories containing files and to generate
man pages or other integrations that will be developed in configure_me
in the future.
So far, this is missing connect because it wasn't used and
configure_me doesn't support parsing multiple arguments into arrays
yet.
Closes #9
Currently blocked on https://github.com/LNP-BP/rust-lnpbp/pull/34

Damn, wanted to avoid wasting resources on CI, which will fail on unmerged serde support by making the PR Draft. That doesn't seem to be what drafts do. :confused:

Not WIP, since serde support was merged.

PR is ok, if you ok with changing default, I'll merge it right after it.

Done.

LNP node is now using configure_me! Thank you!


Command line properties not taking precedence over other property sources

I am using spring-cloud-config-client 1.0.1.RELEASE and it is not taking command line properties over the property file in the cloud config server.
From the spring-boot docs on external configuration, it seems like this should always be the case:
http://docs.spring.io/spring-boot/docs/current/reference/html/boot-features-external-config.html#boot-features-external-config-command-line-args
My current work around is to completely remove the property from my default and profile config files in the config repo, but it would be nice to have this option.
Is there a way to change the property source order?  Or is this the intended functionality?
Thanks.

If I'm not mistaken, this is the intended functionality.

So given a spring-boot application running with cloud-config, the only option to overwrite a property in the default config file is to check in a profile to the config repo with the changed property in it and run with the profile?  I can work with this, but seems like you should have the ability to change it from command line if needed (e.g., to quickly test a different server, but only for a short period of time).  It would be great to understand why this is intended functionality.  Thanks for your feedback and quick response.

You can turn off the config server for a given instance, if you want, spring.cloud.config.enabled=false.

Not quite what I am looking for since I think this would require me to pass all properties in through command line then, I would like to just override one property but still use all other properties from config server (just like it would work with a properties file on the classpath).
After looking through the code, I think I understand what is going on.  In BootstrapApplicationListener (which is in spring-cloud-context), the property order is bootstrap, commandLineArgs, etc...  Since cloud-config URI resides in bootstrap property source, all properties in cloud-config will take priority over all other properties.
What I am looking for is for cloud-config to act the same as if properties were retrieved from a file in the classpath.
In other words, I need cloud-config to not be part of bootstrap property source, but instead have its own property source that resides between 5 and 6 in the [Externalized Configuration order] (http://docs.spring.io/spring-boot/docs/current/reference/html/boot-features-external-config.html). Right now it is at 0 and which makes it impossible for applications to override values.
This doesn't seem to have a work around currently, so I will try to manage it through multiple profiles in the config repo.

I had a similar issue. How I got around it was to provide an override in
the property located in the application's config located in the repo,
something like this:
dialer.connection.port=${port:9125}
Then I could actually override it by passing an argument: --port=9999
On Fri, Jun 5, 2015 at 8:49 AM, SplashAttacks<EMAIL_ADDRESS>wrote:

Not quite what I am looking for since I think this would require me to
pass all properties in through command line then, I would like to just
override one property but still use all other properties from config server
(just like it would work with a properties file on the classpath).
After looking through the code, I think I understand what is going on. In
BootstrapApplicationListener (which is in spring-cloud-context), the
property order is bootstrap, commandLineArgs, etc... Since cloud-config URI
resides in bootstrap property source, all properties in cloud-config will
take priority over all other properties.
What I am looking for is for cloud-config to act the same as if properties
were retrieved from a file in the classpath.
In other words, I need cloud-config to not be part of bootstrap property
source, but instead have its own property source that resides between 5 and
6 in the Externalized Configuration order
http://docs.spring.io/spring-boot/docs/current/reference/html/boot-features-external-config.html.
Right now it is at 0 and which makes it impossible for applications to
override values.
This doesn't seem to have a work around currently, so I will try to manage
it through multiple profiles in the config repo.
—
Reply to this email directly or view it on GitHub
https://github.com/spring-cloud/spring-cloud-config/issues/167#issuecomment-109337810
.

Excellent suggestion, I will try this.  Thanks!


Please Answer..Important!

what is the difference between synergy 1 , synergy 2 and synergy 3?
Thank you!

I'm not the official maintainer of the Synergy project. This question is better asked on the Synergy repository:
https://github.com/symless/synergy-core/issues
I'm guessing Synergy 2 is like a rewrite of the original Synergy app, but I am not sure as i never used it.

Thank You!


various nxstyle fixes linked with the nuttx/audio PR

dependent on https://github.com/apache/incubator-nuttx/pull/836

DO NOT MERGE yet!

I am merging this PR now.  This WILL break the PR checks for awhile.  Sorry.
I will restart the checks on nuttx 838 and if that passes, I will merge it and things will be good.  But if not things will be more difficult.

Thanks


problem

Excuse me. I'm very interested in this article called CMX and read it several times. I have one more question to ask you.According to Figure 4, is there any problem with formula 6？

Hi, in Eq. (6) we use the skip connection, thus it is correct. You could also find that in Fig. 4b.


fby3.5: cl:Added event log

Summary:

Added event log for SYS Throttle, PCH Thermal trip, HSC OC, SOC MEMHOT, SOC VR HOT assert/deassert event.
Added SOC Thermal trip, VR watchdog timeout, system power drop, PCH power drop assert event.


Using "debug" for logging.

Hello,
I think using debug for console will be a good addition.
My current use case involve piping stderr & stdout from a service and there is some console.log that are a bit annoying here. debug is already used by express, it's very convenient to debug some part of the application, without generating to much noise.
Could I make a PR for that ?
I found you package delightful to use, it's exactly what I was looking for. Great job here.

Yes, of course, you could ;) Sorry for the delay but I've had some holidays. I'm going to merge your PR

Merged ;)


SAK-48902 Portal: Direct tool link button position issues

For the corresponding jira, refer to the attached screencast, SAK-48902-ProposedFix-20230517.mp4.

Hi Sean,
I really appreciate you jumping on this so issue so quickly thank you so much. During testing, I found that the 'Direct Tool Link' dialog box's CSS needs updating—it's currently too far down in the portal. I've attached a screenshot for reference.
Thanks

@kunaljaykam - Oops. Sorry about that. Easy enough to fix. I've force pushed a revision to fix that


chore(deps): upgrade concerto deps

Description
Upgrade Concerto Dependencies to latest  v3.13.0

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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<PHONE_NUMBER>:
0.0%

Covered Lines:
202

Relevant Lines:
202

💛  - Coveralls


'@' in @example code block tries to render a custom tag

When documenting code with an '@' symbol in the @example block, jsdoc thinks the '@' is a tag and breaks the rendering of the example code
Input code

/**
 * Creates a LESS file with variable definitions based on the style dictionary
 *
 * @example
 * ```
 * @color-background-base: #f0f0f0;
 * @color-background-alt: #eeeeee;
 * ```
 */
function foo() {

}

JSDoc configuration

No configuration, also tried with the markdown plugin enabled, same thing happens.
JSDoc debug output

DEBUG: JSDoc 3.5.5 (Thu, 14 Sep 2017 02:51:54 GMT)
DEBUG: Environment info: {"env":{"conf":{"plugins":[],"recurseDepth":10,"source":{"includePattern":".+\\.js(doc|x)?$","excludePattern":"(^|\\/|\\\\)_"},"sourceType":"module","tags":{"allowUnknownTags":true,"dictionaries":["jsdoc","closure"]},"templates":{"monospaceLinks":false,"cleverLinks":false,"default":{"outputSourceFiles":true}}},"opts":{"_":["test.js"],"debug":true,"destination":"./out/","encoding":"utf8"}}}
DEBUG: Parsing source files: ["/Users/djb/Dev/jsdoctest/test.js"]
Parsing /Users/djb/Dev/jsdoctest/test.js ...
DEBUG: Finished parsing source files.
DEBUG: Indexing doclets...
DEBUG: Adding inherited symbols, mixins, and interface implementations...
DEBUG: Adding borrowed doclets...
DEBUG: Post-processing complete.
Generating output files...
Finished running in 0.23 seconds.

Expected behavior

Either provide a way to escape an '@' sign with a backslash, or have jsdoc ignore any content in triple ticks so that the '@' character does not trigger a tag.
Current behavior

jsdoc reads any '@' symbol as a tag even if it is within a code example.
Your environment

Software
Version

JSDoc
3.5.5

Node.js
v9.6.0

npm
5.8.0

Operating system
OSX 10.12.6

Pinging this. Is there a workaround for this if this is not a valid use-case?

I've rectified this in a crude way; feel free to expand on it if you'd like.

Note that I am running a custom version of JSDoc in my own src/ folder; I'm not pulling from node_modules/ dir.

Inside jsdoc/lib/jsdoc/doclet.js, I've added the following just before the first docletSrc.replace statement:
docletSrc
+   // replace the following @ symbol groups with escaped @ symbols to
+   // avoid breaking markdown triple-tick code blocks; as JSdoc will
+   // think it's a doclet tag and try to create a tag object.
+   .replace(/(@:)/g, '@:') // @:
+   .replace(/(@import)/g, '@import') // @import
+
    .replace(/^(\s*)@(\S)/gm, '$1\\@$2')

This requires me to add additional .replace statements as needed. Not the best or most elegant solution, but it's at least not breaking my Vue SFC @click statements anymore in rendered code blocks.

any update on this yet? it's relatively difficult to document TypeScript decorators without a way to escape '@' character in example code block.

TypeScript also has this issue.
https://github.com/microsoft/TypeScript/issues/47679
I believe that we should find a solution that is compatible with both. TypeScript and JSDoc already have some differences, but I believe we should minimize these as much as possible and coordinate on the best solutions to be as compatible as possible.

More specifically, I don't think we need to handle @ specifically inside an @example block, but we should be able to escape characters anywhere in a JSDoc comment.
F.e. using \ as an escape character, \@ would render a literal @ in the comment output, and would not consider it the beginning of the tag. We could escape other characters using the same \.

I was able to escape the @ sign by preceding it with a zero width joiner character. It's a bit of a hack. I'm not sure if it will cause issues down the road either.
char:

code:
/**
 * @description handles the setup and teardown of connection for google cloud pubsub
 * @example
 *
 * ```
 * @Connect() // TODO: remove zero width joiner char once jsdocs accommodates
 * connect(): ClientConfig {
 *   const projectId = this.configService.get('SOME_PROJECT_ID')
 *   const credentials = this.configService.get('SOME_CREDENTIALS')
 *   return { projectId, credentials }
 * }
 * ```
 * @returns void
 */

result:

More specifically, I don't think we need to handle @ specifically inside an @example block, but we should be able to escape characters anywhere in a JSDoc comment.

+1
This will solve other issues like https://github.com/gajus/eslint-plugin-jsdoc/issues/710 as well.
The zero-width unicode character workarounds in this thread and others are not viable for examples that are meant as boilerplate, as the copied text will include the character and result in runtime errors like: Unexpected "\u2063".

Hey found this now and my usecase is lit html examples with eventlisteners like
 * @example
 * ```js
 * <cap-trigger
 *   visible="false"
 *   local-trigger="true"
 *   \@beginCollide=${(e) => {
 *     console.info(e.target)
 *   }}
 *   \@endCollide=${(e) => {
 *     console.info(e.target)
 *   }}
 *   geometry="primitive:box;width:8;height:8;depth:8;">
 * </cap-trigger>
 *  ```
 *

what i see the \ escaping is not used, is there any other workerround for now?


Upgrade Aspose, implement OfflineResourceLoader in mail&slides convertors and set default strategy to SKIP

We have made the following custom changes to the framework:

Upgrade Aspose to 23.1
Implement OfflineResourceLoader in MailConvertor & SlidesConvertor
Set default strategy to SKIP

Hmm, something seems to be wrong with the License Loader..
Did not experience this issue in the hybrid setup where framework dependencies were excluded and overriden..

Superseded by #5640.


Support validated links coming from e.com

Please review!
CC: @cead22, @tgolen, @marcaaron, @Luke9389
Details

Fixed Issues
https://github.com/Expensify/Expensify/issues/177739
Tests
See https://github.com/Expensify/Web-Expensify/pull/31977
QA Steps

Log into an account on NewDot Expensify, create a Workspace if you don't already have one using the + on the bottom right.
Log out of NewDot and then log into that same account on OldDot and go to Settings -> Policy and find your Workspace there. It should have a card icon like these:

Click on its name and verify it opens a new tab, automatically logs you in, and brings you to the workspace settings page:

Tested On

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

Screenshots
Included above

PR for adding workspace/new is ready here: https://github.com/Expensify/App/pull/5396/files. This could be its own PR and I could add testing steps for it.
Or I could merge it with this one which might help test that this PR works. I'm also trying to make sure urls where we do exitTo=workspace/<workspaceID>/card also works.
I'm going to work on the Web-Expensify part right now.

This PR was replaced by https://github.com/Expensify/App/pull/5396/


Added About control documentation

I can add documentation regarding subgroups in this PR as well, if you don't mind. Otherwise, I will just create a new PR for that.

@tonnyandersson I will merge this now. Please create a new PR for subgroups.


Feature: Source code view

This is essential for an RTE  IMHO =)

@exside To clarify your feature request, you would like to add the ability to wrap text in  blocks?

I presume you are referring to viewing the HTML source of what you are editing. This is not really a part of the editor; it would be a part of an integration of Squire. Implementing it is trivial: you just call getHTML to get the raw HTML, then replace the Squire instance with a <textarea> and set its value to the HTML. Reverse the process (calling setHTML) when done. This does not belong in Squire itself though.


Can I perform searches on multiple knowledge bases at the same time?

Search before asking

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

Description
No response
Use case
No response
Related issues
No response
Feature Priority
None
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

eg：SPACE_NAME={"知识库一"，"知识库二"}
"chat_mode": "chat_knowledge", "chat_param": "$SPACE_NAME"
The request will fail, how can I search multiple knowledge bases?Thanks

@kevinsummer219 , you mean multi knowledge spaces storage in different storage type, maybe vector or graph, or full text?

@kevinsummer219 , you mean multi knowledge spaces storage in different storage type, maybe vector or graph, or full text?

yes, how can I do this in AWEL,for example:
I hava two index_store: graph_store and vector_store

多创建几个知识库不就行了，在应用中添加多个知识库的资源


Backpressure for RIO

@davidfowl something like this?

@dotnet-build-bot test this please

@benaadams, @davidfowl, do we want to merge this?

So long ago... also would be merged into non-active branch

So... close?


Update C++ standard target

Reference: #826
This PR gives a snapshot of support for C++20 using the currently configured CI compilers. Currently c++20 fails for MacOS, Mingw-w64, and GCC on Ubuntu 18.04. If we use the earlier c++2a target, from before the C++20 standard became official, than some of the older compilers can manage to build successfully. We'll need to drop support for Ubuntu 18.04 though.
Perhaps someone would like to add the corresponding update for Windows.

Closing. Will wait until the full c++20 standard becomes more widespread before upgrading.


Specify strong mode "top level" inference

We've spent a lot of time discussing strategies around how we handle type inference of "top level" declarations—top level variables and static fields. There is what strong mode currently implements, and the revised rules we've considered informally.
The next step is to write up the set of rules we want to commit to.

Proposal is now here for consideration: https://github.com/dart-lang/sdk/pull/28218 .

Tracking bug for implementation here:  https://github.com/dart-lang/sdk/issues/28219

Can we close this one?

No Dart 2 milestone: This is now considered a subtask of #31228, which has a Dart 2 milestone.

I have a pull request out with a specification for this.


Have nowcasting acquisition use issue date from file

Since nowcasting uses sensor values computed as_of certain dates, the issue dates need to be the as_of dates and not datetime.today() like other acquisitions. These as_of dates are specified during sensorization (it calls the covidcast as_of indicator data to generate the sensors) and added to the files to be ingested as a new column.
cc @mariajahja

Just to confirm, when running in production you expect each line of the CSV might have a different issue date?

Just to confirm, when running in production you expect each line of the CSV might have a different issue date?

Just to confirm, when running in production you expect each line of the CSV might have a different issue date?

That's supported here, though the way it's currently implemented only 1 issue gets produced at a time so the CSV files should be homogeneous in that field

Just to confirm, when running in production you expect each line of the CSV might have a different issue date?

That's supported here, though the way it's currently implemented only 1 issue gets produced at a time so the CSV files should be homogeneous in that field

If you don't have a use case that needs a different issue per line, I'd prefer putting the issue into the filename instead of the file content -- saves a ton on disk space for the archived csvs.

If you don't have a use case that needs a different issue per line, I'd prefer putting the issue into the filename instead of the file content -- saves a ton on disk space for the archived csvs.

If you don't have a use case that needs a different issue per line, I'd prefer putting the issue into the filename instead of the file content -- saves a ton on disk space for the archived csvs.

That makes sense to me. Since I'm sharing the covidcast CsvImporter that auto-detects source/geo/signal from the filename, would you recommend embedding the issue into signal name and parsing it out later (e.g. make the signal name _), adjusting CsvImporter, ,writing new code to parse, or something else

If you don't have a use case that needs a different issue per line, I'd prefer putting the issue into the filename instead of the file content -- saves a ton on disk space for the archived csvs.

That makes sense to me. Since I'm sharing the covidcast CsvImporter that auto-detects source/geo/signal from the filename, would you recommend embedding the issue into signal name and parsing it out later (e.g. make the signal name _), adjusting CsvImporter, ,writing new code to parse, or something else

Borrow the issue-specific importer instead 😄
https://github.com/cmu-delphi/delphi-epidata/blob/2395b487feb760c02c056d27f022c10ef43cbbce/src/acquisition/covidcast/csv_importer.py#L82
Format is issue_{issue}/{source}/{normal covidcast filename}

Borrow the issue-specific importer instead 😄
https://github.com/cmu-delphi/delphi-epidata/blob/2395b487feb760c02c056d27f022c10ef43cbbce/src/acquisition/covidcast/csv_importer.py#L82
Format is issue_{issue}/{source}/{normal covidcast filename}

Borrow the issue-specific importer instead
https://github.com/cmu-delphi/delphi-epidata/blob/2395b487feb760c02c056d27f022c10ef43cbbce/src/acquisition/covidcast/csv_importer.py#L82
Format is issue_{issue}/{source}/{normal covidcast filename}

Oh cool, didn't know about that. Thanks for pointing it out, I'll work on integrating it now

Borrow the issue-specific importer instead
https://github.com/cmu-delphi/delphi-epidata/blob/2395b487feb760c02c056d27f022c10ef43cbbce/src/acquisition/covidcast/csv_importer.py#L82
Format is issue_{issue}/{source}/{normal covidcast filename}

Oh cool, didn't know about that. Thanks for pointing it out, I'll work on integrating it now

@krivard added issue specific csv importer, ready for review again

@krivard added issue specific csv importer, ready for review again


Unable to import native nodejs modules using ESM modules

Describe the bug
I want to use ES modules and exclude @aws-sdk libraries from the bundle since those libraries are built-in by default on AWS Lambda
To Reproduce
Steps to reproduce the behavior. If you have an example repository that would be even better:
files:
|- index.js
|- package.json
|- serverless.yml
|- ebuild.config.js

package.json :
...
"type": "module",
...

serverless.yml :
org: keyvhinng
app: aws-node-project
service: aws-node-project
frameworkVersion: "3"

provider:
  name: aws
  runtime: nodejs18.x

custom:
  esbuild:
    config: ./esbuild.config.js

functions:
  function1:
    handler: index.handler

plugins:
  - serverless-esbuild

esbuild.config.js :
module.exports = () => {
  return {
    entryPoints: ["index.js"],
    format: "esm",
    platform: "node",
    outputFileExtension: ".mjs",
    banner: {
      js: `
// BANNER START
const require = (await import("node:module")).createRequire(import.meta.url);
const __filename = (await import("node:url")).fileURLToPath(import.meta.url);
const __dirname = (await import("node:path")).dirname(__filename);
// BANNER END
`,
    },
    external: [
      "@aws-sdk/client-s3",
      "@aws-sdk/s3-request-presigner",
      "@aws-sdk/lib-storage",
    ],
  };
};

But I got the following error:
Error [ERR_REQUIRE_ESM]: require() of ES Module /.../aws-node-project/esbuild.config.js from /.../aws-node-project/node_modules/serverless-esbuild/dist/index.js not supported.
esbuild.config.js is treated as an ES module file as it is a .js file whose nearest parent package.json contains "type": "module" which declares all .js files in that package scope as ES modules.
Instead rename esbuild.config.js to end in .cjs, change the requiring code to use dynamic import() which is available in all CommonJS modules, or change "type": "module" to "type": "commonjs" in /Users/keyvhinng/Developer/playground/aws/lambda/aws-node-project/package.json to treat all .js files as CommonJS (using .mjs for all ES modules instead).

Expected behavior
Import native nodejs modules correctly

OS: Mac
Serverless Framework Version: 3.35.2
Plugin Version: 7.0.4

The problem here is that serverless-esbuild is assuming a CJS environment. However, I'm in the process of migrating to ESLint Flat Config, and so I've upgraded from v0.3.3 of esbuild-plugin-eslint to v0.3.8 as a result of https://github.com/robinloeffel/esbuild-plugin-eslint/issues/11 (supporting v9 of eslint). As such, I've encountered the problem of importing ESM in CJS, thanks to the work from v0.3.4 (migration to ESM).
The ideal solution to this, would be to support ESBuild config files in ESM format (esbuild.config.mjs, or "type": "module" in our package.json). This can be done thanks to the async import function, but would require quite a bit of refactoring in order to use, since it introduces async into the mix...
In order to reduce the immediate burden, it might be worth using something like https://www.npmjs.com/package/promise-synchronizer to prevent the need to refactor and test the plugin in order to support async.
In the meantime, I'm going to continue to use .cjs for my esbuild config, and use promise-synchronizer in order to use the dynamic import to use esbuild-plugin-eslint. This isn't ideal (the ideal solution would be the plugin supporting ESM), but it gets me back up and running.


Can't modify erroneous German translations on Transifex

Some of the translations to German are wrong, unfortunately. I'd like to correct them but even though I'm in the list of translators on Transifex I can't edit them anymore. Probably, because the text segements are already reviewed and I'm not in the list of the reviewers. Can you help, @Fantomas42 or @jezdez?
The issues I've found (and commented on a year ago) are related to featured entries.

Hi @bittner,
I have put you the role of reviewer on Transifex, tell me if it's good for you.
Regards

That worked out, thank you!
I have now corrected the translations. But I'd prefer that someone else reviewed them rather than myself. @jezdez, would you?

sure

Am 26.04.2015 um 10:45 schrieb Peter Bittner<EMAIL_ADDRESS>That worked out, thank you!
I have now corrected the translations. But I'd prefer that someone else reviewed them rather than myself. @jezdez, would you?
—
Reply to this email directly or view it on GitHub.

@jezdez please don't forget! I'd love to see the updated translation in the next release or so.
Specifically, watch out for "backlink" that I translated as "Rückwärtslink".


[Assets] Fully automatic splash generation

@yleflour note: for android, you're supposed to set a color background and center your image so the image isn't stretched out.
Here I chose to guess the needed background color with the top left splash image pixel color.
Of course, it does not work for gradients at the moment


google-cloud-cpp is our of date

Checklist

[x] Checked the nixpkgs master branch
[x] Checked the nixpkgs pull requests

Project name
nix search name: google-cloud-cpp
current version: 1.38.0
desired version: 1.42.0
Notify maintainers
maintainers: @cpcloud
@tobim who did the last update
Note for maintainers
Please tag this issue in your PR.

I don't have time to work on this at the moment.


Added global report title heading and style

is this different from #462 ??

@ashalper-usgs I think all you need to do here is add title(main = "UV Hydrograph") to createPrimaryPlot in uvhydrograph-render.R.

This added a title to each UV Hydro plot, but @zmoore-usgs added the plot titles in the report header in #460 so now this is redundant. @zmoore-usgs will be removing the titles added to each plot object in a new PR.


Update plugin README's with ember-cli-deploy compatibility badges

Update the following repos:
0.5.0 Compatible (I think)

[ ] duizendnegen/ember-cli-deploy-azure-blob
[ ] duizendnegen/ember-cli-deploy-azure-tables
[ ] jrowlingson/ember-cli-deploy-github
[ ] dschmidt/ember-cli-deploy-cp
[ ] minutebase/ember-cli-deploy-git-tag
[ ] minutebase/ember-cli-deploy-notify-firebase
[ ] ember-cli-deploy/ember-cli-deploy-build
[ ] ember-cli-deploy/ember-cli-deploy-gzip
[ ] ember-cli-deploy/ember-cli-deploy-redis
[ ] ember-cli-deploy/ember-cli-deploy-manifest
[ ] ember-cli-deploy/ember-cli-deploy-revision-data
[x] ember-cli-deploy/ember-cli-deploy-s3
[ ] ember-cli-deploy/ember-cli-deploy-json-config
[ ] ember-cli-deploy/ember-cli-deploy-display-revisions
[ ] ember-cli-deploy/ember-cli-deploy-ssh-tunnel
[ ] simplabs/ember-cli-deploy-notifications
[ ] ember-cli-deploy/ember-cli-deploy-s3-index
[ ] ember-cli-deploy/ember-cli-deploy-slack
[ ] pagefront/ember-cli-deploy-pagefront
[ ] dschmidt/ember-cli-deploy-sentry
[ ] arenoir/ember-cli-deploy-rsync
[ ] pagefront/ember-pagefront
[ ] duizendnegen/ember-cli-deploy-azure
[ ] ember-cli-deploy/ember-cli-deploy-lightning-pack

0.5.0 Incompatible

[ ] eddflrs/ember-cli-deploy-ssh
[ ] Kerry350/ember-deploy-s3-index
[ ] he9qi/ember-deploy-alioss
[ ] langalex/ember-deploy-rest-api
[ ] jschniper/ember-deploy-tagging-env
[ ] kinesisptyltd/ember-deploy-rest
[ ] arenoir/ember-cli-deploy-rsync
[ ] leojh/ember-deploy-rest-index
[ ] rtablada/ember-deploy-s3-index
[ ] LevelbossMike/ember-deploy-s3
[ ] treyhunner/ember-deploy-ssh-index
[ ] jschniper/ember-deploy-cloudfiles
[ ] eddflrs/ember-cli-deploy-ssh
[ ] damon-lanphear/ember-deploy-dynamodb
[ ] aptible/dashboard.aptible.com
[ ] LevelbossMike/ember-deploy-redis
[ ] Scalingo/ember-deploy-tagging-scalingo
[ ] jcaffey/ember-deploy-ssh-index
[ ] alisdair/ember-deploy-tagging-git
[ ] jamesfid/ember-deploy-tagging-index-sha
[ ] plyfe/ember-deploy-package-tagging
[ ] dukex/ember-cli-deploy-github
[ ] waltznetworks/ember-deploy-couchbase
[ ] automategreen/ember-deploy-dynamodb
[ ] cnerepo/ember-cli-deploy-redis-cne

kpfefferle/ember-cli-deploy-cloudfront is also 0.5.0 compatible and I would welcome a PR to add a version badge (or instructions how to add one myself).

@kpfefferle please do, https://github.com/ember-cli-deploy/ember-cli-deploy-version-badges
I plan to do some more work there to give us better colored badges but that should not affect the format
you can use
https://github.com/ember-cli-deploy/ember-cli-deploy-version-badges/blob/gh-pages/plugins/ember-cli-deploy-s3.md
as an example
thanks!

:boom: just opened a LOT of PRs


Allow stricter warnings for users (#560)

Per your suggestion in issue (#560) this change should allow users to have stricter warnings in their code.

Sure, thanks for taking the time to look at it and for helping me ignore the warnings locally. Again, thanks for this wonderful library.


added more potential sizes for css

Right now people can use class="w-25" or class="w-50". Adding all 1-100% increments is overkill but I think that a lot of people would at least expect every 10% increment be defined (I sure would find it handy!).
Adds additional classes to be generated

w-10, h-10
w-20, h-20
w-30, h-30
w-40, h-40
w-60, h-60
w-70, h-70
w-80, h-80
w-90, h-90

Deferring this to @twbs/css-review, but if it's gonna be merged the docs need to be updated to reflect the changes.

Any ETA or decision on this? I can update the docs to reflect this as well if that's what's blocking.

The height utilities will probably barely be used. We are also considering using the .col-* classes as width utilities.


[Refactor] Change ALL OLAPStatus to Status

Currently, there are 2 status code in BE, one is common/Status.h, and the other is olap/olap_define.h called OLAPStatus.
OLAPStatus is just an enum type, it is very simple and could not save many informations, I will unify these code to common/Status.
Proposed changes
Issue Number: close #xxx
Problem Summary:
Describe the overview of changes.
Checklist(Required)

Does it affect the original behavior: (Yes/No/I Don't know)
Has unit tests been added: (Yes/No/No Need)
Has document been added or modified: (Yes/No/No Need)
Does it need to update dependencies: (Yes/No)
Are there any changes that cannot be rolled back: (Yes/No)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

We should separate this PR

Add some performance test:
class Status {
public:
Status() {
_length = 0;
_precise_code = 0;
}
Status(int i) {
_precise_code = i % 17;
}
static Status OK() {
return Status();
}
static Status ERROR(int i) {
return Status(i);
}
bool operator == (const Status& st) { return _precise_code == st._precise_code; }
union {
char _state[2048];
  struct {
      // Message length == HEADER(7 bytes) + message size
      // Sometimes error message is empty, so that we could not use length==0 to indicate
      // whether there is error happens
      int64_t _length : 32;
      int64_t _code : 8;
      int64_t _precise_code : 16;
      int64_t _message : 8;       // save message since here
  };

};
};
static void newstatus(benchmark::State& state) {
// Code inside this loop is measured repeatedly
bool result = false;
int i = 0;
for (auto _ : state) {
Status status(i - 1);
++i;
if (status == Status::ERROR(i)) {
result = true;
} else {
result = false;
}
}
}
// Register the function as a benchmark
BENCHMARK(newstatus);
static void olapstatus(benchmark::State& state) {
// Code before the loop is not measured
bool result = false;
int i = 0;
for (auto _ : state) {
//Status status(i - 1);
++i;
if (i == 16) {
result = true;
} else {
result = false;
}
}
}
BENCHMARK(olapstatus);
new status is slower than olapstatus about 3.6%.
Since there are lot of other work during query and the status is returned batch by batch, so that the impact could be ignored.

Add SSB benchmark:
Environment:

ONE BE, ONE FE
CREATE LINEORDER TABLE with bucket num == 1, lineorder table with 600037902 rows.
set vectoried_engine = true;
set scanner thread == 1;
using select count(lo_shipmode) from lineorder; to test

new status:

old olapstatus:

It is very curious new status is a litter better than olapstatus.

I tested with jmeter, 50 concurrency, 1 FE, 1 BE
select * from tbl1 where k1=1 limit 1
The avg qps decreased from 1400/s to 1300/s, about 7% performance impact at high concurrency senario.
So I think it deserved to improve it

I have a question, why print the stack?

For stack, the common practice in the industry is to print when the process exits
By default, the stack will be printed when an error is reported, resulting in a full screen of stack information，which is not good for viewing the log


Configure prettier

What's wrong ?
Prettier was installed but the rules weren't configured properly. There was different coding styles in different files. Lack of uniformity.
What's inside this PR ?
Edit of the configuration file, and formatting of every file regarding the same specification.
Please take a look at the rules to see if it seems coherent to you 🙏

bors try

Also in the contributing.md apparently the linter recognizes that the code is .js thus he does not lint what is not .js.
So you have the following at the bottom of the file:
"version": "X.X.X",

export default 'X.X.X'

Could you replace the double quotes with simple quotes?

In the contributing.md there is still the following:
# Linter with auto-correct
yarn lint-fix
# Prettier for Markdown (auto-correct)
yarn format:md
# Prettier for SCSS (auto-correct)
yarn format:scss

Could you change the lint-fix and also test the format to see if they work properly?

Nice catch ! I tested them and they work perfectly 👌


Adds option to 'find nearest' when target tile is unacceptable

Not sure if this is the most elegant approach, but is my attempt at 'Moving within range of target'
https://github.com/prettymuchbryce/easystarjs/issues/75

Not sure if this is the most elegant approach, but is my attempt at 'Moving within range of target'
#75

you legend, i was literally just about to write this... testing now

Sadly I find several performance issues with the solution, due to the fact it essentially loops over the entire node list multiple times during its calculation. It would be better if this could calculate the check during the first calculate and return early or something similar.

This sort fuction and the following loop are causing the issue.
`instance.storeList.sort(function(a, b){
if( a.simpleDistanceToTarget === b.simpleDistanceToTarget) return 0;
return  a.simpleDistanceToTarget < b.simpleDistanceToTarget ? -1 :1
})
// loop through and find best result
var closestDist = instance.storeList[0].simpleDistanceToTarget;
var targetNode, targetCost;
for(var i = 0; i < instance.storeList.length; i++){
                    if(instance.storeList[i].simpleDistanceToTarget > closestDist ) break;

                    if(!targetNode || targetCost > instance.storeList[i].costSoFar){
                        targetNode = instance.storeList[i];
                        targetCost = instance.storeList[i].costSoFar;
                    }
                }

`

yah, that's not pretty :/
Updated to check 'nearest' during the main look up and cache it ( kills a few of those loops ).
See if that feels better

@Jiminibob Great stuff! Sadly the execution still takes about 500ms. Have you been able to make progress on the performance?

sorry for the old response, i wrote it later down the line and forgot about this.
commit here, if anyone wants it i might make a pr
https://github.com/aJamDonut/CustomStar/commit/ae4e8fdb7e23bd39049a06df1092da55ee4ce3ad

it's been in use for a while now no issues


🔒  Add TLS support

The challenger currently supports wss without TLS termination. Would be needed to support pinned certificates as well in order to avoid mitm during successfull challenges
Client(kcrypt-challenger) needs to be configured for TLS.  Depends on https://github.com/kairos-io/kairos/issues/399

#399 is about changing kcrypt to accept configuration from a file. Here we are talking about kcrypt-challenger. Maybe 399 means the challenger instead? Anyway, I'll check the 2 projects to see what's possible.


Update actions workflows

adds a new CI job to enforce schema validation for the cache's config

.deploy


Add Quota template support in codegen.

Till now codegen work was not considering QUOTA kind template at all, since we were not sure of the signature of the processor interface. Now since there is some clarity, this PR add support for Quota kind in processor interface code generator.

This change is

Jenkins job mixer/presubmit passed

Jenkins job mixer/presubmit passed

Jenkins rebuild mixer/e2e-smoketest

@douglas-reid @geeknoid @ZackButcher  Can we disable e2e-smoketest. It is failing for unrelated reasons and causing submitting of PR very hard. Even @ZackButcher 's PR is blocked due to this.

Jenkins job mixer/presubmit passed

Jenkins job mixer/pilot-regression passed

sadly, i vote 👎 on disabling the smoketests. if you can run locally and demonstrate a PASS, then I'm fine with overriding the check here, but disabling leads to messes.

Jenkins rebuild mixer/e2e-smoketest

Seems to be consistent !
--- FAIL: TestRateLimit (347.09s)
mixer_test.go:383: Bad metric value for rate-limited requests (429s): got 49.000000, want at least 1344.666667
mixer_test.go:398: Bad metric value for successful requests (200s): got 3.000000, want at least 600.000000
FAIL

Jenkins rebuild mixer/e2e-smoketest

No, it's not consistent unfortunately. Consider this other failure:
https://testing.istio.io/job/mixer/job/e2e-smoketest/648/console
On Mon, Jul 31, 2017 at 1:50 PM, Sebastien Vas<EMAIL_ADDRESS>wrote:

Seems to be consistent !
--- FAIL: TestRateLimit (347.09s)
mixer_test.go:383: Bad metric value for rate-limited requests (429s): got
49.000000, want at least 1344.666667
mixer_test.go:398: Bad metric value for successful requests (200s): got
3.000000, want at least 600.000000
FAIL
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/istio/mixer/pull/971#issuecomment-319192406, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AVucHZ1HGclwdKuHcJef_9arK7loZuRQks5sTj4lgaJpZM4On8vP
.

Jenkins job mixer/presubmit passed

Jenkins job mixer/pilot-regression passed

Jenkins rebuild mixer/e2e-smoketest

Jenkins job mixer/e2e-smoketest passed

@geeknoid thanks for the feedback. As per our offline conversation, I am abandoning this PR and creating a new PR that fixes the issue with the Quota template interface.


added fix for nms bug

Hello Experiencor,
Like all guys, I used your code to train a yolo model with my custom dataset. Your code worked well. I got the mAP value and I want to improve the mAP value. So I drew the pred_boxes around the object and I found a weird behaviour. I saw more than 1 bbox around each object. I was not sure at that time why it is printing like that. Then I added few codes to count true positive,true negative,false positive and all. I found that fp is so high. That is because the multiple bboxes around the same object. Later I found the issue. do_nms function nullify the surrounding bboxes with threshold. Nullify means just change the confidence to zero instead of removing it, this is correct if the bbox is associated with some other object also. But this method will fail, when the bbox is only associated with that particular class. do_nms nullify all the confidence value but the bbox will be still there which is wrong. So I wrote a simple fix for that. I also thought why mAP will still high. Later found that mAP value will not be affected by false positive. Thats why you got the results similar to the original yolo repo. I hope you understand the issue and fix done by me. If you like the work please merge this into master. And great thanks to you for spending your time in doing this cool stuffs which saves many people's time.

Hi @experiencor Any update on this


Can't add variables to a new release in v5_1

In v5_1 when creating a new release I get the error:
'Failed to build release: Variable(s) <variables x, x, x> do not exist in the release pipeline at scope: Release. New variables cannot be added while creating a release. Check the scope of the variable(s) or remove them and try again.'
As soon as I switch back to v5_0 it works fine.
Code:
        ado_variables = dict()
        for variable in variables:
            ado_variables[variable] = azure.devops.v5_1.release.models.ConfigurationVariableValue(
                is_secret=False, value=variables[variable])
        release_client = self._connection.get_client(
            'azure.devops.released.release.release_client.ReleaseClient')
        pipelines = self.get_pipelines(project)

        release_start_metadata = azure.devops.v5_1.release.models.ReleaseStartMetadata(
            definition_id=pipelines[pipeline],
            is_draft=False,
            artifacts=None if artifacts is None else artifacts)
        release = release_client.create_release(
            release_start_metadata=release_start_metadata, project=project)

I take that back, I can't get it working in 5_0 either.  Getting the same error about variables being added when creating a release.  My team was under the impression you guys were deprecating draft releases, is that not the case?
        release_start_metadata = azure.devops.v5_0.release.models.ReleaseStartMetadata(
            definition_id=pipelines[pipeline],
            is_draft=False,
            artifacts=None if artifacts is None else artifacts,
            variables=ado_variables)
        release = release_client.create_release(
            release_start_metadata=release_start_metadata, project=project)

I've asked the Release team to take a look at your question.  In the future for questions regarding specific REST API behavior, rather than the python SDK behavior, you will get a faster response by submitting your question on the appropriate REST API documentation page.  For release management, that page is here: https://docs.microsoft.com/en-us/rest/api/azure/devops/release/?view=azure-devops-rest-5.1

Thanks Ted!

@spacekitty76, If you want to override existing release variables at runtime, you can specify those variables in release start metadata, refer this rest api document. New variables you can't add through release start metadata. If you want to add new variables either you can create release and add new the variable to release.


Incorrect routing of log entries.

Environment details

OS: Google, Container-Optimized OS, 81-12871.148.0 stable
Node.js version: 12.16.1
npm version: 6.13.4
@google-cloud/logging version: 8.0.6

Steps to reproduce
We have the following log instances.
const logging = new Logging({ projectId });
export const fooLog = logging.log('foo-log');
export const barLog = logging.log('bar-log');
export const hamLog = logging.log('ham-log');

export async function logMessage(log: Log, severity: string, data: any): Promise<any> {
  return log.write(
      log.entry(
          {
            severity,
            resource: {
              type: 'gce_instance'
            }
          },
          data
      )
  )
}

Later in application code, we have calls writing to those logs.
logMessage(fooLog, 'INFO', {message: 'Foo message'});
logMessage(barLog, 'INFO', {message: 'Bar message'});
logMessage(hamLog, 'INFO', {message: 'Ham message'});

In Stackdriver log viewer, we noticed "Ham message" appearing in bar-log.
This behavior started ever since we upgraded our lib from 7.3.0 to 8.0.6
Thanks!

@pr4v33n thank you for the bug report, we are starting to dig into this.

Hi @pr4v33n,
I fixed the bug in google-gax and released v2.9.1. Please reinstall your dependencies, make sure that npm ls google-gax shows 2.9.1, and check if it helped.
Thank you for the perfect bugreport!


chore: release 1.1.8

Codecov Report
Patch and project coverage have no change.

Comparison is base (7e3a0cc) 83.66% compared to head (aaa0641) 83.66%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #109   +/-   ##
=======================================
  Coverage   83.66%   83.66%
=======================================
  Files          32       32
  Lines         967      967
  Branches      213      213
=======================================
  Hits          809      809
  Misses        157      157
  Partials        1        1

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

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


Fix mislabeling of feeds from Turkey, Kocaeli, Izmit to Turkey, Izmir

fix: Fixed incorrect locations of feeds that were previously mislabeled as Turkey, Kocaeli, Izmit. The feeds are actually for Turkey, Izmir.  (Also fixing this pr)

tr-kocaeli-metro-izmir-gtfs-1824.json
tr-kocaeli-eshot-gtfs-1823.json
tr-kocaeli-izban-gtfs-1828.json
catalogs/sources/gtfs/schedule/tr-kocaeli-tram-izmir-gtfs-1829.json

I am not sure if the files need to be renamed as well.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


IIS Host Module InProcess Performance slow with both V1 and V2 installed

Describe the bug
I've been doing some informal perf testing on a local machine where I've recently tested performance various local simple requests. Orignially I was able to get performance around 37k req/sec out of the InProcess module for both hello world and hello json controllers which is pretty good (much better than OutOfProcess)
Today I came back to find the same exact app and re-testing and found that performance had dropped to ~1.9k req/sec. Same app, no changes. Out of Process hosting, raw Kestrel and IIS static file hosting all were pretty much the same as the old tests, but InProc hosting consistently down to under 2k r/s.
After a lot of trial and error of uninstalling newer SDKs, I finally looked in the IIS Module list to find both the  V1 and V2 modules installed in IIS:

I removed the V1 module from the application and perf for InProc hosting jumped back up to ~37k req/sec as I would expect.
Put the V1 module back in and boom it's back down to 1.9k r/s. Looks like changing the order will also fix the problem.
To Reproduce
Check to see if V1 ASP.NET Core module is installed. If not install it explicitly and compare performance between V1 and V2 in use.
Expected behavior
When V2 module installs it should take precedence over the V1 module to ensure that the right one does get run.
I did reinstall the .NET Core 2.2 SDK last, so the V2 module would have installed or updated last, but even so the V1 module was first in the module list.
If V1 and V2 are compatible then maybe the V1 should be removed when V2 is installed.

Argghhh... looks like it might be something else after all. After reinstalling and adding and removing the issue has returned and removing the V1 module doesn't help. It also looks like in process is working no matter what.
Turns out the issue is related to switching into a Development build and the info logging bogging down the
My error although I haven't been able to trace down where the development flags are coming from. Unrelated so closing. Sorry for false circumstancial alarm (once again).

@RickStrahl I've been having this exact problem. After 2 days of try everything, I've found that, atleast in my case, the problem was the Microsoft Logging category. For example, changing its value from "Information" to "Warning" solved the problem. Now I'm trying to figure out why it can be.


i use laravel 9 do i can use V1

hi , i want to s3 disk in laravel , must to insall flysystem-aws-s3-v3 ???
and what is deff v1 and v3?

They're different major versions with breaking API changes.


SD: Bootstrap mode

When end hosts want to bootstrap, they need to initialize sciond with the appropriate material.
The sciond needs to have topology + current TRC at the very least.
The topology can be fetched from the discovery service. sciond already allows fetching the topology on start up when configured.
The DS should be configured to serve the topology according to https://godoc.org/github.com/scionproto/scion/go/lib/discovery on port 8041 (https://github.com/scionproto/scion/wiki/Default-port-ranges#discovery-service)
see ./bin/sciond --help-config the section discovery for configuration options.
Sciond should have a bootstrap mode, where it can just be configured with an URL in the config file and will then automatically:

fetch the topology and write it to disk
fetch the newest local ISD TRC from the certificate server.

Because we currently do not have support for TAACs, the received TRC has to be trusted without any further verification. Thus, bootstrap mode should not be enabled by infra services.

The devil is in the details here, there are many questions that need clarification (especially related to the trust model and the trust policy) before a bootstrap can be automatic.
Ultimately, this is more easily done manually by writing a custom small script to solve the bootstrap problem with a few curl/cp commands.
So, given that the it looks like too much work for little benefit, (and with @Oncilla's offline blessing) closing this as a Won't Fix.


Is it possible to connect nodes that are not on the same level in the DOM?

First thank you for this very nice project! Let me explain my need on an example. Suppose the DOM tree is
          body
                div
                          div of id A
                div of id B

I want to connect A and B. Is it possible?
I tried to connect it but I get the error
Uncaught (in promise) TypeError: this.drawflow.drawflow[t] is undefined
    getNodeFromId https://cdn.jsdelivr.net/gh/jerosoler/Drawflow/dist/drawflow.min.js:1
    addConnection https://cdn.jsdelivr.net/gh/jerosoler/Drawflow/dist/drawflow.min.js:1

How did you add the node?
When a node is added with the addnode function they are always at the same level.
What if a node can have several levels.
Example:
editor.addNode('node1', 0, 1, 150, 300, 'node1', {}, '
Hi!
');

editor.addNode('node2', 0, 1, 300, 300, 'node2', {}, '

Hii!

');

Thank you for your answer. OK I see. I added the node via the DOM (appendChild), that is maybe one source of the problem. Nodes are created via DOM manipulations.
When a node has a deeper level, its container may have other children too so unfortunately
editor.addNode('node2', 0, 1, 300, 300, 'node2', {}, '

Hii!

');
will not work, because it will consider node2 has the whole container instead of just the inner element.

Do you think it would be possible to adapt the library so that nodes are not at the same level?
Do you think it would be possible to modify DrawFlow in order to refer to elements directly rather than to their ids?
editor.addConnection(element1, element2)         instead of            editor.addConnection(element1.id, element2.id)

Anyway, thank you for your nice library that works when it is used "in the standard way".

What is the reason for having several nodes at different levels?
Maybe there is some other solution.
I think the library could be adapted, even though you would have to touch a lot of code. You can fork and try.

The reason is that the organization of nodes at different levels enable to benefit from CSS layouting. In the same time, I need to connect two nodes. See picture.

Are they multiple nodes?
If there are multiple nodes, have you thought about adding a new editor to a drawflow node?
Maybe that could be your solution.
If it's not multiple nodes and it's just style, does it mean something like this?

Thank you! Maybe your solution could work indeed!
In your example, the connections are:

Yes ====> Yes Randomized Facts
No =====> New Block 5
Right?

Is it possible for the edges (connections) to go over the block "Ask questions"?

Yes it's possible.

In this case I use an output displacement technique.
I use the class link followed by the output to which it is directed to move the element.
    editor.on('nodeCreated', (id) => {
        const links =  document.querySelectorAll(`#node-${id} .drawflow_content_node .link`);
        links.forEach((item) => {
            const target = document.querySelector(`#node-${id} .outputs .${item.classList[1]}`);
            if(target != null) {
                const pos = item.getBoundingClientRect();
                const targetPos = target.getBoundingClientRect();
                target.style.top = `${pos.y - targetPos.y}px`;
                target.style.left = `${pos.x - targetPos.x}px`;
            }
        })
    })

    editor.start();

    editor.addNode('question', 1, 3, 300, 200, 'question', {}, `

Ask question

Nice to meet yout (name)...

Custom API

Yes

No

No Match

    ` );

In this case it is only done for the outputs and new nodes, the same would have to be done with the import event.
I put the complete code of the example.

<!DOCTYPE html>

  <meta charset="UTF-8">
  <meta name="viewport" content="width=device-width, initial-scale=1.0">
  <meta http-equiv="X-UA-Compatible" content="ie=edge">
  <title>Document</title>
  <link rel="stylesheet" href="https://cdn.jsdelivr.net/gh/jerosoler/Drawflow/dist/drawflow.min.css"/>

Thank you! I will study that!
Then my need would be:

to disable all the editing functionalities (in my need, the graph just evolves by clicking on nodes)
to hide nodes (and the related links)
it would be easier also to nodes via DOM like editor.addNode(domElement) and to refer to DOM elements directly editor.addConnection(domElement1, domElement2), because here, it requires the user to handle ids manually (generation of fake ids etc.) But this depends on the usage. Adding as it is now is good too.

Also, your library deserves a nice tutorial!  :)
Have a nice day!

is it possible to add yes  or no buttons like drag & drop ?


[Fix] Migrate integration test to run on a schedule to unblock CI

Motivation
The integration test is currently configured to run on every push, which is timing out and causing ci to fail.
Although this test SHOULD be run regularly, it is more of a healthcheck than a unit test and thus, more suited to run at regular intervals instead of on every push.
Changelog

Moved integration test to run on scheduled runs twice daily
Created a - run_serial_long command with a timeout of 300m designed for tests that are known/designed to be long running. This param could've been changed in run_serial directly, but it would mean that any tests not designed to be long running would hang for hours, hurting iteration time against PRs & costing ci time.

Test Plan

The integration test should be triggered in SnarkOS to ensure it runs correctly

Where can the logs for the scheduled runs be seen?
Also, in the CircleCI run for this PR I saw the following errors:

error 1
error 2

Where can the logs for the scheduled runs be seen?

In the regular list of ci runs.

Also, in the CircleCI run for this PR I saw the following errors:

error 1
error 2

Those errors occurred originally when the PR was in draft, and occurred because the no_output_timeout had been put in the job scope instead of the command scope. It was moved to the appropriate place before the PR was set to green.


Product Update: September 2024

Tasks List

[ ] What's New
[ ] Fixed Bugs
[ ] Coming Up

Added List Templates While Creating A New List


query failed after leader of clickhouse-server instances is stopped

Bug Report
After multiple clickhouse-servers are deployed, the leader of clickhouse-server instances is stopped.  As a result, select, insert and alter sql all failed.
The result you expected
The query, insert, and alter statements can be executed successfully.
How to Reproduce

Component Name
Number of Instances
host

TSO
3
host1,host2,host3

Server
4
host1,host2,host3,host4

default-worker
5
host5,host6,host7,host8,host9

write-worker
3
host10,host11,host12

DaemonManager
1
host1

ResourceManager
1
host1

zookeeper
3
host13,host14,host15

create a table:

CREATE TABLE test.import_test_S3_ttl_on (
did UInt32,
num1 Int8,
num2 Int16,
num3 Int32,
num4 Int64,
unum1 UInt8,
unum2 UInt16,
unum3 UInt32,
unum4 UInt64,
fnum1 Float32,
fnum2 Float64,
a1 String,
dataCycle Date default today(),
updatetime DateTime default now(),
aa Array(Int16))
ENGINE = CnchMergeTree() Partition by toYYYYMMDD(dataCycle)
Order by did TTL dataCycle + toIntervalDay(10)
SETTINGS index_granularity=8192;

insert data into table:

insert into test.import_test_S3_ttl_on
select (rand() % ((1000000 - 1) + 1)) + 1,
(rand() % ((1000000 - 1) + 1)) + 1,
(rand() % ((1000000 - 1) + 1)) + 1,
(rand() % ((1000000 - 1) + 1)) + 1,
(rand() % ((1000000 - 1) + 1)) + 1,
(rand() % ((1000000 - 1) + 1)) + 1,
(rand() % ((1000000 - 1) + 1)) + 1,
(rand() % ((1000000 - 1) + 1)) + 1,
(rand() % ((1000000 - 1) + 1)) + 1,
(rand() % ((1000000 - 1) + 1)) + 1,
(rand() % ((1000000 - 1) + 1)) + 1,
concat('jwx', toString((rand() % ((1000000 - 1) + 1)) + 1)),
(toDate('2023-06-19') + (rand(3)%((90)))),now(),['1']
from numbers(500);

query table:

select count(),dataCycle from test.import_test_S3_ttl_on group by dataCycle;

According to the logs, the selected leader node is displayed:
will reroute query select count(),dataCycle from test.import_test_S3_ttl_on group by dataCycle to <IP_ADDRESS>:4567
so, the leader is <IP_ADDRESS>
4. stop the leader clickhouse-server
here is stop clickhouse-server with port 4567 on <IP_ADDRESS>
5. execute query, insert, alter sql again:
select count(),dataCycle from test.import_test_S3_ttl_on group by dataCycle;

insert into test.import_test_S3_ttl_on
select (rand() % ((1000000 - 1) + 1)) + 1,
(rand() % ((1000000 - 1) + 1)) + 1,
(rand() % ((1000000 - 1) + 1)) + 1,
(rand() % ((1000000 - 1) + 1)) + 1,
(rand() % ((1000000 - 1) + 1)) + 1,
(rand() % ((1000000 - 1) + 1)) + 1,
(rand() % ((1000000 - 1) + 1)) + 1,
(rand() % ((1000000 - 1) + 1)) + 1,
(rand() % ((1000000 - 1) + 1)) + 1,
(rand() % ((1000000 - 1) + 1)) + 1,
(rand() % ((1000000 - 1) + 1)) + 1,
concat('jwx', toString((rand() % ((1000000 - 1) + 1)) + 1)),
(toDate('2023-06-19') + (rand(3)%((90)))),now(),['1']
from numbers(500);

alter table test.import_test_S3_ttl_on drop partition 20230825

those sql all failed, report the error: Connection refused (<IP_ADDRESS>:4567) or Not connected to <IP_ADDRESS>:7890 yet
Version
0.1.0-GA

which clickhouse-server is stopped? Do you mean byconity server? If yes, would you please paste the server configuration in your cnch_config.xml? And are there any exceptions in the server log?

let's use https://github.com/ByConity/ByConity/issues/547 to track the progress of HA feature of ByConity


Services: first build: abs added

Linter check found the following problems:
The following problems have been found:
WARNING: /tmp/abs_4b4k6tt2mg/clone/recipe/meta.yaml:47: missing_doc_source_url: The recipe is missing a doc_source_url
ERROR: /tmp/abs_4b4k6tt2mg/clone/recipe/meta.yaml:47: missing_license_family: The recipe is missing the about/license_family key.
WARNING: /tmp/abs_4b4k6tt2mg/clone/recipe/meta.yaml:47: http_url: http://www.mono-project.com is not https
WARNING: /tmp/abs_4b4k6tt2mg/clone/recipe/meta.yaml:47: missing_license_url: The recipe is missing a license_url
Errors were found