Fix Delta <> Arrow schema timestamp

Description
This fixes an incorrect schema mapping for timestamp from delta to arrow
Documentation
The protocol talks about timestamp being a microsecond precision value with no time zone. The Arrow Time64 only stores the time portion of a timestamp, so the mapping was incorrect.

@fvaleye I have a suspicion that the delta protocol is incorrect on the representation of dates, but I need to check with some test data first. The delta date should be mapped to arrow::DataType::Date32, as Spark uses similar.
I'll address that separately though

@fvaleye I have a suspicion that the delta protocol is incorrect on the representation of dates, but I need to check with some test data first. The delta date should be mapped to arrow::DataType::Date32, as Spark uses similar.
I'll address that separately though

Ok thank you for letting me know!
I think that Delta uses DateType:

DateType: Represents values comprising values of fields year, month and day, without a time-zone.

It worth testing it and have another PR on this.

DateType: Represents values comprising values of fields year, month and day, without a time-zone.

That's the odd thing for me, because Spark says that the date type is stored as days since/before epoch. https://github.com/apache/spark/blob/master/sql/catalyst/src/main/scala/org/apache/spark/sql/types/DateType.scala#L36
Even with the timestamp type, I haven't seen the y,m,d,h,m,s format anywhere.


Can't use Hex Mirror/Proxy

Hi,
I was looking for an option to use a Hex Mirror / Proxy but couldn't find one in the README.md.
Looking in the codebase, it seems that it's hardcoded such that it will always contact https://hex.pm - would it be possible to have this url configurable via the toml?
Trying to use gleam within an org is difficult due to how package fetching currently works 😄
Thanks

Hello! Are you looking to deploy your own instance of this web application? I would love to hear about your use case. Thanks!

Hi @lpil, your response makes me less confident that I've commented on the right project 😆
it's specifically regarding pulling deps via the Gleam executable - reading the docs, I was led to believe that this project was what was used for that functionality but it could be that I'm wrong 😛
With the environment I'm working in, we can't access deps from an external source - instead, we have a security team download files for us which we upload to an internal Hex server.
Ideally, what I'm after is functionality within Gleam itself to specify the mirror to pull deps from 👍
Let me know if I've got the wrong project!

Ah! Fab fab!


Migration for supercool fields -> spicyweb

Hi,
I've recently joined SuperCool, and I am upgrading some of our older projects. These are still on Craft 3, and when I migrate them from supercool/tools to spicyweb/craft-odds-and-ends, the fields are not migrated.
These projects are not using Project Config, and so I do not believe /src/migrations/m220921_060212_update_developer_in_db.php covers this scenario, hence why I've had to run the attached migration manually.
It would be a big help if we could get this approved in helping us migrate to the plugin you now maintain (possibly adding this to your 4.x branch also to keep things consistent, especially with schemaVersions, etc.?)
Thanks,
Jamie (Supercool)

Fair enough - thanks!


[SPARK-5644] [Core]Delete tmp dir when sc is stop

When we run driver as a service, and for each time we run job we only call sc.stop, then will not delete tmp dir create by HttpFileServer and SparkEnv, it will be deleted until the service process exit, so we need to delete these tmp dirs when sc is stop directly.

Can one of the admins verify this patch?

You'll need a JIRA for this. I think it's probably a harmless change, but, a process contains one SparkContext, and isn't generally doing anything else but Spark, so what problem does this cause in practice?

@srowen we run a process as a service which will not stop. In this service process we will create SparkContext and run job and then stop it, because we only call sc.stop but not exit this service process so the tmp dirs created by HttpFileServer and SparkEnv will not be deleted after SparkContext is stopped, and this will lead to creating too many tmp dirs if we create many SparkContext to run job in this service process.

ok to test

Test build #27097 has started for   PR 4412 at commit b2018a5.

This patch merges cleanly.

Test build #27097 has finished for   PR 4412 at commit b2018a5.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27097/
Test PASSed.

Test build #27186 has started for   PR 4412 at commit 1339c96.

This patch merges cleanly.

@srowen thank you, now I add a member to store the reference of the tmp dir if it was created, please help to check again.

Test build #27186 has finished for   PR 4412 at commit 1339c96.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27186/
Test PASSed.

Test build #27199 has started for   PR 4412 at commit b968e14.

This patch merges cleanly.

@srowen thank you, please help to check again.

Test build #27199 has finished for   PR 4412 at commit b968e14.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27199/
Test PASSed.

Test build #27200 has started for   PR 4412 at commit fbbc785.

This patch merges cleanly.

Test build #27200 has finished for   PR 4412 at commit fbbc785.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27200/
Test PASSed.

OK, that's looking good to me. I'll pause shortly to let others review it too.


Pass custom origin information to rpm-ostree

This is useful for admins but the URL is also needed by the MCD to
compare the state of the system.

Example output:
[root@localhost ~]# rpm-ostree status
State: idle
AutomaticUpdates: disabled
Deployments:
  registry.svc.ci.openshift.org/rhcos/os-maipo@sha256:3728e44ceaa831a356239c50802d3bcb35a5a78fc37c3cbf018e747b55088a8b
              CustomOrigin: Managed by pivot tool
                   Version: 4.0.5470 (2018-09-11 18:30:44)

● ostree://rhcos:openshift/3.10/x86_64/os
                   Version: 4.0.4710 (2018-08-23 20:30:00)
                    Commit: 5b5c39225f985af0707dcf65640e9cbc2870a68af8c6251d0532c9f7ca652bbc

I was thinking of using pivot for updates as well for the time being (e.g. until we fold this back into rpm-ostree?). In that case, the canonical imgid passed to rpm-ostree is what will be used by the MCD to check if a host is updated.

Hah I was working on this too, I had
$ git diff cmd
diff --git a/cmd/root.go b/cmd/root.go
index 95a225c..81b3b1f 100644
--- a/cmd/root.go
+++ b/cmd/root.go
@@ -103,7 +103,9 @@ func Execute(cmd *cobra.Command, args []string) {

        // The leading ':' here means "no remote".  See also
        // https://github.com/projectatomic/rpm-ostree/pull/1396
-       utils.Run("rpm-ostree", "rebase", fmt.Sprintf(":%s", rev))
+       utils.Run("rpm-ostree", "rebase",
+               fmt.Sprintf("--custom-origin-url=pivot-oscontainer://%s", container),
+               fmt.Sprintf(":%s", rev))

        // Done!  Write our stamp file.  TODO: Teach rpm-ostree how to encode
        // this data in the origin.

Basically add pivot-oscontainer:// in front I think makes sense, WDYT?

(I was trying to test this but ended up hitting some 9p issue in FCOS that I didn't get to debugging)

Yeah, prefixing sounds good to me! Bikeshed: just pivot://? That, combined with Managed by pivot tool I think is clear enough, no?

Sure, SGTM

(Though in my head "oscontainer" is a transport format, "pivot" is a tool that parses it but eh)


[WIP] Don't move README to index on check if there exists an index file

I removed the logic to move the README to be the index of the documentation.
However this doesn't work straight away, as I tested this with https://github.com/wdv4758h/python-best-practices/tree/468cec7d100af41d4f35baa1728e31e7ba04fb7d
The problem here is that the conf.py is inserted automatically and assumes a index file to exist. The build output is this:
Exception occurred:
  File "/home/gregor/projects/readthedocs.org/user_builds/python-best-practices/envs/latest/lib/python3.4/site-packages/sphinx/environment.py", line 1272, in get_doctree
    f = open(doctree_filename, 'rb')
FileNotFoundError: [Errno 2] No such file or directory: '/home/gregor/projects/readthedocs.org/user_builds/python-best-practices/checkouts/latest/_build/doctrees-readthedocs/index.doctree'
The full traceback has been saved in /tmp/sphinx-err-mhhzzjwp.log, if you want to report the issue to the developers.
Please also report this if it was a user error, so that a better error message can be provided next time.
A bug report can be filed in the tracker at <https://github.com/sphinx-doc/sphinx/issues>. Thanks!
Traceback (most recent call last):
  File "/home/gregor/projects/readthedocs.org/readthedocs/projects/tasks.py", line 129, in run
    outcomes = self.build_docs()
  File "/home/gregor/projects/readthedocs.org/readthedocs/projects/tasks.py", line 353, in build_docs
    outcomes['html'] = self.build_docs_html()
  File "/home/gregor/projects/readthedocs.org/readthedocs/projects/tasks.py", line 369, in build_docs_html
    success = html_builder.build()
  File "/home/gregor/projects/readthedocs.org/readthedocs/doc_builder/backends/sphinx.py", line 146, in build
    bin_path=project.venv_bin(version=self.version.slug)
  File "/home/gregor/projects/readthedocs.org/readthedocs/doc_builder/base.py", line 121, in run
    return self.build_env.run(*args, **kwargs)
  File "/home/gregor/projects/readthedocs.org/readthedocs/doc_builder/environments.py", line 304, in run
    return self.run_command_class(cls=self.command_class, cmd=cmd, **kwargs)
  File "/home/gregor/projects/readthedocs.org/readthedocs/doc_builder/environments.py", line 335, in run_command_class
    raise BuildEnvironmentWarning(msg)
BuildEnvironmentWarning: Command python /home/gregor/projects/readthedocs.org/user_builds/python-best-practices/envs/latest/bin/sphinx-build -T -b readthedocs -d _build/doctrees-readthedocs -D language=en . _build/html failed:
Running Sphinx v1.3.1
making output directory...
loading translations [en]... done
loading pickled environment... not yet created
building [mo]: targets for 0 po files that are out of date
building [readthedocs]: targets for 2 source files that are out of date
updating environment: 2 added, 0 changed, 0 removed
reading sources... [ 50%] README
reading sources... [100%] misc

WARNING: master file /home/gregor/projects/readthedocs.org/user_builds/python-best-practices/checkouts/latest/index.rst not found
looking for now-outdated files... none found
pickling environment... done
checking consistency... /home/gregor/projects/readthedocs.org/user_builds/python-best-practices/checkouts/latest/README.rst:: WARNING: document isn't included in any toctree
done
preparing documents... done
writing output... [ 33%] README

Traceback (most recent call last):
  File "/home/gregor/projects/readthedocs.org/user_builds/python-best-practices/envs/latest/lib/python3.4/site-packages/sphinx/cmdline.py", line 245, in main
    app.build(opts.force_all, filenames)
  File "/home/gregor/projects/readthedocs.org/user_builds/python-best-practices/envs/latest/lib/python3.4/site-packages/sphinx/application.py", line 264, in build
    self.builder.build_update()
  File "/home/gregor/projects/readthedocs.org/user_builds/python-best-practices/envs/latest/lib/python3.4/site-packages/sphinx/builders/__init__.py", line 245, in build_update
    'out of date' % len(to_build))
  File "/home/gregor/projects/readthedocs.org/user_builds/python-best-practices/envs/latest/lib/python3.4/site-packages/sphinx/builders/__init__.py", line 316, in build
    self.write(docnames, list(updated_docnames), method)
  File "/home/gregor/projects/readthedocs.org/user_builds/python-best-practices/envs/latest/lib/python3.4/site-packages/sphinx/builders/__init__.py", line 354, in write
    self._write_serial(sorted(docnames), warnings)
  File "/home/gregor/projects/readthedocs.org/user_builds/python-best-practices/envs/latest/lib/python3.4/site-packages/sphinx/builders/__init__.py", line 362, in _write_serial
    self.write_doc(docname, doctree)
  File "/home/gregor/projects/readthedocs.org/user_builds/python-best-practices/envs/latest/lib/python3.4/site-packages/sphinx/builders/html.py", line 445, in write_doc
    self.handle_page(docname, ctx, event_arg=doctree)
  File "/home/gregor/projects/readthedocs.org/user_builds/python-best-practices/envs/latest/lib/python3.4/site-packages/sphinx/builders/html.py", line 786, in handle_page
    output = self.templates.render(templatename, ctx)
  File "/home/gregor/projects/readthedocs.org/user_builds/python-best-practices/envs/latest/lib/python3.4/site-packages/sphinx/jinja2glue.py", line 138, in render
    return self.environment.get_template(template).render(context)
  File "/home/gregor/projects/readthedocs.org/user_builds/python-best-practices/envs/latest/lib/python3.4/site-packages/jinja2/environment.py", line 989, in render
    return self.environment.handle_exception(exc_info, True)
  File "/home/gregor/projects/readthedocs.org/user_builds/python-best-practices/envs/latest/lib/python3.4/site-packages/jinja2/environment.py", line 754, in handle_exception
    reraise(exc_type, exc_value, tb)
  File "/home/gregor/projects/readthedocs.org/user_builds/python-best-practices/envs/latest/lib/python3.4/site-packages/jinja2/_compat.py", line 37, in reraise
    raise value.with_traceback(tb)
  File "/home/gregor/projects/readthedocs.org/user_builds/python-best-practices/envs/latest/lib/python3.4/site-packages/sphinx/themes/basic/page.html", line 10, in top-level template code
    {%- extends "layout.html" %}
  File "/home/gregor/projects/readthedocs.org/readthedocs/templates/sphinx/layout.html", line 1, in top-level template code
    {% extends "!layout.html" %}
  File "/home/gregor/projects/readthedocs.org/readthedocs/templates/sphinx/sphinx_rtd_theme/layout.html", line 121, in top-level template code
    {% block menu %}
  File "/home/gregor/projects/readthedocs.org/readthedocs/templates/sphinx/sphinx_rtd_theme/layout.html", line 122, in block "menu"
    {% set toctree = toctree(maxdepth=4, collapse=theme_collapse_navigation, includehidden=True) %}
  File "/home/gregor/projects/readthedocs.org/user_builds/python-best-practices/envs/latest/lib/python3.4/site-packages/jinja2/sandbox.py", line 355, in call
    return __context.call(__obj, *args, **kwargs)
  File "/home/gregor/projects/readthedocs.org/user_builds/python-best-practices/envs/latest/lib/python3.4/site-packages/sphinx/builders/html.py", line 776, in <lambda>
    ctx['toctree'] = lambda **kw: self._get_local_toctree(pagename, **kw)
  File "/home/gregor/projects/readthedocs.org/user_builds/python-best-practices/envs/latest/lib/python3.4/site-packages/sphinx/builders/html.py", line 712, in _get_local_toctree
    docname, self, collapse, **kwds))['fragment']
  File "/home/gregor/projects/readthedocs.org/user_builds/python-best-practices/envs/latest/lib/python3.4/site-packages/sphinx/environment.py", line 1238, in get_toctree_for
    doctree = self.get_doctree(self.config.master_doc)
  File "/home/gregor/projects/readthedocs.org/user_builds/python-best-practices/envs/latest/lib/python3.4/site-packages/sphinx/environment.py", line 1272, in get_doctree
    f = open(doctree_filename, 'rb')
