Discord link does not work

I wanted to join discord to contribute but, I could not. Can you regenerate link ?

Thanks for the heads up @Musa-Sina-Ertugrul! Link has been updated in README and docs: https://discord.gg/nT54yATCTy


why run so slowly after update to lightgbm<IP_ADDRESS>

Description

Reproducible example

Environment info
LightGBM version or commit hash:
Command(s) you used to install LightGBM

Additional Comments

Thanks for using LightGBM.
You might find these suggestions helpful: https://github.com/microsoft/LightGBM/issues/5348#issuecomment-1175728860
But "why is my code slow" is not enough information for us to help you. If you can provide a minimal, reproducible example showing how you're using LightGBM, how you're measuring timings, and what specifically you mean by "so slowly", we'd be happy to help.
But without that effort from you, we won't spend time investigating this report.

thanks, my code used lightgbm,and now i update the lgbm's version, lgbm.cv removed some para,and the doc not info about these para,.when i use the new lgbm,run my model with same data and para,i feel so slowly.

That is not even close to enough information for us to help you, sorry. Please review the information at the comment I linked above.


example_vbwkde.cfg unable to be run through pipeline.py

When I try to run pipeline.py with the example_vbwkde.cfg I get the following error:
Stevens-MacBook-Pro-2:pisa steven$ python pisa/core/pipeline.py --pipeline pisa/resources/settings/pipeline/example_vbwkde.cfg --outdir OUTPUT/ProfileScans/MPHYS/ --png -v
PISA running in double precision (FP64) mode.

[ WARNING] Directory "OUTPUT/ProfileScans/MPHYS/" already exists
[   ERROR] Error occurred computing outputs in stage reco / service vbwkde ...
Traceback (most recent call last):
  File "pisa/core/pipeline.py", line 686, in <module>
    pipeline, outputs = main(return_outputs=True) # pylint: disable=invalid-name
  File "pisa/core/pipeline.py", line 598, in main
    outputs = pipeline.get_outputs(idx=stop_idx)
  File "/Users/steven/IceCube/PISA/pisa/pisa/utils/profiler.py", line 92, in profiled_func
    return func(*args, **kwargs)
  File "pisa/core/pipeline.py", line 261, in get_outputs
    outputs = stage.get_outputs(inputs=inputs)
  File "/Users/steven/IceCube/PISA/pisa/pisa/utils/profiler.py", line 92, in profiled_func
    return func(*args, **kwargs)
  File "/Users/steven/IceCube/PISA/pisa/pisa/core/stage.py", line 571, in get_outputs
    nominal_transforms_hash=nominal_transforms_hash
  File "/Users/steven/IceCube/PISA/pisa/pisa/utils/profiler.py", line 92, in profiled_func
    return func(*args, **kwargs)
  File "/Users/steven/IceCube/PISA/pisa/pisa/core/stage.py", line 470, in get_transforms
    transforms = self._compute_transforms()
  File "/Users/steven/IceCube/PISA/pisa/pisa/stages/reco/vbwkde.py", line 977, in _compute_transforms
    self.characterize_resolutions()
  File "/Users/steven/IceCube/PISA/pisa/pisa/stages/reco/vbwkde.py", line 1238, in characterize_resolutions
    feature, weights=weights, **vbwkde_kwargs
  File "/Users/steven/IceCube/PISA/pisa/pisa/utils/vbwkde.py", line 282, in vbwkde
    evaluate_dens=True, evaluate_at=None
  File "/Users/steven/IceCube/PISA/pisa/pisa/utils/vbwkde.py", line 163, in fbwkde
    min_bandwidth=min_bandwidth
  File "/Users/steven/IceCube/PISA/pisa/pisa/utils/vbwkde.py", line 410, in isj_bandwidth
    disp=True
  File "/Users/steven/.virtualenv/standard/lib/python2.7/site-packages/scipy/optimize/zeros.py", line 438, in brentq
    r = _zeros._brentq(f,a,b,xtol,rtol,maxiter,args,full_output,disp)
  File "/Users/steven/IceCube/PISA/pisa/pisa/utils/vbwkde.py", line 538, in fixed_point_numba_loops
    k0 = _K0[s]
IndexError: index 6 is out of bounds for axis 0 with size 5

Seems to me to be not so much an error with that particular cfg as it is with the vbwkde itself.

closing since it didn't seem to bother anyone for years now..


object-curly-spacing doesn't support objectsInObjects

as stated in the eslint rule

It was probably added after we wrote ours. Thanks for the issue!

fwiw @buzinas it's not accepting either objectsInObjects or arraysInObjects

It should now thanks to @peterjwest (https://github.com/buzinas/tslint-eslint-rules/commit/24190b7059b8c947d7df5b731434566fdd6b5468)


Include xpath & severity from axe

Axe includes the xpath in their csv reports, but Purple Hats does not.
Xpaths are useful for identifying where there may be multiple instances across multiple pages. They are also very useful for finding the exact reference which was causing the error.
I also do think that the Critical, Serious, Moderate & Minor impact is useful. You split yours up into the WCAG vs Deque best practices which is useful to. However it should be possible to capture this:
https://github.com/dequelabs/axe-core/blob/develop/doc/issue_impact.md
I also liked how you had in earlier versions what types of disabilities were affected by a given SC.  That was useful.
More data that may be usefu to folks:
https://github.com/CivicActions/accessibility-data-reference

Thanks @mgifford, considering your request.  It does make sense to add XPath and perhaps axe-severity rating to the CSV report for advanced users 👍

Hi @mgifford, we have added xpath and axeImpact rating to the CSV report in https://github.com/GovTechSG/purple-a11y/releases/tag/0.9.40

this is great. Thanks!


Adding 1.16 to Makefile

Description of changes:
Allow 1.16 version builds for EKS Optimized AMI
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm


add support for repairing Sonarqube warnings with sonarqube-repair

See https://github.com/kth-tcs/sonarqube-repair

My task :) . Related to the thesis

closed per #1066 Thanks Henry!


patch to fix saving error related to missing new_objects attribute

