The dialog message is inconsistent when deleting a VM resource group with different status

OS: All
Azure Resource Extension Build Version: 20220427.11 (unified)
Azure VM Extension Build Version: 20220426.1 (unified)
Regression: Not a regression
Repro Steps:

Make sure the resource group by Resource Group.
Right click a VM resource group -> Delete -> Pop up a dialog 1 -> Cancel.
Expand the VM resource group -> Click "Showing * of * resources..." to show all resources -> Right click the VM resource group -> Delete -> Pop up a dialog 2.
Check whether the dialog message of the dialog 1 and dialog 2 is consistent.

Expect:
The dialog message of the dialog 1 and dialog 2 is consistent.
Actual:
The dialog message of the dialog 1 and dialog 2 is inconsistent.

Suggestion:
Regardless of whether the resource group is "shown" or "hidden", it would be better to show all resources in the dialog message when deleting it.

Fixed by https://github.com/microsoft/vscode-azureresourcegroups/pull/311


@load balanced rest Template  404 issue with Eureka

Hi Team,
We are facing issues with @loadbalanced restTemplate defined as below with default configuration as below
@Bean
@LoadBalanced
@Primary
RestTemplate getRestTemplate() {
return new RestTemplate();
}
settings:
spring boot :1.5.11.RELEASE
spring-cloud-dependencies:Dalston.SR1
url1:10 instances
url2:10 instances
url3:10 instances
Scenario:we are calling multiple rest Service registered in Eureka service registry as below
public class RestCallService {

@Autowired
RestTemplate restTemplate;

public String post1(String url1){
	ResponseEntity<String> result = restTemplate.postForEntity(url1, entity, String.class);
	return result.getBody();
}

public String post2(String url2){
	ResponseEntity<String> result = restTemplate.postForEntity(url2, entity, String.class);
	return result.getBody();
}

public String post3(String url3){
	ResponseEntity<String> result = restTemplate.postForEntity(url3, entity, String.class);
	return result.getBody();
}

}

{\"timestamp\":1558530921431,\"status\":404,\"error\":\"Not Found\",\"message\":\"No message available\",\"path\":\"/abc/d/xyz\"

it is observed that 5% of these call giving us 404 not found in prod.! What could be the reason for this.Any suggestion would be appreciated.

We no longer support the Dalston release train.
https://spring.io/blog/2018/06/19/spring-cloud-finchley-release-is-available#lifecycle-announcements-for-release-train-versions
Please use Edgware.SR6 and try again.

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


Enhancement request: Defaults for grinder, new field for burr with defaults plus other suggestions

Would be great to be able to set my default grinder to be used with all profiles.
Also, burr being used with defaults.  Example Baratza steel burr vs ceramic.
A heart icon or favorite icon would be great to label those shots you really liked.
Longer-term:
At some point, it would be great to collaborate with Nic on his web based profile editor. (discussed on diaspora)
If profiles could be uploaded there and linked to from your site that would be helpful. Maybe even you can collaborate and push profiles to his project from your site.
Often tweaks are done to default "base" profiles.  So it would be good to show those deltas from the base profiles and perhaps upload both the base profile and or the changed profile derived from it to Nic's project.
Nice to have but probably lower priority:  Bulk edit of some sort.  Once I have the ability to set burr for all shots I'd want to have some way to easily apply to multiple shots.

Sorry, I somehow missed this 😅

Would be great to be able to set my default grinder to be used with all profiles.
You should use Enrique's plugin. Then you can always have correct values in .shot files themselves.

Also, burr being used with defaults. Example Baratza steel burr vs ceramic.
Feel free to add your vote on new metadata fields discussion.

A heart icon or favorite icon would be great to label those shots you really liked.
I think the "traffic light" slider does this job even better.

At some point, it would be great to collaborate with Nic on his web based profile editor. (discussed on diaspora)
There isn't much info about the profile in the .shot file. Just the name and some parameters. Definitely can't extract the exact steps or modifications from original profile. Maybe another suggestion for that metadata fields? Although I don't see a ton of value in this since you can't really replicate one's shot without exact same coffee, grinder, and settings. I see Nic's tool focused on profiles, and mine on shots, and to me that's input settings vs real-world output so not much overlap.

Bulk edit is a feature that would require a ton of work so I don't have much interest in doing it anytime soon. But PRs are welcomed 😅


Geoconnex phase 2

Describe the feature you'd like and what it will do
This Issue will serve as a landing spot for suggestions and ideas to improve our Geoconnex implementation in Hydroshare
Brainstorm

Toggle to allow mapped layers to have different colors for each item in the layer
Flash of Unstyled Content FOUC
Query geoconnex api using polygons where possible see example 9
Add browser tests for the Geoconnex component

refactor how we set feature properties:
right now we set the feature.NAME on every feature, this could be made more efficient
https://github.com/hydroshare/hydroshare/blob/2.3.2/theme/static/js/hs-vue/geoconnex.js#L1270


Fix missing dot causes confusing sentence in AlertNode doc

The missing dot causes the generated doc to include a confusing sentence:

@minhdanh Thanks!


Dependency specification too restrictive

The current dependency specification on httpx is ^0.23.0 (equivalent to 0.23.*), while pytest-httpx is <0.23.0. My guess is this is a typo on pytest-httpx.  I have a love-hate relationship with poetry dependency specification, as it doesn't work as nicely for pre-1.0 programs. It would be nice to relax the restrictions on httpx to anything >=0.23.0, as I am having trouble building a package that depends on chem-cloud (but I understand that poetry doesn't like this sometimes)
Additionally, it would probably be good to bump the version of Ruff and drop black and isort, as ruff has fully eaten those. For reference, here is the pyproject.toml template that I use for my projects.

Thanks @jevandezande! Will take a look this week. Time to update some dependencies :)


MessageReceived is not called correctly

There is a logical error in the current implementation of "PhotinoWebViewManager".
Since "MessageReceived" is called outside of the UI thread, it is possible to get an exception if the render batch is frequent.
blazor.webview.js:1

Received unexpected acknowledgement for render batch 46 (next batch should be 45)
at Microsoft.AspNetCore.Components.WebView.Services.WebViewRenderer.NotifyRenderCompleted(Int64 batchId)
at Microsoft.AspNetCore.Components.WebView.IpcReceiver.OnMessageReceivedAsync(PageContext pageContext, String message)
at Microsoft.AspNetCore.Components.WebView.WebViewManager.<>c__DisplayClass18_0.<<MessageReceived>b__0>d.MoveNext()

This happens because the queue of кeceived messages is moved due to the external thread.
You can reproduce this issue if you run the MudBlazor example in this pull request #36 and just quickly switch navigation links.

We can confirm that too. Our project is also based on MudBlazor.

