tagsAll being marked as secret stops use of requireOutputValue in other stacks to access infrastructure objects

What happened?
A common pattern in my use of pulumi is to put the object referencing a piece of infrastructure into the output, then pick that up using requireOutputValue in another stack in order to be able to use the various attributes.
In #2791 to fix #2778 a change was implemented to make the output of tagsAll as secret, as it was possible a tag was marked as secret but this would not be carried through to the tagsAll output.
I would argue that this change gives a false sense of security and it would be better, if a change is needed to stop this behaviour, to disallow the setting of tags to a value marked as secret. As remarked in #2778:

The behavior is a bit pedantic since in real world use cases tags are fairly exposed in the cloud, can't think of an easy use case where they should be secret, but it is certainly an odd corner case.

which I would agree with, and for instance in https://docs.aws.amazon.com/secretsmanager/latest/userguide/managing-secrets_tagging.html AWS explicitly say:

Never store sensitive information for a secret in a tag.

In AWS at least tags are exposed all over the place as read only, including to all sorts of roles like billing that should not have access to secrets.
Is there any appetite for having this change marking tagsAll as secret reverted?
And having values passed into tags that are marked as secret instead causing an error and the tag being unable to be set?
If this change can not be reverted I would appreciate some feedback on best practices for passing information about infrastructure between stacks, as I see it there are a few possible ways:

Switch to using requireOutput and wrap everything in an apply
Keep each parameter needed to be passed to another stack in its own output
Pass an id or ARN that can be used with the normal lookup methods to look up the infrastructure object in the other stack

Is there a recommended best practise from these or something else I haven't thought of?
Example
A simple contrived example of how I pass an object to another stack using an output.
Certificates Stack:
  const websiteAlbCertificate = new aws.acm.Certificate(
    "website-alb",
    {
      domainName: "alb.example.com",
      tags: {project: "website",},
      validationMethod: "DNS",
    },
  );

return {
  websiteAlbCertificate: websiteAlbCertificate,
};

Certificate Validations Stack:
const certificatesStack = new pulumi.StackReference("certificates");
const websiteAlbCertificate = await certificatesStack.requireOutputValue("websiteAlbCertificate");

Output of pulumi about
on7000nn902:certificates mwalke25$ pulumi about
CLI
Version      3.105.0
Go Version   go1.21.6
Go Compiler  gc
Plugins
NAME    VERSION
nodejs  unknown
Host
OS       darwin
Version  13.6
Arch     arm64
This project is written in nodejs: executable='/opt/homebrew/opt/node@18/bin/node' version='v18.17.1'
Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

Hi @mwalker-g2u. I'm going to close this as a duplicate of https://github.com/pulumi/pulumi-aws/issues/3265. As a workaround for your specific use case, you can un-secret your return value. I'm not an inspired TypeScript developer, but I have tested this and it works:
import * as pulumi from '@pulumi/pulumi';
import * as aws from '@pulumi/aws';

const b = new aws.s3.BucketV2("bucket");

function sanatize_exports(exports: {[index: string]: pulumi.Resource}): {[index: string]: pulumi.Output<pulumi.Resource>} {
    const result: {[index: string]: pulumi.Output<pulumi.Resource>} = {};
    for (const k in exports) {
        result[k] = pulumi.unsecret(pulumi.output(exports[k]).apply((value) => {
            if ("tagsAll" in value) {
                const tagsAll: any = (value as any)["tagsAll"];
                (value as any)["tagsAll"] = pulumi.unsecret(pulumi.output(tagsAll));
            }
            return value;
        }));
    }
    return result;
}

export const outputs = sanatize_exports({
    bucket: b,
});

Thanks @iwahbe I will try that out and I appreciate you taking the time to provide a workaround, I'm also not an inspired TypeScript developer.
Sorry I missed #3265 and I will add to the discussion there.


Live map should be better at keeping caches in memory

When the map is moved cgeo tends to drop caches necessitating reloading unnecessarly.
This makes "looking around" or "checking the next area" sluggish.

Please reopen this issue if the problem arises again. Due to changes in the meantime the situation should be much more better.

It isn't better, but we have another issue for it.

OK, thanks.
My idea was mostly that if you move back&forth over, say, two or three screens' worth, it should be possible to try and avoid reloading info just a bit more.


[FEATURE]Adding the Contact Us Page

Detailed description
I'll add the form, make it compatible with current UI and make it responsive
Add some graphics to it
Add the email address, location and phone number to reach out
@Deepu178 , the link was there in the footer, but the page is not created, could you assign it to me so that a dedicated page can be added for it
Context
When we click it on footer, we don't find it and it's necessary for contacting, reporting any issues
Possible implementation
No response
Additional information
No response
Code of Conduct

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

@Vishalk91-4  I have assigned the issue to  you and  you can start working on this issue. Good luck!

@Vishalk91-4 I have assigned the issue to you and you can start working on this issue. Good luck!

Hey @Deepu178  I can Work On This issue if He is not working on this issue assign to me can i do in 1 day :)

Hii @Vishalk91-4 if you are not working it. I would like to work on it.

I think similar issue exists #269


Having um in multiple platforms

