Failing to package due to NPM 5.3.0

[x ] I have read the contribution documentation for this project.
[x ] I agree to follow the code of conduct that this project follows, as appropriate.
[x ] I have searched the issue tracker for an issue that matches the one I want to file, without success.

After creating a new project with electron-forge init, or even on an older project where the package command used to work, the electron-forge package command now fails.

Console output when you run electron-forge with the environment variable DEBUG=electron-forge:*. (Instructions on how to do so here). Please include the stack trace if one exists.
WARNING: DEBUG environment variable detected.  Progress indicators will be sent over electron-forge:lifecycle
WARNING: DEBUG environment variable detected.  Progress indicators will be sent over electron-forge:lifecycle

An unhandled rejection has occurred inside Forge:
Command failed: npm prune --production
npm WARN<EMAIL_ADDRESS>No repository field.

npm ERR! May not delete: C:\Users\robertf\AppData\Local\Temp\electron-packager\win32-x64\mbl-agent-win32-x64\resources\app\node_modules\.bin

npm ERR!      C:\Users\robertf\AppData\Roaming\npm-cache\_logs\2017-07-19T09_23_13_479Z-debug.log

Error: Command failed: npm prune --production
npm WARN<EMAIL_ADDRESS>No repository field.

npm ERR! May not delete: C:\Users\robertf\AppData\Local\Temp\electron-packager\win32-x64\mbl-agent-win32-x64\resources\app\node_modules\.bin

npm ERR!      C:\Users\robertf\AppData\Roaming\npm-cache\_logs\2017-07-19T09_23_13_479Z-debug.log

    at ChildProcess.exithandler (child_process.js:270:12)
    at emitTwo (events.js:125:13)
    at ChildProcess.emit (events.js:213:7)
    at maybeClose (internal/child_process.js:897:16)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:208:5)

What command line arguments are you passing?
electron-forge package

What does your config.forge data in package.json look like?
"config": {
        "forge": {
            "make_targets": {
                "win32": [
                    "squirrel"
                ],
                "darwin": [
                    "zip"
                ],
                "linux": [
                    "deb",
                    "rpm"
                ]
            },
            "electronPackagerConfig": {
                "asar": false,
                "dir": "./src",
                "platform": [
                    "linux",
                    "win32"
                ],
                "arch": [
                    "ia32",
                    "x64"
                ],
                "ignore": [
                    ".vscode",
                    "idea"
                ]
            },
            "electronWinstallerConfig": {
                "name": "mbl_agent"
            },
            "electronInstallerDebian": {},
            "electronInstallerRedhat": {},
            "github_repository": {
                "owner": "",
                "name": ""
            },
            "windowsStoreConfig": {
                "packageName": "",
                "name": "mblagent"
            }
        }
    }

I've tried deleting the mentioned folder by hand, but it doesn't change the error. It seems to just fail on the npm prune --production line.
I'm at a complete loss as to what could have changed.

What version of NPM are you using?

5.3.0

For ease of others landing here, this was fixed in npm 5.4.0 so just update your npm with:
npm update -g npm

And you should be good to go.

If you're in Windows, upgrade NPM by running from a Admin PowerShell:
Set-ExecutionPolicy Unrestricted -Scope CurrentUser -Force
npm install -g npm-windows-upgrade
npm-windows-upgrade


Improve functions to report JUnit XML output

We would like to have it same for WA and CF. Here is a link for inspiration:
https://www.ibm.com/support/knowledgecenter/en/SSUFAU_1.0.0/com.ibm.rsar.analysis.codereview.cobol.doc/topics/cac_useresults_junit.html

@ondrejtomas this just got a priority bump, because I discovered that current workaround is ignoring occasional problems / test errors altogether

Merged.


add support for peak regions for ASOC analysis

ASOC analysis needs additional BED file for each sample that indicates peak regions found in each sample
modify prepare_counts-rna.r and create a version (prepare_counts-atac.r?) that finds overlaps with peak regions instead of genes  and reads in BED files instead of GTF file

just attaching an excerpt from our Slack convo about this for the sake of archiving things:

yeah, so in this this little section, we use findOverlaps to intersect the chr:start-end of the SNPs with the chr:start-end of the genes in the GTF file
so instead, we'd probably want to do the same thing but with a bed file instead of a GTF file (edited)
and here's where the GTF gets read into a Granges object
so we would probably just want to read the BED file into a Granges object, instead

update 4/6:
see this pr and this pr for preliminary work on this
it's still unclear as to whether the new config options I added will work with rna-seq, so that's why I'm not marking this as completed yet
we need a test dataset to try things out


v012.0 2023-09-14 firmware seems to just boot loop on Mayhem 1.1 board

So when I try the v0.12.0 2023-09-14 firmware, the ESP32-CAM just will stay in a boot loop.  Resetting, disconnecting, different micro sd card.  Nothing will allow it to work properly.  If I use the "esp32cam_marauder.ino.bin", from the history files (back in March), I can get the older version to at least boot properly.
Interestingly, I can get the version 12 to load with the "vanilla" marauder flash (from and using FZEasyMarauderFlash) to boot just fine.  But it will not find the micro sd card through the ESP32-CAM -- but other functions that do not need the sd card work fine using the unleashed version [ESP32] Wifi Marauder app.
Any ideas why I cannot get your mayhem v1.1 hardware to flash with your files from your current webinstaller?

Can you see what the boot loop is saying? connect it via serial in Arduino IDE for example and 115200 or 230400. Paste here

Sure:

ELF file SHA256: f36ac39c7b69d770
E (520) esp_core_dump_flash: Core dump flash config is corrupted! CRC=0x7bd5c66f instead of 0x0
Rebooting...
ets Jul 29 2019 12:21:46
rst:0xc (SW_CPU_RESET),boot:0x13 (SPI_FAST_FLASH_BOOT)
configsip: 0, SPIWP:0xee
clk_drv:0x00,q_drv:0x00,d_drv:0x00,cs0_drv:0x00,hd_drv:0x00,wp_drv:0x00
mode:DIO, clock div:2
load:0x3fff0030,len:1344
load:0x40078000,len:13964
load:0x40080400,len:3600
entry 0x400805f0
assert failed: do_core_init startup.c:328 (flash_ret == ESP_OK)
Backtrace: 0x40083da1:0x3ffe3ab0 0x4008effd:0x3ffe3ad0 0x400956c9:0x3ffe3af0 0x4010c0e2:0x3ffe3c20 0x400834b5:0x3ffe3c50 0x40079316:0x3ffe3c90 |<-CORRUPTED

Thanks!

Strange, based on
E (520) esp_core_dump_flash: Core dump flash config is corrupted! CRC=0x7bd5c66f instead of 0x0
Rebooting...

I would say the browser failed to download the files and flashed something wrong. If you clear the browser cache and flash again I assume it would work. I have flashed many with the newest version without issues.

Switched to a windows 11 machine (was using MacOS before), same issue on a clean browser.   This is what I am seeing.

I havent seen that before, I would try to upload the firmware using arduino directly and using the "Erase all flash" option in Tools while uploading it. Otherwise the board might be faulty

@rlaltrello have you figure out more about this boot loop?

Sorry for delay, life happened.  I have wiped the flash with the ESP tool and tried again with the same results.  I guess it could simply be a bad board.

I will close this for now since is not replicable, but write back if you find out something else.


Added a function to extend the options used in the initial connection

And synced src/ and release/ .js files
For #45 :-)

@davidwdan if accepted, could you push a new release?

@bondt I'm not clear on what the use case is for this?  When would you need to change the connection options after the initial setup?

@davidwdan I am using wampcra for authentication. To allow the onchallenge handler to sign the request with the user's password, I am setting the onchallenge after the input.
Another use case is written in #45.
And finally I don't see why angular-wamp should limit autobahn's connection options to be set and managed freely.

I'm not a big fan of being able to change the options after the service has been created. It's confusing and allowing the user to open and close the connection while changing the options is a separation of concerns issue.
If anything we should make it easier to create a new instance of the service without having to set options at bootstrap.

I agree with you on both terms, though I think the developer should be capable to managing this. :-)


Demo page is down or removed.

Link from main page returns 404

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/10011290-demo-page-is-down-or-removed?utm_campaign=plugin&utm_content=tracker%2F332399&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F332399&utm_medium=issues&utm_source=github).

Thanks for the note. Link fixed.


Logged Exception Displaying Incorrectly

Hello. I am logging some exceptions on failure and noticed that the Assert values (Expected and Actual) do not output the actually values. It reads like this:
"Assert.AreEqual failed. Expected:. Actual:."
There is indeed data there. I printed the data to the console after logging the exception and the output is correct. Any idea here? This is in C#.

When I edit the index.hmtl the values not shown are surrounded by <>.

I used string.Replace to get rid of any "<" and ">" as a temporary solution.


add iteration fixes of _.keys to _.isEmpty

This is an attempt to fix #1983
I tried reusing parts of _.keys that made sense to detect if an object was truly empty in IE < 9

Is there anything left to do here?


refactor to create Money type

Preparation to handle recipe ingredients which cost karma.
The use of u32 instead of i32 might be controversial, or the removal of Ratio's when dealing with discrete counts. However, both may have been chosen for convenience.
Also fixes the test cases which I've broken w/other commits.
Added sales cost into the crafting total as well in the process as an incidental, not worth splitting out.

Broke the tests w/the rebase. Fixing.


Problem with hex-encode/hex-decode with "bigger" streams

Hi there,
Maybe it's not a bug or just undocumented or it has to do with linux STDIN-buffering
I have following problem with hex-encode and hex-decode using pipes.
user@host:~/tests$ sttr version
0.2.21

## OK case
## 31 kByte File - everything is ok

user@host:~/tests$ dd if=/dev/random of=random_file.bin bs=1k count=31
31+0 records in
31+0 records out
31744 bytes (32 kB, 31 KiB) copied, 0.00068615 s, 46.3 MB/s

user@host:~/tests$ sttr hex-encode random_file.bin | sttr hex-decode > copy.bin

## Error case
## 32 kByte File - error occurs

user@host:~/tests$ dd if=/dev/random of=random_file.bin bs=1k count=32
32+0 records in
32+0 records out
32768 bytes (33 kB, 32 KiB) copied, 0.00306848 s, 10.7 MB/s

user@host:~/tests$ sttr hex-encode random_file.bin | sttr hex-decode > copy.bin
Error: encoding/hex: invalid byte: U+000A
Usage:
  sttr hex-decode [string] [flags]

Aliases:
  hex-decode, hex-dec, hexadecimal-decode

Flags:
  -h, --help   help for hex-decode

## same file decode using a temp-file - no error

