game hangs when i try to eat stuff

*Mod Version:*1.12-<IP_ADDRESS>
*Forge Version:*1.12-<IP_ADDRESS>7
*Link to crash log (if applicable):*ther isent any crashing involved
*Description:*when ever i try to eat any type of food, my game hangs, doesent crash

What mod pack?  And how long did you wait?

its a bundel of mods i chose, heres the list of them:
BetterFoliage
Blur
CreativeCore
CrossHairMod
DurabilityShow
DynamicSurroundings
ItemsPhysic
Optifine
ShoulderSurfing
ToroHealth
i know that ShoulderSurfing mod cusases that proplem too, but nobody really reported it as an issue, so i did, and i didint really wait to see how long my game would last hung.

ok, this is the latest log from when my game crashes
[11:50:06] [main/INFO]: Setting user: ?????
[11:50:07] [main/INFO]: [STDOUT]: [itemphysic] Patched net.minecraft.entity.item.EntityItem ...
[11:50:07] [main/INFO]: [STDOUT]: [itemphysic] Patched net.minecraft.entity.item.EntityItem ...
[11:50:17] [main/INFO]: [STDOUT]: [creativecore] Patched net.minecraft.world.chunk.Chunk ...
[11:50:18] [main/WARN]: Skipping bad option: lastServer:
[11:50:19] [main/INFO]: LWJGL Version: 2.9.4
[11:50:28] [main/INFO]: Reloading ResourceManager: Default, FMLFileResourcePack:Forge Mod Loader, FMLFileResourcePack:Minecraft Forge, FMLFileResourcePack:Better Foliage, FMLFileResourcePack:Blur, FMLFileResourcePack:CreativeCore, FMLFileResourcePack:Custom Crosshair Mod, FMLFileResourcePack:Durability Show, FMLFileResourcePack:Dynamic Surroundings, FMLFileResourcePack:Presets!, FMLFileResourcePack:Shoulder Surfing, FMLFileResourcePack:ToroHealth Damage Indicators, Better Foliage generated, Blur dummy resource pack, additional-elytras-tron-update-1497983847.zip, Odyssey.zip
[11:50:31] [main/INFO]: [STDERR]: org.lwjgl.LWJGLException: Could not locate OpenAL library.
[11:50:31] [main/INFO]: [STDERR]: at org.lwjgl.openal.AL.create(AL.java:156)
[11:50:31] [main/INFO]: [STDERR]: at org.lwjgl.openal.AL.create(AL.java:102)
[11:50:31] [main/INFO]: [STDERR]: at org.lwjgl.openal.AL.create(AL.java:206)
[11:50:31] [main/INFO]: [STDERR]: at org.blockartistry.DynSurround.client.sound.SoundEngine.configureSound(SoundEngine.java:178)
[11:50:31] [main/INFO]: [STDERR]: at org.blockartistry.DynSurround.client.sound.SoundEngine.onSoundSetup(SoundEngine.java:162)
[11:50:31] [main/INFO]: [STDERR]: at net.minecraftforge.fml.common.eventhandler.ASMEventHandler_18_SoundEngine_onSoundSetup_SoundSetupEvent.invoke(.dynamic)
[11:50:31] [main/INFO]: [STDERR]: at net.minecraftforge.fml.common.eventhandler.ASMEventHandler.invoke(ASMEventHandler.java:90)
[11:50:31] [main/INFO]: [STDERR]: at net.minecraftforge.fml.common.eventhandler.EventBus.post(EventBus.java:179)
[11:50:31] [main/INFO]: [STDERR]: at net.minecraft.client.audio.SoundManager.(SoundManager.java:79)
[11:50:31] [main/INFO]: [STDERR]: at org.blockartistry.DynSurround.client.sound.SoundManagerReplacement.(SoundManagerReplacement.java:90)
[11:50:31] [main/INFO]: [STDERR]: at net.minecraft.client.audio.SoundHandler.(SourceFile:71)
[11:50:31] [main/INFO]: [STDERR]: at net.minecraft.client.Minecraft.func_71384_a(Minecraft.java:472)
[11:50:31] [main/INFO]: [STDERR]: at net.minecraft.client.Minecraft.func_99999_d(Minecraft.java:377)
[11:50:31] [main/INFO]: [STDERR]: at net.minecraft.client.main.Main.main(SourceFile:123)
[11:50:31] [main/INFO]: [STDERR]: at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
[11:50:31] [main/INFO]: [STDERR]: at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
[11:50:31] [main/INFO]: [STDERR]: at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
[11:50:31] [main/INFO]: [STDERR]: at java.lang.reflect.Method.invoke(Method.java:483)
[11:50:31] [main/INFO]: [STDERR]: at net.minecraft.launchwrapper.Launch.launch(Launch.java:135)
[11:50:31] [main/INFO]: [STDERR]: at net.minecraft.launchwrapper.Launch.main(Launch.java:28)
[11:50:32] [Sound Library Loader/INFO]: Starting up SoundSystem...
[11:50:33] [Thread-6/INFO]: Initializing LWJGL OpenAL
[11:50:33] [Thread-6/INFO]: (The LWJGL binding of OpenAL. For more information, see http://www.lwjgl.org)
[11:50:38] [main/ERROR]: Unable to parse metadata from minecraft:textures/blocks/i_golden_shovel.png
java.lang.RuntimeException: broken aspect ratio and not an animation
at net.minecraft.client.renderer.texture.TextureAtlasSprite.func_188538_a(TextureAtlasSprite.java:343) ~[cdo.class:?]
at net.minecraft.client.renderer.texture.TextureMap.func_110571_b(TextureMap.java:227) [cdn.class:?]
at net.minecraft.client.renderer.texture.TextureMap.func_174943_a(TextureMap.java:149) [cdn.class:?]
at net.minecraftforge.client.model.ModelLoader.func_177570_a(ModelLoader.java:168) [ModelLoader.class:?]
at net.minecraft.client.renderer.block.model.ModelManager.func_110549_a(ModelManager.java:28) [cga.class:?]
at net.minecraft.client.resources.SimpleReloadableResourceManager.func_110542_a(SimpleReloadableResourceManager.java:121) [cet.class:?]
at net.minecraft.client.Minecraft.func_71384_a(Minecraft.java:512) [bhz.class:?]
at net.minecraft.client.Minecraft.func_99999_d(Minecraft.java:377) [bhz.class:?]
at net.minecraft.client.main.Main.main(SourceFile:123) [Main.class:?]
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:1.8.0_25]
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) ~[?:1.8.0_25]
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) ~[?:1.8.0_25]
at java.lang.reflect.Method.invoke(Method.java:483) ~[?:1.8.0_25]
at net.minecraft.launchwrapper.Launch.launch(Launch.java:135) [launchwrapper-1.12.jar:?]
at net.minecraft.launchwrapper.Launch.main(Launch.java:28) [launchwrapper-1.12.jar:?]
[11:50:38] [main/ERROR]: Using missing texture, unable to load minecraft:textures/items/wooden_hoe.png, java.io.FileNotFoundException
[11:50:38] [main/ERROR]: Using missing texture, unable to load minecraft:textures/blocks/texture.png, java.io.FileNotFoundException
[11:50:38] [main/ERROR]: Using missing texture, unable to load minecraft:mcpatcher/ctm/planks birch/.13.png, java.io.FileNotFoundException
[11:50:38] [main/ERROR]: Using missing texture, unable to load minecraft:mcpatcher/ctm/planks birch/.17.png, java.io.FileNotFoundException
[11:50:38] [main/ERROR]: Using missing texture, unable to load minecraft:mcpatcher/ctm/planks birch/.21.png, java.io.FileNotFoundException
[11:50:38] [main/ERROR]: Using missing texture, unable to load minecraft:mcpatcher/ctm/planks birch/.25.png, java.io.FileNotFoundException
[11:50:38] [main/ERROR]: Using missing texture, unable to load minecraft:mcpatcher/ctm/noteblock/44.png, java.io.EOFException
[11:50:38] [main/ERROR]: Using missing texture, unable to load minecraft:textures/blocks/particle.png, java.io.FileNotFoundException
[11:50:39] [main/ERROR]: Using missing texture, unable to load minecraft:textures/blocks/e_texture.png, java.io.FileNotFoundException
[11:50:39] [main/ERROR]: Using missing texture, unable to load minecraft:mcpatcher/cit/custom elytras/mario wings/lotus_wings_broken_icon.png, java.io.FileNotFoundException
[11:50:39] [main/ERROR]: Using missing texture, unable to load minecraft:textures/blocks/4.png, java.io.FileNotFoundException
[11:50:39] [main/ERROR]: Using missing texture, unable to load minecraft:mcpatcher/cit/custom elytras/mario wings/lotus_wings_icon.png, java.io.FileNotFoundException
[11:50:39] [Thread-6/INFO]: OpenAL initialized.
[11:50:39] [Sound Library Loader/INFO]: Sound engine started
[11:50:40] [main/INFO]: Created: 8192x4096 textures-atlas
[11:50:51] [main/INFO]: [STDOUT]: [itemphysic] Patched net.minecraft.client.renderer.entity.RenderEntityItem ...
[11:50:53] [main/WARN]: Skipping bad option: lastServer:
[11:50:53] [main/INFO]: Narrator library for x64 successfully loaded
[11:51:43] [Server thread/INFO]: Starting integrated minecraft server version 1.12
[11:51:43] [Server thread/INFO]: Generating keypair
[11:51:45] [Server thread/INFO]: Loaded 488 advancements
[11:51:45] [Server thread/INFO]: Preparing start region for level 0
[11:51:46] [Server thread/INFO]: Preparing spawn area: 0%
[11:51:46] [Server thread/WARN]: Keeping entity minecraft:chicken that already exists with UUID 7069479e-4a88-434f-899d-fcf3825f5373
[11:51:47] [Server thread/WARN]: Keeping entity minecraft:chicken that already exists with UUID 45dc0783-91e6-43ab-9829-a62993c51a20
[11:51:47] [Server thread/INFO]: Preparing spawn area: 47%
[11:51:47] [Server thread/WARN]: Keeping entity minecraft:sheep that already exists with UUID 36de5a81-fd27-4df7-8967-5f966b8344cc
[11:51:48] [Server thread/WARN]: Keeping entity minecraft:llama that already exists with UUID d74d7a64-38c9-481e-8304-21c410ad76a0
[11:51:48] [Server thread/INFO]: Preparing spawn area: 99%
[11:51:48] [Server thread/INFO]: Changing view distance to 8, from 10
[11:51:52] [Server thread/INFO]: ??????[local:E:3f5a48a1] logged in with entity id 323 at (-539.6764677787564, 110.0, 56.607451559016596)
[11:51:52] [Server thread/INFO]: ????? land to this planet
[11:51:56] [Server thread/INFO]: Saving and pausing game...
[11:51:58] [Server thread/INFO]: Saving chunks for level 'test'/overworld
[11:51:58] [main/INFO]: Loaded 155 advancements
[11:51:58] [Server thread/INFO]: Saving chunks for level 'test'/the_nether
[11:51:58] [Server thread/INFO]: Saving chunks for level 'test'/the_end
[11:51:58] [Server thread/WARN]: Can't keep up! Did the system time change, or is the server overloaded? Running 2570ms behind, skipping 51 tick(s)
idk why that last line happens, its proabebly the cuse of my game crashing ?

