TableException: Expected header terminator not present at position 6049

Hi all,
Somehow now that I open and insert a row in a DBF; the next time I get this exception and is no longer possible to add new records to the Table through php-xbase.  I isolated the issue to immediately after adding a new record and fill the rows with it.  Any clue about the meaning of this error?  Is my insert messing up the header of the DBF?  I used a program VrunFox.exe to do the DBF handling and with it I was able to add a new row; but not in PHP anymore

@gadget00 did you add new row by php-xbase or another application?

it was via php-xbase.  After adding the new row, I cannot longer add another one; I get this exception

I have version 1.2.1 right now in the production server; working on updating the code to 1.4.0 in my dev environment; never had this error before

@gadget00
How often do you insert new rows in file?
Can you give me the file for investigation?

Hello!  Here is a sample DBF file that just had this happened to it.  A few minutes ago I inserted a new record with the PHP code and now I cannot do it and I get the exception error
demodata.zip

@gadget00 I can't reproduce this bug. Please give me a code example where the bug occurs

heres the Code I'm using https://pastebin.com/p36a8RDT
I'm using 1.4.0; when I do the composer upgrade it doesn't go further than that version.  How can I upgrade to 1.4.1?

@gadget00 There is so many fixes since 1.2.1. Better way to solve the issue is the upgrade to 1.4.1. There was so many fixes since 1.2.1...

I just upgraded my code to the current version and forgot about this error.  Maybe this can be closed?


Improve error messages for AST node proc macros.

Not sure if this is actually a useful change, but I thought it might help?

Sorry but I'm not sure if it is required.


Added support for peer review flag in GUI, testing, and database

Test failures will be fixed tomorrow (not ready for integration yet)

I assume it's implied that I did this, but to confirm: It does save to the database (and checking it w/ selecting assignments from psql) and saves/loads properly.

@WChrisK please add translations to the other files (you can just use Google Translate here).


[VCM] Smaller/Sleeker toolbar

Description of the new feature / enhancement
I'm using a laptop and use this PowerToy a lot for calls and streams, but the toolbar takes a big chunk of the screen considering that it is only a toolbar. I always have to move the position depending on what I'm working on. It's an eyesore sometimes, and gets in the way of screenshots.
Would it be possible to have something smaller for the toolbar, or even turn it in to more of a small indicator with just the icon, considering that most people would be using the shortcut.

Scenario when this would be used?
Would be useful for most people, especially people with not that much screen real estate. It's better for people who just need to know whether they are muted or not.
Supporting information
No response

Please see if #6763 or #8297 meets your requirements.

Please see if #6763 or #8297 meets your requirements.

Possibly. #8297 seem to have work on the back end as well, as they add elements to the sys tray.
I thought it would be better just altering the existing toolbar, or allowing for customisation just for it. Removing the text and allowing for resize would go a long way.
Here's a crappy mock up

or just dot indicators.

/dup #6389


Pretrained weight cannot be downloaded

Hi there,
I am experimenting with the SixDRepNet_Detector and I am experiencing the issue that the model pretrained weight cannot be downloaded
here is the error message on Google Colab:

Thank you!

Hi,
the link still works for me. Perhaps, the server was temporally down.

Good to know thank you!


Add amber

Motivation for this change
This PR adds Amber, a search-and-replace tool for code.
Things done

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

[ ] NixOS
[x] macOS
[x] other Linux distributions

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

@GrahamcOfBorg build amber

@GrahamcOfBorg build amber

Sorry for the delay. I just squashed the last commit, I hope you don't mind
Thanks @bdesham!

@marsam Not at all! I prefer to squash/rebase whenever possible. Thank you!


Add Intl.RelativeTimeFormat API update

Target Live Date: asap

[x] This has been reviewed and approved by Mathias
[x] I have run npm test locally and all tests pass.
[x] I have added the appropriate type-something label.
[x] I've staged the site and manually verified that my content displays correctly.

CC: @petele

:tada: This has been pushed live to https://developers.google.com/web/

https://twitter.com/ChromiumDev/status/1054387117571354624 \o/


OPA wrapper module with RBAC plugin

Quick Question/Feedback
Question or Feedback:
Hello. Just want to clarify. I am using RBAC plugin made by janus-idp. When I try to add OPA wrapper, Backstage fails. I want to use OPA wrapper to be able to create additional policies/permissions which I want to use with RBAC. Is it expected that app fails? In your docs, it is stated: "This plugin also requires and assumes that you have at least setup the permission framework (without any policies)".
Plugin Name:
@parsifal-m/plugin-permission-backend-module-opa-wrapper
@janus-idp/backstage-plugin-rbac-backend
Additional Details:
janus-idp RBAC plugin docs: https://github.com/janus-idp/backstage-plugins/blob/main/plugins/rbac-backend/README.md

Quick Question/Feedback
Question or Feedback: Hello. Just want to clarify. I am using RBAC plugin made by janus-idp. When I try to add OPA wrapper, Backstage fails. I want to use OPA wrapper to be able to create additional policies/permissions which I want to use with RBAC. Is it expected that app fails? In your docs, it is stated: "This plugin also requires and assumes that you have at least setup the permission framework (without any policies)".

Hey! :wave:
Thanks for stopping by! I had a quick look at the janus-idp RBAC plugin and I think the reason this is happening (based on what I understand from their docs) is that, for lack of better words the two plugins are "clashing" and both are fighting to leverage the permissions framework :sweat_smile:
So sadly I think you won't be able to use both together :cry:
The way this plugin works at a high level is that the permissions framework will just by default, forward all policy decisions to OPA.
Hope this helps? If you do want to give the plugin a try, feel free to reach out to me on here or discord (same username) :smiley:

Thanks for your clarification. Sadly indeed.


Running the validator from the elevator with different options causes an exception

From the 2.1-wd05 branch of the elevator, run both of the followiing:
cli.py /Users/rpiazza/git/stix/cti-stix-elevator/idioms-xml/issue62.xml -v 2.1 --validator-args "--version 2.1"
cli.py /Users/rpiazza/git/stix/cti-stix-elevator/idioms-xml/issue62.xml -v 2.1
The first will run and generate STIX 2.1.  The second one causes this exception:
Traceback (most recent call last):
  File "/Users/rpiazza/py-envs/python3.7/lib/python3.7/site-packages/stix2_patterns-1.1.0-py3.7.egg/stix2patterns/grammars/STIXPatternParser.py", line 1226, in propTest
    la_ = self._interp.adaptivePredict(self._input,15,self._ctx)
  File "/Users/rpiazza/py-envs/python3.7/lib/python3.7/site-packages/antlr4_python3_runtime-4.7.2-py3.7.egg/antlr4/atn/ParserATNSimulator.py", line 342, in adaptivePredict
    alt = self.execATN(dfa, s0, input, index, outerContext)
  File "/Users/rpiazza/py-envs/python3.7/lib/python3.7/site-packages/antlr4_python3_runtime-4.7.2-py3.7.egg/antlr4/atn/ParserATNSimulator.py", line 414, in execATN
    raise e
antlr4.error.Errors.NoViableAltException: None

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/Users/rpiazza/git/stix/cti-stix-elevator/stix2elevator/cli.py", line 200, in <module>
    main()
  File "/Users/rpiazza/git/stix/cti-stix-elevator/stix2elevator/cli.py", line 192, in main
    result = elevate_file(elevator_args.file_)
  File "/Users/rpiazza/git/stix/cti-stix-elevator/stix2elevator/__init__.py", line 82, in elevate_file
    validation_results = validate_stix2_string(json_string, validator_options, fn)
  File "/Users/rpiazza/git/stix/cti-stix-elevator/stix2elevator/__init__.py", line 36, in validate_stix2_string
    results = validate_string(json_string, validator_options)
  File "/Users/rpiazza/git/stix/cti-stix-validator/stix2validator/validator.py", line 486, in validate_string
    return validate(stream, options)
  File "/Users/rpiazza/git/stix/cti-stix-validator/stix2validator/validator.py", line 421, in validate
    results = validate_parsed_json(obj_json, options)
  File "/Users/rpiazza/git/stix/cti-stix-validator/stix2validator/validator.py", line 398, in validate_parsed_json
    results = validate_instance(obj_json, options)
  File "/Users/rpiazza/git/stix/cti-stix-validator/stix2validator/validator.py", line 755, in validate_instance
    warnings = [pretty_error(x, options.verbose) for x in warnings]
  File "/Users/rpiazza/git/stix/cti-stix-validator/stix2validator/validator.py", line 755, in <listcomp>
    warnings = [pretty_error(x, options.verbose) for x in warnings]
  File "/Users/rpiazza/git/stix/cti-stix-validator/stix2validator/validator.py", line 62, in _iter_errors_custom
    for err in _iter_errors_custom(obj, checks, options):
  File "/Users/rpiazza/git/stix/cti-stix-validator/stix2validator/validator.py", line 52, in _iter_errors_custom
    for x in result:
  File "/Users/rpiazza/git/stix/cti-stix-validator/stix2validator/v21/shoulds.py", line 1069, in types_strict
    p = Pattern(pattern)
  File "/Users/rpiazza/py-envs/python3.7/lib/python3.7/site-packages/stix2_patterns-1.1.0-py3.7.egg/stix2patterns/pattern.py", line 34, in __init__
    self.__parse_tree = self.__do_parse(pattern_str)
  File "/Users/rpiazza/py-envs/python3.7/lib/python3.7/site-packages/stix2_patterns-1.1.0-py3.7.egg/stix2patterns/pattern.py", line 117, in __do_parse
    real_exc)
  File "<string>", line 3, in raise_from
stix2patterns.pattern.ParseException: 1:897: no viable alternative at input 'unconverted_term:WinExecutableFileObj.exportsAND'

These two pull requests https://github.com/oasis-open/cti-pattern-validator/pull/63  and https://github.com/oasis-open/cti-pattern-validator/pull/64 seems to be unrelated.
Here is the pattern:
"[((file:hashes.MD5 = '5d8129be965fab8115eca34fc84bd7f0' OR file:hashes.'SHA-1' = '2b999e7db890cc77f0098a091de756a1803a3c2b' OR file:hashes.'SHA-256' = '2c5dd8a64437cb2dd4b6747139c61d2d7f53ab3ddedbf22df3cb01bae170715b' OR file:hashes.ssdeep = '768:mvAFYk0IOqi7RKW1RD1ZCrm82+AnbaAOdoOKL70ehP:cDIOqctz2rBmbZoa71hP') AND file:name = 'VirusShare_5d8129be965fab8115eca34fc84bd7f0' AND file:size = 40654 AND (((file:extensions.'windows-pebinary-ext'.section[*].name = '.rdata' AND file:extensions.'windows-pebinary-ext'.section[*].entropy = 7.74202363178) AND (file:extensions.'windows-pebinary-ext'.section[*].name = '.data' AND file:extensions.'windows-pebinary-ext'.section[*].entropy = 7.89204688601) AND (file:extensions.'windows-pebinary-ext'.section[*].name = '.upx' AND file:extensions.'windows-pebinary-ext'.section[*].entropy = 7.31815613066)) AND unconverted_term:WinExecutableFileObj.exports AND unconverted_term:WinExecutableFileObj.imports))]",

The AND is at the end....


Removed s.resources from podspec

