CASM-2670: Refactor to use patched CSM images

CASM-2670: Refactor to use patched CSM images

Job lint-test-scan / scan-images (artifactory.algol60.net/csm-docker/stable/cray-sts:0.7.0) fails because that image does not yet exist until this PR is merged and cray-sts-0.7.0 is released. However, here are the Snyk results from the Jenkins CI image build:
$ snyk container test artifactory.algol60.net/csm-docker/unstable/cray-sts:0.7.0-20211130080124_c68225b

Testing artifactory.algol60.net/csm-docker/unstable/cray-sts:0.7.0-20211130080124_c68225b...

Organization:      shasta-csm-oss
Package manager:   apk
Project name:      docker-image|artifactory.algol60.net/csm-docker/unstable/cray-sts
Docker image:      artifactory.algol60.net/csm-docker/unstable/cray-sts:0.7.0-20211130080124_c68225b
Platform:          linux/amd64
Base image:        alpine:3.15.0
Licenses:          enabled

✔ Tested 27 dependencies for known issues, no vulnerable paths found.

According to our scan, you are currently using the most secure version of the selected base image


After typing in one location, a new on is tricky

There's some kind of bug with the selection.

I don't think this is a problem anymore.


Support csv tables in the table

Showcase csv tables and the correct syntax

Thanks for suggesting this. This is now available in all places:

The Antora editor contains tracks.csv.
The Kata has been enhanced.
The AsciiDoc plugin for IntelliJ docs contain a section about CSV tables.


Fix exception when settings.width is a number

The API on http://www.fixedheadertable.com/ shows settings.width can be a number. I have confirmed that settings.height can be a number. However, the code assumes that settings.width is a string and does not work when it is a number. This fixes that issue.

Thanks for this!


0049-group-anagrams.py's solution returns a dict_values object when a List is expected if using Python3

Title. Solution is correct for Python.

return value should actually be return list(result.values())

Solution has been resolved


[REST] OpenAI Rest API compatible function calling

This PR adds rest API function calling feature. As per the open AI documentation, the recipe for function api calling is as follows:

The basic sequence of steps for function calling is as follows:

Call the model with the user query and a set of functions defined in the functions parameter.
The model can choose to call one or more functions; if so, the content will be a stringified JSON object adhering to your custom schema (note: the model may hallucinate parameters).
Parse the string into JSON in your code, and call your function with the provided arguments if they exist.
Call the model again by appending the function response as a new message, and let the model summarize the results back to the user.

Example:
# test_function_calling
tools = [
    {
        "type": "function",
        "function": {
            "name": "get_current_weather",
            "description": "Get the current weather in a given location",
            "parameters": {
                "type": "object",
                "properties": {
                    "location": {
                        "type": "string",
                        "description": "The city and state, e.g. San Francisco, CA",
                    },
                    "unit": {"type": "string", "enum": ["celsius", "fahrenheit"]},
                },
                "required": ["location"],
            },
        },
    }
]

payload = {
    "model": "gorilla-openfunctions-v1-q4f16_1",
    "messages": [
        {
            "role": "user",
            "content": "What is the current weather in Pittsburgh and tokyo?",
        },
    ],
    "stream": False,
    "temperature": 0.7,
    "tools": tools,
}
r = requests.post("http://<IP_ADDRESS>:8000/v1/chat/completions", json=payload)
print(f"Without streaming:\n{r.json()['choices'][0]['message']['tool_calls']}\n")

Expected output:
[
  {
    "id": "call_y5dfxRxkPvedCUmpfYB7Ur",
    "type": "function",
    "function": {
      "name": "get_current_weather",
      "arguments": {
        "location": "Pittsburgh,PA"
      }
    }
  },
  {
    "id": "call_VFdQdLHUWnw5MewjNFmzzh",
    "type": "function",
    "function": {
      "name": "get_current_weather",
      "arguments": {
        "location": "Tokyo,JP"
      }
    }
  }
]

References:

OpenAI API reference
Cookbook examples
#1341

Comments:
The model gorilla-openfunctions-v1-q4f16_1 is a bit unstable. For example, changing order of functions leads to different outcomes. Also currently, the function calling assumes that all the api parameters are names parameters which the model does not always respect.

Besides, I noted that the gorilla model uses USER: /ASSISTANT:  as the conversation template https://github.com/ShishirPatil/gorilla/tree/fcceefe5c5c8beb25ee28f80331d26d8946ad294/openfunctions#running-openfunctions-locally. We need to make sure to use the correct template when compiling this model and uploading the pre-built gorilla models to HF.

@Kartik14 This is really great work, thank you for your work on that!

Besides, I noted that the gorilla model uses USER: /ASSISTANT:  as the conversation template https://github.com/ShishirPatil/gorilla/tree/fcceefe5c5c8beb25ee28f80331d26d8946ad294/openfunctions#running-openfunctions-locally. We need to make sure to use the correct template when compiling this model and uploading the pre-built gorilla models to HF.

If I want to compile a model locally, how can I also make sure that I am using the right template? Is there a way that I can specify or edit the template before I compile a model?


Toggle change mode button.

There must be a toggle change mode button to change UI of website according to user . @skmirajulislam  Please assign me this under gssoc-extended , hactober fest and with levels, so that a could solve this issue.

@Devanshu1603 ASSIGNED!

Hi @skmirajulislam ,
Can you please assign this issue to me? I'd like to work on it.
Thanks!

@Devanshu1603 is already assigned!


Implement Arbitrary::size_hint for hand-implemented types

What problem does your feature solve?
The Arbitrary trait provides a size_hint method for determining how many bytes to provide. This is mostly implemented by the derive macro, but some of our Arbitrary implementations are written by hand, and do not implement this method.
Fuzzing is more effective with this implemented.
What would you like to see?
Implement.
What alternatives are there?
No implement.

Also needs to be done for soroban-env-common

I've posted a pr to soroban-env. The 1 impl in the sdk doesn't actually need a size hint - the default impl is appropriate.


Adding venv and yarn-error.log to gitignore

venv: in the wiki, the installation is described with creating the venv into "venv", but only ".venv" is in gitignore
yarn-error.log: file is created if yarn fails, it should not be commited to the repo accidentally
This is a preparation for some more pull requests of me soon.

Thanks for contributing.
If PR seems stuck, it is because we lack some time to better test them before merging. The simple ones could be added quickly but larger ones will not be merged that fast.
Anyhow, all help to make PDA become better is really appreciated.


cranelift: Optimize br_table's with 0 or 1 targets into other types of jumps

👋 Hey,
Feature
I had this idea when looking into #5508.
We can optimize a br_table with 0 targets in the JumpTable into a single jump into the default block.
We can also optimize br_table's with 1 target into a brz + jump which should still provide code size benefits.
Benefit
Generated code size improvements. Even a small br_table is huge in terms of code size.
Here's an example for the x86 backend.

`br_table` with 1 target
clif:
function %br_table_with_1_targets(i32) {
    jt0 = jump_table [block2]

block0(v0: i32):
    br_table v0, block1, jt0

block1:
    return

block2:
    return
}

Assembly:
   0:   55                      push    rbp
   1:   48 89 e5                mov     rbp, rsp
   4:   83 ff 01                cmp     edi, 1
   7:   0f 83 20 00 00 00       jae     0x2d
   d:   8b d7                   mov     edx, edi
   f:   b9 00 00 00 00          mov     ecx, 0
  14:   48 0f 43 d1             cmovae  rdx, rcx
  18:   48 8d 0d 0a 00 00 00    lea     rcx, [rip + 0xa]
  1f:   48 63 54 91 00          movsxd  rdx, dword ptr [rcx + rdx*4]
  24:   48 01 d1                add     rcx, rdx
  27:   ff e1                   jmp     rcx
  29:   09 00                   or      dword ptr [rax], eax
  2b:   00 00                   add     byte ptr [rax], al
  2d:   48 89 ec                mov     rsp, rbp
  30:   5d                      pop     rbp
  31:   c3                      ret
  32:   48 89 ec                mov     rsp, rbp
  35:   5d                      pop     rbp
  36:   c3                      ret

Equivalent `brz`+`jump`
clif:
function %brz(i32) {
block0(v0: i32):
    brz v0, block1
    jump block2

block1:
    return

block2:
    return
}

Assembly:
Disassembly of 22 bytes:
   0:   55                      push    rbp
   1:   48 89 e5                mov     rbp, rsp
   4:   85 ff                   test    edi, edi
   6:   0f 85 05 00 00 00       jne     0x11
   c:   48 89 ec                mov     rsp, rbp
   f:   5d                      pop     rbp
  10:   c3                      ret
  11:   48 89 ec                mov     rsp, rbp
  14:   5d                      pop     rbp
  15:   c3                      ret

Here are the savings (in bytes) for the various backends:

x86: 33 bytes
aarch64: 28 bytes
s390x: 30 bytes
riscv: 32 bytes

Implementation
Can we write this as an egraph rule? Otherwise we probably need to use a custom pass.
Alternatives
We can always not do this optimization, I'm not sure how common these two kinds of br_tables are. We can also perform these optimizations in each backend, but that seems to be duplicating some effort.

For cg_clif br_table with less than 3 targets doesn't happen. It uses cranelift_frontend::Switch, which uses brz/brnz in case of two or less targets (excluding the default target)

Right, we have the Switch interface that does a bunch of those optimizations. I've also disassembled a few of the sightglass benchmarks and none of them have a br_table with fewer than 3 targets, so its probably not worth pursuing this optimization.


Spread Operator Fails On Inline Function Calls

The following code:

generates an error at compile time.
It compiles fine using babel, and

 compiles just fine.
As another (less minimal) example:

compiles but then produces
// Generated by CoffeeScript 1.9.3
(function() {
  React.createElement("div", {
    "label": t('audience'),
    "bindState": true,
    "State": true,
    "name": true,
    "me": true,
    "_updateName": true,
    "e": true
  });

}).call(this);

which makes no sense.

Even more puzzlingly:

 (here) does not compile.
but

 (here) compiles fine. The difference is the space between the parameters.


[Feature Request] Comments on highlighted text