The last line is telling you that the server is experiencing lag related problems.  What is interesting is the following"
[11:50:31] [main/INFO]: [STDERR]: org.lwjgl.LWJGLException: Could not locate OpenAL library.

I have seen this sporadically show up for some unknown reason.  Do you think your Minecraft installation is "sane"?

@AsteroidzX Have you updated Forge?  Are you still experiencing the problem?

soory soory, i wasent off for quite a while, really soory for the late responce
yes, i have moved to the "reccomended" version and "lateset" version multiple times, i still get the same problem

Have you tried removing some of the other mods till it stops?  I am currently running the Radon modpack with some other folks and no one is reporting this issue.

i only used your mod and it still gave me the same problem

Can you post a complete client log using only my mod?  Reason is that I can't duplicate and I am hoping that there is something more specific in the log that will give me clues.  Also, what launcher are you using?

normal minecraft luncher, why ?

[17:45:49] [main/INFO]: Setting user: CybernetricKat
[17:45:58] [main/WARN]: Skipping bad option: lastServer:
[17:45:58] [main/INFO]: LWJGL Version: 2.9.4
[17:46:07] [main/INFO]: Reloading ResourceManager: Default, FMLFileResourcePack:Forge Mod Loader, FMLFileResourcePack:Minecraft Forge, FMLFileResourcePack:mercurius_updater, FMLFileResourcePack:Dynamic Surroundings, FMLFileResourcePack:Presets!
[17:46:27] [Sound Library Loader/INFO]: Starting up SoundSystem...
[17:46:27] [Thread-6/INFO]: Initializing LWJGL OpenAL
[17:46:27] [Thread-6/INFO]: (The LWJGL binding of OpenAL.  For more information, see http://www.lwjgl.org)
[17:46:28] [Thread-6/INFO]: OpenAL initialized.
[17:46:28] [Sound Library Loader/INFO]: Sound engine started
[17:46:37] [main/INFO]: Created: 512x512 textures-atlas
[17:46:42] [main/WARN]: Skipping bad option: lastServer:
[17:46:42] [main/INFO]: Narrator library for x64 successfully loaded
[17:49:03] [Server thread/INFO]: Starting integrated minecraft server version 1.12
[17:49:03] [Server thread/INFO]: Generating keypair
[17:49:05] [Server thread/INFO]: Loaded 488 advancements
[17:49:05] [Server thread/INFO]: Preparing start region for level 0
[17:49:07] [Server thread/INFO]: Preparing spawn area: 0%
[17:49:07] [Server thread/WARN]: Keeping entity minecraft:chicken that already exists with UUID 7069479e-4a88-434f-899d-fcf3825f5373
[17:49:07] [Server thread/WARN]: Keeping entity minecraft:chicken that already exists with UUID 45dc0783-91e6-43ab-9829-a62993c51a20
[17:49:08] [Server thread/INFO]: Preparing spawn area: 44%
[17:49:08] [Server thread/WARN]: Keeping entity minecraft:sheep that already exists with UUID 36de5a81-fd27-4df7-8967-5f966b8344cc
[17:49:09] [Server thread/WARN]: Keeping entity minecraft:llama that already exists with UUID d74d7a64-38c9-481e-8304-21c410ad76a0
[17:49:09] [Server thread/INFO]: Preparing spawn area: 99%
[17:49:09] [Server thread/INFO]: Changing view distance to 8, from 10
[17:49:14] [Server thread/INFO]: CybernetricKat[local:E:ce99c4e6] logged in with entity id 397 at (-189.5, 67.0, 258.5)
[17:49:14] [Server thread/INFO]: CybernetricKat joined the game
[17:49:17] [Server thread/INFO]: Saving and pausing game...
[17:49:17] [Server thread/INFO]: Saving chunks for level 'test'/overworld
[17:49:17] [Server thread/INFO]: Saving chunks for level 'test'/the_nether
[17:49:17] [Server thread/INFO]: Saving chunks for level 'test'/the_end
[17:49:18] [main/INFO]: Loaded 155 advancements
[17:49:43] [Server thread/INFO]: [CybernetricKat: Set own game mode to Survival Mode]
[17:49:43] [main/INFO]: [CHAT] Your game mode has been updated to Survival Mode
heres the log, its not showing what happens after i eat something for some reason

[17:45:49] [main/INFO]: Setting user: ????
[17:45:58] [main/WARN]: Skipping bad option: lastServer:
[17:45:58] [main/INFO]: LWJGL Version: 2.9.4
[17:46:07] [main/INFO]: Reloading ResourceManager: Default, FMLFileResourcePack:Forge Mod Loader, FMLFileResourcePack:Minecraft Forge, FMLFileResourcePack:mercurius_updater, FMLFileResourcePack:Dynamic Surroundings, FMLFileResourcePack:Presets!
[17:46:27] [Sound Library Loader/INFO]: Starting up SoundSystem...
[17:46:27] [Thread-6/INFO]: Initializing LWJGL OpenAL
[17:46:27] [Thread-6/INFO]: (The LWJGL binding of OpenAL.  For more information, see http://www.lwjgl.org)
[17:46:28] [Thread-6/INFO]: OpenAL initialized.
[17:46:28] [Sound Library Loader/INFO]: Sound engine started
[17:46:37] [main/INFO]: Created: 512x512 textures-atlas
[17:46:42] [main/WARN]: Skipping bad option: lastServer:
[17:46:42] [main/INFO]: Narrator library for x64 successfully loaded
[17:49:03] [Server thread/INFO]: Starting integrated minecraft server version 1.12
[17:49:03] [Server thread/INFO]: Generating keypair
[17:49:05] [Server thread/INFO]: Loaded 488 advancements
[17:49:05] [Server thread/INFO]: Preparing start region for level 0
[17:49:07] [Server thread/INFO]: Preparing spawn area: 0%
[17:49:07] [Server thread/WARN]: Keeping entity minecraft:chicken that already exists with UUID 7069479e-4a88-434f-899d-fcf3825f5373
[17:49:07] [Server thread/WARN]: Keeping entity minecraft:chicken that already exists with UUID 45dc0783-91e6-43ab-9829-a62993c51a20
[17:49:08] [Server thread/INFO]: Preparing spawn area: 44%
[17:49:08] [Server thread/WARN]: Keeping entity minecraft:sheep that already exists with UUID 36de5a81-fd27-4df7-8967-5f966b8344cc
[17:49:09] [Server thread/WARN]: Keeping entity minecraft:llama that already exists with UUID d74d7a64-38c9-481e-8304-21c410ad76a0
[17:49:09] [Server thread/INFO]: Preparing spawn area: 99%
[17:49:09] [Server thread/INFO]: Changing view distance to 8, from 10
[17:49:14] [Server thread/INFO]: ????[local:E:ce99c4e6] logged in with entity id 397 at (-189.5, 67.0, 258.5)
[17:49:14] [Server thread/INFO]: ???? joined the game
[17:49:17] [Server thread/INFO]: Saving and pausing game...
[17:49:17] [Server thread/INFO]: Saving chunks for level 'test'/overworld
[17:49:17] [Server thread/INFO]: Saving chunks for level 'test'/the_nether
[17:49:17] [Server thread/INFO]: Saving chunks for level 'test'/the_end
[17:49:18] [main/INFO]: Loaded 155 advancements
[17:49:43] [Server thread/INFO]: [????: Set own game mode to Survival Mode]
[17:49:43] [main/INFO]: [CHAT] Your game mode has been updated to Survival Mode
here it is, for some reason it doesent show what happens after i try to eat food

@AsteroidzX I have been unable to repro the problem and I've been looking at the code and can't see where something like this would happen.  I've made a lot of changes with the latest BETAs related to the sound engine crash stuff and would like to know your experience with it.  If you could give it a try and let me know I would appreciate it.

Closing due to inactivity.  If the problem persists let me know.  Thanks!


Warn about memory usage of compiling with "all"

True, and hopefully will be mitigated somewhat by #316 when it gets addressed.  We should maybe additionally explain that --features all is almost certainly not what you want to do.  It's highly unlikely tha anyone needs 40+ AWS services built.

How is:

You can use the Cargo feature "all" to build Rusoto with support for every available service. Warning: building with "all" can require upwards of 5 GB of memory. Most people do not need all 40+ services anyway so use individual features to enable the services you use.

Thanks for the PR.
I agree with @adimarco , we should call out you can build with all but that's probably not the usual use case.  @dtolnay would you be able to add that to the PR?

@adimarco I don't think #316 will help at all. I tried running cargo expand and it finished in a couple seconds using "only" 1.8 GB of memory. (Incidentally the generated code is 78 MB and 1258523 lines.)

Updated.

@dtolnay Yeah, those numbers sound about right.  The code we generate is pretty verbose in the first place, and then it gets passed through serde_codegen for a double-dose of code generation.
Once we get to a working state for all of the AWS protocols, the next major phase of development will be optimizing the codegen.  There's a lot of low-hanging fruit there.


Certain Chinese Characters not showing up in Kivy

See title.  Corresponding user group discussion here
Current characters that are missing:

杀
艳
㑇

Note:  These are all simplified characters that do have a corresponding traditional character (e.g. It's not both simp. and trad.)

This is actually a problem with Apple's Ping Fang font file, not with Kivy.


feat(rate-limiting) global rate-limiting to protect back-end

This minor feature enable to perform global rate-limiting over a given endpoint aggregating count globally on the whole kong instance in case you want to protect the back-end to exceed its limits.
Summary
Current rate-limiting limit_by modes are: consumer, credential, ip.
Default is ip.
A new mode, is implemented and named "global".
It was tested and validated only using policy=local, but should also work on cluster or REDIS as soon as nginx server ip differs between knog/nginx instances.
Full changelog
I did not add new tests. Existing luacheck tests are OK.
Issues resolved
It is linked to the following forum thread:
https://discuss.konghq.com/t/common-rate-limiting-for-all-consumers/2218/5
https://discuss.konghq.com/t/rate-limiting-behaviour/2081/2

 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.

Closing because of lack of activity. We would like to accept this contribution if anyone wishes to pick this back up.


How can I use my own data for test?

I am getting a lot of help from your paper.
I want to test my own data.
(4 MIC configuration DoA estimation)
What parts of the code will need to be modified before testing can proceed?
I would really appreciate it if you could let me know.

Hi @jhkim6757 , it should be possible to process your data by modifying the file test_doa_recorded.py.

Change the code so that the variable mic_array contain the coordinates of the microphones in the microphone array. The variable should be a numpy.ndarray of shape (2, n_mics) where n_mics is the number of microphones.
Modify the code so that the variable speech_signals contain your recorded signals. This is also a numpy.ndarray of size (n_samples, n_mics) where n_samples is the length of the signal in samples.

Some details to pay attention to:

The order of the signals in speech_samples should match that of the coordinates of the microphones in mic_array
To obtain good performance you will most likely need to adjust the parameters of the algorithm to account for dominant frequencies in your signal, microphone spacing, etc. For examples, freq_hz (frequency bands), fft_size, sampling frequency fs, etc.
Pay particular attention to M, this is the maximum Fourier coefficient index (-M to M), used in FRIDA. The algorithm requires that K_est <= M <= n_mics*(n_mics - 1) / 2. A smaller M will provide more robustness, but lower accuracy, and a larger M vice versa.

Thank you for comment,
I would like to ask few more questions.
I got a Error Message
' LinAlgWarning: scipy.linalg.solveIll-conditioned matrix detected. Result is not guaranteed to be accurate.', instead of 'Appling FRI ...'
How can I solve this problem?
Additionally, Should I have to match the filename to 1.wav, 2.wav, ..., 16.wav etc.

@jhkim6757 The message you describe is a warning, rather than an error. It doesn't mean the code is not working, unless some other error occurs, or you are getting garbage results out.
Concerning the filenames, you have to change the code so that filename (see this line) contains the name of the wav file with the data. In our code, we assume all channels are contained in a single wav file. If this is not the case, you have to open every file individually and concatenate the signals into the speech_signals array as described in my reply above.
Hope this helps.

is there any adjustment for 'protocol.json' to test my own data set and 4mic configuration?

The protocol.json file was used to store all the information pertaining to our own recorded data. I think the best way is for you to get rid of this part of the code and only specify what you need.

Due to the warning mentioned above, we are not getting accurate results.
Could you tell me where 'the ill-conditioned matrix detected' warning occurred?
' LinAlgWarning: scipy.linalg.solve Ill-conditioned matrix detected. Result is not guaranteed to be accurate.'
I want to see the DOA results my data.
Thank you


[Doc] Update group_concat default sep to ,

change group_concat default seperator to ,
according to the
code, the default sep is ,  , not comma ,.
What type of PR is this：

[ ] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[x] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto backported to target branch

[x] 2.5
[x] 2.4
[x] 2.3
[ ] 2.2

run starrocks_admit_test

@Mergifyio backport branch-2.5

@Mergifyio backport branch-2.4

@Mergifyio backport branch-2.3


Implement strings extension in Emacs core

If nothing else helps to achieve good string ops performance,
it is possible to add required functions into Emacs C code itself.
Required functions:

substring-shared to make allocation-free substring; this requires strings to be immutable
string-byte-at to index nth multibyte string byte
mapc-string-bytes to loop over multibyte string individual bytes

My understanding is that string-byte-at can be implemented as Emacs module function.
This will be faster for medium/long strings.
Not sure about mapc-string-bytes, but it can also be implemented as a module.
This does not solve O(N) problem for string-byte-at, but the performance will increase
significantly. Need some concrete speed tests though.


Issue with EnableTerminationProtection in DescribeStacks when no StackName are provided

When DescribeStacks is called without filter on the StackName, the property EnableTerminationProtection from the Stack model is filled with an invalid value (False).
Expected Behavior
The EnableTerminationProtection property should be filled with the real value.
Current Behavior
EnableTerminationProtection is filled with the value "False" wheter the Stack is protected or not.
Possible Solution
Steps to Reproduce (for bugs)

Create a stack
Try to make a call to DescribeStacks using the StackName in the parameters
Try to make a call to DescribeStacks without any filter

The result for the same stack is not the same in both cases.
I have attached a console application sample EnableTerminationProtectionIssue.zip.
Launch it as follows :
EnableTerminationProtectionIssue.exe <accesskey> <secretkey> <region name> <stackname>
Context
When I try to display the result for a list of stacks, the result is not accurate.
Your Environment

AWSSDK.Core version used:  3.3.20
AWSSDK.CloudFormation version used:  3.3.9
Operating System and version: Windows 10 Entreprise
Visual Studio version: 2015
Targeted .NET platform: net461

Thanks for detailed report.  This seems to be a service side issue. I forwarded the information to CloudFormation team.

It looks like CloudFormation service is not returning EnableTerminationProtection flag when making DescribeStacks without filter.  We passed the information to the service team.  I'll keep the ticket open until we hear back from them.

Hi, I had a chat with the service team.  This behavior seems to be as designed.  It appears that the documentation doesn't have any information about this so we'll update that.  CloudFormation team is aware of this inconsistency and they are actively working on this.
Since there's no actionable item on our side, I'll close this.


Decide where to do general documentation, tutorials etc.

Since @jhumber  and his Falcon colleagues don't get access to editing our https://dev.commercetools.com site where the tutorials and general pages on the JVM SDK live we could consider writing the HOWTOs and other version-agnostic content somewhere else.
Options:

markdown files in the repository (in the root dir or a "doc" folder).
github wiki feature ( https://github.com/commercetools/commercetools-dotnet-sdk/wiki )
the CT developer website ( via a person at commercetools that does the changes on request)
more ideas?

@jhumber @stmeissner please vote or propose something.
I prefer the github wiki as it's most approachable.  known downside: embedding images is a bit of a hassle.

I think the GitHub Wiki feature is suitable for the documentation.

I also vote for the GitHub Wiki for two reasons

it's
close to the source code
can
be maintained by the developers without me being in the loop

OK, decision made. I'll set up an initial page and a link from the readme.


_SourceKitLSP - invalid character in Bundle Identifier

When use with SPM as a dependency, Swift 5.9 (Xcode 15 RC) is not happy about _SourceKitLSP target name:
warning: invalid character in Bundle Identifier. This string must be a uniform type identifier (UTI) that contains only alphanumeric (A-Z,a-z,0-9), hyphen (-), and period (.) characters. (in target '_SourceKitLSP' from project 'SourceKitLSP')

Tracked in Apple’s issue tracker as rdar://115633096

This doesn’t appear to be an Xcode regression. Xcode 14.3 emits the same warning. The fix should be to declare the product as automatic instead of it being declared as dynamic. That way Xcode won’t build a framework and thus no Info.plist will get created, which causes the issue.


Missing support for resources of type Microsoft.Subscription/aliases

The type "Microsoft.Subscription/aliases" let the validation fail and stops the user from getting an outline.
The resource type is defined here:
https://docs.microsoft.com/en-us/azure/templates/microsoft.subscription/aliases
to achieve this
https://docs.microsoft.com/en-us/azure/cost-management-billing/manage/programmatically-create-subscription-microsoft-customer-agreement?tabs=rest-getBillingAccounts%2Crest-getBillingProfiles%2Crest-MCA

This is only valid for deployment to a tenant, right?  The following template works for me.  Note that the $schema must correctly point to a tenant deployment:
{
    "$schema": "https://schema.management.azure.com/schemas/2019-08-01/tenantDeploymentTemplate.json#",
    "contentVersion": "<IP_ADDRESS>",
    "parameters": {},
    "functions": [],
    "variables": {},
    "resources": [
        {
            "name": "string",
            "type": "Microsoft.Subscription/aliases",
            "apiVersion": "2020-09-01",
            "properties": {
                "displayName": "string",
                "workload": "Production",
                "billingScope": "string",
                "subscriptionId": "string",
                "resellerId": "string"
            }
        }
    ],
    "outputs": {}
}

Please let me know if this doesn't address the issue, thanks!

This warning is new in the current version:

This warning is new in the current version:


Tickets/preops 3141

PR to incorporate revisions to the early science program following the December project replan.  Restructuring of the document to remove duplication and improve readability

Mostly small textual suggestions. I think it would be useful to call out Alert Production in a colored table analogous to DP0/1/2--it's also a significant release of data to the community and we should make it prominent.

Agree, I'll look at making a similar such table


fix(onboarding): reset index on cancelling the process

closes #942

Codecov Report

Merging #949 into master will increase coverage by 0.04%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #949      +/-   ##
==========================================
+ Coverage   74.32%   74.36%   +0.04%
==========================================
  Files         187      187
  Lines        3552     3550       -2
  Branches      665      664       -1
==========================================
  Hits         2640     2640
+ Misses        821      819       -2
  Partials       91       91

Impacted Files
Coverage Δ

src/store/onboarding/actions.ts
88.46% <100%> (+6.31%)
:arrow_up:

Continue to review full report at Codecov.

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


Does ssh-agent on windows have persistent storage of ssh keys?

[Enter feedback here]
This article seems to suggest deleting ssh private keys after adding them to ssh-agent with ssh-add and backing them up to a secure storage medium.  I've only ever seen ssh-agent store keys for a single session of the OS running, so a reboot clears them all out and they have to be re- ssh-add'ed every reboot.
Did Microsoft alter ssh-agent to have persistent storage of added keys?

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

ID: 4c89b7b3-c05f-b517-6109-04e6a97195c3
Version Independent ID: 1e4b742e-d999-2c1e-a396-dc149f7ac76b
Content: OpenSSH key management for Windows
Content Source: WindowsServerDocs/administration/OpenSSH/OpenSSH_KeyManagement.md
Product: windows-server
Technology: administration
GitHub Login: @maertendMSFT
Microsoft Alias: damaerte

Closing and re-opening to test labeling automation so this can be addressed.

Hi @zaaj
Thanks for your dedication to our documentation. In our examples the service was set to manual which required the service to be started at each reboot, although the keys were already persistent. We've submitted a documentation change to update the code examples, setting the ssh-agent to automatic, the updated article should be live by tomorrow. If you have an more queries about the ssh-agent design, please use the (Win32-OpenSSH issues)[https://github.com/PowerShell/Win32-OpenSSH/issues]
#please-close


node vv14.20.1 (64-bit) is not installed or cannot be found.

Issue:
Me and my colleague is trying to use nvm in WIndows using this version.
We are both running windows 11 and we are getting the same error message.
How To Reproduce:

Expected Behavior:
I would expect this to work, as I have installed version 14.  I can also see that it is installed under AppData/Roaming/nvm
Error:
nvm use 14
node vv14.20.1 (64-bit) is not installed or cannot be found.
Desktop:

OS: Windows 11
Terminal: PowerShell && cmd

I get the same exact error when I run nvm use 14.
However when I run nvm use 14.20.1:
nvm use v14.20.1
exit status 5: Access is denied.

exit status 5: Access is denied.

Duplicate of #708. See the Common Issues wiki page for details about exit status codes.


imageUrl incorrect

It is small problem, but still it bothers me. In imageUrl you missed http: on the begining.

It will automatically add "http:" if it is missing at the beginning of the URL.


[turbopack] Non matching path (conversation) provided for /(routes)/conversation

What version of Next.js are you using Turbopack in?
13.4.11
What package manager are you using / does the bug impact?
npm
What operating system are you using?
Windows
Describe the Bug
Non matching path (conversation) provided for /(routes)/conversation
Expected Behavior
Starting dev server
To Reproduce
dev server is not starting
Reproduction Repo
https://github.com/salmansheri/ai-saas

Have the same problem after upgrade next.js to v13.4.12.

yes
Non matching path (dashboard) provided for /(routes)/dashboard
Debug info:

Execution of ::get failed
Non matching path (dashboard) provided for /(routes)/dashboard

This is likely fixed, since you didn't provide a minimal reproduction I'm unable to run the provided reproduction, can you try again with npm install next@canary? I'll close the issue for now as there hasn't been activity for many months, if it still doesn't work let me know and we can re-open the issue. Thanks!


Pull apb templates from their respective git repos

Describe what this PR does and why we need it:
The new ci framework uses the git repos of APBs to carry templates
that will operate the APBs.  This patch will point CI at the templates
in the APB's git repos.
Also, keep the existing templates in tree for cli testing.
Changes proposed in this pull request

Point CI to the APB's git repos

Changes Unknown when pulling 06618b5b01031551dc4567a2a9c76ce2561dc1b6 on rthallisey:use-git-names into ** on openshift:master**.


Fix warnings on disable USETCP_WIN macro

Description
This change fixes warnings which appear on disabling the macro ipconfigUSE_TCP_WIN
This PR is a fix for the customer reported issue
Test Steps
Run the Echo Posix Demo with warnings enabled , the demo runs successfully without these warnings
Checklist:

[x] I have tested my changes. No regression in existing tests.
[x] I have modified and/or added unit-tests to cover the code changes in this Pull Request.

Related Issue
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/bot run uncrustify


Remote management network adapater

On a Dell system the script is picking up the iDrac network adapter:
NIC settings per active adapter:
        Interface Description: Remote NDIS Compatible Device [Ethernet]
                Warning: NIC driver is over 1 year old. Verify you are at the latest version.
                Driver Date: 06/21/2006 00:00:00
                Driver Version: 10.0.14393.0
                Link Speed: 425.984 Mbps
                MTU Size: 1500
                RSS: No RSS Feature Detected.
                IPv6Enabled: True
                Packets Received Discarded: 0

Can this be excluded?

I've sent on the XML file now.

@smwk  is it possible that the name could be different on different OSs?
I found this article, and it looks like on Windows 2016 OS the name of the NIC appears to be a standard that I could exclude, however, I would like to make sure that it would also do it for Windows 2019 for sure as well as older OSs.
https://www.dell.com/support/manuals/us/en/04/idrac-service-module-v3.0.1/ism_3.0.1_users_guide/installing-the-idrac-service-module-on-microsoft-windows-operating-systems?guid=guid-ecb11c9e-dfbc-4b45-ab0c-4ce7f718bf96&lang=en-us

It might be but unfortunately I have no way of checking as we only have Server 2016 deployed at work.


Have 1.11 branch to build from 1.11 k8s

This is part of the branch creation automation we need to have. A card is already created.

Thanks @kwmonroe


Adding second initramfs for microcode

Hi, I can't see an obvious way to add a second initramfs for microcode other than by inserting directly into the cmdline array. This is obviously not the preferable solution - if this functionality existed and I'm missing it or could be added that would be great. Many thanks!

Hi, I can't see an obvious way to add a second initramfs for microcode other than by inserting directly into the cmdline array. This is obviously not the preferable solution - if this functionality existed and I'm missing it or could be added that would be great. Many thanks!

https://github.com/BiteDasher/kesboot/commit/8aefdc340dec42c229b265131e2a2f16a4d88c59
Please try this and ping me if it worked

@BiteDasher Excellent - that seems to work perfectly. Thanks!


Standard-Template-Protocols

Standard-Template-Protocols - Protocols for your every day iOS needs

Let's put this in Utility

Thanks, @dkhamsing

:+1:


Interpolated string in ExecuteSqlCommand do not work as expected

So, I experience this interesting behaviour when using an interpolated string with the ExecuteRawSql method, but also when using the parameter classes.
Case: I want to disabled the checks, remove all data and re-enable all checks for a mssql database (used for testing).
Below is the code that currently works (with only the first of the required three queries per table):
var tables = dbContext.Model
                      .GetEntityTypes()
                      .Select(r => $"{r.Relational().Schema}.{r.Relational().TableName}")
                      .ToImmutableList();

foreach (var table in tables)
{
    dbContext.Database.ExecuteSqlCommand($"ALTER TABLE {table} NOCHECK CONSTRAINT all", table);
}

As you can see, I provide the entityType twice to the ExecuteSqlCommand. I've tried omitting it, or using any of the other possibilities I've found online, but result vary. See below:
// Works, double param
dbContext.Database.ExecuteSqlCommand($"ALTER TABLE {table} NOCHECK CONSTRAINT all", table);

// Works, it seems that the parameter is ignored
dbContext.Database.ExecuteSqlCommand($"ALTER TABLE {table} NOCHECK CONSTRAINT all", "RandomValue");

// Does not work
// System.Data.SqlClient.SqlException : Incorrect syntax near '@p0'.
dbContext.Database.ExecuteSqlCommand($"ALTER TABLE {table} NOCHECK CONSTRAINT all");

// Does not work (no interpolated string)
// System.Data.SqlClient.SqlException : Incorrect syntax near '@p0'.
dbContext.Database.ExecuteSqlCommand("ALTER TABLE {0} NOCHECK CONSTRAINT all", table);
// System.Data.SqlClient.SqlException : Incorrect syntax near '@table'.
dbContext.Database.ExecuteSqlCommand("ALTER TABLE @table NOCHECK CONSTRAINT all", table);

// Does not work (SqlParameter)
// System.Data.SqlClient.SqlException : Incorrect syntax near '@table'.
dbContext.Database.ExecuteSqlCommand("ALTER TABLE @table NOCHECK CONSTRAINT all", new SqlParameter("table", table));
dbContext.Database.ExecuteSqlCommand("ALTER TABLE @table NOCHECK CONSTRAINT all", new SqlParameter("@table", table));
dbContext.Database.ExecuteSqlCommand("ALTER TABLE @table NOCHECK CONSTRAINT all",
                                                     new SqlParameter("@table", SqlDbType.Char) {Value = table});
// System.FormatException : Input string was not in a correct format.
dbContext.Database.ExecuteSqlCommand("ALTER TABLE {@table} NOCHECK CONSTRAINT all",
                                                     new SqlParameter("@table", SqlDbType.Char) {Value = table});

What am I missing?
I see that 2.2 has just been released, so I will check what that does.
Further technical details
EF Core version: 2.1.3
Database Provider: Microsoft.EntityFrameworkCore.SqlServer
Operating system: macOS
IDE: Rider 2018.2

@bobvandevijver It's not 100% clear to me what your expectations are, but two important points are:

SQL does not allow parameterizing the table name in the query. (At least, not without tricks.)
String interpolation with this method attempts to create parameterized queries.

The API here is planned to change in 3.0, which might make it easier to understand what is happening. See #10996


Updates imports from OpenApi to new version

Hi,
The new OpenApi has some minor changes in the import path. This pull request fixes them.
Yours truly,
Iman Tabrizian

Can you also update the minimum version of the jane openapi requirement in order for this to work ?

Sure I will :)

According to the source code of OpenApi v1.3.

QueryParam is deprecated since 1.3 and will be removed in 2.0, please use Joli\Jane\OpenApi\Runtime\Client\QueryParam instead.

It is since v1.3.

Thanks !


tox fails on azure-storage-file-datalake

When running tox in azure-storage-file-datalake
azure-sdk-for-python\sdk\storage\azure-storage-file-datalake>tox -c ../../../eng/tox/tox.in
Get the following error:
ERROR: Could not open requirements file: [Errno 2] No such file or directory: 'dev_requirements.txt'
The dev_requirements.txt file doesn't exist in that service subfolder.

@xiafu-msft Emily, can you please take a look at this bug.

Hi @jongio
We can add dev_requirements.txt if it will be used.
Currently all dependencies for it will be automatically added by running python script/dev_setup.py under azure-sdk-for python, because it is using the same dependencies as some other packages.
BTW what were you trying to generate by calling tox? (what role dev_requirement.txt is performing there?)

tox is used to run tests in venvs configured for various python versions and environment settings. Please see the contributing guide for more info.

https://github.com/Azure/azure-sdk-for-python/pull/8828

The pr has been merged, close this for now, feel free to reopen it if you have any concern! ^_^


azure-cosmos: access_cosmos_with_resource_token.py - Authorization header doesn't confirm to the required format

Package Name:  azure-cosmos
Package Version: 4.1.0
Operating System: Windows
Python Version: 3.8.2

Describe the bug
After generating a token for a user, any operations fail with the following 401 error:
Authorization header doesn't confirm to the required format. Please verify and try again.
To Reproduce
Steps to reproduce the behavior:

Set-up local Cosmos-DB Emulator
Copy sample code and configure host parameters.
sample code: https://github.com/Azure/azure-sdk-for-python/blob/4f96d8a8dfde9b4cac9d2103b5f58d5705b26ded/sdk/cosmos/azure-cosmos/samples/access_cosmos_with_resource_token.py
Execute script

Expected behavior
Expected behavior is a http response of 200 with a body containing requested data.
The sample code successfully does the following:

Creates a database
Creates a container
Creates a user.
Generates a token.

However, when the token is used in any subsequent request, it returns a 401 error with:
Authorization header doesn't confirm to the required format. Please verify and try again.
Example of a token:
'type=resource&ver=1&sig=DojUybVKTy7H4o9f7GUtjw==;3N789ulXD6B5f+Bvyd6nkQ6wgNkGCpoUqn5SvDN/s16DS/NjW6r/nYYcHiTOG423Ue4gPhRqTlw6/BbM3TQID1LA1pvoS9Wqt1oanFBbj/nogj/hI/h3tkrXuA6wcqPrLbHI+hh6UMICaN1XaXtKGyUTJ2d2jZ2YhIVPLs9qvYtGn4aoC561nwB8m6b3PcCuMUyAB4OfqP6PM6pZ9Bc4f0Pa/Ci/IylUPg4kYUclSKg=;'
Additional context
I have also tried using this sample code in a HTTP Azure function. It can generate the token but the same issue persists regarding the required format.

I have actually solved the problem but now face something slightly different. Some reason, the code sample in the original post did not encode the token before using it. I used the urlib library to encode the token before passing to the request.
I have created a permission definition to grant a user access to a specific partition key. However, when i use the token generated with the partition scoped query, I receive the following error:
Insufficient permissions provided in the authorization header for the corresponding request. Please retry with another authorization header
I can successfully read a document when I give a user ALL permissions but this is not something I want to do for security purposes.

Can you post some code that reproduces the issue? "partition scoped query" can be done a couple different ways. You will need to include the partition key in the options object, not in the query itself.

Hi @Mja101, I was able to reproduce your original issue from the sample. I'm now adding the urllib changes, could you link me an updated code sample to reproduce your new issue?
Some docs for reference that I've been using: https://docs.microsoft.com/en-us/rest/api/cosmos-db/access-control-on-cosmosdb-resources#authorization-header

Hi @Mja101, I resolved this issue in the sample and will update it.
The key for each token dictionary needs to be container.container_link instead of container.id, I'll push a PR soon.
cc @srinathnarayanan

Hi @zfoster ,
Thanks for the replies. I have tried changing container.id to container.container_link but I still recieve the same error when I try to access a resource based on a specific partition.
Insufficient permissions provided in the authorization header for the corresponding request. Please retry with another authorization header
The code to create the permissions dict:
        permission_definition = {             "id": PARTITION_READ_PERMISSION,             "permissionMode": documents.PermissionMode.Read,             "resource": container.container_link,             "resourcePartitionKey": [serial_number],         }
The body of the POST:
{   "query": "SELECT * FROM info c WHERE c.serial_number = @serial_number",   "parameters": [{"name": "@serial_number", "value": "0000000392"}],   "partition_key": "0000000392" }
I can only access the data when I change documents.PermissionMode.Read to documents.PermissionMode.All.
I've tried with different users and still receive the insufficient permission error. Any ideas?

Taking a look

@Mja101 the sample is working for me, using this query:
def token_client_query(container, username):
    try:
        for item in container.query_items(
            query="SELECT * FROM my_container c WHERE c.username=@username",
            parameters=[{"name": "@username", "value": username}],
            partition_key=username,
        ):
            print(json.dumps(item, indent=True))
    except exceptions.CosmosHttpResponseError:
        print("Error in querying item(s)")

and this code:
        # Give user read-only permission, for a specific partition
        user_2 = create_user_if_not_exists(db, USERNAME_2)
        permission_definition = {
            "id": PARTITION_READ_PERMISSION,
            "permissionMode": documents.PermissionMode.Read,
            "resource": container.container_link,
            "resourcePartitionKey": [USERNAME_2],
        }
        permission = create_permission_if_not_exists(user_2, permission_definition)
        read_token = {}
        read_token[container.container_link] = permission.properties["_token"]

        # Use token to connect to database
        token_client = cosmos_client.CosmosClient(HOST, read_token)
        token_db = token_client.get_database_client(DATABASE_ID)
        token_container = token_db.get_container_client(CONTAINER_ID)

        # Fails since this client has access to only items with partition key USERNAME_2 (ie. "user2")
        token_client_read_all(token_container)

        # Ok to read item(s) with partition key "user2"
        token_client_read_item(token_container, USERNAME_2, ITEM_3_ID)

        # Can't upsert or delete since it's read-only
        token_client_upsert(token_container, USERNAME_2, ITEM_3_ID)

        # Query for items in a certain partition !!!! this works
        token_client_query(token_container, USERNAME_2)

See the 2nd to last line - this seems to be the same code as what you've pasted?

@zfoster
Finally got it all working. Thanks for all your help mate!

@Mja101 no problem! take care


Update copyright date to 2024

Summary

Checklist

[ ] The Pull Request has tests
[ ] A changelog file has been created under changelog/unreleased/kong or skip-changelog label added on PR if changelog is unnecessary. README.md
[ ] There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Issue reference

Fix #[issue number]

 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.achiel seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.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.achiel seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Added answer_length.py

Simply added the custom metric class for answer length to its own file in the metrics dir - didn't do anything to any other processes and not sure if it should be included anywhere other than here, for reference/use

failed the lint formatting - lmk what you're using locally (if it matters) so I can edit and rerun but feel free to add anything you've been using on your end if no other changes

@j-space-b Thanks!
For linting (for now) we use black. Planning to do a major code clean-up including testing, mypy and ruff soon (ticketed up but unfortunately never got around to it).

just reformatted - getting an error on the PR comment resource (below) - possibly due to git permissions - lmk what I can do to keep adding PRs, otherwise will let you take this one in Error: Resource not accessible by integration

Thanks - should be able to ignore that github action - I'll look to remove it. It hasn't really been used and is more just a way to tell others what to install to test out deepeval!

@j-space-b I see a few extra committed .idea files - are you able to remove those from the PR so we can merge this in? If it's easier - might be better to create a separate PR :)

@ColabDog removed from this PR all good

Thanks @j-space-b
Just wondering with this implementation, it does a super simple count of the length of the number of characters.
Curious on whether you think it's useful to also consider the following:

If there's empty spaces before and after (or if this should be left to the user). I've seen lower-parameter GPT models have a lot of empty spaces before and after the text generation that could inflate this metric.
Are there edge cases that length doesn't capture? I feel if code is being generated, len might not be the best metric? Could you specify in some documentation inside docs/docs/metrics  about the main use case of this?

In addition, could you please also add the following?

[ ] A test so we can capture the desired behaviour any ignore any undesired behaviour
[ ] Any documentation outlining this example

np ^ totally agree with the preceding/trailing spaces, good call, added that!  added documentation that stated original intent (to measure brevity along with any other metrics, but left to user to decide if relevant to other modalities e.g. code).  added to test_utils.py to test output.

Oops - test_utils isn't really a test - I"ll make sure to rename that to avoid the confusion. All tests sit inside the tests folder. As an example  - I would recommend looking at tests/test_bias.py to see where the tests currently lie.


get_document_multi_form doesn't play well with WAGTAILDOCS_DOCUMENT_MODEL.

The function wagtail.wagtaildocs.forms.get_document_multi_form() builds it's Form object from a fixed set of field names: ['title', 'collection', 'tags']. This means that the multiple-upload form can never set any other fields on a custom model that's set through WAGTAILDOCS_DOCUMENT_MODEL.
The PR for this would be easy, but I haven't got time to make it right now. You'd just need to change the function to this:
def get_document_multi_form(model):
    return modelform_factory(
        model,
        form=BaseDocumentForm,
        fields=[field for field in model.admin_form_fields if field != 'file'],
        widgets={
            'tags': widgets.AdminTagWidget,
            'file': forms.FileInput()
        })

That does the equivalent of what get_document_multi_form() does today, while also making sure it includes custom fields.

I've gone ahead and created a PR for this: #3928.

Completed in #4840


refactor RPC -> gRPC

Can we simplify the RPC reature and just use gRPC over a unix domain socket please?

deprecating the RPC feature - i think its no longer needed - had a good run however


Return empty page token for last response

According to protobuf specification, the next token should be empty for
the last.
	// The next pagination token in the list response. It should be used as
	// page_token for the following request. An empty value means no more result.
	NextPageToken        string   `protobuf:"bytes,2,opt,name=next_page_token,json=nextPageToken,proto3" json:"next_page_token,omitempty"`

Signed-off-by: Liron Levin<EMAIL_ADDRESS>

Thanks for the PR @liron-l


(WIP) How to set up SSL/TLS when running a self hosted Citus

Why are we implementing it? (sales eng)
Requirement from security compliance team within Microsoft
Communication goals (e.g. detailed howto vs orientation)

What does Citus do by default? A: for new clusters self-signed certificates, for old clusters no encryption by default
Why are self-signed certificates not secure (and thus also the Citus default)? A: MITM attack is still easily possible
What does a user need to do to get ssl=verify-full working? A: Get certificates from a CA (either company internal PKI or publicly trusted). Install those certificates. Manage key rotation. Do this for all nodes in the cluster.
Mention hostssl for denying unencrypted connections

Good locations for content in docs structure
How does this work? (devs)
Example sql
Corner cases, gotchas
Are there relevant blog posts or outside documentation about the concept/feature?
Postgres docs on setting up SSL/TLS
pg_auto_failover docs on this same topic:
https://github.com/citusdata/pg_auto_failover/blob/release-1.2/docs/security.rst
Link to relevant commits and regression tests if applicable

Due before April 1


Add BIP141 tab

BIP49 uses P2WPKH Nested In P2SH
This is also currently provided as an option in the BIP32 tab.
But BIP141 specifies several other formats for segwit addresses.
https://github.com/bitcoin/bips/blob/master/bip-0141.mediawiki#Examples

P2WPKH
P2WPKH nested in BIP16 P2SH
P2WSH
P2WSH nested in BIP16 P2SH

Maybe the tool could facilitate the generation of these other types of addresses

See

https://github.com/iancoleman/bip39/commit/9f59e99bec5cc5faa63c0296f064a09b0745291f - Update bitcoinjs from v3.1.1 to v3.3.0
https://github.com/iancoleman/bip39/commit/0fd67b5915a08168d31caaa581f55cf86591be73 - Remove segwit from bip32 tab
https://github.com/iancoleman/bip39/commit/c49e881294343b762109bb8104e7c1b45898c894 - Add BIP141 tab for full segwit compatibility

P2WSH and P2WSH-in-P2SH do not apply to this tool.


Incorrect string on rebranding page.

When I go to https://<onlyoffice.mydomain.com>/controlpanel/rebranding/, I found that the description text of dark theme logo is "light":

That's because they use the same variable LogoDarkAbout.
https://github.com/ONLYOFFICE/ControlPanel/blob/99248d6b77e6f2fef87c59f7423bfce7a350476b/web/www/views/rebrandingPartial.pug#L14
https://github.com/ONLYOFFICE/ControlPanel/blob/99248d6b77e6f2fef87c59f7423bfce7a350476b/web/www/views/rebrandingPartial.pug#L29

By the way, you may have noticed that the text of LogoDarkAbout is "light". The root cause is the incorrect text in the resource file:
https://github.com/ONLYOFFICE/ControlPanel/blob/99248d6b77e6f2fef87c59f7423bfce7a350476b/web/www/public/resources/CPWhiteLabelResource.json#L6-L9
Maybe a lot of Dark/Light related variables also have to be swapped, to resolve this issue completely...

Hello @fz0000 yes you are right. We will fix this in future releases.


feat: test very large logs

The logging API defines some limits to log sizes here. We should stress test around these limits to make sure the library behaves consistently with large logs

Great idea :+1:. What would be a best place for putting the tests (per language)? I am thinking about samples/ folder since it is a kind of separate module in each language, so we can use extra dependencies there without influencing the library itself.


Footer UI defects on Redoc sidebar

Hi there, I built the Redoc component as the main page and the footer is showing some glitches as the Redocly watermark shows above it. I'm using the default themes. Any ideas or tips to improve it? Thx

Can recreate this on the example website - https://redocusaurus.vercel.app/docs/nested/nested-1#tag/store_model

Also note that depending on the screen size, when using three colums they stack too soon

Any ideas or tips to improve it?

I don't really have a fix for this. You might have to fix this with some custom css like the custom fixes already added here - https://github.com/rohit-gohri/redocusaurus/blob/main/packages/docusaurus-theme-redoc/src/theme/Redoc/styles.css
Would be happy to merge and release a PR.

Also note that depending on the screen size, when using three colums they stack too soon

That is just default docusaurus theme, nothing to do with redoc.

If you are looking for a quick fix. Then you can add this css to your custom.css file.
.menu-content .scrollbar-container > ul + div {
    position: relative;
}

Thanks @Innders , however I don't see any .menu-content or .scrollbar-container on my code? 🤔

@Innders well I recreated the file as I removed it at some point, but it did not work. I had this when I initialised: https://github.com/jrub/redocusaurus_test/commit/ee18599ee163e3d0949861c170bce6fcf959d566#diff-44813f307e729042af7e70beff6f32ea63a7941bc100043a49e84d229ea2570f so I think it might be a different thing form what you're getting? (as again, those classes are not around). Thanks again!

@jrub make sure you uncomment this line in your docusaurus.config.js file. Otherwise the CSS file won't be included in your build and none of those styles will be applied.
I just tried it in your repo and it worked for me :)


Port local lucy to ci

Porting @MartinMinkov great job in implementing local test executive engine to CI
Closes #14790

!ci-nightly-me

!ci-build-me

!ci-nightly-me

!ci-build-me

!ci-build-me

!ci-nightly-me

!ci-nightly-me

!ci-nightly-me

!ci-nightly-me

!ci-nightly-me

!ci-nightly-me

!ci-nightly-me

!ci-nightly-me

!ci-nightly-me

!ci-nightly-me

!ci-nightly-me

!ci-nightly-me

Test passed: https://buildkite.com/o-1-labs-2/mina-end-to-end-nightlies/builds/1881#018f0b80-1dd9-453e-adc3-e87580583728

!ci-build-me


I don't need two actionpack&activesupport versions in my app.

Hi there! We're doing some open source maintenance work and closing old PRs. If you would still like this PR to be merged in, please re-open it by April 17, 2017. Thanks!


Fix typo

updated readme
stop by to say hi -
[ stop saying hi🙌🏼
Share your issues we are happy to help 💯🚀
Thanks for reading🤗 ]

We don't accept pull requests with style fixes. closing this PR


Move all errors out to one enum

Currently have a bunch of different error types (Key, Layer, etc.), which makes error handling when combining different modules more cumbersome. Not sure if this is the best approach

295fcd4b33d86fe623b71636b96fe48b8e7b06d8


Alert! Alert! Alert! Incompatilbe with leatest vesrion of Kali 2021.2 and Python3 which is wrost . MUST READ!!!!

This Software is not Compatible with the Latest Version of Kali 2021.2 and Python 3 which is wrost. Since I am using an up to date Platform so this Tool should be updated rather forcing the users to downgrade python then install it if you know an Alternative Software/Tool for SS7 attacks then Mention it.

just use a virtual env

just use a virtual env

Hey mate can we connect and have a programmatic relationship.


Undefined property $config in Zend_Http_Client_Adapter_Curl

Hey!
I just upgrade my deps from 1.12.14 to 1.12.15 and it seems there is a break in the curl http adapter. At this line and this one, the code relies on $config whereas the good one should be $_config since the property has been renamed.

Ping @weierophinney…


Intermittent errors using rhyme carriers in the sandbox

Input:

verb::&listen and noun::&listen

Sometimes expected output, like press and dress, but also sometimes getting Unknown error

The unknown error means there is a bug in the query engine. I'll investigate this.


Vision framework compatibility

I am using the Vision framework but the program crashes as soon as I access the pixelBuffer property of the result observation. This is my code:
    VNCoreMLModel * model = [VNCoreMLModel modelForMLModel:fns.model  error:&error];
    VNCoreMLRequest * request = [[VNCoreMLRequest alloc] initWithModel:model completionHandler:^(VNRequest * _Nonnull request, NSError * _Nullable error) {
            if(error) {
                NSLog(@"completionHandler Error %@", error);
                return;
            }
            for (VNObservation * observation in request.results){
                if(observation.class == [VNPixelBufferObservation class]){
                    VNPixelBufferObservation * pxbufobs = (VNPixelBufferObservation*)observation;

                    CIImage *ciImage = [CIImage imageWithCVPixelBuffer:pxbufobs.pixelBuffer]; // <<---EXC_BAD_ACCESS here!!!

                    CIContext *temporaryContext = [CIContext contextWithOptions:nil];
                    CGImageRef videoImage = [temporaryContext
                                             createCGImage:ciImage
                                             fromRect:CGRectMake(0, 0,
                                                                 CVPixelBufferGetWidth(pxbufobs.pixelBuffer),
                                                                 CVPixelBufferGetHeight(pxbufobs.pixelBuffer))];

                    // resulting UIImage
                    UIImage * image = [UIImage imageWithCGImage:videoImage];
                    CGImageRelease(videoImage);
                }

            }
        }];

The fns is a .mlmodel converted with this project and dragged into my xcode project.
Interestingly, the models offered here do not work in xcode 9.1. It complains about the missing header file (although it is there). So, I was not able to test with mlmodels from alternative sources.

I also have bad_access in VNPixelBufferObservation. I think it is because errors on Vision framework side as direct CoreML model usage is ok. I can suggest a simple workaround:

Don't use image output in CoreML model. You need to convert models with these lines https://github.com/prisma-ai/torch2coreml/blob/master/example/fast-neural-style/convert-fast-neural-style.py#L155-L161 commented or deleted. You model will have multi-array output instead of pixelBuffer.
Convert multi-array to image manually. Handy library for array->image function as well as multi-array manipulation functions: https://github.com/hollance/CoreMLHelpers

I create a quick and straightforward example using CoreMLHelpers (only for demo purpose, you need to handle errors more cleverly): https://gist.github.com/opedge/1e3a80528e2d30d2238bc7b18e0a2020
Please note that you need to add bias to output image and convert it from BGR to RGB manually.


Various fixes and improvements from 11-2022 OCPP Plugfest

Reset Handling:

stopping ocpp service properly before killing process on reset
improved handling of Reset.req - Transactions will be stopped properly before the actual reboot
Now sending StatusNotification.req after successful reconnect for all connectors

Authorization Handling:

Configuration key AuthorizeRemoteTxRequests is now rw
improved authorization handling and consideration of keys LocalPreAuthorize, LocalAuthorizeOffline, AuthorizationCacheEnabled, LocalAuthListEnabled
RemoteStartTransaction.req for no connector given now supported

Message Queue:

now dropping non-transaction-related msg queue is paused
maintaining time based order of messages in message queue

Other:

fixed bug that deleted ca cert without checking for parameters  issuerkeyhash, issuernamehash and serial
Now properly erasing references of stopped transactions. This fixes the problem of MeterValues that have been sent also after stopped transactions
MinimumStatusDuration is now a considered configuration key

Should be merged with: https://github.com/EVerest/everest-core/pull/142


AttributeError: 'NoneType' object has no attribute 'event_id' in framework.run()

  line 1473, in <module>
    framework.run()
  File "/home/bruno/miniconda3/envs/betfair_pytorch2/lib/python3.9/site-packages/flumine/flumine.py", line 33, in run
    self._process_market_books(event)
  File "/home/bruno/miniconda3/envs/betfair_pytorch2/lib/python3.9/site-packages/flumine/baseflumine.py", line 151, in _process_market_books
    market = self._add_market(market_id, market_book)
  File "/home/bruno/miniconda3/envs/betfair_pytorch2/lib/python3.9/site-packages/flumine/baseflumine.py", line 214, in _add_market
    self.markets.add_market(market_id, market)
  File "/home/bruno/miniconda3/envs/betfair_pytorch2/lib/python3.9/site-packages/flumine/markets/markets.py", line 21, in add_market
    if market.event_id:
  File "/home/bruno/miniconda3/envs/betfair_pytorch2/lib/python3.9/site-packages/flumine/markets/market.py", line 136, in event_id
    return self.market_book.market_definition.event_id
AttributeError: 'NoneType' object has no attribute 'event_id'

Anymore context to this? Code used etc.

it's happening like 3/4 of the time I run flumine, my strategy doesn't touch event_id or anything internal to flumine, this happens right at the start when framework.run() is caleld

removing streaming_timeout seems to fix it

Which markets? / What is your market and data filter?

    filter=streaming_market_filter(
            event_type_ids=["7"],
            market_types=["WIN"],
            # country_codes=["GB"]
        )

    strategy = PlaceSomeBets(market_filter=filter,
                            #  streaming_timeout=.5,
                             conflate_ms=400
                             )

    framework.add_strategy(strategy)
    framework.run()

uncomment the streaming timeout and play around with the values you should be able to recreate the error

Ok, I can replicate, looks like a race condition due to the conflation and a snap which is sending out data with no data

Fixed in this bflw commit


💡 [FEATURE] - Adding best videos link for every study field

Idea Contribution

[X] I have read all the feature request issues.
[X] I'm interested in working on this issue
[X] I'm part of GSSOC organization

Explain feature request
Along with content it is very necessary to add video link as well beacuse not everyone can understand with the content only
Explain your solution
No response
Any alternative approaches/features
No response
Additional Context
No response

hello @jfmartinz
can you please assign me this issue?


Update @trou's experience with Blue Coath

Fixes https://github.com/disclose/research-threats/issues/8
Thank you for that update @trou!

Wrong repo


feat: rating algorithm

This commit reflects the changes made after reviewing and incorporating feedback.
Resolves #234
Will also resolve #209

This will also resolve #209

Oh, I almost forgot - can you please modify migrateKv() to update current items with their respective itemCounts?

Oh, I almost forgot - can you please modify migrateKv() to update current items with their respective itemCounts?

What is the purpose of this script?
Is it because items already in the database do not contain the commentsCount property?
So I would have to count the number of comments in an item and update the value accordingly?

Yes and yes 🙂

Oh, I almost forgot - can you please modify migrateKv() to update current items with their respective itemCounts?

I have a question regarding the migrate_kv.ts file; it would appear that the item ids are hard coded, I'll assume that those are the items that are currently in the prod. database, however it would appear that there exist more that 13 items on the prod. db (website). Would you like me to not touch the ids and only work on updating their key-values?... which in this case I'll be adding the commentsCount property to the item.

The migration script is used whenever there's a change in the database structure. The script, in its current state, has already been run. You can mould it however you'd like now, including removing those item IDs.

Did you need a hand with the migration script?

I've been reconsidering this feature. I'm not entirely convinced that the value it adds, if any, outweighs the added complexity.
Happy to others' thoughts.
CC @mbhrznr, @lino-levan, @brunocorrea23

I'm pretty neutral on this. I'm not a fan or complexity but this is a nice feature. Up to you and the others.

Yeah.. I agree with you. I'm not so sure about it now.
When I created #234 , I believe we were only sorting by votes, which is bad because very old and irrelevant post might appear first. But now that we have the filters for last week, last month and all time, this becomes less of a problem.
I still think that #209 is a nice feature as it indicates how 'hot' is the topic. If we sliced it only to this issue, would you still think that the value outweighs the complexity?

Yeah, I like the comment count aspect, but the algorithm functionality may not be needed.

as @brunocorrea23 already mentioned, having the filters available reduces the need for the algorithm quite a bit.
however the sorting algorithm itself doesn't seem too complicated, does it?
if we'd slice it to only account for the item's comment count, we'd need the migration and the added complexity in item creation (and deletion) anyway, wouldn't we?
or would we try to find another approach on how to integrate the comment count?

The implementation of the sorting algorithm is straightforward. It's more the algorithm design that could become tedious. For example, why do the weight coefficients have those values in particular? There isn't a clear objective reason why they would use any given values for this use case (Deno Hunt).
The aim is to have items the user likes more likely to appear at the top. Genuinely, IDK that such a solution is that much better than just using votes.
I can see the comments count as helping with that a bit.

if we'd slice it to only account for the item's comment count, we'd need the migration and the added complexity in item creation (and deletion) anyway, wouldn't we?

Yep, that's right. But it may be worth it.

It seems it'll be fine to skip the algorithm functionality on this change, @roberto-morado. However, having the item's comments count is still a good idea.

Understood.
Also; I apologize for the late response, I became quite busy at my day job this past week.
Let me know if there is anything else I can help with. I love the app, I'd like to contribute more.


The genetype page links appear to be broken

I tried some on the rad21 page
https://www.pombase.org/genotype/mis4-C-A-1-191,K873A,K877A-amino_acid_deletion_and_mutation-expression-not_assayed__rad21M-middle%3F-other-expression-not_assayed

It's only multi gene ones

It looks like it's only a problem for the genotypes containing the allele "rad21M(middle?)".  I'm guessing the question mark is what's triggering the bug.  I'll fix that tomorrow.  :-)

Here are some other examples here: https://www.pombase.org/gene/SPBC30B4.01c

well it shouldn't be there really! DOn't worry about this at the weekend.

I need to chase those up too! on my list...

All fixed now.

I'm guessing the question mark is what's triggering the bug.

Question marks need special handling in URLs.


Configuration issue: errors sending email with Strapi 3.3.4

I followed the described setup steps but I found some errors on email sending and the email are not sent (see at the end for the error logs)
It could be useful to have a more detailed setup steps about Gmail/OAuth configuration.
This is my situation:

"Gmail API" enabled for my project
"Oauth consent screen" configured as:
Publication Status: "Test in progress" (not "Production")
User type: "External"
Users added: my personal gmail address and also the gmail address of the account I'm configuring GoogleCloudPlatform
Service Account configured as:
Enabled
Email something like this<EMAIL_ADDRESS>Enable delegation across the G Suite domain: checked
Created a new Private key for your service account
Private JSON key active and downloaded in my local machine
Created file /config/plugins.js following the provided example.
Provider options

module.exports = ({ env }) => ({
  // ...
  email: {
    provider: 'gmail-2lo',
    providerOptions: {
      username: // <-- here which email should I use?
      clientId: // <-- here I put the "client_id" value of my private JSON key
      privateKey: // <-- here I put the "private_key" value of my private JSON key
    },
    settings: {
      defaultFrom<EMAIL_ADDRESS>      defaultReplyTo<EMAIL_ADDRESS>    },
  },
  // ...
});

Error logs:

Using providerOptions.username with Service Account email (like "service-account-name@project-xxx.iam.gserviceaccount.com"):

Error: invalid_request: Principal must be an email address
[{"messages":[{"id":"Auth.form.error.email.invalid"},{"mailer_error":"invalid_request: Principal must be an email address"}]}]

Using providerOptions.username with one of the users added in Oauth consent screen:

Error: unauthorized_client: Client is unauthorized to retrieve access tokens using this method, or client not authorized for any of the scopes requested.
[{"messages":[{"id":"Auth.form.error.email.invalid"},{"mailer_error":"unauthorized_client: Client is unauthorized to retrieve access tokens using this method, or client not authorized for any of the scopes requested."}]}]

What I missed? Can you please help me to correctly configure it?
Strapi version:  3.3.4 (node v12.18.4)

I have the same issue!

Hey @lucap84,
Thanks for trying out the provider and waiting for my response, had some busy days (moved houses and had holidays).
Regarding your first point: the providerOptions.username indeed needs to be an existing email address within your domain/project and not the service-account email.
Are you looking for a "three-legged OAuth", then you should use this provider or a fork of it.
Or do you make use of Google Workspace (prev. G suite), then you need to follow some extra steps: https://github.com/MattieBelt/strapi-provider-email-gmail-2lo/blob/master/README.md#g-suite, following those steps when having G workspace should solve your second error. If this is not the case feel free to let me know.
@josemanuelguzman Are you also use Google Workspace, if so please follow the linked steps from the Readme.

Hey @lucap84,
Thanks for trying out the provider and waiting for my response, had some busy days (moved houses and had holidays).
Regarding your first point: the providerOptions.username indeed needs to be an existing email address within your domain/project and not the service-account email.
Are you looking for a "three-legged OAuth", then you should use this provider or a fork of it.
Or do you make use of Google Workspace (prev. G suite), then you need to follow some extra steps: https://github.com/MattieBelt/strapi-provider-email-gmail-2lo/blob/master/README.md#g-suite, following those steps when having G workspace should solve your second error. If this is not the case feel free to let me know.
@josemanuelguzman Are you also use Google Workspace, if so please follow the linked steps from the Readme.

I hoped this worked for you guys.
Closing this, let me know if you still run into this problem.


bump embedded-graphics-core: 0.3.2->0.4.0

There was some trait bound issue compiling the minimal example for the Raspberry Pi while using the I²C interface (rppal crate with hal feature enabled).
Updating to embedded-graphics-core = 0.4.0 solves the problem and results in a working display setup.

@jamwaffles what do you think?


BUG: showing symbol instead of space in certain occasions (Win 11)

using any effects with dir command on windows 11, a strange symbol appears instead of blank/space. See below example in the summary of bytes/bytes free:

it should actually look like (see last two lines):

Interesting, can you post a screenshot of what that output looks like without running through TTE?  And does this bug happen for all effects?

Interesting, can you post a screenshot of what that output looks like without running through TTE? And does this bug happen for all effects?

@ChrisBuilds: (1) updated bug-post without tte + (2) yes, this happens with all effects


SignalR/CORS: Credential is not supported if the CORS header ‘Access-Control-Allow-Origin’ is ‘*’

Describe the bug
Hi! I just upgraded from ASP.NET Core 2.1.6 to 2.2.0, everything working just fine except my connections to SignalR. In the negotiation phase (using javascript client and over websocket) I get a 204 with the error

Cross-Origin Request Blocked: The Same Origin Policy disallows reading the remote resource at ‘https://localhost:44333/ChatHub/negotiate’. (Reason: Credential is not supported if the CORS header ‘Access-Control-Allow-Origin’ is ‘*’).

Taking a look at the request I have the following

So the server sends out a '*' instead of just sending out the origin from which the request came. The app needs to allow all origins since our users can add a chat to their own site.
In my Startup.cs I have defined CORS rules as follows
services.AddCors(options =>             {                 options.AddPolicy("AllowAllOrigins", b =>                 {                     b.AllowAnyOrigin()                     .AllowCredentials()                     .AllowAnyHeader()                     .AllowAnyMethod();                 });             });
Mind that this was working on 2.1.6 and earlier versions.
In the client side code I can get the chat to work if I set skipNegotiation to true, like this
connection = new signalR.HubConnectionBuilder()                         .withUrl(serverURL + "ChatHub", { skipNegotiation: true, transport: signalR.HttpTransportType.WebSockets })                         .build();
But I'm guessing that will have it's own consequences and is not solving the real problem.
Been reading through so many threads and I just can't get this to work again, anyone have any ideas as to might be causing it?
To Reproduce
Steps to reproduce the behavior:

Using ASP.NET Core 2.2.0
Create a Hub, add CORS settings and use a JS client to try to connect from a origin other then where the server is running

Expected behavior
To not recieve the CORS error, instead have the server return the allowed origin as a URL instead of wildcard '*' since this won't work with credentials, which is needed for SignalR as I understand it (for sticky cookies)

Also, since setting skipNegotiation to true "solves" the problem, what does that actually do? With it set to true chatting works fine - does that mean the CORS settings are working overall, but just not for the negotiation part?

skipNegotiation will skip the HTTP negotiate that does transport fallback and will only run WebSockets. WebSockets and CORS are not compatible so you end up skipping CORS when using that.
As far as your actual problem, in 2.2 AllowAnyOrigin + AllowCredentials now fails on purpose because it is insecure and I believe not allowed by the CORS spec. You need to explicitly specify your allowed origins.

Thank you @BrennanConroy
That's quite bad news then since SignalR requires credentials by default and my app has lots of users with lots of different origins they are calling my app from. Do you have any suggestion for what would be a good solution for this, can the credentials requirements for SignalR be disabled for example (and then removed from my Startup.cs)?
Adding all origins separately is unfortunately not an option as I see it with how our app is working.

@pranavkm For docs.
Currently your CORS does nothing (allows everything), so you could just disable it by writing a custom middleware that returns the "correct" response to the browser.

I am curious what your scenario is that causes you to host the browser client from many different origins.

so you could just disable it by writing a custom middleware that returns the "correct" response to the browser.

Hmm, OK - can't say I have any deeper knowledge on how to do that but thanks I'll start reading up on how to do it!

I am curious what your scenario is that causes you to host the browser client from many different origins.

The app offers users to place chat widgets on their sites, it's a SaaS so users can then over at our app chat with users on their site 😃

@pranavkm For docs.
Currently your CORS does nothing (allows everything), so you could just disable it by writing a custom middleware that returns the "correct" response to the browser.

An update regarding this. Thank you so much for sending me the right way! I implemented a custom CORS middleware and now everything is working just as before. For reference is anyone else stumbles upon the same problem: https://stackoverflow.com/questions/44379560/how-to-enable-cors-in-asp-net-core-webapi

Will there be a fix?

@solojuve1897, @endeffects, I solved the problem through my own Middleware.
CorsMiddleware.cs
public class CorsMiddleware
{
   private readonly RequestDelegate _next;

   public CorsMiddleware(RequestDelegate next)
   {
         _next = next;
   }

   public Task Invoke(HttpContext httpContext)
   {
      if (httpContext.Request.Headers.TryGetValue("Origin", out var originValue))
      {
         httpContext.Response.Headers.Add("Access-Control-Allow-Credentials", "true");
         httpContext.Response.Headers.Add("Access-Control-Allow-Headers", "x-requested-with");
         httpContext.Response.Headers.Add("Access-Control-Allow-Methods", "POST,GET,OPTIONS");
         httpContext.Response.Headers.Add("Access-Control-Allow-Origin", originValue);

         if (httpContext.Request.Method == "OPTIONS")
         {
            httpContext.Response.StatusCode = 204;
            return httpContext.Response.WriteAsync(string.Empty);
         }
      }

      return _next(httpContext);
   }
}

public static class CorsMiddlewareExtensions
{
      public static IApplicationBuilder UseCorsMiddleware(this IApplicationBuilder builder)
      {
         return builder.UseMiddleware<CorsMiddleware>();
      }
}

Startup.cs
public void Configure(IApplicationBuilder app, IHostingEnvironment env)
{
   app.UseCorsMiddleware();
   ...........

@cores-system
Thanx for the answer. What I ended up doing was the following:
In ConfigureServices-method:
services.AddCors(options => options.AddPolicy("CorsPolicy",
builder =>
{
builder.AllowAnyHeader()
.AllowAnyMethod()
.SetIsOriginAllowed((host) => true)
.AllowCredentials();
}));
In Configure-method:
app.UseCors("CorsPolicy");
app.UseSignalR(routes =>
{
routes.MapHub("/hubs/general");
});


"~" option incorrectly trims significant zeroes with "e" option

This does not seem to be correct behavior:
> d3.format("~e")(98765432190)
"9.876543e+1" // <- zero removed here???
> d3.format("e")(98765432190)
"9.876543e+10"

Duplicate of #83. Fixed in #86 v1.4.3 (d3 v5.15.0).


fix lag on toggling mode on some browsers

resolve #19
cause: the loading of the svg images, took the alternate svg out.
there is still albeit negligible lag on test android browser after applying fix.
sorry for messing up the commit history, still learning to use git :)

oops never mind, I am squashing my multiple commits with this merge :)


