chore: import repository https://github.com/q1323829945/test18.git

this commit will trigger a pipeline to generate the CI/CD configuration which will create a second commit on this Pull Request before it auto merges

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign q1323829945
You can assign the PR to them by writing /assign @q1323829945 in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@q1323829945: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

verify
16ddbee21d6b39340e4cbf0482d06e99082783af
link
/test verify

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


for ConsulConfigurationSource if some errors happens in the reload process once, the initialized flag will keep false all the way, which will lead to the getConfiguration function throw IllegalStateException

`public void reload() {
Map<String, String> newConsulValues = new HashMap<>();
List valueList;
try {
  LOG.debug("Reloading configuration from Consuls' K-V store");
  valueList = kvClient.getValues("/");
} catch (Exception e) {
  initialized = false;
  throw new SourceCommunicationException("Can't get values from k-v store", e);
}

for (Value value : valueList) {
  String val = "";

  if (value.getValueAsString().isPresent()) {
    val = value.getValueAsString().get();
  }

  LOG.trace("Consul provided configuration key: " + value.getKey() + " with value: " + val);

  newConsulValues.put(value.getKey(), val);
}

consulValues = newConsulValues;

}`
For this reload function, in my opinion, if no Exception has been caught, then we need to set the initialized as true

fixed in https://github.com/cfg4j/cfg4j/pull/178
set true is incorrect. Retry and fail if not initialized much better. Please review


Update PPP

Codecov Report

Merging #244 (a560c8a) into master (9e08a5e) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #244   +/-   ##
=========================================
  Coverage     77.42%   77.42%
  Complexity      706      706
=========================================
  Files            97       97
  Lines          2224     2224
  Branches        275      275
=========================================
  Hits           1722     1722
  Misses          402      402
  Partials        100      100

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 9e08a5e...a560c8a. Read the comment docs.


DTS-HD not working on Nvidia Shield

As in title, I’m only getting the DTS core with Justplayer. With all other players, even ExoPlayer and Kodi, DTS-HD bitstreams just fine. TrueHD/Atmos does work.

I'm afraid it's same as #99

Any chance this could be resolved? I guess I am stuck with that build of Exoplayer in the meantime, but it doesn't switch refresh rates. Vimu (also Exoplayer based) is too buggy and doesn't really work right.

Share/link the version of ExoPlayer that works with DTS-HD, so I can inspect how they do that. The problem is basically only DTS-HD detection, which can be done by already included ffprobe. So there's a chance.

https://forum.makemkv.com/forum/viewtopic.php?t=21937&start=915
https://drive.google.com/open?id=12nFLa67n3Jee1mGkRBxL2XSIyOIRSk4d
I'm pretty sure this is the build I have installed on my Shield (I couldn't find the original download). I got it from that really long thread. You've probably already seen it, if not they go pretty deeply into Dolby Vision playback on the Shield Pro and getting it to work properly with modded versions of Exoplayer.
I would use Vimu (it's also Exoplayer based) which has more features than JustPlayer (and works with DTS-HD) except it's buggy and very unstable.

The Google Drive link isn't public and the wetransfer link from the forum doesn't contain any file.

The Google Drive link isn't public and the wetransfer link from the forum doesn't contain any file.

https://drive.google.com/file/d/17O1rGpAUAyc-q3PqNoQtQDXHgc3bjgUH/view?usp=sharing

This ExoPlayer apk used DtsProbe in MatroskaExtractor for DTS-HD detection (isHD()). Similar reimplementation would be probably even accepted by the upstream ExoPlayer.

Exoplayer_Chapters_Support_03_08_2020_no_debug.zip
Try that.

when can we expect dts hd support in justplayer?

I tried to reimplement DTS HD detection in a similar way as in the linked version of ExoPlayer (2.11.4) but then I found out than neither Just Player or ExoPlayer 2.11.4 actually play DTS HD for me. DTS HD tracks just do not play anything on my AV receiver, while DTS HD actually works at least in VLC.
So, could you share some sample that does play as DTS HD in the linked ExoPlayer version for you?

https://we.tl/whSFPBN6hT
Hmm. Everything I've tried with a DTS-HD track plays back properly on my Onkyo TX-NR676 receiver. This one works fine.

https://we.tl/whSFPBN6hT
Sorry about that- didn't mean to close it.
Everything I've thrown at Exoplayer has worked fine in DTS-HD MA on my Onkyo TX-NR676 from my Shield. Try that demo file- I just tested it and it worked fine.

dts hd now working in 0.46?

0.46 still DTS only.

Thank you for the sample. Unfortunately I got the same result as with other files I tried - no audio output at all. I have Onkyo TX-NR509 and currently testing the app on Geniatech ATV598Max. This Android TV device might be somewhat obscure (possibly requiring hacks to make things work?) so I will try it with Xiaomi Mi Box instead (will have access to it three weeks).
But please try this version: Just.Player.v0.47-dtshd.apk 🤞 Ideally it should detect and output DTS HD, just like the previously ExoPlayer apk (though the list of audio tracks still shows DTS only for now).

Works! Congratulations! and thank you. :)

DTS X, DTS HD MA all good, DTS HD still plays like DTS.

DTS HD still plays like DTS.

Do you have a sample you could share?

DTS HD still plays like DTS.

Do you have a sample you could share?

https://drive.google.com/file/d/17Ocw2Uddgze0cTPsFk8Z58t1Asxta5ss/view?usp=sharing

DTS X, DTS HD MA all good, DTS HD still plays like DTS.

Despite the name, DTS HD is a lossy format and rarely used. I don’t think I’ve ever played a file that uses it.

