Lost state?

Does helm-tiller maintain state somewhere? On a tiller stop/start, I lose my releases. I can see my release on my cluster but helm release does not return anything. I can also see releases from other namespaces but not in my own. Could it be because I didn't stop tiller using the stop command?
Here is an example of the steps to replicate this issue.

start tiller 👉 helm tiller start
install release 👉 helm install
list releases, release shows 👉 helm list
computer restart
start tiller
list releases, no releases

What am I doing wrong?

If I run kubectl get pods I see the pods for my release.

I tried duplicating the issue without success. How could this disconnect happen? How could helm/tiller not know of a release it has done in the past?

cannot really comment on the issue, not sure why it not work for you


Update commands.md

[ci skip]

I thought mentioning this just below the licensed cache command description would be appropriate.

This PR was automatically closed after I deleted the branch in may fork using the site. The branch is created with another commit. I don't see how I can reopen this PR. So opening a new one...


Pydocstyle lints

closes #21

these tests should fail because theres no docstrings
tox is not running the lint and type envs


Sanitize paths in emitted JSON paths

Didn't want to apply it to all emitted strings, since the fix of simply replacing \ with / probably isn't ideal (for example it would change properly escaped sequences).

I already normalize the full_path variable in mdebug.cpp, which has some additional logic which I seem to remember fixed a problem with Windows paths not being handled properly by fs::path on Linux (grr), so I'll probably break that logic out into its own function use that on relative_path too instead. Does that sound okay or is there another problem?

There were some more problems, so I'll just merge this PR as-is then clean up the remaining problems later.


Sign in / Sign up forms have overflowing fields

Note that I have the Lastpass chrome extension, which might be causing the problem?

I didn’t notice that last night working on those fields, but I will check again.

I doubled checked and the form looks good on my end.

This is fixed.


postfix exporter prometheus queries

Hello,
I see there are a good number of metrics getting generated.
but I was wondering if someone has used those in PromQL to utilize for record and alert based on that.
Please share some ideas/files if you have created .
thanks

Hello, I'll also be grateful if someone shares the alerting rules.

+1


Make proofread boxes available in edit pages

From Debra (email 12 Sept 2023):

if possible I would like the little blue checkboxes that I click when I proofread a record visible in my "edit" view. (They can also be any version of a "yes/no" box or fill-in, as long as I can mark a record proofread as I go through the source.)

Screenshot from chant edit page in OldCantus

According to Debra, these boxes (or something like them) are available in the Taxonomy Manager (I don't have permission to view, so have not seen what they look like there myself), but not in the edit pages, making it a two-window task to proofread/edit and then mark as proofread.

For completeness:
These boxes appear in the proofread pages:

They do not appear in the edit page (which are a good deal easier to access):

They do also appear in the Taxonomy manager like so:

I believe Debra has also requested in the past some sort of way of checking off that all the other information (like genre, position, etc.) about a chant was proofread, while we are on the topic of little boxes...


[TW#14556] Is lwip netconn api supported?

Hi,
I am trying to use lwip's netconn api without any luck, I didn't see any examples either... Is the API supported/tested on the esp32?
Best regards,
Roberto

The net module in the ESP32 version of NodeMCU uses the netconn API these days, you might find some useful ideas/examples there.

Thank you, that was very helpful!


⚠ ClamAV has degraded performance

In 049e59f, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 194 ms

Resolved: ClamAV performance has improved in 589eec2 after 6 minutes.


Fixed broken echo syntax for shitpost join message

Pull requests directly into the master branch should not be accepted. Instead, pull into the nixbot-dev branch which will then get pulled into the master branch.

Since it's just a edit of a string, it is not an issue.


Ability to map route directly to template to render

Possible with the new $http->get('/')->template('index.html') syntax.


In feed.xml, why site.url always is http://localhost:4000?

I followed the Usage to Installation & Development.

Set url: http://domain.com in _config.yml,
Run bundle exec jekyll serve -w
Use chrome open http://<IP_ADDRESS>:4000/feed.xml
I think the link should be http://domain.com/, but it always is http://localhost:4000, whatever is set any value to url: in _config.yml.

@ArchimboldiMao In the production version it will be your url. Example: http://chalk.nielsenramon.com/feed.xml

Thanks for your guide.
Now I know why and how to do, and I should run this: JEKYLL_ENV=production bundle exec jekyll serve

I wasted over 2 evenings to found out that NGINX reverse proxy is not responsible for redirection to localhost.
So, JEKYLL_ENV-production jekyll server
Thanks, man

@nielsenramon, should probably add this to the docs - it isn't obvious.


Is there a reason crypto.subtle is missing in most polyfills including this one?

Hello,
We are using a project which needs crypto.browserify (or its derivatives and the packages it has inspired). However we have hit a problem: crypto.subtle isn't polyfilled. This is a source of incompatibility with all projects that depend on crypto.subtle. What can we do to get crypto.subtle in polyfills?

Not that I know of. A PR to add it, with tests, would probably be what's needed.
#203 seems related.


Windline is_git() broken on nightly

local git_comps = require("windline.components.git").is_git()

got_comps is false when inside a git repo
works on 0.6.1 just fine but on 0.7.0 nightly it breaks and does not work

you need add bufnr .is_git(bufnr)

you need add bufnr .is_git(bufnr)

true not it works


.CANT LOAD LLAMA 3.1 70B due to ValueError: Some modules are dispatched on the CPU or the disk.

I am trying to fine-tune llama 3.1 70b. It has 6 safetensors making a total of  39.52GB.
I have a total of 57.6GB of DISK in kaggle. But the code is raising a value error.
ValueError: Some modules are dispatched on the CPU or the disk. Make sure you have enough GPU RAM to fit the quantized model. If you want to dispatch the model on the CPU or the disk while keeping these modules in 32-bit, you need to setload_in_8bit_fp32_cpu_offload=Trueand pass a customdevice_maptofrom_pretrained. Check https://huggingface.co/docs/transformers/main/en/main_classes/quantization#offload-between-cpu-and-gpu for more details.
Kaggle session matrix:

Oh Kaggle can only fit 22B Mistral Small - 70B sadly is way too large - you need at least a 48GB GPU card to finetune with Unsloth!

It's not loading gemma2 27B how would it load the 22B model?

@MuhammadBilal848 We literally have a notebook for Mistral 22B :) https://colab.research.google.com/drive/1oCEHcED15DzL8xXGU1VTx5ZfOJM8WY01?usp=sharing using T4s 16GB
Gemma 27B is different since the vocab size is different sadly

@MuhammadBilal848 We literally have a notebook for Mistral 22B :) https://colab.research.google.com/drive/1oCEHcED15DzL8xXGU1VTx5ZfOJM8WY01?usp=sharing using T4s 16GB
Gemma 27B is different since the vocab size is different sadly

Thank you. @danielhanchen


Add fetchURI

Proposed Changes
Add FetchURI step to gather the contents of a specified URI and store the content on disk.
Related Issue(s)
Testing
go test
Documentation
pkg/README.md
Checklist

[ ] Ran mage runprecommit locally and fixed any issues that arose.
[ ] Ran mage runtests locally and fixed any issues that arose.
[ ] Curated your commits so they are legible and easy to read and understand.
[ ] 🚀

This is great, thank you! Once the tests and pre-commit hooks pass, we should be in good shape to land this!

@d3sch41n had some comments on this PR, so I'm gonna leave approval in his court


cleanup branches workflow started to fail

I have noticed that "clenaup branches" workflowflow never completes as it stuck on some interactive questions. Moreover it's never times out so it had occupied all agents.

Looks like the problem is a change of the SSH host key on GitHub. So we
need to update the key locally.
https://buildkite.com/llvm-project/cleanup-branches/builds/590#9a7b3321-ecc7-4ae6-8d14-ac18f6b45130
--
Best,
Christian
Mikhail Goncharov @.***> schrieb am Mi., 26. Jan. 2022,
19:03:

I have noticed that "clenaup branches" workflowflow never completes as it
stuck on some interactive questions. Moreover it's never times out so it
had occupied all agents.
—
Reply to this email directly, view it on GitHub
https://github.com/google/llvm-premerge-checks/issues/381, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AEYJMCDJLZJVOQMVHJ7CGN3UYAZOLANCNFSM5M3W7KGQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I have:

documented how to update ssh keys (apparently ed25519 is a current recommendation) and updated secret. Some agents are to be restarted but I am postponing it until I confirm that bolt configuration runs fine.
updated "cleanup" pipeline to run on "service" queue and also added timeout of 10 minutes.
Recent successful run: https://buildkite.com/llvm-project/cleanup-branches/builds/602


fix(Sketch): set correct arg name in build method

Description of changes:
Fix typo in the argument name for build method in the typescript definition file.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

:tada: This PR is included in version 1.17.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


fix: aws cluster schema

The minimal required values for installing Otomi are cluster.name and cluster.provider. In the schema cluster.apiName and cluster.region where still required for the AWS provider. This PR removes this requirement.

I see that cluster.region is used at
values/cluster-autoscaler/cluster-autoscaler.gotmpl
and
jobs/certs-aws.gotmpl
The job-certs-aws depends on aws-alb-ingress-controller.
The cluster-autoscaler autoscaler is always deployed if provider is aws
I am afraid we cannot remove those requirements until we sort out these dependencies.

Requires more changes that need investigation and upgrades first. Closed for now


Suggestion: Slider component

Hi @saadeghi
great work! Are you planning to create a slider component too ?

@m0dch3n here it is: https://daisyui.com/components/form/range


Integrate with artifacthub

https://artifacthub.io/docs/topics/repositories/#helm-charts-repositories

Add some more annotations to the spire chart.
https://artifacthub.io/docs/topics/annotations/helm/


Add ocdmi recipe to master-staging

This is a part of the effort to forward port OpenCDM from LHG's chromium_45 to cromium currently in meta-browser (53.0.2785.143 for chromium-wayland atm).
This patch set is not enough to get anything working, but it lets the chromium build for HiKey to complete OK w/o disabling/removing any optee dependent stuff from meta-lhg (the " ${@base_contains('MACHINE_FEATURES', 'optee', 'optee-aes-decryptor ocdmi portmap', '', d)}" line inrpb-westonchromium-image.bb in particular).