Hi team, love this project!
I have a feature request to add comments to highlighted section of text - much like what's possible with Confluence.
Example:
https://github.com/giscus/giscus/assets/12656294/3664955a-c581-4585-a49c-4b8119c8767f

I am experimenting with using Giscus advanced configuration to manually create a new github discussion for comments on highlighted text (or individual blocks).
The annoying thing is that a floating comment section requires custom styling which is a little tricky in this case. I'm looking at ways I can achieve that


When bootstrapping teuthology, avoid deadlocks

Use Popen.communicate() instead of Popen.wait()
Signed-off-by: Zack Cerza<EMAIL_ADDRESS>

ooooh, do you think it was overrunning the pipe buffer?

@dmick check this out: https://docs.python.org/2/library/subprocess.html#subprocess.Popen.wait

@dmick it was certainly a possibility here

So far so good.  Last night and yesterday, it was to the point where I'd kill the hung pip process, ~100 jobs would start then pip would get hung up again.


[release-1.8] Add ROOT env var

This is an automated cherry-pick of #32083

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio_release-1.8
db3c80a36700ccc331ee2d434bdb5f4ac742a463
link
/test unit-tests_istio_release-1.8

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

/retest


Send token10 and token20 via verifyLookup method

Hello.
As you know, in addition to %token, %token2 and %token3 there is %token10 and %token20 which is suitable for sending strings containing space charachter such as user's name and company's title.
In this situations, it's not possible to use just %token or %token2 or %token3.
In KavenegarApi class -as you can see below-  it's possible to send %token10 & %token20 as 6th and 7th arguments and it works properly:
class KavenegarApi
{

    public function VerifyLookup($receptor, $token, $token2, $token3, $template, $type = null)
    {
        $path   = $this->get_path("lookup", "verify");
        $params = array(
            "receptor" => $receptor,
            "token" => $token,
            "token2" => $token2,
            "token3" => $token3,
            "template" => $template,
            "type" => $type
        );
        if(func_num_args()>5){
            $arg_list = func_get_args();
            if(isset($arg_list[6]))
                $params["token10"]=$arg_list[6];
            if(isset($arg_list[7]))
                $params["token20"]=$arg_list[7];
        }
        return $this->execute($path, $params);
    }
}

I'm wondering how it's possible to do so using this package and notificaiton methods:
class InvoicePaid extends KavenegarBaseNotification
{

    public function toKavenegar($notifiable)
    {
        return (new KavenegarMessage())
            ->verifyLookup('verify_first',['token1','token2']);
    }
}

class KavenegarMessage
{

    public function verifyLookup($method, $tokens)
    {
        $this->method = $method;
        $this->tokens = is_array($tokens) ? $tokens : [$tokens];
        return $this;
    }

}

Any news...?! 🙄

hi,
Is there a solution for this issue?

Solved


add help string to configure so it can be included in docs

@WRansohoff if you think the help string should be different let me know and I'll update it


Possible Unnecessary Object

As far as I can tell, this object just stores a string and a related double value. Could we not just replace it entirely with a dictionary everywhere we use it?
https://github.com/HydrologicEngineeringCenter/HEC-FDA/blob/0c0e82868f7b7f9aa0a6e7aeb857b48f4f6dcad0/ViewModel/WaterSurfaceElevation/PathAndProbability.cs#L4

Probably. I like having the custom object. I think it makes it clearer what the thing is. It is a path and a probability. Dictionaries are can be a bit more confusing because you have to know the keys. I vote we leave this one alone.

understood, I'll defer to your judgement on that.

Sounds like an opportunity to refactor the way we’re storing results from the compute. Duly noted.

On Feb 25, 2022, at 8:55 AM, Brennan Beam @.***> wrote:

understood, I'll defer to your judgement on that.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.


Multimodal model editing

I am very interested in your multimodal editing work, but when using the code, I found that in the COCO caption task, the image indexes involved in caption_train_edit.json and caption_eval_edit.josn are not in the image download link you provided. How can I solve it?

Hi there,
Thank you for bringing this to our attention, and I appreciate your interest in our work!
To make it easier for you and other users to find the necessary datasets, we have placed the links in a more prominent position in our repository. Now, you can access the images download link by scrolling down to the "Dataset for Multimodal" subsection
Should you encounter any further issues or have additional questions, please feel free to reach out.

Thank you very much for your reply. But maybe because my previous statement was unclear, the question I want to ask seems to have not been resolved yet. So I will describe my problem in more detail: I have downloaded the image dataset (the download link you provided), but some of the image indexes contained in caption_train_edit.json and caption_eval_edit.josn (such as COCO_val2014_000000386164.jpg) which are no such pictures in the link you provided. How can I solve it?

Hi,
Thank you for your patience and for providing additional details on the issue. And we apologize for any inconvenience this has caused.
We have now updated the dataset to ensure all the referenced image indexes, including COCO_val2014_000000386164.jpg, are available. Please download the updated image dataset using the same link, which should now include the missing images.
If you encounter any more missing elements or have further questions, do not hesitate to reach out.
Thank you for your understanding and continued support of our work.

Hi buddy, have you solved your issue?

Thank you very much for your reply, my problem has been solved.


Support for signing multiple release files

Following up this PR: https://github.com/r0adkll/sign-android-release/pull/14

Should fix #13.

@r0adkll could you please give us a heads up on when you believe you can review this?

@victorpaleologue Sorry for the delay on this, it LGTM. :shipit:


upgrade dwn-sdk-js to v0.4.6

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

Project coverage is 95.08%. Comparing base (e674187) to head (2c4f799).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #163   +/-   ##
=======================================
  Coverage   95.08%   95.08%
=======================================
  Files          28       28
  Lines        2951     2951
  Branches      395      395
=======================================
  Hits         2806     2806
  Misses        144      144
  Partials        1        1

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


wrong constructor is being picked up

In below code I am trying to construct Target object. Target class has 3 constructors but only one is preffered (the one annotated). This results in compilation error because wire[Target] picks up wrong constructor. I've intentionally provided to much eligible values here in order to show the naughty behaviour:
  trait A
  trait B
  trait C

  class Target(a: A) {

    def this(b: B) = this(new A{})

    @javax.inject.Inject
    def this(c: C) = this(new A{})
  }

  val a: A = new A {}
  val a1: A = new A {}
  val b: B = new B {}
  val b1: B = new B {}
  val c = new C {}
  val sa = wire[Target]

Error:(24, 16) Found multiple values of type [com.softwaremill.macwire.macwireakka.Demo.A]: [List(a, a1)]
val sa = wire[Target]
Error:(24, 16) ambiguous reference to overloaded definition,
both constructor Target in class Target of type (c: com.softwaremill.macwire.macwireakka.Demo.C)com.softwaremill.macwire.macwireakka.Demo.Target
and  constructor Target in class Target of type (b: com.softwaremill.macwire.macwireakka.Demo.B)com.softwaremill.macwire.macwireakka.Demo.Target
match argument types (Null) and expected result type com.softwaremill.macwire.macwireakka.Demo.Target
val sa = wire[Target]

However if you provide type after sa:
  val sa: Target = wire[Target]

it works fine.

Maybe I'm wrong, but is not your problem listed in limitations section of the Readme (https://github.com/adamw/macwire#limitations)?

Right. Due to macro limiations we are not able to implement nicer behaviour. Anyway, we can keep this issue open in order to address it in future. Maybe when scala meta comes to play it solves the problem.

I have another issue that looks like this
class ConfigModule(config: com.typesafe.config.Config) {
	@javax.inject.Inject
	def this(configuration: play.api.Configuration) = this(configuration.underlying) // underlying : com.typesafe.config.Config
}
lazy val playConfig: play.api.Configuration
lazy val configModule: ConfigModule = wire[ConfigModule]

gives
[error] Cannot find a value of type: [com.typesafe.config.Config]
[error]   lazy val configModule: ConfigModule = wire[ConfigModule]

The original issue appears to be already solved in MacWire 2.3.5 and Scala 2.12.11.


Fix stuck report

Issue Number
DL-5493
Issue Description
The submissions report stopped running on October 5. After investigating it locally on production data, there were some null variables that caused the report generation to fail.
Add the following to your docker-compose.override.yml:
report-generation:
  ports:
    - 9999:80

and execute this curl call to run the report:
curl --location 'localhost:9999/reports' \
--header 'Content-Type: application/vnd.api+json' \
--data '{
  "data": {
    "attributes": {
      "reportName": "inzendingen-per-bestuurseenheid"
   }
  }
}'

Inspect the logs (docker compose logs -ft report-generation), and you will see an error about properties being accessed by an undefined variable.
Solution Description

Some eredienst besuurseenheden did not contain a besluit:Bestuurseenheid type, which caused the generateVanBestuurseenheidMetaQuery(uris) to return some empty results. Matching for ere:BestuurVanDeEredienst was added to the query.

Null-checks were added inside the report's execute block to avoid null accesses.

How to Test
Execute the curl command above on this branch and observe the logs. There should be no errors, and a new CSV file should be created under data/files/.