user@host:~/tests$ sttr hex-encode random_file.bin > temp.hex
user@host:~/tests$ sttr hex-decode temp.hex > copy.bin

Okay, I'm able to reproduce one of the cases, looks like something with STDIN-buffering I will check this more !!

tbh  I don't know much about golang. so I don't have a local build pipeline.
Maybe the 64kB limit (hexed-size) is a limit when reading by line, which is not necessary processing hex-streams.
So I think somewhere here is the problem ...
https://github.com/abhimanyu003/sttr/blob/a7d35c1e21c56ac3f6f765c467bd442bd450958e/cmd/processor_hex-decode.go#L29
should be replaced by something like this:
in = io.ReadAll(os.Stdin)

just a thought ...


Fix uncaught error on the Home page which relates to POIs

Problem Description
There seems to be an uncaught error in the JS file that the home page uses to carry out the associated functionalities associated with the home page. This error relates to the table being generated to show POIs from a certain region, where the variable being used to hold the list of POIs is not a JavaScript array, and thus '.forEach' cannot be used on this array.
Analysis of error and possible fix
The variable holding the list of POIs is actually a JSON array, which usually requires a regular for loop to loop through all elements in the array. Either a for loop should be used, or if the results returned from MongoDB is actually a JavaScript array, then there is no need to convert it to JSON when a response is received from the database.
Screenshots (of issue)

(when the data is received back from the database)

(where the error occurs)
Any further comments
There is not much else to add, except that this issue shouldn't be too difficult to fix, please make a branch/make a pull request for this issue. Thanks.

This is caused by the NGINX proxy issue, the API route sometimes returns an error instead of an array.

However, there should be client side error handling for all API calls to ensure that an appropriate error is displayed if the API sends back an error.

changes to the APIs makes this irrelevant

changes to the APIs makes this irrelevant


Android client

Question
While testing the ANDROID client with new values I can still not get the 720p format under the streamingData json as earlier? If I add the X-GOOG-API-FORMAT-VERSION header the request does not go through.
What am I missing here?
Pls. check the request below:
Other details
https://restninja.io/share/b099118328a785e3d04884d4334a3338ee3b3d20/0
Checklist

[X] I am running the latest version.
[X] I checked the documentation and found no answer.
[X] I have searched the existing issues and made sure this is not a duplicate.
[X] I have provided sufficient information.

It was a client version issue :)

Your params are also wrong by the way.

Yes, figured that and I think that was the main reason. Thanks :)


TokenTextSplitter creates document with different id than it's input document

Bug description
when giving a document to TokenTextSplitter after the split the resulting documents have a different id than the input. in a pipeline scenario this is completely unexpected.
Environment
Kotlin + Java 21
Spring Boot 3.2.1
Spring AI 1.0.0-M1 (Also on 1.0.0-SNAPSHOT)
Steps to reproduce
create a document with specific id, pass it to tokenTextSplitter, the returned document will have a different id.
Expected behavior
document id should not change in textSplitter or even any DocumentTransformer for that matter unless explicitly done so.
Minimal Complete Reproducible example
val input = Document("123", "content", emptyMap())
val output = TokenTextSplitter().split(input).first()
assert(input.id == output.id)

As the TokenTextSplitter can return multiple Documents for a given document the id would no longer be unique.
If you are wanting to keep the relationship between the parent document and child chunks, we could update the Metadata of the returned Documents to contain a parent document id for parent document retrieval?

I don't think the uniqueness of the ids of the returned document should not
be a concern of TextSplitter.
as for the parent id in metadata, again feels too much for such simple
class, assuming there is a parent child relationship and modifying the
metadata all without the user not knowing seems too much ambiguity for this
api (especially that metadata get included by default in embedding as I
have found out recently which also seems like should not be the case)
I think it should not change the ids unless explicitly stated.
On Tue, Aug 6, 2024 at 10:37 AM David @.***> wrote:

As the TokenTextSplitter can return multiple Documents for a given
document the id would no longer be unique.
If you are wanting to keep the relationship between the parent document
and child chunks, we could update the Metadata of the returned Documents to
contain a parent document id for parent document retrieval?
—
Reply to this email directly, view it on GitHub
https://github.com/spring-projects/spring-ai/issues/1167#issuecomment-2270706067,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKYUVY5IOA5P6XKVQ7KE4BLZQCDOFAVCNFSM6AAAAABMAC54JCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENZQG4YDMMBWG4
.
You are receiving this because you authored the thread.Message ID:
@.***>

@hamedmonji , I think its fair to have separate id for the splitted documents as your input document will be splitted into multiple output documents and it wont be good to keep the same ids for each output document.
However, do you think an additional field  to map the output id to input id would be good?

@Devansh-Rastogi I don't see any reason why the outputs can't have the same ids, having the same ids even more implies that the documents are sub parts of the original documents, it is not really an issue of losing the id, as in many cases the use case would be something like
documents.map( doc -> TokenTextSplitter().split(doc) )
so i can easily put the ids back in but to me it was strange that the ids are changed, it was unexpected and in fact i realized it in debugging an issue. i expect TokenTextSplitter to only split a document to multiple documents and do nothing else which is why I don't think we can say it's fair to have separate id because it really is not within the boundary of the TokenTextSplitter to be concerned about weather id is unique and therefore it should keep that uniqueness.
as a side note I think defaults like this can become the root of somewhat hard to find bugs, another issue i faced was after splitting my document with TokenTextSplitter and having the chunk size set to maximum length allowed for open ai's ada_002 model, I would get an error saying I have exceeded the maximum size for the model, after debugging I realized EmbeddingModel by default includes the metadata as well as the document content for embedding which caused the model size overflow.

@hamedmonji cool, make sense totally, i have found the issue actually, the tokensplitter doesnt even take a look at id and create a fresh set of documents and thats why the id is been different

#1197

honestly, I can see also another way, I came to the same issue as @hamedmonji  , maybe a predictable id generating with just appending subdocument id. i.e. I have document id0 that would be split into three documents and ids would be id00 id01 id02 or some similar formatting

My perspective is that when you split the large document, for example the text from an entire PDF, you get new documents instances, so they would have new id's.  For tracking purposes, the new documents can have a reference back to the original document from which they came if you want to somehow link them back togther.  i don't quite follow the logic of having sequential IDs or giving them structure.
As far as the surprise to have metadata added, it is true that it is a big of an art to split up documents and metadata is cruical to have so that you can show the user what information was used in a RAG scenario to supply an answer.  Not having metadata means all the entries in the vector store are impossible to distinguish from each other, for example one can't use the query languages (and our portable filter expression syntax) to help the search by picking a subset of all the entries.
In short, I'd propose keeping the linkage and making the docs more clear that some metadata will be added and how to customize it.

To me id is what "semantically" identifies a document, as in if I had a PDF document as you said and transformed it into lets say a CSV document, having the document have the same id is what makes this new CSV document semantically be the same pdf document even though the format or even the content may have changed.
However I can also see your perspective, as I said this was an expectation from my perspective as a user, It could be that it is just me and other users mental model thought the id would change too.
Either way I do think it is good to have something if not the original id, to allow to for recognizing a document even after many transformations.
For the metadata, I did not meant to not include metadata, I meant the EmbeddingModel includes the metadata a document has in the call to the embedding service by default which again from my perspective was not obvious. A document has a content field which in my opinion strongly implies it is what is sent to the embedding service.
embeddingModel.embed(document)
I think this does not say the metadata for that document will also be sent to the embedding service (also metadata could have more private data in it)
My point for this part was that i think this inclusion of metadata in the embeddingModels embedding process should be opt-in rather than Opt-out.
(I may have used the word semantically wrong, not sure)

The way I see it simply the inability to know deterministically the next id. I quite like the @markpollack way, of referencing back the original document, I would personally even do so we can provide either enum in construct, so we can just choose, random or sequential, or some sort of factory to pass in.


aarch64 wheels support

We are using pypolyline in our python projects. Recently we need to migrate them to arm64 servers. Is it possible for you to provide aarch64 wheels?
If not, can you provide source code on pypi.org for us to build wheels?
If not, then I think the only option for us is to pre-build the wheels. I have been reading the build_wheel.sh. It looks like it also depends on polyline-ffi. And there is not aarch binary for it. Do we also need to build it ourselves? Is this the right way to go?
Any information is appreciated.

You'll have to be patient – I'm trying to get manylinux2014 aarch64 builds of libpolyline working, which should make building the wheels relatively straightforward – but it's taking a while and I don't have much spare time at the moment.

Thanks for your response. Is there any way that I can help you with? Although I actually don't know too much about libpolyline, I do have access to arm64 machines to verify any script for you.
Or if you can give me some directions, I can try to come up with the script.

libpolyline (in the polyline-ffi repo) is built using a tool called cross which executes Rust builds in a Docker container, which is required for manylinux compat. At the moment, the linker step is failing, but I don't know why – probably because the linker or target is specified incorrectly. By all means have a look! https://github.com/urschrei/polyline-ffi/commit/7d407c9ceb51ed0239e774cf1ce0be39459c1471

aarch64 wheels now available on PyPI


Do need update by date

Hi, do i need update query date by date?

Short answer: No.
Long answer: The query string is valid for only 5 minutes, after which it expires. Therefore, you need to paste your query in the query.conf file. When running TimeFarm Bot, respond with "yes" to the generate tokens from ./configs/query.conf prompt. The bot will automatically generate a token from the given queries in the query.conf file, store it in the tokens.conf file, and then remove the query from query.conf.
After that, each time you run the program, it uses the tokens instead of the queries. If you have a new query from another account, simply paste it in the query.conf file, run the program, and confirm the prompt to generate a token. The bot will use the new query to generate and store a new token in tokens.conf alongside the existing token, enabling the bot to run both accounts.

hi bro, token available only 1 day, after that need collect new query


add deprecation notice

Description
Add deprecation notice for Log Analysis and Activity Tracker.
Release required?

[x] No release
[ ] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

@shemau thanks for the review. can you merge it?

/run pipeline

/run pipeline

:tada: This PR is included in version 2.15.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Hyperparameters of MLP part should be changed, if it refers to the paper

I appreciate for your code :)
I want to suggest an issue about hyperparameters.
I think, according to the paper,  hyperparameters of MLP part should be changed.
According to Appendix B of the paper, "mlp_hidden_mults" is multiplied to "input_size",
and "l" is shared embedding dimension
The code should be changed as below. (class TabTransformer() - def init())
[Original code]
input_size = (dim * self.num_categories) + num_continuous
l = input_size // 8

hidden_dimensions = list(map(lambda t: l * t, mlp_hidden_mults))