Is there a way to refresh the token?

Hello everyone, I need to refresh tokens when the discord is blocking me from sending messages.  I have a couple of them and when it comes to the error with discord I need to refresh the token and the program should go on. How can I do this?

To refresh your token, change your password. This will log out ALL your devices as the old token is invalid!


Unity Package Manager Error (Update azure-object-anchors/.../mrtk to Unity 2020)

While trying to follow the instructions on the linked site to update the azure-object-anchors mrtk to Unity 2020 (2020.3.13.f1 in my case) the following error appears:

I'm happy for any help, how to resolve this.

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

ID: 0743a042-5182-bdf7-fb63-839841efa387
Version Independent ID: d549dca3-0a4f-7f50-71c1-de65d77c7c7b
Content: Quickstart: Upgrade Quickstart app to Unity 2020 - Azure Object Anchors
Content Source: articles/object-anchors/quickstarts/upgrade-unity-quickstart-to-2020.md
Service: azure-object-anchors
GitHub Login: @RamonArguelles
Microsoft Alias: rgarcia

@LoGunr Thanks for your feedback! We will investigate and update as appropriate.

@SaurabhSharma-MSFT Ok, as far as I got it, problem has something to do with the ssh protection of my institution - I don't know if it's worth it to mention this in the documentation, but you can see this error as resolved.