I find this PR a bit difficult too review. (I know it sounds silly coming from me, because I'm terrible at creating clean commits and PRs. :zipper_mouth_face: I am clearly no authority to give advise, but here goes anyway:)
I encourage working on code quality and making it look good, but I would suggest keeping formatting in separate commits from actual code changes. That way we can follow a commit history where we can ignore huge amounts of changes that are just formatting and quickly focus on the real meat.

Hi,
Thanks!
The first analysis of the reason why the report got stuck was correct. However, that's only the first step. It's good to try to go a couple of steps deeper and ask why these types are missing. And here, you'd have seen this is a problem with the consumer-producer of deltas from OP (=app-organizations) and a well-known bug where too many types get deleted in certain conditions.
So, the proposed solution is not really a solution to the problem. The final solution would be to fix the consumer.
So, key here is, not the fact that a known bug was missed, but rather the fact that troubleshooting should go as far as reasonably possible. In that context, Toyota's five whys might be a good guide for troubleshooting. I would not apply it to the letter, but the principle of going a couple of whys deeper is valuable. (And if you get stuck on a why, which here would be probably on the second one, ask around!)
Anyway, fixing the final problem will take a bit of time, or rather, time should be made for that, so we're left with intermediate solutions. And here, I would not go for the proposed solution, but rather apply what's already known: i.e., re-propagate these types, see https://app.gitbook.com/o/-MP9Yduzf5xu7wIebqPG/s/-M_5cpNQ4inMjiC5hhyJ/code/incident-handling and https://binnenland.atlassian.net/browse/OP-2869 for context.
The benefits compared to the proposed solution:

the code remains unchanged, cause now it adds extra complexity
extra awareness to raise a sense of urgency
avoid silent fail

the report would still work with the bug and the proposed solution, BUT it's actually a good thing it fails. Because the type besluit:Bestuurseenheid is critical for our apps. Users wouldn't be able to log in anymore. So consider the failure of the report as our canary in the coal mine (for now).

In fact, regarding the latter point, while testing the solution after data was fixed, we discovered other problems with the data that required other fixes.
With regard to @benjay10  remark, I agree to keep formatting commits as much as possible out of the PR, but sometimes it just happens.

By the way, I really think it's a very valuable PR actually, considering things we all learn from it:

Approaches to troubleshooting
The value of the report
Some perspectives on how apps are interconnected and the importance of data quality.


_cmsLinkProfiles is declared twice in src/cmscnvrt.c and src/lcms2_internal.h

Subject says it all. The declaration in lcms2_internal.h seems to be a copy/paste from the function definition in src/cmscnvrt.c; the declaration in src/cmscnvrt.c has slightly different names.
This is not problem, but it just seems strange - is there a reason for this replication?

fixed in 2.10 rc1


Set MaxMetaspaceSize to 512m to fix Dokka Gradle build step

Without this configuration, the :core:dokkaJavadoc Gradle build step was failing/hanging with error:
Exception in thread "DefaultDispatcher-worker-7" java.lang.OutOfMemoryError: Metaspace
The Daemon will expire after the build after running out of JVM Metaspace.
The project memory settings are likely not configured or are configured to an insufficient value.
The daemon will restart for the next build, which may increase subsequent build times.
These settings can be adjusted by setting 'org.gradle.jvmargs' in 'gradle.properties'.
The currently configured max heap space is '512 MiB' and the configured max metaspace is '256 MiB'.
For more information on how to set these values, visit the user guide at https://docs.gradle.org/8.0/userguide/build_environment.html#configuring_jvm_memory
To disable this warning, set 'org.gradle.daemon.performance.disable-logging=true'.
Daemon will be stopped at the end of the build after running out of JVM Metaspace

For whatever reason the build worked on master but not current dev branch.
Specifically I was running $ ./gradlew :core:publishToMavenLocal

Thanks, updated.

Thanks for all the feedback on these pull requests, and for the library!


Flutter use image.asset  flicker

When I click on a container to switch pictures(Image widget), the first time I switch pictures, I have a flicker problem. Is this flutter's bug ? I use Stack and Offstage widget solved my problem

can you provide flutter doctor -v and flutter run -v and simple reproduction code?

Container(
constraints: BoxConstraints(maxWidth: 52),
child: Column(
mainAxisAlignment: MainAxisAlignment.center,
children: [
              //  the code below may lead to a flicker problem when call setState func
              currentIndex == index ? Image.asset('images/base_info_nor.png',
          package: StringUtils.packageName) :Image.asset('images/base_info_sel.png',
          package: StringUtils.packageName),
            //  the code above may lead to a flicker problem when call setState func

              Padding(padding: EdgeInsets.only(top: 4)),
              Text(
                item.text,
                overflow: TextOverflow.ellipsis,
                maxLines: 2,
                textAlign: TextAlign.center,
                style: TextStyle(
                    color: currentIndex == index
                        ? hui.colorNeutral2
                        : hui.colorNeutral4,
                    fontSize: hui.infoSizeSupport),
              )
            ],
          ),
        ),

Hi @mlf2020
What we need is a minimal reproducible code sample that demonstrates this issue. You should be able to simply paste into a new project main.dart and (after adding a few random image assets) run it. Please also provide your flutter doctor -v logs.
Thank you

Related #63218

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


"You have to specify endpoint name" error when more than one endpoint is defined

Running
graphql get-schema dev

with this config works:
{
  "schemaPath": "schema.graphql",
  "extensions": {
    "endpoints": {
      "dev": "http://localhost:3080/api/graphql"
    }
  }
}

but with this config (generated by graphql init):
{
  "schemaPath": "schema.graphql",
  "extensions": {
    "endpoints": {
      "dev": {
        "url": "http://localhost:3080/api/graphql"
      },
      "prod": {
        "url": "https://myserver.com/api/graphql"
      }
    }
  }
}

it just errors with
You have to specify endpoint name or define GRAPHQL_CONFIG_ENDPOINT_NAME enviroment variable

Wrong repo
https://github.com/graphcool/graphql-cli/issues/12


How to store realm_access.roles in a local variable?

The console.log(this.oauthService.loadUserProfile());prints these following values:
ZoneAwarePromise {__zone_symbol__state: null, __zone_symbol__value: Array(0)}
...
family_name: "Bob"
given_name: "Bob"
iat:<PHONE_NUMBER>
iss: "http://localhost:8080/auth/realms/myRealm"
preferred_username: "bob"
realm_access:
  roles: (3) ["offline_access", "admin", "uma_authorization"]
  prot: Object
...

How can is store the realm_access.roles array in a local variable?

Ok, easier than expected.
const userProfile = this.oauthService.loadUserProfile();

userProfile.then(value => {
    const roles: Array<string> = value.realm_access.roles;
    console.log('roles: ', roles);
});

Thanks for sharing your solution? Can we close this issue then?


Support for [cdv][ai]t, fixes http://youtrack.jetbrains.com/issue/VIM-264

Adds "t" alias for angle brackets.

Anyone has status on this PR ? I can't wait for this feature!  >:)

The bug with solo tags has been eradicated. Testers appreciated.

I've accepted a different implementation #100. Thanks for the pull request anyway!


Deprecation warning... And BrowserSync

Not an error, just it seems thing are moving forward faster than I think! I got this with last FoundationPress
DEPRECATION WARNING: Passing a string to call() is deprecated and will be illegal in Sass 4.0. Use call(get-function("spin")) instead.
Also, it's been 3 month since I worked with Foundation. I see that in the new version, Browsersync is back to the code you have to add in the footer. The last version I use was opening auto a Browser window.
Just for my information, why choose a version or another. (I prefer the actual version where you have to add a small code, you just have to not forgot to remove it before production!).

Forget about BrowserSync, I never made the link between the fact that when you enter your local dev url in the config.yml, BrowserSync launch your browser... I'm tired today...

@dantahoua what command did you run that showed the deprecation warning? I'm not seeing it on the master branch from npm i or npm start.

I use nvm... So for this one I'm using node v8.11.2 (npm v6.1.0)
As usual I use npm start... It's just a notice... But I thought it was interesting to share it just in case.
Funny because I have another project I installed one month before and I have no Deprecated notice (same npm/node version but older project, one month older... :) )

@dantahoua I cannot reproduce on that version of node. I was running npm 6.2.0 but I'm not sure if that makes a difference. If you rm -rf node_modules and reinstall does it still show up after running npm start?

Nope, it's only on one project... Last one I started one week ago do not have this warning...

Thanks for the quick response. I'll close this one out then since at worst it's a deprecation warning which has no effect on whether or not the gulp task works.


2 improvements for mobile devices

Devices (e.g. mobile phone) without a mouse fire mouseover events on user tap, but never mouseout events. When a context menu is re-opened, the last used item still appears as selected. This PR fixes this, by calling _onItemMouseOut() when an item is selected on devices without pointer support.

On small screens, when the context menu is wider than half the map viewport, and the menu is opened on a point near the center in the left side of the viewport, currently:

When there is no fixed menu width defined, the menu is shrunk so it fits:

When menu has fixed width defined, a part is hidden on the left side of viewport.

This PR fixes this, by pushing the menu on the right so it just fits in the viewport:

Remove a useless _setPosition(pt) call. Is executed unconditionally 3 lines earlier.

Code cosmetic: Consistently use tabs for indentation.

Nice work François!


PC Plot: context menu harmonization

[x] add Filter menu and group entries:
Show Filters
Show Filtered Out Lines
[x] add Columns... entry that will open Select column dialog from where Columns and Log Columns can be selected
[x] add new menu Selection/Active Row with entries:
Show Current Line (Point)
Show Mouse Over Line (Point)
Show Mouse Over Row Group

DG 1.15.1

If a menu in the context-menu opens a dialog, you typically show "..." after the name. For "Columns" the three dots are missing.

DG 1.15.3


Request for UNIX time & request fix date format in Core parts

I've noticed that on of the cores has a wrong date format:
When running this endpoint: GET https://api.spacexdata.com/v2/parts/cores?status=active the original launch date-format differs.
Also, could a *_date_unix time-format be added to other date related models as well, including the cores?

Looks like an issue with the date updating utility, which is adding milliseconds onto the usual UTC timestamps. Looking at fixes

Since the default date in javascript includes milliseconds, I'll be using this utc date format for everything now (2018-06-29T09:42:00.000Z). The formatting should be consistent now.
Working on adding unix timestamps to cores and capsules

Unix times added to cores and capsules 👍

Thanks Jake! :) 👍


Move config repo creation section to deploy guide

I would expect this to be the first page of the deploy guide before „Seed“.
https://osism.github.io/docs/guides/configuration-guide/configuration-repository#creating-a-new-configuration-repository

Makes sense!
Maybe we could add a hyperlink to the „create config repo section“ somewhere here as it’s somehow still part part of the initial deployment? For those who missed the getting started page…


[ACR] Update ACR SDK version

@djyou please rebase this onto master and the CI should pass.

@tjprescott There are still errors which don't seem to be caused by this PR.

@djyou you are right. Now there seem to be issues with tests no longer being accepted. I'm investigating.

@djyou an update was released for msrestazure that no longer works with several test recordings. The CI pulls that version and that's why it is failing. I will rerecord the affected tests and let you know when you can rebase your changes.


Alert notification for Stock Price

Hi ,
I am trying to automate a process where I can set a price limit for a stock and will get a notification on phone if that price is met or below. I am not sure how to get it done.
Hopefully someone will guide me in right direction.
Thanks,
Manoj

