Threshold switch does not keep its signal when the game starts

Describe the Bug
Everytime I reopen my world (single player), all of my threshold switches seem to lose the last value read from the container they're attached to and send the wrong signal, for an instant.
Reproduction Steps

Fill any container with items and use a threshold switch (in this example, I've also used one with the signal inverted)

2.Use the signal to control a funnel attached to another chest -  for instance:

3.Quit the game, open it again and see that one batch (depends on the funnel config) has been released:

My workaround for this is to use a pulse extender configured for 10 ticks, between the switch and the link.
Expected Result
Threshold switches should maintain its signal when the world is restarted.
Screenshots and Videos
No response
Crash Report or Log
No response
Operating System
Fedora 38
Mod Version
0.5.1d
Minecraft Version
1.20.1
Other Mods
./durabilitytooltip-1.1.5-fabric-mc1.20.jar
./lambdynamiclights-2.3.2+1.20.1.jar
./freecam-fabric-modrinth-1.2.1+1.20.jar
./lithium-fabric-mc1.20.1-0.11.2.jar
./minihud-fabric-1.20.1-0.27.0.jar
./BetterF3-7.0.1-Fabric-1.20.1.jar
./sodium-fabric-mc1.20-0.4.10+build.27.jar
./supermartijn642configlib-1.1.8a-fabric-mc1.20.jar
./modmenu-7.2.2.jar
./continuity-3.0.0-beta.4+1.20.1.jar
./Jade-1.20-fabric-11.6.2.jar
./create-fabric-0.5.1-d-build.1161+mc1.20.1.jar
./tweakeroo-fabric-1.20.1-0.17.1.jar
./malilib-fabric-1.20.1-0.16.3.jar
./fabric-api-0.88.1+1.20.1.jar
./cloth-config-11.1.106-fabric.jar
./reeses_sodium_options-1.6.2+mc1.20.1-build.85.jar
./JadeAddons-1.20.1-fabric-5.2.2.jar
./indium-1.0.21+mc1.20.1.jar
./jei-1.20.1-fabric-<IP_ADDRESS>.jar
./litematica-fabric-1.20.1-0.15.3.jar
Additional Context
No response

Thanks for the resolution!
Just one quick question - is the new threshold switch code present on patch v0.5.1.f, or will it be released in the next version?

it'll be released as 0.5.1.j


Create groupby data using dask

This is a PR that adds a script to create the groupby data (no NAs and random) that uses dask. This will run the creation in parallel using the cores available on your machine.
To run the script you need to pass args, you can check the helper to get descriptions of the args as well the corresponding flags.
python create_groupby_data_dask.py -h
An example of how to use it to would be
python create_groupby_data_dask.py -n 1e6 -k 1e2 -nfiles 10 -dir test
I wasn't sure to replace the create_groupby_data that you already have as there are other scripts in the repo that use it, but you can adapt them later on.

@ncclementi - awesome, thanks!!!


Instruct git to ignore emacs backup files

The recommended way to do this is to add it to your home directory's config.  From man gitignore:
   ·   Patterns which a user wants Git to ignore in all situations (e.g.,
       backup or temporary files generated by the user’s editor of choice)
       generally go into a file specified by core.excludesfile in the
       user’s ~/.gitconfig. Its default value is
       $XDG_CONFIG_HOME/git/ignore. If $XDG_CONFIG_HOME is either not set
       or empty, $HOME/.config/git/ignore is used instead.

That said, I don't have a strong reason to oppose this, so I'll merge.  Thanks.

Merged manually.  Thanks.


feat(services/sftp): add append support for sftp

Updated:

add append support for sftp

In the tests, we will receive warning opendal::layers::complete: appender has not been closed, must be a bug. It also exists in github action test for fs implementation. Should we concern about it?

Should we concern about it?

Yes, it's a bug. cc @suyanhanx, we should close the appender by hand.

Should we concern about it?

Yes, it's a bug. cc @suyanhanx, we should close the appender by hand.

fixed in #2298 🥲 This is due to incorrect detection.


TH2-1401. Improve "go to timestamp/id" component

Sometimes when you pass an id to input it fetches item, and doesn't show modal.
After successfully fetching an item and changing timestamp value or even deleting it modal still shows previously fetched item.
Also i think 2 seconds timeout should be only when we initially fetched event or message, not always


Problems with the-unarchiver cask

I'm on a fairly fresh install of 10.10.3, and when I brew cask install the-unarchiver, the app (version 3.9.1) installs, but I can't select or deselect any of the archive types in the app Preferences window.
Perhaps this is related to their issue: https://code.google.com/p/theunarchiver/issues/detail?id=758
Either way, I'm not sure what I can do to fix this. I've tried brew cleanup && brew cask cleanup && brew cask install the-unarchiver but I see no change in behavior.

The current cask is installing https://theunarchiver.googlecode.com/files/TheUnarchiver3.9.1.zip which is the sandboxed Mac App Store version, I think the cask needs to install ``https://theunarchiver.googlecode.com/files/TheUnarchiver3.9.1_legacy.zip` which doesn't have these limitations. I'm going to try this out, and if it works, create a PR.


Simpler way to create the filterNonUnique

Description
The filterNonUnique array option is very nice but there is a simpler way to do it
Current
const filterNonUnique = arr => arr.filter(i => arr.indexOf(i) === arr.lastIndexOf(i))
My suggestion
const filterNonUnique = arr => [...new Set(arr)]
Please let me know if you want a pull request :)

This is a newer way that is finally available. I would have no problem with it being PRed. Consensus?

Seems like it already exists: uniqueElements
So basically an alias for that one...

The two snippets are different. Take this test case for example:
filterNonUnique([1, 2, 3, 4, 1, 1, 2]); // [3, 4]
uniqueElements([1, 2, 3, 4, 1, 1, 2]); // [1, 2, 3, 4]

filterNonUnique will remove every instance of each value that appears twice, while uniqueElements will only keep one of each value.
I'm closing on the basis that the proposed change alters the snippet functionality.


Add publicly accessible configuration file

We need to have a configuration file that can be manually edited by the user. It would be a good idea to create a new folder in the documents directory (taking cross-platform compatibility into account) and then create a configuration file in that folder.

I think it'd be best if we postponed creating a public configuration file. For now, we could just create an internal configuration file, which could be manually edited (before compiling, of course).


zotero uses wrong citation key when exporting tot bibtex file

Even though in the Zotero interface the correct key is displayed (fokkema1993), after exporting my library to .bib file, it uses the default Zotero scheme (fokkema_seismic_1993).
Other settings are also ignored. E.g. the 'fields to omit for export' option is also not working with input 'note' (the note field still appearts in the bibtex file).

Can you try clearing your cache in the Better BibTeX Advanced preferences and try again?