In this patches I moved the ocdmi out of meta-browser. And in porting forward the rest of the LHG's modifications to chromium_45 I was going to move these changes out of meta-browser whenever this is possible.
But do we plan to upstream (a part of) this stuff?
If (a part of) it is OK to go into meta-browser, I would leave it there (wouldn't move to meta-lhg).


Tracking issue: Update doc for release 0.9

Update Concept:

[ ] Elasticsearch
[ ] Memcached
[ ] MongoDB
[ ] MySQL
[x] PostgreSQL
[ ] Redis

Add doc for DBVersion crd:

[ ] Elasticsearch
[ ] Memcached
[ ] MongoDB
[ ] MySQL
[x] PostgreSQL
[ ] Redis

Update Guides:

[ ] Elasticsearch
[ ] Memcached
[ ] MongoDB
[ ] MySQL
[ ] PostgreSQL
[ ] Redis

Update Examples:

[ ] Elasticsearch
[ ] Memcached
[ ] MongoDB
[ ] MySQL
[ ] PostgreSQL
[ ] Redis

Others:

[ ] Make clear that wipOut does not clear snapshot data from local backend. xref: https://github.com/kubedb/project/issues/261
[ ] Fix snapshot backend for local backend. xref: https://github.com/kubedb/project/issues/266
[ ] Document that requests is used for storage resources. xref: https://github.com/kubedb/project/issues/192
[ ] Show how to find available storageclass. xref: https://github.com/kubedb/project/issues/192
[ ] Don't use gitRepo as script source. Use configMap. xref: https://github.com/kubedb/project/issues/170

Update concept:

[ ] Snapshot


Terragrunt destroy ask me to apply each modules of my tree

Describe the bug
When i run terragrunt destroy on a specific module it ask me to apply each external dependency since Terragrunt 0.51.9, reproductible version 0.52.0.
It ask to apply external modules, not linked to my main module where i try to destroy.

To Reproduce
Run this command :
 terragrunt destroy

Here is config of specific module :
terraform {
  source = "git::ssh://git@somesource//mymodule?ref=mytag"
}

include {
  path = find_in_parent_folders()
}

locals {
  common_vars = merge(
    yamldecode(
      file("${find_in_parent_folders("common.yaml")}"),
    )
  )
}

inputs = {
  mydepoutput= dependency.mydep.outputs.myoutput
}

dependency "mydep" {
  config_path = "../mydep"
}

Expected behavior
When i run a terragrunt destroy don't ask me to apply other modules of IaC.
Versions

Terragrunt version: 0.51.9
Terraform version: 1.5.7
Environment details (Ubuntu 20.04, Windows 10, etc.): Windows 11 - WSL2 - Ubuntu 22.04.3 LTS
Tried on Debian 12 on other machine (Win11 WSL2 too)

Additional context
Looks like it try to apply all modules of IaC

me too

Fix included in https://github.com/gruntwork-io/terragrunt/releases/tag/v0.52.1

Thanks a lot, you rock @denis256


Spoofing

Hi,
Correct me if I'm  but it looks like if client sends X-Forwarded-For: <IP_ADDRESS>, <IP_ADDRESS> in request, which then goes via trusted proxy (or several), and middleware is configured using proxy(['ip of trusted proxy']) then the result is that client Ip appears as <IP_ADDRESS>, however the actual client IP (detected by first proxy) should be the 3rd in the XFF list.
In other words, the middleware should check XFF IP addresses one by one from right to left, and return first one that is not in the trusted proxies list.

Yes, you're absolutely correct. Thanks for address this.
Do you like to work in a PR? If don't, I can fix it.

@oscarotero check out the PR above.


Links not working on the description webview page

Open description page for a PR
Click a link, like the PR number

Nothing happens

Nevermind 😄 My OSS code instance was hung which was causing unreponsiveness.


Reviewed and waiting for Representation Implementation: Update getting_started.md

removed spelling mistakes from the getting started

added tokenization functions to the document and process

It will still remain as a draft after everything else in #107 is done 🐘 🧝

@jbesomi waiting here for PR #135 to be merged, so I can include the tutorial link 🥇

Went through this again, fixed some small stuff; tried out all code pieces. Ready to merge from our side now


Two CSS colors missing from validCssColors.ts

Tested against list taken from https://www.tutorialrepublic.com/css-reference/css-color-names.php, two colors are shown during testing as missing:  "navyblue" and "seagreen"

Good first issue - research that these are valid colors according to CSS spec, and update valid colors.

Awesome, I just fixed this with #132. https://github.com/Mirrorful/mirrorful/commit/fbc3e0218d989599f3c678ebe4a4c6abcca8c05d
Closing!


object has no attribute 'get_extension_for_class'

hi, I've got this error while running on my centos 7,
this is my installed version
/m/d/ostack-la >>> linuxacademy-dl -v                                                                                (1) : 18:00:20
linuxacademy-dl 1.0.dev2
/m/d/ostack-la >>>

and here is error :
ERROR    Unknown Exception
ERROR    'Extensions' object has no attribute 'get_extension_for_class'
Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/linuxacademy_dl/cli.py", line 148, in main
    output_folder, args['use_ffmpeg'], args['video_quality']
  File "/usr/lib/python2.7/site-packages/linuxacademy_dl/linux_academy.py", line 118, in __enter__
    self.login()
  File "/usr/lib/python2.7/site-packages/linuxacademy_dl/linux_academy.py", line 134, in login
    "password": self.password
  File "/usr/lib/python2.7/site-packages/linuxacademy_dl/url_templates.py", line 109, in render_url
    return getattr(session, 'request')(**render(dict(url_template)))
  File "/home/adhi/.local/lib/python2.7/site-packages/requests/sessions.py", line 488, in request
    resp = self.send(prep, **send_kwargs)
  File "/home/adhi/.local/lib/python2.7/site-packages/requests/sessions.py", line 609, in send
    r = adapter.send(request, **kwargs)
  File "/home/adhi/.local/lib/python2.7/site-packages/requests/adapters.py", line 423, in send
    timeout=timeout
  File "/home/adhi/.local/lib/python2.7/site-packages/requests/packages/urllib3/connectionpool.py", line 594, in urlopen
    chunked=chunked)
  File "/home/adhi/.local/lib/python2.7/site-packages/requests/packages/urllib3/connectionpool.py", line 350, in _make_request
    self._validate_conn(conn)
  File "/home/adhi/.local/lib/python2.7/site-packages/requests/packages/urllib3/connectionpool.py", line 835, in _validate_conn
    conn.connect()
  File "/home/adhi/.local/lib/python2.7/site-packages/requests/packages/urllib3/connection.py", line 330, in connect
    cert = self.sock.getpeercert()
  File "/home/adhi/.local/lib/python2.7/site-packages/requests/packages/urllib3/contrib/pyopenssl.py", line 324, in getpeercert
    'subjectAltName': get_subj_alt_name(x509)
  File "/home/adhi/.local/lib/python2.7/site-packages/requests/packages/urllib3/contrib/pyopenssl.py", line 171, in get_subj_alt_name
    ext = cert.extensions.get_extension_for_class(
AttributeError: 'Extensions' object has no attribute 'get_extension_for_class'

Hi @adhioutlined, thanks for reporting this issue! I think you are using an older version of cryptography library there! Please upgrade the library (pip install --upgrade cryptography) and try again. If you are seeing any error messages while upgrading the library, please visit this link and follow the instructions!


[WIP] Default to "latest" when tag not defined

This causes the following error:

2018-08-06 19:02:37,678 platform:- - atomic_reactor.plugins.add_labels_in_dockerfile - ERROR - base image was not inspected
2018-08-06 19:02:37,679 platform:- - atomic_reactor.plugin - DEBUG - Traceback (most recent call last):
File "/usr/lib/python2.7/site-packages/atomic_reactor/plugin.py", line 239, in run
plugin_response = plugin_instance.run()
File "/usr/lib/python2.7/site-packages/atomic_reactor/plugins/pre_add_labels_in_df.py", line 272, in run
raise RuntimeError(message)
RuntimeError: base image was not inspected

which I think is being triggered because base_image is tracked without a tag by atomic reactor.

Dropping this in favor the the quickfix https://github.com/projectatomic/atomic-reactor/pull/1054.
Although, we'll eventually need to address this properly.


nSkinz have a bunch of VAC Modules that can lead into Ban

Monitoring the VAC System for 24/7 and I get nearly any Modules now. Last time one VAC Module was very simple it counts VMTs and if it have different count as expected it loads another Module which check Signatures for VMTs if it catches unknown Sigs it streams another Module which I am currently analysing which I am not 100% sure what it does a friend on UC is currently looking into this cascaded Modules. There are different Detection Methods streamed packed into different Modules and they combine them to get a statistical aproach to Ban a player for 60% or 70% suspicious Modules. That is why some get banned using nSkinz and some are not.

This is very helpful, thank you. Do you have any threads on UC about this? I'd like to llok forward to it.

 Also, whatever you mean by counting VMTs does not affect us, all objects I hook are dynamically allocated. Only the checks in random functions that xor table addresses and or a global variable have any effect on us. Feel free to post in this issue again in case you see that happening with any of our objects.


No init() function

The docs use an init() function all over the place but I don't see it in the object passed from the module. Other than that, the module seems to work correctly. Are the docs incorrect?

It should be there. In a Node.js prompt try require('nbind') and it should print:
> require('nbind')
{ Binding: [Function: Binding],
  find: [Function: find],
  init: [Function: init] }

init loads the actual module and returns an object with the exported C++ classes and functions.

I was loading the module directly. It seems I can pass a relative folder name to init() to differentiate between multiple nbind bindings so I'll do that.


initial compilation error fix

Jenkins run: http://wls-jenkins.us.oracle.com/view/weblogic-operator/job/weblogic-kubernetes-operator-javatest/2945/

@maggiehe00 when you have working code, you will be removing the commented code..


Add configuration property to to select the gradle worker isolation strategy

🚀 Description
As discussed in this , add a new property to select the isolation strategy for the gradle worker.
📄 Motivation and Context
Fixes issue
The user is able to select between the default no-isolation strategy and a process-isolation strategy. I remove the classloader-isolation strategy since it does not seem wo work for larger projects, and I wanted to prevent future issues ;)
But it could be easily added, if desired.
🧪 How Has This Been Tested?
Added integration tests for the format and checkFormat task.
The property is set in the example project.
📦 Types of changes

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