FileNotFoundError: [Errno 2] No such file or directory: '/home/gregor/projects/readthedocs.org/user_builds/python-best-practices/checkouts/latest/_build/doctrees-readthedocs/index.doctree'

Any ideas how we could solve this easily and elegantly?
Related #1550.

Could we test in the auto-generated conf.py if a index.rst exists? Or is it possible that the source directory is different to the directory that the conf.py is in?

I think requiring an index file is a reasonable thing -- We also have logic for auto-creating an index file if it doesn't exist, which should be triggered in this case, no?

https://github.com/rtfd/readthedocs.org/blob/master/readthedocs/doc_builder/base.py#L90

Closing this for #1597 -- which I think is the proper solution.


[NSConcreteValue dataUsingEncoding:]: unrecognized selector sent to instance 0x167d8a30

[NSConcreteValue dataUsingEncoding:]: unrecognized selector sent to instance 0x167d8a30
There is a crash when I was in "Textview" enter of quickly.

when UITextView will create a newline, it crash


ci: bump github actions; reduce job permissions to minimum

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.6%) to 47.677%

Totals

Change from base Build<PHONE_NUMBER>:
-0.6%

Covered Lines:
181

Relevant Lines:
343

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.6%) to 47.677%

Totals

Change from base Build<PHONE_NUMBER>:
-0.6%

Covered Lines:
181

Relevant Lines:
343

💛  - Coveralls


can't connect to retroachievements unless ipv6 is disabled

RGB30 running ArkOS v2.0 (11/04/2023) Update.
No idea why, but the device cannot connect to retroachievements.org unless I first disable ipv6.
Disabling ipv6 currently seems to require ssh-ing in and running a few commands via CLI.
It would be great if there were a toggle under Options to disable ipv6 permanently.

That's an issue with your network and retroacheivements.org not the OS.  I have no such issues on my end.

Even so, I imagine it wouldn't be super difficult to implement a simple toggle as I described. I can provide the commands I used, if need be. JelOS already has this implemented.

I may consider this in the future.  Not something I'm interested in implementing at the moment.  I'm aware how this can be achieved already.  Thanks for the offer.


Update datasaturday0005.html

Activating links for call for speakers page.

Question: How do we upload images for the sponsors' logo?
TIA.

Sponsor logos can be uploaded to the /events/assets/0005/ folder. Each event has its own assets folder. Have a look at 0001 for instance.

Sponsor logos can be uploaded to the /events/assets/0005/ folder. Each event has its own assets folder. Have a look at 0001 for instance.

Sponsor logos can be uploaded to the /events/assets/0005/ folder. Each event has its own assets folder. Have a look at 0001 for instance.

@spaghettidba, I do not have the right to create the "0005" folder in /events/assets or upload files.

Sponsor logos can be uploaded to the /events/assets/0005/ folder. Each event has its own assets folder. Have a look at 0001 for instance.

@spaghettidba, I do not have the right to create the "0005" folder in /events/assets or upload files.


⚠ Slack has degraded performance

