[OpenShift] How would I upgrade from 1.4.6 to 1.4.7 via commandline?

Hello! I'm new to helm and I'm having a hard time figuring out how to upgrade from my existing version 1.4.6 SplunkConnect to 1.4.7.
In OpenShift, it does not show there is a version 1.4.7 to upgrade to in the drop down menu, so I'm assuming I have to somehow push new charts that way?
I also attempted to run:
helm upgrade splunk -f values.yaml https://github.com/splunk/splunk-connect-for-kubernetes/releases/download/1.4.7/splunk-kubernetes-logging-1.4.7.tgz
With the exact same values.yaml I customized for 1.4.6. However, I noticed that on spin up, the new daemonset had Privileged set to false, instead of true. If I correct it, my pods start throwing a 'pattern not matched' error. I observed this with 1.4.6 and deleted the whole Helm release and redeployed it, whereby Privileged was initially set to true (it's set to true in the values.yaml file) and it went through. I'm not sure why that flag is switching to false.
What's the right way to do this :(

Have you deployed 1.4.7 now or are you back to 1.4.6?

Have you deployed 1.4.7 now or are you back to 1.4.6?

I'm still at 1.4.6. Do you know which value in the values.yaml corresponds to the pod having the privileged parameter set as true? I want to make sure maybe I'm not missing something.

you can set this .Values.kubernetes.securityContext to true.

Oh perfect. That's what I was missing. Other than that, my original upgrade command worked. Thank you :)


multiple svg images -- property alignment

change the sbgnShapes.draw function to be more specialized for generating the array of svg images.  Also, don't just generate the svg image array, but also the properties for each image

sbgnShapes.draw(node) creates all images and all properties under a single object per sbgn class and caches it.

The other background image properties read from the cached sbgnShapes.draw function and read the specific array of properties from this function.  The function is cached so the images don't have to be regenerated every time.

padding for bg position doesnt seem to work

temporary, easy solution -- generate an empty image if a node doesnt have an item.
e.g node has no clonemarker, no aux items, then the svg image array is:
[linesvg, empty svg image, empty svg image, empty svg image]
Then, each property can assume the convention that the line will always be in index 0, clonemarker index 1, uinfo index 2, statevar index 3, etc.
(still a temporary fix), the caching solution will be less fragile


Rename 'platform' var to something else to avoid conflicts

SUMMARY
Ansible outputs the following WARNING:
[WARNING]: The loop variable 'platform' is already in use. You should set the `loop_var` value in the `loop_control` option for the task to something else to avoid variable collisions and unexpected behavior.
ISSUE TYPE

Feature Idea

COMPONENT NAME
ANSIBLE VERSION
ansible 2.9.6
  config file = /etc/ansible/ansible.cfg
  configured module search path = [u'/root/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/lib/python2.7/dist-packages/ansible
  executable location = /usr/bin/ansible
  python version = 2.7.17 (default, Nov  7 2019, 10:07:09) [GCC 7.4.0]

CONFIGURATION
ANSIBLE_NOCOWS(env: ANSIBLE_NOCOWS) = True
DEFAULT_CALLBACK_WHITELIST(/etc/ansible/ansible.cfg) = [u'profile_tasks']
DEFAULT_HASH_BEHAVIOUR(/etc/ansible/ansible.cfg) = merge
DEFAULT_LOG_PATH(/etc/ansible/ansible.cfg) = /var/log/ansible.log
DIFF_ALWAYS(/etc/ansible/ansible.cfg) = True
DIFF_CONTEXT(/etc/ansible/ansible.cfg) = 3
HOST_KEY_CHECKING(/etc/ansible/ansible.cfg) = False

OS / ENVIRONMENT
Ubuntu 18
STEPS TO REPRODUCE
Import the role from a playbook that has a platform var already set.
EXPECTED RESULTS
The role should work without any issues.
ACTUAL RESULTS
The role outputs a WARNING
[WARNING]: The loop variable 'platform' is already in use. You should set the `loop_var` value in the `loop_control` option for the task to something else to avoid variable collisions and unexpected behavior.

This happens in the following tasks:

lean_delivery.java : Apply security policy patch
lean_delivery.java : Choose platform based task

For the tests I'm doing I've renamed my var to cloud_platform and warning disappears, not sure what is the correct way here, platform is a very generic word that is likely to be used so other people might see the warning too...


Eclipse Active Catalog/Schema width is short for databases with long name

please increase the width of   Active Catalog/Schema to display the database name completely

Do you have any clue how to make it better? Resizing this control isn't the best solution (this will re-layout main toolbar one every connection switch).
I can add tooltip. What else?

I have the same problem. In my case the most important information is at the end.

I think the most intuitive way for the user is being able to resize using the mouse:

There is an option somewhere in preferences that lets you set this, you have to write the number of pixel instead of mouse resize, but it works!

There is an option somewhere in preferences that lets you set this, you have to write the number of pixel instead of mouse resize, but it works!

Thank you. It’s quite hard to find—filtering the options for toolbar, catalog, schema, selector or width does not show it. The path is Window → Preferences → Database → Editors → Schema selector width.
I still think resizing using the mouse would be much easier (also the preferences window could be slightly less overwhelming with the width settings removed).
Not sure if this is related, but in my case after changing the setting DBeaver didn’t start anymore. %userprofile%\.dbeaver4\.metadata\.log shows
!SESSION 2018-11-08 12:09:35.898 -----------------------------------------------
eclipse.buildId=unknown
java.version=1.8.0_181
java.vendor=Oracle Corporation
BootLoader constants: OS=win32, ARCH=x86_64, WS=win32, NL=de_DE
Command-line arguments:  -os win32 -ws win32 -arch x86_64

!ENTRY org.eclipse.ui.workbench 4 2 2018-11-08 12:09:40.060
!MESSAGE Problems occurred when invoking code from plug-in: "org.eclipse.ui.workbench".
!STACK 0
java.lang.NullPointerException
	at org.eclipse.equinox.internal.p2.ui.sdk.scheduler.migration.MigrationSupport.getLastMigration(MigrationSupport.java:236)
	at org.eclipse.equinox.internal.p2.ui.sdk.scheduler.migration.MigrationSupport.baseChangedSinceLastPresentationOfWizard(MigrationSupport.java:197)
	at org.eclipse.equinox.internal.p2.ui.sdk.scheduler.migration.MigrationSupport.performMigration(MigrationSupport.java:77)
	at org.eclipse.equinox.internal.p2.ui.sdk.scheduler.AutomaticUpdateScheduler.earlyStartup(AutomaticUpdateScheduler.java:78)
	at org.eclipse.ui.internal.EarlyStartupRunnable.runEarlyStartup(EarlyStartupRunnable.java:80)
	at org.eclipse.ui.internal.EarlyStartupRunnable.run(EarlyStartupRunnable.java:56)
	at org.eclipse.core.runtime.SafeRunner.run(SafeRunner.java:45)
	at org.eclipse.ui.internal.Workbench$40.run(Workbench.java:2714)
	at org.eclipse.core.internal.jobs.Worker.run(Worker.java:63)

!ENTRY org.eclipse.core.jobs 4 2 2018-11-08 12:09:40.060
!MESSAGE Während "Workbench early startup" ist ein interner Fehler aufgetreten.
!STACK 0
java.lang.NullPointerException
	at org.eclipse.core.internal.runtime.InternalPlatform.getLog(InternalPlatform.java:350)
	at org.eclipse.core.runtime.Plugin.getLog(Plugin.java:270)
	at org.eclipse.ui.internal.WorkbenchPlugin.log(WorkbenchPlugin.java:772)
	at org.eclipse.ui.internal.EarlyStartupRunnable.handleException(EarlyStartupRunnable.java:72)
	at org.eclipse.core.runtime.SafeRunner.handleException(SafeRunner.java:78)
	at org.eclipse.core.runtime.SafeRunner.run(SafeRunner.java:47)
	at org.eclipse.ui.internal.Workbench$40.run(Workbench.java:2714)
	at org.eclipse.core.internal.jobs.Worker.run(Worker.java:63)

I had to delete %userprofile%\.dbeaver4\.metadata\.plugins\org.eclipse.e4.workbench\workbench.xmi to make DBeaver start again.


tvOS support

"Alamofire tvOS" framework + "Alamofire tvOS tests" targets.  Tests all pass and tested in a separate sample app.

Thanks for your PR @bscarano...much appreciated!
I merged a slightly modified version of your PR in 8873379b. I wanted to make sure all the settings were correct, so I created the framework and test targets myself going through each step carefully. I still gave you attribution for the change. I also want to keep this separated from the master branch until it's officially released.
Thanks again! 🍻


After install, component manager broken

After installing this plugin, the component manager is broken ( keeps waiting unlimited time )
When removing the upper layer ( inline edit style ) and click the sync button manualy, resluts in : Couldn't get available versions for package thaiphan/magento2-s3
Guess why that is why the component manager waits unlimited time with wait cursor....
Can it be fixed?
Magento 2.1.7

It can be fixed to get the latest version ( from github ) of the setup/src/Magento/Setup/Model/PackagesData.php  and replace your current ( 2.1.7 ) version of the file.


Saving and reading DateTimeOffset in DocumentDB

Hi,
I am saving some candidate data in Azure DocumentDB with date field as DateTimeOffset as string in the format "2017-01-13T07:30:00+05:30" as per the thread http://stackoverflow.com/questions/31973710/datetime-the-epoch-and-documentdb Document DB supports DateTimeOffset now. But i noticed one thing the data i saved is UTC+5:30 but when i look at the Azure portal its getting saved to UTC by default. Is it an auto conversion done by DocumentDB , because i was able to save data with UTC+5:30 last week , but there was an issue while reading because it was getting converted to UTC. I raised a question in StackOverFlow (StackOverFlow). I am using .NET class DocumentClient to save & read data. Am i missing something?

Any updates on this anyone??/

@ArjunMenonUK Can you please provide the sample code for which you are seeing this issue as I couldn't repro this?
Please see https://gist.github.com/rnagpal/8ac56a8af81f31a41a3c28e080e45b70 for the code that I had for creating/retrieving from documentdb as well as seeing in portal as well.
I'm seeing the same value everywhere.

@ArjunMenonUK Did you got a chance to try this?

Hi @rnagpal, I am out of office till next Tuesday. I had a look @ ur code. Can you try one thing ,change created date property type to datetimeoffset,also did you try running the code locally? When I saved the data to docdb from console app, I had no issue, but observed the UTC conversion once hosted in azure

Whenever I save to doc db from local I see no issue,but once its published to azure its all UTC.

@ArjunMenonUK we are aware of this issue when the DateTimeOffset type is used that it doesn't gets deserialized correctly when reading back, so as a workaround please use this date as string property and convert it into DateTimeOffset after reading the data from DocumentDB. I'll verify the local vs hosted on Azure difference you mentioned and report back. I assume that if you use that as string you are not seeing this issue locally as well as when hosted on Azure.

@rnagpal
Hi,
As of now we have kept date and offset separately and implemented it. String implementation will check and will update you . Anyway thanks for the update

@ArjunMenonUK Closing this issue. Let me know if you still see different values of the datetimeoffset property in local vs hosted on Azure when used as a string property in DocumentDB.

Why was this issue closed? As it stands, DocumentDb has a MASSIVE BUG, let me repeat that...
A MASSIVE BUG
just for clarity
A MASSIVE BUG
in that your SDK is changing data values. Changing the property to a string is an EXTREMELY SHORT TERM WORKAROUND and a horrific one at that. I just downloaded Nuget 1.13.1 and the issue still exists. It is 100% unacceptable that this issue has not been resolved.

@jackbond  i kept my datetime and offset separately as a work around. Felt more of a good method rather than keeping data as string :)

@ArjunMenonUK Yah, their workarounds are terrible. I reported this bug months ago. To be clear, at this point SOMEONE OUGHT TO BE FIRED.
DateTimeOffset is the preferred class for storing dates in .NET. This bug should either be fixed, or the DocumentDb team should stand up and clearly state, WE DO NOT SUPPORT DATETIMEOFFSET.
For this bug to have been closed is simply INEXCUSIBLE.

@jackbond the solution is here
https://github.com/Azure/azure-documentdb-dotnet/issues/292
;)


