Add support for DATE inputs to date extraction functions

Add support for DATE type to date extraction functions:
year, month, day, dow, doy, hour, minute, second, millisecond.
Add support for DATE type to date_trunc function.

@aditi-pandit Looks good. Minor comments below.

Thanks Masha. Addressed the review comments in the latest patch.

@aditi-pandit Aditi, the internal linter raised a few warnings. Would you take a look?

variable 'kSecondsInDay' defined in a header file; variable definitions in header files can lead to ODR violations
unused parameter 'date' in DateTimeFunctions.h:254
unused parameter 'config' in DateTimeFunctions.h:318

@aditi-pandit Aditi, the internal linter raised a few warnings. Would you take a look?

variable 'kSecondsInDay' defined in a header file; variable definitions in header files can lead to ODR violations
unused parameter 'date' in DateTimeFunctions.h:254
unused parameter 'config' in DateTimeFunctions.h:318

Thanks Masha. Have fixed these warnings.

Super clean! I love how the new simple function framework is shaping up :) Thanks Aditi!

@aditi-pandit I still get this warning:
variable 'kSecondsInDay' defined in a header file; variable definitions in header files can lead to ODR violations

@aditi-pandit I still get this warning:
variable 'kSecondsInDay' defined in a header file; variable definitions in header files can lead to ODR violations

As per https://clang.llvm.org/extra/clang-tidy/checks/misc-definitions-in-headers.html making this inline should fix the violation.
Maybe we should add these flags in the invocation of clang-tidy in the format-fix scripts in velox. If you give me a list of these I could add them.

Maybe we should add these flags in the invocation of clang-tidy in the format-fix scripts in velox. If you give me a list of these I could add them.

Indeed. It would be nice to align linter that runs in GitHub with internal one. @mshang816 Michael, any idea how we could approach this? CC: @pedroerp


Get-DbaForceNetworkEncryption multiple suggestions

Is this:
Just place "x" to indicate selection (e.g. "[x]")

[ ] feature
[x] bug:

Steps to Reproduce
-ComputerName parameter should be changed to -SQLInstance to match other commands and the help.
Example 1 of using local host does not work. and even when I put in the server name I get the message below.
There is an example with the -WhatIf parameter, but it is not defined as a parameter.
Attached screen shots/console output

hey @gbargsley - that is a good proposal! I also removed the example.


TCP bandwidth limit

Title:  Is there any way to limit transport bandwidth for tcp-proxy, e.g. 20kb/s.
Description:
I found a bandwidth limit filter, but it seems that the filter is only for HTTP.  Is there a similar filter can be used for tcp-proxy, I think it is useful in many scenarios.
[optional Relevant Links:]
type.googleapis.com/envoy.extensions.filters.http.bandwidth_limit.v3.BandwidthLimit
https://www.envoyproxy.io/docs/envoy/latest/intro/arch_overview/other_features/bandwidth_limiting#arch-overview-bandwidth-limit

is this has been solved?

I'm also interested in that.

/assign


RedirectToPage doesn't trim /Index from URL

RedirectToPage should trim the trailing /Index from the URL given that it's default anyway.
That is, doing this return RedirectToPage("/Index"); should effectively be the same as return Redirect("~/");.
Obviously needs to work for sub-folders and when route values are passed as well.

@rynowak @pranavkm @Eilon

Should we even match /Index - right now we create a route entry for / and for /Index? Is there value to the latter?
If we want to keep both, then / needs to have a better value of order so that its considered first.

In the amazing new world of Razor Pages perhaps it's fine to canonicalize URLs to their simplest form?


a little question about '/currentfilemembersastree'

This endpoint not returning a FileName in Location property. I can easily fix it.
A question is: bug it or not?

It probably should. Although, I expect the value to be the same as the FileName that's passed in with the request.

Sorry! I forget to update you here. This end point is effectively deprecated at this point. You should use the /v2/codestructure end point, which replaces this.


Serializing container elements in XML should not add counter to auto-generated tags

When serialising containers (std::list, std::vector, ...) using the XML archives, Cereal auto-generates the tags for each container member. Currently Cereal uses unique tags, which is very non-standard and un-XML-ish:
<container_name size="dynamic">
some value
some other value
...
the nth value
</container_name>
Instead the normal XML way to serialize such structures is to use the same repeated tag for all members in the container:
<container_name size="dynamic">
some value
some other value
...
the nth value
</container_name>

Relates to existing discussion on mailing list: https://groups.google.com/forum/#!topic/cerealcpp/sDwloQBA-70
Will slate this to be looked at for 1.3.


CDC OOM when update changefeed and resume

What did you do?

TiDB cluster with 3 CDC deployed (32C 64G each)
There are 2 changefeed (memory-quota is set to 16GB), one for single big table, the  other for 4k small tables. the lag for big-table is normal, the lag for 4k-table is 20h
Update changefeed for 4k-table, and resume changefeed