✅ Checklist

[ ] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.

Hi @cortinico ,
Here is my draft PR. Unfortunately I encountered an issue, that I am not sure how to resolve. Maybe you have an idea?
What happened:
I created the PR, all tests are locally passing. Afterwards I applied the config property in the example project for some final testing.
Isolation strategy parallel works as expected.
No Isolation strategy does not work with the error Unable to load class 'com.facebook.ktfmt.format.ParseError'..
The dependency is not resolved and after some investigation this makes totally sense ;)
In the build.gradle.kts for the plugin we have the following snippet:
dependencies {
    compileOnly(libs.ktfmt)

    // Other dependencies...

    testImplementation(libs.ktfmt)
}

So for the tests, the ktfmt plugin is added as implementation dependency and both strategies work since the dependency is always resolved. But obviously not for the production code. When using the no-isolation strategy, the production classpath is used and there the dependency is only compileOnly and therefore missing.
Ideally we could add the dependency after the configuration phase of the plugin to the classpath, when the isolation strategy is no-isolation. I tried to implement this with org.gradle.api.artifacts.DependencyResolutionListener but was not able to make it work.
The only two other solutions I am able to come up with are:

Remove the option again and use either process or no isolation. But options will not make everyone happy as discussed in this issue
Or: Always add the dependency as implementation. Users that have a classpath conflict can exclude the dependency as shown in the accepted answer here

For now: Maybe it ist worth to have a quick fix with just the process isolation, so that the plugin starts working again.
And then we have more time to fix it and think about the solution.

No Isolation strategy does not work with the error Unable to load class 'com.facebook.ktfmt.format.ParseError'.. The dependency is not resolved and after some investigation this makes totally sense ;)

Another option could be to ask users to add a classpath() dependency if they wish to use the NO_ISOLATION

Or: Always add the dependency as implementation. Users that have a classpath conflict can exclude the dependency as shown in the accepted answer here. I have not yet tested, if that would actually work.

That's also another option. I'm not sure how that would work with the plugin{} syntax though.
If you send another PR with the Process Isolation, we can merge it first, do a new release, and get back to this.

Another option could be to ask users to add a classpath() dependency if they wish to use the NO_ISOLATION

That would be in my opinion the better solution. I did not think of that.

If you send another PR with the Process Isolation, we can merge it first, do a new release, and get back to this.

I will do that. Lets first fix it, then make it better :)

I would close this PR for now.
When the new version is released, I will do some performance testing with the new process isolation. I still have the hope, that the performance impact is negligible when the gradlew workers are reused between tasks.
Then we still can think of a better solution if necessary :)


Segmentation fault at build time for Mono [Tool] script

Godot version:

b77a411
OS/device including version:

Ubuntu 16.04 LTS
Issue description:

Segmentation fault at build time for Mono [Tool] script, producing the following error:

ERROR: _unload_scripts_domain: Mono: Domain finalization timeout
At: modules/mono/mono_gd/gd_mono.cpp:681.

Steps to reproduce:

Build the following script
Apply the script to a spatial in a scene
Close the scene
Add/remove some white space
Recompile

using Godot;
using System;

[Tool]
public class CrashEditorScript : Spatial {

    public override void _Ready() {

        MultiMeshInstance multiMeshInstance = new MultiMeshInstance();
        multiMeshInstance.Multimesh = new MultiMesh();
        multiMeshInstance.Multimesh.Mesh = new PlaneMesh();

        AddChild(multiMeshInstance);
    }

}

Minimal reproduction project:
MonoToolCrash.tar.gz
@neikeq

Any error in the terminal other than the one mentioned?


Ivonne - Refactor to make MFB more flexible for the future - how components are built / managed

This is for the Textfield component.


Syntax error with current build

I wanted to try out pestle which seems to be a great idea.
However with the built build or when i build myself with ./build.sh I get the error:
PHP Parse error:  syntax error, unexpected 'function' (T_FUNCTION), expecting identifier (T_STRING) or \ (T_NS_SEPARATOR) in phar:///home/sven/PROJEKTE/Magento_Extensions/pestle/pestle/pestle.phar/modules/pulsestorm/pestle/runner/module.php on line 8
PHP Stack trace:
PHP   1. {main}() /home/sven/PROJEKTE/Magento_Extensions/pestle/pestle/pestle.phar:0
PHP   2. main() /home/sven/PROJEKTE/Magento_Extensions/pestle/pestle/pestle.phar:9

@twosg are you running PHP 5.6 or greater? (php -v from the command line if you're not sure)

@astorm no I am using 5.5.9 currently. Should I update in order to run pestle?

@twosg Ah, that'll do it.  Pestle requires PHP 5.6 or greater.  One of its non-magento goals was/is to implement a better version of namespace/moduled functions, so we require the PHP use function syntax that's only available in PHP 5.6+
Sorry for the ugly break -- we've got an issue open to do a version check and return a better error message in the future. https://github.com/astorm/pestle/issues/139

@astorm Thanks it works now :)


Different parameters for each image

Hi,
thanks for this great library.
Is there a best practice how to deal with parameters that change per image?
Specifically, I want to crop out an object given through a bounding box. So far I would probably get the augmenter by name and overwrite its parameters for every image.
Thanks

If you only want to get the image content of a bounding box you can also just do something like
import imgaug as ia
image = ... load somehow ...
bb = ia.BoundingBox(x1=100, x2=150, y1=100, y2=150)
bb_image_area = bb.extract_from_image(image)

then bb_image_area contains the image content within the bounding box area.
If you want to achieve your goal via augmenters, then what you described is one possible way to do it. You could also just keep the stochastic parameter somewhere and update it. Something like the following should work:
from imgaug import parameters as iap
from imgaug import augmenters as iaa

# rotate by 45deg
param = iap.Deterministic(45)
aug = iaa.Affine(rotate=param)
image_aug = aug.augment_image(image)

# rotate by 50deg
param.value = 50
image_aug = aug.augment_image(image)

and so on.
Similarly, you can save the augmenter in a variable and change its parameters directly, instead of first searching for it via name and only then changing its parameters. Saves the search.
Another way to do it is to reinstantiate the augmenters each time, though obviously that gives up a bit of performance. It also makes background/multicore augmentation and random order augmentation harder to achieve.
def augment(image, rotate, crop_px):
    image = iaa.Affine(rotate=rotate).augment_image(image)
    image = iaa.Crop(px=crop_px).augment_image(image)
    return image

image = ... load image ...
image_aug = augment(image)


Add pdftotext cask

This adds a cask for the pdftotext software. I'm not sure what I need to do for the uninstall bit.

There are scripts and documentation geared for that. Do you need any help?

@vitorgalvao I wasn't exactly sure how to use it.. I went through the docs and some other casks, and set it up this way. It seemed to uninstall fine locally.

Looks good, could you please squash your commits?

Sounds good - I'll update it soon.

@vitorgalvao If this all looks good, I'll squash my commits.

Please do.

@vitorgalvao Here you go.

Thanks.


Error loading docker image

Good evening
I have created the image myself via "docker
The following command triggers an error message:
udocker load -i src.api.tar

