Flaky test: active editor not always correct...

This test started to fail recently. At first I thought it was only in browser but I also see it in electron now:
https://github.com/microsoft/vscode/blob/4978f52ec8d56ae52809b6a3683b80e4144bdf50/extensions/vscode-api-tests/src/singlefolder-tests/window.test.ts#L154
https://github.com/microsoft/vscode/runs/438638035#step:13:397
When I briefly debugged this in the browser I noticed how - when the second editor opens to the side - both editors show blinking cursors and as such I suspect an issue with our focus tracking in the editor. That is also why the first editor shows as active even though the second one opened.
I have for now disabled this test.

At first I thought it was only in browser but I also see it in electron now:

You cannot debug this test because it depends on correct focus-logic - this and other tests mean you cannot touch your keyboard/mouse when running API tests locally. I wonder if playwright has issues with that and that it is failing on the build machines more often because of that

I can reproduce it locally when I run with UI, e.g. not headless. That is how I noticed that 2 editors show a blinking cursor each which hints to focus problem.
As for electron: this has started to happen with the E7 adoption, so we might hit a browser issue now that came into Electron too.

That is how I noticed that 2 editors show a blinking cursor each which hints to focus problem.

There is well known issue (since very long) for those two blinking cursors. @alexdima introduced me to this and I don't remember if its chrome or electron but it happens on mac  when using multiple desktops.

The known issue is #82326 -- having two windows on macOS might lead to both of them believing they have focus after unlocking or after switching desktops, but that is not about two editors in a single window having focus...
@bpasero How can I reproduce locally the two editors with two blinking cursors?

@alexdima steps:

from master with distro mixed in
change this line to suite.only
add a timeout after this line await new Promise(c => setTimeout(() => c(), 10000))
run resources/server/test/test-web-integration.sh --debug

@bpasero I have investigated by adding logging to the relevant code in the editor.  Basically, focus and blur events are not delivered to DOM nodes... so that's why both editors believe they are focused...
I think this is an issue with the automation...

I have created https://github.com/microsoft/playwright/issues/979

@alexdima thanks!


Fail pdo_sqlsrv.

It would be great if it could also be installed in PHP 8.0, https://pecl.php.net/package/pdo_sqlsrv/5.9.0preview1,
Thks for all your work,

@terabytesoftw Patched in #310. It should work now.


Mqtt Code not working

Hi, I am not sure if this is an issue of mqtt or nukihub.
but I tried to set a code via the default_code field in Mqtt device setting. Though, code is not asked before execution. I also tried to change my nuki pin, no difference
any Idea?

Thanks! Great Development!! Love it so far :)

NUKI Hub version: 8.27
run: true
deviceId: 534405045
deviceIdOp: 534405045
mqttbroker: <IP_ADDRESS>
mqttport: 1883
mqttuser: ***
mqttpass: ***
mqttlog: true
lockena: true
mqttpath: nuki
openerena: false
mqttoppath:
maxkpad:
opmaxkpad:
mqttca:
mqttcrt:
mqttkey:
hassdiscovery: homeassistant
dhcpena: true
ipaddr:
ipsub:
ipgtw:
dnssrv:
nwhw: 1
rssipb: 60
hostname: nukibridge
nettmout: 60
restdisc: true
rstbcn: 60
lockStInterval: 1800
configInterval: 3600
batInterval: 1800
kpInterval: 1800
kpEnabled: false
accLvl: 0
regAsApp: false
nrRetry: 0
rtryDelay: 100
crdusr:
crdpass:
pubauth: false
pubdbg: false
prdtimeout: 60
hasmac: false
macb0:
macb1:
macb2:
MQTT connected: Yes
Lock firmware version: 3.6.9
Lock hardware version: 6.11
Lock paired: Yes
Lock PIN set: Yes
Lock has door sensor: No
Lock has keypad: No
Network device: Built-in Wifi
Uptime: 14 minutesHere we go :)____________Am 18.10.2023 um 18:22 schrieb Jan-Ole Schümann @.***>:
Please post the content of the system information page (click on the firmware version)
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

MQTT is connected. I'm not sure what default code is about, but if you want to configure the PIN, set it in the NUKI Hub configuration portal.

but I tried to set a code via the default_code field in Mqtt device setting. Though, code is not asked before execution.

where did you read that field is for setting a code that will be asked when calling lock/unlock/open? or it was simply an assumption? :)

I assumed based on this Article :)Lock Entity | Home Assistant Developer Docsdevelopers.home-assistant.io___Am 19.10.2023 um 17:32 schrieb Alessandro Del Prete @.***>:
but I tried to set a code via the default_code field in Mqtt device setting. Though, code is not asked before execution.
where did you read that field is for setting a code that will be asked when calling lock/unlock/open? or it was simply an assumption? :)
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

Okay. That‘s why I am asking :)Thought its software only, like how the mqtt lock identifies itselfs. Not a code wanted by the lock :)BRAm 19.10.2023 um 19:44 schrieb Alessandro Del Prete @.***>:
I assumed based on this Article :) https://developers.home-assistant.io/docs/core/entity/lock/
"Some Locks"
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

That‘s why I am asking :)

And that's why I'm answering. :)


Some minor feedback

@mosuem I had a quick scan through the code and had some feedback:

Snippets are defined against the JSON language, so they show up for all JSON files (inc. non-ARB).
I don't think there's an easy way to define a new language (ARB) in VS Code and have it inherit the functionality from another language. It would be nice to define "ARB" but inherit everything from "JSON" and then use ARB to target snippets etc., however you can supply snippets via a completion provider which would allow you to check the file extension before providing them. I did something similar in Dart-Code so we could conditionally include Flutter snippets only if the current project is a Flutter project (see https://github.com/Dart-Code/Dart-Code/issues/350).
I was a little surprised that the extension modified my user settings file (to add an association) without any prompt or action (I cam to add something and saw an unfamiliar entry at the end of the file). I think it could be done instead by using contributes/configurationDefaults in package.json to avoid modifying the settings (example of similar in Dart-Code).
It probably doesn't matter too much for such a small extension, but currently in activationEvents is onStartupFinished which means the extension always activates. Generally it's better to only activate when it's certain the user needs the extension (for example onLanguage, or onCommand).. although, since we can't create an ARB language (without losing basic JSON support), onLanguage:json might be the best you can do.
Repository/bugs field in package.json point to https://github.com/mosum/arb-editor (I presume these should be google, but if not, is the username still a typo?)
I noticed in #1 you're using jsonc-parser.. Is the intention to support comments in these files? (I ask because VS Code has a separate "JSON with comments" language, but I'm not sure if it's safe to have comments here if other tools might be reading the files that might not accept them).
FWIW, I noticed the visitor in that parser is already providing offsets which deals with one thing discussed earlier.

fixed by #9, all other in #5.


Glow filters cause text blur in Safari

Appears to effect both Safari on iOS and on desktop. Not sure if this is fixable or a Safari bug.

Also happens with the drag shadow filter.


Linter Metrics

I wanted to share the idea of a metric based on code linter.
There's a tool on GitHub (with an article) that provides an XML output on code quality based on the linter of a C/C++ program.
I haven't used it yet, and I am just providing it for an example.

I like the idea as a code-quality metric.
What kind of requirements would we need from projects?
Is there a standard way that projects declare which formatting style they
use?

Thanks for the reply.
Regarding requirements, there is a large number of problems that can provide the FIX tag. Perhaps files must have a certain ratio of FIX tags to their length in lines - or something like that.
I think it is possible to determine programming language, or relative language, but... It would likely be difficult to determine format specification.
On the other hand, I'm not sure how much of a standard could be imposed on others to identify their format.

This was discussed during today's meeting. Seems like there is interest in
the industry for software quality metrics.

Discussed in the GMD meeting December 5 but not really a good fit at the moment.

I think we discussed about taking note about this in a new file in the chaoss/metrics repo, something like "proposed metrics that we're not currently dealing with", or something like that. If you agree, I can start a pull request there to start that file with this proposal, so that we can close this issue.

How about moving the issue to the metrics repo? We have several metric
ideas living in issues there.
On Wed, Dec 12, 2018, 07:17 Jesus M. Gonzalez-Barahona <
<EMAIL_ADDRESS>wrote:

I think we discussed about taking note about this in a new file in the
chaoss/metrics repo, something like "proposed metrics that we're not
currently dealing with", or something like that. If you agree, I can start
a pull request there to start that file with this proposal, so that we can
close this issue.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/chaoss/wg-gmd/issues/54#issuecomment-446584751, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AIc5RblvrWQFoDKAD2PSoq9Z_gVlq30Kks5u4QH4gaJpZM4Y6MzV
.

This seems to be the logical move based on the Metrics repo intending to now capture such things.


Add section about contributors and license

@friedger you don't want to add your name here too? :)

Test failures are bogus since no code changes.


Llama-2-70b - Tensor Dimension Mismatch Error for GPTQ Engine Build for 1GPU and 4GPUs

Model under test is Llama-2-70b-chat-hf.
Follow the GPTQ instructions here. Was able to generate safetensors file successfully with the following command:
 python GPTQ-for-LLaMa/llama.py ./Llama-2-70b-chat-hf c4 --wbits 4 --true-sequential --groupsize 128 --save_safetensors ./Llama-2-70b-chat-hf_gptq/llama-70b-4bit-gs128.safetensors
However, build engine failed because of the tensor dimension mismatch.
Command:
 python examples/llama/build.py --model_dir ./Llama-2-70b-chat-hf --quant_ckpt_path ./Llama-2-70b-chat-hf_gptq/llama-70b-4bit-gs128.safetensors  --dtype float16 --remove_input_padding --use_gpt_attention_plugin float16              --enable_context_fmha  --use_gemm_plugin float16  --use_weight_only  --weight_only_precision int4_gptq --per_group                 --enable_context_fmha  --output_dir ./examples/llama/out/70b/gptq_1gpu
 python examples/llama/build.py --model_dir ./Llama-2-70b-chat-hf --quant_ckpt_path ./Llama-2-70b-chat-hf_gptq/llama-70b-4bit-gs128.safetensors  --dtype float16 --remove_input_padding --use_gpt_attention_plugin float16              --enable_context_fmha  --use_gemm_plugin float16  --use_weight_only  --weight_only_precision int4_gptq --per_group                 --enable_context_fmha   --world_size 4  --tp_size 4  --output_dir ./examples/llama/out/70b/gptq_1gpu
Error message:
[11/30/2023-22:01:59] [TRT-LLM] [I] Serially build TensorRT engines. [11/30/2023-22:01:59] [TRT] [I] [MemUsageChange] Init CUDA: CPU +13, GPU +0, now: CPU 119, GPU 436 (MiB) [11/30/2023-22:02:01] [TRT] [I] [MemUsageChange] Init builder kernel library: CPU +1809, GPU +316, now: CPU 2063, GPU 752 (MiB) [11/30/2023-22:02:01] [TRT-LLM] [W] Invalid timing cache, using freshly created one [11/30/2023-22:03:53] [TRT-LLM] [I] Loading weights from groupwise GPTQ LLaMA safetensors... Traceback (most recent call last):   File "/code/tensorrt_llm/examples/llama/build.py", line 718, in <module>     build(0, args)   File "/code/tensorrt_llm/examples/llama/build.py", line 689, in build     engine = build_rank_engine(builder, builder_config, engine_name,   File "/code/tensorrt_llm/examples/llama/build.py", line 543, in build_rank_engine     load_func(tensorrt_llm_llama=tensorrt_llm_llama,   File "/code/tensorrt_llm/examples/llama/weight.py", line 923, in load_from_gptq_llama     qkv_part = torch.cat([q_part, k_part, v_part], dim=0) RuntimeError: Sizes of tensors must match except in dimension 0. Expected size 8192 but got size 1024 for tensor number 1 in the list.

same problem，any solution？

UPDATE: with the latest code (0.6.1), I was able to successfully build GPTQ engine for different GPU numbers.

Curious how the latency and throughput is for a GPTQ model if you have any figures handy?

I have not gotten the results for 70b yet, but for 7b, it is 1.5~3x boost (vs fp16) for throughput depending on batch size. Don't have latency data for now.

@taozhang9527 Hi, since it works so I will close this issue. If there is any further problems, you can update here (and I will reopen it) or open an new issue.


Sortling/layout issues when Y coordinates don't exactly match

Hi,
We've been using an old version of this gem (1.4.1) for a little while now and we are looking to upgrade to the latest version. That upgrade broke some of our specs and when looking deeper, it seems like the logic around PageLayout changed.
It might also be bad luck, but the use of the round here (for X and Y coords) will create issues when the PDF generated the texts with slightly different y coordinates.
Below is an example:

In this case, the texts in those boxes/rectangles are slightly lower than the labels from that form, causing some of those texts to be generated on another line:
Claim Number:           PHNP1610102                                     Contact:
Insured:                                                                Phone:
                         Fairfield Boys Club
Address 1:                                                              Email:
                         c/o Bejo Nanni, Treasurer

Another example:

We could monkey patch or fork the repo to make those changes, but please see below the code that we're going to be using. I can create a PR if this repo is still well maintained. Please let me know.
PageLayout
class PDF::Reader
  class PageLayout

    def to_s
      return "" if @runs.empty?
      return "" if row_count == 0
      first_run_at_new_y = nil # remembering a previous run at a new Y coordinate

      page = row_count.times.map { |i| " " * col_count }
      @runs.each do |run|
        x_pos = ((run.x - @x_offset) / col_multiplier).round
        y_ref_run = run # line added
        if first_run_at_new_y && run.similar_y_coord?(first_run_at_new_y) # line added
          y_ref_run = first_run_at_new_y # line added
        else # line added
          first_run_at_new_y = run # line added
        end # line added
        y_pos = row_count - ((y_ref_run.y - @y_offset) / row_multiplier).round # line updated
        if y_pos <= row_count && y_pos >= 0 && x_pos <= col_count && x_pos >= 0
          local_string_insert(page[y_pos-1], run.text, x_pos)
        end
      end
      interesting_rows(page).map(&:rstrip).join("\n")
    end

  end
end

TextRun
class PDF::Reader
  class TextRun

    # def <=>(other)
    #   if similar_y_coord?(other)
    #     x <=> other.x
    #   else
    #     other.y <=> y
    #   end
    # end

    def similar_y_coord?(other, threshold = nil)
      # arbitrary logic below. It could probably safely bumped to a higher number (dividing by 2 for instance)
      threshold = threshold || [self.font_size, other.font_size].min / 3
      (self.y - other.y).abs < threshold
    end

  end
end

Thank you.
EDIT: I updated the code above to properly support for catching multiple texts which could have been drawn on the following line.

Thanks for the well written report.
If you have the code in a fork for your own use, I'd love a PR that I can play with, check our spec suite, etc ❤


Support Update Mutations like count_add or count_reduce in field Int ?

I love this frameWork! But I got some problem when i tried to use it to design a online market. I can't deal with some data in mutations, such as the amount of a product.
Is there some mutations like this:
mutation {
    updateProduct(
        where: {
            id: "ID"
        },
        data: {
           salesAmount: {
               count_add: 1
           },
           stockQuantity: {
              count_reduce: 1
           }
        }
    )
}```

Thanks a lot for this feature request. There are some discussions around current work arounds for this in the Forum, and we are thinking about adding atomic operations directly into the Prisma API. You can follow this feature request to stay notified of our progress here 🙂


Bumps org-policies version

In order to use the last freestyle version, under the frees-io organization.

Thanks @dominv , I had to restart the builds because Travis have been with problems today


[MODEL] Isolate pagination libraries in tests

While debugging the problem described in https://github.com/elastic/elasticsearch-rails/issues/476 I found that when running the test suite, both Kaminari and WillPaginate are included in Elasticsearch::Model::Response.
This make tests pretty indeterministic. For instance, while running test/unit/response_pagination_kaminari_test.rb we should check assertions over responses that only have kaminari pagination enabled, but if the test is run as part of the whole suite, Elasticsearch::Model::Response will have also WillPaginate features included.
I discovered this, while running the suite after a change in the code :
    ERROR (0:00:01.981) test_: Multimodel response pagination records should return previous page and next page.
          undefined local variable or method `max_pages' for #<Elasticsearch::Model::Response::Response:0x007f8438e29050>
        @ /Users/miguel/.rvm/gems/ruby-2.1.6/gems/kaminari-0.16.3/lib/kaminari/models/page_scope_methods.rb:27:in `total_pages'
          /Users/miguel/.rvm/gems/ruby-2.1.6/gems/kaminari-0.16.3/lib/kaminari/models/page_scope_methods.rb:62:in `last_page?'
          /Users/miguel/.rvm/gems/ruby-2.1.6/gems/kaminari-0.16.3/lib/kaminari/models/page_scope_methods.rb:47:in `next_page'
          lib/elasticsearch/model/response/pagination.rb:118:in `next_page'
          test/unit/response_pagination_kaminari_test.rb:425:in `block (3 levels) in <class:ResponsePaginationKaminariTest>'
          /Users/miguel/.rvm/gems/ruby-2.1.6/gems/shoulda-context-1.2.1/lib/shoulda/context/context.rb:413:in `instance_exec'
          /Users/miguel/.rvm/gems/ruby-2.1.6/gems/shoulda-context-1.2.1/lib/shoulda/context/context.rb:413:in `block in create_test_from_should_hash'
          /Users/miguel/.rvm/rubies/ruby-2.1.6/lib/ruby/2.1.0/minitest/unit.rb:1265:in `run'
          /Users/miguel/.rvm/rubies/ruby-2.1.6/lib/ruby/2.1.0/minitest/unit.rb:884:in `_run_anything'
          /Users/miguel/.rvm/rubies/ruby-2.1.6/lib/ruby/2.1.0/minitest/unit.rb:1092:in `run_tests'
          /Users/miguel/.rvm/rubies/ruby-2.1.6/lib/ruby/2.1.0/minitest/unit.rb:1079:in `block in _run'
          /Users/miguel/.rvm/rubies/ruby-2.1.6/lib/ruby/2.1.0/minitest/unit.rb:1078:in `each'
          /Users/miguel/.rvm/rubies/ruby-2.1.6/lib/ruby/2.1.0/minitest/unit.rb:1078:in `_run'
          /Users/miguel/.rvm/rubies/ruby-2.1.6/lib/ruby/2.1.0/minitest/unit.rb:1066:in `run'
          /Users/miguel/.rvm/rubies/ruby-2.1.6/lib/ruby/2.1.0/minitest/unit.rb:802:in `block in autorun'

The test test/unit/response_pagination_kaminari_test.rb:423 was erroring due to a method missing, that was being delegated by WillPaginate (While the test is covering Kaminari pagination :rotating_light:)
However, when run in isolation, the test fails differently:
ERROR (0:00:02.074) test_: Response pagination records should return previous page and next page.
          undefined method `find' for Elasticsearch::Model::ResponsePaginationKaminariTest::ModelClass:Class
        @ lib/elasticsearch/model/adapters/default.rb:18:in `records'
          lib/elasticsearch/model/response/records.rb:58:in `method_missing'
          /Users/miguel/.rvm/gems/ruby-2.1.6/gems/kaminari-0.16.3/lib/kaminari/models/page_scope_methods.rb:27:in `total_pages'
          /Users/miguel/.rvm/gems/ruby-2.1.6/gems/kaminari-0.16.3/lib/kaminari/models/page_scope_methods.rb:62:in `last_page?'
          /Users/miguel/.rvm/gems/ruby-2.1.6/gems/kaminari-0.16.3/lib/kaminari/models/page_scope_methods.rb:47:in `next_page'
          test/unit/response_pagination_kaminari_test.rb:224:in `block (3 levels) in <class:ResponsePaginationKaminariTest>'
          /Users/miguel/.rvm/gems/ruby-2.1.6/gems/shoulda-context-1.2.1/lib/shoulda/context/context.rb:413:in `instance_exec'
          /Users/miguel/.rvm/gems/ruby-2.1.6/gems/shoulda-context-1.2.1/lib/shoulda/context/context.rb:413:in `block in create_test_from_should_hash'
          /Users/miguel/.rvm/rubies/ruby-2.1.6/lib/ruby/2.1.0/minitest/unit.rb:1265:in `run'
          /Users/miguel/.rvm/rubies/ruby-2.1.6/lib/ruby/2.1.0/minitest/unit.rb:884:in `_run_anything'
          /Users/miguel/.rvm/rubies/ruby-2.1.6/lib/ruby/2.1.0/minitest/unit.rb:1092:in `run_tests'
          /Users/miguel/.rvm/rubies/ruby-2.1.6/lib/ruby/2.1.0/minitest/unit.rb:1079:in `block in _run'
          /Users/miguel/.rvm/rubies/ruby-2.1.6/lib/ruby/2.1.0/minitest/unit.rb:1078:in `each'
          /Users/miguel/.rvm/rubies/ruby-2.1.6/lib/ruby/2.1.0/minitest/unit.rb:1078:in `_run'
          /Users/miguel/.rvm/rubies/ruby-2.1.6/lib/ruby/2.1.0/minitest/unit.rb:1066:in `run'
          /Users/miguel/.rvm/rubies/ruby-2.1.6/lib/ruby/2.1.0/minitest/unit.rb:802:in `block in autorun'

The goal of this issue is to gather ideas on how can we setup a controlled, deterministic fixture for running the tests.

Yeah, this seems really brittle. Can we do something as easy as remove_const in the test, and use gem / load to force-load the library needed for the test? (Not sure, just throwing out ideas...)

I'm going to close this as it's no longer relevant with the restructure and refactor of the tests to rspec. See commit https://github.com/elastic/elasticsearch-rails/commit/e72e02ca3eefef1e6af673456ec28b90e0e5c873


Update Armadillo

Library name
Armadillo
New version number
14.0.2
Other information that may be useful (release notes, etc...)
https://arma.sourceforge.net/download.html

Armadillo has had a minor release, it's now at 14.0.3. https://sourceforge.net/projects/arma/files/armadillo-14.0.3.tar.xz

Hey there @WangWeiLin-MV, just want to check on the status of this.  Thanks!

Nudge. Any update?


kie-maven-plugin mixes multiple Maven versions: [3.8.6, 3.8.4]

The kie-maven-plugin mixes multiple Maven versions: [3.8.6, 3.8.4].  We get this warning during the build.  We see this both in 8.45.0.Final and in 9.44.0.Final.
[WARNING]
[WARNING] Plugin [INTERNAL, EXTERNAL] validation issues were detected in following plugin(s)
[WARNING]
[WARNING]  * org.kie:kie-maven-plugin:9.44.0.Final
[WARNING]   Plugin EXTERNAL issue(s):
[WARNING]    * Plugin mixes multiple Maven versions: [3.8.6, 3.8.4]
[WARNING]
[WARNING]
[WARNING] Fix reported issues by adjusting plugin configuration or by upgrading above listed plugins. If no upgrade available, please notify plugin maintainers about reported issues.
[WARNING] For more or less details, use 'maven.plugin.validation' property with one of the values (case insensitive): [NONE, INLINE, SUMMARY, BRIEF, VERBOSE]
To reproduce add -Dmaven.plugin.validation=VERBOSE to the .mvn/maven.config and run a build using the kie-maven-plugin.

Thank you for reporting. Working on https://github.com/apache/incubator-kie-drools/pull/6004


Set transaction mode to READ on GET requests, fixes #547

Quick fix to allow connecting to read replicas.
In the future, functions like app, runWithClaims (anything that runs in the transaction) might need a bit of refactoring and have a common type (continuation passing style) like this, in order to allow composition if needed
ApiRequest -> (ApiRequest -> H.Transaction Response) -> H.Transaction Response
For now i don't think it's worth the effort

I'm going to experiment with code cleanup of this work on another branch.


Domain name has extra letter "c" in it

In the Deep Linking for SharePoint Framework Tabs section of this page, it shows this URL:
https://teamsc.microsoft.com/l/entity/<AppId>/<EntityId>?webUrl=<entityWebUrl>/<EntityName>
It looks like the first section of the domain should be teams and not teamsc

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

ID: 45541337-ecd7-5cf6-cbe4-3f0d2f39aa79
Version Independent ID: 6b4203f0-2844-06c9-eaa3-9cc03e6f76c0
Content: Create deep links - Teams
Content Source: msteams-platform/concepts/build-and-test/deep-links.md
Service: msteams
GitHub Login: @surbhigupta
Microsoft Alias: lajanuar

@surbhigupta12 - Could you please update the below content.
https://teams.microsoft.com/l/entity/<AppId>/<EntityId>?webUrl=<entityWebUrl>/<EntityName>
Contents need to be updated here: https://docs.microsoft.com/en-us/microsoftteams/platform/concepts/build-and-test/deep-links?tabs=teamsjs-v2#deep-linking-for-sharepoint-framework-tabs


UI Revamp

Hi there,
Following my chat with @bnjbvr, I'm opening this ticket to:

keep track of the corrections that are still to be made in the UI
gather feedbacks regarding the latest UI-related evolutions
collect new inputs about what should change in the current interface

The task board is currently available at https://trello.com/b/KlNwlofs. Feel free to add your suggestion here and it will be added to the board.

Hi @AwYiss ,
maybe you could add some recommendation for the styling of pages, and also for accessibility, so that when a new  feature is developped, we can integrate the guidelines in the new code. Thank you


Call on navigator on nested view crashes the app

I am trying to reuse a view created in one coordinator and place it in another one.
The problem I encounter is that the called view is shown as it should, but all actions called on the navigator (FlowNavigator<>) inside subordinate views cause the application to crashes. What's interesting is that the "first" call on the navigator seems to work as expected - the same cell that is inside the list I'm trying to display opens the dependent view without a problem - only nested navigators seem to have a problem.
App crashes due to fatal error:

No ObservableObject of type FlowNavigator found. A View.environmentObject(_:) for FlowNavigator may be missing as an ancestor of this view.

I noticed that setting the key embedInNavigationView to true fixes this error a bit, however the subordinate views are pushed directly to where the coordinator view is located and not as initially in "proper" but not quite working navigationView.
There you can find demo project with the issue:
https://github.com/sparafinski/FlowStacksDemo
And there is a short video presentig the issue:
https://github.com/johnpatrickmorgan/FlowStacks/assets/40520109/e9ae83ff-5ccb-40e0-a9c8-8607e17cdf2c

Thanks @sparafinski for the detailed reproduction. The cause as I understand it is that screens in a NavigationView inherit their environment objects from the NavigationView itself, not the previous screen that pushed them. The error is thrown here because the NavigationView in question is from the parent SecondCoordinator, not the child FirstCoordinator (which does not add its own NavigationView - since embedInNavigationView is false). Since the environment object is added in the child coordinator, not the parent, any pushed screens won't have access to a FlowNavigator<FirstTabViews>.
I'm not sure how best to work around this limitation. You should be fine if you avoid using the FlowNavigator API, instead passing closures to child views. Or you could combine FirstTabViews and SecondTabViews into a single enum used by both coordinators. I appreciate those workarounds might not be ideal though.


https://blog.ipfs.io/ = 404 ||| ipfs.io/ipns/blog.ipfs.io = 404

The Blog is down.
And sadly the blog is not up on ipfs either.

Thanks for the report! We'll investigate immediately

Resolved!


Multi-Part option of karate is adding additional charset=UTF-8 in the content-type of request

Summary: While using multi-part to upload excel file, request is adding charset as UTF-8 by default at the end of content-type section
Steps to reproduce:

url 'baseURL'
Given path 'Path of end point'
header Content-Type = 'multipart/form-data'
And multipart file fileName = {read: 'path of xlsx file', filename: 'name of the file',  contentType : 'application/vnd.openxmlformats-officedocument.spreadsheetml.sheet'}

Actual Result:
In Request Log following content type is appearing i.e. content-type: application/vnd.openxmlformats-officedocument.spreadsheetml.sheet; charset=UTF-8
By default it is adding charset=UTF-8 by default.
Expected Result:
As I am explicitly providing the contentType so I should see :content-type: application/vnd.openxmlformats-officedocument.spreadsheetml.sheet

please ask questions like this on stack overflow: https://stackoverflow.com/a/53474225/143475
if you want to re-open, follow this process: https://github.com/karatelabs/karate/wiki/How-to-Submit-an-Issue


fix(observer): should trigger notify when using set to change value of array length

What kind of change does this PR introduce? (check at least one)

[x] Bugfix
[ ] Feature
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:

[x] It's submitted to the dev branch for v2.x (or to a previous version branch), not the master branch
[ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[x] All tests are passing: https://github.com/vuejs/vue/blob/dev/.github/CONTRIBUTING.md#development-setup
[ ] New/updated tests are included

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:

Hi.@posva May i ask whats the progress of this PR？

Thanks for the PR, I didn't see it, this caveat is documented at https://vuejs.org/v2/guide/reactivity.html#For-Arrays and the recommended approach for Vue 2 is to use array.splice(newLength)


Memfile segfault for 'MapInfo File' driver

The following code segfaults Fiona. The code works with other drivers, thus I assume it is a GDAL issue.
from collections import OrderedDict
from io import BytesIO

import fiona

driver = 'MapInfo File'
schema = {'geometry': 'Point', 'properties': OrderedDict([('position', 'str')])}

with BytesIO() as fout:
    with fiona.open(fout,
                    'w',
                    driver=driver,
                    schema=schema) as c:
        pass

Confirmed with GDAL 3.1.

Oh, I see. We're calling "MITAB" "MapInfo File" in the drvsupport module. I don't remember whether that was the original name of the format driver or just an error.

@sgillies I think it is this driver: https://gdal.org/drivers/vector/mitab.html#driver-capabilities
https://github.com/OSGeo/gdal/blob/73117389a4e0c3ce530b5029c1b5dafed47c7274/gdal/ogr/ogrsf_frmts/mitab/mitab_ogr_driver.cpp#L204

The MapInfo File driver writes a vsimem file that VSIFOpenL fails to open. I don't understand why, but I'll make a change so that we raise OSError instead of crashing from the null pointer.

@rbuffat it occurs to me that the multi-file nature of the format may be involved.

The MapInfo File driver writes a vsimem file that VSIFOpenL fails to open. I don't understand why, but I'll make a change so that we raise OSError instead of crashing from the null pointer.

This is a good point. Using BytesIO() makes only sense for single file formats!

Fixed in Fiona 1.8.17


Reformat contributors.css for clarity

This seems to have some merge conflicts because we merged earlier pull request. If you have time to
solve the conflict we happily merge it :)


Missing "./decorators/custom-element" export in "lit" package

I upgraded<EMAIL_ADDRESS>to 2.0.0
and this error showed up
[vite:resolve] Missing "./decorators/custom-element" export in "lit" package
error during build:
Error: Missing "./decorators/custom-element" export in "lit" package
    at bail (/Users/jerrynim-zigbang/Desktop/lit-blog/.yarn/cache/vite-npm-2.4.4-dac9e9402c-075deeb5f9.zip/node_modules/vite/dist/node/chunks/dep-c1a9de64.js:38264:8)
    at resolve$1 (/Users/jerrynim-zigbang/Desktop/lit-blog/.yarn/cache/vite-npm-2.4.4-dac9e9402c-075deeb5f9.zip/node_modules/vite/dist/node/chunks/dep-c1a9de64.js:38340:10)
    at resolveExports (/Users/jerrynim-zigbang/Desktop/lit-blog/.yarn/cache/vite-npm-2.4.4-dac9e9402c-075deeb5f9.zip/node_modules/vite/dist/node/chunks/dep-c1a9de64.js:38769:12)
    at resolveDeepImport (/Users/jerrynim-zigbang/Desktop/lit-blog/.yarn/cache/vite-npm-2.4.4-dac9e9402c-075deeb5f9.zip/node_modules/vite/dist/node/chunks/dep-c1a9de64.js:38786:26)
    at tryNodeResolve (/Users/jerrynim-zigbang/Desktop/lit-blog/.yarn/cache/vite-npm-2.4.4-dac9e9402c-075deeb5f9.zip/node_modules/vite/dist/node/chunks/dep-c1a9de64.js:38575:11)
    at Object.resolveId (/Users/jerrynim-zigbang/Desktop/lit-blog/.yarn/cache/vite-npm-2.4.4-dac9e9402c-075deeb5f9.zip/node_modules/vite/dist/node/chunks/dep-c1a9de64.js:38459:28)
    at /Users/jerrynim-zigbang/Desktop/lit-blog/.yarn/cache/rollup-npm-2.55.1-800ab6ca2c-504f293723.zip/node_modules/rollup/dist/shared/rollup.js:20197:25
    at processTicksAndRejections (internal/process/task_queues.js:93:5)

I'm using yarn2 Berry, lit, vite

Not sure if Vite has updated their starter templates, but they do support one for Lit so maybe worth comparing against that, or otherwise provide some code (ideally a sample repo) so that everyone can see what is happening.
Probably worth reviewing the discussion in #2198 though as I think if you can provide a reproduction repo and / or coordinate with the Vite team first, it could help uncover the root cause to determine if it is a Vite issue, or Lit issue.
If it is indeed an issue with how Lit is managing export maps or something similar on the npm / package.json side, having an example with something actionable for the Lit team to review will probably yield the best outcomes here in the Lit issue tracker.  (IMO)

You need to import 'lit/decorators/custom-element.js'

thanks @justinfagnani
this occured when i use 'lit/decorators/custom-element'
so, i change it
'lit/decorators/custom-element' => 'lit/decorators/custom-element.js'
'lit/decorators/property' => 'lit/decorators/property.js'
'lit/directives/style-map' => 'lit/directives/style-map.js'

it works nice

You need to import 'lit/decorators/custom-element.js'

It works like a charm. Thanks a lot


[fix] Update is_proposing check

Motivation
This PR attempts to solve the issue where validators are stuck on an old proposal and unable to advance/participate in future rounds of consensus. The current check was preventing the round from advancing if we had a pending proposal, because we were too aggressively assuming the other validators would still sign it. The new logic will now advance the round even if we are proposing if the peer's certificate is already past our proposal round.
If the peer already has a certificate that hit availability threshold, then we can assume they won't be signing for old proposals.

I tested this branch, and this seems to work. On mainnet branch I could still reproduce the issue.


Add Collapsable Rows for Series in the Updates Tab

Describe your suggested feature
Certain series perform bulk updates periodically, leading to a congested and disorganized Updates Tab.
Incorporating a drop-down menu could be beneficial in reducing clutter in the Updates Tab.
I am basing this off of the J2K implementation but I would like to change some things.
Change 1:
I believe that if the number of chapters in the past 24 hours or per day exceeds 3, the rows should be condensed into a single row with a drop-down button. However, if the count is 3 or fewer, it should remain as is. I find a drop-down for just 2 or 3 chapters to be more of a nuisance than a convenience.
Here is a example of the issue when there are only 2 chapters:

Change 2:
I prefer the chapter display in the drop-down to keep the same sorting order as the rest of the Updates Tab. In J2K, the drop-down chapter display rearranges the order such that unread chapters are prioritized and sorted in ascending order by chapter number, while read chapters are deprioritized and sorted in descending order.
Here is a example of the issue:

Change 3:
I believe we should keep the earliest updated chapter visible in the collapsed row. However, upon expanding the drop-down, the initial row should only display the series title, with the drop-down chapter display, as mentioned in Change 2, appearing beneath it.
Kind of like this:

Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an official extension, I should be opening an issue in the extensions repository.
[X] I have updated the app to version 0.16.1.
[X] I will fill out all of the requested information in this form.

Duplicate of #44


tests: add macro warpper in header files

Description
the predefined macros do not have a wrapper,
add wrapper will make the test flexible to different run settings
the user cases is we have CFLAGS set in build time, which can enable the application with our test envrionment.
Checklist:
[ x] I have tested my changes. No regression in existing tests.
[x ] My code is Linted.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Signed-off-by: Hake Huang<EMAIL_ADDRESS>
Description

Checklist:

[ ] I have tested my changes. No regression in existing tests.
[ ] My code is Linted.

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

/bot run checks

/bot run checks


installing molecule

Hi,
I am trying to use molecule for ansible role testing and I am having problems installing it. I installed it once but by mistake I deleted it, so I am re-installing it again. After I ran pip install molecule everything seemed like it was successful. However, when I ran molecule --version , I got the following error:
molecule init test -bash: molecule: command not found
I am running OS X.

@pleegor When you run "pip list" what do you see? Is "molecule" there?

Worked with @pleegor on slack.  This issue was resolved by brew install python.  The python2.6 on El Capitan was an issue.

Per suggestion of my co-worker @tleyden I ran brew install python which solved the problem. Even though, Python interpreter does come pre-installed with OS X, Apple completely locks it down in El Capitan so pip cannot install anything. As soon as you complete brew installation you will see that which python will provide python's location as /usr/local/bin/python instead of /usr/bin

The issues with the system python issues is described here: https://github.com/pypa/pip/issues/3165#issuecomment-152570526

@tleyden 🎸

Just add /Users/${USER}/Library/Python/3.7/bin to the PATH.
~/bash_profile
export PATH=/Users/youruser/Library/Python/3.7/bin:$PATH


Non-descriptive error on type mismatch for functions of scipy.optimize using zeros.h

Many functions in scipy.optimize have an optional maxiter argument. This argument is expected to be an instance of int. Although that's in the documentation, the error when it fails isn't descriptive.
Reproducing code example:
import scipy.optimize

scipy.optimize.bisect(lambda x: x, -1, 1, maxiter=100.0)

Error message:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/lib/python3.7/site-packages/scipy/optimize/zeros.py", line 529, in bisect
    r = _zeros._bisect(f, a, b, xtol, rtol, maxiter, args, full_output, disp)
RuntimeError: Unable to parse arguments

Which says nothing about the problem and blames an internal function.
This is because
scipy/scipy/optimize/Zeros/bisect.c expects an int for iter (and maxiter maps to iter)
double
bisect(callback_type f, double xa, double xb, double xtol, double rtol,
int iter, void *func_data, scipy_zeros_info *solver_stats)

It should be checked beforehand that maxiter is an instance of int to get a more userfriendly error message such as TypeError: 10.0: expected int but got float

I suggest adding:
if not isinstance(maxiter, int):
    raise TypeError("maxiter must be an integer, not %s" %
                     type(maxiter).__name__)

After
if not isinstance(args, tuple):
    args = (args,)

For each function on scipy/optimize/zeros.py

Would it be easier to cast to an int?
maxiter = int (maxiter)
On Sat., 1 Jun. 2019, 17:29 Bocchio<EMAIL_ADDRESS>wrote:

I suggest adding:
if not isinstance(maxiter, int):
raise TypeError("maxiter must be an integer, not %s" %
type(maxiter).name)
After
if not isinstance(args, tuple):
args = (args,)
For each function on scipy/optimize/zeros.py
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/scipy/scipy/issues/10236?email_source=notifications&email_token=AAFLTVSXCDCXFXOI5HFPVSLPYIQL5A5CNFSM4HRFEWCKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWW24GA#issuecomment-497921560,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAFLTVXLXLOO65TCUMXS5JLPYIQL5ANCNFSM4HRFEWCA
.

maxiter = operator.index(maxiter).

Would it be easier to cast to an int? maxiter = int (maxiter)

I really think the function should raise an exception for cases in which maxiter is something like 10.5
This issue may arise silently because int.__truediv__ returns a float. There isn't a type annotation either. So an user searching for roots (maybe using scipy indirectly, without knowing it) might adjust the iterations manually with code along the lines:
maxiter /= 10

And at no point will the traceback inform them that what failed was that argument.

But if the coercion is done in the Scipy function then passing a float
wouldn't result in an error.
On Sat., 1 Jun. 2019, 17:57 Bocchio<EMAIL_ADDRESS>wrote:

Would it be easier to cast to an int? maxiter = int (maxiter)
I really think the function should raise an exception for cases in which
maxiter is something like 10.5
This issue may arise silently because int.truediv returns a float.
There isn't a type annotation either. So an user searching for roots (maybe
using scipy indirectly, without knowing it) might adjust the iterations
manually with code along the lines:
maxiter /= 10
And at no point will the traceback inform them that what failed was that
argument.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/scipy/scipy/issues/10236?email_source=notifications&email_token=AAFLTVWDHRHIG6FHQSG5ICLPYITWNA5CNFSM4HRFEWCKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWW3JLQ#issuecomment-497923246,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAFLTVWJ44LR7Z7PDKOAIB3PYITWNANCNFSM4HRFEWCA
.

I'm not arguing that, but in that case the documentation for maxiter should also change accordingly.
Current one just asks for an int.
maxiter : int, optional
        if convergence is not achieved in `maxiter` iterations, an error is
        raised.  Must be >= 0.

Personally I think that if a user pass something like 72.45, maybe they don't know what they are doing and the best would be to inform them their mistake.
At least python's built-in class range took that path.

This was fixed by #10907.

Yes, thanks, closing.


react npm start error

$ npm start

<EMAIL_ADDRESS>start C:\Users\Fins&Gills\my-app
react-scripts start

'Gills\my-app\node_modules.bin' is not recognized as an internal or external command,
operable program or batch file.
internal/modules/cjs/loader.js:968
throw err;
^
Error: Cannot find module 'C:\Users\react-scripts\bin\react-scripts.js'
at Function.Module._resolveFilename (internal/modules/cjs/loader.js:965:15)
at Function.Module._load (internal/modules/cjs/loader.js:841:27)
at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:71:12)
at internal/main/run_main_module.js:17:47 {
code: 'MODULE_NOT_FOUND',
requireStack: []
}
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>start: react-scripts start
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      C:\Users\Fins&Gills\AppData\Roaming\npm-cache_logs\2020-09-14T13_17_34_346Z-debug.log
This is the error i am getting . I have been trying many ways to correct it , But still nothing is really changing.
It would be a great help if anyone help me to reolve it.
Thankyou .

module is not found Error
Are you in the project directory?
try npm install first to load node modules and then npm start

Yes , I am in the project directory only. But it is showing like this only.
The react-script folder and  'react-script.js' file is still inside the project directory
Tried many ways , nothing is working.

ok.
try to update the node modules and try again this should solve the problem
On Tue, 15 Sep 2020 at 00:43, athz95<EMAIL_ADDRESS>wrote:

Yes , I am in the project directory only. But it is showing like this only.
The react-script folder and 'react-script.js' file is still inside the
project directory
Tried many ways , nothing is working.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/facebook/create-react-app/issues/9642#issuecomment-692256840,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIETYZOZX3ENBVLXD77PSWLSFZTNRANCNFSM4RLTEFDQ
.

Close this, no activity since December 25, 2020. Thank you.


Please exit with nonzero on failed builds

Like #94, but for real. cli.ts has:
const build = debounce(async function build() {
  try {
    // do build
  } catch (e) {
    process.stderr.write(e.stack);
  }
});

build().catch(e => {
  console.error(e);
  process.exit(1);
});

so any exceptions are suppressed and the catch is never executed.

Fixed in v3.19.1.


Add a style guideline - Linter

related #6
This PR will add some config files to guarantee that all developers follow the same code style guide :shipit:
The config files are:

editorconfig

prettier

eslint

Beat me to it lol. Happy Hacktoberfest!


Tesseract producing texts when specifying a language other than original language

I  am trying to detect the language in an image based on the confidence scores , generated by tesseract. I am checking for different languages in each of the images. What I am using :
text1 = pytesseract.image_to_data(full_path, lang='eng', config=tessdata_dir_config, output_type='data.frame')
text1 = pd.DataFrame(text1)
text1 = text1[text1.conf != -1]
text1 = text1[~text1.text.str.contains('-')]
text1 = text1[~text1.text.str.contains(' ')]
text1 = text1[~text1.text.str.contains("\.")]
text1 = text1[~text1.text.str.contains("[0-9]")]
text1 = text1[~text1.text.str.contains("[:,\n,/,%,|,;,!,&,(,),\*,\",\?,\',_,=,é,$,@,#]")]

text2 = pytesseract.image_to_data(full_path, lang='hin', config=tessdata_dir_config, output_type='data.frame')
text2 = pd.DataFrame(text2)
text2 = text2[text2.conf != -1]
text2 = text2[~text2.text.str.contains('-')]
text2 = text2[~text2.text.str.contains(' ')]
text2 = text2[~text2.text.str.contains("\.")]
text2 = text2[~text2.text.str.contains("[0-9]")]
text2 = text2[~text2.text.str.contains("[:,\n,/,%,|,;,!,&,(,),\*,\",\?,\',_,=,é,$,@,#]")]

For English it is generating texts and the respective confidence scores but when I am passing specifying Hindi as the language , I am also getting texts and confidence scores for Hindi ,  whereas the image has only English language in it.
Here is the image : sample
What am I doing wrong? 😞

I am trying to detect the language in an image based on the confidence scores.

That won't work. Nearly all models can recognize English text.

Tesseract does not have a function to detect the language of the text in the image.


Gender neutral updates per #31443

@ryandcoates : Thanks for your contribution! The author, @genlin, has been notified to review your proposed change.

Cancelling and attaching to a future PR due to branching fun!


Upd standard

Fixes a pair of bugs

I tested this last time and it seemed to pass, but obviously I missed something. Would be good for @gittyupagain to test.


Add Ed25519 pubkey in oxend ping

A few times someone has restarted their oxend without restarting lokinet and storage server, but both of those only get the service node private keys at startup, and so if you change keys, it is quite easy to end up with Lokinet & Storage server running with the wrong keys.
The problem is that this isn't easy to notice because oxend keeps sending pings, and only several hours later (once storage and/or lokinet checks fail) does this result in a decommission on the network.
I'd like to fix it by adding the service node ed25519 pubkey into the ping to oxend, so that oxend can verify and, if wrong, reject the ping. This, in turn, will make it stop sending proofs, which will usually get noticed quickly by an admin -- and if they check oxend-node logs, they will see prominent errors about lokinet or SS pings having the wrong pubkey.
(Same issue as https://github.com/oxen-io/lokinet/issues/1892, but for storage server)

addressed in #449


Adds command spo file checkout undo

Closes #4926
@milanholemans I know that we discussed that we would first to the single verb commands, and then the multiple verb commands, but I think that this one has to be underneath the file checkout one, as this is directly related. If you would like to have this changed, feel free to tell me.

Sure, I think it makes sense here to keep all related commands together.

@milanholemans Thanks for the great review. Could you please have another look?

@milanholemans Thanks for the great review. Could you please have another look?

Definitely, I'll try to have another look ASAP.

Merged manually, thank you!


Station Block Enhancements Pack

Reverse direction mode
New option on track switch box and trigger to toggle track's move mode, either one-way forward <-> one-way backward or force forward <-> force backward. Works with global flags.
Demo: https://discord.com/channels/403698615446536203/908809001834274887/1209521637637095464
Dynamic Routing
New option on station block track. When enabled, force forward/backward tracks will recalculate their outgoing direction every time track switch on/off is toggled. This makes it possible to create more complex automated junctions for station blocks.
Demo: https://discord.com/channels/835962669453410354/1129908744009035817/1209619626892201995
Station Block Volume Control
New option on station block. "volume" is a float between 0 and 1 that controls the volume of the station block SFX. Lowering the volume can improve the player experience in rooms with lots of auto-looping station blocks which is quite noisy by default.

Merging now, thanks a lot!


Cow: Watch medium.com/@multiwooduae8

Cow requests the watch of the watch_keyword medium\.com/@multiwooduae8. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

medium\.com/@multiwooduae8 has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by tripleee in Charcoal HQ


Implement SPICE-0009 External Readers

SPICE-0009

Add EvaluatorOptions.ExternalModuleReaders and EvaluatorOptions.ExternalResourceReaders.
Add ExternalReaderRuntime to host the child process side of the external reader workflow.

Thanks @bioball!

I am thinking that the external reader stuff should be moved to a sub-package. This is because it is quite a different use-case; if you are just using Pkl as application config, the external reader stuff is not relevant to you. This means that we'll need to make atomicBool, atomicRandom public. I think we can just do that and move them into the internal package.

I'm not sure how critical this is. If a client application is not using this code it gets eliminated at compile time, whether or not it's in a different package, so this isn't additional cost to users of other pkl-go functionality. Given that the ExternalReaderClient code is just a single file, I'm not sure the refactor is worth the payoff here. If there is interest in more major refactoring, it might be worth considering three packages: core/shared, external reader client, and evaluator client (as an application only using external reader stuff wouldn't need any evaluator code).

It'd be good to add some tests for external readers. Maybe we can have a test fixture external reader that we build. And maybe we can use a //go:generate command to build that fixture?

I'll need to chew on this a little. I would want to ensure this only gets built when testing against Pkl 0.27, and it doesn't looks like pkl-go is set up to test against 0.27 at all yet.

I'm not sure how critical this is. If a client application is not using this code it gets eliminated at compile time, whether or not it's in a different package, so this isn't additional cost to users of other pkl-go functionality. Given that the ExternalReaderClient code is just a single file, I'm not sure the refactor is worth the payoff here. If there is interest in more major refactoring, it might be worth considering three packages: core/shared, external reader client, and evaluator client (as an application only using external reader stuff wouldn't need any evaluator code).

My main concern here is in polluting the namespace. Right now, the methods exported on the pkl namespace only concern using Pkl directly. Creating an external reader runtime is somewhat of a different use-case, and I wonder if we'd end up confusing users who are new to pkl-go if we add that stuff to the pkl package (e.g. what is this for? is this important to me?)

I'll need to chew on this a little. I would want to ensure this only gets built when testing against Pkl 0.27, and it doesn't looks like pkl-go is set up to test against 0.27 at all yet.

Yeah; this is non-blocking.


Module 2 exercises [Chapman Owusu-Sekyere]

Please "drag & drop" exercise 1 pdf file below.

Please "drag & drop" exercise 2 pdf file below.

Please "drag & drop" exercise 3 pdf file below.

Please "drag & drop" exercise 4 pdf file below.

Model 1 Assing 2 (4).pdf

Please UPDATE THE TITLE of this "Issue" in the title bar above with your name.

If you haven't done so, please go back to Module 2 and make sure to take
the post-training survey. Participating in surveys is required for receiving
the "GALUP Certificate" by the University of Florida.

Well done, Chapman. You clearly grasped the material outlined in Module 2 and successfully applied it to these exercises. Thank you for your participation!


overwrite and append mode behavior

Transcribing/Paraphrasing feedback from a separate venue.
When using lance.write_dataset it's not obvious whether the dataset already exists. The current semantics are:
create - if it exists then raises an error, otherwise writes new dataset folder
overwrite - if it exists then it writes a new version, otherwise raises error
append - if it exists then it writes a new version, otherwise raises error

A user reported that "if the file does not exist, I would expect the function to create it, but instead, I see an error that the manifest doesn't exist".
An additional suggestion was "For append, I'd say issue a warning then make a new one"

Can you use write_dataset to concatenate two Lance files, including merging vector indexes? I've got a few hundreds Lance files with ~1M vectors each. Would be great to be able to merge them without having to recompute the index

Current state:

very excited for both of those. The index merging enables large-scale vector search while still building the lance files in a separate nodes / machines / processes.
And, I think, not having to call to_table() means I could look up individual rows across the whole partitioned dataset.... very exciting!

oh but if you have several hundred, we may want to consider a different API for that.
e.g., we could have each process write their lance data and the index to the same destination, and then at the very end, re-write the manifest altogether.
i'll start a separate issue to discuss this

If we use write append dataset_new on dataset_v0, the value of dataset_v1 should be dataset_v0 union dataset_new right? Does our current implementation have a schema check before the union?

closed via #690


MemcpyContinueCopying hint is wrongly implemented

the hint is
	//> n -= 1
        //> ids.continue_copying = 1 if n > 0 else 0

n is a value stored in the scope.
Currently we do
		newN := new(f.Element)
			newN = newN.Sub(n.(*f.Element), &utils.FeltOne)

			if err := ctx.ScopeManager.AssignVariable("n", newN); err != nil {
				return err
			}

this is not the expected behavior as we store a pointer to n, instead of n in scope

Hello @TAdev0! I worked on adding that hint, I think I have enough context to solve the issue, so I will be happy to help!

Hey @danielcdz , someone else is already working on it!


Bundle does not support collection and entries require optional 'request' attribute

Describe the bug
I encountered the following issues that might be of interest to comply to the FHIR R4 standard:

Issue: FHIR-Server does only support Bundle.type 'batch' or 'transaction':

https://github.com/LinuxForHealth/FHIR/blob/b808fd75520dad72ba5cdb6ee3a5ada92b163591/fhir-server/src/main/java/com/ibm/fhir/server/util/FHIRRestHelper.java#L1562
but allows various types:
(from https://www.hl7.org/fhir/codesystem-bundle-type.html)

document
message
transaction
transaction-response
batch
batch-response
history
searchset
collection (desired by me)

Issue: Field 'request' is required but is in fact optional:

https://github.com/LinuxForHealth/FHIR/blob/b808fd75520dad72ba5cdb6ee3a5ada92b163591/fhir-server/src/main/java/com/ibm/fhir/server/util/FHIRRestHelper.java#L1607
(from https://www.hl7.org/fhir/bundle-definitions.html#Bundle.entry.request)
EDIT: It seems that request is required if the Bundle is used in batch/transaction-mode (not sure whether it is really required though). Allowing/implementing support for Bundle-objects of type collection probably requires the validation to keep the request attribute optional.

Environment
Which version of IBM FHIR Server? Tag 4.11.1
To Reproduce
Steps to reproduce the behavior:
FHIR Bundle type

Setup a FHIR-Server in version (e.g.) 4.11.1
Send Bundle to the server:

The following Bundle is not accepted but should be accepted:

curl -lk -v -u 'fhiruser:change-password' \
    -H 'Content-Type: application/json' \
    -X POST --data-binary @- \
    https://<your-fhir-server>/fhir-server/api/v4/ <<EOF
{
  "resourceType": "Bundle",
  "type": "collection",
  "entry": []
}
EOF

Response from the server (400):
{
    "resourceType": "OperationOutcome",
    "id": "ac-13-0-4-a70a92e5-fba3-4c45-bbb1-7ff30b33505b",
    "issue": [
        {
            "severity": "fatal",
            "code": "value",
            "details": {
                "text": "Bundle.type must be either 'batch' or 'transaction'."
            }
        }
    ]
}

FHIR Bundle entry 'request'

Setup a FHIR-Server in version (e.g.) 4.11.1
Send Bundle to the server:

The following request works because it includes:

type = batch
entry[].request has valid data (EDIT: required?/optional? due to batch type)

curl -lk -v -u 'fhiruser:change-password' \
    -H 'Content-Type: application/json' \
    -X POST --data-binary @- \
    https://<your-fhir-server>/fhir-server/api/v4/ <<EOF
{
  "resourceType": "Bundle",
  "type": "batch",
  "entry": [
    {
      "fullUrl": "https://redcap-server/some-questionaire-path-1",
      "resource": {
        "resourceType": "Questionnaire",
        "identifier": [
          {
            "system": "https://.../...",
            "value": "questions-1"
          }
        ],
        "item": [
          {
            "answerOption": [
              {
                "valueCoding": {
                  "code": "1",
                  "display": "item-1-1",
                  "system": "https://.../..."
                }
              },
              {
                "valueCoding": {
                  "code": "2",
                  "display": "item-1-2",
                  "system": "https://.../..."
                }
              }
            ],
            "linkId": "item-1",
            "required": true,
            "text": "item-1-text",
            "type": "choice"
          },
          {
            "answerOption": [
              {
                "valueCoding": {
                  "code": "1",
                  "display": "item-2-1",
                  "system": "https://.../..."
                }
              },
              {
                "valueCoding": {
                  "code": "2",
                  "display": "item-2-2",
                  "system": "https://.../..."
                }
              }
            ],
            "linkId": "item-2",
            "required": true,
            "text": "item-2-text",
            "type": "choice"
          }
        ],
        "name": "questions-1-name",
        "status": "draft",
        "title": "questions-1-title",
        "url": "https://.../..."
      },
      "request": {
        "method": "POST",
        "url": "Questionnaire"
      }
    },
    {
      "fullUrl": "https://redcap-server/some-questionaire-response-path-1",
      "resource": {
        "resourceType": "QuestionnaireResponse",
        "identifier": {
          "system": "https://.../...",
          "value": "QuestionnaireResponse/some-value"
        },
        "item": [
          {
            "answer": [
              {
                "valueCoding": {
                  "code": "1",
                  "display": "...",
                  "system": "https://.../..."
                }
              }
            ],
            "linkId": "item-1",
            "text": "item-1-text:"
          },
          {
            "answer": [
              {
                "valueCoding": {
                  "code": "2",
                  "display": "...",
                  "system": "https://.../..."
                }
              }
            ],
            "linkId": "item-2",
            "text": "item-2-text"
          }
        ],
        "status": "completed",
        "questionnaire": "https://.../..."
      },
      "request": {
        "method": "POST",
        "url": "QuestionnaireResponse"
      }
    }
  ]
}
EOF

The following request fails because the optional request entry is missing, but is expected to work (EDIT) but is in fact required?/optional? for batch mode):

curl -lk -v -u 'fhiruser:change-password' \
    -H 'Content-Type: application/json' \
    -X POST --data-binary @- \
    https://<your-fhir-server>/fhir-server/api/v4/ <<EOF
{
  "resourceType": "Bundle",
  "type": "batch",
  "entry": [
    {
      "fullUrl": "https://redcap-server/some-questionaire-path-1",
      "resource": {
        "resourceType": "Questionnaire",
        "identifier": [
          {
            "system": "https://.../...",
            "value": "questions-1"
          }
        ],
        "item": [
          {
            "answerOption": [
              {
                "valueCoding": {
                  "code": "1",
                  "display": "item-1-1",
                  "system": "https://.../..."
                }
              },
              {
                "valueCoding": {
                  "code": "2",
                  "display": "item-1-2",
                  "system": "https://.../..."
                }
              }
            ],
            "linkId": "item-1",
            "required": true,
            "text": "item-1-text",
            "type": "choice"
          },
          {
            "answerOption": [
              {
                "valueCoding": {
                  "code": "1",
                  "display": "item-2-1",
                  "system": "https://.../..."
                }
              },
              {
                "valueCoding": {
                  "code": "2",
                  "display": "item-2-2",
                  "system": "https://.../..."
                }
              }
            ],
            "linkId": "item-2",
            "required": true,
            "text": "item-2-text",
            "type": "choice"
          }
        ],
        "name": "questions-1-name",
        "status": "draft",
        "title": "questions-1-title",
        "url": "https://.../..."
      }
    },
    {
      "fullUrl": "https://redcap-server/some-questionaire-response-path-1",
      "resource": {
        "resourceType": "QuestionnaireResponse",
        "identifier": {
          "system": "https://.../...",
          "value": "QuestionnaireResponse/some-value"
        },
        "item": [
          {
            "answer": [
              {
                "valueCoding": {
                  "code": "1",
                  "display": "...",
                  "system": "https://.../..."
                }
              }
            ],
            "linkId": "item-1",
            "text": "item-1-text:"
          },
          {
            "answer": [
              {
                "valueCoding": {
                  "code": "2",
                  "display": "...",
                  "system": "https://.../..."
                }
              }
            ],
            "linkId": "item-2",
            "text": "item-2-text"
          }
        ],
        "status": "completed",
        "questionnaire": "https://.../..."
      }
    }
  ]
}
EOF

Combined Issues
The final following request fails due to unsupported type and unsupported optional request (EDIT: request attribute due to batch type?), but should work:

curl -lk -v -u 'fhiruser:change-password' \
    -H 'Content-Type: application/json' \
    -X POST --data-binary @- \
    https://<your-fhir-server>/fhir-server/api/v4/ <<EOF
{
  "resourceType": "Bundle",
  "type": "collection",
  "entry": [
    {
      "fullUrl": "https://redcap-server/some-questionaire-path-1",
      "resource": {
        "resourceType": "Questionnaire",
        "identifier": [
          {
            "system": "https://.../...",
            "value": "questions-1"
          }
        ],
        "item": [
          {
            "answerOption": [
              {
                "valueCoding": {
                  "code": "1",
                  "display": "item-1-1",
                  "system": "https://.../..."
                }
              },
              {
                "valueCoding": {
                  "code": "2",
                  "display": "item-1-2",
                  "system": "https://.../..."
                }
              }
            ],
            "linkId": "item-1",
            "required": true,
            "text": "item-1-text",
            "type": "choice"
          },
          {
            "answerOption": [
              {
                "valueCoding": {
                  "code": "1",
                  "display": "item-2-1",
                  "system": "https://.../..."
                }
              },
              {
                "valueCoding": {
                  "code": "2",
                  "display": "item-2-2",
                  "system": "https://.../..."
                }
              }
            ],
            "linkId": "item-2",
            "required": true,
            "text": "item-2-text",
            "type": "choice"
          }
        ],
        "name": "questions-1-name",
        "status": "draft",
        "title": "questions-1-title",
        "url": "https://.../..."
      }
    },
    {
      "fullUrl": "https://redcap-server/some-questionaire-response-path-1",
      "resource": {
        "resourceType": "QuestionnaireResponse",
        "identifier": {
          "system": "https://.../...",
          "value": "QuestionnaireResponse/some-value"
        },
        "item": [
          {
            "answer": [
              {
                "valueCoding": {
                  "code": "1",
                  "display": "...",
                  "system": "https://.../..."
                }
              }
            ],
            "linkId": "item-1",
            "text": "item-1-text:"
          },
          {
            "answer": [
              {
                "valueCoding": {
                  "code": "2",
                  "display": "...",
                  "system": "https://.../..."
                }
              }
            ],
            "linkId": "item-2",
            "text": "item-2-text"
          }
        ],
        "status": "completed",
        "questionnaire": "https://.../..."
      }
    }
  ]
}
EOF

Expected behavior
Obviously, I'd like to see the combined request to be accepted and processed by the FHIR server.
Additional context
Feel free to tell me in case of misconceptions or mistakes by me.
Thank you for your support and your work on the FHIR server and its software utilities. :rocket:

It seems that the combined response is successful if the request is sent to the "/Bundle" FHIR endpoint.

curl -lk -v -u 'fhiruser:change-password' \
    -H 'Content-Type: application/json' \
    -X POST --data-binary @- \
    https://<your-fhir-server>/fhir-server/api/v4/Bundle <<EOF
{
  "resourceType": "Bundle",
  "type": "collection",
  "entry": [
    {
      "fullUrl": "https://redcap-server/some-questionaire-path-1",
      "resource": {
        "resourceType": "Questionnaire",
        "identifier": [
          {
            "system": "https://.../...",
            "value": "questions-1"
          }
        ],
        "item": [
          {
            "answerOption": [
              {
                "valueCoding": {
                  "code": "1",
                  "display": "item-1-1",
                  "system": "https://.../..."
                }
              },
              {
                "valueCoding": {
                  "code": "2",
                  "display": "item-1-2",
                  "system": "https://.../..."
                }
              }
            ],
            "linkId": "item-1",
            "required": true,
            "text": "item-1-text",
            "type": "choice"
          },
          {
            "answerOption": [
              {
                "valueCoding": {
                  "code": "1",
                  "display": "item-2-1",
                  "system": "https://.../..."
                }
              },
              {
                "valueCoding": {
                  "code": "2",
                  "display": "item-2-2",
                  "system": "https://.../..."
                }
              }
            ],
            "linkId": "item-2",
            "required": true,
            "text": "item-2-text",
            "type": "choice"
          }
        ],
        "name": "questions-1-name",
        "status": "draft",
        "title": "questions-1-title",
        "url": "https://.../..."
      }
    },
    {
      "fullUrl": "https://redcap-server/some-questionaire-response-path-1",
      "resource": {
        "resourceType": "QuestionnaireResponse",
        "identifier": {
          "system": "https://.../...",
          "value": "QuestionnaireResponse/some-value"
        },
        "item": [
          {
            "answer": [
              {
                "valueCoding": {
                  "code": "1",
                  "display": "...",
                  "system": "https://.../..."
                }
              }
            ],
            "linkId": "item-1",
            "text": "item-1-text:"
          },
          {
            "answer": [
              {
                "valueCoding": {
                  "code": "2",
                  "display": "...",
                  "system": "https://.../..."
                }
              }
            ],
            "linkId": "item-2",
            "text": "item-2-text"
          }
        ],
        "status": "completed",
        "questionnaire": "https://.../..."
      }
    }
  ]
}
EOF

I suppose that the issue was to use the non-Bundle-related, generic "/"-endpoint and the phenomena as described above is a valid behavior and not a bug.
Therefore I close this issue and apologize for any inconvenience.


JENKINS-33782: Add support for git-chooser-alternative to the scm helper's StrategyContext

See also JENKINS-33782.

[x] Add an alternative function to StrategyContext.
[x] Update release notes
[x] Add an example to scm_context.groovy
[x] Add a spec for the new alternative function

I've added specs, improved the release notes, and rebased. Ready for review now. :+1:

Thank you for this pull request! Please check this document for how the Jenkins project handles pull requests.


warnings due to wrong types in js-controller 3.3

Hi,
with the new js-controller there are at least two warnings in the log due to wrong datatypes 👍
regards
E
`

hue-extended.1
2021-04-30 13:27:07.413
warn
State value to set for "hue-extended.1.sensors.003-window_esszimmer.state.lux" has wrong type "number" but has to be "string"

hue-extended.1
2021-04-30 13:27:07.413
warn
State value to set for "hue-extended.1.sensors.003-window_esszimmer.state.lightlevel" has wrong type "number" but has to be "string"

`

Same issues here. MEssages like:
State value to set for "hue-extended.0.resourcelinks.peter_sensor_dg.classid" has to be type "string" but received type "number"

Same here


CTS Unit Testing

Contains all the unit tests for the get methods (multiple files, directories) and the modify methods I created (rename). each set of tests goes through the necessary success cases and the failure cases. Unit tests not needing a client for testing (because it would fail before needing the client) don't actually create a ftp connection in order to prevent the server from rejecting requests do to overuse. all test have been ran and are confirmed to pass at the current state. Also in this PR are enhancements for the get multiple, and get directory commands. before you have to pass in a fully qualified file path (root\remotefoldername\filename.txt)
which is not exactly how the commands should work so I enhanced these methods to utilize the FilePath object which stores what remote and local directories we currently are in and now accept the user just entering a file name while "behind the scenes" the remote path is prepended to successfully execute a download of a file or directory. Functionality tested and working.

Approved. Looks good. Squashing.


Instagram post not downloadable

Instagram post not downloadable  just becuase when reading jsoup gives excpetion
http error fetching url
status 404
please resolve this issue

Yes, as it says only profile downloader


Change in behaviour for DK2 VR headsets?

Hi there
Love the library, we use it and the polyfill extensively. I have leveraged it here:
https://github.com/MetaCDN/videojs-vr
I have noticed a change in behaviour in recent versions for the DK2 which is a bit of a negative for us.
Previous versions (< 1.2) were able to 'cast' directly to the Oculus Rift DK2 in direct rendering mode on chromium and firefox nightly w/ VR enabled, which was fantastic. You hit the VR button, a warning came up like it was a native app at the beginning, and it worked & looked great.
e.g. try this link with a DK2 plugged in (I'm running Oculus SDK 0.8, Windows 10,  chromium vr build or firefox nightly w/ VR enabled):
https://s3-ap-southeast-2.amazonaws.com/quicktest-metacdn/vr3.html
The new 0.2.0 behaviour is fantastic with cardboard but not great with DK2.  At one point I was getting cardboard instructions on desktop but I believe this has been resolved in recent commits.
This is with 0.2.0 of the new libs
https://stage.metacdn.com/r/v/vorjbrr/GKIb3Gc/embed?cachebust
The VR button now does side by side rendering on the main monitor. I presume this might still work in extended mode but most folks have moved (really, have been forced to move via SDK changes) to direct mode. Is there any way to get this behaviour back?
cheers
James

It's a bug. I've fixed it in my pull request.


[Bug][Array] cast array element to same type

Proposed changes
Issue Number: close #10978
Problem Summary:
Describe the overview of changes.
Checklist(Required)

Does it affect the original behavior: (Yes/No/I Don't know)
Has unit tests been added: (Yes/No/No Need)
Has document been added or modified: (Yes/No/No Need)
Does it need to update dependencies: (Yes/No)
Are there any changes that cannot be rolled back: (Yes/No)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

LGTM

FE UT need to fix

LGTM


Integrate in docker-compose and document my-pubblic-assets

Sharing my-pubblic-assets in the frontend build is a simpler option for users that integrate a remote bucket to serve custom icons.
Now that we changes YAP to frontend/backend and suggesting using docker compose to start the project, we should integrate in the docker compose the volume, review the README in the my-pubblic-assets and explain a user how to use personal assets in th demo.

I modified the current command in frontend Dockerfile to add the VOLUME instead of in the docker-compose.
Previously we are using wrong directory that's why the command is not working. I just changed it to the correct one and it works.

Also updated the README under my-public-assets and modified the config-test-pictures.json file to use the correct path.
For testing, tried adding more assets under my-public-assets and using it as shown below.

Update from latest commit:

I tried testing again if icons are loaded properly by cleaning up my environment and Marco's comment about the local assets being wiped off based on my previous commit is true.  I think if I just rebuild w/o wiping the data somewhat persist. That's why I was not able to test properly.

I moved away from Dockerfile and used docker-compose to add the VOLUME instead. I used a separate folder named "my-assets" to avoid removing the local "assets" where the local icons are residing.

After cleaning and rebuilding the container. Local Icons were loaded properly.

Loading the configure-test-pictures.json. Icons under "my-public-assets" was loaded successfully.

I updated the README and configure-test-pictures.json properly to align with the change


Docker container suddenly terminates

Hello, my team is currently testing Secor to ingest JSON messages to our data lake. We have not encountered any problem running the application directly on an EC2 instance, but when we tried to dockerize this application, we have faced two problems.

First, the Log4j appenders are not recognized (we are using the default settings from log4j.docker.properties)
Seconds, the application connects to Zookeeper and after a minute, more or less, it suddenly terminates without a WARN or ERROR.

Is there anything that we are forgetting to address on this image?
Dockerfile
ARG SECOR_VERSION=0.29
ARG HADOOP_VERSION=2.9.2

FROM maven:3.6.3-openjdk-11 AS builder

ARG SECOR_VERSION

WORKDIR /root
RUN \
  git clone https://github.com/pinterest/secor.git && \
  cd secor && \
  git checkout v${SECOR_VERSION} && \
  mvn -DskipTests -Pkafka-2.0.0 clean package

FROM openjdk:11-jre-slim-buster

ARG SECOR_VERSION
ARG HADOOP_VERSION

COPY --from=builder /root/secor/target/secor-${SECOR_VERSION}-bin.tar.gz /secor-${SECOR_VERSION}-bin.tar.gz
COPY log4j.docker.properties /log4j.docker.properties
COPY secor.properties /secor.properties
COPY entrypoint.sh /entrypoint.sh

RUN \
  apt update && \
  apt install -y curl && \
  rm /etc/ssl/certs/java/cacerts ; update-ca-certificates -f && \
  # Extract Secor build
  mkdir secor && \
  tar zxf secor-${SECOR_VERSION}-bin.tar.gz -C secor && \
  # Update properties files
  rm -f /secor/*.properties && \
  mv /*.properties /secor/ && \
  chmod 500 /secor/*.properties && \
  # Donwload Hadoop libraries
  curl https://downloads.apache.org/hadoop/common/hadoop-${HADOOP_VERSION}/hadoop-${HADOOP_VERSION}.tar.gz | tar zx && \
  mv hadoop-${HADOOP_VERSION} hadoop && \
  rm -rf hadoop/share/doc/ && \
  # Make Secor data directory
  mkdir /mnt/secor_data && \
  chmod +x entrypoint.sh

ENV LD_LIBRARY_PATH=${LD_LIBRARY_PATH}:/hadoop/lib/native
ENV PATH=${PATH}:/hadoop/bin

WORKDIR /secor

ENTRYPOINT ["/entrypoint.sh"]
CMD [ "java", "-Xmx512m", "--illegal-access=warn", "-ea", "-Dlog4j.configuration=/secor/log4j.docker.properties", "-Dconfig=/secor/secor.properties", "-cp", "/secor/secor-${SECOR_VERSION}.jar:/secor/*:/secor/lib/*", "com.pinterest.secor.main.ConsumerMain" ]

entrypoint.sh
#!/bin/bash
for envvar in $(printenv); do
  key=$(echo $envvar | cut -f1 -d'=')

  if grep -iq "^$key=" secor.properties
  then

    line_num=$( grep -in ^$key= secor.properties | head -n 1 | cut -f1 -d':' )
    key=$(echo $key | sed 's/[][~%^&*()_=+{}\\|;:",<.>'"'"'`/?-]/\\&/g')
    value=$(echo $envvar | cut -f2- -d'=' | sed 's/[][~%^&*()_=+{}\\|;:",<.>'"'"'`/?-]/\\&/g')

    sed -i "${line_num}s/.*/$key=$value/g" secor.properties
  elif $(echo $key | grep -iq "^secor.orc.message.schema")
  then
    printf "\n${envvar}\n" >> secor.properties
  else
    echo "Ignored environment variable: ${key}"
  fi
done

exec "$@"

logs
log4j:WARN No appenders could be found for logger (org.apache.commons.configuration.PropertiesConfiguration).
log4j:WARN Please initialize the log4j system properly.
log4j:WARN See http://logging.apache.org/log4j/1.2/faq.html#noconfig for more info.
Aug 10, 2020 5:51:38 PM com.twitter.ostrich.admin.BackgroundProcess start
INFO: Starting LatchedStatsListener
Aug 10, 2020 5:51:38 PM com.twitter.ostrich.admin.AdminHttpService start
INFO: Admin HTTP interface started on port 9998.

The log4j configuration file not be found in the classpath.  To debug log4j issues, turn on log4j debugging on command line: -Dlog4j.debug
Once log4j issue is fixed, you can turn on DEBUG level logging for kafka/zookeeper/secor in the log4j configuration file to see ZK issue (I think it probably have problems either connecting to ZK or Kafka inside docker)  You can also do a few jstack on secor to see what it was doing.

I changed my Dockerfile to the following and now it works, but I'm still investigating why the previous one did not. Also, I did not change any other settings.
  It's a bit hard to tell the diff.


Fix Issue# 66 Make it easier to visualize collected data

Following changes are implemented in this PR:

elk setup in a container with logstash starting with http input plugin
new emitter in crawler which would output every feature json in a separate input to logstash
and these can be indexed and group by namespace
Makefile changes to manage spinning up these containers at start/

Also, when you are ready, don't forget to run an autopep8 -i -a -a -a on all the files you touched.

my bad, i deleted brach by mistake. Creating new PR


Is this bug?

If this is a bug report, please fill in the following sections.
If this is a feature request, delete and describe what you would like with examples.
What was wrong?
vyper-run  name_registry.vy "register(0x12,0x000000000000000f000001000000000000000001)"
produce me assertion failed  when == and !=. I succesfully call this method from remix

Code that produced the error

registry: map(bytes[100], address)

@public
def register(name: bytes[100], owner: address)->address:
    assert self.registry[name] == ZERO_ADDRESS
    self.registry[name] = owner
    return owner

@public
@constant
def lookup(name: bytes[100]) -> address:
    return self.registry[name]
....
with call
 vyper-run  name_registry.vy "register(0x12,0x000000000000000f000001000000000000000001)"
CODE_TO_REPRODUCE

Full error output
ERROR_HERE

Expected Result
This section may be deleted if the expectation is "don't crash".
EXPECTED_RESULT

Environment
# run this:
$ python -m eth_utils

# then copy the output here:
OUTPUT_HERE

How can it be fixed?
Fill this section in if you know how this could or should be fixed.

it's like assert checks post factum

About init, there is problem vyper-run. There no name for init function (i think it was removed):
for func in abi:
    try:
        func_signature = func['name']
    except KeyError:
        # constructor and fallback functions don't have a name
        continue

This defenitely error.

Find bug with assert! There was extra call (unused)

Fixed in #20, thanks! :)

Will you update version to get it latest version through pip install?


Add instructions to override the destination directory of temporary files

Is your request for enhancement related to a problem? Please describe.
The main analysis is here: https://github.com/zowe/zowe-install-packaging/issues/3236
There's a bug up to Zowe 2.6 that makes zowe ignore overrides to the temporary directory variables.
2.7 will include the fix.
Describe the solution you'd like
Docs should describe the procedure to update the destination temporary directory in the installation and run.
There are three variables that control this behaviour now, the issue 3236 shared above is to improve this by providing only one variable.
The current ones are:
TMPDIR, TEMP_DIR and CATALINA_TMPDIR
Related doc pages
No current doc page covers this topic.
Additional context
This issue is important because customers may have reasons to update the directory to not affect other products that use the main /tmp uss directory.
We confirm that the Zowe installation creates thousands of these temporary files during normal operation.

Addressed with https://github.com/zowe/docs-site/pull/2656


Error during installation from Gallery

Getting error during Install-Package. Running ISE as admin.
PackageManagement\Install-Package : The member 'FileList' in the module manifest is not valid: Cannot find drive. A drive with the name 'O' does not exist.. Verify that a valid value is specified for this field in the
'C:\Users\admin\AppData\Local\Temp\1673533330\PsISEProjectExplorer\PsISEProjectExplorer.psd1' file.

Thanks, will fix in a minute :)

Should be fine now, please let me know if it works for you.

Cool and fast. Works fine now


ollama uasage

what are all the configurations i need to do to use ollama?

Same here on windows. I have installed the platform Ollama, as well as using "open webui" for the Ollama API service for end point. I have tried playing with the .evn config file and searched the web to no avail. Even toyed with the studio light model management option to use my local model. Perhaps its a simpler solution just starting us in the face. either way I was hoping there would be a predefined option or a comment in the .env file for local model use.

Here as well. I am running CrewAI-Studio on a Windows 10 workstation successfully. My Agents can access OpenAI just fine. However, when I try to access our Ollama running on a Linux GPU server, I either get "ERROR: Failed to get supported params: argument of type 'NoneType' is not iterable" or, if using the OpenAI interface method, "LiteLLM call failed: litellm.APIConnectionError: OllamaException - {"error":"model "llama3.1" not found, try pulling it first"}". Nothing seems to work. I can use Postman and post to the Ollama server and I get correct results. I'm sure I'm doing something wrong. We could really use the proper Ollama configuration, or suggestions on where to look! Thanks.

fixed :)

Thanks for fixing it so quickly! Works like a charm and we are off running with scissors :)


Pinned idna==2.8 to satisfy requests[security]==2.21.0.

The existing requests[security]==2.21.0 requires idna>=2.5,<2.9.
However, idna just released 2.9 a few days ago, and it broke Forseti installation on VM startup using terraform-google-forseti.
This PR pins idna to 2.8.
Related: https://github.com/forseti-security/terraform-google-forseti/issues/513

[x] I signed Google's Contributor License Agreement
[x] My code conforms to Google's python style.
[x] My PR at a minimum doesn't decrease unit-test coverage (if applicable).
[x] My PR has been functionally tested.
[x] All of the tests pass.
[x] I have submitted a corresponding PR for documentation in the forsetisecurity.org-dev branch and included this on this PR (if applicable).
[x] My documentation has been functionally tested and used (if applicable).
[ ] I have submitted a corresponding PR in the Forseti Terraform module (if applicable).

These guidelines and more can be found in our contributing guidelines.

Resolves #3655


Implement label matching logic in metric declarations

Description:
This PR extends the features implemented in #2 for MetricDeclarations by adding functionality for filtering metrics by labels using regex. This filtering logic works similarly to how it's done in Prometheus' relabel_config.
Specifically, we include a new optional label_matchers attribute under metric_declarations that defines a list of label matching rules. The surrounding metric declaration rule is applied to a given metric if the metric's labels matches any one of the label matching rules. The API for each label matcher is:
# List of label names to filter by. Their corresponding values are
# concatenated using the configured separator and matched against the configured
# regular expression.
label_names: [ - <string> ]

# (Optional) Separator placed between concatenated source label values.
separator: <string> | default = ';'

# Regex string to be used to match against values of the source labels defined in `source_labels`.
regex: <string>

Testing:
This code change was tested using new unit tests also included in this PR. This change was also tested on CW logs using a sample NGINX Prometheus workload. Given the following config:
exporters:
  awsemf:
    log_group_name: 'awscollector-test'
    region: 'us-west-2'
    log_stream_name: metric-declarations
    dimension_rollup_option: 'NoDimensionRollup'
    metric_declarations:
    - dimensions: [['Service', 'Namespace'], ['pod_name', 'container_name']]
      metric_name_selectors:
      - '^go_memstats_alloc_bytes_total$'
      label_matchers:
      - label_names: ['app_kubernetes_io_name', 'app_kubernetes_io_instance']
        regex: '^ingress-nginx;my-nginx$'

The corresponding exported metrics for go_memstats_alloc_bytes_total were:
{
    "Namespace": "eks-aoc",
    "Service": "my-nginx-ingress-nginx-controller-metrics",
    "_aws": {
        "CloudWatchMetrics": [
            {
                "Namespace": "kubernetes-service-endpoints",
                "Dimensions": [
                    [
                        "Namespace",
                        "Service"
                    ],
                    [
                        "container_name",
                        "pod_name"
                    ]
                ],
                "Metrics": [
                    {
                        "Name": "go_memstats_alloc_bytes_total",
                        "Unit": ""
                    }
                ]
            }
        ],
        "Timestamp":<PHONE_NUMBER>208
    },
    "app_kubernetes_io_component": "controller",
    "app_kubernetes_io_instance": "my-nginx",
    "app_kubernetes_io_managed_by": "Helm",
    "app_kubernetes_io_name": "ingress-nginx",
    "app_kubernetes_io_version": "0.40.2",
    "container_name": "controller",
    "go_memstats_alloc_bytes_total": 0,
    "helm_sh_chart": "ingress-nginx-3.7.1",
    "kubernetes_node": "ip-192-168-69-5.us-west-2.compute.internal",
    "pod_name": "my-nginx-ingress-nginx-controller-77d5fd6977-4zprn"
}

and
{
    "Namespace": "kube-system",
    "Service": "kube-dns",
    "container_name": "coredns",
    "eks_amazonaws_com_component": "kube-dns",
    "go_memstats_alloc_bytes_total": 0,
    "k8s_app": "kube-dns",
    "kubernetes_io_cluster_service": "true",
    "kubernetes_io_name": "CoreDNS",
    "kubernetes_node": "ip-192-168-1-121.us-west-2.compute.internal",
    "pod_name": "coredns-5946c5d67c-zndg7"
}

Note that the second log had no metrics exported because it didn't have the right labels.
Documentation:
The documentation in the README for the awsemfexporter is also updated to include this new API change.

Closing this in favour of https://github.com/open-telemetry/opentelemetry-collector-contrib/pull/1619


Update use of MemoryHandle to react to property rename

Supersedes https://github.com/dotnet/corefxlab/pull/1853
cc @dotnet/corefxlab-contrib

Superseded by https://github.com/dotnet/corefxlab/pull/1896


CATextLayer?

First, thanks for this amazing library.
This is a question though, not an issue.
Doesn't CATextLayer provide a similar functionality? I specifically mean the async drawing bit, great work!
CATextLayer has an inherited drawsAsynchronously property. The docs says:

When this property is set to true, the graphics context used to draw the layer’s contents queues drawing commands and executes them on a background thread rather than executing them synchronously. Performing these commands asynchronously can improve performance in some apps. However, you should always measure the actual performance benefits before enabling this capability.

Aside from caching, what advantage does asynchronously outputting to a CGImage have over what CATextLayer provides?
Thanks!

To be honest, I'm not that familiar w/ CATextLayer. I'd love to try using it. I wonder if we lose some rendering control? Like we still use TextKit to draw. I'm not sure this does. I definitely want to read up more on it.

I wonder if we lose some rendering control

Good point! I didn't think about that.
Another unclear thing about CATextLayer is the possibility of thread explosions. drawsAsynchronously offloads rendering to a background thread, but I don't see how can we control what queue/thread will be used. So, what will happen in case of many CATextLayers?
I prefer your current approach. ☺

Thanks for bringing up the idea!
Sent with GitHawk


Optional Constructor Promoted Property Dependencies not detected

Bug Report

Q
A

Library Version
0.3.20

PHP version
8.1

Current behavior
When I use optional properties promoted in the constructor, the dependencies of the properties are not detected.
How to reproduce
Similar to https://github.com/phparkitect/arkitect/issues/371#issuecomment-1484618340
public function __construct(public ?NotBlank $foo){}
Expected behavior
Optional Constructor Promoted Property Dependencies should be detected correctly.

Thanks @fp-cfr
We will release in next hour the new minor version with the fix of this issue!

You are amazing! Thank you so much 👍


dockerized api not returning float-values

Host:
Ubuntu 18.04.6 LTS
I have the docker container csgofloat up and running and calling  the /stats endpoint works properly.
When sending a post-request to /bulk like shown in the docs, I get the following response:
{
    "error": "You have too many pending requests",
    "code": 3,
    "status": 400
}

Also, other requests just load infinite and don't return at any time.
Furthermore, none of the endpoints except /stats works locally.
I feel like I need to install some more csgo-related dependency for the GC to be able to obtain any information, but that's just a suggestion based on the documentation:
"In order to retrieve float values for weapons in this way, you must have Steam account(s) with a copy of CS:GO."
I have also tried running the csgo container
Here are the docker-logs:

Am I missing something obvious?
Thanks in Advance

Have you setup Postgres to connect to? 5432 is the postgres port.

Hi, thanks for your quick response.
Not yet. In the docs, it is recommendet, but I didn't expect it to be necessary to work. The current 'csgofloat-db' docker-image is deprecated isn't it? Is there any info on how to set up the postgres server (db-schema) for csgofloat?

Update:
Careful if you host the repo yourself, not containerized:
I was using node version 18.x.x which led to this error where node would not connect to postgres.
I resolved this Issue by downgrading to node version v10.24.1.
I now got it up and running by spinning up a postrgres container running on port 5432 and updating the config.jslike mentioned.
It wasn't obvious for me to set it up, because I expected the data to be returned immediately. Maybe you could change the 'recommended' to 'necessary' in the documentation ?
Nevertheless, thanks for your hint and for maintaining such a nice project :)


Adds filtering of partial matches to classifications/detections

Using a wildcard character, you can match to partial classification/detection labels.
For example,
if your config is
{
  "camera": "my-cam",
  "vision": "my-cision",
  "classifications": {
    "COUNTDOWN*": 0.9
  }
}

Then images that have the classification label "COUNTDOWN: 20 s remain!" will also be saved

can you add a note to the README?

can you add a note to the README?

done


ADD Smooth Scroll Effect and transition effect in the website

When user scroll the pages then all items should be translate . For better user experience it is required

Can you show the demo first?

Visit this website
https://twogood.com.au/
like that scrolling effect

Assigned you the issue


[Bug]: Star icons do not show up paper cards and review cards

URL of the Page
No response
What happened?
As mentioned in #87, the star rating icons are hidden when no ratings are present. We want to have a placeholder to let users know that there are no ratings submitted yet, like below:

Steps to reproduce
No response
What browsers are you seeing the problem on?
No response
Code of Conduct

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

I did the implementation for the article card--next to work on the individual review cards


Question on publishing from a website to a silo

Question on publishing from a website to a silo
When bridgy publishes to a silo (say twitter), is it possible to have the address of the published tweet ? I use webmentions.
(Originally published at: https://jp.caruana.fr/notes/2021/04/16/question-on-publishing-from-a/)

Definitely! Bridgy returns JSON in response to webmentions. The url field contains the syndicated post's URL: https://brid.gy/about#response