I am not sure if this is related but I am seeing a similar sort of issue where I keep seeing the errors below when doing random things:
Photino.NET: "Strem".SendWebMessage(__bwv:["NotifyUnhandledException","Received unexpected acknowledgement for render batch 17 (next batch should be 16)","   at Microsoft.AspNetCore.Components.WebView.Services.WebViewRenderer.NotifyRenderCompleted(Int64 batchId)\r\
n   at Microsoft.AspNetCore.Components.WebView.IpcReceiver.OnRenderCompleted(PageContext pageContext, Int64 batchId, String errorMessageOrNull)\r\n   at Microsoft.AspNetCore.Components.WebView.IpcReceiver.OnMessageReceivedAsync(PageContext pageContext, String messa
ge)\r\n   at Microsoft.AspNetCore.Components.WebView.WebViewManager.\u003C\u003Ec__DisplayClass18_0.\u003C\u003CMessageReceived\u003Eb__0\u003Ed.MoveNext()"])
Photino.NET: "Strem".SendWebMessage(__bwv:["NotifyUnhandledException","Received unexpected acknowledgement for render batch 16 (next batch should be 17)","   at Microsoft.AspNetCore.Components.WebView.Services.WebViewRenderer.NotifyRenderCompleted(Int64 batchId)\r\
n   at Microsoft.AspNetCore.Components.WebView.IpcReceiver.OnRenderCompleted(PageContext pageContext, Int64 batchId, String errorMessageOrNull)\r\n   at Microsoft.AspNetCore.Components.WebView.IpcReceiver.OnMessageReceivedAsync(PageContext pageContext, String messa
ge)\r\n   at Microsoft.AspNetCore.Components.WebView.WebViewManager.\u003C\u003Ec__DisplayClass18_0.\u003C\u003CMessageReceived\u003Eb__0\u003Ed.MoveNext()"])

I am using Photino.Blazor v2.1.7 and unfortunately its not easy to reproduce as its quite random, but this instance was triggered by just swapping between pages on an SPA app.


Fix image metadata being used in wrong cell

Fix bug pointed out in https://github.com/executablebooks/MyST-NB/pull/588#pullrequestreview-2040630500 caused by reusing the image_options dict. Fixes #605.
Seems like in my previous PR I didn’t understand the tests enough to realize that I changed them to nonsense, but now I do.
Sorry for that!
cc @OriolAbril

Hi! It would be great to have this fix into the next release 😄.

Thanks for your feedback everyone! I am doing a round of maintenance on the Sphinx stack at the moment, I will aim for a release next week.

Thanks @flying-sheep, this is very helpful. Another contributor also opened a PR https://github.com/executablebooks/MyST-NB/pull/609, so I'm in the fortunate position of having to make a choice between the two! As there's a new test in #609, I'm going to close this in favour of that one :)

@agoose77 I would merge both PRs actually. They do have the addition of the .copy() in common, but that is all, so I would not expect merge conflicts either. This PR fixes the existing tests ehich are as of now testing that the wrong behaviour happens (consequently, all tests are failing on the other PR). The other PR doesn't change anything about existing tests (which needs to happen too in addition to fixing the bug) and adds a new test that is more specific for image metadata.
If you prefer, I can also merge both PRs into one keeping the commits of each author so contributions are properly tracked and open a new with the changes from both PRs

@OriolAbril yep, I'm pulling the changes from the tests to attribute @flying-sheep :)


Bool framework

mabornea created the following on May 23:
merging bool-framework to a branch from master
Original issue: https://github.ibm.com/ai-foundation/PrimeQA/pull/211

duplicate #64


libusb-compat-0_1: 0.1.7 -> 0.1.8

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/libusb/libusb-compat-0.1/releases.
meta.description for libusb-compat-0_1 is: "cross-platform user-mode USB device library"
meta.homepage for libusb-compat-0_1 is: "https://libusb.info/"
meta.changelog for libusb-compat-0_1 is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.1.8 with grep in /nix/store/ghplmd4zrwk1qrkvpi4g286z0vrrxcng-libusb-compat-0.1.8
found 0.1.8 in filename of file in /nix/store/ghplmd4zrwk1qrkvpi4g286z0vrrxcng-libusb-compat-0.1.8

Rebuild report (if merged into master) (click to expand)

187 total rebuild path(s)

187 package rebuild(s)

First fifty rebuilds by attrpath
arduino
arduino-core
arduino-core-unwrapped
audacious
audacious-plugins
audaciousQt5
avrdude
avrdudess
blueman
brscan4
cadence
cqrlog
cubicsdr
cutter
dfu-programmer
diffoscope
direwolf
domoticz
dump1090
dvdstyler
epkowa
fbida
fgallery
fldigi
freedv
fujprog
g15daemon
garmin-plugin
garmindev
garmintools
geeqie
gjay
gnss-sdr
gnuradio
gnuradio3_8Packages.ais
gnuradio3_8Packages.limesdr
gnuradio3_8Packages.osmosdr
gnuradio3_9
gnuradio3_9Minimal
gnuradio3_9Packages.gnuradio
gnuradio3_9Packages.grnet
gnuradioMinimal
gnuradioPackages.gnuradio
golden-cheetah
gpredict
gqrx
gqrx-gr-audio
gqrx-portaudio
grub2
grub2_efi

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/ghplmd4zrwk1qrkvpi4g286z0vrrxcng-libusb-compat-0.1.8 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A libusb-compat-0_1 https://github.com/r-ryantm/nixpkgs/archive/853dd62b67236bcf194943e3c1b6892ea81421a5.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ghplmd4zrwk1qrkvpi4g286z0vrrxcng-libusb-compat-0.1.8
ls -la /nix/store/ghplmd4zrwk1qrkvpi4g286z0vrrxcng-libusb-compat-0.1.8/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review took longer than 45m and timed out

Maintainer pings

@ofborg build gnuradio


Avoid "read" hanging if there is no CAN data ?

Hi,
I have the following code:
Serial.println ("Reading 0");
if(CAN1.read (&id,&ext,&rtr,&fd,&len,dtaGet))
{
  Serial.print ("0 ");
  Serial.println (len);
}

Serial.println ("Reading 1");

It seems to hang if there is no data on the canbus - how can I avoid this please ?
Thanks !

Seeming to have similar issues to this


v3

Description

Test Steps

Related Issue

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

v3


NXP: Configure NXP Kinetis FlexTimer DTS nodes for PWM on board level

Don't assume in the soc level device trees that FlexTimer nodes will always be configured as PWM. Instead, configure FlexTimer nodes for PWM at the board level for NXP Kinetis boards.
The NXP Kinetis FlexTimer IP can be configured as PWM controller, counter,
quadrature decoder, pulse-width measurements and more. Zephyr currently only has support for PWM. This PR is preparation for introduction of Zephyr support for other FlexTimer functionality.

LGTM. Can we remove the pwm-N aliases? Not sure we need them anywhere.

@galak Thanks. I have removed the unused aliases.


Issue with Gemini on Vertex AI

[ x] This is actually a bug report.
[ ] I am not getting good LLM Results
[ ] I have tried asking for help in the community on discord or discussions and have not received a response.
[ ] I have tried searching the documentation and have not found an answer.

What Model are you using?

[ ] gpt-3.5-turbo
[ ] gpt-4-turbo
[ ] gpt-4
[x ] Other (gemini-1.5-pro-002 on vertex ai)

Describe the bug
When executing this example - https://python.useinstructor.com/hub/vertexai/#vertex-ai-client , I am getting
ModuleNotFoundError: No module named 'instructor.client_vertexai'; 'instructor' is not a package
StackTrace
Traceback (most recent call last):   File "c:\Users\A0100109\python\gemini-bq-py-test\instructor.py", line 1, in <module>     import instructor.client_vertexai  as instructor   File "c:\Users\A0100109\python\gemini-bq-py-test\instructor.py", line 1, in <module>     import instructor.client_vertexai  as instructor ModuleNotFoundError: No module named 'instructor.client_vertexai'; 'instructor' is not a package
To Reproduce
Here is Code I used
import instructor

from pydantic import BaseModel
import vertexai.generative_models as GenerativeModel
import vertexai

vertexai.init(project="XXXX", location="us-central1")
client = GenerativeModel.GenerativeModel("gemini-1.5-pro-002")
client = instructor.from_vertexai(client)

class UserDetails(BaseModel):
    name: str
    age: int

resp = client.create(
    response_model=UserDetails,
    messages=[
        {
            "role": "user",
            "content": f'Extract the following entities: "Jason is 20"',
        },
    ],
)
print(resp)

**REQUIREMENTS.TXT
google-cloud-aiplatform==1.71.1
pydantic==2.9.2
instructor==1.6.3
jsonref==1.1.0

Expected behavior
Error should not appear
Screenshots
If applicable, add screenshots to help explain your problem.

Could you try installing and re-installing instructor.

Tried, Different error
$ c:/Users/A0100109/python/gemini-bq-py-test/.venv/Scripts/python.exe c:/Users/A0100109/python/gemini-bq-py-test/instructor.py
Traceback (most recent call last):
  File "c:\Users\A0100109\python\gemini-bq-py-test\instructor.py", line 1, in <module>
    import instructor
  File "c:\Users\A0100109\python\gemini-bq-py-test\instructor.py", line 12, in <module>
    client = instructor.from_vertexai(client)
             ^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: partially initialized module 'instructor' has no attribute 'from_vertexai' (most likely due to a circular import)
(.venv)

Could you try renaming your script to another name? It seems like you're running it from a script called instructor.py

Worked @ivanleomk , Kind of mistake from my side having same name as package for file

Awesome glad to hear that


send message to channel

I'm trying to use this libarary to broadcast a message to my telegram channel.
This is the code I'm using:
public static async Task Send(string message)
{
try
{
var ap = new TelegramAppInfo
{
ApiId = 33438,
AppVersion = "1",
DeviceModel = "pc",
LangCode = "en",
SystemVersion = "1"
};
            var cfg = new TcpTransportConfig("149.xx.xx.xx", 443);
            var key = GetBytes("71d3af5934bf7dttt17ta4bb7c31437c");
            var ccfg = new ConnectionConfig(key, 0);
            var client = new TelegramClient(cfg, ccfg, ap);
            client.Connect();
            while (!client.IsConnected)
                System.Threading.Thread.Sleep(50);

            var sendCodeArg = new AuthSendCodeArgs();
            sendCodeArg.ApiHash = "71d3af5934bf7dttt17ta4bb7c31437c";//App api_hash
            sendCodeArg.ApiId = xxxxx;//API ID
            sendCodeArg.LangCode = "en";//Lan Code
            sendCodeArg.PhoneNumber = "+961xxxxxx";//my account phone number
            sendCodeArg.SmsType = 0;

            var args = new MessagesSendMessageArgs
            {
                Message =message,
                Peer = new InputPeerSelf(),
                RandomId = GetRandomId()
            };
            var send_res = await client.Methods.MessagesSendMessageAsync(args);
        }
        catch (NullReferenceException ex)
        {
            throw ex;
        }
        return "done";
    }

    private static ulong GetRandomId()
    {
        return (ulong) DateTime.Now.Millisecond;
    }

    private static byte[] GetBytes(string str)
    {
        return str.HexToBytes();
    }

I'm getting an exception:
Offset and length were out of bounds for the array or count is greater than the number of elements from index to the end of the source collection.
Stacktrace
[ArgumentException: Offset and length were out of bounds for the array or count is greater than the number of elements from index to the end of the source collection.]
System.Buffer.BlockCopy(Array src, Int32 srcOffset, Array dst, Int32 dstOffset, Int32 count) +0
SharpMTProto.Messaging.MessageCodec.ComputeAesKeyAndIV(Byte[] authKey, Int128 msgKey, Byte[]& aesKey, Byte[]& aesIV, Sender sender) +353
SharpMTProto.Messaging.MessageCodec.EncodeEncryptedMessage(IMessage message, Byte[] authKey, UInt64 salt, UInt64 sessionId, Sender sender) +1259
SharpMTProto.MTProtoConnection.EncodeMessage(IMessage message, Boolean isEncrypted) +200
SharpMTProto.<b__27>d__2a.MoveNext() +226

Hey, library is not implemented. Do not post any question here, plz.


4.0.1 Broken on Windows

byebug 4.0.1 is broken on Windows.
Gem::Ext::BuildError: ERROR: Failed to build gem native extension.

C:/Ruby21-x64/bin/ruby.exe -r ./siteconf20150313-10752-1hz6nzd.rb extconf.rb
C:/Ruby21-x64/bin/ruby.exe: No such file or directory -- extconf.rb (LoadError)

extconf failed, exit code 1

Gem files will remain installed in C:/Ruby21-x64/lib/ruby/gems/2.1.0/gems/byebug-4.0.1 for inspection.
Results logged to C:/Ruby21-x64/lib/ruby/gems/2.1.0/extensions/x64-mingw32/2.1.0/byebug-4.0.1/gem_make.out
An error occurred while installing byebug (4.0.1), and Bundler cannot continue.
Make sure that `gem install byebug -v '4.0.1'` succeeds before bundling.

3.5.1 works fine.

:cry:
The CI build seems fine... at least in the Ruby you're using (in Ruby 2.2 it seems that something is wrong in the appveyor's server ...).
If you want to and have time, could you git bisect this from 3.5.1? I don't have a Windows machine...

Another idea. Clone the gem and replace the extconf.rb file with the one present in '3.5.1'. Then link to the cloned gem in your application and try to install it.
Let me know if you need any help (if you want to help at all, of course).

Did some troubleshooting, turns out Ruby doesn't like autorun on windows for the command prompt.  Not a byebug issue.  Sorry about that!

No problem, thanks for checking that out!


MNIST tutorial data cant be downloaded (404)

Hey, tried to work my way through tf but the download link seems down
https://tensorflow.googlesource.com/tensorflow/+/master/tensorflow/g3doc/tutorials/mnist/input_data.py
404s

The file was moved to tensorflow/examples/tutorials/mnist/input-data.py -- I think our website hasn't updated yet.
You might want to consider just git cloning the repository instead of downloading files individually.


AUTHENTICATION_ANONYMOUS_ACCESS_ENABLED in docker-compose.yml

Hi team,
When using AUTHENTICATION_ANONYMOUS_ACCESS_ENABLED in the docker-compose.yml file, I get an argument missing error when omitting the api_key.
/opt/homebrew/lib/ruby/gems/3.3.0/gems/langchainrb-0.14.0/lib/langchain/vectorsearch/weaviate.rb:20:in `initialize': missing keyword: :api_key (ArgumentError)
	from threatai.rb:8:in `new'
	from threatai.rb:8:in `<main>'

This is the Ruby code, I'm using to initialize:
client = Langchain::Vectorsearch::Weaviate.new(
    url: 'http://<IP_ADDRESS>:8080',
    # api_key: '',
    index_name: "Documents",
    llm: llm
)

This is my docker-compose.yml file:
version: '3.4'
services:
  weaviate:
    command:
    - --host
    - <IP_ADDRESS>
    - --port
    - '8080'
    - --scheme
    - http
    image: cr.weaviate.io/semitechnologies/weaviate:1.26.1
    ports:
    - 8080:8080
    - 50051:50051
    volumes:
    - weaviate_data:/var/lib/weaviate
    restart: on-failure:0
    environment:
      QUERY_DEFAULTS_LIMIT: 25
      AUTHENTICATION_ANONYMOUS_ACCESS_ENABLED: 'true'
      PERSISTENCE_DATA_PATH: '/var/lib/weaviate'
      DEFAULT_VECTORIZER_MODULE: 'none'
      ENABLE_MODULES: 'text2vec-cohere,text2vec-huggingface,text2vec-palm,text2vec-openai,generative-openai,generative-cohere,generative-palm,ref2vec-centroid,reranker-cohere,qna-openai'
      CLUSTER_HOSTNAME: 'node1'
volumes:
  weaviate_data:

Is it possible to use weaviate-ruby to connect with anonymous access?
Thanks!

@ethicalhack3r Yep, I'll fix it for you in a few hours.

Thank you!


Use setup.cfg for project configs

To reduce the littering of configuration files of many tools. For example, coverage and tox. Configurations for mypy and yapf can still be added.

Moved configurations to setup.cfg for

tox in f86c3aaad99f15e56f086f70e241d9be7df540dd;
mypy in aa4ba1fa8114b4b1e0266ea3eacec618255f1afc;
yapf in 5d77562cf92501aa61c16e6388c839f2391f40b0; and
coverage.py in 0c9a7932301defb1d56a905c460ea4fcadc293ea.

Moved configurations to setup.cfg for

tox in f86c3aaad99f15e56f086f70e241d9be7df540dd;
mypy in aa4ba1fa8114b4b1e0266ea3eacec618255f1afc;
yapf in 5d77562cf92501aa61c16e6388c839f2391f40b0; and
coverage.py in 0c9a7932301defb1d56a905c460ea4fcadc293ea.


run @jokeyrhyme/node-init for consistency / CI

ran @jokeyrhyme/node-init for consistency / CI

LGTM, thats a lot of boilerplate you removed from every project, awesome!


Keep the SplitContainer offset value when it is shrinked

What this does:

keep the SplitContainer split_offset value in any situation, even if it is constrained by a too small container. This makes the splitter go back to its original value when the SplitContainer gets resized.
add a new clamp_split_offset() function that allows to manually bring back the split_offset value to ranges depending on the SplitContainer's size.

Fixes #23289.
Note: this is more or less a compatibility breaking change. But I think it is worth getting merged as it wont impact people projects, unless they used weird workaround to this bug. It is worth notifying in the change log though.

I finally cherry picked this one to test.  Looking good, haven't noticed any problems yet.
Glad to have the dock sizes remembered.  X)

Thanks!


[BUG] When you add, remove and try to re-add vGPU profiles on addresses they aren't available

Describe the bug

If you add a vGPU profile, use it, then remove it you can't add another one
To Reproduce
Steps to reproduce the behavior:

Add 4 vGPU profiles to a cluster
assign all 4 to VMs
Remove two of the VMs, then remove the profiles
Try to add them again
Expected behavior

You should be able to add them again
Support bundle

supportbundle_314d015f-bac6-449f-a676-14fd3732e576_2024-05-08T17-15-05Z.zip
Environment

Harvester ISO version: v1.3.0
Underlying Infrastructure (e.g. Baremetal with Dell PowerEdge R630): DL360 single node, and 2 node

Additional context
Add any other context about the problem here.
I was able to add the profiles to addresses on the SR-IOV GPU that hadn't been used before

I'm not sure if this is still an issue. I'm going to check in v1.3.2 and v1.4-head.

I tested this in v1.3.2 with a nvidia A2 card and A2-2Q profile on a 2 node cluster with 1 node having the GPU. I was able to add, remove, and then re-add the profile.


Minor fixes and CI for JavaScript

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


atuin: 0.7.1 -> 0.8.0

Motivation for this change
Version Bump
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[x] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

This has been updated to 0.8.0 in #151275, so I am closing this.
Still thank you for your contribution!


Add new fields to the links index

a field for the OD url (the scanned url, could also be a suburl because it contains the root url)
this allows users to limit the search to specific ODs only, if they prefer that. it would also allow to provide a search link for a specific OD along with the scan results
a field for the file extension.
this makes it easier to filter certain types of files (we could create custom extension groups for video files, audio files, documents, etc., that we can select in advanced search, or have the user input the extension(s) manually)

file extensions are done. i wonder what elasticsearch field type OD urls would be. keyword would be ideal because there are tons of duplicates in that field, but text is also an option.

file extensions are done. i wonder what elasticsearch field type OD urls would be. keyword would be ideal because there are tons of duplicates in that field, but text is also an option.

As long as keyword can be used for exact matching, that should work...

As long as keyword can be used for exact matching, that should work...


Travis CI: Run shellcheck on shell scripts in allow_failures mode

Output: https://travis-ci.org/openaps/oref0/jobs/494772101#L382

Did you find any of this output useful?  I'm not sure it found anything worth fixing, but it definitely found all kinds of stuff that doesn't really matter and therefore I don't care to fix (but wouldn't mind if someone else wants to)...


Getting a Karabiner-Elements Alert with no way to close it

I'm getting this error message upon launching the app.
Title
Karabiner-Elements Alert
Description
System policy prevents loading the kernel extension that is used by Karabiner-Elements. You have to allow the system software from developer "Fumihiko Takayama" in System Preferences > Security & Privacy.
When I go to the System Preferences > Security & Privacy I don't a permission request.
I've restarted the machine couple of times and it still doesn't show up.
OS version
macOS High Sierra
10.13.6
Karabiner version
12.1.0
See attached screenshot:

yes, I did try that before I posted this thread. I gave Accessibility permission to the app inside Privacy.

this is still happening and I haven't found a solution. The link @clemsam posted describes a bug where the tap was not recognized but in my case I don't even see the permission request in Security & Privacy.

That's weird. I replied above after the last stale bot warning and it still closed the issue.

Same situation happened here just yesterday.
Mac OS: Mojave 10.14.6
Karabiner: 12.5.0
Haven't found any solutions yet. Here are what I tried so far:
(1) Re-install Karabiner for the latest version as well as old versions.
(2) Restart computer.
(3) Restart computer with safe mode and re-install Karabiner.

Same issue.


How to use this repository with custom knowledge-base ?

Similar to the RAG in the Transformers library can we use a custom KB?

Did you mean updating custom document entries to the index for retrieval during fine-tuning?

Yes exactly.
[image: Augmented Human Lab] http://www.ahlab.org/ [image: uni]
https://www.auckland.ac.nz/en/abi.html
Gayal Shamane
Ph.D. Candidate
Augmented Human Lab
Auckland Bioengineering Institute | The University of Auckland

Option 3 has been implemented, please check out the readme and see if that matches your need :-)


Crashing

Crashing on oneplus 9 pro with android 13 latest official rom

I will never support 'talking only' without sending proper logs. If you want a fix, then read #bug_report carefully, and send required files to the author.


scheduler-v2: Pre master review

A PR to review the scheduler-v2 implementation, ensuring it addresses issues raised in #1484.
This is a PR into scheduler-v2-base, with the hope that once this is merged, we can PR to master and get more peer review.
/cc @jzila

Bug: requesting to stop multiple jobs can request to stop the same job twice.
t=2015-09-04T20:48:53+0000 lvl=dbug msg="Updating processes" component=scheduler fn=RectifyJobs formation.processes=map[echoer:3] cluster.processes="map[echoer:3 printer:2]"
t=2015-09-04T20:48:53+0000 lvl=info msg="sending event to listeners" component=scheduler event.type=rectify-jobs event.error=nil listeners.count=0
t=2015-09-04T20:48:53+0000 lvl=info msg="requested job stop from host" component=scheduler fn=stopJob host.id=flynn job.type=printer job.id=flynn-631118a2-1cfb-461b-900a-0acc629c351f
t=2015-09-04T20:48:53+0000 lvl=info msg="sending event to listeners" component=scheduler event.type=request-job event.error=nil listeners.count=0
t=2015-09-04T20:48:53+0000 lvl=info msg="requested job stop from host" component=scheduler fn=stopJob host.id=flynn job.type=printer job.id=flynn-631118a2-1cfb-461b-900a-0acc629c351f
t=2015-09-04T20:48:53+0000 lvl=info msg="sending event to listeners" component=scheduler event.type=request-job event.error=nil listeners.count=0

@jzila the main scheduler loop should not block on network calls (i.e. controller / postgres / host clients), instead writing to channels which other goroutines can receive from and do the necessary calls. I am currently working on this.

@jzila I have merged in your fixes, I am currently working on improving the logs.

@jzila I have quite a few log changes I intend to push soon, let me know if you are working on things as it may cause you some rebase pain.

@jzila FYI I have refactored the tests (5565f09) and I am now verifying we have enough test coverage.

@jzila left a few comments related to job backoffs and SyncJobs, let me know in IRC if you start working on them, as I may fix them later today.

In the latest build failure, the main loop of the scheduler leader (job 89d4f3ae-7ac8112a-bd2a-485f-8023-5256e382aabe) got blocked, meaning it stopped reacting to events.
The last action I can see in the logs is stopping a job due to a formation change:
t=2015-09-13T15:26:42+0000 lvl=info msg="rectifying formation" component=scheduler fn=Rectify app.id=427c6ab2-5bed-4b07-864e-dbf8e085ad26 release.id=5953a97b-56a8-466e-a51b-0ad453f08e36
t=2015-09-13T15:26:42+0000 lvl=info msg="existing formation in incorrect state" component=scheduler fn=Rectify app.id=427c6ab2-5bed-4b07-864e-dbf8e085ad26 release.id=5953a97b-56a8-466e-a51b-0ad453f08e36 e
xpected=map[resources:1] actual=map[resources:2] diff=map[resources:-1]
t=2015-09-13T15:26:42+0000 lvl=info msg="requesting removal of 1 job(s) of type resources" component=scheduler fn=sendDiffRequests app.id=427c6ab2-5bed-4b07-864e-dbf8e085ad26 release.id=5953a97b-56a8-466e
-a51b-0ad453f08e36
t=2015-09-13T15:26:42+0000 lvl=info msg="handling job request" component=scheduler fn=HandleJobRequest req.id= req.type=down
t=2015-09-13T15:26:42+0000 lvl=info msg="stopping job" component=scheduler fn=stopJob req.host.id= req.job.id= job.type=resources
t=2015-09-13T15:26:42+0000 lvl=info msg="selected job for termination" component=scheduler fn=stopJob req.host.id= req.job.id= job.type=resources job.id=697e7980-33e63547-9450-4dfd-9b7d-872510f65b1c job.h
ost.id=697e7980
t=2015-09-13T15:26:42+0000 lvl=info msg="getting host client" component=scheduler fn=stopJob req.host.id= req.job.id= job.type=resources
t=2015-09-13T15:26:42+0000 lvl=info msg="requesting host to stop job" component=scheduler fn=stopJob req.host.id= req.job.id= job.type=resources

Everything after that is logged from other goroutines (i.e. RunPutJobs, triggerSyncJobs or triggerSyncFormations) but no actions are being taken.
Looking in the host logs of 697e7980, the StopJob request is logged as started, but is indeed blocked (there is no log of its completion):
t=2015-09-13T15:26:42+0000 lvl=info msg="request started" req_id=9a4db7a4-94c3-4ef6-a2c9-fa05420a1eec component=host method=DELETE path=/host/jobs/697e7980-33e63547-9450-4dfd-9b7d-872510f65b1c client_ip=<IP_ADDRESS>

We should investigate why the host is not returning a response, but this highlights the fact that the scheduler is vulnerable to blocking network requests bringing everything to a halt, something I think we really need to fix (see my earlier comment).
/cc @jzila

@jzila lots of minor comments and a few bigger ones, I think some of them can just be opened as separate issues to fix post merge rather than blocking things.
I also think we should squash to a small number of commits (maybe even one?).

@lmars I'll fix the minor ones and open issues for the major ones. I responded to each comment for both me and you to communicate what I'll be fixing.
As for squashing, I'll handle that once there is no feedback left to implement. I'll squash every set of consecutive commits by one person into 1 commit, so they'll end up alternating.

@jzila latest fixes merged, please open issues for the points we are not handling in this PR.
I will run run the tests a few times and comment with any failures.


Add mention to v-bind directive documentation that v-bind.sync="{...}" is possible

This capability was added here: https://github.com/vuejs/vue/issues/5937

Thanks! This would be in the component documentation and a contribution is welcome. 🙂

Done.
https://github.com/vuejs/vuejs.org/pull/1459

Merged! 😄


Added non-native ordered buffer and memory variants

Also:

Fixed a bug - BaseWritableBufferImpl.checkValidForWrite() used to check resource's read-only flag instead of local read-only flag
Some cleanup across the codebase, added final modifiers to methods and classes

I'll add tests in some time. But you could review the code already.

Coverage decreased (-18.3%) to 78.033% when pulling e12d95ced45a25b885b84c853051ddfedf00afa9 on non-native into 2412d23f2b990597510ca4633c7ddb6f00e09964 on master.

I fixed the above changes.  This merge was a complex one that involved some fixes from PR #67.  This was done locally and pushed to master.

@leerho my authorship of this contribution is lost in master. This is not a way to go.

Sorry about that, it was not my intent.  Your name is in the source code and I certainly recognize your contributions.   I kept waiting for your responses and/or fixes to issues I brought up, and I was at a point where I felt blocked and needed to move forward.   #67 and #69 were intertwined and I had to manually go through both to select the proper code fragments.  This was easier to do locally.  Yes, I could have taken the extra steps to push the updated master and branches back to origin before the final merge.  My mistake.
Nonetheless, there is a lot of test code that needs to be written.  Please let me know promptly if you are willing to contribute to that as I have already started.

I usually don't care about @author tags in Javadoc, Git history is more important for me. Could you please make this commit to look like I've authored it? It's possible to do it as described in https://stackoverflow.com/a/3042512/648955 (setting the author leventov<EMAIL_ADDRESS>and then force push into master.

Please let me know promptly if you are willing to contribute to that as I have already started.

I was planning to do this. If you will do this I'm grateful because you are taking over this work from me. We could still split the efforts (e. g. by test classes) but I don't promise the date of delivery of my part.

I followed your suggested fix to add you as author on abc1190 Merge
non-native impls.
On Wed, Apr 25, 2018 at 10:41 AM, Roman Leventov<EMAIL_ADDRESS>wrote:

Please let me know promptly if you are willing to contribute to that as I
have already started.
I was planning to do this. If you will do this I'm grateful because you
are taking over this work from me. We could still split the efforts (e. g.
by test classes) but I don't promise the date of delivery of my part.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/DataSketches/memory/pull/69#issuecomment-384372854,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMV4wkL6HM1VODW6vV7DjkLBprSfsdfyks5tsLVhgaJpZM4TOJ8m
.


PR: General code refactoring (PEP8 guidelines) and setup.py file added

Fixes #18

[x] Renamed main module to visual_genome
[x]  Added setuptools distribution file
[x] Function and variable naming is now PEP-8 compliant
[x] Imports are now absolute
[x] Docstrings are now defined inside function body
[x] Update README.md instructions
[x] HTTP server requests are now done using requests module
[x] Module is now canonical for both Python 2 and Python 3

Nice job. Sorry took me a while to get to this. Can you solve the merge conflicts and send me another request?

@ranjaykrishna Don't worry! I've merged the last changes from master, does everything looks good?


Add ability to export data

Add ability to export data from the app as a csv file so that Lisa can have a local back-up.

@icodeclean is working on this

I promoted this to MVP as Lisa requested we have some kind of backup strategy.


updated with more words

updated from the wordlist of Pratham shared by @dohliam

Thanks so much for adding the extra entries! :+1:
I'll keep a look out for any other open-licensed sources of Odia wordlists... We have two Odia translations already as part of this storybook project -- if we get more that might eventually be a good corpus to look at.


Adds Healh Analyzer to Chem PDA

About the PR

Chem PDAs can now be used as health analyzers, like all other medical jobs. Not sure why it can't already. Probably contributes to Chem not doing medical stuff as often as they should.
Media

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

Changelog

:cl: Lank

tweak: Chemist PDA's can now analyze patient health.

This isn’t the pr we deserved but it is the one we needed


Responsive utility classes overlapping near breakpoint (ref: #20902)

Raised issue (#20902) has been closed, but I have some more input to help you reproduce the problem. Thus I have created this new issue with additional information. Hope that is alright.
It seems to me, that the responsive utility classes overlap at the breakpoints. If for example I have:

   This is XL

   This is LG

Then, when making the browser the exact width of the breakpoint between xl and lg, I see both texts
"This is XL" and "This is LG".
Same for LG-MD, MD-XS.
I am using Google Chrome latest version on Windows 7.
Follow up:
Please have a look at this JS Bin:
https://jsbin.com/sabedowija/edit?html,console,output
If you make the Output window exactly 1199 pixels in width, you will see:
This is XL
This is LG
You can also see it from the attached image below.

This has to be some odd pixel rounding. I can't reproduce it in that example in Safari and Chrome on macOS. Any kind of browser zooming going on here or anything?

I have no zooming set in the browser (Chrome/Windows).
By the way, I just checked with Firefox/Windows, and it also has the same issue when window is 1199 pixel in width.

I can reproduce this on Bootstrap 3.3.7. This code:
`

XTRA-SMALL
SMALL
MEDIUM
LARGE
`
doesn't work at 767px, 991px or 1199px width. Same happens with hidden-**

Regrettably closing as a can't fix. We can't really do anything about this given the media queries don't have any overlapping boundaries—they go from 1px to the next. Overlapping them would create a brand new set  problems for us.


Better error reporting when interacting with undeployed contracts

Component
Forge
Describe the feature you would like
I have bumped into many EvmError errors that were caused by an attempt to interact with a contract which was either undeployed already or was deployed to a local network and by the time I was making the call, the contract did no longer exist because I had used a fork cheatcode.
I think that this is such a common scenario that it would be extremely helpful (and prevent many angry reports in the Foundry Support group) for Foundry to handle this sort of error more gracefully and display a clearer error, e.g.:

You have attempted to make a call to address 0xCAFE..., but not contract is present there

I keep bumping into this issue every time we make a new deployment which I need to cross-reference in another repository - I forget to bump the block number.
The error message is generic and difficult to reverse engineer:

sg @PaulRBerg — agreed we could probably improve this error. Will take a stab at this during the next week to see if we can improve this


README: whitespace cleanup

Remove a line.

Gerrit code review: https://camlistore-review.googlesource.com/6019

Gerrit code review: https://camlistore-review.googlesource.com/6019 (at git rev 00b0c96)

Gerrit code review: https://camlistore-review.googlesource.com/6019 (at git rev 676bd67)

Gerrit code review: https://camlistore-review.googlesource.com/6020 (at git rev 676bd67)

merge this! :-)

Ah, I looked to quickly. I would like to see the whitespace removed from the LICENSE

The whitespace cleanup in README looks good, but the test.txt file should probably not be here, right?
Should this PR be cleaned up and merged? I see on https://camlistore-review.googlesource.com/#/c/6020/ it says abandoned, but it's open here. It's a good change, aside from test.txt though.


[BEAM-5250] Fix GlobalWindow's max timestamp in Python SDK

This fixes the Python streaming wordcount failing with an
IllegalStateException in the Runner code:
"TimestampCombiner moved element from MAX_TIMESTAMP to earlier time END_OF_GLOBAL_WINDOW"

The Python SDK would create a GlobalWindow with a maximum timestamp of
MAX_TIMESTAMP which in turn would shift the 'element hold' in the Runner code
backwards. This is because the Runner's GlobalWindow maximum timestamp is
smaller than the Python SDK's.
CC @tweise @angoenka
Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---
---
---
---
---

Java

Python

---

---
---
---
---

I wonder if this max timestamp (in the future) could be defined in proto so that SDKs don't repeat it @lukecwik ?

This is very surprising behavior to me and I don't see a good reason for this being the case.  Why is this the case in the Java SDK?  Shouldn't we make the formal spec do the more obvious thing rather than carry over what seems like a longstanding hack from one SDK to the formal spec? (CC: @lukecwik)

CC: @robertwb

Having the end of the GlobalWindow be max - 1 day seems rather arbitrary. If it really needs to be less than MAX_TIMESTAMP, how about its immediate predecessor? (Or is there ambiguity about milliseconds vs. microseconds to deal with here? And even then, 2^63 - 1 micros is not likely representable as an even number of milliseconds...) Or is the concern that windowing from global windows -> fixed windows would overflow? In which case we'd still overflow for days (not unreasonable) and maybe should choose something much larger.
I think it'd be worth doing at least a cursory exploration into what it would take to "fix" Java. If that's too hard, at least we should make this a proto value (in standard windows perhaps)?

Having the end of the GlobalWindow be max - 1 day seems rather arbitrary.

Yes, it is arbitrary. The reason is stated in GlobalWindow (just restating in case somebody didn't have the chance to look it up):
  // Triggers use maxTimestamp to set timers' timestamp. Timers fires when
  // the watermark passes their timestamps. So, the maxTimestamp needs to be
  // smaller than the TIMESTAMP_MAX_VALUE.
  // One standard day is subtracted from TIMESTAMP_MAX_VALUE to make sure
  // the maxTimestamp is smaller than TIMESTAMP_MAX_VALUE even after rounding up
  // to seconds or minutes.

I think it'd be worth doing at least a cursory exploration into what it would take to "fix" Java. If that's too hard, at least we should make this a proto value (in standard windows perhaps)?

I suppose it would work to trigger all windows when the Watermark is at MAX_TIMESTAMP. However, that is a change in semantics, since the max window timestamp normally needs to be larger than the current Watermark. Since we can't make any further progress, that could be ok.
Otherwise, implementing the GlobalWindow timestamp in the RunnerAPI would be the most portable approach.

@lukecwik Thanks for merging this and filing the JIRA.

IIRC there were multiple bugs caused by converting max timestamp from int64 -> double and then back to int64, as high int64 values cannot be precisely represented as an IEEE double (so max int64 comes back as a negative number if you round trip it through a double). I believe the current Java max timestamp was chosen because it's a value that can be represented precisely in both an int64 and a double.


Hand mesh toggle button disables hand input

Describe the bug
Hand mesh toggle button in the toggle features panel disables hand input and freezes HoloLens 2 app.
To reproduce
Steps to reproduce the behavior:

Build HandInteractionExample - UWP
Connect HoloLens 2
Run app through visual studio: Release, ARM, Device
Press hand mesh toggle button in the toggle features panel

Screenshots
Video of Issue - Hand Mesh Toggle
Your Setup (please complete the following information)

Unity Version 2018.4.7f1
MRTK Version v2.1

Target Platform (please complete the following information)

HoloLens 2

fixed with #6238


#144937479 Emojis for chat window

What does this PR do?
It allows users to use emojis in the chat window
Description of Task to be completed?
As a user in a game I want to be able to use emojis in the group chat so that I have a more enjoyable and expressive chat experience
How should this be manually tested?
N/A
Any background context you want to provide?
N/A
What are the relevant pivotal tracker stories?
#144937479 Users should be able to use emojis in the chat window
Screenshots (if appropriate)

#### Questions:
N/A

Changes Unknown when pulling 17b794c4e712ca16c56bb8fc5a1f098c3e238915 on feature/144937479/emojis-for-chat-window into ** on master**.


fix : check themeData transition

Replace this paragraph with a description of what this PR is changing or adding, and why. Consider including before/after screenshots.
Every PR must update the corresponding documentation in the code, and also the readme in english with the following changes.
Pre-launch Checklist

[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making or feature I am adding, or @jonataslaw said the PR is test-exempt.
[x] All existing and new tests are passing.

Thanks for your contribution!
LGTM


Cannot create title with question mark in it

When I find a book in search and try to add it, if it has a "?" in the title, the note fails to create and I get the following error
Error: file name cannot contain any of the following characters:
It would be great if there was an opportunity to edit the proposed title and continue with creation.

Windows 11
Obsidian 0.16.5
Book Search Plugin 0.5.8

Having the same issue, however I have noticed that titles with ampersands are having them deleted automatically and the file is created.  Would it be possible to simply have the file creation delete the question mark?  I'm not sure if colons in a title (title:subtitle) are auto adjusted.

Colons do autodelete for file names.

I was just starting to check this plugin out and I hit this problem on the first book I looked for. It's definitely a show stopper for being able to use this to build a book catalog.

+1, it would be nice if there was a way to clean the illegal characters from the book title.
For example, Movies script and Videogame script do something like this to clean it up:
function replaceIllegalFileNameCharactersInString(string) {
  return string.replace(/[\\,#%&\{\}\/*<>?$\'\":@]*/g, "");
}


Add a Codacy badge to README.md

Current coverage is 37.25% (diff: 100%)

Merging #1 into master will not change coverage

@@             master         #1   diff @@
==========================================
  Files             6          6
  Lines           102        102
  Methods           0          0
  Messages          0          0
  Branches         25         25
==========================================
  Hits             38         38
  Misses           52         52
  Partials         12         12

Powered by Codecov. Last update be9425e...e7ac677


label node with host subnet zone name to specify specific subnet range

To specify specic subnet range for a specific node, first in the
ovn-config configmap, define multiple subnet cidrs in net_cidr with the
format of<EMAIL_ADDRESS>then label the node that that
needs specific subnet range with k8s.ovn.org/subnet_allocator_name=
label.
Signed-off-by: Yun Zhou<EMAIL_ADDRESS>

Pull Request Test Coverage Report for Build 96434858

125 of 155 (80.65%)  changed or added relevant lines in 5 files are covered.
35 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.2%) to 50.46%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

go-controller/pkg/ovn/gateway_init.go
13
15
86.67%

go-controller/pkg/config/utils.go
20
25
80.0%

go-controller/pkg/ovn/ovn.go
25
30
83.33%

go-controller/pkg/ovn/master.go
65
83
78.31%

Files with Coverage Reduction
New Missed Lines
%

go-controller/pkg/ovn/master.go
35
52.76%

Totals

Change from base Build 95583266:
0.2%

Covered Lines:
4172

Relevant Lines:
8268

💛  - Coveralls


buttons on top of progress bar

There seems to be an issue with the buttons' CSS or something similiar. If the horizontal space becomes too small, they are put on top of the progress bar.

Isolating ...
only on Wordpress, not on Drupal, so might be plugin related, BUT the multiple choice example works fine ... hmm ...

Seems to be a problem with JoubelUI when the feedback messages get too long.

It's a problem in the H5P core.


Dinamically set SiteConfig for different locales with i18n

Details
I couldn't find a way to dynamically set SiteConfig for different locales. I need to fetch the infos from a CMS.
The documentation explains :

how to set SiteConfig statically via files but not dynamically
How to do it via plugin or middleware but not for all locales

Is it possible for now and what strategy would you use ?
In either case, I think it would be great to update the docs to mention it.
Thanks a ton for this module! 🫶

Hi, I'd suggest using a Nitro plugin where you fetch the data on init, then add a hook for any request where you update the site config.
The code sample I have in 2 is exactly what I'm talking about.
export default defineNitroPlugin(async (nitroApp) => {
  // fetch the infos from a CM
  const infos = await $fetch('/db/site-config', {
    params: { env: import.meta.env.NUXT_SITE_ENV },
  })
  // init the site config
  nitroApp.hooks.hook('site-config:init', ({ event, siteConfig }) => {
    // presumably info has mapping of all the URLs and data
    const siteData = infos.filter(i => getRequestUrl(event) === i.host)
    siteConfig.push(siteData)
  })
})


[SP-5048] Backport of PDI-17801 - Different output fields of Table Input for MySQL and MariaDB driver (8.2 Suite)

@RPAraujo @ssamora

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -f 'pom.xml' -pl 'core' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'core' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  core/src/main/java/org/pentaho/di/core/database/Database.java
  core/src/main/java/org/pentaho/di/core/database/DatabaseInterface.java
  core/src/main/java/org/pentaho/di/core/database/MariaDBDatabaseMeta.java
  core/src/main/java/org/pentaho/di/core/database/MySQLDatabaseMeta.java
  core/src/main/resources/org/pentaho/di/core/database/messages/messages_en_US.properties
  core/src/test/java/org/pentaho/di/core/database/MariaDBDatabaseMetaTest.java
  core/src/test/java/org/pentaho/di/core/database/MySQLDatabaseMetaTest.java

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes (click to expand)
org.pentaho.di.job.entries.evalfilesmetrics.JobEntryEvalFilesMetrics

Branch Change: -0.4% :small_red_triangle_down:
Complexity Change: -0.5% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.job.entries.evaluatetablecontent.JobEntryEvalTableContent

Branch Change: -1.1% :small_red_triangle_down:
Complexity Change: -1.1% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.job.entries.sql.JobEntrySQL

Instruction Change: -3.4% :small_red_triangle_down:
Line Change: -3.8% :small_red_triangle_down:

org.pentaho.di.trans.steps.dimensionlookup.DimensionLookupMeta

Branch Change: -1.8% :small_red_triangle_down:
Complexity Change: -1.1% :small_red_triangle_down:
Instruction Change: -0.4% :small_red_triangle_down:
Line Change: -0.3% :small_red_triangle_down:

org.pentaho.di.trans.steps.groupby.GroupByMeta

Instruction Change: -0.2% :small_red_triangle_down:
Line Change: -0.3% :small_red_triangle_down:

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change: -3.3% :small_red_triangle_down:
Complexity Change: -3.9% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:
Line Change: -0.4% :small_red_triangle_down:

org.pentaho.di.trans.steps.symmetriccrypto.symmetriccryptotrans.SymmetricCryptoTransMeta

Branch Change: -3.3% :small_red_triangle_down:
Complexity Change: -1.8% :small_red_triangle_down:
Instruction Change: -0.3% :small_red_triangle_down:

org.pentaho.di.trans.steps.textfileinput.TextFileInputMeta

Branch Change: -0.5% :small_red_triangle_down:


Use default generic arguments for some of the Generic* types

This is a review comment split out of https://github.com/Malax/libcnb.rs/pull/28#pullrequestreview-705465664.
If this suggestion ends up being wontfix we can always close this issue out, however I didn't want it to get lost given that the PR was already merged :-)

For some of these Generic* things, I wonder if we can just have default generic arguments.
-- @hone

@Malax Is this now redundant/wontfix after #129?

TBH, I'm not sure what @hone meant with his comment. It might be resolved by #129, but I'm not sure.

I think I now know what this was about. This was a comment from the LayerLifecycle work where we potentially could've set a default for P: Platform to GenericPlatform for example. I think @edmorley is right and this no longer applicable until Rust RFC2532 is finalized.
Feel free to re-open if you think this is still valid.

yeah, not possible right now until that RFC is merged, but we weren't using associated types before.


docs: fixed a link

SUMMARY
Currently link goes to: https://superset.apache.org/docs/docs/installation/installing-superset-using-docker-compose
It should go to: https://superset.apache.org/docs/installation/installing-superset-using-docker-compose
TEST PLAN
Click on the link
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

Thanks for the contribution! 🙏 @ericmand


Support update state by victims in binder for InterPodAffinity and PodTopologySpread plugins

lgtm, cc@binacs. Testing doc: https://xeh61jru4u.feishu.cn/docx/NwrMdx3Hjoi47fxXJf4cXNNOnof. The testing cases related to preemption are at the end.


DMN Engine Rule

Implementation of a DMN Engine Rule in order to show coverage of evaluation of decision tables.

@AntoCuc would you like to propose an implementation in a pull request?

Mmm, I started putting together the code for the registration of the engine events and there appears to be a slight abstraction problem.
The current coverage plugin only works on processes due to the ProcessEngineConfigurationImpl exposing the hooks to register the listeners independently of the initialisation state of the engine. In fact it registers the before initialisation. :(
On the other hand, the DefaultDmnEngineConfiguration does not allow for the registration of the listeners until engine initialisation.
The above can be easily verified by pasting the code below:
<property name="dmnEngineConfiguration.customPostDecisionTableEvaluationListeners">
	<list>
		<bean class="org.camunda.bpm.extension.process_test_coverage.listeners.MyCustomPostDecisionTableEvaluationListener" />
	</list>
</property>

which should, in theory,  register the DMN CustomPostDecisionTableEvaluationListener but fails with:
org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'processEngineConfiguration' defined in class path resource [camunda.cfg.xml]: Error setting property values; nested exception is org.springframework.beans.NullValueInNestedPathException: Invalid property 'dmnEngineConfiguration' of bean class [org.camunda.bpm.engine.impl.cfg.StandaloneInMemProcessEngineConfiguration]: Value of nested property 'dmnEngineConfiguration' is null

Potential solutions:

Expose the DMN-related listeners on the ProcessEngineConfigurationImpl?
Alternatively call the init of the engine a little earlier?

@falko true. But that wouldn't help gathering coverage stats.

What is the expected output of DMN coverage statistics?
You are essentially looking to test inputs and expected output?
Something like: https://github.com/DigitalState/Camunda-Spock-Testing/tree/master/DMN
Would like to understand what the DMN coverage looks like so it can be added to https://github.com/DigitalState/camunda-coverage-generation

Hi @StephenOTT

What is the expected output of DMN coverage statistics?
The expected output would be a percentage assertable in a similar fashion to the BPM process coverage.

You are essentially looking to test inputs and expected output?
Nope. That behavior can already be obtained as in the unit test sample I wrote above.

Would like to understand what the DMN coverage looks like so it can be added to [...]
At the moment I am not trying to address the visual aspect of it. Rather the practical unit testing in order to integrate DMN with Continous Deployment systems.

@AntoCuc but what is your percentage logic based on? The coverage percentage calc seems to be very different compared to bpmn.

@StephenOTT

what is your percentage logic based on?

My understanding is that:
BPMN calculates coverage percentage on a per-method basis and per-class basis vs. the full BPMN diagram.
I am planning to approach DMN in a similar fashion. Ie. the per-method approach will display decisions coverage of a test method in the context of the diagram. In a similar vein the per-class approach will display an aggregate decision coverage in the context of the diagram.
Am I missing something here?

@AntoCuc maybe i am missing something :s .  I am missing what is actually being tested in this case.
What methods and class level stats are being covered?  Are you talking about the expressions and scripts you are adding into a DMN?


Add four-way intersection scenario

Adds the four way intersection scenario with visualization and example.
TODOs:

[ ] Verify collision constraints
[x] Add centerline functions for turns.

Hi Madan, I looked at the video you sent. it seems that the issue was that you initialized a car outside its lane, so its behavior was to rejoin its lane as fast as possible. To get a 4-way intersection with vehicles turning left or right, you can define a lane "turn left", "turn right", "go straight" for each way (east, west, north, south) of the intersection. So that would 12 lanes for this intersection.
The current way to define the lane is not intuitive and quite painful to write down. I took the code from your PR and created a branch in AlgamesDriving.jl "add_4_way_intersection". I wrote a demo with 3 lanes "west straight", "west left", "west right" and tested it so you should be able to run it easily.
You could try to 'rebase' your PR from the branch "add_4_way_intersection" and add the 9 remaining lanes. Then I can merge it to master quickly.
Tell me if you have question about the code in "add_4_way_intersection"

Thanks for adding the turning lanes for west! It was really helpful in understanding how the centerline functions work. I have added the remaining lanes and it seems to be working pretty well. Also updated the visualization code to make it look like the image below.

@simon-lc I noticed this strange behavior where the two cars interchange there positions resulting in a collision, when I increased the amplitude of collision cost. Shouldn't the agents be more cautious about collisions after increasing this value? Also, is there a boolean value that tells if the solution results in a constraint violation? I'd be happy to help, if it doesn't exist yet.
https://user-images.githubusercontent.com/13871064/111500703-46db6a80-876a-11eb-84a4-f6bcb97b3900.mp4

The 4 way intersection visualisation looks good! I'll look at the latest commits in the PR and merge today if everything is okay.
The issue that shows up in the latest video happens when the trajectories of the vehicles are 'entangled'. There might not be any constraint violation because the constraints are only evaluated at the discrete knotpoints on the trajectory.
There are a few ways to avoid this:

increase the number of waypoints,
increasing the radius at which the collision avoidance cost is active (instead of the cost amplitude),
add a bound constraint on the controls to avoid having these large changes in speed and positions.
add a bound constraints on the velocity
For to boolean value indicating the constraint violation at the end of the solve. This should be easy to do. The GameProblem.stats contains all the constraint violations: on the control constraints, the state constraints, the optimality constraints, and the dynamics constraints. You can check the last violations at the end of stats to see what's the constraint
at the end of the solve.


[Bug]:  Issue after updating to 1.3.0

What happened?
Error: Configuration key tenant_id for microso ft-graph mailer is missing. at /httpdocs/vendor/innoge/laravel-msgraph-mail/src/LaravelMsGraphMailServiceProvider.php:53)
PHP: 8.2
Laravel: 9
I believe the issue is in this commit in LaravelMsGraphMailServiceProvider.php:
The requireConfigString method reports missing configuration because of $value = $config[$key]; so for example it looks for $config['tenant_id']; but this is how $config looks at least in my app
[
  'driver' => 'microsoft-graph',
  'host' => '',
  'port' => 587,
  'from' => [
    'address' =><EMAIL_ADDRESS>    'name' => 'Mail',
  ],
  'encryption' => '',
  'username' => '',
  'password' => '',
  'sendmail' => '/usr/sbin/sendmail -bs',
  'markdown' => [
    'theme' => 'default',
    'paths' => [
      0 => '/var/www/vhosts/example.com/httpdocs/resources/views/vendor/mail',
    ],
  ],
  'log_channel' => NULL,
  'mailers' => [
    'microsoft-graph' => [
      'transport' => 'microsoft-graph',
      'client_id' => 'xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxx',
      'client_secret' => 'xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxx',
      'tenant_id' => 'xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxx',
      'from' => [
        'address' =><EMAIL_ADDRESS>        'name' => 'Mail',
      ],
    ],
  ],
]

It used to pull the config this way config('mail.mailers.microsoft-graph.tenant_id', '')
How to reproduce the bug
Update to 1.3.0
Package Version
1.3.0
PHP Version
8.2
Laravel Version
9.0
Which operating systems does with happen with?
Linux
Notes
No response

Hey @galabl thank you for reporting this issue. Unfortunately I'm currently unable to reproduce that. Can you dd() the $config array within the requireConfigString() method in the `LaravelMsGraphMailServiceProvider? and provide me the result?

het @geisi here's the screenshot of dd from requireConfigString().

Below is mail.php config too
<?php

return [

    /*
    |--------------------------------------------------------------------------
    | Mail Driver
    |--------------------------------------------------------------------------
    |
    | Laravel supports both SMTP and PHP's "mail" function as drivers for the
    | sending of e-mail. You may specify which one you're using throughout
    | your application here. By default, Laravel is setup for SMTP mail.
    |
    | Supported: "smtp", "sendmail", "mailgun", "mandrill", "ses",
    |            "sparkpost", "log", "array"
    |
    */

    'driver' => env('MAIL_DRIVER', 'microsoft-graph'),

    /*
    |--------------------------------------------------------------------------
    | SMTP Host Address
    |--------------------------------------------------------------------------
    |
    | Here you may provide the host address of the SMTP server used by your
    | applications. A default option is provided that is compatible with
    | the Mailgun mail service which will provide reliable deliveries.
    |
    */

    'host' => env('MAIL_HOST', 'smtp.mailgun.org'),

    /*
    |--------------------------------------------------------------------------
    | SMTP Host Port
    |--------------------------------------------------------------------------
    |
    | This is the SMTP port used by your application to deliver e-mails to
    | users of the application. Like the host we have set this value to
    | stay compatible with the Mailgun e-mail application by default.
    |
    */

    'port' => env('MAIL_PORT', 587),

    /*
    |--------------------------------------------------------------------------
    | Global "From" Address
    |--------------------------------------------------------------------------
    |
    | You may wish for all e-mails sent by your application to be sent from
    | the same address. Here, you may specify a name and address that is
    | used globally for all e-mails that are sent by your application.
    |
    */

    'from' => [
        'address' => env('MAIL_FROM_ADDRESS', 'hello@example.com'),
        'name' => env('MAIL_FROM_NAME', 'Example'),
    ],

    /*
    |--------------------------------------------------------------------------
    | E-Mail Encryption Protocol
    |--------------------------------------------------------------------------
    |
    | Here you may specify the encryption protocol that should be used when
    | the application send e-mail messages. A sensible default using the
    | transport layer security protocol should provide great security.
    |
    */

    'encryption' => env('MAIL_ENCRYPTION', 'tls'),

    /*
    |--------------------------------------------------------------------------
    | SMTP Server Username
    |--------------------------------------------------------------------------
    |
    | If your SMTP server requires a username for authentication, you should
    | set it here. This will get used to authenticate with your server on
    | connection. You may also set the "password" value below this one.
    |
    */

    'username' => env('MAIL_USERNAME'),

    'password' => env('MAIL_PASSWORD'),

    /*
    |--------------------------------------------------------------------------
    | Sendmail System Path
    |--------------------------------------------------------------------------
    |
    | When using the "sendmail" driver to send e-mails, we will need to know
    | the path to where Sendmail lives on this server. A default path has
    | been provided here, which will work well on most of your systems.
    |
    */

    'sendmail' => '/usr/sbin/sendmail -bs',

    /*
    |--------------------------------------------------------------------------
    | Markdown Mail Settings
    |--------------------------------------------------------------------------
    |
    | If you are using Markdown based email rendering, you may configure your
    | theme and component paths here, allowing you to customize the design
    | of the emails. Or, you may simply stick with the Laravel defaults!
    |
    */

    'markdown' => [
        'theme' => 'default',

        'paths' => [
            resource_path('views/vendor/mail'),
        ],
    ],

    /*
    |--------------------------------------------------------------------------
    | Log Channel
    |--------------------------------------------------------------------------
    |
    | If you are using the "log" driver, you may specify the logging channel
    | if you prefer to keep mail messages separate from other log entries
    | for simpler reading. Otherwise, the default channel will be used.
    |
    */

    'log_channel' => env('MAIL_LOG_CHANNEL'),

    'mailers' => [
        'microsoft-graph' => [
            'transport' => 'microsoft-graph',
            'client_id' => env('MICROSOFT_GRAPH_CLIENT_ID'),
            'client_secret' => env('MICROSOFT_GRAPH_CLIENT_SECRET'),
            'tenant_id' => env('MICROSOFT_GRAPH_TENANT_ID'),
            'from' => [
                'address' => env('MAIL_FROM_ADDRESS'),
                'name' => env('MAIL_FROM_NAME'),
            ],
        ],
    ]

];

FYI:
I tried it on new Laravel project and it works fine, as Laravel 9 reached EOL, I guess this is not at issue at all.

@galabl
This package should be compatible with Laravel 9 and I tested Laravel 9 Projects which work fine with the 1.3.0 version so the issue probably has something to do with your app. From the information provided I do not see any issues with your config. So this should be correct.

@geisi If I remember correctly, my project started as laravel 6 and was updated last time to version 9.
The issue is within mail.php config which I confirmed right now, but my knowledge is not that good to know why is it like that.
As you can see at the start of my mail.php config there's a 'driver' => env('MAIL_DRIVER', 'microsoft-graph'),, I compared it to a fresh laravel mail.php config, and driver doesn't exist so I deleted it and added 'default' => env('MAIL_MAILER', 'log'),(copied from the fresh config file).
After that, this is the dd result from requireConfigString()

Yes this dd is correct and how it should be. Now your problem seem to be solved?

It is, it's just weird that this was solved by renaming driver to default in mail config

Why was this closed?
A "^1.1" dependency on this package proved fatal, as 1.3.0 contains breaking changes, due to commit 2336ce20c which completely changes the supported configuration.
It went from this:
return new MicrosoftGraphApiService(
	tenantId: config('mail.mailers.microsoft-graph.tenant_id', ''),
	clientId: config('mail.mailers.microsoft-graph.client_id', ''),
	clientSecret: config('mail.mailers.microsoft-graph.client_secret', ''),
	accessTokenTtl: config('mail.mailers.microsoft-graph.access_token_ttl', 3000),
);

to this:
return new MicrosoftGraphTransport(
	new MicrosoftGraphApiService(
		tenantId: $this->requireConfigString($config, 'tenant_id'),
		clientId: $this->requireConfigString($config, 'client_id'),
		clientSecret: $this->requireConfigString($config, 'client_secret'),
		accessTokenTtl: $accessTokenTtl,
	),
);

An outdated $config value such as the following would work just fine up until 1.3.0:
[
	"driver" => "microsoft-graph",
	"mailers" => [
		"microsoft-graph" => [
			"client_id" => "foo",
			"client_secret" => "bar",
			"tentant_id" => "baz",
			"from" => [
				"address" =><EMAIL_ADDRESS>				"name" => "Foo Bar Baz"
			]
		]
	],
	"host" => "",
	"port" => "",
	...
	"log_channel" => null
]

So while pre-1.3.0 was a bit specific in its settings access, that's what worked, and legacy Laravel installations that were set up with the "mailers" entry would still work, despite their legacy config format. Now they crash, meaning that while LaravelMsGraphMailServiceProvider now more correctly adheres to Laravel's config system, 1.3.0 was a serious semver violation, and should have been released as a new major version. Upgrading within a major version should be crash safe, period! Composer doesn't care about your READMEs ;-)


Spanish translation

Hi, as an spanish passionate for Cardano I didn't see available Spanish when using Daedalus, so inspired by this thread: https://github.com/input-output-hk/daedalus/issues/637 I decided to translate it into spanish just if you want to add it to the project. Thanks in advance and thank you very much for your work in this project!

I finished this and properly refactored it
https://github.com/input-output-hk/daedalus/pull/716

Closing unmerged in favour of: https://github.com/input-output-hk/daedalus/pull/716


Add the ability to run limited torsion scans on constrained torsions

Description
This PR adds the ability to drive stiff torsions such as rings and double bonds. These changes are general enough to accommodate regular geometry optimization in the future.
Todos
Notable points that this PR has either accomplished or will accomplish.

[x] Find constrained torsions
[x] Generate input for QCFractal
[ ] Submit different kinds of jobs to QCFractal

The JSON specs were changes as follows:

The workflow JSON now includes torsiondrive_static_options and optimization_static_options in the meta section of the workflow and restricted_optimization_options was added to the torsiondrive_input field. An example of the workflow JSON is here
The JSON for the inputs were changed to include a type field to specify if a torsiondrive or optimization job will be run. If it is an optimization job, a constrained_dict was added.
I also changed the name of the jobs to the dihedral it is driving instead of arbitrary names. This will make it easy to see more information at retrieval when querying the database.

I will add the function to submit the jobs to qcfractal once I make the necessary changes there.
Status

[ ] Ready to go

Codecov Report

Merging #22 into master will increase coverage by 0.06%.
The diff coverage is 39.75%.


VSCode Go to Definition

Clicking through to another package in the monorepo should go to src rather than lib.
May be able to do this with paths in jsconfig, but should support jumping to named imports

https://github.com/Andarist/lerna-alias/issues/11

There's a long article here but it mostly relates to TypeScript
https://medium.com/@NiGhTTraX/how-to-set-up-a-typescript-monorepo-with-lerna-c6acda7d4559


sudo is an unspecified dependency

I installed aurutils using makepkg. After setting up a local repository, I tried to build a package using aur sync. The package built fine but it wasn't added to the local database and aur sync terminated with the following error:
/usr/lib/aurutils/aur-build: line 361: sudo: command not found

I believe sudo should be an explicit dependency, or maybe aurutils should fall back to su.

On 24/03/2021 19:09, gillux wrote:

I installed aurutils using makepkg. After setting up a local
repository, I tried to build a package using |aur sync|. The package
built fine but it wasn't added to the local database and |aur sync|
terminated with the following error:
|/usr/lib/aurutils/aur-build: line 361: sudo: command not found |
I believe |sudo| should be an explicit dependency, or maybe aurutils
should fall back to |su|.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/AladW/aurutils/issues/811, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABKASJBTW5HXM4HYVTKNAPDTFITGJANCNFSM4ZXZCAQQ.

AUR packages rely on base-devel being installed, which includes sudo.
I'm not really interested in a fallback to su, as you could write one
yourself and the next guy might ask on a fallback to doas or some other
mechanism.

AUR packages rely on base-devel being installed, which includes sudo.

sudo is not part of base-devel.

My bad, I got confused because I didn’t see it in pacman -Qg base-devel. Just realized I had to run pacman -Sg base-devel instead. Sorry!


Sphere radius bug

Spherical meshes with radius /= 1 do not work.

Closing.  We're only using the unit sphere for now.


Runner cleanup

Automated cleanup on Runner.java

Generify types.
Fix testing loops to terminate after condition is met.
Remove unnecessary casts.
Remove unnecessary returns.
Remove unnecessary boxing and unboxing.
Replace indexOf() == -1 with contains().
Use System.arraycopy where possible.
Use a StringBuilder instead of string concatenation.
Make functions static where possible.

Missed a couple of spots.  Will resubmit.


Azure DevOps Pipeline example

Example of the DevOps Pipeline based on https://github.com/aaronparker/intune-backup-template, but with fixes and improvements (skips jobs in case no change in Intune configuration is detected).
Some variables need to be defined for this pipeline to work (see comments at the YML file top).
Could be useful for the others.

I have added the example to the Wiki instead: https://github.com/almenscorner/IntuneCD/wiki/Getting-started#example-azure-devops-pipeline


Use -isystem instead of -I

Checklist

[ ] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

The outputs section contained an unexpected subsection name. run_exports is not a valid subsection name.
The outputs section contained an unexpected subsection name. run_exports is not a valid subsection name.
The outputs section contained an unexpected subsection name. run_exports is not a valid subsection name.

Seems likely this will fix the referenced issue

@wesm, did this fix your issue?

Yes, it appears to have fixed the issue. Thank you


next/release

:tada: This PR is included in version 3.2.3 :tada:
The release is available on:

v3.2.3
Chrome Web Store
GitHub release

Your semantic-release bot :package::rocket:


death bug

Describe your bug here.
when i die in my mod and wait a few sec then it will crash
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
no

Uploading Friday Night Funkin_ Psych Engine 2023-11-16 19-10-40.mp4…

https://github.com/ShadowMario/FNF-PsychEngine/assets/141778999/90d0dc09-46fe-4aa5-a3b1-44d4f011b324


AssertionError: Expected number of channels to be three, got 3 channels with shape (512, 512, 1).

Hi, when I augment a array with shape (H, W), it is ok sometimes, but failed with the above error sometimes, too. could someone tell me how to avoid it?

@nickshawn You're probably trying to apply augmentations that expect the input image to have 3 color channels. If you have a gray-scale image as your input - which it seems like you do - then you need to make sure you don't use augmentations that work with color channels. For example, an augmenter such as AddToHueAndSaturation would raise that error since it expects the input image to have color channels as far as I can tell.
So the solution is to use augmenters that don't work on color channels or to convert your images to RGB, augment, and then convert back (not ideal performance-wise).


Examples not working - wizard.in cert expired

The examples are no longer working on FF/Chrome due to the wzrd.in ssl cert expiring:
wzrd.in uses an invalid security certificate.The certificate expired on Sunday, November 5, 2017, 3:53 PM. The current time is Monday, November 6, 2017, 2:48 PM.Error code: SEC_ERROR_EXPIRED_CERTIFICATE

This is working on my end. Feel free to re-comment if it's still not working on your end.

The cert was updated, but I'm getting CORS errors now on the examples page on Safari 11.0.3.  On Chrome 63.0.3239.108 it works ok.  Should I open a new issue?
[Error] XMLHttpRequest cannot load http://wzrd.in/multi due to access control checks.
[Error] Cross-origin redirection to https://wzrd.in/multi denied by Cross-Origin Resource Sharing policy: Origin http://regl.party is not allowed by Access-Control-Allow-Origin.
[Error] Failed to load resource: Cross-origin redirection to https://wzrd.in/multi denied by Cross-Origin Resource Sharing policy: Origin http://regl.party is not allowed by Access-Control-Allow-Origin. (multi, line 0)
[Error] XMLHttpRequest cannot load http://wzrd.in/multi due to access control checks.
[Error] Cross-origin redirection to https://wzrd.in/multi denied by Cross-Origin Resource Sharing policy: Origin http://regl.party is not allowed by Access-Control-Allow-Origin.
[Error] Failed to load resource: Cross-origin redirection to https://wzrd.in/multi denied by Cross-Origin Resource Sharing policy: Origin http://regl.party is not allowed by Access-Control-Allow-Origin. (multi, line 0)
[Error] XMLHttpRequest cannot load http://wzrd.in/multi due to access control checks.

I’ll reopen.

Still happening lol.


Repository find() method is missing from type declarations

Environment
No response
Reproduction
N/A
Describe the bug
The docs indicate that there is a find() method directly on the repository:
https://pinia-orm.codedredd.de/api/query/find
It does also work at runtime.
But in the type declarations, the find() method is only available on the query interface:
repo.query().find(id)
It seems to be the type declarations are wrong and the repository interface should have the find() method on it (since it evidently does work at runtime)
Additional context
No response
Logs
No response

This is not a bug but a feature 😉 . To reduce code i tried to use a proxy to query the same functions which are in query. The only thing that was wrong is the type. But this is already fixed in this issue #1863 if you use the edge release channel.
I am also soon releasing 1.9.2. So if you wait i bit you don't need to checkout the release edge channel.

Excellent, thanks @CodeDredd


async http requests with a proxy probably have been leaking for years

async http requests with a proxy probably have been leaking for years. this is a fix for that.

This is not based on snowcone-ltd:main, the diff is not clean

No passive aggressive PRs please


scripts: check_init_priorities: usability updates

Since bb590b5b6e introduced ordinals in the priority sequence, the "same priority" case cannot happen anymore, furthermore the priority value in the script is now the position of the function in the init sequence, so if two devices have the same priority there's something real bad going on.
Drop all the "same priority" handling code and tests, convert the case into ane exception instead. Drop the init stubs as well from the test, they are not required anymore.

The current error messages are a bit cryptic, rework them to make them more meaningful:

add an extra message on the first error to explain what the errors refer to.
rework the error message to be more explicit.
rework the priority string print to use a LEVEL+offset format to somehow highlight that the number is the offset from the level, not the actual priority.
print the init function name in addition to the devicetree path.

Fixes https://github.com/zephyrproject-rtos/zephyr/issues/64386

@kartben took a stab at improving the messages to make this more understandable, unfortunately I'm quite limited in the information I have from the final linker executable and trying to dig more is a bit of a compiler compatibility nightmare, but I think it looks a lot better now, this an example of the new messages:
ERROR: Device initialization priority validation failed, the sequence of initialization calls does not match the devicetree dependencies.
ERROR: /pwmleds <led_pwm_init> is initialized before its dependency /soc/pwm@4001c000 <pwm_nrfx_init0> (PRE_KERNEL_1+7 < POST_KERNEL+4)

What do you think?

Change looks okay, I do not think it should be backported to 3.5 though, it removes a Kconfig (even if the Kconfig does nothing it still removes it) and changes output

I'm happy with either, but this is guaranteed to trigger on some downstream applications so it'd be nice to have the updated message.


Storybook Testing - Add --story flag to run a single test

Replaces --file path/to/story.spec.js with --story StoryName as the way to run
Added documentation on running individual storybook tests and debugging flag

Coverage remained the same at 26.795% when pulling 90e5318bfdddeb0b967a0e04b4f8d4c30adfaea9 on patthiel:storybook-test-enhancements into 46401cc08b88d135cad82a5baa62455aa6315ed7 on linode:develop.


Back to debian in order to support nvidia cards

@netbrain
Just created a docker image by using this branch.
Looks good on my machine with Nvidia driver.
Runs smooth in 4K.
:+1:

@netbrain It's running but I have a strange new effect. If I exit the game, the container still runs in the background.
Seems that the applicartion is not stopping.
Maybe only on my machine?
Would be nice if someone else can test the branch, too.
I see this in the logs:
+ pgrep -f ZwiftApp.exe
+ echo 'Killing uneccesary applications'
+ pkill ZwiftLauncher
Killing uneccesary applications
+ pkill ZwiftWindowsCra
+ wineserver -w
021c:err:combase:RoGetActivationFactory Failed to find library for L"Windows.Devices.Bluetooth.Advertisement.BluetoothLEAdvertisementWatcher"
wine: Call from 00006FFFFF463DF7 to unimplemented function combase.dll.RoUnregisterForApartmentShutdown, aborting
021c:err:combase:RoGetActivationFactory Failed to find library for L"Windows.Devices.Bluetooth.Advertisement.BluetoothLEAdvertisementWatcher"
wine: Call from 00006FFFFF463DF7 to unimplemented function combase.dll.RoUnregisterForApartmentShutdown, aborting

@cmuench Don't know about it not stopping but I did notice my laptop becoming extremely slow after leaving 1.62.0 yesterday. I ended up rebooting the machine just to be able to open firefox. :thinking:

Can confirm the issue of the container not stopping. seems MicrosoftEdgeUpdate is preventing the wineserver from shutting down. Killing it on zwift startup instead.


Literate Programming link goes to a missing page

Literate Programming link goes to a missing page

which one?

Sorry

aaargh. The first link...
Even had a link checker running, digging through all created html - in the docs folder :-)
=> Thanks Ivan!


