Custom Filter

Hello, I have a unique issue with dates and time zones. I have a UTC date in the index and the app allows you to apply any time zone so when I go to a search in the index I apply the time zone offset to match UTC but the issue is related to the daylight savings time since it depends on the date in the index.
It seems like I need to somehow take the date in the index and apply the timezone then do a search. I was thinking that maybe I could accomplish this with a custom filter by getting creating some kind of temporary "virtual" field that has the calculated DateTime with the correct timezone.
Basically, I am looking for a way to query on some field that has manipulated value.
Thanks in advanced

The switch to daylight saving time does not affect UTC.  So the UTC date /time you write into the index when under daylight savings time and when not under daylight savings time will be the same for a given time zone.
So your only issue is that when you perform a search you need to first convert the current date/time for that time zone into it's proper UTC date/time.   And that process will inherently take the daylight savings time (or not) into consideration.

Hello Ron, thanks for the quick reply, you are absolutely right! Except I was in a hurry when writing this and forgot the most important part 🤦 . It's not a DateTime but just a time range so I don't have a month to go off. The more I look into this the more I feel this can't be done since reading the value for each doc and converting it would be too expensive.

Time zone support has been added in #551 and #580 which has been included since 4.8.0-beta00016.
It isn't clear from your question whether you are using DateTools to make the conversions, but there are built-in options for storing dates in the index:

As a number representing milliseconds since the Unix Epoch (Milliseconds since Jan 1, 1970 12:00:00 AM UTC.).
As a number representing the number or .NET Ticks.
As a string in the format yyyyMMddHHmmssSSS.

DateToolsalso allows you to reduce the resolution of the date in certain cases, for example, when the seconds or time component is not important for the search.
As for the time zone, it only really matters if you store the date as a string because numeric dates do not include time zone info, they are always stored as UTC. When you search, your query needs to be built with UTC dates. This can be done using the classic or flexible query parser.
The most common way of dealing with time zones in .NET is to create DateTime instances using DateTimeKind.Local or DateTimeKind.Utc, both of which can be converted to the other. However, if you don't specify what the date is, it will be created with DateTimeKind.Unspecified and will assumed to be local time whether or not you intended that.
https://docs.microsoft.com/en-us/dotnet/standard/datetime/converting-between-time-zones
Using the built-in logic assumes that DateTimeKind.Local is the time zone you are converting to, and it should work for most scenarios (I suspect web apps adjust it automatically based on the user's time zone, although I haven't tested that scenario). However, both DateTools and flexible query parser support setting the current time zone explicitly for scenarios where your computer's local time zone is set to one time zone and you want to search using a different time zone.  Of course, using DateTools, the time zone info (and DST by extension) doesn't matter unless you are storing the dates in the index as a string.
If you store the dates in the index as numbers, the only place time zone matters is when you are building the query to search the index, in which case you will need to convert the date from local time zone to UTC before performing the search. And when you do that, .NET will take care of the DST part automatically as part of the time zone conversion from local time.

Hey guys thanks for the replies. @rclabo you are absolutely right that this isn't a Lucene problem but I'm trying to solve it with Lucene. Right now we use a SQLite DB and we have a custom SQL function that applies the given timezone and then does the query against that. The reason for this is that in the DB there is a full date and time but the query only has a time for example give me everything between 9-5. It's also important to note that in the application you have the ability to specify any timezone so it's not tied to a machine or anything like that. The timezone is easy because it's the same for the entire year but DST is not, so it depends on the month in the Lucene index if DST has an offset
Thanks for the help, I know this is not a normal situation lol

Closing, as this appears to be resolved.


⚠ Bofrost.es has degraded performance

In 7898e25, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1916 ms

Resolved: Bofrost.es performance has improved in a315e28.


Charge gas to unzip wasm code

Uncompressing packed wasm code should be paid in gas to compensate for CPU.
Early version for feedback. Left to do:
- Set proper cost per byte value
Not included is an ante handler to verify gas. But we don't do this for costs per bytes either.

The reasoning behind this one is super-solid.

I have applied the feedback and rebased the branch on top of main.


fairseq-generate RuntimeError: Mask Type should be defined