@vladlenas This should hopefully work better: Just.Player.v0.47-dtshd2.apk (Previous version didn't necessarily load enough required data for detection.)

@vladlenas This should hopefully work better: Just.Player.v0.47-dtshd2.apk (Previous version didn't necessarily load enough required data for detection.)

All is well now.

Does DTS-HD bitstream work for you on non Nvidia Shield devices as well? What is the lowest Android TV OS version it works? Because of issues with the one device I have, I'm considering limiting this DTS-HD detection to Android 8+.

Thank you for the DTS-HD fix, i tested 4 Files and all detected DTS-HD. Good Work.

@vladlenas This should hopefully work better: Just.Player.v0.47-dtshd2.apk (Previous version didn't necessarily load enough required data for detection.)

All is well now.

sorry to bring this up, but can you share this build?  Just.Player.v0.47-dtshd2.apk ?


Apriori_Segmentation_Fault

Segmentation fault with this script:
#pythran export frequent_itemsets(str list list)
import itertools
def frequent_itemsets(sentences):
# Counts sets with Apriori algorithm.
SUPP_THRESHOLD = 100
supps = []
supp = {}
for sentence in sentences:
    for key in sentence:
        if key in supp:
            supp[key] += 1
        else:
            supp[key] = 1
print "|C1| = " + str(len(supp))
supps.append({k:v for k,v in supp.iteritems() if v >= SUPP_THRESHOLD})
print "|L1| = " + str(len(supps[0]))

supp = {}
for sentence in sentences:
    for combination in itertools.combinations(sentence, 2):
        if combination[0] in supps[0] and combination[1] in supps[0]:
            key = ','.join(combination)
            if key in supp:
                supp[key] += 1
            else:
                supp[key] = 1
print "|C2| = " + str(len(supp))
supps.append({k:v for k,v in supp.iteritems() if v >= SUPP_THRESHOLD})
print "|L2| = " + str(len(supps[1]))

supp = {}
for sentence in sentences:
    for combination in itertools.combinations(sentence, 3):
        if (combination[0]+','+combination[1] in supps[1] and
                combination[0]+','+combination[2] in supps[1] and
                combination[1]+','+combination[2] in supps[1]):
            key = ','.join(combination)
            if key in supp:
                supp[key] += 1
            else:
                supp[key] = 1
print "|C3| = " + str(len(supp))
supps.append({k:v for k,v in supp.iteritems() if v >= SUPP_THRESHOLD})
print "|L3| = " + str(len(supps[2]))

return supps

Bug spotted in itertools.combinations when the second parameter is larger or equal to the first parameter size. I'm on it.

Fixed in 9b32420, let's wait it's validated and merged!


lib/ffmpeg/encoderbase: add Intra refresh cases

Add Intra refresh cases to lib/ffmpeg/encoderbase.
Signed-off-by: Li Fan<EMAIL_ADDRESS>

CI failed result for GST not this part change cause

CI failed result for GST not this part change cause

Mengke is helping to add this part into gst. And I will add the parameter to gst cmd after Mengke completes it.

where are the tests that use this feature?

"intra refresh" works when gop > int_ref_cycle_dist > int_ref_cycle_size. The value of cqp doesn't match the rule.


flutter channel shows incorrect Android embedding v2 warning

When I run flutter channel stable (for example) in my root user directory, I see the following warning:

Your Flutter application is created using an older version of the Android
embedding. It is being deprecated in favor of Android embedding v2.

This, however, makes no sense at all because my root user directory is not a Flutter project. It does, however, contain an android directory (which has nothing to do with the android directories you will find in Flutter projects).
Steps to Reproduce

cd to a directory that contains a folder named android
flutter channel stable (or a different channel if you are already on stable)

Click to expand full logs
D:\>flutter channel stable
╔════════════════════════════════════════════════════════════════════════════╗
║ A new version of Flutter is available!                                     ║
║                                                                            ║
║ To update to the latest version, run "flutter upgrade".                    ║
╚════════════════════════════════════════════════════════════════════════════╝

━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
Warning
──────────────────────────────────────────────────────────────────────────────
Your Flutter application is created using an older version of the Android
embedding. It is being deprecated in favor of Android embedding v2. Follow the
steps at

https://flutter.dev/go/android-project-migration

to migrate your project. You may also pass the --ignore-deprecation flag to
ignore this check and continue with the deprecated v1 embedding. However,
the v1 Android embedding will be removed in future versions of Flutter.
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
The detected reason was:

  No `D:\android\AndroidManifest.xml` file
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

Switching to flutter channel 'stable'...

Proposal
Before checking if there is an AndroidManifest.xml in an android directory and if that directory exists, check if the working directory is a Flutter project.
Steps

Check if the current working directory is a Flutter project → check for a pubspec.yaml file.
Check if there is an android directory.
Check for the AndroidManifest.xml file.

Currently, step 2. is missing.

@creativecreatorormaybenot
I tried this case on a sample app and got a similar error but while running flutter run, but didn't get the same while switching to different channel. Is this specific to flutter channel command only or your request for other commands like flutter run as well ?

@darshankawar This is not specific to any flutter command, but it will only happen once and not appear thereafter. I think to reproduce it every time, you might need a clean Flutter installation.

Thanks for the update. After a clean installation, I do see the embedding warning as below while switching the channels:

Your Flutter application is created using an older version of the Android
embedding. It is being deprecated in favor of Android embedding v2.

Treating this as a proposal per above request.


Remove history dep

Ideally react-history will be our only history-related dep in the router. If it doesn't export something we need, it should.
I think right now the only thing it's missing are the location/path creation utilities.

Can it export those from history and just pass things through?

Yep. Working on it now.

Please excuse me while I use GitHub issues as a TODO list :)


Allow scrolling program output

This allows scrolling program output on the right hand side.

let's do overflow: auto; so the scrollbar only shows up when it's needed. Otherwise 👍

Awesome, thanks!


how do I set reject argument for EEG bad epochs rejection

I am a fresh bird for MNE. When I used EEGlab, there is a threshold for artifact rejection, such as (-80 uV, 80 uV). However, in MNE, I did not know how to set the value of the argument to reject as I did not know the units of it. I found that there is a different value in the different example code. eg. eeg=60e-6, eeg=180e-6, eeg=200e-6.

hey @psychoxiatao you might want to try out autoreject. See an example here how the threshold is learned automatically from the data.

the units of reject is always the unit of the channel type so for EEG it's
Volt. To match what you did in EEGLAB it would be reject=dict(eeg=160e-6)
as 160e-6 is for peak to peak amplitudes

the units of reject is always the unit of the channel type so for EEG it's Volt. To match what you did in EEGLAB it would be reject=dict(eeg=160e-6) as 160e-6 is for peak to peak amplitudes

Sorry, I still do not understand. 1 Volt = 1000000 uV,  160e-6=0.00016V=16uV right?  Why did it match(-80 uV, 80 uV) in EEGlab.

No it’s 160uV not 16 so twice 80uV


bug(tabs): Not selected on first click with push change detection

When change detection set to OnPush, clicking on a tab does not activate the tab, until a second click occurs.
http://plnkr.co/edit/PTbPWjuiSU9YMnNTez5D

@trevordaniels Did you find a solution for this?

Hacked a workaround, there's probably a more elegant way to do this..

  Inbox
  Sent
  Archived

constructor(private cd: ChangeDetectorRef) {}

tick() {
  setTimeout(() => {
    this.cd.markForCheck();
  });
}

Nice, I will try that.  I skipped  all together and just bound to a variable.

  Tab 1
  Tab 1
  Tab 1

  One

  Two

  Three


Filter function limited to two args

Addressing https://github.com/microsoft/Power-Fx/issues/1514.
Advantages:

This brings Filter and LookUp into alignment.
Using And to combine predicates is easier to understand and read than knowing that extra arguments are And'd together.
Limiting to two arguments allows us to add a reduction argument like LookUp has.  Which enables us to push shaping of the result up the stack more easily when making a network call.

For C# clients, the Filter function was already effectively limited to two args because the interpreter couldn't handle it (a bug), so no change there.
For Canvas, this is a breaking Power Fx 1.0 change (under Power FxV1CompatibiltyRules flag) that we'll need to manage later, but only ~13% of filter calls use the three or more argument form.  A back compat converter should be possible.

✅ No public API change.

✅ No public API change.

✅ No public API change.


onEnter / onLeave not re-fired after component updates

This becomes an issue if the waypoint ends up with the same position after a component update. Use case is that the component was triggered initially with a position of inside, called onEnter (which then loaded more elements, causing a component update), and then ended up with a position of inside again. In the case of a single waypoint, onEnter / onLeave would not be fired again (as the position was not changed)

FYI, this may be fixed by adding this._previousPosition = null to the componentDidUpdate() function.

I see your issue, and why re-triggering onEnter would solve that. But constantly firing onEnter on component update would quickly become a problem I think. One way you can solve the issue you are describing is to set a key prop on the waypoint that changes when a new "page" has been loaded. This will cause a new waypoint to be mounted instead of updating a previous one.
<Waypoint key={index} onEnter={...} />

@trotzig - I just verified that setting the key prop works perfectly for my use case. Thanks for the tip. It would be great to see an example of this use case somewhere in the docs ;-)

Great! I added some info to the readme in the "Usage" section:
https://github.com/brigade/react-waypoint#usage
Let me know if you think that's enough!

Perfect. Thanks for the quick turnaround!


Guessing of ContentType

When syncing a directory tree to S3 (using sync/dsync) the ContentType is being set as binary/octet-stream on all files.
s4cmd --num-threads=100 --recursive dsync local_app_folder/static/dist/ s3:/my_s3_bucket/app/static/ --API-ACL="public-read" --API-CacheControl="max-age=31536000, public"

As the directory structure contains varying filetypes its not possible to set using the --API-ContentType parameter. I can set the --API-ContentType to a blank string but that actually sets the contentType as blank on S3 which has unexpected behaviour when trying to load some file formats.
s4cmd --num-threads=100 --recursive dsync local_app_folder/static/dist/ s3:/my_s3_bucket/app/static/ --API-ACL="public-read" --API-CacheControl="max-age=31536000, public" --API-ContentType=""

It would be good to support the mimetype guessing like S3cmd

what was the reason behind closing this as not planned?
is there any chance that the mime type guessing is finally merged?


UC price files in wrong location

Hello!
I'd like to suggest a feature and/or bug fix: Could you please move the UC price files to Forge's config directory? ({gamedir}/config/universalcoins instead of ./config/universalcoins)
I recently recieved a mod pack from a friend, and thought the prices were way off but it turns out it wasn't reading from his price files, since he had them in {gamedir}/config/universalcoins, but my Minecraft instance was looking for them at ./config/universalcoins (where it generated the default price files).
Thanks :D

The config files should be in {gamedir}/config/universalcoins. I've run on Windows and Linux without issues. What OS and Java version are you using? Are you using a custom launcher?

I'm on Linux, OpenJDK, and yes a custom 'launcher'. But in the code...
https://github.com/notabadminer/UniversalCoinsMod/blob/master/java/universalcoins/util/UCItemPricer.java#L42
private static String configDir = "." + File.separatorChar + "config" + File.separatorChar + "universalcoins" + File.separatorChar;

It specificly points to ./config/universalcoins, and not {gamedir}. Other mods put their config files in {gamedir}/config somehow, so I thought it would be a fairly simple change that I think would be a really great addition to your mod.
 I looked it up and it turns out Forge tells you what directory you can dump your config files in, which is even better! I believe the correct code is
new File(event.getModConfigurationDirectory(), "Folder name, such as the modID");

This also allows you to watch for config file updates too (if you're not already doing that? I haven't checked actually >.<)
 @SubscribeEvent
public void onConfigFileChanged(ConfigFileChangedEvent event) {
    //update prices please so that I don't have to restart every time!!!! Thanks :D
}

Hi, me again! I looked it up and there is a specific directory Forge has for dumping config things, it's FMLPreInitializationEvent.getModConfigurationDirectory()
http://cmicro.github.io/NeatCraft/forge-javadoc/cpw/mods/fml/common/event/FMLPreInitializationEvent.html#getModConfigurationDirectory()

With the unmodified launcher, "."  is the minecraft folder. Which is why using the code config/universalcoins was used. I wasn't aware of any issues as I haven't used any other launchers. I will change the code to match how NEI gets the config dir and it should resolve the issue.
NEI uses FMLInjectionData.data()[6] which returns the Minecraft home dir.
The new line will be:
private static String configDir = FMLInjectionData.data()[6] + "/config/universalcoins/";

Thanks! That code looks pretty nifty. Now I can just drag and drop one config directory when I get an update from my friends.
Now I'll just need to message the AW2 devs. Maybe they'll use that line of code too.


Can not login to CVAT on kubernetes k3s installed via helm

My actions before raising this issue

[x ] Read/searched the docs
[ x] Searched past issues

Steps to Reproduce (for bugs)

Follow the Helm install instructions without overrides
The install went well and all pods are running
I can port forward the backend server and front end server and hit the login page
I did create a super user

Expected Behaviour
Should be able to login
Current Behaviour

Possible Solution

Context

Your Environment
Running in a VM on a k3s instance kubernetes version 1.25

The issue is defenetly related with ingress, that route all requests to the frontend insdead of backend for a reason, could you provide output of kubectl describe ingress cvat

Thank you for the response as I really want to get this working
Following you will find the output from  kubectl describe ingress -n cvat

I can port forward the backend server and front end server and hit the login page

This will not work, you need expose traefik LB service ports instead of cvat application servece ports, please read k3s docs https://docs.k3s.io/networking

I am not using the Traefik / k3s load balancer
In other words, I disabled the Traefik that ships with k3s and I'm using the Traefik that ships with CVAT with the default configuration and you can see an External IP is set on the LB
Are you saying I have to create IngressRoutes for the services as well ?
Your doc's do not mention anything like that other than a reference to Minikube
With regard to port-forwarding when I disable the load balancer, I should be able to port forward but that does not work as I've tried it.


ModuleNotFoundError: No module named 'langchain.schema'

langchain==0.0.20
from langchain.schema import messages_to_dict
----> 1 from langchain.schema import messages_to_dict
ModuleNotFoundError: No module named 'langchain.schema'

You'll have to upgrade langchain. Version 0.0.20 was ~ 20 days after it was first created

pip install -U langchain


Indirect access inference error

Having fixed problems with data flow inference via relays (issue #89), it is apparent that when data flows via proxies and a proxy acts as an explicit (active) relay (it 'serves' rather than 'relays' the data), some indirect access relationships via the proxy are missed.
This is due to a limitation in the relevant construction pattern (DFuCuS-C-P+u).

What should happen is this.
If a Process acting as a client is the source or destination of a data flow (i.e., an active relay), it means that process must have a trust relationship with services that carry this data flow to or from that client. This need not extend all the way back to the destination or source of the data flow - the client may have a relationship with an intermediary that has its own relationships with the destination or source or other intermediaries. But the client must have such a relationship with a process involved in the data flow.
If the client process is a proxy in a chain, and the other end of the data flow is further down the chain (towards or beyond a back-end service), then the process is not acting as a simple proxy with respect to this data. It should have a trust relationship with at least some service(s) further along the chain.
If the client is an authenticating proxy, this is not a problem as it authenticates on its own behalf with its back-end services. Though the issue with DFuCuS-C-P+u means the fact that it is in a proxy chain prevents the indirect usage relationships being detected.
If it is a transparent proxy, there is a problem because the client does not use its own credentials to authenticate with back-end services. In that situation, should we

transfer the relationshp to the original client of this proxy?
conclude that this is a modelling error?

If the service where the data flow terminates is also a proxy in a chain, then it also is not acting as a simple proxy. At least some of the processes further up the chain (towards the client) must have a trust relationship with this service. Again - this is not a problem if the process is an authenticating proxy (other than fixing the patterns to detect it), but if the service is a transparent proxy, there is a conceptual problem. Again, should we:

transfer the relationshp to a back-end service whence the data has come?
conclude that this is a modelling error?

Now resolved. See this comment in Issue #85.

Now resolved through a new interpretation in which being a proxy is independent on whether the service authenticates (a) locally, (b) by forwarding the connection request to a back-end, or (c) by using a token or code that includes back-end rights. Options (b) and (c) are now referred to as 'credential forwarding/sharing'. See #85.


Preserve URL Query Parameters

Is there any way to preserve query parameters other than ?module and ?meta when requesting a package? I've noticed that adding any other query parameter will result in a 302 redirect removing the query from the url.
My use case is to provide the ability to define an HTML custom element with a custom name upon loading. Currently my project uses a name query parameter from the import specifier to allow clients to customize the name used for the element.
Request: GET https://unpkg.com/tts-element@0.0.2-beta/dist/text-to-speech/defined.js?name=tts-ele
Response: 302 Location<EMAIL_ADDRESS>Are arbitrary query parameters not supported, or am I just missing something? The closest I've seen in regards to this issue is #93.

As a workaround I can provide documentation that suggests using another CDN, for example JSDelivr works without removing query parameters.
GET https://cdn.jsdelivr.net/npm/tts-element@0.0.2-beta/dist/text-to-speech/defined.js?name=my-tts  returns a 200 allowing my element to work as expected when loaded with a query parameter in the specifier.


deprecated message

cell.js:356 [Deprecation] 'window.webkitStorageInfo' is deprecated. Please use 'navigator.webkitTemporaryStorage' or 'navigator.webkitPersistentStorage' instead.
(anonymous)	@	cell.js:356
detect	@	cell.js:354
create	@	cell.js:379
(anonymous)	@	cell.js:404

This is related to https://github.com/intercellular/cell/issues/122 and not a real problem, but it keeps coming up and is kind of annoying to see that warning message every time, so I just got rid of it. Please try the latest develop branch.

thanks


How to use tensorrt_llm for V100 16G version

Our company wants to use TensorRT-LLM to accelerate the ChatGLM2-6B model. Currently, we are using the V100 16GB GPU.
I quantized the model to int8 based on your instructions. However, when using TritonServer to load the model, an error regarding insufficient VRAM occurred.
Are there any options available to reduce the VRAM usage while loading the model?

Can you share the fulll log?

I notice you also reply in my another issue of Tensorrt_LLM, so I reply you both here.

here is my scripts to build the chatglm2 for single gpu:

python build.py --dtype float16 --use_gpt_attention_plugin float16 --use_gemm_plugin float16 --use_weight_only --weight_only_precition int4 --use_inflight_batching
in addition, "use_inflight_batching" is not supported in original build.py, it will lead to this error:   so I use the modified one, here is the .py modified:
examples/chatglm2-6b/build.py
build.txt
/usr/local/lib/python3.10/dist-packages/tensorrt_llm/models/chatglm2_6b/model.py
model.txt
When I start the tritonserver, it tells me out of memory

here is my scripts to build the chatglm2 for multi gpus:

python build.py --dtype float16 --use_gpt_attention_plugin float16 --use_gemm_plugin float16 --use_weight_only --weight_only_precition int4 --use_inflight_batching --world_size 2
And error occur when I build the engine

@byshiue

by the way, chatglm2-6B-int4 is just 3g, but why tensorrt_llm nead more than 16g gpu memory? @byshiue

When you don't use --enable_context_fmha, attention requires lots of memory to put activation (depending on the problem size).
And inflight_batching will enable paged attention, which allocated lots of memory for k/v cache at the beginning.

When you don't use --enable_context_fmha, attention requires lots of memory to put activation (depending on the problem size). And inflight_batching will enable paged attention, which allocated lots of memory for k/v cache at the beginning.

So, how can I use tensorrt_llm for v100 16GB version?

Can you try using latest main branch and set smaller batch size, input length and output length first?

Can you try using latest main branch and set smaller batch size, input length and output length first?

Thanks for dealing with my issue, I change the config you told me, and the GPU memory goes down to 14GB

Thank you for the update. Close this bug. Feel free to reopen it if needed.


Arithmetic operators for decimal timeseries

There are no overloads for the arithmetic operators for the type
Series < DateTimeOffset, decimal >
Would it be possible to add this?

Fixed in https://github.com/fslaborg/Deedle/pull/351


Revert lazy overload (#358, #351) space = overload(shift, space) does not work for FCITX5

I set
space = overload(shift, space)
and worked very smoothly until 13/10/2022 update.
Tapping space bar action allocated to convert by FCITX5 and other IME does not work with this update.

The old/new behaviour is to eagerly emit shift and release it before subsequently producing a space. It was briefly changed for implementation reasons, but this broke a few use cases (e.g click+drag).
I assume FCITX is one of the few programs which struggles with the extraneous shift (though I'm not sure why it would).  One way to avoid this is to use time based overloading (which is probably a superior solution for shift anyway). E.g space = overloadt(shift, space, 150) (modifying 150 to taste) which should also reduce the number of misfires.

space = overloadt(shift, space, 150) with FCITX works!
Thank you.


Change 'split selection into lines' shortcut on Linux

Current combination is a system default for locking the machine/ standby. New combination follows convention used by other editors and should work without affecting any system defaults.

Changed to less ambiguous title


New version: ThreeBodyDecays v0.10.2

Registering package: ThreeBodyDecays
Repository: https://github.com/mmikhasenko/ThreeBodyDecays.jl
Created by: @mmikhasenko
Version: v0.10.2
Commit: 46d5332323fd4bc8ee6b1107940d8aac61c77d95
Reviewed by: @mmikhasenko
Reference: https://github.com/mmikhasenko/ThreeBodyDecays.jl/issues/17
Description: Builder of the three-body decay model using cascade approach

[merge approved]


Server crash with EnderIO Please help

Issue Description:
So i made a modpack to play with 2 of my friends with and i keep getting these crashes on my server
What happens:
After afew minutes of the server being up the server just crashes, ive noticed that everytime i crash it is when i am near or attacking a wither skeleton which apparently according to WAILA is from EnderIO so thats what brought me here
What you expected to happen:
I expected for the server to carry on with normal activity instead of crashing
Steps to reproduce:

Get on the server
Play for a few minutes
Kablam you have a crash
...

Affected Versions (Do not use "latest"):

EnderIO: EnderIO-1.7.10-<IP_ADDRESS>4_beta
EnderCore: 1.7.10-<IP_ADDRESS>_beta
Minecraft: 1.7.10
Forge: forge-1.7.10-<IP_ADDRESS>4-1.7.10

Your most recent log file where the issue was present:
Latest Log: https://gist.github.com/Dartherez1234/03104b895bb4d90804040f572b03e135
Crash Report: http://pastebin.com/GY8Rb137

What makes you think this is related to Ender IO?

There is a config to turn off Ender IO's replacement of Wither Skeletons.  See if changing that to false fixes it.
The log ends with over 200 of the following line.  You might want to look at MFR as well.
[Server thread/DEBUG] [MineFactoryReloaded/MineFactoryReloaded]: [IMCHandler] %s from %s

Thanks for the help really apprecate it will have a look at MFR for you see if anything is wrong

I don't have any reasonably sound ideas.  I'm not a programmer, just someone who has tried making my own personal modpacks and found some things to look for in crash logs.  You should probably poke a programmer from MFR and ask if that repeating line until crash (server log) is as incriminating as it seems.
Since you mentioned Wither Skeletons, are you spawning them in with an Auto-Spawner?  If you are, then try adjusting Ender IO's Wither Skeleton replacement in the config, and get a new skeleton (I have no idea what will happen to the one in that safari net).

9 days? Closing. Hope you could fix your problem.


hcxdumptool: invalid option -- 'o'

Please help. I installed the hcxdumptool tool and it shows errors. Please help
hcxdumptool -i wlan0  -o dumpfile.pcapng -active_beacon -enable_status=15
hcxdumptool: invalid option -- 'o'
hcxdumptool 6.3.2-142-gf977403 (C) 2023 by ZeroBeat
This is a penetration testing tool!
It is made to detect vulnerabilities in your NETWORK mercilessly!
usage:
$ hcxdumptool -h for an overview of all options
$ hcxdumptool --help for an example and trouble shooting

i think you copied this command from this old reddit post
hcxdumptool has changed a lot since then, check hcxdumptool --help to see what options you can use (-o, --active_beacon and --enable_status have all been removed/renamed)

I did not copy anything from anyone, but this happened to me. Can you write me the complete code?
@gemesa

I did not copy anything from anyone, but this happened to me. Can you write me the complete code? @gemesa

The one you use is the old command. Use
hcxdumptool --help
for new command and option list,

First step after an update should always be to take a look the the change log:
03.05.2023
==========
hcxdumptool: changed/added several options:

Second step should be to take a look at the short help "-h" and for additional options "--help"
$ hcxdumptool -h
hcxdumptool 6.3.2-142-gf977403  (C) 2023 ZeroBeat
usage: hcxdumptool <options>

most common options:
--------------------
...
-w <outfile>   : write packets to a pcapng-format file named <outfile>
                  existing dump file file will be overwritten
                  default outfile name: yyyyddmmhhmmss-interfacename.pcapng
                  existing dump file file will not be overwritten
                  get more information: https://pcapng.com/
...

The FAQ are helpful, too:
https://github.com/ZerBea/hcxdumptool/discussions/330
Closed this here, because it is already answered.

Closed this here, because it is already answered.


apxproof breaks fancyvrb macros

Currently, the apxproof package redefines the VerbatimOut environment in a manner, that breaks it for any use other than in apxproof. This seems quite likely to produce severe conflicts with other packages.
How to reproduce
The following file will compile as expected.
But if apxproof is imported, it fails.
\documentclass[a5paper,12pt]{article}

\usepackage{fancyvrb}

\begin{document}

Before VerbatimOut.

\begin{VerbatimOut}{\jobname.x}
First line.
Second line.
\end{VerbatimOut}

After VerbatimOut, before VerbatimInput.

\VerbatimInput{\jobname.x}

After VerbatimInput.

\end{document}

Possible solutions

Defining a package-local version axp@VerbatimOut, instead of redefining the global VerbatimOut environment.
Using an entirely different mechanism for accumulating appendix contents; On modern hardware, an accumulation form \appto{\axp@appendixcontents}{CONTENTS} (or \eappto with \expandonce{\BODY} when using  \NewEnviron) should work too.

Yes, I am very aware of this issue, which is discussed among known issues in the documentation. The first solution is the obvious one, and I should be able to use \let commands so as not to have to redefine the entire behavior of the verbatim package. I am a bit wary of the second option since it means keeping in memory a potentially very large amount of content, but I may be a bit too conservative.
It should not be long to implement the first solution, I will try and do this in the next few days.

Fixed in 1e3759e. It was even easier than I thought, as fancyvrb is very modular.


Code: RegistrationDefinitionInvalidUpdate

Hello, I'm trying to do this deployment and it keeps failing, I didn't find any information related to the error on the internet.
I'm getting the following message:
New-AzDeployment: 2:58:04 PM - The deployment 'AzureADB2C' failed with error(s). Showing 1 out of 1 error(s).
Status Message: The registration definition 'xxxx-xxx-xxx-xxx-xxxxx' property ManagedByTenantId not allowed to update. (Code:RegistrationDefinitionInvalidUpdate)
Any idea of what might be wrong?
I'm sure that I'm using the correct values for the params file.
Thanks in advance

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

ID: 24e34c89-9a70-b5b2-3b38-bc81660f83c1
Version Independent ID: 77fa8d3c-56dd-294f-13ae-ac6ba44018a7
Content: Monitor Azure AD B2C with Azure Monitor - Azure AD B2C
Content Source: articles/active-directory-b2c/azure-monitor.md
Service: active-directory
Sub-service: b2c
GitHub Login: @msmimart
Microsoft Alias: mimart

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

@robermud
Thanks for your patience, looking through the documentation and trying to reproduce your issue, I wasn't able to, due to subscription limitations.
However, can you make sure the ARM template you selected was correct for the type of resource you're trying to onboard, i.e.

If you're still having issues, can you please email me with the below info. I can then enable a one-time free support request for you, so our support engineers can take a closer look into your issue.
Email<EMAIL_ADDRESS>Title: ATTN - James Tran
Body:
SubscriptionID
Link to this GitHub issue
If you have any other questions, please let me know.
Thank you for your time and patience throughout this issue.

@JamesTran-MSFT
Thanks for your response, I already opened a ticket with our Azure support because I've been struggling with this since yesterday.

@robermud
Thank you for the quick response, I apologize for not responding earlier. However, since you're working with our support team, I'll go ahead and close out this GitHub issue at the end of the day, pending any other questions on your end.
Thank you again for your time and patience throughout this issue.

@JamesTran-MSFT Thanks for your response, I already opened a ticket with our Azure support because I've been struggling with this since yesterday.

@robermud
Please refer to the latest note I just left.
Please make sure that there are no other instances on that Log Analytics using the same Instance Name.


Add single pending review mode

I needed this for testing pr https://github.com/conan-io/conan-center-index/pull/16144#issuecomment-1468859516
No idea why it was missing but hopefully will be able to catch the problem in the future 🤞

Codecov Report

Merging #121 (59f63c3) into main (f1c5d0e) will increase coverage by 9.85%.
The diff coverage is 55.69%.

@@            Coverage Diff             @@
##             main     #121      +/-   ##
==========================================
+ Coverage   44.33%   54.18%   +9.85%
==========================================
  Files          15       16       +1
  Lines         609      609
==========================================
+ Hits          270      330      +60
+ Misses        316      243      -73
- Partials       23       36      +13

Impacted Files
Coverage Δ

pending_review/changes.go
49.27% <49.27%> (ø)

pending_review/pull_request.go
57.00% <100.00%> (+32.73%)
:arrow_up:

... and 2 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Problem with Snackbar component size

Duplicates ❌

[X] I have searched the existing issues

Which Figma library is concerned?
Vitamin Android UI Kit
Current behavior 😯
Android Snackbar height size in Figma library is 36.
The right height size is 48
Expected behavior 🤔
No response
Context 🔦
No response
Screenshots / Videos 📸
No response

linked https://github.com/Decathlon/vitamin-design/issues/90

Thanks @Sabrinavigil.
I close this issue in favor of #90 which has more details. And just added "duplicate" label.


⚠ Secret Website 3 has degraded performance

In d728a6a, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 2537 ms

Resolved: Secret Website 3 performance has improved in ad3215f.


User change own password

Hi Thomas,
I have a concern regarding how a user can change his own password.
From my understanding, the APIs are more or less designed for an user admin. While this is fine probably for most use cases, an user might want to change his own password since he does not want the admin to know his password. At the moment, it seems to me that it is not possible for a non-admin role, or did I miss something?
Best regards,
Xiaoyang

@x19chen OpenIddict uses ASP.Net Core Identity for user management.  You wouldn't necessarily use OpenIddict-ui to allow a user to change their own password; openiddict-ui is for an admin role not a user role.
You can most likely use some of the existing ASP.Net Core Identity functionality to do this. You would just need to write an api endpoint that allows a user to change their password similar to how the existing ASP.Net Core Identity functionality does it.
If you've generated all the Identity code for your project you should have a ChangePassword page somewhere in your solution.  For example there is a Change Password page which is most likely located under Areas/Identity/Pages/Account/Manage/ChangePassword.  You can see how this is done via the existing ASP.Net Core web application using the following code:
    public async Task<IActionResult> OnPostAsync()
    {
        if (!ModelState.IsValid)
        {
            return Page();
        }

        var user = await _userManager.GetUserAsync(User);
        if (user == null)
        {
            return NotFound($"Unable to load user with ID '{_userManager.GetUserId(User)}'.");
        }

        var changePasswordResult = await _userManager.ChangePasswordAsync(user, Input.OldPassword, Input.NewPassword);
        if (!changePasswordResult.Succeeded)
        {
            foreach (var error in changePasswordResult.Errors)
            {
                ModelState.AddModelError(string.Empty, error.Description);
            }
            return Page();
        }

        await _signInManager.RefreshSignInAsync(user);
        _logger.LogInformation("User changed their password successfully.");
        StatusMessage = "Your password has been changed.";

        return RedirectToPage();
    }
}

All you will need to do is create an api endpoint that requires authentication and allows the authenticated user to pass in the data to change their password.
Let me know if this helps.
-Robert

@browrp Thanks for answering the question of @x19chen. I couldn't have done it better 👍.
@x19chen For some inspiration you could have a look here or as @browrp mentioned have a look at ASP.NET Core Identity samples.
Have a nice weekend both.
Thomas

@browrp @thomasduft
Thanks for your answer, that was indeed helpful.
It actually confirms my suspicion that I have to create another API for that and the example is clear.
Enjoy your weekend.
Xiaoyang


Major: decoupled task queues and consumer threads, overhauled/simplif…

…ied API interface

moved to main branch


The file still displays on the react file explorer if type a string on the address bar then press 'Enter'

Storage Explorer Version: 1.32.0-dev (93)
Build Number: 20230911.1
Branch: main
Platform/OS: Windows 10/Linux Ubuntu 20.04/MacOS Ventura 13.5 (Apple M1 Pro)
Architecture: x64/arm64/x64
How Found: From running test case
Regression From: Previous release (1.31.1)
Steps to Reproduce

Expand one storage account ->File Shares.
Right click one file share -> Click '[New] Open'.
Upload one file.
Type 'test' on the address bar -> Press 'Enter'.
Check no file displays on the file explorer.

Expected Experience
No file displays on the file explorer.

Actual Experience
The file displays on the file explorer.

@v-xianya The folder doesn't exist in the share. Do you get any error message when you enter the folder name?

@craxal  Yes, there is an error info bar when entering the folder name.

Verified this issue on the main build 20230912.1. Reproduce.

Verified this issue on the main build 20230913.4. Fixed. So, we are going to close it.


Move gradle tests to gradle 3.1

See if this solves the issue.

retest this please

so many ways to write gradle wrongly :)

Retest this please

Seems you found another one?

closing for now


Error: Invalid character in string - how to sanitize data or gratefully fallback?

I noticed, that if string contains data, like Some str\u0010ing then sitemap completely freaks out and throws an error
Error: Invalid character in string: Some string at index 33
    at XMLStringifier.assertLegalChar (/home/.../node_modules/xmlbuilder/lib/XMLStringifier.js:208:19)

Tried to search a way how to sanitize the data (in xmlbuilder issues also) but have no luck.
Is there any recommendations what to do in such cases? Any dedicated libs? I can't sanitize only a \u0010 since there could be dozens of other symbols.

@bdrtsky can you paste a small sample so I may debug? reading the spec from the comments in xml builder it /should/ accept most utf8/16 unicode

It's possible bumping the xml version of the sitemap to 1.1 might help. If you wish to test for yourself you can reach into the source and tweak this line with 1.1. https://github.com/ekalinin/sitemap.js/blob/master/lib/sitemap-stream.ts#L12


Swap gitbook for mdbook

What does this PR do?
Gitbook has changed, and we need to move away. https://github.com/rust-lang/mdBook seems a reasonable alternative, and I didn't need to make many changes.
One downside is it means anybody working on the docs needs to install mdbook too. It was available in community repo in arch though, so maybe available easily? Otherwise downloading from github releases is fairly easy.
This is also useful as gitbook installs a really old version of npm, which itself installs an old version of graceful-fs that doesn't work with node 12+, and that npm gets called when doing a cordova install... that's actually why I started doing this in the first place :)
Links to related issues
Checklist

[ ] added a test, or explain why one is not needed/possible...
[ ] no unrelated changes
[ ] asked someone for a code review
[ ] joined chat.foodsaving.world/channel/karrot-dev
[ ] added an entry to CHANGELOG.md (description, pull request link, username(s))
[ ] tried out on a mobile device (does everything work as expected on a smaller screen?)

Looks good to me + works on my machine


Cannot convert value of type 'Bool' to expected argument type '[Bool]'

i  use webview_cookie_manager 1.0.6

Same issue here. Started popping up today.

Same issue here. Started popping up today.

@michaelgobbers @bert1100 Set webview_cookie_manager version fixed to 1.0.5 and give it a try

@michaelgobbers @bert1100 Set webview_cookie_manager version fixed to 1.0.5 and give it a try

Same issue here ...

Same issue here ...

will push package soon 1.0.7

will push package soon 1.0.7

@fryette in 1.0.5 everything is ok

@fryette in 1.0.5 everything is ok

@bru95 please confirm that in 1.0.7 everything is fine

@bru95 please confirm that in 1.0.7 everything is fine

Yes, 1.0.7 is ok.
Thanks!

Yes, 1.0.7 is ok.
Thanks!


Feature: Render page labels

Description
When creating the logical outline structure for PDF-Navigation, it is required to render an entry for each single Page, with the current page ORDERLABEL attribute displayed.

Done


Convert paramter to wrong type  , and causes very large performances problems

https://github.com/mauricio/postgresql-async/blob/master/postgresql-async/src/main/scala/com/github/mauricio/async/db/postgresql/column/PostgreSQLColumnEncoderRegistry.scala#L33
 classOf[Int] -> (IntegerEncoderDecoder -> ColumnTypes.Numeric),
    classOf[java.lang.Integer] -> (IntegerEncoderDecoder -> ColumnTypes.Numeric),

    classOf[java.lang.Short] -> (ShortEncoderDecoder -> ColumnTypes.Numeric),
    classOf[Short] -> (ShortEncoderDecoder -> ColumnTypes.Numeric),

    classOf[Long] -> (LongEncoderDecoder -> ColumnTypes.Numeric),
    classOf[java.lang.Long] -> (LongEncoderDecoder -> ColumnTypes.Numeric),

Convert  Integer、Long、Short、Float and Double to Numeric , and the attribute of  table is Integer or Long,  If you use preparedStatement ,  and will causes larger performances problems.
e.g
Having table users, and  has attribute uid with primary key.

use statment and find searching by index

explain analyse select * from users where uid in (200,500,800);

use preparedStatement with correct paramter type, and find searching by index

 PREPARE fooplan (int, int, int) AS
  select *
  from users
  where uid in ($1, $2, $3);
  explain analyse EXECUTE fooplan(200, 500, 800);

use preparedStatement with wrong paramter type, and find searching by seq

  PREPARE fooplan1 (numeric, numeric, numeric) AS
  select *
  from users
  where uid in ($1, $2, $3);
  explain analyse EXECUTE fooplan1(200, 500, 800);

related to #212 and #241

I fixed/reverted this in jasync-sql(fork) 0.8.30: https://github.com/jasync-sql/jasync-sql/pull/16 . more details in the issue: https://github.com/jasync-sql/jasync-sql/issues/15


Add append function to array to insert multiple values.

The new API allows to push multiple values to an array. I believe it will be more efficient than insert them one by one using the insert function as it will perform shift only once (which is probably implemented more efficient).
Let me know what you think, thanks.

Hi @MeirShpilraien, thanks for the PR. However, I think append is the wrong name for this method. I'd also rather not add methods that do not already exist on Vec.
The closest method would be Vec::splice - if you want to refactor this PR to implement the same method for IArray then I would be happy to accept that. However, please take care to avoid introducing UB - calling user-provided methods such as those on the Iterator or Into traits whilst in an unsafe block can be dangerous, so look to the libstd implementations for reference.

Thanks @Diggsey , maybe I am missing something (new to rust), but isn't append exists for Vec (https://doc.rust-lang.org/std/vec/struct.Vec.html#method.append)? The difference is that it takes another Vec and leave it empty, is this what you meant? that we are not compatible to Vec?
Good point about the unsafe comment, will fix that (though I thought unsafe is compile time only so not sure if it will effect the user? will it?)

Yes, having a method with the same name that does something different would be even worse than having a method that doesn't exist on Vec at all 😛

Good point about the unsafe comment, will fix that (though I thought unsafe is compile time only so not sure if it will effect the user? will it?)

It's not that there was UB, but if you call into user-code you have to be aware that it might panic (which if called from unsafe could result in broken safety invariants) or might observe the invariants that are temporarily broken by the unsafe code.

You are right, I was intended to implement splice and mistakenly call it append. I will fix that and will also fix your comment about unsafe. Thanks for your help and feedback.

Closing due to inactivity. Feel free to re-open.


Feature Request: increase / decrease spot size

Add command line command to increase/decrease the spot size (with a given delta)

still needs documentation but now
projecteur -c spot.size.adjust=-3
or
projecteur -c spot.size.adjust=+5
work when the related PR is merged.


Automated cherry pick of #2965: feature: support xsky object storage

Cherry pick of #2965 on release/2.11.
#2965: feature: support xsky object storage

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.11@27e8c9f). Click here to learn what that means.
The diff coverage is n/a.