Pp 175

This didn't seem to be matching our documentation which I believe to be the reason we were seeing trailing QR data.
Please confirm: https://escpos.readthedocs.io/en/latest/imaging.html#dynamic-2d-barcode-1d-28-6b-phx

@ncrozi3r does this match what you were talking about on Monday?


question: "todo-comments isn't loaded. Did you run setup()?" How do I run setup()?

Did you check docs and existing issues?

[x] I have read all the todo-comments.nvim docs
[X] I have searched the existing issues of todo-comments.nvim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
v0.10.0-dev-1438+g8405649f9
Operating system/version
5.10.0-26-amd64
Describe the bug
I installed the plugin through Vundle, and when using the plugin (Running :TodoQuickFix), it tells me "todo-comments isn't loaded. Did you run setup()?"
Steps To Reproduce

Install the plugin through Vundle. Write in your .vimrc Plugin 'folke/todo-comments.nvim
Save your .vimrc and run :PluginInstall
Run :TodoQuickFix

Expected Behavior
The plugin should execute the :TodoQuickFix properly. I'm not sure how to run setup().
Repro
No response

UP

Hm?

Add require("todo-comments").setup() to your lua config, is what worked for me.

Im using lazy.nvim and this is my setup:
	{ 		"folke/todo-comments.nvim", 		dependencies = { "nvim-lua/plenary.nvim" }, 		opts = { 			vim.keymap.set("n", "<leader>lt", "<cmd>TodoQuickFix<cr>", { noremap = true, silent = true }), 		}, 	},
