Asura Scans can't download every chapters

Since #762, we can't download any chapter.
The source only downloads one or two pages max.
If we open the chapter without downloading, there's no problem.
I though I was the only one, but more and more people complaining about it on Discord.
Maybe it's a Aidoku issue, but I only have that issue on that source

this is likely something aidoku the app will have to work around. there's an issue on the app repo already


Decorators incorrectly handle emojis

We're running into some bugs around decorating text that has emojis in it. Since a screenshot is worth a thousand words:

Note how the @rauchg mention is misaligned by a couple of characters. However, if the emoji is removed from the initial text it all works perfectly:

Running<EMAIL_ADDRESS>Since I'm a nice open source citizen™ here you have a full reproduction of the bug to play around with: https://codesandbox.io/s/y2rpn3py3x

Good catch!
Should be fixed now.
Bumped version to 0.9.1-beta2 here: https://9ow1j8q1zp.codesandbox.io/
(This version also should have #33 resolved)

I can confirm this is fixed by upgrading to 0.9.1-beta2!


Fixbug arrows.js bug

currentSlide and SlideCount Warnings. #1238
customArrow can not get the class and style

It's fine, but you can try create variable prevArrow and nextArrow from destructuring before creating variables with styles and className. e.g. const { prevArrow } = this.props Thanks to this, the statement will be simpler. e.g. const prevArrowStyle = (prevArrow && prevArrow.props && prevArrow.props.style) || {};

Thank you for your advice, if we want to make the statement simpler, I think we should refactored the render function. e.g.
    const { prevArrow, infinite, currentSlide, slideCount, slidesToShow } = this.props;
    ......
    let prevArrowElement
    ......

Maybe I'll create a new PR after this PR was merged.

That would be the best.

Is there any update on this PR? Because i still get the error.


Fix parent/sub community mapping

See https://jira.duraspace.org/browse/DS-2827

+1 I have tested the before and after of this, and this is a major fix. Before, the XMLUI community-list was in a completely unfamiliar hierarchy sub-communities at the top level, so more like anarchy, and now it is the typical hierarchy.
I have noticed that the sub-communities are not ordered A-Z, but rather Z-A


Unable to run scripts on contracts with linked libraries

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (4a0c8dc 2023-04-15T00:09:32.582313000Z)
What command(s) is the bug in?
forge script
Operating System
macOS (Intel)
Describe the bug
Running scripts fails when including linked libraries. Removing the linked library or converting it to an internal library allows skips the bug. Fails on both solidity 0.8.13 and 0.8.17
Need to use linked libraries here because of contract space constraints.
Logs:

stderr:
tderr: 31
stderr: mThe application panicked (crashed).
Message:  called Option::unwrap() on a None value
Location: /Users/runner/work/foundry/foundry/cli/src/cmd/forge/script/executor.rs:48
This is a bug. Consider reporting it at https://github.com/foundry-rs/foundry
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ BACKTRACE ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
1: __mh_execute_header
at :
2: __mh_execute_header
at :
3: __mh_execute_header
at :
4: __mh_execute_header
tderr: 90m
at :
5: __mh_execute_header
at :
6: __mh_execute_header
at :
7: __mh_execute_header
at :
8: __mh_execute_header
tderr: 90m
at :
9: __mh_execute_header
at :
10: __mh_execute_header
at :
11: __mh_execute_header
at :
12: __mh_execute_header
tderr: m
at :
13: __mh_execute_header
at :
14: __mh_execute_header
at :
15: __mh_execute_header
at :
stderr:
Run with COLORBT_SHOW_HIDDEN=1 environment variable to disable frame filtering.
Run with RUST_BACKTRACE=full to include source snippets.

update: we've fixed it by adding a test with incorrect import syntax (importing a contract that doesnt exist)
no clue why this works. its just a load bearing test now. the bug still exists

Hey @chlwys — could you retry this and see if it's working now? We just fixed a major bug in linking would like to see if this tackled it.

update: we've fixed it by adding a test with incorrect import syntax (importing a contract that doesnt exist)

ping @chlwys could you check if this is still an issue?

Marking as resolved
Was able to successfully deploy a contract with a linked library


Better error message when no kwargs passed to filter_grad

Hi Patrick,
I encountered incorrect error message during using eqx.filter_grad:
import equinox as eqx
from jax import numpy as jnp, tree_util as jtu
@eqx.filter_grad
def f(a):
    return jnp.sum(a)
eqx.filter_grad(jtu.Partial(f, a=jnp.ones((3,2))))()

Throws
Functions wrapped with `equinox.filter_{grad, value and grad}` must have their
first argument passed by position, not keyword. (This is the argument that is differentiated).

which does not make sense in this scenario: there are no keyword arguments!
I simply added a check for no keyword arguments, that now states that ... there must be at least one positional argument ...  when no kwargs are available.

LGTM! Thank you for this improvement. :)


Exception while session.save() "is not a valid entity class. Please check the entity mapping."

Expected Behavior
DB to create objects
Current Behavior
Getting exception during session.save()
Exception in thread "main" java.lang.reflect.InvocationTargetException
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:498)
at org.springframework.boot.loader.MainMethodRunner.run(MainMethodRunner.java:48)
at org.springframework.boot.loader.Launcher.launch(Launcher.java:87)
at org.springframework.boot.loader.Launcher.launch(Launcher.java:50)
at org.springframework.boot.loader.JarLauncher.main(JarLauncher.java:51)
Caused by: java.lang.IllegalArgumentException: Class class DataObjects.Movie is not a valid entity class. Please check the entity mapping.
at org.neo4j.ogm.session.delegates.SaveDelegate.save(SaveDelegate.java:88)
at org.neo4j.ogm.session.delegates.SaveDelegate.save(SaveDelegate.java:40)
at org.neo4j.ogm.session.Neo4jSession.save(Neo4jSession.java:469)
at Controller.SectionController.main(SectionController.java:118)
Possible Solution
No Idea
Steps to Reproduce (for bugs)

Just following the example mentioned here
...
...
...

Context

Your Environment

OGM Version used: 3.1.0
Java Version used:
Java(TM) SE Runtime Environment (build 1.8.0_144-b01)
Java HotSpot(TM) 64-Bit Server VM (build 25.144-b01, mixed mode)
Neo4J Version used:
neo4j 3.3.4
Bolt Driver Version used (if applicable):
Operating System and Version:
mac book, High Sierra 10.13.4
Link to your project:

This is in general not a bug. Without code it is hard to help you or analyse the problem.
I can only guess:

Defined package(s) do not match the Movie class in the SessionFactory creation
Some kind of inheritance issue (but that is very vague)

==============Neo4jSessionFactory.java================================
package DataObjects;
import org.neo4j.ogm.config.Configuration;
import org.neo4j.ogm.session.Session;
import org.neo4j.ogm.session.SessionFactory;
public class Neo4jSessionFactory {
public static final String NEO4J_URL = "http://localhost:7474";
public static final String USERNAME = "neo4j";
public static final String PASSWORD = "pin!20BIT";
Session session;
// prevent external instantiation
public Neo4jSessionFactory() {
     Configuration configuration = new Configuration.Builder() .uri(NEO4J_URL)
            .credentials(USERNAME, PASSWORD) .build();
     SessionFactory sessionFactory = new SessionFactory(configuration, "DataObjects");
     session = sessionFactory.openSession();
}

public Session getNeo4jSession() {
    return session;
}

}
==============Movie.java================================
package DataObjects;
import org.neo4j.ogm.annotation.*;
import java.util.Set;
@NodeEntity
public class Movie {
@Id @GeneratedValue
private Long id;
private String title;
private int released;

@Relationship(type = "ACTS_IN", direction = "INCOMING")
Set<Actor> actors;

public Movie() {
}

public Movie(String title, int year) {
    this.title = title;
    this.released = year;
}

public Set<Actor> getActors() {
    return actors;
}

public void setActors(Set<Actor> actors) {
    this.actors = actors;
}

}
==============Code to kick of creation================================
   Neo4jSessionFactory sessionFactory = new Neo4jSessionFactory();
    Session session = sessionFactory.getNeo4jSession();

    Movie movie = new Movie("The Matrix", 1999);

    Actor keanu = new Actor("Keanu Reeves");
    keanu.actsIn(movie);

    Actor carrie = new Actor("Carrie-Ann Moss");
    carrie.actsIn(movie);

//Persist the movie. This persists the actors as well.
session.save(movie);

Was this issue solved?

Same issue. I have set @NodeEntity(label="Company"), still
Company is not a valid entity class. Please check the entity mapping.
at org.neo4j.ogm.session.delegates.SaveDelegate.save(SaveDelegate.java:88)
at org.neo4j.ogm.session.Neo4jSession.save(Neo4jSession.java:474)
at com.abhsinh2.neo4j.test2.GenericService.createOrUpdate(GenericService.java:28)
I am using 3.1.0 ogm and driver

Regarding the problem of @asinha0884 we found the underlying problem is that the class path scan only works with lower case packages as the Java specs describe. Related SO question: https://stackoverflow.com/questions/12534302/why-should-java-package-name-be-lowercase
Short version: technical it is allowed to use package name with uppercase characters but you shouldn't.
For @abhsinh2 I would like to know who the project looks like and if it would be better suited to post it as a question over at StackOverflow.

I close this issue now. Please re-open it if needed.


Standalone package SCCM OSD deployment information

The Surface UEFI Configurator states it is deployable but throws a 1603 error when deployed in a task sequence. From TechNet forum search this appears to be because the task sequence runs it as the SYSTEM account. Some clarification on how to install this during SCCM OSD Task Sequence would be nice.

Hello, @randomman87
thank you for providing feedback.
The issues section of this repository is intended for product documentation issues only.
Please consider opening a product support ticket by the following link below for your problem so that it gets resolved quickly.
Windows 10 Support
Thank you.


Incorrect refraction elevation cutoff in ephemeris solar position algorithm

The refraction correction used in the ephemeris solar position algorithm has a cutoff at -1 degree, which is not mentioned in the original publication or the provided Fortran code:
https://github.com/pvlib/pvlib-python/blob/6d886dc0fa8e0fa1d011e9332f609cee526c9bbd/pvlib/solarposition.py#L839
To be consistent with the publication I suggest that we remove the lower limit and apply refraction correction to all negative solar elevation angles.

