Make AWS::Logs::Destination.DestinationPolicy take a Token[String]

http://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-logs-destination.html#cfn-logs-destination-destinationpolicy

@jfklingler any reason the constructor was private in the first place?
Also seanmcl, I'm assuming you are changing this to a token since you want to pass in the policy as a parameter?

Hi @tj-corrigan,
http://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-logs-destination.html#cfn-logs-destination-destinationpolicy
My understanding is that the policy is not a reference to an IAM policy (which can only be created by CF by attaching it to a user, group, or role) but the actual content, as a String, of the policy.  The Token part is so I can use functions like Fn::Sub in there.


Add someone to GitHub

Please include:

GitHub ID to add: davidkhala
Project to be added to: fabric-docs-i18n

One of:

Team to be added to:
Role:

Manage

Note: to merge ZH pull request

@ryjones So Ry, which group I am a member of now to grant me approve right?


feat(oxc_transformer): share object define reference for ReplaceGlobalDefines

Closed https://github.com/oxc-project/oxc/issues/7641

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#7698  👈 (View in Graphite)
#7621
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

I'm not very familiar with define plugin. Could someone else review please?

https://github.com/oxc-project/oxc/issues/7641#issuecomment-2525431555


Cliente should be in Caps (Cliente, instead of cliente)

Text formatting issue: should state Cliente

Testado iPhone Xs - Chrome e Safari
Windows - Chrome


typo? "wyoming-wakeword.service"  is "open" missing?

https://github.com/rhasspy/wyoming-satellite/blob/3a91fc468a0d97bad12ecc79f4e4ca2e89cf8839/installer/services.py#L91
your old "tutorial_2mic.md" had us create this file:
sudo systemctl edit --force --full wyoming-openwakeword.service
...
and then Check logs with:
sudo systemctl status wyoming-satellite.service wyoming-openwakeword.service
has this changed?

Since there's only one wakeword system expected to run at a time, I decided to make it a single service name for the installer.


Update API throttling section in reference topic as per release of Adv auditing for M365

Docs Build status updates of commit 9a74011:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

office-365-management-api/office-365-management-activity-api-reference.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Compile time config and graceful exits of unit tests

This PR implements the following:

The feature that is requested in #24 by adding compile time features yubikey and file in Cargo.toml.

Please note that the binary and PAM module have different Cargo.toml and the features can be turned on and off in each independently

It now exits gracefully instead of causing panic, if root permission is not granted or ZFS tools is not installed for unit tests that depend on them.
It up-ticked a minor revision to 0.5.1

LGTM but still needs more testing.


Add color classes

[x] Changes have been thoroughly tested to not break existing functionality.
[ ] New functionality has been documented or existing documentation has been updated to reflect changes.
[ ] Visual changes are explained in the PR description using a screenshot/recording of before and after.

Adds CSS classes to target elements based on their color (gray or custom).
Adds CSS class hooks deprecation comments (https://github.com/filamentphp/filament/issues/8064).

Hm, I think we should remove the deprecated comments to reduce the payload size. Maybe the active sidebar item is ok, but definitely not the button. And we should keep it consistent and not do it anywhere. Maybe a Blade comment is ok though.

Yeah I just added this as a concept, since you mentioned on that issue.
But I'll just move to Blade comments.

Since this breaks the code formatter, we're just going to need to remove those comments for now if you want this merged in.

Worked around it by "escaping" the @.
Have created an issue in the formatter repo: https://github.com/Stillat/blade-parser-typescript/issues/79.

Ready for merge now, @danharrin.


gi\gimodule.c: fatal error C1083: Cannot open include file: 'msvc_recommended_pragmas.h': No such file or directory

Describe the chore that needs to be addressed.

[ ] This issue subsumes the following issues:
when i Run poetry install --no-root  ,message below:

gimodule.c
gi\gimodule.c: fatal error C1083: Cannot open include file: 'msvc_recommended_pragmas.h': No such file or directory
error: command 'C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\BIN\x86_amd64\cl.exe' failed with exit code 2
I am Win10 msys2 installed, and 'msvc_recommended_pragmas.h' in system. but error again.

That's a problem with your PyGobject installation.  Poetry can't install PyGobject on Windows, here are the instructions for installing PyGobject on Windows --> https://pygobject.readthedocs.io/en/latest/getting_started.html#windows-getting-started.


plugin: Upgrade to cap v3

@aacassandra, we're working on updating the install guide. Right now you can install the v3 branch directly via github repo
npm install git://github.com/capacitor-community/capacitor-googlemaps-native.git#v3 --save

hey @hemangsk, i installed this in v3 branch. how do i use this in capacitor v3. isn't the function const {CapacitorGoogleMaps} = Plugins; "Plugins" from the capacitor/core is outdated. how do i get the CapacitorGoogleMaps in capacitor 3. can i use as below?:
import {CapacitorGoogleMaps} from '' capacitor-googlemaps-native "
like other capacitor v3 plugins. new way almost using like this and not using " const {} = Plugins"

Looking into it, import {CapacitorGoogleMaps} from ''@capacitor-community/capacitor-googlemaps-native " should work but it isn't working for some reason.

@aacassandra This works now,
Install from master (it has v3 support merged now),
npm i -S git+https://github.com/capacitor-community/capacitor-googlemaps-native

and
import { CapacitorGoogleMaps } from '@capacitor-community/capacitor-googlemaps-native';

wow that looks great, ok i will try it later and will update my try soon

@aacassandra I'm not familiar with vue,  but see this example if it helps,
If we do something like,
CapacitorGoogleMaps.create({
      width:100,
      height: 300,
      x: 10,
      y: 20,
      latitude: -33.86,
      longitude: 151.20,
      zoom: 12
    })

Then the plugin will draw a map on top left of the screen (and the precise coordinates will be (10,20)). The map's dimensions would be 100x300. Done, and then you don't have to really care about boundingRect stuff in the angular example.

what is the function of this element <div id="map" #map> on angular?
does CapacitorGoogleMaps.create need to be attached to that element to bring up the map? or by using CapacitorGoogleMaps.create it automatically calls the native pagemap?

CapacitorGoogleMaps.create just needs the

starting x,y coordinate of the map view, and
height and width of the map view.

  is used to draw and HTML element on DOM, and then DOMRect() thing grabs the x,y coordinate, height and width of this div.


How to merge array items only if they are in the target?

I want to modify the combineMerge from the ReadMe so that array items only get merged if the inde exists in the target. Example:
x = {
    array: [{
        arrprop_x: 'base_1',
        arrprop_y: 'base_1'
    }, {
        arrprop_x: 'base_2',
        arrprop_y: 'base_2'
    }],
}

y ={
    array: [{
        arrprop_y: 'special',
        arrprop_z: 'special'
    }}
}

Wanted result from deepmerge(x,y, { arrayMerge: combineMerge }):
{
    "array": [
        {
            "arrprop_x": "base_1",
            "arrprop_y": "special",
            "arrprop_z": "special"
        }]
}

(Because only index 0 exists in the target!)
I cannot get my modified combineMerge working for this: I tried to skip the deepmerge in the if (options.isMergeableObject(item)) if-branch, but the items from the source are already copied!
How can I do this?

The solution is to write an arrayMerge function that does what you want it to.
From your description it sounds like you want to change the first line in combineMerge from const destination = target.slice() to const destination = target.slice(0, source.length)
But I would suggest that you start by writing unit tests that assert the behavior that you want.

It works, it works! const destination = target.slice(0, source.length) does the trick. You know your stuff! :-)
The thing I don't understand is why should I limited the destination by the source (via source.length), I thought I want to limit the destination by the target - or are source and target swapped around? If so, why?
Anyway super thanks for your pointer! :-)

You said

array items only get merged if the inde exists in the target

which sounds like "items in the source don't get merged if their index doesn't exist in the target array" :-x

Actually I realised that source and target are in deed swapped to what I had assumed... Makes all sense now.
BTW, it is really cool to have arrayMerge and customMerge hooks: Together they allow for a lot of customisation!


bug: Ajax calls do not work with device or simulator

Type: bug
Ionic Version: 2.x
Platform: mobile browser
I was previously able to make Ajax calls with no issues.
Now they only work when I do
ionic emulate ios -c -l
when I do
ionic emulate ios / android
The Ajax calls do not work. Please tell me what has changed.
The whole project is on github please see this link
https://github.com/alyn000r/testAjax/tree/master/testAjax

Hello! Thanks for opening an issue with us! Are you getting any errors in the console?

I am sorry, I do not know how to check the console logs when I run this command
ionic emulate android
When I run this command
ionic emulate android -c --- No Errors are shown
When I run this command
ionic emulate android -c -l --- Ajax calls work fine

Sorry, i should have been more clear. So, when you run ionic serve does your app work in the browser? Or does it have issues there too?

It works perfectly fine in the browser and when I use live reload.

Device and Simulator without live reload is a no go

Thanks for the info! So to inspect your app when running on your android device you can use chrome://inspect/#devices to inspect your app and see if there are any errors in the console. Im guessing that you may get errors related to cors.

I will try this and get back to you. Thank You

I was reading this blog which says CORS is only an issue when you start up a web server and it should not be an issue when we do run / emulate
However in my case the Ajax works fine when I use live reload or serve
Anything I am missing ?
http://blog.ionic.io/handling-cors-issues-in-ionic/

@alyn000r Hmmm, sorry this is a tough one. I am fixing to do some more testing and then i will get back to you. Thanks for your patience and quick responses!

I saw the error first it was complaining about no <meta "Content-Security"> tag present so I added this to my index.html for android

but then it fires the request as
file:// ....
which fails because the call is supposed to be https://
I did setup a proxy in my ionic.config.json as
{
"name": "testAjax",
"app_id": "",
"v2": true,
"proxies": [{
"path": "/rafitows",
"proxyUrl": "https://rafitoapp.com/rafitows/"
}]
}
What am I missing ? Thanks

@alyn000r Hello! Are you still having this issue with the latest version of ionic?

I am on the same version the issue that I am having is that it keeps the file:// and ignores the proxy which I setup in my ionic project file. Am I missing something or this proxy thing is fixed in the new version ? Thank You

Hello @alyn000r ! So after lots of testing with different versions of Android i cant seem to reproduce this issue at all. My guess is that something is wrong with the web server you are sending requests too. I am very sorry that i do not have a better answer ! I would suggest asking around on our forum (forum.ionicframework.com) or StackOverflow and seeing if anyone has the same issue. If so, please feel free to comment here and i will reopen this issue. Thanks  for using Ionic!


Cannot fetch dictionary by https from a server with valid ECDSA certificate