It works for me.

I don't use lazy.nvim, but @rlpowell's example seems to work. Thanks guys.


premads.com

Domain
premads.com
Link
https://www.premads.com/p/spxchantsr4p1md.html
Version
0.0.0
What browsers are you seeing the problem on?
Chrome
What OS are you seeing the problem on?
Windows
(Optional) Anything else?
No response
Code of Conduct

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

https://www.premads.com/p/spfl23320mg.html


Annotated instances from BQRuntime

Bootique provides the ability to annotate binding of classes, but doesn't provide the ability to get annotated class using BQRuntime.

+1 . Class, Class + annotation are Guice DI key flavors. There are also Key, and TypeRef. Key is the most generic.

@RealElias your PR have conflicts with current master.

I don't see any sense in this PR anymore. There is  public <T> T getInstance(Key<T> diKey) method now.

Closing then.


[WIP] stage1/logging: introduce a new journal logging mode

It is similar to the existing log mode that forwards all logs to systemd, with the difference that output will not show up on the console as well (StanddardOutput|Error=journal instead of journal+console).
In addition to that, a read-only view of the pod journal will be bind mounted inside each app, so one more apps can decide to forward output from the pod reading that journal.
Signed-off-by: Fabio Kung<EMAIL_ADDRESS>

Thoughts on this? I'm trying to find a good way to allow app sidecars (containerized inside the pod) to act as log forwarders for the pod.
There's the existing iomux and output streams + socket activation stuff on systemd, but it seems abandoned and it is unclear it was the intended use case. For now, it seemed simpler to make the pod journal available inside apps, so they can read from it and forward logs.