ICYMI @IoannisSifnaios @kandersolar

The reference publication is here: https://pvpmc.sandia.gov/app/uploads/sites/243/2022/10/Engineering-Astronomy.pdf
It doesn't mention correction below -0.575 degrees, that I can find.
My guess is that the -1 degree limit was added to the Matlab code because we didn't think refraction correction was meaningful when the sun's disc was below the horizon.
I have no objection to removing the limit, or adding a comment pointing back to the Matlab code. I also have no objection to removing the ephemeris function as long as there is an equally fast solar position algorithm in the library.

+1 to removing ephemeris eventually, and leaving it as-is in the meantime.  But good to document the issue regardless.
A bit of context: @AdamRJensen, @IoannisSifnaios, and I have a project underway comparing solar position algorithms.  We plan to contribute implementations of one or more algorithms to pvlib-python, depending on the results of the comparison.  We expect some of them to be better than ephemeris in all respects (speed, accuracy, period of validity, clear reference).


Add Animation when changing the slider

Hi,
First of all, this is an Amazing App!! But, when I change the volume of the device using my keyboard, the transition is quite staccato: There is no transition between the position of the slider. It would be amazing  if you could add a sliding keyframe animation in between the two positions. For reference, this is a video of the slider progressing:
https://we.tl/t-39YRFW1cMb
I know this isn't a very good description, but I tried my best.

Nice suggestion 😃. But that is the default behaviour of the WPF slider control. I don't if other slider controls are animated within Windows. I will try my best to implement fluid transitions all over the app.

Hey @akhilesh-balaji, this is a nice suggestion but the said transition won't be practically good. I can't find a way to smooth it.
So this suggestion may not be implemented


Add SD/MMC support for FRDM-K64F

This pull request allows to build and run the fat_fs example on the Kinetis FRDM-K64F board. The DTS files are modified to include the SPI1 port and its connection to the microSD card slot on the board.

Added missing Signed-off-by, hope it's the last thing needed to merge the PR.

@tilk If you are on a linux machine, running the check_compliance.py script may help you eliminate some of these issues. You can run it like this: python3 ${ZEPHYR_BASE}/scripts/ci/check_compliance.py -c main..HEAD to check all commits between your copy of main and your current HEAD, or run it without -c to only check the latest commit.

@tilk I believe you can resolve this issue by running west update.

@danieldegrasse This indeed solved the issue, thank you.


Generated React Native files do not work with storybook in a vanilla app.

I was putting together a RN app for https://github.com/storybooks/storybook/issues/1012, and ran into. I followed the following steps:
npm install -g react-native @storybook/cli
cd examples
react-native init ReactNativeVanilla

cd ReactNativeVanilla
# test that RN works :+1:
react-native run-ios

getstorybook
yarn run storybook
# storybook server/UI runs on port 7007

# run the app again, should see stories
react-native run-ios

At this stage I should see stories in the simulator, ie, something like

However, what I see is the standard starting app:

And no stories show in the UI.
I believe the problem was introduced by https://github.com/storybooks/storybook/commit/3e2673b96297a948fa11ed040b61b81686385a94, which means that getstorybook creates storybook/index.js rather than storybook/index.[ios|android].js. Changing the filename myself and restarting things fix the issue.
I can't find documentation but I think the core problem is in setting --projectRoot=[configDir,projectDir] here https://github.com/storybooks/storybook/blob/master/app/react-native/src/bin/storybook-start.js#L49, the RN packager will look for and index.ios.js in configDir (i.e. ./storybook), and failing that, revert to the the projectDir.

This issue should be labelled merged but that label is gone, not sure why (@shilman?)

@tmeasday No idea what happened to merged?

This went out last week. Follow up issue: #1277


Release notes for 3.4.15

I think @DrDaveD's comment about this not necessarily being an issue for EL6 applies here

I add EL7 to the note. However, it does state on systems that support the overlay file system, EL6 does not have that capability. So, it is more specific (And over specified.)


Iframe videos are not working on https

we should only accept videos (or images, or any content) only under https URLs. This used to work.


UI test recorder

Provide UI test recorder.
This functionality was available in Xamarin Test Recorder but it was deprecated on March 31, 2018.
Recording test is a quick way to create a simple smoke UI test in few steps.

I'm hoping the feature request gets more reactions, thanks for the feedback.


Quality of Life Updates to Single Device Articles

@tomVcass : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Thanks!
#sign-off

Hi @tomVcass - This pull request was opened in the public repo. Would you make sure updates in the private repo? Thanks! We'll merge this one, though.
Thanks, all.


iOS crashes/quits on 'standard build'