In cbc6e97, Slack (https://status.slack.com/api/v2.0.0/current) experienced degraded performance:

HTTP code: 200
Response time: 214 ms

Resolved: Slack performance has improved in 2569ab2 after 1 hour, 50 minutes.


MarkupLabel total application crash on bad markup

Describe the bug
Bad bbcode involving text size crashes due to line 212 of this file:
https://github.com/kivy/kivy/blob/c28c47b39ae57c97c70cc0398d74774d73a6894b/kivy/core/text/markup.py
As there seems to be no easy way to cleanly handle it(at least without rewriting a lot of my code). the whole app crashes, because there is no error until the rendering phase.
Expected behavior
The app should either be forgiving and use a sane default, or provide a way to cleanly handle it(Peraps with a test_markup function to ensure correctness), as the actual error occurs in a place that can't be handled nicely.
As it is purely visual, and there already seems to be code to handle it in a forgiving way, I suspect some testing code snuck in(It's a raise statement right before the code that would have used a default.
To Reproduce
Try this markup, it is from the official bbcode site:
[size=14]g[/size][size=18]r[/size][size=22]o[/size][size=26]o[/size][size=28]w[/size][size=32]s[/size]!!!
or [style size="10px"]does it?[/style]

Before I even look at the actual bug, the code that is pointed to (and still exists in Master, as of today):
            try:
                if item[-2:] in ('px', 'pt', 'in', 'cm', 'mm', 'dp', 'sp'):
                    size = dpi2px(item[:-2], item[-2:])
                else:
                    size = int(item)
            except ValueError:
                raise
                size = options['font_size']

So, the try block achieves nothing, apart from making the last line unreachable.
If nothing else, this should be refactored.


Added Navbar To Contact Us PageFeature/added navbar

Related Issue
Added a navbar to the contact page to make it easier for users to navigate between sections
Fixes:  #821

Type of PR

[ ] Bug fix
[x] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Checklist:

[x] I have made this change from my own.
[ ] I have taken help from some online resources.
[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers and screenshots after making the changes.

@Abhi-Jeeet Please see the conflicts . Resolve the conflicts . Ensure you pull all the latest changes before pushing your feature/bug fixes to the main branch. After resolving the conflicts, please open a new PR for review.
Thank you!


Only have one set of examples and test that they are valid

We ended up in a state where we had both an examples dir and a
samples dir, and neither were tested, so both were slightly out of
sync with each other and the actual types.
This was b/c:

When we used kubebuilder, we generated our types from an original set
of examples - previously these were in config/samples and they ended
up in samples. We used the validation tests generated by kubebuilder
to test these, but when we removed kubebuilder we removed the tests
Before presenting the API to the Build CRD working group, we wanted to
have some more complex, real world examples, which @tejal29 created
from the kritis project and the k8s guestbook example

This commit makes sure that all of the functionality demonstrated in
samples is in examples, and removes samples. This included:

Fixing passedConstraints to be plural as the types expected
Including references to the PipelineParams in the TaskRun example
(removing the duplicated list of result endpoints, since we'll be
getting this from the references instead)
Fixing clusterBindings to refer to the actual names of the clusters
involved

It also adds a step to the integration tests to deploy the examples,
which will fail if they do not match the expected schema. At this point
they are torn down immediately after creation, but in #108 we can expand
this to actually test that they are working.
Also had to make some tweaks to the types:

actually including ClusterBindings in the Pipeline
Using names which include Resource instead of Source, which we
moved away from in #39

Fixes #20

W1008 16:45:49.586] pkg/reconciler/v1alpha1/taskrun/resources/input_resources.go:37:40: task.Spec.Inputs.Sources undefined (type *"github.com/knative/build-pipeline/pkg/apis/pipeline/v1alpha1".Inputs has no field or method Sources)
```

oo i broke building!

I think now that #86 is merged I should probably fix the conditions as well :thinking:

has a field token that doesn't exist in the type definition

good call!

using prevTask and nextTask that are no longer part of the type definition

ah yeah that's right - I think we're still going to need something to represent the order the tasks executed in :thinking: but i'll remove this till we figure it out!

Should be ready for another look @pivotal-nader-ziada @vdemeester !


raised timeout to 1m from 15s as it always shows timout before finishing fetching

Description
Fetching project takes some time and 15s always result in a timeout. raised timeout to 1 minute
Type of Change

[ ] ✨ feat -- New feature (non-breaking change which adds functionality)
[x] 🛠 fix -- Bug fix (non-breaking change which fixes an issue)
[ ] ❌ ! -- Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🧹 refactor -- Code refactor
[ ] ✅ ci -- Build configuration change
[ ] 📝 docs -- Documentation
[ ] 🗑 chore -- Chore

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


Addressed comments from PR (#7) Search and acquire build tools.

This PR addresses the comments in PR https://github.com/ravimeda/corefx/pull/7
Since there were significant changes, I am sending out a fresh PR. Summary of changes:

Removed named arguments
Repository root is the new argument passed to scripts

A concern I have with these changes is a few arguments passed to scripts are not used anywhere within the script. These unused arguments are being passed due to the contract invoke-extensions maintains. Any suggestions for improvement is greatly appreciated.
@MichaelSimons two comments from PR https://github.com/ravimeda/corefx/pull/7 on .toolversions file have not been addressed since I'll need some clarification.
@dagood PTAL.

@ravimeda, Can you provide a link to the comments that need clarification?

https://github.com/ravimeda/corefx/pull/7#discussion_r110402067
Slightly unclear about the multiple asks in here.
https://github.com/ravimeda/corefx/pull/7#discussion_r110402017
When you say sort on property names, is it like:

DeclaredVersion
DownloadFileLinux
DownloadFileOSX
DownloadFileWindows32
DownloadFileWindows64
DownloadUrl
SearchPathLinuxTools
SearchPathOSXTools
SearchPathsWindows
SearchPathWindows32Tools
SearchPathWindows64Tools
ToolNotFoundMessage

When I previously talked about removing the repo dependency for the local tools directory, I didn't intend for it to be passed in a parameter.  My thought is there should be a new .toolversions property that indicates the local path (probably needs one per OS).  If it isn't specified then we would default it to the repo/tools directory.  This will allow a shared tools location across the dotnet repos if desired.  I would not expect to see the Tools/downloads path segment in the search paths.

@ravimeda, yes that is the order I would expect.

Yes, "Tools\Downloads<tool-name>" is where the scripts will search for a tool. If tool is not found then, scripts will download to that folder. I will update .toolversions and helper functions accordingly.
As clarified offline, repo root is being passed since we cannot assume the scripts to be within the repo in tools-local folder. For example, override scripts can be outside the repo. Hence we need to provide the repo root.  I should have described the changes in more detail. Sorry for any confusion.

In reply to: 292554747 [](ancestors = 292554747)

Summary of changes:

Updated the layout of tool metadata file (.toolversions). Changes include config names are alphabetical, each config name begins with functionality followed by OS, and reduce the overlap in config values.
Ensured arguments to scripts are all required. Thus removing the need for named arguments.

Comments that were not addressed yet revolve around adding helpers.  I marked these as "Pending".
Hence to make efficient use of your time, please focus on .toolversions changes, parsing of toolversions info, and arguments passed to scripts.

Commit https://github.com/ravimeda/corefx/pull/9/commits/1ee4e0b01e65555c05a9610354815c40ae464290 addresses all pending comments. @MichaelSimons @dagood PTAL.

Two changes in https://github.com/ravimeda/corefx/pull/9/commits/6c9bd39f666aa2759b8e8ea636b0b52792818910

Introducing PowerShell scripts
Addressed all pending comments on Shell scripts

@dagood @MichaelSimons PTAL

What do you think about changing the method/script naming scheme to follow PowerShell standard convensions?  Log-Message vs log_message.  Also I think it may be more natural to utilize Write-Error/throw exceptions/return error codes versus returning error messages throughout the scripts.

Weren't we going to start using VSTS to review the powershell scripts?

Log-Message vs log_message

Agree. But the challenge here is many function names will be different between Window and Unix scripts.
Naming convention in PowerShell follows a -, and there are certain prescribed verbs to follow as described at https://msdn.microsoft.com/en-us/library/dd878238(v=vs.85).aspx#RD01
Based on that, search_environment should be find_environment, which is absurd and so it should be renamed as 'find_tool'.
If we are OK with function names that are vastly different between Windows and Unix then, we can take this route.

utilize Write-Error/throw exceptions/return error codes versus returning error messages

Disagree. I think we chatted about this in the past.  Write-Error or Write-Warning returns extra info such as file, line and character where the event occurred, and some formatting.  This was the reason why the existing scripts used Console.WriteLine to return the plain text without extra info.

Weren't we going to start using VSTS to review the powershell scripts?

It was less pain dealing with CodeFlow-GitHub then maintaining a personal VSTS repo.

@ravimeda, I don't recall discussing Write-Error or Write-Warning with you.  I am not as concerned with the constructs used.  What I am more concerned with is the design pattern.  I don't think overloaded return values should be used with PowerShell.  IMO we should be utilizing the power of PowerShell versus making the scripts as identical as possible to the bash scripts.

IMO we should be utilizing the power of PowerShell versus making the scripts as identical as possible to the bash scripts

So is my opinion @MichaelSimons . What I am trying to convey is language's power should be used to fit the need and not vice-versa. Let's Skype.

Please use VSTS based CodeFlow at -
codeflow:open?server=http://codeflow/Services/DiscoveryService.svc&review=raeda-2da4a7c890a74e7d9d5e9d4de27049ce&alert=true


[Feature request] Possibe refinement of tag summary function

When using the below syntax to summarise certainly tags, the images in the insulting view are plain texts instead of rendered images.
include: #tag1 #tag2

Currently workaround
Using the flatten summary button to transport all the search results to the current note, and images will be rendered as they are embedded in the current note.
Problems with a current workaround
One of the possible problems with using the flatten function to summarise headings is that may cause multiple versions of certain headings since one tends to edit the summarised results. Such editing cannot synced to the original heading.
Another problem is that the flattened results will be displayed in the future tag summary syntax causing clutter and might be confusing.
Possible solution

Make images rendered in the tag summary syntax.
(might be harder but more practical) Making an option to embed the heading of the tag summary results in the current note. (I often add tags to the headings of notes, for tags added at the end of a paragraph, maybe embedding the closest heading above the tags could be a possible solution)

Should be fixed now. It's the same versions number, so you'll need to uninstall and reinstall the plugin.

Thank you for the fix. The images is rendered now. is it possible for you to implement "Making an option to embed the heading of the tag summary results in the current note. (I often add tags to the headings of notes, for tags added at the end of a paragraph, maybe embedding the closest heading above the tags could be a possible solution)" if you have time?

Currently, the flatten summary button can copy the summary contents to the current note, but this leaves possible risks of causing duplication in the future summary and version differences between the same content

Hey there. I'm curious to get a better understanding of the use case. What do you mean by the heading of the tag summary - something outside the summary code block? For your second point, the plugin will exclude any queried tags in the note where the summary is generated.

When I read research papers, I often collect all the images of the relevant papers in a single note. Every image has its headings and I can tag the image in its respective heading with the information I'd like to extract from the image. The heading contains a double bracket link to their citekey (Zotero generated) which could help me to identify the source of the image. the following image is an excerpt of such a note.

Since tag buddy allows tags at the end of the paragraph, some of my old notes have such paragraphs that have tags at the end. So I mimic the scenario in the above example.
The above note could be considered my knowledge base
Let's say one day, my professor finds me and tells me that, I found microalgae could be a good research topic, what do you think, can you tell me what resource you need if you have to carry out related research?
Then I will create a new note and conduct a survey about microalgae.
I will have a tag summary query about microalgae first
include: #microalgae

Then it will help me find all the tagged headings and paragraphs

Now let's say we are particularly interested in rapid curve and want to survey how it could be measured or produced

If I use ctrl + hover on the link, I can open the note and edit it, but I have to scroll down to find the tagged heading.

If I use the flatten summary button, I can further edit the rapidcurve part but the note will not sync to the original heading, if I query something in a new note like test3, there will be two results with overlapped information, which is confusing when such notes added up.

Using Heading embed could edit the content in the hover editor and will not cause duplication in future search

This method allows incremental reading of research papers, one can query any tagged contents and further edit it without worrying about causing duplication or version differences. I think it's a great feature.

Wow, super thorough. And I feel you with the need to make it work in just the way you want. It's why I made the plugin, nothing working quite I like I wanted. That said, I think I understand the functionality now. Unfortunately, tagged content is grouped by "blocks." That is, any block of text between empty lines. The 4 lines below are one block.
something #tag
Some heading #anotherTag
Details
This and that.
So that empty space knowledge might be the detail you need to make it work how you want. Otherwise, detecting/pulling a heading from one of the summary items is out of scope right now. However, at one point I generated a heading when baking the summary. I've since moved that to when you flatten the summary to a new note. I could bring that kind of thing back as a setting for baking within the note. But check it out, see if it would work for your needs.
I'm also curious if you've used the dateview/query plugin? It's much more robust and might be better suited.

I see, you mean that Obsidian offers no function to locate headings with tags, and when a tag is found in a note, the whole page will be considered a block, and it's impossible to apply heading embed or heading link in the plugin.

I reinstall the plugin and haven't see any thing like this

Thank you for your time, I will try dataview and see if there is a solution to this

Could you make this link a link to heading instead of note？

The closest query for this that I found could not address the issue, it could only query the headings with fixed name https://share.note.sx/47oiqm8g

I assume test_1 is the note link generated for each of the summary items? If so, I think that could be possible. So instead of it linking to the source note, it would look for the first heading in the block and make an anchor link to that. How does that sound?
About the missing buttons, check setting and switch the visibility options in case the old settings haven't updated. Also, might need to completely uninstall the plugin and reinstall. Only need to do this while I make these minor tweaks with the same version (not best practice, I know :)
I mention testing the buttons because copying a summary item to a section/heading in the current note (or another note) it does anchor link to the heading specified (if it exists).

Thank you for putting your valuable time into this feature. I completely reinstalled the plugin and found the button was refreshed, but the function changed, and some of them not working properly

The copy paragraph link function produce link to the note

"I assume test_1 is the note link generated for each of the summary items? If so, I think that could be possible. So instead of it simply linking to the top of the source note, it would look for the first heading in the block of that summary item and make an anchor link to that. How does that sound?"
this is exactly what I want

Could you make this link a link to heading instead of note？

[[test1]] in the tag buddy summary code blocks
could be
[[test1#heading（where tags were found）]]

The previous example might be to long to demonstrate my use case
here is a more concise one:
note 1

title: test 1
contents:
heading 1
paragraph1
#tag1
#tag2 heading 2
paragraph2
#tag1 heading 3
paragraph3
note2

title: test2
include: #tag1

The query result could be
[[test1#heading 1]]
<tag buddy block for the content of test 1#heading1>
[[test1#tag1 heading 3]]
<tag buddy block for the content of test1#tag1 heading 3>
(note: the # mark will be ignored by the obsidian heading link)
OR
![[test1#heading 1]]
![[test1#tag1 heading 3]]
(note: the # mark will be ignored by the obsidian heading link)
Instead of
[[test1]]
<tag buddy block for the content of test 1#heading1>
[[test1]]
<tag buddy block for the content of test1#tag1 heading 3>
(note: the # mark will be ignored by the obsidian heading link)

test 1.md
test 2.md
test 3.md

For the anchor linking. Should be no problem. I'll put it on the todo list. A lil donation would be appreciated tho :)
When TB finds a tagged block, it will look for the first instance of a heading and used that for the link.

completely reinstalled the plugin and found the button was refreshed, but the function changed, and some of them not working properly

I'd need to investigate the file it's trying to copy to. Can you send it, or let me know if there's any special kind of formatting at the end of the file?

Maybe I misunderstood the button‘s function, I pressed the copy paragraph link button in a new note named untitled 1 and found the resulting link was a link to note the note title instead of the paragraph. maybe the formatting of the test1.md caused the malfunction, Which I have already uploaded.

As for donations, I want to support the development of the plugin. However, I cannot access a visa or any international payment system. I have access to Alipay, which is not supported by the website.

There are two options in the 0.6.8 version of the plugin

Hi. Although I can't donate for the time being. I promise will write a showcasing article in the community forum to share my workflow and help promote the plugin, a game changer that allows using tags in Obsidian to make a section-level search and collection of information, which I have never seen before (Tag summary only allow paragraph level of query and  collection of information)

Hi. Although I can't donate for the time being. I promise will write a showcasing article in the community forum to share my workflow and help promote the plugin, a game changer that allows using tags in Obsidian to make a section-level search and collection of information, which I have never seen before (Tag summary only allow paragraph level of query and  collection of information)

That'd be great. Jammed up with work right now. But I'll check back when I
get into another sprint :)
On Wed, Apr 10, 2024 at 12:35 PM robert zhang @.***>
wrote:

Hi. Although I can't donate for the time being. I promise will write a
showcasing article in the community forum to share my workflow and help
promote the plugin, a game changer that allows using tags in Obsidian to
make a section-level search and collection of information, which I have
never seen before (Tag summary only allow paragraph level of query and
collection of information)
—
Reply to this email directly, view it on GitHub
https://github.com/moremeyou/Obsidian-Tag-Buddy/issues/12#issuecomment-2047174063,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACEOMHG4MQHLYZT2MAPCSMLY4UIYBAVCNFSM6AAAAABFQDKAP6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANBXGE3TIMBWGM
.
You are receiving this because you commented.Message ID:
@.***>

Hey @Ravenclawer,
I've implemented this feature. It will be released with the next update. But if you want to use it right now, download the latest main file and replaces it in the Tag Buddy plugin folder.
Note that the link NAME will still be the file name. But the link itself will go to the closest header in that tagged paragraph. I'd love to hear what you think and/or if you have any issues.
D

Thank you for implementing the function, maybe I have used the wrong file, I have download the main.ts file and changed the file name to main.js. Then, I replaced the file in the obsidian plugin folder. When I came back to obsidian community plugin interface, the Tag buddy cannot be activated.

Hmm, I had to upload the dev environment recently. Maybe it needs
supporting files on my end. Try this pre-made .js file
https://github.com/moremeyou/Obsidian-Tag-Buddy/blob/main/src/main.js
D
On Wed, Dec 4, 2024 at 4:15 PM robert zhang @.***>
wrote:

Thank you for implementing the function, maybe I have used the wrong file,
I have download the main.ts file and changed the file name to main.js.
Then, I replaced the file in the obsidian plugin folder. When I came back
to obsidian community plugin interface, the Tag buddy cannot be activated.
—
Reply to this email directly, view it on GitHub
https://github.com/moremeyou/Obsidian-Tag-Buddy/issues/12#issuecomment-2517727448,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACEOMHF5A744E3XZE3FRHDL2D4MALAVCNFSM6AAAAABS56VT4CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMJXG4ZDONBUHA
.
You are receiving this because you commented.Message ID:
@.***>

Thank you for making a pre-release, currently the generated syntax cannot be rendered and jump to the paragraph. I think it's the addition of tags in the heading that caused the problem. Please check out the attached md file, and put them into an ob
test1-database.md
test2-lookup.md
sidian vault and you can better understand the problem。

Maybe obsidian developed a strategy to omit some special letters in the heading to avoid confusion

Hey, thanks for sharing those files.
I think I've cracked it. Try this main.js file
https://github.com/moremeyou/Obsidian-Tag-Buddy/blob/main/src/main.js.
I think I've reached the limit of how many tweaks I can make to the link
generation. Tbh, I'm surprised these complex links work. Generally I'd
suggest clean headers. Keep any complex formatting out of headers you want
to link to. But I am pleased it seems to be working.
D
On Fri, Dec 6, 2024 at 2:12 PM robert zhang @.***>
wrote:

Maybe obsidian developed a strategy to omit some special letters in the
heading to avoid confusion
—
Reply to this email directly, view it on GitHub
https://github.com/moremeyou/Obsidian-Tag-Buddy/issues/12#issuecomment-2523228840,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACEOMHBI3LS3JE6UJWT3LYD2EGPDHAVCNFSM6AAAAABS56VT4CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMRTGIZDQOBUGA
.
You are receiving this because you commented.Message ID:
@.***>

Thank you so much for your time and effort. It's working like a charm.


Sample micro-services app to exercise EventBridge-Express library.

Closing this ticket. This app is being built and version controlled here: https://github.com/Panda-Whales-21-22/expressbridge-demo-lambdas


The pom.xml of storm project

If you can give us a "pom.xml" example of storm project .
Also it can exclude storm from the Maven Assembly Plugin, otherwise we will get duplicate default.yaml problem.
Thanks

See https://github.com/nathanmarz/storm-starter/blob/master/m2-pom.xml


Push to multiple Users: Return HTTP-200 if no registration was found

In order to simplify consumption, consider this cases as expected and don't throw exception, but return an empty push message result list

Completed


Metro Icon Font sometimes doesnt show up

We use Metro-UI-CSS and its  Metro Icon Font feature in our project.
But sometimes, the fonts do not show up in some embedded browsers.
What might be the causes ? Is there any work around ?
Thanks very much for the  Metro-UI-CSS team!

This is the URL for the page.
http://www.buit2b.com/buit2b/guest/tablecodeseq/8a101c315b426316015b461438050496?tableid=seq

All ok for me

Thanks for confirmation!

which browser are you using?

There are two cases, in which we encountere this problem.
1, In the WeChat embedded browser (The screenshots provided)
2, In our own android app using the embedded browser(webkit). If we reinstall the app, the problem disappears.

At the moment I do not support Metro 3. At least until I gain enough financial support for Patreon. Sorry. No time to support all versions.


Add description for anchor links

@julien-c https://github.com/huggingface/doc-builder/blob/d6126583f078b8b7be60a1c1fbbf2af9853c3eef/src/doc_builder/build_doc.py#L280-L287
the pr is #15

Thank you!! I didn't know that feature, it's very useful.

I guess we added this feature before we wrote the README, but luckily we document it now 😊


'Title' column heading misaligned in explorer for users without 'publish' permission

For users without publish permission, the 'order' column of the explorer is omitted. This results in the 'title' column heading being pushed too far to the left:

Rather than adding a special case in the CSS to give it an extra margin, I think it would be cleaner to consistently include the 'order' column all the time (and leave it blank for users without the relevant permission) - that way we avoid additional special-casing further down the line (e.g. https://github.com/torchbox/wagtail/issues/1583#issuecomment-131078851 - adding a row to contain a help message will require  or ). This involves some rewiring of the table logic, though.

Agree with this approach. Will action.

PR: https://github.com/torchbox/wagtail/pull/1611


Need to provide command to disable the AAD Pod Identity add-on

In this documentation of AAD Pod Identity add-on we also need to provide command to disable the add-on.
https://docs.microsoft.com/en-us/azure/aks/use-azure-ad-pod-identity#clean-up
We might need to add addition command as below either under cleanup section or as a separate section.
az aks update -g podidentityrg -n podidentitytest --disable-pod-identity

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

ID: c3c187c0-27f1-11c9-41af-6c65ca36e77e
Version Independent ID: 249a570b-90ba-4a11-4467-320f0369ec25
Content: Use Azure Active Directory pod-managed identities in Azure Kubernetes Service (Preview) - Azure Kubernetes Service
Content Source: articles/aks/use-azure-ad-pod-identity.md
Service: container-service
GitHub Login: @zr-msft
Microsoft Alias: zarhoads

@saikumar0412 Thank you for your feedback!  We are reviewing the cleanup instructions and will get back to you shortly.

@saikumar0412 Thank you again for your feedback!  I've raised this with the documentation team and they've added this to their backlog this month for further review.  We are going to close this thread as resolved but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.


document installation via curl download

When I installed pusrm (on Linux) into $HOME/.docker/cli-plugins/docker-pushrm my docker installation (version 19.03.12-ce) did not find the plugin.
Therefore I changed the path to /usr/local/lib/docker/cli-plugins/docker-pushrm as it is e.g. described here: https://github.com/docker/ecs-plugin/blob/master/docs/get-started-linux.md#step-by-step-install
Futhermore I added a command to download the latest version with curl. This might be handy for faster installation/updates.

@langfingaz - thanks for this.
Which Linux distro + version are you using and how did you install Docker (i.e. from repos provided by the Linux distro or by adding Docker Inc repos or from https://get.docker.com, etc)?

@christian-korneck you're welcome. Thanks for this great plugin!
I'm on Arch Linux and have docker (and docker-compose) installed with pacman, so via the distro repos: https://www.archlinux.org/packages/community/x86_64/docker/

hi @langfingaz ,

When I installed pusrm (on Linux) into $HOME/.docker/cli-plugins/docker-pushrm my docker installation (version 19.03.12-ce) did not find the plugin.

I just tried on archlinux, but couldn't reproduce your issue. Copying the plugin to $HOME/.docker/cli-plugins worked fine for me.
I have experienced similar issues myself but it always came down to installation mistakes such as:

downloaded file with curl <url> instead of curl -L <url> (which leads to ending up with a html file instead of the executable)
wrong permissions (forget to chmod or chmod'ed as wrong user (i.e. because of sudo))
copied to the wrong user home dir (i.e. because of sudo)
tested with the wrong user (i.e. because of sudo)

I assume something like this could have been the cause for your problem, too?
This is how I tested on archlinux:
Dockerfile:
FROM archlinux
USER root
RUN pacman -Sy --noconfirm docker jq
RUN export FILENAME=docker-pushrm_linux_amd64 && export DESTDIR=$HOME/.docker/cli-plugins && mkdir -p "$DESTDIR" && DOWNLOAD_URL=$(curl --silent --fail --show-error https://api.github.com/repos/christian-korneck/docker-pushrm/releases/latest | jq -r ".assets | map(select(.name == \"$FILENAME\"))[0].browser_download_url") && curl --silent --fail --show-error -L $DOWNLOAD_URL -o "$DESTDIR/docker-pushrm" && chmod +rx "$DESTDIR/docker-pushrm"
CMD /bin/bash

run:
$ docker build -t archtest1 .
$ docker run --rm -ti archtest1 docker --help | grep -i pushrm
   pushrm*     Push Readme to container registry (1.4.2)

Therefore I changed the path to /usr/local/lib/docker/cli-plugins/docker-pushrm as it is e.g. described here: https://github.com/docker/ecs-plugin/blob/master/docs/get-started-linux.md#step-by-step-install

That's a valid solution for installing the plugin for all users (it's already mentioned in the README, under all users install). The reason why the README promotes the installation to the user home dir is that it doesn't require admin/root privileges and "just works" almost anywhere (even on restricted CI runners, Azure Cloud shell, etc).
But the fact that you didn't find the info in the README is good feedback - it has become a bit bloated, I guess. Might need a rework.

Futhermore I added a command to download the latest version with curl. This might be handy for faster installation/updates.
Thanks. I'll keep this PR open for the moment until the README gets reworked.

You're probably right that there might be some mistake from me because of using sudo ;)
I had copied the plugin to $HOME/.docker/cli-plugins/docker-pushrm and /root/.docker/cli-plugins/docker-pushrm but neither of them worked when I ran docker with sudo docker pushrm --help. (For the first I chmod'ed with chmod +x and on the second with sudo chmod +x)
And yes, I've overseen the installation for all users part of the README. Moving this up to the install section would be a good thing I guess. Or maybe put it in a separate all-user-install.md file and just reference it to keep the main RADME not too long?
Thanks for your detailed reply!

@langfingaz - I've now placed the info for how to download and install with curl here:
https://github.com/christian-korneck/docker-pushrm/wiki/unattended-installation-(download-with-curl)
I decided not put it in the README to not further bloat it. The plan (mid term) is to provide packages for popular package managers and make this the preferred install method (roadmap item).
Thanks again for the suggestion (and feel free to contribute to the wiki page).

Looks good! The wiki is indeed the better place for that. Thanks for the update.


Add accessibility statement and menu links

This PR adds the accessibility statement page, as well as links in the main menu and footer. Address issue #67

Looks great! We will just need to go in and add the registration link once it is available.

Great point. I'll hold off on merging this request until we get the registration form link.

Registration link should now be added in commit cc2c857

Looks great! We will just need to go in and add the registration link once it is available.

@acrall - I've aded in the registration link. Do you want to merge the PR?


💡 Add a Rate Us Option

Idea Contribution

[X] I have read all the feature request issues.
[X] I'm interested in working on this issue
[X] I'm part of GSSOC organization

Explain feature request
Hello, my name is A.A.D.Sailaja. I am a Tech Enthusiastic and i'm looking forward to contribute in open source projects. Here in this project i would like to add a rate us option it will allow the user to rate our website and also he can suggest us to improve if any
Can you please assign me this issue under gssoc'24 @jfmartinz
Explain your solution
No response
Any alternative approaches/features
No response
Additional Context
No response

Isnt it 'Feedback' and 'Rate Us' the same thing, i mean we can just include it in the feedback form right? and also my concern about this issue is, is it really necessary to add this rate us? can you tell me more about it? And how youre going to do it?

I would like to make the rate us like this @jfmartinz

I think there is no Feedback form in the website @jfmartinz Nowadays every website is having feedback form right  so i thought it would be better if we too had it

This issue is duplicate @sailaja-adapa  #1331. Are you the one to propose this before?

I cant really tell who is the first one who raised thios issue

NO @jfmartinz

No problem i will close this issue @jfmartinz

This issue is duplicate @sailaja-adapa #1331. Are you the one to propose this before?

no i didn't proposed it @jfmartinz

Okay, thanks @sailaja-adapa


Add support for prepared message templates

There are some messages I end up typing over and over since each contributor needs to learn about our tooling. For example:
@rustbot author

Please write `@rustbot ready` when the PR is ready for another round of review.

Or
This looks good, thanks! Please squash the PR into a single commit.
Use `git rebase --interactive --keep-base ...` to do that so that there's no unnecessary diff.

@rustbot author

It would be nice if I could configure some repo-specific help messages so that I don't have to type the same thing over and over again. (I won't always have the time to be as patient as above on explaining the squashing procedure.)
So for instance:
@rustbot author-help
@rustbot author-squash

could act like @rustbot author but also print a message configured in triagebot.toml. (I am not sure if it is a good idea to tie this to @rustbot author, but both of the most prominent examples in my interaction happen to be of that sort.)

Would the GitHub Saved Replies solve your need? (I use them, too)

I didn't know about that feature. Will give it a try, thanks!


✨  Support input argument of subprocess

Using asyncio.subprocess has less support of subprocess.run...

/standby-release

/standby-release


Build(deps-dev): Bump prettier from 2.6.0 to 2.6.1 (#3210)

Pull request opened by the merge tool on behalf of #3210

Merged in 3ece929737fb2746611094998fca120d08ac7f89


added candump files related with the lane keep assist for Ford Fusion

Here I added candump files related with the lane keep assist. In these files, the lane keep assist was always on.

I am closing this pull request, let's wait until we have files in the format that Motley suggested.


With Expo-router, HoldItem clicked moves the height of the header

Describe the bug
Using Expo-router, <HolItem /> translates on the Y axis the height if a header of a <Stack />, even if the prop disabledMove is set.
To Reproduce
Steps to reproduce the behavior:

Create a basic expo-router file based routing in /app create a _layout.tsx

/app
  _layout.tsx
  index.tsx

In _layout.tsx, create a basic  that returns a navigation

return (<>
  <Stack screenOptions={{ headerShown: true }}>
    <Stack.Screen name="index" options={{ headerLargeTitle: false }} />
  </Stack>
</>
)

In index.tsx, use <HoltItem />

<HoldItem
  activateOn="tap"
  disableMove
  items={[
    { text: 'Reply', onPress: () => {} },
    { text: 'Edit', onPress: () => {} },
    { text: 'Delete', onPress: () => {} },
  ]}>
  <Text>My Action</Text>
</HoldItem>

Click on My Action, this text will translate on Y axis to a distance that is equal to the header's height.

Expected behavior
The My Action text should not move.
Additional context
There is no move by setting headerShown: false in the <Stack />, but I need that header.

everyone leave this lib

@fukemy theres no other good libs that work with expo, v sad lol


Where is the YOURLS Road Map?

Technical details regarding my environment:
YOURLS version: 1.7.3
Plugins enabled:
    admin-ydb/plugin.php
PHP version: 5.6.34
OS: Linux
Web Server: Apache/2.4.29
YOURLS DB Ver: 482

Can someone tell me the URL for the YOURLS Road Map?
I just get a 404 error.

https://github.com/orgs/YOURLS/projects/1


[AIRFLOW-1334] Check if tasks are backfill on scheduler in a join

Dear Airflow maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[x] My PR addresses the following Airflow JIRA issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-1334

Description

[x] Here are some details about my PR, including screenshots of any UI changes: Move query to be a join

Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason: Adds two tests. Both tests pass both pre and post revision.

Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

@aoen @AllisonWang @bolkedebruin

Codecov Report

Merging #2384 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2384      +/-   ##
==========================================
+ Coverage   69.23%   69.24%   +<.01%
==========================================
  Files         146      146
  Lines       11227    11224       -3
==========================================
- Hits         7773     7772       -1
+ Misses       3454     3452       -2

Impacted Files
Coverage Δ

airflow/jobs.py
74.89% <100%> (+0.02%)
:arrow_up:

airflow/www/app.py
96.47% <100%> (ø)
:arrow_up:

airflow/models.py
87.22% <0%> (+0.04%)
:arrow_up:

Continue to review full report at Codecov.

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

Ty and LGTM

Lgtm

You probably want to target this to 1.8.3

@bolkedebruin @aoen PTAL again. We have some properties that alias the instance vars in DagRun that force another query (and don't update internal state), where we should use a refresh_from_db approach like TaskInstance. We should probably eventually remove all these properties.

NVM I gave up on that stuff and I'm just reordering the statements to make the common path faster.

LGTM after tests pass

@saguziel do you have any ballpark data on how much this speeds up the scheduler?


NullPointerException while scanning cvsclient-71-jenkins-11.jar

Describe the bug
scanning cvsclient-71-jenkins-11.jar file withe the latest 6.1.3 causes java.lang.NullPointerException: null
Version of dependency-check used
cli 6.1.3
Log file
https://gist.github.com/wagde-orca/ebb3d217dc9ba5f270f2b6d36a0deb10
To Reproduce
Steps to reproduce the behavior:

mkdir jar_dir
cp cvsclient-71-jenkins-11.jar jar_dir
./bin/dependency-check.sh -s jar_dir
See error

Expected behavior
the scan will finish successfully
Additional context
This bug does not occur in the previous version I used 5.2.4

Thanx @jeremylong
Any ETA for 6.1.4?

should be within 30 minutes... However, what appears to be a random failure already occurred once - so that may delay the release until tomorrow.

https://github.com/jeremylong/DependencyCheck/actions/workflows/release.yml


F-droid release

This looks like an interesting project.
Would you please consider publishing the Android app to F-droid, besides Google Play?

Hi there, is this still being worked on?


Sourcemap adding to css and js after build which causing to make file size bigger (in MBs)

after the gulp dev when i am checking my main.js file size has been increased to 4.6MB

Right, but this only occurs in production and if you have the config.browserify.createSourcemap flag set to true. This can be seen here
If you don't want this functionality, remove line 54 and line 63 of the Browserify Gulp task.


introduced API for setting client version name

Pull Request Description
Provided API ConnectionStringBuilder.SetClientVersionForTracing()
client Version Name will be appended to client version header.
issue ref: #138
Breaking Changes:

None

Features:

None

Fixes:

None

Mmm The structure of go makes this a bit annoying.
I think we should first generalize it into a struct, since we might want to add similar things later (like client request id and such).
Also we can share code between kusto/conn (for queries) and conn/conn (for streaming ingestion).
Also of course add a test.


some question in python.runtime

Expected Behavior
run fast
Actual Behavior
using python.runtime lead slowly than python
Potential Solution
Reproducing the Problem
when using python.runtime,some questions puzzle me, many mistakes happens, like "System.IO.FileNotFoundException","System.IO.FileLoadException",System.IO.FileNotFoundException",maybe this is the reason leads my python code running slow,very slowly,I cant tolerate.my code is just a simple demo.so l wanna know how you resolve this problem,thanks,my python version is 3.6
System Information
win10,python3.6,x64
Checklist

[x] I have completely filled out this template
[x] I have confirmed that this issue exists on the current master branch
[ ] I have confirmed that this is not a duplicate issue by searching issues

[ ] I have provided detailed steps to reproduce the issue

Hi @shmliao, we are constantly working on Lean performance, especially its integration with Python.
Could you please post your questions in the community forum with more detail before you open a GitHub issue?


Use floormod for file bucket calculation in client cache

Merged build finished. Test PASSed.

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

alluxio-bot, merge this please.

alluxio-bot, cherry-pick this to branch-2.2 please.


Bug Report: kade engine crashes on startup

whenever i try building the game, it crashes right after i open it, idk why, im using kade engine 1.4.2

me too

me too

I found out how to fix this you open the file TitleState.hx near line 57 delete the line that says
polymod.Polymod.init({modRoot: "mods", dirs: ['introMod']}); and then lime test windows and see if it works. :)

I found out how to fix this you open the file TitleState.hx near line 57 delete the line that says polymod.Polymod.init({modRoot: "mods", dirs: ['introMod']}); and then lime test windows and see if it works. :)

i found the similar issue post with solve but anyways thx

Where is TitleState.hx??

in the source folder


esify silently drops inner classes

The ShippingLine class is dropped from fulfillments_new_form.coffee.

Fixed by https://github.com/Shopify/decaf/pull/17.  Props to @WillsonSmith for spotting this.


Improve PlanetScale token detection

Description:
This improves the PlanetScale token detection. It add some flexibility in length. There is no guarantee that the length is always 43 characters (in fact, it's very likely to change a bit soon). The length checks here now match what GitHub Secret Scanning does as well.
Additionally, it adds support for detecting oauth tokens as well.
Checklist:

[x] Does your PR pass tests?
[x] Have you written new tests for your changes?
[x] Have you lint your code locally prior to submission?

@dbussink thank you so much for the PR! looks good to me 🚀 this'll go into the next release

@dbussink thank you so much for the PR! looks good to me 🚀 this'll go into the next release


⚠ AtmoLive02 Server has degraded performance

In a5df6b0, AtmoLive02 Server ($ATMOLIVE02) experienced degraded performance:

HTTP code: 200
Response time: 10881 ms

Resolved: AtmoLive02 Server performance has improved in d3d818c.


Build paths are hardcoded

ifeq ($(UNAME), Darwin)
  INSTALL_PREFIX                =       /usr/local
else
  INSTALL_PREFIX                =       /usr
endif

INSTALL_HEADERS                 =       ${INSTALL_PREFIX}/include
INSTALL_LIB                     =       ${INSTALL_PREFIX}/lib

It would be great instead to have those as defaults, if the user hasn't set the install prefix. Hardcoding things like this makes it very hard to package.

It will get overridden if called like make INSTALL_PREFIX=/usr/custom/dir install afaik, so that's one way you could do that. If we alter it to ?=, the environmental variables will also get taken into account, which might not be what we want.

Yes, this one is my bad for not understanding/forgetting that you can override the variables for make like that.


Batch Update fails for enum column

this is my code
`public class BloggingContext : DbContext
{
public DbSet Locations { get; set; }
    protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder)
    {
        optionsBuilder.UseNpgsql(@"balbal");
    }
}

public class Location
{
    [Key]
    public string ID
    { get; set; }

    public bool IsActive { get; set; } = true;

    public string ParentID
    {
        get;
        set;
    }

    public LocationLevel Level

    {
        get;
        set;
    }

    public string Name
    {
        get;
        set;
    }
}

public enum LocationLevel
{
    Province = 1,
    City,
    Area,
    Section
}`

` using (var db = new BloggingContext())
{
var q = db.Locations.Where(t => t.ID == "").Update(t => new Location { Level = LocationLevel.Area });
            Console.WriteLine();

        }`

then  it runs it errors with the following:
The CLR enum type Location must be registered with Npgsql before usage, please refer to the documentation.

Hello @meijinyi ,
Do you think you could provide us a test project with this issue? Including step to support CLR enum type in PostgreSQL (or the link)
It will make easier/faster for my developer to getting started for investigating it.
We now always ask for a project since we found out that most issues are missing some essential information or are resolved by the requestor when creating it
(Even if the issue seem very easy to reproduce, by getting a test project, it allow us to give a faster support and better experience for the support of all our free libraries)
Best Regards,
Jonathan

@JonathanMagnan  i create a very very very simle demo  https://github.com/meijinyi/simpledemo
and  i did not create any enum type in PostgreSQL tks.

Hello @meijinyi ,
Thank for the project, this task has been assigned to a developer
Best Regards,
Jonathan

It's amazingly that such a simple mistake must be corrected for so long

I'm having the same issue. Here's some more info.
If you map the enums to PostGres...
1 - In the DbContextConstructor map the types
var translator = NpgsqlConnection.GlobalTypeMapper.DefaultNameTranslator;                 NpgsqlConnection.GlobalTypeMapper.MapEnum<MyEnum>(translator.TranslateTypeName(nameof(MyEnum)));
2 - In OnModelCreating map the enum values
var translator = NpgsqlConnection.GlobalTypeMapper.DefaultNameTranslator;                 modelBuilder.ForNpgsqlHasEnum(translator.TranslateTypeName(nameof(MyEnum)), Enum.GetNames(typeof(MyEnum)).Select(_ => translator.TranslateMemberName(_)).ToArray());
3  - Then when you try BulkInsert
await context.BulkInsert(myEntities, options => options.IncludeGraph = true).ConfigureAwait(false);
You get an exception:
 {System.Exception: An error occured while retrieving the InformationSchemaTable information. See the inner exception for details. ---> System.Exception: Oops! Unsupported types : USER-DEFINED    at .( , String[] )    at .(Boolean , String[] )    at Z.BulkOperations.BulkOperation.()    --- End of inner exception stack trace ---    at Z.BulkOperations.BulkOperation.()    at Z.BulkOperations.BulkOperation.()    at .[](DbContext this, IEnumerable1 , Action1 )    at .(DbContext this, StateManager , IReadOnlyList1 , Action1 )    at .(DbContext this, StateManager , IReadOnlyList1 , Action1 )    at .(DbContext this, Action1 )
at .[](DbContext this, IEnumerable1 , Action1 )
at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
--- End of stack trace from previous location where exception was thrown ---
at System.Threading.Tasks.Task.ExecuteWithThreadLocal(Task& currentTaskSlot)
--- End of stack trace from previous location where exception was thrown ---
at DbContextExtensions.BulkInsertAsync[T](DbContext this, IEnumerable1 entities, Action1 options, CancellationToken cancellationToken)
`

Hello @bambos-xiouros ,
I believe that issue has been fixed in the last version (v1.8.10)
Your case is similar but very different at the same time.
For the BatchUpdate,  the DbType is provided by Entity Framework and our code was missing the setter for the NpgsqlDbType value which is now specified.
In the case of BulkInsert, we are currently trying to find how to map correctly this column since this time, our library needs to find the right type. However, by posting your issue here, you just give us some idea to maybe find out what we have missed.

Thanks. Is there an issue I can follow for this? Nicer to have the enums if
I can.
On Tue, 25 Sep 2018, 12:19 Jonathan Magnan<EMAIL_ADDRESS>wrote:

Hello @bambos-xiouros https://github.com/bambos-xiouros ,
I believe that issue has been fixed in the last version (v1.8.10)
Your case is similar but very different at the same time.
For the BatchUpdate, the DbType is provided by Entity Framework and our
code was missing the setter for the NpgsqlDbType value which is now
specified.
In the case of BulkInsert, we are currently trying to find how to map
correctly this column since this time, our library needs to find the right
type. However, by posting your issue here, you just give us some idea to
maybe find out what we have missed.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zzzprojects/EntityFramework-Plus/issues/353#issuecomment-424185268,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AM9xknCtk-X77ErN8CjgPXF_jBTd15ejks5ueZK2gaJpZM4TyDDV
.

I ported app to .net core and faced this problem. In ef5/6 version we have this, but in ef core we don't have it.
I write workaround:
private static Action<BatchUpdate> GetBatchUpdateAction<T>()
{
    return update => update.Executing = command =>
    {
        foreach (DbParameter parameter in command.Parameters)
            if (parameter.DbType == DbType.Object && parameter.Value.GetType() == typeof(T))
            {
                var underlyingType = Enum.GetUnderlyingType(typeof(T));
                parameter.Value = Convert.ChangeType(parameter.Value, underlyingType);
            }
    };
}

Usage:
await _db.Assignments.Where(x => inactivatedEmployees.Contains(x.EmployeeId) && x.Task.Deliverable.SOWId == item.Id)
    .UpdateAsync(x => new AssignmentDbModel { Status = AssignmentStatus.Disabled }, GetBatchUpdateAction<AssignmentStatus>());

I hope we get fix soon.

Hello @grehovm ,
Thank you for reporting, we will look at it.
Best Regards,
Jonathan

Performance Libraries
context.BulkInsert(list, options => options.BatchSize = 1000);
Entity Framework Extensions • Entity Framework Classic • Bulk Operations •  Dapper Plus •  LinqToSql Plus
Runtime Evaluation
Eval.Execute("x + y", new {x = 1, y = 2}); // return 3
C# Eval Function • SQL Eval Function

Hello @grehovm ,
The v1.8.17 has been released.
Let me know if that's fixed on your side.
Best Regards,
Jonathan

It's work, thank you.


SBT heroku

@lauraluiz Do we need something like this https://github.com/heroku/sbt-heroku?
@mmoelli mentioned it.
I think for our travis builds we are already deploying an artifact and do not let heroku build it again.

@schleichardt are we currently avoiding Heroku to build the artefact again or do you mean that would be achieved with sbt-heroku?

Yes, the idea is to improve the speed of the heroku deployment. That would be achieved with the plugin, I hope.

That sounds great. So if that's the case, let's try it.

This unfortunately is not compatible with the Heroku button. We could implement it only for Sunrise deployment to staging to improve the speed, but then we cannot guarantee the button works. As the latter is more important right now, let's leave this for now.


Suggestion: add concurrency error on postgres event_store

What do you think about adding concurrency error.
For context the event sourcing event has a version. But is not used to do optimistic locking.
Currently event store is just inserting pending events.
But as shown in CQRS page 43 listing 5.
we could query for the last version stored in posgres and if it does not match with the aggregate current version we return concurrency error and the user should load the aggragate and apply the events again.

@stackus Could you give some suggestions for the version increment which has a race condition?


Azure Dev Spaces is bring retired

I think Azure Dev Spaces is being retired and Bridge to Kubernetes replaces it.

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

ID: d04bb23f-c4d6-ee39-0843-084b1411328b
Version Independent ID: 5aba2c57-770d-4bed-c178-44cd05aab9c2
Content: Application development and deployment - Cloud Adoption Framework
Content Source: docs/innovate/kubernetes/application-development.md
Service: cloud-adoption-framework
Sub-service: innovate
GitHub Login: @sabbour
Microsoft Alias: brblanch

Thanks for reporting - this issue is under review. This is a Microsoft Internal DevOps Bug ID # 376036

This issue has been resolved. Thanks for your feedback!


suport openvpn UDP

suport openvpn UDP please

Sorry, that's not the focus of this project. Please feel free to fork if you'd like to implement it yourself!


📝 Update postponed_annotations.md documentation

More things work on V2, so this is just a removal of non up-to-date information.

please review


Panic in jtag_dtm.rs when using probe-rs in combination with ESP32-C3 and ESP-Prog

Describe the bug
After a couple of days of trying to get probe-rs to work with my chip Espressif ESP32-C3-Devkit-M1 and JTAG ESP-Prog adapter, I'm now at a point where I can successfully enter a "pseudo-debugging" state. VSCode enters the debugger, but upon stepping, I'm getting a panic with a stack trace.
To Reproduce
Steps to reproduce the behavior:

Download the esp-blinky example
Run probe-rs with the following configuration:

[build]
target = "riscv32imc-esp-espidf"

[target.riscv32imc-esp-espidf]
linker = "ldproxy"
#runner = "espflash flash --monitor" # Select this runner for espflash v2.x.x\
runner = "probe-rs run --chip ESP32C3 --speed 10000 --protocol jtag" # Select this runner for probe-rs
rustflags = [ "--cfg",  "espidf_time64"] # Extending time_t for ESP IDF 5: https://github.com/esp-rs/rust/issues/110

[unstable]
build-std = ["std", "panic_abort"]

[env]
MCU="esp32c3"
# Note: this variable is not used by the pio builder (`cargo build --features pio`)
ESP_IDF_VERSION = "v5.1.3"
CRATE_CC_NO_DEFAULTS="1"

Use cargo run

Expected behavior
The debugger hits the breakpoint
Stacktrace
0x105004560 - std::panicking::begin_panic_handler::{{closure}}::h26c73eb75670321f
   8:        0x105002e60 - std::sys_common::backtrace::__rust_end_short_backtrace::hef4773cf0f80485b
   9:        0x105004338 - _rust_begin_unwind
  10:        0x105185aa8 - core::panicking::panic_fmt::h10bd3bfe5bb30d67
  11:        0x104ad45e8 - core::ops::function::FnOnce::call_once::hc66ed890add480ac
  12:        0x1049f6840 - probe_rs::probe::common::<impl probe_rs::probe::JTAGAccess for Probe>::write_register_batch::hc063a306d3fa80dd
  13:        0x104aeb310 - <probe_rs::architecture::riscv::dtm::jtag_dtm::JtagDtm as probe_rs::architecture::riscv::dtm::dtm_access::DtmAccess>::execute::h28718779f760ce86
  14:        0x104aeafe0 - <probe_rs::architecture::riscv::dtm::jtag_dtm::JtagDtm as probe_rs::architecture::riscv::dtm::dtm_access::DtmAccess>::read_deferred_result::hc7f8b50f6b4526ab
  15:        0x104a31854 - probe_rs::architecture::riscv::communication_interface::RiscvCommunicationInterface::read_dm_register::hc0b5771450cdb168
  16:        0x104bc9e80 - <probe_rs::architecture::riscv::Riscv32 as probe_rs::core::CoreInterface>::core_halted::hfa651aeee88430da
  17:        0x104c01704 - probe_rs::session::Session::clear_all_hw_breakpoints::h2fffaf05455d6dbd
  18:        0x104c0e1bc - <probe_rs::session::Session as core::ops::drop::Drop>::drop::h6d4b84f60c4010d0
  19:        0x104e5b228 - core::
ptr::drop_in_place<probe_rs::session::Session>::h62b3dc4874b2aa31
  20:        0x104e5f3b0 - core::ptr::drop_in_place<probe_rs::cmd::dap_server::server::session_data::SessionData>::h7fe1ecafc91c7716
  21:        0x104e76bbc - probe_rs::cmd::dap_server::server::debugger::Debugger::debug_session::hced9676169181247
  22:        0x104e20524 - probe_rs::cmd::dap_server::server::startup::debug::h4a4b53bf2cc69a48
  23:        0x104d93758 - probe_rs::cmd::dap_server::run::he5937137663cb1c2
  24:        0x104cd2af8 - probe_rs::main::h892f4e91d1c28778
  25:        0x104c3ee48 - std::sys_common::backtrace::__rust_begin_short_backtrace::h139193254363bc21
  26:        0x104e59484 - std::rt::lang_start::{{closure}}::hadeb1f7ad991546e
  27:        0x104ff7674 - std::rt::lang_start_internal::haeade8bb0e866fa6
  28:        0x104cdc7fc - _main

thread 'main' panicked at library/core/src/panicking.rs:144:5:
panic in a destructor during cleanup
thread caused non-unwinding panic. aborting.

Desktop (please complete the following information):

macOS Sonoma

Additional context
There is a lot of confusion around useage of the ESP32-C3. Perhaps we should consider opening a tracking issue?
It's also very unlikely to be a hardware issue. OpenOCD works with both ESP-IDF workflow produced binaries and PIO binaries, both with the arduino and espidf frameworks.
Rust toolchain:
[toolchain]
channel = "nightly"
components = ["rust-src"]

launch.json
{
      "type": "probe-rs-debug",
      "request": "launch",
      "name": "Launch",
      "cwd": "${workspaceFolder}",
      "chip": "esp32c3", //!MODIFY
      "speed": 10000,
      "probe": "0403:6010",
      "wireProtocol": "Jtag",
      "allowEraseAll": true,
      "flashingConfig": {
        "fullChipErase": true,
        "flashingEnabled": true,
        "resetAfterFlashing": true,
        "haltAfterReset": true,
        "formatOptions": {
          "format": "idf" //!MODIFY (or remove). Valid values are: 'elf'(default), 'idf'
        }
      },
"coreConfigs": [
        {
          "coreIndex": 0,
          "programBinary": "target/riscv32imc-esp-espidf/debug/${workspaceFolderBasename}", //!MODIFY
        }
      ],
      "env": {
        "RUST_LOG": "warn"
      }
    },

.cargo/config
[build]
target = "riscv32imc-esp-espidf"

[target.riscv32imc-esp-espidf]
linker = "ldproxy"
#runner = "espflash flash --monitor" # Select this runner for espflash v2.x.x\
runner = "probe-rs run --chip ESP32C3 --speed 10000 --protocol jtag" # Select this runner for probe-rs
rustflags = [ "--cfg",  "espidf_time64"] # Extending time_t for ESP IDF 5: https://github.com/esp-rs/rust/issues/110

[unstable]
build-std = ["std", "panic_abort"]

[env]
MCU="esp32c3"
# Note: this variable is not used by the pio builder (`cargo build --features pio`)
ESP_IDF_VERSION = "v5.1.3"
CRATE_CC_NO_DEFAULTS="1"

sdkconfig.defaults
# Rust often needs a bit of an extra main task stack size compared to C (the default is 3K)
CONFIG_ESP_MAIN_TASK_STACK_SIZE=8000

# Use this to set FreeRTOS kernel tick frequency to 1000 Hz (100 Hz by default).
# This allows to use 1 ms granuality for thread sleeps (10 ms by default).
#CONFIG_FREERTOS_HZ=1000

# Workaround for https://github.com/espressif/esp-idf/issues/7631
#CONFIG_MBEDTLS_CERTIFICATE_BUNDLE=n
#CONFIG_MBEDTLS_CERTIFICATE_BUNDLE_DEFAULT_FULL=n

Cargo.toml
[package]
name = "esp32"
version = "0.1.0"
authors = ["jkrieger<EMAIL_ADDRESS>edition = "2021"
resolver = "2"
rust-version = "1.71"

[profile.release]
opt-level = "s"

[profile.dev]
debug = true    # Symbols are nice and they don't increase the size on Flash
opt-level = "z"

[features]
default = ["std", "embassy", "esp-idf-svc/native"]

pio = ["esp-idf-svc/pio"]
std = ["alloc", "esp-idf-svc/binstart", "esp-idf-svc/std"]
alloc = ["esp-idf-svc/alloc"]
nightly = ["esp-idf-svc/nightly"]
experimental = ["esp-idf-svc/experimental"]
embassy = ["esp-idf-svc/embassy-sync", "esp-idf-svc/critical-section", "esp-idf-svc/embassy-time-driver"]

[dependencies]
log = { version = "0.4", default-features = false }
esp-idf-svc = { version = "0.48.1", default-features = false }
anyhow = "1.0.81"

[build-dependencies]
embuild = "0.31.3"

[package.metadata.esp-idf-sys]
esp_idf_sdkconfig_defaults = ["sdkconfig.defaults"]

Probe-rs List output:
![image](https://github.com/probe-rs/probe-rs/assets/27920144/0f50acaa-f0f6-4603-94eb-3d1c3f5dfdb1)

Is an esp-idf binary required to trigger this issue?

Could you also clarify what "jtag adapter" are you using? a ftdi one ore something else? Did you burn the internal FUSE to disable the internal jtag debugger?

@bugadani I am unsure what you mean. I would figure that a flashed binary would be necessary to trigger the debugger?
@Vollbrecht I am using the ESP-Prog as per title. I will include it in the issue :)
And yes, I've burned the FUSE as per documentation. I've also not installed any further ESP-Prog firmware, as per recent documentation Apple ships those by default now.

Is an esp-idf binary required to trigger this issue?

I get what you mean now. I've not tried an esp-idf no_std binary.
OpenOCD works fine with native CPP Esp-idf binaries and arduino based binaries, both in esp-idf and platformio.

dmi_timeout.log
fewer_irs_than_taps.log
flash_alg_fail_verify.log
Here is a couple of error logs I've encountered along the way

@Vollbrecht I've included the exact chip in the issue. It's the Espressif ESP32-C3-Devkit-M1

I've been able to get into a debugging session today, but stepping forward still results in a DMI Timeout.

@MabezDev since you've been playing around with related stuff, do you think these issues will be solvable in the forseeable future? My best guess is that we're still bumping up against some evil setup done by esp-idf and I haven't been following closely to know if anything was done to remedy this.

@bugadani Just out of personal interest, is there a board where probe-rs support is considered to be better?

@juliankrieger Could you test out https://github.com/probe-rs/probe-rs/pull/2164, you can do so by running cargo install probe-rs --features cli --git https://github.com/mabezdev/probe-rs --branch memprot

In my experience C6-based boards work better.
Alright. I will look into buying one.

is there a recommendation on using cargo embed and a remote gdb debugging session

I think these should work about the same. The low-level debug drivers are the same.
Strangely enough, I've been facing a different issue where the debugger halts a couple of times on SIGTRAP and then terminates with the same DMI Access Timeout.

@MabezDev I've been getting two different errors, depending on how I start probe-rs:
Erasing flash beforehand
1 - Running cargo run which leads to probe-rs run --chip ESP32C3 --speed 10000 --protocol jtag results in a flashed binary, a warning about no RTT headers and the command line process hanging.
2 - Running the launch task in VSCode results in the debugger hitting my breakpoint, but I still get the same error of a DMI Access Timeout.
dmi_access_after_patch.log
Not Erasing the flash beforehand
1 - Running cargo run as above results in an immediate DMI access timeout.
2 - Running the launch task results in an immediate DMI access timeout without the debugger hitting the breakpoint.
launch_no_erase.log

@juliankrieger it could be that you are hitting iwdt that are reseting your mcu after hitting the breakpoint. That is because a) esp-idf has code that detects if a debugger is connected b) if so it trys to make a "semihosting" call through the debugger to probe-rs to tell probe-rs "please set the breakpoint for me", but if it fails it will continoue to do so and than after half a second resets the chip.
Currently probe-rs doesnt support the semihosting answering of that call. To test out the hypotesis first try to disable the watchdog interrupt timer by setting the appropriate sdkconfig and test with that binary.