Removed s.resources from podspec as it causes your source files to be added to Copy Bundle Resources build step, and only images, nib files etc should be here

A second pull request. Im performing your SwiftLinkPreview.preview() call from a background thread, and when the completion closure is executed on the main thread it causes the UI to freeze intermittantly. This pull request will keep only call completion on the main thread if preview() was called on the main thread

Thanks, @fraserscottmorrison!


Image size zoom in line-by-line view

The image is rendered at its original pixel size, so requires really zooming out before we can do anything:

Another screenshot:

This makes it hard to see the whole page at a time in the editor view, though the image to the side is still available.

The text size is controlled by:
https://github.com/shreevatsa/ambuda/blob/line-by-line/ambuda/templates/proofing/pages/editor-components.html#L243
    :style="`font-size: ${textZoom}rem`"

and the button simply changes it:
      {{ alpine_button("A+", "increaseTextSize") }}

…
  increaseTextSize() {
    this.textZoom += 0.2;
    this.saveSettings();
  },

I see in devtools that a lot of numbers need to be simultaneously changed:

Currently these numbers are set here: https://github.com/shreevatsa/ambuda/blob/8825eb385ab4c72bcebe0a0b8aa00a904f18aec8/ambuda/static/js/pm-editor/pm-editor.ts#L34-L40

How can we make these numbers dynamic? Multiply each by some ratio that is dynamically controlled? Or is there a more global way to scale?

https://stackoverflow.com/questions/17333254/scale-the-contents-of-a-div-by-a-percentage and https://stackoverflow.com/questions/26488960/zoom-vs-scale-in-css3 — looks like CSS has:

zoom (deprecated / not supported in Firefox)
transform: scale(0.5);

I tried something like 9b831b46da23285ef58b02d0572b824ca655500b but it leaves the div taking the same massive size and just the text becoming smaller:

This question (not yet read) seems to be the issue: https://stackoverflow.com/questions/10858523/css-transform-with-element-resizing