paypal.button.create doesn't generate form

paypal.button.create(business, data, config, parentNode) creates button only, which doesn't do anything.
Is there any additional steps required to get the button working?
sample code:
paypal.button.create(
<EMAIL_ADDRESS>
    'data-button="buynow" data-name="My product" data-amount="1.00"',
    ' ',
    document.getElementById('pay-btn')
  );

It creates div with class "paypal-button-widget" and images inside.

I had the same problem and I included button.js and at least theese:
data-button="buynow"
data-type="form"
data-upload="1"
data-currency="EUR"
data-item_name_1="name"
data-item_number_1="1"
data-quantity_1="1"
data-amount_1="12.45"

Don't forget paypal.button.process CS example


Ensure topologies are DAG

Ensure all topologies are Directed Acyclic Graphs (DAG).

From what I have found the cgraph library does not ensure that the graph is a DAG. The graphviz.StrictDirected is ensuring the graph does not contain multiply direct links from one node to another or self reference links. See Agstrictdirected.
For the first iteration of the library and the use case within Kuadrant the fact we are not guaranteed to have a DAG maybe acceptable. I think the only way we can create loops with the policy-machinery is by adding linking functions that creates the circular reference. With the current Gateway, Route and Attached Policy I don't see how we can create a graph with a loop.
It would be possible to add a check to ensure there is no loops within the graph but this check would be after the creation of the graph. I am not sure if we would be able to resolve any loops at that stage in the process. This post creation check would be a costly check and would only be required when a new node is added to the graph.
What is really needed is someway within the NewTopology to detect that a loop is being created with the link functions. Added that check I believe would be out of scope of getting Kuadrant to GA.
My opinion is this is a manageable risk and should not stop current progress. For external user of the library it will be more of a risk, but for our use case it is manageable.


Revert "[SPARK-35746][UI] Fix taskid in the stage page task event tim…

What changes were proposed in this pull request?
This reverts commit 450b415028c3b00f3a002126cd11318d3932e28f.
Why are the changes needed?
In #32888, @shahidki31 change taskInfo.index to taskInfo.taskId. However, we generally use index.attempt or taskId to distinguish tasks within a stage, not taskId.attempt.
Thus #32888 was a wrong fix issue, we should revert it.
Does this PR introduce any user-facing change?
no
How was this patch tested?
origin test suites

cc @shahidki31 @sarutak @srowen @dongjoon-hyun

With the same reason, I close this.

https://github.com/apache/spark/pull/35155#pullrequestreview-848444699

You may want to open [SPARK-37860][UI] ..., @stczwd .

cc @shahidki31 @sarutak @srowen @dongjoon-hyun

okey


[Availability] Lazily expand type refinement contexts for extensions.

Testing this as a fix for avoiding circularity in macro resolution when extending a nested type of a parent type that has an attached macro that can added nested types, such as a member macro or an extension macro that adds conformances.

@swift-ci please smoke test

@swift-ci please test source compatibility


bug with critical region?

Hello,
I was trying to have a critical region on my sequenceDiagram and was getting a syntax error.
I tried isolating it with one taken straight out of the documentation.
sequenceDiagram
  critical Establish a connection to the DB
      Service-->DB: connect
  option Network timeout
      Service-->Service: Log error
  option Credentials rejected
      Service-->Service: Log different error
  end

In my jetbrains it shows as "syntax error"

But here and in the live editor it shows fine:
https://mermaid-js.github.io/mermaid-live-editor/edit#pako:eNqFUU1PwkAQ_SubORfCQrGyBw9Yb-qFm-ll3Z3CSruD06mKhP_uQiXRxMQ5zcd7b74O4MgjGOjwtcfosAx2zbatokrmOEhwtlF3ndjnJnQbZZWjGNFJoKiElGxQlcsBfrIV8ltwOBrdlEtzwQ5l2p1JjyjvxFsloUXq5S_qt2fUPa0VMhP_Urhl9Bgl2KZTjC-pAfp_ZXyoa-RE-ymI0UMGLXJrg09XOJzSFaSlWqzAJNdb3lZQxWPC2V5otY8OjHCPGfQ7b-VysUtyZ-MTUQrrNN4QgznAB5jRbDYuFtfz-XRSXBV5sdAZ7MHoqR7n-WS-0LNU0TrXxww-zxIJgD4I8cPwpPOvjl-h-4tp
I am using WebStorm 2021.2.4

Hi @rulrok , I am afraid but the other plugin is responsible for that issue. NotesTree just providy the way to create md files,. Mmd files view is served by markdown plugin


when i input :shell ,Eject a cmd window

My system is win10,Normally it where run the terminal in vim.i don't now how to do

when i input :terminal

If you don't want to open a command prompt window when you executing a command, try :set guioptions+=!. (See :help 'go-!' for detail.)
For winpty64.dll, see :help terminal-ms-windows.
Anyway, please ask user questions on the vim_use mailinglist or use vi.stackexchange.com. This here is a bug tracker for problems with Vim. Closing.


Error compiling for board CLP SparkFun Pro Micro 5V, 16 MHz

Good morning developers, I really love your idea and I'm trying to install it on my Pro Micro board,
I'm not an expert in Arduino and I've followed the guide and I've tried different Arduino versions on windows and Mac OS, but I receive always this problem, can someone help me?
Thanks for your work
`Arduino: 1.8.0 (Mac OS X), Board: "CLP SparkFun Pro Micro 5V, 16 MHz, Lighting Node PRO"
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x18 pc=0xa9f2a]
goroutine 1 [running]:
panic(0x153640, 0xc42000e050)
/usr/local/go/src/runtime/panic.go:500 +0x1a1
arduino.cc/builder.(*SetupBuildProperties).Run(0x247c18, 0xc42006d180, 0x247c18, 0x0)
/Users/jenkins/jenkins/workspace/arduino-builder-macosx/src/arduino.cc/builder/setup_build_properties.go:83 +0xf8a
arduino.cc/builder.(*ContainerSetupHardwareToolsLibsSketchAndProps).Run(0x247c18, 0xc42006d180, 0x42055555, 0x0)
/Users/jenkins/jenkins/workspace/arduino-builder-macosx/src/arduino.cc/builder/container_setup.go:59 +0x3c3
arduino.cc/builder.runCommands(0xc42006d180, 0xc42003bcf8, 0x3, 0x3, 0xc42000ce01, 0x1, 0x21a960)
/Users/jenkins/jenkins/workspace/arduino-builder-macosx/src/arduino.cc/builder/builder.go:191 +0xcd
arduino.cc/builder.(*ParseHardwareAndDumpBuildProperties).Run(0xc42003bd58, 0xc42006d180, 0xc42003bda8, 0x15ea80)
/Users/jenkins/jenkins/workspace/arduino-builder-macosx/src/arduino.cc/builder/builder.go:180 +0xfb
arduino.cc/builder.RunParseHardwareAndDumpBuildProperties(0xc42006d180, 0xc42003bda8, 0x247c18)
/Users/jenkins/jenkins/workspace/arduino-builder-macosx/src/arduino.cc/builder/builder.go:227 +0x35
main.main()
/Users/jenkins/jenkins/workspace/arduino-builder-macosx/src/arduino.cc/arduino-builder/main.go:307 +0xcaa
/private/var/folders/m2/3x6j0g_x6xb515sl94fky2yr0000gn/T/AppTranslocation/277FA101-4204-4386-B742-868312699271/d/Arduino.app/Contents/Java/arduino-builder returned 2
Error compiling for board CLP SparkFun Pro Micro 5V, 16 MHz.
This report would have more information with
"Show verbose output during compilation"
option enabled in File -> Preferences.
`

Could you try and update your Arduino IDE version?

Could you try and update your Arduino IDE version?

Sure, the problem started with the latest version of Arduino, i've also tried to downgrade in different machines but no luck

From your log (Arduino: 1.8.0 (Mac OS X)) I would tell you are not using the latest version, which is Arduino 1.8.13. So, please use the latest version and if it still gives you the error, enable "Show verbose output during compilation" option in File -> Preferences.

Thanks again for your support, I've update and enabled the option in Arduino, but same error:
`Arduino: 1.8.13 (Mac OS X), Board: "CLP SparkFun Pro Micro 5V, 16 MHz, Lighting Node PRO"
/Applications/Arduino.app/Contents/Java/arduino-builder -dump-prefs -logger=machine -hardware /Applications/Arduino.app/Contents/Java/hardware -hardware /Users/davidelolli/Library/Arduino15/packages -tools /Applications/Arduino.app/Contents/Java/tools-builder -tools /Applications/Arduino.app/Contents/Java/hardware/tools/avr -tools /Users/davidelolli/Library/Arduino15/packages -built-in-libraries /Applications/Arduino.app/Contents/Java/libraries -libraries /Users/davidelolli/Documents/Arduino/libraries -fqbn=Legion2:avr:promicro5vclp:device=lnp -ide-version=10813 -build-path /var/folders/m2/3x6j0g_x6xb515sl94fky2yr0000gn/T/arduino_build_690051 -warnings=none -build-cache /var/folders/m2/3x6j0g_x6xb515sl94fky2yr0000gn/T/arduino_cache_184918 -prefs=build.warn_data_percentage=75 -prefs=runtime.tools.avr-gcc.path=/Applications/Arduino.app/Contents/Java/hardware/tools/avr -prefs=runtime.tools.avr-gcc-7.3.0-atmel3.6.1-arduino7.path=/Applications/Arduino.app/Contents/Java/hardware/tools/avr -prefs=runtime.tools.avrdude.path=/Applications/Arduino.app/Contents/Java/hardware/tools/avr -prefs=runtime.tools.avrdude-6.3.0-arduino17.path=/Applications/Arduino.app/Contents/Java/hardware/tools/avr -prefs=runtime.tools.arduinoOTA.path=/Applications/Arduino.app/Contents/Java/hardware/tools/avr -prefs=runtime.tools.arduinoOTA-1.3.0.path=/Applications/Arduino.app/Contents/Java/hardware/tools/avr -verbose /Users/davidelolli/Documents/Arduino/libraries/Corsair_Lighting_Protocol/examples/LightingNodePRO/LightingNodePRO.ino
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x50 pc=0x16be5f3]
goroutine 1 [running]:
github.com/arduino/arduino-cli/legacy/builder.(*SetupBuildProperties).Run(0x1eead88, 0xc000055400, 0x1eead88, 0xc000210270)
/Users/jenkins/go/pkg/mod/github.com/arduino/arduino-cli@v0.0.0-20200612152641-f7a6d63d208a/legacy/builder/setup_build_properties.go:89 +0x10a3
github.com/arduino/arduino-cli/legacy/builder.(*ContainerSetupHardwareToolsLibsSketchAndProps).Run(0x1eead88, 0xc000055400, 0x0, 0x0)
/Users/jenkins/go/pkg/mod/github.com/arduino/arduino-cli@v0.0.0-20200612152641-f7a6d63d208a/legacy/builder/container_setup.go:87 +0x464
github.com/arduino/arduino-cli/legacy/builder.runCommands(0xc000055400, 0xc000141cc0, 0x2, 0x2, 0x0, 0x0)
/Users/jenkins/go/pkg/mod/github.com/arduino/arduino-cli@v0.0.0-20200612152641-f7a6d63d208a/legacy/builder/builder.go:192 +0x19f
github.com/arduino/arduino-cli/legacy/builder.(*ParseHardwareAndDumpBuildProperties).Run(0xc000145d40, 0xc000055400, 0x1eeac70, 0xc000271550)
/Users/jenkins/go/pkg/mod/github.com/arduino/arduino-cli@v0.0.0-20200612152641-f7a6d63d208a/legacy/builder/builder.go:183 +0x96
github.com/arduino/arduino-cli/legacy/builder.RunParseHardwareAndDumpBuildProperties(...)
/Users/jenkins/go/pkg/mod/github.com/arduino/arduino-cli@v0.0.0-20200612152641-f7a6d63d208a/legacy/builder/builder.go:215
main.main()
/Users/jenkins/Jenkins/workspace/arduino-builder-mac-catalina/main.go:390 +0x879
/Applications/Arduino.app/Contents/Java/arduino-builder returned 2
Error compiling for board CLP SparkFun Pro Micro 5V, 16 MHz.
`