the sdkconfig option is "CONFIG_ESP_INT_WDT=n" .

On it.

What's a bit weird to me is that cargo espflash has no problem erasing flash. Probe-rs however fails, even when given the following arguments:

@Vollbrecht running cargo run and specifying a log file gives the following error:

When using probe-rs run, can you also supply --catch-reset? That should hopefully solve the DMI timeout issues (if you're still running my branch). I'm not sure if we have an equivalent option for the vscode config.

No change using attach:

No change using run:

No change using run with chip-erase and verify. Sadly, I can not attach a log file, my RUST_LOG=error env var is not respected...
Interestingly enough, passing verify after a flash erasure fails with Flash content verification failed
@MabezDev Can I supply command line options to probe-rs in the launch.json file?

Can I supply command line options to probe-rs in the launch.json file?

Do you expect any difference with the debugger as opposed to running probe-rs from cli? If yes, why?

Well for once, I can not test stepping the debugger if I just use probe-rs run in my console.

Right. If you're in need of hardware, I will see what I can do.

You can try walking through this guide while skipping the xtensa-specific info in there.

@bugadani I there anything left I can do to help? I could clone probe-rs, or try to use it as a library and step through the code there to get to the bottom of this issue.
any more ideas?

..and/or https://github.com/probe-rs/probe-rs/pull/2748 @juliankrieger


[10.x] Refactor retry Function to Use Loop Instead of goto

This pull request aims to improve the retry function within Laravel by replacing the goto statement with a more modern and maintainable approach using a while loop.
Changes:

Replaced the goto beginning statement with a while loop, resulting in a cleaner and more comprehensible code structure.

Maintained the existing logic of the function, ensuring that the behavior remains consistent with the original implementation.

Conducted thorough testing to confirm the correctness and reliability of the refactored code.

Benefits:

Enhances code readability and maintainability by eliminating the use of goto, a construct that can be challenging to follow.

Adopts a widely accepted programming pattern (looping) to achieve the same retry functionality.

Aligns with best practices advocated by the software development community.

See earlier tries on, this. It seems, that goto is faster than the loop constructs... Hence why they went for this. I foresee this one closed.


[Support Request] Yi Dome X BFUS <IP_ADDRESS>A - alt hack?

Hi there.
Is there a firmware that can be used to hack YI Dome Camera X - BFUS - <IP_ADDRESS>A_202001211.  There seem to be a problem with this camera even though it's pretty popular, but maybe other firmware can be used and someone has a sucess with it. Thank oyu.

Check Allwinner:
https://github.com/roleoroleo/yi-hack-Allwinner

Thank you very much.


[minor] typo inindex.js should be in index.js

here: https://docs.microsoft.com/en-us/azure/bot-service/migration/javascript-migration-quickreference?view=azure-bot-service-4.0#v4-3
(in javascript section)
inindex.js should be in index.js
Preview:

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

ID: 8dc19058-d861-8d28-f14f-98517058c9a3
Version Independent ID: b913f984-fe7d-b88f-2435-a8b097cb7b04
Content: JavaScript v3 to v4 migration quick reference - Bot Service
Content Source: articles/v4sdk/migration/javascript-migration-quickreference.md
Service: bot-service
Sub-service: sdk
GitHub Login: @JonathanFingold
Microsoft Alias: v-jofing

Thanks. Fix is in a pull request and should go live soon.


Game width is "jumpy" on Windows

This is how it looks on Chrome on Windows:

This problem was originally reported by @rosald on #2.

@rosald, i've run the game on a machine with Windows and yeah, this issue is totally reproducible :/
I can't think of anything to do though. It seems to be related with the way Windows renders Braille characters. Other systems render all Braille characters with the same width (i had tested the game on Linux mostly, but also on macOS and Android), but Windows seems to render blank Braille character (U+2800) a bit narrower than other Braille characters (which have at least one dot). So it makes this jumpy effect at the of the game board as the snake moves sideways.
I'll leave this issue open for now, though i can't think of a way to fix it. The problem may be inherent to the way WIndows renders fonts... But maybe someone comes up with a solution :) ... If not, i'm going to close it as a wontfix :sweat_smile:

BTW, running this game on WIndows made me realize i had broken compatibility with IE. I fixed tha on 5af76fb7cef7316dba9130b2ebbacd4823fe6350. So thanks for the report! :relaxed:

This issue is outdated. Since then, browsers have started escaping whitespace characters on the URL, making the game unplayable on the addressbar.


Update in scraping mechanism

Google has what seems to be infi-scroll on image search till one point. I updated the scraper so it works now. I also introduced a limit on the number of results it scrapes per keyword.

Testing and merging of pull request would be done by tomorrow. Should resolve the error due to DOM changes in Google. Sorry for the late response. Thanks a lot for your help!


feat: sign presentation using a VC subject DID

Signed-off-by: Filip Burlacu<EMAIL_ADDRESS>

Codecov Report
Patch coverage: 58.06% and project coverage change: -0.53 :warning:

Comparison is base (5cdc0e3) 90.81% compared to head (b734aa9) 90.29%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #289      +/-   ##
==========================================
- Coverage   90.81%   90.29%   -0.53%
==========================================
  Files          51       51
  Lines        2069     2092      +23
==========================================
+ Hits         1879     1889      +10
- Misses        132      141       +9
- Partials       58       62       +4

Impacted Files
Coverage Δ

cmd/wallet-sdk-gomobile/openid4vp/openid4vp.go
78.08% <58.06%> (-15.92%)
:arrow_down:

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Add creation slot to TransactionID