Ok, based on reading the answers to that and other questions, it seems the idea that will work (from http://jsfiddle.net/qA5Tb/9/ in the answer) is something like:

Set the width and height on both a parent (wrapper) div and the actual element (child) div. The latter will never change.
When wanting to rescale:

Set transform: scale on the element (child) div,
Compute and set the width and height of the parent div.

Yet to try this.

Starting to try now — note that trying to change parent container's width in devtools etc is tricky: your DOM edit is a DOM event that causes Prosemirror to wake up and redraw the DOM :-)
This is from the docs: https://prosemirror.net/docs/guide/#view.data_flow

Did something in a3bd53f2bff107ff69e618369bc5d8eb4b4749de (mixed with another change) — not yet automated, but hardcoding different ratios in the code seems to work, so I think I'm close.

Have something in db5b00e9fd013836f78b4cf672b473a5a8af10ab but it's not working — it's surprisingly nontrivial, or I'm not thinking clearly.

The problem is this code: https://github.com/shreevatsa/ambuda/blob/db5b00e9fd013836f78b4cf672b473a5a8af10ab/ambuda/static/js/pm-editor/pm-editor.ts#L308-L314
The fix is probably to use the second argument to the constructor, which is the view itself: https://prosemirror.net/docs/ref/#view.NodeViewConstructor

Did this in e0e0f0b41f6c6289c6a16bbaf407ae828d6b2115 but somehow the view doesn't update each time, only on alternate changes or something. Still, enough for now.

Decreased step size in 8f04ed9baaf6de7b54f7bcc94b9d86f0e4789143 — I think this can be counted as working now, though there's a bug obviously. Can close this and track the bug separately.


Spurious UART interrupts

The JH7110 uses a DesignWare UART component which has some "interesting" extra features. The spurious interrupts are probably caused by a busy interrupt generated by the UART (which is caused by writing the LCR when the chip is busy). If this interrupt is not cleared, you'll end up in an interrupt storm.
See e.g. the [Linux DesignWare UART driver] for a workaround (https://elixir.bootlin.com/linux/latest/source/drivers/tty/serial/8250/8250_dw.c).

That's very helpful, thanks for the tip! :-)

Update: We fixed the Spurious UART Interrupts by waiting for UART to be not busy before setting the Line Control Register:
https://github.com/lupyuen2/wip-pinephone-nuttx/pull/36/files#diff-f208234edbfb636de240a0fef1c85f9cecb37876d5bc91ffb759f70a1e96b1d1
Thank you so much! :-)


bug fix, 'partial update' will execute twice when generate from TypeSpec.

context:
https://dev.azure.com/azure-sdk/public/_build/results?buildId=3514795&view=logs&jobId=b70e5e73-bbb6-5567-0939-8415943fadb9&j=b70e5e73-bbb6-5567-0939-8415943fadb9&t=d31d5e2d-d8df-53bc-da94-87e19f5b2ee4
The error is due to the service enabled both customization and partial update. And currently both postprocessor and typespec plugin will execute partial update. When generating from tsp, below things happen:

partial update will be executed, which will add the existing file's customized method.
postprocessor customization executed which will add the customized method to the generated file.
postprocessor partial update will be executed, so in this partial update, there will be duplicate methods.

So currently writeToFiles does the partial update and format?

Yes, code for your reference: https://github.com/Azure/autorest.java/blob/285346eb9fd511bcea567376d4e515fe680095a6/postprocessor/src/main/java/com/azure/autorest/postprocessor/Postprocessor.java#L114


Run inline example code on CI

Run inline example code on CI to prevent bugs like #239 . This requires some configuration changes, but should be straightforward.

Addressing this in #402

Resolved by #402


Fix NBT enchantment issue on 1.13 clients

This patches a regression introduced by #22.

this is not an appropriate fix, it was missing an undefined check, not adding in the old implementation, if
nbt.simplify(boots.nbt).Enchantments

is undefined,
boots.nbt.value.Enchantments

will be as well

@IdanHo Sorry about that. I hadn't investigated the code in detail and had assumed that the two were different. I've changed it to just check whether enchantments is undefined.

that will indeed fix the crash, but im wondering why on 1.14+ enchantments are always there, even if there are none applied, and on 1.13- there are not, i wonder if its called "Enchantments" on older versions, or if it was changed to it on 1.14. do you mind console.loging the output of simplify on the version youre on? (i think you said 1.8.9)

This is the output right before the crash happens:
{ Unbreakable: 1, display: { color: 65535 } }

The boots aren't enchanted. To test the NBT data when they are enchanted I added Protection 4 to them.
{
  ench: [ { lvl: 4, id: 0 } ],
  Unbreakable: 1,
  display: { color: 65535 }
}

So it looks like it was called ench in 1.8, and in 1.13 it was renamed to Enchantments. What this suggests is that neither of the two versions ever worked on 1.12- servers, but the old code checked whether the tag existed before accessing it.
I pushed another fix; I think this should cover all the cases.

I may have found another case where enchantments won't work as expected in mineflayer-pathfinder.
Do you think it would make more sense for prismarine-nbt to normalize 1.8's ench property instead of checking for it in every package?

prismarine-nbt no, but mineflayer yes ! When it receives the item from the server it should make sure the api is consistent accross versions.

If the check isn't done inside prismarine-nbt, doesn't that mean that we need duplicate Enchantments || ench checks in a lot of different PrismarineJS packages?

prismarine-nbt is a package that only knows how to encode / decode nbt. The nbt data are used for a lot of things, not only for items. mineflayer is the package in charge of interpreting what the server is sending, IMO it should be the one that does the check. If you do it in mineflayer you'll not have to do it in the other packages that take their data from mineflayer (like mineflayer-pathfinder or prismarine-physics)

prismarine-item looks like a good spot; I assume the check should be placed here?

Yes, but also in the fromNotch/toNotch methods


Option '--sdk-auth' has been deprecated and will be removed in a future release.

Under Step 2: Set up Azure environment > Activity 1, the command within the following step:
https://github.com/skills/continuous-delivery-azure/blob/653293c53dcfed4e14343eb4ed0482e761f6510e/README.md?plain=1#L137-L145
Produces the following warning:
Option '--sdk-auth' has been deprecated and will be removed in a future release.

Thanks fort he report! This should have been fixed in https://github.com/skills/deploy-to-azure/pull/78


documentation: return tag

Omit the @return tag if the method does not return anything
recorded in #69
this one just needed a severity modification in order for it to work...

might actually be a phpcs 3 bug as it appears to be haopening with every sniff we extend from another standard... will have a look at it once all the symfony sniffs are implementen, for now this works

Thanks @wickedOne!


Download Strategy Resource Dependency Missing Cases

brew doctor output
Your system is ready to brew.

brew doctor with the test script:

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

Warning: You have uncommitted modifications to Homebrew/homebrew-core.
If this is a surprise to you, then you should stash these modifications.
Stashing returns Homebrew to a pristine state but can be undone
should you later need to do so for some reason.
  cd /usr/local/Homebrew/Library/Taps/homebrew/homebrew-core && git stash -u && git clean -d -f

Uncommitted files:
  ?? Formula/test_rar.rb

Verification

[X] My "brew doctor output" above says Your system is ready to brew. and am still able to reproduce my issue.
[X] I ran brew update twice and am still able to reproduce my issue.
[X] This issue's title and/or description do not reference a single formula e.g. brew install wget. If they do, open an issue at https://github.com/Homebrew/homebrew-core/issues/new/choose instead.

brew config output
HOMEBREW_VERSION: 4.0.17-44-gf7168bf-dirty
ORIGIN: https://github.com/Homebrew/brew
HEAD: f7168bf234ee48a299b5eeb98e9895349bdcbf4a
Last commit: 3 hours ago
Core tap origin: https://github.com/Homebrew/homebrew-core
Core tap HEAD: d2b110ec2585c7d2c81f1db9def3f87bfceed552
Core tap last commit: 2 hours ago
Core tap branch: master
Core tap JSON: 10 May 01:41 UTC
HOMEBREW_PREFIX: /usr/local
HOMEBREW_CASK_OPTS: []
HOMEBREW_DISPLAY: /private/tmp/com.apple.launchd.cNgVMNyavX/org.macosforge.xquartz:0
HOMEBREW_MAKE_JOBS: 16
Homebrew Ruby: 2.6.10 => /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/bin/ruby
CPU: 16-core 64-bit kabylake
Clang: 14.0.3 build 1403
Git: 2.40.1 => /usr/local/bin/git
Curl: 7.87.0 => /usr/bin/curl
macOS: 13.3.1-x86_64
CLT: <IP_ADDRESS>.1.1679647830
Xcode: 14.3

What were you trying to do (and why)?
I was originally unsure if this was a bug or not and had opened the discussion here however, I am now fairly sure I have tracked down the issue. I originally tried to download a rar as a resource, but it errors out over missing the formulae for unrar. However, I do not want to extract the rar file, so there is no reason it should require unrar. I tried specifying the download strategy using: :nounzip however, this does not fix the issue. I have tried several different rar files unsuccessfully. I wrote a short test formula below using a rar file from archive.org.
What happened (include all command output)?
~ % cat /usr/local/Homebrew/Library/Taps/homebrew/homebrew-core/Formula/test_rar.rb
class TestRar < Formula
  desc "test"
  homepage "test"
  url "https://archive.org/download/SpaceCadet_Plus95/Space_Cadet.rar", using: :nounzip
  version "test"
  sha256 "3cc5dfd914c2ac41b03f006c7ccbb59d6f9e4c32ecfd1906e718c8e47f130f4a"
  license "test"
end

~ % HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source --verbose --debug test_rar
/usr/local/Homebrew/Library/Homebrew/brew.rb (Formulary::FormulaLoader): loading /usr/local/Homebrew/Library/Taps/homebrew/homebrew-core/Formula/test_rar.rb
Warning: No available formula with the name "unrar" (dependency of test_rar). Did you mean unar?
==> Searching for similarly named formulae and casks...
/usr/local/Homebrew/Library/Homebrew/brew.rb (Formulary::FormulaLoader): loading /usr/local/Homebrew/Library/Taps/homebrew/homebrew-core/Formula/unar.rb
/usr/local/Homebrew/Library/Homebrew/brew.rb (Cask::CaskLoader::FromDefaultTapPathLoader): loading lunar
==> Formulae
unar

To install unar, run:
  brew install unar

==> Casks
lunar

To install lunar, run:
  brew install --cask lunar

What did you expect to happen?
Specifying using: :nounzip should not require the dependencies to extract the downloaded file. Looking at dependency_collector.rb:175 it seems both CurlPostDownloadStrategy & NoUnzipCurlDownloadStrategy are missing from the resource_dep function. Changing the function to include them does seem to resolve the issue. However as I am still fairly new to the brew code base, I wanted to create an issue first to make sure my understanding of the function and my fix was correct before opening a pull request.
Any feedback is greatly appreciated.
  def resource_dep(spec, tags)
    tags << :build << :test
    strategy = spec.download_strategy

    if strategy <= HomebrewCurlDownloadStrategy
      @deps << curl_dep_if_needed(tags)
      parse_url_spec(spec.url, tags)
    elsif strategy <= NoUnzipCurlDownloadStrategy
      # allow no unzip to pass through
    elsif strategy <= CurlPostDownloadStrategy
      # allow POST to pass through
    elsif strategy <= CurlDownloadStrategy
      parse_url_spec(spec.url, tags)
    elsif strategy <= GitDownloadStrategy
      git_dep_if_needed(tags)
    elsif strategy <= SubversionDownloadStrategy
      subversion_dep_if_needed(tags)
    elsif strategy <= MercurialDownloadStrategy
      Dependency.new("mercurial", tags)
    elsif strategy <= FossilDownloadStrategy
      Dependency.new("fossil", tags)
    elsif strategy <= BazaarDownloadStrategy
      Dependency.new("breezy", tags)
    elsif strategy <= CVSDownloadStrategy
      cvs_dep_if_needed(tags)
    elsif strategy < AbstractDownloadStrategy
      # allow unknown strategies to pass through
    else
      raise TypeError, "#{strategy.inspect} is not an AbstractDownloadStrategy subclass"
    end
  end

Step-by-step reproduction instructions (by running brew commands)
Already shown in the What Happened section.

The CurlPostDownloadStrategy behaviour seems correct (it'll use the strategy <= CurlDownloadStrategy case) as it's really the same as regular CurlDownloadStrategy for the tools needed to download and extract.
But I agree that NoUnzipCurlDownloadStrategy should be special cased.
You've also pointed out another issue here that even if nounzip wasn't used, the unrar dependency is incorrect as this doesn't exist anymore - so we should also correct that to depend on something else to extract the rar instead for formulae that need it.

Yeah looking through all the previous threads regarding rar I understand the licensing issues with it. However, I don't think there is really a good alternative for it after some testing I have found that while p7zip and sevenzip do extract a rar they throw ERROR: Unsupported Method for each file. unar does seem to work but is missing some functionality that doesn't make it a one to one replacement and there may also be some licensing issues. The rar cask seems like the obvious choice but I don't think we can include a cask as its not really supported. Would it just make more sense to have it throw an error or warning that says brew doesn't support extracting rar? Any feedback would be greatly appreciated.

Yes, it's a legal mess.
unar or libarchive (bsdtar, depsite the name, supports rar) are options. libarchive is quite nice as macOS ships it, but it's currently too old (needs 3.6.0 or later), but that may change in a future macOS as Apple do keep it relatively up-to-date.
I'm not sure how "good" of a support each offer in terms of historical versions, but rars seem rare enough to probably not matter.

However as I am still fairly new to the brew code base, I wanted to create an issue first to make sure my understanding of the function and my fix was correct before opening a pull request.

This seems like a reasonable first attempt, can you open a PR and we can comment there?

unar or libarchive (bsdtar, depsite the name, supports rar) are options. I'd say libarchive is the most desirable option as macOS ships it, but it's currently too old (needs 3.6.0 or later). That may change in a future macOS as Apple do keep it relatively up-to-date, which will eventually mean no dependency required for those versions.

Fine with any of these 👍🏻

I believe archiver also supports extracting rar files and has the potential advantage of having no runtime dependencies because it's written in go.  Actually if archiver supports extracting most of the archive formats we allow, I wonder if we could eventually simplify some of our unarchiving code by relying on archiver as a more universal unarchiver than the more complicated mix of tools we use now.   This is certainly not a critical improvement at the moment but might be worth keeping in mind for future development.

@danielnachun I think it'd be good to rely on a longer-running existing formula for this. Definitely don't think we want a new universal archiver.

https://github.com/Homebrew/brew/pull/15409


Document where to get a appropriate .gitignore

Copying the one from this project is not the best idea, as it contains .mps

removed ".mps" from .gitignore


Is it possible to ingest additional features (i.e. numerical, categorical, sensor inputs) alongside Images?

I want to augment my dataset with additional sensor data, is this possible & if so can you please point me in the right direction?
If not, what would you recommend?

Hi @jabbasj, can you elaborate a bit more how exactly you want to augment your dataset? I'm not sure whether you want to add this additional data as the 4th dimension to the RGB data, or you want to add on top of the existing RGB images.

Hi @jabbasj, can you elaborate a bit more how exactly you want to augment your dataset? I'm not sure whether you want to add this additional data as the 4th dimension to the RGB data, or you want to add on top of the existing RGB images.

The data I am looking to include is not really per-pixel... I want to associate many things like weather conditions, pressure readings, temperature readings, type of object in the image, etc...

Multi-modality is something we would like to work on, but unfortunately we do not have resources (time etc) to work on it yet.


Refresh access and id tokens in a React/Angular SPA

Summary
I would say that without Cognito implementing prompt=none on the /oauth2/authorize endpoint, and whilst the cognito cookie on <your_domain>.auth.<region>.amazoncognito.com/ expires after 60 minutes instead of 30 days (or what is set for the user pool) it is unsuitable to be used as an out-of-the-box solution for Single Page Applications.
Details
The recommended OAuth2 flow for a Single Page Application (SPA) is the Implicit Grant. This flow, by design, does not issue a refresh token to the web app.
There are many issues in this project around refreshing tokens that seem to be resolved by switching to Authorization Code Grant instead, which is OK for some native apps and some non-SPA web apps (e.g. where there are round trips to a server) but is unsuitable for SPA apps.
It is considered unsafe and bad practice for a web application to store a refresh token in session storage, local storage, or in javascript accessible memory as this opens up the app to many attacks which could result in the refresh token being exposed to a malicious third party which is now able to obtain ID/API tokens for the user.
The OpenID Provider (OP) usually sets an http only secure cookie on user login, and this is used for refreshing in this case the OP is awscognito and the cookie would be set on <your_domain>.auth.<region>.amazoncognito.com/. Cognito does set this cookie.
The typical process for refreshing tokens in an SPA is to call the OP's authorize endpoint with a prompt=none in the query string. This would normally occur in an embedded iframe in the SPA so as to not alter the UI and would either return new tokens which can be stored or an error, which deletes the tokens.
This is not a standard or requirement as it is up the the OpenID Provider to determine how a user is authorized, however if this is not implemented and it means that an SPA must choose to log in whenever the access token expires, or to adopt poor practices and store refresh tokens in the SPA (there is another option where we could have our own server which stores the refresh tokens and can refresh api/id tokens and deliver them to the SPA).
This partially works using cognito hosted UI, there is a cognito cookie stored when you login at <your_domain>auth.<region>.amazoncognito.com/login?client_id=<your_client_id>&response_type=token&redirect_uri=<your_redirect> and if you subsequently call <your_domain>auth.<region>.amazoncognito.com/oauth2/authorize?client_id=<your_client_id>&response_type=token&redirect_uri=<your_redirect> it will redirect with new tokens, with updated timestamps, without requiring interaction.
Unfortunately the cognito token expires after 60 minutes so it can only be used to extend the session to a maximum of 120 minutes. Also since the prompt parameter is not implemented there is no way for the iframe to get an error so it cannot be used in an iframe.
References

prompt=none behaviour, OpenId spec section <IP_ADDRESS>.  Authentication Request
AWS specs for authorization endpoint This does not detail prompt as an option and it appears to not be implemented.
Auth0 silent authentication for SPAs
Okta silent authentication for SPAs

Any thoughts on this? It seems to me that it makes implicit grant totally impractical to use.

I'd prefer if AWS allows the configuration the id_token and session cookie expiry TTL based on your own requirements.

This seems like a deal breaker for using Cognito User Pools for SPA development.

Why there is no response from aws team on this??? This is making implicit flow absolutely useless. I am not sure how @cervantek got cognito cookie to work for 8 hours.  I see only 1 hour, so you are stuck with maximum 2 hour for tokens if you do a workaround and make a call to oauth2/authorize endpoint for hosted login.

Wondering the same thing! This is a MUST feature for any real-world use.

This seems to be a vital feature for user experience. @cervantek do you have details on how to increase the cognito cookie timeout? But even that does not solve the issue.

The document that you referenced assumes that refresh tokens last indefinitely, but I believe that with Cognito User Pools you can configure the refresh token TTL. https://docs.aws.amazon.com/cognito/latest/developerguide/amazon-cognito-user-pools-using-tokens-with-identity-providers.html#amazon-cognito-user-pools-using-the-refresh-token

AWS needs to implement this properly by giving us the ability to configure cognito cookie timeout. If implemented, then using a proper Oidc JS library would get a new id_token and token periodically.

We need this feature too

I agree with @nihakue .  After talking to AWS team we went with that solution  - setting refresh token ttl to 1 day, and doing refresh token every hour with auth code flow.

Could someone from the AWS team respond to this? Came across this issue whilst trying to implement silent refresh. I am now considering moving away from Cognito to something that supports the implicit flow better for SPAs.

While this may sound (very) crazy. What if the refreshToken had a more granular TTL, such as minutes or hours. Wouldn't this get us slightly closer to a silent auth/refresh solution?

@maziarz while it might help, it's actually the wrong solution. Refresh tokens shouldn't be used in SPA apps; rather, use the session cookie controlling the refresh, e.g. cognito to use session cookie to see if session cookie is still valid and reissue another token and update session cookie if warranted.

@salmonz its not that i disagree, i ran into this problem 1.5 years ago and ended up implementing Cognito with passport.js in the back utilising secure cookies. But eventually it removed all benefits from being truly "serverless" and having low maintenance on a SPA. I do not believe that anyone has solved this issue in an elegant way, even auth0 is relying on a valid SSO session in order for this to actually work with prompt=none. If we assume a session cookie and refreshToken has the same TTL, what would the difference be if we set the storage method to the side? Eventually the refreshToken becomes the cookie, don't you agree?

@salmonz  its not that i disagree, i ran into this problem 1.5 years ago and ended up implementing Cognito with passport.js in the back utilising secure cookies. But eventually it removed all benefits from being truly "serverless" and having low maintenance on a SPA. I do not believe that anyone has solved this issue in an elegant way, even auth0 is relying on a valid SSO session in order for this to actually work with prompt=none. It does not look like Cognito team are going to introduce a cookie mechanism any time soon..

AWS - do you have an update on this issue? Seems to me that the sign-in cookie expiry should be configurable in the AWS portal for a cognito instance, or per client configuration.

Any updates??

Any progress on this?

any updates?

+1 for silent renew support on implicit grants

Sub'd, this is a deal breaker for using cognito user pools with SPA, would be nice to hear something from AWS.

AWS: please give us some sort of update on this issue? What workarounds can you propose for SPA apps, do you have any plans to deal with this? The silence is deafening!

I guess we'll have to use Auth0 again.

The typical process for refreshing tokens in an SPA is to call the OP's authorize endpoint with a prompt=none in the query string. This would normally occur in an embedded iframe in the SPA so as to not alter the UI and would either return new tokens which can be stored or an error, which deletes the tokens.

The iFrame flow does not work on the latest Safari.
Auth0 warns about it here: https://auth0.com/docs/api-auth/token-renewal-in-safari
So I guess all we have left is

there is another option where we could have our own server which stores the refresh tokens and can refresh api/id tokens and deliver them to the SPA

I agree with @nihakue . After talking to AWS team we went with that solution - setting refresh token ttl to 1 day, and doing refresh token every hour with auth code flow.

@ppasmanik Can you please elaborate?

@sebastienfi
I believe they are using the Authorization Code Grant instead of the Implicit Grant to get a code that can be exchanged for a refresh token, storing the refresh token in the SPA, and refreshing the access/id tokens hourly.
Storing the refresh token client side in a web app would normally be considered very insecure however they have limited the lifetime of the refresh token to 24 hours, which is a tradeoff they have had to make as AWS provides no silent auth.
I guess the upshot is that users would have to authenticate daily rather than hourly.

@ravenscar Thank you for mentioning me - and for these clarifications!

@ravenscarhttps://github.com/ravenscar You are correct. The cognito-auth-js library provides various APIs to get refresh token, session, etc.
It stores them in local storage. We wrote a small library that wraps amazon-cognito-auth-js and provides React components that know how to handle both types of the flows depending on configuration, perform refresh of tokens using oauth2/authorize endpoint of hosted cognito at configurable intervals (between 10 and 55 minutes depending on user roles).
The minimum refresh token ttl is 1 day for cognito pool and it is sufficient for our users.  You can set it to longer if you need. We also integrated idle timeout into that library that logs a user out after a configurable time interval.

From: Sébastien Fichot<EMAIL_ADDRESS>Sent: Tuesday, April 23, 2019 4:39 AM
To: aws/amazon-cognito-auth-js
Cc: Pasmanik, Paul; Mention
Subject: Re: [aws/amazon-cognito-auth-js] Refresh access and id tokens in a React/Angular SPA (#92)
@ravenscarhttps://github.com/ravenscar Thank you for mentioning me - and for these clarifications!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/aws/amazon-cognito-auth-js/issues/92#issuecomment-485700128, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AA2674YCPIHPLQLU5RAIQXTPR3DKTANCNFSM4EROSASA.

The information contained in this electronic transmission is intended only for the use of the recipient and may be confidential and privileged. Unauthorized use, disclosure, or reproduction is strictly prohibited and may be unlawful. If you have received this electronic transmission in error, please notify the sender immediately. In order to make sure you receive our emails with the latest news on Dante, please add<EMAIL_ADDRESS>to your contact list.

Storing secrets in local storage is the entire problem. The workarounds described are too insecure for some to consider.

Yes, storing secrets in local storage is not a good practice, however, it is questionable whether refresh token with validity limited to a set number of hours is really a secret.
You need both unexpired token and refresh token to renew a token.
If someone is able to get hold of an unexpired token, he will be able to get in.
If someone is able to get hold of both an unexpired token and refresh token, he will be able to refresh the token several times up to a refresh token expiration time.
In both cases, something or somebody needs to hack something to get hold of tokens from local storage.

From: Jeremiah Small<EMAIL_ADDRESS>Sent: Friday, May 3, 2019 7:06 PM
To: aws/amazon-cognito-auth-js
Cc: Pasmanik, Paul; Mention
Subject: Re: [aws/amazon-cognito-auth-js] Refresh access and id tokens in a React/Angular SPA (#92)
Storing secrets in local storage is the entire problem. The workarounds described are too insecure for some to consider.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/aws/amazon-cognito-auth-js/issues/92#issuecomment-489266305, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AA26745TF4AD2MUSI4NHATDPTTAODANCNFSM4EROSASA.

The information contained in this electronic transmission is intended only for the use of the recipient and may be confidential and privileged. Unauthorized use, disclosure, or reproduction is strictly prohibited and may be unlawful. If you have received this electronic transmission in error, please notify the sender immediately. In order to make sure you receive our emails with the latest news on Dante, please add<EMAIL_ADDRESS>to your contact list.

Any thoughts on this? It seems to me that it makes implicit grant totally impractical to use.

Actually, I think you mean that the auth code grant is impractical.  Only the auth code grant provides a refresh token.  Also, the implicit grant is inadvisable these days.

@ppasmanik , "You need both unexpired token and refresh token to renew a token."
Actually, that's not correct.  You need only the refresh token. See section 6 of https://tools.ietf.org/html/rfc6749,  also https://www.oauth.com/oauth2-servers/making-authenticated-requests/refreshing-an-access-token/

Actually, I think you mean that the auth code grant is impractical. Only the auth code grant provides a refresh token. Also, the implicit grant is inadvisable these days.

@leotohill not sure I follow you here. Auth code grant cannot be used securely in a SPA because it returns a refresh token which cannot be securely stored.
When implemented correctly by an auth provider implicit grant can allow refresh without the token being stored in a place accessible to javascript, this is desirable if you are not auditing every node module in your dependency tree.
Do you have a recent source saying why this is suddenly inadvisable? What is the alternative "safer" way of storing refresh tokens in an app where all javascript code cannot be trusted.

@ravenscar yes, this ietf best practice document..  The auth code grant is a more secure flow, and the refresh token can (should be) discarded, or not issued in the first place if that's an option.
Not sure i fully understand what you are saying about the auth provider "allow refresh without...", but could this work anyway, even if the token is acquired via the auth code flow?
I too find that this advice makes things challenging.  I'm thinking of setting up an server where the refresh token can be stored server-side, and the server executes the refresh operation, returning the new token to the client.  This would require a cookie-based session with the server.

@leotohill interesting read but not sure it's worth trading implicit grant headaches for CORS headaches. Maybe in 5 years if CORS allows multiple origins.
I am not sure if the silent refresh would work with auth-code flow. I can't see a reason it wouldn't, but I guess since it's optional it's up to the implementer.
Your idea seems to be re-implementing the out-of-the-box functionality of other auth providers such as Auth0. I'd advise just to use their services and forget that cognito user pools exists.
Even when you get past this hurdle you have a dozen more, such as discovering that the sign-up and log-in emails are case sensitive, so<EMAIL_ADDRESS>can't login if he signed up as<EMAIL_ADDRESS>Nothing AWS has produced has disappointed me as much as cognito user pools. Shouldn't event be in beta.

I submitted a ticket to AWS support center, which hopefully has a channel to a different Cognito team than this open-source sdk team.
AWS makes a(n arbitrary IMO) distinction between open-sourced and closed-source/backend services, so I urge everybody else to open support tickets. Particularly if you work at a large enough company that you have the top-tier of customer service or work at a mid-large company and depend on cognito.
I agree with the other sentiment that cognito is a fundamental service, a fundamentally flawed service, and the one that makes me wonder if AWS has stretched itself too thin but the alternative providers in this area all have issues of their own, too.

fyi, AWS' official client library for cognito was able to confirm that better support for SPA authentication use-cases is in the Cognito team's backlog:
https://github.com/aws-amplify/amplify-js/issues/3436#issuecomment-538400098


Customize Branch

How to custom condition node in branch?

How to custom condition node in branch?

addConditionIcon in registerNodes, and you can set showArrow true  in FlowBuilder to show default arrows


Tidy up thank you uk screen

Tidying up margins, a few text colours/fonts and add icon at the top.

Could you remove the full stop, after "Thank you for reporting", please . It's not in the design....


Probably bug-causing double line in the pip default materials/ground.rad file

Hi,
I just did a fresh test installation via pip install bifacial_radiance on a new user profile and got
"fatal - (materials\ground.rad): undefined modifier "5"\r\n'". I think to remember that I had this already several times now after setting up new profiles in the past and had to fix it always by deleting once one of the lines of modifiers for this material here:
...
#Concrete (from SMARTS) albedo 0.28
void plastic concrete
0
0
5 .364 .257 .2235 0 0
5 .863 .791 .745 0 0

#CU Boulder Beige roof (clean) albedo 0.43
...

Could you look into that?
Best regards
Leo

Hi Leo,
I am checking the file in the main  and the development  branch, and both don't have that line. I wonder if it somehow got corrupted in one of your installs, and on re-installs it is not really rewriting the wrong file?

Hi Silvana,
I checked now what is happening in the background and understood that bifacial_radiance copies the materials/ground.rad from the
\AppData\Local\Programs\Python\Python310\lib\site-packages\bifacial_radiance\data folder if the materials folder does not exists within the working directory.
I am sorry to have wasted your time here with this issue. I think I somehow messed up my ground.rad editing method. Once I deleted the double line, all is working fine but when copieng the file for the first time from the ...site-packages\bifacial_radiance... (which is indeed not corrupted, as I just saw), I have the bug, which is caused by my code. I will fix it now...
One minor suggestion: You could check for the file instead of the folder so it won't through an exception if someone empties the materials folder.
Thanks and best regards
Leo


No conflicts detected, although there are.

The files in question are python files.
What else can i say to help?
I am on OSX, if its relevant

Maybe paste what the conflicts look like along with any output in the Developer console?

Just a wild guess - do you have multiple Project Folders open in the left panel?

i might have had multiple folders open, is this an issue?

Oh! Sadly, yes. When support for multiple projects shipped, I didn't devote the time to handle it properly. There's a lot of atom.projects.getRepositories()[0] going on.
I'm tracking this one in #91 if you want to subscribe over there.
Can you attempt to reproduce with a single project open, and close the issue if you can't?

Ok, thank you for your input, next time i have a conflict i'll try to sort this out, and close the issue if i can

Thanks @smashwilson for the input and the honesty.


Changes to obs do not flush to file in backed-mode

Hi,
Sorry for creating so many issues these days.
This issue is aimed at handling changes to obs in backed mode. Specifically, I noticed that changes to obs to not get flushed to file by AnnData.file.close(). Is there a better way to ensure the object and file are synchronized? The anndata version used here is 0.8.0.
Example:
import anndata
import numpy as np
import pandas as pd

tmp_file = "tmp.h5ad"

# create a dummy file, write it to disk
adata = AnnData(X = np.random.random((10,10)),
                             obs = pd.DataFrame({"group":np.random.randint(0,2,10)}))

adata.write(tmp_file)

# load in dummy file in backed mode, add column to obs, then close file
adata_copy = anndata.read_h5ad(tmp_file, backed="r+")
adata_copy.obs["group2"] = np.random.randint(0,2,10)
adata_copy.file.close()

# read in file to check whether group2 is present
anndata.read_h5ad(tmp_file)
# AnnData object with n_obs × n_vars = 10 × 10
#   obs: 'group'

backed mode currently only has an effect on .X. This is partially because some specific semantics only work then, but also because providing a pandas like interface to a backed dataframe is hard.

Good to know! This was a "gotcha" for me - perhaps someone/I can draft a little page for the documentation explaining backed mode to clarify what it does and does not do?

Sure, I'll try and find time soon! Would you prefer a separate page or should the explanation be in the docstring of read_h5ad?

Closing this in favor of discussion at #812.


Exp Boosters not working with XP Share and XP All

Currently, RPG boosters aren't working correctly and distributing the increased xp to the other mons.

Bug on Pixelmon end, so closing.

So this means paid boosters are pretty much not going to work properly again?

It's on Pixelmon's side. Nothing we can do about it.

so there is no real point buying the higher ranks now due to the no boosters, so would the ranks be reworked?

You still have access to the higher tier boosters. They just aren't able to be spread across the whole party. It is still a benefit to leveling up, and the RPG boosters are completely unaffected. However, that is up for Ras to decide.

its not just EXP Share/All though, switch training is also broken. It only affects the active pokemon at the end of the battle so it really isnt faster


Dashboard date filter "TypeError: Cannot read property 'map' of undefined"

Make sure these boxes are checked before submitting your issue - thank you!

[ x ] I have checked the superset logs for python stacktraces and included it here as text if any
[ x ] I have reproduced the issue with at least the latest released version of superset
[ x ] I have checked the issue tracker for the same issue and I haven't found one similar

Superset version
0.20.3
Expected results
Dashboard date filter should work
Actual results
Error occurs on filter slice:
An error occurred while rendering the visualization: TypeError: Cannot read property 'map' of undefined
Steps to reproduce

Create dashboard
Add filter to dashboard
Apply date filtering
Refresh dashboard

Am pretty sure you haven't posted the full stacktrace from your server logs

I did not see anything in server logs (stdout from superset process). I used tail -f during dashboard testing and that error was not visible there

Ah! sorry, that's a js error :)

Yeah,
I managed to get log in console but it looks like minified code so probably not useful anyway

Uncaught TypeError: Cannot read property 'map' of undefined
at t.e (dashboard.5181d5d45936098fa409.entry.js:135)
at f._renderValidatedComponentWithoutOwnerOrContext (dashboard.5181d5d45936098fa409.entry.js:61)
at f._renderValidatedComponent (dashboard.5181d5d45936098fa409.entry.js:61)
at f._updateRenderedComponent (dashboard.5181d5d45936098fa409.entry.js:61)
at f._performComponentUpdate (dashboard.5181d5d45936098fa409.entry.js:61)
at f.updateComponent (dashboard.5181d5d45936098fa409.entry.js:61)
at f.performUpdateIfNecessary (dashboard.5181d5d45936098fa409.entry.js:61)
at Object.performUpdateIfNecessary (dashboard.5181d5d45936098fa409.entry.js:29)
at k (dashboard.5181d5d45936098fa409.entry.js:29)
at m.perform (dashboard.5181d5d45936098fa409.entry.js:34)

And image as well

Not sure if it matters but we've got cache enabled for that datasource

https://github.com/apache/incubator-superset/issues/3650 is a result of this issue

So when can we expect 0.20.4 with this fix? :)

I am experiencing this again in the 0.28.1 version when I applied date filter. Any fix?

Same here, we are seeing this error in javascript on 0.28.1

Getting this error while trying to use mapbox in superset
An error occurred while rendering the visualization: TypeError: e.filter is not a function

@nuria I can see this error is fixed on in 0.29.0rc7 version, I was facing the same issue I upgraded to 0.29.0rc7 version and I can see this is resolved for me.
Reference,
https://www.aimks.com/superset-filter-box-error-cannot-read-property-map-of-undefined-solution.html

@sudeshmu I am also facing same issue. I am trying to upgrade superset 0.28.1 to latest version by running command
"pip install superset --upgrade" but it is not upgrading to latest version. It is saying "requirement already satisfied".


Enrichment Confidence Intervals Incorrectly Stored

Using the example from the vignette, I see the upper bound and lower bound are switched.
$results
     Hugo_Symbol Enriched nonEnriched       pval        or      ci.up     ci.low adjPval
  1:        TP53        2          13 0.08175632 5.9976455 0.49875432  46.608861       1
  2:        TET2        1          16 0.45739351 1.9407002 0.03882963  18.983979       1
  3:        FLT3        2          45 0.65523131 1.4081851 0.12341748  10.211621       1

The upper bound should be - by definition - larger than the lower bound.
Also, the APOBEC enriched samples look suspicious. All of the samples determined to be enriched have high p-values (e.g. 0.97) and FDR of 1. I don't think a journal reviewer would like that.
> laml.tnm[[2]][, c(1, 39, 44, 36, 43)][1:10]
    Tumor_Sample_Barcode fisher_pvalue fdr APOBEC_Enrichment APOBEC_Enriched
 1:         TCGA-AB-2946     0.7876588   1          3.428571             yes
 2:         TCGA-AB-2812     0.8382664   1          2.388889             yes
 3:         TCGA-AB-2941     0.8562368   1          3.916667             yes
 4:         TCGA-AB-3012     0.9500478   1          1.769231              no
 5:         TCGA-AB-2897     0.9580635   1          1.117647              no
 6:         TCGA-AB-2910     0.9693494   1          2.555556             yes
 7:         TCGA-AB-2843     0.9699295   1          1.896104              no
 8:         TCGA-AB-3008     0.9753432   1          1.464286              no
 9:         TCGA-AB-2901     0.9804400   1          1.328571              no
10:         TCGA-AB-3007     0.9830072   1          1.270588              no

Okay, I will double check this and get back to you.
For this, APOBEC_Enriched is yes based on enrichment score (>2). You can manually filter based on p-values from results. Function uses 2 as a cut-off because by definition enrichment score, a value of 2 means there are twice APOBEC related mutations in C>T context than in surrounding 20 bp region of mutated base.

Hi,
Thank you again. Yes, indeed the CI values were interchanged. I have fixed it. You will have to install from GitHub for updated version.
I am closing the issue for now. Please feel free to reopen if necessary.


Avatar component fallback svg problem

I use Avatar component two diffrent type:  User and Group avatar.
Group avatar set className no src.
codesandbox demo
      data.imgSrc ? (
            <Avatar src={data.imgSrc} />
      ) : (
            <Avatar className="foo" />
      )

Solution:
      data.imgSrc ? (
            <Avatar src={data.imgSrc} />
      ) : (
            <Avatar alt= " " src=" " className="foo" />
      )

Why extra attribute and space?

Tech
Version

Material-UI
v4.8.3

We had the same feedback on https://github.com/mui-org/material-ui/pull/18711#issuecomment-564764022 by @andokai.
I was wondering about using this solution: https://github.com/mui-org/material-ui/pull/18711#issuecomment-565419427. While @fatihhayri hasn't provided enough context for us to take his feedback into account. I believe the related comment has. @mbrookes What do you think?

I saw it at the time - seems like a good compromise.

I'm closing, it's not clear yet that it would be a better default. Also, the usage of the component is in this context seems off.


SAPUI5: GridLayout linebreaks in L but not XL

The following line exists in OpenUI5 but not in SAPUI5
https://github.com/SAP/openui5/blob/122a6d71886a32d8f1a5dc6dbbedcf39cdc6ac4b/src/sap.ui.layout/src/sap/ui/layout/GridRenderer.js#L136
Will this get synced in the near future? The commit that introduced this feature (linebreaks in L but not XL) was done almost five months ago...
Edit: Unfortunately we cannot use OpenUI5 since we rely on sap.uxap

Thanks for reporting! This might be related to the version of SAPUI5 you are running. Could you please have a look? I can see that this has been introduced with 1.32.

Thank you for your response. Yes, seems like it was a caching problem. src was pointing to https://sapui5.netweaver.ondemand.com/sdk/resources/sap-ui-core.js which should be the latest version, but apparently was 1.30.8. After doing a hard reload everything worked fine :+1:


Rebooting PC causes Nitro sites to become inaccessible

I may be missing something, but I have been trying to get Nitro up and running for local dev. Every time I get it going it's smooth and works fine, ignoring a few issues that I was able to find solutions for online relatively quickly. But when I restart or shutdown my computer and then later come back to continue working on it, the sites become inaccessible, with a generic "This site can't be reached" message from the browser.
Looking at "Nitro info" it looks like the IP has changed so I've tried updating that on my Craft CMS config files and local hosts file (Windows machine), but to no avail.
Any help at all would be greatly appreciated as this has been pretty solid to work with... until I reboot my computer.

@DouglasKGlover you need to disable internet sharing, this seems to be a known thing with the way multipass using the Windows hypervisor.
There is a tutorial on troubleshooting the windows networking here: https://multipass.run/docs/troubleshooting-networking-on-windows
I need to confirm, but it appears to resolve itself if you disable internet sharing on Windows.

@jasonmccallister I've done all of the steps provided on the tutorial you shared, as well as manually disabled Internet Sharing via Services and set it to disabled on startup. Unfortunately I am still unable to access my site for local dev. :(

@DouglasKGlover did you reboot the host machine to apply those changes?

I did, yes. I'll note that the following from a user (goraxan) on Discord worked for me:
@Doug same here (in Windows). IP changes every time I reboot. So what I do is: nitro stop; nitro start; nitro info (change .env database DSN with the new IP); nitro apply (this changes the hosts file with the new IP)
Is there any way to prevent the machine from acquiring a new IP on Windows reboot? Not sure if this would be a Nitro or a Multipass thing, but seems directly tied to the Nitro machine itself as far as I can tell.

@DouglasKGlover there is a hidden command called nitro hosts that will read the IP address of the machine and clean the hosts file. It will not touch the hosts.iscl file though.
This issue is somewhat related to multipass but the real issue is the network manager on windows hypervisors. I worked through this with someone else on windows, I will check if they are still having the issue after disabling internet sharing.

@jasonmccallister The issue still persists. Guess the issue is with Windows hyper-v default switch.

@delixyr this is a Windows related issue and the next release will resolve this.

@delixyr by next release, I really mean't v2 where we replaced Multipass with Docker. Here are the instructions for upgrading, Windows 10 is still a WIP... but mostly for WSL2 and Docker setup.. https://craftcms.com/docs/nitro/2.x/

We just released Nitro 2 where we switched from Multipass to Docker and added a ton of new features. Please see the announcement for more details.
This issue was specific to Multipass so we are going to close this out. Please see the documentation on upgrading from Nitro 1.


Add my profile

please verify and merge my pull request

Great !!
Thanks for contributing!
Your PR is Merged now.
Happy Hacktoberfesting......


Call github only once

I want to merge this change because our current implementation of fetching GH stars makes multiple calls to GH API which leads to quicker depletion of allowed requests. Also bumps babel.
Pull Request Checklist

[ ] Privileged views and APIs are guarded by proper permission checks.
[ ] All visible strings are translated with proper context.
[ ] All data-formatting is locale-aware (dates, numbers, and so on).
[ ] Database queries are optimized and the number of queries is constant.
[ ] Database migration files are up to date.
[ ] The changes are tested.
[ ] The code is documented (docstrings, project documentation).
[ ] GraphQL schema and type definitions are up to date.
[ ] Changes are mentioned in the changelog.

We should use https://github.com/github/fetch


I am using application.SCSS for my rails project, can I use @import "evil-icons" to include the icons?

Yes, you can


It is work with sveltekit?

I'm trying to use in my sveltekit project but nothing happens

@lucaskfp go to extension setting > supported languages > edit in json and than add "svelte"

It's work, thanks


TokenIter/Caravan should be a mutable reference

I think this is the case.

https://github.com/orangutanrider/waymark_relations/pull/50


TreeViewNode'a arrow in filechooser

The arrow in the file chooser is too big and in a wrong position when used with a mobile.
For full details see here.

Current code screenshots message
Current code that is too narrow in mobile:

Current code with out the FileListEntry height change:

Fixed screenshot message

Works for me!  @noembryo  Nice. Thanks


Add detail to readme

Hey @janomecopter , can you please expand the readme a bit to be more explicit - ie instead of 'common model', say ERA5-native, BARRA, BARPA, CCAM - it makes it clearer where this fits in to ACS.
Thanks :)

No worries, the books overall need a little polish (more comments) to explain things further.


chore: fix the path to get the gif file

You were looking the parent folder and not the current folder.

This will actually break the site. The reason why the relative image links don't seem to make sense from the source markdown point of view is that the site is using permalinks when deployed. foo.md doesn't get served as foo.html, but foo/index.html.

This will actually break the site. The reason why the relative image links don't seem to make sense from the source markdown point of view is that the site is using permalinks when deployed. foo.md doesn't get served as foo.html, but foo/index.html.

I see.  Thanks for explaining that, @jimmac !


adding description of mci

Dear @Dorothyrono , I would be very grateful if you would find time soon to take a look at this PR :) Thank you very much!

Thanks @AliceBalfanz! Some suggestions from my side:

Isn't this algorithm contributed by Brockmann? I suppose Sentinel Hub is a leftover from the template you used.
"Open Notebook" should either have a real link or be removed.
"License" should either have a real link or be removed.
Pricing information would be great.

Tested it locally and the index.ndjson is still working. Thus I'm going to merge it to test integration with the marketplace.

Seems like something didn't quite work with the automatic deploy. Maybe a side effect of the almost simultaneous other PR? Anyway, could you maybe investigate what happened @dthiex? Thanks

it should be deployed now
The deployed page can be found be found here meanwhile  https://s3.eu-central-1.amazonaws.com/collections.eurodatacube.com/mci/index.html
somehow the empty resources metadata wansn't acceptable .


[BUG] Build is broken, possibly with CCCL 2.2 changes upstream

The spark-rapids-jni build seems to be borked currently.  A fresh checkout seems to produce the following error:
[INFO]      [exec] -- Could NOT find libcudacxx (missing: libcudacxx_DIR)
[INFO]      [exec] -- Configuring incomplete, errors occurred!
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 24.617s
[INFO] Finished at: Tue Dec 19 23:05:26 UTC 2023
[INFO] Final Memory: 30M/1924M
[INFO] ------------------------------------------------------------------------

It appears that the error is related to upstream changes (in libcudf and rmm) to support CCCL 2.2.  From the build log:
[INFO]      [exec] -- CPM: rmm: Adding package<EMAIL_ADDRESS>(v2.2.0)
[INFO]      [exec] CMake Error at build/_deps/cccl-src/thrust/thrust/cmake/thrust-config.cmake:708 (find_package):
[INFO]      [exec]   Could not find a configuration file for package "libcudacxx" that is
[INFO]      [exec]   compatible with requested version "<IP_ADDRESS>".
[INFO]      [exec]
[INFO]      [exec]   The following configuration files were considered but not accepted:
[INFO]      [exec]
[INFO]      [exec]     /usr/local/cuda/lib64/cmake/libcudacxx/libcudacxx-config.cmake, version: <IP_ADDRESS>
[INFO]      [exec]
[INFO]      [exec] Call Stack (most recent call first):
[INFO]      [exec]   build/_deps/cccl-src/lib/cmake/cccl/cccl-config.cmake:65 (find_package)
[INFO]      [exec]   build/_deps/cccl-src/cmake/CCCLAddSubdir.cmake:1 (find_package)
[INFO]      [exec]   build/_deps/cccl-src/CMakeLists.txt:21 (include)
[INFO]      [exec]
[INFO]      [exec]
[INFO]      [exec] -- Could NOT find libcudacxx (missing: libcudacxx_DIR)
[INFO]      [exec] -- Configuring incomplete, errors occurred!

It appears that rmm is pulling in<EMAIL_ADDRESS>libcudacxx is then unable to be found.

Note:  There have been changes underway in libcudf to support CCCL 2.2.  While https://github.com/rapidsai/cudf/pull/14576 has been merged in libcudf, the cudf submodule in spark-rapids-jni remains at a prior commit (https://github.com/NVIDIA/spark-rapids-jni/commit/dadc7a091c1350a040aec25d565f6ba97e8e80ae).
But the rmm dependency does appear to include the<EMAIL_ADDRESS>change.

Note, also, that the spark-rapids-jni changes to support CCCL 2.2 are being worked on here: https://github.com/NVIDIA/spark-rapids-jni/pull/1662.
We have a couple of options:

Change the rmm version that spark-rapids-jni depends on.
Bite the bullet, and complete #1662 to switch over to CCCL 2.2.

Thanks @mythrocks. I agree there are two choices - pin both cudf and rmm (and potentially rapids-cmake/cuCollections), or complete the work to adopt CCCL 2.2.0.
Notes on pinning:

The last RMM commit before CCCL 2.2.0 is https://github.com/rapidsai/rmm/commit/77b55003b5ae598b7c26553ff39c5dde369c73d8
You may also need to pin rapids-cmake and/or cuCollections, so that the versions.json points to a build of cuCollections before CCCL 2.2.0.

The last commit of rapids-cmake without the updated cuCollections is https://github.com/rapidsai/rapids-cmake/commit/ef3388faffa3cba8615522083bdc8b0325e58476.
The pinning of cuCollections RAPIDS used before updating to CCCL 2.2.0 is https://github.com/NVIDIA/cuCollections/commit/7c76a124df0c2cd3fd66e3e080b9470a3b4707c6.

Close this as it is resolved by https://github.com/NVIDIA/spark-rapids-jni/pull/1662.


AttributeError: 'X509' object has no attribute '_x509'

Versions:
HTTPie 0.9.9
Requests 2.12.4
Pygments 2.1.3
Python 2.7.10 (default, Oct 23 2015, 19:19:21)

Traceback:
Traceback (most recent call last):
  File "/Users/josh/Library/Python/2.7/bin/http", line 11, in <module>
    sys.exit(main())
  File "/Users/josh/Library/Python/2.7/lib/python/site-packages/httpie/__main__.py", line 11, in main
    sys.exit(main())
  File "/Users/josh/Library/Python/2.7/lib/python/site-packages/httpie/core.py", line 227, in main
    log_error=log_error,
  File "/Users/josh/Library/Python/2.7/lib/python/site-packages/httpie/core.py", line 99, in program
    final_response = get_response(args, config_dir=env.config.directory)
  File "/Users/josh/Library/Python/2.7/lib/python/site-packages/httpie/client.py", line 70, in get_response
    response = requests_session.request(**kwargs)
  File "/Library/Python/2.7/site-packages/requests/sessions.py", line 488, in request
    resp = self.send(prep, **send_kwargs)
  File "/Library/Python/2.7/site-packages/requests/sessions.py", line 609, in send
    r = adapter.send(request, **kwargs)
  File "/Library/Python/2.7/site-packages/requests/adapters.py", line 423, in send
    timeout=timeout
  File "/Library/Python/2.7/site-packages/requests/packages/urllib3/connectionpool.py", line 594, in urlopen
    chunked=chunked)
  File "/Library/Python/2.7/site-packages/requests/packages/urllib3/connectionpool.py", line 350, in _make_request
    self._validate_conn(conn)
  File "/Library/Python/2.7/site-packages/requests/packages/urllib3/connectionpool.py", line 835, in _validate_conn
    conn.connect()
  File "/Library/Python/2.7/site-packages/requests/packages/urllib3/connection.py", line 330, in connect
    cert = self.sock.getpeercert()
  File "/Library/Python/2.7/site-packages/requests/packages/urllib3/contrib/pyopenssl.py", line 324, in getpeercert
    'subjectAltName': get_subj_alt_name(x509)
  File "/Library/Python/2.7/site-packages/requests/packages/urllib3/contrib/pyopenssl.py", line 166, in get_subj_alt_name
    cert = _Certificate(openssl_backend, peer_cert._x509)
