Split Message in sample code

If current message size is larger than the maximum size(4M). The message should be abandoned.
If only self-increase nextIndex, there will be "nextIndex=currIndex+1". Later, when subList method is called,  messages[currIndex] will be kept.  As this message can not be sent, maybe the message should be abondoned here.
My question is "does self-increase both nextIndex and currIndex work here ? In this way, the message that has exceed the maximum size will be abondoned。

My question is "does self-increase both nextIndex and currIndex work here ?

It's not working here. If you do this, next() will return a List which does not contain any messages. This situation needs to be avoided as we always want a valid return value when we call next() in an iterator. The operation to abandon an invalid message  should be implemented in the inner iterator


DCT Go SDK for API v<IP_ADDRESS>b1-rc

Context
Autogenerated SDK for the DCT API v<IP_ADDRESS>b1 Release Candidate.

Closing this PR as wont merge.


FIX: Reusing vectors requires reset nulls.

Summary:
Many functions are implemented with the assumption that
not-null do not need to be set, since null are initialized
with all not null.
In HashJoin, a single vector is re-used for the filter results.
This diff reset nulls of the vector before evaluation.
Same thing is changed for vector re-use in string functions.
prepareFlatResultsVector was defined twice, one
definition is kept.
We should look at other places where vector reuse
happens and make sure it's safe if any.
Differential Revision: D32677187

Abandoning PR


import/no-extraneous-dependencies and import/order pretty slow

Performance not great when I useTIMING=1 with eslint, I don't know how these particular rules are implemented but if there's low hanging fruit to make them run quicker it'd be a boon for my project.

This is because the first import rule that needs it has to build up the dependency graph across your entire codebase. I'm open to general performance improvements, but I doubt there's low-hanging fruit.
(I'd suggest using jest-runner-eslint for a large codebase; the parallelization helps a lot)

@ljharb bringing this back - setting aside import/no-extraneous-dependencies, does import/order really need to build up a dependency graph? It seems like it would be able to alphabetize my imports without actually needing that knowledge.

@osdiab it doesn't need the entire graph, but it does need to know what kind of import each thing is, which requires some of the ExportMap to be built up.
There might be a way to only partially traverse in the order rule, and then fill that out later if other rules need the full information, but that sounds like a lot of complexity given that the rules that do require all of it are so critically important.

I just ran into this issue today as well. Our project has the following timings:
Rule                              | Time (ms) | Relative
:---------------------------------|----------:|--------:
import/no-extraneous-dependencies | 15763.183 |    18.2%
import/extensions                 | 11078.163 |    12.8%
import/order                      |  9825.438 |    11.4%
deprecation/deprecation           |  8934.147 |    10.3%
import/no-cycle                   |  7922.312 |     9.2%
import/no-self-import             |  4429.868 |     5.1%
import/no-duplicates              |  4408.646 |     5.1%
import/no-named-as-default        |  3513.506 |     4.1%
import/no-named-as-default-member |  3500.599 |     4.1%
import/named                      |  1678.111 |     1.9%

The total time taken can be extrapolated to be around 86 seconds (time / relative * 100 = total). But here's the thing. I decided to disable the first two, and indeed, roughly the time neccesary for those rules are subtracted:
Rule                              | Time (ms) | Relative
:---------------------------------|----------:|--------:
import/order                      | 12254.752 |    19.6%
deprecation/deprecation           |  9173.969 |    14.6%
import/no-cycle                   |  8027.005 |    12.8%
import/no-duplicates              |  4428.442 |     7.1%
import/no-self-import             |  4406.546 |     7.0%
import/no-named-as-default        |  3487.111 |     5.6%
import/no-named-as-default-member |  3470.090 |     5.5%
import/named                      |  1708.323 |     2.7%
indent                            |  1637.275 |     2.6%
react/jsx-no-bind                 |   771.113 |     1.2%

The total time is now extrapolated as around 62 seconds, which is only a 2 seconds difference from the first set of timings, sans the first two rules. Therefor, I can conclude that, within a margin of error, disabling one or more import rules almost entirely subtracts their time taken from the duration of the lint command.
And if I may, this means the whole discussion about dependency graph being constructed, while certainly not invalid, might be turn out to be only a tiny fraction of the time taken up by the rules that appear to be the slowest. So, not really a factor in why those rules are slow - They are indeed slow, but not because of dependency graph.
But I love to be corrected if anyone has proof to the contrary. That's what science is all about 😀

import/order is really slow

So I'm bringing this back - i have a slightly different proposal. For my project I only really care about three groups - code in the current module, code from another project in my monorepo, and all other external code. I'm thinking this rule could be super fast for me if there were a flag or separate rule altogether, such that I could just decide the import group entirely via text pattern matching. For example:

if it starts with src/ or is a relative path, it's code from my module
if it starts with @mycompany/ then it's part of the monorepo
if it starts with anything else it's external.

this way the rule wouldn't have to do any dependency analysis. Would something like this be amenable to the maintainers here?


Fails with “argument must be inside $GOPATH/src”

Trying to run gobco 0.9.4 on a module I’m developing (outside of $GOPATH), I get:
$ gobco
argument "." must be inside "/home/vasiliy/.cache/gopath/src"

If I set up a $GOPATH specially for gobco, it still doesn’t work:
$ go env GOPATH
/home/vasiliy/tmp/gunison/gopath

$ head -1 /home/vasiliy/tmp/gunison/gopath/src/github.com/vfaronov/gunison/go.mod
module github.com/vfaronov/gunison

$ gobco github.com/vfaronov/gunison
argument "github.com/vfaronov/gunison" must be inside "/home/vasiliy/tmp/gunison/gopath/src"

Hi,
In your last command, the argument github.com/vfaronov/gunison looks misleading to me, as it could be either a relative directory name or a module name. Gobco always interprets it as a directory name, so it may help to just cd /home/vasiliy/tmp/gunison/gopath/src/ before running Gobco. Please try this and tell me whether it worked.
I know that gobco does not support modules. This is just because I never needed support for modules myself. Could you perhaps have a look how to implement them, your advantage is that you have a good test case at hand, which I don't.
I don't expect it to be any difficult. Have a look at the method gobco.rel in main.go, which generates the error you got. The idea of the methods gobco.rel and gobco.prepareTmp is to copy all code that will be instrumented into a temporary directory. If everything goes well, this is the only code you need to touch.
Roland

I got it to work by setting up an actual GOPATH with GO111MODULE=off and all my dependencies (voa go get -t github.com/vfaronov/gunison). Now gobco works if I run it inside gunison or give it the path to that directory.

Could you perhaps have a look how to implement them

Not at the moment, sorry.
Thank you for help and for developing this tool.


Add typeforward for DispatchProxy in netstandard2.1 and add placeholder for 3.1

Fixes https://github.com/dotnet/maintenance-packages/issues/71

@carlossanlop can you double check what happens if you compile a library against netstandard2.0 that exposes DispatchProxy and then reference it in a project targeting netstandard2.1?  I'm not sure a placeholder is going to work here.  Might need a facade for netstandard2.1 and then a placeholder for some version of netX.0.  That's what I was alluding to here: https://github.com/dotnet/maintenance-packages/issues/71#issuecomment-1976998870
Here's a project for you to test out.
dispatchProxy.zip

Per our discussion last week - consider adding a facade to DispatchProxy for netstandard2.1 to fix the version problem then add a placeholder to netcoreapp3.1.

Did you try things out in the scenario we were looking at last week to see if it gives a better experience?

Our beloved APICompat tells us that something is wrong with the netstandard2.1 build.

Our beloved APICompat tells us that something is wrong with the netstandard2.1 build.

Thanks. I fixed it. I confirmed that if I open the nupkg:

The net462 dll contains the DispatchProxy and DispatchProxyGenerator types
The netcoreapp3.1 dll is a placeholder
The netstandard2.0 dll contains the DispatchProxy type
The netstandard2.1 dll has a DispatchProxy typeforward


Add support for Justine's lambda.com interpreter

This PR adds support for pulling in the source code for Justine's lambda.com interpreter, building it, and using it to run tests and the REPL. Its advantages over Blc are that it runs on macOS and supports several debug flags that show the internal state of the Krivine machine as it executes, which comes in handy for exploratory programming with lambda calculus.

Thanks for your pull request! I'm currently asking Justine if it's OK to get all the source files from justine.lol via wget since it may cause lots of traffic. I'll get back to you when we find a way.

Good call!

Got an approval from Justine for getting the sources via wget. Thanks Justine!
I pulled your branch and ran test run-blc-lambda on linux, but examples/lambdacraft.cl ended with a segfault, probably because of the memory limit. The Make recipe tries to specify a larger limit but I suspect that it's not working. Could you check if the test runs for that as well?
Also I just noticed that SBCL on Mac doesn't print a newline at the end of the file which makes diff to fail. That's another issue. We should probably strip the newline at the end for all interpreters.

Thanks Justine!
Unfortunately I can't repro either error locally on macOS 12.1, compiling with Apple clang version 13.1.6 (clang-13<IP_ADDRESS>.3); make test-blc-lambda completes successfully with no segfaults or mismatched diffs. I pushed a change to blc.h to patch out TERMS like for Blc.S, which at least resolves a couple of compile warnings. I should be able to test this branch on NixOS later tonight to see if I can make it segfault there.
With SBCL 2.2.4.nixos from nixpkgs-22.05-darwin (70b6236f3cbc815f2cae8d597418c714cf83d1c8) on macOS, I didn't run into the newline issue. It's certainly possible nixpkgs has patched this issue to make their own tests work, though I couldn't find anything with a quick look at the package derivation.

Thanks for the thorough testing. As for the newline issue, on my Mac I was using SBCL 2.1.4, and on Ubuntu it was SBCL 2.1.11.debian so it seems that the minor version change solves the newline issue. 2.1.4 is from 2021-04-28 according to the release notes so I guess it's better to just update SBCL to solve this issue.
At the moment I tried adding a newline for SBCL outputs and lambda.com worked just fine, so for Mac it seems that everything is cleared.
However on Ubuntu, building lambda.com using the same setup didn't work. Here's what I got on Ubuntu:
if [ -f "test/backquote.cl.in" ]; then \
        ( cat ./bin/lambdalisp.blc | ./bin/asc2bin; cat examples/backquote.cl test/backquote.cl.in ) | ./bin/lambda.com -b > out/bac
kquote.cl.blc-lambda-out.tmp; else \
        ( cat ./bin/lambdalisp.blc | ./bin/asc2bin; cat examples/backquote.cl ) | ./bin/lambda.com -b > out/backquote.cl.blc-lambda-
out.tmp; fi

ERROR:  OUT OF HEAP
   ip:  51431
  end:  95146
 term:  [ [ [[0  0] ⊤] 0] 1] [0 2] [[[ [ [[0  0] ⊤] 0] 2] [0 5]] [[1  [3  [[2 [6 1]] [6 0]]]] 0]]  [[0 put] put]
        0=0
        1=6
        2=0
        3=[0 0]

Setting TERMS to a 10x larger value on Ubuntu made a segfault even for small programs (arithmetic.cl) so I think the term limit is in fact being set properly. I tried using both clang and cc but got the same results.
Here are the clang and cc versions:
$ clang --version
Ubuntu clang version 14.0.0-1ubuntu1
Target: x86_64-pc-linux-gnu
Thread model: posix
InstalledDir: /usr/bin

$ cc --version
cc (Ubuntu 11.2.0-19ubuntu1) 11.2.0
Copyright (C) 2021 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

I also tried getting the pre-built lambda.com from justine.lol but that didn't work either on Ubuntu. My Ubuntu machine has 48GB memory and my Mac has 8GB (it's a rather old machine) so the RAM size itself shouldn't be the issue. I think it's some kind of compilation issue.
Since the main purpose for lambda.com is for running on Macs, I think we could leave it as is at the moment.
Just one last thing before we merge your PR - since the original lambda.com binary runs on multiple OSs, and the one built here is built for Mac only, and the .com is named for multiple-OS binaries (an αcτµαlly pδrταblε εxεcµταblε), it may be better to rename lambda.com to lambda to prevent potential confusion. Could you do a quick patch on the name?