TIP

Closed by https://github.com/iotaledger/iota-sdk/pull/1414


Just a few to start

@hectork01 - Can you please pull these stacks and review these changes?

Does all files supposed to go into root directory?


[FEATURE] support no monorepo package

Is your feature request related to a problem? Please describe.
I want to be able to use codewoners-generator in a single repo and also work with the root pckage.json on a mono-repo project.
Describe the solution you'd like
Generate codeowners based on the root package.json
Additional context
When i generate codeowners from the root package.json the generated codeowners has a / error that could be replaced with * to work

What do you think @gustavkj we did this change a while back, meaning, going from * to /

I think this was the same behavior before the change, and it's only a problem for the root directory. But it should be fixed, I would categories this as a bug rather than a feature.

Is the idea that:
If the CODEOWNERS (contributors + author) come from the root of the repository: we use * ? meaning we replace that / for *?

Is the idea that: If the CODEOWNERS (contributors + author) come from the root of the repository: we use * ? meaning we replace that / for *?

If it's from a package.json in the root, then yes. But I think there probably needs to be a config option to disable this as well, as the * will act as a catchall.

Want me to give it a try next week or you are working on it @gustavkj ?

@gagoar I'd like to take a stab at it. It was just some weekend plans that got in the way. I'll ping you if I get stuck :+1:

@gustavkj Your PR looks good. I was torn between useRootMaintainers  and monoRepo so we could offer the need specifically. But I'm okay with either.

@gustavkj Your PR looks good. I was torn between useRootMaintainers and monoRepo so we could offer the need specifically. But I'm okay with either.

Yeah, I'm a bit unsure about the name too. I was considering if it should be something with fallback or default as that is what it's achieving.

@gustavkj Your PR looks good. I was torn between useRootMaintainers and monoRepo so we could offer the need specifically. But I'm okay with either.

Yeah, I'm a bit unsure about the name too. I was considering if it should be something with fallback or default as that is what it's achieving.

I use 2 ways of naming things.

Specific to the effect that it produces. In this case, something that refers to --useRootContributors.
Use the use case we are covering as a name. --monoRepo.

I don't like default because it doesn't really say anything and will always take you to read documentation to understand. The same goes for `fallback.

@gustavkj Can you release it? 😄

@gustavkj Can you release it? 😄

Sure, is it just npm version <version> on a separate branch and create a PR?

@gustavkj Can you release it? 😄

Sure, is it just npm version <version> on a separate branch and create a PR?

yup.

npm version patch
git push --tags (to the original project not yours)
git branch gustav/bump_version (I name them that way)
PR

you will see that the release is shown when you open the PR

yup.

npm version patch
git push --tags (to the original project not yours)
git branch gustav/bump_version (I name them that way)
PR

you will see that the release is shown when you open the PR
On Tue, Aug 23, 2022 at 3:07 PM Gustav Utterheim @.***>
wrote:

@gustavkj https://github.com/gustavkj Can you release it? 😄
Sure, is it just npm version  on a separate branch and create a
PR?
—
Reply to this email directly, view it on GitHub
https://github.com/gagoar/codeowners-generator/issues/329#issuecomment-1224646112,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABXJ75CSUEQ2FJ3FDN3ARLV2UOOZANCNFSM56FW7QPA
.
You are receiving this because you were mentioned.Message ID:
@.***>


[Bugfix] Compaction task limit should consider warehouse in Lake compaction scheduler

Why I'm doing:
What I'm doing:
Refactor the compaction task limit computation logic to consider warehouse
Fixes #issue
What type of PR is this:

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[ ] This is a backport pr

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.3
[ ] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

Documentation PRs only:
If you are submitting a PR that adds or changes English documentation and have not
included Chinese documentation, then you can check the box to request GPT to translate the
English doc to Chinese. Please ensure to uncheck the Do not translate box if translation is needed.
The workflow will generate a new PR with the Chinese translation after this PR is merged.

[ ] Yes, translate English markdown files with GPT
[x] Do not translate

PTAL @HangyuanLiu @kevincai