It's time to drop IE6 support

IE6 usage in the US has officially dropped below 1%.
http://www.ie6countdown.com/
<1% in US, Czech Republic, Portugal, Philippines, Ukraine and Mexico.
<2% in Canada, UK, Germany, France, Italy, Spain, New Zealand, Australia
<3% in Russia
Though 5.4% in India, 7% in South Korea, 25% in China, 5.5% in Vietnam, 6% in Japan.
When would coffeescript officially drop support of the browser? IMO, the time has come.

Tane is exactly right. Closing this issue until someone suggests a specific improvement that "dropping IE6 support" would entail that wouldn't require us to drop IE7 support as well.

there are statistics that suggest that IE7 usage is even below IE6
http://www.netmarketshare.com/browser-market-share.aspx?qprid=2&qpcustomd=0
(and some that don't :) )
http://gs.statcounter.com/#browser_version-ww-monthly-201101-201201-bar

@archaeron that is a good point, I think in general 6 + 7 are pretty low now - enough to consider pushing them out from general browser support.
But when it comes to javascript, between 6-9 there isn't really much difference until you hit 9 where it is pretty much full ES3.1 support and 8 is still the highest use.  So really what works in 6 works in 8 is probably a good general rule of thumb (again [].indexOf wasn't added until 9 is probably the most blatant issue I can show right now)

