fix: delete assets permanently deleted instead of trash in android R

Changes made
The current implementation of trashing the asset instead of deleting it in Android 30 and above feels inconsistent with the rest of the behavior of the API in older versions of Android and iOS where the asset is irrevocably deleted.
It is changed in this PR to have the same behavior of deleting the asset using MediaStore.createDeleteRequest which based on the documentation calls ContentResolver#delete similar to the handling used in older android versions
Possible improvements
A new method named trashAsset can be added as an feature which would trash the asset instead of deleting it in Android 30 and above along with a method to fetch such assets as well

Fixed the Android CI failing due to the lack of namespace in the example app's build.gradle as well

I need to observe the real performance on the virtual machine, I will test this modification in the near future.

A new method named trashAsset can be added as an feature which would trash the asset instead of deleting it in Android 30 and above along with a method to fetch such assets as well

Introducing new dart api is really a good idea.


Task - Edit Observation Matrix - Add Rows from another matrix

When I try to add OTUs from another matrix, I can select the external matrix, and from the list, I can add few OTUs, but the button does not work.

It was already fixed during the presentation
https://github.com/SpeciesFileGroup/taxonworks/commit/eb4d66afc24f4853479c1a52cfe2dfd422bb2368


Power should be able to be toggled from the web UI

Using Home Assistant for this, so no need for this


Option to change the hotkey for exiting an inventory

I'm currently trying to set up this mod to be similar to another controller mod I was using, however not being the switch the hotkey for closing the inventory is making it impossible for me. I'm used to having the "Y" button on my controller open and close the inventory and the "B" button set to "quick move."

Duplicate of #93


DirectX 12 support

This revision supports DirectX 12.

This feature will be done in CubbyFlow-v0 and Jet Framework.


[PDI-13927] - Checking type before casting, adding null handling in g…

…etDataSourceFromJndi

@e-cuellar (forward?) port to <IP_ADDRESS>


command line utilities should check that api is not None

Otherwise all the commands fail when ever the command attempts to access api. This gives a range of different errors.
The easiest way to fix this would be to fail here but this would mean you can't run pyglottolog without a valid glottolog install. I wasn't sure how to fix this otherwise.

All commands expect args.repos to point to a valid clone or export of the repos, so I guess your proposed fix is the right thing to do.

Ok, not true. We want glottolog help <cmd> to work anyway - I guess.


Add google calendar

When adding event it should also appear at hackerspace google calendar.

https://developers.google.com/google-apps/calendar/quickstart/python


fails with permission denied in subdirectory

In one of our Java apps, we have integration tests for permissions. Long story short, directories get created that are unreadable.
When codecov-bash recursively looks for coverage files, it bombs on these directories. I would expect the script to continue recursing through other directories rather than aborting the entire search.
(url) https://codecov.io
(root) .
==> Jenkins CI detected.
==> Running gcov (disable via -X gcov)
  -> gcov in .
find: `./reach-engine-core/target/test/reachengine/scratch/bd9d1975-3c89-4b4a-b87a-3d51dcb79ece': Permission denied
find: `./reach-engine-core/target/test/reachengine/scratch/7f239d14-c036-43b5-8693-221940985df0': Permission denied
==> Searching for coverage reports
find: `./reach-engine-core/target/test/reachengine/scratch/bd9d1975-3c89-4b4a-b87a-3d51dcb79ece': Permission denied
find: `./reach-engine-core/target/test/reachengine/scratch/7f239d14-c036-43b5-8693-221940985df0': Permission denied
Something went wrong with Codecov.io coverage upload.

FWIW, I do know about -s DIR. I'm going to add this to my CI as a patch, but it interferes with the ease-of-use that I so much like about codecov-bash.

I tried -s DIR and it didn't seem to matter. Does this syntax look correct?
#!/bin/bash

