RFC: Pattern Guards with Bind-By-Move

Rendered view
Tracking issue
Stabilization PR

Now that this RFC is merged, the above "rendered view" link is broken.  The text of this RFC can now be found here:
https://github.com/rust-lang/rfcs/blob/master/text/0107-pattern-guards-with-bind-by-move.md

Any ideas/progress on that? I think this is perfectly valid code:
fn f(x: Option<String>) {
    match x {
        Some(a) if &a == "Hello" => println!("x is Hello"),
        Some(b) => println!("x is not Hello but {}", b),
        None => println!("none"),
    }
}

But borrowcheker currently won't accept it

Isn't this in 1.39? (i.e. Beta and Nightly)

Wow, guys, you're so quick.
I wonder if I ask about GAT and they appear in 2 minutes thereafter... 😄


Add phrasedml section to documentation

We need a phrasedml section in the docs analog to the antimony section:
https://tellurium.readthedocs.io/en/latest/antimony.html#
This will solve also #237

This will include an updated version of the tutorial #237


nixos/unbound: check validity of config file

Description of changes
After bringing down the shared dnscache which is using unbound, I thought I do something about not doing that again.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

There's a NixOS test for unbound that appears to break with this change, could you take a look at it?

error: builder for '/nix/store/jmwmdfwp4hyg48fli78ifc33b506dkbx-unbound-checkconf.drv' failed with exit code 1;
       last 2 log lines:
       > unbound.conf:14: error: cannot open include file '/etc/unbound/extra*.conf': No such file or directory
       > read unbound.conf failed: 1 errors in configuration file
       For full logs, run 'nix log /nix/store/jmwmdfwp4hyg48fli78ifc33b506dkbx-unbound-checkconf.drv'.

That test is now fixed. I've added an option to control the config check which gets disabled when include is used.

@srhb can you recheck that things are now working as expected?

This validity checking fails when setting services.unbound.settings.remote-control.control-enable = true, as unbound-checkconf ends up checking if the certificates exist in /build/state/: https://github.com/NLnetLabs/unbound/blob/be626f7c5330dc414a582a04b537ea79d5c452fb/smallapp/unbound-checkconf.c#L910-L922
Related: #293001.

Similar issue: when specifying an interface name instead of an IP for the interface option, checkconf wants to verify that the interface exists, which isn't the case in the sandbox.

I don't think we can easily check that. I would suggest to just turn the config check off in this cases.


Unclear what Create remote play invite link is supposed to do

It probably doesn't help that nothing is working for me at the moment (https://github.com/m4dEngi/RemotePlayWhatever/issues/44#issuecomment-1059739755), but the Create remote play invite link button could do with some documentation. Is it supposed to copy something to the clipboard, open a new UI, ...?

It's unclear mostly because in it's current state app is broken.

Not an issue with 0.1.14. Behaviour is obvious when the app actually works.


Better paddings for the old feed x2

.Details seem not to exist in my feeds, so I removed it for better compatibility.

Very strange how many variations of this feed GitHub seems to push...


KAFKA-17268: Fix NPE when closing a non-started acceptor

As titile, the serverChannel will be null if Acceptor's thread is not started. Hence, closing a non-started will produce NPE
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

When handling Closeable objects, wouldn't it be better to use Utils.closeQuietly() instead of CoreUtils.swallow()?

When handling Closeable objects, wouldn't it be better to use Utils.closeQuietly() instead of CoreUtils.swallow()?

that is good idea, as we can eliminate a lot of null check. I file https://issues.apache.org/jira/browse/KAFKA-17285 to address it.


Logo and links disappearing when site is resized

The logo and links in the header will disappear when window width is reduced, menu is opened and closed, and width is put back to fullscreen.
Steps to reproduce:
Reduce window width to less than 770px
Open hamburger menu
Close hamburger menu
Resize window to more than 770px

Should be easy to fix. Somehow the menu acually sets an inline style to display: none at the menu:
Window less than 770px:

Menu clicked:

Menu clicked again:

This overwrites the default css. So just removing the last step should solve everything.


chore: add dependabot and codeql

Is CodeQL needed to spilt into a separated file?

thanks! but maybe we should remove the instruction comments? we may keep some links of these instructions though like line 4 in dependabot.yml.

hi i've done some modifications on workflow.yml, is it ok with you?

it should be OK


Update errors_impl.py

Added the Code example for tf.errors.ResourceExhaustedError.
Fixes #29847.

To be a doctest, it should have a way to test the output, the example should show some output

@mihaimaruseac,
Added the output.


Getting "Panicked at 'not implemented: citext'"

What language are you using?
Python 🐍
What version are you using?
Tested 0.3.0 and alpha 0.2.5 (Mac M1 release)
What database are you using?
Postgres
What dataframe are you using?
Pandas
Can you describe your bug?
When trying to:
cx.read_sql(postgres_url, query)

I get:
thread '<unnamed>' panicked at 'not implemented: citext', /Users/runner/work/connector-x/connector-x/connectorx/src/sources/postgres/typesystem.rs:100:22
Same URL and Query has been used with default pandas read_sql and I didn't have any issue.
I'm on a Macbook Pro 2020 M1
OS Version: MacOs Monterey 12.5.1

rustup --version
rustup 1.25.1 (bb60b1e89 2022-07-12)
info: This is the version for the rustup toolchain manager, not the rustc compiler.
info: The currently active `rustc` version is `rustc 1.64.0 (a55dd71d5 2022-09-19)`

What are the steps to reproduce the behavior?
import connectorx as cx

postgres_url="postgresql://user:pwd@localhost:5432/database"
query = "select * from name_of_the_table"
cx.read_sql(postgres_url, query)

Database setup if the error only happens on specific data or data type
Happens in all tables.
Example query / code
select * from users;

What is the error?
thread '<unnamed>' panicked at 'not implemented: citext', /Users/runner/work/connector-x/connector-x/connectorx/src/sources/postgres/typesystem.rs:100:22

Hi @Sirsirious , can you try version 0.3.1-alpha.9 to see whether it works now?

I can confirm that now I don't receive the same error and that I can have the expected results. Is there any scheduling for the stable release?

@Sirsirious Planning to have next stable release 0.3.1 by the end of this month.


cmp: EqLit may break the query

There is a query builder code using the table model  table.SchedRun.UpdateBuilder("status").If(qb.EqLit("status", "RUNNING")), it generates
AAA [query statement="UPDATE scheduler_task_run SET status=? WHERE cluster_id=? AND type=? AND task_id=? AND id=? IF status=RUNNING " values=[FOO 30f3ac8b-4f29-4beb-8b1d-d40020586a96 mock fe0e6697-ad19-47af-a359-8c2f8d7a4e0d eea3301d-275b-11ec-bf87-c85b76f42222] consistency=QUORUM]

Which fails with no viable alternative at input.
As it turns out the issue is the lack of quotes i.e. IF status='RUNNING'.
The workaround is by using EqNamed instead.

From what I have seen using this function, we must not use a single quote ( ' ) when the column type is UUID for an instance. I personally fixed my issue like this:
qb.EqLit("string_id", fmt.Sprintf("'%s'", stringid))
Since we can't have the column type in qb, I don't think there is any good way to fix this function. we could update the docs, or add a new function like
qb.EqLitStr that does like this:
func EqLitStr(column, literal string) Cmp {
	return Cmp{
		op:     eq,
		column: column,
		value:  litStr(literal),
	}
}

type litStr string

func (l litStr) writeCql(cql *bytes.Buffer) (names []string) {
	cql.WriteString("'")
	cql.WriteString(string(l))
	cql.WriteString("'")
	return nil
}


DB backups daily/weekly

Backup database data daily/weekly as .sql inserts or raw data

http://stackoverflow.com/questions/6645818/how-to-automate-database-backup-using-phpmyadmin

Used by 3th party CI too +  AWS Lambda


Equivalent of Shared Worker Port in Service Worker?

Shared Worker uses Message Port to send messages between parent worker and Shared Worker.
Parent worker
var port = new SharedWorker('/worker.js').port;
port.onmessage = function(e){
    console.log(e.data);
};
port.start();
port.postMessage("Hello Shared Worker. I'm Mr. Tab");

Shared Worker (worker.js)
onconnect = function(e){
    var port = e.ports[0];
    port.onmessage = function(e){
        console.log(e.data);
    };
    port.start();
    port.postMessage("Hello Mr. Tab. I'm Shared Worker.");
};

Is there anything similar to Shared Worker Port in Service Worker?

Yep!
Messaging from page to ServiceWorker:
// in the page:
navigator.serviceWorker.controller.postMessage("Hi!");

// in the ServiceWorker:
self.onmessage = function(event) {
  event.source; // instance of Client
  event.data; // "Hi!"
  // reply
  event.source.postMessage("Hey!");
};

// back in the page:

navigator.serviceWorker.onmessage = function(event) {
  event.source; // instance of ServiceWorker
  event.data; // "Hey!"
};

I think Chrome currently sends postmessages from ServiceWorker to Window to window.onmessage, based on an earlier spec.

@jakearchibald My navigator.serviceWorker.controller is always null. Do you have any ideas about this strange behaviour?

 Look at chrome://serviceworker-internals and check for a successful registration & what its scope is

@jakearchibald I've just found my issue. My scriptURL is /app/sw.js, and the page url is /. How can I make it work without moving the script to root? Thank you.

@tresdin You'd have to move the script to root or else use the Service-Worker-Allowed HTTP header. See #468 . BTW if you're using a recent version of Chrome, it has a message explaining this when you try to register for scope '/':
"DOMException: Failed to register a ServiceWorker: The path of the provided scope ('/') is not under the max scope allowed ('/app/'). Adjust the scope, move the Service Worker script, or use the Service-Worker-Allowed HTTP header to allow the scope."

Thank you @mattto . I've added the header Service-Worker-Allowed : / but nothing changed.  I'm using Chrome 41.
BTW, I wonder how the Service-Worker-Allowed header can fix it since there is no http request sent to server once the error occured.

Sorry, that's because Service-Worker-Allowed is available in Chrome 42+. I'd recommend testing on Chrome Dev or Canary channel. Anyway this this getting pretty implementation-specific as opposed to a spec discussion, so if you have further questions about Chrome specifically, you can use https://groups.google.com/a/chromium.org/forum/#!forum/service-worker-discuss or file bugs at crbug.com.

@mattto I've already asked few questions on stackoverflow. However, I finally found it's easier to receive useful replies on Github. :)