[Modified code]
input_size = (dim * self.num_categories) + num_continuous
l = dim // 8 # to be used shared embedding

hidden_dimensions = list(map(lambda t: input_size * t, mlp_hidden_mults))

I think it could be very confusing because the author of the paper used two kinds of "l" parameters (size of the input & dimension of shared embedding)
Other person already created issue about shared embedding, so the code should be modified considering this issue too.
https://github.com/lucidrains/tab-transformer-pytorch/issues/12
Please check whether my opinion is correct or not.
Thank you.

Dear lucidrains,
I checked.
Your commit is better than my requested commit for this whole project 👍
Thank you for adressing this issue. 😄


Added sample ipta notebook

Adding the sample IPTA notebook we've been informally using during the busy week.

Accidentally added a change to the reflagger, but it's literally just DE441 (erroneous value) -> DE440 (correct)

(Which is to say, that should not be a problem)


mavlink_mission_item_to_mission_item_indices never cleared

_mission_data.mavlink_mission_item_to_mission_item_indices is never cleared which means we might have more mavlink_mission_item_to_mission_item_indices than we want if we download a list of items smaller than the previous one
https://github.com/mavlink/MAVSDK/blob/8afd4640cebf4fae242a9ff9650d354cebab2f48/src/plugins/mission/mission_impl.cpp#L241

Thanks for the report.

What's needed to fix this? Is it simply a matter of clear()ing the list before we start inserting into it?

yes only that hehe. I'm not working with the last version of MAVSDK and I don't have the time to upgrade for now with my job but yes if you clear it everything is fine !

Okay, if it's just clearing I can try to get a PR for this soon.


installation issue on symphony 2.2.5

Hello
I've tried to install duplicate entry on symphony 2.2.5 but I'm not abele to do that...
I have upload the duplicate entry folder in the extention folder
Than I went to the extension page in the back end but I got a black page
do you have any advice to fix this issue?
Thanks
Daniele

Closing as this is an old issue and there are no problems with install


Loading local libs or plugins

I'd like to be able to deliver a brackets folder to someone else who doesn't have my customer libs and plugins and let the .tern-project specify that these should be resolved from a different location.
Being able to use both project files and remote files would be extremely useful, at the moment, if I want to share a lib or plugin, I need to send my files, and have the other user put them into the extensions folder.
Something along the lines of :
{
"libs": [
"ecma5", "http://www.somehost.com/a_remote_lib", "${project}/a_project_lib"
]
,"plugins": {
"http://www.somehost.com/a_remote_plugin" : {},
"${project}/a_project_plugin" : {}
}
}

@gabrielpage I like the idea. It should be possible to do.  The only thing we will have some restrictions on is around CORS, which Ajax and importScript live by.
Libs would need to change because I am expecting them to be local. Here are both relevant parts of the code:
https://github.com/MiguelCastillo/Brackets-Ternific/blob/master/LocalServer.js#L12
https://github.com/MiguelCastillo/Brackets-Ternific/blob/master/LocalServer.js#L143
So I will need to add a small check to see if you are specifying a protocol or not... If you are, then I will need to build the url differently.  But this is not a hard change.
The same protocol check would need to happen.  Here is the relevant part of the code.
https://github.com/MiguelCastillo/Brackets-Ternific/blob/master/TernWorker.js#L38
Now, I see that you have some MACRO type of project definition - ${project}.  I am not sure what we can do about that.  If we were to put the macro expansion in ternific then you would need to define it in the .tern-project file itself.  I wouldn't want to rely on external data for that.  Or are you implying that project would be automatically expanded out to the current project's directory?
Would you be interested in PR?

For ${project} I thought ternific would just replace it with ProjectManager.getProjectRoot()
This change is pretty trivial with a replace in the settings before you JSON.parse() but I didn't know if there was already a scheme for this.
Maybe the macro is not required, and another protocol specifier could be used project:///dir/somelib
But we'd need some way of being able to identify the lib should be found in the current folder rather than in the extensions folder. The other option would be to try to resolve it locally first, and only if it's not resolved look in the extensions folder, however, I think I'd prefer to be specific as it could be confusing.
For remote files, is there an option where the remote file is downloaded to the project onProjectLoad. This would get around CORS issues (but would mean we may have to overwrite something)

I like the idea of expanding out ${project} to ProjectManager.getProjectRoot().  This can be done right here where we are processing the settings file.  Maybe we can just use lodash or mustache templates, both of which are provided by brackets.
As far as identifying if the file is from the the extension or some other location goes... I am not too sure quite yet.  The question I have for is...  How frequently do you load plugins/libs from your local project?
You really can't get around the CORS problem.  You either load the tern plugin via script tags or the server would need CORS support enabled, or possibly JSONP.  Loading via script tags is less of an option because currently tern is running in a web worker, which does not support script tags.  All files are loaded via importScript, which is affected by CORS.   I am inclined to make a change where CORS is a limitation.

Actually, this is a more appropriate place for handling templating.

ok, I have the ${project} replacement working, let me tidy it up and I'll send a PR for that.
In fact, for extension vs other location, I think this will cover me. I don't see a need to be able to load from a location other than project or extensions.
I'm going to have a go at downloading the remote file directly into the folder. Once it's downloaded I can load using ${project} :)

@gabrielpage yeah that sounds really good!  Thanks for your help on this :)

merged in. thanks @gabrielpage !


Icons not centered

A picture says a thousand words, but in this case please provide some more information (Atom version, tool-bar version, theme, etc...)


[YouTube Downloader] Time can't be set to longer than an hour

Extension
https://www.raycast.com/vimtor/youtube-downloader
Raycast Version
Latest
macOS Version
Sequoia
Description
I can't download for example a 5 second part from 3 hour long video.

Steps To Reproduce

Enter a longer time frame.
Experience a download error.

Current Behaviour
The video fails to download.
Expected Behaviour
The video should download and only contain the set time.

@vimtor @pernielsentikaer @darmiel @ridemountainpig @remorses

@raycastbot keep this issue open


copy/paste issue in code with key event

The code here: https://github.com/raylib-extras/rlImGui/blob/fcb54869a90d84c6a6306a43127214a767d77428/rlImGui.cpp#L165
and here: https://github.com/raylib-extras/rlImGui/blob/fcb54869a90d84c6a6306a43127214a767d77428/rlImGui.cpp#L175
clearly contains a copy/paste issue:
// line 163
	bool shiftDown = rlImGuiIsShiftDown();
	if (shiftDown != LastShiftPressed)
		io.AddKeyEvent(ImGuiMod_Shift, ctrlDown); // COPY/PASTE ISSUE
	LastShiftPressed = shiftDown;

// line 173
	bool superDown = rlImGuiIsSuperDown();
	if (superDown != LastSuperPressed)
		io.AddKeyEvent(ImGuiMod_Super, ctrlDown); // COPY/PASTE ISSUE
	LastSuperPressed = superDown;

Can you submit a PR for this fix please?

I just told you exactly what is wrong. The fix is obvious. It will be a lot easier for you to simply fix it. I am not sure why you closed the issue before it is actually fixed.


jenkins.sh needs default for COPY_REFERENCE_FILE_LOG

Faced this issue while adding a different entrypoint.
If COPY_REFERENCE_FILE_LOG environment variable is missing, it should assume a sane default i.e JENKINS_HOME/copy_reference_file.log

@dsathyan I just retriggered a build for this PR, because it was failing in CI in various cases. Can you please look into it if this keeps failing to see if this is genuine and possibly related to this change?
Thanks

@batmat the reason for it failing is due to some connection timing out. It had passed within around an hour of creating this request.

Build is now green, let's move on.
Thanks!


Cleanup unneeded return statements identified by clippy

It is more idiomatic in Rust to omit the return keyword when possible.

Coverage decreased (-0.5%) to 76.542% when pulling 73602f888e44b6540a63be2b7b9e7b1db7c0976e on shssoichiro:clippy-unneeded-return into 75a06568cc65b8b80857654c6b54a59c6fa15af2 on xiph:master.


Export QR code cannot be read with dark theme enabled

Checklist

[X] I have used the search function to see if someone else has already submitted the same bug report.
[X] I will describe the problem with as much detail as possible.

App
Thunderbird for Android
App version
8.0
Where did you get the app from?
Google Play
Android version
12
Device model
No response
Steps to reproduce

Activate dark theme in Thunderbird Desktop
Follow steps to import settings from Thunderbird Desktop: https://support.mozilla.org/de/kb/thunderbird-android-import#w_export-thunderbird-desktop-settings
Try to scan the QR code (whose colors are inverted)

Expected behavior
Thunderbird for Android should be able to detect and read the QR code even with inverted colors.
(OR Thunderbird Desktop should not invert the QR codes colors with the dark theme enabled)
Actual behavior
Thunderbird for Android doesn't detect the QR code.
Logs
No response

This sounds like a desktop issue, I see there is https://bugzilla.mozilla.org/show_bug.cgi?id=1928234 which is similar. I'll let them know about this issue so they can fix it there.

@kaivol would you mind chiming in on the linked bug with steps to reproduce?


Add EUR to currencies

Currently, two currencies are available : USDT & DATA
Adding EUR would be nice for EU users.

Hi,
Can I take up this issue if possible?

Hi :) Sure ! Have you work with SvelteKit before ?


auth0-cli: init at 0.11.7