Thanks for your contribution!

Thank you for following up and merging! That's an interesting result for lambda on Ubuntu. I may dig into it a bit more at a later date -- I plan on studying the lambda implementation as a way to expand my knowledge of Krivine machines and lambda calculus as a whole.

 That's exciting. Please feel free to ask any questions if you come across any.
I'm curious about the Ubuntu issue too. I'd be happy if you let me know if you find out something.


add default exports to all packages

We are using Jest to do unit tests on a project that is newly integrating with some code that imports zarrita.
Jest simply can not resolve zarrita imports and we determined that it's because jest can't understand the "export" construct in the package.jsons of the zarrita packages.
We found that just adding a "default" entry suddenly made everything work.
We are unsure if this would break anything else but it seems harmless and allows the module resolution to happen correctly at least in Jest.

Hi there, sorry you are having issues with unit tests, but prior to making upstream changes for a single library I'd like to see if there are alternatives that can suit your choice of using Jest. I assume you are using NodeJS, which version?
The import is a standard field and makes it explicit that zarrita is distributed only as ESM (not commonjs). I believe that adding a default could "trick" node into trying to require zarrita.js, which should not happen because the library is not intended to be consumed that way.
Would you share more context about the project or any relevant code? Some questions I'd like to consider first:

Does your project's package.json use "type": "module"? One issue is that Jest might fallback to node's resolution, and without type module Node resolves to require exports, not import. the type field determines how Node resolves .js files are interpreted. I typically set "type": "module". If you have any scripts that use require, you can change them to .cjs explicitly.

Sometimes changing type: module can break other things if you are relying on commonjs behavior somewhere else. You can "opt-in" to ESM resolution for your tests with NODE_OPTIONS. ESM support in node is now stable, but the flag stil says experimental.

  "scripts": {
    "test": "NODE_OPTIONS=--experimental-vm-modules jest"
  }

A last option is that many package managers allow for rewriting package metadata. For example, pnpm you could apply the patch you have suggested here with a .pnpmfile.cjs.

TL;DR - I'm not totally opposed to these changes, I'd just like to explore options for he

Thanks for the thorough response.
We're using node 20.
To get more specific, we are bringing https://github.com/allen-cell-animated/volume-viewer into https://github.com/simularium/simularium-viewer . The latter package is using Jest.
I'll try to look into all the things you suggested and reply here.  I can confirm that simularium-viewer doesn't use type:"module" yet (but volume-viewer does)...

Well we went for it and got rid of Jest in favor of vitest and everything is working.  I am fine to close this PR.  Thanks again for the info and for the great library.


Add asmERC20 implemetation to fix interaction with bad ERC20 after solc 0.4.22

Fixes no issues in this repo
🚀 Description
Allows smart contracts to interact with bad ERC20 implementations, ie returning void from transfer, transferFrom or approve. Instead of revert introduced methods returns bool.
New methods:

asmTransfer
asmTransferFrom
asmApprove

Info:

[x] 📘 I've reviewed the OpenZeppelin Contributor Guidelines
[ ] ✅ I've added tests where applicable to test my new functionality.
[ ] 📖 I've made sure that my contracts are well-documented.
[ ] 🎨 I've run the JS/Solidity linters and fixed any issues (npm run lint:fix).

I'll do tests/docs/linter after the discussion.

Also we think asmName and asmSymbol could be added to the library to allow handle string and bytes32 return types.

Copying our discussion from gitter:
Anton Bukov
@k06a
12:16
Hi, what do you think of this PR? OpenZeppelin/openzeppelin-solidity#1344
Every contract interacting with arbitrary ERC20 tokens, should use this reliable api.
Leo Arias
@elopio
12:24
After a super quick look, I don't like that it uses assembly and call.
Anton Bukov
@k06a
12:24
There are no another way to avoid revert in solidity after calling transfer which returns void
Leo Arias
@elopio
12:25
A change like this shouldn't be discussed at the ERC level first? If it's important for everybody, everybody should push for a better way to solve it.
Anton Bukov
@k06a
12:26
Since solidity compiler 0.4.22 return size mismatch in function definition and result leads to revert
Leo Arias
@elopio
12:28
Yes, I understand why you have to do it this way. it's just that I always prefer to discuss upstream before making a patch downstream.
Anton Bukov
@k06a
12:28
@elopio agree, will create ERC

Hi @k06a! Thanks for this suggestion.
The purpose of SafeERC20 is to provide helpers that revert when the underlying ERC20 transaction fails. The safety comes from there not being a boolean return value that can be easily ignored. So asmTransfer returning a bool is not a good fit for the library.
That said, I don't think the implementation of asmTransfer is correct. Before the returndatasize opcode was introduced, not returning anything was equivalent to returning false. Some ERC20 implementations could have used this fact to signal failure, yet asmTransfer would be returning true as if the transfer had succeeded. I don't think it's possible to implement such a thing correctly, unfortunately.

Closing in favor of https://github.com/OpenZeppelin/openzeppelin-solidity/pull/1655.


🐛 Mute phone call sound on Android

What were you trying to do?
When I'm recording a video on Android and I receive a phone call the sound of the phone ringing is on the recording.
This was fixed for iOS on this release but the issue continues on Android.
Reproduceable Code
No response
What happened instead?
Instead of continuing the recording with no sound of the phone ringing you can hear the phone notification on the video.
Relevant log output
No response
Device
Samsung S20
VisionCamera Version
2.15.0
Additional information

[ ] I am using Expo
[X] I have read the Troubleshooting Guide
[X] I agree to follow this project's Code of Conduct
[X] I searched for similar issues in this repository and found none.

I'm having the same issue

Hey! I've rewritten the entire Android codebase of VisionCamera from CameraX to Camera2 in the efforts of ✨ VisionCamera V3.
I just now completed the Camera2 rewrite and I believe the core structure is running, but there might be some edge cases to iron out. Can you try and test the PR #1674 for me to see if you can still reproduce this issue here?
Here's an instruction on how you can test that: https://github.com/mrousavy/react-native-vision-camera/pull/1674#issuecomment-1684104217
If the issue cannot be reproduced with that version/PR anymore, then hoorayy, I fixed it! 🎉
Otherwise please let me know and I'll keep this issue open to keep track of it.
Thank you!


Do all the events like NewStateEvent, CallHungUp event are they asynchronous or synchronous when an Event is Fired.

Depends on what you've asked for. You can change the event firing to be Sync or ASync by setting the property:
public bool UseASyncEvents = false;

Of the ManagerConnection class. Default is false!


Add Urdu translation

Adding Urdu support for translation as I believe it would benefit a huge community worldwide. I have experience of translation with few products of Google and Facebook platform itself. I have been a product advocate for Google Map Maker for several years and maintained few online communities during that period. At the moment, it is only myself listed as a maintainer here but I am hoping more people join the effort once I ask the community.

Hi @jabranr that's awesome! Let's keep this up so that other people can find it.

@jabranr you can add me as maintainer. I don't have English to Urdu translation experience but I think I can handle most of the maintainer responsibilities.

I am Professional React Nanodegree graduate from Udacity
Urdu is my first language.

@armujahid Awesome! I have updated the maintainer list to include you.

Here you go! https:github.com/reactjs/ur.reactjs.org/issues/1

Hey @jabranr,
I am the WP expert, Open Source Contributor & translation editor @WordPress for Urdu. I would love to maintain strings for react as well.
Here is my WP.org Profile: https://profiles.wordpress.org/abdullahramzan
Thank you,

@abdullah1908 you can claim any of the core pages mentioned here. Let us know which page you want to translate. We will then update that issue.

Thanks for your response @armujahid  :+1:
You can assign any of the page, would love to suggest Urdu strings :)


pre-commit-em-all

Ran precommit on everything, auto generated a bunch of docs, fixed some formatting.
I removed the k8s-application* from here, they weren't building correctly. @WillBeebe confirm this isn't what customer FOO is using.

Confirmed! They use what's in the provisioner and I'm pulling that into their local repos


php8 compatibility

Description
Currently, this library asks for php7 but seems to work fine as far as I can tell with php8, maybe the composer file/dependencies can be updated to work fine with php8?

It already does:
https://github.com/unicorn-fail/emoji/blob/96be75690f38206a27944d18f1e17cbd863aff2d/composer.json#L51


Audit doesn't highlight on python2

Doing e.g. detect-secrets audit .secrets.baseline won't highlight the relevant secret, which makes it kind of frustrating to use.
I didn't dive into what is causing this, b/c I switched to a python 3 venv, but we should fix it.

Seeing that Python2 will be retired Jan 2020, I'm comfortable closing this issue as "won't do".
After all, audit is used in an adhoc basis, so it doesn't need to be compatible with any legacy systems that might be out there.


Introduction missing for Basic CSS section (Beta site)

Section Name

Beta site Basic CSS
Issue Description

The Basic CSS section needs a landing page with introductory text to start the section. The text was already written in this comment. It needs to be put into a slide challenge at the front of the section.

@dhcodes - as the author of the intro text, would you like to work on this? Figured you should get PR credit 😄

Sure thing, but I'll follow someone else's lead on adding the landing page :)

Sure - I can put a placeholder in for CSS when I drop the HTML section in, or you can copy how it's done for some of the other sections. For example, if you look at the applied-visual-design.json file, you can copy that first challenge into the basic-css.json. It's a challenge type 7 where you just click "Finish" to move on. Then replace the title and text, and pick a few of the digits in the "id" and change them. Unfortunately, to get the paragraph and list formatting, you have to manually add br tags and the ul and li tags, respectively. Let me know!

Indeed!   Bring that shit!!
On Jan 19, 2017 9:02 PM, "Heather K"<EMAIL_ADDRESS>wrote:

Sure - I can put a placeholder in for CSS when I drop the HTML section in,
or you can copy how it's done for some of the other sections. For example,
if you look at the applied-visual-design.json file, you can copy that
first challenge into the basic-css.json. It's a challenge type 7 where
you just click "Finish" to move on. Then replace the title and text, and
pick a few of the digits in the "id" and change them. Unfortunately, to get
the paragraph and list formatting, you have to manually add br tags and
the ul and li tags, respectively. Let me know!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/freeCodeCamp/freeCodeCamp/issues/12626#issuecomment-273965167,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVHZeUMFfUM30zmE0afKdgFvJG6taDGrks5rUCPEgaJpZM4Loyu3
.

@HKuz Ah, didn't realize there were some already in the wild. I can hack that, but it will probably be tomorrow. Thanks for the help.

#TomorrowFace... #WakeStateYou!!!!
On Jan 19, 2017 9:22 PM, "Dylan"<EMAIL_ADDRESS>wrote:

@HKuz https://github.com/HKuz Ah, didn't realize there were some
already in the wild. I can hack that, but it will probably be tomorrow.
Thanks for the help.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/freeCodeCamp/freeCodeCamp/issues/12626#issuecomment-273967736,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVHZeVoqVOFEMwsjSTux5fvnkSko17BKks5rUCiPgaJpZM4Loyu3
.