File "/usr/home/simrah/.local/bin/udocker", line 8, in
sys.exit(main())
File "/usr/home/simrah/.local/lib/python3.9/site-packages/udocker/maincmd.py", line 36, in main
exit_status = UMain(sys.argv).execute()
File "/usr/home/simrah/.local/lib/python3.9/site-packages/udocker/umain.py", line 108, in execute
exit_status = cmdscommand  # executes command
File "/usr/home/simrah/.local/lib/python3.9/site-packages/udocker/cli.py", line 316, in do_load
repos = self.localfileapi.load(imagefile, imagerepo)
File "/usr/home/simrah/.local/lib/python3.9/site-packages/udocker/localfile.py", line 35, in load
repositories = DockerLocalFileAPI(
File "/usr/home/simrah/.local/lib/python3.9/site-packages/udocker/docker.py", line 850, in load
repositories = self._load_image(structure, imagerepo, tag)
File "/usr/home/simrah/.local/lib/python3.9/site-packages/udocker/commonlocalfile.py", line 66, in _load_image
return self._load_image_step2(structure, imagerepo, tag)
File "/usr/home/simrah/.local/lib/python3.9/site-packages/udocker/docker.py", line 795, in _load_image_step2
(json_config_file, layers) = self._get_from_manifest(structure, imagetag)
File "/usr/home/simrah/.local/lib/python3.9/site-packages/udocker/docker.py", line 782, in _get_from_manifest
for repotag in structure["manifest"]:
TypeError: 'NoneType' object is not iterable

can you see if our latest rel 1.3.11 fixes this issues?
https://github.com/indigo-dc/udocker/releases/tag/1.3.11

@mariojmdavid Thank you in advance for the quick reply!
Unfortunately, I must note that the same error still exists. I have verified that it is the latest version

is it possible to send us the tarball src.api.tar, unless if it's private of course
on the other hand can you do
udocker -D load -i src.api.tar    and paste here the full result

or better yet, what is the docker image, (if it's public)

no activity for 3 months, closing
if you still have the issue, feel free to reopen


Decompose<EMAIL_ADDRESS>into recommended directory structure

@grzegorz-wcislo Thank you very much for this 👍. This is good but I am going to close this PR for now so we can re-do the repo utilizing our updated Orb Starter Kit which will create a more consistent config with our other orbs.
Thank you! https://github.com/CircleCI-Public/orb-starter-kit/pull/37


Implements symbologyIdentifier for DataMatrix Code (#214)

Part-implements #214
Implements symbologyIdentifier for DataMatrix Code.
Adds unit test & 2 samples - one for standard DM, one for DMRE.
Also fixes unit tests for Structured Append which must be 1st codeword, and Reader Init sample which wasn't actually Reader Init ([8066b527161e074dcb2025cd60978625de9dc956])

This PR is ready for review.

This PR is ready for review again.

Thanks a lot!


SuggestionResult not declared as module

With version 4.5 many new types where introduced.
I'm missing SuggestionResult returned as Promise from suggestLocations(params, options) as declared module.

SuggestionResult is not a physical module, it's only an interface (see https://github.com/Esri/jsapi-resources/blob/master/4.x/typescript/arcgis-js-api.d.ts#L5937-L5940). To use such a type in your own code, you can use the following:
import esri = __esri;

// You can now refer to any type from the esri namespace
function processResult(result: esri.SuggestionResult) {
}

// Note that this namespace contains type interfaces only, to instantiate types you still need to actually import the correct module


Add iteratorsize

This should fix it with v0.5 of Julia. Re: all iterators are broken.
To be honest I haven't tested this version directly, am doing a quick commit and am relying on TravisCI to test it. If this doesn't CI I'll do a more careful branch and fix.
This problem is referenced in https://github.com/JuliaLang/LightXML.jl/pull/46#issuecomment-219366412
See: https://github.com/JuliaLang/julia/issues/15977 and the linked documentation gist

Looks like this worked.
Failure currently showing is a timeout on one of the appveyer builds.
(I'm going to close and reopen to trigger another build)

Thanks!

Cool.
This is my first code to have merged into someone else project.
Even if it is like 5 lines.
Yay.

Good to hear - I could've sworn you had a base PR merged by now. If this thing with iteratorsize comes up often in other packages it would probably be worth putting it in Compat.jl.

There is something I don't get here: what is the point of defining a fallback iteratorsize(v) for version < v"0.5.0-dev+3305"  which gives an error but is less specific than the actual iteratorsize methods coming later?

@mschauer Good point.
I copied it direct from Iterators.jl
In particular from this PR
When I was doing so, it was on the logic that if iteratorsize was called in v0.4, it would not hit the Xml*Iter's iteratorsize-- which would result in undefined SizeUnknown. but would hit the generic one.
However that does not follow, because multiple dispatch is most specific first.
So it could just be:
if VERSION >= v"0.5.0-dev+3305"
    import Base: iteratorsize, SizeUnknown
end


launch problem

HI,
When I launch quad_gazebo.launch file,it occurs

and Gazebo cannot run.There is only rviz page.

how can i solve this problem?

This snippet isn't enough information to determine the issue on its own. You could try just running gazebo and see if it launches properly, if not then you may need to reconfigure it. Does the image of the terrain show in RViz?

When I run Gazebo alone, it opens fine. But when I run it with roslaunch quad_utils quad_gazebo.launch, Gazebo does not open, Rviz runs and loads the terrain, but the bot does not load properly.

My system is Ubuntu 20.04, The repository is also checkout to the noetic version, and I tried to install libignition-math, but it didn't work.

Closing this due to inactivity.


fix: invalid schema for function 'func_name': None is not of type 'object' (#429)(#432)

Describe the change
The problem recurs when Parameters is a value instead of a pointer. MarshalJSON in Definition does not execute MarshalJSON if Definition is a value because the receiver is a pointer.
func (d *Definition) MarshalJSON() ([]byte, error)

Refs:

https://github.com/sashabaranov/go-openai/pull/433#issuecomment-1629401793
https://github.com/sashabaranov/go-openai/pull/433#issuecomment-1629430142

error message:
 invalid schema for function 'func_name': None is not of type 'object'

request to reproduce:
	req := openai.ChatCompletionRequest{
		Model: openai.GPT3Dot5Turbo,
		Messages: []openai.ChatCompletionMessage{
			{
				Role:    openai.ChatMessageRoleUser,
				Content: "What is the weather like in Boston?",
			},
		},
		Functions: []openai.FunctionDefinition{{
			Name: "get_current_weather",
			Parameters: jsonschema.Definition{ // <= The problem recurs when Parameters is a value instead of a pointer.
				Type: jsonschema.Object,
				Properties: map[string]jsonschema.Definition{
					"location": {
						Type:        jsonschema.String,
						Description: "The city and state, e.g. San Francisco, CA",
					},
					"unit": {
						Type: jsonschema.String,
						Enum: []string{"celsius", "fahrenheit"},
					},
				},
				Required: []string{"location"},
			},
		}},
	}

Describe your solution

MarshalJSON method is executed whether jsonschema.Definition in Parameters is a pointer or a value
Initialization with an empty map if Properties is nil.

Tests

Test MarshalJSON with both value and pointer.
add integration test for function call.

Additional context
Issue: #429, #432

@sashabaranov Please check it.


improve: verify the join reorder search space;

Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[x] No
[ ] I don't know

Has unit tests been added:

[x] Yes
[ ] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[x] No Need

Does it need to update dependencies:

[ ] Yes
[x] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[x] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 40.26 seconds
load time: 587 seconds
storage size:<PHONE_NUMBER>7 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20221020122918_clickbench_pr_31807.html


'KernelConfig' does not contain a definition for 'AddAzureOpenAICompletionBackend'

Getting this error when trying to run the notebooks in VS Code.
Error: (14,19): error CS1061: 'KernelConfig' does not contain a definition for 'AddAzureOpenAICompletionBackend' and no accessible extension method 'AddAzureOpenAICompletionBackend' accepting a first argument of type 'KernelConfig' could be found (are you missing a using directive or an assembly reference?)
(16,19): error CS1061: 'KernelConfig' does not contain a definition for 'AddOpenAICompletionBackend' and no accessible extension method 'AddOpenAICompletionBackend' accepting a first argument of type 'KernelConfig' could be found (are you missing a using directive or an assembly reference?)
The notebooks were running earlier but suddenly it is throwing this error - not sure what I'm missing. Please see below screenshot.

If you re using the newest nuget version <IP_ADDRESS>-preview, the APIs have changed. This PR updates them: https://github.com/microsoft/semantic-kernel/pull/206

This also happened to me this afternoon. After looking around, I found that the following line installs the latest version of Microsoft.SemanticKernel.
#r "nuget: Microsoft.SemanticKernel, *-*"

To fix the issue, you can simply checkout this repo to get updated notebooks.

#206 landed! Closing this issue

The APIs in the Getting-Started-Notebook are not compatible with version 0.9 of the Nuget package.  I had to specify <IP_ADDRESS>-preview to get it to run.

@aditmer This is a good catch! That notebook needs to be updated to include the change in the API

Thanks everyone! It was indeed the version of nuget that was causing this issue. The proper version as pointed out by @aditmer is working fine.


Installation on ubuntu 18:04:repository not found

Hi Igor,
I am on Ubuntu 18.04.4 LTS and I could just install from ppa. Could you please try again? Maybe the service was down.
$ sudo apt-get install dicom2mesh
Reading package lists... Done
Building dependency tree
Reading state information... Done
...
After this operation, 1'567 kB of additional disk space will be used.
Get:1 http://ppa.launchpad.net/eidelen/d2m/ubuntu bionic/main amd64 dicom2mesh amd64 0.8-0ppa0~bionic [195 kB]
Fetched 195 kB in 0s (573 kB/s)
...
Unpacking dicom2mesh (0.8-0ppa0~bionic) ...
Setting up dicom2mesh (0.8-0ppa0~bionic) ...

For me it is difficult to interpret your screenshots without any explanation. My guess is that it is somehow related to docker. Maybe that helps.

Now its install. Thank you very much.


pull request for e2e test at 01/09/2017 08:26:12

:white_check_mark: Validation status: passed
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.


Polygon: closed and right hand rule?

The Polygon type currently holds an exterior ring and one or more interior rings, but otherwise doesn't enforce much.
Currently it is not enforced that the first and last positions should be equal, or the right hand rule as in https://tools.ietf.org/html/rfc7946#section-3.1.6
Should we enforce these in this package? Or is it better to deal with those in packages such as GeoJSON.jl do you think?

That seems like a cheap enough check to just always do it, if it's required ;)


Robust geometry optimization

Use ORCA compound job to handle possible imaginary frequencies.

Putting out of the alpha version. Alpha version is all about a working minimal workflow. This is a crucial thing for a production version, not alpha version.

I have run into this issue right from the start when calculating the pinacolone molecule. Therefore, bringing this back to alpha milestone and hoping it will not be too difficult to make this work.

First step here is to figure out whether the ORCA license even allows us to use the compound job file.
If the license proves to be an issue, we could load the compound job file directly from the orca source directory, modify it, and save to DB, when it is first needed.

To test this, use the pinacolone geometry and PBE/sto-3g method. Should have one negative frequency at -108 cm^-1.
19
Lattice="15.460009977915 0.0 9.46652586712e-16 2.2104281850416e-15 13.745395511257 8.4166273079376e-16 0.0 0.0 14.394446502237" Properties=species:S:1:pos:R:3:masses:R:1 pbc="F F F"
H       10.43848329       6.97057991       8.09154441       1.00794000
H        9.72310167       5.58387614       7.19721927       1.00794000
H       10.43847436       6.97058002       6.30288727       1.00794000
H        7.15445223       6.26191359       9.39445074       1.00794000
H        6.22011754       5.02157788       8.49681784       1.00794000
H        8.01097455       4.95752793       8.50665490       1.00794000
H        8.01098119       4.95751660       5.88781588       1.00794000
H        6.22012407       5.02156772       5.89764175       1.00794000
H        7.15446489       6.26189547       5.00000424       1.00794000
H        4.97847331       6.88348037       7.19722094       1.00794000
H        5.91609508       8.12809978       8.09259490       1.00794000
H        5.91609793       8.12809015       6.30183670       1.00794000
C        9.85377921       6.67939600       7.19721875      12.01070000
C        8.47649807       7.44339167       7.19722605      12.01070000
C        7.15757204       6.53614136       7.19722637      12.01070000
C        7.13959497       5.63730286       8.48155223      12.01070000
C        7.13960178       5.63729227       5.91290779      12.01070000
C        5.91025235       7.47941323       7.19721928      12.01070000
O        8.41996762       8.70291211       7.19721440      15.99940000

The compound method in ORCA is....broken. :anguished:
https://orcaforum.kofo.mpg.de/viewtopic.php?p=40500#p40500
It will be easier (and more generally usefully for other programs) if we just implement it as WorkChain. Shouldn't be too hard, just need to multiply the XYZ geometry by the hessian displacement vector (vibdisp) from ORCA output_parameters. We should have a WorkChain call "RobustOptimizationWorkChain".


"fu" is not detected

regex will no doubt fix this

commit #bb2bfea445a82660cf9c5539fa384263c0cf970d fixes this but not completely

This issue was moved to HulaSamsquanch/swearify#6

fu turns into fu$3 now
wtf

FIXED


Add config for introspection_endpoint_auth_method

It was not possible to set the auth method for introspection through the config.
This was found when trying to use token introspection with ORY Hydra, which requires Basic or OAuth authorisation for introspection.
This PR:

adds introspection_endpoint_auth_method as required by lua-resty-openidc
adds this value to config, schema, readme
adds unit test
does not add a default value for backwards compatibility (will remain as nil)

Pull Request Test Coverage Report for Build 90

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 95.968%

Totals

Change from base Build 89:
0.03%

Covered Lines:
119

Relevant Lines:
124

💛  - Coveralls

@Trojan295 are you interested in help maintaining this project? I'm currently working on features to match Kong's enterprise OAuth Introspection plugin

@tommilligan I'm interested but I don't have the bandwidth to contribute.

@tommilligan, sorry for the late response, but I'm out of this project a bit.
Still, it looks OK from my side. Can you also squash the commits to one?
@phirvone @tsyrjanen , can you try to prepare a new release for LuaRocks? There were a few PR accepted since the last release.

@Trojan295 thanks, no worries at all


[Feature Request]: Add Distinct Callbacks for Buttons in InterstitialScreen

The problem
The InterstitialScreen component currently allows developers to set only the labels for its buttons using the following props:

nextButtonLabel="Next"
previousButtonLabel="Back"
skipButtonLabel="Skip"
startButtonLabel="Save"

However, developers cannot assign separate functions to these buttons. Regardless of which button is clicked ("Skip," "Save," or "X" for Close), the onClose function is called. This creates a limitation where all actions are handled by the same function, making it impossible to differentiate the logic based on the button clicked.
The solution
Introduce new callback props for individual button actions:

onSkip: A function triggered when the "Skip" button is clicked.
onSave: A function triggered when the "Save" button is clicked.
onClose: A function specifically triggered when the "X" (Close) button is clicked.

This change will provide developers the flexibility to define separate logic for each action, improving the component's usability and extensibility.
Design link
No response
Links to other materials
No response
Owner/main maintainer(s)
NA
Second/backup maintainer(s)
No response
Product/offering
No response
Business priority
None
Code of Conduct

[x] I agree to follow this project's Code of Conduct
[x] I checked the current issues for duplicate issues

Should we expose different callbacks for each actions or a unified callback like below for all actions passing the action type. Thus , we may need to introduce only 1 new callback prop
handleActionButtonClick(actionType,e,...).

Acceptance criteria

[ ] Single async callback, returning the action type (potential naming, onAction, onRequestAction)
[ ] Review with Carbon for any naming convenitions
[ ] Include loading indicator
[ ] Include test for new callback prop

*Note, there will be some composability changes to this component in the future (1H 2025) impacting how actions are provided in this component


Populate hook: parentField and childField are confusing

If we have a system of categories and subcategories, we have:
/*
Category (parent)
    v
Category (child)
*/

{
  service: 'category',
  nameAs: 'subCategories',
  parentField: 'category_id',
  childField: 'id',
  include: [ ... ],
}

/*
Category (child)
    v
Category (parent)
*/

{
  service: 'category',
  nameAs: 'parentCategories',
  parentField: 'parent_id', // but this field is on the child in this case
  childField: 'id', // but this field is on the parent in this case
  include: [ ... ],
}

I think that as for relational databases, like mySQL, you should rename parentField to localField and childField to foreignField.
Which will give:
/*
Category (parent)
    v
Category (child)
*/

{
  service: 'category',
  nameAs: 'subCategories',
  localField: 'category_id',
  foreignField: 'id',
  include: [ ... ],
}

/*
Category (child)
    v
Category (parent)
*/

{
  service: 'category',
  nameAs: 'parentCategories',
  localField: 'parent_id',
  foreignField: 'id',
  include: [ ... ],
}

If we are going that direction, I think I would prefer localKey and foreignKey.

I also prefer localKey and foreignKey.

Eric, David and I had several days of discussion before settling on parentField and childField. These were the most clear because you know what comes from the item being joined to and what from the items being joined.
Every other naming convention confused someone on the core team, including names similar to localKey and foreignKey which I proposed.
Also its too late to make such a breaking change even if we wanted to.

I really struggle with child and parent, it does not make sense here.

The parent is the item that is having other items added to it. The child is
the item that is being included in the parent. I do not understand why this
is confusing.
On Sun, Jan 15, 2017 at 7:07 PM, Kévin Berthommier<EMAIL_ADDRESS>
wrote:

I really struggle with child and parent, it does not make sense here.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/feathersjs/feathers-hooks-common/issues/86#issuecomment-272749041,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABezn3bNSMbA8zDF_m4qDbcag75VGtERks5rSrSsgaJpZM4Lbt2G
.


fix: make sure user-data-dir command line parameter is customizable

The --user-data-dir command-line parameter is currently unusable. Its functionality is hampered due to an automatically appended /data directory, rendering the parameter completely ineffective.
The proposed minor modification allows users to accurately customize this variable, enhancing its utility.
The change is already implemented in coder/code-server at https://github.com/coder/code-server/blob/main/patches/integration.diff#L55
This PR is linked to:

https://github.com/gitpod-io/openvscode-server/issues/512
https://github.com/gitpod-io/openvscode-server/pull/511

Seems reasonable to me but I pass it on to @alexdima

Hi there,
I'm commenting here to know if there is something else to do to push this forward.
@alexdima ? @roblourens ?
Thanks!

Dear @alexdima and @roblourens,
Is there anything I can do to push this thing forward?
Thanks!

Ok closing this PR since I don't have any single feedback.
Many thanks, I'm really willing to contribute to this project in the future.


[GH-114] Refactoring of serialization

This PR:

creates Aeutil application and moves serialization code to it.
moves invocations of Aeutil.Serialization to Aehttpclient
moves other request related code to Aehttpclient

This closes #114

@meivantodorov can you approve and merge?


Libre measure review

There is a new AMP listed under the VMP that covers Freestyle Libre.
It doesn't have a BNF code but this should (probably) prompt a review of the Libre measure.

Updated measure appears identical at the moment to the existing one:
https://openprescribing.net/measure/libre/
https://openprescribing.net/measure/preview_libre/

@richiecroker Updated title/description here: https://docs.google.com/document/d/1541kclbCDZ14vayO7SAsLZGOMn7YAltqMnNEJ07Tztc/edit?usp=sharing

Thanks @chrisjwood16 , have made slight amendment (change "device" to "sensor" in the description for prescribing" renamed measure name and have added to #3509 .  Preview here (https://openprescribing.net/measure/preview_cgm_sensors/) with one "device" not changed, but subsequently done in PR.

@richiecroker Initial p tag can be removed to remove the new line after "Why it matter" - now I see why we leave this out. But otherwise looks all good to me.

Is this ready for sharing with our expert users?


do_unmount usage of printf(1) breaks literal % in paths

From RTFS:
https://github.com/OpenRC/openrc/blob/863e1a5c87056bf5b0c4ce6a02ecb818b0b2ba13/sh/rc-mount.sh#L37
with a comment of "Ensure we interpret all characters properly" looks very weird.  I'm not sure what is the original problem it was trying to fix, but it obviously introduces problems for mount points that contain characters interpreted by printf(1), (e.g. %, \, etc).
Try to use a mount point like e.g. /x%s - that printf will results in mnt=/x (there's no argument to substitute for the %s, so it's replaced with an empty string).  E.g. after
mount --bind / /x%s && mount -o remount,ro,bind /x%s

reboot
 * Remounting remaining filesystems read-only ... *   Remounting /x read only ... *   in use but fuser finds nothing
 [ !! ]

it's to handle \### octal escape sequences.  it's standard to serialize things like spaces as \040.  it's a little ironic that this in turn breaks other printf sequences like %.  the getmntent(3) man page discusses this a bit which is how mountinfo gets a list of mounts.
to your point about \, no, that isn't broken, because a path like /foo/\bar is turned into /foo/bar/\134bar which is then expanded properly.
i think % is the only char that messes things up here.  we could adjust mountinfo to encode that as \045 to avoid the issue.  shouldn't be a big deal since this is an internal program that only other openrc code builds off of.

Ah, fair enough, but the fact that mountinfo ("an internal program that only other openrc code builds off of") does encoding of mount point names is, I'd dsay,  highly unobvious to someone who is not familiar with openrc internals and warrants a comment.  That printf screams "format string attack".

it's internal code used by internal code. it doesn't really need to be obvious to users.
i don't know what "printf attacks" you're talking about, but considering this is only used when shutting down, and requires someone to have already done an arbitrary mount, and the only attack is truncating the path (by expanding % sequences into nothing), doesn't really seem like a security concern. this isn't the same as %n printf used in compiled programs.


added README.md to CC_67_2_binary_tree_viz

also added a link to my community variant in the README.md

Yay!


Bdbag minid

Pull Request Checklist:

[ ] Positive Test Case Written by Dev

[ ] Automated Testing

[ ] Sufficient User and Developer Documentation

[ ] Passing Jenkins Build

[ ] Peer Code review and approval

Positive Test Case

[Enter positive test case here]

Hi! What is this? :)

This is a pull request for @hyi to review the bdbag and minid work so we can deploy it to a local server for further testing.

Closing this pull request since it's against the wrong repo.

Build finished.


Refactor/218 date formatted test fix

This fixes #218
Finally.
The problem was the following:
IMAP uses timestamps like this:
10:42 +0200 is not 10:42 UTC plus 2 hours local. 10:42 is already local time and the +0200 defines the time-offset which is already applied
I also renamed the methods to make comments obsolete

Pull Request Test Coverage Report for Build 839

11 of 11 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 74.3%

Totals

Change from base Build 809:
-0.1%

Covered Lines:
1168

Relevant Lines:
1572

💛  - Coveralls


Next Score Shows wrong number

If you add a New Score the value is shown always the Double of the entered value

Closed with https://github.com/TimKolberger/skor/commit/a83090701e196ad0d9e66a3fb02e139b6eb17305


fix: make sure to fetch all tags after checkout

Description:
Release cut was failing (trying to create release-1.0 branch again)
Related:
N/A
Visual:
N/A
TODO:

~[ ] Updated README documents~
[x] Complete PR Body

!merge


Standardise files with files in sous-chefs/repo-management

Signed-off-by: Dan Webb<EMAIL_ADDRESS>

Released as: 4.0.3


logging: re-use existing logs in pulp

Call setup_logger once, from pulp_util, and have all other plugins
that want to use the pulp logger import pulp_util.
Fixes the issue of repeated pulp log entries.
Signed-off-by: Mark Langsdorf<EMAIL_ADDRESS>

Updated draft release notes.


Opinion: can reduce arguments of removeOwner()

From:
https://github.com/coinbase/smart-wallet/blob/adab965145cc7a6f938dd118295c6ae062672838/src/MultiOwnable.sol#L122
It seems a bit redundant to require the caller to pass in both the index and the bytes.  Perhaps two overloading methods would suffice, one for index only and one for the bytes?

Thanks! See https://github.com/coinbase/smart-wallet/pull/42


add GitTime to CommunityProjects

I am using Moya in my project.
GitTime is GitHub client app using ReactorKit architecture.
I am constantly trying to add new features. 🙏

Warnings

:warning:

Consider also updating the Chinese docs. For Chinese translations, request the modified file(s) to be added to the list here for someone else to translate, if you can't do so yourself.

Messages

:book:
macOS: 	 Executed 283 tests, with 0 failures (0 unexpected) in 16.087 (16.441) seconds

:book:
tvOS: 	 Executed 283 tests, with 0 failures (0 unexpected) in 13.271 (13.665) seconds

:book:
iOS: 	 Executed 283 tests, with 0 failures (0 unexpected) in 13.305 (13.764) seconds

Mdspell report on docs/CommunityProjects.md:

Line
Typo

8
SwiftyJSONMapper

13
Moya-JASONMapper

15
MoyaSugar

16
subspecs

16
ReactiveCocoa

18
Moya-Decodable

18
Decodable

19
SwiftMoyaCodeGenerator

23
NetClient/Moya

27
Drrrible

27
iOS

29
iOS

30
Papr

30
iOS.

31
SwiftHub

31
Github

31
iOS

32
GitTime

32
iOS

  Generated by :no_entry_sign: Danger Swift against dd40062fadaf18c44b18bed7ba4ada075530bfa0


[Internal Server Error] <sever seems to be down again?>

Is there an existing issue for this?

[X] Yes, I have searched the existing issues

Product Variant
Cloud (http://rxresu.me)
Current Behavior
"Internal Server Error" displaying when trying to load my previously saved resumes, along with some broken images. I think this happened a few weeks ago or a month back so I presume it means you're not at your PC or something. Please find a way to restart the server remotely when you leave because it's making it a headache when I'm trying to adjust my resume for a job opening.
Thanks.
Expected Behavior
I expect it to work as normal instead of getting a blank page that says "Internal Server Error"
Steps To Reproduce
No response
What browsers are you seeing the problem on?
Microsoft Edge
Anything else?
No response

The issue should be resolved now.


Get-AzSqlDatabaseBackupLongTermRetentionPolicy -current parameter does not exist

Getting -current parameter does not exist with example code, and example code could be cleaned up for $server variables not in use.
Powershell
Version
5.1.17763.1007
See latest docs
https://docs.microsoft.com/en-us/powershell/module/az.sql/get-azsqldatabasebackuplongtermretentionpolicy?view=azps-4.3.0
[Enter feedback here]

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

ID: a40f786f-e6c9-15f5-d80a-b4d4eb7a39f5
Version Independent ID: 12620513-5d3c-b7a2-32c9-294117ff3d57
Content: Azure SQL Database: Manage long-term backup retention - Azure SQL Database
Content Source: articles/azure-sql/database/long-term-backup-retention-configure.md
Service: sql-database
Sub-service: operations
GitHub Login: @anosov1960
Microsoft Alias: sashan

@asears Thank you for bringing this to our attention. I can confirm that the -Current parameters being passed in the two View LTR policies PowerShell examples are not supported. I was able to run the statements individually minus the -Current without issue. Also, the get all LTR policies within a server needs to be returned to include the established variables instead of static values.
Current:

get all LTR policies within a server
$ltrPolicies = Get-AzSqlDatabase -ResourceGroupName Default-SQL-WestCentralUS -ServerName trgrie-ltr-server | `
Get-AzSqlDatabaseLongTermRetentionPolicy -Current

Fix that needs to be made:

get all LTR policies within a server
$ltrPolicies = Get-AzSqlDatabase -ResourceGroupName $resourceGroup -ServerName $serverName | `
Get-AzSqlDatabaseLongTermRetentionPolicy -Current

I will have a PR submitted and approved to have this corrected.

@asears  Thank you for pointing out the issue, we have updated the document content accordingly.
The changes should be live once the PR is approved.
Also the $server variable are used as below. Please let us know if any specific ones are not used and we can correct the document accordingly.

We will proceed to close this issue now.
If there are further questions regarding this matter, please comment and we will gladly continue the discussion.


Client Implementation in Go

Hello 👋
We are thinking about using Privacy Pass for anonymized authorization for a VPN-like service.
Currently we are getting to learn Privacy Pass and the server code a little closer. If we end up choosing Privacy Pass, we would start implementing the client functionality in Go. Here is where I have a couple questions:

Is a client implementation in Go something you would also be interested in?
Would you like a PR for it?
Is there anything you would like us to know before starting? Pitfalls, dragons or the like...

We are naturally not expecting you to help us; we would do the implementation ourselves and have it audited by a cryptographer. Just wanted to communicate and ask beforehand.

Hi @dhaavi , a Go implementation of Privacy Pass is something really useful.
We recommend to keep track on the changes happening on VOPRF draft and PrivacyPass working group, so the implementation remain compliant with the latest version.

Hey @armfazh, thanks for chiming in.
We're not actually creating a new implementation, but just adding some client functionality to what this repo already offers.
Adding client side functions wasn't as difficult as expected, as most of the logic was already in the tests. It just threw me off when the "client" in the tests started to use the server's private key.
I would like to create a PR for this (and a couple other) changes. Can a maintainer comment on that?
(I assume that you, @armfazh, don't have merge rights, correct?)
Also, would you prefer a bigger PR with more changes or many little ones?

@dhaavi can you say more about your use case? In particular:

What sort of requirements do you have for key rotation?
How will you ensure clients get the expected public key needed to verify tokens?
Do tokens need to carry any additional attributes beyond a single bit?
Do the tokens need to be verifiable by parties without the issuance private key?

Hey @chris-wood, thank you for your reply!

@dhaavi can you say more about your use case?

Of course! Here you go:

What sort of requirements do you have for key rotation?

For now, none. The parent system is versioned, which we can use for manual key rotation or breaking changes.

How will you ensure clients get the expected public key needed to verify tokens?

We ship it in the binary of the client.

Do tokens need to carry any additional attributes beyond a single bit?

For now, no. But it would be nice to have some additional attributes in the future.

Do the tokens need to be verifiable by parties without the issuance private key?

Yes, this would be very helpful.

I just realized that I somehow managed to mistake @armfazh for an external contributor. 🤦 Sorry about that!
Thanks a lot both of you!

Christopher Wood @.***> 于 2021年11月5日周五 下午9:57写道：

@dhaavi https://github.com/dhaavi can you say more about your use case?
In particular:

What sort of requirements do you have for key rotation?
How will you ensure clients get the expected public key needed to
verify tokens?
Do tokens need to carry any additional attributes beyond a single
bit?
Do the tokens need to be verifiable by parties without the issuance
private key?

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/privacypass/challenge-bypass-server/issues/17#issuecomment-961915413,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASEIZBQFNIRNJY7UCH7M4ZLUKPWFRANCNFSM5G4IXCLA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Change oj version to fix sensu installation using ruby 2.4

Change OJ version to fix Sensu installation using ruby 2.4.
Tests are passing.
The issue was opened on:
https://github.com/sensu/sensu/issues/1580

:+1:

Going to get this into the 0.28.1 release.


Setting Execution Role ARN in Lambda Integration Request - API Gateway

Describe the bug
I am using Java CDK to stand-up API’s in API Gateway service backed by Lambda function written in Java8.  In my customer’s account, I am not allowed to create Roles.  Roles are created by customer security team and given to the dev’s.
In API Gateway’s Lambda Integration request, I want to assign Execution Role in the Lambda integration request.  However, I’m running into a CDK Java SDK problem.  I can create IRole from a string using the Java IRole interface.  However, the LambdaIntegrationOptions.Builder only accepts a Role object in .withCredentialRole().  I don’t see any way to convert IRole to a Role object, nor I found any other methods in LambdaIntegrationOptions.Builder that will allow me to pass the Role ARN instead of Role object.
To test things, I created CloudFormation template manually and used a Role ARN (given to me by the dev’s) in AWS::ApiGateway::Method.  It worked.
To Reproduce
NA - Related to CDK SDK specifically with LambdaIntegrationOptions.Builder.withCredentialRole() method
Expected behavior
Provide a mechanism in LambdaIntegrationOptions.Builder to accept a Role ARN and not a Role object.  Enterprise customers typically restrict developers to create Roles in AWS accounts; Role creation in AWS account is managed by separate security teams.
So keeping the above use-case in mind, please make a CDK wide change where if you are creating a CloudFormation Resource that accepts Role object, make it accept Role ARN (using IRole interface).
Version:

OS - all
Programming Language - Java
CDK Version - 0.33.0

I am in a similar situation with my ability to create IAM roles via CDK. I am forced to use existing roles when creating stacks. However, for the python CDK SDK I am unable to specify a role for aws_gateway.LambdaRestAPI in order to send logs to CloudWatch. This is something @smoghal you'll want to check for the Java SDK since gateway is in your construct.
FYI the API reference for aws_lambda.Function has a role parameter documented here. Maybe the recent updates cleared things up for you
Version:
CDK: 0.35.0
Programming Language: Python

I have tested latest CDK 0.35.0.  LambdaIntegrationOptions.Builder.withCredentialRole() still doesn't accept a Role ARN.
Also, aws_gateway.LambdaRestAPI Python SDK is slightly different from what I read per your documentation reference.  I have no issue creating a Lambda function that accept Role ARN to send logs to CW.
The issue occurs in later stage when you want to assign the Lambda in APIG integration request.  The integration request also requires a role that APIG assumes to execute the Lambda.  This is where I'm stuck as the CDK SDK only allows you to specify a Role object (that CDK wants to create) instead of a Role ARN (created outside of CDK).

Yes, the credentialsRole property should be typed IRole and not Role.

Hi, is there any eta on this fix?  I have tried to use addPropertyOverride to override generated method resources.  It allows me to replace the generated Integration.Credentials at the method level.  However, the CDK execution still fails because now I have a dummy role in CFN that I can't delete.
I have to create a dummy role initially to properly construct LambdaIntegration object with .withCredentialsRole() which I can then later override with my Role ARN (above).  I haven't found a way to completely delete a resource from generated CDK template.  Apparently, addDeletionOverride only allows you to delete a property and not the entire resource.


nft: Basic operations

This PR introduces the go-nft basic functionality.
The schema follows the libnftables-json specification but does not implement it fully at this stage.
The user can read and apply the configuration using the nft wrapper functions:

Apply the configuration:

config := nft.NewConfig()
config.AddTable(nft.NewTable("mytable", nft.FamilyIP))
err := nft.ApplyConfig(config)

Read the configuration:

config, err := nft.ReadConfig()
nftVersion := config.Nftables[0].Metainfo.Version

Depends on #8

I will try to add a pair of comments for the API

change: Rebase


Pym Chile does height problem

Hi,
My CSHTML file has these lines of code

I noticed that when we navigate to an external URL the IFrame Height will be set to the height of the iframe before the navigation occurs.  What happens is that if I have a list with many items, where I have to scroll down to see all the content and from here navigate to the external url the iframe will have the height of that previous page with the long list of items.  If the previous page has 0 items making the height shorter the external url will be rendered on a shorter height resulting in a problem that only parts of the external url is visible.
To solve this does the external child page have to implement some javascript, pym code to be able to solve this height problem?
Thanks

Yes, if you navigate within the frame, all pages within the Pym parent must also use a Pym child to send their height back, or it won't know how tall to be.

What exactly does the external url page have to include in its code to solve this problem?

It would need to include the Pym script, and (at minimum) run new pym.Child() to initialize it. It might also need to call sendHeight() on the child object if the code inside dynamically changes height outside of a standard window resize.


Bump Go version to 1.22 & controller-gen to 0.14

All Submissions:

[ ] Have you signed our CLA?
[ ] Put closes #XXXX in your comment to auto-close the issue that your PR fixes (if there is one).
[ ] Update docs/release-notes/release-notes.md if your changes should be included in the release notes for the next release.

You need to run make licenses.csv

It seems you might also need to update controller-gen as well.

I think the e2e tests are getting a false cache hit (because we haven't actually changed any dependencies) and are still using 1.21 :(

This will need rebase now before merging, but hopefully it will get simpler after the few fixes in the CI.

We need to run make recompute-licenses and update. I would do that after rebasing from the merge of the release, rerun all suite and merge.


Sharing a virtual printer between unix-windows systems

Hi,
I've written a virtual printer based tea4cups which you guys introduced it to me, and I also wrote simple PPD file for that printer. Everything is fine for unix* systems. But when I want to share it via windows users, it needs a driver and windows default generic drivers does not suit me. How can I solve this problem?
It does not matter what sharing protocol is I mean IPP, SAMBA or even Bonjour. I used some cups' filters and windows drivers don't care about those filter, and they send application/octet-stream files to spooler directory. I don't want that, I want to be like unix system output which is pdf.

Can I write inf file based on PPD file?
Or any other suggestions are welcomed.

Please let me know, if there is any misunderstanding here.
Thanks

Sorry, there is no standard way to get PDF from a Windows client.


Can it add an example which using DaemonSet to deploy logrotate pod in every nodes of the cluster?

Can it add an example which using DaemonSet to deploy logrotate pod in every nodes of the cluster ?
So it can give the cluster enhanced ability to collector logs.

@yan234280533 I don't think it's a great idea overall:

AFAIK, logrotate (or preferrably, docker config) is set up as part of deployment tool for all popular tools, kubeadm, kopf, etc.
Logrotate doesn't allow to collect logs, more often it actually interferes in the logs collection process
Contrib repo is deprecated, we're going to remove it at some point, so let's not add more content there
DaemonSet is a poor choice for something that runs to completion, I'm not even sure API will allow it

I'm going to close the issue due to these reasons, feel free to reopen it if you feel that there's something else to consider


Extra dot "." in version tag release

Not sure if the extra dot in the release tag name v.2.0.3 was intended, instead of v2.0.3

fixed. Thanks :)

Thanks for the ultra quick fix! :rocket:


global refactor "Michalis" --> "Michaelis"

This is to address #99 , and upon @zoltuz 's request.
NB I am not aware of any tests, so non were ran; moreover, my familiarity with this package is limited to the SBtools June2020 bootcamp. Therefore, be as suspicious as possible -- things could be broken badly... :-) .
I ran:
sed -i 's/Michalis/Michaelis/g' ./biocrnpyler/*
and
sed -i 's/Michalis/Michaelis/g' ./examples/Parameters\ loading.ipynb
Verified using:
grep -rnw './' -e 'Michalis'
That all strings have been replaced.
Hope that was not too foolish...

Two comments:

Any code relying on MichalisMenten* , that is, before typo correction, would be broken by this refactoring. One way to avoid this is to include an alias  MichalisMenten* = MichaelisMenten*` ; and include a future deperication warning.

I've re-run the ipynbs from today's bootcamp, replacing MichalisMenten for MichaelisMenten, and all ran well. This is obviously not a complete test, yet an positive indication.

Thanks for your contribution! There are some hoops to jump, but bear with me! Fist, please rebase your branch to dev, so you get the latest updates:
git fetch upstream/dev
git checkout refactor_Michaelis
git rebase upstream/dev

Once you have that you need to force push:
git push -f

After that, we can see whether Travis can check/build your code.

Sure, my pleasure!
Thanks for the detailed instructions.
Here's an error I receive:
$ git fetch upstream/dev

fatal: 'upstream/dev' does not appear to be a git repository
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

$ git config --get remote.origin.url

https://github.com/eldad-a/BioCRNPyler.git

When you have a minute, I'd appreciate your advice.

Sure! Please have a look here: https://github.com/BuildACell/BioCRNPyler/wiki/how-to-sync-your-BioCRNPyler-fork
you should see something like this when upstream is correctly configured:
➜ git remote --verbose
origin  git@github.com:zoltuz/biocrnpyler.git (fetch)
origin  git@github.com:zoltuz/biocrnpyler.git (push)
upstream        git@github.com:BuildACell/BioCRNPyler.git (fetch)
upstream        git@github.com:BuildACell/BioCRNPyler.git (push)

To me it seems similar, what am I missing?
$ git remote --verbose
origin	https://github.com/eldad-a/BioCRNPyler.git (fetch)
origin	https://github.com/eldad-a/BioCRNPyler.git (push)
upstream	https://github.com/BuildACell/BioCRNPyler.git (fetch)
upstream	https://github.com/BuildACell/BioCRNPyler.git (push)

no idea, try this then
git fetch -all
git checkout dev
git merge
git checkout refactor_Michaelis
git rebase dev

Thanks!
Let's hope for the best :-)
Including the outputs for reference
$ git fetch --all
Fetching origin
From https://github.com/eldad-a/BioCRNPyler
   e9289a9..c165e88  master     -> origin/master
Fetching upstream

$ git checkout dev
Branch 'dev' set up to track remote branch 'dev' from 'upstream'.
Switched to a new branch 'dev'

$ git merge
Already up to date.

$ git checkout refactor_Michaelis
Switched to branch 'refactor_Michaelis'
Your branch is up to date with 'origin/refactor_Michaelis'.

$ git rebase dev
First, rewinding head to replay your work on top of it...
Applying: global refactor "Michalis" --> "Michaelis"
Using index info to reconstruct a base tree...
A	examples/Developer Overview.ipynb
A	examples/Parameters loading.ipynb
Falling back to patching base and 3-way merge...
Auto-merging examples/8. Developer Overview.ipynb
Auto-merging examples/5. Parameters.ipynb

$ git push -f
Username for 'https://github.com': eldad-a
Password for<EMAIL_ADDRESS>
Counting objects: 12, done.
Delta compression using up to 4 threads.
Compressing objects: 100% (7/7), done.
Writing objects: 100% (12/12), 1.17 KiB | 1.17 MiB/s, done.
Total 12 (delta 11), reused 5 (delta 5)
remote: Resolving deltas: 100% (11/11), completed with 11 local objects.
To https://github.com/eldad-a/BioCRNPyler.git
 + 4083ea6...427ebfa refactor_Michaelis -> refactor_Michaelis (forced update)

All green! Thanks for your contribution!

All green! Thanks for your contribution!

Following your directions made it effortless  :-)
Thank you!

no idea, try this then
...

May be worth including :
git fetch upstream dev
git checkout <branch_with_amazing_new_feature>
git rebase upstream/dev
git push origin <branch_with_amazing_new_feature>

In how-to-sync-your-BioCRNPyler-fork
for future PRs.
NB git fetch upstream dev without /, while git rebase upstream/dev with.


Allow status items with error background color (fix #110214)

This PR fixes #110214

@misolori I merged this to get it into our November release, feel free to tweak colors as you see fit. For now I set the background to red:
https://github.com/microsoft/vscode/blob/14d59bef9272a309da6c99aa28eee1a26a8bd530/src/vs/workbench/common/theme.ts#L398-L409


Docs README: remove block registration and add -U to pip install

Unless I'm mistaken, there is no block to register.

Closes #
Example

Checklist

[ ] This pull request references any related issue by including "Closes #<ISSUE_NUMBER>"

If no issue exists and your change is not a small fix, please create an issue first.

[ ] This pull request includes tests or only affects documentation.
[ ] Summarized PR's changes in CHANGELOG.md

These tests failed and all I did was change the README, fyi @ahuang11 .


feat: update template

[ ] I have considered whether this PR needs review, and requested a review if necessary.

Fixes issue #
Notes for reviewers
Reviewers can skip X, but should pay attention to Y.

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

Current dependencies on/for this PR:

#47  👈
#46
#44
#43
#41
main

This stack of pull requests is managed by Graphite.


Show compass in home nearby banner

Showing the nearest item is a great call to action.
But it only says like "200 meters", so the hardest thing next is to figure out in what direction to head.
I suggest showing an arrow that shows the direction.
Of course, it will not work great when the phone is held vertically.

hello, I am working on it

As of now showing directions/routes in the app is still debated in #2424.
It might be better to show a directions icon clicking on which fires a map intent with the route. Also, I sometimes feel that we should have more CTA options in the card itself. Maybe the same set of options which are shown on tapping a nearby item in the list.

@maskaravivek While directions are debated, I don't think the present enhancement is controversial?

Is this issue finalized and not assigned?

@mumtazcem Are you still working on this? Please inform us about your progress, or if you have abandoned the task then @ronanb80 will work on it. Thanks! :-)

@ronanb80 If there is no news from @mumtazcem within 24 hours, the issue becomes assigned to you. Meanwhile how about you research how to extract mediawiki captions for #2337 ? :-)

hello, I am deeply sorry that I could not update you guys.. Due to homeworks and midterms for the last two weeks I could not check my emails.
However, I was able to implement 'Bearing' which is explained here -> https://www.movable-type.co.uk/scripts/latlong.html successfully. Using this method I am able to calculate the bearing between given two points.
By checking the bearing value, cardinal points with precision 2 would be printed to the view just as @nicolas-raoul expected and can be observed below. Direction would be printed as string for now, I can update it with compass or with simple arrow.

Thanks for sharing your progress!
Yes, a compass (or simple arrow) is needed, because when walking in the street it can be difficult to tell where North-East is, and on the opposite it is easy to go in the direction of an arrow.
Thanks!

New update, I added compass icon. Icon would be rotated with calculated bearing value. Below you can observe that icon is rotated 73 degrees exactly.

This is the map as well.

Here I added some more examples. If the design and the icon are acceptable for you guys @nicolas-raoul , I can make a pull request for my changes.
1. First example

2. Second example

@mumtazcem
Great start!
If you turn your phone 30 degrees to the right, does the compass turn 30 degrees to the left to compensate?
If it does not, you should use Android's internal compass and include that in the calculation.
The idea is that if you hold your phone horizontally, the arrow points in the direction in which you must walk. So you can start walking even if you have no idea where the north is.

I actually never thought about that :). I assumed that this compass would help user to find out the nearest location in Nearby section. Thank you for your feedback, I will update you as soon as possible.

Thanks, and sorry for my approximate explanation :-)

This might help:
https://stackoverflow.com/questions/8315913/how-to-get-direction-in-android-such-as-north-west

Also:
https://bitbucket.org/alekseyt/compass/src/41d8dc3b1d40f8e5d79920aaffa8ca7c330850cd/src/com/sgr_b2/compass/navigation/Monitor.java#lines-214

This is maybe the closest code to what we are trying to achieve:
https://github.com/metinkale38/prayer-times-android/blob/bfc080918d68e3b0d2bda0ba0ab98b96f614e399/features/compass/src/main/java/com/metinkale/prayer/compass/_2D/Frag2D.java#L69
It displays an arrow that shows the direction of the Kaaba, Muslims can just look at the arrow and pray in the right direction. We would just have to replace the Kaaba's coordinates with the item's coordinates.

Not unfortunately all phones (mine samsung galaxy J5) have a magnetic sensor. I finally borrowed one for this project that is why I could not update you guys...
However, since having a magnetic sensor is possibility, should I alert the user if they have a magnetic sensor or not? If a phone does not have a magnetic sensor, then the compass would work just as I described above in current situation.

If the phone does not have any way to find the direction, then please just hide the arrow.
Thanks! :-)

Can I take it up now?!

@nicolas-raoul  if nobody is working on this issue i would like to take it.

Hi @LuGO0 and @318anushka,
Sorry for the late reply!
@LuGO0: OK feel free to work on it, as mumtazcem seem to not be around anymore. Good luck, let us know about your progress :-)

Hi, is this issue taken? If not I would be happy to try a solution!, btw, if u could give me more details, ill be glad :))

@XxBrezxX Please focus on one issue at a time :-) Thanks for your enthusiasm!

@nicolas-raoul I would like to work on this issue

@ShashwatKedia It is yours, thanks! :-)

@nicolas-raoul I've implemented the required changes. Kindly review the PR :-)


Fix OpenSimplexNoise get_image() swap axes

fix #30356

I'm not against this change, but I want to note that it breaks compat with 3.1

We can switch p_height and p_width instead of i,j in the for loop,right? Because i,j looks nicer than j,i. Just my opinion.

@raphael10241024 Is this still desired? If so, it needs to be rebased on the latest master branch. While there are no conflicts, rebasing is important so that reviewers can easily test the PR.
If not, abandoned pull requests will be closed in the future as announced here.

rebased

I also tend to prefer swapping p_width and p_height in the for loops so that we go through (X, Y) instead of (Y, X). But I also don't have a strong opinion either way, it's equivalent.

Thanks!

Cherry-picked for 3.4.
Note that this breaks compatibility. Users who want to preserve the behavior of 3.3 and earlier can swap the arguments in their call to get_image().


020 update design fields flyouts 020

Docs Build status updates of commit 5fee37d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/assets/images/components/componentAssets_mobile_fields.png
:white_check_mark:Succeeded

msteams-platform/assets/images/components/componentAssets_mobile_flyout_menu.png
:white_check_mark:Succeeded

msteams-platform/includes/design/fields-mobile-image.html
:white_check_mark:Succeeded

msteams-platform/includes/design/flyout-menu-mobile-image.html
:white_check_mark:Succeeded

msteams-platform/resources/design/components/fields.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.


How build Benchmarks project ?

I copy code from repo and build it:
0>Immediate.Handlers.Shared.csproj: Error NU1507 : Warning As Error: There are 2 package sources defined in your configuration. When using central package management, please map your package sources with package source mapping (https://aka.ms/nuget-package-source-mapping) or specify a single package source. The following sources are defined: nuget.org

Without more code, I cannot be certain what is missing in your setup, but I would encourage you to read https://learn.microsoft.com/en-us/nuget/consume-packages/Central-Package-Management, as IH relies on CPM for nuget package version management. Sounds like you have an issue with what you have set up with regards to the CPM expectations of the IH csproj files.

I'm not change code - only clone and run on Rider

I find
https://github.com/ImmediatePlatform/Immediate.Handlers/blob/cfb783a656127921c410d8f642c845450bc9982a/Directory.Packages.props#L3
if false = I can build project

I'm not change code - only clone this repo and run on Rider
You not change nuget.config on you PC ? may be add setting in

No. My computer is standard, and I just opened Rider on the project and it builds fine. Please ensure that you are not using a custom nuget.config anywhere that may be influencing the CPM to demand package source.
Note again the error: "There are 2 package sources defined in your configuration.". If you look, there is no nuget.config in the repository anywhere, so for additional sources to be defined, your machine must be doing so.

Thank you

BenchmarkDotNet v0.14.0, macOS Sequoia 15.0.1 (24A348) [Darwin 24.0.0]
Apple M2 Pro, 1 CPU, 10 logical and 10 physical cores
.NET SDK 8.0.200
[Host]     : .NET 8.0.2 (<IP_ADDRESS>11), Arm64 RyuJIT AdvSIMD
DefaultJob : .NET 8.0.2 (<IP_ADDRESS>11), Arm64 RyuJIT AdvSIMD

Method
Mean
Error
StdDev
Ratio
RatioSD
Rank
Gen0
Allocated
Alloc Ratio

SendRequest_Baseline
39.66 ns
0.478 ns
0.447 ns
1.00
0.02
1
0.0048
40 B
1.00

SendRequest_ImmediateHandler
56.63 ns
0.332 ns
0.278 ns
1.43
0.02
2
0.0048
40 B
1.00

SendRequest_ImmediateHandler_Abstraction
57.41 ns
1.022 ns
1.050 ns
1.45
0.03
2
0.0048
40 B
1.00

SendRequest_Mediator
65.08 ns
0.800 ns
0.748 ns
1.64
0.03
3
0.0048
40 B
1.00

SendRequest_IMediator
88.24 ns
0.414 ns
0.367 ns
2.23
0.03
4
0.0048
40 B
1.00

SendRequest_MediatR
184.28 ns
0.537 ns
0.503 ns
4.65
0.05
5
0.0668
560 B
14.00

Method
Mean
Error
StdDev
Ratio
RatioSD
Rank
Gen0
Allocated
Alloc Ratio

SendRequest_Baseline
38.76 ns
0.457 ns
0.428 ns
1.00
0.02
1
0.0048
40 B
1.00

SendRequest_ImmediateHandler_Abstraction
58.38 ns
0.095 ns
0.089 ns
1.51
0.02
2
0.0048
40 B
1.00

SendRequest_ImmediateHandler
61.40 ns
0.830 ns
0.776 ns
1.58
0.03
3
0.0048
40 B
1.00

SendRequest_Mediator
65.74 ns
0.202 ns
0.179 ns
1.70
0.02
4
0.0048
40 B
1.00

SendRequest_IMediator
77.79 ns
0.319 ns
0.299 ns
2.01
0.02
5
0.0048
40 B
1.00

SendRequest_MediatR
182.14 ns
0.683 ns
0.639 ns
4.70
0.05
6
0.0668
560 B
14.00

FYI, 8.0.2 is not the current version of net8- you should stay current and update to 8.0.10


Fix #2114 Resolve task dependency declaration

The generatePomFileForMavenJavaPublication task is dynamic and added
late, so anything that wants to depend on it must do so at the last
moment.
Also added a task graph visualisation plugin

test this please

The build failed from an issue that is probably linked to message tracker but that is unrelated to the PR. I will look into it. If the other build doesn't pass, someone who has the right can force merge it since it will be helpful to debug other issues.


Refactor rollbar to use new routing api + catalog entity annotations

Hey, I just made a Pull Request!
Refactored rollbar frontend to to use entities and the new catalog router:

Added new annotation rollbar.com/project-slug
Refactored frontend to use entities with rollbar.com/project-slug annotations
Refactored frontend to use hooks
Added EntityPageRollbar + Router to allow for creation of an Errors page for the catalog

NOTE: I have not integrated the router into the example app. This could be done similar to how #2282 is using a switcher or can be implemented by users following the instructions / pattern.

:heavy_check_mark: Checklist

[ ] All tests are passing yarn test
[ ] Screenshots attached (for UI changes)
[ ] Relevant documentation updated
[ ] Prettier run on changed files
[ ] Tests added for new functionality
[ ] Regression tests added for bug fixes

Build currently fails due to the organization prop.

@freben - Thanks! All relevant comments addressed and CI now passing.