WIP, BindReadOnlyPaths is only supported on systemd >= 233.
However, I'm still interested on design feedback and/or alternatives.

There's the existing iomux and output streams + socket activation stuff on systemd, but it seems abandoned and it is unclear it was the intended use case.

It is unfinished, yes. The usecase is to have a custom stdin/stdout/stderr handler for scenario where we don't want to go through journald (e.g. custom log forwarders) or we need to attach/detach to apps (e.g. detachable interactive sessions). Those forwarder/attachers are in the host environment though, not in the pod itself.

Thoughts on this? I'm trying to find a good way to allow app sidecars (containerized inside the pod) to act as log forwarders for the pod.

Except for the systemd version incompatibility, I'm not much of a fan of the bindmount as it generically tries to sidestep and bend app's FS separation for a very specific purpose. Also, you'll soon realize you can't distinguish stdout vs stderr lines.
Some other options worth considering:

teaching the app not to print on stdout/stderr, but use proper logging to sidecar instead. This would also avoid the journald/io overhead, support structured logging, and looks cleaner to me. But it doesn't offer a generic solution baked in the container runtime, of course.
having a custom stage1 image, with a dependency on stage1-coreos, which just overwrites the iottymux binary or install a drop-in for each app with the bindmount. This would be a bit more generic than the above, but still no in rkt stage1-coreos itself.
teaching stream mode that those entries are handled by a sidecar app. This seems clean but I fear it introduces an implicit ordering on application start.