This isn't the right place to ask this kind of question. The Forum or Discord is better suited to this kind of question.

Thanks for guiding, I use the other forum.


Subdural hemorrhage

Add new term:
subdural hemorrhage (a child of intracranial hemorrhage MP:0001915)
DEF: bleeding into the subdural space, the potential space between the dura and arachnoid mater of the meninges around the brain
SYN: subdural haemorrhage
SYN: subdural hematoma
SYN: subdural haematoma
SOURCE: https://radiopaedia.org/articles/subdural-haemorrhage, HP:0100309
Annotations in: J:207884, also COVID-19 related

Added new term:
subdural hemorrhage MP:0031117


deleted the nowrap for interface method sigs

with nowrap the interfaces method sigs would expand at libitum

Thanks @paolino!


Use friendly_name instead of id as the caller id

The friendly_name attribute should have the person's name, whereas id can be random id.
https://github.com/TECH7Fox/asterisk-hass-addons/blob/d6b270dc76071ba494eaba443e9d11579ea4ba64/asterisk/rootfs/etc/cont-init.d/asterisk.sh#L148

PR is welcome, @OnFreund.

There you go: #322

#322


Documentation for likely dev tasks for site search autocomplete

We are looking to the roll out of autocomplete for GOV.UK site search. This will result in the search widget on every page of GOV.UK providing autocomplete suggestions.
This documentation provides 2 guides:

How to follow a relatively manual process on how to update the denylist of terms that the GOV.UK autocomplete is not allowed to suggest (swear words, MP names, test data)
How to disable the autocomplete altogether

This information is added to the dev docs as there are concerns that there could be an urgent push to update the denylist if it is showing something undesirable and this could occur out of hours. The task to disable the autocomplete is provided as a quick means to be able to turn the feature off altogether should a problem occur and to allow time to fix it in office hours.

Thanks @leenagupte - comments are resolved


Mouse move doesn't work, can't enable native events

Hello,
I am trying to move the mouse but it's not working:
05:17:07.548 INFO [14] org.openqa.selenium.remote.server.DriverServlet - Executing: [get: https://www.facebook.com])
05:17:10.658 INFO [14] org.openqa.selenium.remote.server.DriverServlet - Done: [get: https://www.facebook.com]
05:17:10.678 INFO [14] org.openqa.selenium.remote.server.DriverServlet - Executing: [find element: By.id: loginbutton])
05:17:13.250 INFO [14] org.openqa.selenium.remote.server.DriverServlet - Done: [find element: By.id: loginbutton]
05:17:13.270 INFO [14] org.openqa.selenium.remote.server.DriverServlet - Executing: [mousemove: 0 false])
05:17:13.503 INFO [14] org.openqa.selenium.remote.server.DriverServlet - Done: [mousemove: 0 false]
05:17:13.516 INFO [14] org.openqa.selenium.remote.server.DriverServlet - Executing: [click: nothing])
05:17:13.957 INFO [14] org.openqa.selenium.remote.server.DriverServlet - Done: [click: nothing]
I even tried to enable native events:
setPreference('setEnableNativeEvents',true);

$capabilities = array(WebDriverCapabilityType::BROWSER_NAME => 'firefox',
					  FirefoxDriver::PROFILE , $profile);
$driver = RemoteWebDriver::create($host, $capabilities,50000);

$driver->get("https://www.facebook.com");
$login_button = $driver->findElement(WebDriverBy::id('loginbutton'));
$driver->getMouse()->mouseMove($login_button->getCoordinates());
$driver->getMouse()->click();

//$driver->quit();
?>
Please give a hint on how to make the mouse move.
In about:config:
setEnableNativeEvents is true
but
webdriver_enable_native_events is false, and stays false even if I setPreference('webdriver_enable_native_events',true);
Thank you in advance.

I'm encountering the same issue. I was unable to find a solution so far :(

Got it to work by disabling the native events (since they are not supported on the latest browser versions).


chromium: 86.0.4240.183 -> 86.0.4240.193

https://chromereleases.googleblog.com/2020/11/stable-channel-update-for-desktop_9.html
This update includes 1 security fix (no CVE).

Motivation for this change
cc @danielfullmer @thefloweringash
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

There seems to be a failing dependency (doxygen) for aarch64. I’ll look into it tomorrow, but we shouldn’t let that hold up this security fix.

Build and briefly tested on x86_64-linux.

Backported to 20.03 in abe53c3191b5951772bc3564f45e5da10003ac02 (forgot to reference this PR :o).

Was packported in 3af6255b10f94901c7024341a501c6e59f0bf681.

Was packported in 3af6255b10f94901c7024341a501c6e59f0bf681.

@erictapen thanks! Guess this means I should remove the "9.needs: port to stable" in the future, right? (I thought it wouldn't matter if the PR's closed but in hindsight it seems like a good idea to remove the label so that it is clear that the update got backported.)
Or tbh I'll probably just stop using the label for future Chromium updates (or assign "8.has: port to stable" right away so that I don't have to remember to update it) as security updates need to be backported anyway. Would be nice if we had a bot for backports... :)

@erictapen thanks! Guess this means I should remove the "9.needs: port to stable" in the future, right? (I thought it wouldn't matter if the PR's closed but in hindsight it seems like a good idea to remove the label so that it is clear that the update got backported.)
Or tbh I'll probably just stop using the label for future Chromium updates (or assign "8.has: port to stable" right away so that I don't have to remember to update it) as security updates need to be backported anyway. Would be nice if we had a bot for backports... :)

Mh yeah I think it'd make sense to use 9.needs: port to stable and 8.has: port to stable as a method to communicate the state of a PR regarding its backports. They way I read them is that you only should add has port to stable if the commits actually landed in staging-20.09 or release-20.09. This way we can easily detect if a backport was forgotten.
But I'm actually not absolutely sure how it is commonly understood. I opened https://github.com/NixOS/nixpkgs/pull/109263 to document the semantics as I think would make sense.
:+1: for better backport automation.

Mh yeah I think it'd make sense to use 9.needs: port to stable and 8.has: port to stable as a method to communicate the state of a PR regarding its backports. They way I read them is that you only should add has port to stable if the commits actually landed in staging-20.09 or release-20.09. This way we can easily detect if a backport was forgotten.
But I'm actually not absolutely sure how it is commonly understood. I opened https://github.com/NixOS/nixpkgs/pull/109263 to document the semantics as I think would make sense.
:+1: for better backport automation.


Revert "zstd 1.5.2 => 1.5.4"

Reverts vespa-engine/vespa#26349
Opensource build failed with
18:54:57   error: Failed build dependencies:
18:54:57   	vespa-libzstd-devel >= 1.5.4-1 is needed by vespa-8.134.39-1.el8.x86_64

Ah there is more ...


Hot Module Reloading Fixes + More storybooks for consignments

Spec:

Current:

I wanted to get the pure components changed, and figured it can't hurt to keep all the form work in. WRT the pixel perfectness, I know it's not right yet, I need to make some kind of form abstraction next, so I'm ok for this to not look 100% 👌
#trivial

WRT HMR: see #555 for context

Fails

:no_entry_sign:

Missing Test Files:

src/lib/components/storybooks/__tests__/section_browser-tests.tsx
src/lib/components/storybooks/__tests__/story_browser-tests.tsx
src/lib/components/consignments/components/__tests__/text_input-tests.tsx

If these files are supposed to not exist, please update your PR body to include "Skip New Tests".

Warnings

:warning:

Could not find stories using these components:

ConsignmentTODO

StorybookBrowser

  Generated by :no_entry_sign: dangerJS


[5.3] Add seeView() and dontSeeView() test methods

Will assert that a view has been loaded
$this->seeView('home.widgets.trending');

Will assert that a view has not been loaded
$this->dontSeeView('home.widgets.trending');

Testing is being overhauled for 5.4 so will wait until then for new features surrounding testing.

@taylorotwell Could i at least make a pull request to add FileViewFinder::getLoadedViews() ?


Character creation not saving ped

Currently when you create a character, you can still select do not save for the first time, which will bug the character as nothing will save. I have saved a default version as a fail safe to ensure people can access their ped. however on first creation can we remove the cancel option as a player needs to save their customisation.

Feature released in https://github.com/iLLeniumStudios/fivem-appearance/releases/tag/v3.9.0


libffi-3.2.1-r4: BAD signature for the Arm architecture

I am trying to build an image using Alpine for armhf and sometimes I get the error below:
$ docker build --build-arg ARCH=armhf --no-cache -t [secure]/pagekite:armhf .
Sending build context to Docker daemon  92.67kB
Step 1/6 : ARG ARCH
 --->
Step 2/6 : FROM multiarch/alpine:${ARCH}-latest-stable
armhf-latest-stable: Pulling from multiarch/alpine
a59d3bc5b01d: Pulling fs layer
1d001419eaed: Pulling fs layer
1d001419eaed: Verifying Checksum
1d001419eaed: Download complete
a59d3bc5b01d: Verifying Checksum
a59d3bc5b01d: Download complete
a59d3bc5b01d: Pull complete
1d001419eaed: Pull complete
Digest: sha256:68a15a2683b7dcf17a3a191045b5563bf1978bb84bfaf4caebb41fa8c53cca69
Status: Downloaded newer image for multiarch/alpine:armhf-latest-stable
 ---> 6adf271d3db0
Step 3/6 : MAINTAINER Michel Filipe<EMAIL_ADDRESS> ---> Running in b2c9314d7a9c
 ---> 4eb7e4566f93
Removing intermediate container b2c9314d7a9c
Step 4/6 : RUN apk add python
 ---> Running in 1eceefde6b1c
fetch http://dl-cdn.alpinelinux.org/alpine/latest-stable/main/armhf/APKINDEX.tar.gz
fetch http://dl-cdn.alpinelinux.org/alpine/latest-stable/community/armhf/APKINDEX.tar.gz
(1/11) Installing libbz2 (1.0.6-r6)
(2/11) Installing libgcc (6.4.0-r8)
(3/11) Installing expat (2.2.5-r0)
(4/11) Installing libffi (3.2.1-r4)
(5/11) Installing gdbm (1.13-r1)
ERROR: libffi-3.2.1-r4: BAD signature
(6/11) Installing ncurses-terminfo-base (6.1-r0)
(7/11) Installing ncurses-terminfo (6.1-r0)
(8/11) Installing ncurses-libs (6.1-r0)
(9/11) Installing readline (7.0.003-r0)
(10/11) Installing sqlite-libs (3.24.0-r0)
(11/11) Installing python2 (2.7.15-r0)
Executing busybox-1.28.4-r1.trigger
1 error; 53 MiB in 28 packages
The command '/bin/sh -c apk add python' returned a non-zero code: 1