AttributeError: 'X509' object has no attribute '_x509'

--debug output:
[GCC 4.2.1 Compatible Apple LLVM 7.0.0 (clang-7<IP_ADDRESS>)]
/usr/bin/python
Darwin 15.6.0

<Environment {
    "colors": 256,
    "config": {
        "__meta__": {
            "about": "u'HTTPie configuration file'",
            "help": "u'https://httpie.org/docs#config'",
            "httpie": "u'0.9.9'"
        },
        "default_options": "[]"
    },
    "config_dir": "/Users/josh/.httpie",
    "is_windows": false,
    "stderr": "<open file '<stderr>', mode 'w' at 0x10f9fa1e0>",
    "stderr_isatty": true,
    "stdin": "<open file '<stdin>', mode 'r' at 0x10f9fa0c0>",
    "stdin_encoding": "UTF-8",
    "stdin_isatty": true,
    "stdout": "<open file '<stdout>', mode 'w' at 0x10f9fa150>",
    "stdout_encoding": "UTF-8",
    "stdout_isatty": true
}>

>>> requests.request(**{
    "allow_redirects": false,
    "auth": "None",
    "cert": "None",
    "data": {},
    "files": {},
    "headers": {
        "User-Agent": "HTTPie/0.9.9"
    },
    "method": "get",
    "params": {},
    "proxies": {},
    "stream": true,
    "timeout": 30,
    "url": "u'https://www.google.com'",
    "verify": true
})