/cc @alban as they do similar stage1-dependency tricks and may also need some iottymux progress to get rid of journal2cri.

Thanks for the feedback! I'm aware of the stdout/err separation with systemd's journal mode. My plan was  to send a PR to systemd. That doesn't seem too hard to fix (famous last words).
I'm also not as worried with the bind mounts because we do similar things for pod volumes shared across all apps. This would be a special (implicit?) case of that.
As for the options you suggest, I considered some of them. Some more thoughts inline:

teaching the app not to print on stdout/stderr, but use proper logging to sidecar instead. This would also avoid the journald/io overhead, support structured logging, and looks cleaner to me. But it doesn't offer a generic solution baked in the container runtime, of course.

We run arbitrary code, and it would be pretty hard to enforce and/or prescribe that for all apps. It's a different avenue that we may explore at some point, but for now not very realistic.

having a custom stage1 image, with a dependency on stage1-coreos, which just overwrites the iottymux binary or install a drop-in for each app with the bindmount. This would be a bit more generic than the above, but still no in rkt stage1-coreos itself.

I started this that way, but soon realized that customizing units generated by stage1-coreos is not trivial. It is currently done by the <stage1>/init entrypoint, which immediately execs into systemd. Customizing units means patching the source code for /init and maintaining a fork of it. There are no good hooks or good ways to wrap it and add unit drop-ins.
This may be a good area to focus on a different PR: extensibility of systemd units generated by stage1-coreos. In which case I'd be happy to maintain this as an alternative stage1 image, instead of baking it onto vanilla rkt if you are not interested in having apps reading from the pod journal.

