redirects config

added separate redirect (gatsby) config for easier maintenance
add redirect for: https://github.com/substrate-developer-hub/substrate-docs/issues/680

Amazing, thanks @rmnprkrl 🙏🏻


:wrench: Additional settings: OFAC check

This check blocks the ability to buy nft , for wallets that are on the sanction list

if OFAC check successful

if OFAC check declined

OFAC setting setup

OFAC setting edit

@0xPoint thanks for making those changes! Again, really appreciate you doing the heavy-lifting on this PR.
I'm going to merge into upstream feat/ofac, check some things and then merge to main. I'll let you know when it's live 🚀


Add supervisord process to forward NGINX logs to stdout

We're not currently seeing all the log files in Splunk that we do on Kibana. We need have NGINX access logs in stdout of our PaaS app, so it will appear in cf logs and be forwarded to Splunk.
This commit copies the method used in alphagov/digitalmarketplace-docker-base#74, adding a process that continually echoes the contents of the log file to stdout. We need to keep the logfile so that awslogs can keep forwarding to CloudWatch.

@lfdebrux looks OK to me if you can figure out why the build is failing?

@lfdebrux looks OK to me if you can figure out why the build is failing?

Yeah, trying to figure it out, it's a bit confusing...

I've pushed an additional commit to fix what I think was a bug in the tests, although to be honest I'm not sure that it was the cause of the issue we were seeing. This app could definitely use some work, it's a bit over-engineered.

I assume we've not included the environment scrubbing we did in https://github.com/alphagov/digitalmarketplace-docker-base/pull/74/files because the nginx logs do not contain any of those variables?

I didn't add it in this PR because it wasn't already present, it might be worth doing it though.


add timeout to faucet

High Level Overview of Change

My faucet is slow and this PR adds a timeout to the faucet post request.
Context of Change

I used a default timeout of 5 however there are some sub-settings we could add if we wanted to get more fine grained with the timeout.
connect: float | UnsetType | None = UNSET,
read: float | UnsetType | None = UNSET,
write: float | UnsetType | None = UNSET,
Type of Change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Refactor (non-breaking change that only restructures code)
[ ] Tests (You added tests for code that already exists, or your new feature included in this PR)
[ ] Documentation Updates
[ ] Release

Test Plan

I am actually working on adding a memos params (with default values) to all faucet transactions in client libraries, this PR is highly related, would you mind if I base my PR on yours? thanks

I am actually working on adding a memos params (with default values) to all faucet transactions in client libraries, this PR is highly related, would you mind if I base my PR on yours? thanks

Not at all :)

This didn't end up being added as part of the memos change - @dangell7 I think this would make sense as an optional parameter, but not as a default timeout value (because I'm not sure other users want a timeout, and picking a number that works for everyone with sidechain setups seems hard to do).
If you don't want to add that, we can close this potentially.

@dangell7 FYI the Ripple faucet repo is public if you'd rather use that (it might be faster)
https://github.com/xpring-eng/testnet-faucet


Feature/api function separation

Internal metador-core routes don't have separate functions for the routes. I separated these functions and implemented them in routes again. Necessary for invenio-metador package.

Codecov Report

Merging #81 (4e5587e) into dev (0b8e6e8) will decrease coverage by 0.03%.
The diff coverage is 11.53%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##              dev      #81      +/-   ##
==========================================
- Coverage   80.00%   79.98%   -0.03%
==========================================
  Files          56       56
  Lines        4997     5001       +4
==========================================
+ Hits         3998     4000       +2
- Misses        999     1001       +2

Impacted Files
Coverage Δ

metador_core/widget/server/__init__.py
23.96% <11.53%> (+0.89%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


openapi_3 to 2: Copy x- props from schema to param

The parameter in OpenAPI/Swagger 2 has the properties of schema, excluding properties which define objects, and acts like schema.  When translating, copy schema extension properties which are applicable to parameters.  Since I'm not aware of any which are incompatible, copy them all for now.
Add x-ms-enum to test.  Note that Autorest uses the same generated enum type for multiple parameters (and schemas) with the same x-ms-enum.name and values, so duplication is not an issue.
Note: This PR should apply after #231.  Only the last commit is specific to this PR.
Thanks for considering,
Kevin

I rebased the commit on the current master branch so it is now independent of #231.

Thanks!


Bump aptible-auth version to 1.2.4

@almathew can we get an override push to master here? The test failure is preexisting https://app.travis-ci.com/github/aptible/aptible-cli/builds/241267415

@UserNotFound is this the "we also should pin git" issue that I think I heard you talked about, but didn't catch the actual conversation, or something else?

@almathew https://github.com/aptible/aptible-cli/pull/301/files#diff-41f6c6264ecf15b7062d10db7d0457e8797fd597f0aad9be62720a7724998910R28

@aguilinger merged


Update relayer.md

fixed senario to scenario

This repo is being archived in favor of docs.wormhole.com


[Emacs Lisp Question] When to use back quote in defmacro?

Hi @hlissner,
Follow up my last question abt elisp. This week I'm trying to get my head round with defmacro. I'm wondering why you don't use back quote starting from the body of defmacro, but just small part of its body instead?
For example with map!
From line 138 to 238, you just use normal elisp inside the macro and it still works properly. Why did it happen? Did I overlook sth from Intro Elisp Manual?

A lisp macro is essentially a function that returns lisp forms to be evaluated. And lisp forms are simply regular lists. For example...
(defmacro say-hi-1 (name)
  `(message "Hello %s" ,name))

(defmacro say-hi-2 (name)
  (list 'message "Hello %s" name))

(defmacro say-hi-3 (name)
  (let (forms)
    (push name forms)
    (push "Hello %s" forms)
    (push 'message forms)
    forms))

These all expand into (message "Hello %s" "world"), just in different ways. It's metaprogramming. Macros let you write dynamic lisp. You can test macros with macroexpand:
(macroexpand '(say-hi-1 "world")) ;; #=> '(message "Hello %s" "world")

All map! does differently is it constructs these forms using more complex logic. I urge you to try macroexpand on a map! form to see what it produces.
But perhaps more importantly, you should understand something about macros: they expand at compile-time and the expanded code is evaluated at run-time. Take this for example:
(defmacro say-hi (name)
  (message "foo")
  `(message "Hello %s" ,name))

(message "bar")
(say-hi "world")

This produces the following output:
foo
bar
Hello world

Notice that the foo and bar are reversed. To put it simply: the elisp interpreter expands macros (among other things) before actually running your code. So "foo" is printed during compile time, then "bar" and "Hello world" are printed at run time (in the expected order).
If you byte-compile your code, "foo" will get printed out while compiling. Then, when you run the compiled program, the output will be:
bar
Hello world

That's because macros are permanently expanded when elisp is byte-compiled (i.e. the macro isn't needed in the compiled code). I exploit this fact a lot for performance; any time-consuming logic used in your macros are no longer needed.
I hope that helps!

Actually I can't use my computer today, I will test it asap.
One question keeps lingering in my head atm is defmacro return a list to be evaluated. In its body, we must use quote to those we don't want evaluate, for example special forms like if, while, setq.... but in your codes, those quotes are missing.

Quotes are not required in a macro. Quotes are just one of several ways to create a list in lisp. These all produce the same list:

'(1 2 3)
`(1 2 3)
(list 1 2 3)

A macro needs to return a form to be evaluated. This form can be a list of forms, and there are many ways of building this list.

You can do so with quotes: '(if var 5 10)

Or without quotes (list 'if 'var 5 10)

Or with logic (note: the let-form isn't quoted):
(let (forms)
  (push 10 forms)
  (push 5 forms)
  (setq forms (append '(if var) forms))
  forms)

The result of all these methods is a list of forms: (if var 5 10), which will replace the macro call.

Thank you so much @hlissner, it's much clearer to me now. Just one question left. Why in the last example, you dont quote let? I thought lisp would evaluate let as value without quote and producing exception error.

I think I figured it out. If I understand correctly, the returned list form is the most important part of defmacro. My first thought was elisp will evaluate everything inside defmacrobody, from top to bottom. But I'm wrong, aren't I? It just evaluate the returned list forms (or the last list form in the body).
If this is correct pls close this issue. Otherwise I beg you pardon for further explanation 😅

Ah, my bad. I missed the notifications for your messages!
The way you used the word 'evaluate' confused me a little. The Elisp interpreter does evaluate the macro body in order to produce elisp forms, then it evaluates those lisp forms. It is essentially a two-step evaluation, with the possibility of removing the first step if you byte-compile. Does that make sense?

Thank you @hlissner for further explanation.  Now elisp macro is not a thing too dreadful for me as it was 👍 thanks to you!


Proposal: Session Recording Feature Discussion

I've implemented session recording functionality in the development branch (link). This feature allows users to record terminal sessions in Asciicast format. However, there are some aspects that need discussion:

Storage Location: Where should recorded sessions be stored?
Download Capability: Should users be able to download recorded sessions from the server?
Live Viewing: Asciinema supports live viewing of recorded sessions. Should we consider integrating this functionality?

Your feedback on these points would be greatly appreciated.

Hi there!
Thanks so much for your work. I just have a few comments/questions to make before addressing your questions.
This will only work if the user uses the server console rather than the preferred method of jump host (-J) and it wouldnt be possible to record the jump host sessions as it uses the typical ssh method of direct-tcp.
Do you think this is still useful?
Additionally, whats to stop you using ascii-cinema to just record the terminal as you're using rssh? Why does this have to be part of the functionality of rssh?

Storage Location: Where should recorded sessions be stored?
The only place they could be stored is on the server, I'd recommend just putting them within a seperate folder within the data directory, call it recordings just make sure people can path traversal it

I think it would be best to allow this. But how we should do it is by having the record function have a --read flag that dumps the file to stdout if the current connection is not a server console.

This means a user can use the ssh exec handler to do something like this:
ssh your.rssh.server record --read your.recording.date > file

This is definitely outside outside the scope of reverse ssh, it's a bit of a debate whether recording should be part of rssh at all. I just want this project to effectively be something people build off, not a collection of every functionality.

Hello,
Yes, this will only work if the user uses the server console using the --record <filename> flag.
I acknowledge the redundancy of this feature, as asciinema can always be initiated on the host, and then the file can be downloaded...
The only advantage would be on windows, where it's a bit more complicated, and also that the recording occurs on the server and not on the client. However, adding it would slightly divert the project's focus.

Hi! Unfortunately after giving it a bit more of a think, I think this is outside the scope of the project.
Thank you so much for your work though, and please contribute to RSSH in the future!


Compiled css syntax

Fix invalid CSS syntax.

Summary by CodeRabbit

Style

Improved CSS formatting by removing quotes around numeric values and color codes.
Added semicolons to the end of CSS property declarations for consistency.
Updated font-size and border-radius properties for better syntax correctness.

/fast-forward


Remove expected failure from doctests

In order to confirm that examples in the readme are working, let's remove the expected failure tag and ensure that all remaining examples are succeeding

[x] Update number of synaptic connections from 74->90
[x] Unicode for the names of neurons
[x] Backtick syntax error
[x] 302 vs. 333 neurons?

Fixed by #138 !!


Raw block in figure is cut if it's too long

Description
If I nest a raw block in a figure, and the code is longer than the page, the code is clipped. Does someone know how I can wrap the whole code into a figure function with it displaying correctly? The reason I'm doing it is so that it is listed in the figure table.
The typst file looks something like this:
  #set raw(theme: "dark_modern.tmTheme")
  #show raw: it => block(
    fill: rgb("#1d2433"),
    inset: 8pt,
    radius: 5pt,
    text(fill: rgb("#a2aabc"), it)
  )

#figure(
  [
    ```rust
      ///  ... long code ...
    ```
  ],
  kind: image,
  caption: "File typst/crates/typst-ide/src/complete.rs"
)

The theme can be downloaded from this repository
Here is an example of what it looks like:

Please note that the code is taken from the Typst repository and I only use it for this issue.
Reproduction URL
No response
Operating system
Windows 10
Typst version

[x] I am using the latest version of Typst

Try this:
#show figure: set block(breakable: true)


Add 'eye depth' and 'eye folds' DNA to default races (for Asian characters)

For now, it's not easy (or even possible) to generate an authentic looking Asian characters with UMA, because it lacks such DNAs like 'eye depth' and 'eye folds'.
Considering the importance of providing ethnic diversity in such a project, it would be a significant improvement for a relatively small efforts if it could add such morphs to existing default races.

Available in Willb races


Opt out

Is your feature request related to a problem? Please describe.
I don't want people to know anything about me, unless i opt-in.
Describe the solution you'd like
Default opt-out, opt-in an option

You opt in by installing the plugin.

You opt in by installing the plugin. I'll include this warning in the plugin installation.

Well, this is very limiting, what if i want to use the grouping but not want to share my location, run, ...

You opt in by installing the plugin. I'll include this warning in the plugin installation.

Well, this is very limiting, what if i want to use the grouping but not want to share my location, run energy, hp, etc with the group

I've given this some thought, and tbh if they don't want all of those features -- then they can just not install the plugin. I don't have any plans on adding opt-out features. It doesn't make sense to work around it on my end for a relatively small number of players.
They opt-in currently on two occasions:

Installing the plugin
Joining a private or public group.

I don't see relevancy for including an opt-out feature when that defeats one of the main points of the plugin.

This will be added aith runeguard.


Feature Request: Open new screencast tabs when new browser windows or tabs are opened

Is your feature request related to a problem? Please describe.
The current behavior of screencast in VS Code Edge Dev Tools can be confusing for users. When new browser windows or tabs are opened, they are not visible in the UI and require the user to click the "Toggle Screencast" button to see them. However, doing so will replace the currently shown tab with the new target, which may not be the desired behavior.
Describe the solution you'd like:
I would like a new screencast tab be opened within VS Code whenever a new browser window or tab is opened during a screencast. This would allow users to see all open instances/targets in the screencast without having to toggle the screencast on and off or lose the view of the previously shown tab.
Describe alternatives you've considered:
One alternative could be to show a notification or pop-up within the VS Code UI whenever a new instance/target is opened during a screencast. However, this may not be as intuitive as having a separate screencast tab for each instance/target.
Additional context:
This feature would improve the usability and user experience of the screencast feature in VS Code Edge Dev Tools. It would make it easier for users to see all open instances/targets without interrupting their workflow. Thank you for considering this request.

Thanks for describing the scenario in detail. I'll flag this up as a feature request for the team to evaluate and prioritize.


Publish to npm

It would be cool to slot in a package.json file and push this up to npm.

+1

Why on earth would this belong on NPM?  It's a front end only tool, I can certainly see the value of adding it to a project via bower, but that works today.

Many people, including myself, use NPM for frontend and backend in a single package.
Thinking about it, I would perhaps prefer npm be for backend on principle, only because it's so saturated already. I guess it's down to convenience.

NPM is a fantastic package manager, but I think you've exactly identified the reason to seperate them from each other.  And while I certainly don't want to speak to any particular architecture, as a node developer I find that using serve-static("/bower_components") allows me to package, serve, and manipulate all of my front-end packages without allowing someone to forcefully browse to myapp.com/js/some_vulnerable_component

@crispen-smith Tools like Browserify and Webpack bundle packages from npm for web browsers. There is a strong use-case there.

I've never had a problem loading from github with bower, I can't speak to Browserify or Webpack, though if Webpack requires an NPM entry for loading then yes, I'll +1 this.

@crispen-smith Technically speaking any library that depends on npm that load files from github, but you lose the versioning information. The main benefit of pushing to npm would be being able to declare a dependency on a version range.

That does make sense then,  I still feel it muddies the water between front-end and back-end package managers but I think that's a bigger issue than prefix-free.

Are there any other scripts that require a browser environment published on npm?

https://www.npmjs.com/package/jquery-ui

In case you do use Browserify and Gulp as myself, you can simply use gulp-autoprefixer to do the work for you instead of binding in Lea's prefixfree statically into the page.
@LeaVerou there are popular scripts like jQuery and Backbone which are on NPM.

I am interested in consuming the API for inline-style prefixes but stopped short because no package integration.

You can check out Radium's Prefixer, which was inspired by PrefixFree and css-prefix. Could be moved to its own npm module.

@ianobermiller Already found https://github.com/jsstyles/css-vendor

npm package management for frontend projects are gold tbh, bower has it strengths, but not as powerfull as npm.
I would greatly appreciate this module (and conic-gradient) to be published on npm!
+1

You may also be interested in inline-style-prefixer, which has the advantage of working on the server as well.

Yes, this should definitely be published to npm, it's long overdue. Once my laptop is fixed (it died unexpectedly and I'm currently using my partner's to Github for a bit) it's on my to-do list.

Done!
https://www.npmjs.com/package/prefixfree


OCPBUGS-9924: Remove "unsupported" wording from info on console

Description
Simple wording change in console output (removed the wording "unsupported")
Fixes # OCPBUGS-9924
Type of change

[x ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
No testing needed

[ ] Test A
[ ] Test B

Test Configuration:

Firmware version:
Hardware:
Toolchain:
SDK:

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

/jira refresh

/cherrypick release-4.13

/assign @soltysh
PTAL when you get a chance

Done pre-merge test , no issue found

/label qe-approved


[Gally]: master <- dev

Automatically created by Git-Ally

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Using specific translation

Hi,
Is it possible to tell the translator that if source text contains a given word, then translate it to a specific word instead of what the database throws up. I want this as many times translated text uses words which are not in pure form.

I'm not sure you are able to do this currently. If you haven't already, you might want to also try using the new premium (NMT) model as it may give you more natural translations.

Closing issue as it is relatively old and never received a follow-up. Feel free to re-open.


Upgrade to LLVM 3.9.1

Apparently there is no Ubuntu 3.9.1 prebuilt
http://releases.llvm.org/3.9.1/

@SeanTAllen Couldn't we use the one for Debian? http://releases.llvm.org/3.9.1/clang+llvm-3.9.1-x86_64-linux-gnu-debian8.tar.xz

@Praetonus i think so, i'm going to make a change and find out.

Pushed some changes @Praetonus. Let's see how that works out.

@ponylang/committer hmmm the windows version of this isn't going to pass until we update https://github.com/ponylang/ponyc-windows-llvm/
Anyone with vagrant et al installed want to do that?

@Praetonus i switched it but how we have it set right now, that then breaks the other versions because in life we aren't allowed to have nice things. I'll fix this up another time. First I'm going to make sure that 3.9.1 works even if the others don't.

In case I don't make the sync on the 11th. Can we try and get someone to update our Windows LLVM version to 3.9.1 so we can get this merged?

I'm working on Windows build stuff; I'll make sure 3.9.1 gets included.

@kulibali Awesome. Ping me here when I should update and rerun this.

#1515 builds with 3.9.1 for Windows.  If you rebase this on master (without the appveyor.yml changes) things should work.

without the appveyor changes @kulibali ?
I'm a little confused. What am I bringing over?

The master branch is now building 3.9.1 on Windows, so you don't need to change the appveyor.yml in this PR.

This is ready to merge


WordPress autosaves blocked

I was writing a new post but the ModSecurity detected it as "Remote Command Execution". Here is the ModSecurity audit log
--f9af023d-H--
Message: Access denied with code 406 (phase 2). Pattern match "(?:;|\\{|\\||\\|\\||&|&&|\\n|\\r|\\$\\(|\\$\\(\\(|`|\\${|<\\(|>\\(|\\(\\s*\\))\\s*(?:{|\\s*\\(\\s*|\\w+=(?:[^\\s]*|\\$.*|\\$.*|<.*|>.*|\\'.*\\'|\".*\")\\s+|!\\s*|\\$)*\\s*(?:'|\")*(?:[\\?\\*\\[\\]\\(\\)\\-\\|+\\w'\"\\./\\\\]+/)?[\\\\'\"]*(?:l[\\\\'\"]* ..." at ARGS:content. [file "/etc/modsecurity/rules/REQUEST-932-APPLICATION-ATTACK-RCE.conf"] [line "124"] [id "932100"] [msg "Remote Command Execution: Unix Command Injection"] [data "Matched Data: \x0aMODULE_VERSION=v0.8.0\x0aREPO=\x22https://github.com/Icinga/icingaweb2-module-${MODULE_NAME}\x22\x0aMODULES_PATH=\x22/usr/share/icingaweb2/modules\x22\x0agit clone ${REPO} \x22${MODULES_PATH}/${MODULE_NAME}\x22 --branch \x22${MODULE_VERSION}\x22\x0aicingacli module enable \x22${MODULE_NAME}\x22
\x0a\x0a\x0a\x0a

\x0a\x0a\x0a\x0a
IPL module
\x0a\x0a..."] [severity "CRITI
Apache-Error: [file "apache2_util.c"] [line 273] [level 3] [client <IP_ADDRESS>] ModSecurity: Access denied with code 406 (phase 2). Pattern match "(?:;|\\\\\\\\{|\\\\\\\\||\\\\\\\\|\\\\\\\\||&|&&|\\\\\\\\n|\\\\\\\\r|\\\\\\\\$\\\\\\\\(|\\\\\\\\$\\\\\\\\(\\\\\\\\(|`|\\\\\\\\${|<\\\\\\\\(|>\\\\\\\\(|\\\\\\\\(\\\\\\\\s*\\\\\\\\))\\\\\\\\s*(?:{|\\\\\\\\s*\\\\\\\\(\\\\\\\\s*|\\\\\\\\w+=(?:[^\\\\\\\\s]*|\\\\\\\\$.*|\\\\\\\\$.*|<.*|>.*|\\\\\\\\'.*\\\\\\\\'|\\\\".*\\\\")\\\\\\\\s+|!\\\\\\\\s*|\\\\\\\\$)*\\\\\\\\s*(?:'|\\\\")*(?:[\\\\\\\\?\\\\\\\\*\\\\\\\\[\\\\\\\\]\\\\\\\\(\\\\\\\\)\\\\\\\\-\\\\\\\\|+\\\\\\\\w'\\\\"\\\\\\\\./\\\\\\\\\\\\\\\\]+/)?[\\\\\\\\\\\\\\\\'\\\\"]*(?:l[\\\\\\\\\\\\\\\\'\\\\"]* ..." at ARGS:content. [file "/etc/modsecurity/rules/REQUEST-932-APPLICATION-ATTACK-RCE.conf"] [line "124"] [id "932100"] [msg "Remote Command Execution: Unix Command Injection"] [data "Matched Data: \\\\x0aMODULE_VERSION=v0.8.0\\\\x0aREPO=\\\\x22https://github.com/Icinga/icingaweb2-module-${MODULE_NAME}\\\\x22\\\\x0aMODULES_PATH=\\\\x22/usr/share/icingaweb2/modules\\\\x22\\\\x0agit clone ${REPO} \\\\x22${MODULES_PATH}/${MODULE_NAME}\\\\x22 --branch \\\\x22${MODULE_VERSION}\\\\x22\\\\x0aicingacli module enable \\\\x22${MODULE_NAME}\\\\x22
\\\\x0a\\\\x0a\\\\x0a\\\\x0a

\\\\x0a\\\\x0a\\\\x0a\\\\x0a
IPL module
\\\\x0a\\\\x0a..."] [severity "CRITI [hostname "prakash-khadka.com.np"] [uri "/wp-json/wp/v2/posts/2082/autosaves"] [unique_id "YB0MdXr1DEgaGbqiBXwSgAAAAAY"]
Action: Intercepted (phase 2)
Stopwatch:<PHONE_NUMBER>710151 18934 (- - -)
Stopwatch2:<PHONE_NUMBER>710151 18934; combined=13986, p1=1555, p2=12067, p3=0, p4=0, p5=364, sr=224, sw=0, l=0, gc=0
Response-Body-Transformed: Dechunked
Producer: ModSecurity for Apache/2.9.3 (http://www.modsecurity.org/); OWASP_CRS/3.2.0.
Server: Apache
Engine-Mode: "ENABLED"

--f9af023d-Z--

Sorry for the inconvenience @pkhadka56. Could you please check if you have enable the WP rule exclusion package (-> see crs-setup.conf)?
You have a false positive on the content parameter on the autosave, but that rule exclusion package instructs CRS to ignore the content parameter completely for all save actions. You are also not running the latest version of CRS, but I think we covered that in your version already.

After I uncomment the following section in /etc/modsecurity/crs-setup.conf, it seems to be working.
SecAction \
 "id:900130,\
  phase:1,\
  nolog,\
  pass,\
  t:none,\
  setvar:tx.crs_exclusions_wordpress=1

Thanks for your quick response.

You're welcome. And glad you got it to work for you. Good luck!


feat(callbar-button): DLT-2211 add passthrough tooltip props

feat(callbar-button): add passthrough tooltip props

Obligatory GIF (super important!)

:hammer_and_wrench: Type Of Change

These types will increment the version number on release:

[x] Feature

:book: Jira Ticket
https://dialpad.atlassian.net/browse/DLT-2211
:book: Description
Added a few requested tooltip props to callbar_button and callbar_button_with_popover
:bulb: Context
Needed to handle some edge cases in DPM
:pencil: Checklist

For all PRs:

[x] I have ensured no private Dialpad links or info are in the code or pull request description (Dialtone is a public repo!).
[x] I have reviewed my changes.
[x] I have added all relevant documentation.
[x] I have considered the performance impact of my change.

For all Vue changes:

[x] I have made my changes in Vue 2 and Vue 3. Note: you may sync your changes from Vue 2 to Vue 3 (or vice versa) using the ./scripts/dialtone-vue-sync.sh script. Read docs here: Dialtone Vue Sync Script

Changes made, nice find thanks!

Accessibility tests failing due to some installation issue, nothing to do with the PR

Please add invertedTooltip and showTooltip props to callbar_button.stories.js as on Vue 2 storybook doesn't render those correctly and the props don't work as expected.

Weird.. wonder why that only happens in vue 2.


Braze Action - Add optional Attributes to Action Hub

Added a feature to allow for sending optional custom and standard Braze attributes via tagging with braze[].
Also increased api limit from 10,000 to 100,000.

https://github.com/looker/actions/pull/271/files#diff-c5149b4c257c672d74b56fbc8a3d3813R51
I would like executeInOwnProcess = true here. Reason being is that we are now adding some more processing on an unlimited amount of data, getting some potentially very large streams of data off of the main event loop would probably be a good idea
example usage: https://github.com/looker/actions/blob/master/src/actions/segment/segment.ts#L56

looked at the test failure. all that needs to happen is in the Braze test file, use action.executeInOwnProcess = false like in test_segment.ts. We test the queueing in another file and in the Braze action we just want to test the Braze action itself without the queueing

looked at the test failure. all that needs to happen is in the Braze test file, use action.executeInOwnProcess = false like in test_segment.ts. We test the queueing in another file and in the Braze action we just want to test the Braze action itself without the queueing

Thanks. Made the change.


added windows 11

after reading this article, i confirmed windows 11 is supported

Learn Build status updates of commit a6ed823:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

windows/security/threat-protection/security-policy-settings/user-account-control-behavior-of-the-elevation-prompt-for-standard-users.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Model Inheritance Best Practices

I am converting some of my models to be in line with performance_modeling_test.py. I currently have an AircraftPerformance model and a ClimbPerformance model. The ClimbPerformance model inherits from Aircraft Performance. I want ClimbPerformance to have all the constraints that are in AircraftPerformance in addition to its own constraints.
I am not sure how to handle the initialization of the different classes. Thoughts?

@mayork, I think we talked about this, but I don't remember exactly the design pattern you ended up following...could you describe your solution here briefly and then close the issue?

What I ended up doing was having both AircraftPerformance and ClimbPerformance inherit from Model and then each instance of ClimbPerformance creates its own AircraftPerformance model


Make `lp -i' able to restart a job with a job-hold-until attribute

Version: -feature
CUPS.org User: opher
The patch for STR #3130 adds support in the schedulers Restart-Job operation for job-hold-until' attribute. Please add support for thelp -i' command to restart a job with a `job-hold-until' attribute. ex:
lp -i 123 -H hold -H restart

Thanks,
Opher Shachar.

After review we have decided not to address this issue.


Add Context argument to GraphStore interface [phab:D318]

Created by schroederc at 2015-06-04 20:29:28:
Add Context argument to GraphStore interface

fromberger accepted this revision at 2015-06-04 21:27:26

schroederc closed via commit 5cd490692645314878a7a237baf0552e2ad9ab95 at 2015-06-04 21:28:56


Font is blurry in godot 4 compared to godot 3

Godot version
4 beta 10 and 3.5.1
System information
win 11
Issue description
Godot 3

Godot 4

Both have a font size of 12, which I think is the "proper" size of the font. However, the Godot 4 is randomly blurry in some places. No, turning off anti-aliasing doesn't fix the issue, it just makes the blurry pixels completely white so the font looks thicker in some places.
Steps to reproduce
Use the font "power_red_and_green" from pokemon essentials
power_red_and_green.zip
Minimal reproduction project
none

Check the Texture property under CanvasItem and select texture_filter as Nearest.

Check the Texture property under CanvasItem and select texture_filter as Nearest.

no, didn't change anything.

Have you checked the Import Defaults tab for the Font Data importer in your Project Settings?

Duplicate of https://github.com/godotengine/godot/issues/70327.


Test on Python 3.12

#69 rebooted.

Thanks for fixing my failed attempt to merge things.
I still get the error:
Traceback (most recent call last):
  File "/hpc/mydata/eduardo.hirata/.conda/envs/py312/bin/viscy", line 5, in <module>
    from viscy.cli.cli import main
  File "/hpc/mydata/eduardo.hirata/repos/viscy/viscy/cli/cli.py", line 13, in <module>
    from viscy.light.engine import VSUNet
  File "/hpc/mydata/eduardo.hirata/repos/viscy/viscy/light/engine.py", line 9, in <module>
    from matplotlib.cm import get_cmap
ImportError: cannot import name 'get_cmap' from 'matplotlib.cm' (/hpc/mydata/eduardo.hirata/.conda/envs/py312/lib/python3.12/site-packages/matplotlib/cm.py)

Closing for https://github.com/mehta-lab/VisCy/pull/88


Raspberry Pi Bazel Build "Define appropriate PLATFORM macro" error

When building TensorFlow on Raspberry Pi with Bazel, the pre-compiler end up throwing this error:
Define the appropriate PLATFORM_<foo> macro for this platform

From what I've been able to gather, here's the cause of the issue:

Because Raspberry Pi uses an ARM CPU, the IS_MOBILE_PLATFORM macro is defined in `tensorflow/core/platform/platform.h
Later, when trying to compile png.h, gif.h, or jpeg.h, the if/else statements fall through, leaving with the default option to throw an error.

Environment info
Operating System: Raspbian GNU/Linux, version 8.0
Installed version of CUDA and cuDNN: N/A
Installed from source.

The commit hash : e95f4e760c6b6713b6b686ebeff9a1586a5831dd
The output of bazel version :

Build label: 0.2.1-2016-06-13 (@447f7f3)
Build target: bazel-out/local_linux-fastbuild/bin/src/main/java/com/google/devtools/build/lib/bazel/BazelServer_deploy.jar
Build time: Mon Jun 13 19:05:24 2016<PHONE_NUMBER>)
Build timestamp:<PHONE_NUMBER>
Build timestamp as int:<PHONE_NUMBER>

Steps to reproduce

Build TensorFlow from source on Raspberry Pi as described in this guide without deleting this line from tensorflow/core/platform/platform.h, which defines IS_MOBILE_PLATFORM when compiling on ARM devices.

What have you tried?

Deleted the line mentioned above, which fixes the problem for local builds

Logs or other output that would be helpful
define_macro_error.txt

According to this post on StackExchange, it looks like there's no reliable, automatic way to detect if building for Raspberry Pi.
Would it be possible to add in a #if  !defined(PLATFORM_RASPBERRY_PI) line in platform.h and define the macro manually in Bazel with --copt="-D PLATFORM_RASPBERRY_PI"?

@petewarden any thoughts?

No update on this one I'm afraid. I've been focused on the makefile approach to building for the Pi, and I'm not familiar enough with Bazel to suggest the right way to handle this.

Closing this -- we only support Makefile builds right now.

Ok, no worries- I'll keep maintaining my repo for those who want a Python interface on their Raspberry Pi.

Long build process of the Raspberry Pi.


osCommerce config keys helper Missing testparameter in pw 4.0.8

osCommerce config keys helper is missing the parameter 'MODULE_PAYMENT_DIBS_TESTMODE' which renders the module in a constant testmode
Added screenshots of missing and fixed dibs.php file

done in latest release <IP_ADDRESS>


fix: updated gitignore for few defaults files

Thanks, but .gitignores should only contain ignorable files created by the project itself, not just those that might be present on the host system.


Update community page

Following #465, I've added some styling, and a first version of an event list (to be automated), to have something to look at. The Community page was very bare-bones before.
A screenshot of what I did:

And yes, comparing that to the previous page, it is much nicer.

Cool info and give some energy to the page.
How will we maintain this list?

Cool info and give some energy to the page. How will we maintain this list?

The idea would be to hook an API in here. So maybe pulling from the Google calendar (and follow conventions like putting the speaker and talk title in the descr. of the calendar item), or from a (to be created) Meetup group. Until that's decided I'd be happy to maintain this.
We can link names to Twitter accounts or GH (instead of just using usernames). Would need to figure out how we can get that to work in an automated workflow but should be possible.

Why do you think we need some external automation for the twitter names? Do you think something like this -> @laysauchoa would work for this case? I have not found of twitter account can be modified, so I guess they cannot.

Alternatively, a simple solution can be to keep up-to-date info in a google spreadsheet and inject it here as an iframe. (just got this idea from Felix @runwuf :D)

static google spreadsheet can be easily integrated by embed the code generated out of File > Share > Publish to web -> Embed -> Publish, but it is going to be static, to get a live/editable spreadsheet embed is a bit tricky, ping me if needed. I might document the steps after.

The styling improvements by themselves are worth it :) I'm not sure about maintaining a list of events by hand but if you want to do that then we can try, and just remove the section if it doesn't work out!
The build failed because it doesn't know our proper names (also can you use "Lorna Mitchell" for me please?), you can add them to the vocabulary file (info is in the readme) to permit those words.

rebase with the main page

The way that it is, you won't be redirected to the right links in none of the buttons. I had the same issue and one needs to remove the  :type: ref. Leaving here for your reference.
    .. link-button:: https://twitter.com/aiven_io
        :type: ref
        :text: To the Twitterverse
        :classes: stretched-link


Changing the password for a user

Changing your own password with deis auth:passwd is easy.
Changing the password for a user who has forgot their password is not.
http://stackoverflow.com/questions/1873806/changing-password-in-django
$ docker exec -ti deis-controller ./manage.py shell
WARNING Cannot synchronize with etcd cluster
Python 2.7.6 (default, Mar 22 2014, 22:59:56)
[GCC 4.8.2] on linux2
Type "help", "copyright", "credits" or "license" for more information.
(InteractiveConsole)
>>> from django.contrib.auth.models import User
>>> u = User.objects.get(username__exact='theirusername')
>>> u.set_password('iforgotmypassword')
>>> u.save()

There really should be an easier way.

if we were to implement #3200 we could send out an email with a reset code.

:+1: I think this is a fairly easy and reasonable feature request.


improve docs for right navbar item, including onRight

Tried to render a hamburger menu item in the navbar of my initial route:
<Route name="launch" initial={true} component={ChatRoomScreen} wrapRouter={true} hideNavBar={false} title="Home" rightTitle="menu" onRight={Actions.drawer}/>

The docs for displaying the right nav button are a little unclear. With this setup, nothing is displayed, however changing to this works:
onRight={() => {Actions.drawer() }}

3.0 release should solve this problem. However it has a little bit different API comparing with 2.x release. Anyway I'm closing this ticket because 2.x release will not be supported except critical bugs.


Azure Firewall basic - how will this affect the enterprise scale architecture?

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Versions

module: 4.0.1
Description
With v4 of the module you have implemented support for Azure Firewall basic. That's great!
Other than capacity, what features will we be missing? From what I can see in the article Choosing the right Azure Firewall SKU to meet your needs, DNS proxy + Custom DNS is not supported in the basic sku.
How will this affect us using a standardised setup with Virtual WAN? What actual scenarios will not work if changing to basic sku?
Thanks!
Terraform plan - changing from Standard to Basic
Terraform will perform the following actions:

  # module.alz_connectivity.azurerm_firewall.virtual_wan["/subscriptions/xxx/resourceGroups/xxx/providers/Microsoft.Network/azureFirewalls/xxx"] will be updated in-place
  ~ resource "azurerm_firewall" "virtual_wan" {
        id                  = "/subscriptions/xxx/resourceGroups/xxx/providers/Microsoft.Network/azureFirewalls/xxx"
        name                = "xxx"
      ~ sku_tier            = "Standard" -> "Basic"
        tags                = {
            "deployedBy" = "terraform/azure/caf-enterprise-scale"
        }
        # (7 unchanged attributes hidden)

        # (1 unchanged block hidden)
    }

  # module.alz_connectivity.azurerm_firewall_policy.virtual_wan["/subscriptions/xxx/resourceGroups/xxx/providers/Microsoft.Network/firewallPolicies/xxx"] must be replaced
-/+ resource "azurerm_firewall_policy" "virtual_wan" {
      - auto_learn_private_ranges_enabled = false -> null
      ~ child_policies                    = [] -> (known after apply)
      ~ firewalls                         = [
          - "/subscriptions/xxx/resourceGroups/xxx/providers/Microsoft.Network/azureFirewalls/xxx",
        ] -> (known after apply)
      ~ id                                = "/subscriptions/xxx/resourceGroups/xxx/providers/Microsoft.Network/firewallPolicies/xxx" -> (known after apply)
        name                              = "xxx"
      - private_ip_ranges                 = [] -> null
      ~ rule_collection_groups            = [
          - "/subscriptions/xxx/resourceGroups/xxx/providers/Microsoft.Network/firewallPolicies/xxx/ruleCollectionGroups/DefaultApplicationRuleCollectionGroup",
        ] -> (known after apply)
      ~ sku                               = "Standard" -> "Basic" # forces replacement
        tags                              = {
            "deployedBy" = "terraform/azure/caf-enterprise-scale"
        }
        # (3 unchanged attributes hidden)

      - dns {
          - proxy_enabled = true -> null
          - servers       = [] -> null
        }
    }

Plan: 1 to add, 1 to change, 1 to destroy.

Hi @tlfzhylj
The decision on which Azure Firewall SKU to choose is probably one for another forum. In terms of this module we have included the capability, but you should consult the product documentation to ensure you select the correct SKU for your needs.
It looks like you have the correct document link to enable this comparison.

@matt-FFFFFF Okey, thanks. I can agree with that. Thank you for taking the time to answer anyway :-)


[FEATURE]: Improve how to remove country filters once a selection has been done

What areas do you propose needs enhancement or improvement?
When I select multiple countries in a filter, it is unclear how I can adjust my query by, say, removing a few countries. I am trying to get back in, but can't seem to remove any.
Where is the current state/behavior?

Here is my test: https://sdg-innovation-commons.org/test/all?search=&countries=PSX&countries=PAN&countries=PRY&countries=ZAF&countries=TZA&countries=TUR&countries=UGA
How would it improve the project?
What would be great is to have a simply cross next to each filter tag that could be clicked to remove then.
In addition, it would be helpful to also be able to click back on the drop down and deselect them.
Add why this is beneficial.
Easier for the user to tailor their query and play with the data.
Add any other relevant details
No response

This is related to #88 and will be fixed concurrently.


Front fix question ID length

Change the question ID length to 6 bytes.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 60.9%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3003

Relevant Lines:
4931

💛  - Coveralls


Bug: precedent top-level constants conflicting to namespaced constants

The following causes an error:
CONST = 'Hee'
class Foo
  CONST = 'Foo'
end
#=> ERROR: ConstantAlreadyInitializedError: Constant CONST already been initialized. Can't assign value to a constant twice.. At :2

But this works:
class Foo
  CONST = 'Foo'
end
CONST = 'Hee'

Checked with Goby script and REPL. They work on current Ruby.

I think this problem is the same to #500 and it's been fixed.


FA Asset Pack - Door Frame Issue

Hey there, thank you for this export.  I am just getting into DD and working on a project and came across something that you may or may not have the ability to do anything about.  But maybe we can get something changed.
In the Forgotten Adventures asset pack, there is door frames that can be placed separately from doors.  They are in the portal tool, so you end up with in effect, two doors.  I endedup remaking my map to not use them but would love to see support for door frames.
Anything I can do to help let me know.
Thanks

So you're talking essentially about an open door?  Or something else?
The best way to look at this would be for you do make a small map (like my sampleMap), which is basically just a square building with door(s) in the walls.  You want as few different things in the map to show just what you're trying to accomplish without having a whole bunch of other data that we'd have to wade through.   Attach the exported dd2vtt file here.  I would suggest exporting with the lowest pixel count that you can get away with; keeps the file size small.

Test.zip
Ok, here is the file, exported at 25 PPI.
The single corridor is a normal door from the FA pack without using the door frame.  The two double wide corridors use the door frame and door which over laps.

As far as the file can tell, there's 5 portals in there.  Seems that there's overlapping doors.  So, as far as the export is concerned, you've got 2 doors at the same place.  The question now is what are we expecting that to do?  What do you mean by "support door frames"?

Is there anything in the export that calls out that asset by its name maybe? door frames are not doors, so dont need to be opened or closed, and should just block LOS like a wall.   But with it being a portal asset, maybe there is nothing you can do with it, and I should make the suggestion to DD or FA..

But if its a rectangle, then.. I guess that means you cant do anything with it, unless you subtracted the door from the frame object

No, the only differences between the door and the frame is that the frame is a little wider, and is marked as freestanding.   The assets (not surprisingly) are "gone" in the exported file as all the exported file really is, is a static map plus some line-of-sight and lighting details.  So the exported file doesn't even know if it is a "door frame" vs. a "door".  It's just a "portal".
A door frame should block LOS?  I would have sorta expected it to not block LOS (I can look through a door frame), but rely on the door to block LOS.   I was somewhat thinking that in your map, perhaps the frame should have been marked to "allow light".  Which (unfortunately?) turns the frame into a window.  Which would allow light to pass, but would block movement.

I'm sorta feeling like FA.  It would seem that they should be able to have an asset which can break a wall (perhaps some sort of "arch" asset?) which would make a gap in the wall.  And then the door assets that would fit in that gap.
Now, if DD also had a more distinct difference between Doors and Windows (even better, Secret Doors and Illusory Walls too :) ), that would be nice.

Actually its me thats the problem, thats totally it Imagix.  Use the door frame as the portal, then I am supposed to use the door object to place in the door frame gap.  Only 1 portal.  Just me being a noob. Thanks for helping me....  Sorry for the false positive. Thought I was on to something but nope. lol.  Not supposed to use a door frame portal and a door portal.  Just the frame portal and a door object. Nuances in the learning curve.

No problem.  Maybe we were onto a new feature.  It's good.


Terrain Generation

addresses https://github.com/rewin123/space_editor/issues/94

[x] terrain heightmap generation
[x] meshed terrain
[ ] Biomes
[ ] cubic/quadratic terrain grid
[ ] hexagonal terrain grid (hexx crate)
[ ] feature toggle terrain render system (Mesh, quadratic grid, hexagonal grid, just heightmap - for voxels)
[ ] feature for terrain physics/collision
[ ] UNIT TESTS!
[ ] examples
[ ] Improve noise function
[ ] Have this crate as a portable standalone crate for other projects
[ ] Persistence
[ ] Select vertex
[ ] Edit specific vertex information

I suggest bringing this first before merging it into 0.4. I may be join to work on pr tommorrow

I suggest this for first merging into 0.4. I may be join to work on pr tommorrow

My plans for the PR:

[ ] feature toggle terrain render system (Mesh, quadratic grid, hexagonal grid, just heightmap - for voxels)
[ ] feature for terrain physics/collision
[x] meshed terrain
[ ] cubic/quadratic terrain grid
[ ] hexagonal terrain grid (hexx crate)
[ ] UNIT TESTS!
[ ] examples
[ ] Improve noise function

addresses https://github.com/rewin123/space_editor/issues/94

I will come back to this PR on Jan 3

I messed up this branch, sorry

Closing this in favour of a future brand new PR


Return column structure in case of empty resultset

Describe the bug
Return column structure in case of empty resultset
Steps to reproduce

Use the following query "SELECT 1  as 'column1' LIMIT 0"

Expected behaviour
Have a new setting, which will change behaviour in case of an empty result set being returned.
In case of pandas, driver.query_df(ch_query) should return pd.DataFrame({'column1' : [] })
This was resolved in the previous driver here https://github.com/ClickHouse/ClickHouse/issues/33522

There is no easy way to do this using the HTTP interface, as ClickHouse doesn't return the column name or types in Native Format when the result set is empty.  The driver would have to use a different format (such as JSONEachRow) for the query in order to be able to build the column structure.  Note that the native interface used in clickhouse-driver works differently and does return an "empty block" with the column structure even for empty queries, so the implementation is more straightforward in that case.
If you know ahead of time that the query result will be empty (for example, with your LIMIT 0 example), maybe there could be something like a "structure only" query that would use the JSONEachRow format over HTTP.

Hi @genzgd , Thanks for the reply.
Using query_arrow is am able to return the columns, whilst using 'query_df' the column set is empty. Maybe we can use a partial implementation from pyarrow to get the column lists?
I have tagged the part of code which is returning the empty dataframe which would require an update here https://github.com/ClickHouse/clickhouse-connect/pull/119

Eventually it may make sense to switch the Pandas code to use Arrow instead of the ClickHouse Native format, in particular for Pandas 2.x, which is now backed by Arrow, but that's a pretty significant change.  In the meantime I've opened a feature request https://github.com/ClickHouse/ClickHouse/issues/55939 that would make this possible without major code changes, but I don't know if and when the ClickHouse team will get to it.


Remove the field welfaretype.satisfaction_cost_per_point

Satisfaction is bought through personal measures (which can be a percentage of the round income) rather than as a separate, singular item to buy. Therefore, this field will disappear from the database, and has to be removed from editing of the Welfaretype. See https://github.com/averbraeck/housinggame-common/issues/41

Processed for the admin app.


Add fullscreen button for the gamepad page

The PR adds a fullscreen button to the Gamepad page:

I own a Pixel 6 and find it quite disturbing that the gamepad sometimes scrolls (even with the web installed as an app). I always ended up typing in the url bar javascript: document.body.requestFullscreen(); in order to trigger the fullscreen mode.
The idea of the PR is to have a button for that functionality. I just added added the button, I'm not really sure if the position of it is the best, please tell me if you have in mind any other better way to handle this.

Hi :wave:
Thanks for taking the time to open a PR. I'm not a huge fan of the full screen button because I suspect there's a rist of people clicking it accidentally. Therefore, I tried to come up with an alternative solution to the problem of the vertical scroll. Which, just to be clear, I fully recognize as an annoyance that should be fixed.
Here's the branch of a potential fix without a full screen button: https://github.com/jehervy/node-virtual-gamepads/tree/fix/vertical-scroll-on-wide-screens
Could you give that a spin and let me know if that solves the issue for you?

Hi 👋
Thanks for taking the time to open a PR. I'm not a huge fan of the full screen button because I suspect there's a rist of people clicking it accidentally. Therefore, I tried to come up with an alternative solution to the problem of the vertical scroll. Which, just to be clear, I fully recognize as an annoyance that should be fixed.
Here's the branch of a potential fix without a full screen button: https://github.com/jehervy/node-virtual-gamepads/tree/fix/vertical-scroll-on-wide-screens
Could you give that a spin and let me know if that solves the issue for you?
Finally, please excuse the super late response. Sometimes life just does life things™...

Make it a little different (for example, holding the start button make website fullscreen)


Shopping list

Hi @RitaDee ,
Your project is complete! There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! :+1:

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers, and Happy coding! :1st_place_medal:
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


compilation for humble and ubuntu20

I was not able to compile mesh navigation (humble version) on my ubuntu 20. I fixed it by changing the lvr2 imports in the CMakeLists.txts like I did for the mesh_tools.
More info about my setup:

I compiled ROS2 humble from source
lvr2 - current main branch (develop). Put into the "src" folder of my humble workspace
cmake version 3.16.3

I want to check if it is still compiling with ubuntu 22 before merging this PR.

compiles with ubuntu 22 and binary installed ROS2 humble as well


Add resultSelector argument to both first and last operators

Just what it says. This was primarily for

to enable a user to use first to select an index if they so choose, effectively replacing findIndex as any sort of need in CoreOperators.
Consistency, so that first and last have identical signatures...

To be bikeshedded:

the usefulness of the thisArg
argument order

In argument order, I see defaultValue being more common than thisArg or resultSelector.

I think thisArg could be removed entirely, myself. I've never been a fan of thisArg parameters.

Summoning @mattpodwysocki for this

So I've ended up just removing the thisArg. Since .bind() is available in every browser we're targeting, I don't see the need to have the thisArg, when it seems to junk up the API.
However, @staltz, I didn't put the defaultValue first because it made using the resultSelector without the default argument weird... obs.first(predicate, undefined, resultSelector), you can't do null, because someone might want a defaultValue of null... so the order is obs.first(predicate, resultSelector, defaultValue), and if you want a default value with no result selector it's obs.first(predicate, null, 'hi') I agree it's not as pretty, but I think the odds are so high that users are going to do a map right after they do a first or last, that the resultSelector will be a perf win for those cases.

FWIW, I did experiment with having the order be what you suggested, @staltz.

I think we can merge this for now and let users try it out and see how it feels. As long as we settle on a final API by Beta.

Looks good to me. If defaultValue first didn't work, then your solution has to be the only way. I'd merge


Questions about the paper

Hi! The idea of deriving a token-level guidance interesting!
While I have some questions and confusions on the derivations in your paper, and I hope you can provide some intuitions for me.

In Equation (3), the optimal q minimizes the forward KL, KL(p||q) while restricted to be in the set Q which forces q(y|x) to be 0 when C(x,y)=0. However, the forward KL goes to infinity when q(y|x)=0 while p(y|x) is not 0. Since the choice of C(x,y) and p(y|x) is independent, there could be cases when the support of p is larger than that of C, which leads to inifinite forward KL. The form of the optimal q seems similar to the form derived using reverse KL (which is presented in Khalifa et al. (2021)), so it would be helpful to have a proof on Equation (3), which is not presented in the paper.
In the proof of "Closed-form Solution and Decomposition with Soft Constraints" in Appendix A.2, it is also not very intuitive for me to see how the form of the optimal q is derived. And the forwad KL, KL(p||q) is writtern into \sum_i p_i \log(q_i/p_i), which is more confusing, as it seems to be the negative of KL(p||q).
In Equation (7), the Cross-Entropy loss is calculated on R_theta(x,y) and R_p(x,y) which are not even probability distributions. In most scenarios, where the reward is not bounded in [0,1], can the method still be used?

Hi! Thanks for your interests in our work.
In Appendix A.2 the KL divergence is a typo and we missed a negative sign. Thanks for the catch! In this proof, we divide the KL divergence into two terms: C(x,y)=0 and C(x,y)=1. In both of the terms, the sum of q is fixed (r and (1-r)), and the sum of p is also fixed (Rc and (1-Rc)). We can do the optimization separately.
Let's consider the following optimization problem: D(p,q)=\sum_i p_i \log (p_i / q_i), where  \sum_i p_i = A and \sum_i q_i = B. We can normalize p and q as two distribution by set p' = p / A and q' = q / B.  D(p,q)=KL(p'||q') + A * \log A/B. So D get minimal when p'_i=q'_i, which means p is proportional to q.
Therefore, the optimal q* should be q*(y|x) = Ap(y|x) when C(x,y)=0, and q(y|x)=B*p(y|x) when C(x,y)=1. A and B can be solved.
For question 1, Eq(3) is the special case when r=0 or 1 in Appendix A.2. It makes the support set of p and q are different so the proof in A.2 is not rigorous. However, we believe this does not affect the conclusion and the solution. Actually, we can regard the optimization is done only on the common support set.
For question 3, R_theta and Rc are probabilities. Rc is defined in Eq (1) and R_\theta is the neural approximation of R_theta. This is a cross-entropy between two binary distributions. In this work, we don't have 'reward'. The oracle provides the constraint is sissified or not so it's binary signal. This work can be extended to continuous version if the oracle output is "the probability of the sentence satisfying the constraint". For general reward, it need to be re-scaled to [0,1].
Please let me know if the explanation is not clear or further clarification is needed. Again thanks for your interests in our work!.
-Tao


Bump flink version to 1.15.0.

@SteNicholas Please code review , if your time is available.Thank you~


C++/Python manager interop

What
As an integrator of OpenAssetIO into a host, or a developer of an OpenAssetIO manager plugin, I need to be able to freely split my implementation between one or both of C++ or Python (based on CY2022 versions).
I should not be tied into implementing any specific part of my code in one language or there other aside from for performance reasons to avoid issues such as the Python GIL.
This Epic covers a minimal vertical slice that allows this working methodology to be established, constrained to the "Almost Minimum Viable Manager Plugin" - ie, one that just identifies itself, but doesn't get as far as having an answer to managementPolicy.
Why
The majority of "phase one" adoption targets (both managers and hosts) have code bases where critical business logics is only available in one of the two aforementioned languages. As OpenAssetIO is meant to be the "universal connector", the choice of host language should not dictate that of the manager, and vice versa.
In addition, there are many known hosts whose implementation is split between C++ and Python, in such a fashion that developers leverage both languages to complete any specific feature work.
Acceptance criteria

As a host or manager developer, OpenAssetIO is available through both C++ and Python.

Limited to identifier, displayName and info.
A host written in either language should be able to interact with a manager written in the other, and vice versa.
A manager should be free to implement the ManagerInterface API in both languages at a method-wise granularity.
A C++ to C++ implementation should have absolute minimum intermediate code to support the highest possible performance path.

Manager plugins written in any valid language should be loadable from OPENASSETIO_PLUGIN_PATH.

Dual language implementations supplied via discrete build targets can use the identifier to determine if they are part of the same logical plugin.

Out-of-scope for this epic:

Context, Specification et. al.
Other interface methods.
Compiler isolation
Optimisations such as kField_EntityReferencesMatchPrefix
Python docstrings etc

Tasks

[ ] #193
[ ] Core type system for Map<string, string>
[ ]  C++ <-> Python ManagerInterface shim classes.
[ ] C++ plugin loader

It's worth keeping #201 in the back of the mind when working on this.


Fix for https://github.com/vigonotion/hass-simpleicons/issues/255

This seems to have fixed this issue on my system

Thank you! 🎉


Broadcast page - preview button should be displayed after template selection

Describe the bug

preview button should be displayed after template selection or display like sms or notification preview


sys/freebsd: update pf ioctl definitions

Looks like the last update didn't regenerate the syscall definitions?

Do you mean the constants in pf.txt.const? Those are the ioctl numbers and they didn't change.
I'll fix the pfioc_nv definition, but I'm letting it run a few minutes here first, just in case.

Looks like the last update didn't regenerate the syscall definitions?

Do you mean the constants in pf.txt.const? Those are the ioctl numbers and they didn't change. I'll fix the pfioc_nv definition, but I'm letting it run a few minutes here first, just in case.

The CI task is failing because gmake generate on your branch modifies pf.txt. On my system the result looks like this:
diff --git a/sys/freebsd/pf.txt b/sys/freebsd/pf.txt
index a48f2aafb..154ce83b1 100644
--- a/sys/freebsd/pf.txt
+++ b/sys/freebsd/pf.txt
@@ -154,7 +154,7 @@ pfioc_states_v2 {
 }

 pfioc_nv {
-       data            ptr[in, array[int8]]
-       len             int64
-       size            len[data, int64]
+       data    ptr[in, array[int8]]
+       len     int64
+       size    len[data, int64]
 }

I'm not sure why.

The CI task is failing because gmake generate on your branch modifies pf.txt. On my system the result looks like this:
I'm not sure why.

It looks like it's just unhappy with my whitespace choices. Let's try this.


fix(menu-button): use css variables for padding

fixes #18187

Thank you for the PR! Could you combine this PR with #18188 please?


Wiki home page links broken

https://github.com/ZeBobo5/Vlc.DotNet/wiki
The links for Forms and WPF are circular and point back to the same home page.

They haven't written a documentation yet. So try the samples for now:
https://github.com/ZeBobo5/Vlc.DotNet/tree/master/src/Samples

Getting started has now been written


fix for issue #5

Fix for Disposable.off is deprecated https://github.com/abe33/minimap-color-highlight/issues/5

Thanks for the fix, I'll publish it soon!


Automated translation of README.md (GitHub action)

Draft PR for #600
The Automation currently works as described in the issue

This workflow takes approximately 5 minutes to complete.
Links and code blocks are not translated.
This approach uses the deep_translator library. A free web-scraping technique to access the Google Translate service. For better results, the official Google Translate tool may be better, but it is not free.

I am open to discussions on this.

[!IMPORTANT]
Automates translation of README.md into Chinese, Japanese, and French using a GitHub action and Python script.

GitHub Action:

New workflow translate-readme.yml triggers on README.md changes to automate translation.
Uses deep_translator and markdown-it-py for translation and markdown parsing.

Translation Script:

scripts/translate_readme.py translates README.md into README-CN.md, README-JP.md, and README-FR.md.
Handles text in markdown, preserving non-translatable elements like code blocks.
Splits text into chunks to manage API limits.

Misc:

Adds translated README files: README-CN.md, README-JP.md, README-FR.md.

This description was created by  for b363bea32d936c7f78972d6f3b098cf3321ea027. It will automatically update as commits are pushed.

Appreciate the PR but closing in favor of #624


Extension issue

Issue Type: Bug
Extension Name: vsliveshare
Extension Version: 1.0.3577
OS Version: Windows_NT x64 10.0.19042
VSCode version: 1.52.1

:warning:
{
   "messages": [],
   "activationTimes": {
   	"codeLoadingTime": 8195,
   	"activateCallTime": 2,
   	"activateResolvedTime": 29593,
   	"activationReason": {
   		"startup": true,
   		"extensionId": {
   			"value": "ms-vsliveshare.vsliveshare",
   			"_lower": "ms-vsliveshare.vsliveshare"
   		},
   		"activationEvent": "*"
   	}
   },
   "runtimeErrors": []
}

Apologies for the delay here. This issue doesn’t have enough information to be reproduced. Please reopen if the issue persists with necessary details. Thank you


deregister unlock/twiddle handler on audio play

we can deregister the handler safely once audio is playing
even if user has not clicked yet
Enables Scenario:
Given I have a desktop browser
And unlock is activated by ngAudioGlobals.unlock = true
And I open a page which autoplays a track on load
Then the track will start playing
When I click anywhere on the page
Then the track is not paused
While mobile keeps working fine

:snowman: Approved

:tada: thanks :)


[Housekeeping] - Fixed DemoApp compilation

Jira Ticket

Description

Fixes DemoApp compilation

Commit message

chore: Fix Cocoapods release

Screenshot

Testing

[ ] Added unit tests
[x] Ran unit tests successfully
[ ] Added documentation for public APIs and/or Wiki

Commit message chore: Fix Cocoapods release
Is it correct?

Code coverage
Total coverage: 95.50%

Powered by Slather

  Generated by :no_entry_sign: Danger

Commit message chore: Fix Cocoapods release
Is it correct?

The commit message is correct, I just put the wrong one on the description but it has the right one now

Code coverage
Total coverage: 95.50%

Powered by Slather

  Generated by :no_entry_sign: Danger


Multi-account with LocalStack PRO and cdklocal does not honor the custom account ID

export AWS_ACCESS_KEY_ID=000000000001
export AWS_SECRET_ACCESS_KEY=test
export AWS_REGION=us-east-1

cdklocal bootstrap

Expected behavior: LocalStack creates an ECR repository in account<PHONE_NUMBER>01
Actual behavior: LocalStack creates an ECR repository in account<PHONE_NUMBER>00

@dominikschubert apologies for the ping, is this issue still being looked into?

@dominikschubert Just gave this a quick try with the latest version of LocalStack and cdklocal (using CDK version 2.62.2).
$ export AWS_ACCESS_KEY_ID=000000000001 AWS_SECRET_ACCESS_KEY=test AWS_REGION=us-east-1
$ cdklocal init app --language=python
...
$ pip install -r requirements.txt
...
$ cdklocal bootstrap
...
 ✅  Environment aws://000000000001/us-east-1 bootstrapped.
...
$ awslocal ecr describe-repositories | grep repositoryName
...
            "repositoryName": "cdk-hnb659fds-container-assets-000000000001-us-east-1",

Looks like this issue is potentially already solved in the meantime?
@vincenthongzy Are you able to replicate these results? Please note that for ECR to work fully end-to-end, you'll need a LocalStack Pro license (free trial available). Please keep us posed on how it goes.. Thanks!

Hi @vincenthongzy! We just wanted to follow up to see whether your issue has been resolved. Were you able to get it working with the latest version of LocalStack? We would appreciate your feedback!

Apologies for the late reply, but yes this does work with the latest version of localstack-pro and aws-cdk-local !

Thank you for your feedback. We are delighted it's working! If you experience any other issues, please do not hesitate to contact us.


Some light grammar and capitalization fixes

There was small mistake in emails1 but I've changed that. Thanks.


clarify some things I noticed in the eye tracking setup docs

Two different sections discussed workarounds for the GazeInput capability that is not yet available in Unity. I consolidated them into the first section (A note on the GazeInput capability) and stripped the second section (Testing your Unity app on a HoloLens 2) down to a simple reminder. I don't think the information about hand-editing the XML is necessary--people who are interested in that could make the edit in MRTK or Visual Studio and then look into the resulting changes for themselves.
Added step to expand _'Input Data Providers' -- it isn't expanded by default and someone wouldn't see '+ Add Data Provider' if that isn't expanded.
Added step to clone DefaultMixedRealityInputSimpulationProfile.
EyeGazeCursor now seems to be in Foundation, so MRTK Examples aren't needed.

[EDIT: Removed doc changes related to IsEyeTrackingEnabled based on @keveleigh feedback.]

As a reviewer, it is possible to check out this change locally by using the following
commands (substituting {PR_ID} with the ID of this pull request):
git fetch origin pull/{PR_ID}/head:name_of_local_branch
git checkout name_of_local_branch

/azp run mrtk_docs

/azp run mrtk_docs


Feature request: Change notification sound in mobile apps

Requesting feature to change notification sound in iOS and Android-apps. Would also be super nice to have different sounds based on priority.
Would it be possible to implement a feature where the notification sound repeats x-times (or until its read)?

@BeFs You can already define a per-priority sound through Android notification channels. If you long-press on the ntfy icon and then select "App info" -> "Manage notifications", you can define behavior per priority.

Would it be possible to implement a feature where the notification sound repeats x-times (or until its read)?

I don't think this is technically possible, though I suppose alert apps like OpsGenie can do that too. :thinking:

It would also be cool to implement specific notifications on the presence of keywords or regular expressions.

Very related #6

@BeFs You can already define a per-priority sound through Android notification channels. If you long-press on the ntfy icon and then select "App info" -> "Manage notifications", you can define behavior per priority.

Would it be possible to implement a feature where the notification sound repeats x-times (or until its read)?

I don't think this is technically possible, though I suppose alert apps like OpsGenie can do that too. 🤔

On iOS the only alert currently offered is the basic “pinggg” sound.
Many apps offer a range of alert sounds, if not their own at least the iOS built in ones.
Perhaps Ntfy can do the same.

I think I'm closing this in favor of the duplicate ticket #6.


Params has encoded

Hello everyone!
I found this answer from stackoverflow:
scrape_configs:

job_name: 'somename'
params:
prefix: ['finatra']
url: ['http://:9090/admin/metrics.json']

When i do that,i also get params encoded:
url=http%3A%2F%2F%3A9090%2Fadmin%2Fmetrics.json
Is there any idea?

Thanks for your report. It looks as if this is actually a question about usage and not development.
It makes more sense to ask questions like this on the prometheus-users mailing list or our forums rather than in a GitHub issue. In our community channels, more people are available to potentially respond to your question, and the whole community can benefit from the answers provided.


evaluation of stitcher

Because
We want to see if the stitcher/smoothing added via https://github.com/clamsproject/app-swt-detection/issues/33 is doing well, independent from the accuracy of image-level classification model.
Done when
Controlled evaluation is done to measure the effectiveness of the stitcher. The evaluation at high-level, should be measuring the performance difference between raw image classification results and re-constructed image classification results from TimeFrame annotations.
Additional context
Original idea of having this evaluated was proposed by @owencking in his email on 12/15/2023. Here's an excerpt from it.

Suppose we have a set of still frames F from a full video.  And suppose we have human/gold labels of all those frames. Suppose we have a CV model M and a stitching algorithm S.  Then a full SWT app is the composite M+S and can take a video as input.
Generate two sets of image classification predictions for all the frames in F:

The first prediction set P1 just uses M (choosing the label with max score across the output categories) to predict one label for each frame.
The second prediction set P2 is generated by using M+S on the original video to generate time-based annotations, then producing a label prediction for each frame in F according to the label in the annotation for the time period containing the frame.

Compare both P1 and P2 to the gold labels of the frames in F.  Then we can evaluate how good S is according to how much P2 improves over P1.  This gives us a way of evaluating different stitching algorithms (somewhat) independently of the CV model.  So it will allow us to tell whether performance improvements for time-based eval metrics are coming from the image classifier or the stitching algorithm.

Done pretty much as described above with one difference. Trying to mimic the sample rate was impossible since the app at the moment only accept milliseconds and the rate used for the annotation was using some number of frames I think. So I just used a frame for the annotation that was within at most 32ms.


reduced default guzzle set up

set up guzzle client as very minimal, not throwing exceptions for HTTP status codes, not following redirects, not handling cookies
fix #13

:+1:

:+1:

This is actually causing me a problem. Guzzle6HttpAdapter::sendRequest() is returning null. When I look deeper into it, I see the guzzle promise is an instance of GuzzleHttp\Promise\RejectedPromise with reason: "No handler has been specified".
When I revert this change, it works as expected.

Fix is on it's way

@kbond can you try this fix: #21?


Remove assert that verifies if tables were marked

A summary of this would be that having multiple workers and multiple
insert/select on the same table causes a lot of issues. Please refer
yourself to the issue below to have a better idea.
fixes glossier#2

Wrong repo. -- To delete.


Floating point truncated to zero warnings

xsimd/math/xsimd_numerical_constant.hpp:94:5: warning: floating constant truncated to zero [-Woverflow]
     XSIMD_DEFINE_CONSTANT(smallestposval, 1.1754944e-38f, 2.225073858507201e-308)
     ^~~~~~~~~~~~~~~~~~~~~

xsimd/math/xsimd_rem_pio2.hpp:237:17: warning: floating constant truncated to zero [-Woverflow]
                 2.16741683877804819444e-51, /* 0x3569F31D, 0x00000000 */
                 ^~~~~~~~~~~~~~~~~~~~~~~~~~

I have these warnings when I compile my code with included xsimd.hpp under Mingw-G++ 8.1.0

I was surprised myself. When I replaced 1.1754944e-38f and 2.225073858507201e-308 as a test with std::numeric_limits<float>::lowest() and std::numeric_limits<double>::lowest() respectively the first warning went away. If this is novel here, I guess I need to look for source of the issue on my side. Will leave it open for now until I reconfirm.

@lhog I cannot reproduce the issue with 2.16741683877804819444e-51 . I've dig a bit and this is a magic value used internally, we don't want to change it (at all). A reproducer on https://godbolt.org/ would certainly help...

Yep, very strange thing. I expect it might be some compiler quirk. My project is unfortunately too big to fit into godbolt.
P.S. I ended up replacing constant from the first warning with FLT_MIN and DBL_MIN and wrapping constants like this double(2.16741683877804819444e-51L), in the second case just to mute the annoying warning.

@serge-sans-paille I don't remember why we went with these ard-coded constants instead of using std::numeric_limits, I agree it would be more elegant to use this latter.


Resolve #1130 where sed and tee is trying to access files at the same time

Signed-off-by: Peter Zhu<EMAIL_ADDRESS>Description
Resolve #1130 where sed and tee is trying to access files at the same time.
If we use sed to access file while tee, it would sometimes cause a conflict and result in tee creating empty file due to lock being hold by sed.
Therefore, we cat the file, pipe to sed, then tee, would resolve issue.
Also, fixed an error where sed is not correctly setup for the pattern.
Issues Resolved
#1454
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #1458 (b33273f) into main (780c28d) will decrease coverage by 8.25%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main    #1458      +/-   ##
============================================
- Coverage     94.67%   86.41%   -8.26%
- Complexity        0       12      +12
============================================
  Files           120       21      -99
  Lines          3003       81    -2922
  Branches          0        8       +8
============================================
- Hits           2843       70    -2773
+ Misses          160        4     -156
- Partials          0        7       +7

Impacted Files
Coverage Δ

src/build_workflow/builder.py

src/assemble_workflow/bundle_recorder.py

src/build_workflow/build_recorder.py

src/system/temporary_directory.py

...teg_test_start_properties_opensearch_dashboards.py

src/system/execute.py

...rkflow/integ_test/service_opensearch_dashboards.py

src/ci_workflow/ci_test_manifest.py

...orkflow/integ_test/integ_test_runner_opensearch.py

...flow/dependency_installer_opensearch_dashboards.py

... and 131 more

Continue to review full report at Codecov.

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


panic on unwrap when using valgrind

I am trying to profile my code using valgrind, but when I run the compiled benchmark binary I get an unwrap panic (haven’t found the trace yet) If I run without valgrind it works fine, as well as running through valgrind if I use rusts benchmark harness.
My suspicion is that sth changes about stdin/stdout/stderr that makes criterion upset but I haven’t been able to pinpoint it yet.

Hey, thanks for trying Criterion.rs!
Unfortunately there isn't much I can do to help here without more information. You could maybe try running Criterion.rs in debug mode (see here) and see if that produces any useful output. Another option would be to use the --measure-only flag to disable all of the reporting and statistical code (which is generally a good idea when profiling Criterion.rs benchmarks anyway).
Hope this helps! If you find more information then please reopen this issue.


Ability to configure attachment support for models in extra-openai-models.yaml

In ~/Library/Application Support/io.datasette.llm/extra-openai-models.yaml, I have this:
- model_id: or:c35s
  model_name: anthropic/claude-3.5-sonnet:beta
  api_base: "https://openrouter.ai/api/v1"
  api_key_name: openrouter

When I try to use -a some_image.png, I get:
Error: This model does not support attachments, but some were provided

❯ llm --version
llm, version 0.17.1

PS: It works with gemini-1.5-pro-latest accessed directly from Google. The problem is probably with the OpenRouter indirection.

From #593:

I added custom gpt-4o azure model through extra-openai-models.yaml and Api proxy config is
- model_id: 4o-azure
  model_name: azure-gpt4o
  api_base: "http://<IP_ADDRESS>:4000/"
  api_key_name: azure-gpt4o

then command returns
"describe this image" \
      -a IMG_20240908_161441.jpg
Error: This model does not support attachments, but some were provided

Is there option to extra models that indicate model supports attachments or any other function?


no noBacklight() function ?

Hi,
I'm surprised not to see a noBacklight() function. It will be nice to have this feature.
regards,
Jim

Does this function not work?
https://github.com/Seeed-Studio/Grove_LCD_RGB_Backlight/blob/master/rgb_lcd.h#L140

I would rather use the following block, to avoid any blinking issue:
lcd.setPWM(REG_RED, 0);
lcd.setPWM(REG_GREEN, 0);
lcd.setPWM(REG_BLUE, 0);


ValueError: Rank(A) < p using cvxopt and RuntimeWarning: invalid value in data.py

Posting to help any future GPkit'ers debug.
I have a GPkit model (casey.py) that prompts the following error/traceback:
$ ipython casey.py
C:\Python27\lib\site-packages\gpkit\nomials\data.py:189: RuntimeWarning: invalid value encountered in double_scalars
  mmap[i][j] = mag(expmap[exp][j]/c)
variable1 has no upper bound
Using solver 'cvxopt'
Solving for 40 variables.
     pcost       dcost       gap    pres   dres
 0:  0.0000e+00      nan  7e+01    nan    nan
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
C:\Users\philippe.kirschen\Documents\casey.py in <module>()
    191
    192 m = Model(objective, constraints, substitutions)
--> 193 m.solve(verbosity=5)

C:\Python27\lib\site-packages\gpkit\constraints\prog_factories.pyc in solvefn(self, solver, verbosity, skipsweepfailures, *args, **kwargs
)
     78         else:
     79             self.program, solvefn = genfunction(self, verbosity-1)
---> 80             result = solvefn(solver, verbosity-1, *args, **kwargs)
     81             solution.append(result)
     82         solution.program = self.program

C:\Python27\lib\site-packages\gpkit\geometric_program.pyc in solve(self, solver, verbosity, *args, **kwargs)
    180             sys.stdout = self.solver_log   # CAPTURED
    181             solver_out = solverfn(c=self.cs, A=self.A, p_idxs=self.p_idxs,
--> 182                                   k=self.k, *args, **kwargs)
    183             self.solver_out = solver_out
    184         finally:

C:\Python27\lib\site-packages\gpkit\_cvxopt.pyc in cvxoptimize(c, A, k, *args, **kwargs)
     42     g = log(matrix(c))
     43     F = spmatrix(A.data, A.row, A.col, tc='d')
---> 44     solution = gpsolver(k, F, g, *args, **kwargs)
     45     return dict(status=solution['status'],
     46                 primal=solution['x'],

C:\Python27\lib\site-packages\gpkit\_cvxopt.pyc in gp118(K, F, g, G, h, A, b, kktsolver, **kwargs)
    257         else: return f, Df, H
    258
--> 259     return cp(Fgp, G, h, dims, A, b, kktsolver = kktsolver)

C:\Python27\lib\site-packages\cvxopt\cvxprog.pyc in cp(F, G, h, dims, A, b, kktsolver, xnewcopy, xdot, xaxpy, xscal, ynewcopy, ydot, yaxp
y, yscal)
   1967
   1968     sol = cpl(c, F_e, G_e, h, dims, A_e, b, kktsolver_e, xnewcopy_e,
-> 1969          xdot_e, xaxpy_e, xscal_e)
   1970
   1971     sol['x'] = sol['x'][0]

C:\Python27\lib\site-packages\cvxopt\cvxprog.pyc in cpl(c, F, G, h, dims, A, b, kktsolver, xnewcopy, xdot, xaxpy, xscal, ynewcopy, ydot,
yaxpy, yscal)
    786             singular_kkt_matrix = False
    787             if iters == 0:
--> 788                 raise ValueError("Rank(A) < p or "\
    789                     "Rank([H(x); A; Df(x); G]) < n")
    790

ValueError: Rank(A) < p or Rank([H(x); A; Df(x); G]) < n

With MOSEK the error was
/Users/philippekirschen/Documents/MIT/Research/GPkit/gpkit/gpkit/nomials/data.py:189: RuntimeWarning: invalid value encountered in double_scalars
  mmap[i][j] = mag(expmap[exp][j]/c)
variable1 has no upper bound
Using solver 'mosek'
Solving for 40 variables.
Warning: The variable with index '0' has only negative coefficients akj.
 The problem is possibly ill-posed.
MOSEK error 1375: A large value of      inf has been specified in cx for variable '' (11).
Number of Hessian non-zeros: 47
* Solving exponential optimization problem on dual form. *
* The following log information refers to the solution of the dual problem. *
Computer
  Platform               : MACOSX/64-X86

Problem
  Name                   :
  Objective sense        : max
  Type                   : GECO (general convex optimization problem)
  Constraints            : 41
  Cones                  : 0
  Scalar variables       : 87
  Matrix variables       : 0
  Integer variables      : 0

Optimizer started.
Interior-point optimizer started.
Presolve started.
Linear dependency checker started.
Linear dependency checker terminated.
Eliminator - tries                  : 0                 time                   : 0.00
Eliminator - elim's                 : 6
Lin. dep.  - tries                  : 1                 time                   : 0.00
Lin. dep.  - number                 : 0
Presolve terminated. Time: 0.01
Matrix reordering started.
Local matrix reordering started.
Local matrix reordering terminated.
Matrix reordering terminated.
Optimizer  - threads                : 4
Optimizer  - solved problem         : the primal
Optimizer  - Constraints            : 5
Optimizer  - Cones                  : 0
Optimizer  - Scalar variables       : 16                conic                  : 0
Optimizer  - Semi-definite variables: 0                 scalarized             : 0
Factor     - setup time             : 0.02              dense det. time        : 0.00
Factor     - ML order time          : 0.00              GP order time          : 0.00
Factor     - nonzeros before factor : 62                after factor           : 86
Factor     - dense dim.             : 0                 flops                  : 0.00e+00
ITE PFEAS    DFEAS    GFEAS    PRSTATUS   POBJ              DOBJ              MU       TIME
0   0.0e+00  0.0e+00  0.0e+00  0.00e+00   0.000000000e+00   0.000000000e+00   0.0e+00  0.04
Interior-point optimizer terminated. Time: 0.04.

Optimizer terminated. Time: 0.07
Return code from optimize - 1432

Interior-point solution summary
  Problem status  : UNKNOWN
  Solution status : UNKNOWN
* End solution on dual form. *
Transforming to primal solution.
Solving took 0.098 seconds.
---------------------------------------------------------------------------
RuntimeWarning                            Traceback (most recent call last)
/Users/philippekirschen/Desktop/casey.py in <module>()
    191
    192 m = Model(objective, constraints, substitutions)
--> 193 m.solve(verbosity=5)

/Users/philippekirschen/Documents/MIT/Research/GPkit/gpkit/gpkit/constraints/prog_factories.pyc in solvefn(self, solver, verbosity, skipsweepfailures, *args, **kwargs)
     78         else:
     79             self.program, solvefn = genfunction(self, verbosity-1)
---> 80             result = solvefn(solver, verbosity-1, *args, **kwargs)
     81             solution.append(result)
     82         solution.program = self.program

/Users/philippekirschen/Documents/MIT/Research/GPkit/gpkit/gpkit/geometric_program.pyc in solve(self, solver, verbosity, *args, **kwargs)
    199                 " If the problem was Primal Infeasible, you can generate a"
    200                 " feasibility-finding relaxation with model.feasibility()." %
--> 201                 (solvername, solver_out.get("status", None)))
    202
    203         self._generate_nula(solver_out)

RuntimeWarning: final status of solver 'mosek' was 'UNKNOWN', not 'optimal'.

The infeasible solve's result is stored in model.program.solver_out. A result dict can be generated  via program._compile_result(program.solver_out). If the problem was Primal Infeasible, you can generate a feasibility-finding relaxation with model.feasibility().

The (embarrassing) mistake was that one of my substitutions was 'variable7': 3000/3600, which evaluates to zero (should have been 3000./3600).


Error on search empty

db.search({
query: '',
fields: ['field']
}).then(function (res) {
alert(res.total_rows); })
always return 0

pouchdb-quick-search cannot return all? This is one useless project. Please remove this project on github

@nolanlawson Care to reply?


[Lession4] schedule function explanation seems incorrect

Someone can you please help me to understand this explaination of the _schedule?

The algorithm works like the following:
The first inner for loop iterates over all tasks and tries to find a task in TASK_RUNNING state with the maximum counter. If such task is found and its counter is greater then 0, we immediately break from the outer while loop and switch to this task. If no such task is found this means that no tasks in TASK_RUNNING state currently exist or all such tasks have 0 counters. In a real OS, the first case might happen, for example, when all tasks are waiting for an interrupt. In this case, the second nested for loop is executed. For each task (no matter what state it is in) this loop increases its counter. The counter increase is done in a very smart way:
The more iterations of the second for loop a task passes, the more its counter will be increased.
A task counter can never get larger than 2 * priority.

My understanding is if all task are waiting for an interrupt, after the first for loop, 'c'=-1 and the if(c){break} will break the while loop and the second for loop will not be executed.
If my understanding is correct, the above explanation should be corrected.
I pasted the function here for convenient:
void _schedule(void)
{
    preempt_disable();
    int next,c;
    struct task_struct * p;
    while (1) {
        c = -1;
        next = 0;
        for (int i = 0; i < NR_TASKS; i++){
            p = task[i];
            if (p && p->state == TASK_RUNNING && p->counter > c) {
                c = p->counter;
                next = i;
            }
        }
        if (c) {
            break;
        }
        for (int i = 0; i < NR_TASKS; i++) {
            p = task[i];
            if (p) {
                p->counter = (p->counter >> 1) + p->priority;
            }
        }
    }
    switch_to(task[next]);
    preempt_enable();
}

Fairly old post, but the explanation is correct. Since c = -1, the if statement containing the break will not execute and it will fall through to the second for loop.
Normal case is that some process will have a counter value of 1 or more and will have the running state (as of this lesson). In this case, this will be the chosen task, and we will switch to it. In the case that all processes are waiting for an interrupt, the statement: "if (p && p->state == TASK_RUNNING && p->counter > c)" will never be true . Therefore, c will remain -1 and "if (c)" will be false and we will fall through to the second for loop.
There is also the case where all tasks have counter of 0, and as the paragraph mentions, the second for loop will increase their counter within their priority range. We will stay in the while loop until at least on task has a state RUNNING.

I know this is an old response now, but I am responding because I have the same question as the op.
In C, if c=-1, then C interprets this as a truth value, so the if (c) will be true, and we will break the while loop and the second for loop will not be executed as the op said.


make compatible with django 3.2 / django 4.0

Hello,
I've done a few fixes and bumped the version up to 0.8.0 so that the package works again under modern django versions (tested on django 3.2 and 4.0).
Greetings,
jgb

Why remove the pyproject.toml file? It's needed to use poetry to build.

Also, it's deliberate that the package is inside the src directory.

Hello @schinckel, in a first step I simply fix the compatibility with django 4.0, but the package wouldn't install via pip because there are syntax errors in pyproject.toml (strings that start with double quote, but are closed with single quote).
I then fixed those syntax errors, but still the package was unable to build and install (I forgot which error exactly).
Since I got the impression that commit e454e59ba57d5f44646ae362924a63fac0e0813c was a "work in progress" kind of thing, the easiest solution on my side was to simply revert it.
Please feel free of course to correct any mistakes I've made, all I did on my side was just get the package up and running without issues on django 3.2 / django 4.0.
Thanks.

@funkybob @schinckel any updates on this PR?

I'm not willing to just dump poetry.
It would have been better if you'd captured the error you got, and we could have fixed that there.
I'm going to spend some time tonight trying to fix it myself.

I'm not willing to just dump poetry. It would have been better if you'd captured the error you got, and we could have fixed that there. I'm going to spend some time tonight trying to fix it myself.

I'm not asking to drop anything, all I'm saying is that the master branch in its current state is not installable with pip.


TypeError: Cannot read properties of undefined (reading 'length')

I keep getting error with the main branch when I do google searches.

Digging into this a bit more it seems after using the google plugin the selectedConversation is set to an incorrect value somewhere.
Some console logging in Chat.tsx shows it gets set to something crazy at some point but my react is not good enough to debug where this incorrect value is coming from:
Working:
selected conversation:  {id: '2a98be35-b912-4947-b3dd-1319daa5f005', name: 'New Conversation', messages: Array(3), model: {…}, prompt: 'You are a helpful chatbot. Respond using markdown.', …}
Broken:
selected conversation:  (updatedConversation, allConversations)=>{
const updatedConversations = allConversations.map((c)=>{
if (c.id === updatedConversation.id) {
return updatedConversation;
…

I've run a pull request, have a fresh setup, but I keep running into
selectedConversation?.messages.length === 0
chatbotui.com:

and locally:


chore: convert all examples to yaml

There is currently no reason to have the petstore json example and the oas3 one in JSON. Remove and unify.

Where do these *.no-auth.oas3.yaml files come from?

We have them in the repo:
https://github.com/stoplightio/prism/blob/master/packages/http-server/src/__tests__/fixtures/petstore.no-auth.oas3.yaml
https://github.com/stoplightio/prism/blob/master/packages/http-server/src/__tests__/fixtures/petstore.no-auth.oas2.yaml

Where do these *.no-auth.oas3.yaml files come from?

We have them in the repo:

I don't know what I was looking at 😅