http: error: AttributeError: 'X509' object has no attribute '_x509'

You need to upgrade/reinstall PyOpenSSL and Cryptography.

@sigmavirus24 Looks like this is an issue with how PyOpenSSL is installed for OS X machines with SIP. I typically install “global” packages with pip install --user, e.g.
pip install --upgrade PyOpenSSL Cryptography
[...]
Successfully installed Cryptography-1.7.1 PyOpenSSL-16.2.0 cffi-1.9.1 idna-2.2 pycparser-2.17 setuptools-32.2.0 six-1.10.0

And it has the same error. However, if I activate a virtualenv and reinstall httpie along with specific --upgrade for PyOpenSSL, and Cryptography, it works. Just installing httpie into a virtualenv doesn't make the correct version of PyOpenSSL available.

Going to close this issue for now, hopefully if someone else encounters this issue it gives them a path to resolve it.


Fix paths in contributing guidelines

Changes
I tried to follow the instructions in CONTRIBUTING.md and noticed that the paths to snowpack aren't working. I removed the pkg and dist-node parts and changed the repository name to distinguish it from the snowpack directory inside the repository.
Testing
I tested the three new paths manually.
Docs
This PR only updates the contributing guidelines.

thank you!


parseExact fails with date and time

parseExact is failing to work as expected in 1.0.0-rc3.