(there's no error in the log that was submitted so if the clear-cache doesn't help I'll need a new debug log with extended debugging on)

I didn't realize I was able to select Better Bibtex when exporting ><. Sorry for bothering you! Everything is working fine now.

Oh, right, you exported using "BibTeX"? That's the stock Zotero BibTeX exporter.

Yes, I somehow assumed BBT would 'overwrite' the default bibtex features of zotero, but being able to choose between default zotero and BBT in this way is actually pretty good!
Also, thanks for developing this plugin, makes zotero a much more usable.

It did at one time overwrite the standard BibTeX exporter, but that yielded some funky error reports for the Zotero devs who could have no idea who generated the error.


Use tempfile instead of tempdir

tempdir has been deprecated and using tempfile is recommended.

Can do without this dependency? I don't really have the time to review another dependency right now.

The implementation here is hard a bit. Actually, tempfile is using tempdir API (see: https://github.com/Stebalien/tempfile/pull/33) so the behavior shouldn't be changed. And it's just a dev-dependency so the effect is quite small.
But if you still have concerns, I'll close this or leave until you review it.

Removing the dependency wasn't too much work: #1239.

#1239 removed the dependency.


⚠ Bofrost.lu has degraded performance

In 4fde5d2, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2504 ms

Resolved: Bofrost.lu performance has improved in 0d05b9f after 7 minutes.


feat: add back to origin test case

feat: add back to relate origin test case in history record page.
click this button can back to relate origin test case.

If the original test case  is deleted, there will be the following prompt:


.bowerrc file is getting skipped from project root

It appears s2i is skipping dot files like .gitignore, .bowerrc etc. I have not tested this exhaustively but I was not able to use .bowerrc until I renamed it to bowerrc and then renamed it back during assemble.

@gabemontero can you take a look at this?  sounds like maybe we aren't using the right args when we tar up the src to inject it into the builder image.
@nitintutlani can you provide the exact command line you're running so we can try it with your repository and builder image?

@bparees will do

@bparees I have a fairly simple html/js website and an openshift source strategy buildConfig. Inside .sti/bin/assemble script as expected the source is delivered inside tmp/src/* but all dot files are missing including .bowerrc . There really is no command I am running in particular. Unfortunately this particular work is in a private repo.

and internet search of "tar hidden file" unveiled a common theme of appending a dot ( .) to the dir path.
For example:  http://www.digitesters.com/linux-include-hidden-files-in-tar-archive/
If our go based tar implementation holds to this behavior, I believe the solution here could be a simple one - append a dot to the directory parameter that serves as the source for the tar file.
I'll try an experiment and see if it holds.

Hmm ... actually, some of the search links say just specifying the dir, without say a * wildcard, should also include dot/hidden files.  And my experiments with both the current form and adding a . including files like .gitignore.
@nitintutlani could you re-run sti with the --save-temp-dir=true" option?  In the /tmp/stidirectory there should be a tar file calledtar.  Could you then run tar tvf ` on that file and tell us if the dot files in question are listed?
thanks

As per this log: I0820 11:24:52.274283 20065 tar.go:133] Adding to tar: /tmp/sti008846339/upload/src/.bowerrc as src/.bowerrc, /bowerrc file is inside the tmp/src folder.
There is a bug in my code. Due to presence of artifacts I could not move the entire src folder. I need to move regular files and hidden files in two steps:
mv /tmp/src/* ${APP_ROOT}/.
mv /tmp/src/.* ${APP_ROOT}/.

Thanks for your support and sorry for trouble.


[Question]:  Define meta data for each chunks

Describe your problem
Thank you for your excellent work!
We wanna use this Ragflow as our major knowledge base. Our documents have several pattern of structures. Each part in the structure has coherent contents. So we decide to split documents to chunks according to the structure.  How can we mark meta data in each chunk? we only find keywords attribute of chunks. This isn't suitable for marking meta data.

Meta data to chunk has not been supported yet.
You could contact us by<EMAIL_ADDRESS>

Thank you for your quick reply.
Another question: where do we get the sequence and page no of each chunk in the source document? We didn't found in the code and information in the chunk.

It's stored in ES about fields of position.


Support for axlsx  v3

Allow axlsx 3 as dependency
Add appraisal gem to test multiple axlsx versions
Use assert_nil rather than assert_equal nil in tests (latter is deprecated)

I've also added TravisCI config so that the tests will run on every push

@davefp could you bump up the version, so we can release?

@sakovias done! 😃

@sakovias also you may need/want to sign in on travis-ci.org to turn on CI for this repo.


swapd: bug fix, finality thr

Codecov Report

Merging #344 (c76f910) into main (c8c1592) will not change coverage.
The diff coverage is 0.0%.

:exclamation: Current head c76f910 differs from pull request most recent head 3704005. Consider uploading reports for the commit 3704005 to get more accurate results

@@          Coverage Diff          @@
##            main    #344   +/-   ##
=====================================
  Coverage   14.0%   14.0%
=====================================
  Files         27      27
  Lines       7257    7257
=====================================
  Hits        1019    1019
  Misses      6238    6238

Impacted Files
Coverage Δ

src/swapd/runtime.rs
0.0% <0.0%> (ø)

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 c8c1592...3704005. Read the comment docs.


Rya 250 Added Smart URI to the MongoDB Entity Index

Description
Added Smart URI to the MongoDB Entity Index
The Smart URI format:
[subject URI]?ryaTypes=[encoded map of type URIs to type names]&[type1.propertyName1]=[value1]&[type1.propertyName2]=[value2]&[type2.propertyName1]=[value3]
Tests
Unit tests
Tested serializing/deserializing URI
Tested storing/querying
Links
Jira
Checklist

[ ] Code Review
[ ] Squash Commits

People To Review
@kchilton2
@isper3at
@pujav65
@amihalik

asfbot build

looks good to me, about to merge


Update package metadata

This PR updates the package metadata in preparation for publishing a copy of the package under the team's scope.
✅ Updated package name to @metamask scope
✅ Updated GitHub links
✅ Updated package descriptions

Are we deprecating the old package?

Yup, I'll deprecate the old one!


Deployment is still "new_deployment" if the database isn't seeded

In some cases, the pod may exit after the database is migrated, but before it is seeded.  We were claiming that we were in "new_replica" status the next time it came up, but if there are no servers at all, that's not the case, it's still a "new_deployment".

Nice find @bdunne

Backported to morphy in commit b771e0363a9749c8ffb20365de1dec8fe48f4239.
commit b771e0363a9749c8ffb20365de1dec8fe48f4239
Author: Jason Frey<EMAIL_ADDRESS>Date:   Fri Jan 28 17:08:25 2022 -0500

    Merge pull request #21673 from bdunne/fix_new_deployment

    Deployment is still "new_deployment" if the database isn't seeded

    (cherry picked from commit 5fd6ef94a1762996acde9e3a76da68054e49a0c1)


Always escape identifiers in the set(), setUpdateBatch(), and insertBatch()

Fixed a problem where identifiers were not escaped in the set, setUpdateBatch, and insertBatch methods.
Description
I decided that there was no case where the column name in the set clause did not need to be escaped, so I turned off the unescaping process.
The same goes for the table name when inserting.
see: https://github.com/codeigniter4/CodeIgniter4/pull/2487#discussion_r716012806
Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPdocs
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[x] Conforms to style guide

You need to update the PHPdoc.
https://github.com/codeigniter4/CodeIgniter4/blob/ef08b6f4298678da304e5663f5862043338d3451/system/Database/BaseBuilder.php#L1343
And user guide, too.
https://codeigniter4.github.io/CodeIgniter4/database/query_builder.html#set

You need to update the PHPdoc.
And user guide, too.

Thanks, response.
Fixed it.
Thanks to your pointers, I realized that I have a problem similar to this case with the setUpdateBatch and insertBatch identifiers.
Thank you very much.
I've fixed this as well and added a test.

@ytetsuro
Your commit message like the following means the opposite what you did.

fix: removed unnecessary escaping process in setUpdateBatch as well.

It should be like fix: always escape identifiers in setUpdateBatch()

@kenjis

Your commit message like the following means the opposite what you did.

Thanks, response.
You are right.
Fixed it.💪

@ytetsuro
The first commit:

fix: Because we have made escaping unnecessary in places where it is not necessary.

Sorry, I don't know what you are saying.
DeepL translation:

逃げることを必要としない場所では、逃げることを不要としたからです。

Google translation:

必要のない場所での脱出を不要にしたからです。

I don't know what you are saying.

Sorry.
Fixed it.💪

I didn't understand it before, but this change is necessary for the normal use case in Oracle.
The normal use case is when an identifier is quoted and lowercased, like "name".
In this case, if a user specify $escape to false, the identifiers will not be quoted, and Oracle will not find the identifier.
name is not equal to "name".
See https://seeq.atlassian.net/wiki/spaces/KB/pages/443088907/SQL+Column+Names+and+Case+Sensitivity
for Oracle identifier case sensitivity.

@kenjis
I'm sorry to have bothered you.
I've aligned the commit messages.

@paulbalandan
I'm sorry.
I have no experience in writing changelog.
Please tell me.
Is the changelog to be updated using the following procedure?
Is it my CI4 repository to push the release branch?
https://github.com/codeigniter4/CodeIgniter4/blob/0f0c1d6514c0234f4d16bdca0e5e308a39168986/admin/RELEASE.md#codeigniter4

No, not there. I meant here.
https://github.com/codeigniter4/CodeIgniter4/blob/develop/user_guide_src/source/changelogs/v4.1.5.rst

@paulbalandan
Thanks, response.
Fixed it.

Thank you, @ytetsuro


Typo correction

Actual:
"You can override specific host.json values be creating an equivalent value as an application setting. "
Expected:
"You can override specific host.json values by creating an equivalent value as an application setting. "

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

ID: 27150c55-615f-fb44-d5e4-7b5815631c7d
Version Independent ID: 271006dc-afce-99f3-0790-41fa2e35b22b
Content: host.json reference for Azure Functions 2.x
Content Source: articles/azure-functions/functions-host-json.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

Hello @joelmdev - Thank you for reporting this and helping us drive improvements towards MS Docs! I have created a PR that fixes the typo, and once it is merged, this issue will self-close.


Support error as a value for Comparison

Alternative to #54
Instead of removing NilError, keep it around, but still support nil/error in Assert/Check so that Comparisons can be created from any function that returns error.
Also fail instead of panic in the default case.

I think I prefer this one because people are used to NilError (kinda), but I feel #54 keeps the design and usage simpler (i.e. you always use assert.Assert)

Ya, I think we should take this one. I added one more commit which removes cmp.NilError, so to check for error you can do: assert.Assert() or assert.NilError() which are equivalent except that NilError only accepts an error not interface{}, and I documented that these are equivalent. assert.Check() also works if you want a Fail instead of fatal, so there's no reason to keep the cmp.NilError().

We can always decide to deprecate NilError later.


Should OTEL language libraries also support pull Prometheus exporters or reuse the Collector Prometheus exporter

Discussion needed in spec SIG. Will file more detailed issue in spec repo.

A decision taken in the Collector SIG was to use the Collector based PRW Exporter for all Prometheus remote-write push based services.
However each individual language library can still support a pull-based OpenMetrics exposition end point.


Add update_field_value action

As described in #35, creates an action to update_field_value where you can set an existing field to a new value.

Looks good, aside from the tiny YAML file lint issue I noted.
CI is failing, but it's not due to anything in this PR. I'll rerun the CI jobs for this PR once you fix the tiny linting thing and once https://github.com/StackStorm/st2/pull/4819 is merged.

Updated with corrections.

No problem. As the saying goes, a poor craftsman blames his tools. In this case, it was VSCode's fault as it likes to insert blank lines with spaces as a way of being "helpful". :smile:

Thanks for the contribution!


Handle cancelling dbt Cloud jobs

Solves #20
Adding logic to cancel the job that was triggered if the action is still running when we cancel it.
It can be useful when workflow_dispatch is used, but also especially when people trigger CI jobs using this action.
With the additional config
on:
  - pull_request

concurrency:
  group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
  cancel-in-progress: true
...

we can actually trigger dbt Cloud CI jobs from the action and if we push more commits to the PR, it will trigger a new CI job and cancel the previous one.

Quick ping to see if anyone on the fal side still looks after this repo 😃

@chamini2 sorry for the direct ping, but I think that you are part of the fal-ai org in GitHub


DefaultScheduler can cause persistent data loss, when block is restarted

Persistent data can be lost if block is restarted after initial deployment, when DefaultScheduler is used. This happens only if scheduler was not restarted after the initial deployment.
Please see the test in https://github.com/xtenzo/dcos-commons/commit/737b1d113349951bcb72f0b67b1158524483b59f that reproduces this. If "Simulate scheduler restart block" is uncommented, then test passes.
When Plan is initialized for initial deployment from the ServiceSpecification, it does not contain any reservation IDs. During initial deployment reservation IDs are assigned, but they are not reflected in the plan. So when restart happens, a new task is started instead of the old one (which also involves RESERVE and CREATE operations), and soon after that cleaner UNRESERVEs and DESTROYs resources from the original task. However, after scheduler is restarted, DefaultBlockFactory uses TaskInfo from StateStore to create plan with reservation IDs assigned, so restart runs as expected.

I'm not sure this isn't by design.  Restarting a block (now step) should redo the work it did before.  In this case, that's initial launch.  If you want to restart a Task, restarting the deployment block is not the way to do it.
The way to do it is through this endpoint or equivalent implementation.
Then the RecoveryManager will relaunch using the same resources as expected.

@xtenzo, please reopen if that did not answer your question.


[DC-510] Reenable preview dataset

Do not merge until DC-347 is in prod

@fboulnois we actually already seem to track a mixpanel event, so that's a noop!


Stop DFK cleanup in atexit, because this doesn't work in Python >3.12"

Description
atexit handlers work differently in the latest version of Python (version 3.12) and it is not safe to automatically call cleanup any more in Python 3.12
Changed Behaviour
Instead of cleaning up, the code will now emit a WARNING log message to help users understand the change.
Fixes
Fixes # (https://github.com/Parsl/parsl/issues/3104)
Type of change

Bug fix

right now, this PR has a big jumble of changes that are unrelated to issue #3104 - look at https://github.com/Parsl/parsl/pull/3141/files and see them all. i'll work through fixing that with you.

This PR was tangled up, but we untangled things into a new PR #3165 that is now merged. So I'll close this issue.


Doesn't read pixel data for fast parsing

Hello,
This is a question, is it possible to read only meta-data part of (large) files (without pixel data) to optimize processing time ?

Hello! The short answer is not yet, but that is indeed planned and an implementation should arrive soon-ish. This is being tracked in #1.

See that, but I expect there are an option in dicom_object::open_file in the meantime.
Anyway thank you for your fast response and you fabulous work.
Did you have an idea about release time of lazy loading feature?

See that, but I expect there are an option in dicom_object::open_file in the meantime.

I suppose that an additional function which loads a DICOM object in-memory without any pixel data could be added as a quick
workaround, and it should be easy to implement. I admit that the pixel data can be ignored entirely in some use cases. Pull requests are welcome, and I can provide additional guidance if necessary.
#1 would be more generic so that it can serve other use cases, such as reading image frames independently, or for not loading certain heavier data sequences when not necessary. Committing to a deadline has been a bit difficult, as it depends a lot on how much of my free time is invested on this, but I've been wanting to bring this to the library for quite a while. I will try to pick this up this weekend.


Can't load databases with numeric schema names

Description
Loading databases with numeric schema names e.g. 1 lead to following error message:
SQL-Fehler [1064] [42000]: (conn=983) You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near '1 WHERE Table_type <> 'SEQUENCE'' at line 1
  (conn=983) You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near '1 WHERE Table_type <> 'SEQUENCE'' at line 1
  (conn=983) You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near '1 WHERE Table_type <> 'SEQUENCE'' at line 1
    You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near '1 WHERE Table_type <> 'SEQUENCE'' at line 1
    You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near '1 WHERE Table_type <> 'SEQUENCE'' at line 1

DBeaver Version
CE 22.1.4
Operating System
Ubuntu 20.04.4
Database and driver
DB: 10.6.4-MariaDB-1:10.6.4+maria~focal
Driver: MariaDB JDBC 2.7.1
Steps to reproduce
No response
Additional context
No response

Hello @chWagnr
Can you open catalogs with non-numeric names in this connection?

Hi.
Yes, that works. Maybe it's not working because of the missing quotation of
the numeric name?
Anastasiya @.***> schrieb am Di., 9. Aug. 2022, 17:05:

Hello @chWagnr https://github.com/chWagnr
Can you open catalogs with non-numeric names in this connection?
—
Reply to this email directly, view it on GitHub
https://github.com/dbeaver/dbeaver/issues/17313#issuecomment-1209504342,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEJS5HXXDSIKQCFHLUHGNPLVYJXVBANCNFSM56AQTOXA
.
You are receiving this because you ?were mentioned.Message ID:
@.***>

We should quote database names which contains only numeric characters. For database names like 123db it works fine.


package.toml doesn't support quoted or dotted key names.

e.g.
[depends.debian]
"libstdc++-arm-none-eabi-newlib" = {}

gives
simon@Starbuck cubicle % cub package update qmk
Error: error scanning built-in packages in "/Users/simon/Github/cubicle/packages"

Caused by:
    0: could not read manifest for package "qmk": "/Users/simon/Github/cubicle/packages/qmk/package.toml"
    1: data did not match any variant of untagged enum DependencyOrTable for key `depends.debian` at line 1 column 1

same for
[depends.debian]
libusb-1.0-0={}

this running #38 if that makes a difference.

I couldn't repro a failure with your first example; this seems to parse for me:
[depends.debian]
"libstdc++-arm-none-eabi-newlib" = {}

Is it possible you had more stuff elsewhere in the file that was causing the issue?
I did get that error with your second example:
[depends.debian]
libusb-1.0-0={}

which needs to be quoted like this:
[depends.debian]
"libusb-1.0-0"={}

since the dots mean something to TOML.

yeah, looks like a PBK problem, sorry.


Extend documentation for the minimal tflite example.

Add tflite:: prefix for all tflite types for consistency.
Extend documentation of how to actually read the input/output buffers of the interpreter.

Bonus:

Adds missing entry for android_arm64 in .baselrc.

@googlebot I signed it!


There is any plan to adapter the latest react version?

i'm  using the react-localize-redux --- "3.5.3"   react ------ "16.4.1 "
if i update the react version to 16.8.6 ,  the react-localize-redux could't used!
and the translate have  delay in some situation, should be click the browser refresh , the translate  will be Ok
@ryandrewjohnson

Closing due to lack of details in the issue. An updated version is in the works and is being tracked in #140


How to use existing static IP address when installing chart

Hi
How can I use an existing IP address when installing?
Sorry if this question should be for nginx-ingress?
Note: I have a static IP address on Google Cloud Network
helm version
Client: &version.Version{SemVer:"v2.8.1", GitCommit:"6af75a8fd72e2aa18a2b278cfe5c7a1c5feca7f2", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.8.0", GitCommit:"14af25f1de6832228539259b821949d20069a222", GitTreeState:"clean"}

I have the following value.yaml:
# Default values for sample-go.
# This is a YAML-formatted file.
# Declare variables to be passed into your templates.

replicaCount: 1

image:
  repository: gcr.io/my-cloud/sampler-go
  tag: latest
  pullPolicy: IfNotPresent

service:
  type: LoadBalancer
  loadBalancerIP: x.x.x.x
  port: 80

ingress:
  enabled: true
  annotations: {
        kubernetes.io/tls-acme: "true",
    kubernetes.io/ingress.class: "gce",
    kubernetes.io/ingress.global-static-ip-name: "kube-ingress"
  }

    # kubernetes.io/ingress.class: nginx
    # kubernetes.io/tls-acme: "true"
  path: /
  hosts:
    - chart-example.local
  tls: []
  #  - secretName: chart-example-tls
  #    hosts:
  #      - chart-example.local

resources: {}
  # We usually recommend not to specify default resources and to leave this as a conscious
  # choice for the user. This also increases chances charts run on environments with little
  # resources, such as Minikube. If you do want to specify resources, uncomment the following
  # lines, adjust them as necessary, and remove the curly braces after 'resources:'.
  # limits:
  #  cpu: 100m
  #  memory: 128Mi
  # requests:
  #  cpu: 100m
  #  memory: 128Mi

nodeSelector: {}

tolerations: []

affinity: {}

Hi @appsuwear
You can specify a variable at install or upgrade by using --set service.loadBalancerIP="<ip>".

Hi and thanks @jascott1
Just tried that and it did not work. Also I would expect to be able to put this into the values.yaml?
Updated the question to include helm version.

I couldn't figure out which chart you are using since your values look different from the one in the stable/nginx-ingress chart. We need to look at the templates to determine what is happening so if you can, please share the  chart you are using.

I have just installed helm install stable/nginx-ingress and then used helm create <app name>.
The app/Container is pushed to gcr.io//sampler-go. The container is working fine when using Google Cloud provide LoadBalancer.
The problem is just how to use the Nginx-Ingress with a Helm Chart - and again, maybe this is the wrong report to ask the question.
But the Ingress is automatically created with helm create <app name>.
ngress:
  enabled: true
  annotations: {
    kubernetes.io/ingress.class: "nginx",
    kubernetes.io/ingress.global-static-ip-name: "kube-ingress"
  }

Thanks

@appsuwear To be clear you are asking how to use your own Ingress with a service from your own chart? You don't have to use the Ingress resource generated with the helm create command. You can just delete that resource or disable it and configure your nginx-ingress to point to your service.

Let me share what worked for me (I added controller.service.loadBalancerIP) :
helm install --name ng1 stable/nginx-ingress --namespace kube-system --set controller.service.loadBalancerIP="<IP_ADDRESS>"

but the most important was to reserve an External IP addresses, static, in exactly the same cluster region, so if the cluster IPs are on us-central1, create a static IP in us-central1. If you try global, or us-east1, it will not work.


WebTorrent Desktop v0.18.0

I just resolved the conflict. GitHub offered an in-browser editor to do it. Pretty cool, but it created a nasty commit that merges master into this PR branch -- oh well.
I'll do a final test of Windows and Linux tomorrow and merge this PR.


Fix visualize_heatmap Throwing Error for Feature Lists of Length 1

visualize_heatmap can now handle feature lists of any length.
Closes #102.

LGTM


[Search result order] Exact match is not the first card

Describe the bug
When searching in PyPI.org, the exact match is not the first search result.
Example: When searching latex2mathml, the first result is mathml2latex instead of latex2mathml

Expected behavior
Exact matches goes on top
To Reproduce
Search latex2mathml
See: https://pypi.org/search/?q=latex2mathml
My Platform

Additional context

Duplicate of https://github.com/pypi/warehouse/issues/10718.


Training teams: General Manager to ATC/Pilot Training Manager

Just noticed this on the website; the role titles for the ATC/Pilot Training Manager changed away from 'General Manager' ~two years ago (see for example Discord roles).

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


Screenshot in the README

(perfect timing! you seem to have just published this project a day before I started looking for it!)
It would be nice to have a screenshot of the application in the README.


Unsupported architecture

Installation of the add-on fails & I'm getting this error when trying to install to Raspberry Pi 3 B+
19-03-09 10:58:13 ERROR (SyncWorker_13) [hassio.docker.interface] Can't install korylprince/hassio-caddy-armv7:1.4 -> 404 Client Error: Not Found ("pull access denied for korylprince/hassio-caddy-armv7, repository does not exist or may require 'docker login'").

I'm not even sure if this is supposed to be installable, but as armhf architecture Docker images exist for the addon, I guess it could be made to work?
Thanks

It looks like they just added support for armv7 a few days ago. I've updated the add-on to 1.5 and added a build for armv7. Please try to install the add-on now and let me know if you have any issues.


Charge for decoding JSON

PR checklist:

[x] Test coverage for the proposed changes
[x] PR description contains example output from repl interaction or a snippet from unit test output

The proposed change results in a change of gas consumption, which is visible in quite a few golden tests.

[x] Documentation has been updated if new natives or FV properties have been added. To generate new documentation, issue cabal run tests. If they pass locally, docs are generated.

No such natives or properties were added.

[x] Any changes that could be relevant to users have been recorded in the changelog
[x] In case of changes to the Pact trace output (pact -t), make sure pact-lsp is in sync.

I think there are none.

Additionally, please justify why you should or should not do the following:

[ ] Confirm replay/back compat

This has to be done for completeness' sake but there is no reason to think it would fail, because the changes are guarded by the fork flag.

[ ] Benchmark regressions

Not sure what exactly this entails, but I can do it.

[x] (For Kadena engineers) Run integration-tests against a Chainweb built with this version of Pact

CI did this for me for the chainweb-node PR.

Does this PR attach gas costs to operations that haven't been part of public language specification until now?
I wonder whether this makes gas cost dependent on implementation details that are not part of the surface semantics of pact. In particular, does this mean that changing the underlying serialization and storage format would become a forking change?

@edmundnoble This PR needs to be rebased, it contains an inverse of the prettyprinter PR, for example.

@jwiegley it's based on the microgas branch, maybe it doesn't have it yet?
@larskuhtz yeah, it's not nice, but write's already gassed this way.

@edmundnoble In that case, perhaps the PR merge base should be changed to reflect which branch you're changes are a delta for?

The base is microgas. I think the trouble is I merged master and microgas is still out of date. Once that's fixed I can rebase.

Postponing for now.


links in traintrack

In the installation section there is a repeated link:
https://ohbm.github.io/hackathon2022/traintrack/

installation should be updated to this one https://psy6983.brainhackmtl.org/modules/installation/


Bugfix: Bug view didn't open in diff view

Fails with:
TypeError: this.runData is undefined

In which line do you get this error? What scenario leads to this? I can't reproduce this error.

I confirm that it is fixing the issue for me.
Thanks


Tasks are soon wiped when server is unavailable

The problem happens after a period of time (maybe a couple hours) of no connection with the server. All tasks of the remote TaskList on the client (Android) are deleted, including any newly added tasks.
What should happen is the remote TaskList should remain available when offline, then synced when connection resumes.
I'm using CALdroid with a Radicale server on a local wifi only. Everything syncs fine when always connected, but this is impractical for my desired setup.

I just found issue #404 that describes the same problem in one of the comments.


[BEAM-646] Get flink runner out of apply

Just for testing and letting people have a look. I want to refactor this once the lightweight override factories by @tgroh from #2039 are in.

Coverage increased (+0.1%) to 69.288% when pulling 55447b46708c85eac14984739bbbfc56580afc53 on aljoscha:get-flink-runner-out-of-apply into 8cfb3d125d9a5b5950f487e2a31fa6d4a5096a04 on apache:master.

R: @tgroh

@tgroh I'll let you know once this is ready for review again. I first want to get some preliminary stuff in.

@tgroh I'm now happy with the state of this. This sits on top of #2072, #2055 and #2035 but it's probably also fine to review it all here in one go.

Coverage increased (+0.02%) to 69.3% when pulling 9708b34746627f1d9feba2d83bd633eef924513c on aljoscha:get-flink-runner-out-of-apply into 5ca149de2eb7924a996902419ade156ca526412c on apache:master.

Run Flink RunnableOnService

Coverage remained the same at 69.295% when pulling a040219eb1ac6b5d5a51a38ef0e0b266edb9082a on aljoscha:get-flink-runner-out-of-apply into fbaac0fc81d12ca63b35b0e1f9176660c5164275 on apache:master.

Run Flink RunnableOnService

Thanks! It seems Jenkins doesn't want to take commands from me here? Is there something I have to set up for this? I can manually start builds using "Build with Parameters", so I have an account and (it seems) some permissions.

Yeah, I don't really have any idea. Feel free to merge whenever


[Bug] IndexError: too many indices for array with PyAudio

Description
I am attempting to process frames of waveforms generated by pyAudio with the Separator, and unable to figure out how to get my audio data into the right shape for processing
Step to reproduce
I'm using Soundflower to send my audio to a virtual mic jack, and then using pyAudio to pick up that audio and attempt to process it.
My code looks like
import numpy as np
from spleeter.separator import Separator

class AudioProcessor(object):
    FORMAT = pyaudio.paFloat32
    CHANNELS = 1
    RATE = 44100
    CHUNK = 1024
    hop_s = CHUNK // 2  # hop size

    def __init__(self):
        self.separator = Separator('spleeter:4stems')
        self.p = pyaudio.PyAudio()
        stream = self.p.open(format=self.FORMAT,
                        channels=self.CHANNELS,
                        rate=self.RATE,
                        input=True,
                        output=True,
                        input_device_index = self.get_input_device_index(),
                        output_device_index = self.get_output_device_index(),
                        frames_per_buffer = self.CHUNK,
                        stream_callback=self.callback)

        stream.start_stream()

    def get_input_device_index(self):
        for i in range(self.p.get_device_count()):
            if self.p.get_device_info_by_index(i)['name'] == 'Soundflower (2ch)':
                print("Found!{}".format(self.p.get_device_info_by_index(i)['name']))
                return i

    def get_output_device_index(self):
        for i in range(self.p.get_device_count()):
            if self.p.get_device_info_by_index(i)['name'] == 'Built-in Output':
                print("Found!{}".format(self.p.get_device_info_by_index(i)['name']))
                return i

    def callback(self, in_data, frame_count, time_info, status):
        ret = np.fromstring(in_data, dtype=np.float32)

        prediction = self.separator.separate((ret, 2))
        print(prediction.keys())
        return (in_data, pyaudio.paContinue)

if __name__ == "__main__":

    a = AudioProcessor()
    import time
    while True:
        time.sleep(0.1)

I have tried changing channels to 2, and also have tried changing the call to separator to be just self.Separator.separate(ret), and it is the same result either way.
Output
WARNING:tensorflow:From /Users/ben/code/visualization/visualization/lib/python3.6/site-packages/tensorflow_core/python/ops/resource_variable_ops.py:1630: calling BaseResourceVariable.__init__ (from tensorflow.python.ops.resource_variable_ops) with constraint is deprecated and will be removed in a future version.
Instructions for updating:
If using Keras pass *_constraint arguments to layers.
Traceback (most recent call last):
  File "pyglet/audioprocessor.py", line 178, in callback
    prediction = self.separator.separate((ret, 2))
  File "/Users/ben/code/visualization/visualization/lib/python3.6/site-packages/spleeter/separator.py", line 184, in separate
    return self._separate_librosa(waveform, audio_descriptor)
  File "/Users/ben/code/visualization/visualization/lib/python3.6/site-packages/spleeter/separator.py", line 161, in _separate_librosa
    stft = self._stft(waveform)
  File "/Users/ben/code/visualization/visualization/lib/python3.6/site-packages/spleeter/separator.py", line 126, in _stft
    d = data[:, :, c].T if inverse else data[:, c]
IndexError: too many indices for array

Environment

OS
MacOS

Installation type
pip

RAM available
8gb

Hardware spec
2.7 GHz Dual-Core Intel Core i5, Intel Iris Graphics 6100 1536 MB

This is an awesome library that I would love to be able to get working for this project!

Hi @dimdog,
ret = np.fromstring(in_data, dtype=np.float32) returns a NC long 1D numpy array (with interleaved channels) while Separator needs a NxC (2D) numpy array, where C is the number of channels and N the number of samples per channel. The number of channels needs to be equal to 1 or 2 and note that, in both cases, the separator will output stereo waveforms.
So you need to reshape properly the ret array.
Also, be aware that:

separating on very short audio snippets (here, you use 1024 sample long audio snippets) will results in very poor separation performances and in very inefficient computations (since anyway the separator will pad the audio to process 12s long segments). You need to take much longer snippets (at least at the order of 1s), which leads to the second point.
you seem to want to perform separation realtime, while spleeter is not designed for that. Taking sufficiently long segment (which is needed) will result in quite large latency.


Directional lighting and discarded surfaces

In the example above I have a room with a directional light in the scene. Ideally I want the room to be unlit and the outer surfaces to be discarded as they will never be seen and should not be rendered. Is there a better way to go about this or can discarded surfaces be changed to block light rays?

Right now the only way to do this is by putting these discarded surfaces in another (csg) model, select it and set Cast Shadows to "Shadows only", then you won't need to discard them .. they won't be rendered, but they will still cast shadows..

I found a solution for this. A property of the CSG Model script under Cast Shadows I can set it to "Two Sided", now surfaces with normals away from the directional light also catch light rays. You had programmed a solution all along!

Ah, didn't realize that that's what you wanted. Glad that that works for
you :)
On Fri, Nov 4, 2016 at 5:21 PM, Andrew<EMAIL_ADDRESS>wrote:

Closed #54
https://github.com/LogicalError/realtime-CSG-for-unity/issues/54.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/LogicalError/realtime-CSG-for-unity/issues/54#event-848136057,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJpGHW1JZpWVEwjDIlZeCol-NNbfF6Hks5q61uhgaJpZM4KkaIP
.


[Bug] pyatv not found after upgrading by from B94 to B95

What version of Music Assistant has the issue?
B95
The problem
Settings page does not load
logs throw error from module pyatv not being found
this is on x86 Home Assistant OS
How to reproduce
Upgrade from B94 to B95
attempt to view settings page
Relevant log output
No response
Additional information
Manually running pip install pyatv within the addon’s docker container cleared the issue
What version of Home Assistant Core are your running
2024.2.1
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Generic x86-64 (e.g. Intel NUC)

This is known and will be fixed in the next beta

Fixed in b96


Upcased sorts to allow for non-ascii sorting. Updated docs to include…

… require statements.
Currently the sorting order of a group of aliases for example is sorting it by ASCII characters which is the default sorting order of Enum.sort. In an effort to sync this up with Credo I'm adding a String.upcase().
For example, Credo would expect the following block to look like this:
alias Aspire.Patient
alias Aspire.PG.Repo

but currently eunomo would sort it to look like this
alias Aspire.PG.Repo
alias Aspire.Patient

Makes sense, thanks.

@Roberto-XY could we get a new release sent out for this whenever you have a chance please?


Fix for iOS camera orientation issue

Fix for iOS camera orientation issue (https://github.com/Redth/ZXing.Net.Maui/issues/40)

One issue I found with this fix is that it updates appropriately for the device orientation but doesn't account for the application orientation which could be different. E.g. The device is held in portrait when the scan starts but the application only supports landscape will lock the camera view in portrait thus being offset by 90 degrees.
It also doesn't re-apply the layout when the device orientation changes. E.g. Change from landscape left to landscape right keeps the camera view in landscape left, thus resulting in the camera being upside down.
I've created pull request #186 with help from a coworker to handle these two scenarios.


Row grouping with Vertical scroll

I'm submitting a ...  (check one with "x")
[ V] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request => Please do not submit support request here, post on Stackoverflow or Gitter

Current behavior
I know that there is already an open issue about it
but nothing is updated there for a long time, and this is realy urgent to me.
when i set vertical scroll to true and initialize the groupRowsBy parameter -the page just collapses.
i have lots of data, so i cannot give up on this virtual scroll.
and i need this row grouping as well.
any idea would be very appreciated!
Expected behavior
I expect the row grouping to work well with the vertical scroll
Reproduction of the problem
just took the demo of the 10k lines and add the "groupRowsBy" parameter.

Table version:
last version

Angular version: 5

Browser: [all  ]

Language: [TypeScript]

same issue.

me too

Please fix this!

Has anybody found any solution for this ?


Browser module imports don't work

I'm trying to use this lib in the browser without compilation and get this error.

try the following code to reproduce
<!doctype html>
<html lang="en">
  <head>
    <meta charset="UTF-8" />
    <meta name="viewport" content="width=device-width, initial-scale=1.0" />
    <title>Document</title>
  </head>
  <body>
    <script type="module">
      import json from "https://unpkg.com/esptool-js@0.3.2/lib/targets/esp32.js?module";
    </script>
  </body>
</html>

I guess the reason for this this line:
import ESP32_STUB from "./stub_flasher/stub_flasher_32.json?module";
The point is that you don't need to use ?module for json files

@brianignacio5 Could you fix this please?

How about using the resulting bundle file<EMAIL_ADDRESS>?

Yes, that would work. But anyways, it's better be fixed

Also you need to update README then, because it recommends using https://unpkg.com/esptool-js/lib/index.js?module


feat: improve how references are displayed

This PR improves the way references are displayed. It depends on https://github.com/antfu/changelogithub/pull/18.
Before this PR, references are inserted into backticks to display them as code snippets, and they are separated by a single space.
After this PR, references related to an issue or PR are displayed in a list (with commas and a final "and" word when necessary) and prefixed by "in", and references to a specific commit are displayed in parenthesis.
Three examples of the current behavior is:

Support for multiple @ parents (@media, @supports, etc) #1161 by @antfu
Revert variant apply order in #1127 #1162 by @antfu
New preflights and minify options a60a0e9b

After this PR, the changelog would look like this:

Support for multiple @ parents (@media, @supports, etc) by @antfu in #1161
Revert variant apply order by @antfu in #1127 and #1162
New preflights and minify options (a60a0e9b)

This design feels cleaner to me (the 'code snippet' references were attracting too much attention), it's inspired by the way the GitHub-generated release changelog is formatted.

@antfu done!


Drude Installer conflicts with custom boot2docker VM vhost_proxy setting

The drude installer will install the Vagrantfile and vagrant.yml (one level above), but the vhost_proxy setting causes port conflicts on 443 with the way the default docker-compose.yml is set.
See more details on the issue I opened upstream https://github.com/blinkreaction/boot2docker-vagrant/issues/31
Example docker-compose.yml
# Docker and Docker Compose based environment for Drupal.
# See https://github.com/blinkreaction/docker-fig-drupal for more information and documentation.

# Web node
web:
  hostname: web
  image: blinkreaction/drupal-apache-php:2.2-5.5
  ports:
    - "80:80"
    - "443:443"
    # Use IP based mapping when working with multiple projects. See docs (^^) for more details.
    #- <IP_ADDRESS>:80:80
    #- <IP_ADDRESS>:443:443
  volumes:
    # PHP configuration overrides
    - "./.docker/etc/php5/php.ini:/etc/php5/fpm/conf.d/z_php.ini"
    # Project root folder mapping
    - &project_root ".:/var/www"
  links:
    - db
    # Uncomment this and the memcached service definition below to start using memcached.
    #- memcached

# DB node
db:
  hostname: db
  image: blinkreaction/mysql:5.5
  ports:
    - "3306:3306"
    # Use IP based mapping when working with multiple projects. See docs (^^) for more details.
    #- <IP_ADDRESS>:3306:3306
  # DB configuration
  environment:
    - MYSQL_ROOT_PASSWORD=admin123
    - MYSQL_USER=drupal
    - MYSQL_PASSWORD=123
    - MYSQL_DATABASE=drupal
  volumes:
     # MySQL configuration overrides
     - "./.docker/etc/mysql/my.cnf:/etc/mysql/conf.d/z_my.cnf"
     # Permanent DB data storage
     - /var/lib/mysql

# CLI node
# Used for all console commands and tools.
cli:
  hostname: cli
  image: blinkreaction/drupal-cli
  volumes:
    # PHP configuration overrides
    - "./.docker/etc/php5/php-cli.ini:/etc/php5/cli/conf.d/z_php.ini"
    # Project root folder mapping
    - *project_root
    # Host SSH keys mapping. Uncomment one of the lines below based on your setup.
    - /.ssh:/.ssh   # boot2docker-vagrant
    #- ~/.ssh:/.ssh  # Linux
  links:
    - db
    - web
    # Uncomment this and the browser service definition below to start using selenium2.
    - browser

# Memcached node
# Uncomment the service definition section below and the link in the web service above to start using memcached.
# memcached:
#   hostname: memcached
#   image: memcached
#   environment:
#     # Memcached memory limit in megabytes
#     - MEMCACHED_MEMORY_LIMIT=128

# ngrok node
# Uncomment the service definition section below to start using ngrok for sharing your local web server with the world.
# share:
#   hostname: share
#   image: fnichol/ngrok
#   ports:
#     - "4040:4040"
#     # Use IP based mapping when working with multiple projects. See docs (^^) for more details.
#     #- <IP_ADDRESS>:4040:4040
#   links:
#     - web:http

# selenium2 node
# Uncomment the service definition section below and the link in the web service above to start using selenium2 driver for Behat tests requiring JS support.
browser:
  hostname: browser
  image: selenium/standalone-chrome
  links:
    - web
  ports:
    - "4444:4444"

Recommended fix:
Only install an appropriate (pegged) version of the custom boot2docker so that the vm installed is ensured to be compatible out of the box.
Note: From my other tinkering, it looks like the newest versions of the docker-composer.yml file in develop remove the hardcoding of the host port, but they require vhost_proxy to be set to true so the random ports can be identified. There are also other issues I'm hitting when using the develop branch, so  master should be fixed for now until the new stuff is resolved.

Fixed in v1.0.0-rc1


flake in IpVersionsAndStreaming/ShadowPolicyIntegrationTest.RequestMirrorPolicyWithShadowOnlyTimeout

normally I don't post from open PRs but I can't see how a file move is relevant to failure so I think this one's real
https://dev.azure.com/cncf/envoy/_build/results?buildId=133376&view=logs&j=767be981-567e-57d8-68c3-2140ede0a0bd&t=8a34f9ed-4946-5fcf-7cd6-99c93070884d
[2023-04-06 14:39:01.202][335900][critical][backtrace] [./source/server/backtrace.h:96] #10: Envoy::Http::Http2::ServerConnectionImpl::dispatch() [0x12ec0a4]
[2023-04-06 14:39:01.213][335900][critical][backtrace] [./source/server/backtrace.h:96] #11: Envoy::FakeHttpConnection::ReadFilter::onData() [0xab689c]
[2023-04-06 14:39:01.224][335900][critical][backtrace] [./source/server/backtrace.h:96] #12: Envoy::Network::FilterManagerImpl::onContinueReading() [0x156cc90]
[2023-04-06 14:39:01.234][335900][critical][backtrace] [./source/server/backtrace.h:96] #13: Envoy::Network::ConnectionImpl::onReadReady() [0x1566114]
[2023-04-06 14:39:01.245][335900][critical][backtrace] [./source/server/backtrace.h:96] #14: Envoy::Network::ConnectionImpl::onFileEvent() [0x1564210]
[2023-04-06 14:39:01.256][335900][critical][backtrace] [./source/server/backtrace.h:96] #15: std::1::function::func<>::operator()() [0x1552e5c]
[2023-04-06 14:39:01.266][335900][critical][backtrace] [./source/server/backtrace.h:96] #16: Envoy::Event::FileEventImpl::assignEvents()::$1::invoke() [0x1553f8c]
[2023-04-06 14:39:01.277][335900][critical][backtrace] [./source/server/backtrace.h:96] #17: event_process_active_single_queue [0x17e8f14]
[2023-04-06 14:39:01.287][335900][critical][backtrace] [./source/server/backtrace.h:96] #18: event_base_loop [0x17e77b4]
[2023-04-06 14:39:01.298][335900][critical][backtrace] [./source/server/backtrace.h:96] #19: Envoy::FakeUpstream::threadRoutine() [0xab370c]
[2023-04-06 14:39:01.309][335900][critical][backtrace] [./source/server/backtrace.h:96] #20: Envoy::Thread::ThreadImplPosix::ThreadImplPosix()::{lambda()#1}::invoke() [0x17e05e0]
[2023-04-06 14:39:01.309][335900][critical][backtrace] [./source/server/backtrace.h:96] #21: start_thread [0xffffac660624]
Http2::ConnectionImpl 0x490be4dd690, max_headers_kb: 60, max_headers_count: 100, per_stream_buffer_limit: 268435456, allow_metadata: 1, stream_error_on_invalid_http_messaging: 0, is_outbound_flood_monitored_control_frame: 0, dispatching: 1, raised_goaway: 0, pending_deferred_reset_streams.size(): 0
&protocol_constraints:
ProtocolConstraints 0x490be4dd700, outbound_frames: 0, max_outbound_frames_: 10000, outbound_control_frames_: 0, max_outbound_control_frames_: 1000, consecutive_inbound_frames_with_empty_payload_: 0, max_consecutive_inbound_frames_with_empty_payload_: 1, opened_streams_: 1, inbound_priority_frames_: 0, max_inbound_priority_frames_per_stream_: 100, inbound_window_update_frames_: 1, outbound_data_frames_: 0, max_inbound_window_update_frames_per_data_frame_sent_: 10
Number of active streams: 0, current_stream_id_: 1 Dumping current stream:

hitting this on the 1.26 release branch

This should have been fixed by #26605 -- is that PR included in the 1.26 branch? I don't know how to tell.

you can see on the commit that landed https://github.com/envoyproxy/envoy/commit/4b730e47029867d78bac6810a77c4ec83da85a3c
it shows the tags that include it, with the caveat that it doesnt include backports - in this case its there

@phlax what compilation tags should I be running with to get a repro? and what's the frequency you're seeing?

what compilation tags should I be running with to get a repro?

there are some docs here worth checking https://github.com/envoyproxy/envoy/blob/1b906296c2525e9898b9880f0b6be0ed52003b17/test/integration/README.md#deflaking-tests

what's the frequency you're seeing?

not high altho, i think there was at least one on postsubmit to day
https://dev.azure.com/cncf/envoy/_build/results?buildId=135895&view=logs&j=767be981-567e-57d8-68c3-2140ede0a0bd&t=67a76341-37f4-5b49-db38-acbb8a59af82&l=954

The failure you linked above is IpVersionsAndStreaming/ShadowPolicyIntegrationTest.Basic/IPv6_streaming_shadow, not RequestMirrorPolicyWithShadowOnlyTimeout. It ought to be fixed by https://github.com/envoyproxy/envoy/pull/27126.

thanks
ill test backporting it tomorrow
unfortunately i didnt post the link when i hit it on the 1.26 branch and i think it was in presubmit/pr so harder to find, so not sure exactly the error there

closing this - i have backported whatever would pick easily and im not seeing this issue on main or release branches


Add Tusk

Tusk is a open source unofficial evernote client
They have a deb available from their website.
https://github.com/klaussinani/tusk/releases/download/v0.23.0/tusk_0.23.0_amd64.deb

Sorry, we are only accepting software that is actively maintained/released. Please see:

Add guidelines for what packages are acceptable for inclusion


Branch free of tracking

Hi All,
I am wondering which of the branches is free of tracking components? I would like to build on top of that.
Thank you for help!

Hi. The npm package does not include any tracking, it's used just for the excalidraw.com app!

But whht if I need to edit the code itself?

No branch. The docker image is built uses a modified build. But now I'm not sure whether it was being stripped properly (I wasn't on the Docker team nor do I use it), or whether it regressed.
In any case, if you plan to fork the repo to make changes, and you still want to be deploying the excalidraw.com app wrapper as opposed to build your app from the package API, then I suggest you delete the tracking code in index.html, or use the REACT_APP_DISABLE_TRACKING=true ENV var when building which we're adding to be sure: https://github.com/excalidraw/excalidraw/pull/6059.
We're filtering out non-excalidraw hostname tracking data regardless, so not much would happen.


[# macro and doc improvements

Python slices! This one does an injection, but it should be fairly well contained most of the time and it's just too useful to omit. I avoided using it in the macro definitions, although it certainly would have simplified a number of places. I also added to the macro tutorial explaining how to implement this one.
I also reworked the code theme in the docs for better contrast. The default Monokai comments are a little to faint for how much I'm using them.
spy# and time# now pretty-print. This complicates the macros, but just a bit. Not sure if it's worth it yet.
Lots of other doc improvements.

Probably the pretty prints. Need to update some tests.


Windows build support self-contained build (single exe file)

Feature request:
flutter build for windows can build output a single exe file.
build all dll, into one big single exe file, let we can easy to redeploy our programm, copy then run.
much more like Mac .app file, copy, then we can run anywhere.

Duplicate of https://github.com/flutter/flutter/issues/75200

I am said some thing like dotnet self contained file. It's more like a box to map to real  executable file and it's dlls


Automated daily update to rustc (to nightly-2024-12-20)

Automated update to Github CI workflow rust.yml by create-pull-request GitHub action

Pull Request Test Coverage Report for Build<PHONE_NUMBER>3
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 83.037%

Totals

Change from base Build<PHONE_NUMBER>0:
0.0%

Covered Lines:
20422

Relevant Lines:
24594

💛  - Coveralls


upgrade: avoid upgrading unrequested casks

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew tests with your changes locally?

Fixes #8107.
In that issue, I noticed that running brew upgrade with named formulae will now try to upgrade all outdated casks. This is a regression from the old behaviour, where naming things on the commandline ensures that we only upgrade those requested items.
After some investigation, I determined this was introduced in 210d22e8199f2482b28f550838a33cca47380ce4. It turns out that upgrade_outdated_casks, if passed an empty array, will try to upgrade all outdated casks. That's the correct behaviour if no formulae are named either, but the wrong behaviour if at least one formula was named.
This PR fixes that by checking to see if any formulae were named; if at least one formula was named, but no casks were named, we skip the call to upgrade_outdated_casks.
cc @whoiswillma

Thank you for finding this issue,
I just tested it and the mirror of this issue, specifying a cask to upgrade but not specifying any formulae, results in unrequested formulae being upgraded as well.

Good catch! Updated to fix that too.

Sudo prompt saved me from accidentally bumping a whole lot of casks (most of which are intentionally kept on exact versions) 😰
Came here to create an issue but @mistydemeo saves the day with both reporting and fixing! it instantly 👍
Thank you, 100 times!

Thanks @mistydemeo!


[ORC-756] Check null readerschema prior to setting readerSchemaIsAcid && readerColumnOffset

-- Add a check for null readerSchema and using the fileschema before calculating readerSchemaIsAcid && readerColumnOffset
-- resolves "Include vector the wrong length" error for ACID files/tables that don't include OrcConf.MAPRED_INPUT_SCHEMA
What changes were proposed in this pull request?
Setting readerSchema to the fileSchema prior to setting final properties for readerschema.
Why are the changes needed?

Currently SchemaEvolution from a recordreader without OrcConf.MAPRED_INPUT_SCHEMA causes
"Include vector the wrong length" error

How was this patch tested?
Limited Testing on local workstation.

Hey @bilbingham  thats for reporting this! Can you please add a code sample to reproduce this?
It would also make sense to add a TestCase targeting this change but this can be the next step :)

To reproduce. (Sorry this is only a fragment) my test code is currently tied to a specific hdp cluster (hive and conf variables.,  I don't have a generic test case at the moment)
WIth a hive table. based on Hive APIV2 Streaming.  (https://cwiki.apache.org/confluence/display/Hive/Streaming+Data+Ingest+V2 )
CREATE TABLE acidorc ( col1 string, col2 string, col3 string, col4 string, col5 string )
PARTITIONED BY (part1 string)
STORED AS ORC
tblproperties( "transactional"="true", "orc.compress"="SNAPPY",  "orc.bloom.filter.columns"="col1,col2,col3");
Path p = "/path/to/acidorc/files";
OrcInputFormat oif = new OrcInputFormat();
JobConf jc = new JobConf();
// Added the schema to the conf resolves as well
//OrcConf.MAPRED_INPUT_SCHEMA.setString(jc,"struct<operation:int,originalTransaction:bigint,bucket:int,rowId:bigint,currentTransaction:bigint,row:structcol1:string,col2:string,col3:string,col4:string,col5:string>");
//Works fine until you try to INCLUDE_COLUMNS
OrcConf.INCLUDE_COLUMNS.setString(jc,"5");
Job theJob = new Job(jc);
org.apache.hadoop.mapreduce.lib.input.FileInputFormat.setInputPaths(theJob, p);
List<org.apache.hadoop.mapreduce.InputSplit> splits = oif.getSplits(theJob);
splits.forEach(split -> {
try {
org.apache.hadoop.mapreduce.RecordReader rr = oif.createRecordReader(split, tac);
rr.initialize(split,tac);
while (rr.nextKeyValue()) {
OutputStream outputStream = new ByteArrayOutputStream();
JsonWriter jw = new JsonWriter(new OutputStreamWriter(outputStream, "UTF-8"));
OrcStruct row = (OrcStruct)(((OrcStruct) rr.getCurrentValue()).getFieldValue(5));
jw.beginObject();
for (int i = 0; i < row.getNumFields(); i++) {
jw.name(row.getSchema().getFieldNames().get(i));
jw.value(String.valueOf(row.getFieldValue(i)));
}
jw.endObject();
jw.close();
}
} catch (Exception ex) {
System.out.println("Bummer " + ex.getMessage());
}
}

Hey @bilbingham I guess a MR test like this could be used to repro this -- even better if you have some ORC files already.

To reproduce
The Attached file was creating with the hivev2 streaming api into the following table (5000k random values)
CREATE TABLE acidorc ( i int, j int, k int)
STORED AS ORC
acid.orc.zip
tblproperties( "transactional"="true", "orc.compress"="SNAPPY", "orc.bloom.filter.columns"="i,j,k");
package orc.apache.orc.test;
import com.google.gson.stream.JsonWriter;
import org.apache.hadoop.fs.FileSystem;
import org.apache.hadoop.fs.Path;
import org.apache.hadoop.io.NullWritable;
import org.apache.hadoop.mapred.JobConf;
import org.apache.hadoop.mapreduce.RecordReader;
import org.apache.hadoop.mapreduce.TaskAttemptContext;
import org.apache.hadoop.mapreduce.TaskAttemptID;
import org.apache.hadoop.mapreduce.TaskType;
import org.apache.hadoop.mapreduce.lib.input.FileSplit;
import org.apache.hadoop.mapreduce.task.TaskAttemptContextImpl;
import org.apache.hive.streaming.StreamingException;
import org.apache.orc.OrcConf;
import org.apache.orc.mapred.OrcStruct;
import org.apache.orc.mapreduce.OrcInputFormat;

import java.io.*;

public class Break {
    public static void main(String[] args) throws StreamingException, IOException, InterruptedException {
//        CREATE TABLE acidorc ( i int, j int, k int)
//        STORED AS ORC
//        tblproperties( "transactional"="true", "orc.compress"="SNAPPY", "orc.bloom.filter.columns"="i,j,k");

        Path workDir = new Path(System.getProperty("test.tmp.dir", "target" + File.separator + "test" + File.separator + "tmp"));
        JobConf conf = new JobConf();
        FileSystem fs;
//        String typeStr = "struct<i:int,j:int,k:int>";
//        OrcConf.MAPRED_OUTPUT_SCHEMA.setString(conf, typeStr);
//        conf.set("mapreduce.output.fileoutputformat.outputdir", "workDir.toStworkDir.toString()ring()");
//        conf.setInt(OrcConf.ROW_INDEX_STRIDE.getAttribute(), 1000);
//        conf.setBoolean(OrcOutputFormat.SKIP_TEMP_DIRECTORY, true);
        TaskAttemptID id = new TaskAttemptID("jt", 0, TaskType.MAP, 0, 1);
        TaskAttemptContext attemptContext = new TaskAttemptContextImpl(conf, id);
//        OutputFormat<NullWritable, OrcStruct> outputFormat =
//                new OrcOutputFormat<OrcStruct>();
//        RecordWriter<NullWritable, OrcStruct> writer = outputFormat.getRecordWriter(attemptContext);
//
//        // write 4000 rows with the integer and the binary string
//        TypeDescription type = TypeDescription.fromString(typeStr);
//        OrcStruct row = (OrcStruct) OrcStruct.createValue(type);
//        NullWritable nada = NullWritable.get();
//        for(int r=0; r < 3000; ++r) {
//            row.setFieldValue(0, new IntWritable(r));
//            row.setFieldValue(1, new IntWritable(r * 2));
//            row.setFieldValue(2, new IntWritable(r * 3));
//            writer.write(nada, row);
//        }
//        writer.close(attemptContext);
        conf.set(OrcConf.INCLUDE_COLUMNS.getAttribute(), "5");
        FileSplit split = new FileSplit(new Path("/tmp", "acid.orc"),
                0, 1000000, new String[0]);
        RecordReader<NullWritable, OrcStruct> reader =
                new OrcInputFormat<OrcStruct>().createRecordReader(split,
                        attemptContext);
        // the sarg should cause it to skip over the rows except 1000 to 2000
        int count = 0;
        while (reader.nextKeyValue() && count < 5) {
            count++;
            OutputStream outputStream = new ByteArrayOutputStream();
            JsonWriter jw = new JsonWriter(new OutputStreamWriter(outputStream, "UTF-8"));
            OrcStruct row = (OrcStruct)reader.getCurrentValue();
            jw.beginObject();
            for (int i = 0; i < row.getNumFields(); i++) {
                jw.name(row.getSchema().getFieldNames().get(i));
                jw.value(String.valueOf(row.getFieldValue(i)));
            }
            jw.endObject();
            jw.close();
            String x = outputStream.toString();
            System.out.println(outputStream.toString());
        }
    }
}

Hey @bilbingham  thanks the the detailed example, this was super helpful.
Indeed seems that the MR Record reader can not process ACID files when schema is not set as part of the options.
Do believe we should default to fileSchema when its not set -- create a PR following that logic as part of #650
Please let me know what you think!

This was resolved by f3cd500bbe.


Crash when trying to capture a still image

When I run the following code:
CameraManager.sharedInstance.capturePictureWithCompletition({ (image) -> Void in
            self.imageView.hidden = false
            self.imageView.image = image
})

I get the following exception:
2014-12-02 21:27:39.343 Pixync[8928:3126855] *** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '*** -[AVCaptureStillImageOutput captureStillImageAsynchronouslyFromConnection:completionHandler:] - inactive/invalid connection passed.'
*** First throw call stack:
(0x187c7e59c 0x1983d00e4 0x186515140 0x100100360 0x1000ed4b8 0x1008b8e30 0x1008b8df0 0x1008c3854 0x1008bc120 0x1008c575c 0x1008c6f18 0x198bf12e4 0x198bf0fa8)
libc++abi.dylib: terminating with uncaught exception of type NSException

Thanks so much for submitting the issue and all the effort you put into solving the bug. I found the issue and fixed it, will release a new version any time soon.
Thanks again!


Add test for sharing uring across fork

Signed-off-by: Andres Freund<EMAIL_ADDRESS>

Thanks! This is great to have.


Add exercise on @while directive

Write up exercise for @while directive. See docs.

Done with 77dca3f.


Optional property for distribution: cnt:characterEncoding  wrong range

The controlled vocab is not defined yet.
If it is defined, the range should be changed to "skos:Concept(optional)"

the controlled vocabulary is missing.

Our  property "cnt:characterEncoding" is copied from the same property in geoDCAT-AP v2:
According to geoDCAT-AP v2, this property should be using...

...as value the character set names in the IANA register IANA-CHARSETS].

Status 15.08.24:
I executed the proposal above. Usage notes is corrected. Issue to be closed.


Add extra role binding to GCP projects

The team group should have the roles/cloudsupport.techSupportEditor role on the GCP project in addition to the role it currently has.
After this issue has been fixed, the following policies are enforced by Console:

roles/owner: group:<team group email>
roles/cloudsupport.techSupportEditor: group:<team group email>
<custom cnrm role name>: serviceAccount:<team cnrm service account email>

Closing for now.


Enhanced GitHub webhook handler tests.

Summary
This PR is only about adding test coverage to GitHub webhook handler.
Fixes #3358
Review Checklist:

[X] Reviewed my own code for quality and clarity.
[X] Added comments to complex or tricky code sections.
[ ] Updated any affected documentation.
[X] Included tests that validate the fix or feature.
[ ] Checked that related changes are merged.

coverage: 51.524% (+1.5%) from 49.995%
when pulling a65817404ee8e1b73b96bf93670ba1f2d241469c on issue-3358
into 12c5e04a8fb11346245f46bae375de54c2906e07 on main.


fix(fsapp): Basic React Native Navigation Web Wrapper

Not showing modals as modals (pushing the screen instead) and not running the slide menu, but most screen transitions should work.
I'm making this PR with this partial version so that anyone trying to run this branch can build navigation using the wrapper, which should be fully-functional for apps.

@Cauldrath - conflicts. Let me know when you're ready for this to go in so it gets merged before it gets out of date again.

This PR has gotten out of date enough that I'm just going to close it in favor of the one I've been updating: https://github.com/brandingbrand/flagship/pull/915


[Enhencement](Jdbc catalog) Add a parameter to jdbc catalog to allow only the specified database to be synchronized

Proposed changes
Issue Number: close #xxx
Problem summary
In the past, Jdbc Catalog will synchronize all database from source database.
Now we add a parameter called only_specified_database to jdbc catalog to allow only the specified database to be synchronized, eg:
create resource if not exists ${resource_name} properties(
    "type"="jdbc",
    "user"="root",
    "password"="123456",
    "jdbc_url" = "jdbc:mysql://<IP_ADDRESS>:${mysql_port}/doris_test?useSSL=false",
    "driver_url" = "https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/jdbc_driver/mysql-connector-java-8.0.25.jar",
    "driver_class" = "com.mysql.cj.jdbc.Driver",
    "only_specified_database" = "true"
);

if only_specified_database is true, jdbc catalog will only synchronize the database which is specified in jdbc_url.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

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

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 33.52 seconds
stream load tsv:          481 seconds loaded<PHONE_NUMBER>9 Bytes, about 148 MB/s
stream load json:         37 seconds loaded<PHONE_NUMBER> Bytes, about 60 MB/s
stream load orc:          67 seconds loaded<PHONE_NUMBER> Bytes, about 15 MB/s
stream load parquet:          27 seconds loaded 861443392 Bytes, about 30 MB/s
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230301153024_clickbench_pr_106608.html

run feut


[BUG] Error: Cannot find module 'chrome-aws-lambda'

{
    "errorType": "Runtime.ImportModuleError",
    "errorMessage": "Error: Cannot find module 'chrome-aws-lambda'\nRequire stack:\n- /var/task/index.js\n- /var/runtime/UserFunction.js\n- /var/runtime/index.js",
    "stack": [
        "Runtime.ImportModuleError: Error: Cannot find module 'chrome-aws-lambda'",
        "Require stack:",
        "- /var/task/index.js",
        "- /var/runtime/UserFunction.js",
        "- /var/runtime/index.js",
        "    at _loadUserApp (/var/runtime/UserFunction.js:100:13)",
        "    at Object.module.exports.load (/var/runtime/UserFunction.js:140:17)",
        "    at Object.<anonymous> (/var/runtime/index.js:43:30)",
        "    at Module._compile (internal/modules/cjs/loader.js:1015:30)",
        "    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1035:10)",
        "    at Module.load (internal/modules/cjs/loader.js:879:32)",
        "    at Function.Module._load (internal/modules/cjs/loader.js:724:14)",
        "    at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:60:12)",
        "    at internal/main/run_main_module.js:17:47"
    ]
}

Environment

chrome-aws-lambda Version: 5.5.0

puppeteer / puppeteer-core Version: 5.5.0

OS:

Lambda / GCF Runtime: nodejs12.x

I was using aws-chrome-lambda 3 with puppeteer-core 3.0.4 for a long time, now decided to upgrade to 5.5.0 but it looks like layer is seen.
In my js code im importing it this way:
const chromium = require('chrome-aws-lambda');
const puppeteer = require('puppeteer-core');

Are there any known issues with module visibility while upgrading?
Ive checked zip structure, looks good. Correct layer is used. Correct node version. Im out of ideas, to my knowledge it should work as before.

@pavelloz How did you solve this? I am getting the same error.

@pavelloz How did you solve this? I am getting the same error.

Yes, it turned out i checked the wrong zip when investigating. I went to s3, downloaded and it turned out that my serverless framework package had excluded layer files in the .zip file. So i rebuilt how i package layer zip and it works fine now.
Double check the final zip that you are uploading, it should have proper directory structure inside of it: nodejs/node_modules/x

Yes, it turned out i checked the wrong zip when investigating. I went to s3, downloaded and it turned out that my serverless framework package had excluded layer files in the .zip file. So i rebuilt how i package layer zip and it works fine now.
Double check the final zip that you are uploading, it should have proper directory structure inside of it: nodejs/node_modules/x

hey im trying to deploy my  app that is using puppeteer-core to vercel but the same probleme appearing can you help me


Peloton tweaks

Hey @jeremydk I was planning to release 1.1.0 and a new SD card image this weekend. Were you still thinking of opening a PR for the Peloton lag improvements? I can do if you don't have time and are ok with that, full credit to you of course, lmk!

Now that we've also fixed the cadence issue, I can rebase my pi tomorrow
and test how necessary the lag changes are. If they're required still, I
can get a Pull Request out by tomorrow afternoon for you to ship a 1.1.0 on
Sunday if that's cool with you.
On Fri, Oct 9, 2020 at 7:43 PM ptx2<EMAIL_ADDRESS>wrote:

Hey @jeremydk https://github.com/jeremydk I was planning to release
1.1.0 and a new SD card image this weekend. Were you still thinking of
opening a PR for the Peloton lag improvements? I can do if you don't have
time and are ok with that, full credit to you of course, lmk!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ptx2/gymnasticon/issues/19, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADAM6M3ES2JASMBSCFRTKHTSJ7C4ZANCNFSM4SKX2HOQ
.


Use JSON Lines

This is an experiment in using JSON Lines for our harvested JSON data instead of CSV.
This commit moves our sul_pub, dimensions and openalex harvesters over
to writing records as JSON Lines (jsonl). This means we will preserve
the dictionary and list data structures we received from the APIs and
will be able to use them for querying later, e.g. ORCID IDs embedded in
author objects.
Reading in JSON-L can be achieved with Pandas like:
pandas.read_json('data.jsonl', orient='records', lines=True)

and Polars:
polars.read_ndjson('data.jsonl')

or lazily:
polars.scan_ndjson('data.jsonl')

This was a useful experiment to see how the structure of the harvested data could be preserved. But I don't think it's strictly necessary. Perhaps it's something to keep in mind for the future though?


Remove jwt

I think JWT is way too compliated and specially for something as simple. Let's read and discuss other solutions:

https://news.ycombinator.com/item?id=14290114
https://news.ycombinator.com/item?id=11895440
https://www.wolfe.id.au/2012/10/20/what-is-hmac-authentication-and-why-is-it-useful/

We should look into it. I don't think jwt adds much complexity, but haven't really thought what advantage it has as opposed to another kind of token auth. The good thing of token auth in general is that it let's us generate a token at login and then use it to access the rest of the API without sending credentials again. So if we want to, for example, support both user/password auth and google login, then we only need to change the login handler, the rest will work the same.
I haven't really read a lot but I think HMAC doesn't work for our case, since it needs both sides to know the secret, in our case the password; that would mean the server would have to store the password instead of the password hash in the db. See here: http://blog.restcase.com/restful-api-authentication-basics/#hmac

The server can generate the digest as well, since it has all information.
Please note that the "password" is not encrypted on the server, as the server needs to know the actual value. This is why te name "secret" is preffered and not a "password".

Plus it has the same downside as basic auth, if we want to add a different auth method (like google login), we need to support both on every api endpoint.
I'd lean towards sticking with jwt unless we identify some disadvantage it has for our use case; on a quick review of the links I'm not sure most of them apply for us, I'll read with more detail during the week.


Add bigarray interface too

To work not only with strings and to avoid uneeded copy on conversion.

Related to https://github.com/xavierleroy/camlzip/pull/7


[chip-test, sival] chip_sw_rv_core_ibex_fault_dump

Test point name
chip_sw_rv_core_ibex_fault_dump
Host side component
None
OpenTitanTool infrastructure implemented
None
Silicon Validation (SiVal)
Yes
Emulation Targets

[ ] None
[X] CW310
[ ] Hyperdebug + CW310

Contact person
@GregAC
Checklist
Please fill out this checklist as items are completed. Link to PRs and issues as appropriate.

[ ] Check if existing test covers most or all of this testpoint (if so, either extend said test to cover all points, or skip the next 3 checkboxes)
[ ] Device-side (C) component developed
[ ] Bazel build rules developed
[ ] Host-side component developed
[ ] HJSON test plan updated with test name (so it shows up in the dashboard)
[ ] Test added to dvsim nightly regression (and passing at time of checking)
[ ] For SiVal test cases, test is running relevant FPGA or silicon regression

estimate 0.5

rstmgr_cpu_info_test is sufficient to test for this, but blocked on https://github.com/lowRISC/opentitan/issues/20030#issuecomment-1808126208


(SURE-5654)  CAPI cluster-agents do not tolerate the uninitialized taint

richard - I think we can remove this as its for a different team.

Issue was for Fleet and resolved as included in SURE-4552


Consistent Redstone Control

Fans, Vacuums, Crushers, Heaters, etc should all be disabled by a redstone signal.

@Divineaspect Could you provide a complete list of the machines that are not already redstone-disableable that should be?

Not sure how this should be implemented in a nice way that doesn't require adding a bunch of redundant code all over the codebase. Pushing back until a nice solution can be figured out by someone.


[DE DateTimeV2] The datetimeV2 prebuilt entity recognizes the utterance "Montag" as "diese Montag".

Is your feature request related to a problem? Please describe.
For example, if the utterance is "Wie ist das Wetter Montag/am Montag", the user means definitely next Monday but we get the Monday in the current week. This makes a problem when we are in a day after Monday.
Describe the solution you'd like
The solution can be to return both this Monday and next Monday. This happens already in the other case "am 23.7" when no year is specified. In this case, LUIS will return both the date of this year and next year (23.7.2019 and 23.7.2020).
Describe alternatives you've considered
Additional context

This is a bug. "Montag" should return both a past and future resolutions. As in English, for example.

Sorry. Actually this is already solved in the latest code.


Gh 668 named op

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@89975c6). Click here to learn what that means.

@@            Coverage Diff             @@
##             develop     #697   +/-   ##
==========================================
  Coverage           ?   60.66%
  Complexity         ?     2695
==========================================
  Files              ?      393
  Lines              ?    12080
  Branches           ?     1394
==========================================
  Hits               ?     7328
  Misses             ?     4170
  Partials           ?      582

Impacted Files
Coverage Δ
Complexity Δ

...va/uk/gov/gchq/gaffer/function/FilterFunction.java
100% <ø> (ø)
1 <ø> (?)

...n/java/uk/gov/gchq/gaffer/function/ArrayTuple.java
63.15% <ø> (ø)
5 <ø> (?)

.../commonutil/iterable/LimitedCloseableIterator.java
62.5% <ø> (ø)
9 <ø> (?)

.../gchq/gaffer/function/context/FunctionContext.java
80% <ø> (ø)
3 <ø> (?)

...chq/gaffer/operation/impl/get/GetRelatedEdges.java
0% <ø> (ø)
0 <ø> (?)

.../gov/gchq/gaffer/operation/OperationException.java
0% <ø> (ø)
0 <ø> (?)

...affer/operation/impl/generate/GenerateObjects.java
76.47% <ø> (ø)
11 <ø> (?)

...q/gaffer/operation/impl/export/FetchExporters.java
0% <ø> (ø)
0 <ø> (?)

...ation/handler/generate/GenerateObjectsHandler.java
100% <ø> (ø)
2 <ø> (?)

...n/java/uk/gov/gchq/gaffer/data/element/Entity.java
80.48% <ø> (ø)
15 <ø> (?)

... and 128 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 89975c6...5ec9e34. Read the comment docs.


new rule: import name must match imported module name

import foo = require('foo'); // OK
import bar = require('foo'); // Error

This requires a list of exceptions.

This was closed when we implemented import-name rule


Enhancement/version

Hi Pangeran, thank you most kindly for contributing to the project! Wonderful to see.
I like very much your idea of reducing the number of places where the version string is hard-coded - currently we just use the bumpversion program to maintain these strings. I'm not 100% sure whether the regular expression parsing of the version.py is the easiest approach though... wouldn't it be better to just import the version.py module?
Also, I'll have to go through and understand where all the other commits in this pull-request are coming from - I assume that you took the dev branch and are now merging against master? (that's completely fine, but we'll have to just go through all those changes of marko and I to make sure it's all ok)...

Hi Pangeran,
could you please just resubmit this PR, updating your local branch first?
I have since merged all the dev commits into master, so that if you do a PR this time for just the version change, it will be much cleaner.
Thank you very much.
Rob.


[stable/graylog] Chart does not allow volumeName to specified in PersistentVolumeClaims

The volumeClaimTemplates section in statefulset.yaml does not allow a volumeName to the added to the PVC spec.
This, of course, should not be necessary if you are using any storage provider and use a storageClass. However, small scale Kubernetes installations may still require manual PV provisioning. For such cases, the .spec.volumeName field is essential to make sure the correct volume is selected for a given PVC.
This issue requests that volumeName be added to values.yaml as an option to configure PersistentVolumeClaims.

After some time reading about how StatefulSets, PVCs and PVC Claims work, I'll admit there is no way to implement this only by only manipulating chart templates.
The solution proposed here does NOT work at all, because the the controller won't even consider including volumeName field in the PVC spec at runtime. The one declared in volumeClaimTemplates is simply ignored.
Besides, and more importantly, the k8s controller that uses the volumeClaimTemplates to create PVCs makes a number of assumptions to maintain the consistency it promises about StatefulSets.
The concern raised here is one that shouldn't affect many use cases and it's very particular to manual PV provisioning (which should be avoided anyway).
Thank you for the time taken to consider this.


Bootstrap 5 compatibility

Hello, please, can you make bootstrap 5 compatibility, or its WIP?

Duplicate of #2505. Please 👍 to vote and subscribe 🔉 for updates.


[Fab] Add Shadow Color Prop

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Summary 💡
Without monkeying with the theming or without overwriting the entire themed shadow, there isn't a direct way to modify the Fab component shadow. Implementing a shadowColor prop would be a beneficial API choice which would be useful with Dark Mode themes or other scenarios where there is a dark background to enable the "floating" effect.
Examples 🌈
No response
Motivation 🔦
Enhance design against dark backgrounds. Enable a consistent "floating" effect against dark backgrounds, even or especially in the case that the dark background exists isolated from a theme.
Beyond my use, exposing this as a prop could better enable other creative uses of the full color palette for Fab shadows.

If we do this, we should also add similar props to other components with shadows (as Paper).
Considering there is a way of controlling the shadow color (with theme override or by applying custom CSS), we'll unlikely allow this.
cc @mnajdova, @siriwatknp for thoughts.


[#30] Make contractID configurable

Closes #30

Update examples in plugin/nns/README.md

Update examples in plugin/nns/README.md


MLknn

Sorry, I do not have the time to make a pull but there is an "error" and I would like a function:
in mlknn compute_cond, mlknn is counting also the first entry of kneighbors, which is the instance itself, I would only count the neighborhood!
for instance in xrange(self.num_instances):
neighbors = self.knn.kneighbors(X[instance], self.k, return_distance=False)[0,1:]
for label in xrange(self.num_labels):
delta = sum(y[neighbor][label] for neighbor in neighbors)
also the rankings would be nice:
def predict_rankings(self, X):
result = np.zeros((len(X), self.num_labels), dtype='i8')
ranks = np.zeros((len(X), self.num_labels))
for instance in xrange(len(X)):
neighbors = self.knn.kneighbors(X[instance], self.k, return_distance=False)
for label in xrange(self.num_labels):
delta = sum(self.predictions[neighbor][label] for neighbor in neighbors[0])
p_true = self.prior_prob_true[label] * self.cond_prob_true[label][delta]
p_false = self.prior_prob_false[label] * self.cond_prob_false[label][delta]
prediction = (p_true >= p_false)
ranks[instance][label] = p_true/(p_true+p_false)
result[instance][label] = int(prediction)
return ranks,result
Cheers

I'll investigate this, I need to read up on mlknn, for now I've just converted the code to sparse matrices, maybe  @Antiavanti can comment on this? is it a mistake or did you make it so on purpose?
I'll implement predict rankings soon

So I've checked it some more, the fact that we're taking ourselves as neighbour in the fitting, doesn't change much for the fitting conditional probabilities, as it is stable over all instances, and deltas actually account for this additional 1 label always added. It's also a safeguard for the case if we actually have one sample with that one label combination. Do you think this argument makes sense?
Also for predict rankings, I'm still working on it, should be ready during the weekend :)

Hi,
I had some errors with the itself neighbor, that is why I came across but not sure exactly why (it was about 4-5 months back).
It could be that the implementation I used (in matlab) gave other resuls, but within your implementation it is fine.
The ranking issue is easy to fix as it is described in my post. The main point it to weight the p_true with the 1 divided by sum of p_true and p_false.
Cheers,
Fernando

Hi,
so I've done some tests and all, and it does turn out, that my @Antiavanti's implementation was indeed correct - I am getting better scores when using the begaviour with myself being a closest neighbour to myself. But for the sake of usefulness, I've added an extra parameter that allows to ignore as many first neighbours as possible.  I am also adding a predict_proba function that assigns probabilities to labels instead of ranking them, as this is the general spirit of the scikit API. Such a vector can be easily ranked.
Will commit the changes in a moment.


Error in automatic_control.py

Hi,
I got an error when try to run PythonAPI/examples/automatic_control.py. I am really newbie here but my guess is that this is a bug inside the library:
$ python automatic_control.py
pygame 1.9.6
Hello from the pygame community. https://www.pygame.org/contribute.html
INFO: listening to server <IP_ADDRESS>:2000
Example of automatic vehicle control from client side.
Spawning the player
Traceback (most recent call last):
  File "automatic_control.py", line 846, in <module>
    main()
  File "automatic_control.py", line 839, in main
    game_loop(args)
  File "automatic_control.py", line 709, in game_loop    agent.set_destination(agent.vehicle.get_location(), destination, clean=True)
  File "/home/shiba/work/carla/carla-dvs/PythonAPI/carla/agents/navigation/behavior_agent.py", line 119, in set_destination
    route_trace = self._trace_route(self.start_waypoint, self.end_waypoint)
  File "/home/shiba/work/carla/carla-dvs/PythonAPI/carla/agents/navigation/behavior_agent.py", line 159, in _trace_route
    end_waypoint.transform.location)
  File "/home/shiba/work/carla/carla-dvs/PythonAPI/carla/agents/navigation/global_route_planner.py", line 359, in trace_route
    if edge['type'].value != RoadOption.LANEFOLLOW.value and edge['type'].value != RoadOption.VOID.value:
AttributeError: 'int' object has no attribute 'value'

CARLA version: manual build at nearly the latest master (04b5e4fcd4e43bd96fd79b6d45173035ddef6b89 link: https://github.com/carla-simulator/carla/commit/04b5e4fcd4e43bd96fd79b6d45173035ddef6b89 )
OS: Ubuntu 16
python 2.7.12

Could anyone resolve it? Thank you!

Hi,
Is it a sporadic error, or something you've seen at every iteration? I'll give that a look after you provide me with this information!

Thanks, it occurs once, right after invoking the script - I launch the server, "Play" in UE4 Editor, and then run python automatic_control.py. I get this error every time I try to run it.

i tried to fix it by myself, but not sure what is the expected behavior for this script, automatic_control.py. Can you take a look on my PR? @jackbart94

@shiba24 The expected behavior is very similar to manual control, but now the vehicle drives autonomously.
For my part, just did a quick test with both python 3.6 and 2.7, and I saw no errors with the code, I'm able to run it without any problems.
However, with that said, the general approach when using RoadOptions in CARLA isn't to check them by value so, for consistency, I'd vote to change it as here is the only time that the values are used. This isn't the final answer though, I'll do more testing soon.

I saw no errors with the code,
Thank you for testing, @glopezdiest . Hmmm, interesting. Let me investigate too why only I got this.
About PR itself, I'll follow whatever your decision.