You are using a SparkFun Pro Micro, did you install https://github.com/sparkfun/Arduino_Boards#installation-instructions ?
PS: use ``` for multi line code blocks

yes, iv'e tried another time, but nothing

Have you tried to upload some other simple sketch to the Arduino, e.g. the blink example sketch?

I'm getting the same issue with Arduino 1.8.13 on Arch Linux.  I don't think it's an uploading problem, because it happens even when there is no board plugged in.  If this makes a difference, I'm not actually using a Pro Micro.  I'm using adafruit's itsybitsy 5v 16 MHz, which seems like the same thing, with a few different pins.  I was hoping it would just work, but haven't tested it yet.  I'm pretty sure this issue is unrelated to the board mismatch, becasue it doesn't matter if there is a board plugged in or not.
Arduino: 1.8.13 (Linux), Board: "CLP SparkFun Pro Micro 5V, 16 MHz, Lighting Node PRO"

/usr/share/arduino/arduino-builder -dump-prefs -logger=machine -hardware /usr/share/arduino/hardware -hardware /home/jack/.arduino15/packages -tools /usr/share/arduino/tools-builder -tools /home/jack/.arduino15/packages -libraries /home/jack/Arduino/libraries -fqbn=Legion2:avr:promicro5vclp:device=lnp -ide-version=10813 -build-path /tmp/arduino_build_12536 -warnings=none -build-cache /tmp/arduino_cache_95655 -prefs=build.warn_data_percentage=75 -prefs=runtime.tools.avr-gcc.path=/home/jack/.arduino15/packages/arduino/tools/avr-gcc/7.3.0-atmel3.6.1-arduino7 -prefs=runtime.tools.avr-gcc-7.3.0-atmel3.6.1-arduino7.path=/home/jack/.arduino15/packages/arduino/tools/avr-gcc/7.3.0-atmel3.6.1-arduino7 -prefs=runtime.tools.arduinoOTA.path=/home/jack/.arduino15/packages/arduino/tools/arduinoOTA/1.3.0 -prefs=runtime.tools.arduinoOTA-1.3.0.path=/home/jack/.arduino15/packages/arduino/tools/arduinoOTA/1.3.0 -prefs=runtime.tools.avrdude.path=/home/jack/.arduino15/packages/arduino/tools/avrdude/6.3.0-arduino17 -prefs=runtime.tools.avrdude-6.3.0-arduino17.path=/home/jack/.arduino15/packages/arduino/tools/avrdude/6.3.0-arduino17 -verbose /home/jack/Arduino/LightingNodePRO/LightingNodePRO.ino
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x50 pc=0x55d7dbb9576a]

goroutine 1 [running]:
github.com/arduino/arduino-cli/legacy/builder.(*SetupBuildProperties).Run(0x55d7dc3bdca8, 0xc00019a800, 0x55d7dc3bdca8, 0xc000432030)
	github.com/arduino/arduino-cli@v0.0.0-20200612152641-f7a6d63d208a/legacy/builder/setup_build_properties.go:89 +0x10ca
github.com/arduino/arduino-cli/legacy/builder.(*ContainerSetupHardwareToolsLibsSketchAndProps).Run(0x55d7dc3bdca8, 0xc00019a800, 0x0, 0x0)
	github.com/arduino/arduino-cli@v0.0.0-20200612152641-f7a6d63d208a/legacy/builder/container_setup.go:87 +0x45d
github.com/arduino/arduino-cli/legacy/builder.runCommands(0xc00019a800, 0xc0005b1db8, 0x2, 0x2, 0x0, 0x0)
	github.com/arduino/arduino-cli@v0.0.0-20200612152641-f7a6d63d208a/legacy/builder/builder.go:192 +0x176
github.com/arduino/arduino-cli/legacy/builder.(*ParseHardwareAndDumpBuildProperties).Run(0xc0001e9e3e, 0xc00019a800, 0x55d7dc3bdb68, 0xc0003114c0)
	github.com/arduino/arduino-cli@v0.0.0-20200612152641-f7a6d63d208a/legacy/builder/builder.go:183 +0x98
github.com/arduino/arduino-cli/legacy/builder.RunParseHardwareAndDumpBuildProperties(...)
	github.com/arduino/arduino-cli@v0.0.0-20200612152641-f7a6d63d208a/legacy/builder/builder.go:215
main.main()
	github.com/arduino/arduino-builder/main.go:390 +0x914
/usr/share/arduino/arduino-builder returned 2

Error compiling for board CLP SparkFun Pro Micro 5V, 16 MHz.

@jath03 did you try to compile other code for that board? Just to make sure the error is really related to the code of this project.

Hmm, this issue might belong in CorsairLightingProtocolBoards.  When I set the board to the correct value, the same program (the LightingNodePRO example) compiles fine.

And the built-in BareMinimum example doesn't compile on the CLP SparkFun Pro Micro 5V, 16 MHz. Compiling the BareMinimum example fails with the same error on all the CLP boards except the CLP Arduino Leonardo, which compiles fine.

When I set the board to the correct value, the same program (the LightingNodePRO example) compiles fine.

What do you mean with "correct value"?

And the built-in BareMinimum example doesn't compile on the CLP SparkFun Pro Micro 5V, 16 MHz

Can you try to compile it for "SparkFun Pro Micro 5V, 16 MHz" without the CLP in the name.

What do you mean with "correct value"?

The official board from adafruit's boards package.
After getting the boards list from here, The BareMinimum example compiled fine for the SparkFun Pro Micro, configured with the 5v 16 MHz processor

The official board from adafruit's boards package.

Can you send me the link to it, is it different from the SparkFun Pro Mirco?

It seems like adafruit's version of the pro micro.  It has the same processor, but a few different pins.  As much as I'd love to get this device supported, like I said earlier, I don't really think that is the problem with the compilation error.
https://learn.adafruit.com/introducting-itsy-bitsy-32u4/arduino-ide-setup

After getting the boards list from here, The BareMinimum example compiled fine for the SparkFun Pro Micro, configured with the 5v 16 MHz processor

Did you try to compile with "CLP SparkFun Pro Micro 5V, 16 MHz" after you have installed "SparkFun Pro Micro 5V, 16 MHz"?

No, I didn't.  That fixed it!  You need to have the official board installed for the CLP one to work!

Yes this is also documented in the CorsairLightingProtocolBoards repo, but I thinks it's not placed very visible.

@jath03 if you want support for the Adafruit board please open an issue in the CorsairLightingProtocolBoards repo

try to compile for CLP Leonardo board, works for me, I have an Pro Micro 5v board too

try to compile for CLP Leonardo board, works for me, I have an Pro Micro 5v board too

I had this problem, some of the clone boards are labeled as Pro Micro, but when you read their minimal instructions, they say to use Leonardo.


Play live h.264 encoded video from aiortc using ffmpeg-python

Hi,
I am using ffmpeg-python for a project that encodes it into h.264 and stream it using aiortc and then play it live on the other end. But I couldn't find out how I can play the stream live using ffmpeg-python on the other side.
Is this possible using ffmpeg-python?
Thanks in advance!

To play a video you should use ffplay. Its a tool that is installed when you install the FFmpeg. This library compile commands to ffmpeg instead ffplay.
But search more, I can be wrong.

To play a video you should use ffplay. Its a tool that is installed when you install the FFmpeg. This library compile commands to ffmpeg instead ffplay.
But search more, I can be wrong.

@raulpy271 Thank you for the reply! But does this mean I can't use ffplay using the package?

Probably not.
This library converts python commands to a command to be run in terminal calling ffmpeg.  The compiled command is in this way:
ffmpeg -i <media> -f <format> <output>

While using ffplay the command is different:
ffplay -f <format> <input>

I simplified the command to be easy to understand. But anyway, this library doesn't compile in the above way.

I see. I guess I'll go ahead and close the issue then. Thank you @raulpy271 !

Thanks, man! good work!


[FEATURE] Use of configMapRef into GrafanaDashboard

Is your feature request related to a problem?
Our json dashbaord is not available from web server. I want to use configMapRef like:
apiVersion: grafana.integreatly.org/v1beta1
kind: GrafanaDashboard
metadata:
  name: kafka-dashboard
  namespace: grafana-operator
spec:
  folder: Kafka
  instanceSelector:
    matchLabels:
      dashboards: "external-grafana"
  configMapRef:
    name: grafana-kafka-dashboard
    key: json

Describe the solution you'd like
configMapRef is not available into grafana-operator 5.0.0-rc3.
Upgrade it to 5.5.2 will work after a merge request has been accepted into aws-observability here

Linked to https://github.com/aws-observability/aws-observability-accelerator/pull/36. Let me know if this solves the issue


feat(message): Simplify the demo code of the message component

Please modify the branch and resubmit.

Ok


updated blazor topic

Thank you for contributing! Please confirm this pull request meets the following requirements:

[X] I followed the contributing guidelines: https://github.com/github/explore/blob/master/CONTRIBUTING.md
[X] I have no affiliation with the project I am suggesting (as a maintainer, creator, contractor, or employee).

Which change are you proposing?

[ ] Suggesting edits to an existing topic or collection
[X] Curating a new topic or collection
[ ] Something that does not neatly fit into the binary options above

Editing an existing topic or collection
I'm suggesting these edits to an existing topic or collection:

[ ] Image (and my file is *.png, square, dimensions 288x288)
[ ] Content (and my changes are in index.md)

Please replace this line with an explanation of why you think these changes should be made.

Curating a new topic or collection

[X] I've formatted my changes as a new folder directory, named for the topic or collection as it appears in the URL on GitHub (e.g. https://github.com/topics/[NAME] or https://github.com/collections/[NAME])
[X] My folder contains a blazor.png image (if applicable) and index.md
[X] All required fields in my index.md conform to the Style Guide and API docs: https://github.com/github/explore/tree/master/docs

Blazor is a promising web framework from Microsoft. The Blazor topic has more than a 1000 repositories on Github.

Something that does not neatly fit into the binary options above

[ ] My suggested edits are not about an existing topic or collection, or at least not a single one
[ ] My suggested edits are not about curating a new topic or collection, or at least not a single one
[ ] My suggested edits conform to the Style Guide and API docs: https://github.com/github/explore/tree/master/docs

Please replace this line with an explanation of your proposed changes.

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

Hello, thank you for your contribution it looks like Blazor has moved to the ASP.NET Core repo which we already have a topic for so I'm going to go ahead and close this for right now!

Hello, thank you for your contribution it looks like Blazor has moved to the ASP.NET Core repo which we already have a topic for so I'm going to go ahead and close this for right now!

@rmathew1992  ok sorry...

@rmathew1992  ok sorry...


Introducing DataHub Guru on Gurubase.io

Hello team,
I'm the maintainer of Anteon. We have created Gurubase.io with the mission of building a centralized, open-source tool-focused knowledge base. Essentially, each "guru" is equipped with custom knowledge to answer user questions based on collected data related to that tool.
I wanted to update you that I've manually added the DataHub Guru to Gurubase. DataHub Guru uses the data from this repo and data from the docs to answer questions by leveraging the LLM.
In this PR, I showcased the "DataHub Guru", which highlights that DataHub now has an AI assistant available to help users with their questions. Please let me know your thoughts on this contribution.
Additionally, if you want me to disable DataHub Guru in Gurubase, just let me know that's totally fine.

Hi again,
Have you had a chance to check this out?
Additionally, I’d like to update you on the release of the Maintainer Panel feature on Gurubase. With this panel, you can add, remove, or update data sources, as well as change the logo and more. You can find the details here.
In the near future, I’m planning to include analytics insights in this panel, such as the number of questions asked, the most frequently asked ones, and more. I’ll be sure to update this thread once it’s available. However, in case I miss it, I highly recommend joining our Discord channel to stay updated.
If you’d prefer that I don’t update this thread anymore, please let me know.

Hey @kursataktas , this looks pretty neat.
We currently use RunLLM for question answering. It's been working reasonably well so far, so we're likely not going to spend much time looking at / promoting alternatives. At the same time, it's always useful to have alternatives available for our community, so you're welcome to keep pulling in our code/docs/etc as you build out your tool.


Telegram Component Webhook Issue

Home Assistant release (hass --version):
0.42.3
Python release (python3 --version):
3.5
Component/platform:
telegram_webhooks
Description of problem:
I am trying to setup a telegram webhook and I am unable to send any message to my homeassistant.
Home Assistant does not throw an error, but if I set the log level to debug, I see this in the log.
Shouldn't this show up at info log level too ?
It just says connection refused, but I don't understand what the reason could be. I have LetsEncrypt SSL setup. I see that the document also says manually registering webhook, but I don't know how to do that.
DEBUG (Thread-1) [homeassistant.components.telegram_webhooks] telegram webhook status: {'pending_update_count': 15, 'last_error_date':<PHONE_NUMBER>, 'last_error_message': 'Connection refused', 'has_custom_certificate': False, 'url': 'https://myurl:8443/api/telegram_webhooks'}
17-04-13 21:17:22 DEBUG (Thread-1) [homeassistant.components.telegram_webhooks] users allowed: {11111111: 'user1'}

Expected:
Registered and connected webhook
Problem-relevant configuration.yaml entries and steps to reproduce:
Just added the entry and restarted. No specific steps to reproduce
telegram_webhooks:
  api_key: sdfsdfsdf
  user_id:
    user1: 11111111

Traceback (if applicable):
DEBUG (Thread-1) [homeassistant.components.telegram_webhooks] telegram webhook status: {'pending_update_count': 15, 'last_error_date':<PHONE_NUMBER>, 'last_error_message': 'Connection refused', 'has_custom_certificate': False, 'url': 'https://myurl:8443/api/telegram_webhooks'}
17-04-13 21:17:22 DEBUG (Thread-1) [homeassistant.components.telegram_webhooks] users allowed: {11111111: 'user1'}

Additional info:

Perhaps @sander76 could help here.

I have the exact same issue and was wondering if you had any news @kdvlr ? I am now looking into my nginx conf but cannot find anything wrong there.

I see a more detailed error message now. It looks like server certificate validation fails.
telegram webhook status: {'last_error_date':<PHONE_NUMBER>, 'last_error_message': 'SSL error {337047686, error:1416F086:SSL routines:tls_process_server_certificate:certificate verify failed}', 'url': 'https://url.com:8443/api/telegram_webhooks', 'pending_update_count': 0, 'has_custom_certificate': False

OK. I seem to have resolved my issue. All I needed was the full chain certificate. See here - https://www.cubewebsites.com/blog/guides/fix-telegram-webhooks-not-working/

@kdvlr I guess you are using nginx and lets encrypt. If that's the case did you had to create the fullchain.pem file or you used the one generated by let's encrypt?
Also, would you mind sharing your nginx.conf and default site file?
I am still struggling with this issue....
Thanks and lucky you you got it solved!


WebUSB (browser) support

The WebUSB spec is currently being drafted, but is available as an Origin Trial in Chrome 54+.

Chrome Status Page, which links to documentation
Origin Trial sign-up form

With WebUSB, we could make an app that requires no installation and just runs in a browser.

An upshot of using WebUSB and running this code in a browser is that we get full access to the canvas API, which solves #6.

Chrome has announced intent to ship WebUSB! It's closer than I thought: https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/KuXx_k2KIis

WebUSB has shipped in Chrome, so we could begin working on this feature at any time: https://caniuse.com/#feat=webusb

I think for this to happen Elgato needs to ship it in their driver first, similarly to https://github.com/webusb/arduino
In any case, I can't see the device in Chrome. Pity.

It looks like WebUSB specifically does not support HID under normal conditions: https://stackoverflow.com/a/49453250/3903335

WebHID is a very very very early, but might be just what we need to do this https://github.com/WICG/webhid/blob/master/EXPLAINER.md


Fatal error Traceback

This happens after login in.
Fatal error encountered:
Traceback (most recent call last):
File "twitch.py", line 102, in start
File "asyncio\base_events.py", line 616, in run_until_complete
File "twitch.py", line 173, in _run
File "twitch.py", line 653, in fetch_inventory
TypeError: 'NoneType' object is not iterable
Exiting...
Application Terminated.
Close the window to exit the application.

Hello =)
I see. I expect the cause of it is related to there being no available campaigns when the inventory is fetched, and the code expects an empty list while None is returned instead. This should be now fixed in v6, available now on the releases page: https://github.com/DevilXD/TwitchDropsMiner/releases
Please note that you'll need to activate the campaign manually, before the application can begin mining. I've added a "Troubleshooting" section to the README to aid you in this process: https://github.com/DevilXD/TwitchDropsMiner/blob/master/README.md


4.2.beta6.mono iOS build crashes on iOS 12

Godot version
v4.2.beta6.mono.official [64150060f]
System information
iOS 12.5.7
Issue description
iOS 12 builds crash on launch with error
dyld: Library not loaded: @rpath/GodotiOS12Sandbox.dylib
  Referenced from: /var/containers/Bundle/Application/42C3A63F-4D55-4F2E-A5EF-44581F92994C/GodotiOS12Sandbox.app/GodotiOS12Sandbox
  Reason: no suitable image found.  Did find:
	/private/var/containers/Bundle/Application/42C3A63F-4D55-4F2E-A5EF-44581F92994C/GodotiOS12Sandbox.app/Frameworks/GodotiOS12Sandbox.dylib: missing LC_DYLD_INFO load command

Steps to reproduce
Example project included. Export to iOS and test on device running iOS 12 (iPhone 5S for me)
Minimal reproduction project
GodotiOS12Sandbox.zip

Try it with RC1, .Net export for iOS was mostly broken in previous builds.

iPhone 5S

Your project seems to be set to use Mobile Vulkan renderer, it's unlikely it will reliably run on devices with SoC older than A12, but OpenGL (Compatibility) renderer might work.

I will try with RC1, thanks saw you just posted it. Compatibility renderer is set for mobile in project settings.

Tested on RC1 and did not see the issue. Thanks


Avoid repeated connection initializations in the NetworkClient.

Avoid repeated connection initializations for the same request in the
NetworkClient; fail a request immediately if a connection initialized
on its behalf fails.
--
This PR adds two changes:

Prevents repeated connection initializations for the same request, if one was already initiated. Before this patch, when the first request to a node comes in, the network client might end up establishing max number of connections to a destination - because the sendAndPoll() could happen several times before the first connection gets established.

Fails a request if a connection initiated on its behalf fails - similar to failing it if the send had already started. This is a "fail fast" approach - rather than waiting for checkoutTimeout to kick in which would be useless almost all of the time.

Coverage
is 100%. More importantly, added a test to verify the changed functionality.
NetworkClient	100% (2/ 2)    100% (12/ 12)    100% (108/ 108)

Looks good to me except for one minor comment

Finished a pass. Just the one question.

Addressed comments, please take a look.

LGTM. Merging.


EXOMobileDeviceMailboxPolicy

Pull Request (PR) description
Add DSC resource EXOMobileDeviceMailboxPolicy
This Pull Request (PR) fixes the following issues

This change is

@gaelcolas @PlagueHO can you have a look at the failed AppVeyor tests when you get a chance? @malauter and I believe this is a false positive due to the name of the parameter (which contains 'Password'). We do not want to rename the param, which is how the associated cmdlet uses it, to get around this. Please advise. Thanks

False positives in AppVeyor due to variable name


[Marker.io] Animation isn't centred correctly

Animation is sitting slightly to far over to the left

Reported by: Holly<EMAIL_ADDRESS>- Contact via Marker.io
Source URL: https://na13-prezzee.webflow.io/en-us
Issue details: Open in Marker.io
Device typedesktopBrowserChrome <IP_ADDRESS>Screen Size2048 x 1152OSOS X 14.7.1Viewport Size1561 x 912Pixel Ratio@2x

🤖 Marker.io bot - This issue was marked as ✅ Resolved in Marker.io


[oneDNN]: Using oneDNN v3.x by default on Linux and Windows x86 builds

This PR turns ON oneDNN v3.x by default on both Linux and Windows x86 builds.
No change is needed from end-users who build TF from source since oneDNN v3.x will be automatically fetched.
ARM builds will not be affected since they will continue to use oneDNN v2.7.
This PR also removes conflg settings which was recently added to simultaneously support both oneDNN v2.x and v3.x.
Support for oneDNN v2.x in TF will be dropped in a future PR.

NOTE: We also need to add onednn repo in TSL's workspace file.

@penpornk Thank you for reviewing this PR. I have addressed your review comments. Please take a look.

@penpornk Thanks for approving the PR. Please note that we also need to add onednn repo in TSL's workspace file. Would you be able to add it? Thanks.

@bhavani-subramanian Will do. Thank you for the heads up!
Quick notes on CIs for future reference:
ROCm errors are unrelated. I've seen them on other PRs:
ERROR: /workspace/tensorflow/compiler/xla/service/gpu/BUILD:853:11: in deps attribute of cc_library rule //tensorflow/compiler/xla/service/gpu:gpu_executable: Label '//tensorflow/tsl/platform:random' is duplicated

ERROR: /workspace/tensorflow/compiler/xla/service/gpu/BUILD:853:11: Analysis of target '//tensorflow/compiler/xla/service/gpu:gpu_executable' failed

MacOS failure is unrelated. (Test timed out)
[38,742 / 38,767] 771 / 1022 tests, 1 failed; Testing //tensorflow/python/data/kernel_tests:interleave_test (shard 1 of 24); 18s local, remote-cache ... (16 actions running)
[38,742 / 38,767] 771 / 1022 tests, 1 failed; Testing //tensorflow/python/data/kernel_tests:interleave_test (shard 1 of 24); 27s local, remote-cache ... (16 actions running)

ERROR: Aborting VM command due to timeout of 43200 seconds

Py+CPP Ubuntu CPU failure is unrelated. All 6 failed unit tests timed out.
Py+CPP Ubuntu GPU failure is unrelated. The failed test timed out.
//tensorflow/python/kernel_tests/linalg:linear_operator_identity_test_gpu TIMEOUT in 2 out of 5 in 466.9s


Ignore consecutive separators

Thank you so much for creating this library. It's taken away all of the heavy lifting of working with CSV files in C++.
I'm using this library to parse Log ASCII Standard (*.las) files. One of the oddities of these files is that they use consecutive spaces as separators. This makes the values easier for humans to read but challenging for computers. Here's an example, note that I want to get a value like SHT (68.0) from this table:
#MNEM           .UNIT                     VALUE            :                 DESCRIPTION
#----            ------          --------------            -----------------------------
OBPS            .                       -999.25            : (DnMWorkflow)   On-Bottom Pass Selector{S}
A               .                      1.000000            : (Borehole)      Constant A of the Archie Formation Factor - Porosity Equation{F}
ETIP            .ft                    0.000000            : (DNMSESSION)    Elevation of the TIP above MSL{F}
GCSE_RT         .                            BS            : (Borehole)      Generalized Caliper Selection for DnM real time{S}
SHT             .degF                 68.000000            : (Borehole)      Surface Hole Temperature{F}

I didn't find a way to ignore the extra spaces with the current library, so I made the following changes that work well for my project.
 struct SeparatorParams
  {
    /**
     * @brief   Constructor
     * @param   pSeparator            specifies the column separator (default ',').
     * @param   pTrim                 specifies whether to trim leading and trailing spaces from
     *                                cells read (default false).
     * @param   pHasCR                specifies whether a new document (i.e. not an existing document read)
     *                                should use CR/LF instead of only LF (default is to use standard
     *                                behavior of underlying platforms - CR/LF for Win, and LF for others).
     * @param   pQuotedLinebreaks     specifies whether to allow line breaks in quoted text (default false)
     * @param   pAutoQuote            specifies whether to automatically dequote data during read, and add
     *                                quotes during write (default true).
     * @param   pIgnoreConsecutive   specifies whether to ignore consecutive separators data during read (default false).
     */
    explicit SeparatorParams(const char pSeparator = ',', const bool pTrim = false,
                             const bool pHasCR = sPlatformHasCR, const bool pQuotedLinebreaks = false,
                             const bool pAutoQuote = true, , const bool pIgnoreConsecutive = false)
      : mSeparator(pSeparator)
      , mTrim(pTrim)
      , mHasCR(pHasCR)
      , mQuotedLinebreaks(pQuotedLinebreaks)
      , mAutoQuote(pAutoQuote)
      , mIgnoreConsecutive(pIgnoreConsecutive)
    {
    }

    /**
     * @brief   specifies the column separator.
     */
    char mSeparator;

    /**
     * @brief   specifies whether to trim leading and trailing spaces from cells read.
     */
    bool mTrim;

    /**
     * @brief   specifies whether new documents should use CR/LF instead of LF.
     */
    bool mHasCR;

    /**
     * @brief   specifies whether to allow line breaks in quoted text.
     */
    bool mQuotedLinebreaks;

    /**
     * @brief   specifies whether to automatically dequote cell data.
     */
    bool mAutoQuote;

    /**
     * @brief   specifies whether to ignore consecutive separators.
     */
    bool mIgnoreConsecutive;
  };

    void ParseCsv(std::istream& pStream, std::streamsize p_FileLength)
    {
      const std::streamsize bufLength = 64 * 1024;
      std::vector<char> buffer(bufLength);
      std::vector<std::string> row;
      std::string cell;
      bool quoted = false;
      int cr = 0;
      int lf = 0;

      while (p_FileLength > 0)
      {
        std::streamsize readLength = std::min<std::streamsize>(p_FileLength, bufLength);
        pStream.read(buffer.data(), readLength);
        for (int i = 0; i < readLength; ++i)
        {
          if (buffer[i] == '"')
          {
            if (cell.empty() || cell[0] == '"')
            {
              quoted = !quoted;
            }
            cell += buffer[i];
          }
          else if (buffer[i] == mSeparatorParams.mSeparator)
          {
            if (!quoted)
            {
              if (!cell.empty() || !mSeparatorParams.mIgnoreConsecutive)
              {
                  row.push_back(Unquote(Trim(cell)));
              }
              cell.clear();
            }
            else
            {
              cell += buffer[i];
            }
          }
...
      }
    }

Please consider adding this feature in a future release.

Hi @TXCodeDancer - thanks for sharing!
I'll review this and see how/if we can incorporate it.

Hi @TXCodeDancer - so after careful and a bit (too) long consideration I've arrived at not including this functionality in rapidcsv. It's a bit of a corner use case and technically not a CSV format in the traditional sense.
Thanks though for providing the solution above, it may come in handy for other developers googling the problem.
I will proceed to close this issue, but feel free to re-open it if you have follow-up questions.


fix: sipi makefile

resolves #1614

@benjamingeer sorry, this should fix the issue with sipi.

Thanks, could you also add something to knora-api/sipi/README.md to say that you have to set SIPI_VERSION and to indicate what it could be set to?


plain HTML documents not supported error for Markdown content that starts with HTML

If one has some Markdown content that starts with HTML and has no front matter, Hugo will throw the error "plain HTML documents not supported" regardless of whether or not markup.goldmark.renderer.unsafe = true is set in the configuration. If one adds any non-HTML content or even an empty front matter to the document, the error goes away.
What version of Hugo are you using (hugo version)?
First v0.91.2 that is shipped in the Debian repos, then the latest v0.92.1 from Homebrew. Both have the same issue.
$ hugo version
hugo v0.91.2-1798BD3F linux/amd64 BuildDate=2021-12-23T15:33:34Z VendorInfo=gohugoio

$ hugo version
hugo v0.92.1+extended linux/amd64 BuildDate=unknown

Does this issue reproduce with the latest release?
Yes

If one adds any non-HTML content or even an empty front matter to the document, the error goes away.

That is what you need to do. I don't think we're going to complicate the parser to support this case -- I'm certainly not spending time on it and I'm only merging a PR if it's simple.

Duplicate of #7296. See also #6098.


Training resumption with non-empty model dir.

Do not refuse training when no model state is found. So far the training
just died if the model directory already existed but did not contain
a training state. I do not see any good reason to not just start training
from scratch from the existing directory.
Additionally this commit adds a nicer error message for decoding when no
parameter file is found in the model directory.

The assumption is that an existing training dir without training state is actually a finished training, and we don't want to overwrite it (the training state dir is deleted when training ends succesfully). The goal was to avoid overwriting finished systems by mistake. You could change the logic to something like "if not exists(traning_state_dir) and exists(params file) then refuse overwriting".

makes sense. With what you're proposing we get the best of both options: resume if training stopped before the first checkpoint but do not overwrite finished trainings. I'll change the PR accordingly.

updated the PR, let me know if it looks ok now :)


Investigate bootstrap field for component Helm charts

spec.bootstrap is probably something we want to consider for the MetalLB and Endpoint Copier Operator Helm charts (and also future ones necessary for the clusters).

Leaving this here for the future https://github.com/k3s-io/helm-controller/blob/5d91b5fef8482219a93ed6ccfc4ea026431966f8/pkg/controllers/chart/chart.go#L589
It seems when setting it to bootstrap it configures the helm-install job with specific tolerations, node selectors, etc.


Feature Request -  Append to Note

Hi thanks for the great workflow!
I'd like to append to a bullet MD list on a note (or ideally different notes ) do you have any plans to add anything like this to your roadmap?

Can you give me an example of what that syntax would look like? I'm having trouble imagining how that would work.

Hi,
I use the Alfred Drafts workflow and it works really well in this respect.
Drafts Workflow
It would be great if you could replicate something similar ideally with the ability to have a couple of different lists

I took a look but it is still not clear to me what syntax you could use to both specify a note title and specify a bullet, especially onto different notes. It looks like the workflow you linked to just appends everything to a single unnamed document.
How would that work?

I think something like badd    note-name    note-content would work best.
There would obviously have to be a means of setting up the UUID for that note (happy to be in the back end of the workflow) and it would be great to have the ability to have multiple notes (again personally I would be happy to duplicate accordingly in the workflow and replace the UUID)
Thanks

That possible syntax had already occurred to me. But the problem I don't think you are seeing is that it is ambiguous.

If you have any two notes with overlapping names in those note title possibilities, there is literally no way to disambiguate. Further still if you allow badd to create new notes. And I can't even begin to imagine an unambiguous syntax for adding bullets to multiple notes.

I think it could work with a delineator like a | for example. From my use case as I mentioned before I don't this would be adding to every available note, but just to be able to add to one or two notes like a todo list and another bulleted list

I think this type of feature is out of scope for this workflow. I suggest creating a new workflow, possibly collaborating with the author of #20.


Apollo Kotlin should still handle ping messages before connection ack but after websocket is established

Summary
Frameworks like Hasura send a Ping IMMEDIATELY after a socket has been established. Apollo Kotlin will log this when this happens:
unknown graphql-ws message while waiting for connection_ack: 'ping

According to the spec: The Ping message can be sent at any time within the established socket.
Here is the thread over there where I thought it was a Hasura bug, but after reading the spec more closely I agree that this should be handled by clients.
https://github.com/hasura/graphql-engine/issues/9246
Thankfully this client handles it better than dgs but still annoying that it logs warnings all the time :)
Version
3.7.1
Logs
unknown graphql-ws message while waiting for connection_ack: 'ping

Steps to reproduce the behavior
Connect to a framework like Hasura over the GraphQLWsProtocol or just simply send a ping to Apollo Kotlin client before connect_ack message.

It's now available in 3.7.2


Open Vault Brand Refresh

[ ] Change blue header to new purple color.

[ ] Change "WGBH Series" to "GBH Series."

[ ] Under WGBH Series, change "WGBH Media Library and Archives" to "GBH Archives"

[ ] Change any WGBH references to GBH on main page.

[ ] Change Open Vault logo so it says "Open Vault from GBH"

[ ] Remove logo at the bottom right and replace with updated GBH Archives logo

[ ] Replace all WGBH mentions with GBH and "Media Library and Archives" with Archives on this page

https://app.zenhub.com/files/198695627/d7f51d1c-d8d3-47ff-8b0f-b826dc8381dc/download

@foglabs-zen this is looking good to me. Ready to go?


chore(dump): Readability improvements to src/dump

Pull Request
What does this PR do?
Improves readability focused in the /meilisearch-lib/src/dump path
PR checklist

[x] Have you read the contributing guidelines?
[x] Have you made sure that the title is accurate and descriptive of the changes?

Hello @ryanrussell
Thanks for this PR, again!
Looks like the tests are failing, could you fix them before we review? 😄

Hey @ryanrussell,
It looks like you are patching English misspells in version 2 of the bump (bumps V2) and we don't want to do so, we want to be able to parse our mistakes from this version of the dumps. English mistakes must be kept and were fixed in version 3 of the dumps.
I am afraid but I'll close your PR, sorry for that.

You could probably open a new PR that fixes only the English mistakes that aren't related to the dumps?

@Kerollmops
No problem.
Just trying to do a little readability janitor work on down cycles. I've made a lot of use of Meilisearch, just trying to do something small to contribute back :)
Are there other paths that should not be touched like dumps?
I see a few things in tests, index, index_resolver ... was just going to iterate through them as time allows.
All the stuff I'm going for on this initial round will not change functionality... just trying to improve readability.


chore: bdd-spring-1586823957 to 0.0.1

chore: Promote bdd-spring-1586823957 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Add dependency injection and unit tests

This project is now big enough to add a DI container and unit tests.

Which is also good occasion to do a general architecture cleanup :)


update cdh web deploy for pre-production environment

requested new production VMs from PUL (cdh-web3 and cdh-web4)
made cdh-ansible updates so we can deploy springload code to new pre-production servers; uses a new db separate from current cdh production
manually copied current cdh database to new db - got errors dumping the data and restoring so I used this command to create the new db using the old one as a template: https://stackoverflow.com/a/876565/9706217

the cdh-ansible changes are in a PR #220 awaiting review; once that is merged I will update the playbook in Ansible Tower so we can deploy to either staging or preproduction, and make more variables configurable


[Question] no-did-mount-set-state accesing DOM ref

Shouldn't it be allowed to use the setState function inside did-mount/update when accessing a reference to a DOM element? Right now we have to put an ignore comment above the line
componentDidMount() {
  this.setState({ width: this.containerRef.current.clientWidth });
}

The rule (which, it must be noted, is incompatible with server rendering, and so is totally fine to disable) is about setting state - it's irrelevant what state you're setting.


[BUG] Tutorial project not created when using yarn

Describe the bug
Issue occurred while following the getting started guide for version 3.xx.xx
https://refine.dev/docs/tutorial/getting-started/mui/create-project/
Creating the project with npm works fine, but does not work using using yarn, the following command and output can be observed, without any files being generated.
Steps To Reproduce

Install node
Install yarn
Run CLI command to create tutorial project yarn create refine-app -- -o refine-mui tutorial

Expected behavior

Installation wizard to start
Project folder and files to be created

Screenshot
$ yarn create refine-app -- -o refine-mui tutorial
➤ YN0000: ┌ Resolution step
➤ YN0000: └ Completed in 5s 797ms
➤ YN0000: ┌ Fetch step
➤ YN0000: └ Completed
➤ YN0000: ┌ Link step
➤ YN0000: └ Completed
➤ YN0000: Done in 6s 66ms

Desktop
Node version: 18.13.0 (installed using n)
Yarn version: 3.3.1
OS: Ubuntu 22.04.1 LTS
Mobile
No response
Additional Context
No response

Hey @wuju
Thank you for reporting this issue.
I also reproduced the problem. We'll see what we can do.

We currently only support yarn v1 😞 We have added this problem to our work list and we want to solve it as soon as possible.
If you are an expert in Yarn can you help us 🙏🏼


Deprecated Filtering method

The docs show a constructor to use filtering.
It is marked as deprecated, and should be updated.
[Obsolete("This method is obsolete and will be removed in a future version. The recommended alternative is using LoggerFactory to configure filtering and ConsoleLoggerOptions to configure logging options.")]
public ConsoleLoggerProvider(Func<string, LogLevel, bool> filter, bool includeScopes);

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

ID: ed83069f-39a8-e992-95f7-a8b41154b6f4
Version Independent ID: 144fe168-080f-6557-e4b2-a18adb4b89ed
Content: Logging - EF Core
Content Source: entity-framework/core/miscellaneous/logging.md
Product: entity-framework
GitHub Login: @rowanmiller
Microsoft Alias: divega

From https://github.com/aspnet/EntityFramework.Docs/pull/1164#issuecomment-451748026,  ignore the warnings and/or adding a suppression in the code would be enough for 2.2.
And from https://github.com/aspnet/Extensions/issues/615#issuecomment-447061661, We may use LoggerFactory.Create in .Net Core 3.0.
var loggerFactory = LoggerFactory.Create(builder => {
    builder.AddConfiguration(loggingConfiguration.GetSection("Logging"))
        .AddFilter("Microsoft", LogLevel.Warning)
        .AddFilter("System", LogLevel.Warning)
        .AddFilter("SampleApp.Program", LogLevel.Debug)
        .AddConsole()
        .AddEventLog();
});

Thank you, @zerda. I read #1164, and that is the same issue as described here. Also, @0xced has a PR!


Update image in config to most recent

Version of stack logging agent upgraded to 1.9.4.
Version of google-fluentd = 1.13.3

LGTM. Instead of "most recent", could you update PR title to the specific version we are updating to. Since everytime when we update, we are likely going to update to the most recent version, having version number makes it easier to know which version we are updating to for future reference.


[8.x] Add Conditionable Trait to Relation

Hi!
Thank you for all the amazing work 🔥
Overview
This PR adds the new Conditionable trait to the base Relation class.
Currently, the following would not work for a relationship query...
$user->posts()->when($shouldOrder, fn ($query) => $query->orderByPivot('..'))->get()

...because the Relation class does not have a native when/unless. So, it forwards it to the Builder but then this does not have access to the Relation's methods - in this case orderByPivot.
Compatibility
The only thought is that this could be a breaking change in the sense that some relations override some Relation methods - eg. get in BelongsToMany. So if any apps do something along the lines of...
$user->posts()->when($shouldPaginate, fn ($query) => $query->paginate(), fn ($query) => $query->get())

...the get method would now be elevated from the Builder up to the Relation?
Tests
Testing wise, I appreciate there is only one test. I felt it illustrated the feature nicely using an existing set up. But very happy to add more if desired.

Reverted due to breaking tests on master.


-No need for Preamble- just read from Details section of Biography Tab

I think it would be better to be able to just read the Biography sections of the character sheet as opposed to having the separately stored information about the actor in the preamble. This will also make it easier for DM's who already have NPC's with bio's written to start using this module without having to do a ton of copying and pasting.

I think this is a good case for something that should be configurable on a global and per-character level (like several other settings). This sounds like a very good idea for a particular kind of DM. I on the other hand wasn't even really aware of the Biography section, I never used it. :D
Would you say that the preamble still has its merits, and should be read in addition to the bio? Maybe I want an NPC to answer in a particular style, I don't know if this is part of the bio. And maybe you do not want to include the full 500 Line bio you wrote for your favourite NPC.
I imagine something like an "include bio" checkbox in the settings.
I should mention: My time is very stretched thin at the moment due to work stuff, and will be for a couple of months. I am uncertain if and when I will find the time to implement this.

Any issue with me making a branch and working on it then? I already did locally and it works but I removed all the preamble stuff in the process. I think making it a checkbox is a much cleaner idea and also probably does better to keep the module system agnostic.

I will never have objections to anyone branching off and implementing stuff. Thank you! I'll have a look in time.

Thanks for the great work!
The Preamble not only defines the background etc but should also state, that the token is a person and give a background etc. Like "You are an NPC in DnD and speak elvenlike". Therefore I would recommend that, if you want to read from the bio to add a global prompt together with the swich you talked about. Something like: "You are a non-player character in the pen and paper game Dungeons and Dragons. The following Information defines you background and personality. Act accordingly. If you do good you will get a reward."

I have another branch that I made where I actually created a spot in the settings for unkenny where you can type in a instruction line for the LLM where you would indicate to it that it is a non-player character etc etc.  I thought that since this was something that would not change from character to character, it would be a good idea to have it in a central location where the dungeon master can just give a little bit of information about the world setting, that would maintain consistency across all NPCs

I like that idea, I'll create a separate issue for it.


Fix job services localhost example in workflow-syntax-for-github-actions.md

Why:
Because job services localhost example is wrong and misleading.
What's being changed (if available, include any code snippets, screenshots, or gifs):
Current example states that :

When you specify the Docker host port but not the container port, the container port is randomly assigned to a free port. GitHub sets the assigned container port in the ${{job.services.<service_name>.ports}} context. In this example, you can access the service container ports using the ${{ job.services.nginx.ports['8080'] }} and ${{ job.services.redis.ports['6379'] }} contexts.

Which is not true because you either specify container port only and then GitHub Actions will choose random free port on host, or you specify both, host and container port.
To access host port you should use container port as a key. In nginx example it should be  ${{ job.services.nginx.ports['80'] }} and not 8080.
Updated job services yaml example to show more clearly how to access those services.
I tested all this using this yaml
jobs:
  job-a:
    runs-on: ubuntu-latest
    services:
      nginx:
        image: nginx
        # Map port 8080 on the Docker host to port 80 on the nginx container
        ports:
          - 8080:80
      redis:
        image: redis
        # Map TCP port 6379 on Docker host to a random free port on the Redis container
        ports:
          - 6379/tcp
    steps:
      - run: docker ps
      - run: echo '${{ toJSON(job.services) }}'
      - run: |
          echo "Redis available on <IP_ADDRESS>:${{ job.services.redis.ports['6379'] }}"
          echo "Nginx available on <IP_ADDRESS>:${{ job.services.nginx.ports['80'] }}"
      - name: Check Redis
        run: |
          echo -e '*1\r\n$4\r\nPING\r\n' | netcat -w1 <IP_ADDRESS> ${{ job.services.redis.ports['6379'] }}

Check off the following:

[x] I have reviewed my changes in staging, available via the View deployment link in this PR's timeline (this link will be available after opening the PR).

For content changes, you will also see an automatically generated comment with links directly to pages you've modified. The comment won't appear if your PR only edits files in the data directory.

[x] For content changes, I have completed the self-review checklist.

@piotrekkr Thanks so much for opening a PR! I'll get this triaged for review ✨

5545554

@piotrekkr Thank you so much for your patience while our SME team reviewed! 💛 They have approved of your changes after testing your example ✨
Once you are able to resolve the conflicts with this branch, we are happy to get this merged! If submitting a new PR is easier for you, we're happy to merge that instead 🙇

@nguyenalex836 I've rebased my branch on latest code. There is no conflicts any more.

@piotrekkr Thank you! 💛 Getting this merged now!


IDVA5-1311 Fixed wrong page being displayed after an application is rejected

Filtered out rejected applications to prevent the wrong page being displayed when a user attempts to create a new application after a rejected one.
Updated delete application to also delete the transaction associated with when when user selects to create a new application and not continue an old one.
Moved calling POST transaction back to the get request in what type of business controller

CI: No security warnings found


addHTML() Image Size Issue

Hi,
 Noob question. Do I need to define an image size? I'm trying to capture an entire page (to maintain the Bootstrap styling) but when I run the "**NEW: addHTML()" code you recently added, I keep getting...

Error: getJpegSize could not find the size of the image (jspdf.plugin.addimage.js)
Thanks in advance for any help!
Chris

Probably the image you tried to fetch was not a real jpeg.


Enforce Canonical Base64 encoding.

See https://github.com/Cyphrme/Coze/issues/18 for details.

Fixed by c035229.


Student form finder change for EU students

Student form finder change
This is an update to the forms for new, full-time, EU students
Piv: https://www.pivotaltracker.com/story/show/115249327
Zen: https://govuk.zendesk.com/agent/tickets/1275668
Only one file needs to change: outcome_eu_ft_1617_new.govspeak
It's the final commit on this branch. It adds the forms and the standard copy & partials.

We have cherry-picked your commits into PR #2377
Thank you @stwalsh

@stwalsh This content change is currently in production. Please see: https://github.com/alphagov/smart-answers/pull/2377


RFC Parsing Error

While compiling the following RFC, there's a parsing error which is detected at the definition of Object o1 when the assigned values are represented as ENUM tags. However, Object 02's definition using integers does not present any parsing errors.
enum {
  REJECT(1),
  IGNORE(2),
  NOTIFY(3),
  (7)
 } Criticality;

 enum {
  OPTIONAL(1),
  CONDITIONAL(2),
  MANDATORY(3),
  (7)
 } Presence;

enum{
	BANANA,
	APPLE
}Fruit;

struct{
	uint8 number;
}V1;

struct{
	uint16 number;
}V2;

 struct{
 	Criticality 	criticality;
 	Presence 		presence;
 	Fruit 			fruit;
 	select(fruit){
 	case BANANA: 	V1;
 	case APPLE: 	V2;
 	}fruit_body;

 }Object;

Object o1 = {Criticality.REJECT, Presence.OPTIONAL, Fruit.BANANA};

Object o2 = {1, 1, 1};

The parsing error that's generated isn't very informative.

("Object")
("o1")
13
9
("Criticality.REJECT")
Parsing error near ./test.rfc(40,13)-./test.rfc(40,31)

Hi, there is currently no support in QD for creating instances of a defined datatype.
You need to do that in F* or in C, e.g.
#include "Object.h"
#include "Criticality.h"
#include "Presence.h"
#include "Fruit.h"
#include "Object_fruit_body.h"

Object_object o1 = {
  .criticality = Criticality_REJECT,
  .presence = Presence_OPTIONAL,
  .fruit_body = {
    .fruit = Fruit_BANANA,
    { .case_BANANA = (uint8_t) 0 }
  }
};


Boost.Coroutine is now deprecated. Please switch to Boost.Coroutine2.

Hello, and thank you for maintaining this project--it has been really helpful for me.
Since upgrading to Boost 1.62, I now get this warning:
  CL : warning : Boost.Coroutine is now deprecated. Please switch to Boost.Coroutine2. To disable this warning message, define BOOST_COROUTINES_NO_DEPRECATION_WARNING.

It's not causing me any trouble, but I thought you might like to know.

Thank you. If I'm not mistaken though, this is an issue within Boost.Asio?

You refer to Boost.Coroutine in CMakeLists.txt.  The way I interpret this warning, that library is (recently) deprecated and boost is recommending that you use their newer one.  I may have misunderstood though.


cni: fix k/v pairs in UDS logging from plugin

Fixes https://github.com/istio/istio/issues/51494
We dropped these on the floor when we unmarshal into a struct with only
static fields

In response to a cherrypick label: #51546 failed to apply on top of branch "release-1.22":
Applying: cni: fix k/v pairs in UDS logging from plugin
Using index info to reconstruct a base tree...
M	cni/pkg/log/uds.go
M	cni/pkg/log/uds_test.go
Falling back to patching base and 3-way merge...
Auto-merging cni/pkg/log/uds_test.go
CONFLICT (content): Merge conflict in cni/pkg/log/uds_test.go
Auto-merging cni/pkg/log/uds.go
CONFLICT (content): Merge conflict in cni/pkg/log/uds.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 cni: fix k/v pairs in UDS logging from plugin
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: new issue created for failed cherrypick: #51551


feat(money): pin confirmation functionality

Summary
Added the function confirmPin on Swift and Kotlin that compares the pin and pinConfirm values

@JorgeDelgadillo can you add TAD link to the description for more context.


Fix some deliver_to_automate method overries not accepting any args

There were some #deliver_to_automate overrides which didn't accept any arguments causing issues with creating request tasks.
[----] I, [2023-06-06T12:56:22.847006 #13431:a17c]  INFO -- evm: MIQ(ManageIQ::Providers::Vmware::InfraManager::Provision#deliver_queue) Queuing VM Provisioning: [Provision from [ag_template] to [ag-prov-test-2]]...
[----] E, [2023-06-06T12:56:22.847115 #13431:a17c] ERROR -- evm: [ArgumentError]: wrong number of arguments (given 2, expected 0)  Method:[block (2 levels) in <class:LogProxy>]
[----] E, [2023-06-06T12:56:22.847149 #13431:a17c] ERROR -- evm: /home/agrare/src/manageiq/manageiq/app/models/miq_provision.rb:48:in `deliver_to_automate'
/home/agrare/src/manageiq/manageiq/app/models/miq_request_task.rb:143:in `deliver_queue'
/home/agrare/src/manageiq/manageiq/app/models/miq_request.rb:470:in `block in create_request_tasks'
/home/agrare/src/manageiq/manageiq/app/models/miq_request.rb:467:in `each'
/home/agrare/src/manageiq/manageiq/app/models/miq_request.rb:467:in `create_request_tasks'