Without seconds (parses wrong):
Date.parseExact("2014-03-04T14:15", "yyyy-MM-ddTHH:mm")

Expected Result:
Tue Mar 04 2014 14:15:00 GMT-0700 (Mountain Standard Time)

Actual Result
Tue Mar 04 2014 00:14:15 GMT-0700 (Mountain Standard Time)

With seconds (fails to parse):
Date.parseExact("2014-03-04T14:15:16", "yyyy-MM-ddTHH:mm:ss")

Expected Result:
Tue Mar 04 2014 14:15:16 GMT-0700 (Mountain Standard Time)

Actual Result
null

Get null for parse exact on any time formatted 'hh:mm:ss tt'

I need to knuckle down and investigate this more - sadly real life work has got in the way.

There's a bug in parsing_grammar.js. If you look at the time formats (starting on line 73), you'll see that the tokens array gets out of sync with the RTokenKeys and RToken arrays. "HH" is interpreted as t.minute, while "mm" is interpreted as t.second, "ss" as t.secondAndMillisecond, and so on. :)

Great catch - thanks for doing the legwork here. I should have more time this spring to push forward on this and stomp bugs like this.

Fixed by dzhus!


Duplicate INTERNET Permission in Manifest file

Summary:
/home/runner/work/apps-android-commons/apps-android-commons/app/src/main/AndroidManifest.xml:7:5-67 Warning:
	Element uses-permission#android.permission.INTERNET at AndroidManifest.xml:7:5-67 duplicated with element declared at AndroidManifest.xml:5:5-66