When compiling/running my app based on mvcExpress 1.4.2. it crashes without throwing any kind of error/message. It just quits without notice. This behavior is strangely only showing when compiling it with the 'standard build' option in Flash Builder, using the 'fast build' shows no problems and the App works just fine. Also, Android is not affected.
I've managed to trace it down to my MainModule.as file
	public class MainModule extends ModuleCore
	{
		override protected function onInit():void
		{
			trace(this, 'onInit called');
			// it kills the app on the next line
			commandMap.execute(SetupControllerCommand);
			commandMap.execute(SetupModelCommand);
			commandMap.execute(SetupViewCommand);
		}

I'm using Adobe Air SDK 18.

Can you try it with version 2? It might be foxed there, and its release ready.
I will also check 1.4 at weakend.

Hi, thanks for the reply! I've just upgraded to version 2.0rc3 but to no avail, keeps on crashing on the same line.

wierd.. :( will check.

hm.. I failed to reproduce it on my system.. and have to go away for 2 weeks.. :(
I will be able to continue my investigation after I get back...
Sorry for inconvenience.


Config send by mail kuikoo?

I want to config to send mail by kuikoo. This is my config:
var transporter = nodemailer.createTransport({
    host: 'smtp.kuikoo.com',
    port: 465,
    secure: true,
    auth: {
        user: 'user',
        pass: 'pass'
    }
});

When i send mail, There is a error : "[Error: SELF_SIGNED_CERT_IN_CHAIN]". Anyone can help me pls?

This is because the server uses self signed certificate which is not allowed by default for security reasons. If you want to bypass this, use additional config option tls
var transporter = nodemailer.createTransport({
    host: 'smtp.kuikoo.com',
    port: 465,
    secure: true,
    auth: {
        user: 'user',
        pass: 'pass'
    },
    tls: {
        rejectUnauthorized: true // allow invalid certificates
    }
});

@andris9 i tried but it have error:
{ [Error: Invalid login]
  code: 'EAUTH',
  response: '535-5.7.8 Username and Password not accepted. Learn more at\n535 5.7.8 http://support.google.com/mail/bin/answer.py?answer=14257 bw3sm4508881pbb.9 - gsmtp',
  responseCode: 535 }

I want to use mail kuikoo but why there is google support?

@andris9 I have check account mail but user and password is very correct. Why invalid login? :(

This is a response from Gmail, are you sure you do not have the host set as smtp.gmail.com, or the service property set as gmail?

sorry that's my fault. This is error when i try:
{ [Error: Can't send mail - all recipients were rejected]
  code: 'EENVELOPE',
  response: '553 5.7.1<EMAIL_ADDRESS>Sender address rejected: not owned by user<EMAIL_ADDRESS>  responseCode: 553 }


Fix parsing slowness

Resolve #1108

I'm still not sure on the specifics, but there must be some large dependency or something inside the waverider module that was causing the slowness. Maybe some reliance on typescript module or something that is large, according to Google. Either way, we were getting hit by this bug because of our parser setup. This caused slowness and memory usage issues. I did the fix in that thread a few comments down and it is significantly faster to parse and lint waverider module, and as a result, this also fixes slowness in the formula module

You can get timings for specific eslint rules by setting an environment variable TIMING=1, then running npx eslint <files>.
I tested this way on a formula module file, but all rules ran in < 5 ms, so there was no issue with rules. Some comment on Stack Overflow led me to believe it was related to parsing.

You can get timings for parsing/all of eslint by setting an environment variable DEBUG=eslint:*. This produces a lot of fluff, and once I narrowed it down to parsing/linting, I set the env varable DEBUG=eslint:linter.
Running this, it was taking 7s to parse Nilou's entire 600-line sheet. Whereas formula file of 61 lines took 28s

Looking up stuff related to parsing slowness in monorepo, I ended up at the link in the description. Making the changes in this PR changed Nilou to 10s parse, and formula file to 9s parse. There was also clearly noticeable speedups in linting within vscode for formula module. Before, I had to wait 30s-1 minute for lint errors to show up, and now they are showing instantly, similar to our other modules.

Linting time for another PR was 9 minutes https://github.com/frzyc/genshin-optimizer/actions/runs/4833269220
Linting time for this PR was 2.5 minutes https://github.com/frzyc/genshin-optimizer/actions/runs/4834551217

Just as a tldr, before, if we had some dependencies between modules, we would load each dependency everytime, very redundantly. So if we had a dependency like

A depends on C
b depends on C

We would actually have to generate the typing/ast for C and have 3 copies of it in memory. Once for A's dependency, once for B's dependency and once for C itself
The change makes it so our project is generated as a whole for linting and loaded entirely in memory. So it should be okay as long as our project is not humongous, though I don't know what qualifies as humongous

Waiting to see if nx 16 fixes or breaks this


The fine-tuned Gemma model encounters an error when loaded through vllm: KeyError: 'lm_head.weight'

Reminder

[X] I have read the README and searched the existing issues.

Reproduction
deepspeed --include="localhost:0,1,2,3,4,5,6,7" src/train_bash.py
--stage sft
--do_train
--model_name_or_path gemma-7b
--dataset XXX.json
--overwrite_cache
--template gemma
--finetuning_type full
--cutoff_len 3072
--output_dir ${OUTPUT}
--per_device_train_batch_size 2
--gradient_accumulation_steps 4
--lr_scheduler_type cosine
--dataloader_num_workers 5
--logging_steps 10
--save_steps 500
--learning_rate 1e-5
--save_only_model
--num_train_epochs 15
--bf16
--save_total_limit 2
--report_to wandb
--flash_attn
--deepspeed configs/ds_stage_2.json
After fine-tuning all parameters of the gemma-7b model, it can be decoded normally using src/web_demo.py.
However, when loading the model through vllm, the following error is:
  File "/opt/conda/lib/python3.10/site-packages/vllm/entrypoints/llm.py", line 109, in __init__
    self.llm_engine = LLMEngine.from_engine_args(engine_args)
  File "/opt/conda/lib/python3.10/site-packages/vllm/engine/llm_engine.py", line 391, in from_engine_args
    engine = cls(*engine_configs,
  File "/opt/conda/lib/python3.10/site-packages/vllm/engine/llm_engine.py", line 128, in __init__
    self._init_workers()
  File "/opt/conda/lib/python3.10/site-packages/vllm/engine/llm_engine.py", line 181, in _init_workers
    self._run_workers("load_model")
  File "/opt/conda/lib/python3.10/site-packages/vllm/engine/llm_engine.py", line 1041, in _run_workers
    driver_worker_output = getattr(self.driver_worker,
  File "/opt/conda/lib/python3.10/site-packages/vllm/worker/worker.py", line 100, in load_model
    self.model_runner.load_model()
  File "/opt/conda/lib/python3.10/site-packages/vllm/worker/model_runner.py", line 88, in load_model
    self.model = get_model(self.model_config,
  File "/opt/conda/lib/python3.10/site-packages/vllm/model_executor/utils.py", line 52, in get_model
    return get_model_fn(model_config, device_config, **kwargs)
  File "/opt/conda/lib/python3.10/site-packages/vllm/model_executor/model_loader.py", line 86, in get_model
    model.load_weights(model_config.model, model_config.download_dir,
  File "/opt/conda/lib/python3.10/site-packages/vllm/model_executor/models/gemma.py", line 337, in load_weights
    param = params_dict[name]
KeyError: 'lm_head.weight'

The script for loading the model in VLLM is as follows:
 llm = LLM(
      model=model_path,
      trust_remote_code=True,
      tensor_parallel_size=1,
      gpu_memory_utilization=0.9,
      )

Expected behavior
No response
System Info

transformers version: 4.38.2
Platform: Linux-4.19.91-kangaroo.<IP_ADDRESS>c7cc891b.alios7.x86_64-x86_64-with-glibc2.35
Python version: 3.10.13
Huggingface_hub version: 0.21.3
Safetensors version: 0.4.1
Accelerate version: 0.27.2
Accelerate config:    not found
PyTorch version (GPU?): 2.1.2+cu121 (True)
Tensorflow version (GPU?): 2.14.0 (False)
Flax version (CPU?/GPU?/TPU?): not installed (NA)
Jax version: not installed
JaxLib version: not installed
vllm: 0.3.3

Others
No response

Related Issues:
https://github.com/vllm-project/vllm/issues/3323


Cleanup image building jobs

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #
Does this PR introduce a user-facing change?:

Checklist

[ ] Added unit test, integration, and/or e2e tests
[ ] Tested locally
[ ] Updated documentation
[ ] Update Swagger spec if the PR introduce API changes
[ ] Regenerated Golang and Python client if the PR introduce API changes

Please take a look at the early draft PR to automate pyfunc image building cleanup @pradithya @tiopramayudi

Can you update the PR description before merging?


Dashboard

The Prometheus exporter (see havfo/multiparty-meeting#222, prom.md) could serve as the basis for a monitoring dashboard (see havfo/multiparty-meeting#381). Users in environments without existing Prometheus monitoring may benefit from a related new capability inside docker-compose.yml as follows:

activate Prometheus exporter in configs/server/config.js
deploy Node Exporter from prom/node-exporter
deploy Prometheus from prom/prometheus
configure jobs mm-exporter and node-exporter into Prometheus
deploy Grafana from grafana/grafana
configure dashboard Multiparty Meeting with panels Mediasoup (as pictured in havfo/multiparty-meeting#222) and Rooms (e.g. as table panel) into Grafana
configure e.g. dashboard Node Exporter Full into Grafana

@misi I have moved on to other things by now. I had asked for feedback back in May and since there were only two related votes (see above), it did not seem worthwhile to pursue this further in terms of preparing a PR.

@christian-2 I am so sorry I was busy with other duties unfortunately. Thanks for your fast answer!


Validate the existence of fieldA xor fieldB

I started with cue-lang and aim to use it to validate YAML. I want either to have a field called item or items, but not both and I am wondering how I can achieve that...
My cue file looks like this:
item?: string
items?: [...string]

And I suppose I need to add a check that checks if either of them is set, but not both.
(Writing the issue is almost rubber duck debugging - I might resolve the issue tomorrow already; hope that helps someone else)
Valid YAML
item: abc

or
items: [abc, cde]

Invalid YAML
item: abc
items: [abc, cde]

See the numexist builtin function proposed in #822. It's not implemented today, but that's the intended way to cover this case.

You could use the anyOf work around with an upper limit as well.
https://cuetorials.com/patterns/fields/#anyof

Thanks for raising this @ChristianKniep
@seh is right, this is covered in #822 so will close this as a duplicate.


New OP for issuer to reclaim fee pool funds

Two points about the fee pool:

The fee pool of an asset can be drained by anyone that has a share in that asset by creating an order paying the fee in that asset and cancelling the order again. He gets refunded 90% in BTS not the asset.
When creating a new asset, the fee pool is prefunded by 50% of the fee. This fee is deducted from the basic member fee. Recently @Rune has bought a 3-letter symbol (MKR) and payed quite a bit for it with the expectation of a 80% refund after 90 days! He won't get 80% after 90 days but only 40% because the fee pool funds have been deduced from the total amount! He could now drain the fee pool him self (which is cumbersome and time consuming) or we offer a way to collect all of them instantly.

This should really exist as there is no real reason (AFAIK) that should prevent the issuer of an asset to also full control funds in the fee pool.
I propose a new claim_pool_balance_operation that takes

the asset,
the amounts of BTS to claim, and
the account to credit

as arguments.
~I consider this high priority to get this included in the next hard fork (if by any means possible)~

Assume the core exchange rate of the asset is 1 asset = 1 CORE, assume there is X CORE in fee pool, the issuer can claim funds from fee pool easily with following steps as a work around, no matter how much CORE is in the fee pool:

issue X assets to herself
craft an limit_order_create_operation which will never be filled, set the fee field to X assets, sign & broadcast it
cancel the order

I believe @xeroc can make a python script to batch these operations, no need to write C++ code.

@abitmore
The approach you describe is exactly what I meant by "pool draining" in the OP.
I see it as a hacky solution that cannot be used if you have your asset traded already.

@xeroc what did you mean "have your asset traded already"? IMO the issuer can withdraw from fee pool with that approach at any time, anyone else can do that as well if have enough issued assets.

The approach I provided is just not too "cumbersome and time consuming" as you said in OP.

This issue was moved to bitshares/bitshares-core#188


Update handling of workshops for rollover memberships

When a membership is rolled over to a new membership, we have the option of carrying over unused benefits.  The number of workshops that can be rolled over should correspond with the number of workshops of each type in the source membership.
For example, if a member site had 2 inperson and 1 online workshop remaining, they may roll over at most 2 inperson workshops and 1 online workshop.

Location: fiscal/views.py > MembershipCreateRollOver

Update get_initial and get_form_kwargs and form_valid to pass through the rolled over values for all changed fields. You can copy the lines that are there for the original fields.


Resume Cluster sessions

Describe the enhancement you'd like to see
When I'm not designing manifests I usually debug some resources in K8s.
The issue is that I want to have everything reloaded right after Monokle start,
and see last session where I ended - perfectly with recent resources.
It's also about faster reloading of resources (previous issue about use filters when reloading)
Maybe we could start listening on K8s events somehow and reload stuff in background to make Monokle feels more "real-time" with cluster resources.

This is related to https://github.com/kubeshop/monokle/issues/708
I like the idea of saving the session to get back where you ended, thanks for the feedback!

This could be done as a follow-up to https://github.com/kubeshop/monokle/issues/2087

Closing as it has been addressed with cluster mode


React to QeweEntry priority changes

Suggested behaviour:
const queue = new Qewe({ reactive: true });

const firstEntry = queue.enqueue('hello', 1);
const secondEntry = queue.enqueue('world', 2);

console.log(...queue); // [ 'world', 'hello' ]

firstEntry.priority = 3;

console.log(...queue); // [ 'hello', 'world ]

This should be possible by returning a Proxy from .enqueue that wraps the QeweEntry's priority setter.

Working on this over in #8


Update README.md

Hi Leandro, I saw your learning resources and I think are great. I just add a new one from Nadrej karpathy I think it is also a very good resource. Best regards.

Hi Serge! Thanks a lot for your contribution. It's a pleasure!
Happy Learning!!


Fix and improve FluxFM connector

Changelog:

fixed FluxFM connector;
added album field;
added coverart.

Thanks Alex!


[New game]: 2048 single-player sliding tile puzzle

🎮 Game Request
Game logic and basic description: 2048 is played on a plain 4×4 grid, with numbered tiles that slide when a player moves them using the four arrow keys. Every turn, a new tile randomly appears in an empty spot on the board with a value of either 2 or 4. If two tiles of the same number collide while moving, they will merge into a tile with the total value of the two tiles that collided. If a move causes three consecutive tiles of the same value to slide together, only the two tiles farthest along the direction of motion will combine. The game is won when a tile with a value of 2048 appears on the board. Players can continue beyond that to reach higher scores. When the player has no legal moves (there are no empty spaces and no adjacent tiles with the same value), the game ends.
Point down the features
Game points: A scoreboard on the upper right keeps track of the user's score. The user's score starts at zero and is increased whenever two tiles combine, by the value of the new tile.

Select program in which you are contributing
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @thevirengarg !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign

Hey @thevirengarg ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


KOGITO-8178: [SWF Editor] Enable validation tab for Serverless Logic Web Tools

Jira: https://issues.redhat.com/browse/KOGITO-8178
CC: @Sara4994
Special thanks to @caponetto, who helped me finding a solution.
Description:
Validation for YAML is not enabled on WebTools, the "Problems" tab is disabled.
On VS Code, it works fine because this validation comes from VS Code itself.
The problem:
To validate YAML SWF I used "yaml-language-server", which allow you to apply a validation schema.
I passed all the YAML validation tests in "serverless-workflow-language-service" using the Red Hat  "yaml-language-server" directly.
The problem I had was compiling the VSCode Extension and the WebTool, which have problems with the way "yaml-language-server" is importing external libraries.
I tried importing "yaml-language-server" using CJS, ESM or UMD formats, because "yaml-language-server" gives you these options, but it didn't work.
Mainly I had problems due to imports from "prettier".
The solution:
The solution was to create a package named "@kie-tools/yaml-language-server" to bundle the real "yaml-language-server" with a few dependencies.
I tried a lot to create a bundle with Webpack, but I had issues with some dependencies, mostly "vscode-json-languageservice" and "ajv".
In the end, I created a bundle using Esbuild in a similar way to monaco-yaml.
Notes:

In packages/yaml-language-server/package.json I didn't manage to test it using build-env, but only directly with jest. The reason for this is the "type": "module" in package.json, but this option is required by esbuild to make the bundle. Does anyone have a suggestion for this?

@tomasdavidorg this is ready for review, thanks in advance!


Smoother project generation

This new batch file will generate the project and if everything is alright the command line will close itself when it's finished. If there is any error, it will show the error message and then pause.

Batch files aren't cross-platform so I don't think that Cherno would accept it. Hazel was meant to be cross-platform.

This PR is just enhancing the batch file we already have.. I don't see why this is an issue, but I like to see all the projects it generated before manualy closing it. I think this is just a preference...

I like this. It gives people who run the script directly from explorer a chance to catch errors.
@LovelySanta Since I usually have to re-generate the project files now and then, I always keep a terminal open so I can run my project generation and see the output after.

Was it decided this would be merged? It is such a small change but I think it's a great thing, you don't need to worry about the output unless there is an error.
LGTM


Initial jsonschema checkin

nails down v1 for the mac feed, pip(3) install check-jsonschema (in the venv of your choice, hopefully dropping it into your path), then check-jsonschema  --schemafile sofa/v1/v1_mac.schema sofa/v1/macos_data_feed.json for great rejoicing
if it all worked, you'd see
ok -- validation done

/me talking at self NEEDS DOCS ofc, will probably christen the wiki with details like this


Why is Lbry downloading each video to my iMac ??

I'm using the app on my iMac and each video gets downloaded to my local iMac !!!
I certainly do not want this.  How do I stop it?
Many thanks

Just seen the help menu in 'Settings' Hmmm.... If a download message had not appeared for each video, I would have been oblivious that Libry was using my iMac and (limited) bandwidth for it's own purposes.  I think you need to be a bit more clear on doing this.  It feels like resource usage by Stealth.

This is the default mode and can be changed in settings.


Note about s3 "random key names"

"🔸Note that sadly, the advice about random key names goes against having a consistent layout with common prefixes to manage data lifecycles in an automated way."
I think these are two different problems: s3 performance and object lifecycle management.
I think in general the guide should to be less biased about particular implementations and concentrate solely on Amazon's services pros and cons.

The intended point is that you can't have randomness on the start of a key, as well as folder-based lifecycles. You have to pick.
Perhaps you can clarify the concern? This isn't a bias as much as an observation that you can't do both, given S3's design.

Maybe reference to amazon's doc about performance optimization would be useful in this case?
http://docs.aws.amazon.com/AmazonS3/latest/dev/request-rate-perf-considerations.html
There is still a way to have keys prefixed with some sort of naming scheme and have hashes added after, it is mentioned in that guide. So lifecycle rules can be configured based on prefixes (example from aws s3 cli guide):
{
  "Rules": [
    {
      "ID": "Move to Glacier after sixty days (objects in logs/2015/)",
      "Prefix": "logs/2015/",
      "Status": "Enabled",
      "Transition": {
        "Days": 60,
        "StorageClass": "GLACIER"
      }
    },
    {
      "Expiration": {
        "Date": "2016-01-01T00:00:00.000Z"
      },
      "ID": "Delete 2014 logs in 2016.",
      "Prefix": "logs/2014/",
      "Status": "Enabled"
    }
  ]
}

It's probably the best when objects keys are completely random with the metadata information stored in low latency storage systems like dynamodb. Then lifecycle management can be applied easily and consistently to all objects in a bucket.

Based on the explanation offered from AWS on partitioning, in the situation where you have high qps and large numbers of keys, adding the fixed prefixes could have a performance penalty when compared with keys starting with the full randomness (as is the preferred examples in that link). But partitioning seems to be dynamic, too, so it may not be the case. Do you have any concrete perf observations when using prefixes + hashes?
@max-zanko suggests using multiple buckets in this case to support the lifecycle management + high perf. That approach seems safe.
We could elaborate this bullet a little. PR welcome.

👍 good stuff


New UI changes Added

Hey @vim12345  you can't just make any changes and make a pull request without any description. For that you need to raise a issue first.


Redaction errors fixed

More accurate translation for this article and also translation for variables and classes' names

[X] I have read freeCodeCamp's contribution guidelines.
[X] My pull request has a descriptive title (not a vague title like Update index.md)
[X] My pull request targets the master branch of freeCodeCamp.
[X] None of my changes are plagiarized from another source without proper attribution.
[X] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Closed and Reopened this PR to attempt to resolve a specific Travis build failure.

Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 🎉


Release Request (to include Pull Request #174)

Could a new release be made to include Pull Request #174 (already merged). I am working on a project that would greatly benefit from adding file collector to WMI_Exporter.
Thanks,
Mike

@mbibinski-ttd The release has been made, and artifacts should be published in a few minutes!


Support Admin State on Azure Global Load Balancer

Related command

az network cross-region-lb address-pool create
az network cross-region-lb address-pool address add
az network cross-region-lb address-pool update
az network cross-region-lb address-pool address update
Is your feature request related to a problem? Please describe.

Right now customers can set admin state on their regional load balancer through CLI but there is no support for global LB. We need to support admin state for global LB.
Describe the solution you'd like

Support setting admin state on global LB, similar to regional load balancer.
Describe alternatives you've considered

Additional context

Hello @yonzhan, are we able to take this as part of the current milestone?


Typescript

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.53%. Comparing base (d669cc1) to head (11d348c).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #321   +/-   ##
=======================================
  Coverage   99.53%   99.53%
=======================================
  Files          23       23
  Lines         214      214
  Branches       65       65
=======================================
  Hits          213      213
  Misses          1        1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


When starting a job, why is the ec2 configuration always c6i.large？The EC2 model entered on the front-end page cannot be recognized

Dear,
When I configured the PBS+StarCCM application,
I found that no matter how I chose the EC2 model on the front-end,
the machine I started was always c6i.large, and I wanted to know if it was a code bug.thanks!,
Example：

I adjusted the EC2 model to c6i.2xlarge, the required CPU is 16, and 2c6i.2xlarge should be started in the background.
But the actual startup was 8c6i.large

I adjusted the EC2 model to c6i.2xlarge, the required CPU is 16,
and 2c6i.2xlarge should be started in the background.
But the actual startup was 8c6i.large

Hi, ZDYC
Did you set up "Application Management -> 'What type of instance do you want to use? (instance_type)'" as an administrator? This picture is the EC2 selection screen for the web application used in fluent.
In my case, after completing the SW tile in "Application management", the number of nodes for the selected instance was automatically calculated when the user selected the instance type and entered the number of cores.

the code version v2.7.5

Hello @ZDYC -
Can you export your application profiles and send the .json to my Github Username @amazon.com
As Admin User, login to Web Interface > Application Management > Import/Export tab

My problem has been solved, thanks


[vcpkg] update CMake to 3.27

Upstream issue:
~~https://gitlab.kitware.com/cmake/cmake/-/issues/24991~~ fixed in rc2
https://gitlab.kitware.com/cmake/cmake/-/issues/25002
Note to self:
~~https://github.com/jmcnamara/libxlsxwriter/pull/402~~ fixed by another PR
dcmtk regression is baseline (also another PR)
Remove https://github.com/eclipse/paho.mqtt.c/blob/7db21329301b1f527c925dff789442db3ca3c1e7/src/CMakeLists.txt#L190-L191 from paho-mqtt; also unset any variable effecting openssl lookup in vcpkg-cmake-wrapper.cmake CMake will also fix it in a newer release

[ ] Merge #32338 before or together with this!
closes #32676
[ ] wait for osx results but i assume they'll be clean.

CMake Warning (dev) at D:/downloads/tools/cmake-3.27.0-rc1-windows/cmake-3.27.0-rc1-windows-i386/share/cmake-3.27/Modules/FindZLIB.cmake:88 (message):
  ZLIB does not provide any COMPONENTS.  Calling

    find_package(ZLIB COMPONENTS ...)

  will always fail.

This already appeared in another issue recently, with CMake 3.26. They seem to have broken something around version arguments.

@dg0yt If you want to chase it down feel free to do so https://gitlab.kitware.com/cmake/cmake/-/issues/24991#note_1375365 / https://gitlab.kitware.com/cmake/cmake/-/merge_requests/7767 . I am probably just going to patch the few failing ports since that seems less work.

@Neumann-A CMake 3.27 RC2 is out now for testing with, BTW the freebsd link no longer works for 3.20.4

Hmm a lot of android failures.

@Neumann-A CMake 3.27 officially released.
https://github.com/Kitware/CMake/releases
https://cmake.org/download/
https://cmake.org/cmake/help/v3.27/release/3.27.html#cmake-3-27-release-notes

This one is new:
CMake Error at D:/installed/x64-windows/share/directxtex/DirectXTex-targets.cmake:60 (set_target_properties):
  The link interface of target "Microsoft::DirectXTex" contains:

    Microsoft::DirectX-Headers

  but the target was not found.  Possible reasons include:

    * There is a typo in the target name.
    * A find_package call is missing for an IMPORTED target.
    * An ALIAS target is missing.

Call Stack (most recent call first):
  D:/installed/x64-windows/share/directxtex/directxtex-config.cmake:27 (include)
  C:/a/1/s/scripts/buildsystems/vcpkg.cmake:855 (_find_package)
  src/CMakeLists.txt:109 (find_package)

I am not able to reproduce that locally directxtex-config.cmake has everything needed just the order is a bit messed up which CMake shouldn't complain about. @walbourn If this issue persist you probably want to move the find_dependency stuff before the targets include(); Also why is that port switching between find_dependency and find_package in the config.cmake? (That doesn't make sense to me)

if(MINGW OR (NOT WIN32))
    find_dependency(directx-headers)
    find_dependency(directxmath)
else()
    find_package(directx-headers CONFIG QUIET)
    find_package(directxmath CONFIG QUIET)
endif()

This is because with non-MinGW scenarios on Windows, the use of directx-headers and/or direcxtmath package is purely optional. If it can't find them, it just doesn't care and sticks with the Windows SDK headers.
In the case of MinGW, however, the "Windows headers" there are, um, a mess. It has Direct3D 11 headers with some quirks, the DirectXMath headers exist but their contents is basically empty, and there's no DIrectX 12 headers. Therefore, in those cases it needs these packages but I wanted it to pass along the parameters from the including cmake.
In the case of Linux, both packages are also required.

@Neumann-A  Please review https://github.com/microsoft/DirectXTex/pull/380

@walbourn I used
if(VCPKG_TARGET_IS_WINDOWS AND NOT (VCPKG_TARGET_IS_XBOX OR VCPKG_TARGET_IS_MINGW) AND NOT "dx12" IN_LIST FEATURES)
  list(APPEND FEATURE_OPTIONS "-DCMAKE_DISABLE_FIND_PACKAGE_directx-headers=TRUE")
endif()

to fix the problem since this controls the dependency explicitly in vcpkg.
The problem is you can run into a situation where you build directxtex with directx-headers being found but when you consume them directx-headers is not found leading to the error observed.
For the config you probably want something like:
if(@directx-headers_FOUND@)
    find_dependency(directx-headers)
endif()
if(@directxmath_FOUND@)
    find_dependency(directxmath)
endif()

instead since finding the deps is not going to be optional on Windows if the headers were found when building directxtex you will depend on the headers target.
Moving the include() is not necessary.

if(@directx-headers_FOUND@)
    find_dependency(directx-headers)
endif()
if(@directxmath_FOUND@)
    find_dependency(directxmath)
endif()

Ends up if(0) no matter what.

Ends up if(0) no matter what.

Configure log? Did you see the message from https://github.com/microsoft/DirectXTex/blob/4d9d7a8ceba6d6a121cd1aae160a0b856ef03d89/CMakeLists.txt#L206 ?
If not it should be 0. That is the plan here.

Moving the include() is not necessary.

IMO it is necessary: If the included files create imported targets, it shouldn't happen if a find_dependency failed.
However I do agree that the find_package must be turned into guarded find_dependency. Either it is a dependency due to build time configuration, or the side effects are not desired.

Hmm should I update to 3.27.1 or just let this be merged ?

Somebody want to fix the baseline regression of starlink-ast? For me that look like a optional dependency which is only found for release builds.

Somebody want to fix the baseline regression of starlink-ast? For me that look like a optional dependency which is only found for release builds.

The latest ci baseline does not appear starlink-ast:arm64-windows error. https://dev.azure.com/vcpkg/public/_build/results?buildId=92175&view=results

Hm, Mismatched binaries is a recurring pattern with starlink-ast.
warning: Mismatching number of debug and release binaries.
Found 12 debug binaries:

    D:\packages\starlink-ast_arm64-windows\debug\lib\libast.lib
    D:\packages\starlink-ast_arm64-windows\debug\lib\libast_drama.lib
    D:\packages\starlink-ast_arm64-windows\debug\lib\libast_ems.lib
    D:\packages\starlink-ast_arm64-windows\debug\lib\libast_err.lib
    D:\packages\starlink-ast_arm64-windows\debug\lib\libast_grf3d.lib
    D:\packages\starlink-ast_arm64-windows\debug\lib\libast_grf_2.0.lib
    D:\packages\starlink-ast_arm64-windows\debug\lib\libast_grf_3.2.lib
    D:\packages\starlink-ast_arm64-windows\debug\lib\libast_grf_5.6.lib
    D:\packages\starlink-ast_arm64-windows\debug\lib\libast_pal.lib
    D:\packages\starlink-ast_arm64-windows\debug\lib\libast_pass2.lib
    D:\packages\starlink-ast_arm64-windows\debug\lib\libast_pgplot.lib
    D:\packages\starlink-ast_arm64-windows\debug\lib\libast_pgplot3d.lib

Found 8 release binaries:

    D:\packages\starlink-ast_arm64-windows\lib\libast.lib
    D:\packages\starlink-ast_arm64-windows\lib\libast_drama.lib
    D:\packages\starlink-ast_arm64-windows\lib\libast_ems.lib
    D:\packages\starlink-ast_arm64-windows\lib\libast_err.lib
    D:\packages\starlink-ast_arm64-windows\lib\libast_grf3d.lib
    D:\packages\starlink-ast_arm64-windows\lib\libast_grf_2.0.lib
    D:\packages\starlink-ast_arm64-windows\lib\libast_pal.lib
    D:\packages\starlink-ast_arm64-windows\lib\libast_pass2.lib

libast_grf_3.2.lib, libast_grf_5.6.lib, libast_pgplot.lib, libast_pgplot3d.lib were present for release in the successful vcpkg.ci run.

I fail to spot significant differences in starlink config logs. I still feel nervous about
 cl : Command line warning D9002 : ignoring unknown option '-Xcompiler'
 LINK : warning LNK4044: unrecognized option '/Xlinker'; ignored
-LINK : warning LNK4044: unrecognized option '/LD:/installed/arm64-windows/debug/lib'; ignored
+LINK : warning LNK4044: unrecognized option '/LD:/installed/arm64-windows/lib'; ignored

but I have no interest to spend more time before the vcm fixes from #31228.

@JonLiu1993: arm64-windows CI is now green that is why starlink-ast is a baseline regression

Why wouldn't the same issue apply to say directxmesh?

@walbourn If it has the same logic it has the same problem. I only checked the previous ci failing ports.


fix typo in moodle/README.md

What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:

LGTM

/ok-to-test
/lgtm


FluentState::singleton()->getLocale() returns the wrong Locale when not logged into CMS

We have two Locales in one of our web sites (en_US and zh_CN only). And the default locale is en_US.
But the function "FluentState::singleton()->getLocale()" returns the locale en_NZ when you are not logged into CMS. There are correct cookies set in both mode. Please check the screenshots below.
Logged into CMS

Not logged into CMS

@robbieaverill Thank you for the quick response. We are using SilverStripe 4.0 in this project.

@robbieaverill I tried installing the plugin on a fresh SS4 installation, and I could not reproduce this issue. Therefore, I think there must be some issue with the project's code base (not the plugins's).


[FLOC-4523] Add FlockerHub menu links

Fixes: https://clusterhq.atlassian.net/browse/FLOC-4523
I couldn't easily  get the new drop down menus working so opted to stick with the current menu style.

In the interest of expediency, let's just use
FlockerHub
Fli
Flocker
Blog
as the menu options for now.
We can figure out the drop down later.  Mainly want to get rid of VolumeHub and dvol for now.  Thanks!


[Fix #8718] Fix undefined methods of pseudo location

This change aims to fix the error due to undefined methods of RuboCop::Cop::Offence::PseudoSourceRange.
The RuboCop::Cop::Offence::PseudoSourceRange class emulates the Parser::Source::Range class in the parser gem.
(see https://github.com/whitequark/parser/blob/v<IP_ADDRESS>/lib/parser/source/range.rb)
The RuboCop::Cop::Offence#location attribute can have an instance of both these classes.
So, these classes need to have the same interface and semantics for the usage in RuboCop.

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[ ] Feature branch is up-to-date with master (if not - rebase it).
[ ] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Run bundle exec rake default. It executes all tests and RuboCop for itself, and generates the documentation.

I will squash the commits to one commit finally.

Thanks for implementing this fix

Thanks!

@bbatsov Thank you for merging it!


[Five Torches Deep]background images links

Changes / Comments
Updated the links to background images to Roll20 repository :)
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[X] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[X] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.


Velocity v<IP_ADDRESS>: (LATEST) bug-Lighting glitch

This glitch occurred while messing around in the mining game. It resulted after a series of light sources were created, moved, offscreen, then more light sources were placed. The glitch makes almost everything turn bright white and only shows certain colors and the outline of certain sprites.

I may have an insight into this issue.
I will need to do further testing and the elusiveness of this bug makes test case development difficult but I have my suspicions.
I'm currently of the opinion that while the deallocated light sources are awaiting finalization, too many of them pile up and overflow the 128 light sources allowed by the shader. Since this is the case, and no length validation is performed, LVOGL may be writing outside the bounds of the lighting data array. And that may be corrupting other associated data (since out of bounds access is not good) and resulting in maximum brightness calculation for all fragments.
Now that I think about it, this likely is the issue. And it probably has to do with the illuminate function in lit.frag. Validating the light source count and uniform upload from the CPU side would help matters (and prevent the counter uniform from exceeding the length of the uniform LightSource array, so we can avoid a bounds test every fragment). So I'll fix that and merge that into mainline in a couple of weeks.

In the latest LVOGL release, this may have been patched. The transient nature of the bug makes it hard to know, and the work on LVOGL is not complete, so this issue will remain open.

See #8

Tested on Velocity v<IP_ADDRESS>-dev with LVOGL in the Velocity Tilemap Editor. Bug patched.


add nz buildings

Not sure what the problem is here I can download 2.76G from the data url.

Log reports a "MemoryError".


Mac system alerts

@joerlane I stopped merging other folks PRs for the most part if they have write access and can merge it themselves.

@joerlane merging this is fine... but I don't merge other folks PRs anymore if they can merge things themselves.

Yeah, at this rate we could also do the correct thing of making those things into commands.
Beep as a command would beep on all OSes. And the display notification thing is also a thing in windows so if we had a display notification command thing we could do different things on different OSes.

Not sure whether beep special operation should run a beep command - we need to be careful about synchronicity. But in principle, that would be more inline with Meerk40t's command philosophy. Also, I think this PR is because there is no way yet found to do an actual sound beep on mac. (I can't quite image why this is, but since I have not used a Mac for more than 3 decades I have no actual knowledge.)
But I agree that we should implement generic a notify command which will generate some sort of balloon / toast notification, ideally using O/S functionality (so it will also work in non-GUI) but with use of wx.adv.NotificationMessage in the event that the O/S doesn't have this functionality built in. (If running in console mode and no O/S notification, then console message would need to suffice.)

There is a Pypi package for Win10Toasts but this is just Win10 and we probably don't want to have another Pypi dependency - but the code is not complicated and is MIT license, so we can almost certainly copy and tweak the code.
I'll look into creating a tools plugin to provide this functionality as a command.

The only way I can find to produce Windows 10 notifications from Python is Win10Toasts which in turn relies on pywin32. In theory it should be possible to work out how the former uses the latter and how the latter does ctype calls to windows dlls, and thus eliminate these dependencies, but frankly this is a lot of work. What would the issues be for adding these two dependencies to requirements.txt?

Ok - it looks like wx.adv.NotificationMessage actually interfaces with Windows and Mac notification subsystems, so the best choice is to use this if we have GUI availability, and notify to console if not. Working on this now.

I am leaving this targeting main, even after the legacy7 switch. The commands here should probably be both put into extras or dealt with properly. Turning beep and os_notify into proper commands.

I did try to create a PR which put beep and notify into a separate plugin (in 0.7) but the difficulty is in making an o/s agnostic Notify - you cannot create a Windows o/s notify box without having a shortcut with certain parameters, and that means using an installer to install the executable and create a shortcut - and that means it wouldn't work running from source etc.
So the only way to be platform agnostic IMO is to crate our own notification windows that pop up and then disappear after so many seconds and which stack vertically like o/s notifications do - and this was too much work for the time I had available at the time.

Probably won't get merged.


Bundles.Url with non-rooted path fails

Given a bundle with path "~/scripts/example". The following should work, but doesn't:
Bundles.Url("scripts/example");

Need to check for non-rooted path and prepend "~/" or "~"

Fixed in v1.0.1


Global Admin credentials

Please note that the term Global Admin credentials it is no about Azure AD Global Admin, but to the developer portal Global Admin account, that at the end, are to different things. Please address this as this could mislead our customers.

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

ID: 1b50e13d-f920-d392-6be0-c792bceac73c
Version Independent ID: 2c10bcf7-5a2a-26c9-af2d-bdae6845159f
Content: Adding or Removing Users
Content Source: windows-driver-docs-pr/dashboard/adding-or-removing-users.md
Product: windows-hardware
GitHub Login: @aahill
Microsoft Alias: windowsdriverdev

Thanks, @gonzalre. I've edited the opening of the topic to differentiate the two account credentials. Should be live later today.

Great! Thanks!
Gonzalo Reyna (GONZ)
Support Escalation Engineer
Microsoft Azure Rapid Response
Office: +1<PHONE_NUMBER>
gonzalre<EMAIL_ADDRESS>M-F 8:00-17:00 CST
<EMAIL_ADDRESS>
If you have any feedback about my work, please let either me or my manager Harry Storey know at harryst<EMAIL_ADDRESS>From: Eliot Graff<EMAIL_ADDRESS>Sent: Thursday, September 27, 2018 11:35 AM
To: MicrosoftDocs/windows-driver-docs<EMAIL_ADDRESS>Cc: Gonzalo Reyna (GONZ<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/windows-driver-docs] Global Admin credentials (#860)
Thanks, @gonzalrehttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fgonzalre&data=02|01|gonzalre%40microsoft.com|767c6f2c5feb46b05cd408d624973d23|72f988bf86f141af91ab2d7cd011db47|1|0|636736629317274016&sdata=qd5dsinXzbUQ85d1Vni7U7H1Di7%2FOzqmegFtjVcKPpA%3D&reserved=0. I've edited the opening of the topic to differentiate the two account credentials. Should be live later today.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fwindows-driver-docs%2Fissues%2F860%23issuecomment-425159715&data=02|01|gonzalre%40microsoft.com|767c6f2c5feb46b05cd408d624973d23|72f988bf86f141af91ab2d7cd011db47|1|0|636736629317274016&sdata=oDui2H22urPCPs3IN5J0QTPtcNVtFNz69%2BkWLU0DKkk%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FApoA712JEvkIPS3puHKxYKRbK5uyMnosks5ufP5RgaJpZM4W83pH&data=02|01|gonzalre%40microsoft.com|767c6f2c5feb46b05cd408d624973d23|72f988bf86f141af91ab2d7cd011db47|1|0|636736629317284024&sdata=ZP4rq3XSYhxrxqM2kEqKKANsIfQt7MQ%2BfsmSWtMbOuE%3D&reserved=0.


Cannot move waypoint on connector

Preflight Checklist

[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Describe the bug
One of the waypoints on a connector is not moveable. All other waypoints are moveable, as are the end-points, but one is not.
To Reproduce
Steps to reproduce the behavior:

Load the attached diagram.
Click on Purple connector line. Notice that four waypoints are highlighted on the connector.
Click and drag each waypoint to change the shape of the connector. Notice that the top most waypoint at location (240,32) is not moveable.
Try and fix this by right clicking on the connector and clearing waypoints. Notice that the top waypoint now at location (170,32) is still not moveable.

Expected behavior
Expect that waypoints on a connector would be moveable.
Screenshots
Load attached file to use a diagram that exhibits this problem.
draw.io version (In the Help->About menu of the draw.io editor):

draw.io version 13.6.5

Desktop (please complete the following information):

OS: Fedora 32
Browser Chrome 84 & Firefox 79

stuck_waypoint.txt

I figured out how to trigger this type of connector problem:

Select the purple connector and set it as the default style.
Pick any connection point on the shape. There are three white dots on the left, three dark gray on the right and three bumps on the bottom.
Hover over the connection point until you get the four directional connection arrows. (Sometimes I had to click to focus it too)
Select one of the directional arrows. Move over top of it, click to create a connector and begin dragging it to another connection point.
After connecting the end of the connector, go back and try and move the first waypoint on the connector and notice that it is not moveable.

I have another document that exhibits this behavior. Please see attached. (It's listed as a TXT file, but it's really a drawio saved file.)
Stuck_connector_waypoint.txt

Connectors are not added to "regular" shapes but if the connector in question is disconnected and reconnected, it can be moved


Update Team per Recent Changes

Signed-off-by: Ricardo Baeta<EMAIL_ADDRESS>

Thank you so much for the detailed review @dpordomingo :)
@marnovo @smola How should I include Hassen? Thanks is advance!

@ricardobaeta @dpordomingo  True, we need a new sales section w/ Hass + change the Dev Rel section to Mkt and add the new Alexander intern. Please is this is good as it is on the eng side (cc @smola), let's merge and deploy and open a new ticket for the remaining changes w/ me tagged and I'll provide the copy.

ok, so let's wait for @smola approval of team section, as it can be seen at staging:
https://landing-staging.srcd.run/company/#team
I'll deploy once he approves the PR

@dpordomingo we're all good to go! Please be so kind as to merge and deploy to production!

deployed into prod

Thanks @dpordomingo ✨


Bug: issue with serde skip_serializing_if

Current behavior
When using skip_serializing_if on a field the output can become unaligned.
Expected behavior
Skipping a value should maintain the correct incrementatal row number.
Sample code to reproduce
use rust_xlsxwriter::{ExcelSerialize, Format, Workbook, XlsxError};
use serde::Serialize;

fn main() -> Result<(), XlsxError> {
    let mut workbook = Workbook::new();

    // Add a worksheet to the workbook.
    let worksheet = workbook.add_worksheet();

    fn skip_check(value: &str) -> bool {
        value == "Peach"
    }

    //use rust_xlsxwriter::Format;
    let _foo = Format::new().set_bold();

    #[derive(ExcelSerialize, Serialize)]

    struct Produce {
        #[serde(skip_serializing_if = "skip_check")]
        fruit1: &'static str,

        fruit2: &'static str,
    }

    // Create some data instances.
    let items = [
        Produce {
            fruit1: "Peach",
            fruit2: "= Peach?",
        },
        Produce {
            fruit1: "Plum",
            fruit2: "= Plum?",
        },
        Produce {
            fruit1: "Pear",
            fruit2: "= Pear?",
        },
    ];

    worksheet.set_serialize_headers::<Produce>(0, 0)?;
    worksheet.serialize(&items)?;

    // Save the file.
    workbook.save("serialize.xlsx")?;

    Ok(())
}

Note, fixing this may break serialization of array values in a struct. That probably isn't a common use case though.

IIUC skip_serializing_if on one field here basically means to skip serialization of the row containg other fields given a condition.
So the caller can do it directly by iterator filter or continuing in the loop
let slice: &[Produce] = ...;
slice.iter().filter(|p| !skip_check(&p.fruit1)).try_for_each(|p| worksheet.serialize(p));

Oh, I just realized the problem is not that callers can't make it, instead it's if they use skip_serializing_if, the outcome is incorrect 😢

it's if they use skip_serializing_if, the outcome is incorrect 😢

Yes. That is it. ☺


Bug located in MPCOPY and other commands which has unused field

🤓 Before submitting the issue

[X] I have visited the Troubleshooting section.
[X] I have searched among the existing issues.
[X] I am using a Python virtual environment.
[X] I am using the latest version of PyMAPDL and its dependencies (fully updated virtual environment). You can update them using pip install --upgrade --upgrade-strategy eager ansys-mapdl-core in your activated virtual environment.

🔍 Description of the bug
Use of MPCOPY results in TypeError as it has an unused field.  Looks like it was fixed, or supposed to be, in prior pull: https://github.com/ansys/pymapdl/pull/1675
Received error using 0.68.1 and mapdl 24R1.
🕵 Steps To Reproduce
The log will automatically be formatted as Python code! No need to type backticks.

💻 Which Operating System are you using?
Windows
🐍 Which Python version are you using?
3.10
💾 Which MAPDL version are you using?
24R1
📝 PyMAPDL Report
Show the Report!

# PASTE HERE THE OUTPUT OF `python -c "from ansys.mapdl import core as pymapdl; print(pymapdl.Report())"` here

📝 Installed packages
Show the installed packages!

# PASTE HERE THE OUTPUT OF `python -m pip freeze` here

📝 Logger output file
Show the logger output file.

# PASTE HERE THE CONTENT OF THE LOGGER OUTPUT FILE.

The /post26 command ADD and QUOT still have problems due to multiple blank fields on the command.  Also if the data is complex then QUOT using the NAME field returns machine zeros for complex variables.

Interesting... it seems to work for me:
>>> mapdl.mat(1)
>>> mapdl.mp("SONC", 1, 1500)  # sonc in m/s
>>> mapdl.mp("DENS", 1, 1000)  # Density in kg/m3
>>> mapdl.mpcopy(1, 2)

COPY MATERIAL        1  DENS  TO MATERIAL        2

 COPY MATERIAL        1  SONC  TO MATERIAL        2

The /post26 command ADD and QUOT still have problems due to multiple blank fields on the command. Also if the data is complex then QUOT using the NAME field returns machine zeros for complex variables.

I will need an example for testing and debugging this.

There is an unused/beta field in MPCOPY.  So the command should really be mapdl.mpcopy('', 1, 2)

For reference: https://github.com/ansys/pymapdl/discussions/3026#discussion-6554930

This issue is locked until https://github.com/ansys/pymapdl/discussions/3026 gets closure.


Using local DNS

I know from https://github.com/mailcow/mailcow-dockerized that they use a local DNS because of the high amount of DNS requests (to not get blacklisted).
Do you know if this is a topic for kopano/mailserver?
If so, do we want to have local DNS too?
f.e. this is one https://github.com/mailcow/mailcow-dockerized/tree/master/data/Dockerfiles/unbound

No, I cannot think of a component which would do many external dns lookups (besides the postfix part). When using smime you could have some requests for the oscp part, but I think this is all client side.

OK. I did a little bit research and found out that it originally wasn't for the amount of DNS requests, it was for DNSSEC too:
https://github.com/mailcow/mailcow-dockerized/commit/b367ec0ace533ee3f32bd16c70a5e7f7ab52f571#diff-4e5e90c6228fd48698d074241c2ba760

Ok, closing since all mail flow related parts are handled by https://github.com/tomav/docker-mailserver

I think we do have to rethink about the mailserver.
Because otherwise some of the DNS-Blacklist lookups aren't working.
Look here
https://www.spamhaus.org/faq/section/DNSBL%2520Usage#261

If you are using a free "open DNS resolver" service such as the Google Public DNS (<IP_ADDRESS>) and others [...] to resolve your DNSBL requests, in most cases you will receive a "not listed" (NXDOMAIN) reply from Spamhaus' public DNSBL servers

https://github.com/tomav/docker-mailserver/issues/948

If you're using Google's public DNS servers (or several other public DNS servers) they don't allow querying of DNSBL. You can use the new Cloudflare DNS <IP_ADDRESS> or <IP_ADDRESS>
I started working on a companion container that would run Unbound but since Cloudflare released their DNS servers, I gave up.

The linked issue makes the following suggestion:

You can add the following to your docker-compose.yml to fix this:
services:
mail:
image: ...
...
dns: <IP_ADDRESS>

Seems easy enough to include this, although it feels like it should be something else than cloudflare if this project should make a DNS recommendation.

At least we should add a warning to the README that in case somebody wants to use DNSBL , we use a public DNS IP for it.
But you're right that adding the dns was the pretty easiest way to solve it ;-)


Add summaryOnly (true/false) optional arg to GET Contacts

https://developer.xero.com/documentation/api/contacts#summary-only
When set to true, this returns lightweight fields, excluding computation-heavy fields from the response, making the API calls quick and efficient.
https://github.com/XeroAPI/xero-node/issues/523

Closed in xero-node 🥳  with 4.11.3
via: https://github.com/XeroAPI/Xero-OpenAPI/pull/404


Installation issue

pi@raspberrypi:~/wshare-touchscreen $ gcc -std=c99 -Wall ./waveshare.c -pthread -lsuinput -ludev -o waveshare-touch-driver
./waveshare.c:7:21: fatal error: suinput.h: No such file or directory
#include <suinput.h>
^
compilation terminated.

Actually it is "compilation" phase. And you do not have a libsuinput dev library installed...

Since it no longer relevant, I'll close this issue


Add modern metroidvania map

Adds support for "modern metroidvania" map that displays detailed room images instead of simple grid. Such system is featured in games like Ori or GRIME, but this one is specifically inspired by ENDER MAGNOLIA, where rooms are still constrained to rectangles, so you can just slap nice overlay over a grid.
https://github.com/KoBeWi/Metroidvania-System/assets/2223172/4da10227-f817-47d6-868a-80932e8af47c
Important component is a module for grained discovering of room contents.
I'm opening a pull request, because the changes are a mess and it's easier to clean them up this way.

Nice work! Are you able to control the scale of how much of the map is shown ?

Yes, the the overlays can be any size and the discovering resolution is controlled with constants. Though the final display size (the rectangles on map) is determined by the theme, which takes some effort to change.


Oh I cannot put a row with /0 is that going to be an issue?

I should test input with /0 from Python

Users have to replace null terminated characters with an escaped version


Update to 1.19.4

After some changes in toasts BatteryStatusInfo doesn't work anymore in this version, so updating to 1.19.4 would be awesome

You can download a 1.19.4 updated version of the mod here, just extract the zip and add the non-sources jar to your mods folder. I haven't yet pushed this patch to CurseForge/Modrinth as I am working on improving compatibility with different computers (as I've had issues with it on Windows)

A 1.19.4 compatible version of the mod is available on Modrinth and CurseForge


Vue Devtools is not working?

I run dev mode(npm run dev), and then open my browser. But I see nothing in the vue devtools. Is it something wrong?

Works for me, using latest rc8


[Bug] deletepa is not deleting public address

Any command in the valid format always returns something like:
Deleted Person's Public Address: PublicAddressesComposition{{BTC=[DeFaUlT: 2345, idk what: 3456, default but long: 4567, default: 6789, default: 1234]}}
For labels that do not exist, also returns this.
For labels that do exist, it does not get deleted.

Addressed in PR #171


Sia Daemon v1.3.2 Git Revision 72938f5 Loading... (0/6) Loading siad... (1/6) Loading gateway... (2/6) Loading consensus... (3/6) Loading transaction pool... (4/6) Loading wallet... (5/6) Loading host... (6/6) Loading renter... Finished loading in 1.883216582 seconds

BUG REPORT
Stack Trace or error message
Expected Behavior
How to reproduce it (as minimally and precisely as possible)
Environment

Sia version:
OS:

Sia Daemon v1.3.2
Git Revision 72938f5
Loading...
(0/6) Loading siad...
(1/6) Loading gateway...
(2/6) Loading consensus...
(3/6) Loading transaction pool...
(4/6) Loading wallet...
(5/6) Loading host...
(6/6) Loading renter...
Finished loading in 1.883216582 seconds

Sia Daemon v1.3.2
Git Revision 72938f5
Loading...
(0/6) Loading siad...
(1/6) Loading gateway...
(2/6) Loading consensus...
(3/6) Loading transaction pool...
(4/6) Loading wallet...
(5/6) Loading host...
(6/6) Loading renter...
Finished loading in 1.82516918 seconds

How to fix the problem
v1.3.2
Git Revision 72938f5
Loading...
(0/6) Loading siad...
(1/6) Loading gateway...
(2/6) Loading consensus...
(3/6) Loading transaction pool...
(4/6) Loading wallet...
(5/6) Loading host...
panic: Consensus change series appears to be inconsistent - we are applying the wrong block.

@Tyz49
There is no error message here, all the data you've included is normal output.
Try this to find your error message: https://gist.github.com/tbenz9/8066d74853ab28c23bd7bc1d3c7cdeda
@soon1ee
often a restart of Sia will fix that error.  If not, use the link above and follow the instructions for "Delete your transactionpool.db file"


v0.01 beta release-blockers

[ ] File handling: Requirements: No download before conversion finished (even though conversion is v. fast); avoid clashes if two people are downloading files at the same time (e.g. each session needs to be tracked - make a hash of the post.dump html and use directly (or as key to) converted file names.
[ ] Landing Page
[ ] Report Abuses of CoC section
[x] Codify one more Code of conduct

https://github.com/alexmorley/CodeOfConduct.Builder/commit/c95c542ecdce6489f5491953084c3dbc2ae354de


Updated for ontology.iupac.org

Applies the refactored metrology to the fse discipline vocabulary.

Same doc generation problems as https://github.com/opencaesar/jpl-vocabularies/pull/1#issuecomment-708765524


refactor: rename hosts.Parse => hosts.Load + add NewHosts

Codecov Report

Merging #7 (6381956) into main (deeb58c) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main        #7   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           13        14    +1
  Lines          306       310    +4
=========================================
+ Hits           306       310    +4

Files
Coverage Δ

hosts_load.go
100.00% <100.00%> (ø)

hosts_new.go
100.00% <100.00%> (ø)

:mega: Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!


3752 Segmentation fault      node "$basedir/node_modules/gulp/bin/gulp.js" "$@"

Hi,
my gulp setup is working fine for months but all of sudden i am getting this error.
my directory structure is below and it working in my other project whereas gulpfile.js and project settting is same.
+projects
|-- project_1
|-- project_2
+node_module
here is the output of "npm ls chokidar"
$ npm ls chokidar
<EMAIL_ADDRESS>E:\Google Drive
└── (empty)
npm ERR! code 1

Hi,
I checked it further error is with sass there are some variable in my sass partial file like i mention below when i comment them this error do not appear instead variable not found error which is ok.
Now I switched to compass which is working fine.
$orange: #ec7d00;
.orange {
color: $orange;
}
$blue : #3e81de;
.blue {
color: $blue;
}

Looks solved. Closing.

How to solve this problem?

@hualinIT compile sass file with compass to check if the error is in your sass file this is how i solved. Some time gulp-sass dont give correct error.


Fix returns keyword suggestion issue

Purpose

With this fix the following completion issue
Fixes #18931

Check List

[x] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[x] Checked Tooling Support (#)
[x] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[x] Integration Tests
[ ] Ballerina By Example Tests

[x] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

Merging #19288 into ballerina-1.0.x will not change coverage.
The diff coverage is n/a.

@@               Coverage Diff                @@
##           ballerina-1.0.x   #19288   +/-   ##
================================================
  Coverage            15.73%   15.73%
================================================
  Files                   47       47
  Lines                 1265     1265
  Branches               197      197
================================================
  Hits                   199      199
  Misses                1053     1053
  Partials                13       13

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 463f06d...385543c. Read the comment docs.


Add gitignore

Let's hold this submit until ".gitignore,modules/dreamview/fronted: add gitignore,delete redundancy lock file #19" to avoid conflicts.


Upgrade to new google cloud ruby client

Switched to the recommended Ruby gem: https://github.com/GoogleCloudPlatform/google-cloud-ruby

Looks like one of the unit tests depends on the gcloud plugin. I'll fix that soon.


feat: reactable#add_reactions

Codecov Report

Merging #19 (599bf37) into main (d7c252c) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #19   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            5         6    +1
  Lines           59        86   +27
=========================================
+ Hits            59        86   +27

Impacted Files
Coverage Δ

.../acts_as_reactable/reactable/add_reactions_spec.rb
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d7c252c...599bf37. Read the comment docs.


CUDA out of memory with Default Script

Thanks for the great work. I run the default script with an 80GB A100,  but it gives the OOM memory issue.
  sh run_sparse.sh

torch.cuda.OutOfMemoryError: CUDA out of memory. Tried to allocate 39.55 GiB. GPU 0 has a total capacty of 79.10 GiB of which 5.04 GiB is free. Process 232127 has 74.04 GiB memory in use. Of the allocated memory 33.33 GiB is allocated by PyTorch, and 39.47 GiB is reserved by PyTorch but unallocated. If reserved but unallocated memory is large try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF
Do you know any problem behind this?

Hello, you can refer to the discussion here https://github.com/Drexubery/ViewCrafter/issues/23#issuecomment-2396131121

Thanks! Using torch 2.4.0 works for me.


Refactoring of Steps 1,2 and modal component

Containerized the control buttons for the stepper outside of each step
Made components for step 1 and 2
Refactored tests to be more contained
Implemented event handlers for each of the step components
This will make the implementation of the next steps on the stepper a lot easier

Shoutout to Sean Egan for helping me think about how to refactor this code and Jaron Hamlik for explaining event emitters to me 🚀

LGTM!


enhance getAudience method to use also azp

enhance getAudience method to use azp in case audience claim is empty

hi @strehle  what's your opinion - is it okay when Token.getAudiences() returns "azp" in case "aud" is empty? This is also used in context of audience validation.


Remove DOWNLEVEL_PRIOR_TO_WIN8 flag

remove DOWNLEVEL_PRIOR_TO_WIN8/WIL_SUPPRESS_PRIVATE_API_USE/STANDaLONEDLL_EXPORTS flags
hook up telemetry for mrmmin/mrmex code
some improvements to telemetry.

You're removing code that's included if DOWNLEVEL_PRIOR_TO_WIN8 is NOT defined. Shouldn't we keep that code? And instead remove code that's included if DOWNLEVEL_PRIOR_TO_WIN8 IS defined?

That's the correct behavior. The OSS code uses DOWNLEVEL_PRIOR_TO_WIN8 defined.

In reply to: 910843591

/azp run


Local metadata which does not verify should not block updates

If a new root.json is downloaded during an update (e.g. because the local one is expired), and the new root.json has completely different keys, then other local metadata can potentially no longer be verified in Client.getLocalMeta, leaving the client unable to update.
Instead of returning an error when local data does not verify, it should be invalidated and re-downloaded, possibly logging the failed verification somewhere.

Closed by #58.


add spatial query notebook

this PR adds a notebook on how to use the spatial query interface

@giovp , thanks for the tips. The build is now passing.


output: too many empty additionalProperties

Not a huge deal. Guess a side product by integrating the latest autorest runtime
Suggest not emit out empty ones. A single vm show gives me 8 of them, that is too many :)
//CC: @lmazuel @derekbekoe

This one is becoming a non-trivial distraction when checkout the response from complex objects like VMSS, VM, LB, etc. It is everywhere.
Unless there is an objection, I will go ahead to make the change to no longer emit out empty additionalProperties. Though I don't think would ever be a scenario, but for generic update, users can always assume there is an additionalProperties they can update.  But I don't think we should let edge scenarios get into the way of mainline usages.
//CC: @derekbekoe @lmazuel @tjprescott @troydai

For generic_update, currently it warns if you try and set something that doesn't exist. Perhaps those things should be automatically put in "additionalProperties"?

That's what additionalProperties is meant for. On read, everything not declared in the Swagger is put inside. On write (PUT, POST), either the Swagger enables it, or you can set an option to do it per Model:
VirtualMachine.enable_additional_properties_sending = True

finally done #6277


XSD not found ->  https://jolokia.org/jolokia-spring/schema/config/jolokia-config.xsd

Hi, good day. Can you please take a look at the following.
I'm getting a 404 on subject xsd. I have maven dependencies jolokia-spring + jolokia-core + jolokia-client-java all of them in 1.7.1 version. I could not found the xsd on those dependencies
While trying to define Jolokia spring config as the doc shows:

Hi,
a workaround should consist in replacing  http://jolokia.org/jolokia-spring/schema/config/jolokia-config.xsd by https://raw.githubusercontent.com/jolokia-org/jolokia-spring/master/src/main/resources/org/jolokia/support/spring/jolokia-config.xsd

This should be fixed in the doc generation, not sure if this ever has worked

Show soon (after 2.0 release) be available under:

https://jolokia.org/jolokia-spring/schema/config/jolokia-config.xsd
https://jolokia.org/jolokia-config.xsd


Add Headers for Composite Video Mode Update Callback

Add the callback function to send the composite video mode update event to the Upper layer.

Where's the sign off by the architecture team or information on the discussion about it?


Python 3.7?

Is it possible to have a Python 3.7 package generated for pybtex?
https://anaconda.org/conda-forge/pybtex/files

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I rerendered the recipe in conda-forge/pybtex-feedstock#3.

3.7 should be available once https://github.com/conda-forge/pybtex-feedstock/pull/5 is built.


[BACKLOG-21806] Updated tooltips for JMS Steps

Build Completed
:x: This pull request has errors. They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -fn -f 'pom.xml' -pl 'plugins/streaming/impls/jms' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'plugins/streaming/impls/jms' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  plugins/streaming/impls/jms/src/main/java/org/pentaho/di/trans/step/jms/context/JmsProvider.java
  plugins/streaming/impls/jms/src/main/java/org/pentaho/di/trans/step/jms/ui/ConnectionForm.java
  plugins/streaming/impls/jms/src/main/java/org/pentaho/di/trans/step/jms/ui/DestinationForm.java
  plugins/streaming/impls/jms/src/main/resources/org/pentaho/di/trans/step/jms/messages/messages_en_US.properties

Checkstyle violations
The code submitted is not complying with one or more of our guidelines. To perform these checks locally, please use this tool.
/home/buildguy/project/plugins/streaming/impls/jms/src/main/java/org/pentaho/di/trans/step/jms/ui/DestinationForm.java

:heavy_exclamation_mark: Unused import - org.pentaho.di.trans.step.jms.context.JmsProvider.ConnectionType.WEBSPHERE. at Line: 39 Column: 15

Build Completed
:white_check_mark: This pull request has passed all validations.
Build Commands
mvn -B -fn -f 'pom.xml' -pl 'plugins/streaming/impls/jms' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'plugins/streaming/impls/jms' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  plugins/streaming/impls/jms/src/main/java/org/pentaho/di/trans/step/jms/context/JmsProvider.java
  plugins/streaming/impls/jms/src/main/java/org/pentaho/di/trans/step/jms/ui/ConnectionForm.java
  plugins/streaming/impls/jms/src/main/java/org/pentaho/di/trans/step/jms/ui/DestinationForm.java
  plugins/streaming/impls/jms/src/main/resources/org/pentaho/di/trans/step/jms/messages/messages_en_US.properties

@pentaho/ackbar please review


Limit Switches

Make sure it reads that it touches
Makes motor/servo stop

Limit switches for extender are now on extension hubs ports 4 & 5. May be subject to change, please ask.


Should have $fetchState.success

Describe the solution you'd like
$fetchState.success = !$fetchState.pending && !$fetchState.error

I think that's a reasonable solution. Closing as $fetchState is no longer present in Nuxt 3.


:sparkles: post-apply checks

Run post-apply checks and look fordb_port terraform output
Ticket: APPSRE-11228

/retest


Does "fapiao" means invoice ?

[Enter feedback here]
Seems like typo.

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

ID: b7b9f169-db90-943e-afb4-aa90be48acbd
Version Independent ID: 2336b9a6-8d5f-dc27-2d4d-b42a2df6cdbf
Content: View your bill or get a Fapiao in Office 365 operated by 21Vianet - Microsoft 365 admin
Content Source: microsoft-365/admin/services-in-china/view-your-bill-or-get-a-fapiao.md
Service: o365-administration
GitHub Login: @cmcatee-MSFT
Microsoft Alias: cmcatee

@Code2Life Thank you for your feedback. A fapiao is a legal receipt that serves as proof of purchase for goods and services. So this is more like an invoice.
Hope this helps!
@yogkumgit There are no changes required in the document. Please close the issue.
Thanks
Sri

@msbemba Thank you very much for the contribution and sharing this explanation. @Code2Life Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.