bpf: use skb->ifindex for FIB lookup in handle_*_from_lxc()

L7 LB traffic is first redirected to envoy for load-balancing. The corresponding local packet by envoy is then sent back to handle_policy_egress() by the cilium_from_host program via tail-call.
In a config with ENABLE_HOST_ROUTING and multiple devices, it then potentially requires a in-BPF FIB lookup. For this lookup, we currently use ctx->ingress_ifindex (aka skb->skb_iif).
For the normal veth->veth transition, the skb->skb_iif would be set by the __netif_receive_skb() in process_backlog(). But for a local packet (as generated by envoy for L7 LB) skb->skb_iif is not set. Consequently such a FIB lookup for ctx->ingress_ifindex 0 will fail, and the packet gets dropped with DROP_NO_FIB.
Use ctx->ifindex instead. This changes nothing for normal from-pod traffic, but allows L7 LB traffic to perform a FIB lookup.
Note that for the L7 LB case, ctx->ifindex will point at cilium_host (rather than the pod's veth interface).
Fix FIB lookup for traffic to a L7 service backend, when BPF host-routing is enabled and multiple external devices are configured.

/test

Runtime tests flaked in one of LBIPAM tests (opened https://github.com/cilium/cilium/issues/24193). Otherwise all green.


"useSetState" doesn't really act similar to class component's setState. It doesn't have any callback function and i got mistaken because of it, because i wanted to use that feature but it wasn't there.

What is the current behavior?
Steps to reproduce it and if possible a minimal demo of the problem. Your bug will get fixed much faster if we can run your code and it doesn't have extra dependencies other than react-use. Paste the link to your JSFiddle or CodeSandbox example below:
What is the expected behavior?
A little about versions:

OS:
Browser (vendor and version):
React:
react-use:
Did this worked in the previous package version?

Docs says that: "it merges object changes into current state.", that's the current behavior which is correct.
I guess you want to have second parameter(callback function which calls after render) like we do in class components right?
If so let me know and I will start implementing that, and If not you should close the issue because for now everything is working as it should.

Docs says that: "it merges object changes into current state.", that's the current behavior which is correct.
I guess you want to have second parameter(callback function which calls after render) like we do in class components right?
If so let me know and I will start implementing that, and If not you should close the issue because for now everything is working as it should.

Docs says that: "it merges object changes into current state.", that's the current behavior which is correct.
I guess you want to have second parameter(callback function which calls after render) like we do in class components right?
If so let me know and I will start implementing that, and If not you should close the issue because for now everything is working as it should.

Yes, that's exactly what i'm talking about.

Docs says that: "it merges object changes into current state.", that's the current behavior which is correct.
I guess you want to have second parameter(callback function which calls after render) like we do in class components right?
If so let me know and I will start implementing that, and If not you should close the issue because for now everything is working as it should.

Yes, that's exactly what i'm talking about.


Better name for TokenProxy in generic-eternal-storage

TokenProxy in generic-eternal-storage is not specific to token, think of more generic name (EternalStorageProxy?)

yes please !


DIOS-6232: maxBitrate, forcePlayoutDelayMin & forcePlayoutDelayMax

Added maxBitrate to "More Stream Configurations" view on Landing
Passed through maxBitrate, forcePlayoutDelayMin & forcePlayoutDelayMax to the SDK for playback
added .DS_Store to gitignore
some linting

Hi @sheiladoherty-dolby - let's have a chat on the coding style changes that were incorporated into this PR. I used Kodeco/former Raywenderlich styles in this project. Happy to see your thoughts and the style guide you follow. Its best to not mix and match though ?

We have always used SwiftFormat for our code formatting in the Denver office. It has pretty standard code practices that it formats code to & any code I touch will get formatted to these standards. But if you would prefer a different method of formatting for this codebase, I can try to refrain from formatting my code with it when I am working on this codebase.


eDiary PWA Screen size based CSS rules

Screens of the eDiary are made for a PWA, mobile environment but this does not mean that they will adapt properly for every screen. We need to take our time and make proper CSS @media based rules for the most important screen resolutions and devices.

handled by bootstrap and CSS flex rules.
iPad Mini

iPhone SE

Surface Duo

Galaxy S8


Bring model Seed into the format of those three (VMH, HMDB, BIGG)

[ ] Refactor alias extraction
[ ] Reimplement pka/pkb
[ ] Reimplement the correct translation to identifier.org (This needs not be implemented in a common way anyway)
[ ] Reimplement inchi handling

# get the inchi strings
            smiles = mseed.loc[mseed["id"] == met._id, "smiles"]
            smiles = smiles.loc[~pd.isna(smiles)]
            inchi_strings = []
            # there are only smiles in modelseed and rdkit sometimes fails to convert them - report here if that happens
            for smile in smiles:
                try:
                    inchi_strings.append(smile2inchi(smile))
                except Exception as e:
                    warnings.warn("Could not convert smile to inchi for metabolite {met} and {smile}\n". format(met = met._id, smile = smile) + str(e))
            # get the correct inchi_string, if there was more than one
            if len(inchi_strings) > 1:
                inchi_string = findOptimalInchi(inchi_strings, charge = met._charge)
                if inchi_string is None:
                    raise NotImplementedError()

            elif len(inchi_strings) == 1:
                inchi_string = inchi_strings[0]
            else:
                inchi_string = None


Allow empty strings to be evaluated by regex::searchAll

Proposed fix for issue described at #1785. Confirmed working on my set of tests, and now empty strings seems to be evaluated properly by regex::searchAll when using the caret and dollar operators (^$)
ModSecurity: Warning. Matched "Operator `Rx' with parameter `^$' against variable `REQUEST_HEADERS:User-Agent' (Value: `' ) [file "/usr/local/nginx/conf/modsecur$ty.conf"] [line "287"] [id "123456"] [rev ""] [msg ""] [data "Matched data: "] [severity "0"] [ver ""] [maturity "0"] [accuracy "0"] [hostname "<IP_ADDRESS>"] [u$i "/"] [unique_id "152663695450.272548"] [ref "o0,0v56,0"] ModSecurity: Warning. Matched "Operator `Rx' with parameter `^$' against variable `REQUEST_HEADERS:Accept' (Value: `' ) [file "/usr/local/nginx/conf/modsecurity.$onf"] [line "721"] [id "12345"] [rev ""] [msg ""] [data "Matched data: "] [severity "0"] [ver ""] [maturity "0"] [accuracy "0"] [hostname "<IP_ADDRESS>"] [uri "/$] [unique_id "152663695450.272548"] [ref "o0,0v81,0"] ModSecurity: Warning. Matched "Operator `Rx' with parameter `^apache$' against variable `ARGS:param1' (Value: `apache' ) [file "/usr/local/nginx/conf/modsecurity$conf"] [line "722"] [id "1234"] [rev ""] [msg ""] [data "Matched data: apache"] [severity "0"] [ver ""] [maturity "0"] [accuracy "0"] [hostname "<IP_ADDRESS>"] [$ri "/"] [unique_id "152663695450.272548"] [ref "o0,6v21,6"]
Testsuites seems happy as well :)

Merged! thanks!


Multiple GDExtensions can't be registered in iOS export

Godot version
Godot v4.2.rc1
System information
macOS 13.6.1 - GLES3 (Compatibility) - ANGLE (AMD, ANGLE Metal Renderer: AMD Radeon Pro 560X, 版本13.6.1（版号22G313）) - Intel(R) Core(TM) i7-8850H CPU @ 2.60GHz (12 Threads)
Issue description
v4.2rc1 can load a single GDExtension in iOS export successfully. But when two GDExtensions are added, one of them will be registered twice and the other won't get registered.
GDExtensions are statically linked. Xcode 15.0.1 is used. Tested on iOS 12.5.3 iPod touch 6.
GDExtensions work fine in macos editor.
Steps to reproduce
Export the attached sample project to iOS and run it on an iOS device.
Minimal reproduction project
gdbug2.4.zip
There are two GDExtension libraries in this project: ext1 and ext2. They will print logs when registering classes.

CC @bruvzg

I tried the fix in bruvzg/static_multi with my sample project. Both extensions are registered twice. Error messages are logged.
USER ERROR: Attempt to register extension class 'Foo', which appears to be already registered.
   at: _register_extension_class_internal (core/extension/gdextension.cpp:386)
USER ERROR: Attempt to register extension class 'Bar', which appears to be already registered.
   at: _register_extension_class_internal (core/extension/gdextension.cpp:386)


Make sure you've used Screengrab.screenshot() in your tests and that your expected tests are being run.  [!] No screenshots were detected 📷❌.

@fastlaneTeam help me to solve this issue
Macs-Mac-mini:screenshotTestApplication machd$ bundle exec fastlane screengrab
[✔] 🚀
[11:46:55]: Successfully loaded '/Users/machd/Documents/Automator/screenshotTestApplication/fastlane/Screengrabfile' 📄
+-----------------------------+--------------------------------------+
|          Detected Values from './fastlane/Screengrabfile'          |
+-----------------------------+--------------------------------------+
| app_package_name            | com.example.screenshottestapplicati  |
|                             | on                                   |
| use_tests_in_packages       | ["com.example.fastlanex.view.fastla  |
|                             | ne"]                                 |
| app_apk_path                | app/build/outputs/apk/debug/app-deb  |
|                             | ug.apk                               |
| tests_apk_path              | app/build/outputs/apk/androidTest/d  |
|                             | ebug/app-debug-androidTest.apk       |
| locales                     | ["en-US"]                            |
| clear_previous_screenshots  | true                                 |
| test_instrumentation_runner | androidx.test.runner.AndroidJUnitRu  |
|                             | nner                                 |
+-----------------------------+--------------------------------------+
[11:46:55]: Using adb found at /Users/machd/Library/Android/sdk/platform-tools/adb which is not within the specified ANDROID_HOME at ~/Library/Android/sdk/
[11:46:55]: The aapt command could not be found relative to your provided ANDROID_HOME at ~/Library/Android/sdk/
[11:46:55]: Please ensure that the Android SDK is installed and you have the build tools downloaded
+-----------------------------+--------------------------------------+
|                   Summary for screengrab 2.149.1                   |
+-----------------------------+--------------------------------------+
| tests_package_name          | com.example.screenshottestapplicati  |
|                             | on.test                              |
| android_home                | ~/Library/Android/sdk/               |
| locales                     | ["en-US"]                            |
| clear_previous_screenshots  | true                                 |
| output_directory            | fastlane/metadata/android            |
| skip_open_summary           | false                                |
| app_package_name            | com.example.screenshottestapplicati  |
|                             | on                                   |
| use_tests_in_packages       | ["com.example.fastlanex.view.fastla  |
|                             | ne"]                                 |
| test_instrumentation_runner | androidx.test.runner.AndroidJUnitRu  |
|                             | nner                                 |
| ending_locale               | en-US                                |
| use_adb_root                | false                                |
| app_apk_path                | app/build/outputs/apk/debug/app-deb  |
|                             | ug.apk                               |
| tests_apk_path              | app/build/outputs/apk/androidTest/d  |
|                             | ebug/app-debug-androidTest.apk       |
| device_type                 | phone                                |
| exit_on_test_failure        | true                                 |
| reinstall_app               | false                                |
| use_timestamp_suffix        | true                                 |
+-----------------------------+--------------------------------------+
[11:46:55]: Clearing phoneScreenshots within fastlane/metadata/android
[11:46:55]: $ /Users/machd/Library/Android/sdk/platform-tools/adb -s emulator-5554 shell echo $EXTERNAL_STORAGE
[11:46:55]: ▸ /sdcard
[11:46:55]: Cleaning screenshots on device
ls: /sdcard/com.example.screenshottestapplication/screengrab/en-US/images/screenshots: No such file or directory
[11:46:55]: Exit status: 1
ls: /data/user/0/com.example.screenshottestapplication/files/com.example.screenshottestapplication/screengrab/en-US/images/screenshots: Permission denied
[11:46:55]: Exit status: 1
ls: /data/data/com.example.screenshottestapplication/files/com.example.screenshottestapplication/screengrab/en-US/images/screenshots: Permission denied
[11:46:55]: Exit status: 1
ls: /data/data/com.example.screenshottestapplication/app_screengrab/en-US/images/screenshots: Permission denied
[11:46:55]: Exit status: 1
ls: /data/data/com.example.screenshottestapplication/screengrab/en-US/images/screenshots: Permission denied
[11:46:55]: Exit status: 1
[11:46:55]: The aapt command could not be found on your system, so your app APK could not be validated
[11:46:55]: Installing app APK
[11:46:55]: $ /Users/machd/Library/Android/sdk/platform-tools/adb -s emulator-5554 install -t -r app/build/outputs/apk/debug/app-debug.apk
[11:46:55]: ▸ Performing Streamed Install
[11:46:56]: ▸ Success
[11:46:56]: Installing tests APK
[11:46:56]: $ /Users/machd/Library/Android/sdk/platform-tools/adb -s emulator-5554 install -t -r app/build/outputs/apk/androidTest/debug/app-debug-androidTest.apk
[11:46:56]: ▸ Performing Streamed Install
[11:46:57]: ▸ Success
[11:46:57]: Granting the permission necessary to change locales on the device
[11:46:57]: $ /Users/machd/Library/Android/sdk/platform-tools/adb -s emulator-5554 shell pm grant com.example.screenshottestapplication android.permission.CHANGE_CONFIGURATION
[11:46:57]: $ /Users/machd/Library/Android/sdk/platform-tools/adb -s emulator-5554 shell getprop ro.build.version.sdk
[11:46:57]: ▸ 29
[11:46:57]: Granting the permissions necessary to access device external storage
[11:46:57]: $ /Users/machd/Library/Android/sdk/platform-tools/adb -s emulator-5554 shell pm grant com.example.screenshottestapplication android.permission.WRITE_EXTERNAL_STORAGE
[11:46:57]: $ /Users/machd/Library/Android/sdk/platform-tools/adb -s emulator-5554 shell pm grant com.example.screenshottestapplication android.permission.READ_EXTERNAL_STORAGE
[11:46:58]: $ /Users/machd/Library/Android/sdk/platform-tools/adb -s emulator-5554 shell getprop ro.build.version.sdk
[11:46:58]: ▸ 29
[11:46:58]: The aapt command could not be found, so status bar could not be cleaned. Make sure android_home is configured for screengrab or ANDROID_HOME is set in the environment
[11:46:58]: Running tests for locale: en-US
[11:46:58]: $ /Users/machd/Library/Android/sdk/platform-tools/adb -s emulator-5554 shell am instrument --no-window-animation -w
-e testLocale en_US
-e endingLocale en_US
-e appendTimestamp true
-e package com.example.fastlanex.view.fastlane
com.example.screenshottestapplication.test/androidx.test.runner.AndroidJUnitRunner
[11:47:00]: ▸ Time: 0.001
[11:47:00]: ▸ OK (0 tests)
[11:47:00]: Pulling captured screenshots for locale en-US from the device
ls: /sdcard/com.example.screenshottestapplication/screengrab/en-US/images/screenshots: No such file or directory
[11:47:00]: Exit status: 1
ls: /data/user/0/com.example.screenshottestapplication/files/com.example.screenshottestapplication/screengrab/en-US/images/screenshots: Permission denied
[11:47:00]: Exit status: 1
ls: /data/data/com.example.screenshottestapplication/files/com.example.screenshottestapplication/screengrab/en-US/images/screenshots: Permission denied
[11:47:00]: Exit status: 1
ls: /data/data/com.example.screenshottestapplication/app_screengrab/en-US/images/screenshots: Permission denied
[11:47:00]: Exit status: 1
ls: /data/data/com.example.screenshottestapplication/screengrab/en-US/images/screenshots: Permission denied
[11:47:00]: Exit status: 1
[11:47:00]: Make sure you've used Screengrab.screenshot() in your tests and that your expected tests are being run.
[!] No screenshots were detected 📷❌

No its not working even adding it

@fastlaneTeam
help me out to solve this issue i have been trying in many ways but it is not at all working.

@SanjuSona-19 See this from your log:

[11:46:55]: Using adb found at /Users/machd/Library/Android/sdk/platform-tools/adb which is not within the specified ANDROID_HOME at ~/Library/Android/sdk/
[11:46:55]: The aapt command could not be found relative to your provided ANDROID_HOME at ~/Library/Android/sdk/
[11:46:55]: Please ensure that the Android SDK is installed and you have the build tools downloaded


Request support for Geyserwise Max

Good Afternoon
If you could please look at the provided info in order to create support for a Geyserwise Max control unit.
I have the following information. Please let me know if I have omitted anything?

DPID
Description
Example
unit

1
Power
On / Off
-

2
Mode
Holiday or Normal
-

10
Geyser Temp.
31
Deg C

13
Element
On / Off
-

20
Fault
E1 to E9
-

101
Pump
On / Off
-

102
Solar Differential
7
Deg C

103
Block 1
55
Deg C

104
Block 2
55
Deg C

105
Block 3
55
Deg C

106
Block 4
55
Deg C

107
Anti Freeze Temp.
5
Deg C

108
Collector
38
Deg C

109
Element Run Time
10
hours

Product ID:  atagzqpggkegajmw
Thanks in advance

Duplicate of #765.  No new information is given by this report, local protocol values for the mode dp are still unknown.


Release v2.13.1 cherry-pick

[x] https://github.com/kedacore/keda/pull/5452
[x] https://github.com/kedacore/keda/pull/5421
[x] https://github.com/kedacore/keda/pull/5518
[x] https://github.com/kedacore/keda/pull/5519

Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)

Relates to # https://github.com/kedacore/keda/issues/5504

/run-e2e


Add/Edit with default value

update user guide, dev guide
Add/Edit with default value

Coverage decreased (-0.1%) to 89.616% when pulling 0682ca3d9cfe42044837d9c8777d68f08bd53651 on edit-issue-61 into b5ecde450f0d920a76c1d3910fb495b7fff20d58 on master.

Coverage decreased (-0.1%) to 89.616% when pulling 0b429a805b25c98cbbde6a480197e94a4ee1d302 on edit-issue-61 into b5ecde450f0d920a76c1d3910fb495b7fff20d58 on master.


DebugBar Routes Collector throws ErrorException when optional parameter is not passed to controller

Calling a controller method:
    public function edit($id = NULL) {
without specifying the optional parameter $id like in http://localsite/admin/article/edit
throws ErrorException with the white screen of death if the debug bar is active.
The same error was reported in the forum by user qury
https://forum.codeigniter.com/thread-67555.html
This is what I've been able to debug so far...
Undefined offset: 0	in
BASEPATH/Debug/Toolbar/Collectors/Routes.php line 99
var_dump($rawParams);
array(1) {
   [0]=>  object(ReflectionParameter)#308 (1) {
        ["name"]=>  string(2) "id" }
}
var_dump($router->params());
array(0)

How is your route defined?


fix(designer): Expose get connector for runtime url

Exposing the getConnector function to access the connector RuntimeUrl from Ppux for Hybrid Triggers:

Closing this since it looks like a better solution was found


Take over the body of the previous issue

I want to take over the previous Body when creating an issue on a regular basis.
For example, when taking over an unresolved task.
Proposal 1
If use_old_body: true is specified, it will be overwritten with the old Body.
I think you just need to implement it here.
https://github.com/imjohnbo/issue-bot/blob/5fcd53925854d67510f1d1b8e2f8cd07fea3470a/index.js#L220
But not convenient.
Proposal 2
Take over only a specific part.
For example.
take_over_word: "take over"
old issue
# tesk
- [x] aaa
- [x] bbb
- [ ] ccc

#  take over
- [x] hoge
- [x] fuga
- [ ] foo
- [ ] bar

new issue
# tesk
- [ ] aaa
- [ ] bbb
- [ ] ccc

#  take over
- [x] hoge
- [x] fuga
- [ ] foo
- [ ] bar

Template overwrites task.
The location of the specified keyword remains the same.
This is good, but unfortunately I didn't seem to be able to implement it right away.

Hi @soudai! Have you found any workarounds for this? I'm considering adding an additional output, previous-issue-number, that you might be able to use to overwrite the previous issue in a subsequent run` step or action.

Thank you. @imjohnbo
The problem is still not solved.
I think the previous-issue-number idea is great!!
Being able to specify where it can be overwritten solves my problem.
I'd love to see it happen!

Done here! It's available on all semantically relevant tags: v3, v3.2, v3.2.0.


Make orbit_camera use left click to pan to use on laptops

The orbit_camera package uses holding right-click to pan. This is hard to use on most laptops touchpads as it is difficult to hold down right click and move the mouse. Using leftclick allows you to do both the press and drag or the double tap and hold gestures to pan around; none of which are possible with right click.
What do you others think of this? @philpax?

Hmmm, I think I considered this, but it might conflict with existing code that uses left-click (like the screen ray example). The other, bigger, issue is that UI doesn't steal focus by default, so clicking on UI elements will cause the camera to move.
I think this is something we should eventually fix (and generally make the orbit camera more featureful), but I don't think it'll get done in this milestone.


Get config from graphql-config

Currently I have to configure GQL client twice:

graphql.config.yml for IDE language server
nuxt.config.ts for this module

IDE get config by graphql-config which is required by graphql-codegen/cli, requirement of this module, anyway.
Could you add option to use this also? For most use-cases it will be enough to have just one GraphQL config.

Trying to configure multiple clients, it is hell to learn and maintain the same in two files.


Add session expiry

Criteria

Inactive sessions should expire after a certain amount of time

No longer using these types of sessions.


Responsive Examples

I see that there are a few other closed issues requesting there be examples of responsive classes in your demo. It would be very useful if you provided a few of these to help newcomers.

Hi @meowsus. I've added an example of the responsive modifier classes to the demo - http://colourgarden.net/avalanche/. Thanks for supporting the project.


Article on CosmosDB's geospatial types should disclose that EF Core does not support them

It's commonplace for developers to use CosmosDB via Entity Framework Core, however as of late 2021 EF Core still does not support using CosmosDB's geospatial types as complex-value-types in EF entity classes:

https://github.com/dotnet/efcore/issues/17317
https://github.com/dotnet/efcore/issues/15896
https://github.com/dotnet/efcore/issues/14336

And this is becoming a gotcha for users unfamiliar with how EF works and that EF is fully separate from CosmosDB, for example: https://stackoverflow.com/q/60949543/159145
So a simple advisory in this article that makes it clear that using CosmosDB's own complex-types will not work when going through an ORM layer, like EF Core, instead of using CosmosDB's client lib/API directly.

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

ID: 40e7ff0a-c1fc-70e9-10ba-8f8ec57cea2b
Version Independent ID: 8f5c8c81-2d65-eb15-dc34-6f7864460389
Content: Geospatial and GeoJSON location data in Azure Cosmos DB
Content Source: articles/cosmos-db/sql/sql-query-geospatial-intro.md
Service: cosmos-db
Sub-service: cosmosdb-sql
GitHub Login: @timsander1
Microsoft Alias: tisande

Hi @Jehoel, thanks for bringing this to our attention. I have assigned the issue to the content author to evaluate and update this doc as appropriate

#assign:markjbrown

Thanks @Jehoel for your feedback and apologies for the very slow response.
Yes, EF does not yet have spatial support. I have made a note in that article to clearly call out there is no support for it today.
This change is now in a PR and should publish by tomorrow.
Thanks again for your help.
#please-close


Configure MegaLinter to be started automatically for every PR

It would be useful to configure MegaLinter integration following this commonalities guideline: https://github.com/camaraproject/Commonalities/blob/4d05a3dbf1c9b61d3ec42eed117d78e6b7645d82/documentation/API-linting-Implementation-Guideline.md
This allows to make sure that API definition is and stay compliant to the rules.
Example of MegaLinter comment : https://github.com/camaraproject/SimSwap/pull/153#issuecomment-2357854088)

Thanks.
This looks like a CAMARA universal code checking tool.
If I want to configure Megalint for our project, just follow the instructions to configure it, right？

Perhaps we can ping @rartych for help :)

@rartych Could you please provide guidance on how to configure MegaLinter? Thank you very much.


chef-client license accept pre-upgrade to 15 documentation error

Cookbook version
11.2.0
Chef-client version
14.11.21-1 upgrade to 15.0.300-1
Platform Details
Ubuntu 16.04.6
Scenario:
Trying to accept license prior to upgrade to 15.x of the Chef Infra Client.
Steps to Reproduce:
Set attribute:
default[:chef_client][:license] = 'accept'
Upgraded to 15 and prompted for license.
Expected Result:
Expected automatic license acceptance: i.e.
[2019-05-24T20:49:58+00:00] INFO: Persisting a license for Chef Infra Client at path /etc/chef/accepted_licenses/chef_infra_client
[2019-05-24T20:49:58+00:00] INFO: Persisting a license for Chef InSpec at path /etc/chef/accepted_licenses/inspec
+---------------------------------------------+
✔ 2 product licenses accepted.
+---------------------------------------------+

Actual Result:
License prompt
+---------------------------------------------+
            Chef License Acceptance

Before you can continue, 2 product licenses
must be accepted. View the license at
https://www.chef.io/end-user-license-agreement/

Licenses that need accepting:
  * Chef Infra Client
  * Chef InSpec

Do you accept the 2 product licenses (yes/no)?

Workaround:
Set attribute using config attribute path:
default[:chef_client][:config][:chef_license] = 'accept'
Fix:
chef_client::config is referencing attribute:
node[:chef_client][:chef_license]
Either change the documentation to reflect:
node['chef_client']['chef_license'] - Set to 'accept' or 'accept-no-persist' to accept the license before upgrading to Chef 15.
Or change the chef_client recipe to reference the attribute in documentation (line 92):
chef_license: node['chef_client']['license']

@tas50 - fixed in #630, this can be closed.

This has been merged into master


Detect horizontal text only

I'm currently using craft on a project where I am interested only in horizontal text. At the moment I'm discarding the boxes in which the height is greater than the width, but this method fails sometimes. Is there any way to set the craft model to look for horizontal text only?
Thanks in advance.

@guipaiva can you upload an test image so that i can replicate your issue better? Why is a simple aspect ratio check not enough for you?
There is no setting for horizontal only check in craft but a postprocessing for that purpose can be implemented. I just need to understand your need better.

@fcakyon I can't upload an example at the moment because there are sensitive data and I'm not authorized to share, but the aspect ratio check works most of the time but fails when the font size is too big and the text is too sparse to link with other words (almost always a single letter), then I get some boxes with height being larger than width.
It would be useful if there was a parameter that indicated the rotation of the text so I could filter by that.
I'll keep experimenting with aspect ratio and link threshold parameters, thank you for your time.

@guipaiva i totally understand your reasoning however, CRAFT does not predict the character orientation, it only predicts the region and affinity scores.


rough sketch of a potential way to organize a data model and its related cores and indexes

this is a very rough attempt at a class that uses corestore to define a writer, track remote writers (eventually), and create an index of the winning forks of the data.
this is likely to change a lot but wanted to make a pr to check out.
the indexer is just a placeholder that i repurposed from another experiment that i was working on, and would be replaced with the sqlite indexer.

updated this with changes incorporating the wip updates to mapeo-sqlite-indexer.


 But the library doesn't have a simple open/close method.
Check if it can actually connect to multiple devices on the bus...

Keep the bus available for other stuff, now opening a single device seems to lock up the bus. But the library doesn't have a simple open/close method.

only locks it for the detect function (i think)

Check if it can actually connect to multiple devices on the bus...

works fine with 3 ltc4222's sharing a bus

Status streams now looks like this
Streams
SERIAL [metrec|generic:metrec] ttymxc1 | 19200,8,1,none 13s [-1s]
SERIAL [suna|void] ttymxc2 | 19200,8,1,none     13s [-1s]
!!I2C [ltc1|void] 0x58 @3       Not used yet [-1]
I2C [ltc2|void] 0x56 @3 Not used yet [-1]
I2C [ltc3|void] 0x55 @3 Not used yet [-1]
!! in front of the i2c means a failed probe or no connection.

Additionally now data can be requested with i2c:id instead to i2c:forward,id.
This also accepts a regex instead of id to request from multiple devices with one command.
That was beyond the scope of this issue, and all other points were resolved.


Issue with routing when we make a mistake in our own app

Hi!
In your routing you have:
sylius_sitemap_providers:
    resource: .
    type: sitemap

This causes a problem when we make a mistake in our own routing. If so, it returns always this error:
Exception thrown when handling an exception (Symfony\Component\Config\Exception\LoaderLoadException: Do not add the "sitemap" loader twice in . (which is being imported from "/sylius/vendor/stefandoorn/sitemap-plugin/src/Resources/config/routing.yml"). Make sure there is a loader supporting the "sitemap" type.)

If we comment out the route configuration above, we get the correct error: (as example, in my case)
Exception thrown when handling an exception (Symfony\Component\Config\Exception\LoaderLoadException: The routing file "/sylius/config/routes.yaml" must not specify both the "resource" key and the "path" key for "app_custom". Choose between an import and a route definition in /sylius/config/routes.yaml (which is loaded in resource "/sylius/config/routes.yaml").)

And this is the same issue everytime we make an error in our config.
This is kind of very annoying actually.
Can we try to find a solution to avoid this behavior?
What do you think?
Thanks!
How to reproduce
Of course with the sitemap bundle loaded.
In your config/routes.yaml, add these lines:
example:
    resource: "toto.yaml"

Do not create the toto.yaml file.
Reload and you'll get the error …Do not add the "sitemap" loader twice in….
Now, comment the lines in the routing of the bundle like written above in the description.
Reload and you get this error: …The file "toto.yaml" does not exist…
This is much better don't you think?
Thanks!

Thanks @jacquesbh. I've been looking through this, but I find it hard to figure out what is the issue here. I've also reverted to the Symfony docs, and basically it's following the suggestions there: https://symfony.com/doc/current/routing/custom_route_loader.html#using-the-custom-loader. I'm a bit lost where the error is coming from therefore.

@jacquesbh Could you try removing the Exception and letting it load twice? I'm curious what happens to the routing table and performance if the router get's executed twice. Alternative is to see in case it's already loaded, to just return an empty RouteCollection.


How will you handle ordering and skip/take?

First of all, congratulations for taking on this big challenge!
Let's say you want to display a list of top 5 users (ordered desc by points). Given that another user gains enough points to be in top 5, how will you invalidate the results?

In this case, it will probably need to be custom - you'll get to pick from one of three options:

Either always invalidate that list whenever points change on any user (probably not good)
Fire custom invalidations when you know for sure the points are close enough to be in the top list
Render the list via polling and never invalidate it

How is this done currently in Meteor? I believe the server publishes all the users and the client has a reactive query through minimongo, right? But this is not scalable for, let's say, 1M users.

@deiucanta . No, the server does not have to publish all users. The main reason why the current approach is not very scalable (although this is always mentioned very vaguely, I haven't seen any good benchmark yet), is that the server has to process all changes to a database, even if they are irrelevant to the users connected to that server + the server has to cache all the data a client is subscribed to (called the mergebox) in order to only send data once.

@sebakerckhof makes sense
Any ideas on how RethinkDB does it? They can handle thousands of reactive queries.

Would the invalidation (e.g. @stubailo's options 1 and 2) have to be triggered 'manually' or would Apollo do it reactively, accordingly those custom rules?

@batjko That will depend on the exact circumstances. Firing manual invalidations will always have to be an option for performance and flexibility reasons, but in many applications where performance isn't critical, Apollo could do essentially what livequery is doing today and there would be no need to fire invalidations manually.
My guess is that we will have a version 0 where all invalidations have to be fired manually, and then we'll build more and more support for automatic invalidations for different backends.


Fix/predicate parser custom fields

Moved the in with singe value bug to https://github.com/labd/commercetools-node-mock/pull/11, keep it as is for now.


would it be possible for clients to get information about http_api?

Feature

Please describe the feature you are requesting and your use case.

I see there is no way for an irods client to get any information about http-api running in the irods server. It looks like running independently unlike rule engines or msi available in the irods server.
More specifically, may I request that CLIENT_HINTS_AN can contain some information about the http-api? The reason for is that we may offer the api version information to our users programmatically.

Try taking a look at the /info endpoint and letting us know if it meets your needs.

https://github.com/irods/irods_client_http_api/blob/main/API.md#information-operations

But again I first still need to know the  info.

Oh, I misunderstood your question. I don't think that's possible at the moment. If we did offer such a feature, we'd likely need to provide a way for admins to filter the results. Some admins may not want certain things to be discoverable in that way.
Transferring to irods/irods since this is more about the iRODS server / zone.


OrganizationalUnit

I think this needs to be clarified -for Exchange Online, you can't use an on-premises OU path even if the AD object lives in an OU. I can see people expecting this to work that way, e.g. https://www.reddit.com/r/PowerShell/comments/apfwu2/problem_with_getmailbox_organizationalunit/eg829s3/

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

ID: 3b640892-b0a1-4f20-1e25-fe5f0e97f5b6
Version Independent ID: 51774b36-90fa-1d9a-5800-1a54d9f70590
Content: Get-Mailbox (exchange)
Content Source: exchange/exchange-ps/exchange/mailboxes/Get-Mailbox.md
Service: exchange-powershell
GitHub Login: @chrisda
Microsoft Alias: chrisda

@officedocsbot assign @yogkumgit

@chrisda Please merge this PR #4286

@dariomws306  Thank you very much for the contribution and updating the documentation with PR. @adamfowlerit Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Inline app identifier

Changes
Minor improvement to leverage environment variables and provide smoother integration.
Checklist

[x] Read the contribution guide and accept the code of conduct
[x] Readme (updated or not needed)
[ ] Tests (added, updated or not needed)

Thank you for your proposal.
While this reduces a line of code, I would argue that this actually makes the working of it harder to understand.
What's your take on this @davidmfinol?

My motivation wasn’t reducing line count. I accidentally skipped this step
myself and had to figure out what I’m doing wrong. Every additional manual
work leads to more errors, imho.
On 6 Jun 2021 Sun at 10:51 PM Webber Takken @.***>
wrote:

Thank you for your proposal.
While this reduces a line of code, I would argue that this actually makes
the working of it harder to understand.
What's your take on this @davidmfinol https://github.com/davidmfinol?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/game-ci/documentation/pull/183#issuecomment-855453000,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABSAOWH33CQKCDQKVCTPSITTRPGTRANCNFSM46GII2AQ
.

--
Thanks,
Enes.

I like this!
I actually wanted to do it this way, but for whatever reason, I couldn't get it to work.
I'll test this with my personal project, and merge it if it works.

I see your point and agree too.


[Query] Page versioning after migration

Hello,
After migrating static templates to editable templates using modernize tools. Will out-of-the-box functionality: reverting to page version or comparing version with current page works?
Please let us know.
https://docs.adobe.com/content/help/en/experience-manager-64/authoring/siteandpage/working-with-page-versions.html#reverting-to-a-page-version

With v2.0 (forthcoming) before Structure or Full conversions, a new version of the page will be created.
You can use the UI to compare the before/after transformations were made on a page.


Docker - The "CompileSass" task failed unexpectedly. An error occurred trying to start process '/root/.nuget/packages/aspnetcore.sasscompiler/<IP_ADDRESS>/build/../runtimes/linux-x64/sass' with working directory '/source/src/Web'. No such file or directory

Describe the bug
I am publishing a release version of my app to run on Docker (so not trying to develop in Docker), and I am getting the following error:
docker build --pull --rm -f "Dockerfile" -t crm.promosys:latest "."
[+] Building 15.9s (22/23)
 => [internal] load build definition from Dockerfile             0.0s
 => => transferring dockerfile: 973B                             0.0s
 => [internal] load .dockerignore                                0.0s
 => => transferring context: 35B                                 0.0s
 => [internal] load metadata for mcr.microsoft.com/dotnet/aspne  0.3s
 => [internal] load metadata for mcr.microsoft.com/dotnet/sdk:6  0.1s
 => [build  1/15] FROM mcr.microsoft.com/dotnet/sdk:6.0-alpine@  0.0s
 => [internal] load build context                                0.1s
 => => transferring context: 14.80kB                             0.0s
 => [stage-1 1/3] FROM mcr.microsoft.com/dotnet/aspnet:6.0-alpi  0.0s
 => CACHED [stage-1 2/3] WORKDIR /app                            0.0s
 => CACHED [build  2/15] WORKDIR /source                         0.0s
 => CACHED [build  3/15] RUN apk update                          0.0s
 => CACHED [build  4/15] RUN apk add --upgrade apk-tools         0.0s
 => CACHED [build  5/15] RUN apk upgrade --available             0.0s
 => CACHED [build  6/15] RUN apk add --update nodejs npm         0.0s
 => CACHED [build  7/15] RUN sync                                0.0s
 => CACHED [build  8/15] RUN reboot                              0.0s
 => CACHED [build  9/15] COPY Directory.Build.props ./           0.0s
 => CACHED [build 10/15] COPY src/Core/*.csproj ./src/Core/      0.0s
 => CACHED [build 11/15] COPY src/Web/*.csproj ./src/Web/        0.0s
 => CACHED [build 12/15] RUN dotnet restore /source/src/Web/Web  0.0s
 => CACHED [build 13/15] COPY src/Web/. ./src/Web/               0.0s
 => CACHED [build 14/15] WORKDIR /source/src/Web                 0.0s
 => ERROR [build 15/15] RUN dotnet publish /source/src/Web/Web  15.5s
------
 > [build 15/15] RUN dotnet publish /source/src/Web/Web.csproj -c release -o /app -r linux-musl-x64 --no-self-contained --no-restore:
#19 0.504 Microsoft (R) Build Engine version 17.2.0+41abc5629 for .NET
#19 0.504 Copyright (C) Microsoft Corporation. All rights reserved.
#19 0.504
#19 2.643 /usr/share/dotnet/sdk/6.0.300/Sdks/Microsoft.NET.Sdk/targets/Microsoft.NET.Sdk.targets(1114,5): warning NETSDK1179: One of '--self-contained' or '--no-self-contained' options are required when '--runtime' is used. [/source/src/Core/Core.csproj]
#19 5.978   Core -> /source/src/Core/bin/release/net6.0/Core.dll
#19 15.31 /root/.nuget/packages/aspnetcore.sasscompiler/<IP_ADDRESS>/build/AspNetCore.SassCompiler.targets(11,5): error MSB4018: The "CompileSass" task failed unexpectedly. [/source/src/Web/Web.csproj]
#19 15.31 /root/.nuget/packages/aspnetcore.sasscompiler/<IP_ADDRESS>/build/AspNetCore.SassCompiler.targets(11,5): error MSB4018: System.ComponentModel.Win32Exception (2): An error occurred trying to start process '/root/.nuget/packages/aspnetcore.sasscompiler/<IP_ADDRESS>/build/../runtimes/linux-x64/sass' with working directory '/source/src/Web'. No such file or directory [/source/src/Web/Web.csproj]
#19 15.31 /root/.nuget/packages/aspnetcore.sasscompiler/<IP_ADDRESS>/build/AspNetCore.SassCompiler.targets(11,5): error MSB4018:    at System.Diagnostics.Process.ForkAndExecProcess(ProcessStartInfo startInfo, String resolvedFilename, String[] argv, String[] envp, String cwd, Boolean setCredentials, UInt32 userId, UInt32 groupId, UInt32[] groups, Int32& stdinFd, Int32& stdoutFd, Int32& stderrFd, Boolean usesTerminal, Boolean throwOnNoExec) [/source/src/Web/Web.csproj]
#19 15.31 /root/.nuget/packages/aspnetcore.sasscompiler/<IP_ADDRESS>/build/AspNetCore.SassCompiler.targets(11,5): error MSB4018:    at System.Diagnostics.Process.StartCore(ProcessStartInfo startInfo) [/source/src/Web/Web.csproj]
#19 15.31 /root/.nuget/packages/aspnetcore.sasscompiler/<IP_ADDRESS>/build/AspNetCore.SassCompiler.targets(11,5): error MSB4018:    at System.Diagnostics.Process.Start() [/source/src/Web/Web.csproj]
#19 15.31 /root/.nuget/packages/aspnetcore.sasscompiler/<IP_ADDRESS>/build/AspNetCore.SassCompiler.targets(11,5): error MSB4018:    at AspNetCore.SassCompiler.CompileSass.GenerateCss(String arguments) [/source/src/Web/Web.csproj]
#19 15.31 /root/.nuget/packages/aspnetcore.sasscompiler/<IP_ADDRESS>/build/AspNetCore.SassCompiler.targets(11,5): error MSB4018:    at AspNetCore.SassCompiler.CompileSass.GenerateSourceTarget(SassCompilerOptions options)+MoveNext() [/source/src/Web/Web.csproj]
#19 15.31 /root/.nuget/packages/aspnetcore.sasscompiler/<IP_ADDRESS>/build/AspNetCore.SassCompiler.targets(11,5): error MSB4018:    at System.Collections.Generic.List`1.InsertRange(Int32 index, IEnumerable`1 collection) [/source/src/Web/Web.csproj]
#19 15.31 /root/.nuget/packages/aspnetcore.sasscompiler/<IP_ADDRESS>/build/AspNetCore.SassCompiler.targets(11,5): error MSB4018:    at AspNetCore.SassCompiler.CompileSass.Execute() [/source/src/Web/Web.csproj]
#19 15.31 /root/.nuget/packages/aspnetcore.sasscompiler/<IP_ADDRESS>/build/AspNetCore.SassCompiler.targets(11,5): error MSB4018:    at Microsoft.Build.BackEnd.TaskExecutionHost.Microsoft.Build.BackEnd.ITaskExecutionHost.Execute() [/source/src/Web/Web.csproj]
#19 15.31 /root/.nuget/packages/aspnetcore.sasscompiler/<IP_ADDRESS>/build/AspNetCore.SassCompiler.targets(11,5): error MSB4018:    at Microsoft.Build.BackEnd.TaskBuilder.ExecuteInstantiatedTask(ITaskExecutionHost taskExecutionHost, TaskLoggingContext taskLoggingContext, TaskHost taskHost, ItemBucket bucket, TaskExecutionMode howToExecuteTask) [/source/src/Web/Web.csproj]
------
executor failed running [/bin/sh -c dotnet publish /source/src/Web/Web.csproj -c release -o /app -r linux-musl-x64 --no-self-contained --no-restore]: exit code: 1

To Reproduce
Here is my Dockerfile.
# https://hub.docker.com/_/microsoft-dotnet
FROM mcr.microsoft.com/dotnet/sdk:6.0-alpine AS build
WORKDIR /source

# update with dependencies
RUN apk update
RUN apk add --upgrade apk-tools
RUN apk upgrade --available
RUN apk add --update nodejs npm
RUN sync
RUN reboot

# copy Directory.Build.props over as that contains TargetFramework
COPY Directory.Build.props ./

# copy csprojs and restore as distinct layers
COPY src/Core/*.csproj ./src/Core/
COPY src/Web/*.csproj ./src/Web/
RUN dotnet restore /source/src/Web/Web.csproj -r linux-musl-x64

# copy everything else and build app
COPY src/Web/. ./src/Web/
WORKDIR /source/src/Web
RUN dotnet publish /source/src/Web/Web.csproj -c release -o /app -r linux-musl-x64 --no-self-contained --no-restore

# final stage/image
FROM mcr.microsoft.com/dotnet/aspnet:6.0-alpine-amd64
WORKDIR /app
COPY --from=build /app ./

EXPOSE 80
ENV ASPNETCORE_URLS=http://+:80
ENTRYPOINT ["./Web"]

** Screenshots:**
Here is my directory structure in comparison to my Dockerfile etc

Desktop (please complete the following information):
 OS: 64bit Mac OS X 10.15.7 19H1922
 Kernel: x86_64 Darwin 19.6.0
 Uptime: 2h 43m
 Packages: 547
 Shell: zsh 5.9
 Resolution: 1920x1080
 DE: Aqua
 WM: Quartz Compositor
 WM Theme: Blue
 Disk: 11G / 480G (5%)
 CPU: Intel Core i7-3615QM @ 2.30GHz
 GPU: Intel HD Graphics 4000
 RAM: 8217MiB / 16384MiB

dotnet --info
.NET SDK (reflecting any global.json):
 Version:   6.0.300
 Commit:    8473146e7d

Runtime Environment:
 OS Name:     Mac OS X
 OS Version:  10.15
 OS Platform: Darwin
 RID:         osx.10.15-x64
 Base Path:   /usr/local/share/dotnet/sdk/6.0.300/

Host (useful for support):
  Version: 6.0.5
  Commit:  70ae3df4a6

.NET SDKs installed:
  3.1.419 [/usr/local/share/dotnet/sdk]
  6.0.300 [/usr/local/share/dotnet/sdk]

.NET runtimes installed:
  Microsoft.AspNetCore.App 3.1.25 [/usr/local/share/dotnet/shared/Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 6.0.5 [/usr/local/share/dotnet/shared/Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 3.1.25 [/usr/local/share/dotnet/shared/Microsoft.NETCore.App]
  Microsoft.NETCore.App 6.0.5 [/usr/local/share/dotnet/shared/Microsoft.NETCore.App]

To install additional .NET runtimes or SDKs:
  https://aka.ms/dotnet-download

Additional context
Any help would be greatly appreciated. By the way, the workflow with your Sass Compiler library is fantastic by the way.
Works great with donet watch / hot reload 🙏

I'm going to close this as I switched the underlying docker image from alpine to a more standard Ubuntu one, and it now works.
I will once I have time try to debug why the alpine one didn't work and will update here for reference.
Sorry if you wasted any time on this

I have reopened this as I've narrowed down the issue a little and have created a repo to help diagnose over at https://github.com/solrevdev/aspnetcore-alpine-sass

I'm having the exact same issue.
Looking at this issue in the following days.

I will take a look at your repository next

All I could do was compile the Dockerfile, still had some unexplained errors thereafter, looking into those first.

Very interesting!
So that didn't work for me yet your  3 lines worked. Yours uses glibc-2.34-r0.apk where as I grabbed from the source site  glibc-2.35-r0.apk
See my comments
FROM mcr.microsoft.com/dotnet/sdk:6.0-alpine AS build
WORKDIR /source

RUN apk update
- #RUN apk add gcompat

+ # THIS FROM YOURS WORKS! https://github.com/CargoSense/dart_sass#compatibility-with-alpine-linux-mix-sass-default-exited-with-2
+ #RUN wget -q -O /etc/apk/keys/sgerrand.rsa.pub https://alpine-pkgs.sgerrand.com/sgerrand.rsa.pub
+ #RUN wget https://github.com/sgerrand/alpine-pkg-glibc/releases/download/2.34-r0/glibc-2.34-r0.apk
+ #RUN apk add glibc-2.34-r0.apk
+
+ # THIS FROM SOURCE SITE DOES NOT https://github.com/sgerrand/alpine-pkg-glibc#installing
+ RUN wget -q -O /etc/apk/keys/sgerrand.rsa.pub https://alpine-pkgs.sgerrand.com/sgerrand.rsa.pub
+ RUN wget https://github.com/sgerrand/alpine-pkg-glibc/releases/download/2.35-r0/glibc-2.35-r0.apk
+ RUN apk add glibc-2.35-r0.apk

COPY ./ ./
RUN dotnet restore -r linux-musl-x64
RUN dotnet publish -c release -o /app -r linux-musl-x64 --no-self-contained --no-restore

FROM mcr.microsoft.com/dotnet/aspnet:6.0-alpine-amd64
WORKDIR /app
COPY --from=build /app ./

EXPOSE 80
ENV ASPNETCORE_URLS=http://+:80
ENTRYPOINT ["dotnet", "aspnetcore-alpine-sass.dll"]

Very interesting! Happy to see that it compiles successfully now :smile:
Damn version compatibility :joy:

Thanks @a-van-rooyen for your glibc-2.34-r0.apk fix.

Very interesting! Happy to see that it compiles successfully now 😄
Damn version compatibility 😂


Feat/pubnub

Motivation
Needed a way to connect multiple devices with each other. I used "PubNub" for this.
Also added the question page with a question card. The questions will be randomized with the next update.

Beside the maybe unnecessary comment, lgtm 🚀


qemu-vanilla: add patches required in qemu 4.0 to run kata containers

Add kata specific patches for qemu 4.0:

9p optimization
vm-templating patch

fixes #471
Signed-off-by: Julio Montes<EMAIL_ADDRESS>

/test

@sboeuf could yo take a  look and see if can we have both patches in NEMU?

@jcvenegas NEMU is being used as the vehicle to enable virtio-fs, therefore I think we should leave the 9p patches out of the picture for NEMU. One more thing, I thought those 9p patches would never make it upstream anyway, which is a good reason for NEMU not to carry them.
Globally, I'd say that you Kata should treat NEMU the same way it treats QEMU, which means Kata has to apply those very Kata specific patches on top of those codebases.


Feature Request::: add estimated hours required as shown on Map to camper's code portfolio page

It would be great if potential employers, as an example, would see the estimated hours required to complete the work being showcased when you give someone a link to your personal portfolio page. Currently it gives your Current and Longest Streak, the option to View 'my project' and View 'solution' for Projects, Algorithms, and Challenges, but does not give the viewer any idea how much work potentially went into achieving it.
I believe it would be a great selling feature for campers trying to get recognition for all their hard work when having 'outsiders' visiting and reviewing their work.

@rickstewart I think we already have certificates for that. Are you talking about how much estimated from 'theoretical' 2000+ hours?

I'm closing this as a previous feature request for this was closed by Quincy not long ago. You can see the details at #6236. Thanks and happy coding!


Add option to pass features directly into LSTM and skip embeddings

Actually adding one more commit with some docstring edits and asserts!

@jdunnmon Okay added a test, passes locally, ready for quick review!

@jdunnmon take a look at changes, thanks!


Ratings Demo Not Working

This issue was imported from Closure Library's previous home at http://closure-library.googlecode.com
The link to the original issue is:
https://code.google.com/p/closure-library/issues/detail?id=330

Fixed


How to Read URL of websocket upgrade request

Hi @vinniefalco  I want to read URL of websocket upgrade request, I should use this api:
async_accept_ex(http::header<true,         http::basic_fields<Allocator>> const& req,             ResponseDecorator const& decorator,                 AcceptHandler&& handler);
that right ?
and how to construct a http header object ?

Hmm, async_accept_ex is for when you already have the HTTP upgrade request. If you want to perform custom actions based on the request, you should read the request yourself using boost::beast::http::read or boost::beast::http::async_read. Call boost::beast::websocket::is_upgrade to make sure that it is a valid WebSocket Upgrade request. If that function returns true then inspect the target field of the request object. Finally, to accept the upgrade request and establish a WebSocket session call boost::beast::websocket::stream::async_accept_ex and pass the request object as a parameter.
The documentation provides a description and example for this:
http://www.boost.org/doc/libs/develop/libs/beast/doc/html/beast/using_websocket/handshaking_servers.html#beast.using_websocket.handshaking_servers.passing_http_requests
The "advanced" example servers also perform the operation you want:
Line 528:
https://github.com/boostorg/beast/blob/develop/example/advanced/server/advanced_server.cpp#L528

Oh, I was confused. Thanks for your support. I will try custom it.

Now I can get websocket upgrade request but don't know how to read url in this request. Do you have document for this case ?

Call target() on the message to retrieve the URL.

yeah, I got it. Many thanks.


Ensure screenshots are taken and uploaded to Skia gold, or fail the test

From Discord:

Running into some weirdness.
In https://github.com/flutter/engine/pull/50583, I added another set of tests running Android API v28, expecting new goldens. I only saw a few (<5) and assumed it just matched existing digests and moved on.
Now on an entirely unrelated PR, https://flutter-engine-gold.skia.org/cl/github/50601, they are showing up.
This is pretty crucial because these are the only rendering tests for the Android embedder, if I can't rely on the goldens all of this effort might be wasted.

I'll file a separate issue for some possible race-y conditions that result in a screenshot never being taken, but the test should definitely fail if an expected screenshot is not created and uploaded. @gaaclarke had some similar work for the Impeller golden files, and ended up with a diff file approach that seems reasonable to copy or at least be inspired by.
The outcome of this issue would be:

[ ] We're sure that every test is taking and uploading a screenshot
[ ] A missing screenshot (or failed upload) fails the test suite
[ ] In combination with https://github.com/flutter/flutter/issues/143458, logging makes it clear what did/did not happen

I know i just spent time writing up the functionality I needed in the python.  If you need it though it might be worth transitioning it to a dart script that can be used in multiple places
$ dart compare_dir_contents.dart <path to text file> <path to dir>


LanguageServerHost fileExists / readFile no longer optional?

TypeScript Version:  2.4.2, also in master
This issue was encountered when upgrading javascript-typescript-langserver from typescript 2.3.4 (see https://github.com/sourcegraph/javascript-typescript-langserver/pull/356)
Code
When implementing a LanguageServiceHost without implementing optional fileExists or readFile, the compilerHost instance silently skips import statements:
export class InMemoryLanguageServiceHost implements ts.LanguageServiceHost {
  // only required members implemented here..
}

The above implementation compiled without errors against 2.4.2, but all import statements in source files added to the language server failed to resolve, resulting in diagnostics, missing definitions, completions, etc.
Expected behavior:
Either:

Compilation failure indicating that LanguageServiceHost is now incomplete
Module resolver falls back to ModuleResolutionHost to access fileExists / readFile

Actual behavior:
Imports fail to resolve where they do exist, in nodeModuleResolverWorker, following the path:
nodeLoadModuleByRelativeName -> loadModuleFromFile
loadModuleFromFile ->  tryExtension -> tryFile -> state.host.fileExists

Debugger screenshot showing the issue at runtime at https://github.com/Microsoft/TypeScript/blob/21bbdd33c51f07a46663e9b50fff62e9c007c806/src/services/services.ts#L1166

The typings published at https://github.com/Microsoft/TypeScript/blob/master/lib/typescript.d.ts#L3764-L3765 show that the two members are still optional.
This dependency appears to have been introduced at: https://github.com/Microsoft/TypeScript/commit/2ec92b9c02ba6ac6b97d3fb5af830d0ece6dcd92

@andy-ms can you take a look.

Looks like a dupe of #16616

Thanks for the quick update. A comment about why those members are optional in the published typescript.d.ts might be nice for future implementers?
I'm satisfied with the explanation, feel free to close!


Compiler errors

I can't seem to get compile errors to be reported, every build is reported as successful.
I tried to dig into this a little myself, if I allow the window to pop open/stop the stdout redirect I can definitely see the errors.
When stepping through the code, the output from ScriptCompiler simply doesn't appear to be received -- instead I get the message for the compile file:
from pi_OutputDataReceived : Compiling script file F:\_GameJams\Teacart 1k\Game\Resources\Scripts\main.as
Neither single file nor multi file compile reports errors.
Maybe I didn't follow instructions correctly, eg I didn't do this yet:
It's recommended to modify your scriptcompiler to generate the dump files AFTER first compiling the code. This way the header dump will contain definitions for the script classes, functions, and variables found in your files.

Thanks

Is your compiler relatively recent? There were uses with PrintUnicode in Urho3D's ProcessUtils, piping unicode doesn't work.
It should work out of the box ever since this commit (back on April 15): https://github.com/urho3d/Urho3D/commit/ff8f4b75255de42ac41d6c37b24cf3e0cc3df0ee

I need to remove that sentence, it's no longer needed or recommended now that it scans arbitrary angelscript code to get details about script code for type-info, type/function locations, etc.

That probably should have been obvious, but that was the issue!
Thanks.


Checking whether the repo contains any tags when syncing repo between db and registry

Coverage increased (+0.04%) to 57.175% when pulling 1dffe91f91d37c4d8e62059427d020cc60c58ee4 on ywk253100:170526_bug_fix into 565110d9f172859a17e888220fbdf0c40010d158 on vmware:master.


BaseInflector camel2words behavour changed in 2.0.16

What steps will reproduce the problem?
echo Inflector::camel2words("LEAD");
What is the expected result?
It should echo "Lead"
What do you get instead?
It echos "L E A D"
Additional info

Q
A

Yii version
2.0.16

PHP version
7.1.25

Operating system
Ubuntu

Indeed in previous version it returned Lead but it was wrong because description is:

Converts a CamelCase name into space-separated words.
For example, 'PostTag' will be converted to 'Post Tag'.

2.0.16 change has been introduced in https://github.com/yiisoft/yii2/commit/b9aa0001b65400d5986488d2b64c64291dca1eeb#diff-1d93dd129d9cbdd656d16f9f4716e24e but it was not mentioned in UPGRADE file.

@cebe I hope this is considered a BC bug and not just a docs issue. Inflector::camel2words('TEST') should output Test not T E S T. And it has imposed a security bug on Craft, which was using this method to check environment variable names & other things for certain sensitive-sounding words, (e.g. key in S3_KEY).

While I understand the problem from @brandonkelly point of view we've got here this xkcd case and I would suggest using custom method and not the core Yii helper.

Previously the regex was looking for capital letters that weren’t preceded by another capital letter for this exact reason, so converting TEST => Test was the clearly intended behavior; and since there was no mention of this behavior change in the upgrade guide, I think it’s safe to say that the BC break was not intentional.

It was definitely intentional, there are tests to cover this behavior, and this is how camel case works - capital letters indicates word separators, so TEST definitely should not result Test.

How do you explain the (?<![A-Z]) lookahead in the original regex then? Someone fell asleep on the keyboard?

…and done (https://github.com/yiisoft/yii2/pull/17151/commits/c5d51f42e5981fa5aeba213a026e55a327377532).

How do you explain the (?<![A-Z]) lookbehind in the original regex then? Someone fell asleep on the keyboard?

You should ask author of this regexp ;), but there are no tests for this case and this is not how camel case works. In general this should be true:
Inflector::camel2words(Inflector::camelize('t e s t'), false) === 't e s t'

There was a test case to ensure lower_case gets converted to Lower Case. That’s not strictly camel case either, so I don’t think we should get too hung up on the semantics.
But still, I’m sure something like MyAWSBucket would be considered camel case, and I would expect that to be converted to My Aws Bucket, not My A W S Bucket. Latest commit in #17151 will fix that as well.

There was a test case to ensure lower_case gets converted to Lower Case. That’s not strictly camel case either, so I don’t think we should get too hung up on the semantics.

Underscore has no meaning in camel case, capitalized letter has. You're trying to create some own camel case variation which is not even internally consistent (camelize() does not match camel2words() behavior).

You're trying to create some own camel case variation which is not even internally consistent (camelize() does not match camel2words() behavior).

echo Inflector::camel2words('lower_case'); // outputs: Lower Case
echo Inflector::camelize('Lower Case');    // outputs: LowerCase (not lower_case)

Why do you expect that Inflector::camelize('Lower Case') will return lower_case?

I wouldn’t. My point is that there’s no president for those two functions being internally consistent.

Obviously they're not always interchangeable - both transformations are lossy, so you may loss case of letters and type of word separators. What you should not loos is word separator itself.

The primary point of camel2words() is to find the words in the given string, without being too pedantic about whether the input was actually camelCase. Evidenced by the current test cases:

Input
Expected Output

'camelCase'
'Camel Case'

'lower_case'
'Lower Case'

' tricky_stuff.it-is testing... '
'Tricky Stuff It Is Testing'

'ІЦеДійсноТак!'
'І Це Дійсно Так!'

(If it were strict about only converting actually-camel-cased text, only the first test case would pass.)
Given that, I think expected behavior should be that the function does its best to recognize the words regardless of casing. And I suspect most would agree that FOO should be considered a single word, and FOOBar should be considered two words.

(If it were strict about only converting actually-camel-cased text, only the first and last test cases would pass.)

It does not matter if the input string is not correct camel case, as long as the correct camel case is handled as an actual camel case. Supporting everything else is additional value to the method, it does not affect the main purpose of it. But ignoring case (just a reminder - the case of the letters indicates word separators in camel case, so it is crucial for detecting words) and trying to guess words by using fancy regexp is something completely different - you're breaking the main purpose of this method by introducing some magic behavior. It is only a matter of time when someone hits an edge case when this will not work as expected.
If this is all about ALLCAPS strings, I suggest to lowercase them and then use old regexp to detect words - it should cover LEAD and S3_KEY and does not affect most of the edge cases with actual camel case.

Again I remind you that up until 2.0.16, the regex had a specific check for consecutive capital letters. Whether there was a test case for that use case or not, it was clearly the intent of the function to treat consecutive letters as a single word. And there is no mention of the regression in either the upgrade guide or the commit message (b9aa0001b65400d5986488d2b64c64291dca1eeb):

Fixed yii\helpers\Inflector::camel2words() to work with UTF-8

We can argue about whether the function should or should not be worrying about this until the cows come home, but this is a very clear break in backwards compatibility. Knowing how seriously Yii core devs generally take BC breaks, I’m frankly a bit shocked that we’re even debating this.

We can argue about whether the function should or should not be worrying about this until the cows come home, but this is a very clear break in backward compatibility.

Every other argument aside, the fact this breaks backward compatibility can't be denied, therefore should be considered a regression that should be addressed.

@brandonkelly For me, BC break is the only reason we're debating this. Basically, you have a string that obviously is not a camel case. And you're using a method which Converts a CamelCase name into space-separated words. And you're not getting output you're expecting, so you're proposing to change implementation of this method to not treat camel case as camel case and instead try to detect words using regexp based on some arbitrary rules (why 3 consecutive uppercase letters are treated as a word, but 2 consecutive letters is not a word?). The fact that it worked like that before is the only reason why this does not sounds insane. :P But this is pure https://xkcd.com/1172/ - you can't fix incorrect implementation because people rely on it.

why 3 consecutive uppercase letters are treated as a word, but 2 consecutive letters is not a word?

There’s nothing in the regex that cares about number of characters. Just more closely looking for what looks like actual camelCased text, rather than assuming all capital letters = camelCase.
Pre-2.0.16 behavior: Find all uppercase letters that aren’t preceded by another uppercase letter, and put spaces before them.
2.0.16 behavior: Find all uppercase letters and put spaces before them.
My proposed behavior: Find all uppercase letters that aren’t preceded by another uppercase letter or are followed by a lowercase letter, and put spaces before them.

fooBar
FOO_BAR
FOOBar

Pre-2.0.16 output
✅ Foo Bar
✅ Foo Bar
❌ Foobar

2.0.16 output
✅ Foo Bar
❌ F O O B A R
❌ F O O Bar

#17151 output
✅ Foo Bar
✅ Foo Bar
✅ Foo Bar

The fact that it worked like that before is the only reason why this does not sounds insane. :P But this is pure https://xkcd.com/1172/ - you can't fix incorrect implementation because people rely on it.

You’re not the first one to link to that here, but it’s apples and oranges. In the comic, we can assume there was no code specifically instructing the CPU to overheat. In this case, the pre-2.0.16 version of the function was purposefully looking for consecutive uppercase characters to treat them as a single word. My assumption is that @SilverFire just didn’t fully understand the consequences when he removed that regex lookbehind in b9aa0001b65400d5986488d2b64c64291dca1eeb, and was only concerned with ensuring his new test case passed (where ІЦе would be interpreted as two separate words).

You’re not the first one to link to that here, but it’s apples and oranges.

I disagree. Things that signals intentions are:

documentation (PHPDoc),
method name,
tests.

None of these suggests that this method should work in a way you're expecting.

In the comic, we can assume there was no code specifically instructing the CPU to overheat.

If there were no code for that, there would be no CPU overheat. :)
Anyway, I don't see any sense in continuing this discussion. It is not my decision how to deal with that, I just want to point that this method finally works correctly (in a way that name and PHPDoc suggest) and this issue is about reverting valid bugfix (intentional or not). If BC is a reason for this, then so be it (I still hope it will be fixed in Yii 3 and your implementation will be extracted to separate method and cale2words() will handle camel case as camel case), but note that #17151 breaks BC anyway...

I have some other examples where it makes more sense to not split up upper case characters:

generateCSRF -> generate CSRF
findCPU -> find CPU
...

Also we had a similar discussion before: https://github.com/yiisoft/yii2/pull/4166 where we agree on the fact that splitting these does not make sense. It just turned out that no test had been added for camel2words() only for other similar methods.

@cebe These are just bad names - it should be generateCsrf and findCpu. For the same reason why we have Html helper instead of HTML and Json instead of JSON. I was hoping that more strict rules in Inflector will result more consistent and predictable names (including core framework itself).

@rob006 they are acronyms. The comparison does not make sense.

The Wikipedia entry on Camel Case literally has an example of this mixed-case camel-case format under Programing and Coding:

Programming identifiers often need to contain acronyms and initialisms that are already in uppercase, such as "old HTML file". By analogy with the title case rules, the natural camel case rendering would have the abbreviation all in uppercase, namely "oldHTMLFile".

@brandonkelly The same paragraph has explanation why this is problematic and unpractical. Yii already had CJSON episode, I was hopping that we've learned from the mistakes that this is the wrong approach.

Only problematic when there’s a second abbreviation; that doesn’t mean there aren’t cases where it’s valid.

Only problematic when there’s a second abbreviation

Or for consecutive one-letter words. For me its is enough to call it unreliable - loos of case will give less harm than loos of word separators.

I've re-checked everything and spoke with @SilverFire who introduced the change.

The change was to add unicode only. Logic change wasn't intentional.
In @brandonkelly variant it gives more logical results that are both suitable for his use case, URL slugs and other cases.

I think we'll merge it.

I am totally for redesigning these APIs in 3.0 btw. While trying to write tests for this method I found a few edge cases where a different behavior would be expected. Also the second parameter does really weird things on some inputs.

https://github.com/yiisoft/yii-core/issues/175


Example for snowflake credentials rotation?

Is it possible to upload new snowflake credentials into an environment with this project?

Hey @ernestoongaro - you should be able to use the Snowflake Credential resource combined with the Environment data source/ resource to do this by providing the credential_id to the environment I believe.
Credential: https://registry.terraform.io/providers/GtheSheep/dbt-cloud/latest/docs/resources/dbt_cloud_snowflake_credential
Env: https://registry.terraform.io/providers/GtheSheep/dbt-cloud/latest/docs/resources/dbt_cloud_environment
Is it a new or existing env specifically? With that I should be able to add an example for this.
Thanks for this!

Thanks Gary! It would be for an existing environment, if you have an
example that would help me a lot!
On Thu, Feb 10, 2022 at 11:53 AM Gary James @.***>
wrote:

Hey @ernestoongaro https://github.com/ernestoongaro - you should be
able to use the Snowflake Credential resource combined with the Environment
data source/ resource to do this by providing the credential_id to the
environment I believe.
Credential:
https://registry.terraform.io/providers/GtheSheep/dbt-cloud/latest/docs/resources/dbt_cloud_snowflake_credential
Env:
https://registry.terraform.io/providers/GtheSheep/dbt-cloud/latest/docs/resources/dbt_cloud_environment
Is it a new or existing env specifically? With that I should be able to
add an example for this.
Thanks for this!
—
Reply to this email directly, view it on GitHub
https://github.com/GtheSheep/terraform-provider-dbt-cloud/issues/54#issuecomment-1034832212,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAALVKBP7JPAUXXH34C7SSTU2ORLNANCNFSM5N7RF5TQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

Hey @ernestoongaro - In order to add the credential to an existing environment, you would need to import the environment to Terraform such that it can be managed as a resource (I think), in which case, a full example would be something like:
terraform {
  required_providers {
    dbt = {
      source  = "GtheSheep/dbt-cloud"
      version = "0.0.75"
    }
  }
}

provider "dbt" {
  account_id = <ACCOUNT ID>
  token      = "<TOKEN>"
}

data "dbt_cloud_project" "test_project" {
  project_id = <project ID>
}

resource "dbt_cloud_snowflake_credential" "new_credential" {
  project_id  = data.dbt_cloud_project.test_project.project_id
  auth_type   = "password"
  num_threads = 16
  schema      = "DATA"
  user        = "user"
  password    = "password"
}

resource "dbt_cloud_environment" "test_environment" {
  dbt_version   = "1.0.1"
  name          = "test"
  project_id    = data.dbt_cloud_project.test_project.project_id
  type          = "deployment"
  credential_id = dbt_cloud_snowflake_credential.new_credential.credential_id
}

of course, I'd recommend passing the passwords at apply/ plan time as variables rather than storing in code, they're marked as sensitive in the provider, hope this helps 😸

Wow thanks so much! Will give it a go once I have the other pieces in place. Should this be saved into your examples folder somehow @GtheSheep?

@ernestoongaro good call! I've added the credential and environment as relative files in the examples folder 😺

That's awesome! Thank you
(closed in https://github.com/GtheSheep/terraform-provider-dbt-cloud/commit/4bdacab58516349ea37f70b23afacac895426230 and https://github.com/GtheSheep/terraform-provider-dbt-cloud/commit/858f78db4d0532c2b1078de1cc5b56661ae6a1b5)


compiling multiple smart contracts?

it will be really helpful to see an example for compiling (and running) several smart contracts that call each other - e.g univ2 router calling univ2 pool etc.
tried doing it myself - w/o success, I keep getting "cannot compile more functions after finalizing the module" or segfaults.
thank you.

You'd have to add each bytecode to the module individually using translate, keeping track of the IDs and then calling jit_function for each ID https://github.com/paradigmxyz/revmc/blob/22bfaaee29f29e3843e2b8758d9ea3805499fb25/crates/revmc-cli/benches/bench.rs#L74-L88
A module finalizes once you JIT a function or emit an object, after that it has to be created again if you want to add more functions to it

thank you! :)
got partially successful result.
now i'm getting:
Transaction result: Call { inputs: CallInputs { input: 0x0902f1ac, return_memory_offset: 509..605, gas_limit: 982136, bytecode_address: 0xb4e16d0168e52d35cacd2c6185b44281ec28c9dc, target_address: 0xb4e16d0168e52d35cacd2c6185b44281ec28c9dc, caller: 0x7a250d5630b4cf539739df2c5dacb4c659f2488d, value: Transfer(0), scheme: StaticCall, is_static: true, is_eof: false } }

seems that i'm doing something wrong with compiling them together? (i'm trying to simulate swapExactTokensForTokens)
that's the code more or less (all the rest of the code, e.g storage creation, works good with revm):
...

let uniswap_v2_router_bytecode = Bytes::from(hex::decode(include_str!("../uniswap_v2_router.hex").trim_start_matches("0x")).unwrap());
let uniswap_v2_pool_bytecode = Bytes::from(hex::decode(include_str!("../uniswap_v2_pool.hex").trim_start_matches("0x")).unwrap());
let erc20_bytecode = Bytes::from(hex::decode(include_str!("../erc20.hex").trim_start_matches("0x")).unwrap());

...

let router_id = compiler.translate(None, &uniswap_v2_router_bytecode, SpecId::LONDON)?;
let _pool_id = compiler.translate(None, &uniswap_v2_pool_bytecode, SpecId::LONDON)?;
let _token_id = compiler.translate(None, &erc20_bytecode, SpecId::LONDON)?;

...

let router_fn = compiler.jit_function(router_id)?;
let _pool_fn = compiler.jit_function(_pool_id)?;
let _token_fn = compiler.jit_function(_token_id)?;

...

let contract = Contract {
    bytecode: revmc::interpreter::analysis::to_analysed(revmc::primitives::Bytecode::new_raw(uniswap_v2_router_bytecode.clone())),
    hash: Some(B256::from(keccak256(&uniswap_v2_router_bytecode))),
    target_address: router_address,
    caller: fake_address,
    call_value: U256::ZERO,
    input: create_uniswap_router_request(
        U256::from(10).pow(U256::from(18)),
        U256::ZERO,
        &[weth_address, usdc_address],
        fake_address,
        U256::from(21000000)
    ),
};

let mut interpreter = Interpreter::new(contract.clone(), 1_000_000, false);
let mut memory = SharedMemory::new();

let result = unsafe { router_fn.call_with_interpreter_and_memory(&mut interpreter, &mut memory, &mut external_context) };

here is the full project:
https://github.com/0xSt1ng3R/revmc_uniswap_tests/

what is wrong ? if you mean that you're expecting the call to happen then you want to use Evm, please look at the examples

works now! thank you!
I did a quick benchmark simulating a swap via the univ2 router with one pool and two tokens. It takes 0.1ms in both revm and revmc.
Why and how is that? one could argue that native compilation should give faster runtime...

Can you please share the code?

sure - I made the repo public
https://github.com/0xSt1ng3R/revmc_uniswap_tests
I really hope I broke something :)

Yes, quite a few things actually

you're setting code hash to zero, which messes with the internal revm state, meaning the compiled function is never called
if you fix this, you are calling a function that does not exist anymore, because the JITted function is freed when the compiler is dropped

wow nice!! execution time dropped to ~80µs!!
would it be possible to achieve even faster times?
i missed completely the things you pointed out - but for my spouse, the code did work? how's that possible? the output was correct...
noted about alloy_primitives, thanks!

the first run is slower because the CPU, caches, etc are not warmed up, I would recommend using a benchmark framework if you want more precision like criterion
your code worked because it was just using revm normally

will do that!
any recommendations on built-in performance related upgrades? like maybe turning off gas_metering and such?

Yeah, you can check out the methods on EvmCompiler for all the configuration available.


Update pkcs12 files.

We generated the pkcs12 certificate/key bundles with mkcert. mkcert uses pbeWithSHA1And40BitRC2-CBC for encryption which is now not supported anymore. Solution: Update pkcs12 files.

Closed by ac2706e366b9930dca9ba50aeb0b96fef83b8433.


Cannot build project with AudioKit 4.9.4 using Xcode 11.4 beta 3

Hi, I am having issues building my project with Xcode 11.4 beta 3.
There were no issues when using Xcode 11.3.1 with AudioKit 4.9.3. Then I updated AudioKit to version 4.9.4. Still no issues. Then I downloaded Xcode 11.4 beta 3. Now when building my project I get the following errors:

I am using AudioKit through CocoaPods. I also tried to include the framework manually but the same errors still appear. Is there anything I am doing wrong?

In 4.9.4 we enabled Library Evolution which allows the compiled framework to be used among different versions of Swift/Xcode. I believe this is related to this though I have not observed that behavior myself.
In the forthcoming v5 I have some cleanups of the swiftinterface files to avoid some similar issues (which involved basically renaming the AudioKit class) but I didn't see it happening it on 4.9.
Do you want to maybe try our beta pod of v5? https://github.com/AudioKit/Specs

Thanks @megastep for the explanation. I just tried the beta pod 5.0.b1 and the issue is fixed!

The Xcode 11.4 is released, When will this fix release?

Are you having trouble on Xcode 11.4?

@megastep this should be reopened, I can verify the library doesn't work in 11.4

v4.9.4 does not work on Xcode 11.4, I have over 1000 build errors - please re-open. This library breaks overnight without me knowing since Apple automatically updates Xcode.

What is the Podfile wording?  My attempts won't find the pod.  My latest attempt is
pod 'AudioKit', '~> 5.0.b1'
The platform is 13.3

@DanielSelig that is correct however for this to work you need to make sure you also have the source spec:
source 'https://github.com/AudioKit/Specs.git'

Thank you!

In 4.9.4 we enabled Library Evolution which allows the compiled framework to be used among different versions of Swift/Xcode. I believe this is related to this though I have not observed that behavior myself.
In the forthcoming v5 I have some cleanups of the swiftinterface files to avoid some similar issues (which involved basically renaming the AudioKit class) but I didn't see it happening it on 4.9.
Do you want to maybe try our beta pod of v5? https://github.com/AudioKit/Specs

Hi, I am encountering new issues after using pod 'AudioKit', '~> 5.0.b1', do you know how to get rid of it? thanks

@Winton1992 Change "AudioKit" to "AKManager"

@Winton1992 Change "AudioKit" to "AKManager"

Thanks, bro. Have a good day~~~


Fix: predicate pushdown for Pulsar SQL NPE

Motivation
An NPE can be thrown when predicate pushdown for field publish_time is less than any existing entry.  This is caused because OpFindNewest class in ManagedLedger returns null instead of start position if the condition fails on the first check.

rerun integration tests

rerun java8 tests

rerun integration tests
rerun java8 tests


Test seem to fail due hard coded path from underlying lib

https://github.com/samaloney/stixpy/blob/cdb86094995590da36f3ae5e01f4ca4b9aac819c/stixpy/vis/map_reprojection.py#L107-L109
at need to update to something like this but probably an indication I have made some poor design choices 😞
from stixcore.data.test import test_data
Path(test_data.ephemeris.META_KERNEL_POSITION)

Fixed in #23


--watch crashes mocha on startup

I'm getting error below when running mocha in watch mode. As you can see, the file it tries to find is not even js file. Apparently the file link which breaks the mocha is somehow broken. It shouldn't break the mocha though. Without --watch, mocha runs just fine. i'm running mocha version 3.1.2 with following command from package.json:
"test-rc": "mocha 'frontend/**/*.spec.js' --watch"
The error is following:
Error: ENOENT: no such file or directory, stat '/Users/me/code/my-project/public/index-apitest.php'
at Error (native)
at fs.statSync (fs.js:987:18)
at /Users/me/code/my-project/node_modules/mocha/lib/utils.js:255:11
at Array.forEach (native)
at Object.exports.files (/Users/me/code/my-project/node_modules/mocha/lib/utils.js:253:6)
at /Users/me/code/my-project/node_modules/mocha/lib/utils.js:256:17
at Array.forEach (native)
at Object.exports.files (/Users/me/code/my-project/node_modules/mocha/lib/utils.js:253:6)
at Object. (/Users/me/code/my-project/node_modules/mocha/bin/_mocha:408:26)
at Module._compile (module.js:556:32)
at Object.Module._extensions..js (module.js:565:10)
at Module.load (module.js:473:32)
at tryModuleLoad (module.js:432:12)
at Function.Module._load (module.js:424:3)
at Module.runMain (module.js:590:10)
at run (bootstrap_node.js:394:7)
at startup (bootstrap_node.js:149:9)
at bootstrap_node.js:509:3

if following would be wrapped to try-catch, things does not break anymore:
https://github.com/mochajs/mocha/blob/master/lib/utils.js#L255

Simply ignoring errors at that point isn't likely to be correct behavior -- in this case, as far as I can tell that function shouldn't even be getting paths for files that don't exist, so there's almost certainly something else going wrong that should be addressed; more importantly, in general, suppressing errors could hide and/or worsen other problems.
Any idea where Mocha is getting the path public/index-apitest.php from? Do you have a mocha.opts file?

no idea. I'm not very familiar with mocha architecture. Those non-existing files are broken links. We have vagrant system which creates some links, totally non-related to our frontend, and they are broken on host machine where we handle testing the fe. It seems that mocha is going trough every folder in the project, even though it is pointed to look only at the frontend/ folder. Changing non-matching file does not rerun the test suite though. Same behaviour is in the version 2.5.0

It also seems that ignoring file system errors is pretty common pattern all over the mocha code base. should they be ignored in the place i pointed out too? At least non-existing(ENOENT) files.

I Probably identified the problem (a line which calls the files -function)
https://github.com/mochajs/mocha/blob/master/bin/_mocha#L408
This line should most likely use just variable files, OR lookupFiles function from here: https://github.com/mochajs/mocha/blob/master/bin/_mocha#L364
Maybe someone more familiar with mocha could validate if i'm right? Both functions apparently returns a list of file names with a difference that lookupFiles seems more robust.

Ah, I misread the files function looking at that first line you pointed out; I thought the line getting the error had been given a list of files, but it's actually part of finding the list of files based on looking at the directory recursively and then grabbing only the ones with the right file extensions. That explains why it was looking at the php file at all. So that definitely needs to be fixed.
I'm not sure I'd describe lookupFiles as more robust -- between the fact that it's trying to handle at least three different usage methods (globbing, non-recursive directory/file specification and recursive directory specification) and the fact that it's basically written to silently ignore just about any error interacting with the filesystem at all, I worry that it's already bugged.
But more practically, it doesn't look like lookupFiles is ignoring node_modules and .git the way files is, so I don't think we can just switch over to that... Maybe in the next backwards-incompatible release we could if we made lookupFiles ignore node_modules and .git like files does, because that would also make it much easier to use globbing while avoiding dependencies' test files (which comes up occassionally). Otherwise or until then, I guess we're just going to have to do something to get files to avoid blowing up from broken symlinks.
I suppose suppressing errors the way lookupFiles is doing would technically do that, although I'm still rather wary of it -- it'd probably make the two parts more consistently reliable or unreliable, but I'm not so sure which... Maybe as a more robust alternative we could rewrite files to use globbing (directly)?

i think it would maybe also make sense to just go trough directories & files only once, and then subsetting them as needed (globbing etc). Mochas startup was noticeably faster when i just used same set of files than few lines above, returned by lookupFiles.
My current fix for files() was like this:https://github.com/mochajs/mocha/pull/2538

@villesau please see my comments in #2538
I'll tag this confirmed, because even though I haven't tried it, it's highly plausible.  :smile:


Detect when IDL submodule is not initialized and warn user, improve .fake-codegen

This is largely a continuation of #4155 , and is related to / helps resolve #4162 .
When the IDL submodule doesn't have files in it, builds can fail in a variety of obtuse ways.
Currently it fails when running protoc, as there's a symlink from proto/ into idls/, e.g. causing error from #4155 :
uber/cadence/matching/v1/service.proto:213:12: "api.v1.TaskListPartitionMetadata" is not defined.
proto/public: warning: directory does not exist.
uber/cadence/api/v1/history.proto: File not found.
...

Since this needs to be fixed before running a make ..., as its files are used as target prerequisites,
it can't be auto-initialized by the makefile.  Instead, it now fails the build with a descriptive error,
and the user needs to resolve it before trying again.

While building and testing this I also noticed that the fake protoc/thrift binaries were sometimes surprising me,
so I added a warning about them to make clean.  They might be reasonable to automatically clean up as well,
but I'm leaving them in place for now, so e.g. a build process doesn't accidentally fake -> clean -> do a full build.

Pull Request Test Coverage Report for Build 993d3bfe-8d80-4b88-a0ea-a154f8136657

0 of 0   changed or added relevant lines in 0 files are covered.
69 unchanged lines in 6 files lost coverage.
Overall coverage decreased (-0.02%) to 59.749%

Files with Coverage Reduction
New Missed Lines
%

common/task/weightedRoundRobinTaskScheduler.go
1
89.64%

service/history/execution/mutable_state_task_refresher.go
1
75.29%

service/history/task/transfer_active_task_executor.go
2
69.05%

common/persistence/cassandra/cassandraPersistence.go
3
52.8%

service/history/queue/timer_queue_processor_base.go
4
78.83%

common/persistence/cassandra/cassandraPersistenceUtil.go
58
89.35%

Totals

Change from base Build 6f7bf8dd-90e3-4f38-9611-508bce886dc8:
-0.02%

Covered Lines:
87886

Relevant Lines:
147093

💛  - Coveralls

well apparently that label isn't the (only?) reason automerge is failing.
oh well.


I think this is right - MB

I added a line to identify that I'm doing this right. My question here is what do we do when we want to reject the addition? How does that take place in a pull request?

If we want to reject the addition, then we would not merge the pull request. This is one reason why should assign others as reviewers.
Another possible workflow is to only allow the owner of the project to merge PRs. Everyone else can submit them, but you have one person who merges after approval.

Learning is fun! Thanks for that explanation, Zach.
On Thu, Sep 15, 2022 at 6:08 PM ZacharyHimmelberger <
@.***> wrote:

@.**** commented on this pull request.
We should add a blank line to the end of the R script.
—
Reply to this email directly, view it on GitHub
https://github.com/ZacharyHimmelberger/data-processing-in-R/pull/4#pullrequestreview-1109837509,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2ZZUJ7SBK7L2VECYO4CSHTV6OF57ANCNFSM6AAAAAAQNWSC5A
.
You are receiving this because you modified the open/close state.Message
ID: <ZacharyHimmelberger/data-processing-in-R/pull/4/review/1109837509@
github.com>

--
Matt Best
Motivate Lab
Operations and Research Consultant


Add Llama-github python library to Tools/Service list bottom

Add llama-github Python library to the list
Hello maintainers,
I would like to propose adding the llama-github Python library to the awesome-langchain list. Llama-github is a powerful tool that retrieves highly relevant code snippets, issues, and repository information from GitHub based on user queries, transforming them into valuable knowledge context for LLM Chatbots, AI Agents, and Auto-dev Agents.
Key features of llama-github include:

Efficient Coding Knowledge Retrieval: Advanced retrieval techniques ensure the most pertinent information is found quickly and efficiently.
Repository Pool Caching: An innovative caching mechanism significantly accelerates GitHub search retrieval efficiency and minimizes the consumption of GitHub API tokens.
Advanced Question Analysis: State-of-the-art language models analyze user questions and generate highly effective search strategies and criteria.
Contextual Answer Generation: Rich, contextually relevant answers are generated by combining information retrieved from GitHub with the reasoning capabilities of advanced language models.
Asynchronous Processing: Built from the ground up to leverage the full potential of asynchronous programming, llama-github efficiently handles high-volume workloads without compromising on speed or quality.
Flexible Github API Integration: Easily integrate with various GitHub APIs to tailor the library's capabilities to specific requirements.

Additionally, llama-github utilizes the LangChain framework, as illustrated in the architecture diagram within llama-github readme. This integration further enhances its capabilities, making it a valuable addition to the awesome-langchain list.
Thank you for considering this pull request. I look forward to your feedback and the opportunity to contribute to this fantastic resource for the AI development community.
Best regards,
Jet Xu

Hey @JetXu-LLM, thanks for the add. Looks like an awesome project.

Hey @JetXu-LLM, thanks for the add. Looks like an awesome project.

Thank you for accepting the PR! I'm glad to contribute to this fantastic awesome list. Looking forward to more collaborations in the future ^-^


Getting "'Crashlytics/Crashlytics.h' file not found" on SMXCrashlytics.m

I've updated from 0.0.2 to 0.1.0, followed all steps and I'm getting the following error when trying to compile:
'Crashlytics/Crashlytics.h' file not found on line 2 in file SMXCrashlytics.m.
I have the same import in my AppDelegate.m file without any problems or error.
If I go back to 0.0.2 (and removed the SMXCrashlytics library) everything works fine.
I'm using CocoaPods and that's how Fabric and Crashlytics are installed

+1
Just installed today, having the same problem.

+1
Same here.

try to reinstall Fabric

Tried again on iOS, seems to work. I guess I did not complete the Crashlytics configuration correctly on the first time.
Thanks!

i had the same issue. i'm using cocoapods and fabric was installed correctly. here's how i got it to work:

in xcode project select SMXCrashlytics.xcodeproj under Libraries
go to Build Settings -> Search Paths -> Header Search Paths
add $(SRCROOT)/../../../ios/Pods/Headers/Public/Crashlytics as a non-recursive path

now open your xcworkspace and it should build. your relative path might be different from mine, and maybe there's a better way to do this. i'm still new to ios development.
@corymsmith thanks for the awesome module! any thoughts on making this change that will fix things for cocoapods users? not sure if my solution is general enough to merge. for now i'll have to switch to a fork or react-native-fabric so my team doesn't have to go through this process whenever they do a fresh npm install

what if you're not using cocoapods? ..or how would you use cocoapods with this?

oh my bad, i didn't even install crashlytics itself. ..probably should be in the docs for other newbs. ...gotten all the way to the activation step at this point. it hangs for me. probably need to put in some credentials or something.

...yea for anyone else out there expecting a more standard react-native iOS install process, the gotchas were:

install crashlytics separately (i recommend starting with that) by just going to their site and following their directions verbatim
then follow the react-native install directions in this repo (if you do this first and try to run it, you will run into additional problems)
then make sure you follow the final steps here to get your app delegate file triggering crashyltics:

https://fabric.io/kits/ios/crashlytics/install

@faceyspacey It's the second line in the README, if you have some thoughts on how we can make that more clear I'd gladly change the Readme or accept a PR that clarifies that since alot of the issues seem to be caused by Fabric not being installed first.

Just 1 thing, after struggling a bit, cocoapods pod file comes with the default setting for swift it has a line 'use_frameworks!`
Please comment this line for react-native as it is based of objC.

Reinstalled everything, downloaded the latest version of the app, followed all the instructions (not using pods).  Still getting 'Crashlytics/Crashlytics.h' file not found.  Any other ideas?
Using rn 0.29

My problem was that I was trying to put the frameworks in a frameworks directory on disk, looks like that's just not possible for some reason.

if you use framework must add this
$(SRCROOT)/../../../ios/Pods/Crashlytics/iOS
in xcode project select SMXCrashlytics.xcodeproj under Libraries
go to Build Settings -> Search Paths ->  Framework search Paths
and add it

Had the same problem (I just didn't install everything in the right order) so I made some changes to the readme for future users.  Here's the pull request https://github.com/corymsmith/react-native-fabric/pull/63

Well, I had similar problem (well, the error was in my AppDelegate.m not in SMXCrashlytics.m), so here's how I fixed it:
I'm using pods and know nothing about iOS. I randomly found out that pods creates a Pods.xcodeproj (inside ios/Pods folder). So I dragged it and added it to Libraries inside the Project Navigator (the first tab in the left pane of XCode) and the problem was solved.
(P.S: Sorry for being a newbie.)

Still getting this on a clean installation of RN

With latest master，i still have that error in swift project，through added it  Without CocoaPods.
1~ , add pod 'Fabric'   pod 'Crashlytics' in your podfile,
2~, drag SMXCrashlytics.xcodeproj to your project and choose " create folders referrences "
Then you can build pretty!


dragonberry

Description
Dragonberry. Non state-breaking.  Coordinate release.  Thanks.
Tasks

[ ]

References

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Appropriate labels applied
[ ] Targeted PR against correct branch
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards.
[ ] Wrote unit and integration
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added relevant godoc comments.
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Review Codecov Report in the comment section below once CI passes

@faddat Dragonberry security patch is applied in v2.2.0. Can you spare some time to take a look at the changes and see if you think there is more to change for the patch? For your information, this PR is based on main branch which is the latest branch that includes a few different modules and we plan to have them in the coming month or so.
Instead of bumping Cosmos SDK from v0.45.3 to v0.45.9, we decided to add our own ics23 package and use it by changing replace directive in go.mod.

To give you an update, this version v2.3.0 is released, which uses Cosmos SDK v0.45.9 and dragonberry security patch is applied.