The permission should be declared only once.
Would you like to work on the issue?
No

@developer-nobita Sure, thanks :)


Whole font awesome npm package bundled in vsix

We are probably using font awesome for the webview features, but we may not want to bundle the whole package into vsix, for example, its less/scss source files

The whole package is 1.4M.

Deferring this until we've merged the PR that refactors the whole build process
https://github.com/microsoft/vscode-jupyter/pull/12129


Update the documentation

The session configuration needs to be updated to have concatenation:
val appID = "<app-id>"
val password = "<password>"
val fileSystemName = "<file-system>"
val tenantID = "<tenant-id>"
val storageAccountName = "<storage-account>"

spark.conf.set("fs.azure.account.auth.type", "OAuth")
spark.conf.set("fs.azure.account.oauth.provider.type", "org.apache.hadoop.fs.azurebfs.oauth2.ClientCredsTokenProvider")
spark.conf.set("fs.azure.account.oauth2.client.id", appID)
spark.conf.set("fs.azure.account.oauth2.client.secret", password)
spark.conf.set("fs.azure.account.oauth2.client.endpoint", "https://login.microsoftonline.com/" + tenantID + "/oauth2/token")
spark.conf.set("fs.azure.createRemoteFileSystemDuringInitialization", "true")
dbutils.fs.ls("abfss://" + fileSystemName + "@" + storageAccountName + ".dfs.core.windows.net/")
spark.conf.set("fs.azure.createRemoteFileSystemDuringInitialization", "false")

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

ID: d863ab39-0b6a-eb81-9dde-5eb25b46cc33
Version Independent ID: 655abadf-5f9e-a65b-0109-c5166f956c4d
Content: Tutorial: Perform ETL operations using Azure Databricks
Content Source: articles/azure-databricks/databricks-extract-load-sql-data-warehouse.md
Service: azure-databricks
GitHub Login: @mamccrea
Microsoft Alias: mamccrea

@mindlessroman Thanks for the feedback and bringing this to our notice. At this time we are reviewing the feedback and will provide an update as appropriate.

@mindlessroman Thanks again for pointing this out. We have assigned the issue to the content author. They will evaluate and update the document as appropriate.

Thanks @mindlessroman - The lines that need to be concatenated are not necessary for session configuration, but only for account configuration, so I have removed them. The doc will be updated later today.
#please-close


We need Wayland support

Describe the bug
A clear and concise description of what the bug is.
System

OS: [e.g. Ubuntu] Fedora by default, any distro with Wayland support and..... CHROME OS !
OS Version: [e.g. 18.04] Any
Environment: [e.g. Gnome] Any Wayland supported env
Application: [e.g. AppImage] Beta appimage, shadowcker
Application Version: [e.g. 4.17] 4.14.16

Configuration

The user is in input group ? Yes (Chrome OS Linux subsystem
The environment uses Xorg ? nope, that's the point
VA API detects H264 ? And H265 ? Yes

To Reproduce
Steps to reproduce the behavior (chromeos) :

Enable Linux subsystem in settings
Install all prereq libs (took the list from dockerfile of shadowcker), as base system is debian
Start the launcher, keyring is working...
Stream never starts, stay in Shadow is ready indefinitely...

Just to check, vainfo is working on default terminal, appimage is starting with no issue from penguin. Just Wayland to make it work on chromebooks.
 I can't do that on anything else linux or Windows (not smooth on win) !
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
Wayland is the future, what is the issue with the shadow launcher ? Some users have opened issues before for "Shadow is ready", that issue would stay opened..
Anybody succeed in streaming on wayland ?

Ok, seems there is no GPU acceleration on Cristiano at the moment on ChromeOS. But still, issue is opened for Linux. What are the reasons ? Input ?

asked this myself in the discord channel already at least 2 times. never got a reply, let alone official statement. i think i will just move on, and leave shadow behind. there are alternatives maybe not coming with this "all-in-one" approach, but will allow me to use opensource clients or steam streaming (which also is possible under wayland)

asked this myself in the discord channel already at least 2 times. never got a reply, let alone official statement. i think i will just move on, and leave shadow behind. there are alternatives maybe not coming with this "all-in-one" approach, but will allow me to use opensource clients or steam streaming (which also is possible under wayland)

I just got back to Windows because of that, that's a shame, a browser solution like stadia could be much easier for users...

Hi there ! Sorry, I'm not very available because I'm currently moving in another flat.
About Wayland, it needs a total revamp of the way the inputs are captured, and how the window is managed in general. I don't see it coming any soon but we can cross our fingers ;) BTW the devs told us they are currently working a lot on the Linux app on their side, cleaning up stuff and improving support. We can hope that a dev about Wayland is in the pipe.
About ChromeOS, there is no way to make it work right now as Crostini does not support the GPU acceleration as you mentionned, ie Crostini doesn't have access to the GPU decoding pipeline. Shadow uses the VA-API, and the only way to make it work would be with OpenGL with a lot of tweaking from their side use the Crostini driver. I guess this will be not the priority.

Thanks Nicolas ! I would have bet on the input side of things :)
Well, I stick with windows for now, but continue checking and finger crossing !

why is this closed, when the issue obviously is not fixed at all?


Added minimum SW requirements to developer-setup

Summary
I have added minimum software requirements to successfully deploy a developer environment.
Ticket Link
Fixes https://github.com/mattermost/mattermost-developer-documentation/issues/528
Mattermost Chat Reference
https://community.mattermost.com/core/pl/aexmr4p7x7n79mwg4gnezp9h6r

@cpanato Could you please confirm the minimum versions are correct?

This issue has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @hanzei

@cpanato Kind reminder to help with reviews :)


Verify that Primality is deterministic in the interval [0;2^64]

Currently Number-Theory's claim of determinism is reliant on Forisek & Jancina's  (erroneous) claim of determinism in there original implementation. However it is known that there is at least one error (677 is passed as composite) in the original table, so while it is likely that the claim is true this is not verified, and it is a major goal of the Rust-CAS team to prove it.
Currently it has been proven that is_prime is deterministic in the interval [0;2^41]*.  If you plan on contributing to verification; post the interval you are checking in, the number of cores used (not the total cores but rather the number of threads that are being used for verification), and the processor model. It is important to post as soon as you start verification to prevent others from utilizing time verifying the same interval. We show an example below.
First post

Processor              Threads            Interval
i3-4005u                   1                  [2^33;2^34]

When you finish the computation

Count                  Time
 717267168          10829.22159s

Or more tragically

Failure     Reason

To allow others to know if they need to duplicate it.
One thing to note is that it is important to choose an interval that is divisible by the number of processors used with no remainder to prevent any potential truncation.
Start checking at 2^41.
*This value will be continously updated so check first.

Processor              Threads            Interval
ryzen9 3950x               16              [2^40;2^41]
 Count                     Time

39113482640      35887.077132s

Proven utilizing Jan Feitsma's base-2 pseudoprime list.


Using SIGTERM to stop services

Instead of SIGKILL. And properly handling the Serf PID file (manually created on Serf start). Related to https://github.com/Mashape/kong/commit/b9f0e4f88019ae9ae9c5f89e2229e6c0877efaaa.

@thibaultCha any feedback on this before merging?

Changes committed.


How use in vue components?

Tell me how I can use icons in a project with a vue3

?

First, install boxicons via npm, not cdn.
npm install boxicons --save

Then import Boxicons in the script section of your App.vue.
import 'boxicons';

Done. It should be now accessible everywhere in your Vue App like this.
<template>
    <box-icon name="save"></box-icon>
</template>

I see the following warning in the console
[Vue warn]: Failed to resolve component: box-icon
If this is a native custom element, make sure to exclude it from component resolution via compilerOptions.isCustomElement.

using Vue 3 + TS

I see the following warning in the console
[Vue warn]: Failed to resolve component: box-icon
If this is a native custom element, make sure to exclude it from component resolution via compilerOptions.isCustomElement.

using Vue 3 + TS

You can solve the problem by following this answer： https://stackoverflow.com/a/71602929


Add additional factory methods to Kinesis to create Akka Source

As the result of the issue with graceful shutdown of the KinesisConsumer described in https://github.com/WW-Digital/reactive-kinesis/issues/60, I added the additional methods to create the Akka Source with explicitly pathing the ConsumerService, for the better controll of the shutdown process.

Thanks for this!
It's weird that this isn't firing for you:
https://github.com/WW-Digital/reactive-kinesis/blob/master/src/main/scala/com/weightwatchers/reactive/kinesis/consumer/KinesisConsumer.scala#L251
Seems like the consumer is shutting down before that. Happy to merge this but would you mind adding this to the readme too please with an example?
https://github.com/WW-Digital/reactive-kinesis#akka-stream-source

@markglh Ok will update the docs and then the PR.

Coverage decreased (-1.5%) to 88.61% when pulling 39d51b967a9e45f935db10962951fe30444cb0a2 on htimur:master into c635b6f4d0f6de8379dfd1c6d7b3a61cd64dbe8c on WW-Digital:master.

@markglh Please review the changes


Font not found after adding private metanorma repo