[root@tc-ticdc-0 /]# /cdc cli changefeed --server http://<IP_ADDRESS>:8301 update -c 4k-table --sink-uri="kafka://downstream-kafka.cdc-kafka-big-cluster-tps-1712340-1-428:9092/4k-tables?protocol=avro&partition-num=3&replication-factor=3" --config /tmp/changefeed2.toml
Diff of changefeed config:
{Type:update Path:[SinkURI] From:kafka://downstream-kafka.cdc-kafka-big-cluster-tps-1712340-1-428:9092/4k-tables?protocol=avro&partition-num=3&replication-factor=3&enable-tidb-extension=true&avro-decimal-handling-mode=string&avro-bigint-unsigned-handling-mode=string To:kafka://downstream-kafka.cdc-kafka-big-cluster-tps-1712340-1-428:9092/4k-tables?protocol=avro&partition-num=3&replication-factor=3}
{Type:update Path:[Config Integrity IntegrityCheckLevel] From:correctness To:none}
Could you agree to apply changes above to changefeed [Y/N]
y
Update changefeed config successfully!
ID: 4k-table
Info: {"upstream_id":7234390566115419974,"namespace":"default","id":"4k-table","sink_uri":"kafka://downstream-kafka.cdc-kafka-big-cluster-tps-1712340-1-428:9092/4k-tables?protocol=avro\u0026partition-num=3\u0026replication-factor=3","create_time":"2023-05-19T09:35:04.998625017+08:00","start_ts":441580663020716072,"admin_job_type":1,"config":{"memory_quota":17179869184,"case_sensitive":true,"enable_old_value":true,"force_replicate":false,"ignore_ineligible_table":false,"check_gc_safe_point":true,"enable_sync_point":false,"bdr_mode":false,"sync_point_interval":600000000000,"sync_point_retention":86400000000000,"filter":{"rules":["betting_0.*"],"event_filters":null},"mounter":{"worker_num":16},"sink":{"protocol":"avro","schema_registry":"http://<IP_ADDRESS>:8081","csv":{"delimiter":",","quote":"\"","null":"\\N","include_commit_ts":false},"dispatchers":[{"matcher":["*.*"],"partition":"","topic":"tidb_{schema}_{table}"}],"column_selectors":null,"transaction_atomicity":"","encoder_concurrency":16,"terminator":"\r\n","date_separator":"none","enable_partition_separator":true,"file_index_digit":0,"enable_kafka_sink_v2":false,"only_output_updated_columns":null},"consistent":{"level":"none","max_log_size":64,"flush_interval":2000,"storage":"","use_file_backend":false},"scheduler":{"enable_table_across_nodes":true,"region_threshold":50000,"write_key_threshold":0},"integrity":{"integrity_check_level":"none","corruption_handle_level":"warn"}},"state":"stopped","creator_version":"v7.1.0","resolved_ts":441599939071967246,"checkpoint_ts":441580697505497117,"checkpoint_time":"2023-05-19 19:44:39.327"}

What did you expect to see?

cdc should not OOM

What did you see instead?

CDC OOM, cdc memory consumption is 110GB, much larger than memory quota (16GB)

Versions of the cluster
[root@tc-ticdc-2 /]# /cdc version
Release Version: v7.1.0
Git Commit Hash: 29afa87f9af5d388657e0afa303463a6978153f1
Git Branch: heads/refs/tags/v7.1.0
UTC Build Time: 2023-05-19 07:33:51
Go Version: go version go1.20.3 linux/amd64
Failpoint Build: false

/severity major

/assign @asddongmen

The "memory-quota" parameter is valid for  one changefeed in ticdc . The default value is 1 G , which is enough for most cases. The oom issue is caused by the pebble GC when a large "memory-quota" value is set.

/close


Move compression backend to Compressonator

Motivation:

ispc_texcomp hasn't been updated in years, including merging some PRs for significant-looking bugfixes. Compressonator is under active development (for now).
ispc_texcomp requires a fancy custom compiler, which complicates the build process and hinders both distribution and CI/testing. Compressonator can be easily built from source on any platform with a relatively modern C++ compiler.
Compressonator supports more output formats than ispc_texcomp.

A functional Compressonator port is in the https://github.com/cdwfs/img2ktx/tree/compressonator branch. Compressonator requires cloning an enormous pile of dependency, 99% of which are unnecessary for the library-only build. Wait until https://github.com/GPUOpen-Tools/Compressonator/issues/58 is resolved before merging into master.

Current workaround: put local copies of the sole external dependency (half.cpp & friends from ilmbase) in the third_party directory, and configure Compressonator to look there instead of in its own Ext directory. Recursive submodule clone is no longer necessary.

As of Compressonator 3.2, the build headaches have been resolved; I'll merge this branch into master shortly after a bit more local testing.


Add CASM class hash

Description

Adds CasmContractClass class hashing.

closes #815
Checklist

[x] Linked to Github Issue
[x] Unit tests added
[ ] Integration tests added.
[ ] This change requires new documentation.

[ ] Documentation has been added/updated.

Codecov Report

Merging #814 (d2c0dc4) into main (dce8b59) will increase coverage by 0.02%.
The diff coverage is 94.66%.

@@            Coverage Diff             @@
##             main     #814      +/-   ##
==========================================
+ Coverage   91.77%   91.80%   +0.02%
==========================================
  Files          52       53       +1
  Lines       12297    12388      +91
==========================================
+ Hits        11286    11373      +87
- Misses       1011     1015       +4

Impacted Files
Coverage Δ

src/core/contract_address/casm_contract_address.rs
94.66% <94.66%> (ø)

... and 8 files with indirect coverage changes


puppeteer_test: Add missing wait_for_checked call to avoid rare flake.

Link to CZO Thread
This PR fixes a rare flake which was most probably
caused when we clicked on the Check All button, and
we instantly cleared the filter when it was still marking
the user checked, which nullified the effect of clicking
on Check All button.

Testing plan:
GIFs or screenshots:

Merged, thanks @Riken-Shah!


<IP_ADDRESS> - Heal status does not update in UI

Checklist

[ ]  I am not using Home Assistant. Or: a developer has told me to come here.
[ ] I have checked the troubleshooting section and my problem is not described there.
[ ] I have read the changelog and my problem is not mentioned there.

Deploy method
Docker
Z-Wave JS UI version
<IP_ADDRESS>f4cd65
ZwaveJS version
10.7.0
Describe the bug
In <IP_ADDRESS> if doing a heal, and not selecting battery nodes the HEAL column/status on the UI does not update.
The heal actually takes place, though, as shown in the logs.
To Reproduce
Initiate heal, do not select battery nodes.
Watch UI for heal status to change/refresh screen/sort by heal column.
Expected behavior
Heal status should update in the UI.
Additional context
No response

Error - disregard.


How to relate table to another table twice?

I have the following business case:
A user can be the owner or picker of a list. My table Users has ID and the table Lists has the fields owner_id (which comes from the user.id) and picker_id (which also comes from user.id).
In other words, the same user can be the owner of some lists and picker of other lists. Never both.
My code looks like this:

    const User = bookshelf.Model.extend(
        {
            tableName: 'users',
            lists() {
                return this.hasMany(List, 'owner_id');
            },
            listsPick() {
                return this.hasMany(List, 'picker_id');
            }
        }
    );

    const List = app.bookshelf.Model.extend(
        {
            tableName: 'lists',
            user() {
                return this.belongsTo(User);
            },
            picker() {
                return this.belongsTo(User)
            }
        }
    );

When I query users using this code:
User
                .where('id', req.params.id)
                .fetch({withRelated: ['lists']})
                .then(user => res.status(200).json(user))
                .catch(err => res.status(500).send(err))

It returns me the following JSON:
{
    "id": 1,
    "firstName": "Wagner",
    "createdAt": "2019-02-20T01:16:42.780Z",
    "updatedAt": null,
    "lists": [
        {
            "id": 1,
            "createdAt": "2019-02-20T01:18:03.441Z",
            "updatedAt": null,
            "store_id": null,
            "user_id": 1,
            "picker_id": null
        }
    ]
}

Finally, the question is: Sometimes I just want to return lists that the user is the owner. Sometimes lists that the user is the picker. How can I do it when using withRelated keyword?

Looks to me you're already doing it for lists where the user is the owner. If you use listsPick instead of lists in your withRelated array you should get lists where the user is the picker.

Thanks Ricardo.
Question answered.


Bump versions and some tweaks to work with react-native v0.14.2

replacegoog.require('figwheel.connect');
goog.require('rn_test.core');

withadd importJs('figwheel/connect.js');

add min profile, set optimizations to :simple

Looks good! I'm gonna do some clean up a bit later on everything.


Schedule b

#1183 #1184 #1185 #1186 #1189 #1190 #1191 #1194 #1195
This should create the basic schedule B form. Still lacking some fuel code functionality for the sprint.
Changelog:

Added Provisions of the Act Model
Created an operational script to prepopulate the provisions of the act table
Created routes for Schedule B
Consolidated managers that uses the_type
Added front-end for the Schedule B
Added classes to the columns so they're spaced out based on their possible content and so that it gets a horizontal scrollbar when it's too wide
Various fixes to the front-end
Removed auto-save from fuel-code

 Compliance Report B | Fuel Code Column |8|

 Credit Calculation | Adjust confirmation alert labels for credit calculations |1|

 Credit Calculation | Information indicator for the CI Limits Default CIs and Petroleum CIs tables mistakenly refers to revised values as energy densities |0|

 Credit Calculation | Editing or updating an EER value does not save for fuels with only one fuel class |1|

 Fuel Codes | Auto populate data for next fuel code version not functioning periodically |1|

 Compliance Report B | Units Column |1|


Update README.md

xmlns was incorrect in the XAML sample, at least for the 2.2.0 version I just grabbed. I confirmed this resolves properly for the Android and iOS targets.

Hi, sorry the README refers to v3.x.x that is currently in preview. But I'll release the stable version today I hope.

Released! It should be ok with <IP_ADDRESS>


I als fail to reproduce the results

Hello, in the case of consistent parameter settings, the results I reproduced in the CCVID dataset: general mAP=77.9%, CC mAP=75.7%, and the results in your paper cannot be reproduced. Can I compare the experimental results in my paper with the results I have reproduced?

Since the scale of CCVID is not very large, the results in this dataset are not stable. You can test multiple times and take the average result in your paper.


Any way I can use an Uno by doing the possessing elsewhere?

Hi, I have designed and 3d printed my own 2 DOF hexapod based around the Arduino Uno (particularly its dimensions) prior to finding out its lack of processing power for IK. Switching to the mega would mean I need to redesign and print the whole chassis again. Is there any way I could do all the processing on an external laptop or Arduino Mega and just send the servo positions to the Uno onboard the robot over Bluetooth?

The simplest one is probably Processing (https://processing.org/), there are loads of tutorials on connecting it to Arduino. Just keep in mind that even though it looks similar to Arduino IDE, the language is Java, not C/C++.
If you're looking something more professional, I suggest Visual Studio and writing the PC app in C#. Working with GUI elements is much easier than in Processing.

 I just got a cheap Arduino Mega clone and I'm going to design and print a new chassis for it. I want to add some more sensors to the robot later and maybe control it with a smartphone so I will need more processing power anyway. Also, it should be easier since I don't have to convert to Java.

 I can do 3d modeling and electronics but I'm new to programming in C++. This project is mostly for teaching my self C++ and the Arduino so I make design, wire, and program things.


Add pure kwarg to do

Can now specify if a wrapped function is pure or not (default is False). This is important for functions like random or time.
Also changed tokenize to default to incremental token if hashing fails. This means we don't do common-subexpression-elimination as aggressively as we did earlier, but people shouldn't be relying on dask preventing repeat computations anyway. This is probably a more sane default than we had originally.

Seems good to me


improve configure renovate for prisma dependency

Closes #4
If you'd like to extend the packageRules config to update dependencies and devDependencies, here's a snippet:
    {
      "matchDepTypes": ["devDependencies"],
      "matchUpdateTypes": ["minor", "patch"],
      "groupName": "devDependencies",
      "groupSlug": "dev-dependencies",
      "schedule": ["every weekend"]
    },
    {
      "matchDepTypes": ["dependencies"],
      "matchUpdateTypes": ["minor", "patch"],
      "groupName": "dependencies",
      "groupSlug": "dependencies",
      "schedule": ["every weekend"]
    },

You'll need to give renovate access to this repo
I think you can also edit your notification settings for emails here: https://github.com/settings/notifications

Thanks for this! Had to do some edits to the config afterwards as it didn't seem to be working quite right (unless that just happens the first time it's connected), but thanks for the starting point.


[Question] Proof of CLA

hello community,
please excuse if this is the wrong place to raise the issue, but I was wondering what sort of actual proof cla-assistent provides to be presented in front a court of law if it ever comes down to it. Is there any actual signing procedure and/or a way to extract a signed document (pdf)?
I am aware of the (lovely) possibility to export contributors that signed via csv, but I somehow doubt this will hold up at court.
Thank you for your effort and contributions: Eric.

To our current knowledge our current measures are sufficient and our legal team was fine with the taken approach (electronically stored documents in a systems as a record). We had a discussion with Microsoft and they are using DocuSign to go one step further, but so far we have decided to keep this more lightweight. Having said that, we would be happy to see how an integration into DocuSign (or s.th. similar) could look like and check if that is something needed by the community (formally needed by lawyers).


Update load.py

Add check on non-election Conflict type mission completion to only award a MissionPoints to the mission giver instead of both sides (as the MissionCompleted event of a massacre mission for civil war/war conflicts has both factions mentioned in the FactionEffects, so for both fe3 in fe a tally would be added otherwise).

The check is implemented right at the end before branching into the other than election type conflict mission, but I'm thinking it could technically also be done earlier (so it gets caught earlier, every bit of efficiency is good right) like at line 313 instead, changing 313 from
                    if this.MissionLog[p]["MissionID"] == entry["MissionID"]:
to
                    if this.MissionLog[p]["MissionID"] == entry["MissionID"] and this.MissionLog[p]["Faction"] == fe3:
Since that's the branch towards all conflict missions (or rather the ones without INF gains, which is during a conflict) but I'm not sure how this'd act with election missions and their secondary targets where you e.g. delivery election data to, technically those should work fine since the delivery has explicit INF gains in the target system so it's not going down this path, but I honestly haven't checked and thus it's probably safer to do the check on strictly (civil) wars on line 319 unless you know how it behaves with election missions.


feat(#39): Simple implementation of dynamic saving IR into correct places

Save IR files into correct places according with package information.
Closes: #39

PR-Codex overview
This PR focuses on implementing the name() method in the IR interface and its implementations.
Detailed summary

Added name() method in IR interface.
Implemented name() method in BytecodeIr and XmirIR classes.
Updated tryToSave() method in XmirFootprint class to use the name() method of IR objects.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@rultor merge

@rultor merge

@volodya-lombrozo Done! FYI, the full log is here (took me 2min)


Add 'test.sh' test runner script

test.sh invokes 'run_tests.py' in dcos-commons/tools/ for common bootstrap handling

LGTM. We can merge after the build pass.

retest this please.


join() of array with nulls throws NullPointerException

As mentioned in https://github.com/OpenRefine/OpenRefine/issues/6521#issuecomment-2104513492
To Reproduce
Steps to reproduce the behavior:

parseJson("['a',null,'b']".join(',')

Current Results
NullPointerException
Expected Behavior
"a,,b" (but because of a different bug it should currently return "a,b"

I know how difficult I have made it for the team when pushing for separating concerns for NULL versus Empty String.  But, in fact, it is worth our pain and has really helped a lot of folks especially debugging their own internal data workflows and fixing issues where NULL creeps in where they didn't expect it!  (I've heard from 3 folks already in the last 4 years...not a lot, I know, but who knows how many smiled and didn't reach out to me to acknowledge our efforts on that).
Much appreciated @tfmorris !

@thadguidry I wasn't aware that you had an issue open for handling arrays with nulls. Can you please link it here so that we can track it and close it when the fix for this gets merged?
This was just splitting out the issue raised by @iomicifikko.

I don't have a direct issue open.  I was referring to @ostephens previous work on Blank and Null handling that had been already merged.  But our importers still don't treat them separately!
We have the remaining tangential array/null issues:
#4823
#2157  <-- Actually, this can be closed now, I think?  Because null cell values can be checked with a Text Facet and mass edit replaced.  Agree?
To me, what remains is I think a left over "bug", but really original handling of not differentiating between null like ,, and blank/empty string like "" cells.
You can see this remains an issue if you use the clipboard importer or CSV/TSV importer on something like this:
"A","B","C"
"1","","3"
"1",,"3"

The B column row.index(0) should be an empty string blank and row.index(1) should be null.
But what happens is that both row indexes are imported as empty string (blank).
This is a bug to me that remains since we said we would treat null and blank separately throughout OpenRefine's importers and expressions and it's something that still pesters those in the scientific community who view row 2/column B value null as an outlier, but currently OpenRefine makes the value look the same as row 1/column B "" empty string during import.
Shall I open a new issue for that?

Shall I open a new issue for that?

If you think that the behavior of the CSV importer should be changed, yes, please create an issue. Ditto for any other importers.


fail bag update if the proto carries different typed attributes with same name

Refactor attribute update from protobuf to guard against duplicate attribute names.
Addresses TODO comment in original code

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

Needs approval from an approver in each of these files:

mixer/OWNERS

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

@elevran: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
5652157d0bab9c5f3af85d43843d3d63755be8a7
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Remove reference to roadmap which is a dead link

Coverage remained the same at 98.641% when pulling f5977b8ca42ecd809b39297e09a485b92663a67a on isaacbrodsky:remove-roadmap into 975de7a103ae8dfc22a629ef59890de283a5d8d8 on uber:master.


MountTmpfsAtTemp doesn't seem to always have the desired effect

Describe the bug
MountTmpfsAtTemp=false doesn't always seem to take effect.
This causes a lot of instance churn on scale-up.
I suspect there is something else running on startup that writes to the /tmp dir at around the same time that bk-mount-instance-storage.sh is run, blocking the unmount operation and causing it to fail. Is there anything in the AMI that might do that?
note: it's entirely possible that there's something in our custom AMI that we build atop of the buildkite-provided one that is responsible for this. If you can't reproduce this, that is still good info that we need to audit our startup processes
Steps To Reproduce
Steps to reproduce the behavior:

Create a elastic-ci stack on version v6.21.0
Set MountTmpfsAtTemp to false
wait for the Autoscaling Group to come online
observe some instances transition to a InService state, but get terminated after ~1 minute.

Expected behavior
when MountTmpfsAtTemp is set to false, the instance runs systemctl mask --now tmp.mount on startup, which correctly unmounts tmpfs from the /tmp directory. We observe this in a small number of instances that come up:
[yuchuanyuan@ip-10-0-102-149 ~]$ sudo cat /var/log/elastic-stack.log
Starting /usr/local/bin/bk-mount-instance-storage.sh...
Disabling automatic mount of tmpfs at /tmp
Created symlink /etc/systemd/system/tmp.mount → /dev/null.
Mounting instance storage...
No NVMe drives to mount.
Please check that your instance type supports instance storage.

<truncated>

[yuchuanyuan@ip-10-0-102-149 ~]$ systemctl status tmp.mount
○ tmp.mount
     Loaded: masked (Reason: Unit tmp.mount is masked.)
     Active: inactive (dead) since Tue 2024-06-11 20:02:24 UTC; 1h 18min ago
   Duration: 8.556s
        CPU: 7ms

Jun 11 20:02:24 ip-10-0-102-149.ec2.internal systemd[1]: Unmounting tmp.mount - /tmp...
Jun 11 20:02:24 ip-10-0-102-149.ec2.internal systemd[1]: tmp.mount: Deactivated successfully.
Jun 11 20:02:24 ip-10-0-102-149.ec2.internal systemd[1]: Unmounted tmp.mount - /tmp.

Actual behaviour
On a large number of instances, instead we see the following:
[yuchuanyuan@ip-10-0-114-131 ~]$ sudo cat /var/log/elastic-stack.log
Starting /usr/local/bin/bk-mount-instance-storage.sh...
Disabling automatic mount of tmpfs at /tmp
Created symlink /etc/systemd/system/tmp.mount → /dev/null.
Job failed. See "journalctl -xe" for details.
/usr/local/bin/bk-mount-instance-storage.sh errored with exit code 1 on line 33.
Starting /usr/local/bin/bk-configure-docker.sh...
Sourcing /usr/local/lib/bk-configure-docker.sh...

<truncated>

[yuchuanyuan@ip-10-0-114-131 ~]$ systemctl status tmp.mount
● tmp.mount - /tmp
     Loaded: masked (Reason: Unit tmp.mount is masked.)
     Active: active (mounted) (Result: exit-code) since Tue 2024-06-11 20:35:25 UTC; 24s ago
      Where: /tmp
       What: tmpfs
      Tasks: 0 (limit: 9247)
     Memory: 44.0K
        CPU: 6ms
     CGroup: /system.slice/tmp.mount

Jun 11 20:35:25 ip-10-0-114-131.ec2.internal systemd[1]: Unmounting tmp.mount - /tmp...
Jun 11 20:35:25 ip-10-0-114-131.ec2.internal umount[1923]: umount: /tmp: target is busy.
Jun 11 20:35:25 ip-10-0-114-131.ec2.internal systemd[1]: tmp.mount: Mount process exited, code=exited, status=32/n/a
Jun 11 20:35:25 ip-10-0-114-131.ec2.internal systemd[1]: Failed unmounting tmp.mount - /tmp.

Stack parameters:

AWS Region:  us-east-1
Version:  v6.21.0
AMI: built atop ami-04ca34320055d861c
instance types: m5.large, m5a.large, m5a.xlarge, m6a.xlarge

Hey @yyc, I haven't reproduced yet, and I'm not sure what, if anything, we would be running that is using /tmp concurrently with stack setup. However, I have an idea for a workaround in #1327. WDYT?

Hey @yyc, I haven't reproduced yet, and I'm not sure what, if anything, we would be running that is using /tmp concurrently with stack setup. However, I have an idea for a workaround in #1327. WDYT?

@DrJosh9000 Yes I've tested that and it looks like it works! Thanks for the quick fix :)


Use star import for @radix-ui/react-dialog

@radix-ui/react-dialog is packages incorrectly. Because of this, some tools (Next.js) must use a default import to import it, whereas others (Vite, Docusaurus) must use a star import. Since our demos use Vite and Docusaurus, let’s go with star imports.
See https://github.com/radix-ui/primitives/issues/1848

I'm fairly sure I've used star imports with Next.js in the past 🤔 Maybe their bundler/ESM config has changed in the meanwhile?

But did you import it from (real) ESM or from CJS?
Regardless, it’s a bug in Radix UI.


Cleanup: new project docs fix-ups (alternative)

NOTE:
This is an alternative commit/PR to https://github.com/docker/docker/pull/11229 that re-introduces some inline
styles because the docs/base image has not yet been updated for the current docs.
Either this PR or #11229 can be merged. Differences between this PR and 11229 can be seen below
This cleans up the recently added project docs and fixes some minor issues.

remove inline styles where possible
add redirects for renamed/replaced documents
add styles for GitHub labels to match the style on GitHub
fix minor markdown issues causing some code-blocks
to be shown as text
wrap the documents to 80-chars
use 4 spaces in stead of tabs for identing and remove
trailing whitespace/redundant blank lines
optimized 'gordon' image

Differences between this PR and 11229
(also as gist here; https://gist.github.com/thaJeztah/89594c820fa6aa092379)
diff --git a/docs/sources/project/get-help.md b/docs/sources/project/get-help.md
index 94bdeac..38daf09 100644
--- a/docs/sources/project/get-help.md
+++ b/docs/sources/project/get-help.md
@@ -15,7 +15,13 @@ page_keywords: IRC, Google group, Twitter, blog, Stackoverflow
 There are several communications channels you can use to chat with Docker
 community members and developers.

-

+
+
+

   <col width="25%">
   <col width="75%">

@@ -73,7 +79,13 @@ the easiest way to connect to IRC.

 2. Fill out the form.

-

+
+
+

         Nickname
         The short name you want to be known as in IRC.
diff --git a/docs/sources/project/make-a-contribution.md b/docs/sources/project/make-a-contribution.md
index 6a3bbd9..8a89ac0 100644
--- a/docs/sources/project/make-a-contribution.md
+++ b/docs/sources/project/make-a-contribution.md
@@ -3,6 +3,14 @@ page_description: Basic workflow for Docker contributions
 page_keywords: contribute, pull request, review, workflow, white-belt, black-belt, squash, commit

+
+
+

+
 # Work with a development container

 In this section, you learn to develop like a member of Docker's core team.
@@ -36,7 +44,7 @@ To remove unnecessary artifacts.

     You should see something similar to the following:

-

+

         CONTAINER ID
         IMAGE
@@ -58,7 +66,7 @@ To remove unnecessary artifacts.

     You should see something similar to the following:

-

+

         REPOSITORY
         TAG
@@ -120,7 +128,7 @@ environment.

     You should see something similar to this:

-

+

         REPOSTITORY
         TAG
@@ -284,7 +292,7 @@ build and run a `docker` binary in your container.

         $ docker ps

-

+

         CONTAINER ID
         IMAGE
diff --git a/docs/sources/project/test-and-docs.md b/docs/sources/project/test-and-docs.md
index 3fee820..7e725d2 100644
--- a/docs/sources/project/test-and-docs.md
+++ b/docs/sources/project/test-and-docs.md
@@ -2,6 +2,13 @@ page_title: Run tests and test documentation
 page_description: Describes Docker's testing infrastructure
 page_keywords: make test, make docs, Go tests, gofmt, contributing, running tests

+
+
+
 # Run tests and test documentation

 Contributing includes testing your changes. If you change the Docker code, you
@@ -54,7 +61,7 @@ is simply `test`. The make file contains several targets for testing:

-

+

     Target
     What this target does

For info as to why this PR was created, see https://github.com/docker/docker/pull/11229#issuecomment-77698868

@thaJeztah I'm going to use this one for now.  @SvenDowideit just recently handed his infrastructure changes. I'll need to check with the larger team as I am still learning the build system.  We will do this one for now.  I do apologize for not being able to get your hard work on the table fixes.

LGTM

:+1: great, no reason to fuss about it now, you need to get things moving. I'll create a follow-up PR to remove the temporary styles once the docs/base issue is fixed.


(feature request) if using default values for inputs, let stdout know about it

Closing for now as this hasn't garnered sufficient interest to prioritize


Docker ci

Rather than do our own thing to get opam/ocaml and coq.dev, we rely on the prebuilt images (should cut down build time too).

The compilation time is down to 30min (compared to the usual ~50min in the previous setup IIRC, yay!)


Installed 1.51 and still getting problems

First I would like to thank you for your time, but i am still getting issues with 1.51

[10/30/2020, 8:20:34 AM] [homebridge-magichome-dynamic-platform]
Registering existing accessory 'Simple GRB'
Model: 'AK001-ZJ2131'
Unique ID: 'C82E475FF973'
IP-Address: '<IP_ADDRESS>'
Light Version 8
Version Modifier: 51
Protocol Version: 1

[10/30/2020, 8:20:34 AM] [homebridge-magichome-dynamic-platform]
Registering existing accessory 'RGBW Non-Simultanious'
Model: 'AK001-ZJ2101'
Unique ID: '2CF432F4D667'
IP-Address: '<IP_ADDRESS>'
Light Version 8
Version Modifier: 51
Protocol Version: 8
[10/30/2020, 8:20:34 AM] [homebridge-magichome-dynamic-platform]
Warning! Continuing to register cached accessory 'RGBW Non-Simultanious' despite not being seen for 9 restarts.
Model: 'AK001-ZJ2101'
Unique ID: '5002913333B1'
IP-Address: '<IP_ADDRESS>'
Version undefined
Version Modifier: 51
[10/30/2020, 8:20:34 AM] [homebridge-magichome-dynamic-platform]
Warning! Continuing to register cached accessory 'RGBW Non-Simultanious' despite not being seen for 9 restarts.
Model: 'AK001-ZJ2101'
Unique ID: '500291331038'
IP-Address: '<IP_ADDRESS>'
Version undefined
Version Modifier: 51
[10/30/2020, 8:20:34 AM] [homebridge-magichome-dynamic-platform]
Registered 4 Magichome device(s).
New devices: 0
Cached devices that were seen this restart: 2
Cached devices that were not seen this restart: 2
[10/30/2020, 8:20:34 AM] Error: read ECONNRESET
at TCP.onStreamRead (internal/stream_base_commons.js:208:20)

There's a number of issues here that I can see. The most obvious is that you have two lights with versions listed as "undefined". That's never good.  Can you confirm that after updating to v1.5.1 you used the "prune all accessories next restart" function in plugins > homebridge-magichome-dynamic-platform > settings?
I also see the second light discovered as "light version 8" (RGBW) but the plugin has set them to protocol version 1 (GRB). That will cause this exact crash.
Can you please link me the controllers / bulbs that you have. Exactly if you can. It doesn't matter amazon or aliexpress etc... I just need a reference.
Thank you.

Hi,
Yes tried the Prune, but that didn't work, I am using the Magichome wifi controllers and has far has i am aware all my strips are just RGB, will try and find some links for you

[10/30/2020, 8:20:34 AM] [homebridge-magichome-dynamic-platform]
Warning! Continuing to register cached accessory 'RGBW Non-Simultanious' despite not being seen for 9 restarts.

This means that it is registering a cached device. One which has been around for 9 restarts. Please prune the accessories by doing the following:  enable "pruneAllAccessoriesNextRestart" > restart > ensure that accessories were pruned in the log > disable "pruneAllAccessoriesNextRestart" > restart.
There might still be an issue, but this will solve the duplicate problem and that's a start.

@Zacknetic I'm getting the same problem. I did the steps as above with no help.
11/2/2020, 10:40:20 PM [homebridge-magichome-dynamic-platform]
Discovered 1 devices.
11/2/2020, 10:40:21 PM [homebridge-magichome-dynamic-platform]
 Platform.ts.createAccessory():
Assigning controller to device: UniqueId: 'C82E475FC37D'
IpAddress '<IP_ADDRESS>'
Model: 'AK001-ZJ2131'
Original Light Version: 8
Light Version Modifier: 51
11/2/2020, 10:40:21 PM [homebridge-magichome-dynamic-platform] ( Modified) Light version: 1 matches known device type records
11/2/2020, 10:40:21 PM Error: read ECONNRESET
    at TCP.onStreamRead (internal/stream_base_commons.js:205:27)
11/2/2020, 10:40:21 PM HAP Warning: Characteristic 000000E3-0000-1000-8000-0026BB765291 not in required or optional characteristics for service 00000043-0000-1000-8000-0026BB765291. Adding anyway.
11/2/2020, 10:40:21 PM [homebridge-magichome-dynamic-platform]
Registering new accessory 'Simple GRB'
Model: 'AK001-ZJ2131'
Unique ID: 'C82E475FC37D'
IP-Address: '<IP_ADDRESS>'
Light Version 8
Version Modifier: 51
Protocol Version: 1
11/2/2020, 10:40:21 PM [homebridge-magichome-dynamic-platform]
Registered 1 Magichome device(s).
New devices: 1
Cached devices that were seen this restart: 0
Cached devices that were not seen this restart: 0
11/2/2020, 10:40:21 PM Got SIGTERM, shutting down Bridge...
11/2/2020, 10:40:21 PM Error: read ECONNRESET
    at TCP.onStreamRead (internal/stream_base_commons.js:205:27)
11/2/2020, 10:40:21 PM Error: read ECONNRESET
    at TCP.onStreamRead (internal/stream_base_commons.js:205:27)

This is the item I purchased: Outdoor LED Strip

@jzdeb separating the threads as these are different issues

@m0v1em4n closing this issue for the time being, feel free to re-open with an uploaded log file if the problem persists. Thank you.


WIP - Reworked registration form and register_user logic

This is a work in progress PR meant to discuss how to go about implementing https://github.com/jwag956/flask-security/issues/251
Code in here only changes internals, and shouldn't affect any clients.
It adds a way to extract both user model data and all form data from registration form.
The next step would be to expose form data to clients and I would like to ask for ideas on how to do that. Adding that data as say form_data argument to user_registered signal seems most natural. But it would break clients that use this event - as it would require them to add new argument to their user_registered callbacks. Is that fine?
form_data = registration_form.to_dict(only_user=False)
user_registered.send(app._get_current_object(), user=user, confirm_token=token, registration_form_data=form_data)

Any preferences/ideas on how to expose form_data without affecting existing clients? Adding a new signal seems a bit silly.

I think adding an extra argument is fine - users are supposed to define their signal handlers with **extra - flask specifically mentions this case of adding more info:
http://flask.palletsprojects.com/en/1.1.x/signals/

Refactored comments as requested.
I also modified user_registered to include form_data dictionary - with all fields, though if you would like to filter out user fields from there, that can be done.
There was one test (tests/test_registerable.py::test_registerable_flag) I had to modify, but I only added an assert that form_data isn't empty.
A test with some form including specific user-defined input asserting against it would be best, but I didn't find any example test that would expose registration form and go all the way to registering user. Can you point me out to something I can work off?
I also haven't modified docs yet.

Added tests/test_registerable.py::test_form_data_is_passed_to_user_registered_signal


整合 Disqus 給每篇文章

Update in https://github.com/MOLi-rocks/Casper/commit/13bc64ec459e20bc1d4934dadb81c884b79c7f0a, and comments look like


Fixes keybord directionnal keys not moving nodes in the 2D editor

Closes #12482. Thanks @poke1024 for pointing it out :)

@akien-mga @groud Please note that this still doesn't fix the broken plugin key handling at the same code location.

@akien-mga @groud Please note that this still doesn't fix the broken plugin key handling at the same code location.

Oh true, I think I see what you mean. Could you open an issue for that ? I'll assign it to myself.

@groud https://github.com/godotengine/godot/issues/12505


Introduce utils.getImportCfgFilepath method

This will introduce getImportCfgFilepath in utils to get import cfg file
path from the user.
ONE-vscode-DCO-1.0-Signed-off-by: SaeHie Park<EMAIL_ADDRESS>

From draft #70


Add banner to the notification service

What this PR does / why we need it:
,------.                          ,--.
|  .--. ' ,--,--.,--.--. ,---.  ,-|  | ,---.  ,---.
|  '--' |' ,-.  ||  .--'| .-. |' .-. || .-. |(  .-'
|  | --' \ '-'  ||  |   ' '-' '\ `-' |' '-' '.-'  `)
`--'      `--`--'`--'    `---'  `---'  `---' `----'
Parodos Notification Service 1.0.13-SNAPSHOT
Powered by Spring Boot 2.5.14

Change type

[ ] New feature
[ ] Bug fix
[ ] Unit tests
[ ] Integration tests
[ ] CI
[ ] Documentation
[ ] Auto-generated SDK code

Impacted services

[ ] Workflow Service
[x] Notification Service

Checklist

[x] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.

/lgtm


Add Story

Actually I want to know is there any way to add a story/a story with tag using instapy-cli?
However I appreciate your great work. It works like a charm.

Thank you man, it's fulfilling for me to hear your satisfaction. :)
On what you wrote: this could be a great feature, however there are some obstacles to manage.

The first is that we need some tools to intercept/sniff instagram app's traffic (android/ios).

Then post some datas to stories and log the traffic intercepted so we can see how the call to API is composed and we could replicate it on instapy-cli.

Code the endpoints/datas to instapy-cli and release a new feature.

This is the process of reverse-engineering any private API (like in this case for Instagram) from an android/ios app. And with this process we can integrate other functions, like post a story, post a video or send a message.
It's not so complex to do but it need a bit of time.
Anyone who want to collaborate (even only sniffing datas structure and endpoints) is welcome. 🥂

From v0.0.7 there is the support for story, only via a python script (not in cli implemented yet).
This is a feature not well tested, so if there is a problem submit a new issue.
🥂


Tags' PropertyValueConverter isn't playing nicely

The Grid Layout JSON shows that the Umbraco Tags property has been populated but is returning null or empty when attempting to render on the front-end i.e Model.HasValue("filters") is null
{   "grid": 12,   "allowAll": false,   "allowed": [ 	"button", 	"heading", 	"latestItems", 	"htmlBlock", 	"embedSnippet", 	"embedVideo", 	"imagePicker"   ],   "controls": [ 	{ 	  "value": { 		"docType": "6b2b05ba-a3be-4e5a-bdb5-98a8628d45e9", 		"value": { 		  "name": "Latest News", 		  "brandColour": "ffdc00", 		  "limit": "", 		  "filters": [ 			"Scotland" 		  ], 		  "filter": "Scotland" 		}, 		"id": "218a99f2-a73b-447d-40ea-7363bb9ce7fd" 	  }, 	  "editor": { 		"alias": "latestItems" 	  } 	}   ] }
Here is a link to the Umbraco Forum post Umbraco.Tags value is always empty

I am getting the following error:
System.Exception: Type validation failed. The value type: 'WhereSelectEnumerableIterator`2' does not match the DataType in PropertyType with alias: 'tags'
   at Umbraco.Core.Models.Property.set_Value(Object value)
   at Umbraco.Core.Models.Property..ctor(PropertyType propertyType, Object value)
   at Our.Umbraco.DocTypeGridEditor.Helpers.DocTypeGridEditorHelper.ConvertValue(String id, String contentTypeAlias, String dataJson)
   at Our.Umbraco.DocTypeGridEditor.Helpers.DocTypeGridEditorHelper.<>c__DisplayClass1.<ConvertValueToContent>b__0()
   at Umbraco.Core.Cache.NullCacheProvider.GetCacheItem(String cacheKey, Func`1 getCacheItem)
   at Our.Umbraco.DocTypeGridEditor.Helpers.DocTypeGridEditorHelper.ConvertValueToContent(String id, String contentTypeAlias, String dataJson)
   at ASP._Page_app_plugins_doctypegrideditor_render_doctypegrideditor_cshtml.Execute() in \App_Plugins\DocTypeGridEditor\Render\DocTypeGridEditor.cshtml:line 26
   at System.Web.WebPages.WebPageBase.ExecutePageHierarchy()
   at System.Web.Mvc.WebViewPage.ExecutePageHierarchy()
   at System.Web.WebPages.WebPageBase.ExecutePageHierarchy(WebPageContext pageContext, TextWriter writer, WebPageRenderingBase startPage)
   at System.Web.Mvc.RazorView.RenderView(ViewContext viewContext, TextWriter writer, Object instance)
   at System.Web.Mvc.BuildManagerCompiledView.Render(ViewContext viewContext, TextWriter writer)
   at Umbraco.Core.Profiling.ProfilingView.Render(ViewContext viewContext, TextWriter writer)
   at System.Web.Mvc.HtmlHelper.RenderPartialInternal(String partialViewName, ViewDataDictionary viewData, Object model, TextWriter writer, ViewEngineCollection viewEngineCollection)
   at System.Web.Mvc.Html.PartialExtensions.Partial(HtmlHelper htmlHelper, String partialViewName, Object model, ViewDataDictionary viewData)
   at System.Web.Mvc.Html.PartialExtensions.Partial(HtmlHelper htmlHelper, String partialViewName, Object model)
   at ASP._Page_Views___Default___Partials_grid_editors_base_cshtml.Execute() in \Partials\Grid\Editors\Base.cshtml:line 20

@smdooley @Kimtho Apologies for not get back to you sooner on this issue.
Only a quick update to say that I can reproduce the issue (in Umbraco 7.4.2) and that it appears to be an issue (bug!) with Umbraco's Tags PropertyValueConverter.  Unfortunately it's not something that we can fix within DTGE.
I'll leave this issue open for now, as I'm going to dig into the Umbraco Core code to see if I can identify the cause of the issue.

OK, I've spent a few hours digging deep into the Umbraco Core code, and found the cause.  It looks to be an Umbraco issue, not something we can fix within DTGE.
The underlying issue is that the Tags property-editor handles it's value conversion different to other property-editors. This goes beyond the remit of what DTGE can understand or handle.
(The same issue will hit Nested Content too)
I've raised an issue on the Umbraco tracker: http://issues.umbraco.org/issue/U4-8279

Based upon discussions Lee and I have had with Shannon, we have got clarification on the value conversion issue and a suggested workaround for our code has been suggested. However, in digging into the Tags property editor, it is clear now that it has been designed and built in such a way that it is only meant to be used once per page, and really should be used directly on the page doc type itself so in that case is unsuitable for use in DTGE.
With that in mind I'm going to close this issue and simply state that the tags property editor is not compatible with DTGE


[Bug] Float.TryParse little overview for other CultureInfo system

In the function static float readFloatConfig(XElement elem, string attrName) in Skin.cs [line 275 actually], there is a little oversight in the usage of Float.TryParse
My locale says that a floating point number should be written with a comma instead of a dot. Of course, as a dev, I always use a dot, but my computer didn't hear it from that ear, and so did NintendoSpy when reading from && to attributes in some rangebutton configuration in skins.
Thus, there is a little portability issue, as I initially downloaded the skin.xml from whom it worked just fine.
The solutions are simple :

Decide to let it be, warn users in documentation and lose a little bit of portability over consistency for different locale
Use the form float.TryParse (readStringAttr(elem, attrName), System.Globalization.NumberStyles.Any, System.Globalization.CultureInfo.GetCultureInfo("en-US"), out ret) forcing the North American style redaction for floating point number.
Use a middle term solution where 3 Float.TryParse are done in order to try different solutions, and report a fail if all of that didn't work

I've tried the third one, it has allow me to mix a dot-style number with a comma-style number, but it isn't a one line solution.
The functions become :
        static float readFloatConfig (XElement elem, string attrName)
        {
            float ret;
            string attrValue = readStringAttr(elem, attrName);
            if (! float.TryParse (attrValue, System.Globalization.NumberStyles.Any, System.Globalization.CultureInfo.CurrentCulture, out ret) &&
                ! float.TryParse(attrValue, System.Globalization.NumberStyles.Any, System.Globalization.CultureInfo.GetCultureInfo("en-US"), out ret) &&
                ! float.TryParse(attrValue, System.Globalization.NumberStyles.Any, System.Globalization.CultureInfo.InvariantCulture, out ret)){
                throw new ConfigParseException ("Failed to parse number for property '"+attrName+"' in element '"+elem.Name+"'.");
            }
            return ret;
        }

Thanks for pointing this out. I'll add this in a future commit when I get the chance.

hey, when this bug will be fixed? I'm trying to use rangebutton but that error is very annoying and I can't do what I wanted do to

I don't currently have any time to work on this project, so this bug will get fixed as soon as somebody submits a pull request that solves the issue.

oh ok, I will wait, thanks :D


bug: navigation bar doesn't appear on app load...

Type: bug
Platform: all
Type: bug
Platform: all
... but reappears after switching to a new tab.
This only happens on first app load (after closing the app) on both ios and android, and only on the default tab. After making one tab switch, the behavior reverts to normal and there are no bugs until the app is fully closed and reopen again.

Greetings @tebuevd!
My sensors indicate that you need to update your issue through our custom issue form. We are now requiring all issues to be submitted this way, to ensure that we have all of the information necessary to fix them as quickly as possible.
Click Here To Update Your Issue
I will have no choice but to close this issue if it is not resubmitted through the form. Please fill out the rest of the form, so that I may use my friendly robot powers to assist you.
Thank you!

Can you provide a codepen? There is probably a problem with your markup.

Greetings @tebuevd!
My sensors indicate a reply was requested, but as we have not received one yet, I am using my robot powers to close this issue. However, if you are still experiencing this issue, please feel free to reopen it by creating a new issue, and include any examples or necessary information.
Thank you for allowing me to assist you.

Greetings @tebuevd!
My sensors indicate a reply was requested, but as we have not received one yet, I am using my robot powers to close this issue. However, if you are still experiencing this issue, please feel free to reopen it by creating a new issue, and include any examples or necessary information.
Thank you for allowing me to assist you.


Trying to route the map in routes; Works in index.html body but won't work in routed template.

Hi,,
I am currently trying to implement dburles:google-maps and mdg:geolocation packages. When I add the {{> map}} function to the body of index.html everything loads but the problem is that it loads on every single page. When I try to add this to a sub-directory file, (in my case customerhome.html) nothing shows up and the  does not show up in the inspect element. There is no error in the console, but I am just getting this error. Right now I am trying to finish the project ASAP and it is very frustrating.
Thank you in advance.

I just had to put everything inside the div file and it worked!
picard facepalm


ICE on missing attribute

On an invocation site referring to a non-existent attribute I get:
compiler bug: compiler bug: +  [QualifiedMemberExpression] (67:101-67:117) : unknown [X][method or attribute does not exist: 'setX' in type 'TTT']   |  + setX [Identifier] (67:103-67:117)   |  +  [InferredTypeArguments]   |  +  [BaseMemberExpression] (67:101-67:101) : TTT : c : value c => TTT   |  |  + c [Identifier] (67:101-67:101)   |  |  +  [InferredTypeArguments]   |  + . [MemberOp] (67:102-67:102)    has a null target at unknown
compiler bug: expression with no declaration at unknown
That is a little too much for a missing attribute. The invocation isn't that fancy, just inside of a lambda like this:
fn(a, (TTT b) => b.setX(), ...);
Should be easy to fix since the error is identified properly, it just needs digging out.

Moving to 1.3. (Btw, a code example that was more copy&pastable might help speed things along)

I just tested with map({1->2}, (Integer x, Integer y) => x.notThere); and found the bug to be fixed.

Thanks.


[SMALLFIX] Fix metrics test

Followup to https://github.com/Alluxio/alluxio/pull/3813
@peisun1115

lgtm

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/11106/
Test PASSed.


Message deduplication fails and causes mesh network flood for rapid messages

A rate limiter has been introduced, but is not ideal.

This may be resolved with https://github.com/draeder/easypeers/commit/8013967226e1c978096a4941866ba34b5c46feab


Fix a buggy interaction with i3wm; Fill out server information

I'm using i3wm as a window manager and had a bug where self_anchor of a root block behaved as if it's TL regardless of its actual value. This change fixes the bug. I also tested it with another window manager (marco) and it doesn't seem to break anything.
I'm not sure I fixed the correct root cause though, since I still don't get what caused the bug. It makes sense to me that all anchors would behave like TL when self_rect size is zero, but it only happened in some WMs and I don't know why.

I also filled out server information. Cargo environmental variables seem reasonable to me. Some programs use these to decide the content of notifications (In my case it was Telegram not sending icons).

Thanks for the pull request!
I think you're absolutely right about this being the correct fix -- it actually doesn't even work as it is on my machine (bspwm).  I did some digging and I think at some point I added code to predict the size of the window on creation, probably for this purpose:
https://github.com/Toqozz/wired-notify/blob/dae1e3243f5613aa58ca0c6ba1ab686316c7c294/src/rendering/window.rs#L146-L153
But then forgot to update the positions code to make use of it.
I'm puzzled that it worked on any window managers, because that code should definitely always treat Rect::EMPTY as TL.  Maybe there's some config stuff going on that made it look like it wasn't?
Also, thanks for filling out the server information.


[Docs] Update GettingStarted.md with workaround for icu error

Background
https://forums.swift.org/t/error-running-utils-update-checkout-clone-with-ssh/72696
This adds the workaround from https://github.com/swiftlang/swift/issues/70966 to the documentation to help engineers from running into this error again when building the toolchain locally.
Test Plan
This is only a documentation change. No code is modified here.

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test macOS


Marten has started leaking DB connections since 7.x

In martendb 6.x the attached code was fine, as of 7.x this now leaks DB connections until it fails with "Npgsql.PostgresException 53300: remaining connection slots are reserved for roles with the SUPERUSER attribute"
In my migration application, the following code is called in a for loop.
    public override async Task ExecuteAsync(IDatabaseConfiguration databaseConfiguration, CancellationToken cancellationToken)
    {
        var store = BuildDocumentStore(
            databaseConfiguration,
            x => x.UseJavascriptTransformsAndPatching(t =>
        {
            t.LoadJavascript(Name, JsScript);
        }));

        await using IDocumentSession session = store.LightweightSession();
        await DoTransformsAsync(session, cancellationToken);
        await session.SaveChangesAsync(cancellationToken);
    }

    private async Task DoDocumentTransformAsync<TEntity>(IDocumentSession session, CancellationToken cancellationToken)
        where TEntity : IEntity
    {
        string entityName = typeof(TEntity).Name;
        session.Transform(PerformDocumentTransforms<TEntity>);
        await session.SaveChangesAsync(cancellationToken);
    }

Realised this is fixed by changing to:
await using var store = BuildDocumentStore(

So you may want to close this as an application bug, although the behaviour does seem to have changed..

That change happened quite awhile ago in 6.* I believe. All this kind of "am I correctly cleaning up after services" kind of thing goes much, much better if you just pull things in from an IoC tool. Is there any reason why you're building a DocumentStore like that on the fly? That's actually pretty expensive.

We are currently using 6.4.1, which is "6.x" above. The code sample is from in a small JavaScript utility that applies a set of migrations/transformations in turn to an existing database.
The JS transformations are each loaded using the code:
x => x.UseJavascriptTransformsAndPatching(t =>
        {
            t.LoadJavascript(Name, JsScript);
        }

.. hence the creation of the store on the fly, as "JsScript" changes each time.
The performance of the migration code isn't really an issue as this isnt run very often.
For our main Asp.Net web application, everything is created using the standard IoC container


Date doesn't start at current for some members

Whenever downloading using a list,
and a member is being skipped due to no new post criteria,
and the number of new posts already downloaded for this member is 0,
the next member will continue scanning from the date where the previous member was when skipping rather than resetting to the current date.

Which settings in config.ini you are talking about?

Not sure If I understand, do you have the sample with screenshot? Give me more information to check.
Try to disable/enable these config processfromdb, daylastupdated, checkUpdatedLimit, dateDiff.
Ensure you are using the latest releases here: https://github.com/Nandaka/PixivUtil2/releases
The post work date is based on the value given by the javascript value.
Please note that the datetime is based on UTC and not converted to local time.

https://github.com/Nandaka/PixivUtil2/blob/master/PixivModelWhiteCube.py#L235

I set it all to disabled/0 and oddly when it reaches page 2 it jumps from 2009 to 2017

it seemed to ignore the db setting so i wiped it here's the screenshot afterwards at the moment the date leaps

after looking at the screenshot more carefully it would seem it goes by order of image id but sorted as a string rather than a number.

I see, I think it is because the API separate illust and manga. I think I need to sort it after I append the list together in descending order (latest image first)

https://www.pixiv.net/ajax/user/7575/profile/all
If you use search by member id and tags or enable R-18 mode, it will uses different API
https://www.pixiv.net/ajax/user/7575/illustmanga/tag/R-18?offset=0&limit=24

Thanks in advance.
And despite the odd bug i still love the program.
Thank you for making such a neat and useful program.

try https://github.com/Nandaka/PixivUtil2/releases/tag/v20180928

Just got around to testing it.
It works perfectly now.
Thanks you.


updated flutter_calendar_carousel.dart

changed an import to part so that EventList class could be accessed by library user.

@MehndirattaPiyush Thanks for the contribution.


Db test update

What does this implement/fix? Explain your changes
implement tests after the changes to the Database occurred.
Does this reference any currently open issues?
…
Types of changes
What types of changes does your code introduce?
Put an x in the boxes that apply

[x] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. This is simply a reminder of what we are going to look for before merging your code.

[x] I have linted my code and I have run tests locally that pass
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation

Any other comments?
…

@BrendanOswego Have someone on QA merge this, this is outside of my scope.


Date::parse() bug

Date::parse() accepts timestamps, Carbon doen't.
This is cool but the timezone is wrong, always is "+00:00", you can simply use Date::createFromTimestamp() and all works as expected but this made me crazy for a while...
Thankyou!!

:+1:

Even if you pass the timezone?

Can you check if my last commit on master fixes that? :)

Thank you! Now there is no problem.

Done!


Update build & dependencies

upgrade purescript compiler & gulp-purescript
switch deps to use ^ ranges

:+1:


Get rid of using presenter->context

Also get get rid of $this->presenter

Sign presenter, Auth presenter and creation of services.
Services needs to be rewritten into factories.

Solved all except creation of services - not sure if necassary.
Accessing to presenter is ATM necassary in forms. - flashes, redirects.


Add python-face-alignment-pip

This will be required in a package that we plan to release soon.

Releases cannot use pip rules, they require native package manager rules. Will this still be useful?

Yes it'd still be useful. Would be great if we can add it.

Sorry for that. Indentation fixed.
Link: https://pypi.org/project/face-alignment/
Thanks


Lot of duplicated files

Microsoft PowerToys version
0.80.1
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
General
Steps to reproduce
Install the product.
Run gci "C:\Program Files\PowerToys" -Recurse -Force -File | Get-FileHash | Export-Csv c:\temp\powertoysfiles.csv -Encoding utf8 -Force -NoClobber to export the list of files.
✔ Expected Behavior
DLLs should be present only once to preserve space.
❌ Actual Behavior
Most of the DLLs are duplicated both in C:\Program Files\PowerToys and C:\Program Files\PowerToys\WinUI3Apps, which leads to wasted space.
I computed hash of files to be sure this is actual duplicates. Here is the resulting dataset:
powertoysfiles.xlsx
powertoysfiles.csv
Other Software
Windows 10

Thanks to the github bot, found this duplicate : https://github.com/microsoft/PowerToys/issues/24029
Closing this one.


Advanced compilation of ES6 super.someMethod() is passing unobfuscated name "someMethod" to goog.base

goog.module("my.namespace.MyClass");
var MySuperClass = goog.require("my.other.namespace.MySuperClass");
exports = class MyClass extends MySuperClass {
  constructor() {
  }
  /** @override */
  someMethod() {
    super.someMethod();
  }
}

The super call turns into:
Us.prototype.Kf=function(){Us.Er(this,"someMethod");}

and fails:
Uncaught TypeError: Cannot read property 'apply' of undefined

Hm, looks to be doing the right thing here: https://closure-compiler-debugger.appspot.com/#input0%3Dclass%2520MyClass%2520extends%2520MySuperClass%2520%257B%250A%2520%2520constructor()%2520%257B%250A%2520%2520%257D%250A%2520%2520%252F**%2520%2540override%2520*%252F%250A%2520%2520someMethod()%2520%257B%250A%2520%2520%2520%2520super.someMethod()%253B%250A%2520%2520%257D%250A%257D%26input1%26conformanceConfig%26externs%26refasterjs-template%26includeDefaultExterns%3D1%26CHECK_SYMBOLS%3D1%26CHECK_TYPES%3D1%26LANG_IN_IS_ES6%3D1%26MISSING_PROPERTIES%3D1%26PRETTY_PRINT%3D1%26TRANSPILE%3D1
so maybe I need to try from the command line, with both files. It sounds like this is related to class transpilation, not goog.module handling, right?

I'm using Closure Compiler from Java directly, and am using advanced compilation with full renaming of properties and so forth. This works fine except for the goog.base call which passes the method "someMethod" as a string, and it is forgetting to rename this value, presumably because it is now a literal string after transpiling?

I tried the appspot page, how do I enable equivalent of CompilationLevel.ADVANCED_OPTIMIZATIONS and setLanguageIn(LanguageMode.ECMASCRIPT6_STRICT)

Here it is with superclass and no warnings, but I don't know what I need to tick for advanced optimisations equivalent:
https://closure-compiler-debugger.appspot.com/#input0%3Dclass%2520MySuperClass%2520%257B%250A%2520%2520someMethod()%2520%257B%250A%2520%2520%2520%2520console.log(%2522super%2522)%253B%250A%2520%2520%257D%250A%257D%250Aclass%2520MyClass%2520extends%2520MySuperClass%2520%257B%250A%2520%2520constructor()%2520%257B%250A%2520%2520%257D%250A%2520%2520%252F**%2520%2540override%2520*%252F%250A%2520%2520someMethod()%2520%257B%250A%2520%2520%2520%2520super.someMethod()%253B%250A%2520%2520%257D%250A%257D%250Anew%2520MyClass().someMethod()%253B%250A%26input1%26conformanceConfig%26externs%26refasterjs-template%26includeDefaultExterns%3D1%26CHECK_SYMBOLS%3D1%26CHECK_TYPES%3D1%26LANG_IN_IS_ES6%3D1%26MISSING_PROPERTIES%3D1%26PRETTY_PRINT%3D1%26TRANSPILE%3D1

Ah, got it, here's the problem:
https://closure-compiler-debugger.appspot.com/#input0%3Dclass%2520MySuperClass%2520%257B%250A%2520%2520someMethod()%2520%257B%250A%2520%2520%2520%2520console.log(%2522super%2522)%253B%250A%2520%2520%257D%250A%257D%250Aclass%2520MyClass%2520extends%2520MySuperClass%2520%257B%250A%2520%2520constructor()%2520%257B%250A%2520%2520%257D%250A%2520%2520%252F**%2520%2540override%2520*%252F%250A%2520%2520someMethod()%2520%257B%250A%2520%2520%2520%2520super.someMethod()%253B%250A%2520%2520%257D%250A%257D%250Anew%2520MyClass().someMethod()%253B%250A%26input1%26conformanceConfig%26externs%26refasterjs-template%26includeDefaultExterns%3D1%26CHECK_SYMBOLS%3D1%26CHECK_TYPES%3D1%26LANG_IN_IS_ES6%3D1%26MISSING_PROPERTIES%3D1%26PRETTY_PRINT%3D1%26PROPERTY_RENAMING%3D1%26TRANSPILE%3D1%26VARIABLE_RENAMING%3D1

Perfect, thanks. The handling for SuperClass.base is at https://github.com/google/closure-compiler/blob/master/src/com/google/javascript/jscomp/ProcessClosurePrimitives.java#L615 so we need to do something similar for ES6 classes.

If you are using goog.module, you want to enable closure processing anyway.

"closure processing"? Is that an option in the appspot page? I'm not
familiar with this page; the link above was only the minimum options I
could tick to get the "someMethod" problem to arise. My experience is with
using a year-old version of Closure Compiler directly in Java, which (back
then) we customised heavily, configuring most options to maximise advanced
compilation.

The "closurePass" in the Java API.

Thanks. We're using it implicitly, since we have ADVANCED_OPTIMIZATIONS. Additional options in full are:
dependency pruning+sorting, moocher dropping, a few entry points
cross module code+method motion
define replacements
various warning/error levels/checks
check suspicious+symbols+types
closure coding convention
extern exports=false
ES6strict IN
ES5strict OUT
sourcemaps v3

I have a fix in progress for this. We know the name of the superclass so rather than Subclass.base() we can just call the superclass method directly. The only time we don't know the name of the super class is if you extend an arbitrary expression like class X extends (foo()) {... which we don't support anyway.


[WIP]Add READNE.md and executor tests for trino

What is the purpose of the change
Add some guide for Trino EngineConn plugin.
Related issues/PRs
Related issues: #2687
Related pr:#2639
Brief change log

Add Trino executor test class;
Add Trino README.md;
Fix some configurations for Trino.

Checklist

[x] I have read the Contributing Guidelines on pull requests.
[x] I have explained the need for this PR and the problem it solves
[x] I have explained the changes or the new features added to this PR
[x] I have added tests corresponding to this change
[x] I have updated the documentation to reflect this change
[x] I have verified that this change is backward compatible (If not, please discuss on the Linkis mailing list first)
[x] If this is a code change: I have written unit tests to fully verify the new behavior.

2022-10-17T03:14:38.5724692Z java.lang.NullPointerException: session is null
2022-10-17T03:14:38.5727761Z 	at java.util.Objects.requireNonNull(Objects.java:228)
2022-10-17T03:14:38.5731705Z 	at io.trino.client.StatementClientV1.<init>(StatementClientV1.java:96)
2022-10-17T03:14:38.5736995Z 	at io.trino.client.StatementClientFactory.newStatementClient(StatementClientFactory.java:24)
2022-10-17T03:14:38.5743771Z 	at org.apache.linkis.engineplugin.trino.executor.TrinoEngineConnExecutor.executeLine(TrinoEngineConnExecutor.scala:190)
2022-10-17T03:14:38.5750932Z 	at org.apache.linkis.engineplugin.trino.executer.TestTrinoEngineConnExecutor.testExecuteLine(TestTrinoEngineConnExecutor.scala:112)


settings.py might accept ~ (home dir)

Hello,
Home user directory shortcut ~ is not accept as a valid directory in settings.py
Running
$ python qstrader/examples/test_strategy_backtest.py

with this settings file
CSV_DATA_DIR="~/data/random/"
OUTPUT_DIR="~/qstrader/"

raises
FileNotFoundError: [Errno 2] No such file or directory: '~/qstrader/tradelog_2016-06-22'

os.path.expanduser might help https://docs.python.org/3/library/os.path.html#os.path.expanduser
It may be cleaner to create an inherited class for settings
with methods to get CSV_DATA_DIR, OUTPUT_DIR with ~ expanded
Kind regards

Something like
in settings_default.py
import os

class SettingsDefault(object):
    _CSV_DATA_DIR = "~/data"
    _OUTPUT_DIR = "~/qstrader"

    @property
    def CSV_DATA_DIR(self):
        return os.path.expanduser(self._CSV_DATA_DIR)

    @property
    def OUTPUT_DIR(self):
        return os.path.expanduser(self._OUTPUT_DIR)

in settings.py
from settings_default import SettingsDefault

class Settings(SettingsDefault):
    _CSV_DATA_DIR = "~/my_cache"
    _OUTPUT_DIR = "~/my_qstrader"

I made a similar sort of change suggestion for the logging path handling in QSForex in the following pull request:
https://github.com/mhallsmoore/qsforex/pull/39

In that I detect the file location in what I understand to be a pretty general way (independent of how the process was started) and then derive the directory of the logging config file (also in a platform independent way, I hope).
The effect was to prevent the logging error if you start trading.py in the way described in the QSForex readme.md .
Using normpath seems sensible to maximise platform independence.
Hope that helps in some way.

An other approach could be to use a JSON, a INI or a YAML configuration file.
It will be better easier for packaging.

Closed by https://github.com/mhallsmoore/qstrader/pull/48


Fix flaky integ remote store settings integ tests  (#7141)

Signed-off-by: Varun Bansal<EMAIL_ADDRESS>(cherry picked from commit 632eb44a541b28ef16ed261904b45d74b84f3b9f)

Description
Backport for https://github.com/opensearch-project/OpenSearch/pull/7141
Issues Resolved
NA
Check List

[x] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff
[ ] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #7185 (3cb0901) into 2.x (202c3dc) will decrease coverage by 0.12%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##                2.x    #7185      +/-   ##
============================================
- Coverage     70.44%   70.32%   -0.12%
+ Complexity    59751    59681      -70
============================================
  Files          4847     4847
  Lines        287122   287122
  Branches      41717    41717
============================================
- Hits         202260   201924     -336
- Misses        67930    68270     +340
+ Partials      16932    16928       -4

see 494 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


enable tests that use exfiltration data service

Description of Change
Re-enables some tests that were temporarily disabled after the introduction of functionality from the exfiltration data service.
https://github.com/code42/code42-mock-servers/pull/15 is a dependency of this PR so it needs to be merged first before the newly-enabled tests will pass.

@timabrmsn @tora-kozic @ryan-haley-code42 this is ready now that the dependent PR has merged and i fixed the other minor snags in the checks.


OpenMW support

Linux Mint 18 64 bit
OpenMW 0.40
Morrowind GOTY
How to sort mods for OpenMW on Linux
-> Run Morrowind Launcher.exe using wine
-> Enable your mods there
-> run mlox using wine or the python version
-> run OpenMW launcher & import the morrowind.ini settings & loadorder
So it there a possibility to make python version sort the OpenMW config?
Or is this more complicated because of different Linux distributions and different install locations?

I'd love to, but this is just a re-host of the sourceforge svn.
https://sourceforge.net/p/mlox/tickets/
I will agree that OpenMW is in many ways better than the original engine, but am not that familiar with mlox's actual source.
If you want to look at it, here's a possible starting point: https://github.com/EmperorArthur/mlox/blob/master/mlox/mlox.py#L1079

Side note, if you use the mlox-working branch you don't need "Morrowind Launcher.exe"  It's not the easiest, but it works
Your directory structure should have:

Morrowind.ini
"Data Files/"
"mlox/"

Here's what to do:

Make sure [game files] in morrowind.ini has no entries
Run mlox, and save the resulting load order.
Open the file with the new load order, and add "content=" to every line (search and replace is great for this)
Copy those lines to openmw.cfg

Here's what to do:
Make sure [game files] in morrowind.ini has no entries
Run mlox, and save the resulting load order.
Open the file with the new load order, and add "content=" to every line (search and replace is great for this)
Copy those lines to openmw.cfg

This is much more stuff to do :laughing: - I think I will use wine instead


Envoy: Delete the deprecated code to remove older cilium-envoy.log

Undo the code added as part of #3046 when users are not running into this issue anymore.

@manalibhutiyani Why was this closed? We should probably schedule this for an upcoming release, the code has existed since pre-1.0.

My bad. Out for review.


chore: timeout for system test

Set timeout for all system tests to some big (but limited) value. 10 minutes per test is probably enough.
This is an automated PR prepared using github-repo-automation.

Codecov Report

Merging #2 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master     #2   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines           4      4
=====================================
  Hits            4      4

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 56482cd...86fd9f5. Read the comment docs.


Cache null values in storage

When no value is found in the storage for a given key, cache that "null" in the memory and CacheService to reduce hits on the PpropertiesService for future runs. Fixes #144.

Looks good to me!

@grant PTAL


E2e fix locking wallets issue

Fix wallet locking because of wrong tags in some DApp features.
Branch tests results GH here, Allure here

Allure Report
allure-report-publisher generated test report!

processReports: ✅ test report for af041566

passed
failed
skipped
flaky
total
result

Total
32
0
4
0
36
✅


fixed issue #1559

resolved issue https://github.com/langchain-ai/langchain/issues/1559

Hi @siddarthreddygsr, thank you for the patch, but it's better to not make this modification.
The main recommendation for users is to use langgraph for agents. You can use create_react_agent to create pre-built agents.  The agents leverage tool calling capabilities of chat models and will be more robust.
In general, no patch PRs against langchain and/or langchain-core will not be accepted without unit-tests. In this case, however, it's not clear that the patch is meaningful -- any code changes that focus on adjusting the prompt or making it slightly more robust can't guarantee significant performance increase. It's better to rely on chat models tool calling capabilities and/or provide users with the ability to adjust / manipulate the prompt / tool results as they see needed. Both of these are easy enough to achieve with langgraph.


Add DateTimeOffsetGenerator to AutoGeneratorFactory and ensure Order …

…generation utilizes DateTimeOffset.

Hi @jgreene
Apologies. I haven't actively monitored this repo so didn't see your pull request. Seems like a valid addition, so I have actioned the merge.
Thanks,
Nick.


Snake to Camel Case

Currently, the library doesn't do much work on the path name received from swagger.  As an example, it converts POST: /queue/{id}/enqueue_player to something like PostQueuesIdEnqueue_player.  I think I'd like that to be PostQueuesIdEnqueueGroup, which seems like a good default for the library.  Mind if I send a pull request for this?

Yep! Sounds like a good idea, thanks!


khard: passthrough.tests are broken

Describe the bug
When trying to run the khard tests (by running something like nix-build -A khard.tests .), it fails with an error related to ruamel.yaml.
Log:
khard-0.18.0-test-version> building '/nix/store/3qdzdzwbgly64b25wmcgxcll3259ijf8-khard-0.18.0-test-version.drv'
khard-0.18.0-test-version> khard --version returned a non-zero exit code.
khard-0.18.0-test-version> Traceback (most recent call last):
khard-0.18.0-test-version>   File "/nix/store/pb2b805g3z20rqapna029xim3swcq63w-khard-0.18.0/bin/.khard-wrapped", line 6, in <module>
khard-0.18.0-test-version>     from khard.khard import main
khard-0.18.0-test-version>   File "/nix/store/pb2b805g3z20rqapna029xim3swcq63w-khard-0.18.0/lib/python3.10/site-packages/khard/khard.py", line 17, in <module>
khard-0.18.0-test-version>     from . import helpers
khard-0.18.0-test-version>   File "/nix/store/pb2b805g3z20rqapna029xim3swcq63w-khard-0.18.0/lib/python3.10/site-packages/khard/helpers/__init__.py", line 9, in <module>
khard-0.18.0-test-version>     from ruamel.yaml.scalarstring import LiteralScalarString
khard-0.18.0-test-version> ModuleNotFoundError: No module named 'ruamel.yaml'

Steps To Reproduce
Steps to reproduce the behavior:

Just build khard.tests

Expected behavior
Tests passing without error
Additional context
I ran git bisect, and the bug seems to be introduced in 5330b05d9952aad7606d5c0658d65faf1955c659, a commit which updated ruamel.yaml. I also ran hg bisect on that repo, and the tests broke in revision 836:18eb08865c03 of the hg repo, which is a rather big changeset.
Notify maintainers
@matthiasbeyer

Metadata
 - system: `"x86_64-linux"`
 - host os: `Linux 6.4.12-1-MANJARO, Manjaro Linux, noversion, rolling`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.15.1`
 - channels(root): `"nixpkgs"`
 - channels(anton): `"home-manager"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixpkgs`

nix-build -A khard.tests

cannot reproduce on latest master

It was apparently already fixed in 2b2ff2019e44d8aabf8985dce89955a2f8678312. Thanks for checking though.


add check for capacity size before allocating memory

https://github.com/microsoft/SEAL/issues/693#issue-2337681575
Added a check in [/native/src/seal/dynarray.h] to see if capacity > capacity_ before memory allocation

This doesn't implement the correct/expected functionality. While it might work in most cases, it doesn't correspond to the description of the function in the comments. In particular, if the given capacity is smaller than the current size, the function should automatically resize the DynArray to have size corresponding to the new given capacity. With your edit, the function can no longer be used to reduce the size of the DynArray container at all, which is not right either.


[SW-79] DEX transaction – Transaction detail

fixes #1457

Still wrong information

The transaction is actually a +6 WTT

https://explorer.aeternity.io/transactions/th_ahNbwrdfDDeCaSrSBcR2MPnN1f2fRRUpFgEKhXHEwpfdWzXPr

In the transaction list we have visualy more info on reverted remove liquidity transaction

Adjustments:

fix wrong condition for the pool fee in swap route
fix: swapRoute component did not  show hidden AE to WAE swap
fix: swapRoute was working completely different, showing pairs instead of tokens that being swap
fix: swapRoute did not include the route itself (wasnt showing actual route)
fix: there was no AE icon for sendTx transactions
fix: there was no icon for poolToken in reverted transactions

SwapRoute:

type
before
after

Swap token for AE

Swap AE for token

Swap WAE for token

Swap token for WAE

Swap token for token (without complex route)

Swap token for token (with complex route)


Is there a simple way to keep the menu from sliding back to the left

I picked up you platform and really am enjoying it. One thing I am having trouble with though is the menu as it keeps wanting to slide back to the left. Surely I could rediesign the all the li with different class names, but its such a good design. What I have used the left panel for is a form element editor, and every time I click in one of the forms, the menu disappears. Is there a simple way to prevent that?

Additionally, I am using form elements in the main window as well. I was thinking it would be nice to prevent them from moving forward until they fill out the form element. Is there a way I can access that do that. Maybe a function call?

I'm assuming that you are referring to clicking forms within the presentation, rather than in the menu. The menu has been designed to prevent interaction with the presentation while the menu is open. It adds an overlay when the menu is open so that when the user clicks the presentation area it is captured by the menu and the menu closes. So at present there is no way to interact with the presentation while the menu is open. I'll have a look at adding an option to remove this restriction.

I'm closing this issue for now as I don't believe the use case fits the menu plugin. If there are others that think this would be useful I'll have another look.


Expr::Wildcard should use TableReference as the qualifier type

Is your feature request related to a problem or challenge?
Currently, the wildcard expression is defined as Wildcard { qualifier: Option<String> }, and the semantic of the qualifier is unclear. During logical planning, the qualifier string is implicitly parsed via TableReference::from(), which invokes the SQL parser with the generic dialect.
Describe the solution you'd like
It might be a good idea to directly use TableReference as the type for wildcard qualifiers. Doing so will have the following benefits.

It will make the semantic more clear from the type.
It will avoid potential bugs when the qualifier is quoted/escaped in a way not supported by the generic SQL dialect.
It will make Expr::Wildcard more consistent with other variants such as Expr::Alias(Alias) (which has relation: Option<TableReference> as a field).

Describe alternatives you've considered
N/A
Additional context
N/A

take


Docs typo

-r, --record-count <RECORD_COUNT>
should be -n, --record-count <RECORD_COUNT> right?
Elsewhere in the README, -n is mentioned.

Good catch, thanks! That bug was introduced when the argument parser was updated in e7625cf4b4e477a9ecc1fd01d64b56e2c1a10ae0. It's now fixed in 7288f2a225e831e99626a3325e2ee8161085c80d.


Major overhaul of benchmarking

Issue #, if available:
Description of changes:
This is a major clean-up of benchmarking. In fact, things did not work properly before. This contains several fixes.

Reduce arguments in MethodArguments, in favor of scheduler_kwargs, which collect method-specific args
Make sure that scheduler_kwargs is properly modified by extra arguments. This requires recursive merging of dictionaries. The current code does not do the right thing
Simplify things a lot for people creating new experiment in nursery. There are wrappers now for all baselines, you can just pick what you want, not worrying about how arguments are put together based on command line args. Look at any module in nursery, how simple this is now
Updated benchmarking tutorial

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

Integ Tests are currently configured to run with each PR, whereas the End-to-end Tests that have AWS credentials only run manually on-demand and before each release.


fd_controller load error

I use this sdk in ros foxy, Omega.3.
the launch file load three controller, fd_controller, joint_state_broadcaster and ee_pose_broadcaster. However, the fd_controller seems no source code. Thus, the loading occur an error.

missing of fd_controller means can not control Omega by ros command. How to solve it?

ros2_control and ros2_controller should be added in package.xml depend

Hi, you are right ! feel free to propose a PR to fix it.


Open VSX repo has old version

The open vsx repo https://open-vsx.org/extension/Axway/vscode-titanium
still uses the old Axway 0.14.0 version! Could this be updated? It has almost 500 downloads and people who use VSCodium will search there by default

Shoot yeah I forgot to get that removed, will get that done.

Removed the Axway extension and publish the TiDev one. Currently waiting for my namespace claim to go through so that warning will be removed.
Once done I'll make sure some other folks have access and also setup open-vsx publishing in actions

awesome. Thanks for the quick update :+1: I'll close this one since its available now

The one and only @cb1kenobi configured my open-vsx token in actions so next time we publish this it should do both open-vsx and vscode marketplaces at the same time


Make the genesis_cid field in ChainConfig an Option<Cid>

How about we go ahead and make the genesis_cid field in the ChainConfig an Option<Cid>? The more strongly typed the codebase is, the better.
Originally posted by @LesnyRumcajs in https://github.com/ChainSafe/forest/pull/3243#discussion_r1269119289

I got there before you, @jdjaustin.


takeSnapshot not capturing markers

react-native: https://github.com/expo/react-native/archive/sdk-42.0.0.tar.gz
react: 16.13.1
react-native-maps: 0.28.0
I want to get markers as a part of snapshot. When we use takeSnapshot method all markers are ignored.
Please let me know the possibility of this

Feel to reopen with the issue template fully filled out. fwiw, I can see markers in the snapshots when testing with the example app on iOS.

const snapshot = this.viewRefTest.takeSnapshot({
    format: 'png',   // image formats: 'png', 'jpg' (default: 'png')
    quality: 0.5,    // image quality: 0..1 (only relevant for jpg, default: 1)
    result: 'file'   // result types: 'file', 'base64' (default: 'file')
});

<MapView
    ref={viewRefTest => {
        this.viewRefTest = viewRefTest;
    }}
    showsUserLocation={true}
    followUserLocation={true}
>
</MapView>

@christopherdro try using this code

Follow this issue on github as it needs resolution.

@christopherdro Please reopen this issue.


Add BigInt variants of all Int methods

When testing an adder, I noticed that as soon as I went to larger sizes some of the functions I was using did not have BigInt variants.
For example, I needed to write BigInt variants of these three functions:
MeasureInteger
DrawRandomInt
ApplyXorInPlace

(This would all be moot if Q# was big-int-by-default like python!)

@msoeken @guenp: Thoughts on including this as part of the changes with https://github.com/microsoft/QuantumLibraries/issues/337?

@cgranade I agree to include BigInt-variants for all Int methods as part of the changes with #337. In general, I think that having Q# bit-int-by-default as mentioned by @Strilanc is an interesting idea.

It sounds like there's alignment, then, that #377 is the right place to act on that feedback. Will revise that proposal to incorporate and schedule discussion for the December API review.
@msoeken, @Strilanc: Would either of you be willing to file BigInt-by-default as a suggestion on qsharp-language? I'm happy to capture that in qsharp-language if not.

@cgranade I am not sure what BigInt by default would mean for Q#? there is no default; we have literals for each data type, and no automatic cast or conversions. Hence the data type is whatever you define it to be when you instantiate the value. This is more of a library ask I believe.

It means the literal 1 is a BigInt instead of an Int. And slices would be over big ints instead of over ints. Almost everything would be big ints. Basically you would be renaming Int to Int64 and BigInt to Int, and Int64 would almost never be used. That's what I mean by big-int by default.

The slices or rather support for BigInt ranges is a good point. The concern from my side would be performance and whether the amount of work to fully support that throughout all our tools.

There might be some tricks that one can leverage to address performance issues such as a small integer cache as used in Python's C implementation.

Tagging this as a Status-ChildIssue of https://github.com/microsoft/QuantumLibraries/issues/337.


[UI] Payment Triggers UI Improvements

Related to #

Description
Some style updates for the payments triggers section as discussed
Test plan
Preview the site and check everything looks good and functions well

For sure, added


Does it support text formatting?

Hi!
First, wonderful crate, although documentation could be improved.
My question is the following:
I have a spreadsheet generated in LibreOffice with the number 5 with a custom style that adds extra 0 padding. So when read in LibreOffice it reads as:
00005.
For reference, the format is the following:

I tried the following:
let my_num = my_sheet
                .cell(row, 1)
                .unwrap()
                .value
                .as_str_opt()
                .unwrap();

However it fails. Ok, I guess there i no auto conversion. Of course the following works:
let my_num: u32 = my_sheet
                .cell(row, 1)
                .unwrap()
                .value
                .as_u32_opt()
                .unwrap();

However the problem is that I have no idea how to retrieve the formatting.
Does spreadsheet-ods have a way for me to somewhat easily retrieve the formatting and print the value the way the user seeing it? (i.e. retrieve the fact that there are four 0 of padding so I can print 0005 instead of 5).

That's a tricky question.
You can:
    if let Some(s) = sheet.cellstyle(row, 1) {
        if let Some(style) = wb.cellstyle(s) {
            if let Some(format) = style.value_format() {
                if let Some(format) = wb.number_format(format) {
                   for part in format.parts() {
                       part.attr_def("attr-name", "");
                       // ...
                   }
                }
            }
        }
    }

... and work through those attributes to make something up.
But that's admittedly not very user-friendly.
And now that I tried it, I saw that wb.number_format() is broken.
So, short answer: No, sorry. Doesn't work.
A bit longer answer: I once had half of a solution for this, but I threw it
away, because there is no number formating crate that comes even
close to the requirements for ODS. For dates there is chrono, which
should work, but they added localization only recently, so at the time
there was nothing. And the same again for numbers. For those
I recently wrote format_num_pattern which should be sufficient.
And even then there is all the different attributes for formatting.
Calc can really do a LOT of variations of the theme.
Which means the answer is still: No, sorry.
I will keep this issue open, and after I finished my current project
(which may take a while) I'll read this again, and maybe find the
enthusiasm to start this.
PS: Is it really necessary to use the defined format from the spreadsheet,
or can you just guess an appropriate format for your usecase. In that
case I would try format_num_pattern or something similar, and
chronos formatting for dates.

Thanks for the detailed response!
Indeed my needs are different from spreadsheet-ods.
Whereas I am just fine with 0-padding on the left you need something generic & flexible that works with every possible formatting rule ODS supports.
For the time being I'm managing with just hardcoding it for the specific needs. If I get some time, I'll take a look at your snippet to see if I can get the formatting out.
Cheers

You can always contact me, if you want to start something. At least I can give you a few hints.


RDK-50967 : Add RED state notification

Reason for change: Adding RED state notification in the system plugin
Test Procedure: check the RED state feature
Risks: Low
Priority: P1

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


fix: Align PS with Maya/Nuke

Resolves :

CLICKUP-

FT-

SENTRY-

ZENDESK-

[ ] I have added automatic tests where applicable.
[ ] The PR contains a description of what has been changed.
[ ] The description contains manual test instructions.
[ ] The PR contains update to the release notes.
[ ] The PR contains update to the documentation.

This PR has been tested on :

[ ] Windows.
[X] MacOs.
[ ] Linux.

Changes

Name comment in hook
Function name in hook
Tabs in launcher conf, also fix NKS
Python API constraints in project toml
Remove unused import in module init
Docstring plugins
Use temp path util
Add photoshop prefix to plugin names and classes
add exported path validator in publisher
remove from launcher: TEST_ENVIRONMENT_VARIABLE_WITH_ENV

Also:

Added tabs to NKS launcher conf
Fixed bug in PS javascript on opening task in ftrack.
Commented a possible bug in Maya

Test
Do a full PS ETE test

The ps-include.jsx file from the same folder maybe should be removed if not used.

Think we should keep this as an example on how to extend the JS capabilities with extensions, otherwise will be lost for later

I think we shouldn't put changes to other projects in this PR, better to keep those changes in a separated task. In this regard, I found that nuke publisher tool-config is not fully aligned, is missing the common validator exporter path (If you decide to put other project changes in this PR please add this one as well, if not, please add a task so we can pick it)

Task added: https://dev.ftrackapp.com/#slideEntityId=b7047d2f-5fd3-4bd9-9ee8-dda0cb197b4d&slideEntityType=task&view=tasks&itemId=projects&entityId=dc34b754-79e8-11e3-b4d0-040102b7e101&entityType=show&objectType=show&slideView=

Line 17 signal is never called

It is used on format changed combobox

resource/cep should be filled up or removed. But now is empty.

Indeed has content, updated the README a bit.

IMO self._extensions shouldn't be picked from the tool-config, or should be hardcoded or picked from the plugin_hook.

See comment in files

Line 17 signal is never called

It is used on format changed combobox
I think is not used.

on_format_changed = QtCore.Signal(int)