docker run --platform

introduce ability to select platform for image being ran

If you want this to work the same way the current --platform works, it should be using platforms.Only instead of platforms.OnlyStrict :eyes:
(They're very confusingly named, but the difference is that platforms.Only implies the full "vector" of supported images so that users requesting linux/arm/v7 can and will be able to run linux/arm/v6 images if v7 isn't available.)

An easy way to test that would be to use a value like linux/arm/v8 (of which I'm not aware of any images that exist today, so you should get the linux/arm/v7 images instead when you specify that).

Upstreamed by: https://github.com/moby/moby/pull/44958


Implement fancier test utilities and test fixtures

Create some common functionality for use by tests.

generation of raw sql (maybe to be replaced later by query builder as discussed in #48)
consider adding string formatting libraries to dev-dependencies like this or this for ease of generating nice statements
well-documented connection setup/teardown routines, for example setting up multiple connections/transactions or creating/dropping tables with unique names to avoid transaction deadlocks in parallel tests
move test database creation into separate scripts + config (docker/git action could still reference them) to make it easier for users to run tests locally with minimal dependencies,  and/or create some fixed .fdb files corresponding to different fb server configurations
other ideas?

I think there are two points more relevant here:

Generation of unique name tables/resources/transactions to avoid deadlocks

In some tests we already we a static variable to solve this problem. Maybe we can create a standard approach for this.

Environment reproduction.

Maybe we can use some docker images with fb server. We can build a script for run in our local environment or in our github actions, starting the fb servers containers in different ports. With different ports, we can use the same 'test multiplexing' approach, used today for run the all crate features, to run the tests in all this servers

@jairinhohw @fernandobatels now that I have to fix tests for my changes in #75 I'm noticing some of the ways in which our current testing setup is not so flexible/scalable.
I'd like to change it and would like your feedback.
The main improvements I had in mind:

make something more like our ci matrix for different configurations. Right now it's a bit like this, but it is only 1xn matrix ( the for XYZ -> BuilderType list) and there is no concept of include  where one can introduce extra config variables on a case-by-case basis

any suggestions on implementing this? I think I know how to do it with macros but it will be ugly. Maybe const/const fn could be helpful here?

factor out the cbuilder set-up into separate inline modules or macros
create an inline module somewhere whose only purpose is to re-export a common set of items to be included by all tests.
replace hard-coded dyn lib/database file paths to use environment variables, but with default settings (or maybe cargo has a specific feature for these kinds of variables)
move unit tests in src/**/xyz.rs into separate sub-modules like  // in src/statement.rs //
  ...
  #[cfg(test)]
  mod statement_rs_tests {
    use super::*; // to import declarations in statement module
    use crate::common_test_imports::*;
    ...
  }

Also: is there a reason we use src/tests/ instead of just tests/ as recommended  by the official cargo guide ?

* move unit tests in source files into test-specific sub-modules like

I'm ok with this change. What you think @jairinhohw ?

Also: is there a reason we use src/tests/ instead of just tests/ as recommended by the official cargo guide ?

Hmm, the src/tests/ represents more an approach to deal with the 'multi features tests' that a integration test.

create an inline module somewhere whose only purpose is to re-export a common set of items to be included by all tests.

What items are you thinking about?

make something more like our ci matrix for different configurations.

Do you are thinking about the firebird versions?

* make something more like our ci matrix for different configurations. Right now it's a bit like this, but it is only 1xn matrix ( the `for XYZ -> BuilderType` list) and there is no concept of `include`  where one can introduce extra config variables on a case-by-case basis

  * any suggestions on implementing this? I think I know how to do it with macros but it will be ugly. Maybe `const`/`const fn` could be helpful here?

I don't know exactly what kind of data you want to add, but you can use the env! macro to access environment variables at compile time.

* generally try to rely less on macros if possible

Macros are the only way i know of to create multiple versions of the same tests as is done (one test for every firebird client type).
Other than that it is good where possible to remove the macro usage.

* replace hard-coded dyn lib/database file paths by environment variables, but with default settings (or maybe cargo has a specific feature for these kinds of variables)

The hard coded settings are used only for tests, so i don't think it is a problem, however we can use the env! or option_env macro.

* move unit tests in source files into test-specific sub-modules like
  ```
  // in src/statement.rs //
  ...
  #[cfg(test)]
  mod statement_rs_tests {
    use super::*; // to import declarations in statement module
    use crate::common_test_imports::*;

    mk_tests_default!{
      ...
  }
  ```

I personally like to have some tests in the modules where the code the tests test lives, as the ones in the stmt_cache for example, but it is only a preference.


Unsubscribed action is not capture by webhook or Fetch All Subscription

Describe the bug
If customer decided to unsubscribe from saas offer. They can cancel it via Azure portal, however if customer do so, it will show they are unsubscribed but publisher site does not show the new status.
Also if publisher clicked Fetch ALL Subscriptions. It does not update the status.
However if you run postman you will see the status is unsubscribe
To Reproduce
Steps to reproduce the behavior:

Go to 'saas' subscription resources
Click on Unsubscribe and wait to confirm
Go to publisher site, check subscriptions, it will show subscribed
Click Fetch All Subscription
it will show subscribed

Expected behavior
Fetch All subscription to reflect the new status
Unsubscribe from Azure portal should show in publisher portal

Issue was wrongly filed


Support riptide 2

adapt to riptide 2 API
remove support for AsyncRestTemplate
make OAuth optional

I will remove the synchronous stuff again because one can easily build a RestTemplate from the AsyncClientRequestFactory as we have found out.

:+1:


Quantized model using opencv_zoo/quantize throws error in block_repear when src.dims() = 1 and axis = 1

System Information
Python 3.10.10
General configuration for 5.x =====================================
Platform:
Host: Linux (Ubuntu 22.04) x86_64
CMake: 3.22.1
Configuration: Debug Release
Detailed description
cv.dnn.readNetFromONNX() fails for quantized dexined onnx edge detection in opencv 5.0 with error:
net = cv.dnn.readNetFromONNX(args.model)
cv2.error: OpenCV(5.0.0-pre) opencv/modules/dnn/src/onnx/onnx_importer.cpp:1070: error: (-2:Unspecified error) in function 'handleNode'
> Node [DequantizeLinear@ai.onnx]:(onnx_node!up_block_6.features.6.weight_quantized_node) parse error: OpenCV(5.0.0-pre) /media/pincambv/hugedrive1/opencv5_gursimar/opencv/modules/dnn/src/int8layers/quantization_utils.cpp:45: error: (-2:Unspecified error) in function 'void cv::dnn::block_repeat(cv::InputArray, const MatShape&, int, int, cv::OutputArray)'
> > Axis out of range:
> >     'axis >= 0 && axis < src.dims()'
> > where
> >     'axis' is 1

The original model can be downloaded from:
dexined.onnx
Quantization command:
python --input_model=dexined.onnx --block_size=16 --output_model=dexined_compressed.onnx
Quantized model link: quantized_dexined.onnx
Steps to reproduce
Download the quantized model from: quantized_dexined.onnx
net = readNetFromONNX("quantized_dexined.onnx")

import cv2
net = cv2.dnn.readNetFromONNX("quantized_dexined.onnx")

Issue submission checklist

[X] I report the issue, it's not a question
[x] I checked the problem with documentation, FAQ, open issues, forum.opencv.org, Stack Overflow, etc and have not found any solution
[X] I updated to the latest OpenCV version and the issue is still there
[X] There is reproducer code and related data files (videos, images, onnx, etc)

@DaniAffCH Could you take a look at this problem? Looks like the problem is from the block quantization script.

Hi @gursimarsingh
I noticed you are using OpenCV 5.x so this issue is a duplicate of https://github.com/opencv/opencv/issues/25999
I'll open a PR containing a patch for this issue as well as a refactoring of the block quantization as soon as possible. In the meantime, you can check if using OpenCV 4,x the issue persists.

I am going to transfer this issue back to main repo since it is not a problem related to the model itself, but dnn engine instead.


Reccomend a recipe

Make an experience that will ask users a series of questions about what they might like to eat, and then using the results it will recommend a recipe for them.

Questions might include:
What type of meal is this? dinner, lunch, snack

Could also include timing, how much time have you got to prep and cook?
Options might include

<30 mins
<1hr
<1.5hr

1.5hr

Should also include a question about dietary restrictions


Octopod trying to find configuration from hardcoded path in executable

https://github.com/jekyll-octopod/jekyll-octopod/blob/master/bin/octopod#L5
I was unable to octopad setup until changing this line directly inside of the gem. I uncommented line 5 and commented line 4.

If that's the intended fix, just let me know and I can open a PR.
Thanks for putting this together! I am already finding it useful.

:) Thanks @pattex! Glad this was updated and released so quickly.


[2.1] Better handling of joypad device IDs.

Backport of #7862 to 2.1

@Hinsbart, does this not update the joystick handling on android? Looks like that is still working the old fashioned way...

@akien-mga , @Hinsbart did this end up in 2.1.3 ? Or is it only merged into the 2.1 branch? As per my message above, I don't think any changes were made to the android joystick implementation. I wonder if that could explain why its crashing?
I'll see if my new iPhone gamepad can actually sync up to an android phone. i would be surprised but you never know. But my first priority is adding MFI gamepad support to iOS :)


added utf-8 support

Special characters are now displayed correctly in Hastebin.

Thanks for your work! I'll review & test it soon.


Cannot convert Harumachi Clover replays

when i tried to convert my harumachi clover play it didnt convert, instead displaying "HarumachiCloverBan". this is a serious issue that needs attention.

@cab945 update the app


[Game Crash] Song loading crashes every 5-10% (or so) into loading

Steps to reproduce:

clean install the latest release
start Stepmania ...
... Stepmania scans my songs folders and is loading everything
Stepmania crashes
It looks like Stepmania quickly fails when there are too many files.

Sample error message (taken from the console output): Couldn't reopen "/media/blah/games/stepmaniaSongs/D3MIX COMPLETE T/d3mix/d3mix.mp3": Too many open file
I have got 255 742 items / 347.6 GB of data in my songs folders.
Hopefully, when restarting Stepmania it doesn't reload previously scanned files and resume from just before so after many restarts this eventually succeeds. I can confirm after many many restarts all gets loaded and everything eventually goes well. 33 570 songs (631 groups), 3 681 courses.
Latest release (Stepmania-Outfox-Alpha-4.8.1-amd64-date-20200814)
Ubuntu 20.04.1 LTS (uname -a says Linux blah 5.4.0-42-generic #46-Ubuntu SMP Fri Jul 10 00:24:02 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux)
Things are better on Windows 10 (4.8.1 / latest), though I did have to restart Stepmania twice for the loading to finish.

Hiya Kaede! This was a bug due to the way the new Mp3 driver was handling open file handles. I have just fixed this and another hotfix will be released shortly.
You have a long 'parsing' time due to the sheer number of songs you have, but this should not cause any bugs other than you just having to wait a few minutes. Closes #69
Thanks again!


[NPM] ci: NPM Conformance Test improvements

Reason for Change:

NPM Conformance test is being ran more regularly and will use up quota unless clusters are deleted after automated runs.
NPM Conformance was using the latest testing suite, which is not paired with the k8s version of the cluster, potentially bringing in unwanted changes to our subset of tests. This change also allows the test suite to be dynamic and paired to whatever cluster k8s version is in use.
Issue Fixed:

Requirements:

[x] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[x] relevant PR labels added

Notes:

/azp run NPM Conformance Tests

/azp run Azure Container Networking PR


Problem with security in Microsoft.Azure.Management.DataLake.Store

When I try to use Microsoft.Azure.Management.DataLake.Store from AppDomain, I get error:

'Microsoft.Azure.Management.DataLake.Store.Models.AdlsErrorException.GetObjectData(System.Runtime.Serialization.SerializationInfo, System.Runtime.Serialization.StreamingContext)'. Security accessibility of the overriding method must match the security accessibility of the method being overriden.

For AppDomain I have next permissions:
var permisions = new PermissionSet(PermissionState.None);
permisions.AddPermission(new ReflectionPermission(ReflectionPermissionFlag.MemberAccess));
permisions.AddPermission(new FileIOPermission(PermissionState.Unrestricted));
permisions.AddPermission(new WebPermission(PermissionState.Unrestricted));
permisions.AddPermission(new EnvironmentPermission(PermissionState.Unrestricted));
permisions.AddPermission(new SecurityPermission(PermissionState.Unrestricted));
permisions.AddPermission(new SmtpPermission(PermissionState.Unrestricted));

@begoldsm please take a look

@petermzg Just to make sure I understand correctly, the error is when attempting to load that type:
'Microsoft.Azure.Management.DataLake.Store.Models.AdlsErrorException, correct?
If this is the case, I will need some help from @markcowl, since this is a class generated by AutoRest which has the following attribute at the top layer:
[System.Serializable]

@begoldsm Yes! Problem with security attribute [SecurityPermission] on method GetObjectData (Microsoft.Azure.Management.DataLake.Store.Models.AdlsErrorException).  And several other classes also contain this attribute.

Yes, looks like the GetObjectData override needs a SecurityCritical attribute, and the corresponding serialization constructor may slso need a SecuritySafeCritical attribute.  Of course this means the method will not be usable under partial trust.

Is this in runtime or generated code? What are the actual requirements for partial trust of these libraries?

In my app (WebJob on Azure) I load modules to AppDomain. One of modules uses Microsoft.Azure.Management.DataLake.Store

@petermzg can your code run outside webjob scenario?
By design WebJob runs in FullTrust. So at this point could you provide a repro where I can repro the full call stack at my end.

Please reactivate this issue or open a new issue if this is still a problem
Closing.


Stretch issue with a taller geometry ratio on niri wm

The font and border (and presumably all the elements else) stretches weirdly when I set the geometry with a taller ratio (shown by the following screenshot), and this issue happens only on niri wm but not on hyprland..
Maybe this issue is related to #165 (while seemingly I'm not having the scaling issue here).
For example, run tofi with tofi-drun --width=600 --height=800

The remaining contents within the tofi config file are
fuzzy-match=true
drun-launch=true

font-size=18
font="monospace bold"

background-color=#181818
outline-width=0
outline-color=#1a1a1a
border-width=4
border-color=#7cafc2
text-color=#b8b8b8
result-spacing=5
corner-radius=10
width=0

prompt-text="❯   "
prompt-color=#5291e2

selection-color=#ffffff
selection-background=#383c4a
selection-match-color=#ff0000
selection-background-padding=0,10
selection-background-corner-radius=5

Can you try the patch in #189?

Can you try the patch in #189?

Thx, your patch indeed fixes my problem, I missed the part setting anchor=center in my tofi config, which seems to be necessary?

Thx, your patch indeed fixes my problem, I missed the part setting anchor=center in my tofi config explicitly, which seems to be necessary?

anchor=center is necessary only if you want tofi to appear in the center of the screen (but i believe center is the default value, so even if you don't specify it explicitly it should appear in the center). Other possible anchors are top-left, top, top-right, right, bottom-right, bottom, bottom-left, left, but the issue should be solved by the patch regardless of the chosen anchor

@BRS5672023 Could you please try the patch with other anchor values? I'm almost sure that this issue is a duplicate of #165

@BRS5672023 Could you please try the patch with other anchor values? I'm almost sure that this issue is a duplicate of #165

All values work as intended, but if I comment out the anchor line in tofi config, then its position is at top-left and stretched abnormally.

All values work as intended, but if I comment out the anchor line in tofi config, then its position is at top-left and stretched abnormally.

Probably there is something wrong in the patch or in some other part of the code responsible for setting the anchor, thank you! I'll try to update the patch and see what I can do

@BRS5672023 I updated the patch, does it work as expected now?

@BRS5672023 I updated the patch, does it work as expected now?

Yes, it is fixed now, thanks for your patch.


Version 1.10 new bugs

Looks like something went wrong with the new 1.10 version. Here are the newly introduced bugs:

Tapping the icon to scan a picture file opens a page that has a never ending search spinning circle at the bottom. Previous vesrion gave the option to open an image through one of the installed apps.
When trying to make a QR code out of a phone contact, tapping the contacts icon on the top right gives a message that there's no proper app installed. Previously it opened the contacts app.
When trying to make a QR code of an installed app, a list of only just a few of the installed apps appears. Previously all apps were available.
In the settings, the source code link does nothing when tapped. Previously in opened the github page.
When opening a created QR code, the brightness icon on the top right has no effect when tapped. It's supposed to set the brightness of the screen. This is an old bug.
Using stock Android 11, on a Nokia 6.2.

I'm not a programmer, but it looks like to me that the new version may have some trouble identifying installed apps?

Same problem on android 11 on sony. (Sms app not found)


Map shows whole earth when no store has a location

In group 20 there is no store location specified. The map thus doesn't zoom into any location, even though a group location is added.
It would be better if the map zoomed into the group location, if no store location data was provided.
No marker should be displayed at the group location.

isn't that more like a feature ? Because it is behaving like it should - since there is no store location there is no place to zoom in ?

I agree with your proposal :-D -
If store.location = not found
then
use group.location.
i just wouldn't say it's a bug ;-) but i won't bug you with it.


DESC INTEGRATION QUERY AS A DATA SOURCE

Is your feature request related to a problem? Please describe.
when we creating snowflake storage integration we will require to have external id and user information dumped to enable trust relationship. Currently available data source snowflake_storage_integrations does not provide external id and user information. So it will be better to realize the equivalent of desc integration <integration_name>; snowflake statement.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
when we creating snowflake storage integration we will require to have external id and user information dumped to enable trust relationship. Currently available data source snowflake_storage_integrations does not provide external id and user information. So it will be better to realize the equivalent of desc integration <integration_name>; snowflake statement.
Describe the solution you'd like
A clear and concise description of what you want to happen.
when we creating snowflake storage integration we will require to have external id and user information dumped to enable trust relationship. Currently available data source snowflake_storage_integrations does not provide external id and user information. So it will be better to realize the equivalent of desc integration <integration_name>; snowflake statement.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
when we creating snowflake storage integration we will require to have external id and user information dumped to enable trust relationship. Currently available data source snowflake_storage_integrations does not provide external id and user information. So it will be better to realize the equivalent of desc integration <integration_name>; snowflake statement.
Additional context
when we creating snowflake storage integration we will require to have external id and user information dumped to enable trust relationship. Currently available data source snowflake_storage_integrations does not provide external id and user information. So it will be better to realize the equivalent of desc integration <integration_name>; snowflake statement.
Add any other context or screenshots about the feature request here.
when we creating snowflake storage integration we will require to have external id and user information dumped to enable trust relationship. Currently available data source snowflake_storage_integrations does not provide external id and user information. So it will be better to realize the equivalent of desc integration <integration_name>; snowflake statement.

Also need this.


Feature request: Make fjall::WriteTransaction Send

https://docs.rs/fjall/latest/fjall/struct.WriteTransaction.html
It's hard to use across threads in async context, internally it's using a Mutex, is it not possible to implement the Send bound?
It makes it easier to wrap the API.

I think - and maybe someone can correct me on this - this is not possible with std Mutexes, because they need to be unlocked on the same thread that locked them (https://github.com/rust-lang/rust/issues/23465). I don't understand why, but it's a limitation of the std locks. One way to get around this would be using parking_lot's send_guard feature (https://github.com/Amanieu/parking_lot/issues/197), I guess.
Currently, the transactions are single-writer transactions, so each transaction holds the global write lock it gets from the keyspace. With concurrency control (SSI, https://github.com/fjall-rs/fjall/issues/55), the lock would not need to be held before the transaction is committed, so I think SSI transactions would be Send-able.

Mutex works with the underlying data. If T is Send, so is the Mutex.

Mutex works with the underlying data. If T is Send, so is the Mutex.

Yeah you are right, I meant MutexGuard specifically, because the WriteTx holds a guard, not a Mutex.

With #55 transactions should become Send because they will not contain a lock guard.

Should work for SSI transactions, implemented in https://github.com/fjall-rs/fjall/pull/79


Need to correct the Installation example

Here calico_versioned_docs/version-3.26/operations/image-options/alternate-registry.mdx
at the end we have an example of Installation.
registry field should be spec.registry but in the example it is spec.imagePullSecrets[0].registry

Fixed in #861.


Bonfire Steamroller - does not accept solution

Challenge Bonfire: Steamroller has an issue.
User Agent is: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:30.0) Gecko/20100101 Firefox/30.0.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:
//no protect
var res = [];
function steamroller(arr) {
  // I'm a steamroller, baby
  steamroll(arr);

  return res;
}

function steamroll(arr){

  for(var i=0; i<arr.length; i++){
    if(Array.isArray(arr[i])){
      steamroll(arr[i]);
    }
    else{
      res.push(arr[i]);
    }
  }
}

steamroller([1, [2], [3, [[4]]]]);

Sorry, but Issues are not for code problems.  Please ask for help in Help Chat.
Thanks and Happy Coding!

Thanks. Sorry, but I did not not know about the issue report rules. But the solution appears ok. I tested it on all the cases.


Cask: add doctor check for xattr

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

This pull request extends brew cask doctor to help users fix common issues with xattr.
CC: Homebrew/homebrew-cask#69660

Thanks, @amyspark!


Update on-create.sh for Bicep extension installation

What content needs to be created or modified?
The on-create.sh for the devcontainer is currently using an outdated curl call to get.radapp for the Bicep extension. This needs to be modified with a more updated version that uses either GitHub or the VS Code Marketplace.
Describe the solution you'd like
A GHCR pull or a pull from the VS Code Marketplace.
Where should the new material be placed?
https://github.com/radius-project/docs/blob/c9c773cd72aa90b263070a7d3667a168bbe880ad/.devcontainer/on-create.sh#L19
The associated pull request from the Radius code repo, if applicable
Additional context
Related to https://github.com/radius-project/radius/issues/7295
AB#11438

Closing as complete


Crashes Dungeondraft

Dungeondraft v11.0.5
Windows 11 Pro
RasterPainter 1.0.2
Crashes Dungeondraft whenever I try to open a new map or an existing map.
Tried the following combinations but it still crashes)
Large maps ( > 60x60 squares )
Small maps (10x10 squares)
0 asset packs enabled
6 asset packs enabled
20 asset packs enabled

Having the exact same issue.
Windows 10
DungeonDraft v<IP_ADDRESS>
RasterPainter 1.0.2


tools/minidumpserver: refactor codes

Summary

refactor code style and simplify parse method to make code more readable
change mem find algorithm to bi-search

Impact
Testing

well, risc-v failed again.
some links problem
make[1]: warning: jobserver unavailable: using -j1.  Add '+' to parent make rule.
  Configuring...
make[1]: warning: jobserver unavailable: using -j1.  Add '+' to parent make rule.
make[1]: warning: jobserver unavailable: using -j1.  Add '+' to parent make rule.
  Building NuttX...
riscv-none-elf-ld: sotest.o: ABI is incompatible with that of the selected emulation:
  target emulation `elf64-littleriscv' does not match `elf32-littleriscv'
riscv-none-elf-ld: failed to merge target specific data of file sotest.o
riscv-none-elf-ld: attempt to do relocatable link with elf64-littleriscv input and elf32-littleriscv output
riscv-none-elf-ld: sotest.o: file class ELFCLASS64 incompatible with ELFCLASS32
riscv-none-elf-ld: final link failed: file in wrong format
make[4]: *** [Makefile:54: sotest] Error 1

Fixed here: https://github.com/apache/nuttx/pull/10719

macOS build failed


FP8 loader for pix

please add FP8 support for the CLIP

It's not the pix fp8 model, but I was able to get the t5xxl fp8 clip model working like this:

Comfy ExtraModels node:
https://github.com/city96/ComfyUI_ExtraModels
Here's the download link:
https://huggingface.co/comfyanonymous/flux_text_encoders/blob/main/t5xxl_fp8_e4m3fn.safetensors


[kn] Kannada New Language Translations Added

[kn] Kannada New Language Translations Added
and PHPCS formatted

Hello,
thank you
can you translate json file too ?

Ok will do, and will get back to you with PR


Don't capture this unnecessarily

Introduced and unaddressed in https://github.com/nod-ai/iree-amd-aie/pull/812

I added "this" because it didn't pass the options correctly to the rewritter function in my first attempt with the change. But I guess some other changes fixed it and now it works without passing "this"! Thanks for making the change.


Project: Exchange Event Generator

Project name
Exchange Event Generator
Description
A service worker which can create "random" events for every member of an organization utilizing the Microsoft Graph API.
Basically simulating activity within an organization.
Repo URL
https://github.com/Ichychi/ExchangeEventGenerator
Team members
Ichychi

Very convenient for testing @Ichychi! Would you be willing to record a short video to demonstrate the app? It's not required for the hackathon but it would help us showcase your work internally at Microsoft and publicly in the community 😊

Sure thing, here you go:
(I removed some randomness and tweaked some settings for this video)
https://user-images.githubusercontent.com/36235489/225963689-c7cb598f-096a-477e-b848-3051aa35aac5.mp4


intersink/intersrc - A/V delay

having a simple input file  running in a loop, and producing a TCP output.
produces - over time, like 12h+ a significant A/V delay.
it is not really about the time/hours, it seems once the cpu usage on the host is peaking/blocking the x264 encoder, audio continues flawless but video lags slightly behind.
folks at voc2mix, also have suffered that issue, they somehow overcome it with restructuring their pipelines. https://github.com/voc/voctomix/issues/58
however, they cut out the tcp input - for that to work, and don't really support dynamic inputs (in general they drift more in a SDI-only direction)
currently i am playing around with: https://github.com/RidgeRun/gst-interpipe
you have any idea how to solve the problem (overscaling CPU is one thing, but its no guarantee that even the biggest cpu is working 100% of the running time) - restarting (stop/play) the mixer/output solves it but interrupts the service.

Hello @hjanuschka
#55 Covers some of this as well.
Are you able to share any of the aspects of the file and system you are running it on ?
We have a version that now uses interpipe to solve some of the issue of intersink/intersrc due to the way they split buffers and create a new clock. Sadly I've not had time to backport these changes. But we have had much more success with them.

As for the CPU problem you could run brave with one of the GST_DEBUG option running to identify if there are buffers that are get stuck.
https://gstreamer.freedesktop.org/documentation/tutorials/basic/debugging-tools.html?gi-language=c

debugging didnt bring any light, it show sometimes dropped frames, sometimes high latency.
sometimes, frames are missing in the V pipeline, and sometimes they are stuck for a couple of ms and then run in double-speed. its just too unreliable.
i'v been trying to hlssink the whole thing, with a 3-5s delay, and kept restarting the tcp/mixer on a regular basis :o - but that feels way too hacky.
any chances i can give the interpipe branch?
thx for your feedback.
details:
(does not happen on small files with low resolution)
resolution: 1920x1080
sample file: https://vzaar.com/videos/1570191
when testing, with TCP output, it sometimes is "just" client issue, like vlc is lagging.
using "local" gl output, or direkt decklink sink. removes alot of cpu usage, but still introduces lag over time, so the main problem must be the inter* elements.

system: 8 cores, nvidia card (although no nvdec is used and stuff like that)
running: native on ubuntu, also running on k8s (tried with cpu limits and stuff)

any chances i can give the interpipe branch?

I can't promise an ETA but when i can find some time to work on this I'll push some changes.
Out internal version of this is more progressed but since it ties into more BBC systems it a little hard to backport.

system: 8 cores, nvidia card (although no nvdec is used and stuff like that) - using nvdec, might reduce the issue a bit, since it might free a couple of cpu resouces, although it won't solve the problem :/

Brave uses playbin3 for files, this elements combines many smaller elements together to achieve de-muxing and decoding of the streams in the file. Should you have HW acceleration and the correct libraries installed it should use.

@moschopsuk really would appreciate, to see how it can be ported to interpipes, let me know when there is something, i can look at, would also love to contribute to it.
ohh, yeah, thats correct about playbin3, anyway, cannot get rid of the occasianal stutters and resulting delays.

There's discussion on replacing intersrc and intersink with Ridge Runs interpipe? Is therre any update here?
Not sure if interpipe has been approved to be included in gstreamer mainstream as yet. Seems awfully long-time to get it including as a usable element?  I feel interpipe is favourable here..


evict broken connection instead of release to pool

I'm use Hikari jdbc connection pool.
when execute statement witch produce an exception ( for example by network is broken). as following:
try{
   Connection con = pool.getConnection();
   con.executeQuery("....");
}catch(Exception e){
   con.close();
}

does con.close will evict the broken connection, instead of release it to the pool.
if the broken connection is released to pool. it maybe got by following request of getConnection.

Apart from your question, shouldn't you consider using a try-with-resources block (if you have a recent java). Or at least con.close() in the finally {} block?

@slyoldfox my question is not leaking the con.close() action.
I means if the con.close() called, the broken connection will still be release to the Pool. which will be got by next getConnection() call.

@ChinaXing HikariCP checks exceptions for disconnection codes (here).  If a disconnected connection is returned to the pool, or a connection is disconnected while idle in the pool, it will generally be tested before being given to a user.

I think it was not what you said.  the checkException only be called when SQLException was threw in the con.close().
if a disconnected connection is returned to the pool. when given to a user, test only occurred at :
clockSource.elapsedMillis(poolEntry.lastAccessed, now) > ALIVE_BYPASS_WINDOW_MS

@ChinaXing I think someone also replied to your same question on so, you should be closing connection 'correctly' and let pool do its job (of closing or recycling). Hope your doubt is 'clear' now.
If not and you think HikariCP is not doing what 'you' are expecting, post the test code as evidence for investigation. Looking at piece of hikariCP code will not give you 'complete' picture.

@brettwooldridge the following commit change the ALIVE_BYPASS_WINDOW_MS, in this window, a given back broken connection make be got again from the pool.
https://github.com/brettwooldridge/HikariCP/commit/23c1b80bfd7ffa4df21365b6f5d6a80bcdc8f32d

@ChinaXing similar to ALIVE_BYPASS_WINDOW_MS, ...even if pool is ALWAYS returning connection after validating, it is possible that connection gets 'disconnected' when 'client' needs to use it.
'you' need to write 'client' considering such possibilities if required.


Create imprint page

so who "owns" the page legally for now it is r0g right? we need this soon @intertwilight

yes, for now ... default

I'll go for this one (will keep it in english but might need a German version later)
https://www.e-recht24.de/impressum-generator.html


Feature/bootstrap4

Hi, are there any other issues that keep this branch from being merged? I would love to help and resolve them.

Please accept this request, I have reviewed the branch and I have experienced the problems that this pull request resolves.
I could also help if necessary.

Thanks for the PR, we'll test it soon and get back with the feedback.


Attempting to compile

Apologies if this is a very naive question but when I try to build the solution from command line I switch to the current directory and get a: 'genie' is not recognized as an internal or external command, operable program or batch file. error
Looks like the only 2 files in that folder are genie.lua and MTuner.lua
Is there something I'm missing?
thanks

I need to put up more detailed instructions :)
GENie binary is located in 'build' project:
~/build/tools/OS/genie


Never-winners and choice of min_gap/max_gap

When I set  min_control_gap = 1 and max_control_gap = Inf, the package allows me to include never-winners in the sample by using never_treat_action = "keep".
However, if I set min_control_gap = 1 and max_control_gap =1, the code breaks down. Below, I have copy-pasted my code and the resulting error message.
results <- ES( long_data = event_study, outcomevar = "logsales",  unit_var = "tin", cluster_vars="tin", cal_time_var = "tax_yr", onset_time_var = "win_yr", omitted_event_time = -2, min_control_gap = 1, max_control_gap = 1 , control_subset_var=NA, control_subset_event_time=0, never_treat_action = "keep")
INFO [2019-02-15 09:14:52] Beginning ES.
INFO [2019-02-15 09:14:52] Beginning data stacking.
Error in [.data.table(balanced_treated_control, , na.omit(c(outcomevar,  :
column(s) not found: logsales, tin, tax_yr, win_yr, ref_onset_time, ref_event_time, catt_specific_sample, treated
In addition: Warning message:
In max(get(cal_time_var)) : no non-missing arguments to max; returning -Inf

@tmeling What is the desired behavior? Do you want the control group to be the never-treated group and those who are treated at event time +1 only?

Yes, that is correct. I want the control group to include both never-winners and those that eventually win, and I want to loop across various specifications of min_control_gap and max_control_gap to check whether the results differ across specifications.

@tmeling I think this should be possible, but can you explain a bit more a setting where you would want to use, say, those that win shortly after the treatment group as well as those that never win (but exclude those that win long after the treatment group)?
I understand in your exercise (kind of a robustness check), you are just checking something like all combinations of not-yet winners and never winners, but would be useful to have a concrete example where you would really be interested in some of those combinations of control groups.


Moved to .netstandard2.0

Updated ServiceStack dependencies to 5.0.2.  Updated code and tests accordingly.
Changed some tests to remove xunit warnings

Thanks!
ian
On Wed, Feb 14, 2018 at 10:08 AM, Jonathan Nappee<EMAIL_ADDRESS>wrote:

Updated ServiceStack dependencies to 5.0.2. Updated code and tests
accordingly.
Changed some tests to remove xunit warnings
You can view, comment on, or merge this pull request online at:
https://github.com/ticky74/ServiceStack.Authentication.Azure/pull/11
Commit Summary

updated ServiceStack dependencies to 5.0.2. Updated code and tests
accordingly. Changed some tests to remove xunit warnings

File Changes

M src/ServiceStack.Authentication.Azure.OrmLite/
ServiceStack.Authentication.Azure.OrmLite.csproj
https://github.com/ticky74/ServiceStack.Authentication.Azure/pull/11/files#diff-0
(10)
M src/ServiceStack.Authentication.Azure.ServiceModel/ServiceStack.
Authentication.Azure.ServiceModel.csproj
https://github.com/ticky74/ServiceStack.Authentication.Azure/pull/11/files#diff-1
(2)
M src/ServiceStack.Authentication.Azure.Tests/
AzureGraphAuthProviderTests.cs
https://github.com/ticky74/ServiceStack.Authentication.Azure/pull/11/files#diff-2
(63)
M src/ServiceStack.Authentication.Azure.Tests/
GraphAuthServiceTest.cs
https://github.com/ticky74/ServiceStack.Authentication.Azure/pull/11/files#diff-3
(2)
M src/ServiceStack.Authentication.Azure.Tests/
OrmLiteApplicationRegistryServiceTest.cs
https://github.com/ticky74/ServiceStack.Authentication.Azure/pull/11/files#diff-4
(2)
M src/ServiceStack.Authentication.Azure.Tests/
ServiceStack.Authentication.Azure.Tests.csproj
https://github.com/ticky74/ServiceStack.Authentication.Azure/pull/11/files#diff-5
(18)
M src/ServiceStack.Authentication.Azure.Tests/TestAppHost.cs
https://github.com/ticky74/ServiceStack.Authentication.Azure/pull/11/files#diff-6
(2)
M src/ServiceStack.Authentication.Azure/
ServiceStack.Authentication.Azure.csproj
https://github.com/ticky74/ServiceStack.Authentication.Azure/pull/11/files#diff-7
(20)

Patch Links:

https://github.com/ticky74/ServiceStack.Authentication.
Azure/pull/11.patch
https://github.com/ticky74/ServiceStack.Authentication.
Azure/pull/11.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ticky74/ServiceStack.Authentication.Azure/pull/11,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AD9H9_9b-b_kPH6uAJ6v6jLoeyc94cwuks5tUxL7gaJpZM4SFqKc
.

btw I didnt change any version or release notes


Prometheus metrics break config reload

What version of Traefik are you using (traefik version)?
master
What is your environment & configuration (arguments, toml...)?
marathon
What did you do?
Enable marathon
Enable prometheus web stats
Run traeffik
Deploy app
What did you expect to see?
Traeffik should spot the upadtes, regenerate the configuration, and reload
What did you see instead?
An exception after the deploy and config update appearing
DEBU[2017-03-29T15:38:09Z] Creating loadd-balancer connlimit
ERRO[2017-03-29T15:38:09Z] Error in Go routine: duplicate metrics collector registration attempted
goroutine 57 [running]:
runtime/debug.Stack(0x6d21af, 0xc42004c240, 0x1e602ee)
        /usr/local/go/src/runtime/debug/stack.go:24 +0x79
runtime/debug.PrintStack()
        /usr/local/go/src/runtime/debug/stack.go:16 +0x22
github.com/containous/traefik/safe.defaultRecoverGoroutine(0x1c534e0, 0xc4207a7700)
        /go/src/github.com/containous/traefik/safe/routine.go:147 +0x7d
github.com/containous/traefik/safe.GoWithRecover.func1.1(0x1ecd948)
        /go/src/github.com/containous/traefik/safe/routine.go:138 +0x57
panic(0x1c534e0, 0xc4207a7700)
        /usr/local/go/src/runtime/panic.go:489 +0x2cf
github.com/containous/traefik/vendor/github.com/prometheus/client_golang/prometheus.(*Registry).MustRegister(0xc42004d780, 0xc4201e9ec0, 0x1, 0x1)
        /go/src/github.com/containous/traefik/vendor/github.com/prometheus/client_golang/prometheus/registry.go:404 +0x92
github.com/containous/traefik/vendor/github.com/prometheus/client_golang/prometheus.MustRegister(0xc4201e9ec0, 0x1, 0x1)
        /go/src/github.com/containous/traefik/vendor/github.com/prometheus/client_golang/prometheus/registry.go:152 +0x53
github.com/containous/traefik/vendor/github.com/go-kit/kit/metrics/prometheus.NewCounterFrom(0x0, 0x0, 0x0, 0x0, 0x1e5fb41, 0x16, 0x1ea0c0b, 0x48, 0xc4203e4720, 0xc4207a7580, ...)
        /go/src/github.com/containous/traefik/vendor/github.com/go-kit/kit/metrics/prometheus/prometheus.go:24 +0xed
github.com/containous/traefik/middlewares.NewPrometheus(0xc4203a5410, 0x5, 0xc4203e3980, 0xc4203e46f0)
        /go/src/github.com/containous/traefik/middlewares/prometheus.go:42 +0x214
main.(*Server).loadConfig(0xc4207c2ea0, 0xc420abdec0, 0xa, 0x101, 0x0, 0x0, 0x0, 0x0, 0x1e3e68d, 0x5, ...)
        /go/src/github.com/containous/traefik/server.go:733 +0x1c32
main.(*Server).listenConfigurations(0xc4207c2ea0, 0xc4205d4770)
        /go/src/github.com/containous/traefik/server.go:294 +0x36d
main.(*Server).Start.func2(0xc4205d4770)
        /go/src/github.com/containous/traefik/server.go:103 +0x34
github.com/containous/traefik/safe.(*Pool).Go.func1()
        /go/src/github.com/containous/traefik/safe/routine.go:77 +0x3a
github.com/containous/traefik/safe.GoWithRecover.func1(0x1ecd948, 0xc4202e0ff0)
        /go/src/github.com/containous/traefik/safe/routine.go:141 +0x4d
created by github.com/containous/traefik/safe.GoWithRecover
        /go/src/github.com/containous/traefik/safe/routine.go:142 +0x49

after this, additional configuration changes no longer result in the configuration changing.
This should be a fairly easy fix (don't use MustRegister, and check the error), however, there may be a better way to manage the prometheus metrics (using a custom collector, rather than a registry), I'm happy to investigate this.

probably should mention that remove
[web.metrics.prometheus]
Buckets=[0.1,0.3,1.2,5.0]

fixes the problem

Closing, this seems to have been fixed in 1.2, but isn't present on master. Presumably you aren't using the master branch.

I am in the process of merging 1.2 branch back to master branch ;)

@emilevauge the merge hasn't happened yet as far as I can see, so master is still broken. Is the branch/merge strategy documented anywhere?

It's on master now.
A bit of documentation is likely due, but things have been in the flux lately.


replace dmd and color_dmd widgets with display widget

Currently we have dmd and color_dmd widgets. These aren't really needed. We should just create a single widget called display which just shows another display on the current slide. It could have all the properties that are currently in the dmd widgets, like the dot screen, color controls, ability to render colors to mono, etc.
This could possibly replace the slide_frame widget too.
This is a longer term thing (after expo) since it will need a fair amount of testing and probably break configs.

superseded by #186


Request for new version

It has been a while since timescale/timescaledb:0.9.2 was released.
I have been using Docker image created with Dockerfile modified several lines for about 3 weeks and seems to be working well.
Like this:
FROM postgres:10.4-alpine

ENV PG_MAJOR 10.4
ENV TIMESCALEDB_VERSION 0.9.2

I am happy that pg_prometheus follows timescaledb:0.9.2.

Hi @y2sarakawa, I've created a PR for this https://github.com/timescale/pg_prometheus/pull/18
Soon we plan to release a new version with significant performance improvements so keep an eye on the repo ;)

#12(, #18), and #19 resolved this issue.


Subscriptions: RabbitMQ provider

Is your feature request related to a problem?
In our project (I've already reported some issue regarding subscriptions under @cts-tradeit) we are using RabbitMQ to sync multiple instances in a swarm (which inspired me to write docu in #5295 which is still waiting for second approval). As of now there is not out of the box provider for RabbitMQ's PubSub.
The solution you'd like
Creation of ITopicEventReceiver  and ITopicEventSender for RabbitMQ using RabbitMQ.Client.
If you are not currently planning this feature I can submit our solution in PR (reworked as it does not use those standard interfaces of HC but relies on IAsyncEnumerables directly).
Product
Hot Chocolate

Can you get it to work with the standard interfaces?

Yes I can. The only issue is non string topics but as I can see you are serializing them for redis imple to string. This approach can be used for RabbitMQ exchanges aswell.
So, to main (13) or main-12? If interfaces did not change I can do it to main-12 and u can cherry-pick it.

main please ... we can cherry pick it to 12 later.

#5321

close with https://github.com/ChilliCream/hotchocolate/pull/5599


Feature: allow to disable stock management

Closes https://github.com/mirumee/saleor/issues/1979.
This adds a new field handle_stock on the variant model, which toggles to stock handling (allocations and checks).
This also adds a new settings (on the SiteSettings model), that allows to set the default stock handling for variant creation form.
It's only there to allow users to save time and prevent them from accidentally setting an unwanted stock management.
Pull Request Checklist
(Please keep this section. It will make maintainer's life easier.)

[x] Privileged views and APIs are guarded by proper permission checks.
[x] All visible strings are translated with proper context.
[x] All data-formatting is locale-aware (dates, numbers, and so on).
[x] Database queries are optimized and the number of queries is constant.
[x] The changes are tested.
[x] The code is documented (docstrings, project documentation).
[x] Python code quality checks pass: pycodestyle, pydocstyle, pylint.
[x] JavaScript code quality checks pass: eslint.

@elwoodxblues all good for me, I will do the changes. Thanks!


Python encoder

@sppalkia: just as an FYI, not ready to merge.
Major issues right now:

Some benchmarks don't yet work correctly
Encoder / decoder much slower than C++ encoder / decoder

Why would you do serialization / deserialization in Python when you can do it in C/C++? Do you have to copy the memory into a new memory space, or is zero-copy handoff possible?

You might consider using https://github.com/spacetelescope/asv for performance tracking -- this is what we use in pandas for microbenchmarking and catching performance regressions.

The main problem is that we need these two header files:
#include "numpy/arrayobject.h"```

It seemed like finding these header files is highly dependent on how the user's Python has been installed (for example, the user could use be using `virtualenv`), and we thought having everything in Python can help us route around this problem.

I don't think you're going to be able to avoid Python C extensions and linking to NumPy for very long -- best to sort out the build toolchain problems up front.

For Python.h, it depends on your build system. If you are using distutils, this is handled automatically
For numpy/arrayobject.h, you use numpy.get_include

On my system, I have
In [2]: numpy.get_include()
Out[2]: '/home/wesm/anaconda3/envs/py27/lib/python2.7/site-packages/numpy/core/include'

In pyarrow, we are using CMake modules to find these dependencies
https://github.com/apache/arrow/tree/master/python/cmake_modules

FWIW, your primary audience for Weld in the short term are much more likely to be conda users than pip/virtualenv users -- I think you're going to want to figure out conda install weld sooner than pip install weld (which probably needs to involve manylinux1 wheels on Linux)

Ok, great, I will look into this. Thanks for all the help!

Closing.


DAG-1958: Cache the results of evergreen evaluate when creating a n…

…ew branch.

evergreen merge


Callback getting called multiple times with the same attempt number

I get numerous console.log output with the same attempt number before the attempt number gets incremented.
doing 2
doing 2
doing 2
...
doing 3
doing 3
doing 3

Why would the callback get executed again with the same attempt number?
function retryCommand(command) {
  const options = { retries: 3 }
  return promiseRetry(options, (retry, number) => {
    if (number > 1) console.log(`doing ${number}`)
    return exec(command)
      .catch((err) => {
        if (errorIsRetriable(err)) {
          retry(err)
          // should there be a 'return' here? I tried it with and without, but same result
        }
        throw err
      })
  })
}

And then I use it by calling it from inside an async function like await retryCommand('some-executable').
If it's important, here's what exec is (it's Node's built-in child_process.exec promisifed using the pify library).
import childProcess from 'child_process'
import pify from 'pify'

const exec = pify(childProcess, { multiArgs: true }).exec

We have a test for that: https://github.com/IndigoUnited/node-promise-retry/blob/master/test/test.js#L31
Can you debug into promise-retry and log number right here on line 29?

I put a if (number > 1) console.log(doing ${number} - from promise-retry) right before line 29, and in addition to my logging statement from the previous post, the output is like
doing 2 - from promise-retry
doing 2
doing 2 - from promise-retry
doing 2
doing 2 - from promise-retry
doing 2
...
doing 3 - from promise-retry
doing 3
doing 3 - from promise-retry
doing 3
doing 3 - from promise-retry
doing 3

@rightaway I tried to reproduce the error with the following sample:
const childProcess = require('child_process')
const promiseRetry = require('promise-retry')
const pify = require('pify')

const exec = pify(childProcess, { multiArgs: true }).exec

function retryCommand(command) {
    const options = { retries: 3 }

    return promiseRetry(options, (retry, number) => {
        if (number > 1) console.log(`doing ${number}`)
        return exec(command)
        .catch(retry)
    })
}

retryCommand('ls -4')
.then(stdout => {
    console.log(`${stdout[0]}`)
})

and the output was:
doing 2
doing 3
doing 4
(node:87108) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): Error: Command failed: ls -4
ls: illegal option -- 4
usage: ls [-ABCFGHLOPRSTUWabcdefghiklmnopqrstuwx1] [file ...]

Can you provide a full sample that reproduces the error? That would be awesome! 😄
Thanks

@rightaway any updates on this?

@carsy I haven't been able to duplicate this in a full standalone sample, even though I'm getting it in the application, so it has something to do with the specific way my application is using it. I'll reopen the issue when I've identified what it is.


Remove console.log from handleOutsideClick HostListener

@technikhil314 Was working on implementing your component but noticed the console being spammed with CursorEvents. Removed the console.log as it has no obvious added value.

Hey @JustJollyJonah Thank you for catching this. Merging this PR. I will release this in next version.


ci: remove older versions from python tests

addresses reanahub/reana#460

Codecov Report

Merging #161 (1d0df00) into maint-0.7 (5fbd117) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           maint-0.7     #161   +/-   ##
==========================================
  Coverage      25.91%   25.91%
==========================================
  Files             10       10
  Lines            575      575
==========================================
  Hits             149      149
  Misses           426      426


Fix v1beta1 to v1 migration doc

Changes

This commit follows up on #5415 and resolves the respective examples and docs on the migration.
Fixes: #5330
/kind documentation
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[x] Has Docs included if any changes are user facing
[n/a] Has Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[x] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
[x] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
The document has been added that describes the changes needed when migrating from v1beta1 to v1.

/kind documentation

/retest

Can we also move this document elsewhere for the time being ? If we were to release tomorrow, a new page would appear (Migrating from v1beta1) which would be a bit confusing as it would make it look like v1 is already available, when it is not yet the case.

@vdemeester May I ask where would be a good place to put it or shall I just remove it at the moment ?

Can we also move this document elsewhere for the time being ? If we were to release tomorrow, a new page would appear (Migrating from v1beta1) which would be a bit confusing as it would make it look like v1 is already available, when it is not yet the case.

@vdemeester Thanks for the comment. May I ask where would be a good place to put it or shall I just remove it at the moment ?

Hum we definitely shouldn't remove it. Maybe in a dev/docs or anywhere that is not "indexed" by our website.

/retest

/retest

Created https://github.com/tektoncd/pipeline/issues/5478 for the test flakes on this PR


house or House ?

Should the type be Lakehouse or LakeHouse
as per
https://github.com/Azure/azure-sdk-for-net/blob/main/sdk/datafactory/Azure.ResourceManager.DataFactory/src/Generated/Models/LakeHouseTableDataset.cs

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

ID: 5e665b6b-644b-fbe9-5497-35196f51e3e4
Version Independent ID: 959308b5-71eb-fee5-ef19-b525730d3f4d
Content: Copy and transform data in Microsoft Fabric Lakehouse - Azure Data Factory & Azure Synapse
Content Source: articles/data-factory/connector-microsoft-fabric-lakehouse.md
Service: data-factory
Sub-service: data-movement
GitHub Login: @jianleishen
Microsoft Alias: jianleishen

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

Hi @mikeblakeuk
According to this below article Lakehouse is correct.
https://learn.microsoft.com/en-us/azure/data-factory/connector-microsoft-fabric-lakehouse?tabs=data-factory
If there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.

So the Open API is "wrong"?

@mikeblakeuk
Thank you for bringing this to our attention.
I've assigned this task to our content author, @jianleishen, who will carefully review it and provide their valuable insights. The feedback they provide will help guide any necessary adjustments by the PM/ EM Teams.

Hi @mikeblakeuk, and thanks for your feedback. I checked and confirmed in fact, LakeHouse is the correct casing for the API usage, despite it not matching the convention we use in the docs, where Lakehouse is correct. The product team might consider a change to the API to correct that, but to engage them regarding a change to an API, please open a new free support case from the Azure portal, selecting the menu at the top left of the screen to choose its last item, Help + support, and then choosing Create a new support case. Our support team can follow up and the product group might be able to change this to the correct casing. They might hesitate to do so though, too, since it could be a breaking change for existing code that uses it. Since I confirmed for now that this is the correct casing for use of the API (even if you're correct that it's wrong for our usage standard in the docs), I will close this issue out. However, we can continue this discussion here in this thread if necessary, even after it's closed. #please-close


[Bug] App flagged as Trojan

The app gets flagged as Trojan.
Here's the scan result: https://www.virustotal.com/gui/file/fc429c41b70ee65e1f45575a29e93fc9e51afe50e3188594824246c5cda81701/detection

Hey, it's because the app is able to download the newest update and install it. In app update


Log all actions

[x] log speaker actions
[x] log orga actions (add orga flag in logs)
[x] display logs in orga frontend
[ ] display logs in speaker frontend

Since organizers can change speaker data and submission data, displaying a changelog would be nice. This should be located to the right edge of the main part of the page in the speaker's view of her submission.


AppendBytes and mp4?

I'm resurrecting a project that uses NetStream to load mp4 files. I'm currently troubleshooting this process. I'm using the On Demand method for PAD, but I'm receiving the following error: Error #2004: One of the parameters is invalid "appendbytes"
Before I even begin to rebuild the methods handling the video clips and troubleshoot the error, I want to make sure that I can even use AppendBytes, or is this still limited to FLVs? If so, should I try the Fast Follow method? Thanks.

Hi
NetSteam.appendBytes() only supports FLV files - although it is possible to repackage an a/v file from an MP4 container to an FLV one.
But what are you actually trying to do? Is this just for handling an MP4 file, or a stream? If you're using on-demand play asset delivery then you'd need to wait for the whole asset pack to finish downloading before you'd be able to access it, so then could use the MP4 file directly in a request to NetStream.play(), which may be simpler?
thanks

Apologies, I meant Install Time and not On Demand. This is an application for young children, and assets must either be downloaded at installation, or immediately afterward as parents hand off the device to their kids, who won't be able to manage the process. Ideally, we'd want to do the former, but only if the video clips can be repackaged in FLV format without recompression. Otherwise, we'd have to resort to a Fast Follow approach.

I think the play asset delivery piece is independent of the calls to actually play the video... but I think I see what you mean because of the install-time assets being accessed via the AssetFile mechanism... let me see if there's something we can do there.
thanks

Any assistance with the ability to access .mp4 files via Install Time access is greatly appreciated! Thanks.

That sounds like it would work for our needs, thanks. Please let me know when a link to the Beta is available.

Greetings  @ajwfrost , confirmed that readBytes() is working in the latest Beta release. Thank you! When do you expect this to go to production?

Hi
Wasn't aware of any restriction on the length there, so we'll to check on this one. We had been planning one more 51.0 beta, next week, and then the production release hopefully 1-2 weeks later.
thanks


Docs?

Any interest in web documentation?

Docs sound good
On Tue, Jul 18, 2017 at 4:25 PM, Christopher J. Wright <
<EMAIL_ADDRESS>wrote:

Any interest in web documentation?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mrocklin/streams/issues/36, or mute the thread
https://github.com/notifications/unsubscribe-auth/AASszFiaVrooTU50aypAVY-pq6X9Lxp-ks5sPRS-gaJpZM4Ob4Jw
.

I'm going to use doctr if that is ok.

Why not readthedocs?

I guess I'm more familiar with doctr. They also enumerate a couple of reasons to use it over readthedocs.

Neither of the reasons listed seem very motivating to me:
You are limited in what you can install in Read the Docs.
Read the Docs deploys to readthedocs.io. Doctr deploys to GitHub pages. This is often more convenient, as your docs can easily sit alongside other website materials for your project on GitHub pages.

Some reasons I like readthedocs

It has been around for a long time and has become something of a standard
It handles latest and stable versions of documentation, and has nice history about build failures
It is maintained by dedicated developers

However, if I'm going to push readthedocs then I should probably also be willing to put in some effort to set things up.
I suspect that in both cases we're just going to be pushing rst or md files to a docs/ directory in the master branch, yes?

Ok, yes if you don't mind I've not used readthedocs before. Yes, both are standard docs/ based things. As a side note one thing that I've been looking at doctr to do is render latex documents for collaborators. This way there is a CI based standard so even if you are missing a library or some other resource you can always get your paper rendered.

Ah, right, so it looks like I started down this path before but stopped because I ran into the naming issue.  What do we call this thing?  A few classes of options:

streamspy or some not-yet-taken alternative involving streams and python
Some novel name
dask-streams (which sort of forces me to make a dask-powered version at some point)

I have a slight inclination towards 3, mostly because it avoids creativity

I personally like 3, since it means that we might get a dask-powered version at some point (which I'm happy to help out on, its on my todo list but its not so near the top). I know @ordirules was working on getting the streams name on pypi since the existing library seems to be defunct.

I contacted the owner of the name by email but no response: https://github.com/ztane. I sent out this email about 2 months ago.
The package has had no activity since over two years ago.
I suggest going higher up. I have not done so, but I could try to ask in pypi if it's possible to change ownership of a library and delete it and start fresh again.

I wasn't sure how to go about this, so I asked a general question [on the forum] (https://sourceforge.net/p/pypi/support-requests/742/). Is there a better way?

If we're going with dask-streams then I don't think we need to pursue the
streams name
On Thu, Jul 20, 2017 at 9:14 AM, Julien Lhermitte<EMAIL_ADDRESS>wrote:

I wasn't sure how to go about this, so I asked a general question [on the
forum] (https://sourceforge.net/p/pypi/support-requests/742/). Is there a
better way?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mrocklin/streams/issues/36#issuecomment-316699047,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AASszCpgXFqqqEHkRSO6VoDPyodYNUZQks5sP1KlgaJpZM4Ob4Jw
.

ok, great. so where will it land? (dask-streams.github.io ? )

Naming discussion moved to #35

I have alternately used readthedocs and doctr for a couple projects. I think readthedocs is perfectly good for pure-Python projects like dask-streams, and it might be nice to stick with it for consistency with the rest of the dask mini-ecosystem. I would be glad to help with documentation-writing, wherever it lands.

I'm closing this issue, as it was really about the creation of web docs. However, the push for writing user docs continues!


Frontier Exploration Demo for Noetic

Can you please explain why it was removed for Melodic and what needs to be done to get it running for Noetic?

At the time the Husky was upgraded to Melodic, the frontier_exploration package was not usable with Melodic.  Rather than delay releasing the Husky update, or releasing it with a broken dependency, we made the decision to remove it completely.
Since that time, there does appear to be a melodic branch of frontier_exploration available here: https://github.com/paulbovbel/frontier_exploration/tree/melodic-devel.  I have not tested this branch on Noetic, and don't know if it will work or not.
It appears the package has undergone significant refactoring since the Indigo/Kinetic version.  You're welcome to make use of it, but there's limited support we can offer for getting it to work on a current, Noetic Husky.


Cargo xtask install does not support offline installations

I am currently sitting in the airport with broken wifi and this is annoying me.

I'd say manual work-around is the right approach here.


Update for the guidance to TeamSite automount

@nenonix @JohanFreelancer9 Copy editing is needed for this PR. Thanks.


Revert gitwcsub tracking branch for arrow.apache.org to asf-site

Our website deploys from asf-site have stopped working likely on account of this.
See

https://issues.apache.org/jira/browse/INFRA-18987
https://issues.apache.org/jira/browse/INFRA-18914

Let me know when the deploys from the asf-site branch might start working again


Update Readable/WritableNativeArray

Fixes #20256
mUseNativeAccessor is a static property of the ReadableNativeArray class. This fix works by introducing an instance variable iUseNativeAccessor which determines whether the instance should always make native accesses. The array will make native accesses if either  mUseNativeAccessor or iUseNativeAccessor is true. WriteableNativeArray then sets iUseNativeAccessor true in the constructor.
The main problem I faced here is how to make the read methods read natively in the WriteableNativeArray subclass. You can't override variables/static variables. One option I explored was to create a method shouldUseNative which is overridable. However, I thought the performance hit would be too large from requiring a method invocation on every read.
Happy to put more work into fixing this. Let me know what the best way to do things is.
Test Plan:
Re-ran https://github.com/skaplan/thirddebug/ which was used to reproduce the issue. Output is correct.
Release Notes:
Help reviewers and the release process by writing your own release notes. See below for an example.
[ANDROID] [BUGFIX] [WriteableNativeArray] - Fixed bug that cached reads don't respect new writes.

@dcestari @emilsjolander @cjhopman

Thanks for the PR. I'm sorry we didn't get back to you earlier. We are actually planning on getting rid of this code "soon" so we aren't planning on including this change right now. I hope you'll understand and that the new native module architecture will delight you! See https://github.com/react-native-community/discussions-and-proposals/issues/40


[BUG] The annotation "@mixin" in class Hyperf........ was never imported. Did you maybe forget to add a "use" statement for this annotation?

Execute the command and paste the result below.
Command: uname -a && php -v && composer info | grep hyperf && php --ri swoole
Darwin JJiMac.local 18.7.0 Darwin Kernel Version 18.7.0: Tue Aug 20 16:57:14 PDT 2019; root:xnu-4903.271.2~2/RELEASE_X86_64 x86_64
PHP 7.3.20 (cli) (built: Jul  9 2020 23:55:30) ( NTS )
Copyright (c) 1997-2018 The PHP Group
Zend Engine v3.3.20, Copyright (c) 1998-2018 Zend Technologies
    with Zend OPcache v7.3.20, Copyright (c) 1999-2018, by Zend Technologies
hyperf/config                        v2.0.9  An independent component that provides configuration container.
hyperf/config-aliyun-acm             v2.0.9  An aliyun acm adapter for Hyperf config component.
hyperf/constants                     v2.0.9  A constants component for hyperf.
hyperf/contract                      v2.0.9  The contracts of Hyperf.
hyperf/crontab                       v2.0.9  A crontab component for Hyperf.
hyperf/db                            v2.0.3
hyperf/di                            v2.0.9  A DI for Hyperf.
hyperf/dispatcher                    v2.0.3  A HTTP Server for Hyperf.
hyperf/event                         v2.0.3  an event manager that implements PSR-14.
hyperf/exception-handler             v2.0.3  Exception handler for hyperf
hyperf/filesystem                    v2.0.8  flysystem integration for hyperf
hyperf/framework                     v2.0.3  A coroutine framework that focuses on hyperspeed and flexible, specifically use for build microservices and middl...
hyperf/guzzle                        v2.0.3  Swoole coroutine handler for guzzle
hyperf/http-message                  v2.0.3  microservice framework base on swoole
hyperf/http-server                   v2.0.9  A HTTP Server for Hyperf.
hyperf/logger                        v2.0.3  A logger component for hyperf.
hyperf/pool                          v2.0.3  An independent universal connection pool component.
hyperf/rate-limit                    v2.0.3  A rate limiter implemented for Hyperf or other coroutine framework
hyperf/redis                         v2.0.3  A redis component for hyperf.
hyperf/server                        v2.0.9  A base server library for Hyperf.
hyperf/signal                        v2.0.9  A signal library for Hyperf.
hyperf/task                          v2.0.3  A task library for Hyperf.
hyperf/utils                         v2.0.9  A tools package that could help developer solved the problem quickly.

swoole

Swoole => enabled
Author => Swoole Team<EMAIL_ADDRESS>Version => 4.5.2
Built => Jul 18 2020 11:58:40
coroutine => enabled
kqueue => enabled
rwlock => enabled
openssl => OpenSSL 1.1.1g  21 Apr 2020
pcre => enabled
zlib => 1.2.11
brotli => E16777223/D16777223
async_redis => enabled

Directive => Local Value => Master Value
swoole.enable_coroutine => On => On
swoole.enable_library => On => On
swoole.enable_preemptive_scheduler => Off => Off
swoole.display_errors => On => On
swoole.use_shortname => Off => Off
swoole.unixsock_buffer_size => 262144 => 262144

Description:
在启动时，遇到这个错误：
The annotation "@mixin" in class Hyperf\XXXXXX  was never imported. Did you maybe forget to add a "use" statement for this annotation?
Steps To Reproduce:

报错的类有时是 Hyperf\Di\Aop\AstVisitorRegistry， 有时是 Hyperf\Redis

https://hyperf.wiki/2.0/#/zh-cn/annotation?id=忽略某些注解

https://hyperf.wiki/2.0/#/zh-cn/annotation?id=忽略某些注解

谢谢！问题解决，我把~/config/autoload/annotations配置文件遗漏了, 所以造成了这个问题


Is there some kind of verbose logging in development?

I have a problem with menu ids sometimes when I change them o add a new id to the tree, the menu can stop working without any response/log/life-sign.  I would love to see some kind of response from my server why the submenu button does not work
This is my menu tree structure, is there something wrong with my ids?


Backport "bugfix: Completions for named args in wrong order" to LTS

Backports #18702 to the LTS branch.
PR submitted by the release tooling.
[skip ci]

No regressions detected in the community build up to lts-19520.
Reference

No regressions detected in the community build up to lts-19520.
Reference


TerminalMode 90 and 270 degree rotations incorrect

As reported in #47.
In TerminalMode, the text isn't rotated properly in these two rotations. Instead, it's just drawn backwards for 90 degrees:

And like this for 270 degrees:

0 and 180 degrees work fine.

Are graphics mode rotations correct? As far as I can tell by looking at the code, only the x and y coordinates are flipped, so I think the displayed image is horizontally flipped?

Closed by #129 I believe.


Missing half of answer 4.7

The answer to 4.7 is missing the definition of overflow.

Thx for contribution. @psigillito

Thanks man. I'm reviewing C++ and this is really helpful.  I'll add anything else if I notice it.  @Mooophy

Sweet.


(14) feat: alert-ui

Acceptance Criteria
Add a status line on top of the screen in "alert color" to set a visual mark for custom network. The status contains the network and stage value in the following format n=privnet • s=dev.

[!NOTE]
I didn't see need to alter the whole color scheme for the all other pages because the top bar was sufficient clear.

[!NOTE]
I edited some commits to remove module aliases.

Commits:

feat: add HslColor class
feat: add AlertUI theme
feat: add ToplinearBar component
feat: add NetworkStatusBar component
feat: add NetworkStatusBar to App component

https://github.com/HathorNetwork/hathor-wallet-mobile/assets/5992210/c5a936d0-9dc3-44cb-bc4c-9e549dce8711
Security Checklist

[ ] Make sure you do not include new dependencies in the project unless strictly necessary and do not include dev-dependencies as production ones. More dependencies increase the possibility of one of them being hijacked and affecting us.

I think we should use the same background color as in the other warning containers to keep consistency.
Also, can you update the video with the new UI?

Oh! Here I disagree:

The context for the other backgrounds are "warning", here it is "alert"
An Alert situation signal requires to call attention, that's why the color must be stronger and bright. We can't reproduce brightness well, but we can pick a stronger color that drags the eye to it

Also, can you update the video with the new UI?

Ok!

Rebased to resolve conflicts.

The alert component color is based on semantic colors of UI with 4 types : Error (red), success (green), warning (yellow) and information (blue). Alerts warning colors should be yellow, because based on the psychology of signals, yellow for alert, caution, warning. The colors draw attention and highlight crucial elements like buttons, CTAs, or notifications.
Based on the options that we have, we are going to base the decision on WCAG to achieve legibility and accessibility for the users. Another UX criteria are going to be: common sense and consistency.
Both options (strong yellow and light yellow) passes on WCAG contrast test, so it passes the accessibility and legibility criteria. Following common sense heuristics, the suggestion is to use the option dark yellow.
For practical reasons, we are going to prioritise the current yellow (light), so we keep the consistency of the app without having to change all of the others components colors.
We keep this way now, but in the future, following the UX components studies, these colors are going to be reviewed, so we can standardize it.

To complement @anafigueired comment:

light yellow: used in todays warnings
dark yellow: used in the Alert UI proposition

Decision applied at https://github.com/HathorNetwork/hathor-wallet-mobile/pull/371


TypeError: Cannot read property 'match' of undefined at Object.exports.createDeployment

I am encountering this error when running the workflow action. I'm able to manually create a deployment in AWS but not from the action as I get the following error when running the workflow. I'm not sure what I'm missing here so any help would be appreciated.
Run<EMAIL_ADDRESS>🎋 On branch 'aws', head commit c05d4429909487acca2b81f48117d78a2a14392a
💡 Using appspec.yml -> branch_config 'aws' for branch 'aws'
🎳 Using deployment group 'demo'
⚙  Updated deployment group 'demo'
(node:2404) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'match' of undefined
    at Object.exports.createDeployment (/home/runner/work/_actions/webfactory/create-aws-codedeploy-deployment/v0.2.1/dist/index.js:104:66)
    at processTicksAndRejections (internal/process/task_queues.js:93:5)
(node:2404) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:2404) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

My appspec.yml:
version: 0.0
os: linux
files:
  - source: /
    destination: /home/ubuntu/app
hooks:
  AfterInstall:
    - location: ./install-app.sh
      timeout: 300
      runas: root

# This section controls the action:
branch_config:
  aws:
      deploymentGroupName: demo
      deploymentGroupConfig:
          serviceRoleArn: arn:aws:iam::555036407157:role/code-deploy-role
          ec2TagFilters:
                - { Type: KEY_AND_VALUE, Key: name, Value: AppServer }
      deploymentConfig:
          autoRollbackConfiguration:
              enabled: true

My release-aws.yml:
# This is a basic workflow to help you get started with Actions

name: Release-AWS

# Controls when the action will run. Triggers the workflow on push or pull request
# events but only for the master branch
on:
  push:
    branches: [ aws ]

# A workflow run is made up of one or more jobs that can run sequentially or in parallel
jobs:
  # This workflow contains a single job called "build"
  build:
    # The type of runner that the job will run on
    runs-on: ubuntu-latest

    # Steps represent a sequence of tasks that will be executed as part of the job
    steps:
    -   uses: aws-actions/configure-aws-credentials@v1
        with:
            aws-access-key-id: ${{ secrets.AWS_ACCESS_KEY_ID }}
            aws-secret-access-key: ${{ secrets.AWS_SECRET_ACCESS_KEY }}
            aws-region: us-east-1
    -   uses: actions/checkout@v2
    -   id: deploy
        uses<EMAIL_ADDRESS>    -   uses: peter-evans/commit-comment@v1
        with:
            token: ${{ secrets.GITHUB_TOKEN }}
            body: |
                @${{ github.actor }} this was deployed as [${{ steps.deploy.outputs.deploymentId }}](https://console.aws.amazon.com/codesuite/codedeploy/deployments/${{ steps.deploy.outputs.deploymentId }}?region=us-east-1) to group `${{ steps.deploy.outputs.deploymentGroupName }}`.

I am also getting this issue. @NPMoyer Were you able to resolve this?

I'm experiencing this, as well. Link to the build, if it's helpful: https://github.com/datamade/la-metro-councilmatic/runs/1501527170

Has there been a successful deployment before, or was this the very first deployment attempt?
If there was a previous deployment, could you obtain the deployment description (probably the API method is called GetDeployment?) with the AWS CLI?

Hi, @mpdude – Yes, we've had successful deployments with your action, which is great, by the way – thank you! The description of the last successful deployment with the action prior to the error is Created by webfactory/create-aws-codedeploy-deployment (run_number=11). Here is the corresponding build: https://github.com/datamade/la-metro-councilmatic/runs/1500953012.
Interestingly, deployments via action are working again as of this morning.
I'm not sure what would be useful for debugging, but my workflow when I encountered the error was:

Implement a new feature on a branch with no corresponding deployment configuration in appspec.yml
Manually deploy that branch via the AWS Console to our staging site for QA
Post review, merge the branch into master, configured to deploy to staging in appspec.yml
Deployment fails with TypeError https://github.com/datamade/la-metro-councilmatic/runs/1501527170

Let me know if there's any more information I can provide to help out!

Ah, @NPMoyer, I bet this is the issue you resolved in https://github.com/webfactory/create-aws-codedeploy-deployment/commit/0d7a684950dae16883a140dd950257958730e0b5! Looks like that commit is not in your latest release: https://github.com/webfactory/create-aws-codedeploy-deployment/releases/tag/v0.2.1. Do you have plans to cut a new release with the fix, perchance?

@mpdude Thanks for the tip! Didn't realize you could pin an action to a commit – done in https://github.com/datamade/la-metro-councilmatic/pull/684. So far, so good on deployments. Will update this thread if this error recurs (or open a separate issue if we see a new one).

@hancush In fact, when you're ~a bit paranoid~  more security conscious, it would be a good practice to always review the code shipped by an action and then pin it by commit.  Just sayin'...

@NPMoyer I've tagged v0.2.2, hoping that it will resolve your issue.
If it does, could you please close this issue here?
Thank you all for your patience and helping to resolve this!

Hey @mpdude it seems it was tagged as "0.2.2" and not "v0.2.2" since I am getting this error:
Failed to resolve action download info. Error: Unable to resolve action<EMAIL_ADDRESS>unable to find version v0.2.2
It did fix the issue though!

@NPMoyer fixed the release tag (added the correct v0.2.2, but kept the wrong 0.2.2 to avoid breakages).


Latest version requires ping

Latest version requires the ping node module, this is not mentioned in the docs.

Fixed with ea59ac4


PalletteRibbon does not know about Office365 elements

e.g.
public enum PaletteRibbonShape

And there are other places within the ribbon code (e.g. border sizes) that are not covered by office365.

Will look into it, might be a case of adding them in

Yes only Office2013 appears


Improve error message

This one

There is already a TaskGroupV2 with (pkg_name='fractal-tasks-core', version='1.2.0', user_group_id=5).

should at least become

There is already a TaskGroupV2 with (pkg_name='fractal-tasks-core', version='1.2.0', user_group_name="whatever").

And possibly even more user-friendly, like:

User group "whatever" already owns a task group with name "fractal-tasks-core" and version "1.2.0".

In order to fix #1889 as well, we could add a newline saying:

If these tasks are not visible in your task list, it may mean that another task collection is in progress.

In order to fix #1889 as well, we could add a newline saying:

If these tasks are not visible in your task list, it may mean that another task collection is in progress.

As discussed with @ychiucco, we can improve this message and make it clear, since we can actually find out whether a task collection is in progress or not.


Improve logging

Related to https://github.com/austin-taylor/VulnWhisperer/issues/15

I am going to close both #15 and #11 in pro of this ticket (#78).
The objective of this ticket is improve overall logging on the vulnwhisperer project, including status of jobs and error logs in order to track failures on the project.
This will also be easier to track once python tests are implemented in order to make sure that every code addition is not breaking other code parts from the project.


Allow app to set Hive typeIds for at_persistence classes

Is your feature request related to a problem? Please describe.
See https://github.com/atsign-foundation/at_widgets/issues/827
Describe the solution you'd like
See https://github.com/atsign-foundation/at_widgets/issues/827
Describe alternatives you've considered
No response
Additional context
No response

Workaround proposed for now; closing


Add validation of ecsv file using a schema to application validate_file_using_schema

The application validate_file_using_schema allows to validate now:

yaml/yml files (existing functionality)
ecsv file with astropy tables (added functionality)

Examples are:
python simtools/applications/validate_file_using_schema.py \
    --file_name tests/resources/MLTdata-preproduction.meta.yml \
    --schema simtools/schemas/metadata.schema.yml

and
python simtools/applications/validate_file_using_schema.py \
    --file_name tests/resources/tests/resources/MLTdata-preproduction.ecsv \
    --schema tests/resources/MST_mirror_2f_measurements.schema.yml

Hi @GernotMaier. I havent checked yet the changes, but from the description I was wondering if we could point to the schema file already in the metadata of the file. Both ecsv and yaml allow metadata in the header. With that we would not need to point to a schema file, risking pointing to a wrong one.


Filewatcher ignore from .gitignore not working

Verify canary release

[X] I verified that the issue exists in the latest Turborepo canary release.

Link to code that reproduces this issue
https://github.com/gitbutlerapp/gitbutler
What package manager are you using / does the bug impact?
pnpm
What operating system are you using?
Linux
Which canary version will you have in your reproduction?
2.0.7-canary.1
Describe the Bug
A few times a day when trying to spin up my dev tasks (via turbo watch ..) it'll fail to start with the error message "fail to connect to daemon: server is unavailable: channel closed". After running daemon clear and/or daemon restart it usually works again for a while.
I finally noticed that turbo daemon logs will also tail the logs, so I ran this in the background while such an error occured. The error with turbo watch crashign always seems to occur while some Rust code is compiling and will log some output like this:
2024-07-16T12:24:41.025236Z ERROR turborepo_lib::daemon::server: package changes stream closed: channel closed
2024-07-16T12:24:54.905223Z ERROR turborepo_lib::daemon::server: package changes stream closed: channel closed
2024-07-16T12:24:57.837548Z  INFO turborepo_lib::daemon::server: triggering shutdown
2024-07-16T12:25:02.683079Z  WARN turborepo_lib::package_changes_watcher: changed_files: {AnchoredSystemPathBuf("crates/gitbutler-tauri/Cargo.toml")}
2024-07-16T12:25:02.683116Z  WARN turborepo_lib::package_changes_watcher: changed_packages: Ok(Some({WorkspacePackage { name: Root, path: AnchoredSystemPathBuf("") }}))
2024-07-16T12:25:09.210155Z  WARN turborepo_filewatch: encountered error watching filesystem enumerating recursive watch: IO error for operation on /opt/gitbutler/gitbutler/target/debug/build/atk-sys-9458f4fb2e56870d/rmetaahEQrj: No such file or directory (os error 2)
2024-07-16T12:25:09.231142Z ERROR turborepo_lib::daemon::server: package changes stream closed: channel closed
2024-07-16T12:25:40.004386Z ERROR turborepo_lib::daemon::server: package changes stream closed: channel closed
2024-07-16T12:25:45.043916Z ERROR turborepo_lib::daemon::server: package changes stream closed: channel closed
2024-07-16T12:25:48.147472Z  INFO turborepo_lib::daemon::server: triggering shutdown
2024-07-16T12:26:49.347045Z  INFO turborepo_lib::daemon::server: triggering shutdown

Notice the turborepo_filewatch warning line about no such file or directory

This is a Tauri project and the target/.. directory is the Rust compile output, so files are often being written and deleted quickly. This directory is already a part of our .gitignore. I also tried adding !target to the inputs array for the dev task. Unfortunately that didn't seem to ignore that directory either.
From what I understand, the directory should be automatically ignored by being in our .gitignore anyway. Am I missing something here? :thinking:
Expected Behavior
Turbo watcher doesn't crash on rust compiles.
To Reproduce

Clone https://github.com/gitbutlerapp/gitbutler
Install all pnpm dependencies
Ensure you have rust / Cargo setup
Run pnpm tauri dev in the root of the repository

Additional context
No response

Hi, thanks for the issue. We don't filter by .gitignore because the daemon needs to watch files that may be gitignored, such as your task outputs. We do this so we can detect if the task outputs haven't changed and therefore we don't need to do a cache restoration.
That said, clearly such an error shouldn't crash the package changes stream. I'll look into that and also see if we can add some gitignore filtering to the package changes watcher.

@NicholasLYang thanks for getting back to me! If .gitignore content isn't ignored by default, you may also want to reword some of the documentation here.
Also, can you confirm whether or not the inputs: ["!directory"] is supposed to work as I'm expecting? According to the docs you should be able to ignore certain files/directories from acting as inputs and being watched by the filewatcher. Is that right? I tried this as well, but it also didn't seem to work


Set nonLinear size to player size

nonLinearAdSlotWidth and nonLinearSlotHeight must be equal to player size. Otherwise, modal ads (below the picture) not returned because player.height() / 3. This is a nonLinear ads, its not like it seems.
Additionally, if nonLinearWidth and nonLinearHeight is provided like this:
var options = {
  id: 'content_video',
  adTagUrl: 'http://pubads.g.doubleclick.net/gampad/ads?sz=640x480&' +
      'iu=/124319096/external/ad_rule_samples&ciu_szs=300x250&ad_rule=1&' +
      'impl=s&gdfp_req=1&env=vp&output=xml_vmap1&unviewed_position_start=1&' +
      'cust_params=sample_ar%3Dpremidpostpod%26deployment%3Dgmf-js&cmsid=496&' +
      'vid=short_onecue&correlator=',
  'nonLinearWidth': player.width(),
  'nonLinearHeight': player.height()
};
player.ima(options);

and changed player size with video.js api at runtime then these settings is not updated in the videojs-ima plugin.

You can provide these numbers via your settings passed into the plugin. The default should still be player width and 1/3 player height, and you can change as needed by passing in different numbers when initializing the plugin. See the "Additional Settings" section of our README.

Yes, i can provide these settings like my previous comment but this time when i change player size with video.js api, these settings not updated.
For example, my player initialized size is 360 x 280. If a user click the "expand mode" button then i change player size to 640 x 360.
If i left default settings then videojs-ima plugin initialized with nonLinearWidth=360, nonLinearHeight=93 and it cannot be display modal ads (like i said, modal ads technically nonlinear ads). This is a lost.
If i set these settings to initial player size (360 x 280) ima sdk can call modal ads (200 x 200). When a user expand the player (640 x 360), ima sdk call modal ads again 200 x 200 but it must be call more big modal ads like 336 x 280. It's lost too.
As for me we have 2 way to handle this case.

We can set nonLinear size to player size
We can provide an update function for nonLinear size settings.
This pull request use first way and it's no damage other nonlinear ads formats (like text ads).

OK I think the second option you mentioned would actually be better - the IMA SDK suggests a default of bottom-third size, and this change could cause unexpected behavior for anyone currently using the plugin (people only seeing bottom-third ads could suddenly see both bottom third and full slot). Also note that these settings only matter before you make your ad request - if you make your ad request and then change the settings, it will have no effect.

Ok Shawn, i am working on second option and closing this pr. Thanks.


Test regularizationtools v0.6

@jjstickel can you help with this update?

Sure. Some of the numerical minimization methods changed in RegularizationTools that are probably causing some slight differences in the results. I'll try to take a look sometime in the next few days.

@ChrisRackauckas I've made changes to work with RegularizationTools-0.6 in a local branch pulled from your branch here. However, I do not have permission to commit back to this repository. Do I push the branch to my fork and make a new pull request? This is my first time trying to contribute to pull request initiated by someone else.

@ChrisRackauckas, I created PR #108 to bring in my changes to your branch and this PR. Please review.


Enable training with quantization framework

Description
Enables training with the quantization framework.  Functionality is tested via integration test, will add more unit tests.
Check List

[x] New functionality includes testing.
[x] New functionality has been documented.
[x] API changes companion pull request created.
[x] Commits are signed per the DCO using --signoff.
[x] Public documentation issue/PR created.

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

Rolling upgrade failure not related to this PR


Organization Name
TopSport.com.au
Your Name
Peter Schutt
Your current position
Engineer
(Optional) Company logo
https://www.topsport.com.au/assets/images/logo_pulse.svg
(Optional) Use case
No response
Also using other Lambda Powertools languages?

[ ] Java
[ ] TypeScript
[ ] .NET

Thank you so much Pete!!


Added awesome chess

Adding awesome chess, list of chess related links.

Use proper titles for entries, rather than BOT 1, talk 1, etc.

Done!
Any other suggestion would be most valuable.

From the awesome manifest:

Creative Commons licenses are perfect for this purpose. We would recommend CC0. Code licenses like MIT, BSD, GPL, and so forth are not recommended.

Added.

Thanks!
Modified the Licence.


Make 'delete_local_ref' function public

@jechas01 Could you please also bump a version when/if this is merged?

@jechas01 I don't want to be annoying, but could you please give some update for this? :sweat_smile:


Star of Lakshmi

I would like to add Star of Lakshmi Calculator under shape section.
Please assign this to me.

Go ahead
Deadline 6 May,2021


add rmdir command

The rm command does not work while deleting directories.
Micropython 1.8.5 is throwing an "OSError: [Errno 21] EISDIR"
So I added the rmdir command with some tests.

I am currently writing a script which sync's a whole folder structure to a connected board, or can delete recursively folders, so it would be nice to merge this request ;)

Thanks for contributing the command!  Apologies I didn't pull it in because I actually added an rmdir command that's a little different.  I needed a 'forceful' directory removal command that completely wipes out a directory and all its child files.  It turns out this is kind of tricky with MicroPython because it doesn't have a lot of path functions and requires directories to be empty before deleting them.  However with a little bit of extra logic it's possible to add a forceful rmdir command.  I'll close this pull since the last commit for version 1.0 of ampy adds this forceful rmdir command.
Oh and if you're looking at copying a folder to a board check out the last commit for 1.0.  I just added logic to the put command so it can recursively copy an entire folder from the local computer to the board's filesystem.  It's not a proper sync in the sense that it removes files, but it should help more easily move folder structures to a board.


Lack of documentation

Are there more detailed instructions on how to set everything up and train a model to detect objects?

What codebase is this question referring to?

This resource has detailed instructions for setting up test environments to experiment with object detection models. The site includes a couple platforms that demonstrate the material, including Google Colab & GitHub.
The Tensorflow documentation is a formidable educational resource to get started with machine learning, worth looking over.
Hope this can be of some help getting started.

Are there more detailed instructions on how to set everything up and train a model to detect objects?

This resource has detailed instructions for setting up test environments to experiment with object detection models. The site includes a couple platforms that demonstrate the material, including Google Colab & GitHub.
The Tensorflow documentation is a formidable educational resource to get started with machine learning, worth looking over.
Hope this can be of some help getting started.


Changing address.address doesn't change other address attributes

I did an interview where I entered an address of 220 S. Shepherd Ave, Ironton MO, which is an Iron County, a class 3 county.  In the review, I changed my address to 43 S. Prigmor Avenue, which is in Joplin County.
After I changed it, address.address and address.city were updated, but address.county and many of the other attributes were not.

We need to re -geocode the address if the user edits it. Was this via the review screen? You can use the recompute modifier

Added to recompute:
- address_geocoding_reset
- trial_court.county
- trial_court_name


Add ability to curve screen (probably tessellation + vertex shader)

Mesh shaders aren't part of WGPU spec, so, falling back to a nice pre-generated plane


Create 0000-01-02-tsuccar.md

The next field helps you provide a description of the changes you made. Feel free to add a description of what you’ve accomplished so far. As a reminder, you have: created a branch, created a file and made a commit, and opened a pull request

I'm merging the file after approval