Ref: https://github.com/ManageIQ/manageiq-providers-ibm_cloud/pull/455#issuecomment-1578844038
Introduced by: https://github.com/ManageIQ/manageiq/pull/22544
Related:

https://github.com/ManageIQ/manageiq-providers-foreman/pull/116
https://github.com/ManageIQ/manageiq-providers-lenovo/pull/388

Backported to petrosian in commit a640661808ce466aa44183e3e0ec3d9730cdb608.
commit a640661808ce466aa44183e3e0ec3d9730cdb608
Author: Jason Frey<EMAIL_ADDRESS>Date:   Tue Jun 6 16:27:53 2023 -0400

    Merge pull request #22552 from agrare/fix_deliver_to_automate_no_args

    Fix some deliver_to_automate method overries not accepting any args

    (cherry picked from commit 7ec60bc976e61cf497fbb718caaaa8da7bedcfac)


mobile/ci: Set the default GPG key in the Mobile Release job

This addresses job failures in the Mobile Release CI job: https://github.com/envoyproxy/envoy/actions/runs/9255379931/job/25460301873#step:6:33

/assign @phlax


Header shows user avatar when logged in

Mockup:

js part of this should be done :)

Updated the styling a bit for better readability:


Got undefined

TEstes using you code, but sometimes it shows undefined and everything seems normal.
`        $deviceTokens = User::whereNotNull('fcm_token')->pluck('fcm_token')->toArray();
    return Larafirebase::withTitle('Test Title')
    ->withBody('Test body')
    ->withImage('https://firebase.google.com/images/social.png')
    ->withIcon('https://seeklogo.com/images/F/firebase-logo-402F407EE0-seeklogo.com.png')
    ->withSound('default')
    ->withClickAction('https://www.google.com')
    ->withPriority('high')
    ->withAdditionalData([
        'color' => '#rrggbb',
        'badge' => 0,
    ])
    ->sendNotification($deviceTokens);`