It looks like the http client built into clickhouse-server version 1.1.54231 doesn't support ECDSA certificates. It emits the following error in the clickhouse log when trying to fetch a dictionary by https from the server with ECDSA certificate:
ExternalDictionaries: Failed reloading 'foobar' dictionary: Poco::Exception. Code: 1000, e.code()
 = 0, e.displayText() = SSL Exception: error:14077438:SSL routines:SSL23_GET_SERVER_HELLO:tlsv1 alert internal error, e.what() = SSL Exception

Additionally, the https server with the ECDSA certificate hosts multiple domains on the same IP (aka SNI) and refuses clients without SNI support. The issue described above could be related to this fact.
The dictionary URL is properly fetched by curl, wget and all the modern web-browsers.

It looks like the issue has been fixed in the latest stable release 1.1.54304. The previous stable release 1.1.54292 doesn't understand ECDSA certificates.


Create the exercise repo not under coderefinery namespace

Otherwise we will get 100 pull requests, also in-between workshops, which may mask other pull requests.

I will change that after the in-person workshop this week.


keycloak_quarkus: Permissions issue on controller-side install package

SUMMARY
When installing onto a remote node, the locally-created ZIP file appears to be created as root.
It's unclear why this is the case - Ansible is not running as root on the controller node.
ISSUE TYPE

Bug Report

ANSIBLE VERSION

ansible [core 2.15.5]
  config file = None
  configured module search path = ['/root/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/local/lib/python3.10/dist-packages/ansible
  ansible collection location = /root/.ansible/collections:/usr/share/ansible/collections
  executable location = /usr/local/bin/ansible
  python version = 3.10.12 (main, Jun 11 2023, 05:26:28) [GCC 11.4.0] (/usr/bin/python3)
  jinja version = 3.0.3
  libyaml = True

COLLECTION VERSION

# /home/ubuntu/.ansible/collections/ansible_collections
Collection                     Version
------------------------------ -------
community.general              7.5.0
freeipa.ansible_freeipa        1.11.1
middleware_automation.common   1.1.4
middleware_automation.keycloak 1.3.0

STEPS TO REPRODUCE

- name: Provision Keycloak
  hosts: keycloak
  # We gather facts after our VM comes up
  gather_facts: false
  tasks:
    - name: Wait for SSH connectivity
      ansible.builtin.wait_for_connection:
    - name: Gathering Facts
      ansible.builtin.setup:
    - name: Install QEMU Guest Agent
      become: true
      ansible.builtin.yum:
        name:
          - qemu-guest-agent
    - name: Install Keycloak
      vars:
        keycloak_quarkus_admin_pass: "{{ lookup('ansible.builtin.password', hostvars.localhost.keycloak_dir.path + '/keycloak_admin_password') }}"
      ansible.builtin.include_role:
        name: middleware_automation.keycloak.keycloak_quarkus

EXPECTED RESULTS

I expected the installation to continue.
ACTUAL RESULTS

The installation fails, with a permissions issue on the locally-copied ZIP file.
The file itself is owned by root:root.

TASK [middleware_automation.keycloak.keycloak_quarkus : Copy archive to target nodes] ***
fatal: [keycloak]: FAILED! => {"msg": "an error occurred while trying to read the file '/home/ubuntu/keycloak-22.0.3.zip': [Errno 13] Permission denied: b'/home/ubuntu/keycloak-22.0.3.zip'. [Errno 13] Permission denied: b'/home/ubuntu/keycloak-22.0.3.zip'"}

Hello and thanks for reporting; I tried to reproduce the problem several times, but the download was always owned by the user running the playbook on the controller host. Did you try to delete the file and re-run? Can you provide the output of the playbook up the error while passing -vvv to ansible-playbook?


UI - Charts

[x] Use one legend if possible
[ ] Add 10px padding between table, download button and charts
[ ] Change background hover color
[x] Remove borders from legend color squares
[x] Fix bar widths, make total chart size dynamic to adjust to this (use Insurance category as benchmark for bar widths)
[x] Make y-axis labels go to new line if too long (fix y-axis width)
[x] Make font style more consistent with rest of the app → Roboto
[x] Consider abbreviating numbers in the x-axis of Population by Access chart with “K” for thousands or “M” for millions

Not changing background hover color after all


Can't use Filament components on 404 pages

Package
filament/filament
Package Version
v3.2.116
Laravel Version
v11.27.1
Livewire Version
v3.5.6
PHP Version
v8.3.11
Problem description
My error page template is customised so that it's not the standard Laravel 404 page. It's set to use my main layouts/app.blade.php and the page renders using this layout.
This layout includes @filamentStyles and @filamentScripts with the intention being to make use of some Filament components on the page, but when I hit a non-existent page, @filamentScripts fails to load any scripts needed for Filament to operate - they also 404.
If I swap it to use @livewireScripts, everything works.
So it looks like the @filamentScripts tag is preventing loading of these assets on a 404 somehow.
Expected behavior
I expect Livewire, Alpine and Filament scripts to be available to my page whether it's 404 or not.
Steps to reproduce
Create a errors/404.blade.php file that includes @filamentScripts, then visit a URL that isn't handled by a route in your application. See that the necessary <script> tags rendered but result in a 404 header and aren't loaded.

Swap @filamentScripts for @livewireScripts and note that the behaviour is different.

Reproduction repository (issue will be closed if this is not valid)
https://github.com/simonhamp/filament-scripts
Relevant log output
No response

Donate 💰 to fund this issue

You can donate funding to this issue. We receive the money once the issue is completed & confirmed by you.
100% of the funding will be distributed between the Filament core team to run all aspects of the project.
Thank you in advance for helping us make maintenance sustainable!

Run php artisan filament:assets manually.
Doing this in your repo app loads them correctly on the 404.

@awcodes i haven't tried this yet, but can you enlighten me as to why this works?
Feels really strange to me to have separate behaviour.

@awcodes ok, so this fixed the apparent loading errors, but for some reason the actual behaviour is still different - I'll work up an example for my repo to try to express this, but basically Alpine still isn't available for some reason on the 404s

I'm having a similar problem with Alpine. I've noticed that if the current page doesn't contain any Livewire components then Alpine is not available. My scripts fail.
To workaround it I have to create a placeholder Livewire component. I then add that directly to my layout and Alpine is loaded correctly.
So it seems like Filament decides not to load it's scripts when there are no Livewire components on the current page. I think it should always load the scripts because Alpine may be needed on pages that don't have Livewire components.
layout.app.blade

// Just an empty livewire component to force Alpine to load.
<livewire:placeholder>
/body>


[highcharts] report keeps crashing browser

For any serious testing scenarios with a larger number of different tests, the resulting report crashes my browser and is not usable to gain any insight.

Could you share a reproducer, please? How many different request types do you have?

About 25,000 different request types.

Definitively not the intended usage.
Closing as "won't fix".

What is the intended usage then?
Once I know, I could update the documentation.

Get a look at our HTML reports, you'll realize that we generate a table and a menu which both contain one line per request type. It's very likely that your browser is not fond of such huge tables.
Then, I'm not sure why you want to deal with such a humongous number of request types.

I'm aware of how the reports are being generated.
The reason why I want to deal with such a large number of request types is the ability to analyze requests more specifically.
While it's surely overall acceptable to have a general idea on the performance on requests to http://server/content/:contentId/detail, it becomes apparent when response times for requesting detailed information for pieces of content have a significant range. It becomes important to know that the content with the content id abc comes back in between 50 to 70 ms, but the request with the content id bcd in the path never ever returns in under 2 s, and will even take up to 30 s. That is important information. We can not simply put everything in the "http://server/content/:contentId/detail" request type, but need to differentiate.
"http://server/content/_:contentId_/detail"
becomes

"http://server/content/abc/detail"
"http://server/content/bcd/detail"

I totally understand if this does not make sense to you, and are unwilling to put any effort in improving highcharts regarding these points. We'll build our own in-house solution if needed. Luckily parsing the simulation.log is simple enough and gives us all the information we need in order to format the data more suitable for a small front end application.

Yes, I think we won't invest on this for Gatling standard reports.
However, we're trying to support this in FrontLine.


DCLS: deserialize fails if gtadir already exists

The function tadir_insert do not check if the object exists in gtadir and if it exists, the call dies with an error.
Unfortunately, the method cl_dd_utilities=>write_tadir do not accept DCLS but we can copy the approach.
If gtadir exists and we skip tadir_inser in DCLS~deserialize, the corresponding tadir entry is created during activation.

I think GTADIR is used only inside SAP, so its not easy to reproduce in normal systems
pull requests welcome

If SAP wants this fixed, please create a PR since no one else is using GTADIR feature. Closing.


Suggestion: Offer the not implemented methods as aliases?

I was looking at the few methods that are not implemented, but which are in Laravel, and was wondering if it might be a good idea to implemented them as aliases/wrappers that just call the intended method? This would keep users from having to worry about differences to Laravel, since overall it is expected to be the same (noting that this collection manager is always strict).

One issue with that is that implies the base method in each case is not strict to users who have not noticed the readme disclaimer. You could implement the aliases as prototype methods for your own case.
I think it might make sense to include "notStrict" versions of each case, though in JS this can be sticky.

Agreed. As the impetus of this package is to allow Devi to do the same things in JS as in Laravel, I thought it would be really important for everything to match in order to reduce mental debt. I was primarily looking at the toArray() method, since that is used a lot for semantic syntax.

I'm open to add toArray() as an alias for all(). I havn't added it because I thought people might be confused. What do you think?

In Laravel the toArray() and all() methods are synonymous as well, right? Or is the difference that toArray() returns an array of values, while all() returns a key-value array? I never really looked at the differences and have always been implementing toArray to make my code be more expressive (the word all doesn't necessarily imply an array).

I think the toArray() needs to be an alias for values() or all().
values() makes sense if your collection is an object, like this
const collection = collect({ foo: 'bar', baz: 'qux' });

What about:
If the collection is an array, we simply return all items (like all()).
If the collection is an object, we return the values of that object (like values()).

By the way, with the comparison methods you can make them not strict by forcing use of a not strict operator in the second argument.

as long as the toarray() returns an array of values keep this array values implemented which doesn't complicate it with all() values of this object, if it's implemented in both with same value then you'll get an error message

Added toArray().
https://github.com/ecrmnn/collect.js/blob/master/src/index.js#L895

@ecrmnn If you are interested, I can take on the task of adding the appropriate helpers.

Probably should still add those methods, but just add a console.error to inform developers of the intended use.

Sure @mikebronner, make an implementation and I'll have a look. Right now I can't really decide how I'd like it to work.

You probably meant @mikeerickson ? :)

Hah, yes I meant @mikeerickson 😅

@ecrmnn I will get a POC created and we can discuss.


Prevent sending multiple Set-Cookie headers

By default when using the cookie session handler (encrypted or unencrypted), CI sends the entire "Set-Cookie" header each time a new value is written to the session. This results in multiple headers being sent to the client.
This is a problem because if too many values are written to the session, the HTTP headers can grow quite large, and some web servers will reject the response. (see http://wiki.nginx.org/HttpProxyModule#proxy_buffer_size)
The solution is to only run 'sess_save()' one time right after all other headers are sent before outputting the page contents.
Signed-off-by: Aaron Parecki<EMAIL_ADDRESS>

Superseded by #3073.


@warning_ignore(unused_parameter) annotation doesn't remove the warning.

Godot version
4.0.alpha4
System information
Windows 11 Vulkan
Issue description
Writing @warning_ignore(unused_parameter) above/around a function with at least one unused parameter doesn't remove the warning from the list of warnings. Like with other warnings, I'd expect it to disappear, and without having to put an underscore before the parameter's name.
Steps to reproduce
Write a function with at least one unused parameter, and try placing the @warning_ignore(unused_parameter) annotation above it, or even before the func keyword.
Minimal reproduction project
No response

Sincerest apologies, wrote up this report without checking first a similar issue had already been opened.


AOI creation fails with empty gpkg

A valid gpkg containing no features raises error during creation of an AOI instance:

To reproduce:
python GDL.py mode=verify dataset.raw_data_csv="your/dir/data_with_empty_gpkg.csv"
dataset.bands=[1]
debug=False

Files (raster uses pre-existing raster in GDL's test data as dummy):
input data csv: data_with_empty_gpkg.csv
ground truth gpkg: buil_AB11-WV02-20100926-1.zip

#386 may fix this


Helps needed to package correctly Thoth.Json.Net

Disclaimer: I don't know much about how nuspec and nuget package works. In general, they just work for me so I am fine. But with Thoth.Json.Net I have a more complex package  and I am a bit lost.
@forki this issue is about my call for help on Twitter. I tried to sum up the different problem I have and what actions have been taken when so you can have a global vision of the situation. Sorry for the long issue but I tried to be as explicit as possible.
Problem n°1: The dependencies list doesn't reflect the difference depending on the targetted framework.
Since several version Thoth.Json.Net have the following depencies list:
Version 3.4.0

Instead of something like that:
Version 3.0.0-beta-001

This problem appeared when I tried to lower Newtonsoft.Json dependency to >= 11.0.2 instead of 12.0.1.
At that time, I switched from dotnet pack to paket pack using paket.template in project mode so it had the correct dependency for Newtonsoft.Json based on paket.dependencies.
In the past, @forki mentionned that if we didn't need different transitive deps in different framework it was "ok". Comment
However now, I need different deps depending on the Framework.

See how .NETStandard 2.0 require Fable.Core now while .NETFramework 4.6 doesn't use it because Fable.Core is not available here and was generating exceptions.
In order to archieved this result, I use paket.template in file mode. Source
Problem n°2: It seems that even if we have Newtonsoft.Json >= 11.0.2 the dll was using/required 12.0.0.
@bentayloruk raised an issue about that https://github.com/thoth-org/Thoth.Json.Net/issues/18 and because I didn't see any problem with releasing a new version solving his problem we added lowest_matching:true to paket.dependencies
Problem n°3: The AssemblyInfo of version 3.5.0-beta-001 seems to say that the AssemblyVersion is 1.0.0 which seems wrong.

I'm having some issues, but not sure why they are happening yet.  Will investigate.

Side question, 3.5.0 beta 1 is <IP_ADDRESS> in all of the assembly version attributes.  Is that intentional?

Originally posted by @bentayloruk in https://github.com/thoth-org/Thoth.Json.Net/pull/19#issuecomment-500251013
I suppose this is related to the changes I made when using paket.template in file mode. When doing so because the file mode of paket was asking the same info that I already had in my fsproj I removed them from that last file.
Diff of the changes made
My question is should we keep the info in both file? Should I use FAKE to generate the AssymblyInfo file and how can I use that generated file?
Problem n°4: Should we set an upper version restriction on the depencies?

As you can see here Newtonsoft.Json is marked as >= 11.0.2 so people can use version 12+ of it.
While Fable.Core is marked as (>= 3.0.0 && < 4.0.0). What is the correct way to restrict dependencies?

/cc @matthid

Friendly ping @forki :)

While Fable.Core is marked as (>= 3.0.0 && < 4.0.0). What is the correct way to restrict dependencies?

I think given that Fable.Core changes were rather breaking in the past I think this is fair. The only downside is that you need to publish an update here as soon as Fable.Core goes 4.0 (probably even when it releases alpha)
Regarding https://github.com/thoth-org/Thoth.Json.Net/commit/3c18568cc9f356e9fc5907032389c549574e0138#diff-ff6fc11c05b56740bf3cbea14ceab332  - yes this is a bit unfortunate but type file does not use the project settings. You need to be very explicit.

I think given that Fable.Core changes were rather breaking in the past I think this is fair. The only downside is that you need to publish an update here as soon as Fable.Core goes 4.0 (probably even when it releases alpha)

I am ok with that, this is already what I had to do in the past in order to support the new API

Regarding 3c18568#diff-ff6fc11c05b56740bf3cbea14ceab332 - yes this is a bit unfortunate but type file does not use the project settings. You need to be very explicit.

I don't have a problem with moving the info from the fsproj to paket.template. My concerns is more about the fact that the AssemblyInfo seems wrong.
Do I have to put the information in both place? Do I have to generate the AssemblyInfo myself and if yes I do I package the nuget using this generated file?

My concerns is more about the fact that the AssemblyInfo seems wrong.

yeah that's even more unfortunate. The compiler uses he AssemblInfo and/or the fsproj settings. It does not know about paket template files. So you clearly need to change fsproj and/or AssemblyInfo.
And then the type file needs it as well since it does not make any connection to projects. It's basically a completely independent approach that you can use to package stuff that may not even come from any fsproj or csproj

Ok thank you so I will:

Keep using paket.template
Generate the AssemblyInfo file
Keep Fable.Core restriction
Open FSharp.Core to be something like >= 4.6.2

Sorry for not responding sooner but there are a lot of mixed questions in that which takes a while to respond properly.
In Short:

AssemblyVersion has nothing to do with package version
In .NET SDK they use the same field (Version) in msbuild for AssemblyVersion and PackageVersion
Paket doesn't really support different versions based on framework at this time
There is an open issue in the compiler not setting FileVersion (explorer hover/details page), this is not technically required anywhere so the impact is low (but can lead to confustion). Always check if reflection tools like dotpeek

My recommendation:

Do not use paket pack, use dotnet pack and set the msbuild properties as recommended. In fact you can always use environment variables for example from the FAKE script (for example set Version environment variable and msbuild will pick it up)
Do not use AssemblyInfo any longer. Set the corresponding msbuild variables as needed (if you need company and description information)

This is very short, feel free to further refine questions regarding that ;)

Sorry for not responding sooner but there are a lot of mixed questions in that which takes a while to respond properly.

No problem, I know that there is a lot but I am discovering all these problems ^^

Do not use paket pack, use dotnet pack and set the msbuild properties as recommended. In fact you can always use environment variables for example from the FAKE script (for example set Version environment variable and msbuild will pick it up)

I  made the switch to paket pack because dotnet pack wasn't picking the correct version of Newtonsoft.Json. It was picking up >= 12.0.1 instead of >= 11.0.2.
In theory, if you know how to fix this problem when using dotnet pack it should be ok.
And also, if dotnet pack support different deps depending on the framework.

Also, is it possible to mark Fable.Core with (>= 3.0.0 && < 4.0.0) version when using dotnet pack? If yes, how ?

Yes dotnet pack supports all of this, but the combination with paket complicates some of these. the easiest is to overwrite paket by adding dependencies directly into the project file <PackageReference Condition="'$(TargetFramework)' == 'net46'" Name="Newtonsoft.Json" Version="11.0.2" />
This should in theory overwrite paket in this scenario. For upper bounds of dependencies you need to consult the docs as I don't recall the syntax. I think it's similar to Version="[3.0,4.0[".
Note that you still use paket but overwrite it in some occasions.
Warning: I wrote this on mobile and without google so there might be syntax problems or fields called differently

Hum ok thank you both will try to make something from all this discussion and see how it goes :)