I don't face the same problem with regard amd64.

this is a caching issue. the 'latest-stable' was pointing to same package in v3.7 and got cached. in v3.8 the package was rebuilt, but version number didnt change, so url is still the same and you get the old v3.7 package from cache.
You can invaildate the cache with: curl -X PURGE  http://dl-cdn.alpinelinux.org/alpine/latest-stable/main/armhf/libffi-3.2.1-r4.apk
Or you can use v3.8 instead of latest-stable.

Thanks @ncopa! It worked. :+1:


redis::instance: do not exec cp if source and destination are equal

Pull Request (PR) description
Avoid using this copy with destination and source beeing equal
Remark:  generally it's a bad idea to use an exec statement to copy a file. Since this way we cannot see diffs etc.
A better way would be to use a file{ $destination: source=> $source} statement, but running with noop makes it fail then !

Any update about making the cp optional ?


Are they any options to do Software rendering with LWJGL3?

Especially on older PCs LWJGL3 wont't run...
So my Question is can we render the game with the cpu?
I know that the performance won't be so good. But it is better than nothing.

I'm pretty sure that LWJGL will use whatever OpenGL or Vulkan API is provided, so if there is a CPU-based API (which I'm pretty sure there always is for OpenGL), LWJGL can render on the CPU.

And how can I provide an api for lwjgl like swiftshader fron Google

Has no one an Idea?

Did you try the instructions on https://github.com/google/swiftshader#usage ?

"On Windows, most applications can be made to use SwiftShader's DLLs by placing them in the same folder as the executable. On Linux, the LD_LIBRARY_PATH environment variable or -rpath linker option can be used to direct applications to search for shared libraries in the indicated directory first."
Is this working with LWJGL3?

Is this working with LWJGL3?

You could simply try it. LWJGL tries to load vulkan-1.dll first from org.lwjgl.librarypath, then from org.lwjgl.librarypath and if that did not work, from a system-specific path (such as Windows/System32).

Thanks i will try it!

With Swiftshader Libary pressent there is also the Error:
[LWJGL] GLFW_API_UNAVAILABLE error
Description : WGL: The driver does not appear to support OpenGL

Have you read what SwiftShader actually is and that I always mentioned Vulkan above? Of course it does not support OpenGL, as it says so clearly on the repo readme:

Introduction
SwiftShader is a high-performance CPU-based implementation of the Vulkan graphics API

Can you tell me an other option to do it, when SwiftShader isn't the right way?

I tried to put the Mesa3D libary to java.library.path but lwjgl didn't recognize it?
The file is named opengl32.dll.
Is it the right way?

Closing this as out-of-scope for LWJGL. LWJGL does not do rendering, it simply provides bindings to rendering APIs, that may or may not support software rendering.
The correct way to override the OpenGL library used is to set Configuration.OPENGL_LIBRARY_NAME (or the equivalent JVM parameter -Dorg.lwjgl.opengl.libname=<path>). The specified library will be used by both the OpenGL bindings and GLFW (which is the one creating/managing the OpenGL context).

There may exist some software-based implementation of OpenGL or Vulkan, but as Spasi mentioned, this is out of the scope of LWJGL.


Extra top padding when using customview

Hi,
Thanks for this great library.
I am using cutomview to show an image inside a dialog, here is my code :
new MaterialDialog.Builder(this)
                .neutralText("SNOOZE")
                .theme(Theme.LIGHT)
                .customView(R.layout.alarm_dlg, false)
                .cancelable(false);

alarm_dlg.xml :
<?xml version="1.0" encoding="utf-8"?>
<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
    android:orientation="vertical" android:layout_width="match_parent"
    android:layout_height="match_parent">

    <ImageView
        android:layout_width="match_parent"
        android:layout_height="100dp"
        android:scaleType="centerCrop"
        android:layout_marginTop="0dp"
        android:id="@+id/imageView"
        android:src="@drawable/burst_landscape"
        android:layout_gravity="top" />
</LinearLayout>

But there is an extra top padding :

Based on documentation, there should be no padding when using customviews and scrolling set to false. am I doing something wrong ?

That padding is for the title, although I'm not sure if it can be removed.
On May 26, 2015 10:44 PM, "ffallah"<EMAIL_ADDRESS>wrote:

Hi,
Thanks for this great library.
I am using cutomview to show an image inside a dialog, here is my code :
new MaterialDialog.Builder(this)
.neutralText("SNOOZE")
.theme(Theme.LIGHT)
.customView(R.layout.alarm_dlg, false)
.cancelable(false);
alarm_dlg.xml :

<ImageView
    android:layout_width="match_parent"
    android:layout_height="100dp"
    android:scaleType="centerCrop"
    android:layout_marginTop="0dp"
    android:id="@+id/imageView"
    android:src="@drawable/burst_landscape"
    android:layout_gravity="top" />

But there is an extra top padding :
[image: extra-top-padding]
https://cloud.githubusercontent.com/assets/9332653/7827690/71569e32-0454-11e5-987e-b03e8c744cc0.png
Based on documentation, there should be no padding when using customviews
and scrolling set to false. am I doing something wrong ?
—
Reply to this email directly or view it on GitHub
https://github.com/afollestad/material-dialogs/issues/543.

Well, if there is no title, I don't think there should be a padding for it. especially if you are using a custom view.

I raised an issue too regarding this, the owner said that's how it's intended to work

@Murphybro2 this isn't the same.
The padding in your issue is for list dialogs. This issue is for custom views with no title.

To add to the OP, if you use a custom view without any buttons, you'll get a similarly odd pattern on the bottom.

Working on this now

Should be fixed in the latest commit.

An update was made available late last night:
https://github.com/afollestad/material-dialogs/releases/tag/v<IP_ADDRESS>

Thank you for the update. I can confirm the issue is resolved.

Hello, i've seen that this issue has been fixed on may. But right now that extra spacing on a dialog with no title and buttons but a list, is still visible. Is it posibile to remove somehow that extra padding?

View customView = materialDialog.getCustomView();
customView.setPadding(0,0,0,0);
assert customView != null;


[Feature]Does proxyllm support specified versions of models

Search before asking

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

Description
目前用的tongyi_proxy，有api-key,但是却没有地方指定具体使用哪种qwen模型的选项？因为不同模型之间差距也很大
我在.env加了PROXYLLM_BACKEND=qwen2-72b-Instruct,其它没动，提示模型不存在model doesn't exist，不知道还要改动哪些文件
Use case
No response
Related issues
No response
Feature Priority
None
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

/assign @fangyinc


🧑🔬 Prototype: Enforcing MFA for owners of top 100 downloaded gems (UI)

Contributes to https://github.com/Shopify/code-foundations/issues/166
🧪 What problem does this prototype solve?
Please refer to the RFC: Require MFA on the top 100 most downloaded gems
🛠 TL;DR on Technical Approach
Note: This prototype represents the cumulation of various approaches that were spiked in previous prototypes:

🧬 Programmatic - dynamically determine owners of top 100 gems

🧬 Add database column (boolean) - identify mfa-required users by mfa_required field

🧬 Add database column (date) - provide an mfa enrolment deadline, upon deadline mfa enforcement becomes a requirement

🧬 Redirect all users to MFA landing page if MFA is not enabled

This is the outline of the approach:
🔹 Add mfa_required_at column to users table
The date saved to this field will indicate when a user will have MFA enforced.
🔹 Add script to enroll MFA for top 100 most downloaded gem owners
When the script is run, it sets a date in the future for users identified as the top 100 most downloaded gem owners at that given moment.
🔹 Redirects user to MFA page if account is flagged as mfa_setup_recommended
If a user has an mfa_required_at date set in the future, they will be redirected to the MFA landing page upon logging into rubygems.org UI. A flash notice appears encouraging the user to set up MFA. The user can navigate away from the MFA landing page regardless of whether they enable MFA.
🔹 Redirect users to MFA page if account is flagged as being non-compliant to MFA policy.
If a user has an mfa_required_at date set in the future, they will be redirected to the MFA landing page upon logging into rubygems.org UI. A flash notice appears encouraging the user to set up MFA. The user can navigate away from the MFA landing page regardless of whether they enable MFA.
🔹 For users with MFA enforced, they can deregister their device to set up a new one
For users who are required to have MFA enabled, they have the option to deregister a device. For users with no MFA policy enforced, they can continue with the current user flow and disable MFA if it was enabled.
🔹 Disallow ui_only MFA level for popular gem owners
This removes the ui_only level from the api settings dropdown and blocks it on the backend if the user owns one of the 100 most downloaded gems.
🎬 Demos
Below are demos of various user flows
🎬 User flow of a top gem owner without MFA enabled
https://user-images.githubusercontent.com/4270287/149182021-376e44cb-b484-4ee9-96fe-aa1446659d0c.mov
🎬 User flow of a top gem owner deregistering their device (thereby disabling MFA)
https://user-images.githubusercontent.com/4270287/149188379-2a2602ce-de60-400c-9ddc-696f4e8c8137.mov
🎬 User flow of a top gem owner setting up MFA
https://user-images.githubusercontent.com/4270287/149187258-67e9be42-495f-41f8-bb50-e44e5e004063.mov
🎬 User flow of a top gem owner signing in with MFA already enabled
https://user-images.githubusercontent.com/4270287/149187236-eb39b4ed-bd60-427e-bd3a-e91a143900a2.mov
🎬 User flow for users who are not owners of a top 100 gem
https://user-images.githubusercontent.com/4270287/149189785-949d1f37-8f40-4966-8d33-45cd46ee6aba.mov

I'm going to close this out since the work building off this prototype has been merged into the main repo 🎉  🧹