❓ Questions and Help
What is your question?
Upon trying to generate translations from a fairseq transformer model, I get the following error:
fairseq-generate fairseq-bin --path checkpoints/transformer/checkpoint_last.pt --beam 4 > gen.out

  File "/home/marco/anaconda3/envs/fairseq_test/lib/python3.10/site-packages/fairseq/models/transformer/transformer_encoder.py", line 165, in forward
    return self.forward_scriptable(
  File "/home/marco/anaconda3/envs/fairseq_test/lib/python3.10/site-packages/fairseq/models/transformer/transformer_encoder.py", line 294, in forward_scriptable
    lr = layer(x, encoder_padding_mask=encoder_padding_mask_out)
  File "/home/marco/anaconda3/envs/fairseq_test/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1190, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/marco/anaconda3/envs/fairseq_test/lib/python3.10/site-packages/fairseq/modules/transformer_layer.py", line 319, in forward
    output = torch._transformer_encoder_layer_fwd(
RuntimeError: Mask Type should be defined

I tried mostly to follow the examples in the fairseq repo, but I have my own dataset. The dataset and models were prepared as follows:
The dataset I am using was pretokenized and binarized with:
fairseq-preprocess --source-lang de \
--target-lang en \
--trainpref "train" \
--validpref "valid" \
--testpref "test" \
--destdir "fairseq-bin" \
--workers 60

Then a model was trained with:
CUDA_VISIBLE_DEVICES=0 fairseq-train     fairseq-bin     \
--arch transformer \
--optimizer adam \
--adam-betas '(0.9, 0.98)' \
--clip-norm 0.0 \
--lr 5e-4 \
--lr-scheduler inverse_sqrt \
--warmup-updates 4000 \
--dropout 0.1 \
--weight-decay 0.0001 \
--criterion label_smoothed_cross_entropy \
--label-smoothing 0.1 \
--max-tokens 4096 \
--max-epoch 5 \
--save-dir checkpoints/transformer

I could not find any other issues with this exact error, so I have no other information to go off of unfortunately. Any help would be appreciated.
What's your environment?

fairseq Version (e.g., 1.0 or main): 0.12.2
PyTorch Version (e.g., 1.0): 1.13.0
OS (e.g., Linux): Linux
How you installed fairseq (pip, source): pip
Build command you used (if compiling from source):
Python version: 3.10
CUDA/cuDNN version: 11.7

An update: it seems like fairseq-generate actually runs successfully, and that error is thrown somewhere in the middle of the dataset. Upon inspecting gen.out, all of the target side translations are empty, so it might be a preprocessing problem?
S-48923
T-48923	We should be aware of the dangers of the possible use of this clause as a means of discriminatory restriction .
H-48923	-1.3637781143188477	Mr President , Commissioner , ladies and gentlemen , I should like to congratulate the rapporteur on his report .
D-48923	-1.3637781143188477	Mr President , Commissioner , ladies and gentlemen , I should like to congratulate the rapporteur on his report .
P-48923	-4.9875 -0.1501 -0.1830 -4.1271 -0.1791 -1.0477 -0.1719 -0.1189 -0.2278 -2.0717 -2.8891 -0.4386 -0.1652 -3.7101 -1.7975 -1.3977 -1.1950 -1.1933 -1.3379 -1.1148 -0.1355
S-38554
T-38554	If this is the case , then I would very much like to know what the Council position is .
H-38554	-1.3637781143188477	Mr President , Commissioner , ladies and gentlemen , I should like to congratulate the rapporteur on his report .
D-38554	-1.3637781143188477	Mr President , Commissioner , ladies and gentlemen , I should like to congratulate the rapporteur on his report .
P-38554	-4.9875 -0.1501 -0.1830 -4.1271 -0.1791 -1.0477 -0.1719 -0.1189 -0.2278 -2.0717 -2.8891 -0.4386 -0.1652 -3.7101 -1.7975 -1.3977 -1.1950 -1.1933 -1.3379 -1.1148 -0.1355

I updated fairseq to the main branch (from v0.12.2) and that seems to have resolved the issue.


EyeOnWater data: list index out of range

I see many instances of next error in my log from integration:
Logger: custom_components.eyeonwater
Source: helpers/update_coordinator.py:313
Integration: EyeOnWater ([documentation](https://github.com/kdeyev/eyeonwater))
First occurred: 3:19:06 PM (21 occurrences)
Last logged: 8:19:27 PM

Unexpected error fetching EyeOnWater data: list index out of range
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 313, in _async_refresh
    self.data = await self._async_update_data()
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 269, in _async_update_data
    return await self.update_method()
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/eyeonwater/__init__.py", line 59, in async_update_data
    await eye_on_water_data.read_meters(days_to_load=3)
  File "/config/custom_components/eyeonwater/coordinator.py", line 43, in read_meters
    await meter.read_historical_data(
  File "/usr/local/lib/python3.12/site-packages/pyonwater/meter.py", line 78, in read_historical_data
    elif historical_data[-1].reading == self.last_historical_data[
         ~~~~~~~~~~~~~~~^^^^
IndexError: list index out of range

I don't see any usage data yet, but in my case meter have not reported data to the site for about a month. It just started reporting data again - but still no data in ha.

Thank you for reporting the issue.
I've release the blind fix in v2.5.2

Thank you, it works with the new version!


#2322 - IER12 E2E Tests Additional Scenarios

Created two more scenarios from the previous PR (#2461) to-do list.

Has full-time disbursement feedback errors (DISE).
Has a disbursement that withheld funding due to a restriction (DISW).

The first E2E above is invoking the processor without specifying the date as per previous PR comment.

New E2E factory created createDisbursementFeedbackError.

Added an exception in case getUploadedFile was invoked but it was never called. The tests were failing with a not-so-clear error.

Fixed an error in the IER12 query BETWEEN operator and removed a code TODO.

Great to see the level of validation for IER12. Approving with one very minor comment.
I believe there is going to be an upcoming PR for multiple institutions scenario right?

Yes, the idea is still the same: try to finish what we can inside the sprint. I will take this scenario as a priority for the upcoming PR 😉


Test the required option

Hiya,
All of the tests you added last night are awesome :) Thanks.
I just thought we should test 'required' since none of the other tests tried it. Hope that's ok. Short and small test but certainly a worthy one.
Andy

oh yeah, i left out required, error and default. thanks again.

Ah yeah, I didn't see those others too. Oopse. :)


Reorganize native hosting and service API

It is 50% design document, but tentative, also will be updated as it goes. Not a comprehensive list (at least yet).
Background
AAP had started with similar concept to desktop plugin APIs like LV2, which do not have cross-process client-service context. When implementing the framework, I used aap::PluginHost in plugins, not just hosts, because an AAP plugin is a service and needs hosting functionality. It then brought in confusion. It tries to support multiple instances from the same service, or client for different plugin services.
It's not just about those aap::PluginHost. For example, it affected how "known plugins" are managed there.
The hosting role and servicing role should be distinguished. And it would make sense to redesign all those related bits this time.
Problems
plugin management
In the first place, known plugins should not be stored in a global field as a fixed list. It makes sense when querying AudioPluginService packages takes significant amount of time, but so far it isn't.

AndroidPluginHostPAL::initializeKnownPlugins() is an entrypoint that sets plugin list cache.

It calls PluginHostPAL::setPluginListCache()
PluginHostPAL::getPluginListCache() retrieves the list. aap-juce is the only user so far.
PluginHostPAL::getInstalledPlugins() makes use of this list too.

PluginHostManager::updateKnownPlugins() sets the list of plugins, and getPluginInformation() (either by index or identifier) makes use of the list.

PluginHostPAL::getInstalledPlugins() mentioned above is used here.

AndroidPluginHostPAL holds a list of connections that contain AIBinder*, which means that the connections are statically global.

It should be also noted that PluginHostPAL::getInstalledPlugins() is implemented in somewhat complicated way, because it was meant to cover desktop hosting usages and was designed to work like JUCE AudioPluginFormatManager.
There is also no concept of plugin services apart from plugins.
instance management

There is no class for list of client connections that holds AIBinders in each element.

Surprisingly? they were stored globally and directly(!) at AndroidPluginHostPAL::serviceConnections.
And were directly added and/or removed in AudioPluginNatives_jni.cpp.

New design
non-public PAL
Any PAL layer e.g. aap::PluginHostPAL will not be public anymore. It was public due to aap-juce support requirements which was initially it was part of the same repo and split later (therefore confusion). We will dump getPluginHostPAL() global function. It is widely used so far, to retrieve plugin list, manage service connections, and create shared memory.
plugin information structures
There will be a new aap::PluginServiceInformation class, along with aap::PluginInformation and aap::PortInformation.
Later we will introduce aap::ParameterInformation (which is for another redesign issue).
Plugin list can be acquired by aap::PluginListSnapshot::queryServices(). We keep two-folds plugin query steps:

on Android, plugin services are queried list of aap::PluginServiceInformation (new), and then each aap_metadata.xml is parsed to turn it into list of aap::PluginInformation
on Desktop (not under actual development yet), they are queries per AAP_PATHs to query aap_metadata.xml, then follows the same steps as Android.

The list of aap::PluginServiceInformation should be retrieved on demand.
plugin instancing

aap::PluginInstance : one single instance of either a client instance or a service instance

aap::PluginServiceInstance
aap::PluginClientInstance
aap::PluginClientConnection : a new public API surface that represents a plugin client connection.

AIBinder* is stored as an opaque pointer.
It was aap::AudioPluginServiceConnection before (fun!)

aap::PluginClientConnectionList : connection list that should be managed per plugin client.

It cannot be instantiated without PAL aid
on Android, it is mapped to an org.androidaudioplugin.AudioPluginServiceConnector Kotlin object, and...
... it needs to be a live list and its aap::PluginClientConnection entry has to be assigned an AIBinder* at JNI layer.

aap::PluginService : a native service instance that manages multiple instances for a binder connection

aap::PluginClientConnectionList : a list that manages multiple instances for different binder connections.

aap::PluginGraph : represents an audio plugin graph. May be optional single-instance entity at first.

aap::PluginHost : organizes aap::PluginClientConnectionList and aap::PluginGraph to function as a host.

Feature wise, aap::PluginClientConnectionList corresponds to part of aap::PluginHostManager features as and some from AndroidPluginHostPAL. It is designed to hold multiple instances as a client, and used by AAPMidiProcessor and AudioPluginInterfaceImpl,  but never had complicated uses. They only deal with only one plugin.
Active Sensing
aap::PluginClientInstance::activate() starts active sensing, and aap::PluginClientInstance::deactivate() stops it. While processing audio, it is supposed to send active sensing messages. It should have no timestamp content as there may be time mismatch.
AAP  aap::PluginServiceInstance::updateActiveSensingStatus() is called by the native binder based service implementation, and updates latest_active_sensing_time field (time is managed only within the service process). When aap::PluginServiceInstance::process() runs, it checks the time field and if it was way old (which TBD, but it's 300 msec. in MIDI 1.0 Standard), it does not call the plugin's process() function.
Mapping

old API
new API

PluginHostPAL
PluginClientConnectionManager

getPluginHostPAL()
static function will not be provided anymore.

PluginHostPAL::getPluginPaths()
PluginClientConnectionManager::getPluginServices()

PluginHostPAL::getPluginListCache()
PluginClientConnectionManager::getPluginList()

PluginHostPAL::getInstalledPlugins()
PluginClientConnectionManager::getPluginList()

PluginHostManager
PluginListSnapshot

AudioPluginServiceConnection
PluginClientConnection

An emerging issue is that the reorganization did not weigh in how/where Binder instances are managed:

In the old implementation, AudioPluginHost instantiated ALL plugins on the device at startup time, which was no-brainer solution and very inefficient, but had no problem on retrieving them.
AudioPluginMidiDeviceService first acquired Binder implementation by running bindService() and then passed it to native code.
The new native PluginClient implementation begins with the state where no Binder is instantiated yet. It is possible to kick AudioPluginHostHelper.ensureBinderConnected() method, but needs some consistent connection up to aap::PluginClient.

In the current overhaul-native-api branch, aap::PluginClientConnection holds connection_data opaque pointer field which is AIBinder* on Android. When Kotlin AudioPluginServiceConnector.onBindAudioPluginService()method is called, viaAudioPluginNatives.addBinderForClient()`.
This Android client could just call AudioPluginHostHelper.ensureBinderConnected()  before trying to retrieve AIBinder*, but not sure if it should be done all in Android-specific code region (binder-client-as-plugin.cpp) or up to platform agnostic level (aap::PluginClientConnection, aap::PluginClientConnectionList, and aap::PluginClient).

A big bunch of updates are done as 542ee7 (merged overhaul-native-api branch), which covers a lot of the changes I described above.
A missing piece is active sensing, which I may create a separate issue.

^ is copied into the existing issue. Now we can close this issue.


[FEATURES] move dockerfiles to the same docker directory

What feature you'd like to add:
Currently, all the dockerfiles are in the root directory of the project. I think it's better to move them to the same directory.
Why is this feature needed:

LGTM


[misc] Center horizontal sliders horizontally in the question box

Makes 2 consecutive questions with displayMode: slider and min/max value labels of different lengths look better than the current left alignment.

https://user-images.githubusercontent.com/23243244/190437085-32b6aca6-eea2-41ef-97e2-880f06c9f2f9.mp4

Don't test in prems mode, it doesn't exist in this branch. That's why it is crashing.


.zgen at custom path and multi user

Despite source "/custom/.zgen/zgen.zsh" in .zshrc .zgen is still created at ~/.zgen.
Is it possible to completely move anything zgen related to the custom path? And would it be ok to use this custom .zgen path for multiple users?

Is there a zgen pendant to export ZPLUG_HOME=/custom/.zplug?
In zplug you can fix the issue with the above code.


Add Documentation for Dashboards de-couple build

When running yarn build --skip-os-packages it will build 4 tarballs
for Dashboards (2x linux, 1x macOS, 1x windows) and takes 10+min to
do so.
From the PR here, dashboards now support to build one specific platform. Details are:
yarn build-platform --darwin // build only darwin x64 platform
yarn build-platform --linux //build only linux x64 platform
yarn build-platform --linux-arm // build only linux arm64 platrform

Customer can also build multiple supported platforms. For example:
yarn build-platform --linux-arm --darwin // build both darwin x64 and linux arm64 platforms

Below is the updated DEVELOPER_GUIDE.md. Just provide more reference.
### Building the artifacts

To build the artifacts for all supported platforms,  run the following:

yarn build --skip-os-packages

If you want to build a specific platform, pass the platform flag after `yarn build-platform`. For example, to build darwin x64, run the following:

yarn build-platform --darwin

You could pass one or multiple flags. If you don't pass any flag, `yarn build-platform` will use your local environment. Currenly we only support `darwin` (darwin x64), `linux` (linux x64) and `linux-arm` (linux arm64).

Just want to check whether we need to update any document in opensearch.org.

Build information is typically in the repositories themselves -- we don't have any mention of it in the documentation. We cover installation, concepts, usage, public API, settings, etc. We might eventually include plugin development, too, but I doubt we'll ever want to decouple build instructions from the repos themselves.


attribution: disabled button

CR @piensaenpixel
easy one, needed for the feature overlay button

🚀 after CR


Strip query strings from imports when resolving

In some cases imports will contain query strings. The enhanced-resolver plugin for the "exports" field in package.json will fail to resolve these imports correctly. This change will strip any query strings from the import before resolving it.
The actual use case for this is that with Parcel we have created a set of plugins that will support Webpack's "magic comments" for naming async chunks. The way this plugin works is taking the chunk name and appending it to the import, e.g. import('@some/pkg?webpackChunkName=some-pkg' so it can later be accessed by a namer plugin.
RLL will fail on these imports in the rare case that the package contains an exports field, as the full string will be used to match so the query string will break things.
Added a new failing test for this - note as per the comment I was unable to get it working as a "fixture" because it doesn't appear fixtures are processed as modules, so the exports field wouldn't get used.

So it turns out that removing query params in just the babel-plugin causes issues when attempting to do manifest lookups as the query param will still be present inside the module graph.
I've now updated the PR to strip query params in all places we reference a specifier/request.


MINOR: Remove deprecated valueTransformer.punctuate

Also removed the InternalValueTransformerWithKey / Supplier which is used to mock away the deprecated punctuate function.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@mjsax this is ready for reviews now.


Unable to scale existing ACS-Engine cluster

Is this a request for help?:
Yes
Is this an ISSUE or FEATURE REQUEST? (choose one):
Issue
What version of acs-engine?:
v0.15.0-rc.1

Orchestrator and version (e.g. Kubernetes, DC/OS, Swarm)
Kubernetes 1.8.2
What happened:
acs-engine failed to scale the node count for my cluster.
What you expected to happen:
acs-engine to scale the node count for my cluster.
How to reproduce it (as minimally and precisely as possible):
☁  acs-engine [master] ⚡ ./bin/acs-engine scale --subscription-id <redacted> \
    --resource-group <redacted>  --location westus \
    --deployment-dir _output/dfs-kubernetes --new-node-count 3 \
    --node-pool windowspool2 --master-FQDN <redacted> \
    --auth-method client_secret --client-id <redacted> --client-secret <redacted>
INFO[0000] validating...
INFO[0003] Error returned by LoadContainerService: Unknown JSON tag nonMasqueradeCidr
FATA[0003] error parsing the api model: Unknown JSON tag nonMasqueradeCidr

Anything else we need to know:
Cluster was created with ACS-Engine v0.9. Current version of ACS-Engine being run is v0.15.0-rc.1

Hi @neurot1cal, please refer to #2060.

Thanks for the prompt response! My search-fu was not good enough. ;)

@CecileRobertMichon
INFO[0003] Error returned by LoadContainerService: Unknown JSON tag tillerMemoryRequests
FATA[0003] error parsing the api model: Unknown JSON tag tillerMemoryRequests```

Hmm yeah the apimodel definition for addons changed since v0.9.0 as well... You can look at https://github.com/Azure/acs-engine/blob/master/docs/clusterdefinition.md#addons and adapt your apimodel to follow the new definition. Or you can build a new cluster from scratch if you don't mind starting over or migrating workloads since your cluster is pretty old. Sorry you have to go through this there is a lot of work to be done to make upgrade backward compatible with old (< v0.12) acs-engine versions without requiring user manual intervention...

Thanks! This is luckily a POC cluster, testing some theories here. Is there a story or any documentation on migrating PVs between deployments for blue/green upgrades?
Would love to use this model, but we're struggling thinking about moving any persistent data. Would love to partner here to try and create/provide documentation and process if it doesn't exist already.

There isn't any official acs-engine documentation for migrating PVs that I'm aware of, I know people use different tools such as heptio ark but I think it would definitely be a great addition to have a more formal process in place. @jackfrancis might be able to comment on migrating PVs.

Thanks @CecileRobertMichon for mentioning heptio ark.


Add RequireComponent to CursorContextManipulationHandler

Overview
This will prevent the migration step from running, as of https://github.com/microsoft/MixedRealityToolkit-Unity/pull/7244.
This fixes the case where this script assumed the presence of ManipulationHandler. I chose this fix over EnsureComponent, because we have existing code that prevents migration based on RequireComponent.
Changes

Fixes: https://github.com/microsoft/MixedRealityToolkit-Unity/issues/7465

/azp run

/azp run

/azp run

I didn't know CursorContextManipulationHandler existed 😱  - will add follow up item to make sure we're handling this as well for ObjectManipulator


Error: .../infowindow.xml: Resource and asset merger: Layout file

Hi,
Thank you very much for your plugin.
I keep getting this error when trying to run.
[   +5 ms] ERROR: /.pub-cache/hosted/pub.dev/flutter_osm_plugin-0.70.2/android/src/main/res/layout/infowindow.xml: Resource and asset
merger: Layout file
/.pub-cache/hosted/pub.dev/flutter_osm_plugin-0.70.2/android/src/main/res/layout/infowindow.xml is not located in
[/.pub-cache/hosted/pub.dev/flutter_osm_plugin-0.70.2/android/src/debug/res,
/.pub-cache/hosted/pub.dev/flutter_osm_plugin-0.70.2/android/src/main/res]
[  +19 ms]     java.lang.IllegalStateException: Layout file
I have tried copying the file to those 2 destinations, obviously this did not work. I tried changing flutter versions, even going through the .xml file. I can't seem to figure out what the problem is...

add mutlidex to you app gradle

I have tried including enabling jetifier and using androidx but I still get the exact same error.

Perhaps related to my issue https://github.com/liodali/osm_flutter/issues/461?

which gradle version you're using did you try to upgrade it ?
i will remove that xml layout because it part for window popup for marker


怎么去链接自己的PYTHON3

ts@ts:~/opt/vision-master/build$ cmake -DCMAKE_PREFIX_PATH=/opt/libtorch-cxx11-abi-shared-with-deps-1.5.0/libtorch -DCMAKE_INSTALL_PREFIX=/where/to/install/torchvision -DCMAKE_BUILD_TYPE=Release ..
CMake Warning at CMakeLists.txt:13 (find_package):
By not providing "FindPython3.cmake" in CMAKE_MODULE_PATH this project has
asked CMake to find a package configuration file provided by "Python3", but
CMake did not find one.
Could not find a package configuration file provided by "Python3" with any
of the following names:
Python3Config.cmake
python3-config.cmake

Add the installation prefix of "Python3" to CMAKE_PREFIX_PATH or set
"Python3_DIR" to a directory containing one of the above files.  If
"Python3" provides a separate development package or SDK, be sure it has
been installed.
-- Caffe2: CUDA detected: 10.2
-- Caffe2: CUDA nvcc is: /usr/local/cuda/bin/nvcc
-- Caffe2: CUDA toolkit directory: /usr/local/cuda
-- Caffe2: Header version is: 10.2
-- Found cuDNN: v7.6.5  (include: /usr/local/cuda/include, library: /usr/local/cuda/lib64/libcudnn.so)
-- Autodetected CUDA architecture(s):  6.1
-- Added CUDA NVCC flags for: -gencode;arch=compute_61,code=sm_61
-- Configuring done
-- Generating done
-- Build files have been written to: /home/ts/opt/vision-master/build

Hi,
Please describe what you are trying to do and the problem you are facing. Given that there is no information in this issue that is actionable, I'm closing this issue.
For information, we recommend installing torchvision via conda / pypi using the information in https://pytorch.org/get-started/locally/


It should be impossible to inject invalid OWL into models

From https://github.com/geneontology/noctua-models/issues/18#issuecomment-248775569
It seems if the client tries to pass in something like
ClassAssertion("IDA", iri)

The OWLAPI unquestioningly accepts it, but when it saves it results in invalid OWL
Can we first have a unit test to test this hypothesis. If it turns out to be true we need to address. The best strategy is not clear and may be part of a general improvement in CURIE handling

Yes, you can create an IRI in OWL API like "IDA". It will then serialize it without complaint. But it will fail to parse it back in. I'm not sure where best to put a test for this in minerva. Going through JsonOrJsonpBatchHandler.m3Batch seems to check whether the class IRI is known (regardless of whether it is a valid form). So I think on that path the nefarious class would need to already exist in the tbox.
Could these possibly come in through the "Function companion" tool?

The IDs through most paths should be pretty solid. This seems to happen with a couple of widgets where the user just hits return (or tab) immediately without selecting anything.
While it would be nice if minerva would protect itself better in this case (not being able to re-read of disk is pretty bad), we could also implement checks in the client and force the UI to be more consistent in selection.

I think the FC tool should be safe. More likely the ability of curators
to paste in their own classes.
Requiring a class be in the tbox is too strict (we have that turned off,
as some curators need to paste CURIEs of uniprot classes not loaded).
But this would be the right place in the code to check.
On 23 Sep 2016, at 7:33, Jim Balhoff wrote:

Yes, you can create an IRI in OWL API like "IDA". It will then
serialize it without complaint. But it will fail to parse it back in.
I'm not sure where best to put a test for this in minerva. Going
through JsonOrJsonpBatchHandler.m3Batch seems to check whether the
class IRI is known (regardless of whether it is a valid form). So I
think on that path the nefarious class would need to already exist in
the tbox.
Could these possibly come in through the "Function companion" tool?
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/geneontology/minerva/issues/58#issuecomment-249209089

@cmungall I guess you mean pasting a class into the "Edit Instance" panel. I see that you can paste in any text there and it will be accepted as an IRI. In file mode the OWL API will save this. In Blazegraph mode the bad ID can be entered, but when you save some Rio errors will be thrown, e.g. "Subject URI was invalid". The file will not be saved, but nothing seems to be communicated to the front-end (the bad type will still appear until Minerva is restarted).

Aargh but the error is logged and swallowed in the OWL API so it is reported as a successful save. And if there are other changes it actually will be saved, partially...

IIRC, if you throw the error far enough, it should be picked up by the response handler and get the error back to the client.

Yes—in this case the error is only logged inside OWL API. It is swallowed before we can see it.

I think the right place to check incoming IRIs is at CurieHandler.getIRI. At a minimum we can prevent relative IRIs completely by making sure the text passed to IRI.create at least contains a colon (it should have been expanded to http://... by the curie map). To be even more strict we could verify that the result is a valid java.net.URL (it will throw an error for unknown protocols). However this would disallow valid identifiers that are not URLs like urn:uuid:12345. An in-between solution would be to have a small list of valid protocols like http, urn, mailto, and throw an error if the expanded IRI doesn't start with one of these. Would this be missing something?

I'm pretty sure http is only only protocol, but famous last words and
all that. I think extending to the protocols you mention should be safe.
General comment: is the CURIE handling worth spinning out into a
separate project
On 3 Oct 2016, at 11:59, Jim Balhoff wrote:

I think the right place to check incoming IRIs is at
CurieHandler.getIRI. At a minimum we can prevent relative IRIs
completely by making sure the text passed to IRI.create at least
contains a colon (it should have been expanded to http://... by the
curie map). To be even more strict we could verify that the result is
a valid java.net.URL (it will throw an error for unknown protocols).
However this would disallow valid identifiers that are not URLs like
urn:uuid:12345. An in-between solution would be to have a small list
of valid protocols like http, urn, mailto, and throw an error if the
expanded IRI doesn't start with one of these. Would this be missing
something?
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/geneontology/minerva/issues/58#issuecomment-251194286

@balhoff Okay, I'm now testing on minerva at, I believe, a corrected revision: https://github.com/geneontology/minerva/commit/cde48f8595ae06ee5f7fe111752c5d34d8a80f47
I can still add an item, say a gene product for an annoton, using the string "fooble". In the OWL, this comes up as:
Class: <fooble>

It then saves to disk just fine. A restart of minerva works, as well as viewing the model again.
Is this correct behavior?

@kltm I can't reproduce your test. Can you check on the latest version?

I apparently had versioning issues. I can no longer reproduce this when under adult supervision on the production server.


Requeue without error if handleJobWithNoWorkload finds a Workload

What type of PR is this?

/kind bug
What this PR does / why we need it:
It is possible when two reconciliations for the same Job happen in rapid succession for the earlier call of ensureOneWorkload to not find a Workload instance while the subsequent call to handleJobWithNoWorkload fails in Create with an already exists error. This PR special cases this error condition to requeue without an ERROR level log, as it is not actually an error.
We see error being logged during the processing of 5% of all created AppWrappers in our test suite.
As such, the error is frequently occurring  enough that we think it is worth surpassing the error-level log
message that the controller-runtime will generate if we simply return the error and let it requeue due to non-nil error.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

/ok-to-test

rebased.

/assign @tenzen-y @alculquicondor


[Feature] map_column used for hive

Signed-off-by: zombee0<EMAIL_ADDRESS>What type of PR is this：

[ ] bugfix
[x] feature
[ ] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

This PR support read hive maptype column as a column of result.
Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] I have added user document for my new feature or new function

Can't set status; build failed.

[FE PR Coverage Check]
:disappointed: fail : 23 / 32 (71.88%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/external/HiveMetaStoreTableUtils.java
3
10
30.00%
[161, 162, 165, 166, 167, 168, 169]

:large_blue_circle:
com/starrocks/external/hive/Utils.java
20
22
90.91%
[199, 203]

run starrocks_clang_tidy

Please send the design document, thanks.

Please send the design document, thanks.
this's a brief explanation, thank you.
Structure:

There are three Columns used to express MapColumn,

_offsets, store the start position of every map element. it's size is MapColumn's lenght plus 1, Offsets[i+1] - Offsets[i] is the MapColumn[i] (a map)'s size().
_keys, store the keys of every map.
_values, store the value of every key.

Example:
m1[k1:v1, k2:v2, k3:v3], m2[k4:v4, k5:v5, k6:v6]
The Map Column:
_offsets[0, 3, 6]
_keys[k1, k2, k3, k4, k5, k6]
_values[v1, v2, v3, v4, v5, v6]

run starrocks_fe_unittest

run starrocks_fe_unittest

[FE PR Coverage Check]
:heart_eyes: pass : 28 / 34 (82.35%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/external/HiveMetaStoreTableUtils.java
8
12
66.67%
[170, 176, 177, 241]

:large_blue_circle:
com/starrocks/external/hive/Utils.java
20
22
90.91%
[210, 214]

How do we query the map, plz give an example? I don't see any changes about paser and analyzer. and could you explain some of your core changes to  help others review?

How do we query the map, plz give an example? I don't see any changes about paser and analyzer. and could you explain some of your core changes to help others review?

thanks for your review, this is the first step to support hive's map type.
now it supports reading full map column from hive/orc,
and I had did some thing to support []operator of map type which will be pulled once this pr is merged.
I am also doing some thing to support map function such as map_size(), map_keys(), map_values().

[FE PR Coverage Check]
:heart_eyes: pass : 28 / 34 (82.35%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/external/HiveMetaStoreTableUtils.java
8
12
66.67%
[170, 176, 177, 241]

:large_blue_circle:
com/starrocks/external/hive/Utils.java
20
22
90.91%
[210, 214]

run starrocks_be_unittest

[FE PR Coverage Check]
:heart_eyes: pass : 28 / 34 (82.35%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/external/HiveMetaStoreTableUtils.java
8
12
66.67%
[170, 176, 177, 241]

:large_blue_circle:
com/starrocks/external/hive/Utils.java
20
22
90.91%
[210, 214]

[FE PR Coverage Check]
:heart_eyes: pass : 28 / 34 (82.35%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/external/HiveMetaStoreTableUtils.java
8
12
66.67%
[170, 176, 177, 241]

:large_blue_circle:
com/starrocks/external/hive/Utils.java
20
22
90.91%
[210, 214]

run starrocks_admit_test


Asio integration - completion token

@Naios
I would like to use continuable to build Asio composed operations as described https://think-async.com/Asio/boost_asio_1_30_2/doc/html/boost_asio/overview/composition/compose.html but instead of using async_compose() or co_composed() or deferred(), I would like to use this library as i think it provides a better abstraction.
Is this possible?
A few subtleties:

It needs to work with any Asio completion token (i.e, callbacks, futures, coroutines, deferred operations, etc)
It needs to serialize all the intermediate operations, in this case all the continuables, through the completion token's associated executor.

Commit Hash
23a724c

Expected Behavior
What I would like is a transformation algorithm cti::transforms::to_completion_token() which would allow the following to work:
#include <boost/asio.hpp>
#include <continuable/continuable.hpp>
#include <continuable/external/asio.hpp>

using namespace std::chrono_literals;

template <typename CompletionToken>
auto async_wait_twice(boost::asio::steady_timer& timer, CompletionToken&& token)
{
    return cti::make_ready_continuable()
        .then([&] {
            timer.expires_after(1s);
            return timer.async_wait(cti::use_continuable);
        })
        .then([&] {
            printf("First timer wait finished\n");
            timer.expires_after(1s);
            return timer.async_wait(cti::use_continuable);
        })
        .then([] {
            printf("Second timer wait finished\n");
            return cti::make_ready_continuable();
        })
        .apply(cti::transforms::to_completion_token(std::forward<CompletionToken>(token)));
}

int main()
{
    boost::asio::io_context ctx;
    boost::asio::steady_timer timer(ctx);
    auto guard = boost::asio::make_work_guard(ctx);
    std::thread t([&]{ctx.run();});

    async_wait_twice(timer, [] {
        printf("Two timers called\n");
    });

    ctx.stop();
    t.join();

    return 0;
}

Actual Behavior
N/A
Steps to Reproduce
N/A
Your Environment

OS: Ubuntu 22 but should be cross platform ideally.
Compiler and version: gcc 10

Hi,
I assume you want to attach a handler, not a completion token with an associated executor:
Then you can use something like the following:
template <typename Handler>
auto to_completion_token(Handler&& token) {
  return [token = std::forward<Handler>(token)](auto&& continuation) {
    auto executor = asio::get_associated_executor(token);

    return std::forward<decltype(continuation)>(continuation)
        .then(
            [token = std::move(token)]() mutable {
              std::move(token)();
            },
            [executor = std::move(executor)](auto&& work) mutable {
              asio::post(executor, std::forward<decltype(work)>(work));
            });
  };
}

with

template <typename Handler>
auto async_wait_twice(asio::steady_timer& timer, Handler&& token) {
  return cti::async([&] {
           timer.expires_after(1s);
           return timer.async_wait(cti::use_continuable);
         })
      .then([&] {
        printf("First timer wait finished\n");
        timer.expires_after(1s);
        return timer.async_wait(cti::use_continuable);
      })
      .then([] {
        printf("Second timer wait finished\n");
        return cti::make_ready_continuable();
      })
      .apply(to_completion_token(std::forward<Handler>(token)));
}

int main() {
  asio::io_context ctx;
  asio::steady_timer timer(ctx);
  auto guard = asio::make_work_guard(ctx);
  std::thread t([&] {
    ctx.run();
  });

  std::latch latch(1);

  async_wait_twice(timer, [&] {
    printf("Two timers called\n");
    latch.count_down();
  });

  latch.wait();
  ctx.stop();
  t.join();

  return 0;
}

That's a great answer thank you. But I was hopping to have an adaptor (or whatever we want to call it) which would allow me to use async_wait_twice() with any completion token like callbacks, asio::deferred, asio::use_awaitable, asio::experimental::use_promise etc.
But maybe that's too much work. Maybe the better approach is to juse use asio::deferred since it offers similar functionality to this library, i.e. continuations, and you can use it to build asio composed operations which then work with any completion token. It's just that i think continuable is better than asio::deferred

So the following is roughly equivalent using deferred and works with all Asio completion tokens:
template <typename CompletionToken>
auto async_wait_twice(boost::asio::steady_timer& timer, CompletionToken&& token)
{
    return deferred.values()
        | deferred([&] {
            timer.expires_after(1s);
            return timer.async_wait(deferred);
        })
        | deferred([&](std::error_code ec) {
            printf("First timer wait finished\n");
            timer.expires_after(1s);
            return timer.async_wait(deferred);
        })
        | deferred([&](std::error_code ec) {
            printf("Second timer wait finished\n");
            return deferred.values(ec);
        })
        | std::forward<CompletionToken>(token);
}

However i'm ignoring all the error codes. Each deferred handler has to manually handle ec whereas I think continuable manages the error path automatically??


Improve the documentation

This pull request combined with https://github.com/home-assistant/hassos/pull/86 results in two wired network configurations on the documentation page, without a clear explanation on why they are different (they look the same apart from id and uuid) or what they're supposed to do.


Create mailchimp for updates

For general announcements and one-to-many communications
General discussions will occur on the OWASP Slack

Hi @SebaDele can you resend the invitation?

This is done


Replace um_env.py with a yaml file

During the setup stage, um.py currently sets UM environment variables by sourcing the um_env.py file:
https://github.com/payu-org/payu/blob/8a4a1e2e9171f01e37bd34bfd8bb6f209f550873/payu/models/um.py#L115-L125
Arbitrary code could be included in um_env.py and could lead to security issues. Since um_env.py is only used for setting variable values, it might be best to replace it with a yaml file containing the same information.

I'll try to also clarify which of the variables in um_env.py are actually being used, e.g it currently contains:
vars = {
'AINITIAL': '',
'ASTART': 'INPUT/pre-industrial.astart',
...

while 'INPUT/pre-industrial.astart' is not used and isn't included in the vk83 restarts.


Problem with new detections

I'm trying to detect the colorchecker in the image img-colorchecker.jpg and also in new images taken by me, but the model seems to fail in all of the cases. Does the data need some sort of preprocessing or am I supposed to make a specific preprocessing operation over the RAW format version of my pictures in order to obtain good results? It's strange also that using the code in the notebook is not possible to detect anything from img-colorchecker.jpg.
Thank you in advance

Hello,
I am not the maintainer of this repo, but I have installed and using the tools. Can you be more specific about your problem?


Adding an example to flesh this sentence out

The sentence talks about appending a container name to this command:
kubectl logs counter

I've added an example showing the use of an appended container name.
I've also tweaked this to use the -c flag, in keeping with other kubectl commands that require the -c flag. For kubectl logs, it works either way.

Updating in a new PR.


Embedded player fails to properly load video

OS: MacOS 11.4
Browser: Firefox 89.0.2 (64-bit)
Issue: When attempting to open a channel, the embedded video player displays the error:
Whoops! Video playback error.
(Developers: Please specify valid set of arguments)
Found the following errors in the developer console:
11:15:12 AM [ERROR] [auth] Login nonce mismatch. Error: Login nonce mismatch.
i https://static.twitchcdn.net/assets/core-68406e6b87ebcb429524.js:1
v https://static.twitchcdn.net/assets/core-68406e6b87ebcb429524.js:1
v https://static.twitchcdn.net/assets/core-68406e6b87ebcb429524.js:1
k https://static.twitchcdn.net/assets/core-68406e6b87ebcb429524.js:1
209953 https://static.twitchcdn.net/assets/core-68406e6b87ebcb429524.js:1
i https://www.twitch.tv/passport-callback#access_token=7ftausd6ex424h5t6nbhk2w0mijzyc&scope=chat_login+user_read+user_subscriptions+user_presence_friends_read&state={"origin":"full_page","next":"https://www.twitch.tv/embed/Pestily/chat?parent=multi-mon.netlify.app&parent=localhost","nonce":"f3b0419531355027fdc68183d07e20fc"}&token_type=bearer:1
t https://static.twitchcdn.net/assets/core-68406e6b87ebcb429524.js:1
 https://static.twitchcdn.net/assets/core-68406e6b87ebcb429524.js:1
O https://www.twitch.tv/passport-callback#access_token=7ftausd6ex424h5t6nbhk2w0mijzyc&scope=chat_login+user_read+user_subscriptions+user_presence_friends_read&state={"origin":"full_page","next":"https://www.twitch.tv/embed/Pestily/chat?parent=multi-mon.netlify.app&parent=localhost","nonce":"f3b0419531355027fdc68183d07e20fc"}&token_type=bearer:1
 https://static.twitchcdn.net/assets/core-68406e6b87ebcb429524.js:1
a https://www.twitch.tv/passport-callback#access_token=7ftausd6ex424h5t6nbhk2w0mijzyc&scope=chat_login+user_read+user_subscriptions+user_presence_friends_read&state={"origin":"full_page","next":"https://www.twitch.tv/embed/Pestily/chat?parent=multi-mon.netlify.app&parent=localhost","nonce":"f3b0419531355027fdc68183d07e20fc"}&token_type=bearer:1
 https://static.twitchcdn.net/assets/core-68406e6b87ebcb429524.js:1
core-68406e6b87ebcb429524.js:1:1174942
Failed to execute ‘postMessage’ on ‘DOMWindow’: The target origin provided (‘https://www.twitch.tv’) does not match the recipient window’s origin (‘https://multi-mon.netlify.app’). passport-callback
Cross-Origin Request Blocked: The Same Origin Policy disallows reading the remote resource at https://sentry.io/api/1517394/store/?sentry_key=26325509987e418db03e96d12352a731&sentry_version=7. (Reason: CORS request did not succeed).
Failed to execute ‘postMessage’ on ‘DOMWindow’: The target origin provided (‘https://www.twitch.tv’) does not match the recipient window’s origin (‘https://multi-mon.netlify.app’).

Refreshing seemed to resolve the issue

I am unable to replicate this issue, can you provide some more details on what you did to receive this error?

I had authenticated to twitch, it redirected me back to the landing page with the "Twitch" button still being shown. Pressed it again with no results. Tried once more, after a few moments it redirected me to the channel page. I then clicked on a channel and the player appeared with the error.
I updated my comment to include the browser (guess a standalone version isn't very useful lol).


step_9_admins

DOT:
-Create AdminCommand annotation
-Add annotation to /stat command
-Write variables in the property, the value of which will be the nicknames of the admins
-Make a check when pressing /stat, which would check if a user with such a nickname has access to the command or not
DOD:
To make the /stat command only for admins and so that a regular user could not use it

https://github.com/goatnaaz/Basketball-telegram-bot/pull/20/


Ability to specify custom LESS directory

The issue is that as it stands, the addon assumes that you will have all your styles inside of /bower_components/bootstrap/less. This is not great if you want to have your custom less files not be rewritten when updating or deploying.
It would be very useful to be able to pass a path as an option like:
'ember-bootstrap': {
    'importBootstrapCSS': false,
    'additionalPath': '/vendor/less'
}

A am not sure I understand you well.. I guess your are referring to the newly added feature #80 ?
But this only is relevant if you do want to import Bootstrap's original less files. If you do not want to import them, but use some custom files, just put them into your app/styles folder and import them from there. Are do I miss something?

So, my end goal is to import the bootstrap original less files and add my own customizations on top, imported from a folder that is not bower_components, because this folder will be rewritten when doing a deploy.
However, when I try to put my less files into app/styles I get:
The Broccoli Plugin: [LessCompiler] failed with:
undefined

ember-cli-build.js
'ember-bootstrap': {
   'importBootstrapFont': false,
   'importBootstrapCSS': false
}

app.less
@import "ember-bootstrap/bootstrap";
@import "ember-bootstrap/theme";
@import "custom/custom.less";

Ok! I figured it out. It took me some time to parse the actual error, which was indicated just the line above  what the The Broccoli Plugin. Turns out I had the wrong reference inside of my imported less files, and the path was completely fine.
Sorry for the unnecessary issue, it took me a while to wrap my head around the Broccoli paths.


[v2.2] - Turbo module (iOS & Android)

[x] Compatible with RN 0.7+
[x] Compatible with iOS & Android
[x] Limit & offset to improve speed
[x] Removed blur and other properties
[x] NO more event handlers
[x] Permission need to be handled by your own
[x] getAll, getAlbums, searchSongs method works for iOs & Android

@Drazail could you help to test? thanks
Tested on

[x] Real iPhone 13 iOs 16.6.1
[x] Android 11 & 13

How to install
https://stackoverflow.com/questions/33181297/npm-install-from-github-pull-request

@flarocca @Haggaish @anthonytietjen @krishcdbry when you have time, please help testing :)

On Android 14, even after granting READ_MEDIA_AUDIO permission, the getAll function returns an error "Permission denied". I can see the permission being allowed in the settings of the phone. Might want to check this out.

On Android 14, even after granting READ_MEDIA_AUDIO permission, the getAll function returns an error "Permission denied". I can see the permission being allowed in the settings of the phone. Might want to check this out.

hi @adey69, have you tried requesting READ_EXTERNAL_STORAGE instead?

On Android 14, even after granting READ_MEDIA_AUDIO permission, the getAll function returns an error "Permission denied". I can see the permission being allowed in the settings of the phone. Might want to check this out.

hi @adey69, have you tried requesting READ_EXTERNAL_STORAGE instead?

Hi @dante-cervantes-rocketlab, I did try it first but then changed it to READ_MEDIA_AUDIO. Refer to the following:
https://developer.android.com/about/versions/13/behavior-changes-13#granular-media-permissions

On Android 14, even after granting READ_MEDIA_AUDIO permission, the getAll function returns an error "Permission denied". I can see the permission being allowed in the settings of the phone. Might want to check this out.

hi @adey69, have you tried requesting READ_EXTERNAL_STORAGE instead?

Hi @dante-cervantes-rocketlab, I did try it first but then changed it to READ_MEDIA_AUDIO. Refer to the following: https://developer.android.com/about/versions/13/behavior-changes-13#granular-media-permissions

Can you create a minimal reproduction of the issue and details of the device? last time i updated the code was tested on Android 13, probably i can test on Android 14 in a simulator ...

On Android 14, even after granting READ_MEDIA_AUDIO permission, the getAll function returns an error "Permission denied". I can see the permission being allowed in the settings of the phone. Might want to check this out.

hi @adey69, have you tried requesting READ_EXTERNAL_STORAGE instead?

Hi @dante-cervantes-rocketlab, I did try it first but then changed it to READ_MEDIA_AUDIO. Refer to the following: https://developer.android.com/about/versions/13/behavior-changes-13#granular-media-permissions

Can you create a minimal reproduction of the issue and details of the device? last time i updated the code was tested on Android 13, probably i can test on Android 14 in a simulator ...

Here you go for the minimal code. It's based on the example provided by the package owner.
Device information:
Samsung Galaxy S23 Ultra
OS: Android 14
https://github.com/adey69/music_player_test

hi @adey69 i have pushed an update in the master branch that fixes the issue for android 14, please take a look.

Hi @cinder92, working fine now. Thanks for the quick response.


Multiple Tags

Wow this is Gold! Thank you very much! I have one question: Is it possible to display multiple tags, like url.com/tag/tag-1+tag-2 or even better url.com/tagged/tag-1/tag-2?

@fschleifenbaum No it's not possible. And is not in the scope of this extension. The only thing the extension does is to allow SEF URLs for tag pages.


Add integration with Atomkraft for e2e testing

Summary
Add end to end testing support for provenanced that uses the formal Cosmos SDK specifications to ensure compatibility with the base modules.
Problem Definition
Ensuring the provenanced releases are stable and fully compatible with the Cosmos SDK is currently done by running simulation tests.  These tests are useful but not very comprehensive.  A new project from Informal System aims to more fully assert the proper behavior of the system using test specifications written in TLA+.
Proposal
Create a testing environment with Atom Kraft that runs E2E Cosmos SDK/IBC tests against provenanced.  Informal System also created several TLA+ specifications for the Marker and Metadata modules during their 2021 audit of Provenance blockchain code.  These specification would be useful to add to the others for the Cosmos SDK as a way to further assert the proper behavior of the system.

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

marker-tests-audit.zip
These are the generated tests from when Informal created TLA+ tests for the marker module...

There are also E2E tests for IBC-GO which are worth looking into. https://github.com/cosmos/ibc-go/tree/main/e2e


Blur previous focus while Yippy open

When you open Yippy the previous input remains focused. This sometime leads to me forgetting that Yippy is even open, and I end up typing something in there and then when Yippy steals the space key event I wonder what is going on for a second. It would be much smoother if the focus was completely taken by Yippy while it was open.
Presumably this would need to happen if search was implemented anyway.

Yeah this is another valid concern, but my first intention. I designed Yippy to initially fit my need of wanting a fast access stack of recent copies. Mainly for when I code so I move chunks around very quickly. E.g. very quickly opening the window and paste something recent straight in with very few key presses.
When I first built this I didn't have much experience with macOS development so this was the best solution I could find. However, since then I have seen it done in other ways and think it should be possible to get the solution we want:

Taking focus
Still able to auto paste for us
Search

But unfortunately this is fair chunk of work, and don't have time right now, but I will get to it! 😄

I like the feature of Yippy not taking focus! (this is also a feature of other clipboard managers such as Maccy and Flycut.
One small tweak to improve the UX of the unfocused window may be to dismiss it (same dismissal as done when esc is pressed) when something else is interacted with (eg. user clicks on the background or cmd-tabs).
This may make the unfocused behavior easier to live with, at least temporarily.
What are your thoughts @mattDavo ?

I like the feature of Yippy not taking focus! (this is also a feature of other clipboard managers such as Maccy and Flycut.
One small tweak to improve the UX of the unfocused window may be to dismiss it (same dismissal as done when esc is pressed) when something else is interacted with (eg. user clicks on the background or cmd-tabs).
This may make the unfocused behavior easier to live with, at least temporarily.
What are your thoughts @mattDavo ?

I like the feature of Yippy not taking focus! (this is also a feature of other clipboard managers such as Maccy and Flycut.
One small tweak to improve the UX of the unfocused window may be to dismiss it (same dismissal as done when esc is pressed) when something else is interacted with (eg. user clicks on the background or cmd-tabs).
This may make the unfocused behavior easier to live with, at least temporarily.
What are your thoughts @mattDavo ?

Yes that would be awesome! At the moment it is a bit obstrusive.


How to add scroll paging using ES？

How to add scroll paging using ES？

this would imply many things notably an api change for pagination tokens

I think for normal trace browsing, best to use the UI. If you need to scroll through many pages and hold your place between them, it might hint another tool right now. For example, we have a spark job to run through all data to make the dependency graph.
Are you interested in this for normal browsing? How many pages of traces do you think you'd need to see. I think some look at 100 traces (as opposed to 10 pages of 10), as long is the data is small you can consider just loading more.

We now have large amounts of data, millions, and we need a normal paging. That's why I consulted you. Just like using ES query, the'from'and'size' parameter is set.

yes, this is not going to work in the API. I think a user will be unlikely
to scroll through millions of records, but anyway will hold the issue open
for you.
Meanwhile, it is like better to focus on using Elasticsearch tools as if
you need to sift through millions and you cannot through the query we
provide, maybe you can with their tools.

OK, thanks again for your reply. Thank you very much.
We need to customize some functions. But now using the Zipkin you developed is very good.


Collapse short multi-line import groups to one line with I001

Ruff, like upstream isort, expands single-line import groups that are too long to multiple lines.
-from typing import Any, Callable, Dict, List, Mapping, Optional, Sequence, Tuple, TypeVar
+from typing import (
+    Any,
+    Callable,
+    Dict,
+    List,
+    Mapping,
+    Optional,
+    Sequence,
+    Tuple,
+    TypeVar,
+)

Upstream isort, but not Ruff, also does the reverse: it collapses multi-line import groups that are short enough to a single line. I think we should do this too.
-from typing import (
-    Any,
-    Callable,
-    Dict,
-    List,
-    Mapping,
-    Optional,
-    Sequence,
-    Tuple,
-)
+from typing import Any, Callable, Dict, List, Mapping, Optional, Sequence, Tuple

I believe we collapse such imports if you disable split-on-trailing-comma.
I've looked into this before, and isort's documentation states that this setting is enabled for profile = "black", but in practice it appears not to be.

It works for me.
$ cat > a.py
from typing import (
    Any,
    Callable,
)
$ isort --profile=black a.py
Fixing /tmp/a.py
$ cat a.py
from typing import Any, Callable

What I meant is that: the isort documentation suggests that it shouldn't be removing magic trailing commas there. The documentation says that split-on-trailing-comma: true is the default for that profile. But in practice it appears not to be.

To be clear: we support this behavior, it's just not our default.

Got it. Feel free to close if you think this should remain as is.

I'm torn on it but I'm going to leave as-is for now.


[NFC] asm igemm kernels integrate as tar.bz2, instead of plain text files

Currently asm igemm kernels are placed under src/kernels/dynamic_igemm/ as plain text files. And each kernel will have a single .s file. This will lead to MIOpen source file with hundreds ~  thousands of asm files, which may have following pitfalls:

github clone time will be longer.
When up load new PRs for new kernels, there will be hundreds of kernels (like #1035), will make review more difficult. Reviewers may be distracted by hundreds of asm files which may have little interests of, and hard to locate the only several cpp files need to review.
When have bugs or add new features that apply for all asm files and need re-generate from generator (like #1114, #552), the PR will still have lot of asm files.
When future hardware need add asm igemm kernels, above problem may become worse.

Here I propose a solution to this problem.
When put the assembly kernel into MIOpen, put all the kernel inside a folder and compress as *.tar.bz2 under src/kernels/dynamic_igemm/. Untar the kernel when build from source, before add_kernels() function like here:src/CMakeLists.txt.
All the kernels after decompress still can go into inlined_kernels/batch_*.cpp.hpp as inline data. the Hence we can still enjoy 1 kernel 1 asm file, but only group all the kernels into a tarball.
Runtime compilation/cache is not affected.
e.g. src/kernels/dynamic_igemm/igemm_gtc_xdlops_nhwc currently is 27M, 418 files. if we create bz2 of this(tar -cjf igemm_gtc_xdlops_nhwc.tar.bz2 igemm_gtc_xdlops_nhwc), it becauses 727K
pros:

solve above problem.
reduce source code file size

cons:

when build from source may need decompress stage, so build time may be longer.
when someone want to check the asm souce code, they need manually decompress the tar file.
PR review now unable to review inside the asm source code(because everything becomes a tar file.). We need to make sure this in the generator stage.

Need your comments/feedback on this proposal
@atamazov @junliume @JehandadKhan @shaojiewang @asroy

PR review unable to review inside the asm source code(because everything becomes a tar file.).

If so, we could upload a code object, so that decompression will not be needed.

PR review unable to review inside the asm source code(because everything becomes a tar file.).

If so, we could upload a code object, so that decompression will not be needed.

If upload a single code object, we may violate the purpose of splitting kernels (https://github.com/ROCmSoftwarePlatform/MIOpen/pull/957), that multiple kernel inside a single code object may slow down the wall time. Again, if we have a single code object per kernel, there seems no quite difference of current approach. we still has the problem as in the description.

Close this issue since already have a consensus. If still have problem will reopen this issue again


fix: closing h2 with h3

Hey!
Just noticed that the h2 tags in the page layouts are closed with h3 tags. Renders correctly anyway, but gets correctly flagged as an error by most inspection tools.
Best regards,
Kilian

Good pickup, thanks for the PR!


[MRTCore] Add global PrimaryLanguageOverride switch

Changing the application language by setting Windows.Globalization.ApplicationLanguages.PrimaryLanguageOverride to a specific language is not supported for WASDK unpackaged app, only for packaged ones. To support the same behavior for unpackaged applications similar switch is introduced into WASDK MRTCore module.
Sample app used to verify the change: https://github.com/stefansjfw/testwasdk/blob/master/testlocalwinappsdk/App.xaml.cs#L36
A microsoft employee must use /azp run to validate using the pipelines below.
WARNING:
Comments made by azure-pipelines bot maybe inaccurate.
Please see pipeline link to verify that the build is being ran.
For status checks on the main branch, please use TransportPackage-Foundation-PR
(https://microsoft.visualstudio.com/ProjectReunion/_build?definitionId=81063&_a=summary)
and run the build against your PR branch with the default parameters.

/azp run

while doubtful /azp run

/azp run

This is needed for PowerToys to fully enable language override

note: changing namespace to Microsoft.Windows.Globalization.ApplicationLanguages is in progress (as decided during API review)

note: changing namespace to Microsoft.Windows.Globalization.ApplicationLanguages is in progress (as decided during API review)

change pushed

/azp run

This has been merged as #4523


Compression utility should indicate status of compression in filename

Since it seems renaming it had no negative effects to the compression, it should instead be renamed by the compression utility to indicate status. I.E.: file.compressing[*---] and with progress file.compressing[**--]. It should also check what the current main section of the name is and use that when renaming, allowing for a user to rename the first part without losing it on the next name update.
The option of simply hiding it, violates the principle of doing something when a user does something to indicate action and makes canceling via deletion a power user task instead of a normal option.
The option of a pop-up has two downsides:

It could get lost
If it forced top level, it could get in the way of use on what could be a background task for the user

My personal preference for a filename based option is also so that I could potentially use this via cli instead of monitoring the file size changes.

Is it easy to write a program that changes the name with high frequency like this? If so that is an interesting idea.

The [] characters won't work on some file systems though, right?

The [] characters won't work on some file systems though, right?

hmm I was concerned for special characters in cli use, but I was not aware of filesystems that would lack support for them, do you have any examples?

As I mentioned earlier on reddit, it's rather more intuitive to show a progress bar on unfinished zip files (or half-copied files for that matter) to inform the users that the zipping/copying progress is not finished.
Possibly looks like this:

do you have any examples?

I just looked it up. Popular examples of "problematic" filesystems are FAT12/16/32 not supporting [] and NTFS not supporting *. Here is a more comprehensive list.

@rossbridger It's being considered, but it'd be hard to implement (https://youtu.be/ENBu9_xTrWU?t=877)

Oops, didn't mean to close

The square brackets will 1000% cause problems in any script where you so much as look in the directory where that file exists.

@tjkrobertson Good scripts escape/quote filenames (otherwise spaces would also break them)

Last I checked, ls doesn't return a list of quoted files. If you use that on a file with square brackets in the filename, POSIX-compliant shells will bug out because they're the symbol that denotes an if statement.
On Wed, Dec 8, 2021, at 10:45 AM, glibg10b wrote:

@tjkrobertson https://github.com/tjkrobertson Good scripts escape/quote filenames (otherwise spaces would also break them)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/glibg10b/ltt-linux-challenge-issues/issues/32#issuecomment-988929701, or unsubscribe https://github.com/notifications/unsubscribe-auth/AU6CYJCDBOAQ73EP2M5MNU3UP54P7ANCNFSM5JPG6GFQ.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

@tjkrobertson That's what wildcards are for, they automatically escape weird characters and spaces

Temp file names now have a .part suffix


can't install package

Hi,
I tried to install Xamarin.Forms.GoogleMaps - 3.0.3, but got errors:
Could not install package 'Xamarin.Forms.GoogleMaps 3.0.3'. You are trying to install this package into a project that targets '.NETPortable,Version=v4.5,Profile=Profile259', but the package does not contain any assembly references or content files that are compatible with that framework. For more information, contact the package author.
VERSIONS

Xamarin.Forms.GoogleMaps - 3.0.3
Xamarin.Forms - 3.1.0

PLATFORMS

[x]PCL
[x] Android
[x] iOS
[x] UWP

Any ideas?

Seems same as #555
It fixed in 3.0.4.


Contact

Hi, is it possible to contact with you? Maybe email? Or you can write me on<EMAIL_ADDRESS>

@sanchoflat, feel free to send me a direct message on Twitter.

@sanchoflat, now direct messages should work. Sorry, if you tried already, the Twitter 'error' banner is the same color (blue) as 'successful' when changing profile settings and I didn't notice that I put in the wrong password before.
Let me know if this doesn't work for you.


[Manual] - Comment that was added in offline disappears after pull to refresh operation

Commit number: c643e5d
Preconditions:

Test user is signed in;

Steps to reproduce:

Add a comment to a post in offline
Turn on network connection on device under test
Open the posts with added comments
Pull down the screen to refresh
Pull down the screen to refresh

Actual Result:
after step 4: The comment disappears after pull to refresh operation
after step 5: The comment appears after second pull to refresh operation
Expected Result:
after step 4: The comment is displayed for the post
after step 5: The comment is displayed for the post

@Vadim-Bulavin, please make sure that the deletion of the outgoing command is correct.


Azure Event Grid - Thumbnails costing hundreds of £££ on Azure

I set up the Azure Event Grid to create thumbnails and autodeployed the code from GitHub to create a function. I set it all up and it is working wonderfully.
However - after it being up for a month or so I took a look at our azure billing, and it's costing us a fortune! Hundreds of GBP in the 4 weeks we've been using it.
The function is constantly firing, with enormous ingress and egress of data to the storage account.
I've raised this with Azure, but I want others to be aware, what seems simple could end up costing a fortune. I'm not sure if it's just a mis-configuration on our part, but I would have hoped that the code should take account of that.

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

ID: cad5909e-e1c6-aa8e-82f1-f2851c1caced
Version Independent ID: 42292330-aa69-9df0-5bd7-e07c9daed995
Content: Use Azure Event Grid to automate resizing uploaded images
Content Source: articles/event-grid/resize-images-on-storage-blob-upload-event.md
Service: event-grid
GitHub Login: @ggailey777
Microsoft Alias: glenga

Hi @rossvernal
Thank you for sharing your valuable feedback! Please do let us know and update this thread once the root cause for the unexpected amount billed is identified.
We will now proceed to close this thread for now. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


csexec-cbmc: use goto-instrument for cmd line args

This is support for future feature in CBMC (add cmd line arguments).

I know about the tests, they just need refresh. I will do it ASAP!


Upgrade to react-native 0.17.0, babel-core 6.4.5 and jest-cli 0.8.2

Please review and merge

Looks great :) Could you update readme that it requires at least node 4.0?

Thanks!


Added onion rings

About the PR

Added onion rings
Why / Balance

Previously the onion rings sprite was unusued, now you can microwave an onion slice for 15 seconds to make onion rings
Technical details

Changed meal_recipes.yml and misc.yml
Media

https://github.com/space-wizards/space-station-14/assets/70238020/80042358-46d0-46ea-8db4-ffdb2cc7fbcc

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl:

add: Onion rings!

does this break other recipes using onion slices

does this break other recipes using onion slices

just tested the bowl of hot chili (20 sec) and clown tears (15 sec), they came out as usual


feat: signup functionality connected to backend server

Ive created a couple of extra tabs on the drop down menu.
the signup tab takes the user to the signup form and is now functional, once the users details have been submitted that user will be saved to the database .
the login tab still isnt currently functional as it took some time to connect the signup to the backend.
feel free to change the drop down menu layout just try not to alter the logic of the signup and login as it is pretty complicated and took some time to get right

I had copy your code and commit  them for you, you can cancel this.


PHP 7.3 cannot be built

 - system: `"aarch64-darwin"`
 - host os: `Darwin 22.3.0, macOS 13.2`
 - multi-user?: `yes`
 - sandbox: `no`
 - version: `nix-env (Nix) 2.12.0`
 - channels(root): `"nixpkgs"`
 - channels(shyim): `""`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixpkgs`

Generating phar.php
Generating phar.phar
PEAR package PHP_Archive not installed: generated phar will require PHP's phar extension be enabled.
make: *** [Makefile:395: ext/phar/phar.phar] Bus error: 10
make: *** Waiting for unfinished jobs....

See also https://github.com/php-build/php-build/issues/115

Interesting.
I won't be able to work on this since I don't own such architecture at home.

Didn't tested it yet on Linux 😅

What is the command that you're running to get this issue?

Actually, it seems that the issue isn't actually related to the php archive pear.
I think the original cause for the issue is the pcre2.
For php7.3, we should use the --with-pcre-regex flag for external pcre2, but the generic.nix (in the nixpkgs) only uses the --with-external-pcre flag.
So I have added configure flag for the php7.3.
If I use the following flake, the php was built successfully.
{
  inputs = {
    nixpkgs.url = "github:nixos/nixpkgs/nixpkgs-22.05-darwin";
    phps.url = "github:fossar/nix-phps/master";
  };

  outputs = { self, nixpkgs, phps }:
    let
      pkgs = nixpkgs.legacyPackages.aarch64-darwin;
      php-base = phps.packages.aarch64-darwin.php73.overrideAttrs(prev: prev // {
        configureFlags = prev.configureFlags ++ [ "--with-pcre-regex=${pkgs.pcre2.dev}" ];
      });
    in {
      devShells.aarch64-darwin.default = pkgs.mkShell {
        buildInputs = [ my-php ];
      };
    };
}

But unfortunately, the php-calendar wasn't built, as it had the error:
ld: symbol(s) not found for architecture arm64

The final error was related to this issue.
I could build the php 7.3 (on M1 Macbook Pro) with following flake.
{
  inputs = {
    nixpkgs.url = "github:nixos/nixpkgs/nixpkgs-22.05-darwin";
    phps.url = "github:fossar/nix-phps/master";
  };

  outputs = { self, nixpkgs, phps }:
    let
      pkgs = nixpkgs.legacyPackages.aarch64-darwin;

      # These attributes should be overriden
      php-base = phps.packages.aarch64-darwin.php73.overrideAttrs(prev: prev // {
        configureFlags = prev.configureFlags ++ [ "--with-pcre-regex=${pkgs.pcre2.dev}" ];
        patches = prev.patches ++ [ ./php7-libtool.patch ];
      });

      my-php = php-base.withExtensions(
        {enabled, all }: enabled ++ (with all; [ apcu ])
      );
    in {
      devShells.aarch64-darwin.default = pkgs.mkShell {
        buildInputs = [ my-php ];
      };
    };
}

You should add php7-libtool.patch like this:
diff -ru a/build/libtool.m4 b/build/libtool.m4
--- a/build/libtool.m4
+++ b/build/libtool.m4
@@ -340,7 +340,7 @@ AC_DEFUN([_LT_REQUIRED_DARWIN_CHECKS],[
      _lt_dar_allow_undefined='${wl}-undefined ${wl}dynamic_lookup' ;;
    10.[[012]]*)
      _lt_dar_allow_undefined='${wl}-flat_namespace ${wl}-undefined ${wl}suppress' ;;
-   10.*)
+   *)
      _lt_dar_allow_undefined='${wl}-undefined ${wl}dynamic_lookup' ;;
       esac
     ;;
@@ -5799,7 +5799,7 @@ _LT_EOF
              10.[[012]])
                _LT_AC_TAGVAR(allow_undefined_flag, $1)='${wl}-flat_namespace ${wl}-undefined ${wl}suppress'
                ;;
-             10.*)
+             *)
                _LT_AC_TAGVAR(allow_undefined_flag, $1)='${wl}-undefined ${wl}dynamic_lookup'
                ;;

PR is up, please let me know if it works.
To build locally, please clone this repo, and once you're in the directory, run nix build .#php73.


Validate: Warn if kube-proxy ipvs mode is detected

Fixes issue: https://github.com/submariner-io/submariner-operator/issues/1150
Signed-Off-by: Sridhar Gaddam<EMAIL_ADDRESS>

OVN job timed out during deployment stage itself.


Allow ledgers to be used in complex aggregate config stores

fixes #21277
In Istio 1.4.x, Pilot uses only one Config Store at a time, based on the helm variables supplied by users. In Istio 1.5.x, Istiod will use multiple Config Stores, presenting the necessity of merging ledgers across Config Store instances. This PR allows that merge, with the assumption that any data existing in the ledger before the merge may be safely discarded.
master fork of #21296

@therealmitchconnors: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-k8s-tests_istio
5f697c99e0160b0b5e55ceecf148b3f7f49ccdcf
link
/test integ-pilot-k8s-tests_istio

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

/test integ-pilot-k8s-tests_istio

@therealmitchconnors: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-conformance-k8s-tests_istio
5f697c99e0160b0b5e55ceecf148b3f7f49ccdcf
link
/test integ-conformance-k8s-tests_istio

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

/retest


N-32 dependencies of the Repo

Here we collect/discuss dependencies and track issues.
adding scule gave me a warning on dependency-issues
Should there anything be changed?

As long as it's a warning, it's fine. These issues will be addressed in Nuxt 4, and everything is currently functioning properly.


Feature: Google Sheets Geocoordinate Lookup Conversion

Brief: Small task, <1 hour.
Current State
The existing google sheets does not have a way to specify long/lat - it's more likely that they specify a city/state/location, which must be later converted into a longitude/latitude. Determine how to set this up
TODO

[x] Explore geocoordinate lookup APIs, what methods available for free
[x] Implement on Google Sheets

The Geocode object on App Script allows up to 1000 calls per day on a standard account. The implemented App Script can be found here:
https://script.google.com/u/2/home/projects/1jHI960r9bh2KCD1BMQLscMh4xT1Yx7jqpLVh2LckHGQ1HucSmmvh8XYj


Feature request: Add password protection to the url

With the automatic1111 web ui you have the option to use these parameters:
--listen --gradio-auth someUsername:somePassword
This broadcasts web ui in the lan and allows other computers to connect with it, but once they do, they have to input the correct username and password. However this extension broadcasts a url like: http://<IP_ADDRESS>:7860/infinite_image_browsing which does not use the gradio auth username and password. This is a security issue, as simply having the correct url means anyone can view (and delete) any and every file on your device. If someone knows you use stable diffusion, it's not a difficult thing to guess.
Please add an option to either:

Disable the ability to access this extension directly through a url
Add password protection to the url

Note this should also apply to every other api access as well, such as:
http://<IP_ADDRESS>:7860/infinite_image_browsing/file?filename=C%3A\sd-outputs\image.png

It seems to be an issue. I will try to add it later when I have time. If more people need it, I can implement it earlier.

Thank you for your feedback. I will try to add this feature at some point in the future. As the current usage is low and I haven't used the feature myself, the priority is not very high. If more users need this feature, I will implement it as soon as possible.

hello, first of all thank you for your hard work.. this extension really does awesome thing, because it helps browsing images so fast and stable.
but I agree with @YamiZee, sadly it is really security risk using this extension, in my opinion, it's not about high-low usage of this scenario.
It can become a huge problem, not only on internal LAN, but also when we open our webui to the big bad internet like cloudflare tunnel or gradio app, everyone can access all of our files in all drives, and possibly can delete it.
I just realize this, that everyone can access my files when adding :7860/infinite_image_browsing on my IP or website/tunnel, without any login prompt at all.
take your time, but I do hope it will be addressed sometime later. many thanks.

Thank you for your feedback. I will try to add this feature at some point in the future. As the current usage is low and I haven't used the feature myself, the priority is not very high. If more users need this feature, I will implement it as soon as possible.

hello, first of all thank you for your hard work.. this extension really does awesome thing, because it helps browsing images so fast and stable.
but I agree with @YamiZee, sadly it is really security risk using this extension, in my opinion, it's not about high-low usage of this scenario.
It can become a huge problem, not only on internal LAN, but also when we open our webui to the big bad internet like cloudflare tunnel or gradio app, everyone can access all of our files in all drives, and possibly can delete it.
I just realize this, that everyone can access my files when adding :7860/infinite_image_browsing on my IP or website/tunnel, without any login prompt at all.
take your time, but I do hope it will be addressed sometime later. many thanks.

I am aware that this feature is important. However, there are several major issues with the current extension, so I have not released an official version yet. This feature will be implemented at the same time as #116, which is also aimed at enhancing security. I will release an official version after completing these two features.
This extension is also quite strange to me. I don't know why there are so many feature requests, several of them are submitted on GitHub every day, it feels like there are more requests than most projects with around 1000 stars. Moreover, all the requests from users are useful, so I try to implement them all. Since this feature is more complex to implement than others and can be partially avoided, I will not be able to complete it quickly.
Currently, my priorities are:

A new caching mechanism that is more secure and does not require special settings
Adding support for custom folders in "quick move" and image search

If you are using a public network, I recommend not leaking your URL.

@YamiZee @thesomeotherguy
This feature has already been implemented.
see https://github.com/zanllp/sd-webui-infinite-image-browsing/issues/131


Is there a way to turn off errors.

Hi
I get the error: "[VueSlider error]: The "value" cannot be greater than the maximum."
because I allow the user to recalculate the min max after component mount.
It doesn't cause any problems
Is there a way to turn off errors.?

Can set silent to true.

I'm getting the exact same bug. And it clearly is a bug. At no time is the value higher than the max. What is causing this?

@catsclaw Is it because of this situation? #343

I don't think so. My logic is:
Calculate the new max, and the new value. Ensure they are correct.
If the current max is less than the new max, set the new max.
Set the new value.
Set the new max.
So at every step, the values should be correct. I've even tried setting the value to[0, 1] before setting the max and the new value.

@catsclaw  Can you reproduce it in the online example?
https://jsfiddle.net/NightCatSama/2xy72dod/10339/
If this does not affect the use, set silent to true.


Type mismatch in src/types/input/input_device.rs:28

The compilation of wlroots-rs master fails because of mismatched types:
Compiling wlroots v0.0.0 (file:///home/tallgren/git/way-cooler/wlroots-rs)
error[E0308]: mismatched types
--> src/types/input/input_device.rs:28:18
|
27 |     pub fn vendor(&self) -> c_int {
|                             ----- expected i32 because of return type
28 |         unsafe { (*self.device).vendor }
|                  ^^^^^^^^^^^^^^^^^^^^^ expected i32, found u32
In the file wlroots/include/wlr/types/wlr_input_device.h, device is defined as unsigned int, so the return type of vendor should be c_uint

This should be fixed now.


Enable context extension for Search operations - search support for common STAC clients

Many thanks to maintainers of this cool project.
I'm not sure if this this a feature request or bug report, nonetheless, this is what I have noted on /search functionality
I have tried catalog search using both sat-search and pystac-client but none seems to have native/compatible support.
While sat-search expects /search to return statistics of features matching query with no payload of feature items when limit is set to 0, stac-fastapi responds with 404  Error: Not Found
curl -X 'POST' \
  'http://<IP_ADDRESS>:9092/search' \
  -H 'accept: application/json' \
  -H 'Content-Type: application/json' \
  -d '{
    "collections": ["ls8_sr_c2_l2"],
    "bbox": [-180, -90, 180, 90],
    "datetime": "2021-01-01/2021-08-01",
    "limit": 0
}'

Response
{
  "detail": "No features found"
}

Without limit=0
{
  "type": "FeatureCollection",
  "links": [
    {
      "rel": "next",
      "type": "application/json",
      "method": "POST",
      "href": "http://<IP_ADDRESS>:9092/search",
      "body": {
        "collections": [
          "ls8_sr_c2_l2"
        ],
        "bbox": [
          -180,
          -90,
          180,
          90
        ],
        "datetime": "2021-01-01/2021-08-01",
        "token": "next:c51cb097-e074-5768-bdcc-f1eeb1b6bd31"
      }
    },
    {
      "rel": "root",
      "type": "application/json",
      "href": "http://<IP_ADDRESS>:9092/"
    },
    {
      "rel": "self",
      "type": "application/json",
      "href": "http://<IP_ADDRESS>:9092/search"
    }
  ],
  "features": [
    {
      "id": "d0261e13-6fed-5c75-9dc4-5878aceae1cc",
  .....
}]

Executing a similar request on https://earth-search.aws.element84.com/v0/search with limit=0 on sentinel-s2-l1c collection yields
{'type': 'FeatureCollection',
 'stac_version': '1.0.0-beta.2',
 'stac_extensions': [],
 'context': {'page': 1, 'limit': 0, 'matched': 1731850, 'returned': 0},
 'numberMatched': 1731850,
 'numberReturned': 0,
 'features': [],
 'links': [{'rel': 'next',
   'title': 'Next page of results',
   'method': 'POST',
   'href': 'https://earth-search.aws.element84.com/v0/search',
   'merge': False,
   'body': {'datetime': '2021-01-01/2021-08-01',
    'intersects': {'type': 'Polygon',
     'coordinates': [[[-180, -90],
       [180, -90],
       [180, 90],
       [-180, 90],
       [-180, -90]]]},
    'collections': ['sentinel-s2-l1c'],
    'page': 2,
    'limit': 0}}]}

In addition to this disparity,  result payload from stac-fastapi is missing  context and/or numberMatched key. This makes sat-search return 0 count for items matching search terms. Please see this fork for my temp fix on sat-search side.
The incompatibility is not a sat-search problem because pystac-client is failing on search with NotImplementedError

If help is needed, I can dirt hands on this.

I think it all comes down to the Context extension being enable or not on the API https://github.com/stac-utils/stac-fastapi/blob/master/stac_fastapi/extensions/stac_fastapi/extensions/core/context.py

Thanks for reverting Vincent.
Help me get your response clearly. Does it mean one (stac-fastapi user) has to implement the context extension

@jaysnm Exactly, you'll need to implement or Enable the Context extension for the DB backend you are using (PGSTAC or SQLachemy or ...).
FYI the context extension has been added to the SQLAlchemy backend but not in PGSTAC I think. if you are using SQLAlchemy backend you just need to add ContextExtension() to https://github.com/stac-utils/stac-fastapi/blob/master/stac_fastapi/sqlalchemy/stac_fastapi/sqlalchemy/app.py#L31

Thanks @vincentsarago
Surprisingly, it now works for sat-search but not pystac-client

@jaysnm both pystac-client and stac-fastapi are in development as the stac-api specification, all those 3 are moving at different speed so sadly I think errors like this are expected.

I admittedly have never tried calling stac-fastapi from any of the client side libraries, but I'd like to eventually get to a point where we are doing integration tests between stac-fastapi and other tooling libraries like pystac-client etc.
I will say that we are working on building a python client for stac-fastapi which will likely be pushed to this repo as a new submodule but that doesn't forgo the need for integrating with other client libraries.

Thanks a lot for heads up. I will the next couple of hours looking at pystac-client/stac-fastapi for a possible hack. If I'm lucky, I will share my working. Otherwise I will wait for "the submodule" as Jeff puts it.

Hi @vincentsarago and @geospatial-jeff
I have found out that the whole issue is on hard checks by pystac-client which is enforcing conformance to STAC Spec by looking at specific urls. From https://github.com/stac-utils/pystac-client/blob/main/pystac_client/client.py#L24-L32, the client is expecting that the STAC API root to have whatever links in conformsTo to consist any of https://api.stacspec.org/v1.0.0-beta.2/item-search or https://api.stacspec.org/v1.0.0-beta.1/item-search. Otherwise conformance check fails due to False result in https://github.com/stac-utils/pystac-client/blob/main/pystac_client/stac_api_io.py#L244 thus crashing the program as shown above.
How to get it working

Pass ignore_conformance=True when calling Client.open(). Caution: https://github.com/stac-utils/pystac-client/blob/main/pystac_client/client.py#L64
Stop advertising conformsTo completely https://github.com/stac-utils/pystac-client/blob/main/pystac_client/client.py#L65
Add https://api.stacspec.org/v1.0.0-beta.2/item-search into conformsTo list

There are some changes in https://github.com/stac-utils/stac-fastapi/pull/165 to return the proper conformance URLs, this should let the API pass the pystac-client conformance checks.  Fixing the conformance classes (ref https://github.com/stac-utils/stac-fastapi/issues/169) isn't relevant to the purpose of that PR (adding the filter extension) so maybe it's worth pulling that code into a different PR and getting it merged into master.

Thanks @geospatial-jeff
I have had a quick look at the PR #165 and #169 which seem to have cool fixes bringing stac-fastapi closer to full conformance with STAC Spec. While ContextExtension class exposes required context key on sqlalchemy side, it's not the case for pystac

I'll note that the context extension is not required by the STAC API spec, nor pystac_client in it's most recent version. pgstac doesn't support the context extension in the version currently used in the rep, but does enable it in the most recent releases.

Thank you all for great comments and kind responses. I'm closing this issue for obvious reasons, it is resolved by #211.  In case you are of a differing opinion, please feel free to re-open the issue or open a new ticket. Thank you all and sorry for the inconvenience.


SIGNATURE VERIFICATION FAILED!

By trying to install tor on my laptop with Lubuntu 18.04 system i repeatedly get this error message:
SIGNATURE VERIFICATION FAILED!
Error Code: GENERIC_VERIFY_FAIL
can someone help me to solve the problem?

This is a duplicate of #349


Unable to load identifications csv file unless it is created in Excel

Hello,
I am an odd issue loading a CSV file I created (using Python in Ubuntu) which contains identifications. As far as I can tell, the format matches the example file.
Here is the file:
EL4_4K_R1_250M_280214_1_PSMs_to_annotate.csv
The odd thing is, when I open the file in Excel (actually WPS, which is an Office clone) and save it without making any modifications, then it works. Here is the file after doing so:
EL4_4K_R1_250M_280214_1_PSMs_to_annotate.csv
Perhaps it is something with Windows-style line endings that are absent when I create the file, but get added by the spreadsheet software? That's all I can think of, since the files are slightly different sizes. It isn't really a big issue for me, but it is weird.
Best wishes,
Kevin

Hello again,
To follow up, I created my CSV file using \r\n as the line ending instead of \n and that fixed the problem, so the issue is in fact parsing files without Windows line endings.
I found the same issue occurs with the modifications file, and is also fixed by using \r\n as the line endings.
Best wishes,
Kevin


qml_video example did not stop after the window closed

when I close the aml_vedio example ,the application does not stop, it seems that the QMutexLocker is locked and waiting forever

Hi, thanks for contributing. Could you please share is it Android or Windows? What Qt version is used? Thanks

thanks! Qt 5.15.2, Windows 10, msvc64

I have figured out that in QAVDemuxer::unload(), we  need unlock before avformat_close_input(&d->ctx);, otherwise it will lock forever in decode_interrupt_cb

https://github.com/valbok/QtAVPlayer/pull/412


Simultaneous opening of modals causes unexpected freeze.

Platforms

Both
Versions

react-native-modal: 11.4.0
react-native: 0.61.2
react: 16.10.1

Description

Simultaneous opening of modals causes unexpected freeze of one of the modals.
Example:
Have 2 buttons, click them at the same time, one of the modals will freeze and can not be opened again.
I have tried using setTimeout and InteractionManager.runAfter.. but they dont seem to work.
Help will be appreciated.

https://github.com/react-native-community/react-native-modal#i-cant-show-multiple-modals-one-after-another

I've read the documentation, and am aware of this limitation.
Just so I understand, how can https://github.com/jacklam718/react-native-modals allow the opening of multiple modals and React Native Modal (singular) cannot?


typescript fetchpriority in image

Describe the bug
In a webpage sometimes you have a large contenful paint with images, when this happend you dont want to use loading lazy property in your image instead you want to use fetchpriority
Your Example Website or App
https://github.com/LuisCoderDev/solid_image
Steps to Reproduce the Bug or Issue
Run the proyect and see the error type in the img tag
Property 'fetchpriority' does not exist on type 'ImgHTMLAttributes'
Expected behavior
img should have the posibility to add fetchpriority='high'
i found more info here in the mdn docs maded by mozilla
Screenshots or Videos
No response
Platform

OS: Windows
Browser: Opera
Version: 1.7.3

Additional context
No response

I was going to open a PR but it seem it is already added:
https://github.com/ryansolid/dom-expressions/blob/1d04cc14bbde62e06d852eac9e85252359c6b304/packages/dom-expressions/src/jsx.d.ts#L860
By this PR https://github.com/ryansolid/dom-expressions/pull/237
But not merged because of this error: https://github.com/ryansolid/dom-expressions/actions/runs/4694305232/jobs/8322332238

any update ?