@dhcodes - no problem. One minor comment on the text - since this section uses the style tag to hold CSS, should we mention that as a method to apply CSS? Maybe add it in to that paragraph right after the bullets that starts "CSS can be applied both as inline styles..."? Not critical, just a thought.

I'm closing this issue because PR #12676 with the intro slide and text was merged.


Feature: rework link usage

Type of feature
🍕 Feature
Current behavior
changes in https://github.com/vercel/next.js/pull/36436 require an opt-in to keep using anchors or switch to the new behaviour
Suggested solution
remove the wrapped <a> tags :pizz
Additional context
No response
Code of Conduct

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

Contributing Docs

[X] I agree to follow this project's Contribution Docs

Would also be great if we make a base Link component in our atoms to be used across the application 😇. This will help keep the link usage consistent.

Fixed on https://github.com/open-sauced/insights/pull/758


Add jupyter-lab-professor

This PR adds jupyter-lab-professor as a project inside the jupyter-helx-notebook repo. I've tested building and pushing the image and everything works. The image is very small currently, around 5.44GB, but we haven't added the extension yet so it will balloon a bit once that happens.

Ok, the very first question I have is, "is this appropriate for projects that that nothing to do with eduhelx?"  because the repo is generic.

This repo just houses all of our jupyter notebook container projects across Helx, regardless of Eduhelx or not. And jupyter-lab-professor is definitely one of those, so I figured it fit here. If we have other eduhelx courses that need unique versions of jupyter lab they could be here too because they would be a "jupyter helx notebook".