@MangelMaxime I will test the package again when you think you're ready.  I don't think we can rely on the results of my last test, given that the AssemblyVersion is incorrect.  I must admit to not being aware of all the nuances around assembly resolution in .NET Core and a lot of the information in this issue, so I am learning a lot too!

@forki @matthid Thank you both
I think I manage to fix all the issues I had.
In the end, here are the action I took:

I remove paket.template
Use dotnet pack
Override Fable.Core dependency

Use <PackageReference Condition="'$(TargetFramework)' == 'netstandard2.0'" Include="Fable.Core" Version="[3.0,4.0)" /> in the *fsproj
Use nuget Fable.Core ~> 3 framework: netstandard2.0 in paket.dependencies

I now have the proper deps displayed on nuget:

I also tested the package againt Bookstore and it was working so hopefully it's ok for everyone now.
@bentayloruk You should be able to check again the package againt your problem if you need. It should still be compiled againt the lower version of Newtonsoft because I kept it in paket.dependencies.

@MangelMaxime this is great and I will try it out at my end as soon as I can.  It wont be until next week now as I'm off this week, but it all sounds good to me.  Based on my understanding of .NET Core assembly binding, compiling against the lower version will offer the most flexibility to consumers in .NET Core because it should work with compatible newer versions of Newtonsoft.Json, whereas it would not support working with older versions as it was before.


Feat/cymk

Lymui
Totoro is going to review your PR

What does this PR accomplish

[x] Add a new feature supporting a new color type
[x] Provide a fix to the current project
[x] Add new test

We will check that

[x] Make sure to have unit test
[ ] Make sure that any test is passing
[ ] Make sure to respect that when converting a type T to return a type T Rgb

🐷 Thanks for contributing ! 🐷

Pull Request Test Coverage Report for Build 57

58 of 68 (85.29%) changed or added relevant lines in 6 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-3.02%) to 93.281%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lymui/helper.c
8
9
88.89%

lymui/main.c
4
5
80.0%

lymui/hex.c
1
9
11.11%

Files with Coverage Reduction
New Missed Lines
%

lymui/hex.c
1
66.67%

Totals

Change from base Build 54:
-3.02%

Covered Lines:
236

Relevant Lines:
253

💛  - Coveralls


as_nonblock should be into_nonblock

I think the convention was to only give the name as to functions that do absolutely nothing in terms of the generated code (after inlining), so the function should probably be named into_nonblock.

You are correct that as_* is probably not the right name. I'm thinking of just ::non_block() or ::non_blocking(). Thoughts?

Although that would be possible too, I'd prefer the into prefix because that makes it clear that it consumes self.


