dt: init at 1.2.3

Description of changes
An initial package for dt (Repo, Website)
I also am the upstream owner
Things done

Built on platform(s)

[x] x86_64-linux
[x] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Please set meta.mainProgram and use hash instead of sha255

Done, thanks for the feedback @Gerg-L !

Done, thanks for the feedback @Gerg-L !

I will build and approve when I can get to my computer

@booniepepper now that https://github.com/NixOS/nixpkgs/pull/247437 is merged, could you remove the zigHook = zigHook.override line?

@doronbehar I rebased and tried just to make sure, but no, I cannot remove the zigHook = zigHook.override line. Just like poop requires it in the PR you linked (https://github.com/NixOS/nixpkgs/pull/247437), dt also only can build with Zig 0.11 and requires the override to build. (Unless there's some other way to configure the version of Zig)

I see. Usually, similarly to e.g gcc9Stdenv and other versions of stdenvs, the natural thing to do is to have a global zig0_11Hook that both packages use. It'd be nice to do that treewide, even in a separate PR.
Also, since there are ~20 packages using zig all togehter in Nixpkgs, I think you should make zig_0_11 the default zig, and override the packages that really have no choice but to use an older zig inside them - use zig0_10Hook inside the package, and not in the callPackage. This way, package maintainers will notice their package uses an old version of zig, and will hopefully ask upstream for latest zig support.
If you are willing to work on that, I'd be happy to help you.

I'm a rookie when it comes to Nix, I probably don't know enough of the basic syntax to get it done yet. @figsoda I've seen you working on Zig related commits, do you have any plans or opinions on changing the Zig hooks?

I'm still a bit unsure how to expose the hooks, whether it should be within the zig package, as a passthrus (zig.hook, zig_0_9.setupHook), a separate attribute (zigHook, zig0_11Hook, zigHook0_9), or be separated into multiple hooks that handle each phase

There's also the option of using zig.setupHook = ./setup-hook.sh, without a passthru. This is the most common option in nixpkgs of them all, just like simply adding cmake to nativeBuildInputs automagically makes mkDerivation use cmake.setupHook's phases to build the package.

There's also the option of using zig.setupHook = ./setup-hook.sh, without a passthru.

This is what I meant by "within the zig package"

There's also the option of using zig.setupHook = ./setup-hook.sh, without a passthru.

This is what I meant by "within the zig package"

That's what I assumed, but I think that defining it in the passthru will not produce the behavior we depict, but I'm not sure.

ah that was supposed to be a different option
to clarify, here are the options:

within the zig package
as passthrus (zig.hook, zig_0_9.setupHook)
as separate attributes (zigHook, zig0_11Hook, zigHook0_9)
be separated into multiple hooks that handle each phase

Could I help capture any of the conversation above on how to package Zig somewhere else...? Some of it's going over my head right now, but should it be in an issue or a wiki or something?

There's also the option of using zig.setupHook = ./setup-hook.sh, without a passthru.

And a useless (and possibly mass) rebuild as a gift. Nope, I'll pass.

Don't forget to squash the commits into one.

I think you should make zig_0_11 the default zig, and override the packages that really have no choice but to use an older zig inside them

#248153


The class_id of bboxes_ex_msgs::msg::BoundingBoxes that is pubbed does not change when I have it inferred with custom data.

I use translation software
When inferring custom data, I added the newly created txt file to class_labels_path in yolox_tensorrt.launch.py, but the class_id to be pubbed is still existing. Do you know of a solution?

Sorry, it solved itself.
Sorry for submitting the issue without looking at the code carefully.
I added a new .hpp file so that yolox_cpp::COCO_CLASSES is defined in coco_names.hpp and it worked.
Is this approach correct?

You are right and I apologize for my lack of maintenance.
On the other hand, the essence of yolox is not the retrieval of class_id (e.g id='person') but the id (e.g id=0), and that information is not included in the input onnx or trt. Therefore, it is only a drawing utility and the boundingbox subscriber should use id.
We are currently experimenting with a revised interface using pluginlib, so please wait a little longer for the fix! 🙂
(ref1 : https://github.com/StrayedCats/detector2d)
(ref2 : https://github.com/StrayedCats/yolox_trt)

Thanks for the reply.
Thank you for making yolox available for ros2, Ar-Ray-code's blog is very informative.
Thank you for sharing the essence of yolox.


Windows 20210330.2 image causes NuGet restore to fail when using a MSBuild.Sdk.Extras project

Description
Using new Windows 20210330.2 image causes NuGet restore to fail when using a MSBuild.Sdk.Extras project. May be caused by using the new MSBuild v<IP_ADDRESS>03
The old Windows image 20210316.1 uses MSBuild v<IP_ADDRESS>03 and this works
Area for Triage:

Scripting and command line, Packages
Question, Bug, or Feature?:

Bug
Virtual environments affected

[ ] Ubuntu 16.04
[ ] Ubuntu 18.04
[ ] Ubuntu 20.04
[ ] macOS 10.15
[ ] macOS 11.0
[ ] Windows Server 2016 R2
[x] Windows Server 2019

Image version

Image version where you are experiencing the issue.
20210330.2
Expected behavior
To restore the MSBuild.Sdk.Extras correcly
Actual behavior
Runtime error describing the MSBuild.Sdk.Extras cannot be found when restoring
Starting: NuGet Restore
==============================================================================
Task         : NuGet
Description  : Restore, pack, or push NuGet packages, or run a NuGet command. Supports NuGet.org and authenticated feeds like Azure Artifacts and MyGet. Uses NuGet.exe and works with .NET Framework apps. For .NET Core and .NET Standard apps, use the .NET Core task.
Version      : 2.179.0
Author       : Microsoft Corporation
Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/package/nuget
==============================================================================
"C:/Program Files (x86)/Microsoft Visual Studio/2019/Enterprise/MSBuild/Current/Bin/msbuild.exe" /version /nologo
SYSTEMVSSCONNECTION exists true
SYSTEMVSSCONNECTION exists true
C:\Windows\system32\chcp.com 65001
Active code page: 65001
Detected NuGet version <IP_ADDRESS>34 / 5.9.0+685acd761481e83715ba76ecddc680f5ed2378b5.685acd761481e83715ba76ecddc680f5ed2378b5
SYSTEMVSSCONNECTION exists true
C:\hostedtoolcache\windows\NuGet\5.9.0\x64\nuget.exe sources Add -NonInteractive -Name NuGetOrg -Source https://api.nuget.org/v3/index.json -ConfigFile D:\a\1\Nuget\tempNuGet_516201.config
Package source with Name: NuGetOrg added successfully.
C:\hostedtoolcache\windows\NuGet\5.9.0\x64\nuget.exe restore D:\a\1\s\[path.to.project].sln -Verbosity Detailed -NonInteractive -ConfigFile D:\a\1\Nuget\tempNuGet_516201.config
D:\a\1\s\[path.to.project].csproj : error : Unable to find package MSBuild.Sdk.Extras. No packages exist with this id in source(s): Microsoft Visual Studio Offline Packages
D:\a\1\s\[path.to.project].csproj : error : C:\Program Files\dotnet\sdk\5.0.201\Sdks\MSBuild.Sdk.Extras\Sdk not found. Check that a recent enough .NET SDK is installed and/or increase the version specified in global.json.
D:\a\1\s\[path.to.project].csproj : error MSB4236: The SDK 'MSBuild.Sdk.Extras/2.0.54' specified could not be found.

NuGet Version: <IP_ADDRESS>34
MSBuild auto-detection: using msbuild version '<IP_ADDRESS>03' from 'C:\Program Files (x86)\Microsoft Visual Studio\2019\Enterprise\MSBuild\Current\bin'. Use option -MSBuildVersion to force nuget to use a specific version of MSBuild.

Repro steps
Use Azure Pipelines to restore a MSBuild.Sdk.Extras project. See the NuGet Restore step fails

Hello, @Maxgamerboy1
Could you please check the workaround - https://github.com/xamarin/XamarinCommunityToolkit/pull/1173 ?

I have applied this change and it appears to have worked! Thanks for the quick response!
Do you know if this will be changed in a future Windows image?

I have applied this change and it appears to have worked! Thanks for the quick response!
Do you know if this will be changed in a future Windows image, so we can remove this workaround?

It can unpredictable happen if you don't have a nuget.config file in your repro -  https://github.com/NuGet/Home/issues/10586#issuecomment-783689013 . Example nuget.config - https://github.com/actions/setup-dotnet/issues/155#issuecomment-761195782

Awesome, I'll look into adding this. I'm happy to close this issue for now. Thanks again.


node: allow to override enable encapsulation on a per-node basis

Introduce the possibility to configure whether we need to enforce encapsulation-related routes and alike on a per-node basis, to allow further flexibility, useful for instance in testing environments.

/test

Conformance Ginkgo hit https://github.com/cilium/cilium/issues/29236 and https://github.com/cilium/cilium/issues/29108. Succeeded after rerun.


Added D3.js courses

Hacktoberfest notes:

due to volume of submissions, we may not be able to review PRs that do not pass tests and do not have informative titles.
please read our contributing guidelines
be sure to check the output of Travis-CI for linter errors
if this is your first open source contribution, make sure it's not your last!

What does this PR do?
Add resources.
For resources
Description
Added resources:

Data Visualization with D3.js: Full Tutorial Course 2019 (https://www.youtube.com/watch?v=_8V5o2UHG0E&t=23737s)

Learn the basics of data visualization with D3 (https://www.codecademy.com/learn/learn-d3)

Why is this valuable (or not)?
It is valuable for whoever would like to learn data visualization with the JavaScript library.
How do we know it's really free?
By accessing the 2 links provided.
For book lists, is it a book? For course lists, is it a course? etc.
They are courses.
Checklist:

[Y] Not a duplicate
[Y] Included author(s) if appropriate
[Y] Lists are in alphabetical order
[N] Needed indications added (PDF, access notes, under construction)

Heads up! Where the pull request template has * [ ] you can replace the space with an x to make it a checked checkbox!

[x] Like this!

It seems you've changed the table of contents from asterisks to hyphens.
It may be nice to change them back to maintain consistency with the rest of the repository.

Some text editors with automatic formating like to do that, try to disable that.
In VisualStudio Code, there is a handy feature to only enable format for the parts of the documents you have changed or turn it off completely while you work on this file.

Heads up! Where the pull request template has * [ ] you can replace the space with an x to make it a checked checkbox!

[x]  Like this!

It seems you've changed the table of contents from asterisks to hyphens.
It may be nice to change them back to maintain consistency with the rest of the repository.

Some text editors with automatic formating like to do that, try to disable that.
In VisualStudio Code, there is a handy feature to only enable format for the parts of the documents you have changed or turn it off completely while you work on this file.

I noticed that and have disabled it.
VS Code (Mac):
Preferences > Search "toc.updateOnSave" in settings > Disabled it > Done!
Anyway, thanks!

Heads up! Where the pull request template has * [ ] you can replace the space with an x to make it a checked checkbox!

[x]   Like this!

It seems you've changed the table of contents from asterisks to hyphens.
It may be nice to change them back to maintain consistency with the rest of the repository.

Some text editors with automatic formating like to do that, try to disable that.
In VisualStudio Code, there is a handy feature to only enable format for the parts of the documents you have changed or turn it off completely while you work on this file.

I noticed that and have disabled it.
VS Code (Mac):
Preferences > Search "toc.updateOnSave" in settings > Disable it > Done!
Anyway, thanks!

You are very welcome!

Be sure to check the output of Travis-CI for linter errors. Alphabetical order and spacing are the most common problems.


event listener & unmount

I receive the following error:
Uncaught TypeError: Failed to execute 'removeEventListener' on 'EventTarget': 2 arguments required, but only 1 present.
I believe it has something to do with this section of code in /es/components/popups/PopupWindow.js

PopupWindow.prototype.componentWillUnmount = function componentWillUnmount() {
this.scLauncher.removeEventListener(this.interceptLauncherClick);
};
PopupWindow.prototype.render = function render() {
var _this2 = this;
var _props = this.props,
isOpen = _props.isOpen,
children = _props.children;

return React.createElement(
  'div',
  { className: 'sc-popup-window', ref: function ref(e) {
      return _this2.emojiPopup = e;
    } },
  React.createElement(
    'div',
    { className: 'sc-popup-window--cointainer ' + (isOpen ? '' : 'closed') },
    React.createElement('input', {
      onChange: this.props.onInputChange,
      className: 'sc-popup-window--search',
      placeholder: 'Search emoji...'
    }),
    children
  )
);

};

I don't know why, but the git editor keeps ignoring my markup... sorry about the formatting...
I receive this error when I click to a page that doesn't have the chat widget running

i ended up putting it on every page of my app to get rid of the error, but it'd be great if you could select which pages it's on

@dharness Can you release latest master to npm? I think that may help.
@tylerehc Sorry and thanks for the report! I'll get back to you more later.

Thanks Heather! I hope this is helpful in some way :) I'm content to have it run on all pages for now, and I love the component!!!

Here is the easy fix:
PopupWindow.prototype.componentWillUnmount = function componentWillUnmount() {
  this.scLauncher.removeEventListener('click', this.interceptLauncherClick);
};`

@tylerehc 1.2.0 was just released, let us know if that doesn't solve your problem
@heatherbooker thanks for staying on top of this!

hey @dharness thanks for the update! I circled back to a situation where I was having this error, and after reinstalling the library, the error no longer occurs! Thank you for maintaining this fun and useful library.


Refactor; remove dupe function definition

Biggest change in this PR is breaking up the indicators.js file into multiple smaller test files that are easier to read and edit and, more importantly, are more tightly focused.
Inadvertently defined a function twice. Hilarity ensued. Fixed that.

Current test status:
Number of specs: 38

144 passing (240.80s)
140 skipped
2 failing

1) Program Indicators table should toggle table when a PI Indicators button is clicked:
expected true to equal false
running firefox
running chrome
AssertionError: expected true to equal false
    at Context.<anonymous> (/home/kwall/Work/TolaActivity/test/js/test/specs/indicators_table.js:51:14)
    at new Promise (<anonymous>)
    at new F (/home/kwall/Work/TolaActivity/test/js/node_modules/core-js/library/modules/_export.js:35:28)


Release checklist: add announcement to docs

It appears that we have a blog section on the docs site, which has two release announcement and a… sample post‽ We should either update our release checklist to add an entry there, or delete it completely.
I seem to be dumb — I can't understand the document structure in https://github.com/cloudfoundry-incubator/kubecf/tree/master/doc/release to figure out which is the actual release checklist (there seems to be two overlapping documents plus a template for the GitHub release text).

I think we should delete the blog section.


Rechunking arrays with empty chunks

It looks like our current rechunk implementation can fail when there are empty chunks present. For example:
import dask.array as da

# Create array with empty chunks
x = da.zeros((7, 24), chunks=((7,), (10, 0, 0, 9, 0, 5)))
# Rechunk array
y = x.rechunk((-1, -1))
print(f'x.shape, x.compute().shape = {x.shape}, {x.compute().shape}')
print(f'y.shape, y.compute().shape = {y.shape}, {y.compute().shape}')

Rechunking shouldn't impact the output shape of y. However, running this example on the current master branch gives:
x.shape, x.compute().shape = (7, 24), (7, 24)
y.shape, y.compute().shape = (7, 24), (7, 10)

I was able to track down the problem to _intersect_1d (in dask/array/rechunk.py) not properly incrementing the chunk index for the "old" (pre-rechunking) chunking scheme in the case where empty chunks are present. This PR adds logic to ensure the chunk index is always incremented, even when there are empty chunks.

[x] Tests added / passed
[x] Passes black dask / flake8 dask

I found visualizing the task graph for y in the above example to be useful. With the changes in this PR, y includes all the non-empty chunks from x (i.e. the (0,0), (0,3), and (0,5) x chunks):

On master, rechunk isn't keeping track of the correct chunk index in x which correspond to non-empty chunks. In this case, y doesn't include the (0,0), (0,3), and (0,5) x chunks, but instead includes the first three chunks (two of which have zero size):

hence why the output shape of y.compute() on master is (7, 10) instead of (7, 24).

thanks @jrbourbeau


FileNotFoundError: [Errno 2] No such file or directory: '/data/checkpoint/config.yaml'

Execute the command：bash docker/2_predict.sh /usr/local/docker/big-lama /usr/local/docker/LaMa_test_images /usr/local/docker/output device=cpu
There is an error：FileNotFoundError: [Errno 2] No such file or directory: '/data/checkpoint/config.yaml'
I don't know where this file comes from, could you please answer it? Thank you！

Could you please show the output of ls  /usr/local/docker/big-lama?

This script maps files on your filesystem to virtual volumes within docker and then spawns a docker container - so '/data/checkpoint/config.yaml' is a path within container. /data/checkpoint is supposed to point to the actual folder you specify as the first argument of 2_predict - /usr/local/docker/big-lama in your case.
The correct value for this argument would be a path to a folder which contains config.yaml and models/best.ckpt files (which together constitute the checkpoint in our pipeline)

I'm closing the issue for now. If you have any other questions, feel free to reopen it or create a new one.


HotChocolate response doesn't specify charset

Describe the bug
responses from the HC graphql endpoint return the content-type header as application/json. according to RFC 7230 content must be in ISO-8859-1/US-ASCII (single octet) when no other encoding is specified.
To Reproduce
Steps to reproduce the behavior:

make a request to a HC endpoint
check the reponse header content-type. it's "application/json"

Expected behavior

make a request to a HC endpoint
check the reponse header content-type. it's "application/json; charset=utf-8"

Additional context
We are seeing encoding issues when querying HotChocolate endpoints with PowerShell which doesn't happen with other .Net GraphQL implementations.
ASP.Net core returns json content with charset=utf-8 by default when using their MVC controllers.

This one is now fixed and we will ship it with 10.5.0-preview.8


Fix compile error for exercises using boost:date_time lib

Overview
This PR is following up on cpp: gigasecond does not compile and a fix for cpp-test-runner which updated the test runner dockerfile to add a missing boost::date_time lib used in "gigasecond" and "meetup" CPP exercises.
PR 38 did the following:

install boost in test runner docker container
add minimal tests for the test runner ensuring the library can be used by the runner

Unfortunately, this didn't fix the problem completely because the existing two exercises that use boost::date_time  require the static versions of boost in their CMakeLists.txt files.
Also related is #491.
Proposed solution
This pull request updates those two exercises by removing the static lib requirement and I expect this will resolve the problem.

Thanks! I think this will do the trick.


Watchers were built in order to provide flexibility as to when a React component is supposed to "react" (pun intended) to certain mutations to the node the component is bound to.
This puts on the developer the burden on deciding which logic is needed for which use-cases. However, in 99% of the use-cases, users will expect a component to react should any of the props from a state tree node they rely on change.
In order to provide an even better DX (less boilerplate and concepts to master), this issue proposes deprecating the watcher architecture and in favor of a simpler useArbor hook API which internally will keep track of prop accesses within the context of a React component, automatically reacting to mutations targeting the tracked props and ignoring mutations on props not being used by the component.
A proof-of-concept was built in this codesandbox.

The most recent version of Arbor now ships "path tracking" which allows one to create tracking scopes anywhere in the state tree.
@arborjs/react leverages path tracking seamlessly in order to compute optimal re-rendering, where only components listening to specific paths within the state tree are re-rendered when mutations affecting these paths are triggered. One can still override the "tracking" logic with a "watcher" function which tells useArbor when it should re-render the component.


[elasticsearch] Create or allow setting a serviceAccount to be used

Hopefully most clusters have something along the lines of restricted for the default service account in a namespace.  By default, with no serviceAccount specified, the default serviceAccount is used.
Two values are required, one for the initContainer, which is already requiring privileged permissions and root access, and one for the es container, which doesn't require the same level.

I'm not following, though there are gaps in my kubernetes knowledge. Do you mean PodSecurityPolicy, instead of service account?

A pod security policy is a cluster level resource.  They can be assigned to pods via (cluster) roles and bindings.  One way to add a role to a pod is through service accounts.  Hopefully most cluster admins have implemented the restriced psp linked above and applied it through a CRB to the default service account (like a default least privilege thing)
Every namespace has a default  service account that is, by default, applied to every pod if a service account is not supplied in the manifest .
Services that use the default namespace service account, like the elasticsearch chart, will auto-inherit the psp assigned to the default service account.
Allowing this chart to set its own service account allows for granular control of security policies throughout a cluster.

It isn't currently possible to specify serviceAccounts on a container level https://github.com/kubernetes/kubernetes/issues/66020 is the issue tracking adding this in to Kubernetes.
#38 is the pull request where this was added for Kibana. The only difference being that we will also want to allow the securityContext to be set on a pod level (like in the Kibana PR) and for both of the containers.
@jordansissel https://kubernetes.io/docs/tasks/configure-pod-container/security-context/#set-the-security-context-for-a-container might clear it up for you a bit.

This was implemented in https://github.com/elastic/helm-charts/pull/265, so I'm closing this issue.


resumable uploads

Hi, react-dropzone support the resumable uploads or uploads by parts/chunks?
or how to add this feature, thanks.

react-dropzone is a client-side library that enables a drag'n'drop interaction with files. It doesn't support any of uploading methods that you should implement yourself via callbacks.


Fix Amberscript Transcription Jobs Hanging

This patch fixes the problem that AmberScript transcription jobs will never finish if the event has been deleted between starting and finishing the transcription.
The code checked if a snapshot exists, skipping the operation and repeating this on the next run again. An existing event will always have at least one snapshot, since one is created at ingest or even earlier if the event was scheduled. If you delete the event, however, there will be no snapshot, but there will obviously also never be any snapshot. That is why the jobs will just wait forever.
How to test this patch

Configure AmberScript
Add a new media (triggering the transcription service)
Delete the event as soon as it finished processing

Your pull request should…

[x] have a concise title
[x] close an accompanying issue if one exists
[x] be against the correct branch
[x] include migration scripts and documentation, if appropriate
[x] pass automated tests
[x] have a clean commit history
[x] have proper commit messages (title and body) for all commits
[x] explain why it needs to be merged into the legacy branch, if it is targeting the legacy branch

If you test this, you may also want to test #6299 at the same time

Thanks for testing this. Merging.


Significant Performance Drop in Model Replication

Firstly, I would like to express my sincere gratitude for your impactful work. I've encountered a challenge while attempting to replicate your work on a V100. The accuracy post-training is significantly lower than what is reported in your paper. For the GQA, the paper indicates an accuracy of 56.1, but my replication only achieved 52.98.
I carefully followed the steps outlined in the readme, starting with the first phase of training and then proceeding to the second phase, which involved training using the LoRA method. After training, the model was merged using the merge_lora function, and the accuracy was subsequently evaluated. At this point, I am uncertain whether the issue lies in the training phase or during the “merge_lora ”stage. Below are the loss details from both stages of training.
the first phase：

During the second phase of the training process, I observed that the loss stabilized around 0.85
I am seeking your guidance to pinpoint where the problem might have occurred in the training process. Additionally, would it be possible for you to share your training logs and detailed information about the model merging process? Such insights would be immensely helpful in resolving the discrepancies I am facing.
Thank you very much for your time and assistance.
Best regards,

this is the first phase settings:
 {
 "_name_or_path": "models/MobileLLaMA-1.4B-Chat",
  "architectures": [
    "LlamaForCausalLM"
  ],
  "bos_token_id": 1,
  "eos_token_id": 2,
  "freeze_mm_mlp_adapter": false,
  "hidden_act": "silu",
  "hidden_size": 2048,
  "image_aspect_ratio": "square",
  "image_grid_pinpoints": null,
  "initializer_range": 0.02,
  "intermediate_size": 5632,
  "max_position_embeddings": 2048,
  "max_sequence_length": 2048,
  "mm_hidden_size": 1024,
  "mm_projector_type": "ldpnet",
  "mm_use_im_patch_token": false,
  "mm_use_im_start_end": false,
  "mm_vision_select_feature": "patch",
  "mm_vision_select_layer": -2,
  "mm_vision_tower": "models/clip-vit-large-patch14-336",
  "model_type": "mobilevlm",
  "num_attention_heads": 16,
  "num_hidden_layers": 24,
  "num_key_value_heads": 16,
  "pad_token_id": 0,
  "pretraining_tp": 1,
  "rms_norm_eps": 1e-06,
  "rope_scaling": null,
  "rope_theta": 10000.0,
  "tie_word_embeddings": false,
  "torch_dtype": "float32",
  "transformers_version": "4.33.1",
  "tune_mm_mlp_adapter": true,
  "use_cache": true,
  "use_mm_proj": true,
  "vision_tower_type": "clip",
  "vocab_size": 32000
}

this is the second phase settings:
{
  "_name_or_path": "mobilevlm/mobilevlm1.7b_20240115_223212/mobilevlm-2.finetune-lora",
  "architectures": [
    "MobileLlamaForCausalLM"
  ],
  "bos_token_id": 1,
  "eos_token_id": 2,
  "freeze_mm_mlp_adapter": false,
  "hidden_act": "silu",
  "hidden_size": 2048,
  "image_aspect_ratio": "pad",
  "image_grid_pinpoints": null,
  "initializer_range": 0.02,
  "intermediate_size": 5632,
  "max_position_embeddings": 2048,
  "max_sequence_length": 2048,
  "mm_hidden_size": 1024,
  "mm_projector_type": "ldpnet",
  "mm_use_im_patch_token": false,
  "mm_use_im_start_end": false,
  "mm_vision_select_feature": "patch",
  "mm_vision_select_layer": -2,
  "mm_vision_tower": "models/clip-vit-large-patch14-336",
  "model_type": "mobilevlm",
  "num_attention_heads": 16,
  "num_hidden_layers": 24,
  "num_key_value_heads": 16,
  "pad_token_id": 0,
  "pretraining_tp": 1,
  "rms_norm_eps": 1e-06,
  "rope_scaling": null,
  "rope_theta": 10000.0,
  "tie_word_embeddings": false,
  "torch_dtype": "float16",
  "transformers_version": "4.33.1",
  "tune_mm_mlp_adapter": false,
  "use_cache": true,
  "use_mm_proj": true,
  "vision_tower_type": "clip",
  "vocab_size": 32000
}

After completing the second phase of training, I used the merge_lora function in scripts/mergelora.py provided by the authors to further merge the trained parameters based on the MobileVLM-1.7B model。
#scripts/mergelora.py
model_base = 'models/MobileVLM-1.7B'
model_path = 'mobilevlm1.7b_20240115_223212/mobilevlm-2.finetune-lora'
save_path = 'mobilevlm1.7b_20240115_223212/mobilevlm-recurrent-v2'
merge_lora(model_base , model_path , save_path)

# scripts/mergelora.py
model_base = 'models/MobileVLM-1.7B'     # should change this path to 'models/MobileLLaMA-1.4B-Chat'
model_path = 'mobilevlm1.7b_20240115_223212/mobilevlm-2.finetune-lora'
save_path = 'mobilevlm1.7b_20240115_223212/mobilevlm-recurrent-v2'
merge_lora(model_base , model_path , save_path)

The model_base in scripts/mergelora.py is the chat-bot language model you used, I notice that you use models/MobileLLaMA-1.4B-Chat in the first phase setting.
Please change model_base to correct path and evaluate your merged-model again.

According to the author's suggestion, I reutilized the model 'models/MobileLLaMA-1.4B-Chat' as the model_base for the merge_lora process, and then conducted an evaluation again; the resulting GQA evaluation score remained at 52.98

It seems that GQA is evaluated on cache result. Remove the original result and evaluate again see whether the same value. It should have some fluctuation at least.

I greatly appreciate the help from the authors. Indeed, it turns out that I was using cached results. Now, the accuracy for GQA aligns perfectly!


Fix the browser html code

Version info

Milligram:
Other (e.g. normalize.css, node.js, npm, bower, browser, operating system) (if applicable):
Browser : Chrome
Test case

Steps to reproduce
Just Visit the milligram.io page

Expected behavior
$ bower install milligram

Actual behavior

Hello,
I checked the website with different web browsers and the errors as shown above do not appear, is the issue resolved?

Do you want behaviour like this

if yes I can fix this assign this issue to me

Do you want this behavior:

If yes, I can fix this.
Just assign this issue to me.

This happens when you visit a site for the first time with uBlock Origin turned on.
Browser console:

Links:

https://cdn.carbonads.com/carbon.js?zoneid=1673&serve=C6AILKT&placement=milligramgithubio (this one you can't follow through with UBO on)
https://browser.sentry-cdn.com/5.5.0/bundle.min.js

Site screenshot:

Workarounds:

turn ad blocking off for this site;
soft-refresh (F5) the page multiple times (this also works for some reason).

I guess there's a load/race condition between these scripts and the main script of the site which fixes code blocks after loading but fails for some reason.


add hint messages to use boolean and/or

Thank you for the review comments. I have added the test scenarios as mentioned and committed the changes. Regarding the 'Scope: covering all operand types', as discussed, we will consider this later depending on the need to provide hint message for various operand types.


What about svgz?

Compression can reduce the size ~2x times.

I'm going to say no on this for three main reasons.

Web servers will do the gzipping already - so there are no performance gains there.
I wasn't able to open any of the .svgz files directly in the browser.
I'd like for the code to be readable on GitHub.


Deterministic serialisation for cross binary communication

Prior to this commit, serialisation used the type_id of the types that it serialised and the type_ids assigned to types can vary depending on the program being compiled. This makes it impossible for cross binary serialisation even if the same pony version and types are used.
This commit changes things so that serialisation now uses a determnistically assigned serialise_id instead of the type_id. This new serialise_id is guaranteed to be the same for the same pony version and type (see details for caveats).
The changes include:

adding a size_t size serialise_id to pony_type_t
hashes the reach_type_t->name to generate either a 63 bit or 31 bit serialise_id depending on target platform
the hash for the serialise_id uses a key that is based on the blake2 encoding of the pony version, the target data model, and the endianness (i.e. pony-0.58.7-ac1c51b69-lp64-le). This is to ensure that only programs with the same key will have the same serialise_id to prevent silent data corruption during deserialisation due to subtle differences in platforms. This means that cross binary serialisation will only work for binaries of the same bit width (32 bit vs 64 bit), data model (ilp32, lp64, or llp64), and endianness (big endian or little endian)
serialisation now uses the serialise_id instead of the type_id
deserialisation now uses the serialise_id to look up the type_id to get the entry in the descriptor table instead of looking up the entry in the descriptor table directly based on type_id

yes, this is a ressurrection of https://github.com/ponylang/ponyc/pull/2949

Hi @dipinhora,
The changelog - added label was added to this pull request; all PRs with a changelog label need to have release notes included as part of the PR. If you haven't added release notes already, please do.
Release notes are added by creating a uniquely named file in the .release-notes directory. We suggest you call the file 4567.md to match the number of this pull request.
The basic format of the release notes (using markdown) should be:
## Title

End user description of changes, why it's important,
problems it solves etc.

If a breaking change, make sure to include 1 or more
examples what code would look like prior to this change
and how to update it to work after this change.

Thanks.

What happens in the case of hash collisions with ids? That appears to be a possibility, yes?

yes, hash collisions are possible.. no special handling is done for those in the current implementation but debug builds should fail with an assertion on startup due to (https://github.com/ponylang/ponyc/pull/4567/files#diff-8791fb0991855d735f325e9ce5c0adf9bb9f71331cc032c8e38d913979483c55R85-R97):
bool ponyint_serialise_setup(pony_type_t** table, size_t table_size,
  desc_offset_lookup_fn desc_table_offset_lookup)
{
#ifndef PONY_NDEBUG
  for(uint32_t i = 0; i < table_size; i++)
  {
    if(table[i] != NULL)
    {
      pony_assert(table[i]->id == i);
      pony_assert(desc_table_offset_lookup(table[i]->serialise_id) == i);
    }
  }
#endif

Personally I find it strange to take on additional maintenance burden and implementation complexity for an undocumented/non-public "feature" (which kind of makes it not a feature?).
I'm assuming this is making some use cases possible which weren't possible before, so I'd prefer to document those and mark them as a feature, even if it means we need to include caveats about edge cases that break things.
Without that in place, then somebody could easily revert this PR later because "the docs already say that cross binary serialization doesn't work, so it's safe to simplify this code away", and they would be justified in doing so.

I'm curious to hear what @SeanTAllen thinks on this point, because it sounded like he was the one suggesting to remove the release notes.

I view this as an improvement on the existing functionality. It isn't "you can use this" level where it should be public, but it is an improvement on the functionality that exists and a path towards someone eventually bringing it fully home.

Personally I find it strange to take on additional maintenance burden and implementation complexity for an undocumented/non-public "feature" (which kind of makes it not a feature?).

fair point.. this doesn't have to be merged if it's not considered worth the tradeoff you mention..

I'm assuming this is making some use cases possible which weren't possible before, so I'd prefer to document those and mark them as a feature, even if it means we need to include caveats about edge cases that break things.

nope.. not at all (assuming folks are following the recommended practices as mentioned in the serialise package docs).. at least not as the changes in this PR currently stand..

Without that in place, then somebody could easily revert this PR later because "the docs already say that cross binary serialization doesn't work, so it's safe to simplify this code away", and they would be justified in doing so.

yep..


Interface for steady-state detection

Let's merge it in here, before we submit it to nipype: https://github.com/poldracklab/mriqc/blob/9667137e42dbb1c4c70172d4e18ba358c5761c86/mriqc/utils/misc.py#L46
Comes from: https://github.com/poldracklab/fmriprep/issues/360
One nice addition to this interface is plotting the average signal across time.

I think for the FMRIPREP issue, what we'll want is something like the existing nipype interface, and to accept n_volumes_to_discard as a parameter to CompCor, so that we don't have to make another copy of the file to run (see https://github.com/poldracklab/fmriprep/pull/577#issuecomment-312009856).
So I'm not entirely sure how to match goals here.

Done here https://github.com/poldracklab/fmriprep/pull/603, closing.


[Windows] Install-SQLPowerShellTools.ps1 failing on 404s from download.microsoft.com

Description
Install-SQLPowerShellTools.ps1 is failing due to 404s from download.microsoft.com.  I suspect that the URLs to download from have been rotated.
Broken URL: https://download.microsoft.com/download/8/7/2/872BCECA-C849-4B40-8EBE-21D48CDF1456/ENU/x64/SQLSysClrTypes.msi
Area for Triage:
Databases
Question, Bug, or Feature?:
Bug
Virtual environments affected

[ ] Ubuntu 16.04
[ ] Ubuntu 18.04
[ ] Ubuntu 20.04
[ ] macOS 10.15
[ ] macOS 11.0
[ ] Windows Server 2016 R2
[X] Windows Server 2019

Expected behavior
I expect my windows2019.json provisions to complete without error! :smile:
Actual behavior
My windows2019.json provision is failing during Install-SQLPowerShellTools.ps1. ☹
2020-10-28T00:46:52.9004085Z ==> managedimage: Provisioning with powershell script: D:\a\1\s\virtual-environments\images\win/scripts/Installers/Install-SQLPowerShellTools.ps1
2020-10-28T00:46:59.9149546Z     managedimage: Downloading SQLSysClrTypes.msi...
2020-10-28T00:46:59.9227522Z     managedimage: Downloading package from: https://download.microsoft.com/download/8/7/2/872BCECA-C849-4B40-8EBE-21D48CDF1456/ENU/x64/SQLSysClrTypes.msi to path C:\Users\packer\AppData\Local\Temp\SQLSysClrTypes.msi .
2020-10-28T00:46:59.9240019Z     managedimage: There is an error during package downloading:
2020-10-28T00:46:59.9241367Z     managedimage:  Exception calling "DownloadFile" with "2" argument(s): "The remote server returned an error: (404) Not Found."
2020-10-28T00:46:59.9242218Z     managedimage: Waiting 30 seconds before retrying. Retries left: 19
2020-10-28T00:47:29.9702077Z     managedimage: Downloading package from: https://download.microsoft.com/download/8/7/2/872BCECA-C849-4B40-8EBE-21D48CDF1456/ENU/x64/SQLSysClrTypes.msi to path C:\Users\packer\AppData\Local\Temp\SQLSysClrTypes.msi .
2020-10-28T00:47:29.9705055Z     managedimage: There is an error during package downloading:
2020-10-28T00:47:29.9714236Z     managedimage:  Exception calling "DownloadFile" with "2" argument(s): "The remote server returned an error: (404) Not Found."
2020-10-28T00:47:29.9721033Z     managedimage: Waiting 30 seconds before retrying. Retries left: 18
[...snip...]
2020-10-28T00:56:00.3357817Z     managedimage: Downloading package from: https://download.microsoft.com/download/8/7/2/872BCECA-C849-4B40-8EBE-21D48CDF1456/ENU/x64/SQLSysClrTypes.msi to path C:\Users\packer\AppData\Local\Temp\SQLSysClrTypes.msi .
2020-10-28T00:56:00.3359927Z     managedimage: There is an error during package downloading:
2020-10-28T00:56:00.3371454Z     managedimage:  Exception calling "DownloadFile" with "2" argument(s): "The remote server returned an error: (404) Not Found."
2020-10-28T00:56:00.3385514Z     managedimage: Waiting 30 seconds before retrying. Retries left: 1
2020-10-28T00:56:29.8770349Z     managedimage: Downloading package from: https://download.microsoft.com/download/8/7/2/872BCECA-C849-4B40-8EBE-21D48CDF1456/ENU/x64/SQLSysClrTypes.msi to path C:\Users\packer\AppData\Local\Temp\SQLSysClrTypes.msi .
2020-10-28T00:56:29.8773497Z     managedimage: There is an error during package downloading:
2020-10-28T00:56:29.8793380Z     managedimage:  Exception calling "DownloadFile" with "2" argument(s): "The remote server returned an error: (404) Not Found."
2020-10-28T00:56:29.8823203Z     managedimage: File can't be downloaded. Please try later or check that file exists by url: https://download.microsoft.com/download/8/7/2/872BCECA-C849-4B40-8EBE-21D48CDF1456/ENU/x64/SQLSysClrTypes.msi
2020-10-28T00:56:29.8832877Z ==> managedimage: Provisioning step had errors: Running the cleanup provisioner, if present...

Repro steps
Run a Windows Server 2019 provisioning run, and observe the error.  Appears to be consistent repro, and I can repro the 404 on my own machine as well.

Looks like we caught the same in nightly build: https://github.visualstudio.com/virtual-environments/_build/results?buildId=86752&view=logs&j=f3215316-5135-57cb-291b-0903c8ad86f7&t=363259d8-0f0a-5fb4-2a1f-9a5fbec03447&s=f70a4ffd-001e-530b-b6de-8d07b3f9171f

PR to update base url: https://github.com/actions/virtual-environments/pull/1921


Limit pagination in YT gem

I am using
videos = Yt::Collections::Videos.new
videos.where(q: 'star war', safe_search: 'none')

but i can't find any variable to limit pagination like my search has 500+ videos and i just want first 100.
videos.where(q: 'star war', safe_search: 'none', max_results: 10)

it will just limit the per page request. Can you please update documentation if there is any way.

Hello
videos = Yt::Collections::Videos.new
videos.where(q: 'star war', safe_search: 'none').first(10)

This will only make 1 request. Is that what you need? About documentation, we should work on it. Thank you

Hello
videos = Yt::Collections::Videos.new
videos.where(q: 'star war', safe_search: 'none').first(10)

This will only make 1 request. Is that what you need? About documentation, we should work on it. Thank you

Yes exactly I was looking for that and found help in closed issues. But I think it will be great if you put this in documentation.
Thank You!!!


Typo fix in docs

Coverage remained the same at 95.219% when pulling eed1f392740acee955543d8dacf48b9677f599d3 on gsiou:doc-typo into b90afea58d76f0183c930c9f6b041044e0596835 on Vincit:master.

Coverage remained the same at 95.219% when pulling eed1f392740acee955543d8dacf48b9677f599d3 on gsiou:doc-typo into b90afea58d76f0183c930c9f6b041044e0596835 on Vincit:master.

Coverage remained the same at 95.219% when pulling eed1f392740acee955543d8dacf48b9677f599d3 on gsiou:doc-typo into b90afea58d76f0183c930c9f6b041044e0596835 on Vincit:master.

Coverage remained the same at 95.219% when pulling eed1f392740acee955543d8dacf48b9677f599d3 on gsiou:doc-typo into b90afea58d76f0183c930c9f6b041044e0596835 on Vincit:master.

Thanks!


Restart =D

Hey, @BedirT
What was the last week you updated and started to add?
I have some free time now, and missed coding, I will try to complete. Please contact me via WhatsApp or email ✌🏼

Oh well well 😂 the king has returned I see.
I will talk about the details in the text. I am happy that you are willing to grab the torch once again :)


Add _data/participants.yaml

I guess we recognizing everybody by their twitter handle
Put in their name here
Also put in their crypto address. (With 2-way proof, e.g. link to a tweet confirming it, and link to an ENS or other reversible attestation that this is their twitter account)
Update the token gating file with these addresses (for import for collabland in Su Squares Discord)

Done at https://github.com/community-service/hour.gg/commit/79ed5b71b63117d2440d0fa00f55f8cb0e896e32


Interested to implement the idea of clustering and new feature (image search along with text)

@ManuelFay

Instead of doing cosine similiarity between every patch and every token embedding, if we performed clustering for the patches embeddings then we can calculate the cosine similarity for every cluster and for every token. Then for every token we can consider top3 cluster then move into that cluster embeddings.

If the user given any image as input then we can use vision encoder which we used in the offline phase . So, that we get the embeddings of that image right. Then we need to apply linear projection for getting down to the 128 to match the image patch embedding dimensions. As mentioned above, we need to calculate the cosine similairty for the user input image embedding with the cluster embedding . Then we need to select top 3 clusters and then calculate the cosine similarity between user input image embedding with that selected cluster embeddings. The main usage of clustering is to reduce the millions of calculation (lets us think that we have million image patches).

Yes that concept exists - it's called token pooling and it's implemented in this library!

@ManuelFay what about the image search feature along with the text?


Appium Crosswalk

Hi,
it is not working for me when using Appium. Any idea how I could make it work without changing the driver? Any help would be appreciated :)

Hi @dioxmio - Did you mean to log this on the main cordova-plugin-crosswalk-webview repo?


Fix race condition in mesh config update

git cherry-pick d314d4b71a3dba93c12b4748797390af7102cae1
git cherry-pick e5db42ae909939b281c742e785b7947eb95dda59
Fixes #32073

Hi @xichengliudui. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

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.

ok
/close


Update logitech-options.rb

Update to 6.35.142
If there’s a checkbox you can’t complete for any reason, that's okay, just explain in detail why you weren’t able to do so.
After making all changes to the cask:

[ ] brew cask audit --download {{cask_file}} is error-free.
[ ] brew cask style --fix {{cask_file}} reports no offenses.
[ ] The commit message includes the cask’s name and version.

Additionally, if adding a new cask:

[ ] Named the cask according to the token reference.
[ ] brew cask install {{cask_file}} worked successfully.
[ ] brew cask uninstall {{cask_file}} worked successfully.
[ ] Checked there are no open pull requests for the same cask.
[ ] Checked that the cask was not already refused in closed issues.

@blackmetal23 Please fill the template. It will walk you through what is wrong.


Autorest Regen Preview to 3.0.0-alpha.20230608.1 by Dapeng Zhang from refs/pull/3470/merge

Triggered from https://github.com/Azure/autorest.csharp/pull/3470

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.Verticals.AgriFood.Farming


rustdesk-server: 1.1.11-1 -> 1.1.12

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for rustdesk-server is: RustDesk Server Program
meta.homepage for rustdesk-server is: https://github.com/rustdesk/rustdesk-server
meta.changelog for rustdesk-server is: https://github.com/rustdesk/rustdesk-server/releases/tag/1.1.11
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.1.11 with grep in /nix/store/9201bc9gdghw5w7mb66dqrzl4jd3rshz-rustdesk-server-1.1.11
found 1.1.11 in filename of file in /nix/store/9201bc9gdghw5w7mb66dqrzl4jd3rshz-rustdesk-server-1.1.11

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

rustdesk-server

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/9201bc9gdghw5w7mb66dqrzl4jd3rshz-rustdesk-server-1.1.11 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A rustdesk-server https://github.com/r-ryantm/nixpkgs/archive/995167a5428ff968603925440ec6666fb062acc4.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/995167a5428ff968603925440ec6666fb062acc4#rustdesk-server

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/9201bc9gdghw5w7mb66dqrzl4jd3rshz-rustdesk-server-1.1.11
ls -la /nix/store/9201bc9gdghw5w7mb66dqrzl4jd3rshz-rustdesk-server-1.1.11/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

rustdesk-server

Maintainer pings
cc @gaelreyrol @tjni for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for rustdesk-server is: RustDesk Server Program
meta.homepage for rustdesk-server is: https://github.com/rustdesk/rustdesk-server
meta.changelog for rustdesk-server is: https://github.com/rustdesk/rustdesk-server/releases/tag/1.1.12
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.1.12 with grep in /nix/store/16wq74bjzvh7crjz1n1xj3in9rr3c3gm-rustdesk-server-1.1.12
found 1.1.12 in filename of file in /nix/store/16wq74bjzvh7crjz1n1xj3in9rr3c3gm-rustdesk-server-1.1.12

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

rustdesk-server

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/16wq74bjzvh7crjz1n1xj3in9rr3c3gm-rustdesk-server-1.1.12 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A rustdesk-server https://github.com/r-ryantm/nixpkgs/archive/58a8067afbb0a25e973a737dfe12b8fd371ed90a.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/58a8067afbb0a25e973a737dfe12b8fd371ed90a#rustdesk-server

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/16wq74bjzvh7crjz1n1xj3in9rr3c3gm-rustdesk-server-1.1.12
ls -la /nix/store/16wq74bjzvh7crjz1n1xj3in9rr3c3gm-rustdesk-server-1.1.12/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 1 package built:

rustdesk-server

Maintainer pings
cc @gaelreyrol @tjni for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.


[Masthead Search Typeahead] - link is not opened when using keyboard navigation

[Masthead Search Typeahead links do not open when they are selected by keyboard navigation]:
We are using the Masthead component with Search and custom typeahead API. When the user enters a search text, the typeahead with relevant results is shown. If the user clicks on one of the result under a labelled section, they will be redirected to the link that the result holds.
Steps:

open the page https://www.ibm.com/docs/en
type "aix" into masthead searchbar
wait for typeahead to appear
click on the first item - "AIX" in the "Product documentation landing pages" section
you will land on the product page of AIX, based on the URL given in the typeahead item
This is the correct behaviour

Describe the issue:
The problem
Now, try to do the same with keyboard navigation:
Steps:

open the page https://www.ibm.com/docs/en
type "aix" into masthead searchbar
wait for typeahead to appear
navigate with keyboard arrow on the first item - "AIX" in the "Product documentation landing pages" section and hit ENTER
it will ignore the link  in the item  - will not redirect there, and will execute search instead
This is incorrect

What did you expect to happen? What happened instead?
I expect that the user can navigate to an item in the typeahead with a keyboard, and they will be redirected to the link fo that item after hitting ENTER.

Component(s) impacted? Please include screenshots, or code and relevant URLs.
https://www.ibm.com/standards/carbon/react/?path=/docs/components-masthead--default

What browser(s)are you working in?
FF, Chrome, Safari, Edge

What version of Carbon for IBM.com are you using?
"@carbon/ibmdotcom-react": "1.23.1",
"@carbon/ibmdotcom-styles": "1.23.0",

What application/website do you work on?
IBM Documentation (Knowledge Center team)

Please provide any additional notes:
*Priority
Medium Priority = upcoming release but is not pressing
https://user-images.githubusercontent.com/33804471/132516459-60d730a9-229f-4817-bfb5-17f4ba038f83.mp4

Thank you

For clarification:
when using mouse to click "AIX" under "Product documentation landing pages" in Masthead search then the user is correctly re-directed to the landing page
when using keyboard and hitting enter/return on "AIX" under "Product documentation landing pages" in Masthead search then the user is incorrectly re-directed to the search page

@ariellalgilmore exactly, thank you

@Daynssjal I believe this issue can be resolved at the app level, but I just need a bit more info to be sure. can you provide a reduced test case in Code Sandbox showing how you're implementing the search suggestion event listeners?

Hi @emyarod , we are using Carbon for IBM.com customTypeaheadApi, just like in the documentation
https://www.ibm.com/standards/carbon/react/?path=/docs/components-masthead--default
This is the implementation
 <Masthead
                navigation={mastheadLinks}
                platform={{
                  name: kcGlobals.translation['masthead.IBMDocumentation'],
                  url: `${window.location.origin}/${contextPath}/${language}`,
                }}
                hasProfile={true}
                hasSearch={true}
                placeHolderText={placeholder}
                searchOpenOnload={searchbarOpen}
                searchNoRedirect
                customTypeaheadApi={docsTypeahead}
                multiSection
                initialSearchTerm={searchQuery ? searchQuery : null}
              />

const docsTypeahead = async (query) => {
    const data = await getMastheadTypeahead(query, language);
    if (!data.hints) data.hints = [];

    data.hints = data.hints.slice(0, 6);
    const searchResults = [
      {
        items: data.hints.map((hint) => Object.assign({}, { name: hint })),
      },
      {
        title: kcGlobals.translation['search.typeaheadLandingPages'],
        items: data.products
          ? data.products.map((p) => {
              return {
                name: p.name,
                href: `${window.location.origin}/${contextPath}/${language}/${p.href}`,
              };
            })
          : [],
      },
    ];
    const i = 1;
    return searchResults;
  };

data from the search API:
{products: [{name: "AIX", href: "ssw_aix"}, {name: "aix72", href: "aix72"},…],…}
hints: ["aix", "aix or", "aix or linux", "aix and", "aix systems", "aix linux"]
0: "aix"
1: "aix or"
2: "aix or linux"
3: "aix and"
4: "aix systems"
5: "aix linux"
products: [{name: "AIX", href: "ssw_aix"}, {name: "aix72", href: "aix72"},…]
0: {name: "AIX", href: "ssw_aix"}
1: {name: "aix72", href: "aix72"}
2: {name: "COBOL for AIX", href: "SS6SGM"}
3: {name: "XL C for AIX", href: "SSGH2K"}
4: {name: "XL C/C++ for AIX", href: "SSGH3R"}
5: {name: "XL Fortran for AIX", href: "SSGH4D"}

If we click on one of the "products" items, it will correctly take us to the href of that product item.
If we want to achieve the same but with tab navigation, it will incorrectly take us to the search page, not to the href of that product item.

@Daynssjal I can't see where you are handling the typeahead events to fetch the search result data (getMastheadTypeahead maybe?)
would you be able to provide that for some more context? is it similar to what's implemented in this example? https://codesandbox.io/s/angry-shannon-sw961

@emyarod hi, yes it is the getMatheadTypeahead function:
const getMastheadTypeahead = async (query, lang, categoryies = '', limit = 6) => {
  try {
    if (query.trim().length < 3) {
      return [];
    }
    const res = await abortableFetch(
      SERVICES.TYPEAHEAD,
      buildPath(
        `/api/v1/suggest?query=${encodeURIComponent(
          query.trim()
        )}&lang=${lang}&categories=${categoryies}&limit=${limit}`
      )
    );
    const data = await res.json();
    return data;
  } catch (err) {
    return [];
  }
};

But I do not think this is something you would need to solve this bug. The typeahead was created by Kenny for our project. The typeahead is setup the way we were instructed, also based on the documentation. The search results that have a "href" should follow the href, as they do onclick event - we just need the same behaviour for when the user get to that result via Tab button and hits enter.

@emyarod  hi, yes - onSearchNoRedirect.
useEffect(() => {
    document.addEventListener('onMegaMenuToggle', (e) => {});

    // search icon clicked to open search input
    document.addEventListener('onOpenSearch', () => {});

    // close icon clicked to close search input
    document.addEventListener('onSearchCloseClicked', () => {});

    const searchClicked = (e) => {
      if (
        e.target.children &&
        e.target.children[0] &&
        e.target.children[0].getAttribute('data-href')
      ) {
        window.location = e.target.children[0].getAttribute('data-href');
      } else {
        window.location = `/${contextPath}/${language}/search/${encodeURIComponent(searchText)}${
          getSearchScope(window.product) ? `?scope=${getSearchScope(window.product)}` : ''
        }${getSearchBookTag(window.searchTag) ? `&tag=${getSearchBookTag(window.searchTag)}` : ''}`;
      }
    };
    // search icon clicked to perform search
    document.addEventListener('onSearchNoRedirect', searchClicked);

    // search input onchange
    document.addEventListener('onSearchValueChanged', (e) => {
      searchText = e.detail.value;
    });
  }, []);


Failover metrics scope improve

What changed?

Provide detail failover metrics with domain tags

How did you test it?

Potential risks

Release notes

Documentation Changes

Pull Request Test Coverage Report for Build 11633ff8-db79-4ddc-b34b-029ef657b680

6 of 10 (60.0%)  changed or added relevant lines in 2 files are covered.
40 unchanged lines in 14 files lost coverage.
Overall coverage increased (+0.005%) to 56.439%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

common/domain/failover_watcher.go
2
4
50.0%

service/history/failover/coordinator.go
4
6
66.67%

Files with Coverage Reduction
New Missed Lines
%

common/domain/failover_watcher.go
1
61.67%

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

client/history/client.go
2
41.25%

client/history/metricClient.go
2
47.05%

common/persistence/historyManager.go
2
66.33%

common/task/weightedRoundRobinTaskScheduler.go
2
89.64%

common/types/mapper/thrift/shared.go
2
64.98%

service/history/execution/mutable_state_builder.go
2
69.86%

service/history/handler.go
2
48.15%

service/matching/taskListManager.go
2
74.09%

Totals

Change from base Build 3963ea27-5204-4d7b-ad45-4bcec71b5360:
0.005%

Covered Lines:
78715

Relevant Lines:
139470

💛  - Coveralls


[CSL 50] Replace Request w/ Needle

Updates:

Replace request with needle

Pros:

Needle works somewhat similar to request

Cons:

Needle throws an error when there are no files passed when making a multipart request

Implemented a kind of ugly workaround for the error handling

Multipart requests are a little more verbose

Tests updated. All tests pass.

looks great to me, @jjl014. Nice work on this :)


fix: link for edit action

The Edit link on page https://preset.dev/docs/concepts/actions/#actions is broken. It should be /edit-files/ instead of /edit/

Thanks!


What is the Application function?

My question is, where is that function defined? Where is it documented? Is it built it?
I can't find anything on it, but that could be because I don't know where to look.

Oh. It's JavaScript Apple Script.

Correct. Also worth mentioning my module osa2 is used to bridge between nodejs and Apple's Javascript Automation runtime. @jbis9051

@wtfaremyinitials Yep, saw that. That is a very cool project, could be used for so much! Thanks for the help.


display whitespace indenting in minimap

can sent pull request

:+1:

@ryanoasis check my fork.

@mipmip looks like your fork has your latest contributions?
hopefully @severin-lemaignan is still maintaining this repo

Yes it has. I tried to contact @severin-lemaignan but I think he's very busy.

Merged! Thanks for the contribution!


Config file problem

The code doesn't contain config.cfg file

Please ensure that you are following the proper installation steps mentioned and also satisfy the requirements for it
Python 3.9.12
Visual Studio Build Tools (install C++ Desktop Development)
And follow the installation steps in readme
Config.cfg file will be generated from your side of you satisfy this requirements and follow the steps of installation


Support for load_balancing: priority with autodiscovery: true

Hey guys.
Disclaimer, I didn't dig deep into the codebase to understand this constrain yet nor am Cassandra expert, so maybe this is possible only in my imagination :)
Would it be possible to add support for it?
If so, would you accept PR adding support for it?
Anything I should be aware of before I start ? :)

Actually this might be what I am looking for: https://github.com/lexhide/xandra/pull/186
Long story short, we have distributed clusters across geographical locations, and I want to connect to all, but have a preference of a co-located one.
Currently I can pass a list of fqdns, and it connects nicely and is able to autodiscover peer nodes, but then I am stuck with load balancing random, that makes the experience sub-optimal in normal operation (increased latency). I would like to keep connection to all for redundancy (in case of regional failure), but have a preference of the first node in nodes in normal operation (if available).

Closing this now that we have custom load-balancing policies. We'll implement something like DCAwareRoundRobin.


Make checks optional

Adds a new feature to skipChecks allowing mongodb atlas connections to occur with out authn or server version checks.

Codecov Report

Merging #89 (5bf7d8d) into issue-84-get-server-info-login (bd6d68d) will increase coverage by 0.06%.
The diff coverage is 75.00%.

@@                        Coverage Diff                         @@
##           issue-84-get-server-info-login      #89      +/-   ##
==================================================================
+ Coverage                           87.67%   87.73%   +0.06%
==================================================================
  Files                                   8        8
  Lines                                 779      783       +4
==================================================================
+ Hits                                  683      687       +4
  Misses                                 96       96

Impacted Files
Coverage Δ

lib/authn.js
79.34% <70.58%> (+0.22%)
:arrow_up:

lib/config.js
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Before doing a full review here -- we should see if this solves the problem we're trying to solve.

I have tried with a connection string at the free tier from mongodb atlas and it worked @mattcollier has the connection string that failed. So we need confirmation from him then?


Targeted Sentiment returns blank

Hello, I have no issues with other endpoints like sentiment analysis and keyword extraction, but when I loop through it shows as blank.
results.each do |data|
@data = "#{data}"
@targets = "stranger"
ts = AlchemyAPI.search(:targeted_sentiment_analysis, text: "#{@data}", targets: "#{@targets}")
puts ts
end

See #31


Update MD file on new branch 2017-23-22_07-11-28

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestCon.md
:white_check_mark:Succeeded

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


Align hex grid preview to v10 hexes

Closes #95

This seems to be working nicely for me :)

Thank you for this PR!


Adjust link colors for most themes

In this PR:

Lightened link colors that were too dark
Darkened colors that were too light
Changed link colors for Oshkosh and Superior to colors that can be found on their respective home pages
All link colors meet WCAG AA contrast standards

Note: I did not change the black link color for UW-Milwaukee. Looking at uwm.edu, they use the color black without underline for all their links, so the current MyUW theme would be in keeping with their home page. I'd rather put the onus on someone from UWM to change it if they feel it's necessary.
Example screenshots

This is really pretty cool, making theming more meaningful.

Changes Unknown when pulling 39aa9633c90174f9a792e85005db88e7599f53cf on thevoiceofzeke:themes into ** on UW-Madison-DoIT:master**.


New colonial-split cultures

It is kinda unsatisfactory to see that, while English colonies in the New World will fall under "Yankee" or "Dixie" culture, different from the "British" one, colonizing them as a German or as an Indian would make them of your culture, thus making the de-colonization process more difficult and causing cultural cores going weird.
Therefore, I am currently doing a way to convert new colonial cultures, three for each Vic2 culture group but natives (one for Americas, one for Oceania, one for Africa).
I'll also edit the cultural union decision to avoid weird situations where to form Poland-Lithuania you would have to conquer (e.g.) Couronian Fiji.
EDIT: Vanilla post-colonial cultures remain untouched. Yankees and Dixies still exist.

YES!
How were you thinking of doing this exactly?  I had an idea of editing the cultures files from the code and creating the cultures that were needed, but I'm not really ready to implement that yet (I need to get a bit happier with the parser yet).  If I know what you're doing there might be a "hybrid" solution that would be easier than a pure code or pure data files solution.
Also for the cultural unions it might be best for there to be a decision for each continent, or at least limit the existing one to the Old World (the only vanilla "cultural union" I can think of in the New World is Gran Columbia) and/or continuous territories.

I acted the same way as creating a new culture and mapping it thru the use
of regions.
About Gran Colombia, it has already a special decision, so I can restrict
the generic cultural unions to the Old World.
Il 04 Set 2017 20:42, "kratostatic"<EMAIL_ADDRESS>ha scritto:

YES!
How were you thinking of doing this exactly? I had an idea of editing the
cultures files from the code and creating the cultures that were needed,
but I'm not really ready to implement that yet (I need to get a bit happier
with the parser yet). If I know what you're doing there might be a "hybrid"
solution that would be easier than a pure code or pure data files solution.
Also for the cultural unions it might be best for there to be a decision
for each continent, or at least limit the existing one to the Old World
(the only vanilla "cultural union" I can think of in the New World is Gran
Columbia) and/or continuous territories.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Idhrendur/paradoxGameConverters/issues/345#issuecomment-327013918,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXab-JEjJHiBvQGebK7B6aNUHYig1jr6ks5sfEShgaJpZM4PLNws
.

Implemented in 1.0H.


Update README.md

Add ->toArray method to code examples, when usung EventData object. Also added avaliable plugins, because their names are different from original Full Calendar Docs, for example: 'resourceTimeline'.

ty!


F-string formatting - unmatched '[' in OllamaInstructorClient

The following exception is being thrown when using chat_completion from OllamaInstructorClient and OllamaInstructorAsyncClient (ollama_instructor\ollama_instructor_client.py)

I was able to resolve this by changing the F-Srtring to use double quotes

after change:

Fix is released with newest version v0.5.2.
Thanks for the issue! I will close it now


chore: upload deliverables for sprint 8

pls. Merge just a branch with the deliverables' data- thx

Ill have to update the dependencies in the planning documents.
Also:

it should be planning-documents.pdf instaed of planning-document.pdf
it should be .jpg not .png


Follow-Up Calendar Events

Hey there,
is there a way to create an Event which will be added every week the same time in the schedule? I could not find it but maybe I am just blind. If this is not possible, it would be a cool thing to add.
Thanks

No, that's not supported.
Dunno when and if ever will be tho.


added my name and github profile link to index.html

Updated index.html

Thnx for contributing @umang-sinha  make sure you follow me and star this repo so that you don't miss the updates and contribute during Hacktoberfest 2020.
Also resolve the conflicts.


Applications dependencies cleanup

Merged, thanks!

ty, just another small PR that is safe to merge #6587


fix!: ignore nusery rules

👍 This should help us catch regressions if anyone turned on nursery rules.


Ability to stop task and auto_recrawl

If there any right way to stop task from executing again, in proccess callback? I explain why i need this. I handle actual base of another site. And when i find on page that object already sold i am not interested in it anymore and i want stop auto crawl and mark in my base, that object already sold.

submit the task again with auto_update=False, and force_update=True
change task['scheduler']['auto_update']=False in your callback(self, response, task) when processing the task.

Did not get this. There is some change in api, today is not auto_recrawl but auto_update? I searched in sources, but there no any auto_update. I need return task var or it is enough just change that var?

auto_recrawl and force_update
ignore the second method, it wouldn't work after restart.

What you mean exactly "it wouldn't work after restart"? Second method was exactly what i search. If i find in callback that i have 404 code or sold type, i stop auto_recrawl of object.

then submit response.orig_url with self.crawl, changing the value in task object may not updated into database, yes, it can affect next turn of scheduler, but after restart, it will auto_recrawl agian

Let's do this again, submitting task with same url again is the only way to stop automatic recrawl? So to stop crawl we must do request again?

And changing task is not working at all right?

is the designed mechanism
should work with current implement, but not guarantee while further development.

Can you answer please about additional fetch query and last part of message, pls. I have around 200k tasks now, and it is very major for me to know if fetch will be fired again or not.

parameters in function call have higher priority, so self.crawl(response.orig_url, callback=self.parse_object, auto_recrawl=False) will override auto_recrawl=True in @config
but you need use force_update=True at same time, as the task is in active status.
I added a new parameter in a80e1745897e169f6f57223929f88cc81bfc6e6d (not released), that you can cancel the task immediately override that last one fetch.


[Flaky Test] test_solution_consistency

These tests were found to be flaky (intermittently failing on main or failed in a PR with unrelated changes). As per [the docs](https://github.com/apache/tvm/blob/main/docs/contribute/ci.rst#handling-flaky-failures, these failures will be disabled in a PR that references this issue until the test owners can fix the source of the flakiness.
Test(s)

tests/python/unittest/test_arith_solve_linear_equations.py::test_solution_consistency

Jenkins Links

https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/main/2759/tests
https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/main/2758/tests
https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/main/2764/tests
https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/main/2764/tests

This one fails on ARM/i386 nodes only, #10591 is possibly the culprit. Oddly python tests/scripts/ci.py i386 --tests tests/python/unittest/test_arith_solve_linear_equations.py::test_solution_consistency does not reproduce this failure after several attempts

duped in #11458


curl_easy_escape  errror

Hope you can provide some advise to my problem below.
I am having problem loading the mod_auth_openidc.so
Below is the error when I try to load Apache.
httpd: Syntax error on line 175 of /usr/local/httpd/conf/httpd.conf: Cannot load modules/mod_auth_openidc.so into server: /usr/local/httpd/modules/mod_auth_openidc.so: undefined symbol: curl_easy_escape
This is what I use to build and it seem to work with no errors.
./configure --with-apxs2=/usr/local/httpd/bin/apxs CURL_CFLAGS=/usr/local/bin CURL_LIBS=/usr/local/lib
/usr/local/bin  is where the curl binary that I just build today.
/usr/local/lib is where the curl library that I just build today.
Thank you.
Y

You should most probably use CURL_CFLAGS=-I/opt/local/include and -L/opt/local/lib -lcurl assuming that libcurl cannot be found by pkg-config automatically.

OS= Redhat 7.1 EN
These are my build options:
./configure --with-apxs2=/usr/local/httpd/bin/apxs CURL_CFLAGS=/usr/local/curl/include CURL_LIBS=/usr/local/curl/lib/ OPENSSL_CFLAGS=/usr/local/openssl/include OPENSSL_LIBS=/usr/local/openssl/lib APR_CFLAGS=/usr/local/lib64/apr/include APR_LIBS=/usr/local/lib64/apr/lib JANSSON_CFLAGS=/usr/local/jansson/include JANSSON_LIBS=/usr/local/jansson/lib
These are what it come back with:
checking for pkg-config... /bin/pkg-config
checking pkg-config is at least version 0.9.0... yes
checking for CURL... yes
checking for OPENSSL... yes
checking for APR... yes
checking for JANSSON... yes
checking for PCRE... yes
checking for HIREDIS... no
configure: creating ./config.status
config.status: creating Makefile
These are some of the warning while running make:  But not error at the end of the make.
gcc: warning: /usr/local/openssl/include: linker input file unused because linking not done
gcc: warning: /usr/local/curl/include: linker input file unused because linking not done
gcc: warning: /usr/local/jansson/include: linker input file unused because linking not done
Thanks,
Y

There's a binary package for Centos7 on the Releases page. There's also a Dockerfile that shows how to build the package on Redhat/CentOS 7 here: https://github.com/zandbelt/mod-auth-openidc-dist/blob/master/centos/7/build/Dockerfile You should not need to provide the environment variables manually if you've installed the required "-devel" packages

I am using Redhat 7.1 OS distribution but all my Apache, APR, Openssl was manually build with the latest  release.  It is bit customize to meet our Web server requirements.
Any ideas where to start looking for the problem?
Cannot load modules/mod_auth_openidc.so into server: /usr/local/httpd/modules/mod_auth_openidc.so: undefined symbol: curl_easy_escape
The config.log seems to indicate that it find all the nescessary path and pkg for CURL
config.log:  $ ./configure --with-apxs2=/usr/local/httpd/bin/apxs CURL_CFLAGS=/usr/local/curl/include CURL_LIBS=/usr/local/curl/lib/ OPENSSL_CFLAGS=/usr/local/openssl/include OPEN          SSL_LIBS=/usr/local/openssl/lib APR_CFLAGS=/usr/local/lib64/apr/include APR_LIBS=/usr/local/lib64/apr/lib JANSSON_CFLAGS=/usr/local/jansson/include JANSSON_LIBS=/usr/local/jansson          /lib
config.log:PATH: /usr/local/curl
config.log:PATH: /usr/local/curl/lib
config.log:PATH: /usr/local/curl/include
config.log:ac_cv_env_CURL_CFLAGS_value=/usr/local/curl/include
config.log:ac_cv_env_CURL_LIBS_value=/usr/local/curl/lib/
config.log:pkg_cv_CURL_CFLAGS=/usr/local/curl/include
config.log:pkg_cv_CURL_LIBS=/usr/local/curl/lib/
config.log:CURL_CFLAGS='/usr/local/curl/include'
config.log:CURL_LIBS='/usr/local/curl/lib/'
Thanks,
Y

You're passing directory paths in the environment variables, but they should be linker flags that consist of a combination of compiler specific options as in: CURL_CFLAGS="-I/opt/local/include" and "CURL_LIBS=-L/opt/local/lib -lcurl" etc.

The error is coming from ../src/util.c  file.
However, I am not quite sure what need to be done to fix the error.
Regarding the linker flags:  Is this what I need to use?
./configure --with-apxs2=/usr/local/httpd/bin/apxs CURL_CFLAGS="-I/usr/local/curl/include" "CURL_LIBS=-L/usr/local/curl/lib -lcurl  ??
Thanks,
Y

yes, with a closing double-quote after -lcurl

I have modified the build options as follow: and all the Linker warning gone away.
./configure --with-apxs2=/usr/local/httpd/bin/apxs CURL_CFLAGS="-I/usr/local/curl/include" "CURL_LIBS=-L/usr/local/curl/lib -lcurl" OPENSSL_CFLAGS="-I/usr/local/openssl/include" "OPENSSL_LIBS=-L/usr/local/openssl/lib/openssl -lopenssl" APR_CFLAGS=-I/usr/local/lib64/apr/include "APR_LIBS=-L/usr/local/lib64/apr/lib -lapr" JANSSON_CFLAGS=-I/usr/local/jansson/include "JANSSON_LIBS=-L/usr/local/jansson/lib -ljansson"
But I got a new errors:  Here is the last line:
/session.lo src/authz.lo src/util.lo src/config.lo src/crypto.lo src/proto.lo src/oauth.lo src/cache/lock.lo src/cache/shm.lo src/cache/memcache.lo src/cache/file.lo src/mod_auth_openidc.lo
/bin/ld: cannot find -lopenssl
collect2: error: ld returned 1 exit status
apxs:Error: Command failed with rc=65536
.
make: *** [src/mod_auth_openidc.la] Error 1

I guess your path is "/usr/local/openssl/lib" instead of "/usr/local/openssl/lib/openssl"? I'd also add "-lcrypto", so: OPENSSL_LIBS="-L/usr/local/openssl/lib -lopenssl -lcrypto"

Here are some of the extract data from the config.log file.
It seems not able to find information for PCRE and APXS2_OPTS.
Any suggestion?
collect2: error: ld returned 1 exit status
apxs:Error: Command failed with rc=65536
-----------
Core tests.
-----------
configure:2367: checking for PCRE
configure:2374: $PKG_CONFIG --exists --print-errors "libpcre"
configure:2377: $? = 0
configure:2391: $PKG_CONFIG --exists --print-errors "libpcre"
configure:2394: $? = 0
----------------
Cache variables.
----------------
ac_cv_env_APR_CFLAGS_set=set
ac_cv_env_APR_CFLAGS_value=-I/usr/local/lib64/apr/include
ac_cv_env_APR_LIBS_set=set
ac_cv_env_APR_LIBS_value='-L/usr/local/lib64/apr/lib -lapr'
ac_cv_env_APXS2_OPTS_set=
ac_cv_env_APXS2_OPTS_value=
ac_cv_env_CURL_CFLAGS_set=set
ac_cv_env_CURL_CFLAGS_value=-I/usr/local/curl/include
ac_cv_env_CURL_LIBS_set=set
ac_cv_env_CURL_LIBS_value='-L/usr/local/curl/lib -lcurl'
ac_cv_env_HIREDIS_CFLAGS_set=
ac_cv_env_HIREDIS_CFLAGS_value=
ac_cv_env_HIREDIS_LIBS_set=
ac_cv_env_HIREDIS_LIBS_value=
ac_cv_env_JANSSON_CFLAGS_set=set
ac_cv_env_JANSSON_CFLAGS_value=-I/usr/local/jansson/include
ac_cv_env_JANSSON_LIBS_set=set
ac_cv_env_JANSSON_LIBS_value='-L/usr/local/jansson/lib -ljansson'
ac_cv_env_OPENSSL_CFLAGS_set=set
ac_cv_env_OPENSSL_CFLAGS_value=-I/usr/local/openssl/include/openssl
ac_cv_env_OPENSSL_LIBS_set=set
ac_cv_env_OPENSSL_LIBS_value='-L/usr/local/openssl/lib -lopenssl'
ac_cv_env_PCRE_CFLAGS_set=
ac_cv_env_PCRE_CFLAGS_value=
ac_cv_env_PCRE_LIBS_set=
ac_cv_env_PCRE_LIBS_value=
pkg_cv_APR_CFLAGS=-I/usr/local/lib64/apr/include
pkg_cv_APR_LIBS='-L/usr/local/lib64/apr/lib -lapr'
pkg_cv_CURL_CFLAGS=-I/usr/local/curl/include
pkg_cv_CURL_LIBS='-L/usr/local/curl/lib -lcurl'
pkg_cv_JANSSON_CFLAGS=-I/usr/local/jansson/include
pkg_cv_JANSSON_LIBS='-L/usr/local/jansson/lib -ljansson'
pkg_cv_OPENSSL_CFLAGS=-I/usr/local/openssl/include/openssl
pkg_cv_OPENSSL_LIBS='-L/usr/local/openssl/lib -lopenssl'
pkg_cv_PCRE_CFLAGS=' '
pkg_cv_PCRE_LIBS='-lpcre  '
-----------------
Output variables.
-----------------
APR_CFLAGS='-I/usr/local/lib64/apr/include'
APR_LIBS='-L/usr/local/lib64/apr/lib -lapr'
APXS2='/usr/local/httpd/bin/apxs'
APXS2_OPTS=''
CURL_CFLAGS='-I/usr/local/curl/include'
CURL_LIBS='-L/usr/local/curl/lib -lcurl'
JANSSON_CFLAGS='-I/usr/local/jansson/include'
JANSSON_LIBS='-L/usr/local/jansson/lib -ljansson'
OPENSSL_CFLAGS='-I/usr/local/openssl/include/openssl'
OPENSSL_LIBS='-L/usr/local/openssl/lib -lopenssl'

My suggestion would be to use the prebuilt binary from the releases page and make sure that you include the path to the special built libraries in to the LD_LIBRARY_PATH before the paths to the regular versions of those libraries.


Add overflowing shl and shr to u64 module in stdlib

Addresses #168

@willyrgf - is this PR ready for review? Or is there anything still outstanding?

@willyrgf - is this PR ready for review? Or is there anything still outstanding?

Sorry @bobbinth, apparently I wrongly marked the conversation as resolved. Please can you check here: https://github.com/maticnetwork/miden/pull/191#discussion_r868118363?

@willyrgf - ah! you are correct. They are indeed not equivalent. But now that I think about it, we probably want to define the behavior like I have it in the comment. Specifically:
overflowing_shr(a,b) should output (c, d) such that c = a >> b (i.e., regular right shift), and d = a << (64 - b). The stack diagram would look like this:
[b, a, ...] -> [d, c, ...]

@willyrgf - is this PR ready for review? Or is there anything still outstanding?

@bobbinth - all points have been fixed, I think we are ready for a review.

I like the solution! Well done! I left just a couple of minor comments inline.

@bobbinth - Nice! All these comments are solved.


Investigate ingesting Open Street Maps

closes #13

@mindrones I need to write a short script to automate the downloading of the zipped RDF files. Where in the dir structure should I put this script? My guess is:
ontologies
  OSM
    queries
      stress
        ...
      explore
        ...
    results
      stress
        ...
      explore
        ...
    bin
      fetch.js
...

I'd say:
/ontologies
	OSM/
		queries/
      		stress/
			explore/
		results/
      		stress/
			explore/
	.../
/src
	bin
		OSM/
			fetch.js
		.../
	utils/
		...

We could have a data dir within each ontology dir so that e.g. src/bin/OSM/fetch.js would put the fetched data in ontologies/OSM/data/*:
I'd say:

/ontologies
OSM/
data/
queries/
stress/
explore/
results/
stress/
explore/
.../
/src
bin/
OSM/
fetch.js
.../
utils/
...

I've left it for now, as the data won't be stored on the repo either way, and the script in question maintains the flexibility of being able to output the files to any given directory by supplying the path as an argument.

On second thought, I’d add closes #17 to the commit message so we can get rid of #17 too!

closes #17

closes #17


Make room be a pivot data point

Allow to search tenants and receipts by room

completed in https://github.com/jcuna/roomanize/commit/399535ee81160a6bb440a37dc830c8bce258c61e


Run v0.1.0 of the conformance tests

This allows us to evolve the conformance tests without immediately breaking Spin CI.

I think that might become difficult to manage for a few reasons:

We will likely want to bump the major version of the tests when we introduce a new schema of test description (i.e., the json file that describes the test)
Other runtimes will evolve separately and at different paces, and I'm not sure we want the version of Spin CLI to be the thing that ties them together.

I personally think, at least for now, it makes sense for the tests to evolve as they do and not tie them to any external versioning scheme. In the future, we can revisit this question when the rate of change of the tests slows and we have a better picture of how versioning across the entire Spin ecosystem might work.


#152 Helpfile clarifications

Closes #152

@Constantino-Carreto-Romero Can you take a look at this to see that everything is in order before merging, please?

@jorpppp I already checked and left some comments

@Constantino-Carreto-Romero I agree with your suggestions from your review about the overidoptions. Can you review and make sure that the helpfile here and the helpfile in EventStudyPackage coincide? That may mean some edits both here and in the EventStudyPackage repo.
I am not sure what your suggestion is for the redundant imputation Case. Did we clarify this already in eventstudypackage?

@jorpppp

@Constantino-Carreto-Romero I agree with your suggestions from your review about the overidoptions.

Should I implement the changes in my review above?

Can you review and make sure that the helpfile here and the helpfile in EventStudyPackage coincide? That may mean some edits both here and in the EventStudyPackage repo.

I've found some slight differences between the help file in the paper draft and the help file here. The paper draft is more updated. I think I should first propagate the changes from my review above to the paper draft, then propagate the recent edits in the paper draft to the help file here.

I am not sure what your suggestion is for the redundant imputation Case. Did we clarify this already in eventstudypackage?

We already edited it in the paper draft as in the attached image. I think that description is clearer than the one here in the help file. Should I change the impute description here in accordance to the description in the paper draft?

@jorpppp

@Constantino-Carreto-Romero I agree with your suggestions from your review about the overidoptions.

Should I implement the changes in my review above?

Can you review and make sure that the helpfile here and the helpfile in EventStudyPackage coincide? That may mean some edits both here and in the EventStudyPackage repo.

I've found some slight differences between the help file in the paper draft and the help file here. The paper draft is more updated. I think I should first propagate the changes from my review above to the paper draft, then propagate the recent edits in the paper draft to the help file here.

I am not sure what your suggestion is for the redundant imputation Case. Did we clarify this already in eventstudypackage?

We already edited it in the paper draft as in the attached image. I think that description is clearer than the one here in the help file. Should I change the impute description here in accordance to the description in the paper draft?

Agreed, thank you @Constantino-Carreto-Romero. Let's proceed as you suggest then: Let's propagate these minor changes to the paper, then bring the paper's helpfile here.
@jmshapir FYI

Are we ready to merge @Constantino-Carreto-Romero ?

@jorpppp

in https://github.com/JMSLab/xtevent/pull/158/commits/b7bf272abaf0b05e026799febc2eeb155be85f1b I implemented the changes from my review avobe
in https://github.com/JMSLab/xtevent/pull/158/commits/980fbf2c1c3cb82e33fb50672db759dfd8ece291 I made minor edits to the help files, so now they coincide with the commands' description in the stata paper draft.


Implement Antolope signing transaction for payment with UCW in the demo app

https://www.figma.com/design/Aedi8DrZwwParsirLortGt/Tonomy-Demo-Website?node-id=1732-3638&t=8JVlATcP0VYCBLRp-4
Acceptance criteria:

[ ] small design update to consider the WalletConnect/Antelope in-view popup
[ ] user can use UCW to sign Antelope transaction
[ ] works on Mobile and QR code (desktop)

https://www.figma.com/design/Aedi8DrZwwParsirLortGt/Tonomy-Demo-Website?node-id=1736-2640&t=p1YcAF7UDhb3fiF7-4


Python 3.10: TypeError: add() got an unexpected keyword argument 'context'

Running the example i'm getting this error (tested with Python3.9 and Python3.10)
 $ rp2_us -m fifo -o output -p crypto_example_ crypto_example.config crypto_example.ods
INFO: Country: us
INFO: Accounting Method: fifo
INFO: Configuration file: crypto_example.config
INFO: Input file: crypto_example.ods
INFO: Processing BTC
ERROR: Fatal exception occurred:
Traceback (most recent call last):
  File "/home/rndstr/venv/lib/python3.9/site-packages/rp2/rp2_main.py", line 93, in _rp2_main_internal
    input_data: InputData = parse_ods(configuration=configuration, asset=asset, input_file_handle=input_file_handle)
  File "/home/rndstr/venv/lib/python3.9/site-packages/rp2/ods_parser.py", line 128, in parse_ods
    _create_and_process_transaction(
  File "/home/rndstr/venv/lib/python3.9/site-packages/rp2/ods_parser.py", line 168, in _create_and_process_transaction
    transaction: AbstractTransaction = _create_transaction(configuration, current_table_type, internal_id, row_values)
  File "/home/rndstr/venv/lib/python3.9/site-packages/rp2/ods_parser.py", line 277, in _create_transaction
    transaction = InTransaction(**argument_pack)
  File "/home/rndstr/venv/lib/python3.9/site-packages/rp2/in_transaction.py", line 52, in __init__
    super().__init__(configuration, timestamp, asset, transaction_type, spot_price, internal_id, unique_id, notes)
  File "/home/rndstr/venv/lib/python3.9/site-packages/rp2/abstract_transaction.py", line 41, in __init__
    self.__spot_price: RP2Decimal = configuration.type_check_positive_decimal("spot_price", spot_price)
  File "/home/rndstr/venv/lib/python3.9/site-packages/rp2/configuration.py", line 276, in type_check_positive_decimal
    if result < ZERO:
  File "/home/rndstr/venv/lib/python3.9/site-packages/rp2/rp2_decimal.py", line 59, in __lt__
    return not self.__ge__(other)
  File "/home/rndstr/venv/lib/python3.9/site-packages/rp2/rp2_decimal.py", line 48, in __ge__
    return (self - other).quantize(CRYPTO_DECIMAL_MASK).__ge__(ZERO)
  File "/home/rndstr/venv/lib/python3.9/site-packages/rp2/rp2_decimal.py", line 69, in __sub__
    return RP2Decimal(Decimal.__sub__(self, other))
  File "/usr/lib/python3.9/_pydecimal.py", line 1257, in __sub__
    return self.__add__(other.copy_negate(), context=context)
TypeError: __add__() got an unexpected keyword argument 'context'
INFO: Log file: ./log/rp2_2022_04_10_15_52_55_045185.log
INFO: Generated output directory: output
INFO: Done

Which version of RP2 are you using? If not the latest, could you upgrade and try again? Also make sure the example ODS and config are the latest version.

I set everything up yesterday and was using 0.9.29 installed with pip.

It looks like you are overriding Decimal.__add__() but do not provide the context param which is in the Decimal class in Python3.9/3.10:
https://github.com/python/cpython/blob/89697f7374ea947ebe8e36131e2d3e21fff6fa1d/Lib/_pydecimal.py#L1157
and then being called virtually
https://github.com/python/cpython/blob/f33e2c87a83917b5139d97fd8ef7cba7223ebef5/Lib/_pydecimal.py#L1257
not sure why it would work for others--if that's the case--though.

Interesting: haven't seen that before. The crypto_example.ods test is ran as part of continuous integration at every checkin on Python 3.7, 3.8, 3.9 and on Linux, Windows and Mac. A few more questions:

what OS and OS version are you using?
did you install Python as described in the docs or did you use a Python that came with the OS?

The unit tests are failing in the PR:
Traceback (most recent call last):
  File "/home/runner/work/rp2/rp2/src/rp2/rp2_main.py", line 93, in _rp2_main_internal
    input_data: InputData = parse_ods(configuration=configuration, asset=asset, input_file_handle=input_file_handle)
  File "/home/runner/work/rp2/rp2/src/rp2/ods_parser.py", line 128, in parse_ods
    _create_and_process_transaction(
  File "/home/runner/work/rp2/rp2/src/rp2/ods_parser.py", line 168, in _create_and_process_transaction
    transaction: AbstractTransaction = _create_transaction(configuration, current_table_type, internal_id, row_values)
  File "/home/runner/work/rp2/rp2/src/rp2/ods_parser.py", line 277, in _create_transaction
    transaction = InTransaction(**argument_pack)
  File "/home/runner/work/rp2/rp2/src/rp2/in_transaction.py", line 81, in __init__
    self.__fiat_in_with_fee = self.__fiat_in_no_fee + self.__fiat_fee
  File "/home/runner/work/rp2/rp2/src/rp2/rp2_decimal.py", line 64, in __add__
    return RP2Decimal(Decimal.__add__(self, other, context))
TypeError: expected 1 argument, got 2

The context is probably something that got added in Python 3.10 (unit tests are running on 3.7, 3.8 and 3.9).

Odd, I see it in 3.7 as well https://github.com/python/cpython/blob/3.7/Lib/_pydecimal.py#L1157

Is there some sort of non-strict mode in Python that would allow extra arguments in this particular instance even though it's not part of the signature?

Not sure... but it looks like RP2 is not working yet on Python 3.10. If you find a way to make the PR work in a backward compatible way, I'll accept it. For now the (not ideal) workaround is to use Python 3.9.

I tried with Python 3.10 on my personal machine and I couldn't repro this problem. I also added Python 3.10 to continuous integration on Windows, Mac and Linux and it all works well (see https://github.com/eprbell/rp2/commit/0d7b01a7649d8f50c65c28c85e323e92ad5ce2d5). I think the issue you are seeing might be due to some specific problem with your Python installation.

Here is continuous integration running on Python 3.10:

Windows: https://github.com/eprbell/rp2/runs/5993615909?check_suite_focus=true
Mac: https://github.com/eprbell/rp2/runs/5993616027?check_suite_focus=true
Linux: https://github.com/eprbell/rp2/runs/5993615670?check_suite_focus=true

You may want to try uninstalling python and reinstalling it.

I don't think there is anything else actionable on this issue. OK to close?

FWIW, I think it has to due with the fact that there are two possible backends for the decimal module, see cpython/Lib/decimal.py.
I confirmed that on Arch Linux it's loading from _pydecimal. I assume that elsewhere it's loading from _decimal instead.

Found it!
> python
Python 3.11.8 (main, Feb 12 2024, 14:50:05) [GCC 13.2.1 20230801] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from _decimal import *
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ImportError: libmpdec.so.4: cannot open shared object file: No such file or directory

> pacman -Ss mpdec
extra/mpdecimal 4.0.0-2
    Package for correctly-rounded arbitrary precision decimal floating point
    arithmetic
> sudo pacman -Syu mpdecimal
...
> python
Python 3.11.8 (main, Feb 12 2024, 14:50:05) [GCC 13.2.1 20230801] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from _decimal import *
>>>

...and rp2 appears to be working after this process as well.
TLDR: mpdecimal is a required dependency of rp2. I guess on Ubuntu it's either installed by default with the OS or with the distribution of Python. On Arch it must be installed separately.

Fantastic! Thanks for the good detective work :-)

I filed a bug against Python; maybe this could be fixed upstream.

Good idea. If you also want to submit a PR to add a dependency for RP2 I'll approve it.

This should now be resolved (see #108).