CoffeeScript wants to continue support for IE6 until it's really 100% gone. I would love for that day to come as soon as possible, but it ain't here yet.
For example, on Tuesday night we had to run a JavaScript widget on the NYTimes homepage to display Iowa Caucus results:
http://elections.nytimes.com/2012/primaries/results/IA/widget?slug=home_page_511&template=web_results_wide&exclude_others=yes&candidates[]=bachmann&candidates[]=gingrich&candidates[]=paul&candidates[]=perry&candidates[]=romney&candidates[]=santorum&limit=6&state=IA
We didn't happen to write it in CoffeeScript -- but if CoffeeScript had dropped support for IE6, we wouldn't have been able to write it in CoffeeScript.
Keeping IE{6,7,8,9} support is one of CoffeeScript's biggest and most important features.

@jashkenas have you thought about a node coffeescript version that would allow coffeescript to evolve with javascript, (for example, http://disnetdev.com/contracts.coffee/) I believe V8 has Proxy support and something like contracts would be super powerful.  Just curious...

I'm personally of the opinion that when JavaScript begins to truly evolve rapidly in a cross-platform way, and everyone can use JS.next features at the same time ... there won't be much need for a CoffeeScript.
That said, I'd also be willing to make a bet that never actually happens.
But if you feel differently, I'd love to see a fork that targets new JS features.

@jashkenas yea, but why NYTimes doesn't drop ie6 support, like Google etc did? Do you folks care about 0.8% users?
If yes, you should also care about UC browser for PocketPC, NetFront with the similar market share.

Also, IE5 isn't 100% gone. Netscape 4 isn't 100% gone. IMO, that's absurd to target them today.

Unfortunately, it's not just a pure numbers game -- it also matters who has the browser installed. For us, there are a significant number of reporters and editors in the building running IE6. And our number isn't 0.8% -- it varies widely for different websites.

@jashkenas Doug Crockford agrees with you - https://plus.google.com/118095276221607585885/posts/4Dtj2251Mry


Comments for whenNotPaused and whenPaused are confusing

https://github.com/OpenZeppelin/zeppelin-solidity/blob/b07466a4777297a47bed894c487847a4ce201076/contracts/lifecycle/Pausable.sol#L19-L27
 /**
   * @dev modifier to allow actions only when the contract IS paused
   */
  modifier whenNotPaused() {

/**
   * @dev modifier to allow actions only when the contract IS NOT paused
   */
  modifier whenPaused() {

i think should reverse these two comments

Thank you @kkx! Has been fixed in #428.


when it try to read twice file that fromFile will happen Uncaught Promise Error

when I use fromFile to read CSV file.
In the application some time happened read same file because its interactive program.
its simple code.
module.exports = function() {
  return {
    readcsv: async function(path) {
      const json = await csvReader.fromFile(path);
      return json;
    },
  };
};

I had this error
[2020-04-25 19:49:31] Uncaught Promise Error:
[2020-04-25 19:49:31] Error [ERR_STREAM_WRITE_AFTER_END]: write after end
[2020-04-25 19:49:31]     at writeAfterEnd (_stream_writable.js:265:14)
[2020-04-25 19:49:31]     at Converter.Writable.write (_stream_writable.js:314:5)
[2020-04-25 19:49:31]     at ReadStream.ondata (_stream_readable.js:695:22)
[2020-04-25 19:49:31]     at ReadStream.emit (events.js:310:20)
[2020-04-25 19:49:31]     at addChunk (_stream_readable.js:286:12)
[2020-04-25 19:49:31]     at readableAddChunk (_stream_readable.js:268:9)
[2020-04-25 19:49:31]     at ReadStream.Readable.push (_stream_readable.js:209:10)
[2020-04-25 19:49:31]     at internal/fs/streams.js:210:12
[2020-04-25 19:49:31]     at FSReqCallback.wrapper [as oncomplete] (fs.js:488:5)

maybe, I wrong to use promise. if it is true, please close this issue.
Thank you.

@Minori-Akizuki What you need is to execute the csvReader function for each file. something like this:
const csvReader = require('csvtojson') // and not const csvReader = require('csvtojson')()
module.exports = function() {
  return {
    readcsv: async function(path) {
      const json = await csvReader().fromFile(path);
      return json;
    },
  };
};


cmake/UseDoxygen.cmake problem

I believe we had the same issue in ALPSCore and fixed it there: CMake's UseDoxygen complains:
CMake Warning (dev) at cmake/UseDoxygen.cmake:138 (get_target_property):
Policy CMP0045 is not set: Error on non-existent target in
get_target_property.  Run "cmake --help-policy CMP0045" for policy details.
Use the cmake_policy command to set the policy and suppress this warning.
get_target_property() called with non-existent target "doc".
Call Stack (most recent call first):
CMakeLists.txt:28 (include)
This warning is for project developers.  Use -Wno-dev to suppress it.

Thanks! Fixed with d0fa6ae


Update CONTRIBUTING.md

Added emojis to the text

kmk

I don't know if the icons are propriate for the given headlines


Can we have a release from the current state on master to be able to update to php8.0 without having to rely on dev-master?

Yes, we will today or tomorrow max

Yes, we will today or tomorrow max


LOG-13509 remove support info

bump-patch

The same goes on these public repos as on our private ones btw @rsheng-r7.
2 approvals needed.


chore: remove top level promise await behavior

TODO: types
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[ ] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[ ] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

Tests

[ ] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[ ] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpm changeset and following the prompts. Changesets that add features should be minor and those that fix bugs should be patch. Please prefix changeset messages with feat:, fix:, or chore:.

It looks like maybe the generated types don't allow void as a return type for load. Not sure what we think about whether that should be allowed or not. @dummdidumm?

the site deploy failure looks like a real bug, though i'm a bit confused by it

oh it's because the site has a load that returns top-level promises 🤦 fixed

this LGTM — should it still be draft, or can we merge?

I think we can merge, AFAIK it was in draft Mode because of type failures which I fixed


save and load document_type when serializing / deserializing documents

This was implemented in #346 and https://github.com/ChristophAlt/pytorch-ie-hydra-template/pull/130.


feat(scan-pkg-manifest) Add new scan-pkg-manifest job, refactor container scan job

New scan-pkg-manifest job for integration with Lacework Host vulnerability capability
Refactor vuln-scan-run job to ctr-vuln-scan to be more explicit about the job funtionality
Update base container image to use lacework/lacework-cli

Signed-off-by: Scott Ford<EMAIL_ADDRESS>

Moved to https://github.com/lacework/circleci-orb-lacework/pull/11 since we need to point to Alpha first to do a dev release of the Orb.


Slim 2933 test key recovery

Changes by Jan in SLIM-2933 branch

retest this please

retest this please

Retest this please

retest this please

Successful run of the nightlies: https://ci.opensmartgridplatform.org/view/Nightly/job/OSGP_open-smart-grid-platform_nightly/1323/cucumber-html-reports/overview-features.html


encoder/decoder for new row format

Signed-off-by: niedhui<EMAIL_ADDRESS>What have you changed?
implement the encode and decoder of  row format codec
What is the type of the changes?
New feature
How is the PR tested?
unit test
Does this PR affect documentation (docs) or should it be mentioned in the release notes?
no
Does this PR affect tidb-ansible?
no
Refer to a related PR or issue link (optional)
2018-07-19-row-format.md

I think encoders are not necessary.. We only need decoders?

Right, the encoding part should be done in TiDB. I checked the exists v1 encoder table::encode_row in TiKV is only used for preparing data for test. Do we also need preparing encoding data of other versions for test cases?

@niedhui Nice work, add some partial decode benchmark please.

Close for #5725


Supported currency list consistency across instances

Adding to the config.toml I noticed that all of the supported currencies are included in the order they are added to the list and then sats is  appended at the end out of any order that may have been intended by the user. Additionally a user can add sats separately to the list and then see a duplicate sats on the list of currencies.
This issue is based on PR #124 where a discussion came up on how to best organize this input across all instances of Satsale for consistency
I am testing two possible fixes to adjust the user experience for this in config.py line 99
1.An iteration to appends sats if the user did not already input sats as a currency and then organizes the entire list alphabetically
2. Another iteration that organizes the list alphabetically as before but then places sats next to BTC to better demonstrate their relation to one another

Would like some insight on how best to organize this input


setup service shell

Basic service shell implementation
Add a basic docker file for dev env
Use the service shell to send incoming events to the worker app

Closes https://github.com/gldraphael/gracker/issues/12

Got to update the dockerfiles...


Bug center

Bug in Razo 17.0:
Some may show version 16.0.
I think it is Python's problem.

I think this is because of Python.


Authentication does not work on lightninglogin.live

Steps To Reproduce
Steps to reproduce the behavior:

Open https://www.lightninglogin.live
Click on the QR-Code to trigger login dialog (handled by Alby)
Confirm login in Alby
Success notification is displayed, however you are not logged in as you still se the QR-Code. Reloading does not help either.

Expected behavior
After confirming the login you should see the "success" page:

Information About Alby

Alby Version: 1.6.0
Alby installed through Firefox Add-On-Store
Wallet connected with Alby: LNBits

Device Information:

OS: Windows
Browser: Firefox
Browser Version: 97.0.2

Are you working on this issue? No.

Unfortunately there is nothing Alby can do here, the issue is within the auth library used on lightninglogin.live.


Rename ToolboxOverlay to MyToolboxOverlay

This tripped me up for a bit. I thought ToolboxOverlay was a class in the DevTools API. Calling it MyToolboxOverlay would make the example more clear.

What specific example do you have in mind?
Honza

The DomInspector example is the one I'm looking at.
On Tue, Jan 12, 2016 at 1:03 AM, Jan Odvarko<EMAIL_ADDRESS>wrote:

What specific example do you have in mind?
Honza
—
Reply to this email directly or view it on GitHub
https://github.com/firebug/devtools-extension-examples/issues/5#issuecomment-170818217
.

Renamed for the following examples:

DomInspector
NetAnalysis
TitleManager

Thanks for the report!
Honza