Find frequency of a character in a .txt file

Find frequency of a characters and words in a .txt file
Desc
frequency of user input character
frequency of words
frequency of all characters using switch case
Language
C++
Issue
Create a rasterized program
Proper comments
@amritansh22 Can I do this?

@Tushar11Raj Sure. I have assigned the issue to you.

I would like to contribute to it as a part of Hacktoberfest.

@Tushar11Raj Sure

assign me please

@Pratap2018 This issue is already assigned. You may look for other issues or open your own here.


use use_inline_resources only if defined

use_inline_resources doesn't work in all versions of Chef
NameError: undefined local variable or method `use_inline_resources'

The minimum required version of Chef for the chef-cookbooks is now Chef 11 so we're not going to put any backwards compatibility in for Chef 11 at this point. You can continue to pin to the older release of this cookbook that doesn't include the use_inline_resources attribute. At this point though I'd highly recommend upgrading to Chef 12, which includes several years of improvements and bug fixes.


Refactoring near by link appraoch related code and example notebook

This PR should refactor the nearby link approach.
Todos:

[x] remove function instantaneous_to_minmax_data()
[x] move the calculation of max(pmin), deltaP and deltaPL to nearby_wetdry()
[x] update nearby_determine_reference_level() and nearby_correct_recieved_signals() to make the usage of pmax more intuitive
[x] update of the example notebook and its descriptions to match the changes in the functions above
[x] update test_nearby_wetdry()
[x] update test_nearby_rainfall()

Thanks for the review comments! Ready to be merged from my side.

Thanks a lot again 👏 👏 👏 .
Really good to now have this updated implementation of the nearby-link approach from RAINLINK in Python! 🎉


[FE] Auto-login feature based on remember-me flag

@anzbez Unknowingly suggested auto-login feature which would work based on remember-me checkbox on Login-Screen. If user has remember-me enabled, app could auto-login him based on saved credentials (preferably encrypted in local storage).
We already automatically log-in user if his tokens are valid (not expired). In best case scenario, if user would use Laguna regularly, refresh token would never expire (every time access token expires, we get new access and refresh token). In worst case scenario, user would be logged out every time.
This is how most of web apps probably behaves. We could go beyond tokens, and keep user logged-in indefinitely. Exceptions would be if user manually logged out (we would remove tokens from storage), or if user haven't been logged in before (again, no tokens in storage).
Not sure if this is good practice or good user experience. What are your thoughts?

auto-login**_ feature which would work based on remember-me checkbox on Login-Screen.

I think we don't actually need "remember me".
I think that should be the default behaviour instead of "blocking" behaviour we have ATM.

We already automatically log-in user if his tokens are valid (not expired).

On BE we have auto-refresh of tokens, so technically logging-in w/ expired x-access-token and/or x-refresh-token will result in new token pair being sent back to FE via headers.
This means that we dont actually need to ever "programmatically" logout user on FE (by checking exp field on client). New pairs are generated on-demand when expiry is due on BE. All FE really needs to do it overwrite existing pair with new pair in encryped local storage.

Exceptions would be if user manually logged out (we would remove tokens from storage), or if user haven't been logged in before (again, no tokens in storage).

Yup. This however keeps the annoying nullability on local storage service logic.

Not sure if this is good practice or good user experience. What are your thoughts?

I think its kinda common with JWTs. We don't have a concept of "backend session", we just pass tokens back and fourth without explicit invalidation (only on exp expiry field check) (which is annoying to do w/ JWTs, but the tradeoff being no DB needed). Unlike session-based-auth which has invalidation.


UI view for import

Part of https://github.com/treeverse/lakeFS/issues/3137
Implement a new UI view described in #3137

Done.


New Java File Got Added In The Project

Hi Team,
Please review this code have added few changes related to server

This is done we can close the PR


Crash when  archive contains a binary file

The library is behaving correctly when you try to read an archive containing any binary file like an image, see the error below:
rak -z live /tmp/test.7z                               09:56:34
A worker in a parallel iteration (hyper or race) initiated here:
  in sub show-results at /Users/phil/.rakubrew/versions/moar-2022.07/share/perl6/site/sources/1636BBF2572059F76159DF50C03D5DD52979E294 (App::Rak) line 806
  in sub rak-results at /Users/phil/.rakubrew/versions/moar-2022.07/share/perl6/site/sources/1636BBF2572059F76159DF50C03D5DD52979E294 (App::Rak) line 677
  in sub action-per-line at /Users/phil/.rakubrew/versions/moar-2022.07/share/perl6/site/sources/1636BBF2572059F76159DF50C03D5DD52979E294 (App::Rak) line 2891
  in sub main at /Users/phil/.rakubrew/versions/moar-2022.07/share/perl6/site/sources/1636BBF2572059F76159DF50C03D5DD52979E294 (App::Rak) line 423
  in block <unit> at /Users/phil/.rakubrew/versions/moar-2022.07/share/perl6/site/resources/A0BA54D3D8B6443A1DA0C68A01CA42A3FC242435 line 3
  in sub MAIN at /Users/phil/.rakubrew/versions/moar-2022.07/share/perl6/site/bin/rak line 3
  in block <unit> at /Users/phil/.rakubrew/versions/moar-2022.07/share/perl6/site/bin/rak line 1

Died at:
    Malformed UTF-8 near bytes 8a 53 98
      in sub show-results at /Users/phil/.rakubrew/versions/moar-2022.07/share/perl6/site/sources/1636BBF2572059F76159DF50C03D5DD52979E294 (App::Rak) line 806
      in sub rak-results at /Users/phil/.rakubrew/versions/moar-2022.07/share/perl6/site/sources/1636BBF2572059F76159DF50C03D5DD52979E294 (App::Rak) line 677
      in sub action-per-line at /Users/phil/.rakubrew/versions/moar-2022.07/share/perl6/site/sources/1636BBF2572059F76159DF50C03D5DD52979E294 (App::Rak) line 2891
      in sub main at /Users/phil/.rakubrew/versions/moar-2022.07/share/perl6/site/sources/1636BBF2572059F76159DF50C03D5DD52979E294 (App::Rak) line 423
      in block <unit> at /Users/phil/.rakubrew/versions/moar-2022.07/share/perl6/site/resources/A0BA54D3D8B6443A1DA0C68A01CA42A3FC242435 line 3
      in sub MAIN at /Users/phil/.rakubrew/versions/moar-2022.07/share/perl6/site/bin/rak line 3
      in block <unit> at /Users/phil/.rakubrew/versions/moar-2022.07/share/perl6/site/bin/rak line 1

-z means --auto-decompress in my config. The library is not testing the type of file but assume it is utf-8 encoded. Not sure how you can manage this as it would be good to be able to search in binary file as well with Rak.
Any idea ?

it seems to be related to IO::Path support of binary file:
[1] > my $f="/tmp/test.7z".IO;
"/tmp/test.7z".IO
[2] > my $proc := run <7z e -so>, $f, :out;
(my \Proc_3193317352336 = Proc.new(in => IO::Pipe, out => IO::Pipe.new(proc => Proc_3193317352336, path => IO::Path, chomp => Bool::True, nl-in => "\n", nl-out => "\n", encoding => "utf8"), err => IO::Pipe, exitcode => Nil, signal => Any, pid => 94418, command => ("7z", "e", "-so", IO::Path.new("/tmp/test.7z", :SPEC(IO::Spec::Unix), :CWD("/Users/phil/Dropbox/tools/DarkRecon/scan/lib/IO-Path-AutoDecompress")))))
[3] > $proc.out.lines()
Malformed UTF-8 near bytes 8a 53 98

[4] > $proc.out.lines(True,:latin1)
Malformed UTF-8 near bytes 8a 53 98

[5] > my $proc := run <7z e -so>, $f, :out;
(my \Proc_3193317352944 = Proc.new(in => IO::Pipe, out => IO::Pipe.new(proc => Proc_3193317352944, path => IO::Path, chomp => Bool::True, nl-in => "\n", nl-out => "\n", encoding => "utf8"), err => IO::Pipe, exitcode => Nil, signal => Any, pid => 94428, command => ("7z", "e", "-so", IO::Path.new("/tmp/test.7z", :SPEC(IO::Spec::Unix), :CWD("/Users/phil/Dropbox/tools/DarkRecon/scan/lib/IO-Path-AutoDecompress")))))
[6] > $proc.out.lines(True,:latin1)
(PK)
[7] > $proc.out.lines(True,:latin1).say
(ᦊS_rels/PK)
[7] > $proc.out.lines(True,:latin1).[5]
Malformed UTF-8 near bytes 8a 53 98
  in block <unit> at <unknown file> line 1


How to add icon in text field

my grid enable insert, and I would like to ask how to customize the field to add an icon (e.g., search icon) when I clicked will pop-up a dialog (jquery dialog) and paste back the selected item to the text input.

You can redefine the content of the field with itemTemplate function. Return the content of the cell from it, obviously it can be anything. Checkout fields configuration http://js-grid.com/docs/#fields


Hi @mahimakaushik-oss

Hi @mahimakaushik-oss
The error is related to Gradle, this is usually due to corrupt or incomplete Gradle setup
Go to the root directory and locate the hidden .gradle folder and delete it.
If you're on a mac, open terminal and execute
cd ~
rm -rf .gradle

and then run your app connected to Android emulator or device, don't cancel and make have a stable internet connection, this will download fresh Gradle
If the problem persists, Can you please provide your flutter doctor -v and flutter run --verbose logs, and a minimal complete reproducible code sample?
Thank you
Originally posted by @TahaTesser in https://github.com/flutter/flutter/issues/73852#issuecomment-759494547

Hello please can anyone help me out am new here .. i keep getting error messages while running my android studio via sdk emulator
Running Gradle task 'assembleDebug'...
Exception in thread "main" java.util.zip.ZipException: zip END header not found
at java.base/java.util.zip.ZipFile$Source.zerror(ZipFile.java:1567)
at java.base/java.util.zip.ZipFile$Source.findEND(ZipFile.java:1462)
at java.base/java.util.zip.ZipFile$Source.initCEN(ZipFile.java:1469)
at java.base/java.util.zip.ZipFile$Source.(ZipFile.java:1274)
at java.base/java.util.zip.ZipFile$Source.get(ZipFile.java:1237)
at java.base/java.util.zip.ZipFile$CleanableResource.(ZipFile.java:727)
at java.base/java.util.zip.ZipFile$CleanableResource.get(ZipFile.java:844)
at java.base/java.util.zip.ZipFile.(ZipFile.java:247)
at java.base/java.util.zip.ZipFile.(ZipFile.java:177)
at java.base/java.util.zip.ZipFile.(ZipFile.java:191)
at org.gradle.wrapper.Install.unzip(Install.java:214)
at org.gradle.wrapper.Install.access$600(Install.java:27)
at org.gradle.wrapper.Install$1.call(Install.java:74)
at org.gradle.wrapper.Install$1.call(Install.java:48)
at org.gradle.wrapper.ExclusiveFileAccessManager.access(ExclusiveFileAccessManager.java:65)
at org.gradle.wrapper.Install.createDist(Install.java:48)
at org.gradle.wrapper.WrapperExecutor.execute(WrapperExecutor.java:128)
at org.gradle.wrapper.GradleWrapperMain.main(GradleWrapperMain.java:61)
Exception: Gradle task assembleDebug failed with exit code 1

I tried deleting the gradle folder still not getting it

Hi @kofoshi
Looks like you're probably deleting Gradle folder in your project.  You need to go user directory on your system, find hidden .gradleand delete it then run your app without interruption or cancelation with a stable internet connection as this will download fresh Gradle.

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


SQLite fail to load TPCDS datasets: Rusqlite("database is locked")

Describe the bug
TPCDS datasets can't be fully loaded into a single SQLite file. The error is shown as below

To Reproduce
spicepod.yaml
version: v1beta1
kind: Spicepod
name: tpcds-sf0.01.db
datasets:
  - from: duckdb:call_center
    name: call_center
    params:
      duckdb_open: tpcds-sf0.1.db
    acceleration:
      enabled: true
      engine: sqlite
      mode: file
  - from: duckdb:catalog_page
    name: catalog_page
    params:
      duckdb_open: tpcds-sf0.1.db
    acceleration:
      enabled: true
      engine: sqlite
      mode: file
  - from: duckdb:catalog_returns
    name: catalog_returns
    params:
      duckdb_open: tpcds-sf0.1.db
    acceleration:
      enabled: true
      engine: sqlite
      mode: file
  - from: duckdb:catalog_sales
    name: catalog_sales
    params:
      duckdb_open: tpcds-sf0.1.db
    acceleration:
      enabled: true
      engine: sqlite
      mode: file
  - from: duckdb:customer
    name: customer
    params:
      duckdb_open: tpcds-sf0.1.db
    acceleration:
      enabled: true
      engine: sqlite
      mode: file
  - from: duckdb:customer_address
    name: customer_address
    params:
      duckdb_open: tpcds-sf0.1.db
    acceleration:
      enabled: true
      engine: sqlite
      mode: file
  - from: duckdb:customer_demographics
    name: customer_demographics
    params:
      duckdb_open: tpcds-sf0.1.db
    acceleration:
      enabled: true
      engine: sqlite
      mode: file
  - from: duckdb:date_dim
    name: date_dim
    params:
      duckdb_open: tpcds-sf0.1.db
    acceleration:
      enabled: true
      engine: sqlite
      mode: file
  - from: duckdb:household_demographics
    name: household_demographics
    params:
      duckdb_open: tpcds-sf0.1.db
    acceleration:
      enabled: true
      engine: sqlite
      mode: file
  - from: duckdb:income_band
    name: income_band
    params:
      duckdb_open: tpcds-sf0.1.db
    acceleration:
      enabled: true
      engine: sqlite
      mode: file
  - from: duckdb:inventory
    name: inventory
    params:
      duckdb_open: tpcds-sf0.1.db
    acceleration:
      enabled: true
      engine: sqlite
      mode: file
  - from: duckdb:item
    name: item
    params:
      duckdb_open: tpcds-sf0.1.db
    acceleration:
      enabled: true
      engine: sqlite
      mode: file
  - from: duckdb:promotion
    name: promotion
    params:
      duckdb_open: tpcds-sf0.1.db
    acceleration:
      enabled: true
      engine: sqlite
      mode: file
  - from: duckdb:reason
    name: reason
    params:
      duckdb_open: tpcds-sf0.1.db
    acceleration:
      enabled: true
      engine: sqlite
      mode: file
  - from: duckdb:ship_mode
    name: ship_mode
    params:
      duckdb_open: tpcds-sf0.1.db
    acceleration:
      enabled: true
      engine: sqlite
      mode: file
  - from: duckdb:store
    name: store
    params:
      duckdb_open: tpcds-sf0.1.db
    acceleration:
      enabled: true
      engine: sqlite
      mode: file
  - from: duckdb:store_returns
    name: store_returns
    params:
      duckdb_open: tpcds-sf0.1.db
    acceleration:
      enabled: true
      engine: sqlite
      mode: file
  - from: duckdb:store_sales
    name: store_sales
    params:
      duckdb_open: tpcds-sf0.1.db
    acceleration:
      enabled: true
      engine: sqlite
      mode: file
  - from: duckdb:time_dim
    name: time_dim
    params:
      duckdb_open: tpcds-sf0.1.db
    acceleration:
      enabled: true
      engine: sqlite
      mode: file
  - from: duckdb:warehouse
    name: warehouse
    params:
      duckdb_open: tpcds-sf0.1.db
    acceleration:
      enabled: true
      engine: sqlite
      mode: file
  - from: duckdb:web_page
    name: web_page
    params:
      duckdb_open: tpcds-sf0.1.db
    acceleration:
      enabled: true
      engine: sqlite
      mode: file
  - from: duckdb:web_returns
    name: web_returns
    params:
      duckdb_open: tpcds-sf0.1.db
    acceleration:
      enabled: true
      engine: sqlite
      mode: file
  - from: duckdb:web_sales
    name: web_sales
    params:
      duckdb_open: tpcds-sf0.1.db
    acceleration:
      enabled: true
      engine: sqlite
      mode: file
  - from: duckdb:web_site
    name: web_site
    params:
      duckdb_open: tpcds-sf0.1.db
    acceleration:
      enabled: true
      engine: sqlite
      mode: file

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Runtime Details
Spicepod
Add the relevant spicepod.yml section.
Output of the describe table
Add any relevant describe table output here.
Output of explain query
Add any relevant explain query output here.
E.g. explain select 1
+---------------+--------------------------------------+
| plan_type     | plan                                 |
+---------------+--------------------------------------+
| logical_plan  | Projection: Int64(1)                 |
|               |   EmptyRelation                      |
| physical_plan | ProjectionExec: expr=[1 as Int64(1)] |
|               |   PlaceholderRowExec                 |
|               |                                      |
+---------------+--------------------------------------+

spice, spiced, OS info

spice version: spice version
spiced version: spiced --version
OS info: uname -a

Have you tried this on the latest trunk branch?

[ ] Yes
[ ] No

If possible, run spiced with DEBUG log level
By setting the environment variable SPICED_LOG="task_history=DEBUG,spiced=DEBUG,runtime=DEBUG,secrets=DEBUG,data_components=DEBUG,cache=DEBUG,extensions=DEBUG,spice_cloud=DEBUG"
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Closed by: https://github.com/spiceai/spiceai/pull/2855


[ISSUE] DND 5E 3.0.0+ Compatibility (Deprecation Warnings)

Action Pack on DND 5E 3.0.0+ is giving multiple deprecation warnings on token actor selection. Module is working fine, otherwise.
Foundry Version: 11.315
System Version: DND 5E 3.0.1
Module Version: 11.2
Only module enabled is Action Pack
Steps to reproduce: enable action pack, click on actor token on the canvas. Console will show the following error:
Error: The `system.components` property has been deprecated in favor of a standardized `system.properties` property.
Deprecated since Version DnD5e 3.0
Backwards-compatible support will be removed in Version DnD5e 3.2
    at Object.logCompatibilityWarning (commons.js:2059:19)
    at get components [as components] (spell.mjs:171:19)
    at lookupProperty (handlebars.min.js:27:16153)
    at eval (eval at createFunctionContext (handlebars.min.js:29:1808), <anonymous>:20:263)
    at h (handlebars.min.js:27:17870)
    at c (handlebars.min.js:27:6899)
    at Array.<anonymous> (handlebars.min.js:27:7289)
    at Array.c (handlebars.min.js:27:19853)
    at eval (eval at createFunctionContext (handlebars.min.js:29:1808), <anonymous>:11:49)
    at h (handlebars.min.js:27:17870)

Module is working fine otherwise.

I added a couple pr's, it's just a few lines to fix


Bynder assets extension breaks the translation fallback system

There is currently an issue with the way the Bynder assets plugin is storing the values. The plugin already stores an empty array, even if no image was selected.
This breaks the fallback system, as Contentful thinks that the image is translated.
You can see the issue in the screenshot. When we now perform a graphql query for this entry, we receive an empty array instead of the default language's picture.

I created a pull request at https://github.com/contentful/extensions/pull/155

Closed by #155


Update README.md

Fixing Marc.

Thanks. It'll still be broken on the site and bower listing till I finish the next release, but at least it'll be fixed on the GH home page.

:heart:


perf(ui/collapse): add call

Codecov Report

Merging #369 (46cc2af) into dev (ba68dc9) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##              dev     #369   +/-   ##
=======================================
  Coverage   96.39%   96.39%
=======================================
  Files         209      209
  Lines        4907     4908    +1
  Branches     1047     1045    -2
=======================================
+ Hits         4730     4731    +1
  Misses        177      177

Impacted Files
Coverage Δ

packages/varlet-ui/src/collapse/Collapse.vue
98.43% <100.00%> (+0.02%)
:arrow_up:

Continue to review full report at Codecov.

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


Secondary Colour converted to hex

Mez didn't fix my bipolar code so I did

This also let's you change the text colour even after you initiate the api so that's a bit trolly :trollface:


Include the un-minified built files for easy debugging...

Building from source was a snap, but not everyone wants to install node.  Also, I'm on v0.6.5 and had to run make twice:
➜  rickshaw git:(master) make
npm install clean-css
<EMAIL_ADDRESS>../node_modules/clean-css
└──<EMAIL_ADDRESS>cat src/css/detail.css src/css/graph.css src/css/legend.css  > rickshaw.css
/Users/Rubikzube/node_modules/.bin/cleancss rickshaw.css > rickshaw.min.css
node.js:284
throw new Error('process.stdout cannot be closed');
^
Error: process.stdout cannot be closed
at SyncWriteStream. (node.js:284:15)
at output (/Users/Rubikzube/node_modules/clean-css/bin/cleancss:43:7)
at /Users/Rubikzube/node_modules/clean-css/bin/cleancss:24:5
at [object Object]. (fs.js:115:5)
at [object Object].emit (events.js:64:17)
at afterRead (fs.js:1111:12)
at Object.wrapper [as oncomplete] (fs.js:254:17)
make: *** [rickshaw.min.css] Error 1
➜  rickshaw git:(master) make
npm install uglify-js
<EMAIL_ADDRESS>../node_modules/uglify-js
cat src/js/Rickshaw.js src/js/Rickshaw.Graph.js src/js/Rickshaw.Fixtures.Color.js src/js/Rickshaw.Fixtures.RandomData.js src/js/Rickshaw.Fixtures.Time.js src/js/Rickshaw.Fixtures.Number.js src/js/Rickshaw.Color.Palette.js src/js/Rickshaw.Graph.Ajax.js src/js/Rickshaw.Graph.Annotate.js src/js/Rickshaw.Graph.Axis.Time.js src/js/Rickshaw.Graph.Axis.Y.js src/js/Rickshaw.Graph.Behavior.Series.Highlight.js src/js/Rickshaw.Graph.Behavior.Series.Order.js src/js/Rickshaw.Graph.Behavior.Series.Toggle.js src/js/Rickshaw.Graph.HoverDetail.js src/js/Rickshaw.Graph.JSONP.js src/js/Rickshaw.Graph.Legend.js src/js/Rickshaw.Graph.RangeSlider.js src/js/Rickshaw.Graph.Renderer.Line.js src/js/Rickshaw.Graph.Renderer.Stack.js src/js/Rickshaw.Graph.Renderer.Bar.js src/js/Rickshaw.Graph.Smoother.js src/js/Rickshaw.Graph.Unstacker.js src/js/Rickshaw.Series.js  > rickshaw.js
/Users/Rubikzube/node_modules/.bin/uglifyjs rickshaw.js > rickshaw.min.js

This is all set with commit 043f6cbca6053bba90144167fadec7155de32ffb


added logging metrics

from https://mattermost.atlassian.net/browse/MM-22786

Summary

Ticket Link

Thanks, @thefactremains - LGTM.


fixes godep save failure

godep save ./... fails not finding package gopkg.in/v2/yaml as
the same package is availble as gopkg.in/yaml.v2 through Godeps.json

Thanks Ashwin! However, we need a signed CLA on file for you before we can accept pull requests. See CONTRIBUTING.md for details.

Cool! working on getting the CLA signed...

@a-robinson could you please confirm that you received the contributor agreement signed?

I don't see a CLA for ashw7n. Did you use that username on it or something else? Can you log in to https://cla.developers.google.com/ and confirm that your CLA is listed?

It shows the eBay agreement.  I have both ashw7n and<EMAIL_ADDRESS>in the google groups required for the CLA.  So what do I do now?
Ashwin
Ashwin
On Tue, Feb 17, 2015 at 9:30 AM, Alex Robinson<EMAIL_ADDRESS>wrote:

I don't see a CLA for ashw7n. Did you use that username on it or something
else? Can you log in to https://cla.developers.google.com/ and confirm
that your CLA is listed?
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/pull/4358#issuecomment-74710944
.

I checked again with your email ID and I still cant find a CLA.
@lavalamp to see if he has any ideas.

Sometimes the corporate ones take a little bit to show up for us. I'll email around if it doesn't show up in a little bit.

Actually I just realized that the google groups I had to create for the CLA
has my email ids<EMAIL_ADDRESS>and<EMAIL_ADDRESS>The google
groups itself is named ebay-kubernetes and has few other folks in it.
Ashwin
Ashwin
On Tue, Feb 17, 2015 at 10:50 AM, Daniel Smith<EMAIL_ADDRESS>wrote:

Sometimes the corporate ones take a little bit to show up for us. I'll
email around if it doesn't show up in a little bit.
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/pull/4358#issuecomment-74726200
.

Ah, there we go. Both of those emails are good to go. Sorry for the trouble!


Improve footer

Description
Increase footer size and add social media icons

I would like to work on this issue.

Can I please work on this issue.?

Can I please work on this issue.?

This issue has already been assigned, if he don't make a pr, it'll be assigned to you

Hey @2024-SANDHYA! I think I can improve the footer part to a good extent. Please assign me the task, will be happy to help! :D


Avoid infinite loop

Pull Request
Related issue
Fixes #45
What does this PR do?
This PR prevents the code from falling into an infinite loop when the requested URL (/health) cannot be retrieved. Additionally, I have also included a "status":"available" check if the URL returns a 200 OK status (though I am not sure if this is required). Do let me know if everything is okay. I'd be happy to make the requested changes.
(I used this as reference as cited in #45 )
P.S.: I am unsure why the indent is not shown properly in GitHub's commit tab, but it is proper in the code.
PR checklist

[x] This PR fixes an existing issue
[x] Contributing guidelines have been read
[x] The title is accurate and descriptive of the changes (to my knowledge)

@alallema unfortunatly this will break the update script while checking for the health status after the instance has started with the new dump


fix: show line breaks in curriculum explainer

Description
-- Show line breaks for curriculum explainer statements on Overview tab in curriculum view
How to test

Go to https://deploy-preview-2008--oak-web-application.netlify.thenational.academy/teachers/curriculum/maths-secondary/overview
You should see line breaks in the explainer statement

Screenshots
How it used to look (delete if n/a):

How it should now look:

Checklist

[x] Added or updated tests where appropriate
[x] Manually tested across browsers / devices
[x] Considered impact on accessibility
[x] Design sign-off
[x] Approved by product owner
[ ] Does this PR update a package with a breaking change

:tada: This PR is included in version 1.275.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Basic bench

This PR adds an extremely low-res benchmarking suite to slow-odgi.
I ripped out lots of the stuff that was useful for actually ensuring correctness but was a pain in terms of Make and turnt engineering.

validate is just run against valid graphs; I've thrown out the infrastructure where we perturbed the graph to make nice test-cases. This means validate is less of a delicate flower when it comes to setup, oracle, and test: it can just be added to the usual list of oracles and targets.
For inject and overlap I've stashed precomputed input files under slow_odgi/setup/; this means that Make (and thus turnt) does not need to run setup environments for these.
turnt's use is now purely cursory. I just use it to line up all my stuff and orchestrate it. All the outputs are sent to /dev/null, and all the oracle files (e.g. k.depth, k.inj) are empty. Turnt's oks are 100% vacuous.

I don't really think this stuff can/should be merged; the changes here are too destructive. In milestone 0.2 I'll do a more careful version that brings back testing and is therefore mergeable.
At the risk of repeating myself: fair warning, there's no testing going on over here!! If someone wanted to speed slow-odgi up using an automated tool that guarantees the preservation of program semantics, that is fine because the code as written was tested relatively rigorously before I ripped the testing infrastructure out. In order to make more thoughtful changes that really get into the slow-odgi code, please wait for me to improve this stuff in milestone 0.2.
Here are the numbers I got:
time make fetch
# grabs .gfa files from odgi's repository.

real	0m1.149s
user	0m0.228s
sys	0m0.095s

time make og
# runs `odgi build` on each .gfa file, creating .og files.

real    0m0.571s
user    0m0.570s
sys     0m0.065s

time make slow-odgi-all-oracles
# runs each odgi command against each .og file.

real    0m8.376s
user    0m13.965s
sys     0m1.462s

time make slow-odgi-all-tests
# runs each slow-odgi command against each .gfa file.

real    10m7.876s
user    9m35.973s
sys     0m29.469s

$ time make slow-odgi-all-oracles
real    0m8.376s
$ time make slow-odgi-all-tests
real    10m7.876s

8 seconds versus 10 minutes! slow-odgi truly is living up to its name, isn't it? 😂
Anyway, awesome to see that it's not enormously difficult to get some basic timing numbers. To facilitate easy/reproducible benchmarking in the same setup as testing, maybe we want to think creatively about what features we'd add to Turnt to collect that data for us.

Oh yes yes, if Turnt were interesting in getting into the benchmarking business, I'd be super keen to brainstorm on that. Here I was, crying myself to sleep, afraid I'd have to leave its warm embrace...

I have brought testing back!

I have carved out the fussier but more thoughtful tests (e.g. making interesting GFAs for validate via link-dropping, handmade files...) into a separate Make target called test-slow-odgi-careful. The benchmarking is against the clean, basic suite. To test with full care, run make test-slow-odgi-careful. To test the basic suite, run make test-slow-odgi.
In earlier commits of this branch, I was piping outputs to /dev/null, thus avoiding a lot of the overhead of printing to screen. I have scrapped that, and am printing to screen again. That hack, at least the way I did it, was too destructive. In a future version, I'll use Python's subprocess to orchestrate my benchmarking, and then stdout=subprocess.DEVNULL will achieve this in a cleaner way.

Here's how to tun the benchmark, and the results I see for now:
time make fetch
real    0m0.610s
user    0m0.200s
sys     0m0.078s

time make og
real    0m0.475s
user    0m0.468s
sys     0m0.059s

time make slow-odgi-all-oracles
real    0m27.687s
user    0m35.349s
sys     0m3.729s

time make slow-odgi-all-tests
real    7m26.518s
user    7m7.473s
sys     0m19.073s


Can I print the IP address for the device?

Can I print the IP address for the device?
For the rack, you specified cf.cf_name.
I am not fully aware of what this is.
But is there a way to specify the IP address of the device.
I have tried something like this, but this is probably wrong:
            'text_fields': [
                'name',
                'primary_ip.display'
            ]

hi @LHBL2003 ! You can print primary IP address for device.
'text_fields': [
                'name',
                'primary_ip'
            ]

cf_name it is for Custom Fields

primary_ip works Thank you


Proposed new rule: Trailing commas are required for multi-line initializers.

Category: Maintainability?
Rule: Trailing commas are required for multi-line initializers.
Description: All elements in object initializers and array initializers that span multiple lines must be followed by a comma.
Rationale: For multi-line object initializers and array initializers, requiring a trailing comma for each element has some advantages:

Elements can be reordered easily (Alt + Up/Down in most cases).
Adding a new element at the end doesn't change unrelated lines.

This rule would fire for the following:
var x = new Barnacle
{
    Age = 100,
    Height = 0.2M,
    Weight = 0.88M
};

var x = new []
{
    Tuple.Create("A", 1),
    Tuple.Create("B", 2),
    Tuple.Create("C", 3)
};

The rule would not fire for the following (although other rules might):
var x = new Barnacle
{
    Age = 100,
    Height = 0.2M,
    Weight = 0.88M,
};

var x = new { Min = 0, Max = 0 };

var x = new [] { Tuple.Create("A", 1), Tuple.Create("B", 2), Tuple.Create("C", 3) };

I think it makes sense. If we implement it, I'd like to see it paired with a rule that trailing commas shouldn't be included in single-line initializers.

:+1: I think this is a good proposal

:+1:

Grabbing this :+1:

@nbarbettini Sounds good. I'm not planning to include any other rules for RC 1 other than the three that are already assigned to that milestone, but we can include it afterwards.

@sharwell Cool. I'll PR to master when I'm done.

I'd like to see it paired with a rule that trailing commas shouldn't be included in single-line initializers.

Still want to do this as well, as a separate rule?

Still want to do this as well, as a separate rule?

Not sure yet. I'll have to think about it some more after RC 1.


Terraform - Visible by default

PR Summary
Making the terraform tab visible by default
PR Checklist

[x] PR has a meaningful title
[x] Summarized changes
[x] This PR is ready to merge and is not Work in Progress
[x] Link to a filed issue

Going to leave this one as draft until @TIGHEDEV helps our css styling out 👍


Provide an example of configuration with kotlin

The gradle configuration with kotlin is relatively less straightforward than the one with groovy.
I believe that it would be helpful to include an example of configuration with the kotlin.
I could make a pull request if you think that it's a good idea.
An example of a working (kotlin) configuration is :
tasks {
	compileThrift{
		thriftExecutable = "../thrift/libs/0.17.0/thrift.${osdetector.classifier}"
		sourceItems.from(project.layout.files("../thrift/src"))
	}
}

You mean providing kotlin example? SGTM 👍


[Windows] Set OVS services always restart on failure

This patch updates OVS installation script to:

Set OVS services always restart on failure.
Set ovs-vswitchd service depend on ovsdb-server service.

Signed-off-by: Rui Cao<EMAIL_ADDRESS>

Codecov Report

Merging #1495 into master will decrease coverage by 25.54%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master    #1495       +/-   ##
===========================================
- Coverage   67.99%   42.44%   -25.55%
===========================================
  Files         165       95       -70
  Lines       13107    10821     -2286
===========================================
- Hits         8912     4593     -4319
- Misses       3254     5827     +2573
+ Partials      941      401      -540

Flag
Coverage Δ

integration-tests
?

kind-e2e-tests
?

unit-tests
42.44% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/agent/agent_linux.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/ovs/openflow/default.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/agent/config/node_config.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/apis/controlplane/helper.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/apis/controlplane/v1beta1/register.go
0.00% <0.00%> (-92.86%)
:arrow_down:

pkg/ovs/openflow/logs.go
9.52% <0.00%> (-90.48%)
:arrow_down:

pkg/controller/networkpolicy/tier.go
0.00% <0.00%> (-90.00%)
:arrow_down:

pkg/apis/controlplane/register.go
0.00% <0.00%> (-85.72%)
:arrow_down:

pkg/ovs/openflow/ofctrl_group.go
0.00% <0.00%> (-84.38%)
:arrow_down:

pkg/ovs/openflow/ofctrl_packetout.go
0.00% <0.00%> (-80.00%)
:arrow_down:

... and 129 more

/skip-all

LGTM - --service-monitor does not work?

--service-monitor means try to restart 2 times on failure and stop after that.


Cookies are not sent

This issue pertains to the following package(s):

[ ] GraphQL Playground - Electron App
[x] GraphQL Playground HTML
[ ] GraphQL Playground
[x] GraphQL Playground Express Middleware
[ ] GraphQL Playground Hapi Middleware
[ ] GraphQL Playground Koa Middleware
[ ] GraphQL Playground Lambda Middleware

What version of graphql-playground(-electron/-middleware) are you experiencing the issue(s) on?
1.6.1
What is the expected behavior?
When I set request.credentials: include in settings, cookies should be sent
What is the actual behavior?
Cookies are not sent.
When I downgrade to<EMAIL_ADDRESS>everything works just fine.

same here. the latest version doesn't send cookies, settings are ignored. 1.4.9 works fine.

I am using graphql-playground-middleware-koa and I have the same issue. @michalkvasnicak how do you pass the request.credentials: include settings to the middleware?
Also wouldn't it make sense to send all cookies by default?

@capaj the middleware is responsible for rendering GraphQL Playground, so I just set request.credentials to include in the settings of GraphQL Playground (in browser).

@michalkvasnicak it didn't help. Request doesn't include cookies.

@rocknrolla777 yeah I know, I just answered the question of @capaj. We need to wait until the new version is released.

The problem still exists with latest graphql-playground-middleware-express.
I can confirm that downgrading this package to 1.4.9 fixes things.

Fixed in the latest version. Please let us know if you still have that issue, then we reopen it!

Tested 1.7.2 in different browsers. Cookies are not sent 😞

npm install graphql-playground-html@latest -> 1.6.13
npm install graphql-playground@latest -> 1.3.17
The latest graphql-playground here at GitHub is 1.8.10
Which version is the latest? 1.8.10? Why NPM is so much older?
No idea what's going on here 😨

👍 I have same issue - setting a "Cookie" in the header and the request.credentials to "include" does not send the cookie

Hi @maxwellsmart84, did you use Set-Cookie header to set cookie?

@yoshiakis   I have my settings like so:

Am I missing a setting?

@maxwellsmart84 You can't set cookie from the HTTP HEADERS panel. You need to receive Set-Cookie header from a server in advance to set cookie.


Update not started run

Would be awesome if we make it possible to update map, field, round or other stuff for a run that has not begin scoring.
I.e. update the run view http://<IP_ADDRESS>/admin/5859b7f8a68cf45b14b50b52/runs

Was updated in rewrite branch


[bugfix] duration toString with mixed signs, fixes #3960

This should fix #3960 with output along the lines of "-P1MT-1H-59S" and "P1MT-1H-59S".

close/open

@ashsearle  you fixed an ie8 compatibility problem in another PR and now you introduce Math.sign which is not supported by any ie, not just 8 ;-)
Can you please add a few more tests, and rewrite this sign checking the old school way. Also I'm a bit worried with the current code behavior towards 0 and if you rewrite it it should hopefully feel more correct.

@ichernev I don't think the old school way is readable:
  var ymSign = ((this._months < 0 && total >= 0) || (this._months >= 0 && total < 0)) ? '-' : '';
  var daysSign = ((this._days < 0 && total >= 0) || (this._days >= 0 && total < 0)) ? '-' : '';
  var hmsSign = ((this._milliseconds < 0 && total >= 0) || (this._milliseconds >= 0 && total < 0)) ? '-' : '';

Instead, I've updated to use local copy of Math.sign polyfill from MDN and reordered comparisons for consistency.

@ashsearle or just multiply and see if its negative? Just be careful with edge cases with 0 :)

@ichernev Do you need any more changes before accepting this pull request?
(as you quite rightly noted, using multiplication to determine whether two numbers have opposite signs is complicated by zero, and it becomes confusing to read and write.  Current code is (IMO) easy to read, so I'm not planning on changing it.)

Merged in 1ba87ac6cc638682e7a848522f6e30019b1d40b1


Model details

Thank you for having an open dataset of angiography. I have several questions about the details of the U-net model and YOLOv8 model you implemented.
For U-net: which part of the data did you use to train, validate and test the model?
Also, you mentioned a gradient-based edge detection algorithm in the U-net. What is the algorithm and is there any code that you implemented it in the original U-net model?
For YOLOv8 model: you mention there are two stages the localization and label assigning; and the segmentation. I wonder if it is an end-to-end process or the two stages are performed separately? Is it possible for you provide your code with this?
Thanks,
Ren Yu

Thank you for having an open dataset of angiography. I have several questions about the details of the U-net model and YOLOv8 model you implemented.
For U-net: which part of the data did you use to train, validate and test the model? Also, you mentioned a gradient-based edge detection algorithm in the U-net. What is the algorithm and is there any code that you implemented it in the original U-net model?
For YOLOv8 model: you mention there are two stages the localization and label assigning; and the segmentation. I wonder if it is an end-to-end process or the two stages are performed separately? Is it possible for you provide your code with this?
Thanks, Ren Yu

你好，关于U-net的算法，请问你找到了吗？


(codeblocks) Updated to embedd installer

U r missing import to core. OR we na ditch that mechanism and preinstall it on AV. It will not be the same as package can use unpublished stuff this way but that rarelly happens and that package can still manually import.

Since this one doesn't use au to download the the file, a reference to core extension shouldn't be needed.

Oh, yes, forgot about it, sorry.
On Dec 5, 2016 4:18 PM, "Kim Nordmo"<EMAIL_ADDRESS>wrote:

Since this one doesn't use au to download the the file, a reference to
core extension shouldn't be needed.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/chocolatey/chocolatey-coreteampackages/pull/450#issuecomment-264880806,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAFPB7rotuTrUPFgeEecXz2o1cSIBwyWks5rFCtLgaJpZM4LETw5
.

@majkinetor
Is there anything else?

Looks clear


Fix/connection (#1)

Fixes #27
The project just had to be packaged again.
Consider squashing.

Oh, very good catch, thank you! :)


Derived FIelds Loader issue: loadRelated missing from type store.

Which packages are impacted by your issue?
@graphprotocol/graph-cli
Describe the issue
Utilizing version 0.58.0 win32-x64 node-v16.20.0 of the CLI, when attempting to utilize the derived fields loader I get the following error when attempting to build:
Property 'loadRelated' does not exist on type '~lib/@graphprotocol/graph-ts/index/store'.
in generated/schema.ts(7346,23)

In my mapping I am attempting to load the child entities and loop through:
  let silos = _lendingProtocol.silos.load();

  for (let i = 0; i < silos.length; i++) {
    log.info("loop # {}", [i.toString()]);
  }

Reproduction
local environment
Steps to Reproduce the Bug or Issue

Utilize the .load() function on an array for a derived field.
Run graph build

Expected behavior
I expected to be able loop through the array of entities after loading them to a variable.
Screenshots or Videos
No response
Platform

OS: Windows
NodeJS: node-v16.20.0
@graphprotocol/* version(s): 0.58.0

Subgraph Manifest
No response
Subgraph GraphQL Schema
type LendingProtocol implements Protocol @entity {

  " All markets that belong to this protocol "
  silos: [Silo!]! @derivedFrom(field: "protocol")

...
}

type Silo @entity {
  " {Smart contract address of the market} "
  id: ID!

  " Protocol that the silo belongs to "
  protocol: LendingProtocol!
...
}

Additional context
No response

thanks - @saihaj @incrypto32 is this similar to / a duplicate of https://github.com/graphprotocol/graph-tooling/issues/1449 (I think this example from @Data-Nexus also involves interfaces?)

What version of graph-ts are you using? For this feature to work you need v0.31.0+ https://github.com/graphprotocol/graph-tooling/releases/tag/%40graphprotocol%2Fgraph-ts%400.31.0

I had ^0.31.0 listed in my package.json and after running npm install the issue no longer appears when I attempt to build.
I also assisted a hacker at ETHGlobal NY ran into the same issue and he had newly installed the CLI.
My schema does utilize an interface, although that was not the portion that was broken. The hacker at ETHGlobal only had 2 entities and we moved away from using the derived field and storing a manual array so we could loop through them.

I had ^0.31.0 listed in my package.json and after running npm install the issue no longer appears when I attempt to build.

@Data-Nexus are you saying that this issue worked with 0.31.0?

Correct, this can be closed.
The ETH Global hackers repo was on version 0.30.0 so the error seems common between us.


Unify function decorators for function calling, reply, termination and hook functions

Why are these changes needed?

Add function decorators for registering functions for reply, termination and hook and unify API
Describe the solution you'd like
Add the following function decorators:
@agent.register.for_execution
@agent.register.for_llm
@agent.register.for_reply
@agent.register.for_termination
@agent.register.for_hook

Related issue number

Closes #1214
Checks

[x] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[x] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[x] I've made sure all auto checks have passed.

Codecov Report
Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (2e519b0) 30.63% compared to head (14ed733) 18.91%.

Files
Patch %
Lines

autogen/agentchat/conversable_agent.py
75.00%
2 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1215       +/-   ##
===========================================
- Coverage   30.63%   18.91%   -11.72%
===========================================
  Files          32       32
  Lines        4365     4373        +8
  Branches     1015     1070       +55
===========================================
- Hits         1337      827      -510
- Misses       2932     3451      +519
+ Partials       96       95        -1

Flag
Coverage Δ

unittests
18.91% <75.00%> (-11.68%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Enable statistics in web UI in non-_system databases in cluster mode.

Scope & Purpose
Backport of https://github.com/arangodb/arangodb/pull/13614
In cluster mode, the web UI dashboard did not display statistics properly when not being logged in to the _system database. For all other databases than _system, no statistics were displayed but just some "No data..." placeholders.
Statistics for non-_system databases were not properly displayed since 3.6.10 due to an internal change in the statistics processing.
In addition, a new startup option --server.statistics-all-databases was added that controls whether cluster statistics are displayed in the web interface for all databases (if the option is set to true) or just for the system database (if the option is set to false).
The default value for the option is true, meaning statistics will be displayed in the web interface for all databases.

[x] :hankey: Bugfix (requires CHANGELOG entry)
[ ] :pizza: New feature (requires CHANGELOG entry, feature documentation and release notes)
[ ] :fire: Performance improvement
[ ] :hammer: Refactoring/simplification
[x] :book: CHANGELOG entry made

Backports:

[x] No backports required

Testing & Verification

[x] The behavior in this PR was manually tested
[x] This PR adds tests that were used to verify all changes:

[x] Added new integration tests (e.g. in authentication)

Link to Jenkins PR run:
http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/14132/

http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/14198/


[Feature] Fading Navigation Bar on Home Screen

This PR implement the fading navigation bar on the home screen.

Where is the difference to just setting the isTranslucent property of the UINavigationBar to true?


Feature/perm policy group clients

Re: https://github.com/NikiforovAll/keycloak-authorization-services-dotnet/issues/35

Will this ever be merged? I need these api routes, and I was planning to implement more of them, but I'm not sure if it is worth the effort if it never gets merged :/

Will this ever be merged? I need these API routes, and I was thinking about implementing more of them, but I'm not sure if it is worth the effort if it does not get merged :/

I think this PR has to be abandoned.
I made further changes to the code but pushed it to a different branch.
I will need to apply those changes to the appropriate branch, resolve conflicts, then re-open the PR.


Change organization name to ballerinax and module name to jira

Description:
We need to change the organization name of this connector to ballerinax. we need to change the module name of this connector to jira

Already fixed


UART not working on BB Blue

It seems that the UART cannot send/receive anything.
I am able to open each of the available serial ports but no data is sent nor received.
Are the UARTs simply not available, or have anybody else had similar issues?

@JesperChristensen89 lets see your current software stack for debuggin..
Pleas run:
sudo /opt/scripts/tools/version.sh

Regards

Also please provide the conditions of your test, which port are you using? Also are you testing with a loopback jumper or o-scope? UART is working fine for me right now.

Yes, sorry - I will provide the information below.
I have tested with two seperate BeagleBone Blue Rev A2. Both are freshly flashed with the newest Stretch IoT image, and the librobotcontrol is the newest version from here.
Software stack:
git:/opt/scripts/:[73593ebe3b7d3cc381eeb502d45ccb33a6ec5e78]
eeprom:[A335BNLTBLA21712EL001598]
model:[TI_AM335x_BeagleBone_Blue]
dogtag:[BeagleBoard.org Debian Image 2018-08-30]
bootloader:[eMMC-(default)]:[/dev/mmcblk1]:[U-Boot 2018.03-00002-gac9cce7c6a]:[location: dd MBR]
kernel:[4.14.67-ti-r73]
nodejs:[v6.14.4]
uboot_overlay_options:[enable_uboot_overlays=1]
uboot_overlay_options:[uboot_overlay_pru=/lib/firmware/AM335X-PRU-RPROC-4-14-TI-00A0.dtbo]
uboot_overlay_options:[enable_uboot_cape_universal=1]
pkg check: to individually upgrade run: [sudo apt install --only-upgrade <pkg>]
pkg:[bb-cape-overlays]:[4.4.20180914.0-0rcnee0~stretch+20180914]
pkg:[bb-wl18xx-firmware]:[1.20180517-0rcnee0~stretch+20180517]
pkg:[kmod]:[23-2rcnee1~stretch+20171005]
pkg:[librobotcontrol]:[1.0.2-git20180829.0-0rcnee0~stretch+20180830]
pkg:[firmware-ti-connectivity]:[20170823-1rcnee1~stretch+20180328]
groups:[debian : debian adm kmem dialout cdrom floppy audio dip video plugdev users systemd-journal i2c bluetooth netdev cloud9ide gpio pwm eqep admin spi tisdk weston-launch xenomai]
cmdline:[console=ttyO0,115200n8 capemgr.enable_partno=BB-UART1,BB-UART2,BB-UART4,BB-UART5 bone_capemgr.uboot_capemgr_enabled=1 root=/dev/mmcblk1p1 ro rootfstype=ext4 rootwait coherent_pool=1M net.ifnames=0 quiet]
dmesg | grep pinctrl-single
[    1.085780] pinctrl-single 44e10800.pinmux: 142 pins at pa f9e10800 size 568
dmesg | grep gpio-of-helper

I have tested all of the UARTs both by O-scope and with a loopback jumper. I have both been using the rc_uart_test program, and I have manually opened serial ports tested that.
ttyS0 works (as a serial login console), ttyS1, ttyS2 and ttyS5 are completely silent (no data being sent nor received - verified by o-scope).

Additional observation this morning:
UT5 (UART 5, ttyS5) works just fine. However, still nothing from UT1 or UART 2 from GPS.

Phew, good to know it's just 1&2, that means the pinmux just needs to be set up for those pins. Please try the following and let me know if those ports wake up:
for UART2:
echo uart > /sys/devices/platform/ocp/ocp:P9_22_pinmux/state
echo uart > /sys/devices/platform/ocp/ocp:P9_21_pinmux/state
for UART1:
echo uart > /sys/devices/platform/ocp/ocp:P9_24_pinmux/state
echo uart > /sys/devices/platform/ocp/ocp:P9_26_pinmux/state
This can also be done with the pinmux module in this library. Since the ports are labeled uart it is probably best that these are in uart mode by default. I've updated the device tree to reflect that with this commit: abf7369df272267db6d9120b78b381bde8db1651
Please clone my repo as it stands now (with this commit included) and install the new device tree as follows:
cd librobotcontrol/device_tree
sudo bash install_dt_manually.sh
select 2 and press enter
sudo reboot
then give it a try, they should work without having to change the pinmux. Let me know your findings, if this works we can update the device tree in the beaglebone images.


Session based optimizer configuration

Summary of the changes / Why this improves CrateDB
This adds session settings for rule optimisation adressing https://github.com/crate/crate/issues/9749
This is a draft, documentation should be updated as well.
Deactivate an optimizer rule:
SET SESSION optimizer_merge_aggregate_and_collect_to_count = false;

Activate it back:
SET SESSION optimizer_merge_aggregate_and_collect_to_count = true;

All the optimizer rules will show up like any other session setting:
cr> show all;
+---------------------------------------------------------+-----------------+-------------------------------------------------------------------------------------+
| name                                                    | setting         | description                                                                         |
+---------------------------------------------------------+-----------------+-------------------------------------------------------------------------------------+
| search_path                                             | pg_catalog, doc | Sets the schema search order.                                                       |
| enable_hashjoin                                         | true            | Considers using the Hash Join instead of the Nested Loop Join implementation.       |
| max_index_keys                                          | 32              | Shows the maximum number of index keys.                                             |
| server_version_num                                      | 100500          | Reports the emulated PostgreSQL version number                                      |
| server_version                                          | 10.5            | Reports the emulated PostgreSQL version number                                      |
| optimizer_merge_aggregate_and_collect_to_count          | false           | Indicates if the optimizer rule MergeAggregateAndCollectToCount is activated.       |
| optimizer_move_filter_beneath_order                     | true            | Indicates if the optimizer rule MoveFilterBeneathOrder is activated.                |
| optimizer_remove_redundant_fetch_or_eval                | true            | Indicates if the optimizer rule RemoveRedundantFetchOrEval is activated.            |
| optimizer_rewrite_collect_to_get                        | true            | Indicates if the optimizer rule RewriteCollectToGet is activated.                   |
| optimizer_move_filter_beneath_project_set               | true            | Indicates if the optimizer rule MoveFilterBeneathProjectSet is activated.           |
| optimizer_move_order_beneath_nested_loop                | true            | Indicates if the optimizer rule MoveOrderBeneathNestedLoop is activated.            |
| optimizer_deduplicate_order                             | true            | Indicates if the optimizer rule DeduplicateOrder is activated.                      |
| optimizer_move_filter_beneath_group_by                  | true            | Indicates if the optimizer rule MoveFilterBeneathGroupBy is activated.              |

Checklist

[x] Added an entry in CHANGES.txt for user facing changes
[ ] Updated documentation & sql_features table for user facing changes
[x] Touched code is covered by tests
[x] CLA is signed
[x] This does not contain breaking changes, or if it does:

It is released within a major release
It is recorded in CHANGES.txt
It was marked as deprecated in an earlier release if possible
You've thought about the consequences and other components are adapted
(E.g. AdminUI)

I decoupled the rule session settings from the SessionSettingsRegistry using a Service Locator, added more type safety and prevented the rules from streaming around.

The Optimizer would then get these loaded instances instead of creating new instances of the rules when creating the optimizer.

I had a similar idea in mind, what blocked me was that some of the rules are depending on Functions which has a lot of dependencies including guice etc. However, I am happy to give this a try.

The Optimizer would then get these loaded instances instead of creating new instances of the rules when creating the optimizer.

I had a similar idea in mind, what blocked me was that some of the rules are depending on Functions which has a lot of dependencies including guice etc. However, I am happy to give this a try.

We could add the Functions as parameter to apply (Or give the rule access to the full planner context)