Ok got it, I see the subdirectories now.  Ok in that case.  You should probably also fix the other ones and put them in their own directories to avoid confusion maybe.  Or if in the case of the default (as well as the other ones make sure that those dockerfiles don't copy the other directories, I see maybe the beginnings in that.
Up until now, what's the one that we've been using?  This one or another, if another, better reconcile.
I'm not sure it's a good idea to have per-course dockerfiles (why would we need this, but see below).
I suggest, take out the kaniko references,  as it will never be used.
Remember; don't depend on the USER variable, this will almost certainly be overridden with a different number (ignored) in actual deployments -- especially on openshift.  Also, don't depend, be careful about home directories; same reason.
If you must, perhaps there's a section for each applicable project in the readme, but otherwise, I don't like calling them out by name.  But on the other hand, I suppose there can be specializations, in that case yes, maybe those should be pointed out.  For example, no one coming in from the internet will know what an "ordr(d)" is -- yea that's how it's spelled, this was a note in Wed's meeting.


Fix _reference.js files again

It looks like #1046 only fixed the _reference.js files on my local machine. When I synced to master to pick up the change, the files were corrupted again.
This time, I didn't use VS for editing and force converted everything to utf-8, while also adding a .gitattribute to force js files to use text diffs (these corrupted files currently get treated as binary files: http://stackoverflow.com/questions/6855712/why-does-git-treat-this-text-file-as-a-binary-file.)
Closes #1036

:+1:


fix: add semantic release docker

release to docker hub
fixes #55

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


Refactor openvino inference section

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Update capitalization on field names to current

[ ] This is a small change
[ ] This change has been discussed in issue #<?> and the solution has been agreed upon with maintainers.

Description:

Thanks! That should have been updated for a very very long time! :+1


Update MDTimePicker to require a callback function

MDTimePicker now requires a callback function when initialized. The callback function is called when the 'ok' button is pressed, and the time is automatically passed as an argument.

MDSwitch now initializes properly. Before the thumb would not be in the correct position when the widget initialized. This fixes that issue for when the MDSwitch is made with both active: True or active: False.

You should also change the MDTimePicker calls in the Kitchen Sink application.

Good idea, I didn't think about that. Done!

Better then do this:
class MDTimePicker(
    ThemableBehavior, FloatLayout, ModalView, RectangularElevationBehavior
):

    """MDTimePicker.

    :Events:
        `on_get_time`
    """

    def __init__(self, **kwargs):
        super().__init__(**kwargs)
        self.register_event_type("on_get_time")

    def on_get_time(self):
        return self.time

    def close_ok(self):
        self.current_time = self.ids.time_picker.time
        self.time = self.current_time
        self.dispatch("on_enter")
        self.dismiss()```


[FRONTEND][PYTORCH] Support for nn.SiLU added

Although it is simple to write a tvm-friendly and exportable SiLU function in torch, it would be nice to have a built-in tvm support for nn.SiLU. I also added the necessary unit test. Thank you for reviewing.

thanks @alprnbg !


setContents removes ID attribute

Describe the bug
ID attribute disappears in h1 tag when setting contents using setContents function. This bug affects anchors mechanism, which uses id to find a tag to scroll to.
To Reproduce
The string I used to test this behaviour is
TEST TEXT
. It is possible to reproduce this bug using a react component or using core function setContents.
You can also use replit from the screenshot:
<EMAIL_ADDRESS>Expected behavior
Tag renders with filled ID attribute.
Screenshots

Additional context
This element also has a class attribute, which does appear.

Solved using attributesWhitelist


Reversing a queue

Description 📜
Reversing a queue using cpp language.
Fixes #6722

Type of change 📝

[ ] Au#dio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)
[ ] Vi#deo (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[x] Documentation (Content Creation in the form of codes or tutorials)
[ ] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[ ] Android Dev #(Flutter)
[ ] Android Dev #(Java)
[ ] Android Dev #(Kotlin)
[ ] Backend Dev #(Java)
[ ] Backend Dev #(.NET)
[ ] Backend Dev #(PHP)
[ ] Backend Dev #(Python)
[X] C/CPP
[ ] Competitive #Programming
[ ] Cyber #Security
[ ] DS#A
[ ] Data#base
[ ] Datascience with #Python
[ ] Datascience with #R
[ ] Frontend Dev #HTML/CSS/JS
[ ] Frontend Dev #React/Angular/Vue
[ ] Go#lang
[ ] Interview #Prep
[ ] Java_#Domain
[ ] JavaScript_#Domain
[ ] ME#RN
[ ] Machine #Learning
[ ] Open #Source
[ ] Python_#Domain
[ ] Ru#st
[ ] Statis#tics
[ ] UI/#UX

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

@Suvraneel .Please review.

@vasu-1 Please review.

@vasu-1 .Made the suggested changes please review once.

Thank you :)

Made the changes.

@stutimohta20  Excellent ❤  LGTM 🚀  :octocat:


Add vLLM functional test in CI

While working on the addition of vLLM https://github.com/instructlab/instructlab/pull/1442, I tried adding func test to the e2e test since the runner has a CUDA GPU. Unfortunately, it does not have enough GPU memory. This is a Linux VM with a single Tesla T4 GPU.
I ended up with the following error:
ERROR 2024-06-25 12:52:41,612 serving_backend:0: flush [rank0]: Traceback (most recent call last):
[rank0]:   File "/home/runner/work/instructlab/instructlab/venv/bin/ilab", line 8, in <module>
[rank0]:     sys.exit(ilab())
[rank0]:              ^^^^^^
[rank0]:   File "/home/runner/work/instructlab/instructlab/venv/lib/python3.11/site-packages/click/core.py", line 1157, in __call__
[rank0]:     return self.main(*args, **kwargs)
[rank0]:            ^^^^^^^^^^^^^^^^^^^^^^^^^^
[rank0]:   File "/home/runner/work/instructlab/instructlab/venv/lib/python3.11/site-packages/click/core.py", line 1078, in main
[rank0]:     rv = self.invoke(ctx)
[rank0]:          ^^^^^^^^^^^^^^^^
[rank0]:   File "/home/runner/work/instructlab/instructlab/venv/lib/python3.11/site-packages/click/core.py", line 1688, in invoke
[rank0]:     return _process_result(sub_ctx.command.invoke(sub_ctx))
[rank0]:                            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[rank0]:   File "/home/runner/work/instructlab/instructlab/venv/lib/python3.11/site-packages/click/core.py", line 1688, in invoke
[rank0]:     return _process_result(sub_ctx.command.invoke(sub_ctx))
[rank0]:                            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[rank0]:   File "/home/runner/work/instructlab/instructlab/venv/lib/python3.11/site-packages/click/core.py", line 1434, in invoke
[rank0]:     return ctx.invoke(self.callback, **ctx.params)
[rank0]:            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[rank0]:   File "/home/runner/work/instructlab/instructlab/venv/lib/python3.11/site-packages/click/core.py", line 783, in invoke
[rank0]:     return __callback(*args, **kwargs)
[rank0]:            ^^^^^^^^^^^^^^^^^^^^^^^^^^^
[rank0]:   File "/home/runner/work/instructlab/instructlab/venv/lib/python3.11/site-packages/click/decorators.py", line 33, in new_func
[rank0]:     return f(get_current_context(), *args, **kwargs)
[rank0]:            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[rank0]:   File "/home/runner/work/instructlab/instructlab/venv/lib/python3.11/site-packages/instructlab/utils.py", line 635, in wrapper
[rank0]:     return func(*args, **kwargs)
[rank0]:            ^^^^^^^^^^^^^^^^^^^^^
[rank0]:   File "/home/runner/work/instructlab/instructlab/venv/lib/python3.11/site-packages/instructlab/model/serve.py", line 123, in serve
[rank0]:     backend_instance.run()
[rank0]:   File "/home/runner/work/instructlab/instructlab/venv/lib/python3.11/site-packages/instructlab/model/backends/vllm.py", line 58, in run
[rank0]:     server(
[rank0]:   File "/home/runner/work/instructlab/instructlab/venv/lib/python3.11/site-packages/instructlab/model/backends/vllm.py", line 142, in server
[rank0]:     engine = api_server.engine = AsyncLLMEngine.from_engine_args(
[rank0]:                                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[rank0]:   File "/home/runner/work/instructlab/instructlab/venv/lib/python3.11/site-packages/vllm/engine/async_llm_engine.py", line 398, in from_engine_args
[rank0]:     engine = cls(
[rank0]:              ^^^^
[rank0]:   File "/home/runner/work/instructlab/instructlab/venv/lib/python3.11/site-packages/vllm/engine/async_llm_engine.py", line 349, in __init__
[rank0]:     self.engine = self._init_engine(*args, **kwargs)
[rank0]:                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[rank0]:   File "/home/runner/work/instructlab/instructlab/venv/lib/python3.11/site-packages/vllm/engine/async_llm_engine.py", line 473, in _init_engine
[rank0]:     return engine_class(*args, **kwargs)
[rank0]:            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[rank0]:   File "/home/runner/work/instructlab/instructlab/venv/lib/python3.11/site-packages/vllm/engine/llm_engine.py", line 236, in __init__
[rank0]:     self._initialize_kv_caches()
[rank0]:   File "/home/runner/work/instructlab/instructlab/venv/lib/python3.11/site-packages/vllm/engine/llm_engine.py", line 326, in _initialize_kv_caches
[rank0]:     self.model_executor.initialize_cache(num_gpu_blocks, num_cpu_blocks)
[rank0]:   File "/home/runner/work/instructlab/instructlab/venv/lib/python3.11/site-packages/vllm/executor/gpu_executor.py", line 86, in initialize_cache
[rank0]:     self.driver_worker.initialize_cache(num_gpu_blocks, num_cpu_blocks)
[rank0]:   File "/home/runner/work/instructlab/instructlab/venv/lib/python3.11/site-packages/vllm/worker/worker.py", line 195, in initialize_cache
[rank0]:     raise_if_cache_size_invalid(num_gpu_blocks,
[rank0]:   File "/home/runner/work/instructlab/instructlab/venv/lib/python3.11/site-packages/vllm/worker/worker.py", line 378, in raise_if_cache_size_invalid
[rank0]:     raise ValueError("No available memory for the cache blocks. "
[rank0]: ValueError: No available memory for the cache blocks. Try increasing `gpu_memory_utilization` when initializing the engine.

I then tried using different flags like:
dtype="half",
enforce_eager=True,  # TODO: remove me. CI run into issue: see: https://github.com/vllm-project/vllm/issues/2248
tensor_parallel_size=1,  # TODO: remove me. CI run into issue: see: https://github.com/vllm-project/vllm/issues/2248

But that did not help.
Here is the test I added to scripts/basic-workflow-tests.sh:
commit 52f76f821b362aae225f904527bfd90692f3c26b
Author: Sébastien Han<EMAIL_ADDRESS>Date:   Fri Jun 21 21:34:41 2024 +0200

    ci: add func test for vllm

    The e2e runner has all we need to run vLLM. So the e2e test has a small
    vLLM test that validates:

    * the backend starts correctly with `ilab model serve`
    * the client can chat with the server with `ilab model chat`
    * the client can chat with a temp server (when `ilab model serve`is not
      running and only `ilab model chat` was called)

    Signed-off-by: Sébastien Han<EMAIL_ADDRESS>
diff --git a/.github/workflows/e2e.yml b/.github/workflows/e2e.yml
index b7010b0..6db5a5c 100644
--- a/.github/workflows/e2e.yml
+++ b/.github/workflows/e2e.yml
@@ -107,7 +107,7 @@ jobs:

       - name: Install Packages
         run: |
-          sudo apt-get install -y cuda-toolkit git cmake build-essential virtualenv
+          sudo apt-get install -y cuda-toolkit git cmake build-essential virtualenv expect
           nvidia-smi
           ls -l /dev/nvidia*

diff --git a/scripts/basic-workflow-tests.sh b/scripts/basic-workflow-tests.sh
index 107bfec..7141512 100755
--- a/scripts/basic-workflow-tests.sh
+++ b/scripts/basic-workflow-tests.sh
@@ -23,6 +23,28 @@ BOLD='\033[1m'
 NC='\033[0m' # No Color

 SCRIPTDIR=$(dirname "$0")
+PID_SERVE=
+
+trap 'cleanup "$?"' EXIT QUIT INT TERM
+
+cleanup() {
+    set +e
+    if [ "${1:-0}" -ne 0 ]; then
+        printf "\n\n"
+        printf "\e[31m=%.0s\e[0m" {1..80}
+        printf "\n\e[31mERROR OCCURRED\e[0m\n"
+        printf "\e[31mFunction: %s\e[0m\n" "${FUNCNAME[1]}"
+        printf "\e[31mExit Code: %s\e[0m\n" "$1"
+        printf "\e[31m=%.0s\e[0m" {1..80}
+        printf "\n\n"
+    fi
+    if [ -n "$PID_SERVE" ]; then
+        kill "$PID_SERVE"
+        wait "$PID_SERVE"
+    fi
+    cat serve.log
+    set -e
+}

 step() {
     echo -e "$BOLD$* - $(date)$NC"
@@ -191,6 +213,60 @@ test_exec() {
     kill $PID
 }

+test_vllm_backend(){
+    task "Testing VLLM backend"
+    task "Downloading the model"
+    ilab download --repository instructlab/merlinite-7b-lab
+
+    ilab model serve --model-path models/instructlab/merlinite-7b-lab/ &> serve.log &
+    PID_SERVE=$!
+
+    wait_for_server
+
+    expect -c '
+        set timeout 120
+        spawn ilab chat
+        expect ">>>"
+        send "this is a test! what are you? do not exceed ten words in your reply.\r"
+        send "exit\r"
+        expect eof
+    '
+
+    kill $PID_SERVE
+    task "VLLM backend test passed for serve/chat"
+
+    task "Testing VLLM backend for chat only"
+
+    sed -i 's|models/merlinite-7b-lab-Q4_K_M.gguf|models/instructlab/merlinite-7b-lab/g' config.yaml
+    # shellcheck disable=SC2016
+    expect -c '
+        set timeout 20
+        spawn ilab chat --model models/instructlab/merlinite-7b-lab/
+        send "hello!\r"
+        send "exit\r"
+        expect {
+            "Traceback (most recent call last):" { set exp_result 1 }
+            default { set exp_result 0 }
+        }
+        if { $exp_result != 0 } {
+            puts stderr "Error: ilab chat command failed"
+            exit 1
+        }
+    '
+}
+
+wait_for_server(){
+    if ! timeout 120 bash -c '
+        until curl <IP_ADDRESS>:8000|grep "{\"message\":\"Hello from InstructLab! Visit us at https://instructlab.ai\"}"; do
+            echo "waiting for server to start"
+            sleep 1
+        done
+    '; then
+        echo "server did not start"
+        exit 1
+    fi
+}
+
 usage() {
     echo "Usage: $0 [-m] [-h]"
     echo "  -m  Run minimal configuration (run quicker when you have no GPU)"
@@ -228,5 +304,7 @@ while getopts "cmgh" opt; do
     esac
 done

+ilab sysinfo
+test_vllm_backend
 set_defaults
 test_exec
diff --git a/src/instructlab/model/backends/vllm.py b/src/instructlab/model/backends/vllm.py
index f99f198..f61eb6c 100644
--- a/src/instructlab/model/backends/vllm.py
+++ b/src/instructlab/model/backends/vllm.py
@@ -134,6 +134,7 @@ def server(
     engine_args = AsyncEngineArgs(
         model=model_path,
         max_model_len=4096,  # TODO: make this configurable
+        dtype="half",  # TODO: this is hardcoded for the e2e test to pass, remove once we have extra flags
     )

     engine = api_server.engine = AsyncLLMEngine.from_engine_args(

We should revisit this if we ever get a more powerful runner.

https://github.com/instructlab/instructlab/actions/runs/9662539697/job/26652643014?pr=1442

Might we be able to bump the size of the runner?

Might we be able to bump the size of the runner?

It's a special runner, I'm not familiar with the env, @russellb knows more about it.

https://github.com/instructlab/instructlab/pull/1556 introduces the next size up. Once that's in, maybe we could just configure that job to use vllm instead of llama-cpp as its backend?  maybe a new script arg -v that enables vllm ...

@russellb awesome! How is the job triggered at the moment?

once merged, manually only via the github UI - similar to how e2e worked at first before we enabled it to run on PRs automatically

We merged part of this but it’s not turned on in a CI workflow yet

https://github.com/instructlab/instructlab/pull/1801 addresses vLLM installation and enablement in the A10 workflows.


Tutorial commands failing

In RStudio 0.99.887 (OSX 10.12):
> install.packages("treebase")
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0 17 4208k   17  735k    0     0   770k      0  0:00:05 --:--:--  0:00:05  769k 64 4208k   64 2703k    0     0  1383k      0  0:00:03  0:00:01  0:00:02 1383k100 4208k  100 4208k    0     0  1724k      0  0:00:02  0:00:02 --:--:-- 1725k

The downloaded binary packages are in
	/var/folders/f9/b9snshhs35sb95fd60kt8x5w0000gn/T//RtmprnwJR8/downloaded_packages
> library(treebase)
Loading required package: ape
Warning messages:
1: package ‘treebase’ was built under R version 3.2.5
2: package ‘ape’ was built under R version 3.2.5
> both <- search_treebase("Ronquist or Hulesenbeck", by=c("author", "author"))
http://purl.org/phylo/treebase/phylows/study/find?query=dcterms.contributor=Ronquist%20or%20dcterms.contributor=Hulesenbeck&format=rss1&recordSchema=tree
Opening and ending tag mismatch: HR line 1 and body
Opening and ending tag mismatch: HR line 1 and html
Premature end of data in tag body line 1
Premature end of data in tag html line 1
Error: 1: Opening and ending tag mismatch: HR line 1 and body
2: Opening and ending tag mismatch: HR line 1 and html
3: Premature end of data in tag body line 1
4: Premature end of data in tag html line 1

thanks for the report. Treebase search is down right now. In touch with them to try to get it back up.

I've got in touch with someone from Treebase, but it's still down. Sorry for the down time, will update here when it's back up


add domain to the header to share between

I was trying to share cookies between subdomains, and have found the explanation to add this possible feature.

I will not merge that as is, because I do not think consenting to cookies for all subdomains is okay.
Maybe this can be behind an option, but I do not as the default.


brew upgrade aborted, brew now thinks everything is up to date

Please follow the general troubleshooting steps first:

[x] Ran brew update and retried your prior step?
[x] Ran brew doctor, fixed as many issues as possible and retried your prior step?

Bug reports:
What I did:

I ran brew update and then brew upgrade
I cancelled brew upgrade right at the beginning with ^C

What I expected:
After running brew upgrade again, it would resume upgrading (like apt-get or any linux package manager)
What actually happened:
It suddenly acted like all upgrades were already applied. When running brew cleanup, it noted that it wouldn't clean up some packages because they're not upgraded to their latest version yet.
Neither brew update, brew doctor or running brew upgrade on a single package did anything.
Outputs:
brew update
Already up-to-date.

brew upgrade
<no output>

brew doctor
Please note that these warnings are just used to help the Homebrew maintainers
with debugging if you file an issue. If everything you use Homebrew for is
working fine: please don't worry and just ignore them. Thanks!

Warning: You have unlinked kegs in your Cellar
Leaving kegs unlinked can lead to build-trouble and cause brews that depend on
those kegs to fail to run properly once built. Run `brew link` on these:
  sshfs

brew link sshfs
Linking /usr/local/Cellar/sshfs/2.8...
Error: Could not symlink bin/sshfs
Target /usr/local/bin/sshfs
already exists. You may want to remove it:
  rm '/usr/local/bin/sshfs'

To force the link and overwrite all conflicting files:
  brew link --overwrite sshfs

To list all files that would be deleted:
  brew link --overwrite --dry-run sshfs

brew cleanup
Warning: Skipping net-snmp: most recent version 5.7.3 not installed

brew upgrade net-snmp
Error: net-snmp 5.7.3.pre3 already installed

After looking around I found the Locks directory and there were .brewing files for several packages and also an update lock file as per https://github.com/Homebrew/legacy-homebrew/issues/8555
brew apparently seems to ignore those.

To help us debug this issue can you explain:

What you were trying to do (and why)
What happened
What you expected to happen
Step-by-step reproduction instructions (with as minimal input data as possible)

Closing due to no response.


Issue with file upload on android cordova

I have file_id field in my schema which displays an upload box perfectly on web browsers and mobile browsers but the android app doesn't allow for file upload. Nothing happens when the box is clicked.
I am using quickForm
Is there something I'm doing wrong? Is this a known bug? Do you require any further info.

What do you get on the server/phonegap logs? I am currently having an issue with upload on Cordova Android app not uploading too, cant tell if its the same issue without some more info but it might be -> https://github.com/CollectionFS/Meteor-CollectionFS/issues/499


Blueprint files (.bp) should have syntax highlighting, but not hog up the extension

I've associated .bp files as .lua files as they are essentially one large Lua table with a global call around it. The global call is used by blueprints.lua to understand where it should register the blueprint.
When I did this the extension suddenly had to take into account a whole lot more files, even though the blueprint files do not really add to the intellisense part of the extension. I just want to syntax highlight them.

Should be resolved with https://github.com/FAForever/fa/pull/3887


Keeping up dependencies up to date

As tech moves forward, it's a smart idea to introduce renovate or dependabot to keep some dependencies up to date, sometimes even minor versions have important fixes.

I don't think I need it. There are two situations when a dependent library is upgraded:

I don't need to modify the code, I just need to modify the version number. In this case, you can rely on the new version when using it
I need to modify the code to adapt to new destructive changes. In this case, I need to upgrade the version number and modify the code to adapt. In this case, dependabot is meaningless

This is just my current idea. Welcome to further communication


Erro 500

[root@mon01 zabdash]# tail -f /var/log/httpd/error_log
[Wed Nov 21 14:56:18.006178 2018] [:error] [pid 19234] [client <IP_ADDRESS>:55065] script '/usr/share/zabbix/xiaoma.php' not found or unable to stat
[Wed Nov 21 14:56:18.248877 2018] [:error] [pid 19234] [client <IP_ADDRESS>:55065] script '/usr/share/zabbix/xiaomae.php' not found or unable to stat
[Wed Nov 21 14:56:18.491658 2018] [:error] [pid 19234] [client <IP_ADDRESS>:55065] script '/usr/share/zabbix/xiaomar.php' not found or unable to stat
[Wed Nov 21 14:56:18.734768 2018] [:error] [pid 19234] [client <IP_ADDRESS>:55065] script '/usr/share/zabbix/qq.php' not found or unable to stat
[Wed Nov 21 14:56:18.977553 2018] [:error] [pid 19234] [client <IP_ADDRESS>:55065] script '/usr/share/zabbix/data.php' not found or unable to stat
[Wed Nov 21 14:56:19.220786 2018] [:error] [pid 19234] [client <IP_ADDRESS>:55065] script '/usr/share/zabbix/log.php' not found or unable to stat
[Wed Nov 21 14:56:19.463967 2018] [:error] [pid 19234] [client <IP_ADDRESS>:55065] script '/usr/share/zabbix/fack.php' not found or unable to stat
[Wed Nov 21 14:56:19.706824 2018] [:error] [pid 19234] [client <IP_ADDRESS>:55065] script '/usr/share/zabbix/angge.php' not found or unable to stat
[Wed Nov 21 15:34:35.288612 2018] [:error] [pid 19241] [client <IP_ADDRESS>:42750] PHP Fatal error:  Call to undefined function ZabbixApi\posix_getuid() in /usr/share/zabbix/zabdash/lib/ZabbixApiAbstract.class.php on line 498, referer: http://<IP_ADDRESS>/zabdash/
[Wed Nov 21 15:41:18.390952 2018] [:error] [pid 19236] [client <IP_ADDRESS>:43000] PHP Fatal error:  Call to undefined function ZabbixApi\posix_getuid() in /usr/share/zabbix/zabdash/lib/ZabbixApiAbstract.class.php on line 498, referer: http://<IP_ADDRESS>/zabdash/

Hello! I'm using CentOS 7 with Zabbix 4.0
After installing php-process, I no longer have the error. But when I try to login, it goes back to the login screen

Do you make the changes in config.php file?


Add placeholders for values that come from NodePool

bmcAddress, bmcCredentialsName, bootMACAddress, and nodeNetwork.interfaces[*].macAddress are meant to come from the NodePool (hardware) and should not be exposed in the ClusterRequest when hw plugin is fully utilized.
Placeholders for these fields have been added to pass the ClusterInstance dry-run validation since they are required fields. These placeholders will be replaced with data retrieved from NodePool later on.
Before hw plugin is fully utilized, these fields can still be provided through the ClusterRequest.

Placeholders for these fields have been added to pass the ClusterInstance dry-run validation since they are required fields. These placeholders will be replaced with data retrieved from NodePool later on.

Are these placeholders permanent? i.e. will we always need them in order to pass dry-run validation? If so, this should be commented well in the code.

Placeholders for these fields have been added to pass the ClusterInstance dry-run validation since they are required fields. These placeholders will be replaced with data retrieved from NodePool later on.

Are these placeholders permanent? i.e. will we always need them in order to pass dry-run validation? If so, this should be commented well in the code.

Yeah, they are permanent. I was considering to add a comment in this yaml file but it will also ne included in the created configmap which is visible to everyone, but I think that might be fine?

Yeah, they are permanent. I was considering to add a comment in this yaml file but it will also ne included in the created configmap which is visible to everyone, but I think that might be fine?

If we aren't using the created configmap for anything, I think it is best to just remove it. And then it would be good to add comments to the yaml file explaining why these defaults are required.

If we aren't using the created configmap for anything, I think it is best to just remove it. And then it would be good to add comments to the yaml file explaining why these defaults are required.

Sounds good. We are not actually using the configmap. It's created just for information which seems unnecessary. I will remove it and add comments in this yaml file.


docs: fix typo in textMatch example(#101)

What: Fix typo in textMatch example
Checklist:

[X] Documentation
[X] Tests
[X] Ready to be merged
[X] Added myself to contributors table

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Bug]: Reload adds margin

The data grid has a reload bug:
Possible solution: remove every box-component around the data-grid!

Fixed by removing any boxes/divs around the data-grid. Applied margin and max-width to the data-grid directly to solve this strange bug.


GH-14937: [C++] Add rank kernel benchmarks

Closes: #14937

cc @WillAyd

For both rank & sort do we have any kind of UTF8 benchmarks?

For both rank & sort do we have any kind of UTF8 benchmarks?

We don't.

I would be happy to try and add that in a follow up


feat: update route to founders page

Closes: #974
Demo

@shelegdmitriy please mark this as draft so it's not merged before the coorresonding components PR


Don't reverse alpha premultiplication, it's fundamentally wrong

I have looked at your code and that's what I have to say: you are fundamentally wrong in dealing with alpha-premultiplied images. Alpha premultiplication was introduced to eliminate interpolation error on images with alpha channel. If image is not premultiplied you shoud  premultipy it first before interpolation. Your code does opposite. https://en.wikipedia.org/wiki/Alpha_compositing#Description

Thanks for having a look into the source code, any feedback is always appreciated! Regarding alpha premultiplication I'm pretty sure that the current approach is okay, but of course I might be wrong there. Could you explain and give me some details and/or code examples on why you think the current approach is wrong?

 Get the last greenish sample and try to downsample it to 300x300 px with you favorite image editor. Then compare it to image downsampled by your code and mine. You will see difference.

Thanks! And sorry, saw your PR after I've written the response. Will check it out ASAP.

Generally speaking alpha channel in not a color channel. It is a pixel's weight which must be taken into account on all linear operations on pixels (both interpolation and alpha blending can be seen as computation of linear combination). https://en.wikipedia.org/wiki/Linear_combination

But that's only for pre-multiplied alpha, right? Because that's what is done right now: If the image type is using pre-multiplied alpha (e.g. image.RGBA and image.RGBA64) then the RGBA values are transformated to no longer include the alpha-weight in the RGB channels. I.e. I'm reversing the alpha-premultiplication to be able to treat the alpha channel like a color channel afterwards. Otherwise scaling operations on images with translucency will result in slightly off colors (Issue #26 was about that).

Fixed with PR #42. Thanks @e-asphyx!


I can't use Executing channel handlers on background threads

Steps to reproduce
I am try using executing channel handlers on background threads like flutter docs. That use for swift like :
In Swift:
public static func register(with registrar: FlutterPluginRegistrar) {
let taskQueue = registrar.messenger.makeBackgroundTaskQueue()
let channel = FlutterMethodChannel(name: "com.example.foo",
binaryMessenger: registrar.messenger(),
codec: FlutterStandardMethodCodec.sharedInstance,
taskQueue: taskQueue)
let instance = MyPlugin()
registrar.addMethodCallDelegate(instance, channel: channel)
}
I know a issue before that is fixed by gaaclarke form mer https://github.com/flutter/engine/pull/47348
But i don't know how to use it, and what flutter version support it.
Actual results
Swift throw err : "Value of type '() -> any FlutterBinaryMessenger' has no member 'makeBackgroundTaskQueue'"

Logs

Logs

Swift Compiler Error (Xcode): Value of type '() -> any FlutterBinaryMessenger' has no member 'makeBackgroundTaskQueue'
/Users/phamngocthang/working/video_compress_v2/ios/Classes/VideoCompressV2Plugin.swift:14:42

Flutter Doctor output

Doctor output

Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.16.8, on macOS 13.6.3 22G436 darwin-arm64, locale en-VN)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 15.1)
[✓] Chrome - develop for the web
[!] Android Studio (not installed)
[✓] IntelliJ IDEA Community Edition (version 2023.3.2)
[✓] VS Code (version 1.85.2)
[✓] Connected device (3 available)
    ! Error: Browsing on the local area network for TasTech 📱. Ensure the device is unlocked and attached with a cable or associated with the same local area network as this Mac.
      The device must be opted into Developer Mode to connect wirelessly. (code -27)
[✓] Network resources

Hi @thangpn0207
That PR's commit https://github.com/flutter/engine/commit/1e66c0ae7bdaa1ff75b4f3acb75fc9b605413265 is available on beta and master channels. It hasn't landed to stable yet. Please try switching to these channels:
flutter channel master
flutter upgrade

Closing the issue as fixed. If you still see the issue on beta or master channels, please write in the comments.


Add installation instructions

Done. I have changed the URL a bit to allow releases (e.g. latest or a tag) to be selected.


If I want to train one epoch only, how to set lr?

Optimizer:
name: AdamW
beta1: 0.9
beta2: 0.999
epsilon: 1.e-8
weight_decay: 0.05
no_weight_decay_name: norm
one_dim_param_no_weight_decay: True
lr:
name: Cosine
learning_rate: 0.001 # 8gpus 192bs
warmup_epoch: 5

Hi,
can't you just achieve this by setting the lr in the config to your preferred value and removing the warmup_epoch?
Optimizer:
  name: AdamW
  beta1: 0.9
  beta2: 0.999
  epsilon: 1.e-8
  weight_decay: 0.05
  no_weight_decay_name: norm
  one_dim_param_no_weight_decay: True
  name: Cosine
  lr: 0.001 # 8gpus 192bs
  warmup_epoch: 0

if batch size = 4, what is suitable lr?


RHIROS-909 fixed color codes for occurance table

Fixes/RHIROS-909 :boom:
Why do we need this change? :thought_balloon:
the color codes in the breakdown of occurence section of executive report were different from the pie chart. This PR fixes the issue
Jira: https://issues.redhat.com/browse/RHIROS-909
Documentation requires update? :memo:

[ ] Yes
[x] No

Security Checklist :lock:
Upon raising this PR please go through RedHatInsights/secure-coding-checklist
:guardsman: Checklist :dart:

[x] Bugfix
[ ] New Feature
[ ] Refactor
[ ] Unittests Added
[ ] DRY code
[ ] Dependency Added

Additional :mega:

Thanks @r14chandra for the review. Merging this PR!


[SVE] TC-SU-2.5 ApplyUpdateRequest is still sent

Problem

TC-SU-2.5, step 1
Action field is set to "Proceed"
DelayedActionTime is set to 0
DUT still sends ApplyUpdateRequest after transfer of image before rebooting

tc-su-2.5-part1-otap.txt

The attached log shows a single ApplyUpdateRequest received by the provider. This is correct behavior.
The test step states: "Once the update is finished, verify the software version from the Basic Information cluster on the DUT to match the version downloaded for the software update. Verify on the OTA-P/TH that there is no other ApplyUpdateRequest from the DUT."  -- the implication being that we have one instance of ApplyUpdateRequest command as a part of the software update flow but no others. The log shows exactly that.
This is working as expected.

Cert Blocker Review: @pankore we believe this behavior is correct. Please re-open if necessary.


Remove libraries from README

Description (required)
Fixes #1985 Update libraries used in README
What changes did you make and why?

Moved list of libraries to wiki top avoid clutter in README
Updated list of libraries to include ones that were missing and remove ones that weren't being used
Reordered parts of app/build.gradle so libraries used for similar purposes roughly grouped together
Removed unused mediawiki api dependency

Tests performed (required)
Checked all the links are up to date and work in library list
All automated tests pass
Tutorial, login, home, upload media (only on beta), view media, nearby, achievements, notifications, settings, about
Tested 2.8.5-debug-update-libraries-docs~ccf03c78 (beta) on Samsung Galaxy S6 with API level 25.
Tested 2.8.5-debug-update-libraries-docs~ccf03c78 (prod) on Samsung Galaxy S6 with API level 25.
All working

Codecov Report

Merging #1986 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #1986   +/-   ##
======================================
  Coverage    4.36%   4.36%
======================================
  Files         211     211
  Lines       10649   10649
  Branches      954     954
======================================
  Hits          465     465
  Misses      10149   10149
  Partials       35      35

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 72e7439...ccf03c7. Read the comment docs.

@domdomegg The following two changes doesn't seem to be related to #1985. Can you please open a separate PR for the these changes. :)

Reordered parts of app/build.gradle so libraries used for similar purposes roughly grouped together
Removed unused mediawiki api dependency

Opened:

PR #1988
Issues #1989, #1990
Will submit PRs for #1989 and #1990 shortly


updated dependencies,datatable bug fixed

updated dependencies,datatable bug fixed

I wasn't able to build Android, said that it needs API 31, tried few ways but didn't work

I wasn't able to build Android, said that it needs API 31, tried few ways but didn't work

国人？


[CORE] Unpin Custom Elements from 1.0.0

Signed-off-by: Cotton Hou<EMAIL_ADDRESS>

@jeeyun is this PR good to go?


where to find the topics to control clima and doorlock?

Describe the solution you'd like
A clear and concise description of what you want to happen.
I set up the volvo2mqtt on a proxmox-machine and it works perfectly. I receive all the data from my car (btw: XC40 recharge ER MY24). I do not use Homeassistant, just the mqtt feature to get the data.
Publishing the OTP to volvoAAOS2mqtt/otp_code worked. But how can I switch the clima on and off or lock/unlock the car? I can't figure out the right topics. Could you please put them into the description?
Thank you very much for your work.

This isn't supported, but here you go:
Climate state topic:
homeassistant/switch/_climate_status/state
Climate command topic:
homeassistant/switch/_climate_status/command
Climate Payload:
ON/OFF
Lock state topic:
homeassistant/lock/_lock_status/state
Lock command topic:
homeassistant/lock/_lock_status/command
Lock Payload:
LOCK/UNLOCK

Thank you for the fast answer. I got it working now, also found the topics to change the update_interval:
homeassistant/number/(VIN)_update_interval/command
Payload:
300 600 3600 whatever in seconds
and to force an update:
homeassistant/button/(VIN)_update_data/command
Paylod:
PRESS
however with the update_interval I'm not shure if its possible to set higher values than 30min (1800 sec), due to the renewal of the authentication. If its the same loop the setting cant be higher than almost 30 min, if not it wouldn't matter.
If it is the same timer maybe you could make the authentication seperate?
Once more thank you very much.


Fix analyze issues in FirebaseCore

Fix FirebaseCore issues raised in #1735.
Missing explicit nil checks and using an instance variable in a dispatch_once.

Test failure seems to be flakiness coming out of Messaging.

Yep, it was a flake. The tests are green after a rerun.


preemptive support for solution tag header requirement

Description
Provide the new tag header that the solution API is about to start requiring on push. For now, provide this header on all commands that deal with a remote name: push, download, fork, validate. This set may be revised in the future.
Type of Change

[X] Bug Fix
[ ] New Feature
[ ] Breaking Change
[ ] Refactor
[ ] Documentation
[ ] Other (please describe)

Checklist

[X] I have read the contributing guidelines
[ ] Existing issues have been referenced (where applicable)
[X] I have verified this change is not present in other open pull requests
[X] Functionality is documented
[X] All code style checks pass
[ ] New code contribution is covered by automated tests
[X] All new and existing tests pass

Codecov Report

Merging #48 (f400795) into main (240f3b7) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main      #48   +/-   ##
=======================================
  Coverage   34.32%   34.32%
=======================================
  Files          25       25
  Lines        2983     2983
=======================================
  Hits         1024     1024
  Misses       1875     1875
  Partials       84       84

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


width and height of video element are not consistant with the canvas element

After the image is captured from the video element and drawn into canvas the width and height of the canvas should have been the same as the video element according to this. But that is not the case.

you could download the JS file 'webcam-easy.js' and edit it from 'scrollHight' to 'videoHight' and so to 'scrollWidth'.
remember to delete the word 'export default' from the top of the file.
I'm not familiar with javascript, I just tried and so far so good.


Add automated test following waypoints

Add under ExternalTest dir: automated test turtlebot3_tests package
When the test is deployed with CI, an auto-shutdown Sim command should be called after the test is finished.

@tetorea The launch_testing package is here, which should be installed default as part of ros2' launch? If not, it could be done by sudo apt install ros-foxy-launch-testing, I will update README.

seems this script start gazebo.

Right, this test invokes nav2_bringup's tb3_simulation_launch.py, which indeed starts with headless as False by default here. Updated.

yu@system76-pc:~/turtlebot3-UE/ExternalTest/turtlebot3_tests$ ./RunTB3WaypointFollowerTest.sh
mkdir: cannot create directory ‘maps’: File exists
ln: failed to create hard link 'maps/Turtlebot3_benchmark.pgm': File exists
ln: failed to create hard link 'maps/Turtlebot3_benchmark.yaml': File exists
Warning: The parameter 'node_executable' is deprecated, use 'executable' instead
Warning: The parameter 'node_executable' is deprecated, use 'executable' instead
Warning: The parameter 'node_name' is deprecated, use 'name' instead
usage: launch_test [-h] [--package-name PACKAGE_NAME] [-v] [-s]
                   [--junit-xml XMLPATH]
                   launch_test_file
                   [launch_arguments [launch_arguments ...]]
launch_test: error: malformed launch argument 'tb3_model:=', expected format '<name>:=<value>'

Yeap you need to pass it the <tb3_model> as burger or waffle, as updated here 31e1356

When I executed ./RunTB3WaypointFollowerTest.sh burger,  rviz started but nothing happen. anyother step required to execute test?

Yeah that has been observed and indeed does happen occasionally, like even as usual:

When you manually run the tb3 nav2 command, then if Rviz could not subscribe to the published topics (like odom, scan), typically, the steps would be:
-> Restart the Sim.
-> Then only after the map is fully loaded with TB3's odom + scan on, the nav command script should be run.

As for the prerequisite conditions, there should be prior tests that check for Sim state's service server, clock, laser scan, which are essentially required in advance of further functional tests (like nav in this case for Tb3). As such, those tests, actually being added for io-amr-ue, should be also added for tb3-ue here, but rather in another PR.


⚠ Thumbnails API Endpoint has degraded performance

In 338616a, Thumbnails API Endpoint (https://thumbnails.roblox.com/v1/assets?assetIds=82738847&format=Png&isCircular=false&size=30x30) experienced degraded performance:

HTTP code: 200
Response time: 2109 ms

Resolved: Thumbnails API Endpoint performance has improved in 18cdbf6.


Panic with consistent message on TCP failure

To allow downstream crates to sniff out these panics and downgrade them, if appropriate.

Looks good to me, thanks!


TypeError: Cannot read property 'defaultView' of undefined

TypeError: Cannot read property 'defaultView' of undefined
	    at Keyboard.createEventFromKeystroke (/project/node_modules/keysim/dist/keysim.js:176:31)
	    at Keyboard.dispatchEventsForKeystroke (/project/node_modules/keysim/dist/keysim.js:291:31)
	    at Keyboard.dispatchEventsForInput (/project/node_modules/keysim/dist/keysim.js:240:16)

Using Chrome 67.0.3396.62 (in headless mode) on macOS

Thanks for reporting! Can you include some of the code that triggers this? Instructions to reproduce would be very helpful as well.

My mistake. I had sent in a jQuery instance. Thank you for the quick response!


error: Error -3 while decompressing: incorrect header check

it is downloading log files but in Fail folder. Getting the error below within the /Var/Log file. I am not using any compression etc. The downloaded file format is LEEF and now I am stuck.
Qradar and Imperva
020-07-11 10:48:29,734 INFO Saving file XXX_XXXXXX.log locally to the 'fail' folder Error -3 while decompressing: incorrect header check Traceback (most recent call last):
File "LogsDownloader.py", line 222, in handle_file
decrypted_file = self.decrypt_file(result[1], logfile)
File "LogsDownloader.py", line 280, in decrypt_file
uncompressed_and_decrypted_file_content = zlib.decompressobj().decompress(file_log_content)
error: Error -3 while decompressing: incorrect header check

Should have read the manual thoroughly :). This error comes if the compression is set to No. The script expects the data to be compressed.


Update to the new Formatter behaviour

This is now updated and ready to roll with the new benchee Formatter
behaviour.
This doesn't yet output the memory measurements, but I figured I'd
do that in a separate PR after this.

Coverage remained the same at 100.0% when pulling f0ff1716aabc3011d91f32b4a19d19ef577516b9 on update-to-new-benchee into de6a54505ea898cadbfb58c0d4c2c1df31f37911 on master.

Thanks a lot @devonestes ! I wanted to get to these soon myself, benchee should really get a new release soon.
Thanks for taking care of this :green_heart:


[QUESTION] How to create tooltip on a column?

It would be good to know how to get rid of "..." and to show a full string instead.

One possible approach is the following one.
When the user hovers on an item, a tooltip is displayed.
It would be good to know how to configure the tooltip. Thank you in advance for any info!

The following solves the question.
gb.configure_default_column(tooltipField="quotes")
quotes is the name of the column

Can you leaf example code for this?


StageGL antialias invalid. If scale!= 1 or rotation!=0.This bug will appear

StageGL antialias  invalid. If scale!= 1 or rotation!=0.This bug will appear
var stage = new createjs.StageGL(canvas,{"antialias":true});

let canvas = document.getElementById('canvas')   // let stage = new createjs.Stage(canvas);   let stage = new createjs.StageGL(canvas, {antialias:true});   stage.setClearColor('#d1f1ff')   createjs.Ticker.timingMode = createjs.Ticker.RAF   createjs.Ticker.addEventListener("tick", stage);   let shape1,shape2,container   container = new createjs.Container()   stage.addChild(container)   shape1 = new createjs.Shape()   shape2 = new createjs.Shape()   shape1.graphics.beginFill('#52a3fc').drawCircle(0,0,30).endFill()   shape2.graphics.beginFill('#52a3fc').drawCircle(0,0,30).endFill()   shape1.cache(-30,-30,60,60)   shape2.cache(-30,-30,60,60)   shape1.scaleX = 0.8   shape1.scaleY = 0.8   shape1.x = 100   shape1.y = 100   shape2.x = 200   shape2.y = 100   container.addChild(shape1,shape2)

def add(a, b):
    return a + b

result = add(2, 3)
print(result)

I am not seeing an error with that code.  I tried our version of CreateJS, the CDNjs version and CreateJS CDN - here is my test file:  https://zimjs.com/test8/stagegl.html

the rendering of the scaled shape is definitely pixelated and not anti-aliased

Oh - so sorry - misread the post.  I thought you were getting an error code.
Yes - I see what you mean.  It seems to be fixed when the scale is added to the cache (I switched the scale to 1.8 but setting both to .8 is also good):
shape1.cache(-30,-30,60,60,1.8)
shape2.cache(-30,-30,60,60)
shape1.scaleX = 1.8
shape1.scaleY = 1.8

Does anyone know a reason why this would be happening?  I do not really know anything about gpu coding.


How to convert PanNuke dataset to the format of CoNSeP?

Hi @vqdang, Thanks for sharing this amazing code.
I want to train the code with the PanNuke dataset?  Do you have any idea to convert PanNuke dataset to the format as CoNSep? Thanks in advance

Hi,
Please refer to #123 and #125


ac:image macro

This PR is created for the issue #245. Feel free to edit or change commits.

I'd really like to use this!


styled-component should be able to take theme props from the app it being consumed

I am developing a sharable UI component library and want to use component across different app. Component should be able to take theme props from the app in which they will be used.
I have publish the ui component library using npm link and I am able to export the component but it is throwing error.

Component look like this
export const ItalicTitle = styled.span`
font-size: ${({ theme }) => theme.fontSize.LARGE};
font-weight: bold;
color: ${({ theme }) => theme.palette.DARKGREY3};
font-style: italic
`;

Top level app which is consuming above component
import React, { Component, Fragment } from 'react';
import { ThemeProvider } from 'styled-components';
import theme from './theme';
import { ItalicTitle } from 'ui-component';

import './App.css';

class App extends Component {
  render() {
    console.log('Button', Button)
    console.log('ItalicTitle', ItalicTitle)
    return (
      <ThemeProvider theme={theme}>
        <Fragment>
          <ItalicTitle>sdfsaf</ItalicTitle>
        </Fragment>
      </ThemeProvider>
    );
  }
}

export default App;

theme.js
export const getColor = color => ({ theme }) => theme.palette[color];

export default {
  flexboxgrid: {
    gutterWidth: 0,
    outerMargin: 0,
    container: {
      sm: 46,
      md: 61,
      lg: 76,
    },
    breakpoints: {
      xs: 0,
      sm: 48,
      md: 64,
      lg: 75,
    },
  },
  palette: {
    INVALID_ROW: '#f3e9ea',
    TRANSLUCENT: 'rgba(255,255,255,0.3)',
    // Colors based on styleguide
    ACTIVE: '#0079c1',
    ACTIVE_HOVER: '#008bdb',
    HIGHLIGHT: '#f2f8fc',
    POSITIVE: '#8bc34a',
    NEGATIVE: '#ec1a3b',
    WARNING: '#ffc107',
    INFORMATION: '#90CAF9',
    BLACK: '#000',
    DARKGREY1: '#333',
    DARKGREY2: '#666',
    DARKGREY3: '#999',
    LIGHTGREY1: '#ddd',
    LIGHTGREY2: '#eee',
    LIGHTGREY3: '#f5f5f5',
    WHITE: '#FFF',
  },
  border: {
    normal: color => `1px solid ${color}`,
  },
  fontSize: {
    MEDIUM: '1rem',
    LARGE: '1.5rem',
  },
  navigationHeight: '60px',
};

Environment
npx: installed 1 in 2.699s

## System:
 - OS: Linux 4.14 Amazon Linux 2
 - CPU: (2) x64 Intel(R) Xeon(R) CPU E5-2686 v4 @ 2.30GHz
 - Memory: 133.10 MB / 3.85 GB
 - Container: Yes
 - Shell: 4.2.46 - /bin/bash
## Binaries:
 - Node: 8.12.0 - /usr/local/bin/node
 - npm: 6.4.1 - /usr/local/bin/npm
## npmPackages:
 - styled-components: ^4.0.2 => 4.1.1

Actual Behavior
Should take prop from consumer app

If you're working on a component library, check out the docs about best practices with a ThemeProvider for Shared Component Libraries

npm link

It's highly likely this is the problem. If two instances of styled-components are in the bundle due to linking, then the app will also be using two different contexts.
See this for more info: https://www.styled-components.com/docs/faqs#how-can-i-fix-issues-when-using-npm-link-or-yarn-link


WIP: Refactor to add magic link support

Issue #HAV-151
Signed-off-by: Christopher Mundus<EMAIL_ADDRESS>Description
Now adds magic link support to the login options.
Discussion list or pending tasks

[x] Add base magic link support
[ ] Adjust registration funnel to use magic links

How to test
Go to the login page and try to use the magic link option with a current user.
Checklist

[x] My pull request is based on the latest master branch
git pull --rebase origin master.
[x] My code is formatted with elm-format.
[x] All new and existing tests passed.
[ ] For visual changes, I've attached before and after screenshots

The apitest is failing in CircleCI but is not showing the keycloak logs - the error is that keycloak is not listening, so I suspect keycloak crashing on startup during plugin load.
Running locally I see this message in the keycloak server logs:
keycloak_1            | 18:41:15,033 INFO  [org.jboss.as.server] (Thread-2) WFLYSRV0220: Server shutdown has been requested via an OS signal
keycloak_1            | 18:41:15,043 ERROR [org.jboss.msc.service.fail] (ServerService Thread Pool -- 56) MSC000001: Failed to start service jboss.undertow.deployment.default-server.default-host./auth: org.jboss.msc.service.StartException in service jboss.undertow.deployment.default-server.default-host./auth: java.lang.RuntimeException: RESTEASY003325: Failed to construct public org.keycloak.services.resources.KeycloakApplication(javax.servlet.ServletContext,org.jboss.resteasy.core.Dispatcher)

also further down
Caused by: java.lang.RuntimeException: Unable to resolve auth flow binding override for: browser

Superseded by #305 Closing out.


Rename kNumberOfValues to kValueCount

Done as it reads a bit better.

@pifon2a this address https://github.com/googlecartographer/cartographer/pull/1531#discussion_r263442118

@wally-the-cartographer merge

Merge requested by authorized user pifon2a. Merge queue now has a length of 2.


Design issue

How do i put buttons and other components beside filter dropdown? I can not find any way to do it. Any help or suggestion would be helpful.

duplicate of #96

But i want to set button beside this filter component

It is not available at this moment, after consideration we are planning to add such a feature in the next release.

Thanks for understanding my request! : )


wasm-pack test --headless --firefox fails if I update to wasm-bindgen >= 0.2.93

Describe the Bug
I have an existing project with some browser-based tests. I find when I update my wasm-bindgen dependency to anything >= 0.2.93, tests can no longer run.
Steps to Reproduce

I made a test project with a Cargo.lock set to 0.2.92: https://github.com/eric-seppanen/debug-wasm-bindgen-tests . It contains no code, and the unit test does nothing.
I execute wasm-pack test --headless --firefox; and it succeeds.
Update the wasm-bindgen dependency to 0.2.93 (or above).
wasm-pack test --headless --firefox now fails.

Expected Behavior
Tests succeed. Example output with wasm-bindgen 0.2.92:
$ wasm-pack --version
wasm-pack 0.13.1

$ wasm-pack test --headless --firefox
[INFO]: 🎯  Checking for the Wasm target...
   Compiling proc-macro2 v1.0.89
   Compiling unicode-ident v1.0.13
   Compiling wasm-bindgen-shared v0.2.92
   Compiling once_cell v1.20.2
   Compiling bumpalo v3.16.0
   Compiling log v0.4.22
   Compiling wasm-bindgen v0.2.92
   Compiling cfg-if v1.0.0
   Compiling scoped-tls v1.0.1
   Compiling dummy-test v0.1.0 (.../debug-wasm-bindgen-tests)
   Compiling quote v1.0.37
   Compiling syn v2.0.87
   Compiling wasm-bindgen-backend v0.2.92
   Compiling wasm-bindgen-macro-support v0.2.92
   Compiling wasm-bindgen-test-macro v0.3.42
   Compiling wasm-bindgen-macro v0.2.92
   Compiling js-sys v0.3.69
   Compiling console_error_panic_hook v0.1.7
   Compiling wasm-bindgen-futures v0.4.42
   Compiling wasm-bindgen-test v0.3.42
    Finished `dev` profile [unoptimized + debuginfo] target(s) in 4.83s
[INFO]: ⬇  Installing wasm-bindgen...
    Finished `test` profile [unoptimized + debuginfo] target(s) in 0.02s
     Running unittests src/lib.rs (target/wasm32-unknown-unknown/debug/deps/dummy_test-9771a554381dc469.wasm)
no tests to run!
     Running tests/web.rs (target/wasm32-unknown-unknown/debug/deps/web-61c35450dab3534c.wasm)
Set timeout to 20 seconds...
Running headless tests in Firefox on `http://<IP_ADDRESS>:40171/`
Try find `webdriver.json` for configure browser's capabilities:
Not found
running 1 test

test web::empty_test ... ok

test result: ok. 1 passed; 0 failed; 0 ignored; 0 filtered out

Actual Behavior
Tests fail. Output when updating to wasm-bindgen 0.2.93:
$ cargo update -p wasm-bindgen --precise 0.2.93
    Updating crates.io index
    Updating wasm-bindgen v0.2.92 -> v0.2.93
    Updating wasm-bindgen-backend v0.2.92 -> v0.2.93 (latest: v0.2.95)
    Updating wasm-bindgen-macro v0.2.92 -> v0.2.93 (latest: v0.2.95)
    Updating wasm-bindgen-macro-support v0.2.92 -> v0.2.93 (latest: v0.2.95)
    Updating wasm-bindgen-shared v0.2.92 -> v0.2.93 (latest: v0.2.95)
note: pass `--verbose` to see 5 unchanged dependencies behind latest

$ cargo clean
$ wasm-pack test --headless --firefox
[INFO]: 🎯  Checking for the Wasm target...
   Compiling proc-macro2 v1.0.89
   Compiling unicode-ident v1.0.13
   Compiling wasm-bindgen-shared v0.2.93
   Compiling once_cell v1.20.2
   Compiling bumpalo v3.16.0
   Compiling log v0.4.22
   Compiling wasm-bindgen v0.2.93
   Compiling cfg-if v1.0.0
   Compiling scoped-tls v1.0.1
   Compiling dummy-test v0.1.0 (~/debug-wasm-bindgen-tests)
   Compiling quote v1.0.37
   Compiling syn v2.0.87
   Compiling wasm-bindgen-backend v0.2.93
   Compiling wasm-bindgen-test-macro v0.3.42
   Compiling wasm-bindgen-macro-support v0.2.93
   Compiling wasm-bindgen-macro v0.2.93
   Compiling js-sys v0.3.69
   Compiling console_error_panic_hook v0.1.7
   Compiling wasm-bindgen-futures v0.4.42
   Compiling wasm-bindgen-test v0.3.42
    Finished `dev` profile [unoptimized + debuginfo] target(s) in 5.67s
[INFO]: ⬇  Installing wasm-bindgen...
    Finished `test` profile [unoptimized + debuginfo] target(s) in 0.01s
     Running unittests src/lib.rs (target/wasm32-unknown-unknown/debug/deps/dummy_test-5db8d1540a1f48dc.wasm)
no tests to run!
     Running tests/web.rs (target/wasm32-unknown-unknown/debug/deps/web-4b78f4eba41bde34.wasm)
Set timeout to 20 seconds...
Running headless tests in Firefox on `http://<IP_ADDRESS>:43887/`
Try find `webdriver.json` for configure browser's capabilities:
Not found
Failed to detect test as having been run. It might have timed out.
output div contained:
    Loading scripts...

driver status: signal: 9 (SIGKILL)
driver stdout:
<PHONE_NUMBER>580	geckodriver	INFO	Listening on <IP_ADDRESS>:43887
<PHONE_NUMBER>694	mozrunner::runner	INFO	Running command: MOZ_CRASHREPORTER="1" MOZ_CRASHREPORTER_NO_REPORT="1" MOZ_CRASHREPORTER_SHUTDOWN="1" "/snap/firefox/current/firefox.launcher" "--marionette" "-headless" "-no-remote" "-profile" "/tmp/rust_mozprofilem4F9UU"
    console.warn: services.settings: Ignoring preference override of remote settings server
    console.warn: services.settings: Allow by setting MOZ_REMOTE_SETTINGS_DEVTOOLS=1 in the environment
<PHONE_NUMBER>946	Marionette	INFO	Marionette enabled
<PHONE_NUMBER>009	Marionette	INFO	Listening on port 42243
    Read port: 42243
    [GFX1-]: RenderCompositorSWGL failed mapping default framebuffer, no dt
    console.error: ({})
<PHONE_NUMBER>990	Marionette	INFO	Stopped listening on port 42243

driver stderr:
    *** You are running in headless mode.
    Gtk-Message: 12:52:42.717: Not loading module "atk-bridge": The functionality is provided by GTK natively. Please try to not load it.
    JavaScript error: http://<IP_ADDRESS>:37419/run.js, line 9: SyntaxError: The requested module 'http://<IP_ADDRESS>:37419/wasm-bindgen-test' doesn't provide an export named: '__wbgtest_cov_dump'

Error: some tests failed
error: test failed, to rerun pass `--test web`

Caused by:
  process didn't exit successfully: `~/.cache/.wasm-pack/wasm-bindgen-b608dc45898c1197/wasm-bindgen-test-runner ~/debug-wasm-bindgen-tests/target/wasm32-unknown-unknown/debug/deps/web-4b78f4eba41bde34.wasm` (exit status: 1)
note: test exited abnormally; to see the full output pass --nocapture to the harness.
Error: Running Wasm tests with wasm-bindgen-test failed
Caused by: Running Wasm tests with wasm-bindgen-test failed
Caused by: failed to execute `cargo test`: exited with exit status: 1
  full command: cd "~/debug-wasm-bindgen-tests" && CARGO_TARGET_WASM32_UNKNOWN_UNKNOWN_RUNNER="~/.cache/.wasm-pack/wasm-bindgen-b608dc45898c1197/wasm-bindgen-test-runner" GECKODRIVER="/snap/bin/geckodriver" WASM_BINDGEN_TEST_ONLY_WEB="1" "cargo" "test" "--target" "wasm32-unknown-unknown"

Additional Context
Tests were performed on Ubuntu 24.04 with rust 1.82.0. I also tried 1.81.0 and 1.80.0; they fail the same way. wasm-bindgen 0.2.95 also fails the same way. I also tried the wasm-bindgen master branch at f1d60abe8244; it also fails.

I found the source of the issue: wasm-bindgen 0.2.93 .. 0.2.95 are not compatible with wasm-bindgen-test 0.3.42.
If I upgrade both at the same time, the test passes. If I only update wasm-bindgen, it fails with the symptom above.


Rename NLP library to something that reflects its purpose better

Summary
lib.nlp and lib.nlp.nlp.NLP should be renamed to something better as all internal libraries use natural language processing and spacy in some way or another and the main purpose of the NLP library is to select correct modules. The naming should reflect this instead.
Examples
I propose:
lib.nlp -> lib.selector
lib.nlp.nlp.NLP-> lib.selector.selector.ModelSelector
Context
n/a

This is being worked on in the branch 131-task-piping for issue #131 as the NLP module needs to be worked on anyway


update the images folder with recent ones

people eyes are hungry for the recent screenshots :1st_place_medal:

I added them in the 2021-09-15.
I didn't make mobile screens, but if you need a specific usecase, I can make screens for it as well.


Fix building the comparison tool

Oops, one sec, this is not enough.

There.

Now the question is: why did the CI not catch the issue?


Pipework in AWS?

We've been using Pipework to great effect on our LAN, to provision container interfaces that configured addresses via the LAN DHCP server.  Works well, no issues, etc...
Same setup in an AWS VPC, the interface (eth1) is configured via pipework, but never receives a response via DHCP.  I can see the request going out on the VPC network, using tcpdump.
Just wondering if you've tried pipework in AWS successfully.  I did some searching to see if there is something that needs to be done to get a response from the AWS VPC DHCP server (certain MAC range or something), but nothing so far.
EDIT:  My unconfirmed (so  far) suspicion is that AWS uses some sort of network registry to limit the MAC addresses of interfaces it will assign IP's to or allow in the VPC.  Can't find any information in those regards, but I'm trying to test it out.

You are right—AFAIK, AWS VPC cannot have arbitrary MAC addresses. So you have multiple options:

use ENI and move a whole ENI to a container (but that'll only work for a small amount of containers per machine, since you'll dedicate an ENI to the container)
if you only need intercontainer communication, use some kind of overlay network

Sorry!


How can I only change comment color?

I want to make comment color with more bright, what config items do I need to add in .vimrc file?

highlight Comment guifg=#ffffff

I want to make comment color with more bright, what config items do I need to add in .vimrc file?

You can look at :h :highlight (also found here) for information on how to change highlight group colors.
Then, there is also a small explanation on how to have an automatic setup that will only the ayu theme on my maintained fork. Although I only explain how to use my function to modify the colors, you can also use the :highlight command in there.
The most important thing to remember is that this modification needs to be done after you have loaded your colorscheme with colorscheme ayu.


Shared memory between same process types

Currently as i see it is possible that two parallel processes will override each other changes since they will look at same shared memory according this code. I think it is better to use a a process PID since it is unique per given machine.
@huyanping what are your thoughts on this one? Also can you elaborate why you are using file there?

All cache classes implements the CacheInterface. I thought that the cache object shouldn't care about how different processes use it to add, get or update the data. The SharedMemoryCache is just like RedisCache, so you can modify the value in different processes when you use any of the cache classes. If you want to keep different processes' cache in different namespace, you can extend the cache class to do it.
I am not sure which file you are talking about. I guess it is about this code, this file is used to make a IPC key which is required for initializing the shared memory resouce

Ok, thanks for clarifying this one


Update third-party_client_libraries.md

Update third-party_client_libraries.md with Elixir libs
I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Could you please add also to docs/en?

Done


Release 0.1.17

Could a new crate be released that includes #46?
It's currently blocking an upstream rust-native-tls PR to add similar support: https://github.com/sfackler/rust-native-tls/pull/56

Any update on this?

Just published 0.2.0


Feature/add user to workspace#24

I pulled develop, then I updated my branch to be inline with the changes on develop, then I created this PR. Build failed during the unit tests stage. Did I miss something?

There is a problem z with .Net version in the pipeline. It's not your fault. Don't worry.


Does this support .NET 5.0? Application instantly closes on executing a command without exception or error

Hi, I've been trying to use this library, I've duplicated the code in the Windowless Sample in my .NET 5.0 project, and although a tray icon is shown, and a menu can be accessed, the application will close without exception or error when attempting to execute any of the commands.
My project with is available at https://github.com/JordanAnthonyKing/KeyboardUsurper/tree/NotifyIconIssue

Yes, 1.1.0 works fine with .NET 5.0 and your test project also works for me (after I fix it to work without a configuration)
I see a disable, if I click it, I see an enable in the menu.

Thanks for the response, would you be able to share the fix you made?

@JordanAnthonyKing Hello, you need to set ShutdownMode="OnExplicitShutdown" in App.xaml

That behavior occurs because your application doesn't have any window on startup and default value is OnLastWindowClose. No windows creating during application start. Only popups on mouse over, when they are vanished, last "window" (popup) closed and your application will closing.


API Key does not revoke/delete from the API explorer

Tried out the swagger as soon as I could with a quick little test key. Unfortunately, revoking a key does not seem to work (it remains listed) and still looks to remain working against the API.

The delete button has a confirm step - did you click it twice?

Actually, I am seeing that in prod now. Thanks for the bug report

I think this is still there ? I created a key - used it today, posted on a public forum for some help, then promptly deleted it and generated a new one - but the new one looks like the old one still (in fact is the old one ?)
i.e. my key is out in the wild at the moment
Craig

Correct - I haven't fixed the issue yet

It's now fixed.


Autoclose HTML replace the key ':' by '>'

Having this package enabled each time I do type colon ':' it is replaced by gather than '>'
Environment: Linux mint 17.3 (Ubuntu 14)
Keyboard: Spanish

Same issue in hungarian keymap.

Please use the "Legacy/International Mode" option see #128


PSR-4 support

Since PSR-0 is marked as deprecated, support for PSR-4 which involves namespaces and per-file classes would be nice.

that would definitely break existing implementations. would need to refactor for a 3.x version

indeed, this would justify a 3.x release. Do you got time for this? I might be able to help :+1:

not right now, but will eventually. the classes need to be split into files, then namespaced accordingly, if you can help, would be great :)