teaching stream mode that those entries are handled by a sidecar app. This seems clean but I fear it introduces an implicit ordering on application start.

Ordering on app start is something I will need to work on anyway, because even with journald, graceful shutdown of a Pod means we need to stop the log uploader sidecar last, and guarantee that all shutdown logs get shipped as well.
My reservation about using the stream mode is having to re-implement a bunch of stuff we get from journald "for free": structured logging, buffering and rate limiting, inspection from the host, configuration knobs, and (most importantly) avoid blocking apps writing to stdout/err.

@fabiokung the following code adds some mounts via systemd units for each app without patching /init, using a systemd templated unit:
<EMAIL_ADDRESS>https://github.com/kinvolk/stage1-builder/blob/master/builder#L133

@alban ah, drop-ins on<EMAIL_ADDRESS>is a neat trick I didn't know! Thanks, that works.

Closing this, I'll keep it as a separate stage1 image, and break up the /etc/machine-id changes in stage2 into another PR.


source.js.jsx scope set even when not in a JSX block

AFAICT from reading the source language-babel only emits the source.js.jsx scope for Javascript, and not the source.js scope. This means that linters that don't grok JSX like JSHint will refuse to lint any of the buffer.
Would it be possible to set the scope source.js outside of JSX blocks?

I doubt I'll change this. I use eslint but I believe you can toggle on 'Lint JSX files' and then jshnt will work with my grammar.