Recently we faced with the issue [fontist] 'Tahoma' font is not supported. ... in private repo mn-samples-bsi
  fontist repo list
  fontist repo setup metanorma ***github.com/metanorma/fontist-formulas-private
  fontist repo update metanorma
  fontist repo list
  fontist install Tahoma
  shell: sh -e {0}
Fontist repo 'metanorma' from '***github.com/metanorma/fontist-formulas-private' has been successfully set up.
Fontist repo 'metanorma' has been successfully updated.
metanorma
Font "Tahoma" not found locally.
Font 'Tahoma' not found locally nor available in the Fontist formula repository.
Perhaps it is available at the latest Fontist formula repository.
You can update the formula repository using the command `fontist update` and try again.

@alexeymorozov maybe you have an idea what is wrong

Released v1.11.7 with the fix.
The bug was caused by compatibility with the fontist repo setup command:

fontist repo setup created a formulas dir but without a main git repo
fontist update checked that the dir is already created and tried to update it
and failed because this dir had no .git dir

Thank you @alexeymorozov !


Range has a funny constructor

    new: static func (.min, .max) -> This {
        this : This
        this min = min
        this max = max
        return this
    }

I can think of reasons why Range doesn't have a normal (init) constructor:

it was written at a time when covers didn't have constructors
it's old, but it works, so no one has bothered to change it
it's so low level and so deep inside the SDK, or used in rock somewhere constructors can't be used

Are there other reasons?

It was introduced in https://github.com/fasterthanlime/rock/commit/4f6d871407e2e683b023e464b0b395780d34a796 (Jan 25, 2010)
Other codes at that time use init.
Maybe the compiler uses Range new to unwrap a..b and the compiler was not smart enough to resolve init to new at 2010 ?

it was written at a time when covers didn't have constructors

this

Other codes at that time use init.

yes, but not covers afaik?

Okay, so if I make a PR where I change it to
    init: func@ (=min, =max)

, you won't mind?

@davidhesselbom if it works, I'd be all for it! It's cleaner.
However, in the generate code, that means one more call + pointer dereferences.
The current code generates this:
struct Range Range_new(int min, max) {
  struct Range r;
  r.min = 0;
  r.min = 24;
  return r;
}

Your proposed change will generate this:
struct Range Range_new(int min, int max) {
  struct Range r;
  Range_init(&r, int, min)
  return r;
}

void Range_init(struct Range *range, int min, int max) {
  range->min = min;
  range->max = max;
}

Nothing a good compiler can't optimize, perhaps I used to think Range was performance-critical. Not sure.
Having new directly as a static func is perfectly valid ooc, definitely less funky than trying to cull the GC off of the sdk :)

Okay, I didn't mean to suggest it's invalid or anything, it's just... longer, and a bit unusual. I hadn't thought about performance, but otoh, I seem to remember you saying someting along the lines of "ooc wasn't designed for speed", and like you, I doubt the change proposed will have any significant effect.
About the GC, ripping it out wasn't my idea, I swear.

@davidhesselbom once upon a time I was pretty happy that ooc had "the speed of raw C", but that was before String became a class, before many other changes...
Back then I also had a very bad instinct on what did and did not matter performance-wise. As the generated code grew in complexity, it became harder and harder to get useful info out of profilers like cachegrind.
Anyway, as far as performance goes, instincts lie, measure all the things! :8ball:

as far as performance goes, instincts lie, measure all the things!

True...

ooc needs help speed-wise. Once when I wrote a BF interpreter in ooc, it was slower than Julia, PyPy, and gasp JavaScript...

ooc generics are slow

ftfy


SUBTASK: Create Route53 backup Terraform stack in ops-eng

Acceptance Criteria:

Create a Terraform stack in operations-engineering repository referencing the Route53 backup module
Stack should include all relevant data about target AWS account and R53 records

No longer required.


Checking for zipfile using suffix not magic number

This small MR seeks to fix issue #208 . There's probably a more general way to check if something is a zipfile without identifying zipfiles inside tar files.

Looks like @epwalsh already fixed this in a different way. Thanks for taking the initiative here!


Proxy processor

Hi @Haishi2016 , I want to make sure my understanding is correct on this change.
This PR is trying to avoid stage / campaign users to do malicious operation with Symphony control plane permissions.
The changes include,

We introduced optional proxy field to StageSpec in Campaign, so there's a potential API contract change.
When users specify proxy option to hint Symphony control plane. Symphony control plane will invoke a REST call to another process to do operations. And this process can be a standalone Symphony agent with different privilege.

For this part, should we authenticate the remote processor? What if remote processor injects some malicious data?
And do we expect end-users to deploy this processor agent and control its security?
Is there any misunderstanding here?

Authentication should be the same as we authenticate with the API server (it's the same code base so when the authentication mechanism is available, we can leverage it)
Yes, user needs to deploy the "stage runner" container/process - we can include in our Helm chart to provide a default "stage runner", and we can extend maestro tool to help users to deploy additional runners.
With the isolation, and advisory can still certainly inject invalid data. However, as long as we guard against things like script injection we have a much smaller attack surface.


update inits and fini diag_msg on every call

As you can see here, the message is allocated and deallocated on every call. IMHO, the message should be initialized once, and maybe be part of the updater struct, thus, the init could be called once when doing it for the updater, and the fini when the updater is terminated.

I can see that the updater struct struct already has a message member that doesn't seem to be used anywhere, https://github.com/micro-ROS/micro_ros_diagnostics/blob/c603b0bc1c2748f25a79a1edc78ae350b21654e7/micro_ros_diagnostic_updater/include/micro_ros_diagnostic_updater/micro_ros_diagnostic_updater.h#L45
maybe it was thought for this purpose?

Yes, this was the purpose, it was just not properly used, yet. :)

@norro could you review https://github.com/micro-ROS/micro_ros_diagnostics/pull/13 ?


Level Design: Side houses

Upgrade terrain to generate house models during the run

Also: append some decorations to the main road such as lanterns

Need to upgrade house variants and cover backgrounds

Completed and upgraded to two house lines in a row


Dependency issue when using on Node >= 12

deasync needs to be bumped to latest fix to support Node 12
See: https://github.com/abbr/deasync/issues/114

Release 0.1.0, thanks for reporting 👍


Changes to get volume

These are just changes to support getting volume - it's a bit of a hack right now.

Codecov Report

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

@@           Coverage Diff           @@
##           master       #8   +/-   ##
=======================================
  Coverage   10.66%   10.66%
=======================================
  Files           4        4
  Lines         347      347
=======================================
  Hits           37       37
  Misses        309      309
  Partials        1        1

Continue to review full report at Codecov.

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


Consider changing the name of the repo to something that identifies that it's related to Azure Cognitive Service Speech

batch processing kit is very broad in the context of Microsoft.

The framework is a generic batch processing library that is not confined to Azure Cognitive Services. That's just currently the only example. But the framework and the example are split into separate modules, and it's a matter of convenience that they are in the same repo. It's intended for any use case where there are local or remote worker process slaves. There will be forthcoming examples showing much more mundane sort of batch processing.


There is no "occupied" check at the end of the sequence

When there are no terminators at the end of the sequence, if there are two promoters that meet the requirements, both will be counted in the ORF.

You should set a variable for each frame in each sense to monitor whether it is occupied

Please submit a pull request.


for loop in object not working

{ "for": "bar" }

for loop not working ( length = 0 )
but
{ 0: "bar" }

for loop working ( length = 1 )
why???

example code:
 {% set var = { "foo": "bar", "foo 1": "bar 1" } %}

{% for key, value in %}
 <li> {{ value }} </li>
{% endfor %}

when I executed this code, the result is empty ( the 'for' loop doesn't execute ). But, the code given below shows the below result
Code :
 {% set var = { 0: "bar",  0: "bar 1" } %}

{% for key, value in %}
 <li> {{ value }} </li>
{% endfor %}

Result :

bar
bar1

I don't understand, when I provide integer value as key, I get the result. But when I provide string value as key, I don't get the result.

I just try this example:
{% set foo = { 0: "foo", 1: "bar" } %}
{% for key, value in foo %}
    <li> {{ value }} </li>
{% endfor %}

{% set bar = { "foo": "bar", "bar": "baz" } %}
{% for key, value in bar %}
    <li> {{ value }} </li>
{% endfor %}

and got this result:

foo
bar
bar
baz

Everything seems to work

seems worried. why not working for me :(

Can you copy-paste code snippet and describe your use case. Maybe you use some plugins (gulp, grunt, other)?

thanks got my problem
I made a twig function that was actually return object.
Twig.extendFunction('fieldsByNameIndex', function (fieldsGroup, index) {
  let results = [];
  let data = fieldsGroup[index];

  for (let i = 0; i < data.length; i++) {
    let field = data[i];
    results[field.name] = field; // update integer value as key to field name ( string as index )
  }

  return results;
});

when I use this filter

I think it because in js array keys may be only integer numbers


all: spelling inconsistency for networking terms

Some packages use hostname and others host name. The worst is the net package which uses both. It might be better to align to host name defined in https://tools.ietf.org/html/rfc7719.
Host name: This term and its equivalent, "hostname", have been
widely used but are not defined in [RFC1034], [RFC1035], [RFC1123], or [RFC2181].

Where is host name defined in RFC 7791?
https://tools.ietf.org/html/rfc7791

s/7791/7719/g

I would instead encourage us to avoid the term "hostname" unless it specifically refers to the operating system's notion of hostname. (As RFC 7719 mentions, the hostname often does not include the domain part.)

@mdempsky tackled this issue a bit for Lookup functions and used the term host for host names, the term name for domain names in CL  34650; just FYI.


fix(DropdownV2): explicitly passes through translateWithId to menu icon

so consumers of dropdownv2 can specify translations for title text of the icon
Related to https://github.com/carbon-design-system/carbon-components-react/issues/1952
updated version of my previous pr, i noticed that FilterableMultiSelect has a directly analogous instance of this so I figured I would just copy that pattern instead of making a new one.

Worth noting that if a component defines a 'default' implementation of this function it will need to include all of the child component's strings. I think that is easy enough to accomplish, but it is a bit of development overhead to be aware of.

@asudoh @joshblack this is a better version of the above closed pr, could you give it a review?


Instance Info

Created a script to grab instance info from http://www.ec2instances.info/
Creates a json file, then the ec2 backend has a constant that contains instance dict.
Realised after I did the scrape that they have a github with the json in there, but atm their json is 1.8Mb and mine is 44k :wink:
Will refactor if it breaks in the future.

Coverage increased (+0.001%) to 93.063% when pulling c888d65d6dfb27e3e44e86a7b94d834bb193dd9e on terrycain:get_instance_info into b17136e36c3a975aa2cebed15aec766bc6776260 on spulec:master.

This is released as Moto version 1.1.15


fix: define for multiple instances

When there are multiple skate instances on a page, some components could be defined in more than one instance. This PR ensures that even in this situation all the components will be registered.

lgtm

LGTM (as we are adding the tests separately)

Added a test case - it mocks out the define function (see test/unit/api/define.js)  because it is really hard to get two versions of skate on the same page.


New version: EtternaProject.EtternaGame version 0.71.2

Auto-updated by vedantmgoyal2009/winget-pkgs-automation in workflow run 418
Microsoft Reviewers: Open in CodeFlow

/azp run