Bug about ComiRec

It seems that 'his_vectors' in this line should be 'his_pos_vectors'.
When 'self.add_pos' is set to True,  we should use 'his_pos_vectors' instead of 'his_vectors'.
In my experiments, alter 'his_vectors' into 'his_pos_vectors' can lead to significant performance improvement when setting 'self.add_pos' to True.

We also think it is more effective to aggregate 'his_pos_vectors', but we chose to follow the official code of ComiRec:

Thanks for your response. It fully addresses my concern.


Can not install on latest Alpine image 3.21

Installation is ok on Alpine image 3.20 but not on 3.21:
/ # ./venv/bin/pip install netifaces-plus
Collecting netifaces-plus
  Using cached netifaces_plus-0.12.4.tar.gz (30 kB)
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
  Preparing metadata (pyproject.toml) ... done
Building wheels for collected packages: netifaces-plus
  Building wheel for netifaces-plus (pyproject.toml) ... error
  error: subprocess-exited-with-error

  × Building wheel for netifaces-plus (pyproject.toml) did not run successfully.
  │ exit code: 1
  ╰─> [55 lines of output]
      <string>:13: SetuptoolsDeprecationWarning: Deprecated API usage.
      !!

              ********************************************************************************
              As setuptools moves its configuration towards `pyproject.toml`,
              `setuptools.config.read_configuration` became deprecated.

              For the time being, you can use the `setuptools.config.setupcfg` module
              to access a backward compatible API, but this module is provisional
              and might be removed in the future.

              To read project metadata, consider using
              ``build.util.project_wheel_metadata`` (https://pypi.org/project/build/).
              For simple scenarios, you can also try parsing the file directly
              with the help of ``configparser``.
              ********************************************************************************

      !!
      running bdist_wheel
      running build
      running build_py
      creating build/lib.linux-x86_64-cpython-312/netifaces
      copying netifaces/__init__.py -> build/lib.linux-x86_64-cpython-312/netifaces
      copying netifaces/__init__.pyi -> build/lib.linux-x86_64-cpython-312/netifaces
      copying netifaces/py.typed -> build/lib.linux-x86_64-cpython-312/netifaces
      running build_ext
      checking for getifaddrs...found.
      checking for getnameinfo...found.
      checking for IPv6 socket IOCTLs...not found.
      checking for optional header files...netpacket/packet.h linux/atm.h linux/llc.h linux/tipc.h.
      checking whether struct sockaddr has a length field...no.
      checking which sockaddr_xxx structs are defined...in in6 un ll atmpvc atmsvc llc.
      checking for routing socket support...no.
      checking for sysctl(CTL_NET...) support...no.
      checking for netlink support...yes.
      will use netlink to read routing table
      building '_netifaces' extension
      creating build/temp.linux-x86_64-cpython-312/netifaces
      gcc -fno-strict-overflow -Wsign-compare -DNDEBUG -g -O3 -Wall -fPIC -DNETIFACES_VERSION=0.12.4 -DHAVE_GETIFADDRS=1 -DHAVE_GETNAMEINFO=1 -DHAVE_NETPACKET_PACKET_H=1 -DHAVE_LINUX_ATM_H=1 -DHAVE_LINUX_LLC_H=1 -DHAVE_LINUX_TIPC_H=1 -DHAVE_SOCKADDR_IN=1 -DHAVE_SOCKADDR_IN6=1 -DHAVE_SOCKADDR_UN=1 -DHAVE_SOCKADDR_LL=1 -DHAVE_SOCKADDR_ATMPVC=1 -DHAVE_SOCKADDR_ATMSVC=1 -DHAVE_SOCKADDR_LLC=1 -DHAVE_PF_NETLINK=1 -I/venv/include -I/usr/include/python3.12 -c netifaces/_netifaces.c -o build/temp.linux-x86_64-cpython-312/netifaces/_netifaces.o
      netifaces/_netifaces.c: In function 'ifaddrs':
      netifaces/_netifaces.c:1259:5: warning: this 'if' clause does not guard... [-Wmisleading-indentation]
       1259 |     if (addr->ifa_name == NULL || strcmp (addr->ifa_name, ifname) != 0)
            |     ^~
      netifaces/_netifaces.c:1262:7: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'if'
       1262 |       PyObject* ifinfo = getifaddrsinfo(addr);
            |       ^~~~~~~~
      netifaces/_netifaces.c: In function 'gateways':
      netifaces/_netifaces.c:2006:9: error: initialization of 'int' from 'void *' makes integer from pointer without a cast [-Wint-conversion]
       2006 |         NULL,
            |         ^~~~
      netifaces/_netifaces.c:2006:9: note: (near initialization for 'msghdr.__pad1')
      netifaces/_netifaces.c:2041:33: warning: comparison of integer expressions of different signedness: '__u32' {aka 'unsigned int'} and 'int' [-Wsign-compare]
       2041 |         if (pmsg->hdr.nlmsg_seq != seq || pmsg->hdr.nlmsg_pid != sanl.nl_pid)
            |                                 ^~
      error: command '/usr/bin/gcc' failed with exit code 1
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
  ERROR: Failed building wheel for netifaces-plus
Failed to build netifaces-plus
ERROR: ERROR: Failed to build installable wheels for some pyproject.toml based projects (netifaces-plus)

Perhaps it came from the gcc version:

Alpine image 3.20: gcc (Alpine 13.2.1_git20240309) 13.2.1 20240309
Alpine image 3.21: gcc (Alpine 14.2.0) 14.2.0

Hi @tsukumijima,
i think that the issue did not came from Alpine but more by the GCC version.
netifaces-plus ( and netifaces) compilation fail with GCC >14.


Import an image to a new frame without deleting the rest of the frames

I do not seem to be able to load a new image to a new frame with out deleting the original image on the other frame. Currently I am clicking on "+ Add new frame" then "Import" then loading the image. That gives me "Import as single image" and "Import as spritesheet". However clicking "Import" says "This will erase your current Piskel. Continue?"
How do I import WITHOUT erasing my current piskel?
Thank you

Hi !
If you want to import a new animation (spritesheet, or gif, or piskel) the problem is a bit complex. I talk about it in this comment https://github.com/juliandescottes/piskel/issues/285#issuecomment-109729785.
If you just want to import a static image (png, bmp), you should be able to drag and drop it in your canvas directly. !! It needs to be smaller than your canvas resolution, because it will not be resized !!
Let me know if this helps, otherwise give me more details about what you're trying to do :)

No updates from logger, duplicate from #285
Closing.


Bug report: No such file or directory - /usr/local/Library/Taps/caskroom/homebrew-cask

Already followed the pre bug report guide. Also did $ brew uninstall --force brew-cask; brew untap phinze/cask; brew untap caskroom/cask; brew update; brew cleanup; brew cask cleanup as instructed by brew cask doctor.
Description of issue
I get "No such file or directory - /usr/local/Library/Taps/caskroom/homebrew-cask" error with brew cask doctor, brew cask list gives me a (!) after each installed cask, and brew cask search gives me nothing. Also impossible to install a cask.
Complete outputs below.

Output of `brew cask search text --verbose`
No Cask found for "text".

Output of `brew doctor`
Your system is ready to brew.

 Please run:

      brew uninstall --force brew-cask; brew untap phinze/cask; brew untap caskroom/cask; brew update; brew cleanup; brew cask cleanup

  If this doesn’t fix the problem, please report this bug:

      https://github.com/caskroom/homebrew-cask#reporting-bugs

/usr/local/Library/Homebrew/cask/lib/hbc/version.rb:6:in `chdir'
/usr/local/Library/Homebrew/cask/lib/hbc/version.rb:6:in `full_version'
/usr/local/Library/Homebrew/cask/lib/hbc/cli/doctor.rb:13:in `run'
/usr/local/Library/Homebrew/cask/lib/hbc/cli.rb:108:in `run_command'
/usr/local/Library/Homebrew/cask/lib/hbc/cli.rb:146:in `process'
/usr/local/Library/Homebrew/cask/cmd/brew-cask.rb:8:in `<top (required)>'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:55:in `require'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:55:in `require'
/usr/local/Library/Homebrew/brew.rb:19:in `require?'
/usr/local/Library/Homebrew/brew.rb:93:in `<main>'

Looks like there's a different copy o this repo inside https://github.com/Homebrew/brew/tree/master/Library/Homebrew/cask and that's all I have installed now. Should I still do $ brew tap caskroom/cask?

Can you try cd $(brew --repo); git fetch; git reset --hard origin/master; brew update and then retry cask-related commands?

Looks like there's a different copy of this repo inside https://github.com/Homebrew/brew/tree/master/Library/Homebrew/cask and that's all I have installed now. Should I still do $ brew tap caskroom/cask?

The core has been move to the Homebrew/brew repo, the casks are still in the caskroom/cask tap.

@adidalal, my $(brew --repo) was already at master and clean. Running the git commands above gave me
HEAD is now at 7395ff2 Merge pull request #758 from reitermarkus/cask-dev-scripts
Already up-to-date.

and running the brew cask commands above gave me the same output as I reported earlier.
@reitermarkus, so one still needs to do $ brew tap caskroom/cask to get the casks? Doing so gives me a "should be descendant of Homebrew taps directory" warning in brew cask doctor.
 We're not auto-tapping caskroom/cask anymore, which breaks assumptions elsewhere in the core code that we have the repo tapped.
 Please run:

      brew uninstall --force brew-cask; brew untap phinze/cask; brew untap caskroom/cask; brew update; brew cleanup; brew cask cleanup

  If this doesn’t fix the problem, please report this bug:

      https://github.com/caskroom/homebrew-cask#reporting-bugs

/usr/local/Library/Homebrew/cask/lib/hbc/version.rb:6:in `chdir'
/usr/local/Library/Homebrew/cask/lib/hbc/version.rb:6:in `full_version'
/usr/local/Library/Homebrew/cask/lib/hbc/cli/doctor.rb:13:in `run'
/usr/local/Library/Homebrew/cask/lib/hbc/cli.rb:108:in `run_command'
/usr/local/Library/Homebrew/cask/lib/hbc/cli.rb:146:in `process'
/usr/local/Library/Homebrew/cask/cmd/brew-cask.rb:8:in `<top (required)>'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:55:in `require'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:55:in `require'
/usr/local/Library/Homebrew/brew.rb:19:in `require?'
/usr/local/Library/Homebrew/brew.rb:93:in `<main>'

Looks to be solved, auto-tap was readded and updated pre bug report directions


Sequence doesn't increase on data insert

System information:

Windows 10
DBeaver <IP_ADDRESS>009201907

Connection specification:

Postgrel SQL 12
PostgreSQL JDBC Driver

Describe the problem you're observing:
I insert data as manual to postgresql database. Sequence not increase automatically. I'm using a ORM(EntityFramework Core) for my project. When I insert data with orm, sequence is increasing automatically but it's not increase on dbeaver.
Steps to reproduce, if exist:
Insert data as manual.
Insert data with an orm. You will get an error about the duplicate identity key error.

I cannot reproduce the issue. Could you please write steps/DDL example. Here what I do in DBeaver:

there is no update on the issue for a long time.
If the issue is still actual for you - feel free to reopen it


add button to invert y-axis in side and main views

codenano uses Cartesian coordinates, and its 3D rendering crucially assumes y increases up.
cadnano uses screen coordinates, and cadnano users will expect helices to increase in index moving down.
To make these easy to switch between, add a Boolean that can toggle them.

Probably the easiest way to do this is to apply a second permutation array after helices_view_order, which reverses the order. So helices_view_order can still be specified, but if this option is checked, then the reverse of helices_view_order is used to put the helices in order.


Bad results on SeaThruNeRF dataset (i.e. Curasao)

Thanks for your great work for fast and photorealistic rendering. Recently, I put my efforts into underwater rendering by training f2-nerf. However, the result is not satisfactory. The SeaThruNeRF dataset is a llff-liked. Below are some experimental result figures. I hope that amidst your busy schedule, you can help me resolve my confusion. Thank you once again for your excellent work.

When I resize the images, it seems to work. But the result is still unsatisfactory.


open-channel does not work

When trying to open the channel with the following payload, I get an error.
Should be 120k sat
{
  "local_funding_amount": 120000,
  "node_URI": "024a8228d764091fce2ed67e1a7404f83e38ea3c7cb42030a2789e73cf3b341365@<IP_ADDRESS>:9735",
  "target_confs": 20
}

The error:
{
  "detail": [
    { "loc": ["query", "local_funding_amount"], "msg": "field required", "type": "value_error.missing" },
    { "loc": ["query", "node_URI"], "msg": "field required", "type": "value_error.missing" }
  ]
}

Does not work with CLN either

Was my bad, fixed by https://github.com/fusion44/blitz_api/issues/122

Needs to have a symlink for CLN:
  sudo ln -s /mnt/hdd/app-data/.lightning /home/blitzapi/

adding it to the install script.

Opened a channel from the UI with CLN, but it opened while an error was displayed likely related to: https://github.com/fusion44/blitz_api/issues/126#issuecomment-1171963281


In OperatingReport, Clarify what each EnergyAC value represents.

We need to clarify various instances of EnergyAC values that exist within OperatingReport and what they will represent. May need to add Usage Tips.

This could be a shortfall of the Editor itself. Doesn't allow users to add clarification to a specific use of an element or object. The definitions and usage tips will be applied generally to all instances of an element in the Taxonomy.

@cwhanse It looks like you have addressed this in your Github page. Do you see any development outside the pages that needs to be done to address this issue?

Yes, https://github.com/Open-Orange-Button/Orange-Button-Editor/issues/29 and #215 could help. I don't think we want context-specific Documentation strings; an element should mean one and only one thing. But adding Documentation to Arrays would help, as would adding Usage Tips to the Objects.


[Suggestion] Save messages to a file

Is your feature request related to a problem? Please describe.
Suggestion. Save messages to a file . Sometimes there is useful information in messages.
Describe the solution you'd like
One file per chat. Implement an option of what should be saved. For example, message ID, sender, sender's ID, message text itself. For a message that is a video or photo, write the name of the file with which it is saved to disk.
Describe alternatives you've considered
Saving manually T_T

Is your feature request related to a problem? Please describe. Suggestion. Save messages to a file . Sometimes there is useful information in messages.
Describe the solution you'd like One file per chat. Implement an option of what should be saved. For example, message ID, sender, sender's ID, message text itself. For a message that is a video or photo, write the name of the file with which it is saved to disk.
Describe alternatives you've considered Saving manually T_T

@Teomit  hi, More want to export the details of the message? What if it is written in the log？

@Teomit hi,I understand what you mean, pass the review.


Show direct links before giving consent, allow usage without JS

I wanted to make the iframemanager accessible without JS and add a direct link to the website, so here's a PR for that. New features:
Allows using any HTML tag
You can now use any HTML tag and not just <div>.
Works without JS
Without JS, the page can show any content inside the element, e.g. a normal link to a YouTube video.
When JS is enabled, everything in innerHTML automatically gets removed.
<div
    data-service="youtube"
    data-ratio="9:16"
    data-id="fJ9rUzIMcZQ">
    <a href="https://www.youtube.com/watch?v=fJ9rUzIMcZQ" target="_blank" rel="noopener">
        Watch on YouTube
    </a>
</div>

There is also a new attribute data-style to allow inline styling for the frame only when JS is enabled, to prevent showing large empty containers when JS is disabled.
Direct link when consent is not set
This shows a new "Open SERVICE_NAME" button next to the Load and Don't ask again buttons.
You can either set a link with the data-open-url attribute or, if you have the href attribute on the element or in any child element, it takes it automatically from there.
Service name is set in service definition under the name key, If not set, defaults to the capitalized key of the service definition.
Alternatively, you can change the text shown with the openBtn string in language config.

I do support the idea of direct linking ability (or whatever you want to show) when javascript is disabled (but I have not looked at the code at all). I actually got the same idea when looking at the code and trying testing add it to our site. We want something to make it work for browsers with js disabled too and got some similar ideas.

Hi @AviDuda,
I like the idea of adding a default "placeholder" in case javascript is not enabled, but I think it could be implemented in a simpler and more performant way compared to the numerous (and, in my opinion, some of them, not required) changes suggested by this pr. I'd like to keep the same "iframe-div-block" (not just a link) even when javascript is not enabled — with the placeholder message centered vertically and horizontally — to avoid any possible layout recalculation the moment that javascript kicks in.


Output file even on non-zero exit code

At the moment if the process returns a non-zero exit code then fsatrace does nothing. I find that surprising - I'd expect it to always produce the output, and then bubble the exit code back as well. Assuming you want to keep the default behaviour, an option would be useful.

Thanks!


Deprecate methods that are deprecated in Mockito.

Such as anyCollectionOf, anyListOf, etc.

Fixed in #67


How to use env options in VM options?

I want to use env variable in VM options. So i setup below is my config, but this will not working..

VM options not support variable value.
-Dtest1111=envTest1

VM options not support variable value.
-Dtest1111=envTest1

Thanks! oh...Tomcat in eclipse doing the trick for me.

@zengkid does this plugin have some internal variable for project location?

@zizifn  why not use Env variable directly?

@zizifn why not use Env variable directly?

It seems I can't get Env variable in my log4j.xml and tomcat_home\conf\catalina.properties, but i can get VM args in these file by using ${vaiableName}.

@zizifn please try to add the variable into your computer Environment.

@zizifn please try to add the variable into your computer Environment.

Thanks! Add into window env and echo variable in cmd, restart IDEA and still tomcat_home\conf\catalina.properties can't get variable value.

what do you config in tomcat_home\conf\catalina.properties?
and what do you want?

and I have debug the  VM value from env.
you can try to check your idea console the first line. the plugin call to start the tomcat by command line.

and I have debug the VM value from env.
you can try to check your idea console the first line. the plugin call to start the tomcat by command line.

Thanks! Set env in computer + reference env in VM args, work fine.(I forget reference env in my testing). Is possible Smart Tomcat can make env work when I set in IDEA run/debug configuration?

@zizifn  In fact, IDEA SDK only get computer Env to pass the data. I will try my best to find a way to support the feature in next version.

@zizifn In fact, IDEA SDK only get computer Env to pass the data. I will try my best to find a way to support the feature in next version.

Thanks very much! I'm fine with this and please close this issue if you want.

@zizifn Another alternative you can add into Path Variable.


Default to number of storm worker nodes instead of 2 for ackers/workers

This means we default to the number of nodes in config.json for sparse submit and 1 for sparse run (since it's only running on a single machine).
This also fixes an issue where 0 would be overwritten when someone passed it for --ackers or --workers.

Coverage decreased (-0.1%) to 45.59% when pulling 1768a5c80409427aae89cf22c5cfb9e39c0d66a3 on feature/change_worker_acker_default into<PHONE_NUMBER>5b44555e1f5284ee247b174ecdb695 on master.


Fix bug in dropoutnet.md

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Root package.json scripts should also apply to root dir

[ ] lint
[ ] format

What is there to be linted/formatted in the repo root? Both steps are done for *ts files only

I guess linting's not an issue, but Prettier can make sure code blocks in .md files are formatted nicely. (And less importantly .json files).

Now that lint-fix can take care of our source code, we can use yarn format for formatting configs. So I propose the following path:

Create lint-fix scripts in all packages and root (see ponferrada) for manual execution
Remove yarn format, as this is covered by yarn lint/yarn lint-fix
Create new format script that formats configs .json / .yml and does not run on files that are covered by eslint. (This one I'm not exactly sure about. Is there a good reason to keep jsom/yml/md out of the default prettier+autofixer setup?)

Is there a good reason to keep jsom/yml/md out of the default prettier+autofixer setup?

Best I can think of is that the prettier formatters for these kinds of files seem to be a bit more bug-prone than for .js/.ts, but I don't think that's a good enough reason not to include them by default.


Intercept as keda user

We're using keda and we're proud of it!

Awesome to hear, thanks for opening the PR!
One nit though, can you move it in alphabetical order please?

Done! :)


delete not working on 4.0.0-rc.1

Issue

Describe your issue here

Tried the release candidate, redisclient.delete just doesn't work.

Environment

Node.js Version: 12.20.0

Redis Server Version: 6.0.5

Platform: Amazon Linux 2

The command name is DEL, not DELETE. client.DEL and client.del should work.

This issue should be reopened, because the root issue is that typing is not up to speed.

RedisClientType<RedisModules, RedisLuaScripts> doesn't expose explicit method definitions, it's too easy to make a mistake and not notice until using a real redis server.


MCR1-1 unstake Logic Error

MCR1-1 unstake Logic Error
Auditor: Movebit
Code: MCR Settlement Contract
Severity: Major
Status: Pending
Code Location:
protocol-units/settlement/mcr/contracts/src/MCRLegacy.sol#248
Descriptions:
The unstake function does not check that the argument amount is greater than the
current epochStakes and the next epoch of epochUnstakes , resulting in the user being able to call the unstake function twice to make the number of epochUnstakes greater than the epochStakes .
Suggestion:
It is recommended to consider the number of users that have unstaked.

The MCRLegacy.sol script is no longer in use and will be removed in #537.


Refactor Trillian client with exported methods

This allows the Trillian client to be used in other parts of the codebase besides the api package. The changes included exporting the Response fields and all but one of the struct methods. Also removed ranges since these weren't used outside the api package.

Summary

Release Note

Documentation

Context is starting work on creating a stable checkpoint, where a goroutine outside of the api package will periodically fetch the latest checkpoint.

Moved to draft because I'm not certain I need this yet.

Ready for review!


Unit test toNumber.js

In the scope of this issue, unit testing is done toNumber.js utility library function.


SQL NullString in Where statement not being interpreted as  IS NULL in SQL.

GORM Playground Link
https://github.com/go-gorm/playground/pull/242
Description
When doing a Where clause with sql.NullString it will not add a WHERE field IS NULL to the query. I realize this is a limitation of you using zero/nil values to show a value doesn't exist in a field. But it would be nice to have a better way to have an IS NULL type kind of like what you have with the soft delete type gorm.DeletedAt

So I guess this a kind of feature request, because I understand this is a design limitation

Hello @omarhachach
You could use map with a zero value, just make it works with Valuer
https://gorm.io/docs/query.html#Struct-amp-Map-Conditions
And will enhance the Where API to make it works like Where(User{Nickname: sql.NullString{String: "Value", Valid: true}}, "Nickname")

Added
 Where(User{Nickname: sql.NullString{String: "Value", Valid: true}}, "Nickname")

The enhanced API would be great, since there 3 issues with using a map. The map won't have type safety, you can misspell the field, and it will do field = NULL not field IS NULL as is recommended in the SQL docs.

https://gorm.io/docs/query.html#Specify-Struct-search-fields

Oh, just in time! Very nice, thank you!

Do you have an ETA of when a tag including the new feature will be released?

About every 2 or 3 weeks


⚠ Fetchers has degraded performance

In 9df5b13, Fetchers ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 258 ms

Resolved: Fetchers performance has improved in 4266f60.


Fixes yield proc scheduling

The behavior is now the same for either spawn or sleep:

delay < 0: Proc will continue immediately if nothing is scheduled in the same tick. Otherwise, same as delay == 0.
delay == 0: Proc scheduled on end of same tick.
delay > 0: Proc scheduled after n ticks.

Closes #1262
This also fixes the ProcScheduler being able to still have queued procs when running multiple tests in serial.

Apparently this is still unperformant for the issue in question

Somethings still fucky, which is why the TGS tests are failing


buf and win -api calls not working

Describe the bug
When trying to call the win -api calls, I get a lua error that says :
Error executing lua callback: ...local/share/nvim/lazy/neominimap.nvim/lua/neominimap.lua:56: module 'neominimap.command.window' not found:
To Reproduce
Using require("neominimap").winOff({}, { win = win }) or winOff
Expected behavior
no error
Please complete the following information:

OS: macOS
Neovim Version: 0.10.1

I noticed the modules for neominimap/command/window and neominimap/command/buffer have been renamed to win and buf in this patch here:

So I did some changes in the neominimap.lua file and this seem to help:

Have a nice weekend :^)

Thank you for pointing this out!


PP-4462 Stripe setup middlewares

WHAT
This PR adds two middelwares which can be used for Stripe setup page validations:

Restrict to live Stripe account middleware
Check bank details not submitted middleware

with @stephencdaly

Reviewed under :fire:


[HELP] - (CORS policy?) "Error initializing environment"

Describe the bug
an error appears in the right pane:
Error initializing environment, please raise an issue using the help link.
In chrome dev tools I seen an error
Access to XMLHttpRequest at 'https://api.play-with-go.dev/guides' from origin 'https://play-with-go.dev'
has been blocked by CORS policy: Response to preflight request doesn't pass access control check: No
'Access-Control-Allow-Origin' header is present on the requested resource.

To Reproduce
navigate to https://play-with-go.dev/retract-module-versions_go119_en/
error appears on the right
Expected behavior
no error, site loads correctly
Screenshots
N/A
Desktop (please complete the following information):

OS: OSX
Browser: chrome
Version: 129.0.6668.101 (Official Build) (x86_64)

Additional context
N/A

Looks like the API server is down, I am getting a 502 response from https://api.play-with-go.dev:
curl -v 'https://api.play-with-go.dev/'
* Host api.play-with-go.dev:443 was resolved.
* IPv6: (none)
* IPv4: <IP_ADDRESS>
*   Trying <IP_ADDRESS>:443...
* Connected to api.play-with-go.dev (<IP_ADDRESS>) port 443
(...)
* using HTTP/2
* [HTTP/2] [1] OPENED stream for https://api.play-with-go.dev/
* [HTTP/2] [1] [:method: GET]
* [HTTP/2] [1] [:scheme: https]
* [HTTP/2] [1] [:authority: api.play-with-go.dev]
* [HTTP/2] [1] [:path: /]
* [HTTP/2] [1] [user-agent: curl/8.7.1]
* [HTTP/2] [1] [accept: */*]
> GET / HTTP/2
> Host: api.play-with-go.dev
> User-Agent: curl/8.7.1
> Accept: */*
>
* Request completely sent off
< HTTP/2 502
< content-type: text/html; charset=UTF-8
< referrer-policy: no-referrer
< content-length: 332
< date: Tue, 22 Oct 2024 06:09:49 GMT
< alt-svc: h3=":443"; ma=2592000,h3-29=":443"; ma=2592000
<

<meta http-equiv="content-type" content="text/html;charset=utf-8">
<title>502 Server Error</title>

Error: Server Error

The server encountered a temporary error and could not complete your request.
Please try again in 30 seconds.

* Connection #0 to host api.play-with-go.dev left intact


Issue with priority of headers

Custom request headers should replace default headers, if provided. This PR fixes this issue.

excellent, thank you :+1:


Alex/further cleanup

Moves the custom operators out of the global scope now that Swift 3 supports overloaded operators within extensions.
Updates the project settings to comply with Xcode 8's recommendation.

are there tests?

Outside of the pre-existing tests, I didn't add any myself in this PR.
I plan to add more test coverage in a future PR.

Once the spacing issues in function declarations (i.e., replacing ==( with == ( ) are addressed, I'm good with the changes.  There are some good ideas in here for some re-implementations of the algorithms, but those shouldn't happen in this PR - let's keep this one scoped to simply "moving" the implementations into the "correct" place.

Noting down the conversation and proposed changes for a separate PR.


context export command

Context should be exported to contextName.json in the current folder.
command example: sasjs context export <contextname> -t targetName

:tada: This issue has been resolved in version 1.11.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Thycotic source has incorrect data

https://splunk.github.io/splunk-connect-for-syslog/1318/sources/Thycotic/
Thycotic source has Tenable information as the header and product information.
Currently has this
Vendor - Tenable
Product - Tenable.nnm

Ref
Link

Splunk Add-on
https://splunkbase.splunk.com/app/4060/

Product Manual
https://docs.tenable.com/integrations/Splunk/Content/Splunk2/ProcessWorkflow.htm

Should have this
Vendor - Thycotic
Product - Thycotic

Ref
Link

Splunk Add-on
https://splunkbase.splunk.com/app/5327/

Product Manual

:tada: This issue has been resolved in version 2.0.0-next-major.23 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


v5: Move to Ubuntu 24

This PR is an attempt to move to use Ubuntu 24 by default in CI images.

Your development orb(s) have been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link(s):
https://circleci.com/developer/orbs/orb/geos-esm/circleci-tools?version=dev:c2d521cfb56beb3ab4da788a7d3982a363426ee3
https://circleci.com/developer/orbs/orb/geos-esm/circleci-tools?version=dev:alpha

Your development orb(s) have been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link(s):
https://circleci.com/developer/orbs/orb/geos-esm/circleci-tools?version=dev:c2d521cfb56beb3ab4da788a7d3982a363426ee3
https://circleci.com/developer/orbs/orb/geos-esm/circleci-tools?version=dev:alpha

Your orb has been published to the CircleCI Orb Registry.
You can view your published orb on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/geos-esm/circleci-tools?version=5.8.0


Post-Login automation does not pass command to another shell

What steps will reproduce the problem?

I am trying to do a Wake-on-WAN on my Pirelli PRGAV4202N-Router.

1. Set up the SSH-Connection to my Router properly and confirm it's working

2. I set the following in Post-Login automation:

system shell
ether-wake -b -i br0 [MY MAC-ADRESS]
[IMAGINE A LINEBREAK HERE]

3. Connect

What is the expected output? What do you see instead?
"system shell" calls a downgraded busybox shell inside the routers firmware.
This works properly via automation. The ether-wake command should then be
executed, but isn't. The Problem is that all further commands after "system
shell" are not executed nor are they written.
They simply do not "survive" the call of another shell.

The command works when typed in manually!

The Output is:
----------------------------------------------------------
OpenRG> valu [***PASSWORD***]

Returned 0
OpenRG>
OpenRG>
OpenRG> system shell
Temporary setting log_level off

BusyBox v1.01 (2005.09.07-07:38+0000) Built-in shell (lash)
Enter 'help' for a list of built-in commands.

/ #
----------------------------------------------------------

What version of the product are you using (you can see this by using Menu
-> About in the Host List)?
ConnectBot 1.7.1-pslib (Just because of the PW-autosave function).
The original version doesn't work either.

What type of system are you trying to connect to?
Pirelli PRGAV4202N Wireless Router

If you are able to connect, what is the output of "echo $TERM", "uname -a",
and any other relevant information on the host?

ECHO $TERM: "VT102"
uname -a: "uname: No such file or directory"

The Router uses something like a "Jungo OpenRG"-Software
I can not determine, what the system exactly is.

Please provide any additional information below.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 4 Feb 2012 at 1:50

This is still an issue

This is still an issue


Added example math server

Is there a reason why we're using the Math example for C# rather than the Route Guide we've used for most/all of the other languages? Or are we going to add a Route Guide too?

Math server is our internal development thing (actually, other languages like ruby implement it as well). Route guide is more a public tutorial (and it's in different repo and should depend on Grpc package, not the source code itself). I am totally planning to add route guide for C# like other languages do.

Gotcha - thanks


crc32 computation partly redundant?

While profiling attic, I found that crc32() calls make up 4.2s of a total of 60s "attic create" runtime.
It seems that attic computes a crc32 over some object metadata (id, length, tag) AND the payload data.
Isn't this redundant considering that the payload data already has a sha256 (unencrypted repo) or even a HMAC (encrypted repo) inside? So, could the payload data be excluded from the crc32 computation without losing anything?

Is really 7% of all cpu time spent in crc32 during a normal "attic create", that sounds very high?
the crc32 is very useful for server side repository checks and repairs without needing access to any encryption keys.

BTW I would hope that is really crc32c since that is supported in hardware
by current processors.
On Thu, Apr 16, 2015 at 6:12 PM, Jonas Borgström<EMAIL_ADDRESS>wrote:

Is really 7% of all cpu time spent in crc32 during a normal "attic
create", that sounds very high?
the crc32 is very useful for server side repository checks and repairs
without needing access to any encryption keys.
--
Reply to this email directly or view it on GitHub
https://github.com/jborg/attic/issues/281#issuecomment-93851221.

The numbers are from a cProfile run and ofc may vary depending on the hw and attic version used. I used my code which spends quite some less cycles overall, so the (relative) percentage of time spent for crc32 might be higher than in release attic while the (absolute) amount of cycles is likely similar.

Related:
http://stackoverflow.com/questions/26429360/crc32-vs-crc32c
https://cloud.google.com/storage/docs/gsutil/addlhelp/CRC32CandInstallingcrcmod
https://code.google.com/p/googleappengine/source/browse/trunk/python/google/appengine/api/files/crc32c.py?r=170

Guess we can close this one, (server side) repo checks w/o having to decrypt/decompress is a good thing.


An exception occurs when running anonymizer.py on the Raspberry Pi 4B&USB Accelerator

During the operation of anonymizer.py, when everyone re-entered the screen, there was a problem: the body line drawing in each frame before will not disappear, and it has been piled up in the screen.

Hi, please provide the following details:

python version
Operating system
steps to reproduce the issue.

Hi
raspi4B + ubuntu Mate20.04 + Coral USB Accelerator + python3.8.8
aaa@aaa:~/work/coral/00_posenet$ python3 anonymizer.py
/home/aaa/work/coral/00_posenet/gstreamer.py:15: PyGIWarning: Gst was imported without specifying a version first. Use gi.require_version('Gst', '1.0') before import to ensure that the right version gets loaded.
from gi.repository import GLib, GObject, Gst, GstBase, GstVideo, Gtk
/home/aaa/work/coral/00_posenet/gstreamer.py:15: PyGIWarning: GstBase was imported without specifying a version first. Use gi.require_version('GstBase', '1.0') before import to ensure that the right version gets loaded.
from gi.repository import GLib, GObject, Gst, GstBase, GstVideo, Gtk
/home/aaa/work/coral/00_posenet/gstreamer.py:15: PyGIWarning: GstVideo was imported without specifying a version first. Use gi.require_version('GstVideo', '1.0') before import to ensure that the right version gets loaded.
from gi.repository import GLib, GObject, Gst, GstBase, GstVideo, Gtk
/home/aaa/work/coral/00_posenet/gstreamer.py:15: PyGIWarning: Gtk was imported without specifying a version first.
Waiting for everyone to leave the frame...
Waiting for everyone to leave the frame...
Waiting for everyone to leave the frame...
Waiting for everyone to leave the frame...
Waiting for everyone to leave the frame...
Waiting for everyone to leave the frame...
Waiting for everyone to leave the frame...
Waiting for everyone to leave the frame...
Waiting for everyone to leave the frame...
Waiting for everyone to leave the frame...
Waiting for everyone to leave the frame...
Waiting for everyone to leave the frame...
Waiting for everyone to leave the frame...
Waiting for everyone to leave the frame...
Background set.
The result on the screen is like this：

@3344393853  try the following command let us know still you are facing issues.
python3 anonymizer.py --mirror

@3344393853 try the following command let us know still you are facing issues.
python3 anonymizer.py --mirror

@hjonnala I try the following command still encounter the issue

@yishinhung Thanks for confirming. It is bug on our side, we are working on it. Will let you know once it is resolved.


💡 Working on flawlessly on Windows

Describe the feature
Because the readme mentioned that testing is done on Linux, I just wanted to mention that I installed on windows just now, and it worked without any hickups. Feel free to close this ^_^
An idea of implementation
No response

Thanks for the heads up @KronosTheLate ! Nice cross-platform job from the tauri team :D


Pages: add new page about Twitter Impersonation

In https://github.com/bitcoin-dot-org/bitcoin.org/issues/1842 @midnightmagic requested a page explicitly mentioning the Twitter impersonation accounts are fake.  This PR adds a short statement to that effect, including mentioning Bitcoin Core's official twitter account:

The page is added to the Contact menu:

Note that I reported one of the accounts (@BcoreProject) to Twitter about a month ago and it's still there, so I'm not sure we can expect much action from Twitter on this issue.

It is already expliacitly mentioned on https://bitcoincore.org/en/2017/08/18/btc1-misleading-statements/. Hopefully that is enough for whatever Twitter needs to take down the other account.

@TheBlueMatt fair enough.  I'll leave this issue open for another day to see if @midnightmagic has anything to add, since the request was his.

Mentions about the official twitter and/or webpages are indeed there. I have been actively complaining about the silly impersonation accounts for weeks. It does not appear to be enough to convince twitter. They need a phone number, etc. The procedure is public to close down impersonation, non-parody accounts.
Perhaps they're just not listening to me.

I think we should a) merge this, b) update the text to note that this is a pervasive issue on twitter, not just bitcoin-core-specific. I think this represents an important community service announcement noting that people should generally not trust anything they read on twitter is coming from the person it claims to be without very careful analysis

Reopening.  Will make suggested edits presently.

Updated page content:

Maybe mention something about not trusting any download links or update announcements specifically, given the security issues there best to be very explicit imo. Otherwise, ACK.
On October 10, 2017 2:54:24 AM EDT, "David A. Harding"<EMAIL_ADDRESS>wrote:

Closed #447.
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/bitcoin-core/bitcoincore.org/pull/447#event-1285418766

Added notice about downloading software, a link to the download page (/bin for now; will either update this or #452 depending on which gets merged first), and a link to instructions for verifying binaries using GPG.  In addition, I added a paragraph telling people that if they received a security announcement whose origin they weren't sure about, they can just shut down their node to safely take time to investigate.
Also made another edit suggested out of band.
If this looks good, I'll squash before the final merge.

ACK

utACK 10ce3ceaafea6deedcfaebbcfd9602ab028c4962

ACK

Rebased and squashed with no changes from 10ce3ce


Abstract classes with implemented methods

The fact that methods inside abstract classes can have an implementation is suggested but I don't think it is obvious enough.
The abstract modifier indicates that the thing being modified has a missing or incomplete implementation.
I think this is not enough, more should be said about this or / and an example could be shown where an abstract base class has an implemented function.

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

ID: c00e23c2-618d-0b84-8ea8-0b0b4de1101a
Version Independent ID: 8c00f924-bf5a-a748-331d-3809321e5547
Content: abstract - C# Reference
Content Source: docs/csharp/language-reference/keywords/abstract.md
Product: dotnet-csharp
Technology: csharp-language-reference
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

That's a good suggestion @66Gramms
I've added it to our backlog to address when we next update this sectionn.

I'd like to do this if possible. I can make a PR containing the newly edited document either this weekend or at some time next week.


Fix running tests from VS by adding redirects for Castle.Core and Newtonsoft.Json

@natemcmaster

:shipit:


Typo "Azure file storage"→"Azure File Storage"

https://docs.microsoft.com/en-us/azure/architecture/example-scenario/file-storage/moodle-azure-netapp-files
https://github.com/MicrosoftDocs/architecture-center/blob/main/docs/example-scenario/file-storage/moodle-azure-netapp-files-content.md
#PingMSFTDocs

@arntdg
Can you review this PR?
Note that Microsoft branded products/services include Microsoft Azure Storage or Azure Storage ... (NOT Azure File Storage)... it's an umbrella term for Blob Storage, Azure Files, Queue Storage, and Table Storage. Thanks.
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

I'm confused. The article talks about 'Azure NetApp Files', which is a first party Microsoft service, and a such a Microsoft product. What is the issue, what should change in the article, kindly expand. Thank you.

@arntdg - responded via email.

@arntdg - merging this now. thanks.


Sync CHANGELOG.md, Atmosphere releases (and Github releases if you're feeling generous)

This is a polite request to please keep release history updated.
As a justification by example, I need to roll back to the last release on Beta 0. Atmosphere lists a history of release versions but no information for each (that I can find). The description links to CHANGELOG.md in Github but that file only lists versions 0.3.0 and 0.3.2. Even the description for those two is vague (ie, "Angular 2 upgraded to the last version").
By looking at commit history I think I need 0.3.7, but this seems like more effort than you want new adopters to go through. I think having dep versions in the CHANGELOG.md is reasonable.
thanks, and thanks for the great work!

I will change the title to make it a task, let me know if it makes sense.
Generate changelog automatically from commit messages.

We'll  have a proper look on the change log from now on (auto log is added as well)


Fluid Navigator Transition not working in iOS?

I have used the transition tag to change the size of the image when the navigation takes place. The transition is working fine in Android but it doesn't seems to work in iOS? Can you please tell me why this is happening?
React native ver: 0.61.5
React navigation version: 4.0.10,
react-navigation-fluid-transition: 0.3.2

I downgraded React-Navigation to 3.13.0 and it worked.


Modifying Smart TV section

For the smart TV part, putting software like Kodi is not correct. The operating system is the main issue

So.... you just removed everything and for some reason only added and left LineageOS? FOR TVs???????

Yeah . Smart TV is in the operating systems section. So kodi is not for this part. Lineage OS can be installed for some Android boxes.

I would like to keep kodi and others, maybe we should create a new section for them?

I think you should put them in "Media Streaming Platforms" O_o?


[BUG]  slow working

Expected Behavior

Current Behavior

Possible Solution

Steps to Reproduce

Screenshot

Log output

Log Output
paste log output...

Context (Environment)

Browsers Affected

[ ] Chrome
[ ] Firefox
[ ] Edge
[ ] Safari 11
[ ] Safari 10
[ ] IE 11

Duplicate, closing in favour of: https://github.com/webb-tools/webb-dapp/issues/1210


Disable fieldGroup based on the parent field

I'm submitting a ...  (check one with "x")
[ x] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request

Current behavior

To disable all the fields of a fieldgroup, need to add disable property/expression at each field level.

Adding disabled attribute at parent fieldGroup, not making its children disabled.
Adding disabled property via expression properties at parent works only on the initialization of parent field.

Expected behavior

If the parent fieldGroup is disabled, make all the fields disabled based on the parent disabled property
Minimal reproduction of the problem with instructions

https://stackblitz.com/edit/angular-qzjgpu
What is the motivation / use case for changing the behavior?

Please tell us about your environment:

Angular version: 5.0.X

Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

Language: [all | TypeScript X.X | ES6/7 | ES5]

Node (for AoT issues): node --version =

already implemented by #1088,
see my comment when fieldGroup and key are set https://github.com/formly-js/ngx-formly/issues/704#issuecomment-364661614
I looking to make model value consistent to avoid a such confusion but it'll introduce a breaking change which I don't know how to deal with it for now


Use setting to determine which type of accounts support seasonal addresses

For this and similar functionality we should probably have a method that gets an account record type ID and returns true or false depending on whether functionality is active for that type. Something like:
public static Boolean isSeasonalAddrActiveForRecType(ID recTypeID)

**lurch: attach W-009782

Tracking W-009782

Included in beta release version 1.8 (Beta 5) and higher

Included in production release version 1.8


Occasional fails with circleci

I have seen errors along these lines a couple of times now:

cistern: Provider circleci-0: monitorPipeline failed with parsing time "" as "2006-01-02T15:04:05Z07:00": cannot parse "" as "2006" (https://circleci.com/gh/sunpy/sunpy/36079?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link)

It seems to happen on a newly pushed branch, maybe force pushes? Let me know if there is  anything I can do to get more details when it happens.

Thanks for testing this. I just merged my changes in master.


add basisgold.eth space

add basisgold.eth space

finish review and add basisgold.eth space

finish review and add basisgold.eth space

The images file size should not exceed 50KB. Please reduce their weight.

And should be squared format.

The images file size should not exceed 50KB. Please reduce their weight.

And should be squared format.

please check it,   thank you

please check it,   thank you


I cannot get these merges through the site. Shucks.

I'm going to be using this as a bookmark to indicate what commits to the main I have also paired into my fork.

As of this comment, all of the above changes have been merged.

Done as of 12/1/2021 5:35 PM Local

Done as of 12/8/2021 10:22 AM Local

Done as of 12/12/2021 10:18 AM Local

Done until "Add thread names"

Done as of 12/30/2021 at 8:08 PM Local


set up /teach page and css tests

Fixes #[Add issue number here]
Hi Lauren! I've set up a teach page with some tests going on, and I have a question about the 3rd image below:
Screenshots of the change:

css tests on root/teach page

added an a tag to Home > Get Involved > Teach a workshop or class.

I have added "&" to Learn-title,
and I was going to add a "Teach" link on the src/pages/learn/index.hbs like in the image above but it seems like the the hbs file is sort of autogenerated and I couldn't save the newly added "Teach" link. Do you have any idea on this problem? Thank you :)

Oh and I wished to add Qianqian as a reviewer, but I couldn't find her on the reviewers list. Is there a way I could add Q as a reviewer? Thank you :-)

hi @yinhwa ！

@Qianqianye you can merge this whenever it looks good to you


Compile SCSS from custom styles in browser

SCSS is correctly exported in code.
Since it's not compiled in browser, wysiwyg editor won't correctly show it.
It only correctly shows plain css at the moment.
We should compile scss -> css before applying classes to editor.

We might be able to do this with emotion


No bicep processing with AZURE_PARAMETER_FILE_EXPANSION

Description of the issue
As explained in "Using parameter files" docs, it should be possible to let PSRule discover a parameter file, and analyse the associated bicep file using the parameter file. This obviously does not work (at least in the versions stated below).
To Reproduce
Steps to reproduce the issue:
Here is an example.
# file: ps-rule.yaml

include:
  module:
    - PSRule.Rules.Azure

requires:
  PSRule: '>=2.7.0'
  PSRule.Rules.Azure: '>=1.24.0'

configuration:
  AZURE_PARAMETER_FILE_EXPANSION: true
  AZURE_BICEP_FILE_EXPANSION: false

execution:
  suppressedRuleWarning: false
  notProcessedWarning: false

// file: storage.bicep

@description('Storage Account type')
param storageAccountType string

@description('Location for the storage account.')
param location string = resourceGroup().location

@description('The name of the Storage Account')
param storageAccountName string = 'store${uniqueString(resourceGroup().id)}'

resource sa 'Microsoft.Storage/storageAccounts@2021-06-01' = {
    name: storageAccountName
    location: location
    sku: {
        name: storageAccountType
    }
    kind: 'StorageV2'
    properties: {}
}

# file: storage.test.bicep

targetScope = 'resourceGroup'

param storageAccountType string
param location string = resourceGroup().location

@description('Minimum configuration.')

module sa 'storage.bicep' = {
    name: 'storage'
    params: {
        storageAccountType: storageAccountType
        location: location
    }
}

file below: "storage.test.parameters.json"
{
  "$schema": "https://schema.management.azure.com/schemas/2015-01-01/deploymentParameters.json#",
  "contentVersion": "<IP_ADDRESS>",
  "parameters": {
    "storageAccountType": {
      "value": "Standard_LRS"
    }
  }
}

Expected behaviour
Now when you run PSRule, the bicep files should be analysed, and according rule processing messages should be printed by PSRule.
This does not happen. Instead, PSRule does not print anything. Note that I also tried the usage of the "metadata"-attribute, as described in the docs, which made no difference.
When you instead reconfigure the files above with "AZURE_BICEP_FILE_EXPANSION: true" for regular bicep file expansion, PSRule performs as expected, and prints rule evaluations as expected.
How does this need to be configured, to have parameter file expansion plus analysing the associated bicep files, given the example above? To me, the configuration above looks ok (given the docs), so this looks like a bug.
Module in use and version:

Module: PSRule.Rules.Azure

Version: 1.24.2

Module: PSRule

Version: 2.7.0

Captured output from $PSVersionTable:
Name                           Value
----                           -----
PSVersion                      7.3.2
PSEdition                      Core
GitCommitId                    7.3.2
OS                             Microsoft Windows 10.0.19044
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Additional context

@khaliddermoumi Thanks for raising the issue.
The example above does not include the metadata property. Are you able to try the following as per the instructions:

Alternatively, you can choose to expand and test a Bicep module from JSON parameter files by metadata.

{
  "$schema": "https://schema.management.azure.com/schemas/2015-01-01/deploymentParameters.json#",
  "contentVersion": "<IP_ADDRESS>",
  "metadata": {
    "template": "./storage.bicep"
  },
  "parameters": {
    "storageAccountType": {
      "value": "Standard_LRS"
    }
  }
}

@khaliddermoumi Thanks for raising the issue.
The example above does not include the metadata property. Are you able to try the following as per the instructions:

Alternatively, you can choose to expand and test a Bicep module from JSON parameter files by metadata.
(...)

Hi Bernie,
I also tried to use the "metadata" property, but it does not work too - in the same way, no bicep analysis.

Alternatively for the storage.tests.bicep you can just set a default for storageAccountType. You can find some examples here: https://github.com/Azure/PSRule.Rules.Azure-quickstart/blob/main/bicep/modules/keyvault/v1/.tests/main.tests.bicep

I would really prefer a lot to get it to run with parameter files, because we already have a lot of them. For our setup, we would need exactly this feature.
How do we get it to run?

@khaliddermoumi Your ps-rule.yaml configuration is fine.
Are you able to provide some additional information:

Are you using the DevOps extension tasks/ GitHub Action/ or PowerShell command-line?
Are you able to provide a snippet of your pipeline or PowerShell command-line?
Are you able to provide an example of the output?

Hi,
I was using the PS CLI, and this command: "Invoke-PSRule -InputPath .".
Using your command line, I get the same output as you do - thanks a lot.
I was assuming "Invoke-PSRule -InputPath ." was ok, as this worked with the "bicep expansion".
So I see, the command was obviously the reason it didn't work.
Thanks a lot for your help!


ListPipeline omits details when a user doesn't have access

When a user does ListPipeline and requests details, we attempt to deserialize the spec commit for every pipeline into the response. A user may not have permissions to view every spec commit - previously this caused ListCommit to fail. Instead, this PR omits the details from a pipeline entry when the user doesn't have access. This shouldn't have much user-facing impact since pachctl list pipeline doesn't request the details or display any information from them.
This also removes the --spec flag for pachctl list pipeline, because (presumably for performance reasons) pachctl list pipeline no longer requests the spec commits for every pipeline, which meant the command was useless and stack-traced.

Dropping this in favour of making spec repos world-readable, since they're an implementation detail.


Option to change PHP path

It would be nice if there were an option to change the following option without having to uninstall => install:

This is crucial if you have multiple Xampp installations.
Refs https://github.com/composer/composer/issues/6277

Sorry, but this is not an installer issue.
You need to change your PATH environment variable so that it includes the location of the version you want to use. You can use the Control Panel to do this, or you can do it from your shell.
(You can also run Composer Setup again, there is no need to uninstall first)

I didn't say it's an installer issue, the screenshot is to show that I'm not talking about Composer but PHP path.
Is there any doc that shows how and where to change this PATH ?

Yes, it is called the internet.

You can change php version of composer without uninstalling it, follow these steps :

Search for system environment variables in cortana.
Click on the button "Environment variables".
Under "System variables" select path and click on edit, you will see one entry like this "C:\wamp\bin\php\php5.6.13".
Just change this to the folder name of the php located at your wamp/bin/php7.1.9, here bin7.1.9 is folder name.
Replace php5.6.13 with bin7.1.9, it will look like these "C:\wamp\bin\php\php7.1.9", just click ok on all the boxes.
You are done.
To verify, first close all the cmd windows, than open cmd and type "php -v", press enter and you should see php7.1.9.
If you don't see change in php version than just restart your pc and run php -v again in cmd , it will work.

thanks, i use this to change the php path, worked without typing the number version 👍


[5.4] Attempt to simplify validation with an edge case

This PR attempts to solve the issue with using required_* rules while the ConvertEmptyStringToNull middleware is applied.
Given the following data & rules:
validator(
    [
        'address_one' => null,
        'address_two' => null,
    ],
    [
        'address_one' => ['nullable','required_with:address_two', 'min:6'],
        'address_two' => ['nullable','required_with:address_one', 'min:6'],
    ]
);

With this PR, using nullable won't skip the required_with validation if the value is null and will still fail, this solves the case when you accept null as a value for all rules but rules that imply the field is required.

Doesn't nullable mean NULL is acceptable in addition to other valid values? This fix has broken my app assuming that NULL is always a valid value even in presence of implicit checks, and I have to patch my validator to revert to the pre-fix behaviour.

@themsaid is this affecting required rule or it is only for conditional required rules?


Images matched with themselves in pairselection.py

During the matching phase each image is also matched agains itself.
The problem shows when using bin/opensfm match_features <folder>
In file pairselection.py
line 298 pairs = set([tuple(sorted([i, j])) for i in images_ref for j in images_cand])
A quick fix should be
line 298 pairs = set([tuple(sorted([i, j])) for i in images_ref for j in images_cand if i!=j])

fixed. Thanks, @santelelle


treewide: Reformat all nix blocks in markdown files

Description of changes
Follow-up from #299554. Currently a draft since we need to rework CONTRIBUTING.md first (and also add a GH workflow).
cc @drupol @DanielSidhion @fricklerhandwerk since you may be interested in the attached script
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Ah also cc @hsjobeki since you said this task would be either hard or impossible (not sure what you said) ;)

Very nice work! If you'd be up for the challenge, we could also check the generated .md files. I thought of those that are generated from nixdoc from doc-comments ( here This would ensure doc-comments that declare nix examples would actually evaluate. But i feel like this has some overlap with the doc-tester idea. And didn't yet look into its details.

@hsjobeki not sure how well that would work tbh. I checked and I see blocks like:
cli.toGNUCommandLine {} {
  data = builtins.toJSON { id = 0; };
  X = "PUT";
  retry = 3;
  retry-delay = null;
  url = [ "https://example.com/foo" "https://example.com/bar" ];
  silent = false;
  verbose = true;
}
=> [
  "-X" "PUT"
  "--data" "{\"id\":0}"
  "--retry" "3"
  "--url" "https://example.com/foo"
  "--url" "https://example.com/bar"
  "--verbose"
]

cli.toGNUCommandLineShell {} {
  data = builtins.toJSON { id = 0; };
  X = "PUT";
  retry = 3;
  retry-delay = null;
  url = [ "https://example.com/foo" "https://example.com/bar" ];
  silent = false;
  verbose = true;
}
=> "'-X' 'PUT' '--data' '{\"id\":0}' '--retry' '3' '--url' 'https://example.com/foo' '--url' 'https://example.com/bar' '--verbose'";

That's not really valid nix but I see why it is like that, I cannot really change that. Any idea how to improve on that? Maybe split one block into two where the first one is valid nix and the second one a plain one?

how well that would work tbh. I checked and I see bloc

Right seems rather complicated to somehow parse those example since they are all different and cannot be parsed into an AST. Without a proper convention.
We need to introduce a good convention for those examples before we try to parse them.
One idea. Since nix doesn't use the fat arrow => we could split the example there and see if the head is parseable.
Since this => is already kind of established. One difficulty is that we would need some context awareness to make sure => comes after the expression. (Means we have to custom build our own nix example parser :smile:  )

@hsjobeki I strongly urge to avoid piling up syntax or mechanisms and just as strongly support @dasJ's suggestion: markdown already parses code blocks, and that can easily be leveraged for making the distinction at the presentation  layer with CSS (we do this successfully on nix.dev). It can, with slightly more effort, be used for extracting input-output-pairs for tests.
We don't even have to fix that now. Should be enough to agree on a general direction and merge strict improvements on that path independently.

👍 That's what I was implying here.


weights in cross entropy function

def cross_entropy_one_hot(input, target):
_, labels = target.max(dim=1)
return nn.CrossEntropyLoss(weight=class_weights)(input, labels).
What is weight=class_weights for different datasets

Here, it was unnecessary so I removed it but in general weights can be in the case of imbalanced datasets where there is an unequal distribution of classes within dataset.


Added functionality in livecoms.cls to break URLs on hyphens.

This affects both the main document and bibliographies. Addresses issue #88

Thanks, @dwsideriusNIST !


[Q] OOM error with sweep (with videos input)

I am running a sweep in my code (which uses video input data), and running into out-of-memory (OOM) error.
Is there intensive memory allocation when for just running sweeps? Plus, what is the garbage collection before a next sweep starts? Is there anything I can add to my code to avoid this or should this already be handled by wandb?
I recognise that my code has heavy-weight architectures / input data / computational graph and in case memory for those are not cleared between successive runs it will cause OOM error.

Hello @shivanimall thank you for writing in and happy to help. could you please share the following:

code snippet of your code or a toy example for reproducing this on our end
how are you configuring your sweep

Hi @shivanimall , since we have not heard back from you we are going to close this request. If you would like to re-open the conversation, please let us know!


Use Inline Keyboard for chat configurations

Telegram Bot
Implement Inline Keyboard for language and algorithm configurations.
Display the list of options in Inline boxes and apply the option selected by the user.

Completed on https://github.com/nminaya/grammar-nazi-bot/commit/56a9475a063de5f5df65aeadcf8bc23e16f388ec

Completed on https://github.com/nminaya/grammar-nazi-bot/commit/56a9475a063de5f5df65aeadcf8bc23e16f388ec


expose version/release on footer

Would be helpful to have the release version displayed in the footer near Powered by JIT Access

Version 1.4.1 now includes this feature.


Paste menu item doesn't work in sandboxed item

We use monaco editor in an iframe sandbox that doesn't permit the navigation.clipboard API. (We don't own the parent frame).
So the ctrl + V paste option work but the Paste option in the context menu doesn't work because the frame doesn't allow it.
Is it possible to change the implementation of the paste menu context item so it'll work like the ctrl + v shortcut?
Paste menu item:

Sandbox:

When pressing Ctrl+V, the editor doesn't treat in any way the keydown/keypress events, so it doesn't call preventDefault() on them, which:
This causes a paste event to be created by the browser and we use that event to get the paste data. That is why using Ctrl+V can paste without any browser prompts.
The editor context menu is custom-drawn via dom nodes and all the actions in there, including Paste are basically simple dom nodes. Clicking on Paste does not generate a browser paste event, so that is why we must reach the clipboard somehow differently. So we first call document.executeCommand('paste') and if that fails we fall back to navigator.clipboard.readText.
In this particular case, for which I pushed a repro test case, navigator.clipboard.readText is defined inside the iframe but invoking it always throws an error: DOMException: The Clipboard API has been blocked because of a Feature Policy applied to the current document. See https://goo.gl/EuHzyv for more details
The best we could hope for in this case is to hide Paste from the context menu, but unfortunately I could not find anything on the navigator or on navigator.clipboard which would indicate that navigator.clipboard.readText will permanently throw. I have also followed the link https://goo.gl/EuHzyv to https://sites.google.com/a/chromium.org/dev/Home/chromium-security/deprecating-permissions-in-cross-origin-iframes and got to read a bit https://w3c.github.io/webappsec-permissions-policy/ , but the advertised way of finding out the current policies (document.permissionsPolicy) does not appear to be implemented yet.

Do you know of a way to detect that the current window is a restricted cross-origin iframe? If we could detect that, we could at least hide Paste from the context menu.

Thanks for driving this.
+Adding folks in our end as we have a customer meeting today on the status of this issue.

מאת: Alexandru Dima<EMAIL_ADDRESS>נשלח: יום רביעי, דצמבר 30, 2020 1:23 לפ׳
אל: microsoft/monaco-editor
עותק: Raz Ronen; Author
נושא: Re: [microsoft/monaco-editor] Paste menu item doesn't work in sandboxed item (#2264)

When pressing Ctrl+V, the editor doesn't treat in any way the keydown/keypress events, so it doesn't call preventDefault() on them, which:
This causes a paste event to be created by the browser and we use that event to get the paste data. That is why using Ctrl+V can paste without any browser prompts.
The editor context menu is custom-drawn via dom nodes and all the actions in there, including Paste are basically simple dom nodes. Clicking on Paste does not generate a browser paste event, so that is why we must reach the clipboard somehow differently. So we first call document.executeCommand('paste') and if that fails we fall back to navigator.clipboard.readText.
In this particular case, for which I pushed a repro test case, navigator.clipboard.readText is defined inside the iframe but invoking it always throws an error: DOMException: The Clipboard API has been blocked because of a Feature Policy applied to the current document. See https://goo.gl/EuHzyv for more details
The best we could hope for in this case is to hide Paste from the context menu, but unfortunately I could not find anything on the navigator or on navigator.clipboard which would indicate that navigator.clipboard.readText will permanently throw. I have also followed the link https://goo.gl/EuHzyvhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgoo.gl%2FEuHzyv&data=04|01|Raz.Ronen%40microsoft.com|9de0dcd7c1884c563f2b08d8ac50be45|72f988bf86f141af91ab2d7cd011db47|1|0|637448810096677547|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=1gI5buBzCFTe6gNzOLEBOkd4r5oCbueuGxN69DOCgEU%3D&reserved=0 to https://sites.google.com/a/chromium.org/dev/Home/chromium-security/deprecating-permissions-in-cross-origin-iframeshttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fsites.google.com%2Fa%2Fchromium.org%2Fdev%2FHome%2Fchromium-security%2Fdeprecating-permissions-in-cross-origin-iframes&data=04|01|Raz.Ronen%40microsoft.com|9de0dcd7c1884c563f2b08d8ac50be45|72f988bf86f141af91ab2d7cd011db47|1|0|637448810096687638|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=F9k2avh2nP7iE7O%2FMOTAkQI%2FEobNXmMU7pF4ffihRFk%3D&reserved=0 and got to read a bit https://w3c.github.io/webappsec-permissions-policy/https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fw3c.github.io%2Fwebappsec-permissions-policy%2F&data=04|01|Raz.Ronen%40microsoft.com|9de0dcd7c1884c563f2b08d8ac50be45|72f988bf86f141af91ab2d7cd011db47|1|0|637448810096687638|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=10AlYQcJgjtqKdAAq39n8xOtUyFncVunraajUztN2zg%3D&reserved=0 , but the advertised way of finding out the current policies (document.permissionsPolicy) does not appear to be implemented yet.

Do you know of a way to detect that the current window is a restricted cross-origin iframe? If we could detect that, we could at least hide Paste from the context menu.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fmonaco-editor%2Fissues%2F2264%23issuecomment-752274000&data=04|01|Raz.Ronen%40microsoft.com|9de0dcd7c1884c563f2b08d8ac50be45|72f988bf86f141af91ab2d7cd011db47|1|0|637448810096697439|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=YxjtzPzdfOe7kEaytsoGa61PYJTOBoWf16DJM4gxzRQ%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAMDNM67FFE2Y76C67SYBFZLSXJQG5ANCNFSM4U2TOEUQ&data=04|01|Raz.Ronen%40microsoft.com|9de0dcd7c1884c563f2b08d8ac50be45|72f988bf86f141af91ab2d7cd011db47|1|0|637448810096697439|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=bWKeHag%2Bh31qf0vd44YuMf0Te6yhDYHbnfV7pSirIFI%3D&reserved=0.

Thanks.
I still claim we can and should support this feature, as I’ve written in the OS question I’ve posted:
ibiza - Can clipboard access be enabled for iFrame blades? - Stack Overflow at Microsofthttps://stackoverflow.microsoft.com/questions/236911
<EMAIL_ADDRESS>My only doubt is some objection from Portal team, but since browser has it’s own protection I want to believe they’ll approve this change. I’ll be able to continue contacting them next week post their vacations.
Thanks,
Ido.
From: Raz Ronen<EMAIL_ADDRESS>Sent: Wednesday, December 30, 2020 8:45 AM
To: microsoft/monaco-editor<EMAIL_ADDRESS>microsoft/monaco-editor<EMAIL_ADDRESS>Cc: Author<EMAIL_ADDRESS>Dan Hadari<EMAIL_ADDRESS>Roy Gal<EMAIL_ADDRESS>Ido Schreier<EMAIL_ADDRESS>Subject: Re: [microsoft/monaco-editor] Paste menu item doesn't work in sandboxed item (#2264)
Thanks for driving this.
+Adding folks in our end as we have a customer meeting today on the status of this issue.

מאת: Alexandru Dima <notifications<EMAIL_ADDRESS>נשלח: יום רביעי, דצמבר 30, 2020 1:23 לפ׳
אל: microsoft/monaco-editor
עותק: Raz Ronen; Author
נושא: Re: [microsoft/monaco-editor] Paste menu item doesn't work in sandboxed item (#2264)

When pressing Ctrl+V, the editor doesn't treat in any way the keydown/keypress events, so it doesn't call preventDefault() on them, which:
This causes a paste event to be created by the browser and we use that event to get the paste data. That is why using Ctrl+V can paste without any browser prompts.
The editor context menu is custom-drawn via dom nodes and all the actions in there, including Paste are basically simple dom nodes. Clicking on Paste does not generate a browser paste event, so that is why we must reach the clipboard somehow differently. So we first call document.executeCommand('paste') and if that fails we fall back to navigator.clipboard.readText.
In this particular case, for which I pushed a repro test case, navigator.clipboard.readText is defined inside the iframe but invoking it always throws an error: DOMException: The Clipboard API has been blocked because of a Feature Policy applied to the current document. See https://goo.gl/EuHzyv for more details
The best we could hope for in this case is to hide Paste from the context menu, but unfortunately I could not find anything on the navigator or on navigator.clipboard which would indicate that navigator.clipboard.readText will permanently throw. I have also followed the link https://goo.gl/EuHzyvhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgoo.gl%2FEuHzyv&data=04|01|ischrei%40microsoft.com|12f59865c3ab4c82347d08d8ac8e7b36|72f988bf86f141af91ab2d7cd011db47|1|0|637449075239406032|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=k5KVQUqnh8H5HNt24oC%2Bl5HZO3rPvDYXl9lj5OfBO%2F4%3D&reserved=0 to https://sites.google.com/a/chromium.org/dev/Home/chromium-security/deprecating-permissions-in-cross-origin-iframeshttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fsites.google.com%2Fa%2Fchromium.org%2Fdev%2FHome%2Fchromium-security%2Fdeprecating-permissions-in-cross-origin-iframes&data=04|01|ischrei%40microsoft.com|12f59865c3ab4c82347d08d8ac8e7b36|72f988bf86f141af91ab2d7cd011db47|1|0|637449075239406032|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=mM9Y0nnsl748rJPGEV8awm2WwpLQzKbLJw7z3zqplzw%3D&reserved=0 and got to read a bit https://w3c.github.io/webappsec-permissions-policy/https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fw3c.github.io%2Fwebappsec-permissions-policy%2F&data=04|01|ischrei%40microsoft.com|12f59865c3ab4c82347d08d8ac8e7b36|72f988bf86f141af91ab2d7cd011db47|1|0|637449075239415988|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=fpO7KrJNE9CkJgULvP%2Fm6ENTAbP2WT%2FMMWMZlGnjKEs%3D&reserved=0 , but the advertised way of finding out the current policies (document.permissionsPolicy) does not appear to be implemented yet.

Do you know of a way to detect that the current window is a restricted cross-origin iframe? If we could detect that, we could at least hide Paste from the context menu.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fmonaco-editor%2Fissues%2F2264%23issuecomment-752274000&data=04|01|ischrei%40microsoft.com|12f59865c3ab4c82347d08d8ac8e7b36|72f988bf86f141af91ab2d7cd011db47|1|0|637449075239415988|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=EX1mbMd4%2BcfFmz2xNW%2By4Kbk%2B0THQgn5%2B%2B1xvdVhY6Q%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAMDNM67FFE2Y76C67SYBFZLSXJQG5ANCNFSM4U2TOEUQ&data=04|01|ischrei%40microsoft.com|12f59865c3ab4c82347d08d8ac8e7b36|72f988bf86f141af91ab2d7cd011db47|1|0|637449075239425944|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=2CoojaeXtz%2B2QMfniGXIJsiBL%2Bzb%2FHRkKAgOkrCrc%2B8%3D&reserved=0.

BTW.. also noticed that the paste option isn’t available in Firefox:
<EMAIL_ADDRESS>Can you please tell us why and how can we enable it?
From: Ido Schreier
Sent: Wednesday, December 30, 2020 11:31 AM
To: Raz Ronen<EMAIL_ADDRESS>microsoft/monaco-editor<EMAIL_ADDRESS>microsoft/monaco-editor<EMAIL_ADDRESS>Cc: Author<EMAIL_ADDRESS>Dan Hadari<EMAIL_ADDRESS>Roy Gal<EMAIL_ADDRESS>Subject: RE: [microsoft/monaco-editor] Paste menu item doesn't work in sandboxed item (#2264)
Thanks.
I still claim we can and should support this feature, as I’ve written in the OS question I’ve posted:
ibiza - Can clipboard access be enabled for iFrame blades? - Stack Overflow at Microsofthttps://stackoverflow.microsoft.com/questions/236911
<EMAIL_ADDRESS>My only doubt is some objection from Portal team, but since browser has it’s own protection I want to believe they’ll approve this change. I’ll be able to continue contacting them next week post their vacations.
Thanks,
Ido.
From: Raz Ronen <Raz.Ronen<EMAIL_ADDRESS>Sent: Wednesday, December 30, 2020 8:45 AM
To: microsoft/monaco-editor <reply<EMAIL_ADDRESS>microsoft/monaco-editor <monaco-editor<EMAIL_ADDRESS>Cc: Author <author<EMAIL_ADDRESS>Dan Hadari <danha<EMAIL_ADDRESS>Roy Gal <roygal<EMAIL_ADDRESS>Ido Schreier <ischrei<EMAIL_ADDRESS>Subject: Re: [microsoft/monaco-editor] Paste menu item doesn't work in sandboxed item (#2264)
Thanks for driving this.
+Adding folks in our end as we have a customer meeting today on the status of this issue.

מאת: Alexandru Dima <notifications<EMAIL_ADDRESS>נשלח: יום רביעי, דצמבר 30, 2020 1:23 לפ׳
אל: microsoft/monaco-editor
עותק: Raz Ronen; Author
נושא: Re: [microsoft/monaco-editor] Paste menu item doesn't work in sandboxed item (#2264)

When pressing Ctrl+V, the editor doesn't treat in any way the keydown/keypress events, so it doesn't call preventDefault() on them, which:
This causes a paste event to be created by the browser and we use that event to get the paste data. That is why using Ctrl+V can paste without any browser prompts.
The editor context menu is custom-drawn via dom nodes and all the actions in there, including Paste are basically simple dom nodes. Clicking on Paste does not generate a browser paste event, so that is why we must reach the clipboard somehow differently. So we first call document.executeCommand('paste') and if that fails we fall back to navigator.clipboard.readText.
In this particular case, for which I pushed a repro test case, navigator.clipboard.readText is defined inside the iframe but invoking it always throws an error: DOMException: The Clipboard API has been blocked because of a Feature Policy applied to the current document. See https://goo.gl/EuHzyv for more details
The best we could hope for in this case is to hide Paste from the context menu, but unfortunately I could not find anything on the navigator or on navigator.clipboard which would indicate that navigator.clipboard.readText will permanently throw. I have also followed the link https://goo.gl/EuHzyvhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgoo.gl%2FEuHzyv&data=04|01|ischrei%40microsoft.com|12f59865c3ab4c82347d08d8ac8e7b36|72f988bf86f141af91ab2d7cd011db47|1|0|637449075239406032|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=k5KVQUqnh8H5HNt24oC%2Bl5HZO3rPvDYXl9lj5OfBO%2F4%3D&reserved=0 to https://sites.google.com/a/chromium.org/dev/Home/chromium-security/deprecating-permissions-in-cross-origin-iframeshttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fsites.google.com%2Fa%2Fchromium.org%2Fdev%2FHome%2Fchromium-security%2Fdeprecating-permissions-in-cross-origin-iframes&data=04|01|ischrei%40microsoft.com|12f59865c3ab4c82347d08d8ac8e7b36|72f988bf86f141af91ab2d7cd011db47|1|0|637449075239406032|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=mM9Y0nnsl748rJPGEV8awm2WwpLQzKbLJw7z3zqplzw%3D&reserved=0 and got to read a bit https://w3c.github.io/webappsec-permissions-policy/https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fw3c.github.io%2Fwebappsec-permissions-policy%2F&data=04|01|ischrei%40microsoft.com|12f59865c3ab4c82347d08d8ac8e7b36|72f988bf86f141af91ab2d7cd011db47|1|0|637449075239415988|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=fpO7KrJNE9CkJgULvP%2Fm6ENTAbP2WT%2FMMWMZlGnjKEs%3D&reserved=0 , but the advertised way of finding out the current policies (document.permissionsPolicy) does not appear to be implemented yet.

Do you know of a way to detect that the current window is a restricted cross-origin iframe? If we could detect that, we could at least hide Paste from the context menu.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fmonaco-editor%2Fissues%2F2264%23issuecomment-752274000&data=04|01|ischrei%40microsoft.com|12f59865c3ab4c82347d08d8ac8e7b36|72f988bf86f141af91ab2d7cd011db47|1|0|637449075239415988|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=EX1mbMd4%2BcfFmz2xNW%2By4Kbk%2B0THQgn5%2B%2B1xvdVhY6Q%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAMDNM67FFE2Y76C67SYBFZLSXJQG5ANCNFSM4U2TOEUQ&data=04|01|ischrei%40microsoft.com|12f59865c3ab4c82347d08d8ac8e7b36|72f988bf86f141af91ab2d7cd011db47|1|0|637449075239425944|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=2CoojaeXtz%2B2QMfniGXIJsiBL%2Bzb%2FHRkKAgOkrCrc%2B8%3D&reserved=0.

When pressing Ctrl+V, the editor doesn't treat in any way the keydown/keypress events, so it doesn't call preventDefault() on them, which:
This causes a paste event to be created by the browser and we use that event to get the paste data. That is why using Ctrl+V can paste without any browser prompts.
The editor context menu is custom-drawn via dom nodes and all the actions in there, including Paste are basically simple dom nodes. Clicking on Paste does not generate a browser paste event, so that is why we must reach the clipboard somehow differently. So we first call document.executeCommand('paste') and if that fails we fall back to navigator.clipboard.readText.
In this particular case, for which I pushed a repro test case, navigator.clipboard.readText is defined inside the iframe but invoking it always throws an error: DOMException: The Clipboard API has been blocked because of a Feature Policy applied to the current document. See https://goo.gl/EuHzyv for more details
The best we could hope for in this case is to hide Paste from the context menu, but unfortunately I could not find anything on the navigator or on navigator.clipboard which would indicate that navigator.clipboard.readText will permanently throw. I have also followed the link https://goo.gl/EuHzyv to https://sites.google.com/a/chromium.org/dev/Home/chromium-security/deprecating-permissions-in-cross-origin-iframes and got to read a bit https://w3c.github.io/webappsec-permissions-policy/ , but the advertised way of finding out the current policies (document.permissionsPolicy) does not appear to be implemented yet.
Do you know of a way to detect that the current window is a restricted cross-origin iframe? If we could detect that, we could at least hide Paste from the context menu.

How about "window.isSecureContext"?

When pressing Ctrl+V, the editor doesn't treat in any way the keydown/keypress events, so it doesn't call preventDefault() on them, which:
This causes a paste event to be created by the browser and we use that event to get the paste data. That is why using Ctrl+V can paste without any browser prompts.
The editor context menu is custom-drawn via dom nodes and all the actions in there, including Paste are basically simple dom nodes. Clicking on Paste does not generate a browser paste event, so that is why we must reach the clipboard somehow differently. So we first call document.executeCommand('paste') and if that fails we fall back to navigator.clipboard.readText.
In this particular case, for which I pushed a repro test case, navigator.clipboard.readText is defined inside the iframe but invoking it always throws an error: DOMException: The Clipboard API has been blocked because of a Feature Policy applied to the current document. See https://goo.gl/EuHzyv for more details
The best we could hope for in this case is to hide Paste from the context menu, but unfortunately I could not find anything on the navigator or on navigator.clipboard which would indicate that navigator.clipboard.readText will permanently throw. I have also followed the link https://goo.gl/EuHzyv to https://sites.google.com/a/chromium.org/dev/Home/chromium-security/deprecating-permissions-in-cross-origin-iframes and got to read a bit https://w3c.github.io/webappsec-permissions-policy/ , but the advertised way of finding out the current policies (document.permissionsPolicy) does not appear to be implemented yet.
Do you know of a way to detect that the current window is a restricted cross-origin iframe? If we could detect that, we could at least hide Paste from the context menu.

How about "window.isSecureContext"?

Want to know how to hide the "paste" menu. I didn't find the way.

Want to know how to hide the "paste" menu. I didn't find the way.

I have tried window.isSecureContext and it is set to true in this case, so the inner iframe is considered a secure context.

I have tried window.isSecureContext and it is set to true in this case, so the inner iframe is considered a secure context.

Is there any update on this? I am facing the same issue

Changing navigator.clipboard.readText() to window.top.navigator.clipboard.readText() fixes this bug for me. @alexdima, how about it?


Tour does not work if name contains spaces

If the tour's name has space in it. Tour will not work. My assumption is the local storage retrieve the value using the dot notation (  localStorage.key ).  If key has space in it, game over. It should be  localStorage[key].

Finding this issue helped me solve my own problem so thank you, however, the docs do explicitly say spaces are not allowed:

The name should contain only alphanumerics, underscores and hyphens.

My assumption is that is using  {}.key  instead of  {}[key].  because [key] will work with key that has space, but  .key will not. Thank you for letting me know that API says it.  I'll close this.

you can't use dot notation with variable names ;) it has to be using square brackets or it wouldn't be working at all


fix(datepicker): Fix init-date

When passing 'init-date' through datepickerOptions, need to create Date object again
Also, this allow passing 'init-date' as string

This PR has a lot of bit rot - can you update the PR and squash your commits?

Actually, closing this as I think 98e2bdfc8b4dd108acdf3fac75a795675889bbc7 fixes all of these issues.


No development teams detected

I've peeked through the code but the check for development teams goes through the certificates on my machine?
In my case it can't find anything:
Detected development teams:
  -- none --
  Enter an index for a team above, or enter a team ID manually.

To manually enter this it's also not entirely clear where I could find this ID. developer.apple.com does not really list something like this.

Yeah you need a developer ID certificate installed in your keychain. Usually the name will be displayed like Developer ID Application: APP_NAME (TEAM_ID_HERE). If you do have a certificate installed, you'll be able to find it in the Keychain Access app.

OK. That fixed that.
(Not sure anybody really does that anymore since Xcode manages everything automatically these days.)

Was your certificate installed already? If so that's an issue on this crate.

No I got it manually from the Apple Developer portal and installed it into the login keychain.

Well if there's a way to automate this I would love to investigate.


fix #66: generateASCII with cellsize 1 and margin 0

Fix #66: when margins are 0 and cell size is 1, make sure all needed lines are rendered and do not add any filler line.

Thanks a lot!

Thanks :).


ProjectViewer: project hasOwnProperty('update') check may cause problems

What if we subclass an intermediate project type and don't define update in our subclass, but it should still be called in the intermediate?

Replaced with staticMode


Support IPv6

Add support IPv6 support

@mvandeberg
Hey, I'd like to work on this issue. Any insights on how to get started?
Thanks

This issue is stale because it has been open for 30 days with no activity.

@sgerbino
Makes total sense. Will do the same and see if it works then proceed from there.

@sgerbino
Makes total sense. Will do the same and see if it works then proceed from there.

@sgerbino
Makes total sense. Will do the same and see if it works then proceed from there.


Decouples maliput types compare methods from test.

🎉 New feature
Goes on top of #592
Related to https://github.com/maliput/maliput/pull/593
Summary

Moves up the comparison logic for maliput::api types to maliput api library
Point all the tests to use this instead of the old maliput_types_compare.

Checklist

[x] Signed all commits for DCO
[x] Added tests
[ ] Added example and/or tutorial
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] Consider updating Python bindings (if it affects the public API)

Going in


Where to find galactics.parallel fork of John Dubinsky galactics code?

The readme in the runtime folder mentions a file "galactics.parallel.tar.gz" do you know where one could download it?

Was there a status update for this?

Hi,
I saw you last achievements, and I think you need to keep up the good work! Here is some helpful information for you http://powethosi.henriettethehorseshoecrab.com/xyjivudu
<EMAIL_ADDRESS>


feat(binder): convert to constant lookup when case when expression is in simple form

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
As titled, related: #14317, specifically the following pattern.

In addition, if users write a case-when expression that can be converted to the simple form, we can also normalize it in optimizer easily:
SELECT a,
       CASE WHEN a=1 THEN 'one'
            WHEN a=2 THEN 'two'
            ELSE 'other'
       END
    FROM test;

Also, base branch is set to #15077 to avoid any potential merge conflict, should merge after merging base branch.
Planner tests & e2e_test will be added later.

Checklist

[x] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[x] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

Can you elaborate on the relationship between this PR and #14965?

Can you elaborate on the relationship between this PR and #14965?
#14965 aims to optimize the pattern like below, e.g.,

select case 1 when 1 then 114514 (...) else 1919810 end;

to a simple constant (i.e., 114514 in the above example) during frontend binding phase, while this PR is particularly for detecting some pattern that can be transformed to optimizable case-when expression (e.g., constant lookup), an example would be the pattern in the above description.

BTW, would you like to have a try of https://graphite.dev for the stacking workflow? 😄

I heard of graphite before (but never actually try it) and I just realize how powerful it is, will start use it today. 😄
Thanks for the recommendation!

Can you add some planner test? to cover most branches of your function.


Multiple definition error

I get multiple definition of 'flatbush::detail::calculateNumNodesPerLevel(unsigned int, unsigned int)'
When I try to compile with flatbush, with includes in more than one file, is there a special way I'm meant to include it?

Thanks, I'm using this and https://github.com/jlblancoc/nanoflann to accelerate geometric queries for a SQL-like thing I'm writing to search for things in HD digital maps at Zoox, thought you might like to know.


Chore/docs input gateway

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

Docs update

What is the current behavior? (You can also link to an open issue here)

What is the new behavior (if this is a feature change)?

Other information:

Status

[ ] Code documentation for the relevant parts in the code have been added/updated by the PR author
[ ] The functionality has been tested by the PR author
[ ] The functionality has been tested by NRK

Codecov Report
Base: 58.37% // Head: 75.69% // Increases project coverage by +17.32% :tada:

Coverage data is based on head (957af2b) compared to base (8420458).
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@              Coverage Diff               @@
##           release49     #846       +/-   ##
==============================================
+ Coverage      58.37%   75.69%   +17.32%
==============================================
  Files            460       26      -434
  Lines          72005     5270    -66735
  Branches        3620      519     -3101
==============================================
- Hits           42031     3989    -38042
+ Misses         29923     1279    -28644
+ Partials          51        2       -49

Impacted Files
Coverage Δ

meteor/lib/collections/Timeline.ts

meteor/server/security/studio.ts

meteor/server/migration/1_38_0.ts

meteor/lib/collections/ExternalMessageQueue.ts

meteor/server/migration/upgrades/showStyleBase.ts

meteor/lib/collections/WorkerThreads.ts

meteor/server/api/client.ts

meteor/server/security/noSecurity.ts

meteor/server/worker/workerStatus.ts

meteor/lib/collections/Parts.ts

... and 424 more

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.