@@              Coverage Diff               @@
##             release/2.11   #2978   +/-   ##
==============================================
  Coverage                ?   7.05%
==============================================
  Files                   ?     475
  Lines                   ?   76354
  Branches                ?       0
==============================================
  Hits                    ?    5384
  Misses                  ?   70517
  Partials                ?     453

Flag
Coverage Δ

#aFlag
7.05% <ø> (?)

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 27e8c9f...dd1ae16. Read the comment docs.

/approve
/lgtm


Fork me on github banner

I think the website need a github banner in a corner
https://bit.ly/2BZ3Iom

Can I add this?

@frextrite Go ahead :)


ios build fails

Environment
Library version: 5.0.0
OS version: iOS 17.4
Affected platforms

[ ] Android
[X] iOS

Current behavior

❌  Undefined symbols for architecture arm64
┌─ Symbol: _AvoidSoftInputViewCls
└─ Referenced from: _RCTThirdPartyFabricComponentsProvider in libReact-RCTFabric.a[38](RCTThirdPartyFabricComponentsProvider.o)

❌  ld: symbol(s) not found for architecture arm64

❌  clang: error: linker command failed with exit code 1 (use -v to see invocation)

Expected behavior
Build succeeds
Reproduction
I'm just posting the bash prompt and if that doesn't help feel free to close this