yeah, seems feasible. Am I correct that no tests are in place yet?
On 25 June 2015 at 10:32, Paulo Cesar<EMAIL_ADDRESS>wrote:

not right now, but will eventually. the classes need to be split into
files, then namespaced accordingly, if you can help, would be great :)
—
Reply to this email directly or view it on GitHub
https://github.com/pocesar/phery/issues/17#issuecomment-115162009.

there aren't, because it's really hard to 'cross test' between PHP and the browser in an automated way (tried and failed)


feat(tornado.py): add ignored endpoints

http_filters moved outside of wrappers, and some tests were fixed as well.

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


security problem

We found a potential security vulnerability in one of your dependencies.
The yajl-ruby dependency defined in Gemfile.lock has a known high severity security vulnerability in version range < 1.3.1 and should be updated.
Review vulnerable dependency
Only users who have been granted access to vulnerability alerts for this repository can see this message.
Learn more about vulnerability alerts

https://github.com/brianmario/yajl-ruby/issues/176


ensure roks_url is correct

post backup and restore, this ensures ROKS_URL in platform-auth-idp configmap is updated with right url incase post backup and restore ROKS_URL holds the old cluster url
Ref: https://github.ibm.com/IBMPrivateCloud/roadmap/issues/62929

Test scenario:
I have tested this usecase by keeping ROKS_URL to different one and patched ibm-iam-operator image which has the fix.
noticed that ibm-iam-operator is able to update the ROKS_URL with proper one


Layouter RefCell

@duguorong009 @4rmanc I make a lot of progress in debugging and I found several places where the native and the halo2 version differs, but I want to delay this a little bit and focus on other tasks


Add ((.../...)) autolink shortcuts for elements/attributes.

This patch adds the fairly lame double-parens shortcut for elements and
element attributes. That is ((element)) will produce a link to the
element named 'element', and ((element/attribute)) will produce a link
to the attribute 'attribute' for the element 'element'.
Closes #323.

I stripped out some of the complexity around the !! syntax, as I don't think we need it for attributes and elements. WDYT, @tabatkins?

Ping. Crazy? Not crazy?
Would <{.../...}> be better or worse?

Sorry for delay - I don't look at email over the weekend.
The code looks fine to me at a glance.  I'm fine with whatever shorthand you like more, just decide on one and I'll merge.
And yes, no need for the !! stuff - this shorthand is fully typed already from the syntax, so there's never a need to specify the type explicitly.

Honestly I kind of like <{.../...}> if that counts for anything. I am pretty unlikely to be using this syntax though so don't take me too seriously :).

@mikewest Do you prefer the <{foo}> syntax as well?  If so, I'll pull these edits in and switch it over.

Updated the patch with the <{.../...}> syntax. WDYT?