# upload to codecov
{
  bash <(curl -s https://codecov.io/bash) -t "R-E-D-A-C-T-E-D" -X coveragepy -s "reach-engine-core/target/site/jacoco"
} || {
  echo "Something went wrong with Codecov.io coverage upload."
}

@jfairley the -s argument is used to add new folders where coverage reports may be located. In this case, its not necessary.
This is the first I've seen it failing. I think adding -readable -prune to the find command should fix this.
Is the repository open source? Here is the uploader I would like to try:
bash <(curl -s https://raw.githubusercontent.com/codecov/codecov-bash/fd4e06dd33950c6ef1a415cdfe6fac1e53bb5152/codecov)  -t "R-E-D-A-C-T-E-D" -X coveragepy

It's not open-source. I'll give it a try, and let you know.

This time it printed nothing after "Searching for coverage reports", but it still threw some sort of error. Last time, I saw those errors a second time.
(url) https://codecov.io
(root) .
==> Jenkins CI detected.
==> Running gcov (disable via -X gcov)
  -> gcov in .
find: `./reach-engine-core/target/test/reachengine/scratch/e6b0b3e8-5fa7-4d90-9acc-de0ce89c72b0': Permission denied
find: `./reach-engine-core/target/test/reachengine/scratch/b642aa49-062b-4e41-8e4f-53bbda0d863a': Permission denied
==> Searching for coverage reports

Hey @jfairley sorry for the delay. I'm curious if this is still occuring in the newer releases. I believe we have a flag to ignore permissions issues established.

Yes. Our CI still uses the same bash as commented above, minus the -s switch. It still throws.
Is there a new switch that I should enable?

@jfairley can you run this on the current release. I'm curious what the errors are now.
bash <(curl -s https://codecov.io/bash)

I'm looking at this thread and have implemented here. Thanks!

I thought I should revisit this issue, so I read through that bash file to see what it does. I understand what you were saying about -s being used "to add new folders". Even with -s, it still also searches in ".".
I may have found a workaround after reading through that bash file. I'm currently setting a custom project root, and that seems to work.
-p "reach-engine-core/target/site/jacoco"

I understand that this would also interfere with finding the "codecov.yml" and some other meta files, but I don't have any of those, so I can't see a negative impact based on my repo and what the bash file does today.
If -s were updated to replace the search paths rather than append, I could use that.

The other thought I had was whether using a codecov.yml would "fix" this by bypassing some of that problematic bash code. I haven't configured a codecov.yml for any of my projects yet though, I'm not familiar with its options.

Hey @jfairley thanks for the details on this. I think your idea of replacing the search paths will work well for this. I'll make the adjustments accordingly.

It doesn't seem this is deployed yet. (I just performed a curl locally.) Could you say when that might be?

Hey @jfairley just deployed now. Thank you for your patience 👍

I believe this is still a problem.  I recently ran the bash script and this was my output:
(venv-system) ubuntu@box100:~/shentel$ bash <(curl -s https://codecov.io/bash) -t <snip token>

  _____          _
 / ____|        | |
| |     ___   __| | ___  ___ _____   __
| |    / _ \ / _` |/ _ \/ __/ _ \ \ / /
| |___| (_) | (_| |  __/ (_| (_) \ V /
 \_____\___/ \__,_|\___|\___\___/ \_/
                              Bash-bb2adfe

==> Circle CI detected.
    project root: .
    Yaml not found, that's ok! Learn more at http://docs.codecov.io/docs/codecov-yaml
==> Running gcov in . (disable via -X gcov)
find: `./.tox/py27/storage-shentel': Permission denied
==> Searching for coverage reports in:
    + .
    + /tmp/circle-artifacts.ocCY2ux

The script returns a "1" exit code and, as you can see, nothing gets uploaded.

    Uploading to S3...
    https://codecov.io/github/level12/shentel/commit/de097a38e05c0ca8fefb24a893ea8824fc4eadd3

The permission issues are there, but they don't kill the entire script.
Going back to the bash script, if I disable some of the features, the error messages move but the upload still fails.
(venv-system) ubuntu@box100:~/shentel$ bash <(curl -s https://codecov.io/bash) -X coveragepy -X gcov -X search -t e3daf3ea-914a-44d9-bf00-78ef252a6e5d -f coverage.xml

  _____          _
 / ____|        | |
| |     ___   __| | ___  ___ _____   __
| |    / _ \ / _` |/ _ \/ __/ _ \ \ / /
| |___| (_) | (_| |  __/ (_| (_) \ V /
 \_____\___/ \__,_|\___|\___\___/ \_/
                              Bash-bb2adfe

==> Circle CI detected.
    project root: .
    Yaml not found, that's ok! Learn more at http://docs.codecov.io/docs/codecov-yaml
==> gcov disable
==> Searching for coverage reports in:
    + .
    + /tmp/circle-artifacts.ocCY2ux
    -> Found 1 reports
==> Python coveragepy disabled
==> Detecting git/mercurial file structure
==> Reading reports
    + coverage.xml bytes=248685
find: `./.tox/py27/storage-shentel': Permission denied
==> Appending adjustments
    http://docs.codecov.io/docs/fixing-reports
find: `./.tox/py27/storage-shentel': Permission denied
find: `./.tox/py27/storage-shentel': Permission denied

Gotcha! Solved it by adding || echo '' to the find commands

@stevepeak thanks for the quick turnaround, that's encouraging.

I'm sorry it took so long. I was not fully aware of what the issue was.
Deployed 👍 Let me know if its working properly.

@stevepeak the changes didn't solve the problem :(
bash <(curl -s https://codecov.io/bash) -t <snip>

  _____          _
 / ____|        | |
| |     ___   __| | ___  ___ _____   __
| |    / _ \ / _` |/ _ \/ __/ _ \ \ / /
| |___| (_) | (_| |  __/ (_| (_) \ V /
 \_____\___/ \__,_|\___|\___\___/ \_/
                              Bash-0834ffb

==> Circle CI detected.
    project root: .
    Yaml not found, that's ok! Learn more at http://docs.codecov.io/docs/codecov-yaml
==> Running gcov in . (disable via -X gcov)
find: `./.tox/py27/storage-shentel': Permission denied
==> Searching for coverage reports in:
    + .
    + /tmp/circle-artifacts.2hrfL33
    -> Found 1 reports
==> Detecting git/mercurial file structure
==> Reading reports
    + ./coverage.xml bytes=248755
find: `./.tox/py27/storage-shentel': Permission denied
==> Appending adjustments
    http://docs.codecov.io/docs/fixing-reports
find: `./.tox/py27/storage-shentel': Permission denied
find: `./.tox/py27/storage-shentel': Permission denied

Darn, didnt add it to report fixes. Doing that now. Thank you!

@rsyring can you try this commit:
bash <(curl -s https://raw.githubusercontent.com/codecov/codecov-bash/34f4a561ac6c889237f76c123f47be42e4a9b134/codecov)

Thank you!

@stevepeak that fixed it, thanks.


Use conda to install anaconda-client

Fixes a bug with uploading to Anaconda.org seen at https://github.com/astropy/astropy/issues/13348. anaconda-client is intended to be installed by conda (not pip, as some of its dependencies are not on PyPI). This PR switches the install method for this package from pip to conda.

[x] Delete TMP commit before merging

cc @pllim

By temporarily removing the Anaconda upload condition, anaconda-client is installing successfully with conda and the anaconda command is available: https://github.com/OpenAstronomy/github-actions-workflows/runs/6941673520?check_suite_focus=true#step:9:298
This should be ready to merge now. Then I'll do a bugfix release.

Thanks! So, once you release this, astropy wheel job would automatically work again?

Yes, although I think it might still fail due to https://github.com/astropy/astropy/issues/13197#issuecomment-1146674168


adds functionality so  will show up in message mimicking how it does …

What's this PR do?
Updated the watchdog log to provide the details in the log message.
How should this be reviewed?

Clear your dblogs.
Make a reportback.
Refresh dblogs and make sure that the transaction id is recorded. It should look like the following:

Relevant tickets
Fixes #7135
Checklist

[ ] Documentation added for new features/changed endpoints.
[ ] Tested on staging.
[ ] Pinged a PM if this is a larger PR that would benefit from some additional testing love
[ ] Post a message in #phoenix if this includes something that causes a rebuild!

…in laravel logs in rogue

vunderful! 🚢


How can it be 2047 unique cloners and only 18 stars

Perhaps there are 2047-18 unsatisfied users?

But I shouldn't worry when LuaJIT (such a great piece of code and with so many users) has only 2500 stars...

I guess only those binding library authors would star this project. Now the number just makes sense and cimplot has approximately 18 language bindings. 😉

@Gnimuc  Thanks for taking care. That's why I tagged the issue "wontfix"

I'm not at fault, I starred it :muscle: :smile:


Exit code from educate

Trollop.educate exits with code 0.  I would like to use Trollop in an existing script, but usage errors must exit with a code N != 0.  As a workaround, I can reopen Trollop:
module Trollop
  def self.educate_without_exiting
    @last_parser.educate
  end
end

# somewhere in my code
Trollop.educate_without_exiting
exit(N)

Is there a better way for me to exit with a code N of my choosing?
PS: I'm using Trollop for the first time and I really like it so far.  Thanks for the lib!

I'm mixed on this one.
If you specify --help on the command line, are you expecting a non 0 return status?
In your example, is help coming from a misuse, or an error?

If you specify --help on the command line, are you expecting a non 0 return status?

Good point.  If I'm requesting --help, I'd expect 0.

In your example, is help coming from a misuse, or an error?

Misuse, i.e. any time the usage is printed except for when --help is requested.

@jaredbeck do you have a test case (not formally)
4 liner parse and opt blurb with a description of what you pass in on the command line and when you expect to get errors?
I can convert into an official test, just want to make sure I am coding up your case.
--Keenan

Sure, let me see what I can come up with.  What if we consider cp?  It takes a variety of options and (usually) two arguments, a source path and a target path.  When only one argument is given, it prints usage and exits with a specific non-zero code.
It's my understanding that trollop is responsible only for options.  After parsing options, only the arguments are left in ARGV, right?  So, if we were implementing cp, we'd check the length of the remaining arguments.  If there were only one argument, we'd print usage and exit with that specific code.
Is that a good example?  Do you want me to take the next step and write an opt blurb for it?

@jaredbeck No code. I had misunderstood you. pr 58 helped a lot.
I have what I consider a good fix, but am in the middle of a refactor in the test area.
If I don't merge in a fix for your issue by next friday, please ping me to keep me honest.

After I get #61 in, I'll tack this on


v3.7.0

Description

Added Optional flag to BasicAuthenticator and JWTAuthenticator.
Added support for database.options for the SQLite driver.
Added response.Hijack(). Therefore, *goyave.Response now implements http.Hijacker. Note that status handlers and middleware (requests finalization step in their lifecycle) will still work for hijacked connections.
Added websocket support.
Added shutdown hooks.
Added goyave.BaseURL().
Added config.LoadJSON(). This can be used to load configuration from an embedded configuration file using Go's 1.16 embed directive.
Static file serving will no longer print "no such file or directory" to the error logger.
Static file serving optimization: check file existence once instead of twice.

TODO:

Remove docs
Migrate to go-goyave organization

Possible drawbacks
None.
Related issue(s)
List all related issues here:

#126
#130

Pull Request Test Coverage Report for Build 586785998

254 of 255 (99.61%)  changed or added relevant lines in 10 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.09%) to 98.921%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

router.go
7
8
87.5%

Files with Coverage Reduction
New Missed Lines
%

router.go
1
99.25%

Totals

Change from base Build 450772976:
0.09%

Covered Lines:
4032

Relevant Lines:
4076

💛  - Coveralls


kubectl connection error

$ microk8s.kubectl get all
The connection to the server <IP_ADDRESS>:8080 was refused - did you specify the right host or port?
$ snap list
Name                 Version                    Rev   Tracking  Developer      Notes
microk8s             v1.11.0                    104   beta      canonical      classic

Hi @wallyworld ,
Can you make sure you do not have any other service running on port 8080. Could you also share the output of journalctl -u snap.microk8s.daemon-apiserver.service?
Thank you for reporting this.

Idon't seem to have anything running on port 8080.
The service log does seems to show an issue starting up due to no default routes.
https://pastebin.ubuntu.com/p/RCwhf5wbvp/
(pods, nodes, etc.), comma separated. The individual override format: resource[.group]#size, where resource is lowercase plural (no version), group is optional, and size is a number. It takes effect when watch-cache is enabled. Some resources (replicationcontrollers, endpoints, nodes, pods, services, apiservices.apiregistration.k8s.io) have system defaults set by heuristics, others default to default-watch-cache-size
Jul 20 22:08:56 wallyworld microk8s.daemon-apiserver[2978]: error: Unable to find suitable network address.error='No default routes.'. Try to set the AdvertiseAddress directly or provide a valid BindAddress to fix this.
Jul 20 22:08:56 wallyworld systemd[1]: snap.microk8s.daemon-apiserver.service: Service hold-off time over, scheduling restart.
Jul 20 22:08:56 wallyworld systemd[1]: snap.microk8s.daemon-apiserver.service: Scheduled restart job, restart counter is at 5.
Jul 20 22:08:56 wallyworld systemd[1]: Stopped Service for snap application microk8s.daemon-apiserver.
Jul 20 22:08:56 wallyworld systemd[1]: snap.microk8s.daemon-apiserver.service: Start request repeated too quickly.
Jul 20 22:08:56 wallyworld systemd[1]: snap.microk8s.daemon-apiserver.service: Failed with result 'exit-code'.
Jul 20 22:08:56 wallyworld systemd[1]: Failed to start Service for snap application microk8s.daemon-apiserver.

@ktsakalozos
@tvansteenburgh was seeing this yesterday as well. He saw it on multipass with xenial and bionic.


Update to use new xcode version for framework Mac build_tests

This is a follow up of https://github.com/flutter/infra/pull/320
Related issue: https://github.com/flutter/flutter/issues/72823

Is this intended to fix all the build_test failures happening now, or should I file a new issue?
/opt/s/w/ir/cache/osx_sdk/XCode.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer/SDKs/iPhoneOS14.0.sdk/usr/lib/swift/Swift.swiftmodule/armv7-apple-ios.swiftinterface:21488:21: error: module 'Builtin' has no member named 'convertUnownedUnsafeToGuaranteed'
                   return try body(Builtin.convertUnownedUnsafeToGuaranteed(fakeBase,
                                   ^~~~~~~ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
               <unknown>:0: error: failed to load module 'Swift'

https://logs.chromium.org/logs/flutter/buildbucket/cr-buildbucket.appspot.com/8860163589006361024/+/steps/run_test.dart_for_build_tests_shard_and_subshard_0/0/stdout

Is this intended to fix all the build_test failures happening now, or should I file a new issue?

Actually mac_build_tests ran 7 times starting yesterday https://ci.chromium.org/ui/p/flutter/builders/prod/Mac build_tests/1173/overview and eventually succeeded, so maybe it was fixed by https://github.com/flutter/infra/pull/320

s intended to fix all the build_test failures happening now, or should I file a new issue?

This is only happening on try now.

Is this intended to fix all the build_test failures happening now, or should I file a new issue?

Actually mac_build_tests ran 7 times starting yesterday https://ci.chromium.org/ui/p/flutter/builders/prod/Mac build_tests/1173/overview and eventually succeeded, so maybe it was fixed by #320

I fixed the problem yesterday night. The problem was that the ios 14.0 added to the xcode 11.6 was not compatible with the swift included on xcode 11.6.

or should I file a new issue?

https://github.com/flutter/flutter/issues/72899


Move to Graphql

Expected Behavior
The app is currently designed as a rest api however seeing as this is pretty greenfield project how about we leverage graphql from the get go.
Actual Behavior
We currently serialize our views into json, we would likely replace these with graphql mutations
Acceptance Criteria
App should be using Graphql
Steps to Reproduce the Problem

Specifications

Version:

In order for us to achieve this, we will need to implement a number of items

[ ] context-based authentication for our API
[ ] Transfer user controller actions to a user resolver
[ ] Get rid of unused controllers and views
[ ] Create a graphql schema & endpoint


[PoC] Use R_NewEnv

R_NewEnv is available as of R 4.1, and it's already exported in the dev version of libR-sys. So, the implementation using this is very straight forward. But, I have no idea how to detect the user's R version, either on compilation time or on runtime, in order to decide whether to use R_NewEnv or to call new.env() with call! macro.

Info about the R version is available in the bindings:
https://github.com/extendr/libR-sys/blob/7418fade4bca9723ac1c1382c5d570a0fdb083c8/bindings/bindings-linux-x86_64-R4.0.rs#L285-L293
You could do a simple test of whether R_VERSION exceeds some numeric value. We just have to figure out what the appropriate value would be.

Thanks, but I think we need to switch on compilation time, where we cannot refer to these variables, iiuc. Fortunately, it seems we can use DEP_R_R_VERSION_* variables.
c.f. https://doc.rust-lang.org/cargo/reference/build-script-examples.html#conditional-compilation

While I think we should wait until the next version of libR-sys gets published, this pull request should be ready to review now. Feedback is welcome.

Do we need to publish a new version of libR-sys? Should we publish one whenever a major version of R gets released?

Do we need to publish a new version of libR-sys?

I think so.

Should we publish one whenever a major version of R gets released?

I thought we should wait until the new API is really needed, but now I come to feel one publish per major release is a good idea.

This should run green with the PRSEEN fix in place.


[oom] out of memory

After I debug, I find the oom is caused by code at:

more info: https://github.com/ota-meshi/eslint-plugin-json-schema-validator/issues/220

Thanks for reporting, I'll take a look tomorrow.


Google Cloud Services - PubSub(quarkus-google-cloud-pubsub) does not expose QuarkusPubSub class

I was trying to following example given here for pub-sub
https://docs.quarkiverse.io/quarkus-google-cloud-services/main/pubsub.html
But unfortunately QuarkusPubSub class no where to find
quarkus-google-cloud-pubsub is published correctly?

It is not yet released.
I'll create a new release today if I found the time as it's a long time since I didn't make one.

The new version is published, it's the 2.8.0 which will be integrated in the next platform release (should be 3.8.3).

Actually I am using 3.8.3, but the QuarkusPubSub  class is still not available :(

Yes sorry I release too late for 3.8.3. It will be in 3.9 and 3.8.4.


Make mhpost action consistent with mhget action.

Currently the mhpost.js action is exported as a nodejs module, can we change it like the mhget.js action, and thus remove the package.json file? @krook . Perhaps this is more convenient for deployment for tools like wsk cli or wskdeploy.

sorry, after review the code, seems we'd better put it as a zip file, since openwhisk already support zip actions,  any comments?

mhpost.js depends on an external npm module: message-hub-rest, while mhget.js has no external dependencies... which is why these are different.
We are already creating the mhpost action using the OpenWhisk support for zipped actions:
npm install
wsk action create kafka/mhpost-action actions/mhpost/mhpost.zip --kind nodejs:6

So I see no change required @lzbj, unless you still see something I don't?

Sure, so you mean we create the zip file when we want the zip file, ok, got it. thank you.


Manual App clones OTel Repos in Dockerfile not workflow

Description
Instead of installing the opentelemtry-python-core and opentelemetry-python-contrib GitHub repos as part of the Integration Tests workflow, we install as part of the manual app's Dockerfile RUN commands since it is the only one that needs these repos. (It tests against the very latest code).

The motivation for me was to reduce the "language-specific" behavior in the workflow. Instead, we could put language specific things in their own Dockerfile instead of branching in the workflow steps. For example cloning "opentelemetry-python-contrib" is very specific to Python -> manual instrumentation which could be transparent to the workflow.
This way workflows could ideally be more consistent across the different languages.

Given that all languages have an upstream core and contrib repo, I'm not sure this is one of those things that can get much less language-specific. If we have to change the language names when migrating the workflow between repos, IMO that's an acceptable cost towards portability.

Instead, we could put language specific things in their own Dockerfile instead of branching in the workflow steps.

Just to clarify, are you saying we'd have 1 step per language in the workflow like we do for Lambda? I'm not sure this would be the case since each repo would have its own workflow file.

If we have to change the language names when migrating the workflow between repos, IMO that's an acceptable cost towards portability.

It's not so much about the language name, it's about the action of "cloning the GitHub repo if this is a manually instrumented Sample App because this Sample App will directly install those packages". "Installing the packages straight from the GitHub repo" is a requirement for the Python Manual Sample App, but I didn't assume it would be applicable to the JS/Java/.NET/etc. Manual Sample apps.
I know you CAN download from the GitHub repo in JS, but I don't think the JS sample apps we have today do that. They don't seem to distinguish between "auto" and "manual" instrumentation either. That's what I mean about this step in the current workflow being a very "aws-otel-python"-esc step.

Just to clarify, are you saying we'd have 1 step per language in the workflow like we do for Lambda? I'm not sure this would be the case since each repo would have its own workflow file.

Sorry, it would have probably been better to say "instrumentation-type specific thing" (although like I mentioned above Python seems to be the one that is particularly doing this anyways).
No we shouldn't need a 1 step per language in the workflow like Lambda. Instead, every language adds the step they need to get their sample app to work. (For Python, that means a step to clone the GitHub repo but ONLY for the "manual" instrumentation type Sample App). If we can defer these language-specific/instrumentation-type specific actions (e.g. cloning a repo) to the Sample App, I think we should for the sake of consistent workflows. But like you said if every language needs to clone a GitHub repo then maybe we should be embracing that in the workflow, I just haven't seen that to be the case. (It's also not the case across all instrumentation-types, i.e. "auto" does not need to clone a repo so it feels weird to do it in the workflow).
I'm just saying if we can make the workflow as generic as possible, then that we should.

I see, I agree with @anuraaga that this would slow down the workflow, but it would be a fairly negligible slowdown compared to the overall duration of the workflow. If these steps are indeed specific to only the Python repo and not needed for other languages, then I agree with moving this to the Dockerfile.

This is similar to a comment on the soak tests PR, but I think it is generally difficult to create something generic proactively - you really have to see the final state and knobs required to know that it's possible to share anything at all even. Until then, focusing on each piece individually and using the standard idioms will usually be simpler. One other issue that I realized is that docker will cache the git clone invocation if it can - caching may not be set up to work on github actions so it could work, but for example if you debug the Dockerfile locally you may run into a very mysterious situation. So there is usually risk from diverging from idioms (the checkout action which we always use for checking out code) and I don't know if we've convinced ourselves of the reward here.

I can defer to Nathan to decide if this change is worth the possible pains in breaking from using the checkout action for the sake of possibly more consistent workflows. I agree that there's not much benefit since there will likely be other slight variations between repos anyway.
A comment explaining this behavior is unique to Python could possibly accomplish the same thing.

The docker cache is a good point, (and something I've run into before with other issues), but I still think the right move is to push it down to the sample app.
On one end, the OpenTelemetry Python SIG has very recently decided to stop cloning another repo when running tests on Core/Contrib. Users have reported that this is "too magical" and confusing for them.
That's the same case for our Sample App, you "just have to know" to have a clone of the repos right next to the Dockerfile when building the manual-instrumentation app today otherwise you'll get errors that say
 => [3/4] COPY . .                                                                                0.0s
 => ERROR [4/4] RUN pip install -r requirements.txt                                                                                2.8s
------
 > [4/4] RUN pip install -r requirements.txt:
#9 2.118 ERROR: Invalid requirement: './opentelemetry-python-core/opentelemetry-semantic-conventions' (from line 5 of requirements.txt)
#9 2.118 Hint: It looks like a path. File './opentelemetry-python-core/opentelemetry-semantic-conventions' does not exist.
------
executor failed running [/bin/sh -c pip install -r requirements.txt]: exit code: 1

I think the cache issue is a problem, but would only be for developers, and would hopefully be easy to detect because none of the changes you are making to the app would take effect due to this caching (i.e. even if you clone the repos, if caching gets you, then changing values in the files used by the COPY . . wouldn't get picked up).
HOWEVER, I tested out the solution they used for OpenTelemetry Python and because there are so many packages we need to download, if we install the packages from GitHub using pip our Docker Image build gets much much slower because every pip install installs the whole repository.
If we copy OpenTelemetry Python's methodology:
 => [4/4] RUN pip install -r requirements.txt                                 198.0s

Compared to cloning the repo:
 => [4/4] RUN pip install -r requirements.txt                                 22.9s

So this means we should continue to clone the repo. Our best alternative to doing it in the integration-testing.yml workflow is still to git clone the repos in the Dockerfile.
The reward I'm hoping to get is a scoping of responsibilities for the integration-testing.yml workflow file, and the Dockerfile sample app Docker Image. By pushing the fetching of the packages down to the Sample App, we're giving it work that allows it to stand alone as "a Sample App that uses the absolute latest packages" instead of having a side effect of "this only works if you clone the GitHub repos as you can see in our integration-testing.yml workflow file". Otherwise we should probably update the README.md for that app.
I'm not too sure about GitHub caching the Docker Image... these lines give some doubt, but I don't think this PR will change the caching behavior?
https://github.com/aws-observability/aws-otel-python/blob/c73e0f6ab1e4075bbad4bff83cb68fc14128e9bf/.github/workflows/integration-testing.yml#L42-L48
It's just another way of getting the opentelemetry-python-core and opentelemetry-python-contrib folders in the right place.
What do you all think about this then?

By the way, for more comparison:
The most recent run of the integration-testing workflow took 2 seconds and 1 second respectively to clone the core & contrib repos:

The code from this PR which installs the repos in the Dockerfile took 20 seconds and 3 seconds respectively to clone the repos when I tested it on a workflow on my forked repo:


GIT submodule & .git folder

Good day everyone!
I'd like to mention two things here and would like to request fixing them (at least the one with .git folder):

When I'm specifying .git/ folder to be excluded while deployment it still appears on the server (I've tried EXCLUDE: ".git/" and EXCLUDE: "/.git/" - both options end up with the same result).
I've got submodule in my project and after deployment folder appears to be empty at all. However in fact it points to another git folder which also must be deployed.

Update module version to 2.1.6 works for me. The verison on Readme doesn't updated yet
uses<EMAIL_ADDRESS>

Hey @golee
I'll try that, thanks!

Still doesn't work :(

This should be fixed.


sequencer: implement typed errors

the sequencer crate should have typed errors for easy error matching.
see https://github.com/astriaorg/astria/pull/206 comments for some places to start

Hey I am currently learning rust, as it is a good first issue can you please assign this to me.

@ShivangRawat30 sure, thanks for the interest!


Improving gender classification

Hello, I recently ran some tests using the following image which incorrectly classified the gender as a man:
https://www.sheknows.com/wp-content/uploads/2018/08/strong-black-woman-feature_h42gdn.jpeg
via
img = r'images\strong-black-woman.jpeg'
obj = DeepFace.analyze(img_path = img, actions = ['age', 'gender', 'race', 'emotion'])
print(obj["age"]," years old ",obj["dominant_race"]," ",obj["dominant_emotion"]," ", obj["gender"])
Is there any documentation or suggestions for improving the gender model with more training data?

Hello, failing individual test instances is not important in machine learning studies. Gender model got 97.44% accuracy, 96.29% precision and 95.05% which is acceptable. Here you can find its tutorial: https://sefiks.com/2019/02/13/apparent-age-and-gender-prediction-in-keras/ It covers training stages.


Admin cannot access all teams via API

Gitea version (or commit ref): 1.4.0
Git version: 2.1.4
Operating system: Debian 8
Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[x] Not relevant

Log gist:

Description
I am admin on my Gitea installation and created an API key for me. Although having the highest access permission, I cannot access the API orgs/{org}/teams if I'm not member of this organisation (example: "other-org"):
$ curl -s -X GET "https://git.example.org/api/v1/orgs/other-org/teams" -H  "accept: application/json" -H "Authorization: token ce106f4de28xxxx"
{"message":"Must be an organization member","url":"https://godoc.org/github.com/go-gitea/go-sdk/gitea"

In the web interface I can see all information about the organization and the teams. Accessing the exact same information via API for organisations I'm member of works perfectly fine.
This behaviour is unexpected because the Admin should be able to access the same information via API as via web interface.

I think this has been fixed by #6483.


ClientScene.localPlayer is never null once spawned

Describe the bug
Accessing ClientScene.localPlayer e.g. for its netId works once it has been spawned even after destruction and due to its static nature, from wherever you want.
To Reproduce
Steps to reproduce the behavior:

Access ClientScene.localPlayer.netid in your networkscene after second connection try

Expected behavior
Accessing ClientScene.localPlayer should throw null. Either the static field gets nulled after the localPlayer was destroyed (would match the unique assignment to that field for it), or the property getter performs a null check on its own. In any case, the LocalPlayer should be null if its null

Just to make sure I understand, are you saying that ClientScene.localPlayer is still set after the object is destroyed? and Instead it should be set to null after the localPlayer object is destroyed?

@James-Frowen that's true. Accessing a MonoBehaviour without explicitly checking for null bypasses Unity's code duality checks and works since its a regular class instance. You can see that it's supposedly null because Visual Studio shows it as being string null ("null") and explicitly checking for null also shows it being null. Therefore its no problem to read all the variabled the class had.
And yes, the static reference is explicitly set when the localPlayer is created, it should either be nulled when it's again destroyed, or the static getter gets a null check included


Separate the sign up process into 2 steps

This PR removes the mobile number from the original registration step and creates and additional page for the mobile number entry.  This is in preparation for the potential of using Static Security Codes.
🎥  🎉  ...............................Get your popcorn ready! .......................................🎉  🎥

Okay, going to update the tests tomorrow to change the assertion now that the flow of the sign up has changed


[HOLD][$250] Migrate RequestorOnfidoStep.js to function component

Class Component Migration
Filenames

[ ] src/pages/ReimbursementAccount/RequestorOnfidoStep.js

type: React.Component
has state values: false
has refs: false
has context: false
lifecycle methods: none

Task

We currently have some class components in our codebase that we would like to refactor to a function component.
Here's a link with some general advice on how to refactor a class component to a function component: https://react.dev/reference/react/Component#alternatives
If you need additional guidance, please ask in #expensify-open-source
Test for any regressions and verify that there are no breaking changes

I'd love to work on this.

I can work on this!

I can work on this issue.

I am able to work on this task.

i will work on this

Hi! I'm Bartek from Callstack - an expert contributor group and I would like to work on this one.

Contributor details
Your Expensify account email<EMAIL_ADDRESS>

Proposal
Issue
Migration from RequestorOnfidoStep.js class to function
Solution
Create a function corresponding the class and migrate the functionalities to it in addition to updating the references to the class
for instance if there is some import in a component like below
import {RequestorOnfidoStep}
then it should become
import RequestorOnfidoStep
Upwork Profile: https://www.upwork.com/freelancers/rosemiller
Expensify Email<EMAIL_ADDRESS>

Someone has already been assigned to work on this

This isn't overdue right? Also, is any payment from me needed? Sobit takes New.Dot and @barttom invoices us right?

Hey @jliexpensify!
This issue is on me, I just need to merge main branch into mine fetuje branch, but I’m on OOO until end of this week

No worries! Thanks for letting us know. @barttom just to confirm that you invoice us for payment, correct?

@jliexpensify I’m a part of callstack team, so I afaik they are care of invoices 😉

Cool, in that case - Payment Summary:

Contributor: @barttom Invoiced payment
C+: @sobitneupane $250 New.Dot payment

No Upworks job needed here.

Hi @sobitneupane @barttom just checking: this hasn't been competed/implemented right?

@jliexpensify It has been merged

@barttom As it is still in the regression period, I found a console warning and reported here.
Reason: We switched to useLocalize and removed withLocalize but StepPropTypes expects a prop withLocalizePropTypes which is now missing.

Hmm just to confirm: there's a regression related tot his GH? So payment should be adjusted to $125?

Payment Summary (assuming there's a regression)

Contributor: @barttom $125 (Callstack invoiced payment)
C+: @sobitneupane $125 (New.Dot payment)

NO UPWORKS JOB

@jliexpensify I wasn't assigned to the PR, and I didn't review it. Therefore, no payment is due to me.

Fixed, thanks for letting me know!


Convert readme.creole to markdown README.md

Since your newer projects are using Markdown and Markdown is pretty much the defacto standard, I am
submitting this simple conversion if you desire to make the switch.

It looks like @wbond had a change of heart from creole to Markdown in Aug 2013.
https://github.com/wbond/package_control_channel/commit/e858c4db848efe86164fa16685a89b36d2090bfd
@wbond Thoughts on this PR? I am fine landing it if you approve.
Side note: We should update the text in a following PR to not say ST2 anymore x_x

Yeah, Markdown is great. It has basically become the de facto standard.

Cool, will land within the next week :+1:

Upon re-reviewing, I would prefer to stick with the # to keep things consistent among @wbond's repos.
https://raw.githubusercontent.com/wbond/package_control_channel/7d0858948c54f9dbfa46646f759c0100c3e3d5ab/readme.md
@tekwiz Can you update this PR to line up more with that? (e.g. use # for headings, name it readme.md instead of README.md)

+1

Closing PR due to inactivity. Replacing with PR #100.


upload test result to upstream job if exists

Signed-off-by: Tianle Huang<EMAIL_ADDRESS>Description
Follow up on https://github.com/opensearch-project/opensearch-build/pull/2024#issuecomment-1106743524
When non-prod build job triggers the integ test, it will override the result from prod build job if the build number happens to be same. Thus add the logic to upload the test result to its upstream job if it exists. This can avoid such conflict and also allow us to use latest url to access the test result.
Issues Resolved
Check List

[x] Commits are signed per the DCO using --signoff

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

Codecov Report

Merging #2028 (ebb54d8) into main (7012178) will increase coverage by 5.82%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##               main     #2028      +/-   ##
=============================================
+ Coverage     94.17%   100.00%   +5.82%
=============================================
  Files           195         6     -189
  Lines          3860       105    -3755
  Branches         29        19      -10
=============================================
- Hits           3635       105    -3530
+ Misses          219         0     -219
+ Partials          6         0       -6

Impacted Files
Coverage Δ

src/test_workflow/integ_test/service.py

src/test_workflow/test_component.py

tests/jenkins/jobs/AssembleUpload_Jenkinsfile

src/run_bwc_test.py

src/build_workflow/build_target.py

...workflow/integ_test/integ_test_start_properties.py

...RunBwcTestScript_OpenSearch_Dashboards_Jenkinsfile

src/ci_workflow/ci_check_package.py

...sts/jenkins/jobs/DetectTestDockerAgent_Jenkinsfile

...ests/jenkins/jobs/AssembleManifest_tar_Jenkinsfile

... and 179 more

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 7012178...ebb54d8. Read the comment docs.

Hi @tianleh , can you resolve the merge conflicts and confirm that this is working as expected?

Will revisit when there is bandwidth


Creative exporters crash the server when used on inventories connected to an external storage

I’ve found that the server crashes whenever I use a creative exporter to export items into an inventory that is connected to an external storage. This has happened on three separate occasions:

I had a Mekanism dynamic tank filled with brine (a liquid). I was exporting brine into the tank with a creative exporter, and as soon as I connected an external storage to that tank, the server crashed.

I had a Sophisticated Storage barrel with max stack upgrades. The barrel had about 40K raw iron in it (in one slot). I was exporting the raw iron into it with a creative exporter. Attaching an external storage to the barrel also crashed the server.

I had a Functional Storage storage controller connected to several drawers. It had an external storage connected to it. Attaching a creative exporter to it and then adding raw copper to the filter crashed the server.

I reported this on the Refined Storage issue tracker, but I’ve since realized that this is a problem with Cable Tiers, from what I can tell, since this only happens when using a creative exporter from Cable Tiers. I first thought it had something to do with the inventories containing a large number of items, but I’m not sure. In the drawer situation, the creative exporter worked fine until I added a fourth Netherite upgrade to the drawer, from what I remember, so maybe it has something to do with transferring a large number of items instantly, in conjunction with there being an external storage attached to the inventory. I’m not sure.
I tried again several times with each of these inventories, but it always crashes the server each time. I’m not playing on a server, by the way; this is in single-player. When I say it crashes the server, I mean the server hangs. I.e., you can’t interact with anything, drop items, pick up items, etc. You’re forced to restart, and leaving the world crashes the game. No crash report was generated any of these times, nor did I find anything in the latest.log. Nothing new shows up in the latest.log when the server crashes.
Using ultra exporters is fine, but as soon as I use a creative exporter on any inventory that has an external storage connected to it, the server immediately crashes. It seems as though Refined Storage might have trouble with exporting items instantly when they’re also in the storage system using an external storage.
I’m using the latest version of both Cable Tiers and Refined Storage. Refined Storage isn’t on the latest version in the modpack I’m playing, but I updated it manually. This also happened when using version 1.10.2 of Refined Storage and version 0.5461 of Cable Tiers.
Versions

Modpack: All the Mods 7 (version 0.4.27—latest)
Minecraft: 1.18.2
Forge: 40.1.68
Cable Tiers: 0.547 (latest)
Refined Storage: 1.10.3 (latest)

The server may just be stuck (a hang) instead of an actual crash.
Can you please
a) try to reproduce in All the Mods 8 or in 1.19.2 + the latest version of Cable Tiers and Refined Storage?
b) run spark profiler start --timeout 60 before attempting to reproduce (and try to reproduce the hang in that time)?

I can say that in ATM8 1.19.2 I created a fresh new world just to replicate the above and I did. The barrel is empty and there isn't anything even in the network. The second you slap a Creative Exporter on the inventory of something with an External Storage on it and then attach a cable to bring it into the network the "Server" (also I'm on single player just to quickly replicate it) goes into Waiting for server... and never stops. I had to delete the single player world and just start over to get back in.

I have been having this issue for a while now. I am able to use 1 creative importer with no issues. As soon as a second is added I get the "waiting for server" prompt.
I am currently playing on ATM8 1.0.15

I believe we have a server that is running into this problem. Here's a flamegraph of what the server thread is doing after running 7 minutes with the watchdog disabled:

You can see that it's stuck in the loop where doItemInsertion is called until it returns a false.
The 2 main calls it makes are to:

com.refinedmods.refinedstorage.api.network.INetwork.extractItem
net.minecraftforge.items.ItemHandlerHelper.insertItemStacked

This suggests that it is descending into this block:
https://github.com/starforcraft/Cable-Tiers/blob/afbc975e7b4970e3cb7fa0af90cca710f16e9e2a/src/main/java/com/YTrollman/CableTiers/node/TieredExporterNetworkNode.java#L222-L231
And retuning true on line 241 as the method is exiting and being run again.

We need to figure out the scenario that allows this to happen.
It could just be caused by a lot of items in this case. There are a lot of calls to retrieve and compare capability data. I've seen these to cause problems for RS when storage contains items that have a lot of varying capabilities (and NBT data).
I'm leaving the server running for a few hours to see if it ever completes.

There needs to be a limit on the number of times a Creative Exporter or Importer tries to insert or extract items or fluid, else the server may enter an infinite loop if doItemInsertion, doFluidInsertion, doItemExtraction or doFluidExtraction repeatedly returns true for a given context. Applying this change would also fully fix https://github.com/starforcraft/Cable-Tiers/issues/38, not just fix it for specific blocks.


Add Trino-datasource in Grafana has error "this.getRef is not a function"

when i use this new plugin as your post here (https://github.com/starburstdata/grafana-trino/releases/download/v1.0.0-beta.1/trino-datasource.zip), i got a error when add a trino-datasource.
the error is : "this.getRef is not a function"
see the error picture as i attached.

Grafana version: 8.2.4
Look forward to your help...

Indeed Grafana 8.2 is missing the DataSourceApi.getRef method: https://grafana.com/docs/grafana/v8.2/packages_api/data/datasourceapi/
I'll try to see what was used before, but in the meantime, can you upgrade to a newer version?

@nineinchnick  thanks for the update. yeah. after upgrading to 8.5, the issue has been fixed. thanks.


multiple arrays?

Can I have multiple arrays at the same time?
For instance does the following sequence work?
cbor_encoder_create_map(&stream, &map, 2);
// Create first array.
cbor_encode_text_stringz(&map, "array1");
cbor_encoder_create_array(&map, &array1, CborIndefiniteLength);
// Create a second array.
cbor_encode_text_stringz(&map, "array2");
cbor_encoder_create_array(&map, &array2, CborIndefiniteLength);
cbor_encode_int(&array1, data1);
cbor_encode_int(&array2, data2);    // this overwrites array1.
Are there other ways to do this? Can you provide an example on how to manage two arrays within one cbor stream?

You cannot do that. TinyCBOR is not meant to be a DOM structure, so it will not hold the multiple values. It's a streaming encoder (StAX). You need to finish the first array before beginning the second.
If you need to manage two arrays at the same time, do so prior to encoding. If you want, I can add a feature so that you can write to two buffers in parallel, and then just copy what has already been encoded in one onto the other.

Thanks for the reply.
How about a cbor_encode_merge function()? It is essentially the same thing as you proposed. However this will be more generic. Sort of a strcat function; we can have two arrays managed separately in their own streams and can be merged later.
Let's assume stream 1  is open and stream  2 is closed, I would like to call something like
cbor_encode_merge(stream1, stream2)
which can copy the map from stream2 in to stream1's tail.


chore(deps): update common-dev-assets digest to eee03c3

This PR contains the following updates:

Package
Update
Change

common-dev-assets
digest
00b5683 -> eee03c3

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

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


Logo gets distorted

Logo gets distorted due to the both width and height being defined in the style.

If the width is omitted, then the logo aspect ratio remains correct.

This would work fine when logo is provided as a square image. (ex. 512x512 pixels.)

On the other side, in different scenarios, it may be that a short but wide image is "cropped" with your suggestion. Perhaps setting the max-height and max-width should solve the original problem of not showing a distorted image, no matter its height/width ratio, e.g.

Fixed as part of https://github.com/adiralashiva8/robotframework-metrics/pull/65 pull request

Seems this issue not fixed, Modifying sidebar will fix this issue

Modified sidebar which fixes image distorting in this commit https://github.com/adiralashiva8/robotframework-metrics/commit/3cb7a5f156789c0e7f784d105a13c2f71142e712


[qos] enable OpenThread to provide quality of service

Features Overview:
<1> Provide API to create message with specified priority level
<2> Set the IPv6 DSCP value according to the message priority level
<3> Drop low priority messages when there are no enough buffers
<4> Packets with high priority send/forward first

You originally had PriorityEntry struct/list which were used to track the priority/tag/offset of received fragment frames. In the current code this is fully removed and all frames are added in the reassembly list.
We have two categories of received frame:

The ones we are receiving.
The one we need to forward without IP header parsing, i.e., frames that have MeshHeader

For first category, existing code already parses the IP header and if message is fragmented, the fragments are placed in reassembly list.  We can derive the message priority from the IP header, and all info is tracked by Message in the reassembly list.
For second category we need to determine the priority to be able to add the frame correctly to the send priority queue. So we need to parse the IPv6 header and if message is fragmented somehow keep track of the fragment tag and priority to assign the same priority to next fragments. In https://github.com/openthread/openthread/pull/2752#discussion_r193891288 I suggested that we limit the use of PriorityEntry struct to track only these messages and not messages that are already added in reassembly list.
My suggestion is (instead of adding the second category frames to reassembly list)  to consider defining a new list/queue to keep track of priority/tag/etc for such frames

Either the original PriorityEntry list or,
If you want to use Message to keep the info, using a different message queue instead of mReassemblyList).

@zhanglongxia here is what I would suggest for tracking of the offset/length for MeshHeader received message.

Let's keep the Message APIs exactly as they are. I don't think you need a change.
For MeshHeader frames, we can use a different queue to store a Message containing only the first fragment (to keep the fragment tag, and priority associated with the message) - or alternatively you can use a queue of  PriorityEntry struct (like what you had earlier) instead of a queue of Messages.
For any received MeshHeader fragmented message, we can get both length and offset from the fragment lowpan header. So there is no need to track it in the message. I guess we need the length and offset to determine if the frame is the last fragment so we can delete the corresponding entry from the queue.

@abtink Thanks for your suggestions.
I got your meaning now. Your suggestion is storing the fragment header in the message to avoid modifying the Message APIs. When the node receives subsequent fragment, it updates the fragment header stored in the message.
The length and offset are used to determine when to free the messages from the queue.

@zhanglongxia  that can work. But I guess we do not need to update the offset in the message (in the queue). We can get the length and offset from the received next fragment itself. Lowpan FragmentHeader for the newly received frame contains tag, datagram length and offset.
We need to find the corresponding message in the queue (matching the tag). This would give us the message priority. Beyond that we can use the offset and length (from lowpan fragment header) to determine if this is the last fragment, and if it is, we can free the corresponding message from queue.

@abtink I have updated the code following your suggestion. Please review it at your convenience.

This PR has been splited into the following small PRs: #3015, #3132, #3159, #3199, #3201, #3317, #3318 .

This PR has been splited into the following small PRs: #3015, #3132, #3159, #3199, #3201, #3317, #3318 .


Docs: Fix links to ECE API APM payload definition (v1.0.0-beta2)

We're doing some reorganization of the ECE book, so this fixes the APM link that will break as a result of that change. I believe we'll have to merge this at the same time as the ECE changes to avoid a docs build break.

Closing...  this branch is no longer being built.


frontend: Correctly abbreviate long token amounts

Closes #698, closes #964
default beahavior for the new format() function:

for values >= 1 shows maximum 5 digits totally or at least 2 decimals
for values < 1 show 8 decimal places max
for values < 1 if decimal places would only be 0 show a less than sign (e.g. < 0.001)

Or displays the given decimals from the options.
@GabrielBuragev this does not include the tooltip for displaying the full amount yet. I simply didn't have enough time. Could you add it either here or in a new PR? It should be displayed for the token balance and in the headers in the transfer history

Also, i didn't add the tooltip on the transfer expandable card inside the activity tab since that info is already there when you expand it.


Model deprecation clarification

[Enter feedback here]
Regarding the model versioning details are unclear specifically what model deprecation means. Might model deprecations and ChatML upgrades break APIs or are they going to be backwards compatible? Or simply won't be able to deploy that model anymore but deployed models will still be available? What level of support is offered for a model that won't be offered for a deprecated model?
Thank you,
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 13994f3e-a96a-8804-4deb-7e0adad7f479
Version Independent ID: e087d8ce-bf3a-f12c-41dc-3ea924c6f089
Content: How to work with the Chat Markup Language (preview) - Azure OpenAI
Content Source: articles/cognitive-services/openai/how-to/chatgpt.md
Service: cognitive-services
GitHub Login: @dereklegenzoff
Microsoft Alias: delegenz

@shayben
Thanks for your feedback! We will investigate and update as appropriate.

@shayben - the deprecation date means the model will no longer be available and you'll need to upgrade to a newer version of the gpt-35-turbo model before the deprecation date. So if you were to go to production today with version 0301 of gpt-35-turbo, you should keep an eye on the deprecation date and be prepared to upgrade to a newer model before that date. As of right now, this means you would need to create a deployment with a newer version of the model, test out the new model for your scenario, and then potentially make some tweaks to the syntax in your prompts (if ChatML syntax changes in the newer version). We're mostly setting faster deprecation windows so we can continue to onboard newer versions of the model.
We'll be making some updates to make this easier in the future but I can't comment on those in a public forum currently. Stay tuned!

@shayben
If there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.

@shayben
We are going to close this thread  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.


add alex1 school domain

@edriso I am sorry, this school seems to be not an organization that provides an education in software development. Therefore, most of the students there do not need a free license for a software development tool. Given a serious level of abusing similar domains in Egypt, I have to reject the request.
Also, the name of the organization is incomplete in your request, and I am not sure how to correctly identify it.
If you personally needs a software development license for your study, please apply for free educational license with an official document which proves that you are a student. You can do it using "OFFICIAL DOCUMENT" tab at https://www.jetbrains.com/shop/eform/students.
I have to close the request without merging.
Thank you for your understanding!

Hello, @philipto, thank you for your thoughtful response. I appreciate your feedback. I realize that in my previous message, I omitted the complete domain; it's 'alex1.moe.edu.eg'. I wanted to bring to your attention that some students at this particular region are actively engaged in programming, and schools are hosting programming events. I thought it would be beneficial to highlight JetBrains products specifically, considering the interest in programming activities.
Upon further investigation, I discovered that 'alex3.moe.edu.eg' already exists, representing similar schools in a different region. It's worth noting that if there is another pull request related to 'alex1', it could positively impact students at that location who are interested in programming.
I hope you have a fantastic day, and once again, I'm thanks for your valuable response.


Fix recommended import styles for js, ts, jsx and tsx files

What kind of changes does this PR include?

[ ] Minor content fixes (broken links, typos, etc.)
[X] New or updated content
[ ] Translated content
[ ] Changes to the docs site code
[ ] Something else!

Description
This PR updates how we recommend importing JavaScript, Typescript, JSX and TSX modules based on consultation with @Princesseuh last week.
The one potential point of confusion is importing .ts which in general should be done without an extension to bring in types etc., but in <script> tags should include the extension. I’ve gone with recommend the basics, flag up the <script> tag exception, but I’d like to know if we need to maybe explain in the standard case that omitting the extension is good because we get types etc.

Looks great, and we should also update our strong wording in a couple of places that components must be imported using relative file path including full extension. I guess now the instruction is "except for ..."

framework-components
troubleshooting

Just looking for a few other places where it might be helpful to hit the "no extension thing." (Bold emphasis just for highlighting a suggested place for additional not, not to be bold in final copy). These are the two other places an additional note could be helpful?
Imports Page:
JSX / TSX
Section titled JSX / TSX
import { MyComponent } from './MyComponent';
Astro includes built-in support for JSX (*.jsx and *.tsx) files in your project. Import them into any .astro component without file extension, and the JSX syntax is automatically transpiled to JavaScript.
TypeScript Page
Astro ships with built-in support for TypeScript. You can import .ts and .tsx files in your Astro project (no file extension required), and even write TypeScript code directly inside your Astro component.

In my opinion, such heavy emphasis shouldn't be put on the no extension thing, as importing with a .jsx / .js (for TSX and TS respectively) extension is also totally valid
Additionally, soon we might migrate to using TS  4.7's node16 module resolution which requires the extension (so .js for .ts, .jsx for .tsx) so it's best to not hammer the no extension thing too hard (or we'll run in the same situation we have now where people use .ts extensions because they think it was required)

Importing a .ts file with .js should work, it was fixed a week ago: https://github.com/withastro/astro/pull/3518

Importing a .ts file with .js should work, it was fixed a week ago: https://github.com/withastro/astro/pull/3518

Ah, OK. I tried in the docs repo and we’re stuck on an old beta version at the moment. I see in that issue that that doesn’t work in <script> tags, yet. Hmm.
You know what? The current advice doesn’t break anything — you just get some red squigglies importing TS with the extension. I think it might be a good idea to leave it as is and wait for script tags to reach parity so we can give consistent advice everywhere about using .js/.jsx for Typescript and everything else remains unchanged.


go/parser: StructType.End() can overflow the File

As encountered in #48249, StructType.End() can overflow the file (such that File.Offset panics).
This is due to the error recovery here:
https://cs.opensource.google/go/go/+/master:src/go/parser/parser.go;l=675;drc=0406d3a8e5301bd5fd697018e6e8fbb9c75eeb42
Which will set the StructType.Fields.Closing to the end of the file, and then StructType.Fields.End overflows:
https://cs.opensource.google/go/go/+/master:src/go/ast/ast.go;l=252;drc=master
This is a low priority (the logic that gopls was using should have been more robust), but I feel like it should be an invariant of nodes that if Pos() or End() is not token.NoPos, they have a valid offset.
Are there counter-arguments or contradicting precedent?
CC @griesemer @stamblerre

Related: https://github.com/golang/go/issues/57490


SHMT::create support Iterable objects as $array

construct a large array in php is memory-consuming, if we can write a Iterable class to read database and use it's instance as an array, life could be much easier. is it possible to do it? thanks.

I do not really understand your work-flow. Can you describe your work-flow more precisely? Which kind of array do you use in your code (normal or associative array) for creation of a SHMT?

Sorry, I'm trying to describe it more precisely. Imaging we have a table like this:
CREATE TABLEExampleData(  key-field varchar(255) NULL ,  value-field varchar(255) NULL  );
The current way to use SHMT:
$data = []; $db_statement = $db->query("SELECT * FROM ExampleData"); while ($row = $db_statement->fetch()) {   $data[$row['key-field']] = $row['value-field']; } SHMT::create($filename='map.shmt', $data);
If the table is big enough, $data could consume a lot of memory. And the Iterator way will be like this:
`
class DataIterator implements Iterator {
private $key = null;
private $value = null;
private $stm = null;
public function __construct($stm) {
    $this->stm = $stm;
}

function rewind() {
    $this->next();
}

function current() {
    return $this->value;
}

function key() {
    return $this->key;
}

function next() {
    $row = $this->stm->fetch();
    if ($row) {
        $this->key = $row['key-field'];
        $this->value = $row['value-field'];
    } else {
        $this->key = $this->value = null;
    }
}

function valid() {
    return !is_null($this->key);
}

}
$db_statement = $db->query("SELECT * FROM ExampleData");
$data = new DataIterator($db_statement);
SHMT::create($filename='map.shmt', $data);
`
The advantage of this way is: we didn't construct a large array in PHP, and $data still can be used as an array (support foreach statement). Even more, we can break the SQL query to small queries (to reduce the pressure at MySQL side).
It's just my thought, if there is something wrong please let me known. Thanks a lot.

Sorry, but SHMT intern has no lists of keys or values for any iterations. SHMT calculates the number of bucket by the hash-value of the given key and shows in the bucket for the position in the file where to find the value (if the bucket is not empty).
May be you need APCu or Memcached for you work-flow.

OK, Thanks very much.

Workaround: You can try to work with MySQL limits and to create several *.shmt files. The SHMT has extremely fast initial load for *.shmt files, so you can try to run your lookups in a loop.


Update logos on documentation homepage

As written in email

I have noticed partners’ logos are modified at http://api-dev.freme-project.eu/doc/ e.g. Agro-Know and Tilde. Could you please use original logos – see below, thank you in advance

What is exactly meant by modified?
The "original" logos simply look worse, see
New:

Old:

Or is there some kind of copyright-metadata that was changed which I am unaware of?
I will push these now anyways.

@josauder simply do not strecht the logos. fix one of dimension - width or height not both.
Original logo for Tilde is http://www.tilde.lv/sites/tilde.lv/files/logo_sources/tilde_logo.png
See the difference, on the right are the original logos:

You can use css properties max-width and max-height to scale the images without stretching them.


output dir are not the same when using nobara

Once it's compile the output dir are in cmake-build-release instead of build-cmake on nobara (fedora fork).
cmake version: 3.27.7

ok same problem with actions so it's can be a problem of the makefile

ok it's just the make who output not in the same directory need some precision on how to use the makefile


Complete multi-region work for 'aws_vpc'

There is one last behavior that I am not sure if it should be handled for the user in the mapper or if it's related to a future refactor an existing TODO is already mentioning and should be tackled in a follow-up PR.
If you "move" an Endpoint Gateway between regions, the auto-populated routeTableIds array needs to be NULLed out or it will fail because it is referencing things in the old region and then fails. This PR could be modified to filter out invalid route table IDs instead of failing on it, but that could cover up other mistakes by the user?
There's a TODO stating that the route table ought to be turned into an entity in its own right, and that would make this go away (because we could constrain on the region and force a DB error if you don't detach them before changing the region, and we can also guarantee that the values in there are "real" in a way we currently can't). It should eventually be done, but whether the current behavior is good enough or not is still debatable.


Flatten validation messages from schema validation

Fixes the unnecessarily (and confusingly) nested structure of validation messages from schema validation.
Before:
Error

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

Test name
Commit
Details
Rerun command

lint_istio
911b3981ae0864e912be0e03f70e2c211ebdeff3
link
/test lint_istio

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

@sushicw: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_istio
911b3981ae0864e912be0e03f70e2c211ebdeff3
link
/test lint_istio

integ-telemetry-k8s-tests_istio
911b3981ae0864e912be0e03f70e2c211ebdeff3
link
/test integ-telemetry-k8s-tests_istio

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

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

Test name
Commit
Details
Rerun command

integ-new-install-k8s-tests_istio
255f6751c6bf2bd53e32db08be43286d7d603a14
link
/test integ-new-install-k8s-tests_istio

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

@sushicw: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-new-install-k8s-tests_istio
255f6751c6bf2bd53e32db08be43286d7d603a14
link
/test integ-new-install-k8s-tests_istio

integ-istioio-k8s-tests_istio
255f6751c6bf2bd53e32db08be43286d7d603a14
link
/test integ-istioio-k8s-tests_istio

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

@sushicw: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-new-install-k8s-tests_istio
255f6751c6bf2bd53e32db08be43286d7d603a14
link
/test integ-new-install-k8s-tests_istio

integ-istioio-k8s-tests_istio
255f6751c6bf2bd53e32db08be43286d7d603a14
link
/test integ-istioio-k8s-tests_istio

integ-mixer-k8s-tests_istio
255f6751c6bf2bd53e32db08be43286d7d603a14
link
/test integ-mixer-k8s-tests_istio

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

In response to a cherrypick label: new pull request created: #18305


Github not creating new repos (mainly on Edge)

I can almost never get MakeCode to create a new repo from a project anymore.  It always takes several times of trying, nothing happening, trying again, getting a message that says "This repo already exists", deleting the emtpy repo, and trying again.
Here's the error from the console when this happens:

Related?:
https://forum.makecode.com/t/why-is-github-not-showing-any-files/22988

I am unable to reproduce this issue. Creating both private\public Github repo works fine through the github icon in my machine.
@kiki-lee  does it still reproduce for you?

It's sporadic.  Sometimes it works fine, then other days it won't work even if I try 15 times in a row. I'll see if I can narrow in on any other details that are always true when things go haywire.

Related?: https://forum.makecode.com/t/why-is-github-not-showing-any-files/22988

Yep, that's exactly the same thing that's happening for me.

Are you able to create repo and files in your free account outside MakeCode? We have plenty of users on free account who can create repos. I don't believe this issue is tied to free\pro accounts

My free git account is working fine, with github web and command line git. I just created a repo and added a file to it from the command line.

On Oct 10, 2023, at 12:49 PM, Abhijith Chatra @.***> wrote:
Are you able to create repo and files in your free account outside MakeCode? We have plenty of users on free account who can create repos. I don't believe this issue is tied to free\pro accounts
—
Reply to this email directly, view it on GitHub https://github.com/microsoft/pxt-arcade/issues/6096#issuecomment-1755848446, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAYEXR6VA76ZZIFRCOCNNELX6V4CBAVCNFSM6AAAAAA5ND6226VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONJVHA2DQNBUGY.
You are receiving this because you commented.

How do I tell make code which git account to use?

On Oct 10, 2023, at 12:49 PM, Abhijith Chatra @.***> wrote:
Are you able to create repo and files in your free account outside MakeCode? We have plenty of users on free account who can create repos. I don't believe this issue is tied to free\pro accounts
—
Reply to this email directly, view it on GitHub https://github.com/microsoft/pxt-arcade/issues/6096#issuecomment-1755848446, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAYEXR6VA76ZZIFRCOCNNELX6V4CBAVCNFSM6AAAAAA5ND6226VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONJVHA2DQNBUGY.
You are receiving this because you commented.

Some more info -
on edge my makecode is connected to my paid github account, and it is working. I don’t see how to tell it to use a different github account. On Mozilla Firefox, it is connected to my free account. When I add a project to git, an empty repo is created (the project has lots of code) and the github button always gives the “create a repository” dialog.
I’m looking into having our students in a TEALS class use this but it looks too complicated for them I think.

On Oct 10, 2023, at 12:49 PM, Abhijith Chatra @.***> wrote:
Are you able to create repo and files in your free account outside MakeCode? We have plenty of users on free account who can create repos. I don't believe this issue is tied to free\pro accounts
—
Reply to this email directly, view it on GitHub https://github.com/microsoft/pxt-arcade/issues/6096#issuecomment-1755848446, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAYEXR6VA76ZZIFRCOCNNELX6V4CBAVCNFSM6AAAAAA5ND6226VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONJVHA2DQNBUGY.
You are receiving this because you commented.

more info - firefox makecode was not signed in to a makecode account. Now I signed in to the makecode account and the repo was created and populated with code. Maybe my issue was that I had not signed in to makecode?????

On Oct 10, 2023, at 2:09 PM, John Holland @.***> wrote:
Some more info -
on edge my makecode is connected to my paid github account, and it is working. I don’t see how to tell it to use a different github account. On Mozilla Firefox, it is connected to my free account. When I add a project to git, an empty repo is created (the project has lots of code) and the github button always gives the “create a repository” dialog.
I’m looking into having our students in a TEALS class use this but it looks too complicated for them I think.

On Oct 10, 2023, at 12:49 PM, Abhijith Chatra @.***> wrote:
Are you able to create repo and files in your free account outside MakeCode? We have plenty of users on free account who can create repos. I don't believe this issue is tied to free\pro accounts
—
Reply to this email directly, view it on GitHub https://github.com/microsoft/pxt-arcade/issues/6096#issuecomment-1755848446, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAYEXR6VA76ZZIFRCOCNNELX6V4CBAVCNFSM6AAAAAA5ND6226VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONJVHA2DQNBUGY.
You are receiving this because you commented.

Unfortunately, I think it's less predictable than that.

We can also setup a call to investigate this. Please email to<EMAIL_ADDRESS>and I will setup a quick call to debug.

I don’t see that unlink github option anywhere………..

On Oct 10, 2023, at 2:54 PM, Abhijith Chatra @.***> wrote:
on edge my makecode is connected to my paid github account, and it is working. I don’t see how to tell it to use a different github account.
You can go to settings->unlink github to logout from github and login with the new account
https://user-images.githubusercontent.com/6107272/274031861-a7697bf7-7010-453c-ae6f-5bf0de9dad19.png
more info - firefox makecode was not signed in to a makecode account. Now I signed in to the makecode account and the repo was created and populated with code. Maybe my issue was that I had not signed in to makecode?????
You don't need to login to makecode to use the github integration feature.
—
Reply to this email directly, view it on GitHub https://github.com/microsoft/pxt-arcade/issues/6096#issuecomment-1756050115, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAYEXR5BERXKQDDK2NML5BLX6WKUXAVCNFSM6AAAAAA5ND6226VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONJWGA2TAMJRGU.
You are receiving this because you commented.

This means you have not logged in to github through MakeCode. Try to export the project and see if you are able to login.
Also go to github.com and see if you can login in that browser.

I believe I’m logged in to both make code and git hub.
￼
￼

On Oct 10, 2023, at 3:01 PM, Abhijith Chatra @.***> wrote:
This means you have not logged in to github through MakeCode. Try to export the project and see if you are able to login.
Also go to github.com and see if you can login in that browser.
—
Reply to this email directly, view it on GitHub https://github.com/microsoft/pxt-arcade/issues/6096#issuecomment-1756060787, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAYEXR3FPMIDPZYCT7U5XF3X6WLRHAVCNFSM6AAAAAA5ND6226VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONJWGA3DANZYG4.
You are receiving this because you commented.

I am in github in another tab (this is in edge) as well

On Oct 10, 2023, at 3:01 PM, Abhijith Chatra @.***> wrote:
This means you have not logged in to github through MakeCode. Try to export the project and see if you are able to login.
Also go to github.com and see if you can login in that browser.
—
Reply to this email directly, view it on GitHub https://github.com/microsoft/pxt-arcade/issues/6096#issuecomment-1756060787, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAYEXR3FPMIDPZYCT7U5XF3X6WLRHAVCNFSM6AAAAAA5ND6226VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONJWGA3DANZYG4.
You are receiving this because you commented.

It  seems flaky, I just made a change and pushed it to github via make code and it didn’t work, then I pushed it again and it did.

On Oct 10, 2023, at 3:01 PM, Abhijith Chatra @.***> wrote:
This means you have not logged in to github through MakeCode. Try to export the project and see if you are able to login.
Also go to github.com and see if you can login in that browser.
—
Reply to this email directly, view it on GitHub https://github.com/microsoft/pxt-arcade/issues/6096#issuecomment-1756060787, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAYEXR3FPMIDPZYCT7U5XF3X6WLRHAVCNFSM6AAAAAA5ND6226VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONJWGA3DANZYG4.
You are receiving this because you commented.

right now things seem to be working well…….I don’t see a way to change github accounts though. But I am using edge and makecode and my free github account and it is all working.

On Oct 10, 2023, at 3:01 PM, Abhijith Chatra @.***> wrote:
This means you have not logged in to github through MakeCode. Try to export the project and see if you are able to login.
Also go to github.com and see if you can login in that browser.
—
Reply to this email directly, view it on GitHub https://github.com/microsoft/pxt-arcade/issues/6096#issuecomment-1756060787, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAYEXR3FPMIDPZYCT7U5XF3X6WLRHAVCNFSM6AAAAAA5ND6226VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONJWGA3DANZYG4.
You are receiving this because you commented.

no option to disconnect from GitHub.

I sent some emails from apple mail to reply to this that had screenshots but the screenshots didn't seem to make it here.....

@abchatra - can you see if we need to investigate this more?


How to generate .pth file?

./data/
├──.mp4 # original video
├──ori_imgs # original images from video
│  ├──0.jpg
│  ├──0.lms # 2D landmarks
│  ├──...
├──gt_imgs # ground truth images (static background)
│  ├──0.jpg
│  ├──...
├──parsing # semantic segmentation
│  ├──0.png
│  ├──...
├──torso_imgs # inpainted torso images
│  ├──0.png
│  ├──...
├──aud.wav # original audio
├──aud_eo.npy # audio features (wav2vec)
├──aud.npy # audio features (deepspeech)
├──bc.jpg # default background
├──track_params.pt # raw head tracking results
├──transforms_train.json # head poses (train split)
├──transforms_val.json # head poses (test split)
not found obama_eo.pth and obama.json file after training

me too

should be in trial_/checkpoints/

应该在 trial_filename/checkpoints/ 例如：trial_obama/checkpoints/

I couldn't find it. Can you provide a screenshot

I also couldn't find it using the notebook file explorer at first, but it was there when I did ls /notebooks/RAD-NeRF/trial_gary/checkpoints in the notebook.


Extensibility

Could it be possible to attach to custom knowledge sources? (Like existing vector DB, graph DB, relational DB and other stuff?)

autogpt has a knowledge base, so i guess we'll be the same

I would suggest to add some capabilities and extend to external pluggable knowledge

Considering real-life development scenarios, we should have a workspace consisting of a ready-made or blank code repository and a portion of design and requirement documents. If we want OpenDevin to be practically available, rather than just staying in the demo stage, this requirement is very important and reasonable.

This sounds like a good idea, but TBH I think it is a bit too underspecified. Because of this, I'm going to convert it to a discussion. We can discuss the different knowledge sources that we would like to incorporate, and then create concrete issues for specific knowledge sources.


Drop uneeded scripts

add-cloud-init.sh not used anywhere
azure+gce: https://github.com/kairos-io/AuroraBoot/pull/146
ipxe template: https://github.com/kairos-io/kairos/pull/3097
update-os-release and os-release template: https://github.com/kairos-io/kairos/pull/3097
netboot.sh: already dropped in a previous PR

This should come after the gce/azure PR has been merged


Mark consul instances OUT_OF_SERVICE

Spring-cloud-consul can think of eureka change status?
such as , status = OUT_OF_SERVICE

when I change the status uping of eureka. The services can still work.
It's not a real shutdown service.
I try to use the method deregister of spring-cloud-consul but the serivce remove from the consul server.

@spencergibb  Could you tell me how to operate it?

@spencergibb I'm sorry.   I read the source code and Know how to handle.


Making declarations of EmailAddress available on Java

Description
We would like to make declarations of the EmailAddress type, introduced by the issue #339, available for Java users.
This means that all declarations introduced by the issue #339 should be tested in the subproject java-compatibility and should have a documentation indicating their usage from Java code.
Dependencies
This issue is blocked by the following ones:

[ ] #339

Checklist

[ ] Make the property EmailAddress.Companion.regex available for Java users, test its compatibility, update the public API binaries and update the unreleased changelog.
[ ] Make the function EmailAddress.Companion.orNull(String) available for Java users, test its compatibility, update the public API binaries and update the unreleased changelog.
[ ] Create an issue for stabilizing the EmailAddress type in the next minor release.

The equals, hashCode and toString functions are already visible for Java users through the Object type (see https://github.com/kotools/types/issues/339#issuecomment-1875153542).

This issue duplicates https://github.com/kotools/types/issues/339#issuecomment-1876982815.


Emtpy EXECStart command in systemd service config when there is pipx ensurepath warning

Sometimes, the GPUStack service startup command will be empty in the systemd service config file when there is pipx ensurepath warning in the installation script output. At this time, the "which gpustack" will return empty and the service config file will be like the below:

We already did a force run "pipx ensurepath" in installation script. As the log showed above, not sure if we should force a "source ~/.bashrc" or "source ~/.zshrc" to make sure it take effects during our installation.

Reproduce steps:

Install Python 3.10-3.12, Pipx. For example, create a conda env, then pip install pipx.
Ensure that python3,pip3 and pipx are available in PATH. Ensure that pipx environment --value PIPX_BIN_DIR is not included in PATH.
Run installation script.

Verified in main c73f01e32677e562106224f3b1ada41a9770d567


Skip checking results for tasks that are not loaded

In accordance to issue https://github.com/IBM/tekton-lint/issues/1 , this would fix the undefined issue by checking if tekton.tasks is undefined.
There's also an added exclamation mark in tekton.tasks[matchingTask.taskRef.name]!.spec, to prevent linter issues.

@madbence smart move, will make it more readable in the future!


Dataset

Hello,
Where to find the dataset: 'df_ldt_per_engie_final.pkl'?

okay, thanks for the reply

No problem. I can refer you to a webpage which might be helpful for you: this contains open wind farm data from an onshore wind farm in case you wanted something similar to explore and is publicly available. But there are no labelled fault records though, it only contains SCADA features and Met Data. https://opendata-renewables.engie.com/explore/index

Thank you very much...


Exported functions are detected as props

The export function myFunc() {} syntax is treated the same way as export let myFunc = () => {}, even though it shouldn't be. As mentioned in the Svelte docs, export function () {} makes it a read-only export that exposes that function to the parent component (to be used with bind:this). However, I have the following component:
<script>
  export function showSnackbar(options) {
    // some code
  }
</script>

<slot {showSnackbar} />

(irrelevant code removed - full component can be found here)
And this generates typings as if showSnackbar is a prop that accepts a function, and uses the full implementation code as the default value. .d.ts:
/// <reference types="svelte" />
import { SvelteComponentTyped } from "svelte";

export interface SnackbarContainerProps {
  /**
   * @default () => { const { component = Snackbar, props = {}, duration = 4000 } = options; const key = { component, props }; key.props.closeCallback = function close() { clearTimeout(key.timeoutID); removeSnackbar(key, true); }; key.timeoutID = setTimeout(removeSnackbar, duration, key, false); registeredSnackbars.add(key); registeredSnackbars = registeredSnackbars; return { close: key.props.closeCallback, expired: new Promise(resolve => (key.resolveExpiredPromise = resolve)), }; }
   */
  showSnackbar?: () => any;
}

export default class SnackbarContainer extends SvelteComponentTyped<
  SnackbarContainerProps,
  {},
  { default: { showSnackbar: () => any } }
> {}

I think this might be a Svelte limitation (as I don't see how can this be properly represented with SvelteComponentTyped), but even then maybe it's best to ignore function exports.

On the other hand, I just found out that renamed exports are not documented. If I have
let _class = null;
export { _class as class };

for example (which I do have in many components), neither _class nor class make it to the generated declaration file.

On the other hand, I just found out that renamed exports are not documented. If I have
let _class = null;
export { _class as class };

for example (which I do have in many components), neither _class nor class make it to the generated declaration file.

This is a blocker for me as well right now 👍🏻
But for me it is exporting an object containing functions.
  /** @type {AccountCreateActions} */
  export const actions = {
    create: async (email, password, name = "") => {
      try {
        const response = await Appwrite.sdk.account.create(
          email,
          password,
          name
        );
        dispatch("success", response);
        return response;
      } catch (error) {
        dispatch("failure", error);
      }
    },
  };

is generated into:
    export interface CreateProps  {

      /**
* @constant
* @default { create: async (email, password, name = "") => { try { const response = await Appwrite.sdk.account.create( email, password, name ); dispatch("success", response); return response; } catch (error) { dispatch("failure", error); } }, }
*/
      actions?: {     create: async (email, password, name = "") => {       try {         const response = await Appwrite.sdk.account.create(           email,           password,           name         );         dispatch("success", response);         return response;       } catch (error) {         dispatch("failure", error);       }     },   };
    }

Using export let actions makes it generate valid definitions und uses my @typedef at least, but still sets the wrong @default notation.

in v0.8.1, function declarations are now typed as accessors on the SvelteComponentTyped interface.
Input
<script>
  export let fnA = () => {};

  export const fnB = () => {};

  export function add(a, b) {
    return a + b;
  }

  /**
   * Multiplies two numbers
   * @type {(a: number, b: number) => number}
   */
  export function multiply(a, b) {
    return a * b;
  }
</script>

<h1 on:click={fnA}>Heading</h1>

Output
/// <reference types="svelte" />
import { SvelteComponentTyped } from "svelte";

export interface InputProps {
  /**
   * @default () => {}
   */
  fnA?: () => {};

  /**
   * @constant
   * @default () => {}
   */
  fnB?: () => {};
}

export default class Input extends SvelteComponentTyped<InputProps, {}, {}> {
  add: () => any;

  /**
   * Multiplies two numbers
   */
  multiply: (a: number, b: number) => number;
}

Great! I can confirm that the exported function is now treated as an accessor, but for some reason it still emits a @default annotation for it that contains the full implementation of the function, as outlined in the original comment on this issue.

@aabounegm The @default comment notation is omitted in v0.9.0


Use semver ^5.1.0 as reference

Tests against npm semver.
Issue with pre-releases (#1) isn't showing up
(on npm 3.6.0 and node v5.7.1). Introduces a
dev dependency on semver.

Based on this test, issue #1 doesn't occur (at least with above-quoted versions).

Per travis-ci tests pass on node 0.10 and 0.8 also


New package: Nurbs v0.3.0

Registering package: Nurbs
Repository: https://github.com/HoBeZwe/Nurbs.jl
Created by: @HoBeZwe
Version: v0.3.0
Commit: d251bcf084e7aa6e7c467f0083507b3fb6c947c5
Reviewed by: @HoBeZwe
Reference: https://github.com/HoBeZwe/Nurbs.jl/commit/d251bcf084e7aa6e7c467f0083507b3fb6c947c5#commitcomment-94848799

I noticed you changed the name from NURBS to Nurbs here. I think this is a case where the fully upper-case name is better (I assume you made the change to appease the automated checks).

I guess you are right. I will revert the changes and start the registering process again. Thank you for the fast feedback!

Is a manual merge now justified? Or is there anything else I need to do?
I guess the question about the package name is settled: all upper case letters make sense here since the abbreviation NURBS is the established name for non-uniform rational B-splines as well as curves and surfaces based on these splines.


Add examples of deep repairs in README

https://github.com/Micront/htsjdk/blob/4a38df1ce15e65da27f4b610ad510240aeb48c60/src/java/htsjdk/variant/variantcontext/CommonInfo.java#L105-L107

This one has a neat repair https://github.com/netty/netty/blob/c78afbc4f13ac4d6cb488fb14b31379960f03fff/codec-http/src/main/java/io/netty/handler/codec/http/multipart/DefaultHttpDataFactory.java


vcpkg CONTROL file has incorrect curl version

Hello,
the vcpkg CONTROL file for teemo should be specifying Build-Depends: curl[non-http,openssl] instead of Build-Depends: curl[non-http].
As it is, builds will fail on systems without a system-wide OpenSSL installation. The OpenSSL dependency was automatically met when cpprestsdk was still a dependency of this project, but it was since removed.

thank you, I will update as soon as possible!


Imports block detection needs to be smarter

The current imports block detection need to be improved,  for example, this is a typical and common import block in applications that supports several versions of some library or different versions of python even:
try:
    import ujson as json
except ImportError:
    import json

from collections import defaultdict
import sys

def main():
    print(os.uname())
    a = defaultdict(list)

This is translated to
import json
import sys
from collections import defaultdict

import ujsonas json

try:
    import ujson as json
except ImportError:
    import json

from collections import defaultdict
import sys

def main():
    print(os.uname())
    a = defaultdict(list)

That is a complete disaster, in the other hand, the scope is fine:
print(scope.find_unresolved_and_unreferenced_symbols())
(set(['os.uname']), set(['sys', 'json', 'main']))

The importer has no clue about what is really imported in the file as its not able to know where the imports block really ends.

any updates?


Fix flaky object controller test

Fixed the flaky test mentioned in #282
The flaky test was caused by the design of the test. We need to make
sure the objects get propagated before checking if it's modified from
the source object.
Tested with "make test NOC=1" and it's not flaky any more.

/lgtm
/approve


Binding controller buttons to keyboard keys

I'm not sure if this is a known issue or not, or if it's just for me, but no matter what I try in the settings, binding a controller button to keyboard key doesn't seem to work. For example, if I bind "X" button to the keyboard key "a", pressing X won't write the letter "a".

Apparently, that's a bug from the razer synapse, closing it will fix it.


Change other NFT objects fields to v3

Current dependencies on/for this PR:

main

PR #257

PR #258

PR #262   👈

This comment was auto-generated by Graphite.


New Release?

There are a lot of new features/bugfixes (like FastBoot compatibility) in this addon since the last release in Nov 2017. Can we cut a new release so that people trying this addon for the first time don't get a broken experience because they're not on master?

@elwayman02 I'd love to, but I do not have permissions on npm. @briangonzalez any chance I could get those permissions to publish?

@elwayman02 looks like @briangonzalez published it for you. 💪

@yowainwright I do not think it has been published. The last publish on npm was two months ago. We also are mismatched in versions. GitHub's latest is 0.4.0 and npm has 0.5.0, so the tag must not have been pushed to GitHub. We should get the tag pushed and also release a new 0.6.0 to pull in the latest changes.