I cleared build cache and deleted node modules and now it works


Setup nodejs server ei wo mi sau

It took me quite a while to find out how to get my own Node.JS web server running with your hosting package "EiWoMiSau" and I wanted to help others get there faster with this tutorial.
I have read and understood the Contributor's Certificate of Origin at the end of the template and I hereby certify that I meet the contribution criteria described in it.

Thank you very much for your submission, it has just been finally reviewed and was accepted into our repository. However, I must inform you that a check of the mail address you have attached to your Github profile has found that said address is not currently connected to any existing netcup customer account. Could you please add the mailing address you have used to register at netcup to your Github account and add a comment on here, so I may forward that to the department responsible for handling vouchers? Alternatively, you may also send a mail to me at<EMAIL_ADDRESS>if that is preferred.

Dear Christoph,
Thank you very much for your feedback on my tutorial.
My Netcup e-mail address is @.*** and my Netcup ID: 224079.
I also see that you have deleted the English version of the tutorial.
"netcupChristophW: Delete community-tutorials/setup-nodejs-server-EiWoMiSau/01-en.md"
What is this all about?
Best regards
Mike

PS: Let's try this way now. @.*** address did not work.

The following message to @.> was undeliverable.
The reason for the problem:
5.1.0 - Unknown address error 550-'5.1.1 @.>: Recipient address rejected: User unknown in virtual mailbox table'