Please provide more details otherwise this issue should be closed.


Put a picture in the code comments

It will be nice to add pictures in the code comments (base64 strings can be used).
Imagine that we can put more information in the code comments: "A picture is worth a thousand words" is a common phrase, to describe this. This feature can be used to clarify something etc. The VS Code editor must render the image to the developer.

Duplicate of #41775 and #3220


biber --tool silently fails/crashes with utf8 chars

I hesitate to file this, b/c my understanding of what biber is supposed to do with utf-8 is so poor...
This file containing a single utf-8 char causes biber --tool to crash:
https://bugzilla.redhat.com/attachment.cgi?id=1076155
This is a Fedora bug report:
https://bugzilla.redhat.com/show_bug.cgi?id=1265601
Its reported downstream against biber 1.8, but I think I can confirm with dev branch.

Might need a bit more information - if I take that entry into `testx.bib" and run:
biber --tool testx.bib
with the 2.3 DEV version, it's fine.

Oops, yes its fine, even in 2.1, sorry for the noise.


Alternative for a more precise indication of live event starting times

This is an alternative to #614, which uses different wording ("Live in...", "Live for..."), nested translations for these and a bunch of individual translations for the corresponding time units. I wasn't sure which wording fits best, so I decided to open this as a draft PR for potential review.
Closes #508

Well I just realized this didn't need to be a draft PR as I could instead have linked the branch from my fork in the original PR.

I think we should go with #614. And if I understood you correctly, you agree. So I'm closing this. If we decide differently, we can still reopen.


Add support for openstack to get cluster command

Towards feature parity for OpenStack, this PR adds support for get clusters on OpenStack MCs. We may want to improve the output in the future, but this is a first step.
I tested getting cluster by name, getting clusters in a namespace, and getting clusters in all namespaces.
I also checked that AWS and Azure output was not affected.
Sample output:
kubectl-gs get clusters -A
NAMESPACE           NAME     AGE    CONDITION   CLUSTER VERSION                                  PREINSTALLED APPS VERSION   ORGANIZATION    DESCRIPTION
giantswarm          gamma    10d    READY                                                                                n/a             n/a
org-multi-project   flux01   7d1h   READY       0.5.0-cbde804a0c497463cb0cc37262268667bb015b51   1.8.0                       multi-project   test

As the creator of a pull request, please consider:

[x] Describe the goal you are trying to accomplish here, above the line.
[x] Add a user-friendly description of your change to CHANGELOG.md.
[ ] Provide a link to the issue you are solving or working towards, if available.
[x] Request SIG UX for review. They care about almost all user-facing changes.
[ ] Update the public kubectl-gs documentation to reflect the changes here.
[ ] add the breaking-change label to the PR if the change you are making changes the existing behaviour. Examples: removal of a flag, removal of a command, change of a default value. (Such changes should be released with a major version bump.)

Feel free to remove this checklist when done.

⚠ Headsup! Branch master has been renamed to main
Your PR has already been modified to merge to main.
If you have a local clone, you can adapt that one like this:
git branch -m master main
git fetch origin
git branch -u origin/main main
git remote set-head origin -a

I'm not adding this to the docs currently until we have the output structure nailed down for helm-based clusters.


Tweak Request: Don't clear prompt textbox when switching models.

9 times out of 10, if I'm switching to a new model, it's specifically to compare and visualize prompt sizes between different encoders and models, on the same prompt. Just a little tweak that would personally save me about 100 copy and pastes every day.

Hello @JonathanFly!
Sorry for the long delay. Currently, the prompt textbox is only cleared when switching from a chat model to an autocomplete model (eg. gpt-3.5-turbo to cl100k_base), which I believe does make sense?
If I undestand the issue correctly, you are comparing the ChatML prompt messages between different encoders?

@dqbd looks different to me, the textbox is always cleared no matter which model I switched, even from gpt-4 to gpt-3.5-turbo


Add swag-for-dev?

Creating an issue for people to chime in on whether we should add https://github.com/swapagarwal/swag-for-dev to this list (as per https://github.com/swapagarwal/swag-for-dev/issues/12)..

Sure. It's a cool list. Just make sure you follow the guideliens and then do a PR ;)


add file clean and SafeOpen utils

Proposed Changes

Added CleanPath function to clean paths and mitigate potential path traversal attacks.
Introduced CleanPathOrDefault function to clean paths or return a default path.
Implemented ResolveNClean function to resolve and clean paths.
Created ResolveNCleanOrDefault function to resolve and clean paths or return a default path.
Added SafeOpen function to safely open a file in read mode after cleaning the path.
Introduced SafeOpenAppend function to safely open a file in append mode after cleaning the path and creating any missing directories.
Implemented SafeOpenWrite function to safely open a file in write mode after cleaning the path and creating any missing directories.
Created SafeWriteFile function to safely write data to a file after cleaning the path and creating any missing directories.
Added FixMissingDirs function to create any missing directories in a given path.
closes #263

TODO

[ ] add fuzzing test using know LFI payloads from here


Deterministic Annotations

Hi, so now we have a bunch of data dictionary descriptions etc in our tables I have changed the serialisation method so that Annotations and Partitions are in the table.json file as well as putting relationships by their starting tables.
While doing a few imports and fixing things I found that the TabularEditor_Relationships annotation jumped from one position to another
The new file looks like it is now sorting the annotations into order correctly and possibly this was something that was fixed in the last version to make it deterministic but if not just a heads up.
Previously TabularEditor_RElationships was at the bottom where now with the new version from today it is putting it after TabularEditor_InPerspective and before my VR_* annotations

Annotations are not deterministic at the moment, so this is a good point. I'll make sure that they are sorted alphabetically by annotation name going forward.

On second thoughts - perhaps this should be an optional setting. By default, TOM arranges objects in the JSON in the order they are inserted, so suddenly forcing alphabetical ordering in the JSON files might cause a lot of unexpected diffs for people. And this goes for all object types - not just annotations.

That is true probably wise to have a control to allow people to switch when they want but default for new models. You really want this to be deterministic otherwise you get strange changes as a result of scripts running that basically produce the same logical result but git sees it as a change

We need to think carefully about this. Today, a simple change such as cutting a table away (Ctrl+X) and subsequently pasting it back in (Ctrl+V) would be enough to change the current ordering within the metadata. Interestingly, noone has complained about that yet. Forcing alphabetical order would of course prevent a change of ordering in this case.
On the other hand, alphabetical order might cause a change of ordering in cases when objects are renamed... I think that might actually be worse, since that probably happens more often.

I think there is still room for improvement here. I often find that annotations are re-ordered in objects that I have made no changes to. For example, just now I deleted several columns from my model. When I save it out, I am seeing annotation re-ordering in objects I haven't touched.
Specifically, I deleted Table1[ColumnA], and Table2[MeasureB] has its annotations re-ordered. The only relationship between these things is that they exist in the same perspectives, but I can't confirm that this is the causal factor.
It seems that there is room for a stricter preservation of existing ordering of annotations in existing objects, regardless of whether there is any capability to enforce ordering.
Here's a sample from my diff (scrubbed names):
I see this sort of thing on a semi-regular basis (maybe around new releases of TE? I did just update today)
--- a/path/cube/tables/table1/measures/measure.json
+++ b/path/cube/tables/table1/measures/measure.json
@@ -34,13 +34,13 @@
         ")"
       ]
     },
-    {
-      "name": "TabularEditor_InPerspective",
-      "value": "[\"persp1\",\"persp2\",\"persp3\"]"
-    },
     {
       "name": "persp2_Expression",
       "value": "[MeasureReference]"
+    },
+    {
+      "name": "TabularEditor_InPerspective",
+      "value": "[\"persp1\",\"persp2\",\"persp3\"]"
     }
   ]
 }

This is at least partially deterministic, because it will continue to occur when I make similar unrelated changes. Thus I am forced to either continuously exclude changes introduced by TE, or accept them and muddy up my version history. While BISM Normalizer is a great tool, it is not a good solution to use as part of a normal development cycle. I want to be able to git diff and see changes. Setting my difftool to BISM Normalizer would introduce integer-minute delays into a workflow that typically takes seconds currently.


[Package Request]:

Package Requested

[ ] I would like someone else to build the manifest.
[ ] I would like help so I can submit the manifest.
[X] I have performed a search and couldn't find this package.
[X] I have checked that the installer for this package is not a standalone or portable executable or a .zip file
[ ] I think there is a new version available and I have provided the URL.

Please provide the following information
Publisher: Smile
Publisher Url: https://smilesoftware.com/
Package Name: Text Expander
Description: Create snippets of text for support responses, sales outreach, or even common emails and save them in TextExpander.
Package Version: 7.2 (720.16)
Installer URL: https://textexpander.com/download
Current Package Identifier (for new versions):

Attempting to merge now #65116


Automation apply error

I am getting the following error when applying an imported automation.  There seems to be some issue with the 'expect' field in the trigger event.  It seems there is some inconsistency with the ordering that is causing errors.  I had successfully applied one of the automations but when i went to replan to try to change the order of the other one I am now seeing a similar issue on the one that worked.
So I switched those back and was able to apply that change successfully again but I assume if I replan I'll get that error.  It seems like the API and terraform are reordering the list of items and doing it in some different way.
I've tried reordering the states in various order but I still get errors, though the number of errors varies, so there might be some magical order out there that works.
Plan:
  # prefect_automation.xxxxxx will be updated in-place
! resource "prefect_automation" "xxxxxxx" {
        id                 = "xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx"
        name               = "xxxxxxxxxxxxxxx"
!       trigger            = {
!           event = {
!               after         = [
+                   "prefect.flow-run.Cancelled",
+                   "prefect.flow-run.Completed",
                    "prefect.flow-run.Crashed",
                    "prefect.flow-run.Failed",
-                   "prefect.flow-run.Completed",
-                   "prefect.flow-run.Cancelled",
                ]
                # (7 unchanged attributes hidden)
            }
        }
!       updated            = "2024-12-18T18:44:40Z" -> (known after apply)
        # (5 unchanged attributes hidden)
    }

apply error:
Error: Provider produced inconsistent result after apply
│
│ When applying changes to prefect_automation.xxxxxxxxxxxxx,
│ provider "provider[\"registry.opentofu.org/prefecthq/prefect\"]" produced
│ an unexpected new value: .trigger.event.after[0]: was
│ cty.StringVal("prefect.flow-run.Cancelled"), but now
│ cty.StringVal("prefect.flow-run.Crashed").
│
│ This is a bug in the provider, which should be reported in the provider's
│ own issue tracker.
╵
╷
│ Error: Provider produced inconsistent result after apply
│
│ When applying changes to prefect_automation.xxxxxxxxxxxxxxxxx,
│ provider "provider[\"registry.opentofu.org/prefecthq/prefect\"]" produced
│ an unexpected new value: .trigger.event.after[1]: was
│ cty.StringVal("prefect.flow-run.Completed"), but now
│ cty.StringVal("prefect.flow-run.Failed").
│
│ This is a bug in the provider, which should be reported in the provider's
│ own issue tracker.
╵
╷
│ Error: Provider produced inconsistent result after apply
│
│ When applying changes to prefect_automation.xxxxxxxxxxxxxxxxxxxxx,
│ provider "provider[\"registry.opentofu.org/prefecthq/prefect\"]" produced
│ an unexpected new value: .trigger.event.after[2]: was
│ cty.StringVal("prefect.flow-run.Crashed"), but now
│ cty.StringVal("prefect.flow-run.Completed").
│
│ This is a bug in the provider, which should be reported in the provider's
│ own issue tracker.
╵
╷
│ Error: Provider produced inconsistent result after apply
│
│ When applying changes to prefect_automation.xxxxxxxxxxxxxxxxx,
│ provider "provider[\"registry.opentofu.org/prefecthq/prefect\"]" produced
│ an unexpected new value: .trigger.event.after[3]: was
│ cty.StringVal("prefect.flow-run.Failed"), but now
│ cty.StringVal("prefect.flow-run.Cancelled").
│
│ This is a bug in the provider, which should be reported in the provider's
│ own issue tracker.

Sorry, I said the 'expect' field but its actually the 'after' field in this example.  I saw a similar error also in the expect field.  Just using the CLI to inspect the automation looks like the API returns these lists in a random order.  Each time you run:
prefect automation inspect xxxxxx

the list fields come back in a different order

@jbnitorum we've just cut a new patch release with this fix, could you give this a whirl with 2.13.3 when it's up in the TF registry?
https://github.com/PrefectHQ/terraform-provider-prefect/releases/tag/v2.13.3


Can't rebuild image due to python3-bodhi-client

After bumping the base image to F37 we need to change the hardwired installation of python3-bodhi[-client] from Koji  to F37 packages.
Problem is that bodhi-5.7.5-1.fc37 was built before F37 switched to Python 3.11 so it still requires 3.10.
Depsolve Error occurred:
 Problem 1: conflicting requests
  - nothing provides python3.10dist(click) needed by python3-bodhi-client-5.7.5-1.fc37.noarch
  - nothing provides python3.10dist(koji) needed by python3-bodhi-client-5.7.5-1.fc37.noarch
  - nothing provides python(abi) = 3.10 needed by python3-bodhi-client-5.7.5-1.fc37.noarch
  - nothing provides python3.10dist(python-fedora) >= 0.9 needed by python3-bodhi-client-5.7.5-1.fc37.noarch
 Problem 2: conflicting requests
  - nothing provides python3.10dist(setuptools) needed by python3-bodhi-5.7.5-1.fc37.noarch
  - nothing provides python(abi) = 3.10 needed by python3-bodhi-5.7.5-1.fc37.noarch

I'm not sure it would even be possible to request rebuilding it because the source bodhi (v5) package has already been replaced by a bodhi-client (v6).
What are the options now? Build and install it from Copr? Install it from PyPI? @TomasTomecek do we still need the bodhi v5?

Hups, that's totally a mistake, we should be using bodhi v6 now. I probably forgot to remove that hardwiring, sorry about that.

I'm really confused right now: there is no bodhi in here: https://github.com/packit/packit-service/actions/runs/3707470585/jobs/6283921336 (in the installation output)

okay, checking the actual image right now:
$ podman run --rm -ti quay.io/packit/packit-worker:latest bash

bash-5.2# rpm -qa |grep bodhi

bodhi-client-6.0.1-4.fc37.noarch

did you already push it? ^

$ podman... quay.io/packit/packit-worker:prod bash
Trying to pull quay.io/packit/packit-worker:prod...
Getting image source signatures
Copying blob a7c24621a4a9 done
Copying blob f158c0cca9e0 done
Copying blob 4d9ab5d4e424 done
Copying config fdd30820cc done

bash-5.1# rpm -qa |grep bodhi
bodhi-client-6.0.0-2.fc35.noarch

glad to see we're really on v6: no idea about that dependency hell to be honest

heh, I'm talking about packit, not packit-worker :-)
https://github.com/packit/packit/actions/runs/3701983708/jobs/6291106015

heh, I'm talking about packit, not packit-worker :-) https://github.com/packit/packit/actions/runs/3701983708/jobs/6291106015

we should be using bodhi v6 now. I probably forgot to remove that hardwiring

I can do that, no problemo, see #1813

thanks! that's what really confused me: that we still have a standalone packit container image


A fix for abnormal augmented grid features

There was a bug in the deeprank.generate.DataGenerator.DataGenerator._rotate_feature method. This has been solved an tested.
Several unit tests have been added to be sure that the charge feature isn't given any extreme values. The 5bzztest.parq file has been preprocessed and visualized with the grid_view.py script. The features grids look OK now.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

60 of 70 (85.71%)  changed or added relevant lines in 7 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 72.685%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

deeprank/generate/DataGenerator.py
20
30
66.67%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
2528

Relevant Lines:
3478

💛  - Coveralls


General Naming Conventions

Camel Casing or Snake Casing or ?
Any ideas

I realy love Camel Casing but lately have have been falling in love with Snake casing.

One good thing that snake casing has it uses _ with look like spaces with do prevent confusion.

Would say something like that for PODs/Classes
struct <?>LexerContext;
and for functions i would propose:
example_test()
methods probably equivalent.

I've been mostly using Java/c#/kotlin so something like exampleTest() or struct LexelContext<?> feels more familiar

Did add the Question mark because how knows if we want to use a prefix or not was probably not a good idea.

oh.. I though you were referencing generics

I think the best convention is as follows:
var snake_case: PascalCase = camelCase()

It allows you to easily differentiate functions, variables, and types, which is useful in a language with UFCS and generics.

I would like methods and variables to have the same convention

why tho

methods sould be snake case

but what objective advantage does that have

Honestly, I prefer variable names and function names to be camelCase and classes to be PascalCase because of consistency. I also think that having all three be different styles is a good argument to. Myvar though, I agree with k1tt3hk4t. There is really no advantage to having functions and variables the same style and types different.


[Pager] Add migration guide to new Pager API in androidx.compose.foundation

Thanks for the issue. We do this as part of the deprecation process which will happen when Compose foundation Pager is released in a stable release. I'll make sure I tag this issue when the guide goes up.