@tresdin can you link me to your stackoverflow questions? I'll answer them there, and make sure they're picked up in my searches in future.

@jakearchibald I just have these two questions.

The one you've already answered here
http://stackoverflow.com/questions/29230470/equivalent-of-shared-worker-port-in-service-worker
And the other if you're interested in.
http://stackoverflow.com/questions/28882289/service-worker-vs-shared-worker

@jakearchibald I've just applied your solution on Chrome 43. And event.source is null on ServiceWorkerGlobalScope.

@tresdin I guess that hasn't been implemented yet - that part of the design wasn't done until fairly recently. You can hack around it by passing in a channel https://jakearchibald.github.io/isserviceworkerready/demos/postMessage/

@jakearchibald Actually, none of your solutions is an equivalent of Shared Worker Port. See my answer for more information.
http://stackoverflow.com/questions/29230470/equivalent-of-shared-worker-port-in-service-worker/29320291#29320291

@jakearchibald Actually, none of your solutions is an equivalent of Shared Worker Port. See my answer for more information.
http://stackoverflow.com/questions/29230470/equivalent-of-shared-worker-port-in-service-worker/29320291#29320291

Thanks @jakearchibald (u have great blog btw), @tabatkins


Fix to keep first triplet from DRKG in DataFrame

Need to use header=None for pd.read_csv() in order to ensure that first triplet in drkg.tsv is not treated as header of table

@mufeili Would you like to review this?

Could you take a look? @classicsong I believe you authored this notebook.

LGTM


Publish 11/05/2021, 3:30 PM

Docs Build status updates of commit 8fe73c7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.gitignore
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/Get-DlpPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-DlpPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Remove-DlpPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-DlpPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

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 docs.microsoft.com contributor guidesPost your question in the Docs support channel


Change the semantics of exprs/for loops allocations strategy

This commit adjusts how exprs and for loops are handled within rsx. This is a breaking change in terms of codegen, but has slight semantic changes as well.
Now, when exprs/for loops are allocated, they are given a temporary. The temporary is elided to the <'a> lifetime of the bump, to satisfy the borrow checker. This fixes issues with signals where exprs/for loops mapping vecs out of RefCells would be caught up without a temporary lifetime.
Specifically, before, this component could not compile:
fn app(cx: Scope) -> Element {
    let todos = use_signal(cx, || vec!["hi"]);

    render! {
        ul {
            for todo in todos.read().iter() {
                li { "{todo}" }
            }

            todos.read().iter().map(|todo| rsx! {
                li { "{todo}" }
            })
        }
    }
}

The error we were getting is:
error[E0597]: `todos` does not live long enough
  --> src/main.rs:13:25
   |
9  |     let todos = use_signal(cx, || vec!["hi"]);
   |         ----- binding `todos` declared here
...
13 |             for todo in todos.read().iter() {
   |                         ^^^^^^^^^^^^
   |                         |
   |                         borrowed value does not live long enough
   |                         a temporary with access to the borrow is created here ...
...
22 | }
   | -
   | |
   | `todos` dropped here while still borrowed
   | ... and the borrow might be used here, when that temporary is dropped and runs the destructor for type `std::cell::Ref<'_, std::vec::Vec<&str>>`
   |
   = note: the temporary is part of an expression at the end of a block;
           consider forcing this temporary to be dropped sooner, before the block's local variables are dropped
help: for example, you could save the expression's value in a new local variable `x` and then make `x` be the expression at the end of the block
   |
11 ~     let x = render! {
12 |         ul {
 ...
20 |         }
21 ~     }; x
   |

For more information about this error, try `rustc --explain E0597`.
error: could not compile `demo-signals-todo` (bin "demo-signals-todo") due to previous error