Hi, following patch fixes an error in Django 1.8.8 where saving any change causes a missing attribute new_objects traceback.
diff --git a/mongodbforms/documents.py b/mongodbforms/documents.py
index 0a432a8..0397c2d 100644
--- a/mongodbforms/documents.py
+++ b/mongodbforms/documents.py
@@ -703,9 +703,16 @@ class BaseDocumentFormSet(BaseFormSet):
Saves model instances for every form, adding and changing instances
as necessary, and returns the list of instances.
"""
+
saved = []

   self.new_objects = []

   self.changed_objects = []

   self.deleted_objects = []

   for form in self.forms:

       if not form.has_changed() and form not in self.initial_forms:

       changed = form.has_changed()

       new = form not in self.initial_forms

       if not changed and new:
           continue
       obj = self.save_object(form)
       if form.cleaned_data.get("DELETE", False):

@@ -715,6 +722,11 @@ class BaseDocumentFormSet(BaseFormSet):
# if it has no delete method it is an embedded object. We
# just don't add to the list and it's gone. Cool huh?
continue

           self.deleted_objects.append(obj)

       elif new:

           self.new_objects.append(obj)

       elif changed:

           self.changed_objects.append(obj)
       if commit:
           obj.save()
       saved.append(obj)

Sorry that patch is mongodbforms. I'll put it there as well.


Cookies with SameSite=None need to be set with Secure too

Browsers like Mozilla Firefox and Google Chrome start to inform the user that cookies that are set to SameSite=None need to be set to Secure as well. This is currently not the case in the PWA so one gets a lot of warning logs in the browser console for the cookie handling. This needs to be adapted.
Firefox

Cookie “apiToken” will be soon rejected because it has the “sameSite” attribute set to “none” or an invalid value, without the “secure” attribute. To know more about the “sameSite“ attribute, read https://developer.mozilla.org/docs/Web/HTTP/Headers/Set-Cookie/SameSite

Chrome:

A cookie associated with a cross-site resource at http://x.y.z/ was set without the SameSite attribute. A future release of Chrome will only deliver cookies with cross-site requests if they are set with SameSite=None and Secure. You can review cookies in developer tools under Application>Storage>Cookies and see more details at https://www.chromestatus.com/feature/5088147346030592 and https://www.chromestatus.com/feature/5633521622188032.

Note

a new cookies library could be used to address this issue:
https://github.com/stevermeister/ngx-cookie-service#set-name-string-value-string-expires-number--date-path-string-domain-string-secure-boolean-samesite-lax--strict--none--void
Server Side Rendering support is necessary (https://github.com/stevermeister/ngx-cookie-service/issues/1)

This is not an issue with cookies set by the PWA itself as it only concerns cookies you intend to use as third-party cookies on other websites.
Some cookies from google analytics and youtube lead to this warning but this is out of our control as you are only responsible for cookies from your own domain.
There is an issue filed with youtube concerning this: https://issuetracker.google.com/issues/151754736

Firefox is still complaining, just for the record. We need to keep this in mind I guess.


flipperzero: I2C interface

This adds a Rust API for using both the external and internal I2C bus with 7-bit addresses.

For now this just uses the helper interfaces that the Flipper Zero SDK provides for 7-bit addresses, but those are all themselves implemented in terms of the underlying 10-bit-capable raw interface which is also exposed. So if that's desired then we could re-implement the helper methods in Rust. However, furi_hal_i2c_is_device_ready is only exposed for 7-bit addresses.

Force-pushed to use Duration instead of u32.

Force-pushed to remove the convenience methods from i2c::Bus.

At some point in the future I'll experiment with 10-bit interface support, either once I have a 10-bit device to test with, or as part of adding embedded-hal support for #113. But this is fine to merge for now.


Dynamic whatsapp contact us fixed button

It seems like the whatsapp contact us button is too big for mobile view, how about do we make it more dynamic using bootstrap grid?

fixed on branch https://github.com/garasijogi/mamoto-website/tree/fix/jumbotron-and-portfolio with this commit https://github.com/garasijogi/mamoto-website/commit/8a563eb7e562c1a1499fd242c620b06a8a5728c5


Laravel 7.x update

This PR updates the Package for compatiblity with Laravel 7.x
Create new release v7.0.0 for compatibility with the corresponding PR on lucid-architecture/laravel.

@Mulkave @harris21 Are there any issues with this?


Added Lwaftr benchmark

src/apps/lwaftr/benchmark.lua: Added benchmark. Send a packet during n seconds. Aftr conf file and input pcap file with packet are passed as parameters. Settings of NICs should be adjusted according to aftr configuration file. Run as:

sudo ./snabb snsh apps/lwaftr/benchmark.lua \
   ../tests/apps/lwaftr/data/binding.table \
   ../tests/apps/lwaftr/data/icmp_on_fail.conf \
   ../tests/apps/lwaftr/data/tcp-frominet-bound.pcap

src/apps/lwaftr/lwaftr.lua: Put print and debug messages under a flag. Print out messages to the console affects benchmarking.
src/apps/lwaftr/lwutil.lua: Refactored print functions to dump + print.

I'm happy with benchmark.lua. I'm against adding a 'log' mechanism of the sort this patch introduces; these are short-term debug messages, and having that overhead makes no sense in the near future.

Printing out messages to console slows down the benchmark. I cannot progress in this task while there are messages printed out in stdout. In addition, some print messages are already guarded by a debug flag, why this logging mechanism is a worst solution? I can keep this commit on my local branch and keep progressing on the benchmark task but I wouldn't like to have to modify the source code again is new print messages are introduced. So, if you wouldn't like to integrate the patch, could you propose a solution to avoid printing logging messages?

I've turned debug off by default, made sure the tests still pass, and made all prints conditional on debug. If you modify the patch to just include the benchmarking code, I'll merge it. Thanks!

Pushed a 550-byte size version of 'tcp-frominet-bound.pcap'. Performance is almost the same as with normal version of the packet:
sudo ./snabb snsh apps/lwaftr/benchmark.lua ../tests/apps/lwaftr/data/binding.table ../tests/apps/lwaftr/data/icmp_on_fail.conf ../tests/apps/lwaftr/data/tcp-frominet-bound-550.pcap

Processed 0.2 million packets in 5.01 seconds (11620350 bytes; 0.02 Gbps)
Made 651 breaths: 255.00 packets per breath; 7690.58 us per breath
Rate(Mpps): 0.033

Looks better. I'd like to see usage info, and for pcidevA to be a parameter rather than hardcoded.

Updated PR with suggested changes.

Thanks! This is a quite useful first step in benchmarking. (We'll need to test with more varied packets at some point; this benchmark presumably paints an overly rosy view because the traces are always taking exactly the same conditionals, etc).


(MODULES-4648) Prevent zombie procs from timeout

Prior to this commit if a timeout occurs while waiting for the
connection to a named pipe then a zombie process is left on the
system as the code to close or kill it is never called. This commit
ensures that such zombie processes are removed before the error is
raised.
This commit also adds an RSPEC test to validate that if we try to
create a manager with an unreasonably low timeout value, that the
manager will not have time to open the pipes server and it will
fail to create the manager and further verify that no zombie
processes are left running.
Paired-with: Bill Hurt<EMAIL_ADDRESS>

Whoa... @RandomNoun7 paired with himself on this one @michaeltlombardi !

Poo. I kinda thought it would be a little more obvious that this was just a copy of @michaeltlombardi 's original PR. Guess not.

@RandomNoun7 ➕ ➕ for self-pairing!
This code looks good.... suspiciously good....

Ad-Hoc passed, merging.


Update documentation site

[ ] Update roadmap (out-of-date)
[ ] Update use cases with small examples with code snippets
[ ] Add an announcements tab or section - include new news
[ ] Add twitter link
[ ] Replace updated logo
[ ] Update dashboard image

Please update gearpump version  from 0.7.1-SNAPSHOT --> 0.7.6

See https://issues.apache.org/jira/browse/GEARPUMP-28


ACM-4277: docs: Add L2Advertisement CR to the Handling Ingress section in the agent docs

What this PR does / why we need it:
Fix docs
The last update to Handling Ingress section in create agent cluster (#1836) replaced AddressPool with IPAddressPool, unlike AddressPool IPAddressPool is used only for the ip assignement part. In order advertise the IP we should create L2Advertisement.
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #
https://issues.redhat.com/browse/ACM-4277
Checklist

[x] Subject and description added to both, commit and PR.
[x] Relevant issues have been referenced.
[x] This change includes docs.
[ ] This change includes unit tests.

/retitle ACM-4277: docs: Add L2Advertisement CR to the Handling Ingress section in the agent docs

/assign @davidvossel

e2e-kubevirt-aws-ovn is broken


x/tools/gopls: specify Options.LangVersion when invoking gofumpt

gofumpt's API has an option to specify the Go language version to target: https://pkg.go.dev/mvdan.cc/gofumpt/format#Options
Unfortunately, this seems to not be wired into gopls: https://github.com/golang/tools/blob/e9a2c0706a5b9cc8ee5965d5a50c996ffb255a7b/gopls/internal/hooks/hooks.go#L26
This results in gofumpt having to fall back to assuming the code has to comply with Go 1.0 as a language, not enabling rules which affect newer language features such as octal numbers.
This should be an easy fix, I think. When run from the command line, gofumpt uses the version specified in go.mod, so gopls could do the same.
cc @stamblerre @ianthehat

If possible, this would be a good opportunity for a general mechanism to wire passing flags through gopls to formatters (so, for example with gofumpt, -extra).

This was done in #51327, so closing this as a dupe.

Huh, apologies for filing that twice.


The front page looks bad on amd RX460

I was reading the webgl 2 fundamentals normally and I thought the frontpage looked normal in my desktop machine... then I tried the frontpage on my laptop and realized that it looked better: the floating meshes (2s) had black spots in them that make look kind of bad in the desktop machine.
My desktop machine (where the frontpage looked bad) is:
cpu: Intel 930 i7
gpu: Amd RX460
ram: 8GB
Os: Windows 10 and ubuntu linux (yes both)
Browser: Chrome and Firefox (yes both, in both OSs)
My laptop
OS: Windows 10
Browser: Chrome and Firefox
cpu Intel core i3-4000M
gpu: integrated
ram: 2GB
Screenshot from my desktop machine:

I know it has to do with my hardware but i want to know how to fix it and this is the best place I can find to get a fix, and is weird that the problem persist in all systems (browsers and OSs).

try it now - might have to clear your cache or turn it off in your browser's dev tools
I don't have an AMD to test on unfortunately

Now it works, thanks.

Thanks for reporting it. It was a pow(negativeNumber, ...) issue which I know better but didn't notice.


Multi endpoint support

This PR supercedes #3654
Adds the initial layer of multi-endpoint functionality by extracting out host information to a separate Endpoints model from ExtManagementSystem.
As the initial work on more work to come, this PR is backwards compatible with the pre-multi-endpoint ManageIQ utilising delegates and virtual columns. You shouldn't notice a thing ;)
Please check this @blomquisg @Fryguy

K, I found it for real this time.
The problem is that the down migration is looking for endpoints with a specific role and resouce_type.  But, the spec doesn't create the endpoint_stub instance that way.
I tried this locally and it led to a number of cascading changes.  If I can get collaborator access to the branch, I can push my changes and you can see what I did.

The problem is that the down migration is looking for endpoints with a specific role and resouce_type. But, the spec doesn't create the endpoint_stub instance that way.

Ah thanks @blomquisg that makes sense.

@juliancheal @Fryguy @blomquisg  do you have a dot image before and after?  I forget the details.  Maybe we should put it in the description of the PR.

I can put one together.

I'm testing this out locally now.  So far, I've validated:

adding an OpenStack provider (just default credentials, no AMQP)
adding a RHEVM provider (both default and C&U credentials)
inventory collection for both providers

I've noticed two things that I'm trying to track down:

validating credentials on the "Add New Provider" page creates an Authentication record

I'm checking this on master to see if this is an existing problem or something introduced here

MetricsCollectorWorkers for both providers start, but don't seem to do anything (perf_capture is never called)

this is likely specific to my dev environment, but I'm not sure what's causing the problem

MetricsCollectorWorkers for both providers start, but don't seem to do anything (perf_capture is never called

Make sure the authentication is validated... remember, we don't start ems based workers unless the auth is valid.

Make sure the authentication is validated... remember, we don't start ems based workers unless the auth is valid.

Thanks, I'll double check.  I'm pretty sure they were valid, but worth a second look for sure.

The only other thing I noticed was that the ApiController specs didn't change due to the changes in provider (verify_ssl, url, etc).  We should just verify that those API values, if they were being returned previously, were not broken by this PR.  cc @abellotti

@juliancheal I also noticed your verify last commit message (cd2d8ba) has WIP in it...can you remove that?

validating credentials on the "Add New Provider" page creates an Authentication record
I'm checking this on master to see if this is an existing problem or something introduced here
EDIT This is happening on master ... I added an issue.

This is getting fixed on this PR https://github.com/ManageIQ/manageiq/pull/6377

@blomquisg please review

YAY @juliancheal Thanks for getting this over the finish line :trophy: :runner:


Quest 2 tracks fine, nothing being streamed. Stuck in white void.

Description

On my Oculus Quest 2, ALVR connects fine, tracks fine (at least in SteamVR Home on my monitor), but the screen in the headset is stuck. No video is being streamed to it, just stuck in the white void. In the server logging, I noticed this error:
error in encoder thread: Failed to find encoder libx264
General Troubleshooting

[X] I carefully followed the instructions in the README and successfully completed the setup wizard
[x] I read the ALVR Wikis here and here

Environment
Quest 2
ALVR (v18.2.1, Linux portable version)
Hardware
Hardware Probe Link
CPU: Ryzen 7 4800H
GPU: GTX 1660 Ti Mobile
Audio:...Laptop Speakers?
Installation
ALVR Version: 18.2.1 (alvr_server_linux_portable)
SteamVR Version: 1.21.11
Install Type:

[ ] Packaged (exe, deb, rpm, etc)
[X] Portable (zip)
[ ] Source

OS Name and Version (winver on Windows or grep PRETTY_NAME /etc/os-release on most Linux distributions):
openSUSE Tumbleweed (All packages updated to latest at time of issue and posting)

You need to install the x264 dependency it seems.

Install libx264. If you have nvidia control panel (Nvidia X Server Settings), go to VDPAU information and check if your gpu supports H264.
Also, run steam with your dedicated gpu, as you seem to have integrated graphics. (With prime-run steam).

I was finally able to get past the "Stream will begin soon" message by finding and placing all required libraries in my ~/lib64 directory, but now I am stuck in a green screen void. I feel so close to Linux gaming, but I need help to get past this last (I assume) obstacle.

I'm having the exact same issue, and possibly due to flatpak steam, not sure though.

Could you try native steam not flatpak?

@ColdIce1605 I would really not like to, since normal Steam dumps all its contents all over my filesystem. It also helps that I can simply drag and drop my ~/.var/app/com.valvesoftware.Steam onto a new Linux installation, and have everything work as if on a  previous system, which is exactly what I did after nuking my original Tumbleweed install for unrelated reasons...
Also, although not applicable to me since I run Tumbleweed, users of distros like Fedora Silverblue and openSUSE MicroOS with immutable filesystems have an even harder time setting up dependencies for this, having to reboot for each and every update.
I hope that, someday, users like mentioned above will be considered for this project.

One problem, ALVR cannot work with how flatpaks guarantees security.

At least for this project, what does flatpak specifically do to "guarantee" security? If the answer is file isolation, then technically that would by a non-issue because one could instead modify the RPATH of the executable and portable libraries to point towards a folder containing all other runtime sub-dependencies just for ALVR. In fact, that is exactly what I have done to have ALVR even start inside of Steam's flatpak folder.

sandboxing.

It is still a problem for me, unfortunately.

I have been packaging it for Solus from source, because I wanted to include it in its repos too. I tried the nightly many times but stopped trying some time ago, because there hasn't been any update on this issue.
Was there any commit that was supposed to fix it?


Possible bug when using union types

I have the following code (available here):
type SquareState = {
  value: number,
};

type InternalSquare = {
  value: number | null,
};

export function getMax(squares: Array<InternalSquare> | Array<SquareState>): number {
   return squares.reduce(function(a : number, b : InternalSquare | SquareState): number {
    if (b.value === null) {
      return a;
    } else {
      return Math.max(a, b.value);
    }
  }, 0);
}

Which is generating this error:
 9: export function getMax(squares: Array<InternalSquare> | Array<SquareState>): number {
                                                             ^ number [1] is incompatible with null [2] in property `value`.
   References:
   2:   value: number,
               ^ [1]
   6:   value: number | null,
                        ^ [2]

As far as I understand, there should be no error, because there's a guard against null. The code works fine with either:
export function getMax(squares: Array<SquareState>):  number

or
export function getMax(squares: Array<InternalSquare>): number

But not with Array<SquareState> | Array<InternalSquare>
Also, if I replace the reduce with a for loop, it works:
export function getMax(squares: Array<InternalSquare> | Array<SquareState>): number {
  var max = 0;
  for(var i = 0 ; i < squares.length ; i ++) {
     if (squares[i].value === null) {
       continue;
     } else {
       max = Math.max(max, squares[i].value);
     }
  }
  return max;
}

Is this a bug, and if not, what am I missing?

This probably can be closed.
/cc @vkurchatkin


.NET core and ASP.NET

This repo doesn't accept issues. Please see README.md in the root for more details.
We un a report of vulnerabilities and find the following vulnerabilities;
Security Updates for Microsoft .NET core and ASP.NET (Bypass) (July 2018)
Security Updates for Microsoft .NET core and ASP.NET (DoS) (July 2018)
Security Updates for Microsoft .NET core and ASP.NET Core (DoS) (September 2018)
Security Updates for Microsoft .NET core (May 2022)
Security Updates for Microsoft ASP.NET Core (December 2021)
Security Update for .NET Core (February 2022)
Security Updates for Microsoft .NET core (June 2022)
Security Updates for Microsoft .NET Core (August 2022)
Security Updates for Microsoft ASP.NET Core (September 2022)
Security Updates for Microsoft .NET Core (September 2022)
Question:
By installing the latest update, would this fix all previous vulnerabilities?
Thanks
Luis

I think you should not create an issue in a public announcement repository for .NET. (although it's possible 🤔)
Regarding your question:
The latest updates include all previous security fixes, but I recommend to read the announcements for them for details 😉
Sometimes there are packages that need an upgrade too. (e.g. one of the latest security announcements, e.g. NuGet.CommandLine)


Add function to change/fill request page required filter fields and properties

Could you please add a function that makes it possible to change/fill the requested filter fields after opening the request page?

I want to fill the fields from a setup table when running the report.

Change the properties of the report, e.g. disable save values.

Hi! This issue is beyond the scope of the AL language and development tools. Please go ahead and post this to our Ideas forum at https://aka.ms/BusinessCentralideas, or vote up the idea if its already there. We're constantly monitoring top Ideas and will consider them for a future release.


Add Made with Lush section to Readme

Probably just a short paragraph linking to /made_with_lush/readme.md.
Themes sorted alpha numeric.
Possible contribution guidelines:

--- or  ### theme name
link in form user/repo
max ... 50? word description ("Melange is a dark color scheme with a warm, moderate contrast color palette. It supports tree-sitter and regular highlighting (works with Vim8)."
max two images

one showing some consistent code block (part of lush that wont change much?), for more consistent comparison between themes?

might not be realistic

one showing anything else
some reasonable max size (gh caps readme at 900px wide)
named <gh_user_name>_<gh_repo>_1/2.png

a8008d0ff39d9eecc14f63cb099cbf1fd1978ce8


save function when updating the missing product is not working

Current behavior

Expected behavior

Steps to reproduce the issue

Can you handle fixing this bug by yourself?

[ ] YES
[ ] NO

Environment details

Browser:
OS:
Code Version:

Additional information

POo0012.csv
Uploaded this CSV for dev-hc, got result of 1 missing product. I mapped it with the right SKU, it moved to the completed tab but did not get saved.


Where exactly are these "mentioned" samples?

Advanced .NET with WebAPI, OS calls, PowerShell calls... I'm wanting to see the example that used Swagger and WebAPI locally that was shown in the demo video on YouTube.  Is this published somewhere?

Unfortunately we had to drop the "Advanced .NET Sample" around native dialogues and powershell access, etc, due to limited resources. We still have the project and who knows, at some point we might revive it.


Just updated to 2.1.7 but I can't find anything about this release in the repo.

Just got a quasi-suspicious "user needs confirmation before executing code in local library" pop-up. So I went to see what the latest version's code is to verify it's legit.  I cannot find anything about 2.1.7 in this repo. Can someone point me in the right direction?

From: https://marketplace.visualstudio.com/items?itemName=dbaeumer.vscode-eslint

@dbaeumer

Will be merged in soon.

What if I have multiple projects in the workspace that all have their own eslint installations?
projects/
  projectA/ # eslint installed here
  projectB/ # eslint installed here

The prompts only ask to use the global eslint installation, or asks to use one of projectA/projectB's eslint for the whole workspace. It doesn't ask to use A's eslint to lint A, and B's eslint to lint B. is this possible?

Will be merged in soon.

Appreciate it.  I use vscode-eslint constantly and it makes my job easier. Thank you and all the other contributors for the hard work they do.
I was really hoping to see a release tag or PR that I could diff to understand the changes and more easily scrutinize security. It's unfortunate that such a popular extension seems to get updated behind the scenes.

Please add an option to disable the prompts. Adding them is a significant change that I would have not expected on a patch release. And there is no mention in the release notes of an option allowing us to disable the prompts to restore the old behavior.

@tjohnsonBCM please see the discussions here on why that got added: https://github.com/microsoft/vscode-eslint/issues/1012#issuecomment-659097449
I have merged the changes into the release/2.1.x branch in this repository.


Gotify integration, integers in JSON PostAgent

Hi,
I'm trying to post alerts to Gotify and would like to dynamically specify a priority value. The Gotify API expects one for creating a message, but I'm having a problem specifying one.
My Gotify Poster agent is set up as follows:
{
  "post_url": "https://my.gotify.box/message",
  "expected_receive_period_in_days": "365",
  "content_type": "json",
  "method": "post",
  "payload": {
    "title": "{{title}}",
    "message": "{{message}}",
    "priority": "{{priority}}"
  },
  "headers": {
    "X-Gotify-Key": "{% credential gotify_access_token %}"
  },
  "emit_events": "true",
  "output_mode": "merge",
  "no_merge": "false"
}

The credential definitely works. If I remove the priority the agent works, but then I can't use this for different classes of notification. If the priority is still set, it sends it as a string, but the Gotify API wants an integer. I had a look here but as_object didn't work for me. This is what I get back with the priority field set:
{
  "message": "This is a test message",
  "title": "Testing",
  "priority": "8",
  "body": "{\"error\":\"Bad Request\",\"errorCode\":400,\"errorDescription\":\"json: cannot unmarshal string into Go struct field MessageExternal.priority of type int\"}",
  "status": 400,
  "headers": {
    "Content-Length": "148",
    "Content-Type": "application/json",
    "Date": "Sat, 04 Jan 2020 12:35:10 GMT"
  }
}

Any thoughts on how I can ensure that priority is sent via the post as an integer, not a string?

Hi,
this should convert the string to a number:
...
    "message": "{% assign message = message | plus: 0 %}",
...

The variable I'm using is called priority, so I swapped message for priority. Unfortunately that doesn't look like it's changing it, this is what I get back:
[
{
"message": "This is a test message",
"title": "Testing",
"priority": "8",
"body": "{"error":"Bad Request","errorCode":400,"errorDescription":"json: cannot unmarshal string into Go struct field MessageExternal.priority of type int"}",
"status": 400,
"headers": {
"Content-Length": "148",
"Content-Type": "application/json",
"Date": "Sun, 05 Jan 2020 13:11:04 GMT"
}
}
]

Woups, sorry about that, does this work?
...
    "priority": "{{priority | plus: 0 | as_object }}"
...

Hmmm... It looks like there's something about the way the JSON is structured in the editor. I'm off to bed now but will try to take another look tomorrow and will report back once I have something.

Press Toggle View and change "priority": "8" to "priority": 8

@jarlave I don't think that will work because the Agent configuration has to be valid JSON.

I testet it now and you are right. It only works if you "hardcode" the number in without an liquid syntax.

I found a solution and did more digging to try and find out what's going wrong. For anyone who's stuck, the thing to do is to create an agent that sets the priority value and message as normal strings. This can then be passed to a post agent that looks like this:
{
  "post_url": "https://my.gotify.box/message",
  "expected_receive_period_in_days": "365",
  "content_type": "json",
  "method": "post",
  "payload": {
    "title": "{{title}}",
    "message": "{{message}}",
    "priority": "{{priority | plus: 0 | as_object }}"
  },
  "headers": {
    "X-Gotify-Key": "{% credential gotify_access_token %}"
  },
  "emit_events": "true",
  "output_mode": "merge",
  "no_merge": "false"
}

The key part is the "priority": "{{priority | plus: 0 | as_object }}". It's not ideal but it works and this approach works in general for things that need integers.
From what I've been able to work out, the reason for this is that the text editor either adds quotes or needs quotes to validate the JSON.
I guess there are two reasonable fixes:

Support integers being added in the agent configuration editor.
Document the workaround so people don't trip up.

I would be inclined given discussions I've seen about a new version of Huginn, to suggest documenting the workaround and adding to the wiki. I'm happy to do so if that helps. Least I can do. Also I'm not good enough at Ruby or JSON to add integer support ;)

@stevelord Adding documentation about this to the Wiki would be great!
We added the as_object "hack" because Liquid was designed as a HTML template engine and we are kind of abusing it. Since the JSON syntax is as it is and Liquids default is to return strings, I don't think we have a lot of other options to enable use cases like this.


Transactions needed

Hi.
I tested your lib and it works perfectly and very simply, also in the try catch construct using async/await. Thank you!

I know that ADODB object can export three connection's methods: BeginTrans, CommitTrans and RollbackTrans.
ADODB interface
Is it possible to extend your lib with these three methods?
Thank tou in advance for your answer!

@nuintun great repo... since transaction seem to be added in December 2020 by #382 I wanted to ask if the current master can be published to NPM and this ticket closed.
Let me know how I can help - I did not see any "npm pulish" command inside this Repo / Actions so I am not sure if I can do something.

Hi.
Now I can't test the new feature for the transactions. I work on an another project. But for me the issue is closed.

Hi, no worries... I am somewhat also interested. since transaction could improve the speed of several sequential executes I am currently using on a project with npm and typescript.
I am missing the publish to npm-repo and could then test if working according Readme.


First hack at directly accessing the item network.

If the cursor is empty:
left-click takes one item.
shift+left-click takes on stack.
control+left-click takes everything.
If the cursor is not empty:
left-click dumps the cursor content into the network.

Fixed #54

Rebased.

Merged! All I changed was I cleaned up the tooltips, refactored the sprite generation logic and added the "deposit into network" button in. I also fixed a bug where the view couldn't handle invalid item names.


DRILL-8468: Drill doesn't perform drill.exec.storage.action_on_plugins_override_file action

DRILL-8468: Drill doesn't perform drill.exec.storage.action_on_plugins_override_file action
Description
The issue is caused by plugin registry refactoring: https://github.com/apache/drill/pull/1988
After this change, drill.exec.storage.action_on_plugins_override_file doesn't work as described in our documentation: https://drill.apache.org/docs/configuring-storage-plugins/#configuring-storage-plugins-with-the-storage-plugins-overrideconf-file
storage-plugins-override.conf is neither removed nor renamed on Drill restart.
Documentation
No updates
Testing
Manual

Rebasing should fix the CI here too.

Are the two removed null checks not needed?

@jnturton Yes, they are redundant.
        if (locatorPlugins != null) {
          bootstrapPlugins.putAll(locatorPlugins);
        }

It is redundant, because StoragePlugins#putAll  himself checks the argument nullability:
https://github.com/apache/drill/blob/15c446d7e8ebe6d97fc879103e1710d733318c6d/exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/StoragePlugins.java#L114-L117
      if (upgraded != null) {
        upgraded.putAll(locatorPlugins);
      }

It is redundant, because upgraded is just initialized with a constructor just a few lines above and not overridden.

@rymarm okay let's rebase, get clean runs and merge!


Unable to update - possible cache issue?

Checklist:

[ x] I updated to the latest version available (attempted and failed
[ x] I cleared the cache of my browser

Release with the issue:
latest version (1.3.3)
Last working release (if known):
1.3.2b
Browser and Operating System:
Windows 10 Edge and Chrome, Windows 11 Edge

Description of problem:

Javascript errors shown in the web inspector (if applicable):

Additional information:

On the 25 Sep 2021 the repository was relocated to a new GitHub account with the release of v1.3.4. If you currently have it installed using HACS you will need to go into HACS and remove the old custom repository (the three dots up the top right) and add the new repository and then update the card. The link below has been updated to the new location. Your existing configs will not be lost.
https://github.com/Makin-Things/bom-radar-card

I did actually re-add the repository using the new one in HACS and it still tried to install from the old one.  I don't know if that is a HACS issue or what.
Is there a cache in HACS that may need to be cleared?    Try restarting HA and then try the reinstall.

I'll uninstall, restart HA then reinstall and see if it triggers it to look in the right place.

That seems to have worked.  Removed the repository, rebooted and then readded the updated repository.
Thanks for your help.  I appreciate it.
I'll close this issue now


Fixes#592 Hamburger menu nav list and button fixed

Fixes Issue🛠
Closes #592
Description👨💻
The hamburger menu was not working on the component category page, after adding js files to the page it was working fine.
Type of change📄

[ ] Bug fix (non-breaking change which fixes an issue)

How this has been tested✅
The changes has been through tested by visiting one by one and using Chrome Developer tools
Checklist✅

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added demonstration in the form of GIF/video file
[ ] I am an Open Source Contributor

Screenshots/GIF📷

As told in the PR you closed to make the space same between Contributors and Contact for every pages, pls see and do the changes in this PR only. Not needed to close the previous PR or this PR now.

As told in the PR you closed to make the space same between Contributors and Contact for every pages, pls see and do the changes in this PR only. Not needed to close the previous PR or this PR now @pb0126

Can I know why are you closing the PRs again and again, as you can make any changes in the same PR @pb0126

I have discarded the commit as I have other issues fixed, they are also lining up with them.

Also, I want to know where is the problem coming, I have fixed the margin between the components and contact

You can check all the components category pages and figure out the problem @pb0126


branch info didn't update after refresh button clicked after changing remote url(git remote set-url xxx) except reinstall

GitLens Version: 8.1.1
VSCode Version: 1.20.0
OS Version: win7 64bit

Steps to Reproduce:

change remote url via terminal (git remote set-url origin xxx)
click "refresh" button , but braches information didn't update
reinstall gitlens and branches information are right

I can replicate that it won't refresh the branches after setting the url, because you'd need to do a git fetch <remote> first. Then GitLens should update the view appropriately.

@gemmi does that resolve the issue for you?

I am closing this issue because it needs more information and has not had recent activity.  Please re-open this issue when more details can be provided.
Thanks!


Test Helper Fails To Find Select Element

Version of x-select: 3.0.1
Ember Version / Ember CLI Version: 3.0.1
Ember Version: 2.12.0
Ember CLI Version: 2.12.1
Expected Behavior
The select test helper should be able to find the element when given the selector, and the test should pass.
Actual Behavior
The select test helper cannot find the select. It reports that it could not find any options in the select. https://gist.github.com/krazylegz/355ae971a8734b28337efb4e51c66b95
Steps To Reproduce:

Use the selecthelper in a test configured to pass.
Run ember test.

Ember Twiddle / Example repo / Failing test: https://github.com/krazylegz/emberx-select-test
Please create a reproduction of your bug with this ember-twiddle. This is extremely helpful and speeds up the debugging process.

Thanks for the issue & example repo! It looks like the example the select isn't rendered yet, so it can't select anything from it.

So this did raise the fact that the helper isn't throwing an error like it should be. So I'll get a PR up to fix that.
I was able to get the tests to pass by moving the select helper down into the andThen.


packer.nvim does not load typst.vim on Neovim with ft = { 'typ' }

Installing the plugin via the snippet below succeeds, but when you edit a .typ file, packer does not load the plugin.
require('packer').startup(function(use)
    use {'kaarmu/typst.vim', ft = {'typ'}}
end)

However, removing ft = {'typ'} does make it work, packer loads the plugin.
I'm on macOS Ventura 13.2.1, and my Neovim version is v0.9.0.

When you open a .typ file and then run set ft? does it  detect typst or sql?

It detects typst, both when loaded, and not loaded.

Oh, yes you should do ft = {'typst'}

I updated the README.

That worked, thanks.


CLI autocompletion (tab completion)

Is your feature request related to a problem? Please describe.
All daytona commands should get auto-completed on pressing Tab key.
Should also provide suggestions for custom inputs like command flags/usage/syntax/mandatory and optional args, a one liner for what the invoked command does etc.
It will also help new users in differentiating easily with the available manual and TUI modes for some of the commands like create.
Overall, makes the cli more user-friendly.
Describe the solution you'd like
Perhaps some modification in the existing cobra based implementation should be sufficient else samber/lo package can be used.
Describe alternatives you've considered
None or N/A.
Additional context
None

@abhishek818 There is an existing daytona autocomplete command, have you tried using it?
Please do so and then edit the issue title and description with that in mind and add a couple specific examples of what you feel like is missing to make sure I understood you

Closing this due to inactivity (and also because our stale bot ignores enhancement issues 😄 )


[WIP] Rewrite import insertion

This is my attempt at refactoring the import insertion #3947. I hope what I created here is somewhat in line with what was requested, it wouldn't surprise me .
common_prefix is a copy from merge_imports.rs so those should be unified somewhere, try_merge_trees is also copied from there but slighly modified to take the MergeBehaviour enum into account.
MergeBehaviour should in the end become a configuration option, and the order if ImportGroup probably as well?
I'm not too familiar with the assist stuff and the like which is why I dont know what i have to do with insert_use_statement and find_insert_use_container for now.
I will most likely add more test cases in the end as well as I currently only tried to hit every path in find_insert_position.
Some of the merge tests also fail atm due to them not sorting what they insert. There is also this test case I'm not sure if we want to support it. I would assume we want to? https://github.com/rust-analyzer/rust-analyzer/pull/5935/files#diff-6923916dd8bdd2f1ab4b984adacd265fR540-R547
The entire module was rewritten so looking at the the file itself is probably better than looking at the diff.
Regarding the sub issues of #3947:

#3301: This is fixed with the rewrite, what this implementation does is that it scans through the first occurence of groupings and picks the appropriate one out. This means the user can actually rearrange the groupings on a per file basis to their liking. If a group isnt being found it is inserted according to the ImportGroup variant order(Would be nice if this was configurable I imagine).
#3831: This should be fixed with the introduced MergeBehaviour enum and it's Last variant.
#3946: This should also be fixed
#5795: This is fixed in the sense that the grouping search picks the first group that is of the same kind as the import that is being added. So if  there is a random import in the middle of the program it should only be considered if there is no group of the same kind in the file already present.
the last point in the list I havent checked yet, tho I got the feeling that it's not gonna be too simple as that will require knowledge of whether in this example ast is a crate or the module that is already imported.

Does this fix #5913?

It seems to fix it, instead of erroring it turns the import into use token::TokenKind::{self::*, self};

There is also this test case I'm not sure if we want to support it. I would assume we want to?

It's OK to not support weird cases initially, if we don't do something completely crazy.

For config, I don't think we should handle it in this PR necessary, but it should work like this:

define

struct InsertUseConfig { ... }

add it as a field to AssistConfig

add required user-visible options in package.json and ConfigData

thread them to AssistConfig in https://github.com/rust-analyzer/rust-analyzer/blob/master/crates/rust-analyzer/src/config.rs#L23

Almost all of the current test failures are due to the tests using check_assist which panics on an Option::unwrap in https://github.com/rust-analyzer/rust-analyzer/blob/7f79dbc84f5d69ec6f48fa8d67bdfa0525c5dd34/crates/assists/src/tests.rs#L84 which I assume comes from me not using the AssistContext/TextEditBuilder anymore.

bors r+
:tada:


Add Tigera to community page

This change is

Need you to sign the new CLA.

@devin-donnelly It claims I already signed it.

You are already authorized to contribute code to this project through your membership with CNCF - Tigera.

@devin-donnelly OK - CLA has been signed.

@devin-donnelly looks like the partners section has been removed in master.  Any clues as to where it went?

@caseydavenport I'm not sure how that commit got merged; I know I merged no such thing. I also don't see a PR for it. I'll try and investigate.

@devin-donnelly thanks for the context :)
I've re-opened this here: https://github.com/kubernetes/kubernetes.github.io/pull/1581
please take a look!


COMPRESS-404: Use java.nio.Path in TarArchiveEntry

I wan't to use the NIO API more in my application and therefore would need some additional methods which support the NIO API. Found this already existent issue and thought this would be a good starting point.
The first commit handles the internal migration to the Path object. The second adds additional information to the TarArchiveEntry. Before adding more functionality (e.g. symlinks) I would like to get some feedback for the error handling. Since the old File constructor didn't throw an exception the now delegating constructor also shouldn't throw an exception. But with NIO most file access operations are throwing one.

Coverage decreased (-0.09%) to 87.137% when pulling de37805ca1b58805c784d8e4a7b56be8c352bbf4 on theobisproject:COMPRESS-404 into 69de512db43c9ca35da11664a1502702353a6fdd on apache:master.

The thrown exception of NIO seems difficult to handle if you are not throwing an exception. Haven't got any good ideas.

Is there anything that would prevent us from throwing the IOException from the old constructor.
If we for example can't get the information of the file because we have not enough rights to access it we would now get an exception which the user can handle. I haven't checked what the old File API does in this case (fail silently or a runtime exception?).

Is there anything that would prevent us from throwing the IOException from the old constructor?

I think throwing new IOException in newly added constructor is fine - as it's a new constructor, no existing code needs to be changed. For those existing constructors, existing code calling it may need to be changed. Maybe we need a discussion in developer mailing list?

You must not break binary compatibility within a major version.

In the last commit I removed the delegation from the File constructor to the Path constructor so we keep the backwards compatibility.
Also this should address all previous made comments.

@bodewig @PeterAlfredLee Is there anything left I need to do to get this integrated?

This PR looks good for me now. WDYT @bodewig ?

Yep, looks good.

And thank you for your contributions @theobisproject .


Sort draw calls by mesh (inside material)

Otherwise we sometimes get cloned meshes being rendered without instancing (when instancing is enabled).
Useful even without instancing to reduce number of buffer switches.

Done in https://github.com/playcanvas/engine/pull/631


Add prettier as pre-commit hook

@pnicolli @nzambello as discussed here the PR
Fixes  #17

Approved and tested changes, @jaydp17 look on that eslint configuration and then we can merge this.

@nzambello check 7fa9f17
I hope that's what you were referring to


Google lighthouse complains about font-display property

Ensure text remains visible during webfont load
Leverage the font-display CSS feature to ensure text is user-visible while webfonts are loading.

Recently font-display: block; has been added in #21. I think block is better than swap for icon fonts as you don't want to see ligatures while font is loading. If you are using Sass, it can be changed using the variable $material-icons-font-display.


Allow removing all morph listeners

I had an issue with the library when I needed to do this, because the old listener wasn't removed properly for some reason.
I added this in my work and since then, the morph listeners work just as I want them to.
The thing don't want to have a global morph listener and re-add it or remove it when needed.
Thanks in advance

I'm 100% sure this an user error, but I've added it anyway in https://github.com/rubensousa/FloatingToolbar/commit/5381664a4da87be7b7d69708b80ea525d1de9ee2.


Fix #351 UDPMux read/write race condition

Improvements to interface flexibility, and IPv6 compatibility

Hey @Sean-Der, take a look specifically at https://github.com/pion/ice/pull/352/files#diff-07afbb40d3dc2699797d28c0689ba544f67ca642d0d3efcff3c02a0bbb3604fdR952
It's an attempt at fixing some strange issues that I had ran into; where the outgoing/incoming ports through the mux didn't match remote candidates. I believe this is due to findRemoteCandidate assuming all remote candidates are IPV4. Let me know your thoughts.


Undoing nil Manifest from newManifest

Consequent to changes in the way otpl flags are processed,
newTargetManifest could (effectively) return a nil Manifest. None of its
consumers expect this behavior, so I'm reverting it, ensuring that an
empty Manifest (with default DeploySpecs) is always returned.

This seems reasonable, just curious what led to this change/what does it solve outwardly in Sous?

There was a test that had been added (ironically, the couldn't-reproduce case that I added for the "-repo is ignored" bug) that was failing. The alternative was to add guards for the nil case in update (which has the opposite guard (I.e. for an empty, missing manifest) already), and to greatly expand or remove the test case from ci_test.
From: Sam Salisbury <notifications<EMAIL_ADDRESS>Reply-To: opentable/sous <reply<EMAIL_ADDRESS>Date: Thursday, January 26, 2017 at 04:45
To: opentable/sous <sous<EMAIL_ADDRESS>Cc: Judson Lester <jlester<EMAIL_ADDRESS>Author <author<EMAIL_ADDRESS>Subject: Re: [opentable/sous] Undoing nil Manifest from newManifest (#308)
This seems reasonable, just curious what led to this change/what does it solve outwardly in Sous?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/opentable/sous/pull/308#issuecomment-275380185, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAHyPL8KmtUDYKD_bbkYpuJdcdMjx2BTks5rWJV0gaJpZM4LuC_Z.

Ah, I see :)


[Improvement][Audit log] Audit log improvement design

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
1. Goal
Record user operation logs like create, update, delete, run, and stop jobs, etc. Easy to track item state.
1. Previous Design
https://github.com/apache/dolphinscheduler/issues/5822
1. Realize Method

Plan a: publish and subscribe mode . After user completes the modification operation, the operation behavior information is published as the form of message
Plan B: implementation using AOP

https://github.com/apache/dolphinscheduler/issues/5822#issuecomment-882023608

AOP can be used to get the relevant operation records in the form of log after the end of user operation, but it is not convenient to judge whether the operation is successful.
If use the publish subscribe mode , a message can be sent to the subscriber after the user's operation is successful.
If only need to record the successfully configuration modification, it will be better to use publish and subscribe mode.

Previous design choose public subscribe mode. The reason why is just want to record success operation. And AOP can support this function.
POC:
private void saveLog(ProceedingJoinPoint joinPoint, Object object) {

        Result result = (Result)object;

        // We can judge by this code. 0 means the api is successful, other codes means we can skip this request.
        System.out.println(result.getCode());
}

Publish and subscribe mode has high code intrusive, we need to add log method anywhere. So we thought AOP it's more better.
2. Schema Design

User Name
Resource Type
Project Name
Operation Type
Time

admin
PROJECT
NewProject
CREATE
2023-12-28 10:40:23

admin
USER
NewUser
CREATE
2023-12-28 10:40:23

2. Deficiencies
The recorded operation granularity is too rough. Not clear enough to show the level like workflow and project.
3. Classification
Project

Project (Create, Update, Delete)
Workflow (Create, Update, Delete, Import, Export, Copy, Start, Online, Offline)
Workflow Instance (Edit, Rerun, Stop, Kill, Pause)
Task (Create, Update, Delete, Move, Switch version, Delete version)
Task Instance (Force success)
Schedule(Create, Update, Delete, Online)

Resource

Floder (Create, Delete, Edit, Rename)
File (Create, Delete, ReUpload, Edit, Rename, Upload)
UDF Floder (Create, Delete, Edit)
UDF (Upload, Edit, Delete)
UDF Function (Create, Edit, Delete)
Task Group (Create, Update, Switch status)

Datasource

Datasource (Create, Update, Delete)

Security

Tenant (Create, Update, Delete)
User (Create, Update, Delete, Authorize)
Alarm Group (Create, Update, Delete)
Alarm Instance (Create, Update, Delete)
Worker Group (Create, Update, Delete)
Yarn Queue (Create, Update)
Environment (Create, Update, Delete)
Cluster (Create, Update, Delete)
K8s namespace (Create, Update)
Token (Create, Update, Delete)

4. Extract
We use mutili level of object like:

Level 1: Project.

Level 2: Workflow.

Level 3: Workflow Instance.

Level 1: Resource.

Level 2: Folder.

Level 3: File.

Demo:

Object Type

Project
Resource
Datasource
Security

Object Id/Enum

Workflow
Folder

Detail Id/Enum

Workflow instance
File

5. New Schema Design
Java Code Enum Design
    PROJECT(0, -1, "Project", true),
    RESOURCE(1,-1, "Resource", false),
    DATASOURCE(2,-1, "Datasource", true),
    SECURITY(3,-1, "Security", false),
    WORKFLOW(4,0, "Workflow", true),
    WORKFLOW_INSTANCE(5,4, "Workflow instance", true),
    WORKFLOW_INSTANCE1(6,5, "Workflow instance1", true);

    private final int code;
    private final int parentCode; // support multi level
    private final String name;
    private final boolean hasLogs; // if this object has not value, in search button, only can choose `All Sub-Levels logs`
    private int level; // project 0, workflow 1, workflow instance 2, task 2, task instance 3

6. UI design
Search field:

User

Object Type(Project, Security)

Scope(All Sub-Levels logs, current level logs).  If Object Type hasLogs is false, only can choose All Sub-Levels logs

Object Name

Operation Type

User Name
Parent Type
Parent Name
Object Type
Object Name
Operation Type
Detail
Time

admin
PROJECT
ds-test
PROJECT
ds-project
Create
v-project
2023-12-28 10:40:23

admin
Security

USER
NewUser
Delete

2023-12-28 10:40:23

admin
PROJECT
ds-test
Workflow
ds-workflow
Create
ds-workflow
2023-12-28 10:40:23

NewUser
Workflow
ds-workflow
Workflow-instance
Workflow-instance-1
Run

2023-12-28 10:40:23

NewUser
PROJECT
ds-test
Workflow
ds-workflow
Update
add new task...
2023-12-28 10:40:23

Database Design
CREATE TABLE `t_ds_audit_log` (
  `id` bigint(11) NOT NULL AUTO_INCREMENT COMMENT 'key',
  `user_id` int(11) NOT NULL COMMENT 'user id',
  `object_type` int(11) NOT NULL COMMENT 'resource type',
  `operation_id` int(11) NOT NULL COMMENT 'operation id',
  `operation_type` int(11) NOT NULL COMMENT 'operation type',
  `time` datetime DEFAULT CURRENT_TIMESTAMP COMMENT 'create time',
  `detail` text DEFAULT NULL COMMENT 'detail',
  PRIMARY KEY (`id`) /*T![clustered_index] CLUSTERED */
) ENGINE=InnoDB DEFAULT CHARSET=utf8 COLLATE=utf8_bin AUTO_INCREMENT=1;

Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

PTAL @caishunfeng @EricGao888 @Radeity @ruanwenjun @SbloodyS

Table data

Search data, you can multi choose object type

Search data about operation type

 Search data about operation type

@qingwli if user deletes 2 process defintions in a single batch-delete operation, will there be 2 audit records in the table? or just 1 record with 2 objects

 Search data about operation type

@qingwli if user deletes 2 process defintions in a single batch-delete operation, will there be 2 audit records in the table? or just 1 record with 2 objects

Will record in two lines

Compatibility, Deprecation, and Migration Plan

Compatible with current system, aspect all api method, but just create, delete, method will into record log logic, other query method will keep same. Just record log not change anything about api request.

It seems is not compatible with current system, since you need to change the schema in database.


appcenter distribute release returns non-json response on failure (with --output json option)

The following command appcenter distribute release returns a text response even with --output json option passed.
In my use case, the command throws an error because the app with the specified name doesn't exist, so am expecting a valid JSON response with the status code, so that appropriate action can be taken.
Sample command:
appcenter distribute release -f [PATH_TO_FILE]  --release-notes 'release_notes' -g 'Collaborators' --app 'org/app-name' --output json
Actual Response:
Error: failed to create release upload for [PATH_TO_FILE].ipa. Not found. Correlation ID: d7136eb9-6f2a-4c0f-98bd-xxxxxxxxx
Desired Response
{"message":"Not found. Correlation ID: d7136eb9-6f2a-4c0f-98bd-xxxxxxxxx","statusCode":404,"code":"Not Found"}

It is fixed in the 2.9.0 release.

@russelarms this also happens with the add command

Hi @varemenos , could you elaborate on your point? Why would you like to change the response format in case the deployment was added successfully?

@varemenos it is still unclear what exactly would you expect to receive as a result of success. Json data is displayed when there was requested some data. In this case the command codepush deployment add creates a deployment with success or failure. All such commands behave the same in terms of output.
If you still think this exact command doesn't output json while other similar commands do, please describe why and provide an example.
if you think all such commands that upload or create something should output a json output, please file a separate feature request.


updated the import statement in the method update_grade_entry_items

Motivation and Context

It upgraded the code to use a more recent means of bulk upserting and removed the need for the active-record import gem.

#5280
Your Changes

We adapted the method 'grade_entry_form.rb#update_grade_entry_items' to use 'upsert_all' instead of the 'import' gem. This is because of the introduction of bulk inserts/upserts with Rails 6. This change was done in collaboration with @JunruL
Type of change (select all that apply):

[x] Refactoring (internal change to codebase, without changing functionality)

Testing

We first tested the UI by creating a dummy assignment, then navigating to the grades section, and uploading a sample grades report (that we downloaded from a pre-loaded assignment). This action called the method 'from_csv' which called the helped 'update_grade_entry_items', which was the method we updated.
Checklist

[x] I have performed a self-review of my own code.
[x] I have fixed any Hound bot comments.
[x] I have verified that the TravisCI tests have passed.
[x] I have reviewed the test coverage changes reported on Coveralls.

Hey David, I think I closed in on the root cause. I noticed that (as you said) filtering of the Total column should be automatic. After some debugging and inspection I noticed that the filtering case was not executing as it would require show_total => true. However, as per default instantiation show_total was false which makes sense as we don't want the total to be included. So I changed the conditional being evaluated to be negated and that solved the problem much cleaner.


subtree-push nightly-2024-09-10

Bumping the toolchain version as part of a git subtree push
current toolchain (nightly-2024-08-17):

1.82.0-nightly (506052d49 2024-08-16)

latest toolchain (nightly-2024-09-10):

1.83.0-nightly (c2f74c3f9 2024-09-09)

Running the Diff Check


Failing purchased

Describe the bug
Hi, recently I tried to implement my IAB using Poolakey library but I think something's wrong here.
After a successful purchase in payment onActivtyResult method, right in the purchaseFailed callback I faced with an odd error message in localizedMessage:
purchase was hijacked and it's not a valid purchase
Expected behavior
I thought that after a successful purchase I should back to purchaseSucceed callback but I didn't. though all thing are sounds good.
Versions

Version of the library that your are using: 1.0.0-alpha03
API Version of the Android device that you have tested on: API 28

Hi,
There has to be an issue with your RSA public key, make sure that you have provided that correctly through PaymentConfiguration class. This error happens because of invalidity of the received data or inability to verify the data which in this case, it's probably something wrong with your RSA public key.

Thanks for your quick response. you were right, there were a mistake in providing RSA key.


Path to hm3-file mixes absolute and relative

Coord fails with the --only-hm3 option due to the path to the hm3-file mixing relative and abolute path elements.

Thanks Jakob, I fixed this now.
Cheers,
Bjarni


Stacked suggestions

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: phschiele:x: TransurgeonTransurgeon seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Support sparse vectors in atoti

Description
Support the concept of sparse vectors in atoti
Context
Some calculations such as IMA-DRC use large vectors of up to 1m elements, however most of the values may be 0 (or other value where they are all the same).  We have previously created custom SparseVector implementations to work around this in AP projects, and these feature in a number of our finance accelerators. It would be great if these were available in atoti.
Another use case is vectorised cashflows in liquidity projects which benefits from using sparse vectors.
Other information (if relevant)
You can search for sparse vector details on ActiveViam's confluence page for more information about this.

It would be even better to have it supported in the core ActivePivot !
Then I think we should always detect normal vectors but let the user force a sparse type if necessary.
Some links about it:

Confluence page about performance (slower but really less memory)
ReadOnlySparseVector implementation in Market Risk Accelerator

Fastest way to get something in the core is to ask for it in atoti 😉

That sounds true :eyes:

Indeed, this would need to be done in ActivePivot first.

Related internal issue: https://support.activeviam.com/jira/browse/PIVOT-4621


Performance improvements based in Web Framework Benchmarks

Improve performance for Web Framework Benchmarks tests that use ActiveJDBC:

db
fortunes
queries
updates

We can also generalize the current Model.toJson() implementation so it can be used with non-Model classes and use it as a faster alternative to jackson in the test:

json

keep in mind, that AJ ois not a bottleneck for these tests, so focusing on AJ might not make sense.

I could make the profiler connect remotely to the app server in an 3-server configuration (app, client, db) to run the benchmarks. But the profiling results were not precise as it could not profile the classes of the activeweb-benckmark implementation itself (the app.* packages).
Test fortunes
A profiling of the test (running from an unit test) showed it spends:

43.4% - 39.9% of the time with freemarker.* methods
41.9% - 38% of the time in the LazyList.hydrate() method, more specifically

25.3% - 21.6% of the time in DB.find() self time.

Optimization in the web part of the implementation can be achieved using a faster alternative to freemarker.
The db part comes from the Model.findById() method call, which could be improved though all its call hierarchy. For example, removing these lines from DB.find():
    if(query.indexOf('?') == -1 && params.length != 0) throw new IllegalArgumentException("you passed arguments, but the query does not have placeholders: (?)");
    if(!SELECT_PATTERN.matcher(query).find()) { throw new IllegalArgumentException("query must be 'select' query"); }

made it improve from 25.3% - 21.6% to 19% - 18.8% of the time. The verifications above are already done by the JDBC drivers. As a "pass-though" framework, and considering these verifications are somehow heavy and already implemented in the JDBC drivers (DRY?), we could remove them. What do you think, @ipolevoy?

@ericbn, let me understand. By removing these verifications, you improved from 25.35 to 21.65%? That is the improvement is less than 4%? If this is the case, I would rather look at a lower hanging fruit.
While the framework is pass-through, it needs to have  some value :) These messages are usful in guiding development in the right direction.

I agree the performance improvement is not so strong in this point (though, in the long run, a lot of time will be spent uselessly with these checkings, sometimes even checking queries created by ActiveWeb itself, as is the case for all the tests here).
But there are other factors: it's redundant, as the JDBC drivers already do all the checking. And they respond with the proper error to help the developer too. One common error I had at the begining was doing MyModel.find("WHERE ..."); and then I got errors from the db showing the query was "SELECT * FROM my_model WHERE WHERE ..." so I learned...

understand, but  with all the changes and performance reports, I lost track of where we are. Is it possible that you create a latest report of benchmark tests and top methods where time is going for each. This will allow us to see where we need to focus attention. I'd rather spend 20% of effort to get 80% of improvement, rather than the opposite.

Pushed LIMIT 1 optimization to my fork: https://github.com/ericbn/activejdbc/commit/165c3806c90e7cae4c5c2d1e4db367fa1b376320

so... the code is not as easy to follow anymore (performance improvements are known to cause this), but as far as I see, you removed the limit, and only reading a first result. Is this really going to improve speed??? I think that the DB is a lot better at optimizing these cases. Do you know for a fact that this is actually faster for all databases?

I think we need to refactor LazyList at some point, as it grew a little bit unordered.
The speed improved because MySQL takes more time to execute the query with "LIMIT 1" than without it. I just tested this for MySQL using the Web Framework Benchmarks in a Mac. Yeah, we should make sure this is faster in all DBMSs. We should setup our own benchmark tests and run them in our server.

maybe this is faster on MySQL. Question is how much faster? I want to make sure that writing obscure code is worth it

This fast: https://github.com/javalite/activeweb/issues/197#issuecomment-70635370

so... db test got twice as fast? that is good

Yeah. But I agree we should make sure this is correct for most or all DBMSs (and not worst for any) before using this solution.


feat(batch):  parallelize spill hash agg IO and decouple compute/io

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?

Related issue https://github.com/risingwavelabs/risingwave/issues/16615
Parallelize spill hash agg IO.
decouple compute and io for spill hash agg.

Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

I am a bit concerned about this enhancement, especially the channel between executor and IO tasks.
As we know, IO is usually slower than CPU, so it's likely that channels are completely filled peak traffic comes. Here is a rough estimation:
20      // DEFAULT_SPILL_PARTITION_NUM
* 8     // DEFAULT_IO_CHANNEL_SIZE
* 1024  // batch chunk size
* 1 KB  // assume each row is 1KB. Exterme cases can be much larger
* 32    // assume the CN has 32 cores, thus 32 HashAgg executors
= 5.24288 GB

Keep in mind that spill-out only happens when memory is insufficient to avoid OOM. Such a big memory usage can cause OOM. Apparently, stability is prioritized over performance under the theme of spill-out.

I am a bit concerned about this enhancement, especially the channel between executor and IO tasks.
As we know, IO is usually slower than CPU, so it's likely that channels get completely filled when peak traffic comes. Here is a rough estimation:
20      // DEFAULT_SPILL_PARTITION_NUM (recursive spill-out is not considered yet)
* 8     // DEFAULT_IO_CHANNEL_SIZE
* 1024  // batch chunk size
* 1 KB  // assume each row is 1KB. Exterme cases can be much larger
* 32    // assume the CN has 32 cores, thus 32 HashAgg executors
= 5.24288 GB

Keep in mind that spill-out only happens when memory is insufficient to avoid OOM. Such a big memory usage can cause OOM. Apparently, stability is prioritized over performance under the topic of spill-out.

I can set DEFAULT_IO_CHANNEL_SIZE to 1. Each HashAgg has a single input producer and 20 partitioned output consumers, so the bottleneck would shift from the consumer to the producer. To fully utilize each channel, the producer needs to be very fast. However, in most cases, the input task is not purely CPU-bound but also IO-bound. Therefore, I believe it is acceptable to decouple CPU and IO tasks, especially if we plan to spill out to an object store in the future.

Close due to inactivity. @chenzl25 Please feel free to reopen if needed.


Ensure rogue frameworks get removed

Description of Change

Issues Resolved

fixes #

API Changes

None
Platforms Affected

All
iOS
Android
UWP

Behavioral Changes

None
Testing Procedure

PR Checklist

[ ] Rebased on top of the target branch at time of PR
[ ] Changes adhere to coding standard

Thanks - This seems like a better answer to System.Drawing.Common than what I was originally told.
PS - I'm reserving the right to not fill out your PR templates in the future :)

This actually breaks the build.  I need some form of System.Drawing in there.  I either have to take modern System.Drawing.Common or Open-TK

ohhh i see, my fault you use it! this is if you don't use it.


SKCanvasView Always trigger redraw when invalidate is true

Fixes #261

Thanks. Tested it against various cases (with and without dependsOn) and it looks good!


Create a guideline how to run PipelineDP on AWS

PipelineDP runs on the top of Apache Spark. It would be great to have a guide (and/or examples) how to run Pipeline on AWS.

I want to work on that.

Thanks, go ahead!
Please let me know if you have any questions!


Improve closure variable detection

Purpose
Allow closure variable assignment from the scope which created it.

Closing as same changes will be introduced from https://github.com/ballerina-platform/ballerina-lang/pull/16647


[Feature Request] Locally run Stable Diffusion on X Elite devices

When I run the stable-diffusion example, it appears to be using S23 to directly upload to AI Hub. I haven't found a way to run the model locally on my device.

Thanks for the feature request. We're aware of this and are working on some samples scripts that will allow you to run models locally on the X Elite.

Thanks for the feature request. We're aware of this and are working on some samples scripts that will allow you to run models locally on the X Elite.

hi,  is there any demo yet?  I want to run model locally too


Cannot build 1.1.0 on OS X

I'm working on the latest OS X with clang version:
$ clang --version
Apple LLVM version 6.0 (clang-600.0.57) (based on LLVM 3.5svn)
Target: x86_64-apple-darwin14.1.0
Thread model: posix

cppformat 1.0 works fine but when I upgraded to 1.1.0 I get the following error:
Undefined symbols for architecture x86_64:
  "fmt::BasicFormatter<char>::format(fmt::BasicStringRef<char>, fmt::ArgList const&)", referenced from:
      fmt::BasicWriter<char>::write(fmt::BasicStringRef<char>, fmt::ArgList) in libshoguncore.a(Daemon.cpp.o)
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

The function is defined here and explicitly instantiated here if FMT_HEADER_ONLY macro is not defined. Please check that FMT_HEADER_ONLY macro is used consistently in the project. It should be either defined or undefined everywhere. Note that the macro can be defined by a third-party library like spdlog.

@ruipacheco Have you found out if the link error is indeed caused by inconsistent use of FMT_HEADER_ONLY? If not could you please post an example to reproduce the error?

Yes it is. I added the macro and everything went away.

On 13 Mar 2015, at 05:21, Victor Zverovich<EMAIL_ADDRESS>wrote:
@ruipacheco https://github.com/ruipacheco Have you found out if the link error is indeed caused by inconsistent use of FMT_HEADER_ONLY? If not could you please post an example to reproduce the error?
—
Reply to this email directly or view it on GitHub https://github.com/cppformat/cppformat/issues/133#issuecomment-78791678.

Thanks for the confirmation. Closing the issue now.

Had the same issue.
I someone else finds this post - here is my solution:
My FindFmt.cmake
# - Try to find fmt source lib
#
# Once done this will define
#  fmt_FOUND - System has args
#  fmt_INCLUDE_DIRS - The args include directories

find_package(PkgConfig)

find_file(fmt_INCLUDE_DIR format.h
        PATH fmt
        HINTS fmt
        )

include(FindPackageHandleStandardArgs)
# handle the QUIETLY and REQUIRED arguments and set LIBXML2_FOUND to TRUE
# if all listed variables are TRUE
find_package_handle_standard_args(fmt DEFAULT_MSG
        fmt_INCLUDE_DIR
        )

mark_as_advanced(fmt_INCLUDE_DIR)

# Avoids "Undefined symbols for architecture" Error
add_definitions(-DFMT_HEADER_ONLY)

set(fmt_INCLUDE_DIRS ${fmt_INCLUDE_DIR} )

include_directories(${fmt_INCLUDE_DIR})

Then in CMakeLists.txt
find_package(fmt)

Thats it!


Add a way to generically go back on both Material and Cupertino scaffold pages

I couldn't find a nice way to hit test unfortunately. Cupertino page transitions actually puts a gesture detector on top of the back button for detecting back swipes. We'd need to run the gesture arena, not just a hit test to see if anything is in front of it.
Needed to fix the smoke test breakage in #15324


implement configuration settings for the finished platforms (tvOS, macOS, watchOS)

tcOS and macOS covered, watchOS should use iOS framework.


Add output buffer (and termination?) for TX

Just wanted to log this issue here so that we can keep track of it.  I'm assigning @CansWang since he is the TX designer, and it would be great to hear thoughts from @sjkim85 as well because he designed the differential output buffer used in the digital core.

The output buffer in DCORE is just inverter chain for monitoring purpose and its output impedance is not matched at all.
I thought our TX contains output driver stage to drive channel. Am I right Can?
-Sung-Jin
Sent via the Samsung Galaxy S8+, an AT&T 5G Evolution capable smartphone
Android용 Outlookhttps://aka.ms/ghei36 다운로드

From: Steven Herbst<EMAIL_ADDRESS>Sent: Saturday, October 24, 2020 12:45:34 PM
To: StanfordVLSI/dragonphy2<EMAIL_ADDRESS>Cc: Sung-Jin Kim<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: [StanfordVLSI/dragonphy2] Add output buffer (and termination?) for TX (#137)
Just wanted to log this issue here so that we can keep track of it. I'm assigning @CansWanghttps://github.com/CansWang since he is the TX designer, and it would be great to hear thoughts from @sjkim85https://github.com/sjkim85 as well because he designed the differential output buffer used in the digital core.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/StanfordVLSI/dragonphy2/issues/137, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGWDJIT7I5M76VL3YVUJFDLSMMVF5ANCNFSM4S5ZR6GQ.

The output buffer in DCORE is just inverter chain for monitoring purpose and its output impedance is not matched at all. I thought our TX contains output driver stage to drive channel. Am I right Can?
…
-Sung-Jin Sent via the Samsung Galaxy S8+, an AT&T 5G Evolution capable smartphone Android용 Outlookhttps://aka.ms/ghei36 다운로드
________________________________ From: Steven Herbst<EMAIL_ADDRESS>Sent: Saturday, October 24, 2020 12:45:34 PM To: StanfordVLSI/dragonphy2<EMAIL_ADDRESS>Cc: Sung-Jin Kim<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: [StanfordVLSI/dragonphy2] Add output buffer (and termination?) for TX (#137) Just wanted to log this issue here so that we can keep track of it. I'm assigning @CansWanghttps://github.com/CansWang since he is the TX designer, and it would be great to hear thoughts from @sjkim85https://github.com/sjkim85 as well because he designed the differential output buffer used in the digital core. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub<#137>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGWDJIT7I5M76VL3YVUJFDLSMMVF5ANCNFSM4S5ZR6GQ.

The Tx output stage needs to be designed together with the termination and integrate into tx_top.sv.
There are essentially two ways of designing it (?)
since we want to minimize the delay and number of stages, also make the on resistance + termination to be closer to 50 ohm (single ended)

design this in virtuoso
synthesize it, (I am not familiar with this method), but we should be able to specify the fanout (parasitic at the output) and unit size (determined by sheet resistance of diffusion)

Correct me if anything doesn't make sense to you, thanks!

Closing as this was resolved in #146


[] build failure

Host Environment

OS:  windows
Compiler: visual studio 2019

To Reproduce
Steps to reproduce the behavior:
`./vcpkg install intel-mkl:x64-windows-static
Failure logs
PS D:\vcpkg> ./vcpkg install intel-mkl:x64-windows-static
Computing installation plan...
The following packages will be built and installed:
intel-mkl[core]:x64-windows-static
Detecting compiler hash for triplet x64-windows-static...
Starting package 1/1: intel-mkl:x64-windows-static
Building package intel-mkl[core]:x64-windows-static...
Could not locate cached archive: C:\Users\tang\AppData\Local\vcpkg\archives\88\8841d97f465e29d6f62b9b16d035945721e7ab66.zip
CMake Error at ports/intel-mkl/portfile.cmake:16 (message):
Could not find MKL.  Before continuing, please download and install MKL
(20200000 or higher) from:
  https://registrationcenter.intel.com/en/products/download/3178/

Also ensure vcpkg has been rebuilt with the latest version (v0.0.104 or
later)
Call Stack (most recent call first):
scripts/ports.cmake:79 (include)
Error: Building package intel-mkl:x64-windows-static failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with .\vcpkg update, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: intel-mkl:x64-windows-static
Vcpkg version: 2020.06.15-nohash
Additional context
I installed the intel parallel studio in "D:\IntelSWTools", however, the default locatioin in ports/iintel-mkl/portfile.cmake is set(INTEL_ROOT $ENV{${ProgramFilesx86}}/IntelSWTools/compilers_and_libraries/windows).
Even I set it to : set(INTEL_ROOT D/IntelSWTools/compilers_and_libraries/windows)
compiling is still error.

Hi @rockydut!
I tried to use your modification method to modify the portfile file, but the result is the same as you, I can’t find the installed MKL. I am curious why you don’t install it directly under the C drive. I installed it on the C drive and can successfully install intel-mkl :x64-windows-static, I looked at the portfile file, it seems to look for it in the C drive file by default.
PS F:\test4\vcpkg> ./vcpkg install intel-mkl:x64-windows-static
Computing installation plan...
The following packages will be built and installed:
    intel-mkl[core]:x64-windows-static
Detecting compiler hash for triplet x64-windows-static...
Starting package 1/1: intel-mkl:x64-windows-static
Building package intel-mkl[core]:x64-windows-static...
Using cached binary package: C:\Users\vzhli17\AppData\Local\vcpkg\archives\de\dee7e480e14f1f81ff55caec35b77e9b3b3f8c42.zip
Building package intel-mkl[core]:x64-windows-static... done
Installing package intel-mkl[core]:x64-windows-static...
Installing package intel-mkl[core]:x64-windows-static... done
Elapsed time for package intel-mkl:x64-windows-static: 263.1 ms

@rockydut ,can you try this?
Modify VCPKG_ROOT/ports/intel-mkl/portfile.cmake line 10, from
set(INTEL_ROOT $ENV{${ProgramFilesx86}}/IntelSWTools/compilers_and_libraries/windows)
to
set(INTEL_ROOT ${YOUR_MKI_INSTALLED_ROOT}/IntelSWTools/compilers_and_libraries/windows)

I'm closing this issue for now since there is no response.
Please feel free to reopen this if it is still a problem for you.


run on kubernetes  Pod Terminated with 139

CrashLoopBackOff (back-off 10s restarting failed container=rauthy pod=rauthy-0_rauthy(8fb376da-7437-4f5c-8916-e1cece8ccf39)) | Last state: Terminated with 139: Error, started: Thu, Oct 24 2024 5:19:22 pm, finished: Thu, Oct 24 2024 5:19:23 pm

kubernetes and os version
v1.26.13+k3s1
CentOS Linux 8
containerd://1.7.11-k3s2.26

Hmm... I carefully reviewed the documentation and checked the configuration in the config, and I missed LISTEN_SCHEME=http.
Perhaps it would be better to have a separate text block for required configuration fields in the documentation.

Perhaps it would be better to have a separate text block for required configuration fields in the documentation.

Usually, you only need to copy & paste the reference config and adjust it to your needs. It has all the mandatory and the likely-to-change variables out-commented by default.
I mention to copy the reference into place for both docker and kubernetes getting started, but I'm glad it's resolved.

Thank you for developing such a great project; I really like it!

Thank you for developing such a great project; I really like it!

Thanks! :)


implement reconstruct

use https://github.com/tsuraan/Jerasure/blob/master/src/jerasure.c#L1453.

@xiang90 looks like we need to hook up the server to initiate request to other servers for reconstruction right? looks like  you've already hook up jerasure, perhaps I can help doing the calls part

@tnachen Right. Basically we want to pull source files from other cfs nodes to /dev/shm (tmpfs), then reconstruct the target files onto disk.

@xiang90 can you add me to the org? I can't assign myself this

@tnachen We start to roll up the wheel again. Are you still working on this one?

@yichengq hey good to know! I have a WIP patch in my repo that I can try to finish this few days. I forgot about this a bit and did other stuff, I have time to work on this now so I can look at it.

@tnachen :+1:

@tnachen How is it going? I could work it with you if you meet problems with it. Or I can continue with your WIP patch too.

@yichengq sorry got occupied again earlier :( I do have some time but I think it's safer to give it to you if you're more available, or we can even work together. Yes please take over from my WIP patch


DOC: syntax changes for data access

minor documentation issue

Coverage decreased (-71.19%) when pulling 859922b0ee3d2e9b58ce968138443ff89ddd524c on stnava:master into 29fb561f38ec3e324fb1d770dd077514acd5d3aa on JuliaStats:master.

strange : the change is minor
https://github.com/JuliaStats/SVM.jl/pull/30/files
brian
On Wed, Jan 7, 2015 at 11:42 AM, Coveralls<EMAIL_ADDRESS>wrote:

[image: Coverage Status] https://coveralls.io/builds/1706099
Coverage decreased (-71.19%) when pulling 859922b
https://github.com/JuliaStats/SVM.jl/commit/859922b0ee3d2e9b58ce968138443ff89ddd524c
on stnava:master into 29fb561
https://github.com/JuliaStats/SVM.jl/commit/29fb561f38ec3e324fb1d770dd077514acd5d3aa
on JuliaStats:master.
—
Reply to this email directly or view it on GitHub
https://github.com/JuliaStats/SVM.jl/pull/30#issuecomment-69060004.

Coverage statistics are being fixed as we speak. Without any code change, you could get a coverage change.


Fix the instruction text in the Accessing Nested objects challenge

This commit fixes the instruction text in the challenge to
Use bracket notation for properties with a space in their name.
from
Only use object notation for properties with a space in their name.
I have removed the extra Only on purpose for sake of grammar.
closes #6422

:+1:


Add schema relations to postgres test

Closes #157 .
Signed-off-by: Hanif Bin Ariffin<EMAIL_ADDRESS>

My bad, updated the script @christoshadjiaslanis .

Thanks for taking the time to finish this off @hbina . Much needed! We should figure a way to automate the creation of the schemas/data for these tests.


Introduce option to execute go get commands sequentially

When environment variable GPM_GET_SERIAL is set, dependencies are downloaded sequentially.
This fixes issue #89.

Hey @philippthun !
Thank you so much for your contribution, I've been absolutely swamped with work lately and haven't had the time to consider this properly, but I am aware of it and grateful that you're both using my software and taking the time to improve it. :)
I can't give you an ETA, but will make time to give the issue a proper followup as soon as I can catch a break.
Thanks again! 🎉


Improve access modifiers

Mostly makes access more restricted. internal Drawable<->Container
composability overrides have been made sealed. Drawable.Hovering
has gotten a public getter and documentation.

:x: Build osu-framework 0.1.0-PullRequest.536+2212.build.1751 failed (commit https://github.com/ppy/osu-framework/commit/651486b4b1 by @Tom94)

:x: Build osu-framework 0.1.0-PullRequest.536+2212.build.1752 failed (commit https://github.com/ppy/osu-framework/commit/651486b4b1 by @Tom94)


Scaling issues

Fixed all the errors that I got. However, the main one appeared, which I do not know how to solve.

The picture has moved out and it is not possible to use the application correctly. However, there is enough control for my little pranks.

Which resolution are you running the tool with?

Where are the settings? Inside the application itself, they are not, at startup, too.

Unfortunately there is no resolution setting in the configuration menu. You can adjust the size of the window by dragging the bottom-right corner of the application, like so:

Please note that the minimum supported resolution is 600x400.


Hide / Disable Segment Control for external Items

Hi,
i use your Picker and it works great. There is one last thing, that i can not solve.
I use your Picker like:
let imagePicker = OpalImagePickerController() imagePicker.imagePickerDelegate = self         present(imagePicker, animated: true, completion: nil)
https://ibb.co/bLsoJ8
How can i hide or disable the Segment Control for external Items? I dont want to use external Items

You need to delete the optional delegate method imagePickerTitleForExternalItems. That determines whether or not the segmented control shows up on top.


BHM Target selection issues - CSC priorities+values to be updated

This is a reminder that the BHM CSC priorities+values are currently set to their v0 levels (high numerical priority, low value). This was appropriate for earlier iterations when CSC targets were not confirmed as forming part of 'core' SDSS-V science. However, CSC is now core, and so priorities should be adjusted accordingly.
Updates to target_selection code expected, but are pending outcome of discussions within the BHM + CSC teams.

Initial proposal is to set the following priority floor + deltas:

bhm_csc_boss_dark:
priority_floor: 1920
dpriority_has_spec: 1

bhm_csc_boss_bright:
priority_floor: 2920
dpriority_has_spec: 1

bhm_csc_apogee:
priority_floor: 2930


Resolving signing issue

Resolving signing issue

[ ] You've read the Contributor Guide and Code of Conduct.
[ ] You've included unit or integration tests for your change, where applicable.
[ ] You've included inline docs for your change, where applicable.
[ ] If any gains or losses in performance are possible, you've included benchmarks for your changes. More info
[ ] There's an open issue for the PR that you are making. If you'd like to propose a new feature or change, please open an issue to discuss the change or find an existing issue.

Summary of the changes (Less than 80 chars)
Description
{Detail}
Fixes #{bug number} (in this specific format)

working build: https://identitydivision.visualstudio.com/IDDP/_build/results?buildId=1327328&view=logs&j=fa1208be-7322-5e86-1637-d1f5bee81ceb&t=66c47e1f-c910-55e4-d14d-cdd2ccaa27ce


Investigate joint cloud events with Tekton

Describe the solution you'd like
Right now Keptn cloudevents are designed for keptn only. We want to work with the tekton team to agree on a see of shared events that can be reused.
Additional context
Issue in Tekton: https://github.com/tektoncd/pipeline/issues/742

Very exciting!!! 😻

We have to check out the catalog of tasks, i.e. https://github.com/tektoncd/catalog
Here we can find information on the inputs for tasks. These inputs are similar to Keptn's events.

What has been the outcome of this issue?

As we are actively working on standardizing events in CI/CD in the "Events in CICD Workstream" of the CDF Sig on Interoperability where I am a member, I am closing this issue.


[Feature]: Adapt AMS for Streaming-Inline-Upsert

Description

mysql add column to identity table type
one table need multi change. need multi change location

Use case/motivation
No response
Describe the solution
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Subtask of #256


Treat AutoResolveIntegrationRuntime as a default artifact

When creating a new Synapse Workspace, an Integration Runtime named AutoResolveIntegrationRuntime is automatically created.

The workspace manages the integration runtime in Azure to connect to required data source/destination or external compute in public network. The compute resource is elastic allocated based on performance requirement of activities.

@lordozb - Thoughts?

Hello @owenfarrell
Sorry for delay in getting back.
Changes will also be required in Here because integration runtime is of type Microsoft.Synapse/workspaces/integrationRuntimes.
Can you please also add some positive and negative unit tests Here

Hello @owenfarrell
Thank you for contributing to the project. We have asked our partner team which manages integration runtime to let us know if skipping autoresolve IR will cause any issue or not. We are waiting for their response.

@lordozb - What's the verdict?

Hello @owenfarrell
Apologies for delay in getting back. We can go ahead and mark it as a default artifact. However, due to the recent commit there has been some merge conflicts. If you can resolve the conflicts I can go ahead and merge the PR. If you are unavailable I can resolve the conflicts and merge your PR.

@lordozb - I can resolve the conflicts. But I have a couple questions about #28.

Did you just copy the DefaultArtifact class that I added here and paste it in to your PR? Or is there something genuinely different?
Is there a way to selectively disable the deployment of managed private endpoints? Your PR (as written) seems to duplicate some automation capabilities that exist elsewhere (namely https://github.com/hashicorp/terraform-provider-azurerm/pull/9260). And Terraform is adding more a la carte capabilities for Synapse artifacts, courtesy of @ms-henglu.
I guess my question really is: is this CI/CD plugin trying to complement other automation tools? or compete with them?

@owenfarrell Currently, the code deploys all the artifacts in the template. There is no way to do selective deployment as of now.

Closing this PR since the code was copied/committed elsewhere