Gesendet: Dienstag, 30. Januar 2024 um 07:57 Uhr
Von: "Christoph" @.>
An: "netcup-community/community-tutorials" @.>
Cc: "mzaiprog" @.>, "Author" @.>
Betreff: Re: [netcup-community/community-tutorials] Setup nodejs server ei wo mi sau (PR #159)

Thank you very much for your submission, it has just been finally reviewed and was accepted into our repository. However, I must inform you that a check of the mail address you have attached to your Github profile has found that said address is not currently connected to any existing netcup customer account. Could you please add the mailing address you have used to register at netcup to your Github account and add a comment on here, so I may forward that to the department responsible for handling vouchers? Alternatively, you may also send a mail to me at @.*** if that is preferred.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Analyzer does not provide correct VCS revision of Pub dependencies

This issue happens when analyzing a Pub dependency, which is part of Dart-lang sdk.
The pubspec.lock file contains the following:
packages:
  _fe_analyzer_shared:
    dependency: transitive
    description:
      name: _fe_analyzer_shared
      url: "https://pub.dartlang.org"
    source: hosted
    version: "22.0.0"

dart pub get is invoked, and it correctly downloads the dependency in the .pub-cache directory. From package_config.json:
"packages": [
    {
      "name": "_fe_analyzer_shared",
      "rootUri": "file:///root/.pub-cache/hosted/pub.dartlang.org/_fe_analyzer_shared-22.0.0",
      "packageUri": "lib/",
      "languageVersion": "2.12"
    },

That folder contain the pubspec.yaml that has been downloaded by pub.dev/packages, which contains:
name: _fe_analyzer_shared
version: 22.0.0
description: Logic that is shared between the front_end and analyzer packages.
homepage: https://github.com/dart-lang/sdk/tree/master/pkg/_fe_analyzer_shared

environment:
  sdk: '>=2.12.0 <3.0.0'
dependencies:
  meta: ^1.0.2
dev_dependencies:
  test: ^1.3.4

The analyzer takes the value of homepage to build the VCS info for the dependency, leading to the following entry in analyzer-result.json:
          "vcs_processed" : {
            "type" : "Git",
            "url" : "https://github.com/dart-lang/sdk.git",
            "revision" : "master",
            "path" : "pkg/_fe_analyzer_shared"
          }

The revision is set to master, but that does not correspond to the required 22.0.0 version.
Scancode then fails to find the required version 22.0.0 among the tags of the repository, because these tags refer to the SDK version, with the error  No matching tag for version '22.0.0' found in [2.18.0-89.0.dev, 2.8.0-dev.12.0, ....
I looked into pub.dev but I could not find any reference of the revision corresponding to the correct dependency version, nor could I find how to obtain that information from dart pub get. I noticed that pub.dev contains a source artifact for that version, but my attempt to add a source artifact to the analyzer-result.json did not solve the issue.

@bennati, can you please check whether @maklostermann's https://github.com/oss-review-toolkit/ort/pull/6384 solves this?

@sschuberth no, that change does not solve the issue because the issue appears for packages with source 'hosted', not source 'git'
------------------------------------------
 ______________________________
/        \_______   \__    ___/ The OSS Review Toolkit, version 67b3cb8e00.
|    |   | |       _/ |    |
|    |   | |    |   \ |    |    Running 'analyze' as 'ort' under Java 11.0.18 on Linux
\________/ |____|___/ |____|    with 8 CPUs and a maximum of 27648 MiB of memory.
Environment variables:
ORT_CONFIG_DIR = ort-configuration
ORT_DATA_DIR = /builds/oss/oss-review-toolkit/ort-gitlab-ci/.ort
JAVA_HOME = /opt/java/openjdk
ANDROID_HOME = /opt/android-sdk
Looking for ORT configuration in the following file:
	/builds/oss/oss-review-toolkit/ort-gitlab-ci/ort-configuration/config.yml (does not exist)
Looking for analyzer-specific configuration in the following files and directories:
	/builds/oss/oss-review-toolkit/ort-gitlab-ci/project/pkg/_fe_analyzer_shared/.ort.yml (does not exist)
	/builds/oss/oss-review-toolkit/ort-gitlab-ci/ort-configuration/resolutions.yml
The following package managers are enabled:
	Bower, Bundler, Cargo, Carthage, CocoaPods, Composer, Conan, DotNet, GoDep, GoMod, Gradle, Maven, NPM, NuGet, PIP, Pipenv, PNPM, Poetry, Pub, SBT, SpdxDocumentFile, Stack, Unmanaged, Yarn, Yarn2
Analyzing project path:
	/builds/oss/oss-review-toolkit/ort-gitlab-ci/project/pkg/_fe_analyzer_shared
Found 1 Pub definition file(s) at:
	pubspec.yaml
Found 1 definition file(s) from 1 package manager(s) in total.
08:59:05.722 [DefaultDispatcher-worker-1] ERROR org.ossreviewtoolkit.analyzer.PackageManager - Resolving Pub dependencies for path 'pubspec.yaml' failed with: IOException: Running '/builds/oss/oss-review-toolkit/ort-gitlab-ci/.ort/tools/flutter-2.10.5-stable/flutter/bin/dart pub get' in '/builds/oss/oss-review-toolkit/ort-gitlab-ci/project/pkg/_fe_analyzer_shared' failed with exit code 1:
The current Dart SDK version is 2.16.2.
Because _fe_analyzer_shared requires SDK version >=2.17.0 <3.0.0, version solving failed.
Writing analyzer result to '/builds/oss/oss-review-toolkit/ort-gitlab-ci/ort-results/analyzer-result.json'.
The analysis took 1m 11.288294s.
Found 1 project(s) and 0 package(s) in total (not counting excluded ones).
Applied 0 curation(s) from 2 provider(s).
Resolved issues: 0 errors, 0 warnings, 0 hints.
Unresolved issues: 1 error, 0 warnings, 0 hints.
There is 1 unresolved issue with a severity equal to or greater than the WARNING threshold.

I noticed that pub.dev contains a source artifact for that version, but my attempt to add a source artifact to the analyzer-result.json did not solve the issue.

FYI, https://github.com/oss-review-toolkit/ort/pull/8178 adds support for source artifacts hosted on a package registry.

@mnonnenmacher, are you planning to look into this issue, too, since you seem to be working on the Pub front now?

@mnonnenmacher, are you planning to look into this issue, too, since you seem to be working on the Pub front now?

Not really, I just wanted to make the quick fix for the source artifacts as this fixes many issues for us.
But I had a look at my local pub cache and it seems common that packages set their repository URL with the main/master branch if they want to point to a subdirectory in the repository. Just a few examples:
protobuf-2.1.0/pubspec.yaml:repository: https://github.com/google/protobuf.dart/tree/master/protobuf
device_info-2.0.3/pubspec.yaml:repository: https://github.com/flutter/plugins/tree/master/packages/device_info/device_info
shelf_packages_handler-3.0.1/pubspec.yaml:repository: https://github.com/dart-lang/shelf/tree/master/pkgs/shelf_packages_handler
shelf_packages_handler-3.0.2/pubspec.yaml:repository: https://github.com/dart-lang/shelf/tree/master/pkgs/shelf_packages_handler
webdriver-3.0.2/pubspec.yaml:repository: https://github.com/google/webdriver.dart
html-0.15.4/pubspec.yaml:repository: https://github.com/dart-lang/html
archive-3.3.2/pubspec.yaml:repository: https://github.com/brendan-duncan/archive
pool-1.5.1/pubspec.yaml:repository: https://github.com/dart-lang/pool
path_provider_foundation-2.2.1/pubspec.yaml:repository: https://github.com/flutter/packages/tree/main/packages/path_provider/path_provider_foundation
connectivity_macos-0.2.1+2/pubspec.yaml:repository: https://github.com/flutter/plugins/tree/master/packages/connectivity/connectivity_macos
path_provider_android-2.0.27/pubspec.yaml:repository: https://github.com/flutter/packages/tree/main/packages/path_provider/path_provider_android
plugin_platform_interface-2.1.4/pubspec.yaml:repository: https://github.com/flutter/packages/tree/main/packages/plugin_platform_interface
_fe_analyzer_shared-58.0.0/pubspec.yaml:repository: https://github.com/dart-lang/sdk/tree/main/pkg/_fe_analyzer_shared

I could not find a single package where this pointed to the correct revision. So I think we should just remove the revision for packages with source == "hosted" and leave it empty. I could not find the correct revision anywhere in the available metadata. Keeping it should not lead to the wrong sources being scanned as the downloader does not accept branch names and searches for a tag instead, but having it in the analyzer result is still wrong.
Regarding the "_fe_analyzer_shared" package from the issue description I think there is nothing the analyzer can do to find the correct revision, but with the source artifact fix mentioned above this should not be much of an issue anymore.

I could not find the correct revision anywhere in the available metadata.

Thanks for checking!

For reference, pub.dev uses the exact same URL pointing to the main/master branch for the repository link if you look at the version of a package, check for example https://pub.dev/packages/protobuf/versions/2.1.0.


Datetime bugfix

This fixes the bug raised in issue #5. I have tested locally, and everything displays nicely. @wikkiewikkie

Bump @wikkiewikkie


fix(@embark/core): ensure 0x0 address are extended to full zero addre…

…sses
This was a regression introduced in a web3 upgrade where 0x0 addresses where no longer
accepted as valid addresses and have to be replaced with full zero addresses.
This commit ensures that, if Embark apps still make use of 0x0 addresses in
their configs, they are properly replaced or extended to satisfy web3's APIs.
Fixes #956

This should be patched back to v3.2.x as well @embark-framework/core-devs

Updated!

CI is still failing while tests work fine locally. Investigating.

Alright, turns out that npm test doesn't transpile our code. So I was bascially running tests of an older version. This is now fixed. CI should be happy now.


Does my coding partner follow JS best practices?

Does my coding partner follow JS best practices?
In this milestone

Recreated the To-do-list repo to get a review from coding partner.

Hello @newhoteng
You have done a great job on this project, keep it up 👍 🥇., however there are some issues that you still need to work on to go to the next project, but you are almost there!
Required Changes

Kindly fix issues with newly created elements, so you don't have to reload the page when you create a new task


Sync Fork from Upstream Repo

Pull Request Test Coverage Report for Build b8350ba18939c931f4e00c8a6f968e3f10d13e3d-PR-20

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 98.756%

Totals

Change from base Build bd606d2652fed07261af24e0a4ba4ebdbb31bfff:
0.0%

Covered Lines:
527

Relevant Lines:
529

💛  - Coveralls


Can't play titles since version 2.2.2

Hello Philippe,
first of all, thank you very much for this plugin.
To my problem: The last version which worked for me is 2.2.1 . All versions after that don't start the title. I can select the music and add it to the playlist, but it don't start to play.
I use LMS 8.4 on an debian 12 container on promox and test with a sonos s5 and a iotavx np3 (wiim player).
Do you have an idea, or is there a log file i can check?
Thank you very much

Did you added the ARL token in the text box after adding your account into the plugin settings page? See the little information icon next to "Account".

I had the same issue, and this fix problem.
What is strange is that without arl token , you can browse and switch account
To get the arl token I follow this tuto : https://www.dumpmedia.com/deezplus/deezer-arl.html

Yes, you don't a token for browsing. But you won't play without it


fix mobile layout responsiveness for video resources

What are the relevant tickets?
Closes https://github.com/mitodl/hq/issues/2408
Description (What does it do?)

Fixes mobile layout responsiveness for video resources (titles going out of div boxes if they're bigger in length)
It also adds ellipsis -- 2 lines limit for Desktop view, 4 lines limit for Mobile view (xs screen only)
Also gives max-width of 150px to img thumbnails. Now for larger thumbnails, the div container will not grow bigger.

Screenshots (if appropriate):

How to test it

Checkout to this branch.
Use any course (just run yarn start course).
Go to Video Lectures.
Try to break responsiveness of the video div containers eg. try different combination of long names, or very short names, or different size images for thumbnail. Experiment a little, make sure nothing breaks for all screens.
Also, we've kept the font-size, and line-height same as before for screens larger than mobile (xs). Confirm that it works as expected.

@pdpinch Can you please look at the screenshots and confirm the design changes. For desktop view, we have added 2 lines of text per video div. But for mobile view we have added 4 lines of text.

Thanks @ibrahimjaved12 this looks good. Let's go ahead.


Output broken on Windows

Hi @candrewlee14 ,
thank you for this project.
I have an ugly output on a Windows cmd shell (note: other apps output is managed well, even when using colors or other ANSI escape codes)

The best option would be fix this behaviour, but imho it is acceptable to allow user to disable the colored output.
Cheers, Enrico

Windows terminal was able to fix it for me.
On win10, both cmd and powershell cannot correctly display the codes used to move the cursor, which leads to this.

What do you think about using https://github.com/mattn/go-isatty to identify whether color should be used or not as well?

Merged in #55


[4.5] UPSTREAM: docker/distribution: 3131: Add S3 virtual hosted styl…

…e paths for custom region endpoints
Signed-off-by: Corey Daley<EMAIL_ADDRESS>

/assign @adambkaplan @dmage

our registry uses the branch image-registry-4.0-distribution-16128bb

/close


977 framingham

Thank you for your Pull Request! We have developed this task checklist from the Development Process Guide to help with the final steps of the process. Completing the below tasks helps to ensure our reviewers can maximize their time on your code as well as making sure the admiral codebase remains robust and consistent.
Please check off each taskbox as an acknowledgment that you completed the task or check off that it is not relevant to your Pull Request. This checklist is part of the Github Action workflows and the Pull Request will not be merged into the devel branch until you have checked off each task.

[x] Code is formatted according to the tidyverse style guide
[x] Updated relevant unit tests or have written new unit tests - See Unit Test Guide
[x] If you removed/replaced any function and/or function parameters, did you fully follow the deprecation guidance?
[x] Update to all relevant roxygen headers and examples
[x] Run devtools::document() so all .Rd files in the man folder and the NAMESPACE file in the project root are updated appropriately
[ ] Address any updates needed for vignettes and/or templates
[ ] Update NEWS.md
[ ] Build admiral site pkgdown::build_site() and check that all affected examples are displayed correctly and that all new functions occur on the "Reference" page.
[ ] Address or fix all lintr warnings and errors - lintr::lint_package()
[ ] Run R CMD check locally and address all errors and warnings - devtools::check()
[ ] Link the issue so that it closes after successful merging.
[ ] Address all merge conflicts and resolve appropriately
[ ] Pat yourself on the back for a job well done!  Much love to your accomplishment!

First draft - please review. Thanks!

I can add AGE, SEX, SMOKEFL, DIABETFL, and TRTHYPFL to globals after review.  which is causing CMD/lint failure. (I don't want to add unless you are ok with functions)

Hi Alice,
I'm happy to review, but I think @thomas-neitmann or @bundfussr should also have look as well.

@JMcGavigan32 Hey Jack It looks like you can do tables in Roxygen headers

Awesome! I will get that done. Thanks for confirming that for me 😄

Here are the two sections I followed on the roxygen website
https://roxygen2.r-lib.org/articles/rd-formatting.html?q=table#tables-1
https://roxygen2.r-lib.org/articles/rd-formatting.html?q=table#tables

Closing in favor of Pull Request #1144


Frequency count is zero for PIN=8068.

The next lowest frequency count is 36, so a count of zero seems incorrect (or the perfect 4-digit PIN).
https://github.com/danielmiessler/SecLists/tree/master/Passwords/Common-Credentials)/four-digit-pin-codes-sorted-by-frequency-withcount.csv

It is the last value.
MR: https://github.com/danielmiessler/SecLists/issues/169
Source: https://www.reddit.com/r/hacking/comments/7ujzkf/complete_list_of_four_digit_pins_ranked_in_order/
File history: https://github.com/danielmiessler/SecLists/commits/master/Passwords/Common-Credentials/four-digit-pin-codes-sorted-by-frequency-withcount.csv

May want to ask the original author for more information about it.


소피의 아틀리에 ～신비한 책의 연금술사～ [PCSH00231]

App summary

App name: 소피의 아틀리에 ～신비한 책의 연금술사～
App serial: PCSH00231
App version: 1.01

Vita3K summary

Version: v0.1.6
Build number: 3101
Commit hash: https://github.com/vita3k/vita3k/commit/8623b516
CPU backend: Dynarmic
GPU backend: Vulkan

Test environment summary

Tested by: 75Y
OS: Windows 11
CPU: Ryzen 4300u
GPU: AMD Radeon vega 8
RAM: 15 GB

Issues
works well on native resolution, has no graphics bug on.
maybe need some more test but work flawless until tutorial and opening.
Screenshots

Log
Recommended labels

Playable
graphics bug

The english title is Atelier Sophie: The Alchemist of the Mysterious Book.


openchat3.5 training data formatting

Congrats to the authors on the great achievement!
Trying to understand your great work a bit more. In the inference examples, there are prompts like GPT4 Correct User, Code User. What are other conditional prompts used in training? What does Correct mean here? Thanks!

Correct means verified correct answers. Besides, GPT4 and Human were also used, indicating data with unknown correctness.

@imoneoi thank you for prompt response! I have two follow-up questions.

Are only the following prompts are used in training?
a). GPT4 Correct User and GPT4 Correct Assistant
b). GPT4 User and GPT4 Assistant
c). Code User and Code Assistant

If the answer to previous question is Yes, does it mean you only used the GPT-4 data and did not use GPT-3.5 data?

Really appreciate your answers!

@bpucla

Yes, and Human User Human Assistant
Yes. GPT-3.5 data is discarded in the 3.5 version

@bpucla

Yes, and Human User Human Assistant是的，还有 Human User Human Assistant
Yes. GPT-3.5 data is discarded in the 3.5 version是的。 GPT-3.5数据在3.5版本中被丢弃

Can more details be told? Because in that case V3.5 surpassed ChatGPT by using some secret datasets and it became hard for us to know what on earth made the biggest progress...

Are you using 0.1 of weight for the data with unknown correctness and 1.0 of weight for correct one? If not, could you please reveal more details?


Add formal def to hygrosensory neuron and hygrosensory organ [sf#16]

Reported by djs93 on 2013-08-19 17:07 UTC
Following this ontology design pattern, add a Equivalent Class definitions to 'hygrosensitive sensory organ and 'hygrosensitive neuron.
This is dependent on the following GO term request, which also contains a link to a relevant reference: https://sourceforge.net/p/geneontology/ontology-requests/10320/

Updated by djs93 on 2013-08-19 17:08 UTC

Priority: 1 --> 5
Milestone: FB_2013_04 --> FB_2013_06

Updated by mmc46 on 2013-11-08 16:00 UTC

status: open --> closed


Last delivery date - Should not allow date after today

@Sheeri  it's done. here is PR :) https://github.com/WildAid/o-fish-android/pull/482


docs: Change default font to Inter

Is your feature request related to a problem? Please describe.
Default docs font on Firefox/Linux can be better.
Describe the solution you'd like to see
Change the default font to Inter.
Describe alternate solutions
Does not need an alternate solution.
Additional information
N/A

Can you provide a screenshot on how it looks on firefox linux

Can you provide a screenshot on how it looks on firefox linux


Graph Traversal is very slow, suggestions needed

My Environment

ArangoDB Version:        3.4.3
Storage Engine:             RocksDB
Deployment Mode:       Cluster
Deployment Strategy:   ArangoDB Starter
Configuration:               8 Core, 32GB Nodes * 6
Infrastructure:               AWS
Operating System:        AWS AMI Latest
Total RAM in your machine:        32GB
Disks in use:        SSD
Used Package:              SUSE or RedHat .rpm

Component, Query & Data
Affected feature:
AQL Traversal
AQL query (if applicable):
with mio_company
FOR v, e, p IN 1..5 INBOUND 'mio_company/4ae148aa1faca8eb0541532566ba2106' GRAPH 'mio_company_graph'
options {
    uniqueVertices: "global",
    bfs: true
}
filter MIOTECH::MULTIPLY(p.edges[*].holdPct) > 0.02
limit 10000
return p

The UDF: MIOTECH::MULTIPLY is a function to multiply the values in array:
function multiply(num_array) {
	return num_array.reduce((accumulated, current) => accumulated * current);
}

AQL explain (if applicable):
Query String:
 with mio_company
 FOR v, e, p IN 1..5 INBOUND 'mio_company/4ae148aa1faca8eb0541532566ba2106' GRAPH 'mio_company_graph'
 options {
     uniqueVertices: 'global',
     bfs: true
 }
 filter MIOTECH::MULTIPLY(p.edges[*].holdPct) > 0.02
 limit 10000
 return p

Execution plan:
 Id   NodeType          Site  Est.   Comment
  1   SingletonNode     COOR     1   * ROOT
  2   TraversalNode     COOR     1     - FOR v  /* vertex */, p  /* paths */ IN 1..5  /* min..maxPathDepth */ INBOUND 'mio_company/4ae148aa1faca8eb0541532566ba2106' /* startnode */  GRAPH 'mio_company_graph'
  3   CalculationNode   COOR     1       - LET #7 = (MIOTECH::MULTIPLY(p.`edges`[*].`holdPct`)   /* user-defined function */ > 0.02)   /* simple expression */
  4   FilterNode        COOR     1       - FILTER #7
  5   LimitNode         COOR     1       - LIMIT 0, 10000
  6   ReturnNode        COOR     1       - RETURN p

Indexes used:
 By   Type   Collection            Unique   Sparse   Selectivity   Fields      Ranges
  2   edge   mio_company_company   false    false            n/a   [ `_to` ]   base INBOUND

Functions used:
 Name                Deterministic   Cacheable   Uses V8
 MIOTECH::MULTIPLY   false           false       true

Traversals on graphs:
 Id   Depth   Vertex collections   Edge collections      Options                                                Filter conditions
  2   1..5    mio_company          mio_company_company   bfs: true, uniqueVertices: global, uniqueEdges: path

Optimization rules applied:
 Id   RuleName
  1   optimize-traversals

Dataset:
A financial datasets with 71,187,289 nodes and 7,212,539 edge(s)
Size of your Dataset on disk:
20G on SSD * 6 nodes
Replication Factor & Number of Shards (Cluster only):
Replication Factor: 2
Number of Shards: 12
Problem:
The Query is very slow, which requires more than 300 seconds to execute.
Expected result:
Optimize the query

Closing as outdated.


Pull v2022.11.30 PUDL data from AWS

This PR updates the MCOE table to use data from 2021.
pudl.sqlite is now pulled from AWS instead of Zenodo because it's 1) faster 2) easier to specify the version and 3) doesn't download all of the PUDL code and data.
@TrentonBush I'm getting a data validation error on line 251 in dbcp.data_mart.projects. Looks like there are more than 2 locations found for 2 projects. I'm not sure why this changed because this PR doesn't touch the project data. Maybe the pandas update changed the behavior of a function?

I'm a little confused by the asset statement on 251. It's checking to make sure a project doesn't have multiple locations but projects is used to create two location columns for a project.

That assert is checking that there aren't more than 2 locations for any project, because the wide format data creates county_1 and county_2 columns. I can check what project is turning up with more than 2 locations and try to work backwards to figure out why that happened.

I don't know why this only happened with the PUDL update, but there was one project that got split into 4 entries because bad location information -> geocoding failure -> null locations that didn't get dropped. I never got to the root cause, but it's only one project so I just fixed it manually.

Thanks for fixing it! It could have been a pandas 1.4 change that broke something. Should we merge it in?


Upgrade Pillow to 7.2 to address security vulnerability #384

https://github.com/jmathai/elodie/network/alert/requirements.txt/Pillow/open

Coverage increased (+0.3%) to 90.942% when pulling 8a083aca20636abebd8ba0dfc0ee29b01a3940c4 on pillow-384 into 579d94c7af57147be4358d5dfe5c65a06b62f51a on master.

Fixed in #394


Adjust change of directory in circle.yml to be a modifier

Just a small experiment. Docs for it are here https://circleci.com/docs/configuration/#modifiers
Keep it if you like it, reject it if you don't :smile:

💯 thanks


[Launcher]: Support Container Request Timeout Mechanism

Build mapping among Task, ContainerRequest and Container
Add FrameworkEvent and TaskEvent
Support Container Request Timeout Mechanism
Remove AntiaffinityAllocationManager

@sterowang is added to the review. #Closed


POST to _changes timeout when request.body is included

Description
When POSTing to the _changes endpoint with a request.body, and not using one of the builtin filters (doc_ids for example) the connection always times out.
@rnewson has looked into this, and says "the code starts the response, and may even send it, but the web server itself (mochiweb) won't proceed because it still needs to read the request body first. we should send a 400 Bad Request if you send a body when it's unexpected"
Steps to Reproduce
curl 'foo:bar@localhost:15984/db1/_design/foo'
{"_id":"_design/foo","_rev":"1-1cfd4ef84175f09eb3d41bda94062244","filters":{"bar":"function() { return true; }"}}

curl 'foo:bar@localhost:15984/db1/_changes?style=all_docs&since=0&limit=100&filter=foo/bar' -XPOST -Hcontent-type:application/json --data-raw '{"doc_ids": ["foobar"]}'
Expected Behaviour
Return a 400 Bad Request error in preference to waiting for a timeout if a req.body is provided but unexpected (see enhancement request https://github.com/apache/couchdb/issues/3086 which may solve this bug as a side-effect)
Your Environment
Couchdb 3.1

I think it is preferable to solve this by always reading the body rather than changing the existing behaviour to a 400 Bad Request. The existing POST /{db}/_changes endpoint that has been released in the wild for a long time permits other requests (i.e. not just those not using a filter) and any change to that behaviour would break existing code/users.
For example, whilst it is perhaps unusual to POST a {} body it's not completely unexpected given the way the API is described in the docs:

Requests the database changes feed in the same way as GET /{db}/_changes does, but is widely used with ?filter=_doc_ids

Nothing stated about it being valid only for filters.

@ricellis after discussing this with some of the maintainers, it has become apparent neither this, or the related enhancement is considered a high priority item, and is accordingly unlikely to be fixed by the volunteer maintainers in their free time. They did suggest that they'd consider fixing it as a paid gig.
In the meantime, I have found a workaround by having an API endpoint maintain a design document filter for each user. This mechanism allows me to pass what I would have hoped to pass in a req.body as a doc instead. Then I restart the replication with a reference to that updated filter. Replicating that design document to the client allows me to keep all clients associated with that user in sync.

Just want to drop a note here that I've found the root case for this bug and going to open PR with the fix after Holidays.

Fixed in https://github.com/apache/couchdb/pull/3373


Add concept content from Safely Drain a Node to API-initiated Eviction

The content in Safely Drain a Node for the Eviction API can be used to expand the conceptual information in API-initiated Eviction.
Proposed by @alculquicondor.
/assign
/sig node
/sig docs
/language en

/triage accepted

/remove-lifecycle stale

Bad bot
/remove-lifecycle stale

Nooo
/remove-lifecycle stale
On Sat., Feb. 26, 2022, 12:03 Kubernetes Triage Robot, <
@.***> wrote:

The Kubernetes project currently lacks enough contributors to adequately
respond to all issues and PRs.
This bot triages issues and PRs according to the following rules:

After 90d of inactivity, lifecycle/stale is applied
After 30d of inactivity since lifecycle/stale was applied,
lifecycle/rotten is applied
After 30d of inactivity since lifecycle/rotten was applied, the
issue is closed

You can:

Mark this issue or PR as fresh with /remove-lifecycle stale
Mark this issue or PR as rotten with /lifecycle rotten
Close this issue or PR with /close
Offer to help out with Issue Triage
https://www.kubernetes.dev/docs/guide/issue-triage/

Please send feedback to sig-contributor-experience at kubernetes/community
https://github.com/kubernetes/community.
/lifecycle stale
—
Reply to this email directly, view it on GitHub
https://github.com/kubernetes/website/issues/28072#issuecomment-1052319131,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHH4EFVFN77EWAGFMATLWD3U5EBWLANCNFSM45H3FVBA
.
You are receiving this because you were mentioned.Message ID:
@.***>


ContextModule issue

Hello Rob,
First of all, I really like this project and the simplicity of using a library vs an entire framework like quarkus.io.
But I also have a question/issue (not sure if this is the right place but could not find a better one :) ).
I have created a CoffeeApp like application with modules. Now I can run it inside IntelliJ and one module will load the other module with the ContextModule and resolve the dependencies.
But I also tried making a Jar file and then it failed, I get the following error:
Module [<applicationmodule>] has unsatisfied dependencies on modules: [<library module>]. Modules that were loaded ok are:[]. Consider using BootContext.withIgnoreMissingModuleDependencies() or BootContext.withSuppliedBeans(...)
After some searching I found that the issue is that this file is overwritten in the META-INF/services folder:
io.dinject.core.BeanContextFactory
When I add both _di$Factory lines manually to the 's version it will load.
So my question is: Is it possible to create a complete file with the modules in the correct order when building the project? Or am I creating the jar file incorrectly?
Best regards,
Tijs

But I also have a question/issue (not sure if this is the right place b

This is the correct place.  We don't have a discussion forum etc so yes asking questions here is the right thing to do.

issue is that this file is overwritten in the META-INF/services folder:

Hmmm ok, that does not sound correct.

Is it possible to create a complete file with the modules in the correct order when building the project? Or am I creating the jar file incorrectly?

I think we are talking about creating a runnable jar ... and then it not working when it's run.
Are you able to put this project on github somewhere or fork one of the examples so then I could reproduce the problem.
Thanks, Rob.

Hallo Rob,
Sorry for the late reply, I was not home this weekend.
But I cloned the Coffee app and changed as little as possible: https://github.com/Tijs-2/CoffeeApp
First I build it with "mvn package"
Then copy the jar file like this: "copy coffee.app.main\target\Service.jar ." (I am in the root of the project)
Finally I run the application like this: "java -jar Service.jar" then I get the following error:
Caused by: java.lang.IllegalStateException: Module [coffee-main] has unsatisfied dependencies on modules: [coffee-pump] [coffee-heater]. Modules that were loaded ok are:[]. Consider using BootContext.withIgnoreMissingModuleDependencies() or BootContext.withSuppliedBeans(...)
To fix it I changed the "target/classes/META-INF/services/io.dinject.core.BeanContextFactory" file
to this before running the "mvn package" command:
coffee.app.heater._di$Factory coffee.app.pump._di$Factory coffee.app.main._di$Factory
Tijs

Hi Rob,
I was wondering if you had the time to look at this?
And/Or can I do something to improve my example?
Thanks,
Tijs

I haven't got to this yet. I've got busy this last week or so  preparing the next release for Ebean ORM -  https://github.com/ebean-orm/ebean
I should get to look at this on the weekend or maybe before.

No problem was just wondering if I could do something to help
But I also like your ebean library, must admit as far as I know, the best database library out there. Also the reason why I found this one :)

Hi Rob,
I found a solution for the problem, and it is a very simple one.
I have added the following lines to the "maven-shade-plugin" plugin:
`

META-INF/services/io.dinject.core.BeanContextFactory

'
It then combines the files instead of overwriting the files
I do have another question: Is this software usable for production software and are you maintaining it. Or is it a sample project to test of this functionality works? Because I think it is quite good now I am trying it out some more.
Tijs

> maven-shade-plugin
Ok cool. I rarely use the shade plugin these days.
*> Is this software usable for production software *
Yes. I'd say so, I personally have a lot of faith in it because it is
literally so simple and small - there really isn't much to it.
*>  are you maintaining it. *
I do my best. That said it is nice and simple with no more features to add
so to date there it hasn't needed much time.
Cheers, Rob.
On Sat, 28 Mar 2020 at 03:32, Tijs-2<EMAIL_ADDRESS>wrote:

Hi Rob,
I found a solution for the problem, and it is a very simple one.
I have added the following lines to the "maven-shade-plugin" plugin:
`
META-INF/services/io.dinject.core.BeanContextFactory
'
It then combines the files instead of overwriting the files
I do have another question: Is this software usable for production
software and are you maintaining it. Or is it a sample project to test of
this functionality works? Because I think it is quite good now I am trying
it out some more.
Tijs
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dinject/dinject/issues/37#issuecomment-605032955, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AABTATPYF6GD5XFL3N5UJMDRJS2IJANCNFSM4K5NXMAQ
.

I think we can close this issue.

Yes, sorry was completely behind some other work that I forgot to close this issue.
But thank you for your support!
Tijs


Report load/store address and memory size on out-of-bounds memory traps

Feature
When a Wasm program traps due to an out-of-bounds memory access, report the address that the Wasm attempted to access and the memory size, eg:
wasm trap: out of bounds memory access
    memory size (bytes) = 65536
    trap address = 999999

Benefit
This will make debugging such bugs much easier.
Implementation
We can get the faulting address via si_addr which I thiiiink is all we need. Wasm can't access two different memories with the same native address (or else what should have been a trap for one memory might accidentally succeed in accessing a different memory) so guard pages should always be associated with one particular memory. We just have to grab the current instance, iterate over its memories and find which one this guard page is associated with. Once we have that, we do a little arithmetic to translate the native address to a Wasm address.
Hopefully mach ports give us similar info. Completely unsure about windows. This seems like the kind of thing where it isn't the end of the world if we only get this info on some platforms, since it is just a debugging helper.
When we are using explicit bounds checks, it seems like we can just emit code to embed this data into the trap directly.
Alternatives
Build valgrind-esque tooling into Wasmtime? Add a mode to trace all heap accesses before we actually do the access and potentially trap?

trap address = 999999

I think it should be a hex address. That is how pointers are often shown.

We should also be able to report if the access was a read vs. a write.

This is a great idea!
One possible complication is what to do about memories with explicit bounds-checks (and no or minimal guard regions); in such a case, the generated code contains a compare/branch and an explicit trap instruction in the failing case. It should be reasonable to add a register argument to the trap instruction to provide the faulting address (Wasm or native) in a fixed register, though; then we could grab this from the signal machine context in the same way we get PC, etc.

I thiiink with mach ports we want the faultvaddr field of the <arch>_exception_state_t struct, which can be grabbed via thread_get_state.

Oh, we should also report the memory index.
The root cause of the bug that I was debugging and was annoyed with how little info I had, prompting me to open this issue, was that I was accessing the wrong memory.


SSCS-10832 - Provide TCW’s permission to access ‘the process audio/vi…

…deo evidence’ event
Before creating a pull request make sure that:

[ ] commit messages are meaningful and follow good commit message guidelines
[ ] README and other documentation has been updated / added (if needed)
[ ] tests have been updated / new tests has been added (if needed)

Please remove this line and everything above and fill the following sections:
JIRA link (if applicable)
Change description
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ ] No

Versions

it just going to test, can do the version bump after the po review


antidote update no longer works when run non-interactively

as of 1.6.3, antidote will no longer update bundles when running with zsh -c
example:
$ zsh -c 'source $ZDOTDIR/.antidote/antidote.zsh && antidote update'
Updating bundles...
Updating antidote...
antidote version 1.6.3 (679502a)

a bit of a nitpick, but i like using antidote this way to update it easily through non-zsh (bash) scripts

That’s not a nitpick. It seems perfectly reasonable that antidote should work in non-interactive mode. I’ll dig in soon and let you know what I find. Assuming there’s a reasonable fix, I’ll add a test to protect this functionality from breaking going forward. Thanks for the bug report.

TLDR;
I think you customized antidote, and running zsh -c doesn't pick those customizations up. Prior versions of antidote used aggressive globbing (**) that handled both 'friendly names' and the default names. I suspect running zsh -ci 'source $ZDOTDIR/.antidote/antidote.zsh && antidote update' will fix your issue in the short term. Please confirm.
The longer version
I was AFK for most of yesterday, but finally got around to looking into this this AM. At first I was not able to reproduce the issue, which leads me to believe that antidote is working as expected. As I dug deeper, I think I can guess what's going on. I'm going to bet that you have customized antidote's config by:

choosing a new ANTIDOTE_HOME
set the zstyle ':antidote:bundle' use-friendly-names 'yes'

When you run zsh -c, your .zshrc isn't run, meaning that antidote is using its defaults instead of your customizations.  Unfortunately, I don't think you can easily export your zstyles the same way. So, you have a couple options.
The first is to make sure your zsh command configures antidote the same way:
$ export ANTIDOTE_HOME=/path/to/your/new/home
$ zsh -c 'source $ZDOTDIR/.antidote/antidote.zsh; zstyle ":antidote:bundle" use-friendly-names yes; antidote update'

The second option is to run zsh -ci (i for interactive) which will pick up your .zshrc config:
zsh -ci 'source $ZDOTDIR/.antidote/antidote.zsh && antidote update'

You were able to get away with this in prior versions of antidote because I just used aggressive globbing for updates. I switched from that to make sure submodules didn't get updated. I'm debating moving back to aggressive globbing and checking for submodules, but for now zsh -ci is the recommendation. Let me know if this make sense.

both solutions worked! while i didn't have ANTIDOTE_HOME set, i did have use-friendly-names and adding that to a -c command or just using -ci replicated the same behavior as previous versions. i can also see why excessive globbing was removed, and imo that's a great solution and it should probably stay that way.
and just a sidenote: i'm not sure if  zsh -ci would be the best recommendation here since at worse, it could have unintended side effects depending on what is in .zshrc (maybe something like p10k's instant prompt being run, running neofetch, etc) that would detract from just updating the bundles; at the very least though, it would most likely cause slowdowns since it would be sourcing all of your plugins. i found putting the zstyle line in .zshenv (since that will always be sourced) worked the best as you wouldn't need to use zsh -ci or write the line in the zsh -c arguments
thanks for the help!


Gift Entry Payments: handle system errors

Critical Changes
Changes
Issues Closed
Community Ideas Delivered
New Metadata

Custom Labels

commonCriticalError
commonReviewForm
geErrorCardChargedBDIFailed
geErrorRequestTimedout
geErrorUncertainCardChargePart1
geErrorUncertainCardChargePart2
geErrorUncertainCardChargePart3
geErrorUncertainCardChargePart4

Lightning Web Components

gePurchaseCallModalError
utilCustomErrors

Deleted Metadata

**lurch: attach W-039744

Tracking W-039744

@psadie-sfdo and @ptewson-sfdo I added suggestions to the custom labels based on our many discussions today, and as long as those are implemented, I'm signed off on this PR. Thank you for all your help and patience getting to the bottom of these tricky scenarios!

@psadie-sfdo and @ptewson-sfdo I added suggestions to the custom labels based on our many discussions today, and as long as those are implemented, I'm signed off on this PR. Thank you for all your help and patience getting to the bottom of these tricky scenarios!

@melissabarber - proposed custom labels are updated along with the necessary logic update.