This is fixed now, fixing a major papercut of signals and the work being done in fermi to allow truly global state.
This works because we now do a codegen from
make_node(#expr)

to
{
    let node = #expr.make_node(cx);
    node
}

It technically worked before, apparently, but I've updated it to do the temporary coercion for both.
I think there's no performance impact since the temporary should get merged during codegen.


[Lean Core] Add deprecation warning for status-bar

Summary
Adds a deprecation warning for using status-bar package
Lean Core Issue: #23313
Changelog
[General] [Deprecated] - Deprecated NetInfo as it has now been moved to @react-native-community/netinfo
Test Plan
Open the RNTester app to the StatusBar examples and see the deprecation warning yellow box.

I have signed  CLA

@dmtrKovalenko can you double check your Changelog? Seems to mention NetInfo instead.

@ericlewis Fixed


Supporting Arrays

Any plans to support arrays, this is somewhat of a big issue

Agreed. This would save a LOT of heartache.
I'm using houston for a project that relies on arrays of objects and the collection view containing the array is becoming unusable.

:+1:  To be clear, this is arrays for New Items or attributes.  If an array is already setup with some content it works fine to edit, but if you try to populate an item with an empty array and object is created instead.


Typo in README.md

In the README.md file there is a typo. It should be "your README" rather than "you README" imo.

Thanks for catching that!

Fixed in ed32b8a!


Add the OpenSSF Scorecard GitHub Action

Proposal
Hello, I'm working on behalf of Google and the Open Source Security Foundation to help essential open source projects improve their supply-chain security. Given prometheus’ importance to monitoring and alerting, we'd like to recommend an improvement to the projects security posture.
Would you consider adopting an OpenSSF tool called Scorecards? Scorecards runs dozens of automated security checks to help maintainers better understand their project's supply-chain security posture. It is developed by the OpenSSF, in partnership with GitHub.
I see prometheus’ already follows security best practices, such as cryptographic release signing and CI/CD through CircleCI. Adding Scorecards could help ensure other best practices, such as the usage of Token Permissions mentioned in https://github.com/prometheus/prometheus/pull/11285.
The Scorecard GitHub Action is very lightweight and runs on each change to the repository. The results of its checks are available on to the project's security dashboard, and include suggestions on how to solve any issues (see examples below). Over 1600 projects have added the action already.
Would you be interested in a PR which adds this Action? Optionally, it can also publish your results to the OpenSSF REST API, which allows a badge with the project's score to be added to its README.

Examples

It is interesting, feel free to open in a pr.


column reference "region" is ambiguous (SQLSTATE 42702) running control.foundational_security_ecr_1

Describe the bug
When running the foundational_security benchmark, we are always getting the error "column reference "region" is ambiguous (SQLSTATE 42702)"  at the control control.foundational_security_ecr_1
Powerpipe version (powerpipe -v)
Example: v0.1.3
Steampipe version (steampipe -v)
Example: v0.22.0
Plugin version (steampipe plugin list)
hub.steampipe.io/plugins/turbot/aws@latest | 0.136.1 | aws
hub.steampipe.io/plugins/turbot/steampipe@latest | 0.10.0 | steampipe
To reproduce
executing the following command:
steampipe check benchmark.foundational_security --export foundational_security_{ACCOUNT_NAME}_2024-06-20.csv --progress=false
Expected behavior
The error doesn't occur running the benchmark.
Additional context
The Benchmark is executed in a container using the latest turbot/steampipe image.

Hi @semeiner,
Sorry for the radio silence on this issue. Could you please answer the following questions to make it easier for us to replicate/debug this issue?:

Which container are you using to run the Benchmark?
Did you try using the powerpipe command? Does it throw the same error?
Thanks.

Hello @khushboo9024,
thank you for the response, using powerpipe command fixed the issue.
I tried running the benchmark inside a fargate container.
I appreciate the help!


MultiIndexable: Make #shape_internal mandatory and #shape optional (currently the opposite)

Right now, we expect MultiIndexables to implement #shape, which should return a non-mutable version of their shape. Usually, this means that an implementor has to write the following:
def shape
  @internal_shape.dup
end

def shape_internal
  @internal_shape
end

We did this because not all MultiIndexables neccessarily keep their shape around as an internal buffer, and so shape_internal is not always meaningful. In those cases, MultiIndexable provides a default implementation of #shape_internal that just returns the output of #shape.
I think that the opposite makes more sense - namely, #shape should by default return a copy (or ReadonlyWrapper) over #shape_internal. Then, the implementor only has to worry about defining #shape_internal, even if it's not actually exposing a member variable.

I've been playing around with making this change, and having #shape return a ReadonlyWrapper around shape_internal - it's kind of neat, but it introduces a fair bit of user-facing complexity that I'd like to avoid for now. I think we should stick to arrays for the time being.

For now, I've made the proposed change (without involving ReadonlyWrapper).


My project does not use Tailwind but it requires it to build

Describe the bug
I have a project that does not use Tailwind, but when I run npm run build, I see the following:

Now, when I try to deploy my project to production, it fails during the build process because it requires the tailwindcss package.
I already deleted and recreated my package-lock.json with no luck.
Here's the contents of my package.json:
{
  "name": "admin",
  "version": "0.0.0",
  "scripts": {
    "dev": "vite",
    "build": "vite build",
    "serve": "vite preview"
  },
  "dependencies": {
    "@fullcalendar/core": "^5.9.0",
    "@fullcalendar/daygrid": "^5.9.0",
    "@fullcalendar/interaction": "^5.9.0",
    "@fullcalendar/list": "^5.9.0",
    "@fullcalendar/timegrid": "^5.9.0",
    "@popperjs/core": "^2.9.3",
    "@rollup/plugin-yaml": "^3.1.0",
    "@sindresorhus/slugify": "^2.1.0",
    "@tinymce/tinymce-vue": "^4.0.4",
    "@vue/cli-plugin-router": "^4.5.13",
    "autoprefixer": "^10.3.1",
    "axios": "^0.21.1",
    "bytes": "^3.1.0",
    "codemirror": "^5.62.2",
    "color": "^4.0.1",
    "cropperjs": "^1.5.12",
    "date-fns": "^2.23.0",
    "dayjs": "^1.10.6",
    "diff": "^5.0.0",
    "dompurify": "^2.3.1",
    "dotenv": "^10.0.0",
    "escape-string-regexp": "^5.0.0",
    "html-entities": "^2.3.2",
    "joi": "^17.4.2",
    "jsonlint-mod": "^1.7.6",
    "lodash": "^4.17.21",
    "lodash-es": "^4.17.21",
    "marked": "^3.0.0",
    "md": "^0.6.0",
    "micromustache": "^8.0.3",
    "mime": "^2.5.2",
    "mitt": "^3.0.0",
    "nanoid": "^3.1.25",
    "pinia": "^2.0.0-rc.5",
    "qrcode": "^1.4.4",
    "qs": "^6.10.1",
    "resolve-url-loader": "^4.0.0",
    "sass": "^1.38.0",
    "tinymce": "^5.8.2",
    "uuid": "^8.3.2",
    "vue": "^3.2.4",
    "vue-flatpickr-component": "^9.0.4",
    "vue-i18n": "^9.1.7",
    "vue-loader": "^15.9.8",
    "vue-router": "^4.0.11",
    "vuedraggable": "^4.0.3"
  },
  "devDependencies": {
    "@vitejs/plugin-vue": "^1.3.0",
    "@vue/compiler-sfc": "^3.2.4",
    "vite": "^2.4.4"
  }
}

Reproduction
It happens when I install the project dependencies and try to run npm run build.
System Info
#10 1.402 ><EMAIL_ADDRESS>build /app/frontend/admin
#10 1.402 > vite build
#10 1.402
#10 2.311 vite v2.5.0 building for production...
#10 2.431 transforming...
#10 6.810 ✓ 190 modules transformed.
#10 6.813 [vite:css] Loading PostCSS Plugin failed: Cannot find module 'tailwindcss'
#10 6.813 Require stack:
#10 6.813 - /app/frontend/admin/noop.js
#10 6.813
#10 6.813 (@/app/postcss.config.js)
#10 6.813 file: /app/frontend/admin/src/app.vue?vue&type=style&index=0&scoped=true&lang.scss
#10 6.814 error during build:
#10 6.814 Error: Loading PostCSS Plugin failed: Cannot find module 'tailwindcss'
#10 6.814 Require stack:
#10 6.814 - /app/frontend/admin/noop.js
#10 6.814
#10 6.814 (@/app/postcss.config.js)
#10 6.814     at load$1 (/app/frontend/admin/node_modules/vite/dist/node/chunks/dep-98dbe93b.js:19511:11)
#10 6.814     at /app/frontend/admin/node_modules/vite/dist/node/chunks/dep-98dbe93b.js:19536:16
#10 6.814     at Array.map (<anonymous>)
#10 6.814     at plugins (/app/frontend/admin/node_modules/vite/dist/node/chunks/dep-98dbe93b.js:19535:8)
#10 6.814     at processResult (/app/frontend/admin/node_modules/vite/dist/node/chunks/dep-98dbe93b.js:27837:14)
#10 6.814     at /app/frontend/admin/node_modules/vite/dist/node/chunks/dep-98dbe93b.js:27951:14
#10 6.814     at async resolvePostcssConfig (/app/frontend/admin/node_modules/vite/dist/node/chunks/dep-98dbe93b.js:28681:22)
#10 6.814     at async compileCSS (/app/frontend/admin/node_modules/vite/dist/node/chunks/dep-98dbe93b.js:28504:27)
#10 6.814     at async Object.transform (/app/frontend/admin/node_modules/vite/dist/node/chunks/dep-98dbe93b.js:28193:50)
#10 6.814     at async ModuleLoader.addModuleSource (/app/frontend/admin/node_modules/rollup/dist/shared/rollup.js:19708:30)
#10 6.840 npm ERR! code ELIFECYCLE
#10 6.841 npm ERR! errno 1
#10 6.849 npm ERR<EMAIL_ADDRESS>build: `vite build`
#10 6.849 npm ERR! Exit status 1

### Used Package Manager

npm

### Logs

_No response_

### Validations

- [X] Follow our [Code of Conduct](https://github.com/vitejs/vite/blob/main/CODE_OF_CONDUCT.md)
- [X] Read the [Contributing Guidelines](https://github.com/vitejs/vite/blob/main/CONTRIBUTING.md).
- [X] Read the [docs](https://vitejs.dev/guide).
- [X] Check that there isn't [already an issue](https://github.com/vitejs/vite/issues) that reports the same bug to avoid creating a duplicate.
- [X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/vue-next instead.
- [X] Check that this is a concrete bug. For Q&A open a [GitHub Discussion](https://github.com/vitejs/vite/discussions) or join our [Discord Chat Server](https://chat.vitejs.dev/).
- [X] The provided reproduction is a [minimal reproducible example](https://stackoverflow.com/help/minimal-reproducible-example) of the bug.

Hi,
I have more information about the issue.
My project has the following structure:

root...
frontend/admin
frontend/surveys
package.json
resources/js
tailwind.config.js
...

So, about the above:

frontend/admin is one standalone app.
frontend/surveys is another standalone app.
inside resources/js lives another standalone app

The resources/js uses tailwind, it also uses the package.json and tailwind.config.js files from the root dir.
The frontend/admin does not uses tailwind, and it is built with npm i && npm run build.
To test my theory, I moved the application that does not require TailwindCSS to a completely different directory, and then it will not require TailwindCSS during build.
So, the issue seems to be that, for any reason, the Vite build command is looking at the directories above.
Just to clarify, when I build the frontend/admin I do cd frontend/admin && npm install && npm run build. So it should not consider anything that's outside that directory.

Ok, I have a repo to reproduce the issue: git@github.com:maxacarvalho/vite-tailwind-issue.git
It happens once I place a postcss.config.js on the root folder. For any unknown reason (at least to me) the Vite build process is "reading" the postcss.config.js that's located two levels above and taking that into account.

That's the expected behavior of PostCSS:
Vite uses https://github.com/postcss/postcss-load-config/ underlyingly, which depends on lilconfig to find the project PostCSS configuration.
lilconfig tries to mimic cosmiconfig whenever possible.
Per cosmiconfig documentation:

Cosmiconfig continues to search up the directory tree, checking each of these places in each directory, until it finds some acceptable configuration (or hits the home directory).

Not sure how we can improve the error message, but it's not a bug.

Hi @sodatea, thanks for your reply and explanation.
Just to be clear, I think this is an amazing project and I'm not trying to BS it at all.
But, although technically that's not a bug, as you said it relies on tools and those tools use that as normal behavior, it is (IMHO) a very problematic one.
Let's imagine a shared server that has all sorts of crazy filesystem structures? Any postcss.config.js above the application's directory would trigger that possible error.
The optimal solution in my opinion would be to have the builder only scan the root directory. But, maybe, as an "easier" solution, a "fake" postcss.config.js could be added to the root directory, to be safe. It could be a part of the docs perhaps?

@sodatea I think this issue relates to the fact that Tailwind is missing from your devDependencies in your package.json. To add it do a npm install tailwindcss -D OR yarn add tailwindcss -D

This looks expected to me. If you place postcss.config.js in the root, all tooling that uses postcss-load-config would load that by default. The docs already documented Vite uses postcss-load-config.

Also agreed that this is expected. Vite doesn't modify how tools load their own config files where possible. Please create a discussion to keep discussing though, and maybe a concrete bug report or feature request could be created from there in the future.


bug-fix:checkbox checked attr on unchecked

[x] I have followed (at least) the PR section of the contributing guide.

What problem are you trying to solve? It's usually best to create an issue and discuss it first.

I am closing the PR, please create an issue describing the problem first.


Add device ID for Adaprox Fingerbot (y6kttvd6)

Just a small bit of copy-paste work really to support the Adaprox branded Fingerbot (non-plus) as well.

Update please README.md as well.

Split it into Fingerbot and Adaprox Fingerbot because the Adaprox has a built-in battery unlike the Tuya one that uses a CR2. Is that okay?

Yes, thank you.

One more question. Since your device has the built-in battery, has it a datapoint for charging state? Could you please check it?  Here is an instruction how to obtain datapoint ids for Tuya devices: https://www.zigbee2mqtt.io/advanced/support-new-devices/03_find_tuya_data_points.html#_9-find-your-data-point

One more question. Since your device has the built-in battery, has it a datapoint for charging state? Could you please check it? Here is an instruction how to obtain datapoint ids for Tuya devices: https://www.zigbee2mqtt.io/advanced/support-new-devices/03_find_tuya_data_points.html#_9-find-your-data-point

Sorry for the delay, but I could not my Tuya devices to show up in their dashboard and retrieve the DP IDs for you.

I even tried creating a new account and adding the Fingerbot with and without Bluetooth bridges, but I could not get the devices to show. I also have a project in the IoT dashboard for Home Assistant, but that also shows zero devices, even though they do load in Home Asssistant. I also tried tuya-cli but it couldn't connect to Bluetooth child devices (using a bluetooth bridge)
Sorry!


Scroll touch

There is no way to scroll the list by touching the touch input device
https://youtu.be/5aZ2e52IVZM

it works fine
https://jsfiddle.net/Fega/wpmxj5uq/3/


Surelog paths need to be stripped.

On a side note, modern (built today) Surelog seems to have broken the post_process step by adding double quotes in the paths written in file.lst so you might want to add path = path.strip('"') at the beginning of the for path in filelist.read().split('\n'): loop. (around line 140 in surelog.py )
Originally posted by @suzizecat in https://github.com/siliconcompiler/siliconcompiler/issues/849#issuecomment-1015402219
This note was really helpful. I'm also using a fresh compiled surelog, I had the same error and after adding
path = path.strip('"')
it vanished.

If this is a bug in Surelog, please report a bug to Surelog and we'll endeavour to fix them....

I'm not sure if this is a bug in Surelog. What changed is that their slpp_all/file.lst which lists the *.v files now prints double quotes around the paths.

Closing; fixed in #898 (thanks for the contribution)! We aren't updating the "official" bundled Surelog yet, but this change should be harmless on old versions.


add: recipes

It works in an engineyard instance, though, it could depend on automake.

User should first make sure of automake installed, and install automake might not be the task of this chef file.

Thank you.

Is using the master branch safe?

It could be a problem.

I now see there is release br., then I shoild use this.br..

I take it as bit 'pressure' on releasing :).
in any sense, I have no reason for reject merging it to master.
btw, I'd like to change name of the directory that might named including 'chef'.
Do you have any idea on naming?

It can be handled by berkshelf if the project has recipes directory.

hm... then it'll be better to keep it.
Thanks.


stream video on mobile CORE-8463

The current video flow involves first downloading the video locally, and then playing it back from the file. This has a number of disadvantages, the primary being it takes up a lot of space, and it is isn't even clear that playing from the encrypted file is fast enough to work at all. Patch does the following:
1.) Adds a new function to attachments.Store, called StreamAsset. The point of the function is to return an io.ReadSeeker that can be fed into http.ServeContent to run the video playback.
2.) Add a new class, signencrypt.decodingReadSeeker which will bring an io.ReadSeeker interface to a signencrypt'd payload. It takes an underlying io.ReadSeeker source which will feed the ciphertext into it. Read will figure out the "chunk" of the stream that we need to fetch from the source and decrypt them.
3.) Add a LRU cache for use with signencrypt.decodingReadSeeker that allow us to keep a fixed amount of data in memory for use with repeated requests for the same chunk. Currently we use 20MB. A typical pattern of requests (from io.Copy) is for Read requests of 32KB. Each of those will hit the same chunk many times, and so this cache is essential.
4.) Add a minimum number of chunks to Read on a cache miss so that we don't make too many calls to the underlying source (AWS S3). Right now this is 4MB, but we probably will want to play around with this going forward.
5.) Change the frontend flow on mobile to play the video on first tap. We also now download the file before Save and Share do anything. Also add the size to the duration text.

cc @malgorithms


Issue formatting bottom for Year

As mentioned in https://github.com/awakesecurity/purescript-precise-datetime/issues/32, I think this is probably because bottom for Year is negative: https://github.com/purescript/purescript-datetime/blob/ffdd94fa0c4ac5d89107b0437aece0556970dcef/src/Data/Date/Component.purs#L28
> import Data.Formatter.DateTime (FormatterCommand(..), format)
> import Data.List (fromFoldable)
> format (fromFoldable [ YearFull ]) bottom
"000-271820"

Possibly related to https://github.com/slamdata/purescript-formatters/issues/45


Add onboarding to plotly treemap

closes #97

Added the onboarding to show the chart title.
Created the treemap.ts to the package/core/src.
Created the treemap.ts to packages/plotly/src.
treemap is added to EVisualizationType

screenshots:

The cause of the scatterchart only showing the stage reading is in the switch / case statement in index.ts of our core package:

If you forget a break then the script will run from the case where the criterion is met and will run the cases after that regardless if a criterion was met.

So we need to add a break in every case statement, so that the cases after the case with the met criterion are not run.


Add a Gitter chat badge to README.md

underscoreio/doodle now has a Chat Room on Gitter
@davegurnell has just created a chat room. You can visit it here: https://gitter.im/underscoreio/doodle.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

I've added the "Join the Chat" badge manually.


suggesting edits to readme - 1

@nahusznaj same here. It's pointing to an old version of the tutorial. Open it again pointing to the main branch if changes apply. Thank you!


Avoid cancel when mic closed + update node to v10

It appears that AoG has changed how it behaves when given input outside of the action. Previously it would ignore that input and leave the conversation closed. Now it appears to open a new conversation which confuses the tests.
Consequently, always saying "cancel" after every test will cause problems when there are multiple tests in a single file.
This PR detects when the mic is closed and skips saying "cancel" if so.
Another change that appears to have happened to AoG is that the debug_info content is no longer being returned. This means that much of the content (including the previous flag for the microphone being closed!) is no longer available.
This PR grabs the microphone_mode value from the dialog_state_out which is still returned to discover if the mic has been closed.
Additional changes in this PR
Added a prepare lifecycle step to package.json so that the project can be used as a dependency straight from a github branch.
Updated node to v10 and typescript output to ES2017 to match -- node 8 is now deprecated and node 10 is the minimum LTS. Completely understand if you want to do the node update as a separate PR.

argh - committed with wrong email. moving to https://github.com/actions-on-google/actions-on-google-testing-nodejs/pull/50


database/sql: incorrect count for DBStats.MaxLifetimeClosed for connections not closed in background goroutine

What version of Go are you using (go version)?

$ go version
go version go1.13.8 darwin/amd64

Does this issue reproduce with the latest release?
Based on the latest master, yeah.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/colinking/Library/Caches/go-build"
GOENV="/Users/colinking/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GONOPROXY="github.com/segment*"
GONOSUMDB="github.com/segment*"
GOOS="darwin"
GOPATH="/Users/colinking/dev"
GOPRIVATE="github.com/segment*"
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/Cellar/go/1.13.8/libexec"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/Cellar/go/1.13.8/libexec/pkg/tool/darwin_amd64"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/colinking/dev/src/github.com/segmentio/cell-ctlstore/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/p6/62gms5fd2sd51_yf9qzbrs9c0000gn/T/go-build662696632=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
We were tracking the number of closed connections over the lifetime of a process with MaxLifetimeClosed from DBStats after setting a max connection lifetime with SetConnMaxLifetime.
What did you expect to see?
We expected MaxLifetimeClosed to be incremented once for every connection that was closed due to SetConnMaxLifetime.
What did you see instead?
Only connections closed via the background goroutine (connectionCleaner) were counted. Connections that are closed in conn (here and here) because it determines they are expired (and currently free) were not counted.
If this is the expected behavior, then I'm happy to open a PR with a fix.

@bradfitz @kardianos @kevinburke

Yes, I believe you are correct. Those are closed due to max lifetime, but are not recorded in Stats.
Please keep this CL in mind if you send a patch.


Supporting a flexible dataset schema

Xpublish-EDR currently support a very narrow dataset schema. For example, coordinates must be named X, Y, and T corresponding to longitude, latitude, and time. When datasets do not conform to this, there is a somewhat confusing error raised about CF conventions:
https://github.com/xpublish-community/xpublish-edr/blob/019e53acd2e0ad5a1d909d1acfe9863f2e90e51b/xpublish_edr/plugin.py#L79-L85
Of course, there are many ways a dataset can be CF complaint (with or without these coordinates present). My understanding of the EDR spec is that it is agnostic to the names of these coordinates in the underlying data source. So Xpublish-EDR should be able to find a way to bridge between coordinate names provided they are real CF.
Can we work on making Xpublish-EDR more flexible here? CF-Xarray could offer a shortcut here.

@abkfenris and I have talked about this before, because a lot of the (not super thought out) abstractions for xpublish-wms overlap a ton with EDR functionality, and same with our newer subsetting prototype plugin.
Both of these are built on top of cf-xarray, which as you mentioned is probably the most important piece here.

Xpublish-EDR should be using CF-Xarray's coordinate criteria to figure out which coordinates correspond to X, Y, and T, and not require datasets to have coordinates be named exactly that.
Here's the attributes on one of my datasets that is playing nice with Xpublish-EDR/CF-Xarray (though my Xpublish server is currently being cranky, so I gotta go figure that out):

Thanks both for your input here. I'm realizing now that I jumped to a conclusion about CF that may not have been correct. I'll do a bit more digging and return with a more concrete request.

I think there is probably a place for configuration on both the plugin and dataset sides.
We could enable setting up the ERD plugin with some custom coordinate criteria so folks could enable matching on patterns that are common in their datasets.
On the dataset side, it would probably make sense to have a standard attribute pattern that can prompt plugins but doesn't get sent to users by plugins that would otherwise share attributes.
ds['funky_time'].attrs['_xpublish']['edr']['axis'] = 'T'

After looking at this in more detail, I understand the problem -- our dataset is simply missing the axis (or similar) attributes for X, Y, and T.
The feature request I would offer here then is to have some sort of verification tool that runs when registering a dataset with a router. If an error or warning had been issued at startup, this would have been much easier to debug.

Hmm, I don't think we'd want to make it the default, since with dataset providers Xpublish doesn't necessarily know of all datasets at startup. We also don't have a hook currently to call plugins on startup, but after datasets may be available.
How about an endpoint to check if the dataset has the right attributes?
There is an existing issue in CF-Xarray for some sort of checker https://github.com/xarray-contrib/cf-xarray/issues/366 but I don't think we need to be blocked by that. I think we can do a simpler query to see if CF-Xarray has identified the axes.


Twitpic content provider

Quite simple

I should have rolled back first

I will rollback the changes and launch another pull request


Updated Hybrid data manager version number to 1.0.0

Only the last commit has been made after merge to master.
Older PR Link

@t-prta Please resolve the merge conflicts


On roster student show page, add list of repos for which student is an owner, or a collaborator

[ ] Create a new route in the controller at, for example https://ucsb-cs-github-linker.herokuapp.com/courses/31/roster_students/2514/repos
[ ] At that new route, have a placeholder page that repeats some of the information from the roster_student show page: First Name, Last Name, github id, email, as a table with one row of headers, and one row of data.   Email should be a link to the email, github should be a link to github.com/username
[ ] Use the github api to fetch a list of repos that the student is an owner of that are from the organization.   Make the list an HTML table with column headings repo name, public/private.  The reponame should be a link to the repo.
[ ] Use the github api to fetch additionally, any repos on which the student is a collaborator.  Add a column heading for owner/collaborator.  Indicate in that column whether the student is an owner or a collaborator on the repo.

A few notes:
If you look at https://github.com/project-anacapa/anacapa-github-linker/blob/dcbf82b4503e3fdc862ca3b0164735326fe3becb/app/controllers/courses_controller.rb#L185 you'll see this code:
    def machine_user
      client = Octokit_Wrapper::Octokit_Wrapper.machine_user
    end

That private method (which we should probably factor out into some kind of helper mix-in) gives you a handle to the machine_user, which is a Github User that has access to the organization, and that can do Github operations on behalf of the application.
Next, look at line 139 in the same file.  We see that machine_user can be used to do Github API calls through Octokit.  The documentation for Octokit is here: https://github.com/octokit/octokit.rb and here http://octokit.github.io/octokit.rb/
Note that the key to unlocking the mystery is thatmachine_user is an instance of Client in the Octokit api, so the methods of Client are the ones you can invoke on machine_user.
Those are here: https://www.rubydoc.info/github/pengwynn/octokit/Octokit/Client
result = machine_user.create_org_hook(
        @course.course_organization,
        {
          :url => webhook_url,
          :content_type => 'json',
          :secret => ENV['GITHUB_WEBHOOK_SECRET']
        },
        {
          :events => ['*'],
          :active => true
        }
      )

In your case, you want to look up repos in an organization that belong to a particular user.  You'll probably be looking for method such as these:

Did a lot of research into the GitHub repos display for student page and it's actually quite a bit more challenging than it would appear on its surface. The problem is that GitHub's API is structured in a way that requires a huge amount of data to be requested to get at this pretty narrow scope. Here are the options I've explored and why they either don't work or are resource prohibitive:

Asking the GitHub API for a list of a user's repos only returns their personal repos, not any repos they've created or contributed to in any organizations.

Another option is to fetch all of the repos in the organization, but the output lists the organization and not the user as the owner, so you would have to make a separate request for every single org repo to find out which users contribute to the repository and then list only those repos which the user contributes to. As you might imagine, this can be hundreds of API calls for a single user's repo list which makes it tremendously inefficient.

GitHub has an Events API which allows you to get certain activity data within the past 90 days about some user or organization. There is an API endpoint which allows you to get all the events in an organization that would be visible to a specific user. You could search through this list of events for a set of repositories that the user has triggered commit/push/other write events for without making tons of requests. Unfortunately, you can only make this request if your authentication token is specifically for the user in question, meaning that a different user cannot get this narrow information set even if they own the organization.

You could use the GitHub Events API to fetch all events in the organization in the past 90 days and then sift through these for write events by the user to figure out which repos they were members of. This would request EVERY organization event so it would require requesting a huge amount of information just to get the user's organization repos. And of course, if there is any repo that was last modified more than 90 days ago it won't show up (not an issue for the quarter system but could cause occasional issues for other schools).

Given these poor options, I don't believe it's doable to create these lists when a TA/instructor/admin goes to view the user's repos. It may be most feasible to simply maintain these lists for each users in the course of real-time webhook integration so that when a user triggers a commit/push/other write event, the program adds the repo the event was made in to the user's repo list unless the repo is already in the list. It's of course not ideal and requires a much more thoughtful implementation but it does not seem to me at the moment that there is an easy way to do this unless I'm missing something.

Closing this issue as this functionality has now been implemented.


Shedulet print starts immediately after non sheduled print is completed

I have had at least two instances of Print Scheduler starting another print right after non-scheduled print completed.
Steps to reproduce.

Use "M80" G-code action to turn on the printer (it fails for whatever reason)
Manually start a print
Manually started print completes and the scheduled print is started leading to a crash.

Was it a print that was scheduled to happen between the time you powered on the printer and the manually started print completed? That would be expected in this case, because the plugin has no checks for bed clearing, etc

No, print was scheduled to start at 8:00, at around 8:20 I noticed that it didn't start and manually powered up the printer and started the print. After the print was finished, another print started and crashed the printhead into the prints.

Have you confirmed the timezone/date and time settings are correct on your pi? What you describe is kind of what I would expect for a missed scheduled print. The plugin checks every minute on a loop and will start any that are earlier than when it checks and is not printing. To troubleshoot further it would benefit having debug logging enabled for the plugin in OctoPrint's logging settings and that will save some additional information to octoprint.log to diagnose.
Like I mentioned before, the plugin doesn't have any way of knowing if the bed is clear or not and ready to go. If you have a camera you could utilize my Bed Ready plugin and add an additional line to your slicer's start gcode to check the bed prior to starting.

I don't use RPi
My clock appears to be set up correctly, at least when looking at PrintJob History.

I don't use RPi

The underlying OS doesn't really matter, what matters is local time is correct. If you are running any linux variant, this may be helpful in verifying. https://linuxize.com/post/how-to-set-or-change-timezone-in-linux/

I don't use RPi

The underlying OS doesn't really matter, what matters is local time is correct. If you are running any linux variant, this may be helpful in verifying. https://linuxize.com/post/how-to-set-or-change-timezone-in-linux/

Time, date and timezone are set correctly

Yeah, so the plugin is working as expected. It's even called out in the warnings on the plugin listing and the readme here that this is something to look out for.

WARNINGS:

This plugin currently only has knowledge of your printer’s state. If your printer is showing operational in OctoPrint, the next scheduled print job will run even if something is on the bed. Take caution when adding multiple prints to the scheduler for this reason.

Using something like Bed Ready or Bed Check AI plugins to avoid this is the only thing currently available. I could potentially add a pre-print confirmation prompt on the OctoPrint UI side, but that ends up defeating the purpose of scheduling prints if it requires user interaction.

Yeah, so the plugin is working as expected. It's even called out in the warnings on the plugin listing and the readme here that this is something to look out for.

WARNINGS:

This plugin currently only has knowledge of your printer’s state. If your printer is showing operational in OctoPrint, the next scheduled print job will run even if something is on the bed. Take caution when adding multiple prints to the scheduler for this reason.

Using something like Bed Ready or Bed Check AI plugins to avoid this is the only thing currently available. I could potentially add a pre-print confirmation prompt on the OctoPrint UI side, but that ends up defeating the purpose of scheduling prints if it requires user interaction.

The warning states that next print job will happen, not that any previously scheduled job that wasn't already started will be started. This is different. I wouldn't have an issue if I started a print manually at 5:00, it completed at 7:00 and PJS crashed my printer at 8:00 when the print was scheduled to start. The issue is that any not yet started job will be started.
If job fails to start, maybe within 5 minutes, the task should be considered failed and a message displayed to the user.

The warning states that next print job will happen, not that any previously scheduled job that wasn't already started will be started.

Fair point, your interpretation of next scheduled print job is probably more accurate than what I meant, which was next job in the list that is in the past.
You're idea of a timeout is something I'd consider to integrate, thanks for the suggestion.


Enhancing returned errors for specialresourcemodule controller.

This is part of the logging enhancement effort which says that:

all errors should be wrapped with its context
error logging should only be done at the reconciliation loop level
info and warning logs can be everywhere in the code

Signed-off-by: Yoni Bettan<EMAIL_ADDRESS>

/cc @yevgeny-shnaidman

/retest

@yevgeny-shnaidman PTAL

/lgtm

Created a follow up task for adding info logs to this controller.


agent cannot handle Linux environment variables [CLI]

Description
when I tried to pass a local environment variable to the program, it cannot interpret the variable.
command:
  - ${env}
  - python3
  - '-m'
  - ${program}
  - '--data_dir=$SLURM_TMPDIR'
  - ${args}

it supposed tot run as /usr/bin/env python3 -m scripts.train --data_dir=/home/local.scratch/*/  but it runs /usr/bin/env python3 -m scripts.train --data_dir=$SLURM_TMPDIR
Environment

Linux
Environment: Cluster
Python Version: 3.8.*

Hey @MohammadJavadD , thanks for reporting this. Can you confirm that this is the behavior with the latest wandb version (0.12.0) ?

@tyomhak I run with wandb version 0.11.2. should I update to 0.12.0?

Thanks! We're looking into this issue.

Hi @MohammadJavadD , my apologies for the delay. We've opened a ticket for this internally and will let you know in this thread when we fix this issue.


Inverting condition on example

Inverting error handling condition on README.md example to fix issue #198

Nice! Thank you 👏


The value of device orientation property is constant

Problem description
The value of device.orientation property does not change when the orientation changed.
Expected behavior
The value of device.orientation property should change when the orientation changed.
Environment

Tabris.js version: 3.2.1
Device: Any
OS: Any

Code snippet
import {contentView, device, TextView} from 'tabris';

contentView.append(<TextView padding={16} markupEnabled font={'16px'} lineSpacing={1.4}/>);

device.onOrientationChanged(({value}) => renderDeviceProperties());
renderDeviceProperties();

function renderDeviceProperties() {
  $(TextView).only().text =
    <$>
      Orientation: {device.orientation}

    </$>
  ;
}

The orientation was cached in JS, the fix for that is currently in gerrit. However, in Android it still does not give the correct value. It seems to give the previous orientation. Thoughts @mpost ?

@tbuschto Running the device.jsx snippet and rotating the device works as expected. what aspect does not work for you?

@mpost I can still reproduce by running device.jsx on an Android device. This could happen at first try or after multiple tries. These are the screenshots from the latest Tabris.js app:

I can confirm your observation. We will look into it.

I've built an app uses the Tabris.js nightly and the value of orientation property is always unknown now.

Agreed. It works as expected in the emulator but not on a real device. We will look into it.


teams: create Android team

We have needed this for a while, especially as it relates to coordinating Android repo updates.
Create it with the people who expressed interest in: https://github.com/NixOS/nixpkgs/issues/359918
CC: @hadilq @adrian-gierakowski @RossComputerGuy @tadfisher

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/creating-the-android-team/57914/1

I think there's some process for making a GH org so we can get pinged. I'll look into it.

I can make the team but there's someone else with org perms who'll need to enable stuff.

Thanks Tristan!

https://github.com/orgs/NixOS/teams/android been created

Are there any plans to make the Android team a codeowner of things? If so, we'll need someone to come in and change stuff.

Updated to add @NixOS/android to teams.nix, and reflect who's in it.

@RossComputerGuy At some point, probably. I don't think we need to do so preemptively until we get an automated update/test pipeline going for androidenv packages.

Cool, then everything is good to go.

Anything else or good to merge?

Should be good. If anyone else wants to be added, we'll do that in subsequent PRs.


fix: improve generic quote evasion detection

do not detect single quotes as evasion
detect multiple single or double quotes as evasion attempt
detect quoted content like kill '-'9

Fixes #3397

Before:
Freq.   ID #   Paranoia Level
    822 932240 PL2

After (this PR):
Freq.   ID #   Paranoia Level
    229 932240 PL2

Thanks. Still not perfect but much better at least.

Thanks. I get the feeling this is acceptable for a release, but let's ask the team.

I think this is a good solution to test for ' inside a word to detect evasions. I tested the new regex and it's better that the old one.

hello'world'xx -> match
hello'world' xx -> no match
pyth''on -> match
hello 'world'x -> match
hello 'world' -> no match

I've improved on the detection of things like kill '-'9 to fix the FPs reported by @xanadu.

@theseion Awesome! Can we chalk this one up as a victory? :man_dancing:

Freq.
ID #
Paranoia Level

11
932240
PL2

I sure hope so :) I just need an approving review from somebody.

If that's not a win, then I do not know!
It looked horrible in the beginning and now it's a homerun.

Thank you, good work!


external symbol _NLG_Dispatch2

hmm i tried 140 build tool same thing..
1>aaa.exp : error LNK2001: unresolved external symbol _CxxFrameHandler4
1>aaa.exp : error LNK2001: unresolved external symbol _NLG_Dispatch2
1>aaa.exp : error LNK2001: unresolved external symbol _NLG_Return2

The vcruntime.cpp file (with implementation) not included in the compilation or a non-x64 assembly is being linked.


[BUG] - Modal not re-rendering childrens

Discussed in https://github.com/nextui-org/nextui/discussions/503

Originally posted by Ampex May 31, 2022
Hello, I found an issue, that Modal on opened state is not re-rendering { children } if they changed. My idea is to dynamically change a content of Modal body with context API, but now it is impossible... what do you think ?

Was a my mistake :)


Re-vamp with the new version of .NET Aspire

Add more stuff like https://github.com/fvandillen/futuretech-dapr-aspire
Adding more APIs:

https://sampleapis.com/api-list/coffee
https://www.infoq.com/articles/webber-rest-workflow/

https://editor.swagger.io/?url=https://jargon.sh/user/jargon/Cafe/v/working/artefacts/openapi/render.json

Close it, because we're addressing it in #2


SpringBoot support

Hello @rkonovalov . I found your project and it was just what i needed for the project that i'm currently working on. Unfortunately you project does not work with spring boot, so i took a look at the source code, and the changes that need to be done to make it compatible are fairly minimal. The only issue i have now is that i have some tests that are failing and i can't seem to figure them out. Are you planning on supporting spring boot yourself or should i make a pull request and we can figure out the tests together?

Hello @ZoidbergVA
Thank you for interesting. I have tested JFilter module with Spring Boot 2.1.0.RELEASE version.
Maybe your issue is related with different version or other dependencies in your project.
Anyway, I have added jfilter-spring-boot repository for test. Link: https://github.com/rkonovalov/jfilter-spring-boot
Could you please run tests in MainControllerTest. It should be passed without errors.

Hello @rkonovalov. I tried your test repository and indeed the tests seem ok. Perhaps you are correct that the issue is a combination of dependencies. I will list here all the issues that i had in my project, and i will also try to make a repository to show them, but that will take a few days.
I'm using spring boot version 2.0.0.RELEASE.
Issues:

Cannot subclass final class com.jfilter.components.FileWatcher - this was solved by simply making the class not final.

Is not really an issue with your project, more like weird spring boot behavior, but because of the @ComponentScan annotation, my path get's messed up and other dependencies do not get picked up / initialized correctly. - this was solved by putting @ComponentScan on EnableJsonFilter.

This is 2 issues caused by the same thing. Firstly @JsonFormat was ignored and i had an issue with LocalDate, LocalTime. LocalDateTime being serialized as objects instead of strings. - this was solved by autowiring the ObjectMapper in FilterJsonConverter / FilterXmlConverter and then in ConverterMapper in setIgnoreModifier instead of overwriting the SerializerFactory i got the SerializerFactory allready on the mapper and just added the SerializerModifier.

I'm using springfox for an autogenerated swagger documentation and FilterRegister was ( i think ) overwriting some configurations. - this was solved by simply removing FilterRegister.

I will leave another reply in a few days with a sample repository as well, to show all these issues.

Hello @ZoidbergVA
Thank you for details. Bellow comments regarding your issues:

I have removed final keyword and created Case #4

After some tests I have found that you can try to use ComponentScans annotation instead of ComponentScan. As example

Instead of using:
@ComponentScan({"com.jfilter.components", "com.your-project...."})
You can use:
@ComponentScans(value = {@ComponentScan({"com.jfilter", "com.jfilter.components"}),
@ComponentScan({"com.your-project...."})})
It could help to eliminate issue with weird spring boot behavior. But, I have decided on your advice to move ComponentScan anntation in EnableJsonFilter annotation and created Case #5

I need more time to fix this issue. Maybe you can make repository with this issue.

FilterRegister using for adding FilterJsonConverter and FilterXmlConverter in Spring Converters list.
Without this component JFilter will be useless. And if this issue is critical for you, I also need repository with issue, for find a way how we can fix it.

Hello @rkonovalov.
I'm glad you were able to solve issues 1 and 2.
Regarding issues 3 and 4, i'm going to try and make a repository tomorrow, but if you have any questions i can answer no problem.
For case 3, you create a new ObjectMapper. I'm not sure in a non - spring-boot environment how this affects things, but in a spring-boot environment there already is a ObjectMapper bean created. I think it's best if you just autowire it and just return the autowired one. This way, in a spring-boot environment the one that is automatically created will be used, and in a non - spring-boot environment you just have to declare a bean of that type. This adds a bit of extra work for people who use jfilter but it's not really a big deal.
For case 4, i just added a Component annotation on FilterJsonConverter and FilterXmlConverter and they were picked up.

Hello @AdrianViman
I have added changes based on your suggestion for case 3.
For case 4 I need more details. I tried to add Component annotation but it didn't work for me.
I think it's not enough to just add Component annotation. Also need register FilterJsonConverter and FilterXmlConverter in HttpMessageConverter list what actually does FilterRegister class.

Hello @rkonovalov .
Sorry to be back so late. I created a repo (it's kind off messy, sorry about that) to show my issues. https://github.com/ZoidbergVA/json-ignore-issues
Issues 1 and 2 are solved with the latest changes from jfilter, issue 3 and are not.
To see issue 3 in action, build the repo, start the app and in a browser open localhost:8080/api/users/1 and you will see that the LocalDateTime is serialized as an object.
To see issue 4 in action, build the repo, start the app and in a browser open http://localhost:8080/api/swagger-ui.html
and you will see basic login page and in logs this line:
WARN 5888 --- [nio-8080-exec-1] o.s.w.s.PageNotFound : No mapping found for HTTP request with URI [/api/swagger-ui.html] in DispatcherServlet with name 'dispatcherServlet'
but if you remove EnableJsonFilter from AppConfig you will see that the page loads correctly.
If you have any questions, please let me know, i'm here and wanting to help. Thanks for the support.

Hello @ZoidbergVA
I have released 1.0.10 version with fixed issues 3, 4 (I Hope :))
Also for fixing case 3 with incorrect date format please add next annotation: @JsonSerialize(using = LocalDateTimeSerializer.class)
As in example:
...
@JsonFormat(pattern = "yyyy-MM-dd HH:mm")
@JsonSerialize(using = LocalDateTimeSerializer.class)
@ApiModelProperty
private LocalDateTime localDateTime;
...
I haven't found a better solution yet.

Hello @rkonovalov.
I can confirm that case 3 and 4 are working now. For case 3, indeed i need to add the JsonSerialize, and while not ideal, it's not really a big problem.
You can close the issue. Thanks for the support.

Hello @ZoidbergVA
I have released 1.0.13 version which should fix issue with LocalDateTime without using @JsonSerialize(using = LocalDateTimeSerializer.class) annotation


Create journal_abbreviations_jcr.csv

Hi, is there a reason why you close your PR again?

Hi, it is because "PR run (Tests workflow run) failed", and I thought it did not meet your requirements. I am sorry if undesirable consequences flow from my misoperation.

No, sorry for the misunderstanding. The workflow is failing because of the markdown in the readme but this is not your fault! (It was broken before)
The CSV is not checked, as long as you stick to the format described in the readme it should be fine

Got it. The CSV file does conform to your format. Should I make a pull request again?


Firefox U2F not working

If I set security.webauth.u2f = true, myetherwallet.com doesn't fully load.
Here's the console error:
TypeError: setting getter-only property "u2f"
Error: [$injector:modulerr] Failed to instantiate module mewApp due to:
[$injector:nomod] Module 'mewApp' is not available! You either misspelled the module name or forgot to load it. If registering a module ensure that you specify the dependencies as the second argument.
http://errors.angularjs.org/1.6.5/$injector/nomod?p0=mewApp
And here is a screenshot: https://www.dropbox.com/s/tokli4k2ohlp2sy/Screenshot 2017-12-12 18.37.23.jpg?dl=0
I'm using FirefoxNightly 59.0a1 (2017-12-12) (64-bit).

#1282


UNotifications index top away can't click element under, how to fix?

Description

Same issue. Custom position configuration doesn't work.
export default defineAppConfig({
  ui: {
    notifications: {
      // Show toasts at the top right of the screen
      position: 'top-0 right-0'
    }
  }
})

Removing top-0 solved the issue for me...

Placing the Notifications as the documentation suggests is still not working.
I think the issues is tailwing-merge is not merging the default bottom-0 end-0 with top-0 right-0 will result in all 4 classes present in class, and thus adding a transparent overlay to the right side of the screen.
tailwind-merge not merging top-0 with bottom-0 makes sense.
Probably something like we have for button variants needs to be implemented for positioning.

I think I find a workaround:
    notifications: {
      // Show toasts at the top right of the screen
      position: 'top-10 end-0 bottom-auto'
    }

bottom-auto cancels out the bottom-0


What is the difference between API version kafka.jikkou.io/v1beta2 and 1 ?

What is the difference between API version kafka.jikkou.io/v1beta2 and 1?
The documentation has examples with each and it is not clear what should be used.

Hi @njlr, Jikkou API is currently under heavy refactoring (btw, I hope to release a new version in a couple of week that may include some breaking changes).
There are errors in the documentation that I will fix but you should use the kafka.jikkou.io/v1beta2.

Thanks @fhussonnois that clears things up!


About the user's choice of charger type

Please recommend a guide phase where the user selects a fast charger and a slow charger.

The charge cost of a slow charger is lower than that of a fast charger
In the case of slow - about six hours until the full discharge in full charge
fast charging by filling 80 percent of total discharge takes 15 to 30 minutes


Software calculated battery percent is inaccurate

The software calculated battery remaining is a simple 1:1 slope degradation where the actual curve has a cliff around 3.6v. A potential solution is a LUT (loosely documented in /supporting material that correctly models the curve.

LUT table implemented


input shape of anti_spoofing.h5

If I want to use anti_spoofing.h5 separately, what image dimension should I pass to the model?
The model shows the following summary:
Model: "sequential"
_________________________________________________________________
 Layer (type)                Output Shape              Param #
=================================================================
 conv2d (Conv2D)             (None, 112, 112, 18)      504

 activation (Activation)     (None, 112, 112, 18)      0

 batch_normalization (BatchN  (None, 112, 112, 18)     72
 ormalization)

 conv2d_1 (Conv2D)           (None, 112, 112, 18)      2934

 activation_1 (Activation)   (None, 112, 112, 18)      0

 max_pooling2d (MaxPooling2D  (None, 56, 56, 18)       0
 )

 batch_normalization_1 (Batc  (None, 56, 56, 18)       72
 hNormalization)

 conv2d_2 (Conv2D)           (None, 56, 56, 36)        5868

 activation_2 (Activation)   (None, 56, 56, 36)        0

 batch_normalization_2 (Batc  (None, 56, 56, 36)       144
 hNormalization)

 conv2d_3 (Conv2D)           (None, 56, 56, 36)        11700

 activation_3 (Activation)   (None, 56, 56, 36)        0

 max_pooling2d_1 (MaxPooling  (None, 28, 28, 36)       0
 2D)

 batch_normalization_3 (Batc  (None, 28, 28, 36)       144
 hNormalization)

 flatten (Flatten)           (None, 28224)             0

 dense (Dense)               (None,<PHONE_NUMBER>

 activation_4 (Activation)   (None, 128)               0

 dropout (Dropout)           (None, 128)               0

 dense_1 (Dense)             (None, 2)                 258

 activation_5 (Activation)   (None, 2)                 0

=================================================================
Total params: 3,634,496
Trainable params: 3,634,280
Non-trainable params: 216
_________________________________________________________________

What is the input shape?

Okay, found it, it's (None, 112, 112, 3)


INDY-339: Fixed test_getting_started_guide.py

Fixed broken printing of claim contents.
Corrected printing of proof requests.

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


Error with Bootssistexplorer and lunarcrush assistant

Hello again!
After not using this bot helper for a long time, I'm surprised that I can't use the Lunarcrush and 3Ctools assistant.
I refreshed with a new lunarcrush api (legacy.lunarcrush) and the 3c api just in case, but this didn't help
Here the errors:
botassistexplorer.py (renamed to ba.py)
.ini :

console error :

altrank.py (renamed to ar.py)
.ini :

console error :

What I can be doing wrong?

3c-tools website is not reachable because you appear to use a proxy server in between, check and fix connectivity to www.3c-tools.com (using webbrowser or something else first.
Galaxyscore isn't working because it looks at the filename to decide what to do (galaxyscore or altrank) they share the same code. So you need to rename it back to galaxyscore.py, you can add something behind it if you want multipe scripts though for example galaxyscore_text.py

3c-tools website is not reachable because you appear to use a proxy server in between, check and fix connectivity to www.3c-tools.com (using webbrowser or something else first.
Galaxyscore isn't working because it looks at the filename to decide what to do (galaxyscore or altrank) they share the same code. So you need to rename it back to galaxyscore.py, you can add something behind it if you want multipe scripts though for example galaxyscore_text.py

Thanks, I solved the Lunarcrush problem but not the 3ctools problem since I use pythonanywhere and for sure the site is blocking the website 😕

You can open a ticket for support at pythonanywhere to open www.3c-tools.com for you, or let them check your permissions, I once had a false network profile which prevented me from connecting to other sites too.

proxy server

You can open a ticket for support at pythonanywhere to open www.3c-tools.com for you, or let them check your permissions, I once had a false network profile which prevented me from connecting to other sites too.

You can open a ticket for support at pythonanywhere to open www.3c-tools.com for you, or let them check your permissions, I once had a false network profile which prevented me from connecting to other sites too.

Thanks I'm on it


detectBrowserLocale is not working

Environment

Operating System: Windows_NT
Node Version:     v16.20.0
Nuxt Version:     3.5.2
Nitro Version:    2.4.1
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      -
Runtime Modules:  -
Build Modules:    -

Reproduction
https://stackblitz.com/edit/nuxt-starter-ee43ob?file=package.json,nuxt.config.ts,i18n.config.ts,pages%2Findex.vue,mod%2Fprefix.ts,package-lock.json
Describe the bug
It seems that i18n_redirected isn't set
Additional context
No response
Logs
No response

cc @ineshbose

wrong repo ...


bug: create statement with record id errors

Example

True for both statements:
CREATE chat CONTENT { customer: "guest:ht28nge5czf39lrs39ox" };
CREATE chat SET customer = "guest:ht28nge5czf39lrs39ox"

Same for
CREATE c SET A = 42;

but not for
CREATE c:ex SET A = 42;

@mathe42 can you out down the exact queries/function calls that you used to get to that error after a fresh browser page load?

I used the node build in my vscode extension. It only runs a USE NS default DB default before that.

Ok cool. Where you calling db.query() yeah?

Yes.

Just USE NS default DB default;\nSQL_FROM_ABOVE\n

Acutaly no I call db.sql.

Hey @mathe42 I can't replicate this...

Will have a deeper look next week.

I'm also running into this with a simple test setup. I'll see if I can put together a minimal repro repo.

@mathe42 and @scotttrinh Kindly try the latest main branch and let me know if you still run into this. I cannot reproduce it myself.

This is fixed by #17! Thanks for reporting!


一部の加算ダイスでシークレットにならない問題を修正

加算ダイスの先頭がダイスロールリテラルでない時に加算ダイスと認識しない問題があった

Codecov Report

Merging #230 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #230   +/-   ##
=======================================
  Coverage   87.67%   87.67%
=======================================
  Files         218      218
  Lines       22458    22459    +1
=======================================
+ Hits        19690    19691    +1
  Misses       2768     2768

Impacted Files
Coverage Δ

src/bcdiceCore.rb
81.60% <100.00%> (+0.02%)
:arrow_up:

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 cf8ead5...aada47c. Read the comment docs.


Remove gulp-uglify

webpack v4+ will minify your code by default in production mode.

Is there way to get webpack not to neaten up my js until it's in production mode/build?

@Nsokyi  Inside of the frasco.config.js you can set the tasks for webpack to false.
  tasks: {
    browsersync: true,
    eslint:      true,
    imagemin:    true,
    sass:        true,
    watch:       true,
    webpack:     false,
  },

Thanks, I tried that and oddly enough it doesn't have any effect?

@Nsokyi If you set webpack to false, webpack will not do anything, so you have to directly edit your js files in the assets/js folder. **Note: if you reset webpack to true, your js files in assets/js will be overwritten by files with the same name in assets/_js.
I'm sorry that for now, there's no way to switch minification between development and production.

OK, thanks for getting back to me.

@Nsokyi On the latest release, I removed gulp-uglify because Webpack automaticlly compress js files. For now, you can add a Webpack configuration devtool: 'source-map' to frasco.config.js to enable source maps. This might make your debug easier.
  webpack: {
    mode:   'production',
    devtool: 'source-map',
    module: {
      rules: [],
    },
  },


Made code compatible with Visual Studio 2015:

changed CTTI_PRETTY_FUNCTION_BEGIN/END to _LEFT, _RIGHT specifying how much to trim (old code didn't work due to a different macro evaluation (?))
removed string::substr(), added trim_* functions
string::c_str() isn't null-terminated properly anymore!

Todo: Ugly warning C4307 due to overflow in sid_hash(), don't know how to get rid of it.
Your bug was indeed caused by substr(), it returned a pointer to a local array. ;)
It is fixed now at the cost that c_str() isn't properly null-terminated.

I have been working this morning on a similar approach, using begin and end member pointers. But as your PR, leaves the substring without null end. This is a problem since the type name is not correct. Of course works for hashing.
One solution is to encourage users to use ctti::detail::string always as the default compile time string type, and never expose the pointer but a runtime to std::string conversion for the case anyone wants to do something else from printing to cout or alike.
Another solution is to use a fixed size array for ctti::detail::string internals, at the cost of constexpr copying of strings always, instead of pointer passing. I have checked some disassemblies yesterday, both GCC and Clang didn't generated a .bss entry, but chunks of mov $character al; mov al i(dest) due to variadic pack expansion in substr().

Regarding the null termination issue:
I think encouraging ctti::detail::string is a good way. If the user calls c_str() with out checking length, it doesn't harm - there is just the ugly prefix at the end. This is also the way Boost.TypeIndex does it:
Storing an array is an alternative, but I've heard about issues with MSVC.
If constexpr would support static local variables, your substr() approach would work, instead of a string class, there is just a templated type name function, where the appropriate area of the function signature is copied. But sadly, they don't. It could be offered as an alternative, though.

What VS version were you using? I have had to change sid_hash() from return n > 0 ?... to return *str > 0 ?... to get it evaluated at compile time. May be a compiler bug, I'm still with the 2015 RTM. I changed to n > 0 to avoid overflow warnings from GCC, exactly the same MSVC gives to me after the "fix".

The normal Community version, no RTM. The overflow warnings I'm getting are due to the multiplication overflow in the function later.

Maybe it's time for a #pragma push disable warning...

Doesn't work. Warning occurs in the code that leads to the hash function, i.e. the ctti::type_id call!

Let's ignore it for now then. I'm more worried about the null terminator.


Changes to Redirects aren't working well

Here's what I did:

On zw.goodinternet.org, I created a redirect from /en/sandbox/your-health/ pointing to a page which was located at /en/sandbox/your-health/
I changed the stub so that the page was located at /en/sandbox/your-health2
The redirect successfully pointed /en/sandbox/your-health2/
I then changed the page stub to /en/sandbox/your-health/
The redirect continues to point to /en/sandbox/your-health2/
Even after deleting the redirect, /en/sandbox/your-health/ redirects to /en/sandbox/your-health2/

I tried changing the redirect to point to another page, tried changing the Page stub to multiple other settings, the Redirect always seems to point to /en/sandbox/your-health2/
Desired behavior:
Changes to a Redirect should be successful.
If they are very delayed (eg 30 mins?) that should be clear to the Admin. If this delay is the case, right now it's difficult to understand because new Redirects are implemented immediately.

@ChrisMarsh82 Would like to discuss this in the afternoon call

We identified that this is almost certainly due to browser caching 301 permanent redirects.
The general consensus seems to be that for most of our common purposes 302 redirects are actually better. Our most common use cases involve promotions, testing, shortening for QR codes, etc, rather than permanently relocating a resource on the website. We do sometimes permanently relocate resources on the website, ie moving Pages, but the right way to handle that would be to offer the user an option to create a 301 redirect when the move the Pages.
Maybe the biggest issue is that Admins are shown that they have the ability to change or delete a permanent redirect, but this won't work for users that already accessed the link.
Let's make the default for redirects to NOT be permanent.
Can we please add text below the permanent check box:
"Permanent redirects are normally stored by the user's browser until they delete their browser cache. If you change or even delete a permanent redirect after you create it, the users who already accessed it will still continue to behave as if the original redirect is in place. Your browser may also store the permanent redirect until you clear its cache."


Remove qctrl integration tests

Summary
These tests haven't been passing for over 4 months and we're removing support soon
Details and comments
Fixes #

Can we try to fix them and keep them until we stop to support?

Going to remove all of q-ctrl code in a separate PR


zig errors on variables that need to be treated as const instead of var

akhil@ad:~/practice/immich$ zig build
/home/akhil/practice/immich/zap/build.zig:217:9: error: local variable is never mutated
    var announceybot_exe = b.addExecutable(.{
        ^~~~~~~~~~~~~~~~
/home/akhil/practice/immich/zap/build.zig:217:9: note: consider using 'const'
/home/akhil/practice/immich/zap/build.zig:203:9: error: local variable is never mutated
    var pkghash_exe = b.addExecutable(.{
        ^~~~~~~~~~~
/home/akhil/practice/immich/zap/build.zig:203:9: note: consider using 'const'
/home/akhil/practice/immich/zap/build.zig:15:9: error: local variable is never mutated
    var zap_module = b.createModule(.{
        ^~~~~~~~~~
/home/akhil/practice/immich/zap/build.zig:15:9: note: consider using 'const'

Very likely a zig version mismatch, you're probably using a nightly master build of zig instead of the required released version of zig.
See the readme.
There is a point in using the release version of zig: zap projects don't need to update their code and dependencies as zig master keeps evolving.
Released zig is not an old version of zig that you should avoid, it's the release version. Zig master is not even alpha.


Update rustc to nightly-2022-05-15

Update rustc to nightly-2022-05-15.
@vakaras could you take care of this?

bors r+

bors r+


Change the posix logging implementation

This commit changes the posix logging implementation to write the entire log line in a single fprintf() command. This change helps keep the open thread logs separate from other wpantund logs (syslog)
in simulator (posix) run.

LGTM 👍


New package: SaleaeInc.Logic version 2.4.14

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.6 schema?

Resolve #143041

For moderators
This account is automated by Github Actions and the source code was created by CoolPlayLin. If you have any questions about any pull request, don't hesitate to ping Exorcism0666.

[!warning]
With the new version of Komac, numerous Pull Requests may contain bugs or unwanted elements. If such issues arise, please tag russellbanks so that these bugs can be addressed in future updates.

[!important]
Please carefully review these Pull Request before merging. If it is a Pull Request for removing incorrect content and the URLs are issue, free upon manual checking, please close this Pull Request directly. (It is best to inform Exorcism0666 of the closure.)

良い一日をお過ごしください！

Automatic Validation ended with:
(Automated response - build 869.)

Manual Validation ended with:

Is there an installer switch to have the package install silently?

Manual Validation ended with:
Is there an installer switch to have the package install silently?

I just can't find them 😭 .

The package uses Advanced Installer

Saleae FAQ: Is Silent Installation Supported?
Advanced Installer: How do I create a silent installation MSI package?
 Even though the title says MSI, it is still true for the EXE)

Using these flags however does not resolve the problem of Windows needing user confirmation to allow installation of the included drivers.

[Policy] Blocking-Issue

Once the install can complete without user interactivity the package may be added to the community repository.


Publish test reports in GHA workflow

The test reports of the GHA workflow should be published to debug the tests if they fail.
The following action could be helpful:
https://github.com/marketplace/actions/test-reporter

@brioux, Could you please take a look at this CI run? The error says that the open function requires atleast 1 parameter, and none were supplied. I took a look at the type definitions for the open function, and in the node_modules/ws-wallet/dist/src/index, the open function has the following function signature:
open(sessionId: string, endpoint?: string): Promise<WsOpenResp>;

However, the function signature given in the secure-identities directory is https://github.com/hyperledger-labs/blockchain-carbon-accounting/blob/b5c662f06e4acff396ba20178c6d4063160cdce0/secure-identities/ws-wallet/src/wallet.ts#L118
This CI run on the main repo passed successfully since it took the latter function definition. However, since the package ws-wallet is imported at the top of the file, and not ../../../../secure-identities/ws-wallet/src, it should have taken the function definition in node_modules. It seemed to have passed anyway, whereas my CI run is failing because of taking the function definition in node_modules (which is what is to be expected since we are not importing it from the secure-identities directory).
Could you please take a look and let me know what is to be done?


Xamarin Forms UWP Null Reference Exception On Navigation

When you navigate from a page playing an animation and go back to that page a null reference exception is thrown.
Com.Airbnb.Xamarin.Forms.Lottie: 2.5.1
LottieUWP: 2.5.1
System.NullReferenceException: Object reference not set to an instance of an object.
at LottieUWP.LottieDrawable.CanvasControlOnDraw(ICanvasAnimatedControl canvasControl, CanvasAnimatedDrawEventArgs args)

Reproduction project: https://drive.google.com/file/d/184cMHR9icG2NfUvPIwnsKrRDaE6M-D8P/view?usp=sharing

Fixed by c84843ae0b3b67ed1d9e546b0b2141e932bd66cd.


Add swastik pattern in python, pattern 36

Description
Description
Task
For the given input 7 the following pattern should print:
Pattern Image

Filename
Pattern36.py

Can you assign this to me?

ok you can work on this issue

pr merged #56


Added Happy number checker

This PR solves the issue Happy Number Checker #601.
Kindly review my PR

DM me on discord, need to discuss something related to your PRs


Node tasks should honour workDir property

Setting the workDir property on the node object doesn't also set the workingDir property on the default node tasks (such as npmInstall). This results in npm looking for my package.json in the directory of the project I'm running from rather than in my working directory. I've had to work around it by making a task npmInstallCustom as follows:
task npmInstallCustom(type: NpmTask) {
    execOverrides {
        it.workingDir = nodeBuildDir
    }

    setArgs( ['install'] )

    inputs.file { file("$nodeBuildDir/package.json") }
    outputs.dir { file("$nodeBuildDir/node_modules") }
}

Additionally the inputs file defined for the NpmInstallTask is a (possibly non-exstient) package.json in the project directory. Line 18 of NpmInstallTask.groovy reads:
getInputs().file( new File( this.project.getProjectDir(), 'package.json' ) )

It'd be good if the NpmInstallTask (and perhaps all NodeTasks or all NpmTasks?) defaulted its/their exec working directory and inputs directory to the working directory set on the node object.
(I've raised a similar issue with the grunt plugin.)

Thanks for the tip. Will look into this (in both projects).

I've submitted a PR that should address this https://github.com/srs/gradle-node-plugin/pull/52

This is now merged into master. Closing this issue.


Integration with ipfs-share-files

Part of UX improvements for sharing local files (#342)

Shipping ipfs-share-files with Companion did not happen for two reasons:

We had to stop bundling Web UI due to reproducibility issues, and for the same reason Share app won’t be bundled any time soon.
ipfs-share-files itself requires some improvements around content discovery speed (at infra, websockets, webrtc), which did not land yet

I think we will switch approach and instead of opening uploaded resource at the gateway, display "success screen" with direct link to the gateway and “get share link” option that copies link to share.ipfs.io.  This would also address #423.

No longer relefant: after successful import we open resource in Web UI and link is automatically copied to clipboard, removing the need for ipfs-companion-specific UI.
If ipfs-share-files gets picked up again, we could use its URLs for shareable links, but for now this can be closed.


Paper Wallets

The width of the paper wallet images should equal the page width and it would look a ton better. =)

They print perfectly now. Closing.


Document test Kubernetes cluster creation

Follow–up: https://github.com/iterative/terraform-provider-iterative/pull/237#discussion_r755000380 & https://github.com/iterative/terraform-provider-iterative/pull/189#issuecomment-916911104
https://github.com/iterative/terraform-provider-iterative/blob/813894ef029930bb9b10d725f2f6b13f27ece4da/.github/workflows/smoke.yml#L22-L49
https://github.com/iterative/terraform-provider-iterative/blob/813894ef029930bb9b10d725f2f6b13f27ece4da/.github/workflows/smoke.yml#L98-L107
Setting up Azure Kubernetes Service (Notion)

Installing the Azure command-line interface tool
First of all, we need to install the az tool following the official documentation on the Microsoft documentation portal: https://docs.microsoft.com/en-us/cli/azure/install-azure-cli
Enabling experimental features
In order to automatically provision GPU nodes for our cluster, we'll also need to enable the following experimental features through the aks-preview extension:
az extension add \
  --name aks-preview

az provider register \
  --namespace Microsoft.ContainerService

az feature register \
  --namespace Microsoft.ContainerService \
  --name GPUDedicatedVHDPreview

Creating a test cluster
The following commands will create an AKS cluster with a single node, keeping everything into a a new resource group for easier deletion of the resources:
az group create \
  --name testKubernetesResourceGroup \
  --location eastus

az aks create \
  --resource-group testKubernetesResourceGroup \
  --name testKubernetesCluster \
  --node-vm-size Standard_NC6 \
  --node-count 1 \
  --aks-custom-headers UseGPUDedicatedVHD=true

Click to reveal a budget-friendly cluster configuration without GPU...
az aks create \
  --resource-group testKubernetesResourceGroup \
  --name testKubernetesCluster \
  --node-vm-size Standard_A2_v2 \
  --node-count 1

Retrieving the credentials
Azure has some wrappers for Kubernetes authentication and will generate for us the required credentials. The following command will produce a full-fledged kubeconfig string that can be directly stored in the KUBERNETES_CONFIGURATION secret of your continous integration system of choice:
az aks get-credentials \
  --resource-group testKubernetesResourceGroup \
  --name testKubernetesCluster \
  --file -

💡 If you skip the --file option, settings will be saved to your computer's ~/.kube/config file, that will be automatically used by kubectl if you ever need to run any sanity check manually on the cluster.

Deleting the test cluster
Once you've finished testing you can run the following command to delete the entire resource group, which includes the cluster and all its nodes:
az group delete \
  --name testKubernetesResourceGroup

⚠ Please delete the entire resource group as soon as you finish using the cluster. They are really pricey, and you may end spending a lot of money just on testing. 🔥  💵

Kubernetes users should already have a working cluster and, to my mind, documenting the cluster configuration process seems a bit out of scope.
I wrote the documentation above just for testing this provider on AKS, but external contributors might prefer to use EKS, GKE or on–premises solutions.

@casperdcl, where (if any) would you like to publish this information?

I'd suggest a new file under https://github.com/iterative/terraform-provider-iterative/tree/master/docs/guides.


After the update of craft 2.6.2974

Right after my update i wanted to create a new field supertable, but when i click on the new block button nothing happen and for the already existing fields 'supertable' i can't modify or view or do anything is this just me or something happened when i updated craft ?

Have you tried clearing Craft's caches? Are there any JS errors in the console for your browser?

i did clear the cache and this is what i ve got in my console i don't think is related though, still i have an earlier version of the site on a diffrent domaine the supertable work fine it's just after this update i had the probleme so it's just me not a bug or something?

i did clear the cache and this is what i ve got in my console i don't think is related though, still i have an earlier version of the site on a diffrent domaine the supertable work fine it's just after this update i had the probleme so it's just me not a bug or something?

If you have any Javascript errors in the control panel, it may very well cause issues with other plugins. Are you able to disable the SEO plugin and try again?

yep it worked thank you so i have to correct the probleme of this seo things ... still thank you very much !