Since this went into hacker news a few weeks back I've been watching the project. I think it would be beneficial to set a direction for the project, so as not to acquire any unwanted responsibilities and make sure the project stays lean and matching its purpose.
There are a few things going on at the same time, mostly trying to get um to be multiplatform, namely enabling it to run on Linux, Windows and FreeBSD. This has been somewhat conflated with making it "not rely on pandoc" (see #10, #6, #4 and #13).
I think this is a mistake, and perhaps you'll agree and we can restructure the issues and organize the efforts to increase um's availability without squandering man-hours. It's possible I'm mistaken - I welcome the learning opportunity.
um is written in Ruby, which can run on all the desired platforms. Except it depends on pandoc. But this is not really a hurdle since pandoc is distributed in all desired platforms.
The availability requests are actually packaging requests for the most part, not really requiring the project to change. um shouldn't have the burden of managing its dependencies. That's what package managers are for.
Packages are projects unto themselves, and need not be conflated with the main project. A good direction to multi-platform availability of um is #16.
I think if anyone wants to make this project available to other platforms, they should do so by packaging it, and packages are separate projects. These are examples of what could be done:

Windows (#4)? Build a choco package (as it was done for pandoc), or build an MSI or equivalent installer
macOS/Linux (#16) can be kept by maintaining the Homebrew package up to date, can be enhanced by building packages for the different distros.
FreeBSD can be managed by making a pkg package.

Just a note—as of 4.0.0, Pandoc is no longer a dependency. The Kramdown gem is instead. As @iajrz pointed out, this wasn't necessary in order to make um available on other platforms. But hopefully it makes it marginally easier given that Pandoc was a pretty heavy dependency.
Leaving the issue open in the hope that somebody who knows more about these other platforms will be able to do the packaging work required.

Also FYI, see UPGRADE.md for more information about the switch to Kramdown, which is a breaking change if you've made much use of Pandoc's Markdown extensions (which the default .md um page template does).

I agree with @iajrz that most of the issues about ports should not burden um as a project. Ideally for each distribution there should be someone packaging um according to how the distribution expects packages to be.
With the change to a Gem this has gotten tremendously easier! Since for most distributions there already exist other projects that are gems, thus each packager can take clues from those packages (or there even exist official guidelines how to package a Gem).
I have recently created a package for Arch Linux in the User Repository AUR: https://aur.archlinux.org/packages/um-git/
Since um is a Gem the package is a quite clean. Note that for the package kramdown is not installed by the Ruby Gem, but with the package manager of Arch Linux. This is one of the examples where the package deviates from the official version, because this is the way how to package on Arch. One should rely on the Arch package manager for dependency resolution.
If you want, you can add a link to the AUR package to the README for fellow Arch users. Maybe in the future more people will package um and you can add those to the README as well.

@He-Ro If you'd like to add info about how to install your package on Arch, feel free to make a PR changing the README.
Thanks for your hard work!

Going to leave this issue open for now in case anyone comes along feeling inspired to package um for more platforms. As @iajrz pointed out, um isn't yet available for Windows as a choco package or for FreeBSD as a pkg package.


MCUnet testing script (for accuracy of models in TF-Lite format) crashing - DefaultCPUAllocator: not enough memory

Hello there I am trying to test the accuracy of int8 models in TF-Lite format, in the mcunet subfolder, so after invoking the corresponding python script I get the following RuntimeError:
Traceback (most recent call last):
File "jobs/eval_tflite.py", line 97, in
images = torch.cat([v[0] for v in val_loader_cache], dim=0)
RuntimeError: [enforce fail at C:\cb\pytorch_1000000000000\work\c10\core\impl\alloc_cpu.cpp:81] data. DefaultCPUAllocator: not enough memory: you tried to allocate<PHONE_NUMBER>0 bytes.
I invoke the jobs/eval_tflite.py script with the following parameter options: python jobs/eval_tflite.py --val-dir dataset/imagenet/val --tflite_path assets/tflite/mcunet-320kb-1mb_imagenet.tflite --workers 4
##I have installed the tensorflow for cpu only edition , since my laptop/system has no GPU whatsoever. In the following snapshot you can see my system's specs

Has anyone else also experienced a similar error? Any help will be invaluable!
--Antonios.

Hi Antonios, it seems that the caching of the validation set exceeds the memory (the server I tested on has a much larger memory). You may want to skip building val_loader_cache and directly feed the data from the val_loader to the model for testing.

Close due to inactivity. Feel free to post issues under the new repo: https://github.com/mit-han-lab/mcunet.


pyodbc don't find and open exist driver

Environment

Python: 3.6.8 (pyenv enviroment)
pyodbc: 4.0.26
OS: Ubuntu 18.04.2 LTS
DB: MSSQL 2016 (remote connection)
driver: "ODBC Driver 17 for SQL Server"

Issue
pyodbc.Error: ('01000', "[01000] [unixODBC][Driver Manager]Can't open lib '/opt/microsoft/msodbcsql17/lib64/libmsodbcsql-17.3.so.1.1' : file not found (0) (SQLDriverConnect)")
I reinstalled the driver several times by

Instruction: https://docs.microsoft.com/en-us/sql/connect/odbc/linux-mac/installing-the-microsoft-odbc-driver-for-sql-server?view=sql-server-2017
Just install deb package: https://docs.microsoft.com/en-us/sql/connect/odbc/download-odbc-driver-for-sql-server?view=sql-server-2017

❯ odbcinst -j                                                                                [12:55:26]
unixODBC 2.3.7
DRIVERS............: /etc/odbcinst.ini
SYSTEM DATA SOURCES: /etc/odbc.ini
FILE DATA SOURCES..: /etc/ODBCDataSources
USER DATA SOURCES..: /home/varvara/.odbc.ini
SQLULEN Size.......: 8
SQLLEN Size........: 8
SQLSETPOSIROW Size.: 8
❯ odbcinst -q -d -n "ODBC Driver 17 for SQL Server"                                                                                [12:55:28]
[ODBC Driver 17 for SQL Server]
Description=Microsoft ODBC Driver 17 for SQL Server
Driver=/opt/microsoft/msodbcsql17/lib64/libmsodbcsql-17.3.so.1.1
UsageCount=1
I tried just open file
❯ /opt/microsoft/msodbcsql17/lib64/libmsodbcsql-17.3.so.1.1                                                                                [12:55:33]
[1]    18755 segmentation fault (core dumped)  /opt/microsoft/msodbcsql17/lib64/libmsodbcsql-17.3.so.1.1
Please help me understand the problem.
I investigated all existing error reports (StackOverflow, Microsoft drivers GitHub, askUbuntu, etc), re-checked docs pyodbc  - don't resolve problem
And method from
https://github.com/mkleehammer/pyodbc/issues/240
Not work fo me
Thanks! c:

Does isql work? Try this:
isql -v -k 'Driver=ODBC Driver 17 for SQL Server;server=<your server>;UID=<your user>;PWD=<your pass>'
If that does not work and fails with the same error, then strace it to find out which file it cannot open.
If that works, but pyODBC does not, then you will need to strace your Python script for the same purpose.

Also can you do ldd /opt/microsoft/msodbcsql17/lib64/libmsodbcsql-17.3.so.1.1 just to make sure nothing suspicious is going on there

@v-makouz
❯ ldd /opt/microsoft/msodbcsql17/lib64/libmsodbcsql-17.3.so.1.1      [11:46:18]
linux-vdso.so.1 (0x00007fff2b1bc000)
libdl.so.2 => /lib/x86_64-linux-gnu/libdl.so.2 (0x00007f8067eba000)
librt.so.1 => /lib/x86_64-linux-gnu/librt.so.1 (0x00007f8067cb2000)
libodbcinst.so.2 => /usr/lib/x86_64-linux-gnu/libodbcinst.so.2 (0x00007f8067a97000)
libcrypto.so.1.0.0 => /usr/lib/x86_64-linux-gnu/libcrypto.so.1.0.0 (0x00007f8067654000)
libkrb5.so.3 => /usr/lib/x86_64-linux-gnu/libkrb5.so.3 (0x00007f806737e000)
libgssapi_krb5.so.2 => /usr/lib/x86_64-linux-gnu/libgssapi_krb5.so.2 (0x00007f8067133000)
libssl.so.1.0.0 => /usr/lib/x86_64-linux-gnu/libssl.so.1.0.0 (0x00007f8066ecb000)
libuuid.so.1 => /lib/x86_64-linux-gnu/libuuid.so.1 (0x00007f8066cc4000)
libstdc++.so.6 => /usr/lib/x86_64-linux-gnu/libstdc++.so.6 (0x00007f806693b000)
libm.so.6 => /lib/x86_64-linux-gnu/libm.so.6 (0x00007f806659d000)
libgcc_s.so.1 => /lib/x86_64-linux-gnu/libgcc_s.so.1 (0x00007f8066385000)
libpthread.so.0 => /lib/x86_64-linux-gnu/libpthread.so.0 (0x00007f8066166000)
libc.so.6 => /lib/x86_64-linux-gnu/libc.so.6 (0x00007f8065d75000)
/lib64/ld-linux-x86-64.so.2 (0x00007f80684c3000)
libk5crypto.so.3 => /usr/lib/x86_64-linux-gnu/libk5crypto.so.3 (0x00007f8065b43000)
libcom_err.so.2 => /lib/x86_64-linux-gnu/libcom_err.so.2 (0x00007f806593f000)
libkrb5support.so.0 => /usr/lib/x86_64-linux-gnu/libkrb5support.so.0 (0x00007f8065734000)
libkeyutils.so.1 => /lib/x86_64-linux-gnu/libkeyutils.so.1 (0x00007f8065530000)
libresolv.so.2 => /lib/x86_64-linux-gnu/libresolv.so.2 (0x00007f8065315000)
✔ varvara ⮀ ~ ⮀
❯ ldd /opt/microsoft/msodbcsql17/lib64/libmsodbcsql-17.3.so.1.1 | grep 'not found'                                                              [16:36:07]
✘ varvara ⮀ ~ ⮀
❯
I apologize for not writing right away. Checked this option

@v-makouz
Your command is work.
https://github.com/mkleehammer/pyodbc/wiki/Module#connect
But this same credentials do not work for pyodbc.connect :c

@Varvara-Gornostaeva In the image you included in your last message, the username and password to your database are visible in the error message at the bottom of the image.  You should probably change that password.

Since isql works but Python doesn't, strace your Python process and find which of the files it cannot open, and why.

@keitherskine Yes, I noticed it late (very stupid). Changed creds and deleted messages

Sorry guys, I found the problem
I upgrade the python version from 3.6.3 to 3.6.8 -> interaction with the driver stopped workingApparently, this is a problem either in pyodbc (wait for the update) or in ubuntu or in pyenv.
Or the reason is in the chaos of dependencies
Thank you all for the answers!
I am closing the issue for now, as it is easier to roll back to the old version.

I just stumbled into this bug and spent some hours investigating what was happening. The reason was that we built on top of docker container python:3.7-slim, and that was updated recently to be based on debian buster instead of debian stretch. The binary msodbcsql17 driver requires libssl1.0.2 which is not available in debian buster out of the box.
For others hitting this bug (and using docker): The simple fix is to switch your dockerfile to be based on python:3.7-slim-stretch.


database/sql: add support for PutData and GetData

Is there any idea to add support for Putdata() and Getdata() in the SQL package. Users store large data in the database and they want to send them to the database by chunks, where Putdata() is used. In the same way Getdata() is used to get chunks of data from the database.
Thanks.

Which database or ORM framework uses these function calls?
You can probably do what you are hoping to with the existing framework - no reason Query or QueryRow can't return you tens or hundreds of megabytes of data.

I think every Database has this kind of function calls. As I work on Db2(IBM database), it has SQLPutData() and SQLGetData().
The above API's takes Statement handle as one of the parameters and variables in statement object (st,err := db.Prepare(query) ) are unexported and I can't use it even if I extend the sql.Stmt.

/cc @kardianos

Can you point me to the docs on what you are trying to do? Are you trying to upload large amounts or rows like a bulk row upload? Maybe an example that demonstrates how you would use this?

package main

import (
    "database/sql"
    "fmt"
    _"github.com/ibmdb/go_ibm_db"
)

func main() {
    con := "HOSTNAME=localhost;PORT=50000;DATABASE=dbname;UID=uname;PWD=password;"
    db, err := sql.Open("go_ibm_db", con)
    if err != nil {
        fmt.Println(err)
    }
	defer db.Close()
    st,err:=db.Prepare("Insert into putdata(a,b) values(?,?)")
	if err != nil{
	fmt.Println(err)
	}
	defer st.Close()

	// If user wants to send the data at execute time.
	// He will send the some empty string or empty byte array or something to notify the execute api
	// that data will be send later.
	st.Query("","")
	// From Here Its My vision of functions how to send the data.
	st.ParamData()
	//Here I am reading the normal data. He may read chunks of Data from file
	st.PutData("Akhil")
	st.PutData(" Ravuri")
	//This paramData function will set the next parameter to be inserted.
	//If there are no Parameter's the it will execute the query.
	st.ParamData()
	st.PutData("Sai")
	st.PutData(" Ram")
	st.ParamData()
}

In the example I have taken the string, suppose if the user wants to store lob data  in the database which is of 2GB (As of I know a buffer can't handle 2GB of Data at once) , then he wants to send them in chunks of data then he will call multiple Putdata's and then store the data.
In the same way, while getting the Data from the Database GetData is used.

Ah, yes, that makes much more sense. Thanks for the context. This is a usecase I'd  like to enable with database/sql/v2, but the current API doesn't really allow for this at the moment. You'll need to use the direct DB2 API I think. I'll look into that driver.

Thanks for the quick turn around, As many users are store lob data in the database it will be useful and when Go is planning to release database/sql/v2.

If I understand your usecase correctly, after thinking about it for awhile, you can probably implement this in your driver with the current interface.
The key is to implement the https://godoc.org/database/sql/driver#NamedValueChecker Named Value Checker, then allow the user to pass in some type of reader, possibly an io.Reader or other chunk based reader. Then the driver can upload parameters as it reads it off in chunks.

NamedValueChecked is just declared in the driver file and no sign of it in the sql file. How to call that function. Is there any example on how to use it.

Fair point. I don't remember if we have examples for that driver member. Here is an example for a SQL Server driver, which supports a number of non-stdlib types:
https://github.com/denisenkom/go-mssqldb/blob/master/mssql_go19.go#L66

Thanks for the information. But that did not help me. Correct me If I am wrong, As of what I know If a user writes an application using the driver the flow is
Application(Database program) --->database/sql/sql.go -----> database/sql/driver/driver.go(where the function are declared) -----> imported Driver.
So if the user wants to use it(NamedValueChecked) at the application side it should be present in the database/sql/sql.go or else it will produce an error that function is not defined in the sql.go
So, what is the equivalent function in the in sql.go for NamedValueChecked?
I will go through the docs to check whether it is helpful for my case or not.

If I extend the interface and add a function to the Stmt handle it will not work because the PutData and get Data functions Require SQLHSTMT as one of the parameters and which is stored in the driver.Stmt. Since driver.Stmt is not exported in the Stmt I cannot access SQLHSTMT.
If there are Read, write specifiers like export and unexport then it will be too easy.

I'm going to close this request. For this specific of capabilities, please sue the Conn.Raw method.


/Discover (404)

Fresh install, docker, windows 11
Clicking on the discover link, returns 404

According to issue #208, The Discover Feature is not yet implemented and under Development.
I also don´t have it, and it is not yet present in the sourcecode.

According to issue #208, The Discover Feature is not yet implemented and under Development. I also don´t have it, and it is not yet present in the sourcecode.

Okay, thank you


Make env route dynamic

Possibly closes https://github.com/penumbra-zone/dex-explorer/issues/121
By default, the api/env route was being built statically at build time. It looks like Nextjs recognized that it depended on static variables (env vars) and thought it a good idea to pre-build the route's response #facepalm.
Before:

After:

Nice! This change resolved the specific problem reported in #121, where testnet asset lookup was failing. Running this new code, I see in the browser console that the correct lookup is performed via GET:

which confirms that the server-side env var is now properly configuring the app behavior, as we wanted. Problems still remain in terms of overall app stability, but will file follow-up issues for those so they can be addressed appropriately.


SDAG Rdma example /examples/rdma/stencil3D failing on nightly build because of migration

Original issue: https://charm.cs.illinois.edu/redmine/issues/1614

Running on 2 processors:  ./stencil3d 64 32 +balancer RefineLB +ppn 2
charmrun>  /usr/bin/setarch x86_64 -R  mpirun -np 2  ./stencil3d 64 32 +balancer RefineLB +ppn 2
Charm++> Running on MPI version: 2.1
Charm++> level of thread support used: MPI_THREAD_SINGLE (desired: MPI_THREAD_FUNNELED)
Charm++> Running in SMP mode: numNodes 2,  2 worker threads per process
Charm++> The comm. thread both sends and receives messages
Converse/Charm++ Commit ID: 3383678
CharmLB> Load balancer assumes all CPUs are same.
Charm++> Running on 1 unique compute nodes (8-way SMP).
Charm++> cpu topology info is gathered in 0.020 seconds.
[0] RefineLB created
STENCIL COMPUTATION WITH BARRIERS
Running Stencil on 4 processors with (2, 2, 2) chares
Array Dimensions: 64 64 64
Block Dimensions: 32 32 32
[respect:01590] *** Process received signal ***
[respect:01590] Signal: Segmentation fault (11)
[respect:01590] Signal code: Address not mapped (1)
[respect:01590] Failing at address: 0x2aaabc29c5e0
[respect:01590] [ 0] /lib/x86_64-linux-gnu/libpthread.so.0(+0xfcb0) [0x2aaaaacdfcb0]
[respect:01590] [ 1] ./stencil3d(_ZN7Stencil13processGhostsEiiiPd+0x268) [0x4ddfa8]
[respect:01590] [ 2] ./stencil3d(_ZN7Stencil9_serial_0EPN15Closure_Stencil23receiveGhosts_5_closureE+0x26) [0x4dcb86]
[respect:01590] [ 3] ./stencil3d(_ZN7Stencil7_when_0Ei+0x1f3) [0x4dc4a3]
[respect:01590] [ 4] ./stencil3d(CkDeliverMessageFree+0x92) [0x5271d2]
[respect:01590] [ 5] ./stencil3d(_ZN8CkLocRec11invokeEntryEP12CkMigratablePvib+0x54) [0x54e1a4]
[respect:01590] [ 6] ./stencil3d(_ZN8CkLocMgr10deliverMsgEP14CkArrayMessage9CkArrayIDmPK12CkArrayIndex11CkDeliver_ti+0x3d1) [0x54e5e1]
[respect:01590] [ 7] ./stencil3d(_Z15_processHandlerPvP11CkCoreState+0xbe9) [0x52d979]
[respect:01590] [ 8] ./stencil3d(CsdScheduleForever+0x48) [0x62f5b8]
[respect:01590] [ 9] ./stencil3d(CsdScheduler+0x2d) [0x62f82d]
[respect:01590] [10] ./stencil3d() [0x62d7d2]
[respect:01590] [11] ./stencil3d() [0x62dd97]
[respect:01590] [12] /lib/x86_64-linux-gnu/libpthread.so.0(+0x7e9a) [0x2aaaaacd7e9a]
[respect:01590] [13] /lib/x86_64-linux-gnu/libc.so.6(clone+0x6d) [0x2aaaabf1638d]
[respect:01590] *** End of error message ***
The failure is due to the closure structure being pupped incorrectly by the generated code. An rdmawrapper should be pupped as an array when inside a closure structure. (i.e marshall the array contents into the buffer  as rdma pointers are invalid on a remote processor).

Original date: 2017-06-23 16:35:11

Fix: ~~https://charm.cs.illinois.edu/gerrit/#/c/2726/~~ https://github.com/UIUC-PPL/charm/commit/7c8a032369b593b630c33e45a680c24df15f46a7
This bug was caught in the nightly build for target mpi-linux-x86_64-smp on respect(machine). The crash happens on unpacking and using the rdma buffer after the closure structure has been unpacked on the new PE after migration. The fix was to adjust the rdma pointers before and after the migration in the pup method of the closure structure generated by charmxi.


Minor: Zerops Guide in Cookbook (WIP)

Overview
What is it?

[ ] Cookbook / Guide

Description
This is a pull request for a zerops guide for qwik's cookbook in docs.
Link to Zerops: https://zerops.io
Use cases and why

Deployment guide for zerops to deploy qwik without encountering any problem.

Checklist:

[ ] My code follows the developer guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] Added new tests to cover the fix / functionality

Thanks for opening this PR.
Are you still working on this or is it abandoned?

Still got some crumbs left ~ will try to complete this by friday 👍🏼


add pfm testing scripts

a->b->c pfm works. to verify:

open two terminal windows as described below

terminal 1 monitors the sender on A: while true; do gaiadl q bank balances $(gaiadl keys show gval1 -a)  && echo "--------------" && sleep 1; done
terminal 2 monitors the receiver on C: while true; do osmosisdl q bank balances osmo1uk4ze0x4nvh4fk0xm4jdud58eqn4yxhrqyeqwq  && echo "--------------" && sleep 1; done

run sh dockernet/scripts/pfm/pfm_abc_default.sh
watch the terminal windows to see C's balance increase after A's decreases.

pfm'ing + autopilot coin A thru chain A will work after upgrade but pfm'ing + autopilot coin A thru B won't work
-there's a script that sends from atom from CH -> stride then from Stride -> osmosis, then tries to autopilot LS that ATOM from Osmosis. If fails.
-there's a script pfm_to_autopilot_ls.sh that tries to PFM (a -> b -> autopilot-LS-on-a: stride -> osmosis -> stride (autopilot LS). It doesn't work right now. Because the autopilot step doesn't work.

I think since pfm was merged you can change the base branch and merge right to main


Adjustments should allow you to increase inventories and not just decrease them

Currently, Adjustments only allow users to decrease inventories, but we need to be able to increase them as well. This may require changes within the Adjustment model in addition.

[ ] Rename "Adjustemnt" to "Inventory Adjustment" in UI
[ ] Add a more explicit adding method for Inventories
[ ] Add test coverage


New a way to disable crds from installation

we are now support helm3 and put all crds under /crd . since /crd does not support templates, there is currently not an easy to to disalbe crds from base.
One possible solution is to keep only crds under base and move all others under istio-control/isito-discovery since those sa, role binddings and cluster role bindings, validation webhook configuraiton will be used by istiod.

@howardjohn @linsun WDYT about above solution?
As if we only change helm.go it will not support helm3 any way.

@howardjohn I consult with helm team and get response like below:
Unfortunately, manifests only apply in the /crds directory and you would not have the templating for adding the enable/disable logic. You could put the CRDs in the template directory as long as there are no resources that use these CRDs in the chart. This may however circumvent API validation and not sure if it is an approved way to do it. You would also need to test this method against your chart to make sure it works as expected.

https://github.com/helm/helm/issues/8821
I moved crds under /templates and test this out via helm3 isntall it works well.   As we are in the case You could put the CRDs in the template directory as long as there are no resources that use these CRDs in the chart.
So suppose we are safe to move crds under /templates?
Any known issues for this?

cc / @sdake

cc / @nmittler @stevenctl @linsun
This is the item we discussed regarding to Split crds and validation webhook configurations from base via a single flag to support remote-less cluster I am stuck on a good solution here. Any suggestions you can offer?
By the way, I will be out until 10/10 so will get back to this when I am back.

@howardjohn I consult with helm team @hickeyma and get response like below:

Unfortunately, manifests only apply in the /crds directory and you would not have the templating for adding the enable/disable logic. You could put the CRDs in the template directory as long as there are no resources that use these CRDs in the chart. This may however circumvent API validation and not sure if it is an approved way to do it. You would also need to test this method against your chart to make sure it works as expected.

helm/helm#8821
I moved crds under /templates and test this out via helm3 isntall it works well. As we are in the case You could put the CRDs in the template directory as long as there are no resources that use these CRDs in the chart.
So suppose we are safe to move crds under /templates?
Any known issues for this?

The conditional here (as long as there are no resources that use these CRDs) is highly dangerous in my opinion. We have been completely cratered by past helm conditionals like this. I would encourage more exploration.
Cheers,
-steve

actually nevermind - too early - or late - brain still booting. We do not use CRs in any charts where we install CRDs. This leads to race conditions. This is why we have a base chart to install the CRDs. Other charts then may install CRs. I haven't looked at your specific PR, but will take a look when my day begins.
Cheers,
-steve

we are now support helm3 and put all crds under /crd . since /crd does not support templates, there is currently not an easy to to disalbe crds from base.
One possible solution is to keep only crds under base and move all others under istio-control/isito-discovery since those sa, role binddings and cluster role bindings, validation webhook configuraiton will be used by istiod.

One potential issue with the possible solution is how it works with multiple control planes as some of these are cluster wide configurations such as cluster role.

@howardjohn I consult with helm team @hickeyma and get response like below:

Unfortunately, manifests only apply in the /crds directory and you would not have the templating for adding the enable/disable logic. You could put the CRDs in the template directory as long as there are no resources that use these CRDs in the chart. This may however circumvent API validation and not sure if it is an approved way to do it. You would also need to test this method against your chart to make sure it works as expected.

helm/helm#8821
I moved crds under /templates and test this out via helm3 isntall it works well. As we are in the case You could put the CRDs in the template directory as long as there are no resources that use these CRDs in the chart.
So suppose we are safe to move crds under /templates?
Any known issues for this?

@nrjpoddar @jacob-delgado are you aware of any known issue with this?  I'm leaning towards this approach atm.

I’m not completely following all the use case and what we are trying to achieve here. As far as I can tell we should keep CRDs under the CRD directory and let Helm manage the lifecycle of CRDs. This will cause behavioral changes for customers as I’m adding doc to support Helm3 in 1.8.

@nrjpoddar the user case is that we need to provide an option to disable CRDs install for remote cluster as they are not needed.  Only the config cluster needs the CRDs.  We don't find a good way to do this, a few options:

move non crds out of base to istio-discovery.  This has implications for multi-revisions for istio-discovery.
move crds to templates so we can add conditions.
split base component to a dedicated component for crd, and another component for non-crds.

Thanks @linsun, what tool are you using to install Istio in remote cluster? If you're using istioctl or Operator you have other ways to accomplish this, correct?

Yes, we do plan to use istioctl or operator - are you suggesting we work around this issue via operator code? if so, helm user won't have it for remote cluster.

Yes, we do plan to use istioctl or operator - are you suggesting we work around this issue via operator code? if so, helm user won't have it for remote cluster.

May be. There's no downside to installing CRDs in remote cluster afaict, so if folks want to use Helm for remote "istiod" setup, they can live with few CRDs or you can add a step to remove CRDs after installing the base chart which doesn't feel right but achieves what you want.
I'm also fine with other solutions around moving cluster resources inside istio-discovery chart as long as it works with canary upgrades.

@nrjpoddar Thanks for looking at the issue!    Basically we want to a way to exclude CRDs when we have base enabled in the IOP file.  Now since crds are under /crd directory in `base' charts and it does not support templates, so there is no way to exclude crds..
The scenario for this is we have a separate config cluster and all remote clusters will not need CRDs installed but the sa, role bindings are needed.
So no matter helm or istioctl , if there is good way to exclude CRDs from base?
Thanks!

I see, Helm install/upgrade provides an option `skip-crds’ option which should meet your needs.

@nrjpoddar Thanks, it is good to know helm has such an option. I think it would be reasonable for istioctl to also introduce an equivalent option.
cc @ostromart @irisdingbj


Create UserAccountEnabledDisabled_10m.yaml

Identifies when a user account is enabled and then disabled within 10 minutes. This can be an indication of compromise and
an adversary attempting to hide in the noise.
Fixes #
Proposed Changes

 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.:x: samikroy sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Upload-Offset vs Range?

Why was Upload-Offset chosen of simply using Range to indicate position?

You can read an in-depth discussion about it here: https://github.com/tus/tus-resumable-upload-protocol/issues/2
The short version is that with we would violate the HTTP specification if we used the Range header for these purposes.


Replacing services with ES modules

My project is bundled with webpack so I tried to replace services with regular ES modules. I found few reasons to use actual services but one of them was crucial:

Services has access to application, which is required way for communicating with it (as opposed to Box.Application). For that I created dumb service returning application which I can use in regular ES modules.
The lifecycle of services is synchronized with application, so the global config works there just fine. But in regular ES modules it breaks, because they executed early while Box.Application.init(config) is still not executed so the global config is empty there.

So I guess in current t3js state it is better to just use services?
P.S.: also I found this repo, but it looks abandoned

ES6 modules replace most of the functionality that a service/module registry system, such as t3js, provides. For us at Box, we've opted to move wholly towards ES6 modules and webpack and have stopped active use of t3js to avoid the complexities that you are encountering.
I recommend avoiding trying to use the Box.application outside of framework components/services due to the synchronization and underlying assumptions that services are built on. The only attach point external modules should be utilizing is the event system.
So yes, in this case, continue to use t3js services. You can still use webpack to help bundle your application but code splitting will be more complex without modules.


V4 Getting two matches, one for :id and another not

<Match pattern="/projects/new" exactly render={this.renderProtectedMatch(ProjectsNew)} />
<Match pattern="/projects/:id" exactly render={this.renderProtectedMatch(Project)} />

and url: /projects/new results in matching both.

This is expected. If only one should render, use a <MatchRoutes>. The documentation for it explains this exact situation.

I ran into the same problem. The solution is super, so the matching is simpler. Can you bump a more current version to npm since in v4 alpha6 there is no MatchRoutes and a git source is not working since a build step is required.

+1


panic following tutorial-app, on reload "no such file or directory"

On Fedora 38 using ds-dev v0.11.0, I followed the tutorial app but every time I save app.ts the ds-dev panics:
set job to finished and sending
2023/10/11 19:02:03 sandbox/sandbox.go:579 a:11 v:1000.0.0 as:15 (Sandbox, Graceful()) sending twine.Graceful()
2023/10/11 19:02:03 sandbox/sandbox.go:584 a:11 v:1000.0.0 as:15 (Sandbox, Graceful()) graceful complete
2023/10/11 19:02:03 sandbox/sandbox.go:864 a:11 v:1000.0.0 as:15 (Sandbox, setStatus()) Sandbox 2 set status from 4 to 5
2023/10/11 19:02:03 sandbox/sandbox.go:661 a:11 v:1000.0.0 as:15 (Sandbox, cleanup()) cleanup complete
2023/10/11 19:02:03 sandbox/sandbox.go:864 a:11 v:1000.0.0 as:15 (Sandbox, setStatus()) Sandbox 2 set status from 5 to 6
panic: stat /home/trevor/Desktop/dropserver/tutorial-app/4913: no such file or directory

goroutine 41 [running]:
main.(*DevAppWatcher).watch.func1()
	/home/runner/work/Dropserver/Dropserver/cmd/ds-dev/appwatcher.go:179 +0x22d
created by main.(*DevAppWatcher).watch
	/home/runner/work/Dropserver/Dropserver/cmd/ds-dev/appwatcher.go:165 +0xdf

I tried a few variations on the command line with relative an full paths as well as setting or not setting the appspace option. It seems to make no difference.

Hi Trevor, thanks for reporting this.
Do you have an idea what /home/trevor/Desktop/dropserver/tutorial-app/4913 file might be? It doesn't sound like a code file you'd be creating. Maybe a temporary file?
I'm wondering if it's a file that gets created and then deleted in very quick succession, which trips up the file watcher algorithm. I can get around this is that's what it is.

Hello. 4913 isn't a file that I've created, either manually or in code, so I'm not sure what it's meant to do or why it would appear in the tutorial-app directory. The name, "4913", is consistent though so it doesn't seem to be a per-process lockfile named after a process id.

Hmm. Weird. I can't reproduce this on Arch or Mac.
Are you using vim by any chance? I just found this post that indicates vim creates a temporary file called 4319.  I'll have to dive into this post and integrates the author's findings.

I am using vim, well, nvim. I just read the linked post and indeed, I think that could explain what is happening.

Thank you.


AD&D 1e update

Changes / Comments

added an announcements section to show important information on upcoming sheet changes
fix for Spells Known roll
small adjustments to the sub-section hide/expand carrot/nav button
changed non-repeating attribute names to lowercase as well as title text.  Purely aesthetic and for attribute naming consistency.

Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[ ] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.


Saving model does not save output_shape in deconv2D

Hello everyone.
I use keras last version and tensorflow 0.10.
I encoutered an issue with model serialization on disk.
I defined a model like :
model.add(Deconvolution2D(64,3,3, output_shape=(32, 8, 8, 64),border_mode='same', activation='relu'))
model.add(Deconvolution2D(64,3,3, output_shape=(32, 8, 8, 64), border_mode='same', activation='relu'))
model.add(UpSampling2D(size=(2,2)))

(I couldn't use a viariable batch size in output_shape like (None, 8, 8,64) but that's another issue)
everything is fine, i could train, predict, etc.
but when i try to save on disk with model.save(...) and load with keras.models.load_model(...) i got the following :
TypeError: Expected binary or unicode string, got None
So i checked the saved config file and i got this for the layer previously described:
{
    "config": {
        "activity_regularizer": null,
        "W_regularizer": null,
        "b_constraint": null,
        "init": "glorot_uniform",
        "nb_filter": 64,
        "nb_row": 3,
        "border_mode": "same",
        "name": "deconvolution2d_1",
        "nb_col": 3,
        "W_constraint": null,
        "b_regularizer": null,
        "trainable": true,
        "output_shape": [
            null,
            8,
            8,
            64
        ],
        "bias": true,
        "activation": "relu",
        "subsample": [
            1,
            1
        ],
        "dim_ordering": "tf"
    }
}

Note the "output_shape": [null,...
Also, model.summary() gave me : deconvolution2d_2 (Deconvolution2(None, 8, 8, 64)
So i have two question :

Can i save the real fixed ouput_shape ?
Is there a way to use a dynamic batch number in output_shape with conv2d and tensorflow ? (using None gave me the same error in layer definition)

Thanks for all and for this wonderful framework

Hey, I just ran across the same problem. Here is how I solved it!
First of all, when define the model you should not give it a specific batch_size. But I guess if you use None while defining the model, you will get the same error. Same here.
It actually has nothing to do with keras. If you look at the python trackback carefully, you will see that the error occurs in tensorflow. Notice one line in the traceback:
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/nn_ops.py", line 276, in conv2d_transpose
So the real problem is that conv2d_transpose in tensorflow does not accept None as its shape. My solution is to use 'symbolic None' instead. For example, the input the this layer has shape [None, h, w, nb_channel], then you can write deconv_shape = [tf.shape(x)[0], new_h, new_w, new_channel]. Pass this deconv_shape to the Deconv layer should work. I use functional API in my model so you may need a bit more tricks to make it work in yours.


sls dynamodb install fails

Actual Behaviour
Serverless Error ---------------------------------------

  Serverless plugin "serverless-dynamodb-local" not found. Make sure it's installed and listed in the "plugins" section of your serverless config file.

Steps to reproduce it
yarn add -D serverless-dynamodb-local
sls dynamodb install
LogCat for the issue
info All dependencies
└─<EMAIL_ADDRESS>Done in 3.09s.
➜  api sls dynamodb install

  Serverless Error ---------------------------------------

  Serverless plugin "serverless-dynamodb-local" not found. Make sure it's installed and listed in the "plugins" section of your serverless config file.

  Get Support --------------------------------------------
     Docs:          docs.serverless.com
     Bugs:          github.com/serverless/serverless/issues
     Issues:        forum.serverless.com

  Your Environment Information -----------------------------
     OS:                     linux
     Node Version:           10.10.0
     Serverless Version:     1.35.1

I have the following in serverless.yml
plugins:
  - serverless-webpack
  - serverless-dynamodb-local
  - serverless-offline

I am also using a yarn workspace. That might be a problem with paths?

was the wrong cli command -> https://github.com/99xt/serverless-dynamodb-local/issues/174


Electrical pop-art scroll driven animation

Ik wil dit toepassen op de webinar page de achtergrond veranderen.

Electrical pop-art

Felle kleuren (neon)
Achtergrond patroontjes (vooral stippen)
Cartoon style

Wireflow

On desktop since the nav is at the top you will see the previous pattern on there.

I decided to use css instead of svgs for the backgrounds. I found a website were you can generate css backgrounds and I used these backgrounds in the keyframes. I gave the html scroll-timeline: --page-scroll block; this way we can use the property --page-scroll in animation-timeline and it will time the animation to take as long as the page is.
https://github.com/itsValyria/Oncollaboration/blob/cf293933e8eb8d9cf88c1933308f1989466d5719/src/routes/%2Blayout.svelte#L30-L31
https://github.com/itsValyria/Oncollaboration/blob/cf293933e8eb8d9cf88c1933308f1989466d5719/src/routes/%2Blayout.svelte#L46-L

I added an animation to change the background color of the cards when you scroll, by using a custom property that changes in the animation. If you want the colours to flow into the next on you have to use @property with inherits: true; this way the browser knows that this property can change. I used scroll(self inline) on the carrousel so that when you scroll trough it the colours change.
https://github.com/itsValyria/Oncollaboration/blob/d2351b14c0cb43a32f258cf08b3895afd7299e2a/static/css/global.css#L90-L112
https://github.com/itsValyria/Oncollaboration/blob/d2351b14c0cb43a32f258cf08b3895afd7299e2a/src/routes/%2Bpage.svelte#L72-L77
https://github.com/itsValyria/Oncollaboration/blob/d2351b14c0cb43a32f258cf08b3895afd7299e2a/src/routes/webinars/%2Bpage.svelte#L32-L37

I also added a shake animation to the webinarcards on the webinarspage. Every card has a delay on it. I used the index of the loop and gave that to a custom property --index. With the index I calculated the delay calc(var(--index) * 0.3s)
https://github.com/itsValyria/Oncollaboration/blob/c9bf99262d0658e801c7eb9cc0380f49f5e13559/src/routes/webinars/%2Bpage.svelte#L19-L23
https://github.com/itsValyria/Oncollaboration/blob/39ffc5860218aa1760b931beccaf74599fb76a71/src/routes/webinars/%2Bpage.svelte#L44-L73


Timeout in register scan

Per discord discussion, I'm not sure my device is ever going to respond...but register_scan.py does not catch timeout error.  So somewhere in the middle of the 10,000 address scan it stopped.  Would be nice to catch this and retry, or at least tell user what the last address scanned was.
Traceback (most recent call last):
  File "C:\Users\Admin\git\pysolarmanv5\examples\register_scan.py", line 27, in <module>
    main()
  File "C:\Users\Admin\git\pysolarmanv5\examples\register_scan.py", line 11, in main
    val = modbus.read_input_registers(register_addr=x, quantity=1)[0]
  File "C:\Python310\lib\site-packages\pysolarmanv5\pysolarmanv5.py", line 224, in read_input_registers
    modbus_values = self._get_modbus_response(mb_request_frame)
  File "C:\Python310\lib\site-packages\pysolarmanv5\pysolarmanv5.py", line 178, in _get_modbus_response
    mb_response_frame = self._send_receive_modbus_frame(mb_request_frame)
  File "C:\Python310\lib\site-packages\pysolarmanv5\pysolarmanv5.py", line 172, in _send_receive_modbus_frame
    v5_response_frame = self._send_receive_v5_frame(v5_request_frame)
  File "C:\Python310\lib\site-packages\pysolarmanv5\pysolarmanv5.py", line 163, in _send_receive_v5_frame
    v5_response = self.sock.recv(1024)
TimeoutError: timed out

My device uses a similar but different scheme.  You could try my code at https://github.com/austinmroczek/modbus_mapper


Dockerfile is incomplete.

The Dockerfile provided appears to be incomplete; it does not copy any files into the image to be run and therefore when the image is run it fails.

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

ID: 4c7470d2-9a57-7654-e13a-4f912898f7aa
Version Independent ID: 58d57b81-4ca6-6536-8788-f212402b41a7
Content: Containerize an app with Docker tutorial - .NET Core
Content Source: docs/core/docker/build-container.md
Product: dotnet-core
Technology: dotnet-docker
GitHub Login: @IEvangelist
Microsoft Alias: dapine

Re-reading it the copy command is there, my apologies, not sure how I missed it!


Can we enable vertical and horizontal scroll while we are dragging the event?

Closing as duplicated of #174


fix(websocket): avoid data race on client request

Motivation and Context
This PR ensures that we don't modify the client request concurrently in multiple goroutines. We copy the request before registering the subscription. This change doesn't come with a big performance penalty because we were already forced to clone the headers. Furthermore, I moved URL parsing out of executeSubscription to do it only once in the websocket handler.

TODO

[ ] Tests or benchmark included
[ ] Documentation is changed or added on https://app.gitbook.com/
[ ] PR title must follow conventional-commit-standard

LGTM. So to clarify, the main fix is in parsing the preHandler.parseRequestOptions earlier, so that it's handed down in the stack-trace and not done across goroutines, as well as explicitly cloning the request?

Yes


adding palindrome checker in python

Okay.
On Mon, Oct 14, 2019, 3:00 AM Otacilio Saraiva Maia Neto <
<EMAIL_ADDRESS>wrote:

@OtacilioN requested changes on this pull request.
Hey,
I think during the time of pull request review someone did this algorithm
before, I don't see any problems in having two versions of that algorithm
in the same programming language.
So since we have a git conflict here, can you rename your file adding a 2
at the end of the file? Example if your file has the name "bogosort.js"
rename to "bogosort2.js" and commit again.
Thanks.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ReciHub/FunnyAlgorithms/pull/336?email_source=notifications&email_token=AK5E3JLWOU7B2TKUEWHINV3QOOHQTA5CNFSM4I7LZC52YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCHY7AKI#pullrequestreview-301068329,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AK5E3JJBNVRTBVNCDL24VGDQOOHQTANCNFSM4I7LZC5Q
.

Can you please review it again?
On Mon, Oct 14, 2019 at 9:51 AM vinayak jaiswal<EMAIL_ADDRESS>wrote:

Okay.
On Mon, Oct 14, 2019, 3:00 AM Otacilio Saraiva Maia Neto <
<EMAIL_ADDRESS>wrote:

@OtacilioN requested changes on this pull request.
Hey,
I think during the time of pull request review someone did this algorithm
before, I don't see any problems in having two versions of that algorithm
in the same programming language.
So since we have a git conflict here, can you rename your file adding a 2
at the end of the file? Example if your file has the name "bogosort.js"
rename to "bogosort2.js" and commit again.
Thanks.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ReciHub/FunnyAlgorithms/pull/336?email_source=notifications&email_token=AK5E3JLWOU7B2TKUEWHINV3QOOHQTA5CNFSM4I7LZC52YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCHY7AKI#pullrequestreview-301068329,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AK5E3JJBNVRTBVNCDL24VGDQOOHQTANCNFSM4I7LZC5Q
.


Added flake8 for checking Python syntax

Fixes #1561
Signed-off-by: Abhijeet Kasurde<EMAIL_ADDRESS>

I don't think this PR is worth merging unless it also includes a commit that:

configures flake8 in a sane way (e.g. don't complain about unused imports in __init__.py and ignore anything outside the flask package)
fixes all problems

Using before_script doesn't even run the tests if flake8 fails. I don't think this is a good idea. Especially for pull requests it's not very helpful if failing style checks skip the unit tests.

Would it make sense to run flake8 from tox.ini?
pytest-pep8 could also be an option

OK. Let me try flake8 from tox.ini

@ThiefMaster How we are going to handle existing pep8 errors?

Fixing them all might be the easiest option, with # noqas where they make sense.
@ThiefMaster recommending fixing them all here

Another option is to pass --exit-zero to flake8 for now, until all the issues can get cleaned up

continued in #1702


add questionnaire-response back

Summary
This PR add US Core QuestionnaireResponse test back to US Core 5.0.1 Test Kit.
Don't merge this PR till ONC confirms test procedure for QuestionnaireResponse
Testing Guidance
Launch US Core 5.0.1 Test Kit
Verify that Questionnaire Response test group exists

After discuss with ONC, we decide not to include QuestionnaireResponse as a normal test group.


Audio registers flags

Added some convenience flags for audio registers.

Flags for specific registers right below the related register.
Flags common to multiple registers in a separate section.
Decided to name SO1/SO2 as LEFT/RIGHT in flags (after looking which terminal is which for the Nth time).
All are tested, except AUDVOL_VIN_LEFT/RIGHT which I added for completeness (following hardware.inc notes).

LGTM. Can't review or push, though, but I'd approve if I could.

Let me try this review thingamajig


Add new script to zip GCS file trees and upload them to Zenodo

Run via analysis-runner so it has direct access to our buckets, zips via Python's zipfile library, and uploads directly to a (pre-made) Zenodo deposit via Zenodo's API.
Context: https://centrepopgen.slack.com/archives/C018KFBCR1C/p1730142264294839?thread_ts=1728943125.475979&cid=C018KFBCR1C

@dancoates It's a good point — In this case, there's ~15,000 files going in to each zip archive, for a total of almost 400,000. I don't have much test data in a test bucket, so I guess the thing to do will be to merge it and see what happens… 🤞


Option to change math separators

I can't change my vscode plugin (Markdown All in One) to work with the Math separator syntax which the script is using. In my opinion it should be possible to have a option that ankdown also works with $ and $$ by substitute it later in the process to \( \) and \[ \] with a regular expression.
When I have the time I would do a pull request. Would be nice to get your opinion about it.
Thank you for your work.

So, I think in order to get this to work you would need to parse out your custom separators and replace them with \(,\), etc., since ankdown has switched to using Anki's builtin mathjax support (which forces you to use those separators). I'd definitely accept a PR that did this is a nice way (one that's extensible to alternate separators and easy to configure).

Thank you for your fast answer
the question is how to get this extensibility  done.
Something like this maybe:
ankdown -r [directory] -p [package filename] -s={'$', '$$'} [change math separator to {X, XX}]
don't know if the syntax is totally right but i hope you get the gist out of it.

I think it would be fine if there were a flag for "--dollar-separators"; the main thing is just to make it easy in the code to change them / make them configurable later.

Done in #14; thanks @jay-pee!


access-token rejected saying "the request could not be verified"

I get the oauth_verifier through request-token request. But access-token request comes back with 401 status and message "oauth_problem=token_rejected&oauth_problem_advice=The%20Request%20could%20not%20be%20verified".
I have confirmed that consumer-key & consumer-secret of my public account, are correct and valid - by logging into developer.xero.com/myapps.
I actually tried with 2 different public accounts - both were in working condition as of last week.
I know that XERO is moving to OAuth2, but it would be greatly appreciated with any help to get around this.

Hi @pkrajanand
Thanks for letting us know about your issue.
As you've mentioned that you're using Consumer Keys and Consumer Secrets, the app you're using on the My Apps page of the developer Center will be an OAuth 1.0a app and incompatible with this Postman tutorial. You'll need to create a New OAuth 2.0 app to get a Client ID and Client Secret and use those keys instead.
If you're still having issues after generating a new OAuth 2.0 app, please get in touch with us at<EMAIL_ADDRESS>with the new Client ID you've received and we'll be happy to assist further.
Cheers
Matt

Thanks Matt for your response.
Yes, you are right.. I am using consumer key & secret of of a 'public' OAuth 1.0a app from developer.xero.com/myapps - which was working perfectly fine with Postman until when I tried last on this 7th July.
I am surprised to hear that this tutorial turned incompatible with 1.0a app keys - especially when I can still see this note from the ReadMe of this project.

Xero-Postman
A Postman collection for authenticating to the Xero API. Please note that this collection provides access using OAuth 1.0a. Our newer collection using OAuth 2.0 can be found here(https://github.com/XeroAPI/xero-postman-oauth2).

I have a web-application that talks to XERO using OAuth1.0a app keys. Migration to OAuth2 cannot be done immediately and we have a roadmap for that.
I usually use PostMan to learn and verify XERO API features before I add those to my application. I tried with 2 different public 1.0a app keys- which had been working until last week. Now, it fails even with Request-Token invocation that just sends back a 401 !!! Actually this call should have redirected me to a login page after confirming the just the existence of the keys. Now, no login page is shown, so I deduce that the XERO-api does not even recognise the keys.
Officially, XERO's deadline is until Dec 2020 for existing OAuth 1.0a apps to migrate to OAuth2 - as per this note from https://developer.xero.com/faq/migration/.

Can I/should I still get my OAuth 1.0a app certified?
We encourage anyone building new apps to use OAuth 2.0 but we will continue to certify OAuth 1.0a integrations until the end of March 2020. Please be aware that if you choose to launch with OAuth 1.0a you will be required to migrate it to OAuth 2.0 before December 2020

So this looks more like a mistake from their side for Public apps. I have heard from a colleague that OAuth 1.0a Partner apps do not face an issue like mine.
Yea, I understand that I need to take it direct to XERO, as this Project is meant only to provide Postman setup to interact with Xero 1.0a endpoints.
Thanks for your time, greatly appreciated.

Hi @pkrajanand
Apologies, I had my Postman collections mixed up.
You will be able to use the Migration endpoint, but only if your app has been through the certification process and is a Partner App. If you do have a Private or Public app, you will have to connect each Org manually to your new OAuth 2.0 app and without the use of the migration endpoint.
As you would only have 30 minute access on Public apps in either case, you can simply provide the new OAuth 2.0 link to your users, rather than the older public app connection link. If you've included the Offline_Access scope you'll also be able to maintain a connection by refreshing your access tokens with refresh tokens.
If you do have a Partner App, please get in touch with us at<EMAIL_ADDRESS>with its Consumer ID and we'll be happy to help diagnose any errors that may be occurring.
Kind Regards
Matt


Closet skeletons can have their entity deleted but still remain on the ghost role list.

Description
Im not 100% sure but it looks like something is happening to the Closet skeleton and deleting its body. If you try to request or follow it via the ghost role list your game bugs out.
Screenshots
This is what happens when i try to follow or request the body,

This was on fland station about 1 hour into the round.
I was never able to find out exactly where the skeleton was spawned or what caused the issue.

Had another instance of this happening on Kettle when i ran addgamemode closetskeleton.
These were the logs when trying to teleport to it.


fixed UncaughtErrorEvents not working (#2574)

I found that there are two separate causes of issue #2574:

commit cde0386f6f70728854084b19b37fadf380f86aef made it impossible to enable uncaught error handling, because the try/catch in ApplicationMain is run only if the user had already added an event listener - but that is not possible, as ApplicationMain is the first code run by the application;

commit 6b58d04cecadde99f46e847d610d0dd3640f7faa made it impossible to enable uncaught error handling in debug mode. This line in include.xml:

<define name="openfl-disable-handle-error" unless="openfl-enable-handle-error" if="debug" />

... seems to only work when openfl_enable_handle_error is defined explicitly via the build command line (e.g. -D  openfl_enable_handle_error), but it doesn't have any effect if is defined in the local Project.xml. Probably because definitions in Project.xml are executed later than include.xml. That means that there is no way to configure a project to activate uncaught error handling in debug mode.
This PR moves the "unless enable" logic from include.xml to the class UncaughtErrorEvents, by initializing the static var __enabled to true if  openfl_enable_handle_error || !openfl_disable_handle_error is defined. This fixes both issues.
After the PR, the logic becomes:

in debug mode: error handling disabled by default, enabled by defining openfl-enable-handle-error;
in non-debug mode: enabled by default, disabled by defining openfl-disable-handle-error.

The logic of "disable the error handling if there are no listeners" remains untouched.
This seems to me to be the intention behind the two original commits.

Closed with #2656


failing to start at all if ifconfig is not available for the user

on Debian by default ifconfig is in sbin, so it's visible to root user only.
while the requirement is listed, it'll be good to:

not to fail at all if the ifconfig is not available
add the recommendation: sudo ln -s /sbin/ifconfig /bin/ifconfig

Now using ip via ip -s link and outputing the error via console.error().
Cheers


CI: Set Android build debug keystore

This change configures the project to use a single, consistent debug keystore, enabling the installation of testing builds without the need to uninstall previous versions of the app. This ensures a smoother testing process by maintaining the same keystore across builds, allowing for easier updates and debugging

Thanks!


Boot Simulator Directly and connect to Framebuffer

SimulatorKit has a very interesting opportunity for FBSimulatorControl. The primary motivation for using Simulator.app instead of booting directly with -[SimDevice bootWithOptions:error] was that there is a framebuffer connected when Simulator.app launches.
Using a similar methodology to create a SimDeviceFramebufferService as Xcode Playgrounds does, it's possible to create a Framebuffer, boot the simulator, then listen for events on the framebuffer. These events are CGImageRefs so can be recorded to images and video.

[x] FBSimulatorLaunchConfiguration control
[x] Basic implementation in bootSimulator
[x] Lots and lots of code-level documentation for how and why this works.
[x] Updates to the README to describe the differences between 'direct' and 'container' launches.
[x] Event Sink events for framebuffer lifecycle.
[x] Add framebuffer to the simulator instance.
[x] Debug NSWindow canvas for live framebuffer view.
[x] Video Framebuffer recorder.
[x] Use Scale variable on Framebuffer to encode at scaled resolution.

lgtm!

wooooo


Fix symlinks on different volumes on Windows

Related to https://github.com/huggingface/huggingface_hub/issues/1385. See CI logs in dependent library.
If we try to create a symlink between 2 volumes, common_path will fail with ValueError("Paths don't have the same drive"). Let's not create symlinks in such a case on Windows (still ok on Linux/MacOS).
cc @mayankjobanputra can you please try to run your test suite with this fix (pip install git+https://github.com/huggingface/huggingface_hub@1385-fix-symlinks-on-different-volumes) and report here if it definitely fixed your issue? Thanks in advance.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

@Wauplin My tests are passing with this branch :)
Thanks a lot :tada:

Good, thanks for confirming!


add fictract docs to ignore links as it is not working temporarily

It seems that this will take a bit longer than we expected.
https://github.com/rjdmoore/fictrac/issues/30
I will also remember that there is another broken link.

OK, @CodyCBakerPhD. I think it is better to link to the repo as the repo as the other link that is failing and it will be updated to some point. I also fix the other links that were broken. This is ready to go on my side.

@h-mayorquin Agreed, thanks for handling this


Using subranges to define a bigger range

Vensim allows the following:
dim: A, subdim ~~|

subdim: B, C ~~|

this is similar to:
dim: A, B, C ~~|

subdim: B, C ~~|

we should implement a loop when creating the subscript dict to define all subscripts ranges with subscripts and avoid including the subranges.

Solved in branch dev


fix: use infura secret

Updates the tests to use a project secret (as the requests are server-originated and may not include a referrer)
Fixes react test to use act, fixing a warning during tests
Throws an error if infura returns a 403, to facilitate remote debugging

This has passed tests. I do not know why they are not showing up.
https://github.com/Uniswap/redux-multicall/actions/runs/2054085366.


How can I get my cookies ?

https://pypi.org/project/genshinstats/#how-can-I-get-my-cookies
doesn't lead anywhere.

Links to headers don't work on PyPI, the documentation is written with the expectation that you're either viewing it on Github or locally.
https://github.com/thesadru/genshinstats#how-can-i-get-my-cookies

Links to headers don't work on PyPI, the documentation is written with the expectation that you're either viewing it on Github or locally.
https://github.com/thesadru/genshinstats#how-can-i-get-my-cookies

Thanks for the info.


Add Kotsu

Another day, another static site generator :)

pinging @bdougie, just in case...

another try... @biilmann

Need to rebase master @ArmorDarks. There was an unrelated error causing this build to fail.
https://github.com/netlify/staticgen/commit/0399a59ffbe903a8e937154cb4f666883bb5ec14

I can no longer fast-forward to current master. New PR here https://github.com/netlify/staticgen/pull/315


Dockerise - PSD-2221

https://regulatorydelivery.atlassian.net/browse/PSD-2221

Updates to ruby 3.2.2
Pins yarn & node versions
Draft working production docker image - more work will be needed once we can test deployment
Working docker-compose local dev setup, all components talking to each other

Not needed at present.


Producing archives with non-standard extensions

Hello
Would you consider taking a PR that made it possible to specify an alternative file extension? I'd like to use jarchivelib to create "dump" files for Neo4j, but we don't want to advertise the file format in the extension.
Thanks
-Ben

Sure. I would suggest modifying CommonsArchiver#createNewArchiveFile.
The condition could check if there is any file extension instead of the specific one.
The ArchiverCompressorDecorator has a different mechanism for generating file extensions.
Maybe you can come up with a solution.

Although I believe the chance is very small, it may break peoples code.
A cleaner solution would probably be to extend the API with a function with the signature File create(File target, File sources...) (and the adjacent one)
However, this would be significantly more effort, as it would involve restructuring parts of the Archive implementations.

Thank you. I've decided in the end to code against Commons Compress directly as well don't need the generality and flexibility that jarchivelib provides.


[Question] How to write not contains using gremlin on hugegraph?

Problem Type (问题类型)
struct / logic (架构 / 逻辑设计问题)
Before submit

[X] 我已经确认现有的 Issues 与 FAQ 中没有相同 / 重复问题 (I have confirmed and searched that there are no similar problems in the historical issue and documents)

Environment (环境信息)

Server Version: 1.0.0 (Apache Release Version)
Backend: RocksDB x nodes, HDD or SSD
OS: xx CPUs, xx G RAM, Ubuntu 2x.x / CentOS 7.x
Data Size:  xx vertices, xx edges

Your Question (问题描述)
How to write not contains using gremlin on hugegraph? Below is the contains syntax，But I don't know the grammer of not contains?
g.V().has('student','object_key',within('610541189818181616')).bothE().and(has('stu_edges','object_key',
Text.contains('610541189818181619'))).otherV().hasLabel('student').limit(10);

I have tried serveral ways to implement, But this sentance was not complied，Please give some suggestion，Thanks.
// add `not()` function outer of `has()` function, failed to executed.
g.V().has('student','object_key',within('610541189818181616')).bothE().and(not(has('stu_edges',
'object_key',Text.contains('610541189818181619')))).otherV().hasLabel('student').limit(10);

Use notContaining function which is provided in standard gremlin grammer, but this function is not supported.
g.V().has('student','object_key',within('610541189818181616')).bothE().and(has('stu_edges','object_key',
notContaining('610541189818181619'))).otherV().hasLabel('student').limit(10);

Vertex/Edge example (问题点 / 边数据举例)
No response
Schema [VertexLabel, EdgeLabel, IndexLabel] (元数据结构)
No response

use__.not(clause)could resolve this problem，Below is the full sentance.
g.V().has('student','object_key',within('610541189818181616')).bothE().and(__.not(has('stu_edges','object_key',
Text.contains('610541189818181619')))).otherV().hasLabel('student').limit(10);

Close my question.

resolved, close my question.

Resolved, close my question.


feat: add rye

This MR adds support for rye, as per https://github.com/renovatebot/renovate/issues/25273#issuecomment-1768042170
Everything should be here, but for some reason I can't get it to pick up TOOL_VERSION? Not sure what's going on there

@viceice  I've moved Rye to it's own container since it doesn't need python at all. It manages all of that by itself.

@vhdirk Please resolve fixed conversations 🤗

@rarkins rye is installing python on the fly when invoking. it's downloading a pre build python version from GitHub. this can cause issues on air gapped renovate installations.
the source doesn't seem to be configurable.
but we can register custom python versions, so rye can use it instead of downloading one on the fly.
should we disable auto install by default for containerbase?
I also think if we should use those pre build python versions instead of building ourselves.
just republish the binaries to our prebuild repo for easier installs.
https://gregoryszorc.com/docs/python-build-standalone/main/

If the source needs to be configurable, I could open an issue/PR on rye to enable it to do that?
Rye should be configurable to use the system python, though it does seem to download a python version anyway? (That might be something to do with my system, though)

Ideally could be configured to never download python itself

It seems to be possible to configure toolchains (https://rye.astral.sh/guide/toolchains/#registering-toolchains). However, any project that pinned a toolchain with a .python-version is able to override this (https://rye.astral.sh/guide/toolchains/#pinning-toolchains).
We could also just prefetch every downloadable toolchain upfront: https://rye.astral.sh/guide/toolchains/#fetching-toolchains ?

It seems to be possible to configure toolchains (https://rye.astral.sh/guide/toolchains/#registering-toolchains). However, any project that pinned a toolchain with a .python-version is able to override this (https://rye.astral.sh/guide/toolchains/#pinning-toolchains).

that's why I think we should download the python version ourselves and register it. disabling auto downloads.

We could also just prefetch every downloadable toolchain upfront: https://rye.astral.sh/guide/toolchains/#fetching-toolchains ?

too much and too slow, because most of the time we only need one chain

Ideally rye is configurable to never download python and use the system python instead

Ideally rye is configurable to never download python and use the system python instead

If the project under consideration (for renovate, that is) pinned a python version using .python-version, rye will always download that python version if it doesn't match the system python version. Even when only locking and not actually installing the dependencies.

Ideally rye is configurable to never download python and use the system python instead

If the project under consideration (for renovate, that is) pinned a python version using .python-version, rye will always download that python version if it doesn't match the system python version. Even when only locking and not actually installing the dependencies.

but it can be disabled and we can already ensure a proper python version.

I am patiently awaiting rye support in Renovate. To help speed this along I've looked at the latest unresolved conversations and provided answers to the questions raised by @viceice. Hope it helps :)
Thanks @vhdirk for all the work you've put into this!

I've had a bit of a spelunk in the Rye codebase. It doesn't seem possible at this time to prevent Rye from installing a Python version that's not there (there's no configuration or option check in that codepath), you can only prevent it by registering it. That should be straight forward, but is a bit more fragile. Would that be reasonable?
Strictly speaking Rye shouldn't need any Python installation to update the lockfiles. It actually does that all by itself. Rye currently only install Python so it can initialise the virtualenv, it initialises the virtualenv because the rye lock command is actually just a simple wrapper around rye sync that would ordinarily install the locked packages as well. It seems a better fix for this would be to make an improvement in Rye upstream so that locking only doesn't create a virtual env and doesn't install Python at all.

I've had a bit of a spelunk in the Rye codebase. It doesn't seem possible at this time to prevent Rye from installing a Python version that's not there (there's no configuration or option check in that codepath), you can only prevent it by registering it. That should be straight forward, but is a bit more fragile. Would that be reasonable?

That was my idea, yes.

Strictly speaking Rye shouldn't need any Python installation to update the lockfiles. It actually does that all by itself (or rather it uses uv to do it). Rye currently only install Python so it can initialise the virtualenv, it initialises the virtualenv because the rye lock command is actually just a simple wrapper around rye sync that would ordinarily install the locked packages as well. It seems a better fix for this would be to make an improvement in Rye upstream so that locking only doesn't create a virtual env and doesn't install Python at all.

That's even a much better option / solution.

I'm abandoning this in favour of https://github.com/containerbase/base/pull/3107


Variables tab

Closes #4516
Proposed Changes

Variables Application Menu

Variables Tab Keyboard shortcut
(For Mac OS): Press Command + Shift + V

Variables Bottom Tab

Checklist
To ensure you provided everything we need to look at your PR:

[x] Brief textual description of the changes present
[x] Visual demo attached
[ ] Steps to try out present, i.e. using the @bpmn-io/sr tool
[x] Related issue linked via Closes {LINK_TO_ISSUE} or Related to {LINK_TO_ISSUE}

We should call this Toggle variables tab

I'd also add Toggle problems tab now that we have the other entry.

I'd suggest that we decide on another key for the shortcut. Ctrl/Cmd+V is the traditional paste shortcut, and in some editors a combination of Cmd+Shift+(Option)+V triggers "past and match formatting". Why don't we use a letter without strong OS link, e.g. U or M.
https://github.com/user-attachments/assets/a90c0cc8-c132-48ae-931f-6f96a5a4f4f7

I'd suggest that we decide on another key for the shortcut. Ctrl/Cmd+V is the traditional paste shortcut, and in some editors a combination of Cmd+Shift+(Option)+V triggers "past and match formatting".

I subscribe to this. CC @lmbateman.

I agree that we should stay away from Cmd-V variations. Looks like for many MacOS apps (but not the Desktop Modeler), Cmd-M minimizes the window. Cmd-U is a common shortcut for underline, but since that's a fairly specialized feature, I think Cmd-U will work, and Ctrl-U should be fine for Windows. (Should we add a Cmd-M shortcut?)

BTW the recording I shared displays the vscode shortcuts. I think the Shift usage in that tool is wise as it allows to avoid conflicts. Learning from Cmd+P (print or toggle properties panel 🤡 ).

MacOS tests are passing now. Will review.

@abdul99ahad Could you adjust the PR according to my suggestion?

add Toggle bottom panel entry with Ctrl+B shortcut
remove Reset properties panel entry (was a workaround created at the time where we had issues with its size, no longer needed)
decide whether to add individual entries for each tab

I'd imagine a toggle entry for the bottom panel (we could use Ctrl+B) and (optional) entries for the individual tabs.

I agree but the core issue here is the visibility of the Variables tab. One option could be to implement toggling for the entire bottom panel. Should we address this within the scope of this issue, or would it be better to open a new issue specifically for panel toggling and keep the focus here solely on the Variables panel? thoughts?

@abdul99ahad Could you adjust the PR according to my suggestion?

add Toggle bottom panel entry with Ctrl+B shortcut
remove Reset properties panel entry (was a workaround created at the time where we had issues with its size, no longer needed)
decide whether to add individual entries for each tab

Okay. Regarding point 3, I think for now, we can add variables tab as individual entry and see in future if users also want Output and Problems tab in application menu.

Regarding point 3, I think for now, we can add variables tab as individual entry and see in future if users also want Output and Problems tab in application menu.

I'd say: Don't add a toggle for variables panel for now, but in the future automatically open variables tab from various places. Much more important that this is properly wired, i.e. with the FEEL editors.

Additionally, adding the individual entries including disabled states is a bit more work since opening a tab doesn't trigger a menu update at the moment. So I'd say let's keep it for later or just don't add them if no one ever asks.

Let's restore the previous bottom panel state (if applicable), once I toggle it open (again). This is standard behavior, and ensures I can temporarily close things, re-open if needed, and end up in the previous setup.
Right now I always end up on the Output tab, which is not desirable.

Done. Now, the bottom panel maintained state and re-open the same tab that was previously opened.

All changes are incorporated, and this PR is ready for review. Here’s a summary of the updates:

Variables Tab: Now accessible via the bottom tab button.
Toggle Bottom Panel:

Added to the application menu.
The panel now preserves context when opened/closed.
Accessible with the keyboard shortcut Command + B (MacOS) or Ctrl + B (Windows).

Removed: The “Reset Properties” option from the application menu.

@abdul99ahad Regarding https://github.com/camunda/camunda-modeler/pull/4638/commits/aa5d66d9043355041d1eb7ee2ae84242320e5c5c: The usual pattern is two empty lines separating two specs, separating specs and setup, separating two describes.
We don't add two empty lines between describe and child spec or describe, so this is the standard way of formatting you'll find across 99% of our tests:
describe('foo', function() {

  beforeEach(...);

  beforeEach(...);

  it('first test');

  it('second test');

  describe('nested describe', function() {

    describe('deeply nested describe', function() {

      it('nested it');
    });

  });

  describe('another nested describe', ...);

});

@abdul99ahad Regarding aa5d66d: The usual pattern is two empty lines separating two specs, separating specs and setup, separating two describes.
We don't add two empty lines between describe and child spec or describe, so this is the standard way of formatting you'll find across 99% of our tests:
describe('foo', function() {

  beforeEach(...);

  beforeEach(...);

  it('first test');

  it('second test');

  describe('nested describe', function() {

    describe('deeply nested describe', function() {

      it('nested it');

    });

  });

  describe('another nested describe', ...);

});

Understood. If that's the convention, then I think the test cases are perfectly aligned with the convention.

Beyond a minor style comment (#4638 (comment)) this still seems to not work as expected:

What I'd expect is that in any view I'm able to toggle the bottom tabs open and close.
What I see is that this PR manually registers the action for a subset of our views, so it is not available in forms, welcome page, etc.

Agreed. I assumed it should function like the properties panel, but since the bottom panel is more global, it makes sense to make it universally available. I've implemented the necessary changes, and the bottom panel is now accessible throughout the modeler.

Ok, this works properly now, across all file types. One issue that remains is that we don't properly fall back to another "known tab" when we switch to a view where a particular tab is not present:

This issue, however, can already be reproduced in released versions of the modeler, so not a regression.

I think it is a good solution we ended up building here :clap:

Please add an entry to the CHANGELOG, linking to the issue this closes.

Merging with some minor squashes.

Question: Did we agree to move the menu entry? I would expect it to be grouped with the Toggle properties panel entry.

Question: Did we agree to move the menu entry? I would expect it to be grouped with the Toggle properties panel entry.

Initially, the "Toggle Bottom Panel" option was grouped with Toggle Properties Panel because both were associated with specific views like BPMN and DMN diagrams. However, since the Bottom Panel is universally available across the entire application, its corresponding menu button needs to be accessible everywhere. Therefore, we’ve decided to place it alongside other universally applicable options in the application menu.

Okay, understood. 😃


Make it easier to run a global before_each before all before_each blocks or a global after_each after all after_each blocks

Currently, it is easy to achieve a global before_each after all before_each blocks using
busted.subscribe({'test', 'start'}, ...)

Sorry, accidentally pressed Ctrl+Enter halfway writing issue #737.


Import issue

Hi! Thank you for developing this module! I experienced an error when trying the Laplace module on Google colab. It says " cannot import name 'Laplace' from 'laplace' (/usr/local/lib/python3.7/dist-packages/laplace/init.py)". Kindly seek your assistance in this issue. Thank you!

Hi! Thanks for letting us know about this problem. How did you install the package in google colab? I briefly tried with the following command in the first cell which ran fine:
!pip install laplace@git+https://github.com/AlexImmer/Laplace.git

And after that from laplace import Laplace worked for me.

Hi! Great thanks for the solution! It works for me as well.
Previously, I used !pip install laplace without the git.
Thanks!


UPDATE DOCs

Tiny updates in the API, need to update docs accordingly;

mtgo platform url checker output mtgo and no more www.mtgo when possible
type guard checker for format, level_of_play and platform added
limited is added in the formats
Examples to gather data should be updated

Should add the new Schemas because of the new API
Should create an adapter to the old schemas just in case of legacy usage
Should create few examples using SQLite

Latest draft with commit a38f4b1c82b9cac7d782675d828db20e5760a2c3


FIXED #935: using properties instead of attributes whenever possible

This pull request is for fixing the issue #935 (wrong usage of attributes instead of properties).

Thanks, just pushed out 4.1.0 with these changes

Many thanks. That was fast :-)
Le 21 août 2015 19:38, "Assaf Arkin"<EMAIL_ADDRESS>a écrit :

Thanks, just pushed out 4.1.0 with these changes
—
Reply to this email directly or view it on GitHub
https://github.com/assaf/zombie/pull/938#issuecomment-133506690.


Using C1 or C2 of ResNet as the input featue map for FasterRCNN

Hi, I have a question about feature map. Is it possible to replace the C4 with C2 or C1 as the input feature map for FasterRCNN? If yes, which part should I modify? Only the configuration file?
Thank you!

I suppose that, although we have not tried so.

I suppose that, although we have not tried so.

Thanks, I will try and get back then.

Feel free to reopen it if there is any related issues.


Add delete endpoint for comments

Umm, this would allow anybody to delete anybody else's comments, right? That's definitely not what we want.
We want a dashboard-like thing where ONLY the admin has the power to delete comments. Therefore, this must not be an exposed endpoint. Especially without authentication.
Am I missing something?

Yeah, uhm, we need authentication, but no endpoint has one right now.

Yeah, we can't add delete functionality unless we can identify that the person deleting the comment is:

the author
the site admin

For the first one, we would have to store some sort of identification on the client side. That amounts to a level of tracking and I'll have to think about that.
The site admin thing is more feasible - with a password-protected moderation dashboard, the site admin can choose to delete comments at will. I will be implementing the dashboard in a while, so this will certainly be one of the most important features there.
For now, this PR is blocked/won't merge.

It might be reasonable to accept this as-is with the single addition of making it only accept requests from localhost (<IP_ADDRESS>) so that you could issue these rest calls directly as an owner of the server, but not from anywhere.

@chuyskywalker that might work, but I'm still uncomfortable merging this. The proper way to do this would be to use an authentication token and we would be able to do that when there's a moderation dashboard (which will happen soon).

One more problem with accepting requests from localhost: if you run Commento behind nginx, for example, all requests will be from localhost. If we're whitelisting IPs, we should also make sure X-Forwarded-For is empty. There are other additional checks as well, probably.


added tada

test pr

Coverage decreased (-0.2%) to 99.224% when pulling 6446a4b56851f8691401b88034227d16e7d2ddbf on dummy_pr into 402c7a51f72c38fdae78f2f68536cea52245609c on master.


Support for \BlueLaTeX on Windows

Theoretically, nothing prevents \BlueLaTeX from running on a Windows computer, but this was never tried. Adding may be direct or may require some more work. In any case, documentation must be added for this.

Is this project officially dead?

Hi,
There has not been much activity for the last two years because most of the contributors have been doing other projects.
It is however not officially dead, and may start over again if people start getting interested in contributing.
Are you interested in \Blue on Windows? I have no windows at home to test but as far as I remember the problem was to port the service manager for Windows.

I wanted to try it.
I thought it was an online service.
But the demo link is dead.


Undocumented feature: filter records from stream by returning None from post_process()

Migrated from GitLab: https://gitlab.com/meltano/sdk/-/issues/233
Originally created by @aaronsteers on 2021-10-07 19:06:50

Per this slack thread, we should better document this capability. At minimum, this should be in the docstring, which then populates the reference docs.

View 5 previous comments from the original issue on GitLab


Add shantanu's file

Layout slide added as second slide. Created a branch, created a file, made a commit and opened a pull request

Finalized all  tasks


The CSRF token has expired

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

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

Superset version
0.23.3
Expected results
Create a dashboard, set it to auto refresh every 1 munute,
Actual results
After about 1 hour get error at UI
Unexpected error.
Check the sueprset server log find this
flask_wtf.csrf:The CSRF token has expired
Steps to reproduce

You can set WTF_CSRF_TIME_LIMIT= None in your superset_config.py for a never-expiring token as documented here: http://flask-wtf.readthedocs.io/en/stable/config.html?highlight=csrf


Set up continuous integration

Bottled Water should have a continuous integration build so we have centralised, automated checks that:

master always builds
the Dockerfile always builds (to catch things like #48)
pull requests merge and build cleanly against the latest master

This would also let us do things like:

run the automated test suite, once we have one :) (#3)
build distribution packages (#6)
test against various versions of our dependencies (e.g. libpq-dev, librdkafka) to catch regressions and incompatibilities.

I've just merged PR #72 which added a test suite and Travis configuration for Bottled Water.  The Travis setup builds Bottled Water (as well as Avro, librdkafka etc) via Docker, then runs the test suite against the built Docker images.  It should be relatively straightforward to extend it to build distribution packages, or run against different dependency versions.
I don't have admin rights to this repo, so I can't turn on Travis builds of confluentinc/master.  Could I interest somebody with admin rights in doing that?  /cc @ept @ewencp @gwenshap  Turning on the Travis build can be done on Confluent's Travis profile page; that should also enable us to use Travis to validate pull requests going forward.

@samstokes I think I just gave you admin rights, can you try now?

Epic, thanks @ept !  I've enabled the Travis build, so subsequent PRs and pushes to master should get automatically tested.


Fix error handling when fail to get auth token

When the ApiClient tries to upload policies and fails to get the Bearer auth token, the thrown error won't have response because the actual request for uploading the policy hasn't been sent. The original logic will cause an error when accessing err.response.data.
Fix the error handling so that when there's no response, use the original error message as the wrapped error message.

Good catch

Published 1.0.6 on npm, i'll add github actions sometime this week. @hanshsieh thanks!


chore(deps): upgrade marked to 1.2.9

Related Ticket(s)
#7931
Description
This upgrades the marked packaged to v1.2.9.
Changelog
New

marked upgraded to v1.2.9
add smartypants option to marked, allowing replacement of normal quotes, dashes, etc. to their 'smart' versions

Changed

update React and web components Storybook markdown to fix indentation. As of >=1.2.3, lists were updated to match the CommonMark spec where sub-item lists must be indented to the column of at least the first non-whitespace character (see example), e.g.:
1. list 1
   - list item a
     - list item a.a

@kennylam thank you for jumping on this! Are the diffs in Percy intentional? I saw that you also updated the example content too.

Are the diffs in Percy intentional?

@jeffchew Yep! The diffs are all the changes from normal to smart quotes, now that the smartypants option has been enabled.

@kennylam looking at one of the downstream diffs, looks like the card section carousel got affected, wondering if this would have caused this change somehow?

wondering if this would have caused this change somehow?

No, nothing from this PR would make those layout changes. Perhaps something related to #7871?

wondering if this would have caused this change somehow?

No, nothing from this PR would make those layout changes. Perhaps something related to #7871?

Yeah possibly, I was in the middle of reviewing another PR and noticed the same thing happening. Going to approve this one!

@kennylam a request was made by @oliviaflory , can you take a look?

@oliviaflory Good catch! Should be fixed now.


Add missing line to quickstart doc; fixes #646

Merged in master and will be redeployed soon. Thanks!


Ability to define events as produced/consumed

Signed-off-by: Tihomir Surdilovic<EMAIL_ADDRESS>Many thanks for submitting your Pull Request :heart:!
Please specify parts this PR updates:

[x] Specification
[x] Examples
[x] Schema
[x] Roadmap
[ ] Use Cases
[ ] Community
[ ] Website
[ ] SDK
[ ] TCK
[ ] Other

What this PR does / why we need it:
This PR adresses Issue: https://github.com/cncf/wg-serverless-workflow/issues/29
It allows users to define an event definition as "produced" or "consumed" with a new "kind" property of the event definition.
If kind is "produce" the "source" property of the event definition is not required. Implementations
must set this property of the CE when the event is produced.
Special notes for reviewers:
Additional information (if needed):

@manuelstein can you please take a look? Thanks.

fixed typos
consume->consumed
produce->produced

@manuelstein yes i think the cases where the same event type is produced and consumed might be very small in which case defining them separately (one consumed and one produced) would be ok.
I understand your point regarding absence of the source param to determine produced / consumed. It would certainly make the model shorter, but IMO less deterministic, as it could clash with user mistakes during modeling for example. For example if I am to write lets say a visualization tool or a linter for this scenario I would not know if user forgot to add the source or its on purpose in order to make it a "produced" one. WDYT?

rebased

rebased

rebased

@manuelstein mind reviewing again after latest comments? thanks.

rebased

@manuelstein yes i think the cases where the same event type is produced and consumed might be very small in which case defining them separately (one consumed and one produced) would be ok.
I understand your point regarding absence of the source param to determine produced / consumed. It would certainly make the model shorter, but IMO less deterministic, as it could clash with user mistakes during modeling for example. For example if I am to write lets say a visualization tool or a linter for this scenario I would not know if user forgot to add the source or its on purpose in order to make it a "produced" one. WDYT?

Sorry, just added my thumbs up to your comment. Fair enough, so LGTM


vulkan-worker Android: fail gracefully and warn user when storage permission is not set

Fixed by #189


Can't the performance of the algorithm be reproduced on curvelanes?

I only got poor performance with F1-measure 82.90% by using the official code provided here (curvelanes small), which is far worse than the checkpoint provided with F1-measure 85.09%. Please help me answer why.

@Jinming-Su  I also get the similar F1 and cannot reproduce the result.
{'f1_measure': 0.8287583304495372, 'precision': 0.8762067744600496, 'recall': 0.7861847678752966, 'pr_num': 84730, 'gt_num': 94432

@ztjsw @Jinming-Su When I reproduce in curvelanes, I first changed the directory, and then entered the command
”python tools/condlanenet/curvelanes/test_curvelanes.py configs/condlanenet/curvelanes/curvelanes_small_test.py ./weight_path/curvelanes/curvelanes_small.pth --show“
then I encountered a problem

. I would like to ask you how to solve it. If you can answer, I will be very grateful


How to search with not sensitive case

I use this library to search query but i don't know how to search with not sensitive case. Please help me

show me a example first ?


chore(ci): Update builder to 24.04

Thank you for contributing to the Universal Blue project!
Please read the Contributor's Guide before submitting a pull request.

As long as everything builds with a checkmark, I'm happy to approve this. Unless there is a reason it shouldn't be due to kernel skew.