fix(TCOMP-615):fix equals and hash for JarRuntimeInfo

What is the current behavior? (You can also link to an open issue here)
the cache for classloader does not consider different RuntimeInfo with different list of dependencies or this is problem cause these lead to 2 differents classloaders.
What is the new behavior?
JarRuntimeInfo now checks only for ClassName and the list of dependencies for equality.
Please check if the PR fulfills these requirements

[x] The commit message follows Talend standard
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features) ?
[x] The code coverage on new code >75%
[ ] The new code does not introduce new technical issues (sonar / eslint)

What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build / CI related changes
[ ] Other... Please describe:

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Pull request test result: failure :x:

Pull request test result: success :star:

Pull request test result: success :star:

Pull request test result: success :star:


2.2.1 still showing for version 2.3.0

Just updated to version 2.3.0 by following the same installation process as 2.2.1 but overwriting files
The version 2.2.1 is still showing in the plugins list for version 2.3.0
Did I do something wrong?

You're not wrong.
But the version number is just a small number.
You can change your self if you want to.

Just updated to version 2.3.0 by following the same installation process as 2.2.1 but overwriting files
The version 2.2.1 is still showing in the plugins list for version 2.3.0
Did I do something wrong?

My bad, thanks for catching this. All fixed now thanks to @master3395


docs: update README.MD and FAQ Page for Superset 1.0 launch

SUMMARY
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
In preparation for Superset 1.0 launch next Tuesday, lets update the visuals in the repo!
Swapped out screenshots & switched slice/dice screenshot for a GIF

![slice_and_dice](https://user-images.githubusercontent.com/801507/104490439-d7e16a00-559e-11eb-8260-a4d9c1222487.gif)

TEST PLAN

Look for any inaccuracies, spelling / grammar issues, etc.
Test images & GIF's rendering in Github

ADDITIONAL INFORMATION

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

instead of pngs can you attach jpgs here? There should be no need to 5MB image files in the README, so let's reduce the size to a reasonable amount (otherwise it'll take forever to load the page)

Good point, I'll compress to JPG. Not much we can do about the GIF though, should I just remove it?

instead of pngs can you attach jpgs here? There should be no need to 5MB image files in the README, so let's reduce the size to a reasonable amount (otherwise it'll take forever to load the page)

Good point, I'll compress to JPG. Not much we can do about the GIF though, should I just remove it?

Codecov Report

Merging #12499 (de1fe07) into master (7aba4c2) will decrease coverage by 2.76%.
The diff coverage is 81.81%.

@@            Coverage Diff             @@
##           master   #12499      +/-   ##
==========================================
- Coverage   66.29%   63.53%   -2.77%
==========================================
  Files        1015      486     -529
  Lines       49554    29929   -19625
  Branches     5079        0    -5079
==========================================
- Hits        32854    19015   -13839
+ Misses      16562    10914    -5648
+ Partials      138        0     -138

Flag
Coverage Δ

cypress
?

javascript
?