Description of changes
Add auth0-cli (https://github.com/auth0/auth0-cli)
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@jonringer Thanks so much for nix-template, it helped so much with this. I was able to get this CLI going quickly so I could use it. Then I figured I'd might-as-well contribute it. I imagine that's the whole point of nix-template haha
I think this PR would be good to go if I could figure out how to best run make test and whether it should be in passthru.tests. Any advice?

Result of nixpkgs-review pr 171352 run on x86_64-linux 1

1 package built:

auth0-cli

Closed as #189335 has been merged


Use CFF file as a source for DOI and authors+ORCID

As CFF (citation file format) is a best practice recommended by fair-software.eu, we should aim to use this meta-data captured in a CFF file to further enhance Gimie output. For instance, extracting DOI (into schema:identifier).
Note: Depends on #97
Objective: Add parser for CFF files to extract DOI
Requirements:

[ ] New gimie.parsers.CffParser follows the gimie.parsers.Parser interface
[ ]  is added to gimie.parsers.PARSERS
[ ] Tests verify that parser works as expected.

Next steps:

Discuss if other properties should be extracted 's  (e.g. authors  in some properties such as m4di:orcidId , schema:author)

Not really gimie's domain - we can get the information associated with a DOI using different services. Gimie is for repositories, not for publications.

closed by #107


[Feature] Longer context length

Motivation
Hi, nice job.
Due to the MHA kernel written by NVIDIA: https://github.com/InternLM/lmdeploy/blob/2dec28aeafd159879b4f437de25c10f7d7139679/src/turbomind/kernels/decoder_masked_multihead_attention/decoder_masked_multihead_attention_128.cu#L39
Dynamic Shared memory size is up to 162kb on A100 which is about to encapsulate context of 28k long.
Will you consider improve this kernel to further increase context length.
Related resources
No response
Additional context
No response

MHA kernel supports infinite context length will be released soon.

It has been supported since v0.1.0a0


⚠ Quirk Club Argentina (API endpoint) has degraded performance

In 93783ed, Quirk Club Argentina (API endpoint) (https://us-central1-quirkclub-dev.cloudfunctions.net/api/check/api) experienced degraded performance:

HTTP code: 200
Response time: 9600 ms

Resolved: Quirk Club Argentina (API endpoint) performance has improved in 5f71464.


Debug stack not removed on sst remove

Fixed in https://github.com/serverless-stack/serverless-stack/pull/42


refactor(app): Add and Display USB port info for Modules

Overview
This PR wraps up the frontend changes for Multiples of a Module. See design

closes #7199
Changelog

add module USB port info in redux
display usb port in Robot > Modules, and combine modules connected to the same hub in the same card
module deck map should now display usb info for each attached module if your robot server is up-to-date. The same info should also be displayed on the calibrate page side panel
module control on the run page now only allows you to control modules that are required in the protocol
you can now see the slots the modules are in for the module control section to reduce confusion when multiples of a module are required in a protocol
update CollapsibleItem and Module in the Components Library

Review requests
Robot > Modules Page:

[ ] Module controls are working as expected
[ ] Module info (engage height/temperature data) are correct
[ ] USB info are correct (You can check against the http GET /modules response from the robot)
[ ] (out-of-sync robot server) you shouldn't see the usb info but everything else stated above should be correct

Calibrate Page after uploading a protocol requiring multiples of a module:

[ ] deck map should show the correct number of modules attached
[ ] correct USB info and should update when you switch ports
[ ] for multiples of the same module, the module connected to the left-most USB port should be assigned to required module with the lower deck slot number
[ ] (out-of-sync robot server) you shouldn't see the usb info and you can't expect MoaM to work correct, so we prompt user to update their robot with a tooltip on the side panel

Run Page:

[ ] only modules required in the protocol will be shown
[ ] you should be able to see the slot number associated with each module, and should match the info on the Calibrate page
[ ] Module controls are working as expected
[ ] Module info are correct

Lastly, we should upload a protocol with multiples of a module and try different USB port combos and make sure the modules are in fact linked to the correct ones in the protocol.
Risk assessment
Medium. Modules info on the Calibrate page is now gated behind a feature flag. However, this PR does affect the way the Modules Page and the Run Page look so we should smoke test everything. Also I've made some changes to the component library, so we should also make sure PD is not affected by them.

hey @nusrat813 could you share screenshots when you have a moment so I can take a look?


[semantic-arc] When compiing with -enable-sil-ownership, run the Owne…

[semantic-arc] When compiing with -enable-sil-ownership, run the OwnershipModelEliminator right after SILGen and verify with SIL Ownership Enabled.
rdar://28685236

@swift-ci Please smoke test and merge


Missing SideNav NPM Package

I noticed the SideNav package within the @react-spectrum storybook project. Yet, I'm unable to find the package within react-spectrum NPM. It looks like this package has been in development and closed for a while (refer to #1095).
What is the best way to access this package and is there a way to import from NPM/Yarn?

Just reference, this issue moved to discussions (https://github.com/adobe/react-spectrum/discussions/2753)


Tube operator generates some twists at 45+n*90 degree intervals

I was applying the Tube operator to the example.lines file at http://visitusers.org/index.php?title=Lines3D and I found that it would insert twisted geometry at spans that cross 45+n*90 degrees (45, 135, 225, ...).
-----------------------REDMINE MIGRATION-----------------------
This ticket was migrated from Redmine. As such, not all
information was able to be captured in the transition. Below is
a complete record of the original redmine ticket.
Ticket number: 1219
Status: Pending
Project: VisIt
Tracker: Bug
Priority: Normal
Subject: Tube operator generates some twists at 45+n90 degree intervals
Assigned to: -
Category: -
Target version: -
Author: Brad Whitlock
Start: 10/25/2012
Due date:
% Done: 0%
Estimated time:
Created: 10/25/2012 12:24 pm
Updated: 11/06/2012 01:27 pm
Likelihood: 2 - Rare
Severity: 2 - Minor Irritation
Found in version: 2.5.2
Impact:
Expected Use:
OS: All
Support Group: Any
Description:
I was applying the Tube operator to the example.lines file at http://visitusers.org/index.php?title=Lines3D and I found that it would insert twisted geometry at spans that cross 45+n90 degrees (45, 135, 225, ...).
Comments:


[i31] - copy over SetParentDefaultThumbnailJob from LV

ref #31
Copied over and tested the SetParentDefaultThumbnailJob.
This job should set the parent's thumbnail to be it's first child's thumbnail, if a thumbnail doesn't already exist.

LV has a spec for this: https://github.com/scientist-softserv/louisville-hyku/blob/42a0fc837fd9ef4832a9741731672200e0097305/spec/jobs/bulkrax/set_default_parent_thumbnail_job_spec.rb


Added rust version of bogo sort

You were faster the slice changes than I was with the review :) Still,

@Gustorn Pretty sure you're the only active contributor who knows Rust well enough.

Fair enough


Nwb to bids - try fixing tests

This PR is just for fixing the tests in https://github.com/INT-NIT/AnDO/pull/73 and should not be merged, but incorporated in #77 once tests are passing

Hello @JuliaSprenger! Thanks for opening this PR.

In the file ando/tools/generator/nwb2bidsgenerator.py:

Line 37:42: E226 missing whitespace around arithmetic operator
Line 37:52: E226 missing whitespace around arithmetic operator
Line 94:101: E501 line too long (101 > 100 characters)
Line 96:39: E226 missing whitespace around arithmetic operator
Line 97:45: E226 missing whitespace around arithmetic operator
Line 145:101: E501 line too long (101 > 100 characters)
Line 154:32: E226 missing whitespace around arithmetic operator
Line 155:29: E226 missing whitespace around arithmetic operator
Line 159:37: E226 missing whitespace around arithmetic operator
Line 168:38: E226 missing whitespace around arithmetic operator
Line 174:36: E226 missing whitespace around arithmetic operator
Line 178:38: E226 missing whitespace around arithmetic operator
Line 185:39: E226 missing whitespace around arithmetic operator
Line 194:35: E226 missing whitespace around arithmetic operator
Line 194:45: E226 missing whitespace around arithmetic operator
Line 202:20: E225 missing whitespace around operator
Line 208:26: E226 missing whitespace around arithmetic operator
Line 212:41: E127 continuation line over-indented for visual indent

In the file ando/tools/generator/tests/test_nwb2bidsgenerator.py:

Line 11:24: E226 missing whitespace around arithmetic operator
Line 28:9: E722 do not use bare 'except'
Line 29:48: W292 no newline at end of file

Do see the Hitchhiker's guide to code style

Pull Request Test Coverage Report for Build 786494076

211 of 251 (84.06%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 83.774%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

ando/tools/generator/tests/test_nwb2bidsgenerator.py
44
45
97.78%

ando/tools/generator/nwb2bidsgenerator.py
138
146
94.52%

ando/tools/generator/bidsconverter.py
29
60
48.33%

Totals

Change from base Build 773479310:
0.1%

Covered Lines:
666

Relevant Lines:
795

💛  - Coveralls


Code sample is broken

I can get the template to deploy successfully after changing the API version from "2015-03-20" to "2017-03-15-preview" (this was to get past this error: "This oms query language is not supported on current workspace anymore. The query cannot be converted 09:33:14 - to new language. Check query syntax.)"
I can view the enabled alert rule fine in the portal, but when I drill into the details of it I get a broken page on the Azure portal. The weird thing is, there's no error in my ARM template deployment.  Do you have a working code sample for this scenario?

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

ID: 926fc77a-c2ae-4fce-899d-d8feb2729edb
Version Independent ID: dc8fce2e-038e-5ff3-c872-bf736e155ae3
Content: Saved searches and alerts in management solutions
Content Source: articles/monitoring/monitoring-solutions-resources-searches-alerts.md
Service: monitoring
GitHub Login: @bwren
Microsoft Alias: bwren, vinagara

@toddrob This may have changed with the new alert experience.  I'll follow up with PMs.

#in-progress

@bwren thanks and I'm happy to test

Sorry this has taken some time.  We created a work item to rewrite this article based on the new alerts.  I still need to verify some things, but it's expected to be based on this article which should give some guidance there.  https://docs.microsoft.com/en-us/azure/monitoring-and-diagnostics/monitor-alerts-unified-log-template.  I'm going to go ahead and close this issue since we're tracking this separately, but know that it's being worked on.

#please-close

@toddrob We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.

It is the above documentation legit or does it need the redo. I tried to make it work on my log analytics template but it doesnt work at all.


CB-20186 [FedRamp] | Skip rpms on aws gov installation

Originally it was similar but with extra conditions regarding RHEL8 and errors occured. I am not familar with salt but seems there are limitations on complex conditions. That is why I brought it to outside and inside remained the variable equal check as in other places.
I tested it here:
http://build.eng.hortonworks.com/job/cloudbreak-multi-packer-image-builder-salt-v3/207/consoleFull
based on the logs the minifi and request-signer were not installed.


Add coding and decoding for IndexSets

This PR adds support for IndexSets which are stored as an array of ints in firebase
For instance:
let reminderSettings = ReminderSettings()
reminderSettings.time = 12.30
reminderSettings.daysOfTheWeek = IndexSet([1, 2, 5, 6])

would be stored as
{
    "time": 12.30,
    "daysOfTheWeek": [1, 2, 5, 6]
}

Of course!!!
On 15 July 2019 at 15:19:33, Oleksii Dykan<EMAIL_ADDRESS>wrote:

@alickbass requested changes on this pull request.
Thanks a lot for writing this PR! Could you please as well add a tests
case that uses IndexSet in unit tests? This way we will make sure that the
logic is correct 😉
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/alickbass/CodableFirebase/pull/75?email_source=notifications&email_token=AAAU3GFT6GEVO2NJJZG55ALP7SBPLA5CNFSM4IDVYZF2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB6NYYQY#pullrequestreview-261852227,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAU3GHHUZJDELSKR6QBUV3P7SBPLANCNFSM4IDVYZFQ
.

Updated with basic tests that mirror the existing ones for other types. Let me know if you want more.


[Feature]: Automating the authentication for azure_consent_url for Snowflake Storage Integration (Blob) with terraform

Use Cases or Problem Statement
Currently, when creating storage integration on Snowflake for Azure blob storage, there is a manual intervention required to "Accept" the connection with azure_consent_url. The ask is to add a feature in terraform to automate this step, so as to avoid manual intervention of any kind.
https://registry.terraform.io/providers/Snowflake-Labs/snowflake/latest/docs/resources/storage_integration#azure_consent_url
Category
category:resource
Object type(s)
resource:storage_integration
Proposal
Possibly add another parameter (boolean) which can be default value as 'accept' which terraform can use for accepting the grants with azure_consent_url.
How much impact is this issue causing?
Medium
Additional Information
No response
Would you like to implement a fix?

[ ] Yeah, I'll take it 😎

Hi @sfc-gh-amdeshpande 👋
Storage integration is a resource that will be redesigned (see the roadmap). We'll examine this during the rework. We welcome testing and implementation on the provider side by following our contribution guide.


proposal: testing: testing.T.IsNil

Proposal Details
Go Wiki explains that assertions are bad because they either

stop the tests early
omit useful information.

Often, in tests, we need to check the errors on nil. If it's not, it indicates a function failure.
This is a classic way of handling errors in tests.
func TestDo(t *testing.T) {
    out, err := Do()
    if err != nil {
        t.Fatalf("Do error: %s", err)
    }
}

If the error happens there's no point in going further
Errors carry a description of what went wrong.

I propose to add the IsNil method to the common testing type.
It would result in code like this:
func TestDo(t *testing.T) {
    out, err := Do()
    t.IsNil(err)
}

I would also like to point out that the nil or zero value doesn't carry any information and perhaps IsZero method should be added as well.
However, to prevent any possible abuse of assertions, we could limit the parameter type to the error interface.
E.g.
func (c *common) ErrorNil(err error) {
    if !reflect.ValueOf(err).IsNil() {
        c.checkFuzzFn("Fatalf")
        c.log(fmt.Sprintf("error: %s", err))
        c.FailNow()
    }
}

Related Issues

testing: Assert(...error) #7009 (closed)
proposal: testing: add conditional-error helpers to T and B #45650 (closed)
proposal: spec: permit type assertion to nil #11635 (closed)
proposal: testing: verify failure (rather than skip) #28488 (closed)
proposal: testing: add assertion methods #52555 (closed)
proposal: errors: Is resolve nil error problem #40674 (closed)
proposal: assert of errors #32997 (closed)
proposal: testing: Collect test failure logs on the testing.common struct and expose collected logs via a new method on testing.TB #61860

Related Documentation

Go Wiki: Go Code Review Comments > Useful Test Failures

(Emoji vote if this was helpful or unhelpful; more detailed feedback welcome in this discussion.)

I don't like that it encourages checks / tests to fail without context

For context you can find third party implementations of this and more:

NoError
NotNil
NotZero

@seankhliao what about the .IsErrorNil(error)? You would see the name of the test and the error that failed it.

Hi @glossd, I suspect this has already been proposed, discussed, and declined.
Would you mind at least skimming the discussions in the issues linked by the @gabyhelp robot above (e.g., #7009) to help check if that suspicion is true, as well as review this FAQ:
http://golang.org/doc/faq#testing_framework
Also note that in the interests of efficiency and to help avoid re-hashing old discussions, the Go project generally does not reconsider prior proposal decisions unless there is significant new information (as described for example in the "Reconsideration" section here).

@thepudds, the person asked for assertions. I feel this is a special case because the nil/zero value does not carry any information which you can report in the tests.

I wrote my own assertion library, and I don't think this makes sense for the standard library. In my own code, I'm happy to follow my own philosophy about test errors and how they should be reported. For the standard library though, I feel like there should really just be universal, low level tools people can use to build their own systems instead of something that imposes its own opinion on users.

I think this is a dupe of #7009, and for the same reason we encourage error wrapping with details, we encourage failing tests with sufficient context. a test failure with a non nil error like "unexpected eof" is not very informative on it's own.

@seankhliao that Issuer is talking about helper assertion functions and has zero arguments.
The errors should be descriptive by themselves, if it says "unexpected eof" and you don't understand the context, then your function has poor error handling.
t.IsErrorNil(err) would actually enforce writing descriptive errors.

Hi @glossd, did you have a chance to at least skim through all the prior issues that the @gabybot found, as requested above?
Also, would you say that there is overlap with your are proposal and comments like https://github.com/golang/go/issues/45650#issuecomment-823773976:

I think this can be only about err != nil. As mentioned in the original proposal, that's very common in tests. A lot of time you don't care about the error, just that it didn't happen.
I think the message may not be needed, maybe it's ok to just stop at that line?
func (t *testing.T) NoError(err error) {
  if err != nil {
    t.Fatalf("Unexpected error: %v", err)
  }
}

Would you say that the rationale for closing #45650 in https://github.com/golang/go/issues/45650#issuecomment-823773976 for example would apply to your proposal here as well? In other words, even if you view some of the details as being different, does that rationale still apply dispite any differences?

If you find yourself using these five lines often, then it's easy to put them in your own library:
func (t *testing.T) Check(err error, msg string) {
  if err != nil {
    t.Fatalf("%s: %v", msg, err)
  }
}

Personally, I would do something different, to make sure that the arguments to Check and t.Fatalf are different. And others would do different things too, as you've noted. The diversity in helpers is fine. Package testing only needs to provide the shared abstractions on which they all build.

In any event, I do believe your proposal has been discussed before and that Sean is correct to close this.

@thepudds my proposal is about checking values that do not cary any information with IsNil, IsZero methods.
Method checking error on nil seems like something highly desired by other developers, and I agree with them.


DOC: How to configure Prometheus alerts

Document or decide if its still valid:
https://github.com/epiphany-platform/epiphany/blob/cleanup/docs/home/howto/MONITORING.md#how-to-configure-prometheus-alerts

@seriva this issue is to document how to enable Prometheus alerts or how to configure Alertmanager?
Some time ago I've added few basic alerts and documented it: https://github.com/epiphany-platform/epiphany/blob/develop/docs/home/howto/MONITORING.md#how-to-enable-provided-prometheus-rules
not sure if this issue is still valid.

Looking at the history and what was added I think this is done:

When I made this issue I removed some old invalid documentation and made this story to rewrite it which clearly has been done in the last 2 years:)


Enhancement: Android?

Please check for duplicates or similar issues before creating this issue.
What is your suggestion, and why should it be implemented?

Android build?

The Android Build is WIP

Wait so there's an official one?

I have a workflow that I pr in here
https://github.com/YoshiCrafter29/YoshiCrafterEngine/pull/277
But there’s an error on most of the builds
source/Discord.hx:4: characters 8-30 : Type not found : discord_rpc.DiscordRpc
14
source/dev_toolbox/ToolboxHome.hx:5: characters 8-30 : Type not found : discord_rpc.DiscordRpc
15
source/TitleState.hx:71: characters 11-17 : Type not found : Thread
16
source/Main.hx:37: lines 37-315 : ... Defined in this class
17
source/ModSupport.hx:1: characters 8-30 : Type not found : discord_rpc.DiscordRpc
18
source/PlayState.hx:2279: characters 3-11 : Expected }

withheld for now

K


README in Persian/Farsi

added the Persian version (that rhymed!) of the README.md file

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

Thanks for taking the time to contribute this localization pull request. However, we are not able to accept contributions to this historical reference repository at this time.


adjust background_dark accent color for dark mode

Currently 'color.background' and 'color.background_dark' are set to the same
value, when specifiying vim.g.gruvbox_baby_background_color = "dark".
I've adjusted the 'color.background_dark' accent color to be visible
again.

Let's try this and see how it looks


Traditional Chinese translation for README

Added Traditional Chinese translation for README.md

@rom1v
I can review this PR in weekend.

@npes87184 Did you find some time to review it? :wink:
@RaenonX (I'm sorry, it's outdated because I just released v1.15, but it's not blocking for merging it)

@rom1v No problem, I could update it to v1.15 later, probably within 24h.

@npes87184 Did you find some time to review it? :wink:
@RaenonX (I'm sorry, it's outdated because I just released v1.15, but it's not blocking for merging it)

Sorry, I am quite busy recently. I will find time to review it.

Finished translating the README for v1.15 and also reflect the changes after I created the first translation.

@RaenonX
Can you squash the commits for better clear? IMO, the first, third and 4th commits can be squashed.

@npes87184 Squashed the commits (actually all of them because I didn't clone it) and created a PR #1644

@RaenonX Thanks.
However, creating a new PR is not needed. Just git push origin master -f (since this PR uses your master branch), it will update this PR (and you could close the new one).

@rom1v Actually I didn't clone the repo when I was doing the translation actually, but for simplicity, I will clone one then.

Pushed the squashed one.

Complete modifications in all suggestions with little changes. Also seeking better technical terms in CHT if any.

Completed suggested changes above.

Completed suggested changes above.

Merge branch 'master' into master

Please never merge in a PR (rebase instead if you want) :)
For this PR, just push force the previous commit to remove the merge commit. (If it cause trouble for you, don't worry, I'll just take the diff and squash in the end)

@rom1v For some reason, I copy-pasted the README.md from the current master branch, and it always yields Unresolved conflict after I forced push my commits. That merge commit was created by GitHub. Probably because of the CRLF difference between OS? (I am using Windows)

LGTM

Thank you :+1:
I rebased on current master, and moved the translations to the bottom of the file (with a link at the top): 8081e9cc11ac96b4dbe452991865d762f3dec468
rendered: https://github.com/Genymobile/scrcpy/tree/pr1626
If everything is ok for you, I merge into master.

If everything is ok for you, I merge into master.

I merged it, and published a new release (sorry :wink:).
Thank you for the translation (and for the review), it's a lot of work :+1:


doc: Fix typo monoid -> group

Notice all the other documentation comments refer to group and not monoid


Improve user documentation

Update documentation (via READMEs for now) to reflect guidance and details needed for a user to use Adelphi for an end-to-end workflow, with a focus on integrity testing.
Provide quick start type guidance, architectural design details, and any other relevant details.
It should be enough that a new user can find Adelphi, grab onto it, and start using it with minimal friction.

Resolved in #97


UP-4414 handle guest access no cookie support in browser

https://issues.jasig.org/browse/UP-4414
Found that disabling cookies in the browser when user agent is set for Googlebot gets into an infinite HTTP 302 loop.  This change allows for cookies disabled in browser when user-agent is Googlebot or other configured user agent that skips cookie checks to render the guest page, at least with the default data set on my local desktop.  TBD if this actually allows googlebot to work properly.

Found Googlebot's user agent string is not just 'Googlebot'.  Adjusted to regex pattern for user agent comparison.  Merging.


Migrate to swift5.0

Hi @dirtmelon, thanks for creating this PR. I didn't have a chance to take a look at it yet. But CI is failing to compile and thus the tests are not passing. If you can take a look at that it'd be great. Otherwise, it will have to wait until I can take a look at it.
Thanks!

Thanks for reply.
I have checked the CI, and here is the error message:
None of your spec sources contain a spec satisfying the dependencies: `RxSwift (= 5.0.0, ~> 5, ~> 5.0), RxSwift (~> 5.0)`.
You have either:
 * out-of-date source repos which you can update with `pod repo update` or with `pod install --repo-update`.
 * mistyped the name or version.
 * not added the source repo that hosts the Podspec to your Podfile.

I will try to fix that.

@leandromperez done


CSS for the New HTML Layout

Issue: "Apply custom CSS to improve the HTML version's UI (chat box design, buttons, fonts)."
note: do this only if the html file has been created

I would like to work on this issue

can you  assign this to me I will work on it


Updating to 1.2.0 without prior reset

Hi,
I would like to update from 1.0.0 to 1.2.0. The changelog tells me to run sudo ./ambar.py reset. This will remove the data directory and thereby remove (correct me, if I am wrong):

all indexed documents
all registered users (just one, but still)
all configured crawlers

I am wondering whether there is a way to keep any of the above while still updating to the most recent version? Or is 1.2.0 just a breaking change in all dimensions?
Thanks!

Hello, @tim-kiefer ! Unfortunately you need to reset ambar to update to 1.2.0 because we changed ES indexes. I suggest you to wait till the next release (this week) as it requires reset too.


Oracle-java-installer ppa is discontinued

I got this mistake when I tried to build the docker image:
Package oracle-java8-installer is not available, but is referred to by another package.
This may mean that the package is missing, has been obsoleted, or
is only available from another source

E: Package 'oracle-java8-installer' has no installation candidate
The command '/bin/sh -c echo oracle-java8-installer shared/accepted-oracle-license-v1-1 select true | debconf-set-selections &&   add-apt-repository -y ppa:webupd8team/java &&   apt-get update &&   apt-get install -y oracle-java8-installer &&   apt-get install -y dos2unix &&   rm -rf /var/lib/apt/lists/* &&   rm -rf /var/cache/oracle-jdk8-installer' returned a non-zero code: 100

I have found that oracle-java8-instaler ppa is discontinued, due to a recent modification of Oracle licenses. The following can be read on the webupd8 website (http://www.webupd8.org/2012/09/install-oracle-java-8-in-ubuntu-via-ppa.html):

The Oracle JDK License has changed for releases starting April 16, 2019.
The new Oracle Technology Network License Agreement for Oracle Java SE is substantially different from prior Oracle JDK licenses. The new license permits certain uses, such as personal use and development use, at no cost -- but other uses authorized under prior Oracle JDK licenses may no longer be available. Please review the terms carefully before downloading and using this product. An FAQ is available here.
Oracle Java downloads now require logging in to an Oracle account to download Java updates, like the latest Oracle Java 8u211 / Java SE 8u212. Because of this I cannot update the PPA with the latest Java (and the old links were broken by Oracle).
For this reason, THIS PPA IS DISCONTINUED (unless I find some way around this limitation).

Is it possible to find a solution to this problem?
Thank you.

@Python-Trading
ah, yes - Oracle's new licensing is wonderful ..
seems the time is now for much needed refactoring of this Docker file - probably should extend from an OpenJDK  image at this point.
i will look to check on this before week end.
cc: @krumware

Not a bad call, openjdk has a ton of image options. Here is another I have contributed to recently, for reference of a build with similar needs.  https://github.com/carlossg/jenkins-swarm-slave-docker

Unfortunately, IBController does not like "non oracle" java
https://github.com/ib-controller/ib-controller/blob/99410bd0782e1634bf7f71f198e175841e7ebb21/resources/Scripts/IBController.sh#L369-L376
This issue claims that TWS "doesn't work properly with non-oracle java"
could have been true in the past, but at this point in time i am pretty sure that the Oracle Java is built from OpenJDK sources and then sprinkled with licenses and classpath restrictions or what not.
all that said, none of this may matter, according to the user guide on IBController:

However starting with TWS 952, the TWS installers for Windows and Linux include a hidden version of Java which Interactive Brokers have used for developing and testing TWS. This version also runs IBController perfectly, and the IBController scripts ensure that it is used.

currently, it looks like it gets dropped here /opt/i4j_jres/1.8.0_152
ongoing ....


Automatic deploys w/ CalVer

Fixes #3.
Switches to https://pypi.org/project/calver/ for versioning and auto-publishing with https://github.com/pypa/gh-action-pypi-publish.

IIUC, a second run of Release in a day would result in a failure, due to the same filenames being uploaded. Is our plan to just-be-very-careful for those cases?

Yes. If we need to we can still make a manual postrelease.

In that case, I'd suggest that we document the release process (including the exact CLI to be run), just to make sure we avoid any issues due to potential mishaps around that area.


Environment for Minecraft 1.12 branch won't work correctly

Hi devs!
This might be an issue, but it's probably my own incompetence.
I want to start developing for ArchitectureCraft on MC 1.12.2. I have a feature I'd like to add.
So I forked the repo, checked out the 1.12 branch and configured my git/eclipse/jdk.
After running the runClient task everything seems to go fine. Minecraft starts, but when I generate a new world it ends with minecraft shutting down and seeing the latest.log complain about missing variants.
Crash report: https://gist.github.com/TvL2386/07ddc8122875625b58f8f09aa5431f7e
I'm not sure if there is something wrong with the state of the current repo or if I'm doing something dumb.
I'd greatly appreciate it if you could tell me what's going wrong.

Minecraft version: 1.12.2
Forge version: <IP_ADDRESS>47
ArchitectureCraft version:  <IP_ADDRESS>
See the crash report for more details.

Kind regards,
TvL2386

I think it has to do with JEI. Commented it as a dependency in project.gradle and now it starts the world fine


[BUG] In DialogListAdapter, addItem(dialog) adds the item to end of the list but updates the first item

On adding a new dialog, item gets added but UI isn't updated.
/**
     * Add dialog to the end of dialogs list
     *
     * @param dialog dialog item
     */
    public void addItem(DIALOG dialog) {
        items.add(dialog);
        notifyItemInserted(0);        <=== should be notifyItemInserted(items.length() - 1)
    }

Hi!  I can't understand how we missed it. I'll fix it. You can use method addItem(int position, DIALOG dialog) while I haven't update library.

Yup! I am using the addItem(int position, DIALOG dialog) for now.


feat: slack app

This PR implemented the Slack App with similar functionalities to the Discord App.

close https://github.com/npi-ai/npi/issues/40


Jetson Wi-Fi Access Point is not working outside

Just before flying, I created a wifi acces point. When I am at the office, I can connect to this AP without any problem. However, when I go to the field to launch ros nodes and fly the drone, I cannot connect to the access point through wifi. I have tried that I connected to AP in the office and went outside, this time it disconnected.
I thought that the problem is related to GPS and it corrupts the wifi network. However, I can also have nearly 15 GPS in indoor like outdoor. I could not figure out how to solve this problem for a while. Why can't I connect to jetson+orbitty when outside ?

I don't remember having that particular problem in our case. In the open field, WiFi range was about 50-100 meters depending on which WiFi antenna we used in our laptop (i.e. internal vs. external USB high-gain). In the forest it was about 20-30 meters.
We had some issues with USB3.0 connection (e.g. ZED camera) creating interference which affected PixFalcon/GPS but it was fixed after we shielded it properly.

I am suffering from this interesting problem and cannot fix it. Even my pc and jetson are nearby (outdoor  environment), I cannot ssh into it. Thus, range is not the issue I guess. Can you provide any suggestion with respect to your knowledge ?

One thing that you can try is to connect Orbitty to your laptop via Ethernet cable, go outside, ssh using Ethernet IP and check/debug WiFi interface/logs/etc.


exceptions.Exception does not take keyword arguments

My playbook:
    - copy:
        content: ''
        dest: /etc/docker/daemon.json
        force: no
        group: docker
        owner: root
    - jsonpatch:
        src: /etc/docker/daemon.json
        operations:
          - op: add
            path: "/insecure-registries"
            value: "[]"
          - op: add
            path: "/insecure-registries/0"
            value: "reg:5000"

Created file is not a valid json. You need to fix copy: { content: '{}' }.
- name: Create daemon.json file
  copy:
    content: '{}'
    dest: /etc/docker/daemon.json
    force: no

- name: Apply json changes
  json_patch:
    src: "/etc/docker/daemon.json"
    operations:
    - op: add
      path: "/log-opts"
      value:
        max-size: "10m"
        max-file: "3"
  register: JSON_RESULT

- name: Restart dockerd
  systemd:
    name: dockerd
    state: reloaded
  when: JSON_RESULT.changed

This is a common usage pattern, should be documented in the readme.md too.

@garkin thank you!


Does not appear to be compatible with Angularjs

I added the class to an element to a rendered template created by the ng-view directive, and there is no formatting.
The formatting appears when added to an element outside the <div ng-view></div>
see http://docs.angularjs.org/api/ngRoute.directive:ngView for details.

Hello every 1.. i wanna how to link the css and script file for odometer in angulerjs
please ..any 1..


graph_scene: correct placement of labels

Currently, when users use add_end or add_start in order to lengthen the axes a bit before/after the last tick, the labels may be misplaced, especially when using arrow tips. This PR fixes the problem.
List of Changes

For the x axis, do not use axis tip or ticks to determine where to put the label. Instead, use get_corner.
For the y axis, specify aligned_edge when using next_to, i.e. if for any reason someone wants the label to be below the y axis, it should be aligned w.r.t. its top edge.

Motivation
I just don't like misplaced axis labels :)
Testing Status
Tested locally with several combinations.
Acknowledgement

[X] I have read the Contributing Guidelines

My bad, I forgot to recreate the reference data for the test. Will do so ASAP

Not quite sure I understand. You mean add a test that makes use of the add_start and/or add_end functionality?

Yes

Done. I came across some strange behavior (not related to my changes though) that I will have to look at more in depth. Basically, the label is sometimes put at the wrong place, e.g. when there is "not enough space" above the y axis. I will open a separate issue for that, providing more details.

I'd say tests/test_graphical_units/test_axes.py has that on L15-16 and L20-21. :)

After having digged deeper, I suggest to fix #663 directly with this PR as soon as we have a consensus on what behavior to adopt.


filedrive: wrap ListDir to exclude future dated files

I accidentally added code to remove the driver I had created, but it's probably needed. You should review it before merging this into the future-actions branch.

I don't understand why your change is needed, so I'm going to merge what I verified worked locally.


Chart fade in from top left

Expected behavior
When I initialize my page and then switch the site focus to another chart, the chart should fade in from the bottom.
Current behavior
Anyways when I change the site focus the chart is fading in from the top left. I also found this problem in your documentations here.
Reproducible sample
https://www.chartjs.org/docs/4.4.0/configuration/animations.html
Optional extra steps/info to reproduce
When you first open the site and then click on the Hide and show [mode] button to change the chart it fades in from the top left and not as expected from the bottom.
Possible solution
No response
Context
I am trying to accomplish the normal fade in from the bottom when changing the site focus like in your documentation linked above but somehow it just does not work when first initializing it.
chart.js version
v4.4.0
Browser name and version
Microsoft Edge for Business Version 118.0.2088.61 (Official build) (64-bit)
Link to your project
No response

This is working as expected, the hide and show animation set the target for appearing and disappearing to x: 0 and y:0 which is the topleft of the canvas

Are you sure?
Here is the same phenomenon but without setting the values you mentioned.
When you go to the page and switch to the IMAGE chart is also coming from the top left instead of doing the nice circle animation. Same thing happens when you then refresh the page and then go back to the COLOR chart.

I have the same problem. Initial animation starts from the top left corner. Further data updates are animated correctly. Bars growing / shrinking.
I am using the ng2-charts directive wich uses Charts.js 3.9.1. For me, it would be ok, when I would be able to disable the initial animation. But I like the update animation which I want to keep. Does anybody an idea how to solve it?

same issue in v4.4.1

Any news?

Maybe someone has workaround for this behavior?

Setting the container to display: block and THEN rendering the canvas solved the issue of my bars in a bar chart floating in from the top left. Rendering the chart and then setting the visibility of its container caused the issue.

I've found a workaround for this registering an "afterDraw" plugin to reset the initial animation before it shows for the first time.
chart.config.plugins.push({
    afterDraw: (() => {
        let itsNotDrawn = true;
        return () => {
            if (itsNotDrawn) {
                chart.reset();
                chart.update();
                itsNotDrawn = false;
            }
        };
    })()
})

It worked fine for me in v4.4.6.

I've found a workaround for this registering an "afterDraw" plugin to reset the initial animation before it shows for the first time.
chart.config.plugins.push({
    afterDraw: (() => {
        let itsNotDrawn = true;
        return () => {
            if (itsNotDrawn) {
                chart.reset();
                chart.update();
                itsNotDrawn = false;
            }
        };
    })()
})

It worked fine for me in v4.4.6.

I had the same issue with the animation. Your workaround solution works for now.


Hidden feeds cause 'mark-read' to move cursor too far

Newsboat version (copy the output of newsboat -v or the first line of git show):
$ newsboat -v
Newsboat 2.25.0 - https://newsboat.org/
Copyright (C) 2006-2015 Andreas Krennmair
Copyright (C) 2015-2021 Alexander Batischev
Copyright (C) 2006-2017 Newsbeuter contributors
Copyright (C) 2017-2021 Newsboat contributors

Newsboat is free software licensed under the MIT License. (Type `newsboat -vv' to see the full text.)
It bundles:
- JSON for Modern C++ library, licensed under the MIT License: https://github.com/nlohmann/json
- optional-lite library, licensed under the Boost Software License: https://github.com/martinmoene/optional-lite
- expected-lite library, licensed under the Boost Software License: https://github.com/martinmoene/expected-lite

Newsboat 2.25.0
System: Linux 5.10.81 (x86_64)
Compiler: g++ 10.3.0
ncurses: ncurses 6.2.20200212 (compiled with 6.2)
libcurl: libcurl/7.79.1 OpenSSL/1.1.1l zlib/1.2.11 brotli/1.0.9 libssh2/1.10.0 nghttp2/1.43.0 (compiled with 7.79.1)
SQLite: 3.36.0 (compiled with 3.36.0)
libxml2: compiled with 2.9.12

Config file (copy from ~/.newsboat/config or ~/.config/newsboat/config):
browser "mybrowser %u"
auto-reload yes
reload-time 30

(plus some ignore-article lines which are personal)
Steps to reproduce the issue:

Create a hidden feed partway through your urls list (with a tag starting with !)

Try 'mark-read' (A by default) from the feed list above your hidden feed. Observe that the next feed is now selected. Reasonable.

Try 'mark-read' (A by default) from the feed list below your hidden feed. Observe that the selected feed moved 2 feeds down. Oops.

Other info you think is relevant:
I was confused about the source of this behavior until I added a second hidden feed and suddenly the number of lines moved went up. Light bulb moment.
Presumably the new cursor position is being calculated based on the order of feeds in the list, and doesn't take account of hidden feeds.

I can reproduce this.
It looks like this issue is already present since at least r2.10.
I might have some time later today to look for a fix.


Inconsistent parameter passing in Oracle scripts

I'm writing Ansible provisioner (see #148) and run into inconsistency issues in the Oracle configuration scripts. E.g.
ACCEPT new_hr_pass CHAR PROMPT 'Please enter a password for the hr schema: ' HIDE

vs.
define new_user_name = '&1'

You've adapted oracle_config.sh to different calling conventions, but I'm won't be such liberal in my Ansible branch. I'll adapt substitution variables for all the scripts.
Also only some scripts have
whenever sqlerror exit sql.sqlcode

and
begin
  if upper(user) != 'SYS' then
    raise_application_error(-20000, 'User must be SYS');
  end if;
end;

even, AFAICS, all scripts should be run as SYS and bail out in failures. Well in fact all the failures in oracle_config.sh will go unnoticed.

@tschf will we make any changes for this? If not can we close it?

I'm in favour of a consistent style, but what are the specific examples? If it's a reusable script (outside the installation), it makes sense using the ACCEPT-HIDE mechanism (to avoid password arguments in command history). I haven't gone through to find which style is where to use a consistent approach...
I think there's actually 3 styles.

ACCEPT
define
Using sed/perl to update values in the script before running

@tschf let's try to avoid sed/perl for script configuration. I like the fact that some of our scripts are re-runnable (ex create a user, apex workspace, etc)


ArrayOf(A) + MinLength generates invalid code

Using the attached design file, goagen app generates invalid code. The exact errors at compile time are:
app/contexts.go:73: cannot use tmp1 (type []string) as type string in argument to goa.InvalidLengthError
app/media_types.go:46: cannot use mt.Strings (type []string) as type string in argument to goa.InvalidLengthError
app/media_types.go:67: cannot use source.Strings (type []string) as type string in argument to goa.InvalidLengthError
app/media_types.go:104: cannot use tmp7 (type []string) as type string in argument to goa.InvalidLengthError

If there's something I'm doing wrong here, I'd love to know what it is. Otherwise, this looks like a bug?

package test

import (
	. "github.com/raphael/goa/design"
	. "github.com/raphael/goa/design/dsl"
)

func init() {
	API("test", func() {
		Title("reproduce a bug, maybe")
	})

	test := MediaType("application/vnd.test+json", func() {
		attrs := func() {
			Attribute("strings", ArrayOf(String), func() { MinLength(1) })

			Required("strings")
		}

		Attributes(attrs)
		View("default", attrs)
	})

	Resource("test", func() {
		Action("create", func() {
			Routing(POST("/"))

			Payload(test)

			Response(Created, func() { Media(test) })
		})
	})
}

Oops. Yes definitely a bug.

Thank you for the quick fix!

On Dec 19, 2015, at 8:46 AM, Raphaël Simon<EMAIL_ADDRESS>wrote:
Closed #128 via 38fb856.
—
Reply to this email directly or view it on GitHub.


Still doesn't really work on macOS

In my last issue, I reported a bug in the install script. That has been fixed, thank you very much, I appreciate that.
But I found another issue.
Now sf installs without any problems, but when you run it, it doesn't really work, because there is no lscpu command on macOS and no uptime -p option. uptime on macOS and I think on BSD, in general, doesn't have any flags.
I think that will be broken on FreeBSD too, maybe I will try it out later, but I don't know because I'm too lazy to install BSD.
Also, the colors don't work, and I don't know why. I thought it was because I use fish shell but I tried in both bash and fish and it didn't work either.
I tried to fix the uptime and I looked at the neofetch script and how it deals with uptime on macOS, and apparently, it uses
sysctl -n kern.boottime on macOS to get a value in seconds and then converts it into minutes/hour/days.
That would be complicated to implement and would require significant changes in the sf script.
For the CPU neofetch uses sysctl -n machdep.cpu.brand_string which on my system returns Intel(R) Core(TM) i7-6700K CPU @ 4.00GHz
At least this value is usable and doesn't have to be formatted any further.
For the Colors, I don't have an idea yet, I will look into this later.
I don't have time right now, but later I will sit down and try to fix some of the issues, and if I find a solution I will submit a pull request.

Update:
I thought that adding export CLICOLOR=1 to /etc/profile would fix the issue with the colors, but it doesn't work.

I will fix this, I am currently looking a way into replacing this linux commands with some commands that both support or even, check if the os is MacOS.

Please tell me, does this work in macOS? sysctl -a

tell me if the lscpu problem is fixed: https://github.com/mauro-balades/sf/commit/a54505d85f2a806323e66a1d2bfd1384f535b227

and the uptime: https://github.com/mauro-balades/sf/commit/e60ee50b3eee060d4e4ca9bdcf3c47f9c64e1066

and colors: https://github.com/mauro-balades/sf/commit/de4cf1e7f876c12eef15dfe82f7dea79cedbfead

CPU and Uptime work perfect, but still no colors. Thank you very much for the fast response.

what output do you get?

I just blured my username since it contains my real name and I don't want to share that on the Internet

try now: https://github.com/mauro-balades/sf/commit/73dd7a3188bb8484b78c1efe7075a1adc2f7b372
btw, if you have a cute, little icon for macOS (ASCII art), I will add it instead of the UFO

I will try it out tomorrow, I have to reinstall macOS because I accidentally removed all files from my /usr/local folder

It's much better, but not completely fixed

Regarding the ASCII Icon, you wrote that you already use some ASCII Art from ufetch, so I think you should use the macOS Icon from ufetch as well.

It's much better, but not completely fixed

are colors allowed in your terminal?

also, I can see that the scapes codes are not updated. Did u check that SF is completely updated?

are colors allowed in your terminal?
Yes, it works for all other CLI Applications

also, I can see that the scapes codes are not updated. Did u check that SF is completely updated?
Yes, I deleted /usr/local/bin/sf and ~/.config/sf and then ran the install script

hm...

any ideas?

Yes, it exists.
I looked into the config file and I saw that it contains values like BGRAY="\e[1;30m"
I changed it to  BGREEN="\033[1;32m" BMAGENTA="\033[1;35m" BCYAN="\033[1;36m" BBLUE="\033[1;34m" BRED="\033[1;31m" BGRAY="\033[1;30m" and now it works.

oh... you are right! I forgot to change the config file also!!! I am really sorry.

btw, did u liked the fetcher?

btw, did u liked the fetcher?

I'm sorry, I didn't see your message.
Yes I like sf, its a nice small fetcher and it does everything I need, and it's a great replacement for pfetch.

Thanks :)


ignite-cloud seems to be missing from the 2.14.0-release onward

At least it is missing from maven central. Also, there seems to be no release note mentioning where else the functionality went.

Sorry, I missed the note stating "Moved ignite-cloud IP Finder to the Ignite Extensions project."


PublickKey Equals method can throw "Object reference not set to an instance of an object."

The Equals method implemented on PublicKey (/src/Cryptography/PublicKey.cs, Line 67) does not appear to check for null prior to checking for the equality of algorithm (line 78).  This can result in an "Object reference not set to an instance of an object." exception thrown.  The "object" version of the Equals method (line 97) uses an "obj as PublicKey" cast to this method, which is one example of how null can be passed into the method to start with.
Discovered this error when using an array of public keys in a windows WPF application, the change detection algorithim was enumerating keys to determine if it was necessary to update, a portion of the stack trace follows:
Object reference not set to an instance of an object.

   at NSec.Cryptography.PublicKey.Equals(PublicKey other)
   at NSec.Cryptography.PublicKey.Equals(Object obj)
   at System.Windows.Controls.ItemsControl.EqualsEx(Object o1, Object o2)
   at System.Windows.Data.BindingExpression.OnDataContextChanged(DependencyObject contextElement)

Good catch! Thanks a lot for reporting this. Fixed in https://github.com/ektrah/nsec/commit/653ad7b0e1cf00d51c20c792db79b2504ddd32a9.


HELYAR1: new volunteer via Mozilla Science Lab Collaborate

I like to help people in every possible wayThis issue was created by @HELYAR1 via Mozilla Science Lab Collaborate

Sorry for the long wait! If you're still interested in participating, we'd love to have you. Is there anything in particular you'd be interested in working on?


⚠ Bofrost.at has degraded performance

In 1d6b641, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1904 ms

Resolved: Bofrost.at performance has improved in b4e9f3a.


Initial changes for 1.23

Description of changes:

Added projects (except etcd, which Jackson already added)

Did NOT add Kubernetes patches because we're still figuring those out

Update version conditional statements throughout

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

checksums will fail. I'll fix them

/hold for jobs updates

/test kubernetes-1-23-presubmit
/test kubernetes-release-1-23-presubmit

/test kubernetes-1-19-presubmit

Do we need to add 1-23 to that supported releases file? Or do we not do that until we ready for prod release?
In the past, we did that as a separate PR, but I added it


Retrain with new language

Hi, I have some questions (sorry if this is some kind of beginners question, I am new in this field). I want to change the word embedder to the BERT that is pretrained with my language (Indonesia, using indobert). Can you give some tips on how to change the embedder to my language? Thanks!

Okay thanks, will try them. But I have another issue when I try to change the save_dir of the model.

This also happens when I use the englsih version and I change the random seed

Hi, this may be due to prediction failure of the model which leads to no prediction file, could you see if there is any error message further up and attach the full error log?

I thinkk this is the main reason that the model.tar.gz file is not created
2022-08-08 20:34:14,521 - CRITICAL - root - Uncaught exception Traceback (most recent call last):   File "/home/randy.suchrady/anaconda3/envs/span-aste/bin/allennlp", line 8, in <module>     sys.exit(run())   File "/home/randy.suchrady/anaconda3/envs/span-aste/lib/python3.7/site-packages/allennlp/__main__.py", line 34, in run     main(prog="allennlp")   File "/home/randy.suchrady/anaconda3/envs/span-aste/lib/python3.7/site-packages/allennlp/commands/__init__.py", line 118, in main     args.func(args)   File "/home/randy.suchrady/anaconda3/envs/span-aste/lib/python3.7/site-packages/allennlp/commands/train.py", line 119, in train_model_from_args     file_friendly_logging=args.file_friendly_logging,   File "/home/randy.suchrady/anaconda3/envs/span-aste/lib/python3.7/site-packages/allennlp/commands/train.py", line 178, in train_model_from_file     file_friendly_logging=file_friendly_logging,   File "/home/randy.suchrady/anaconda3/envs/span-aste/lib/python3.7/site-packages/allennlp/commands/train.py", line 242, in train_model     file_friendly_logging=file_friendly_logging,   File "/home/randy.suchrady/anaconda3/envs/span-aste/lib/python3.7/site-packages/allennlp/commands/train.py", line 466, in _train_worker     metrics = train_loop.run()   File "/home/randy.suchrady/anaconda3/envs/span-aste/lib/python3.7/site-packages/allennlp/commands/train.py", line 528, in run     return self.trainer.train()   File "/home/randy.suchrady/anaconda3/envs/span-aste/lib/python3.7/site-packages/allennlp/training/trainer.py", line 966, in train     return self._try_train()   File "/home/randy.suchrady/anaconda3/envs/span-aste/lib/python3.7/site-packages/allennlp/training/trainer.py", line 1001, in _try_train     train_metrics = self._train_epoch(epoch)   File "/home/randy.suchrady/anaconda3/envs/span-aste/lib/python3.7/site-packages/allennlp/training/trainer.py", line 716, in _train_epoch     batch_outputs = self.batch_outputs(batch, for_training=True)   File "/home/randy.suchrady/anaconda3/envs/span-aste/lib/python3.7/site-packages/allennlp/training/trainer.py", line 604, in batch_outputs     output_dict = self._pytorch_model(**batch)   File "/home/randy.suchrady/anaconda3/envs/span-aste/lib/python3.7/site-packages/torch/nn/modules/module.py", line 727, in _call_impl     result = self.forward(*input, **kwargs)   File "/srv/nas_data1/text/randy/finetune-cxa/Span-ASTE/span_model/models/span_model.py", line 374, in forward     metadata,   File "/home/randy.suchrady/anaconda3/envs/span-aste/lib/python3.7/site-packages/torch/nn/modules/module.py", line 727, in _call_impl     result = self.forward(*input, **kwargs)   File "/srv/nas_data1/text/randy/finetune-cxa/Span-ASTE/span_model/models/relation_proper.py", line 223, in forward     relation_scores = self._compute_relation_scores(pruned_o, pruned_t)   File "/srv/nas_data1/text/randy/finetune-cxa/Span-ASTE/span_model/models/relation_proper.py", line 454, in _compute_relation_scores     embeds = self._compute_span_pair_embeddings(a, b)   File "/srv/nas_data1/text/randy/finetune-cxa/Span-ASTE/span_model/models/relation_proper.py", line 422, in _compute_span_pair_embeddings     c = self._make_pair_features(a, b)   File "/srv/nas_data1/text/randy/finetune-cxa/Span-ASTE/span_model/models/relation_proper.py", line 418, in _make_pair_features     x = torch.cat(features, dim=-1) RuntimeError: CUDA out of memory. Tried to allocate 16.00 MiB (GPU 0; 31.75 GiB total capacity; 28.63 GiB already allocated; 10.06 MiB free; 30.24 GiB reserved in total by PyTorch)

Hi, this may be a GPU issue if you are sharing with other people and programs that are also using GPU memory, or it may be a dataset issue if the long input texts cause the transformer model to use large amounts of GPU memory. In the case of long input texts, you may need to truncate the texts or split into multiple sentences.


fix: use split dns google domain

This modifies the base uri for API calls from https://www.googleapis.com/dns/v1 to https://dns.googleapis.com/dns/v1
Just dipping my toes in the domain splitting waters to verify the domain is correct.

Unsure if this should be merged, since this hasn't landed in discovery service yet.

@broady it's a good question.  Good news is that this PR ran all the sample and system tests, and everything appears to be passing.  I think the rollout on discovery was just for updates of the rootUrl field to signify which path to use by default, but 🤷♂   Can you follow up with the OP folks to make sure?

@shinfan should this be ready to go?


Add maas external check for neutron metadata proxy - kilo

The neutron metadata agent proxy provides access from within neutron
network namespaces to the neutron metadata agent and thus to the
various sources of metadata provided to instances on <IP_ADDRESS>
at port 80 when cloud-init is used.
The metadata agent has monitoring in RPC but has been reported to be
running but non-responsive on some occasions (see issue #342). The
new check tests the agent's proxy from within the namespace to
monitor responsiveness. Because the agent will use the client host
IP to look for metadata but the request is not originating from an
instance a 404 Not Found response is expected from the agent.
Because local checks are expected to operate from the host rather
than inside a container, and the host does not have ready access to
neutron network namespaces, the monitor must probe for an appropriate
container and resorts to executing a curl request through the network
namespace within the selected container for each network that is
expected to be proxied (only those networks which have ports). This is
deemed easier to verify and understand through command shell
operations than alternative solutions.
This solution seems to work regardless of the value of OSA's
nova_force_config_drive setting or the use of the config-drive feature
in Nova.
Fixes #342

:+1: LGTM

@prometheanfire I think it makes more sense to reference the commit from which this was cherry picked. Appending kilo to the end of the commit message title doesn't provide any additional information.

A direct cherry-pick wasn't possible as the originating files changed location.  see #414 for more info...
not having monitoring merged is making backports hard

Personally, I vote we always cherry-pick -x for backports. If it's not clean, fix the merge conflicts, and commit with the "Conflicts" comments provided by git uncommented. It;s the best way to ensure track-ability.

ok, cherry picked

Looks good to me, but I'd like to get it tested first.  Any volunteers?

this was already done in master I thought.  manual tests SUCK

rebased and it does look like it was tested

:+1: genuine to the original
this looks like the second, no dissents other than process at this point, so pushing the button.


[DT-10] Update parsing urls.ipynb

dealt with DT-10

will merge PR, the additional columns @fallendynasty discovered at ['i', 'ind', 'cou', 'indust', 'o', 'sch', 'course_of'] are all emtpy lists so they are dropped