python
63.53% <81.81%> (-0.56%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

superset/databases/schemas.py
99.45% <ø> (-0.55%)
:arrow_down:

superset/db_engine_specs/druid.py
88.63% <33.33%> (+27.27%)
:arrow_up:

superset/db_engine_specs/postgres.py
84.21% <89.47%> (-1.76%)
:arrow_down:

superset/db_engines/hive.py
0.00% <0.00%> (-85.72%)
:arrow_down:

superset/sql_validators/postgres.py
50.00% <0.00%> (-50.00%)
:arrow_down:

superset/db_engine_specs/hive.py
54.61% <0.00%> (-29.24%)
:arrow_down:

superset/db_engine_specs/presto.py
70.56% <0.00%> (-11.48%)
:arrow_down:

superset/db_engine_specs/sqlite.py
65.62% <0.00%> (-9.38%)
:arrow_down:

superset/utils/celery.py
96.42% <0.00%> (-3.58%)
:arrow_down:

... and 546 more

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 40a334a...de1fe07. Read the comment docs.

Codecov Report

Merging #12499 (de1fe07) into master (7aba4c2) will decrease coverage by 2.76%.
The diff coverage is 81.81%.

@@            Coverage Diff             @@
##           master   #12499      +/-   ##
==========================================
- Coverage   66.29%   63.53%   -2.77%
==========================================
  Files        1015      486     -529
  Lines       49554    29929   -19625
  Branches     5079        0    -5079
==========================================
- Hits        32854    19015   -13839
+ Misses      16562    10914    -5648
+ Partials      138        0     -138

Flag
Coverage Δ

cypress
?

javascript
?

python
63.53% <81.81%> (-0.56%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

superset/databases/schemas.py
99.45% <ø> (-0.55%)
:arrow_down:

superset/db_engine_specs/druid.py
88.63% <33.33%> (+27.27%)
:arrow_up:

superset/db_engine_specs/postgres.py
84.21% <89.47%> (-1.76%)
:arrow_down:

superset/db_engines/hive.py
0.00% <0.00%> (-85.72%)
:arrow_down:

superset/sql_validators/postgres.py
50.00% <0.00%> (-50.00%)
:arrow_down:

superset/db_engine_specs/hive.py
54.61% <0.00%> (-29.24%)
:arrow_down:

superset/db_engine_specs/presto.py
70.56% <0.00%> (-11.48%)
:arrow_down:

superset/db_engine_specs/sqlite.py
65.62% <0.00%> (-9.38%)
:arrow_down:

superset/utils/celery.py
96.42% <0.00%> (-3.58%)
:arrow_down:

... and 546 more

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 40a334a...de1fe07. Read the comment docs.

Hey Srini, thanks for updating the doc! Could you replace Explore screenshot with the latest below? Lots of changes have taken place lately, thanks! @srinify

Hey Srini, thanks for updating the doc! Could you replace Explore screenshot with the latest below? Lots of changes have taken place lately, thanks! @srinify

Can we change the Release Notes link in README.md to point to RELEASING/README.md, then add a section near the top where we can add links to the release notes for major releases?

Can we change the Release Notes link in README.md to point to RELEASING/README.md, then add a section near the top where we can add links to the release notes for major releases?

@srinify i would like to include it for 1.0 release wdyt?

@srinify i would like to include it for 1.0 release wdyt?

@rusackas thanks for approving Evan! I'll implement Junlin's / Robert's changes and then we can merge! And resolve the merge conflict

@rusackas thanks for approving Evan! I'll implement Junlin's / Robert's changes and then we can merge! And resolve the merge conflict


Support for Darling project in WSL (ability to run Mac apps on Linux)

Hi,
in my try to compile iOS Metal apps from Windows (I use http://www.pmbaty.com/iosbuildenv/ that supports iOS compilation on Windows for example OpenGL apps) I see that for Linux we have:
https://github.com/darlinghq
which can run Metal shader compilers (macos command line binaries) on Linux
(see for success: https://github.com/darlinghq/darling/issues/326)
so can support ios Metal apps compilation (http://www.pmbaty.com/iosbuildenv provides Linux support also)..
so I asked Darling developers WSL support and pointed me to:
https://github.com/darlinghq/darling/issues/260
they say:
As of now, it's impossible to run Darling in WSL for various reasons, the two most important ones being:
1)Darling needs to load its own kernel module. Since there's no Linux kernel in WSL, it's impossible to use our module there. To make it work, the functionality would need to be reimplemented in some other way.
2)We make use of various features that WSL does not support, such as mount namespaces (last time I checked, WSL didn't have any support for mounting anything, let alone namespacing that).
From:
https://msdn.microsoft.com/en-us/commandline/wsl/release_notes
seems mount namespaces is implemented now..
how can they workaround "kernel module" support..
would be nice if Microsoft devs explore Darling Project support under WSL so make command line binary macos apps runnable on Windows via Darling+WSL..
thanks..

I commented over there the difficulties, but as long as it relies on a module, it's not going to be possible for now.  We'll see in the future.

Depends a lot on the nature of the kernel module and what functionality it provides; contrast with say WINE (which is the spiritual reverse of Cygwin). You can do a Metal compiler on Linux just fine. What you would do with the result is another matter. The request is not dissimilar to #1783.

Since WSL2 early preview released today, Can anyone confirm if it works with WSL2?

Yes, Darling seems to work in WSL 2.

Thanks. This one was submitted before loading kernel modules was in the cards, and back then probably should have been punted user voice or closed tag 'discussion'. Since it is still open we'll chalk this fixed-in-wsl2.
If there are remaining gaps in the scenario (and I am sure there are) they probably wouldn't fall into the WSL actionable sphere. Once you are compiling your own kernel and/or loading modules, that is pretty off road. But if something comes up feel free as always to open a new issue.


Add WithMipGap trait

Add a WithMipGap trait and implement it for coin_cbc, highs, and lp_solvers.  This trait mimics the trait of the same name that we recently added to the lp-solvers crate.
Implementing WithMipGap for lpsolve and scip appears to be non-trivial so I haven't done those, and minilp doesn't support integer variables.
I created a macro for generating tests of the WithMipGap trait.  This is my first experience writing a Rust macro so hopefully it's okay.  It does work as intended.

@mmghannam any idea how hard it would be to implement mip gap in russcip ?

I think implementing MIP gap in russcip should be as simple as calling model.set_real_param("limits/gap", gap) but set_real_param is implemented only when the model is in the Unsolved state and the state changes almost immediately after the scip() function creates it:
let mut model = Model::new()  // state is Unsolved
        .hide_output()  // state is Unsolved
        .include_default_plugins()  // state is PluginsIncluded
        .create_prob("problem") // state is ProblemCreated
        .set_obj_sense(match to_solve.direction {
            ObjectiveDirection::Maximisation => ObjSense::Maximize,
            ObjectiveDirection::Minimisation => ObjSense::Minimize,
        });  // state is ProblemCreated

A with_mip_gap method might be able to call model.try_new() to get the model back into the Unsolved state so we can call set_real_param but then we'd have to get the model back into the ProblemCreated state before returning.  I haven't investigated any farther yet.

@mmghannam what do you think? Maybe russcip could allow setting parameters in the ProblemCreated state? Or would that introduce the possibility for an inconsistent state ?
@mfuhr : anyway, I don't see an issue with with merging this PR without russcip support now, and adding it later.

@mmghannam what do you think? Maybe russcip could allow setting parameters in the ProblemCreated state? Or would that introduce the possibility for an inconsistent state ?

Setting parameters in ProblemCreated state should be fine as far as I know. I added an issue https://github.com/scipopt/russcip/issues/105. I'm just not sure when I'll have time to work on it.

@lovasoa I'd be happy to make any more changes you suggest.  Thanks for the help -- I'm pretty new to Rust and to GitHub collaboration so your feedback has been very useful!


feat: Change ion-fab css position to fixed

Type: feat
Ionic Version: 2.x
Platform: all
Currently the ion-fab buttons to not scroll with the container because they are position: absolute. If we change this to position: fixed it will stick to the bottom of the screen during scrolling.

Hello! Thanks for the feature request. I'm going to move this issue over to our internal list of feature requests for evaluation. We are continually prioritizing all requests that we receive with outstanding issues. We are extremely grateful for your feedback, but it may not always be our next priority. I'll copy the issue back to this repository when we have begun implementation. Thanks!

This issue was moved to ionic-team/ionic-feature-requests#234

@jgw96
The subsequent ticket should be closed. The FAB appears to be working correctly. Making the position fixed works fine on android but blows up on iOS. For scroll content the FAB needs to be placed outside of the ion-content tag


⚠ Bofrost.fr has degraded performance

In 4659224, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1258 ms

Resolved: Bofrost.fr performance has improved in d3776f2.


A suggestion of Highlight

An amazing theme based on hexo !
But it looks like the current version doesn't work with prismjs themes. It would be better if adding more hightlight themes.
Or users can load .css/.js file by themselves.

In addition, the the current version do not support line_number option. I'm not sure whether my configuration is right:
highlight:
enable: false
prismjs:
enable: true
preprocess: true
line_number: true
tab_replace: ''

Thanks you for your support! Yeah I am planning on adding the option to change Prism themes.

replicate #298

很需要的改进，如果能在右上角一键复制就更好了


Exception on empty text

It looks like truncateWords is throwing an exception when text is empty.

Type error: Argument 1 passed to DOMWordsIterator::__construct() must be an instance of DOMNode, null given

called in line 65
It would be nice to avoid such situation, maybe with an additional control

same error in truncateLetters()


Audio DAC tas2563

This adds the driver for the Audio DAC TAS2563 and a speaker sample.

FWIW I am likely not the right assignee for this (but I will probably keep getting assigned whenever changes are pushed to your branch), but Audio area is not officially maintained so I'm being assigned by default as a maintainer for the code samples.
Pinging audio collaborators here again for awareness though: @lyakh @lgirdwood @marc-hb @kv2019i


Add example and documentation for Flake Id Generator

Flake Id Generator example and documentation is added.

Linux test PASSed.

Windows test PASSed.

Why does this PR has Estimation: L label? Do we consider this PR with its implementation and is that why it has this label?

@SaitTalhaNisanci Corrected estimation to S.

Linux test PASSed.

Windows test PASSed.


Portal Moves on its own

Descriptions: Portal moves when one is in the overworld and the other is in the nether
Steps to Reproduce:
Shoot one portal in the overworld, then go into the nether and shoot a portal in there.
Go through the portal in the nether and then back through again a couple times
Results:
The portal in the nether should teleport into the overworld
Expected Results:
It should not move dimensions

should be fixed in v1.1.0

fixed


Commented Danger rules

Temporary removed some danger rules, we should keep improving them onto danger branch. Once they are configured add them to master.

          2 Messages

  :book:
  Executed 101 tests, with 0 failures (0 unexpected) in 1.209 (1.398) seconds

  :book:
  Executed 97 tests, with 0 failures (0 unexpected) in 1.321 (1.570) seconds

  Generated by :no_entry_sign: danger

Current coverage is 30.25% (diff: 100%)

Merging #265 into master will not change coverage

@@             master       #265   diff @@
==========================================
  Files            45         45
  Lines          2003       2003
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            606        606
  Misses         1397       1397
  Partials          0          0

Powered by Codecov. Last update f860951...81d0a91

@goktugyil?

@lfarah  yep?
Why is this PR a merge of a lot of PRs?

@goktugyil because our master branch contains merge commits, I was rebasing on master branch

@goktugyil we need pullrequest template, so we won't apply rules if user inform what he did is fails rules but he aware of it

@piv199 #276


Verify that "Muncelu" is listed in the dropdown menu when searched

Description:
"Muncelu" should appear in the dropdown menu with correct spelling and locality information.
Preconditions:
The website is up and running.

Step 1
Type "Muncelu" in the search bar.
Expected results:
The dropdown is listing "Muncelu" as the first location in the dropdown menu.

Step 2
Observe the dropdown menu.
Expected results:
"Muncelu" is displayed and selectable from the dropdown menu.


Fix App deprecation check mechanism

Description
The app deprecation check mechanism was not working as it required firebase login with web3 account. This requirement is not required for the version check.
Atm we need to register every single version and mark it deprecated in the firebase DB. Now a minimum version in firebase db should do the task
Testing
The deprecation system was tested on the device by:

Changing the deprecation flag of the testing version (1.5.0) in the firebase db. The format versions > 1 > 5 > 0 > deprecated > true was used.
Adding a minVersion flag to versions and providing it a higher value than the current version

In both the cases the deprecation screen was showed at app start.
Related issues

Fixes #1235

Backwards compatibility
It is backwards compatible.

As I asked in my last review, please add info about how this was tested.

Added the Testing information in the description:
Testing
The deprecation system was tested on the device by:

Changing the deprecation flag of the testing version (1.5.0) in the firebase db. The format versions > 1 > 5 > 0 > deprecated > true was used.
Adding a minVersion flag to versions and providing it a higher value than the current version

In both the cases the deprecation screen was showed at app start.


go sum update

Build failing due to. Updated by regenerating go.sum
go: verifying<EMAIL_ADDRESS>checksum mismatch
        downloaded: h1:v3H7yHgF+94suF7Xg6V7Haq6Anac3X6WosuKGTTJCGM=
        go.sum:     h1:WyMdmwfw02n2i3gTkzy+HBR2dsh1axjxn808iAe9PLI=

Checks are failing

Checks are failing

This was due to version differences. Have updated Dockerfile and travis config.
Can squash the commits.

Cool squashing and merging


Improve docker info against 1.7

ping @aluzzardi @abronan i'd like to include this one in the rc2

@aluzzardi PTAL

LGTM


Adding sign and inspect blob commands

Adding sign and inspect blob commands.
Spec Pr used for ref: https://github.com/notaryproject/notation/pull/811

Codecov Report
Attention: 11 lines in your changes are missing coverage. Please review.

Comparison is base (31c9e84) 64.09% compared to head (2ed55c6) 64.45%.
Report is 23 commits behind head on main.

Files
Patch %
Lines

cmd/notation/blob.go
88.29%
11 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #856      +/-   ##
==========================================
+ Coverage   64.09%   64.45%   +0.35%
==========================================
  Files          40       46       +6
  Lines        2284     2788     +504
==========================================
+ Hits         1464     1797     +333
- Misses        698      835     +137
- Partials      122      156      +34

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


fix(r): sub-day precision Date should be floored when treated as integer

Fix #665

This is to better align with how base R normally handles this (although I think either are OK).
typeof(as.Date("2001-02-03"))
#> [1] "double"
typeof(Sys.Date())
#> [1] "double"

Indeed, it seems that when a number is converted to Date with as.Date(), the original numeric type is preserved.
So I think it does not seem surprising that the output of nanoarrow would be an integer.
typeof(as.Date(1L))
#> [1] "integer"

Created on 2024-11-03 with reprex v2.1.1


Investigations on "ON CASCADE DELETE" statement and its implementation in JPA.

Tests in the database reveled that delete is not possible due to foreign key constraints in the project table. Therefore, it is necessary to change the entities to allow deletion and share the deletion with child items in other tables.

Up to this point, the API can CRUD projects. The deletion of a project involves three classes: Project, ManageProject, and ProjectTime. Each one are related to an entity in the database. To delete a project:

collect the information of a project: project id, name, description, starting date, and deadline;
delete the father elements in Project and ProjectTime to delete automatically the children in ManageProject.


Updated length() method

Size method does not exist for priority queues. It has a length() method.

@neptunius


How to disable button back in the last tab?

@agilBAKA You can do it via refs
https://jsfiddle.net/bt5dhqtf/691/
It's not a very clean solution because you have to write the button yourself and also have the ref, but this will be possible in the next release with scoped slots

  <button v-show="!props.isLastStep" @click="props.nextTab()>Back</button>

The only disadvantage is that once you customize this stuff, you will have to wire up the button clicks and the UI but it will be fully customizable. Until the next release you could use the solution from above.

@agilBAKA
The other option is possible now due to the impleemntation from #27
Example
 <template slot="footer" scope="props">

          <wizard-button v-if="props.activeTabIndex > 0 && !props.isLastStep" :style="props.fillButtonStyle">Previous</wizard-button>

          <wizard-button @click.native="props.nextTab()" class="wizard-footer-right" :style="props.fillButtonStyle">Next</wizard-button>

      </template>

Fiddle https://jsfiddle.net/bt5dhqtf/710/
It's up to you which option you chose, but the last one is more modular and allows changes easier as well as does not depend on refs which have their issues.

Released https://github.com/cristijora/vue-form-wizard/releases/tag/v0.6.2
Also see docs about this part

Thank you @cristijora

I should disable the button at the last step. Can you give a complete example using the "wizard-button" ?
Thanks


ARROW-6861: [C++] Fix length/null_count/capacity accounting through Reset and AppendIndices in DictionaryBuilder

Some more work is needed in ARROW-6869 to move the memo table resetting logic out of DictionaryBuilder::Reset but we can leave that for 1.0.0 work

I think if this is disabling the current delta support then this shouldn't go into 0.15.1. Is there a way to solve the Parquet issue without touching delta support (which can be done in another PR for a later version)? For example by adding e.g. a FullReset() method on DictionaryBuilder).

@pitrou I just pushed a much simpler fix that does not touch the dictionary delta logic. I'll rebase a patch I have done for ARROW-6869 after this is merged into master

+1. Merging this


HBASE-22801 Maven build issue on Github PRs

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
43
Docker mode activated.

_ Prechecks _

+1
dupname
0
No case conflicting files found.

0
shelldocs
0
Shelldocs was not available.

+1
@author
0
The patch does not contain any @author tags.

_ master Compile Tests _

_ Patch Compile Tests _

+1
shellcheck
0
There were no new shellcheck issues.

+1
whitespace
0
The patch has no whitespace issues.

_ Other Tests _

+1
asflicense
16
The patch does not generate ASF License warnings.

130

Subsystem
Report/Notes

Docker
Client=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-459/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/459

Optional Tests
dupname asflicense shellcheck shelldocs

uname
Linux 157edd693833 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-459/out/precommit/personality/provided.sh

git revision
master / eef8e140c5

Max. process+thread count
47 (vs. ulimit of 10000)

modules
C: . U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-459/1/console

versions
git=2.11.0 maven=2018-06-17T18:33:14Z) shellcheck=0.4.4

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
38
Docker mode activated.

_ Prechecks _

+1
dupname
0
No case conflicting files found.

0
shelldocs
1
Shelldocs was not available.

+1
@author
0
The patch does not contain any @author tags.

_ master Compile Tests _

_ Patch Compile Tests _

+1
shellcheck
0
There were no new shellcheck issues.

+1
whitespace
0
The patch has no whitespace issues.

_ Other Tests _

+1
asflicense
17
The patch does not generate ASF License warnings.

127

Subsystem
Report/Notes

Docker
Client=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-459/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/459

Optional Tests
dupname asflicense shellcheck shelldocs

uname
Linux 8549f4bc1cd5 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-459/out/precommit/personality/provided.sh

git revision
master / 3b0013ded9

Max. process+thread count
46 (vs. ulimit of 10000)

modules
C: . U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-459/2/console

versions
git=2.11.0 maven=2018-06-17T18:33:14Z) shellcheck=0.4.4

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.


Allow customizing smallfile publisher image

Allow customizing smallfile publisher image

Results for crc_jjb

Test
Result
Retries
Duration (HH:MM:SS)

test_smallfile
Pass
0
00:04:45

This would be useful for benchmark developers because they would not have to rebuild benchmark operator just to get it to use a different image name. Is that the intent here? if so, I'd like to do it for fs-drift and fio as well.

@bengland2 this functionality was added to the other workloads. It was just missed in the publisher section of smallfile. @rsevilla87 ++ for catching it


Make sure examples pass as part of unit test suite

Sometimes I make backwards-incompatible changes, and this should be made clear by running all the example code with the unit test suite.

Fixed in 22383ff


Improve error messages for Unable to find response value from in cases where size too big

You get a cryptic error message when the API returns a response that is too large to be included on chain.
It looks like the value can't be found in the response, when in reality the response is just too big.

Is this about https://discord.com/channels/758003776174030948/765618225144266793/976838180684824669 or something else?

It's from https://api3dao.atlassian.net/browse/AN-599 that @aquarat opened. @aquarat?

AN-599 was created 09-MAR and is perhaps related to #752, for which @vponline implemented some tests here: https://github.com/api3dao/airnode/pull/818/commits/9bb995a4224460b73fce85436f0d2f6b4199c312 as part of #818.

@aquarat Can you comment on this?

Ping @aquarat ^

Sorry everyone, I'm working on managing my Github notifications better.
I am investigating now.

@Siegrift implemented the original response size limiting. This is circa 4th December 2021. I was tasked with testing the completed work - which I did at the time. I found that the error message in the logs didn't surface the underlying error and I noted it at the time both to Emanuel directly via Slack and apparently in the Jira issue you linked.
Looking at the code in airnode-node it looks like this bug no longer exists by virtue of processSuccessfulApiCall wrapping the call to extractAndEncodeResponse inside a try/catch, where the catch returns the error as a log array.
So this issue can be closed, at least in terms of what it originally referenced.


Update vehicles.md

Removed text snippet about cars not yet shipped.
Current API doesn't include any car-references until the vehicle has been delivered. Instead an empty response is the only output.

Thanks!
I think this may be on the products API now? Unfortunately, it's rather expensive to test...


rename getInput() to pollButtons()

Or pollButtonState()?  pollButtonStatus()? getInput is too generic and makes me write library code like:
uint8_t button_state = getInput();
if (button_state & LEFT_BUTTON)

I do that because to me getInput is opaque and I want clarity.  What input exactly am I getting?  But then I realized the real problem is the name of the method, it could be more descriptive.
Not many should use this directly but we should still have the best name possible

readButtonState() // ?
readButtons() // ?

I could imagine read used here.

Like I said in PR #23, I'm kind of like just buttonState().

The word state is very programmery, but so is poll.  Condition is silly though.  Is there  much better word I'm not thinking of?

The word state is very programmery

I don't know if I agree about that. State has pretty much the same meaning outside of programming. E.g. "State of the Union", "In a confused state". So, I think buttonState is fairly descriptive. Even the Doxygen comments for getInput currently begin with:
// get current state of buttons (bitmask)
And @yyyc514, you yourself used button_state = getInput() when you wrote function pressed().
Besides, people using the function will be programmers, or if they're "programmers in training" they should be expected to become familiar with the meaning of state in this context.
I agree that condition doesn't sound right. The only other I can think of  is buttonPositions(), which I'm not too keen on.

buttonsState()  I like.  It's clear that it's polling multiple buttons (not just returning one) and again this is low-level and people using it should understand what it does.

buttonsState() I like

That's fine with me then. :smile:


Improving remove-usage

Stop
On December 27, 2017, at 10:47 AM, David Leitner<EMAIL_ADDRESS>wrote:
You can view, comment on, or merge this pull request online at:
https://github.com/robbyrussell/oh-my-zsh/pull/6516
Commit Summary
Update extract.plugin.zsh
File Changes
M plugins/extract/extract.plugin.zsh (2)
Patch Links:
https://github.com/robbyrussell/oh-my-zsh/pull/6516.patchhttps://github.com/robbyrussell/oh-my-zsh/pull/6516.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.￼
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/robbyrussell/oh-my-zsh","title":"robbyrussell/oh-my-zsh","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/robbyrussell/oh-my-zsh"}},"updates":{"snippets":[{"icon":"DESCRIPTION","message":"Improving remove-usage (#6516)"}],"action":{"name":"View Pull Request","url":"https://github.com/robbyrussell/oh-my-zsh/pull/6516"}}}


Search: Only 10 sites are shown, even if there are more than 10 sites

Search query: https://developers.sitecore.com/search?q=Content Hub ONE CLI

10 sites are shown, but not YouTube
However, Video type shows 55 results.

Filtering by Video: https://developers.sitecore.com/search?q=Content Hub ONE CLI

1 site is shown (YouTube).

This indicates that the facet group is hiding additional facets for the site.

This is fixed.
There was a max return setting of 10 being replied, i've bumped this up to 250 to ensure we always have all sites returned in the facet data.


Update icons from materialdesignicons.com

Update new icons from materialdesignicons.com.
Contains a few extra icons that could be useful for a lot of people like garage and garage-open.

Awesome!


Failed to extract the starterkit

Hi guys, Im new in gulp and emulsify, using windows 10 1703, php 7.1 and nvm for windows.
In our project, we need to use emulsify, but I have problem to install it. Everything looks ok when emulsify is installing, but when pattern-lab is installing, there is problem with extracting starterkit:
suggested starterkits that work with this edition:

  1: pattern-lab/starterkit-twig-base
  2: pattern-lab/starterkit-twig-demo

downloading the starterkit. this may take a few minutes...
finished downloading the starterkit...
failed to extract the starterkit. easiest solution is to manually download it and copy <path>./dist</path to ./source/...
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>postinstall: `rm -rf pattern-lab && composer create-project -n pattern-lab/edition-twig-standard pattern-lab && rm -rf pattern-lab/source && ln -s ../components pattern-lab/source && git clone https://github.com/drupal-pattern-lab/bem-twig-extension.git && cp bem-twig-extension/bem.function.php components/_twig-components/functions && rm -rf bem-twig-extension/`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

I tried to download and copy dist from git, but when I type npm start, there is error for missing 'clean' task in gulpfile.js
Thanks for suggestions

Related: #11


sngrep: 1.7.0 -> 1.8.0

Description of changes
A new version was released: https://github.com/irontec/sngrep/releases/tag/v1.8.0
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 275658 run on x86_64-linux 1

1 package built:

sngrep


Eventhubs track2 preview1 fix2

Remove azure-core dependency
Add azure-identity in dev_requirements.txt

Can one of the admins verify this patch?


feat: switch to multicall3

Closes https://github.com/joshstevens19/ethereum-multicall/issues/45
I didn't update the ABI to include the new Multicall3 methods since it's not deployed on etherlite. I confirmed it built successfully, but I didn't see any tests so I'm not sure how to verify I didn't break anything 😅
All chains are still present and many new ones added, but the sort order is now based on that of the Multicall3 readme so the diff is a bit wonky.

Thanks for this.. Haha yep sorry > https://github.com/joshstevens19/ethereum-multicall/issues/1 I did this repo a while ago and never had or should I say made time to add tests.. but I can easily do an E2E test il fetch and test this out over weekend thanks for doing it!
one thing could you update the readme supported chains ?

README updated!

this is now deployed thanks a lot

Awesome, thank you! 🙌


Is the 'OS_RELOAD_ENABLED' environment variable still used?

It appears all integration tests and the softlayer_creator unit test explicitly set the OS_RELOAD_ENABLED environment variable to TRUE. I'm trying to understand why that's being set as I can find no production code that references it in this release or its dependencies.
If the code is unused, it should be removed as it makes it harder to reason about what's going on in the code.
(Moved here from https://github.com/cloudfoundry-incubator/bosh-softlayer-cpi-release/issues/26)

So this was added early during development as we were investigating os_reload feature. Perhaps @mattcui can help answer where this is now?

Yes, this logic has never been tested, and no one ever used it. It would make code look complicated and ugly, I agree to remove this functionality. I will open a tracker.

@mattcui closing this based on tracker issue. Please add a reference to keep things connected.