cmd/go: GOPROXY credentials exposed in case of errors

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

$ go version
go version go1.12 linux/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/leitzler/.cache/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/leitzler/go"
GOPROXY="https://user:pass@myproxy"
GORACE=""
GOROOT="/usr/local/go"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build942859588=/tmp/go-build -gno-record-gcc-switches"

What did you do?
I'm running a CI environment that uses GOPROXY to fetch dependencies, the proxy (athens) has basic auth enabled since it has access to private repos that all users shouldn't be able to access.
The basic auth credentials are passed as a part of the GOPROXY, as https://user:pass@myproxy.
If go get fails to fetch a dependency (read: if someone just injects a faulty dependency), go get will include username and password from the proxy in the logs readable by the user.
What did you expect to see?
The URL without credentials, or credentials masked (for all go commands that uses GOPROXY)
What did you see instead?
Plain text username & password.

CC @marwan-at-work @hyangah @jayconrod

Don't pass credentials in the GOPROXY environment variable. Please, just don't do that.
Proper HTTPS basic auth support is coming in #26232. At that point you won't need to encode it in the URL, and logging the URL will not leak anything. So I'm closing this issue as a duplicate of #26232, and in the interim please find some other way to inject your credentials (maybe another layer of GOPROXY?).

Passing credentials via GOPROXY is indeed a bad workaround for not being able to do "Proper HTTPS basic auth". I'm aware that this is being worked on, but I assume (and correct me if I'm wrong), that #26232 won't land before 1.13 and that is quite far away.
As seen in https://github.com/gomods/athens/issues/1046 this is an issue that others already have noticed. The fact that Athens supports basic auth will increase the risk that others go down this path since proper auth isn't in place yet.
Even when proper auth is in place, will there be anything that prevents the user from still setting credentials in their GOPROXY?
(CC @FiloSottile who might have something to add since he asked me to open this issue when I asked about it)

Ah, we can probably at least detect and reject credentials in the URL.

FWIW Once Go Modules supports proper authentication sending headers to GOPROXY we will make Athens accept that mechanism and probably remove the basic auth option.
That said, I also agree that Go should detect and reject credentials in the URL or at least detect and obfuscate them in logs.


small visual glitch with inertia

click left mouse button
move mouse quite fast
before releasing the button, wait  a second or two
the image now moves a bit, it would be more natural if it didn't

It should be fixed now. The mousemove event, which was updating the speed, stops firing when the mouse stops moving, so abrupt stops left a large speed. I'm now clearing the speed when the mouse button is released if mousemove hasn't fired recently.

I still have the impression that Flash-based player like http://openpano.org/demo/ are a bit smoother when panning the view, but this is fixed. Thanks!


Is there any way to go next slide using code?

Hi, i want to know if  is there any way to go next slide using code?
Thank you.

I believe it is pager.setCurrentItem(slide_number);
@jeffersonlicet Make sure to close your issue if the above resolves it <3

It works!. I think this is not in the documentation as well as doneButton.callOnClick();

thanks for this!


Allow ng-trim to work on radio buttons

I'd like to have the option to use ng-trim on radio inputs, not just text inputs. I'm using Angular 1.2.x, but have confirmed the feature is not available in 1.3.x or 1.4.x.
JSFiddle example against 1.4.x: http://jsfiddle.net/rgewfgbL/5/

ngTrim is used to trim user input, i.e. trailing / leading whitespace in text inputs. If the value for radio buttons contains white space is in 90% of all cases at the discretion of the developer. If you use user defined values for your radio, I can see whitespace being an annoyance, but this is simply not what ngTrim is about. It's about trimming user input text. So I don't think ngTrim for radios is something we want.

@Narretz while I agree with your analysis, it looks odd that this radio button is not selected, no?

Yeah, that's odd ...

When the value is retrieved from the element, whitespace is automatically trimmed by the browser (Firefox at least). We could trim the $viewValue in this case, too  to match this behavior. However, I don't know what the spec says about this.

It's also possible that the trimming happens when angular creates the attrs for the directive.

I've created an updated fiddle that shows the value as seen by the browser, and proves this to be an Angular problem, not a browser one: http://jsfiddle.net/rgewfgbL/6/

Please fix this one.
<input type="radio" ng-model="format" ng-trim="false" value=" " /> sets $scope.format to an empty string (""), not to a " ". Using the   and similar tricks does not work.


[BugFix] dont update bytes_read when peek fails

Fixes #issue
Since we have zero-copy feature, we use peek to check if we can do zero-copy.
If peek does not  return OK, it means we can not use zero-copy. And in this case

we don't need to update bytes_read(and io_count)
and since peek is very fast, we don't need to time it

we use select * from nation as test case
before PR, you can see ApppIOBytesRead(11.2KB) > BlockCacheReadBytes(9.33KB), which is not normal when all data is from cache.

and after PR, you can see two values are equal

What type of PR is this:

[X] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.0
[ ] 2.5
[ ] 2.4
[ ] 2.3

@mergify backport branch-3.0


Count of attachments is wrong

Describe the bug
When I have an email that contains both plain text and an HTML body the attachment count shows up as 1 instead of zero. When I iterate through the attachments I notice that the non-attachment does not contain the text: 'Content-Disposition: attachment;'
When I examine the email source I see the following entries:
Content-Type: multipart/mixed;
Content-Type: multipart/alternative;
Content-Type: text/plain;
Content-Type: text/html;
The non-attachment showed the contents of the multipart/mixed entry;
A real attachment contains the lines:
Content-Type: application/pdf; name="blahpdf"
Content-Disposition: attachment;
Expected behavior
I expect the count of attachments to be accurate.
Suggested Fix
I stepped through the code and found this at line 288 in part.php
if (($this->subtype == null || in_array((strtolower($this->subtype)), ["plain", "html"])) && $this->filename == null && $this->name == null) {
I changed it to ["plain", "html", "mixed"] and this fixed the problem.

I've just started using this module, and what is really annoying is that I get different attachment counts each time I run the same script on the same message!

Hi @TonyMarston,
Thanks a lot for reporting this issue. I really appreciate it! However, in order to help you out, it would be great if you could provide an anonymized version of the problematic message.
Without that, it's quite tough for me to debug the issue accurately.
If you're using an older version of the library, I recommend updating to the latest version and giving it another shot. There's a chance that the problem might have already been fixed in the newer release.
It might also be related to https://github.com/Webklex/php-imap/issues/418
Once again, thanks for taking the time and effort to make this library better! If you have any more questions or need further assistance, feel free to let me know.
Best regards and happy coding!

Attached is the source of an email message which is reported as having 1 attachment but actually has none.
Tony Marston
http://www.tonymarston.net http://www.tonymarston.net
http://www.radicore.org http://www.radicore.org
From: Webklex @.>
Sent: Friday, June 23, 2023 4:00 PM
To: Webklex/php-imap @.>
Cc: Tony Marston @.>; Mention @.>
Subject: Re: [Webklex/php-imap] Count of attachments is wrong (Issue #412)
Hi @TonyMarston https://github.com/TonyMarston ,
Thanks a lot for reporting this issue. I really appreciate it! However, in order to help you out, it would be great if you could provide an anonymized version of the problematic message.
Without that, it's quite tough for me to debug the issue accurately.
If you're using an older version of the library, I recommend updating to the latest version and giving it another shot. There's a chance that the problem might have already been fixed in the newer release.
It might also be related to #418 https://github.com/Webklex/php-imap/issues/418
Once again, thanks for taking the time and effort to make this library better! If you have any more questions or need further assistance, feel free to let me know.
Best regards and happy coding!
—
Reply to this email directly, view it on GitHub https://github.com/Webklex/php-imap/issues/412#issuecomment-1604405257 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ADRYNOEVYQGED7XBUZGGGMTXMWVNJANCNFSM6AAAAAAYZKL4EM .
You are receiving this because you were mentioned.  https://github.com/notifications/beacon/ADRYNOAIJUTPDILOYIJSUKLXMWVNJA5CNFSM6AAAAAAYZKL4EOWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTS7UFEAS.gif Message ID: @.*** @.***> >

Here is the attachment.
<EMAIL_ADDRESS>

Hi @TonyMarston ,
many thanks for the quick followup. Unfortunately I'm unable to replicate the behavior (see the referenced commit above).
Best regards and happy coding,


Seems like appointment title <FS Umschreibung Ausländischer FS> is not accepted by the buro <Führerscheinstelle> any more Please check of issues on Github and create one if not reported yet

This looks fine now? Not sure what was the problem, maybe the buro website was down? Feel free to reopen if the issue appears again...


Dynesty solver

Implement dynesty solver with "dynamic" and "static" nested sampling options. Remarks:

It would be nice to actually test the "dynamic" mode here, but that takes minutes to complete, which I find too long for the test. @rozsasarpi are there dynesty options to speed that up?
Nested samplers do not work with the logprior, but the prior quantile function. This is not always part of the scipy distributions, e.g. for the MVN. I specifically implemented the quantile function for the case of a uncorrelated MVN within the solver. Is that OK for now?
The solver outputs arviz.InferenceData with only the posterior samples, which seems to be sufficient for our visualization. Is more information needed/preferred?

Hi Thomas, thanks a lot for setting up the dynesty-solver so quickly! It looks pretty good to me. The only things I just added were some run_dynesty arguments in the integration tests which were missing, and I did the version increase from 1.0.14 to 1.0.15


Leading zeros are great for readibility

Trailing zeros, not so much. I would make leading zeros required.
// Very clear a decimal is in use
.element {
  padding: 2em;
  opacity: 0.5;
}

// Not so clear
.element {
  padding: 2em;
  opacity: .5;
}```

I don't like having them. I am not sure why.
I'll think about this.

This is definitely as important or more important than doing things like quoting strings and wrapping calculations in parenthesis. Trailing zeros I can understand omitting. Leading zeros for decimal points are a must.

I feel like this is the a subjective point.
I've been omitting leading zeros for years now and I find it easier to read but I can totally understand why someone would prefer having them.
Maybe mark that point as not important as long as it's consistent throughout (with linting) ?

It's a well understood best-practice in the math, science, and engineering fields that you should always have a leading zero on a decimal point (1: 10.5.2) (2) (3). This is why this looks odd to me.
I understand that in this case we aren't going to kill anyone by someone misreading a drug dosage. However if it's subjective and could go either way: why not go with the more established, logical convention that helps readability? That seems to be in the spirit of the rest of the project.

In most cases of using a decimal there's a leading number. It seems inconsistent to have 1.5 but not 0.5.

The pull request fixed only the recommendation itself, now there are some places were the guide contradicts itself. Example:
// Color formats
color: rgba(0, 0, 0, .1);

// Magic numbers
top: .327em;

@jjeising - good point, I might be able to do that super quick!

@jjeising ^ booyah

You still have it here:
// Yep
.element {
  color: rgba(0, 0, 0, .1);
  background: hsl(300, 100%, 100%);
}

// Nope
.element {
  color: rgba(0,0,0,.1);
  background: hsl( 300, 100%, 100% );
}

@nikolay, I took care of that in my PR

@JulienMelissas :heart:

Power and beauty of advanced tools is that we can have it recognize each form equally.


Add logging for deployment scripts along with the quiet flag

Partially solves #825
Introduced changes

added logging for deployment scripts along with --quiet flag

Checklist

[x] Linked relevant issue
[ ] Updated relevant documentation
[x] Added relevant tests
[x] Performed self-review of the code
[ ] Added changes to CHANGELOG.md

@abulenok Please sync changes with master.

After internal discussion, we decided to close deployment script logging PRs, as the user can use newly added print! and println! Cairo macros for logging necessary information


replace mangleC with Cprefix

Removes an unnecessary function and an unnecessary reference to the GC. I've never encountered a C mangling method that did more than simply add a prefix. Having an abstraction for a possibility that will never happen is useless bloat.

I am confused by the fact this even exists...

I am confused by the fact this even exists...

It's used only by a hack in rt.config to get C name mangling on an identifier declared in a template. The identifier is declared in a template in order for it to be put into a COMDAT section so it can be optionally overridden by a regular declaration. Names in templates are not given C mangling because then they would collide with other names generated by other instantiations of that template.

Wasn't  mangleC  part of the public API?

Yes. But I suspect it is never used other than for that one hack, as I can think of no other purpose for its existence. It could arguably be marked deprecated, but since demangle is inserted into every D executable, I was looking for ways to do a shrink on the executable size, and also ways to reduce the dependence of druntime on the GC.

In any case,  Cprexfix  should be  cPrefix  according to our naming conventions.

Fixed.

Good to merge? @klickverbot @ibuclaw

Auto-merge toggled on


Moved Interactive Docs Example into Examples Folder

to: @vjeux @browniefed
Includes the working interactive docs example, refactored to use the code in this repo.
Notes:

uses browserify
Commit includes a bundle file, which includes react as well (so it is quite large). We can gitignore this if we want, but then the example wouldn't run on its own :/

Can we use react from the cdn?

Sort of yes and no...
The Animated source code runs require('react') in a few places. Additionally, the way i grabbed CSSPropertyOperations module in the DOM target causes issues here: https://github.com/animatedjs/animated/blob/master/src/targets/react-dom.js#L13
Because i'm using require('react/lib/foo'), It's not super straightforward to specify some "external" modules for browserify to ignore.
I'm open to suggestions?

I don't see anything obvious right now, ship it :)


Floornumber and nickname parameters not recognized

I can get the Set-Place command to work perfectly, with the exception of the floornumber and nickname parameters. Those parameters cannot be found. I don't really need them but I would sure like to know why.

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

ID: c8ef4fb7-736b-99e8-796b-c1d02410a7e4
Version Independent ID: 0fb75415-2bd2-a880-aa9e-a06c76d9ddc4
Content: Set-Place (exchange)
Content Source: exchange/exchange-ps/exchange/mailboxes/Set-Place.md
Service: exchange-powershell
GitHub Login: @chrisda
Microsoft Alias: chrisda

@officedocsbot assign @yogkumgit

@chrisda Please merge the PR #3858

@dariomws306  Thank you very much for the contribution and updating the documentation with PR. @homerggg Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Create Crashes when loaded with Paradise Lost (Aether fabric port)

Describe the Bug
When I try to launch the game with both Create and Paradise Lost, the game fails to load and crashes before reaching the loading screen.
Reproduction Steps

install Fabric API 0.51.1 (latest)
install Create 0.4.1+534 (latest)
install Paradise Lost 1.6.8 (latest)
launch game with Fabric Loader 0.13.3

Expected Result
The game should load to the title screen rather than crashing.
Screenshots and Videos
No response
Crash Report or Log
https://pastebin.com/ut7Vy5dC
Operating System
Windows 11
Mod Version
0.4.1
Minecraft Version
1.18.2
Other Mods
Fabric API, Paradise Lost
Additional Context
No response

I'm not sure if this is a Create issue or an issue with Paradise Lost, so I'm going to put the issue on both repositories.

the error does not mention Create at all, test without it.

This is likely caused by the aether's boreal wisteria leaf shader

Close this thead, I put it in a modpack with paradise and nobody reports issues

will close tomorrow if no response from author


Created Code of Conduct

Hope you are good,
Thank you for the opportunity to contribute and please consider it under hacktoberfest2024 if found useful

Thanks for your contribution.


Round function implemented for integers

Oh! I removed that doc by mistake. @uranusjr can go ahead with his implementation as that is standard implementation. I am closing this.


Message Signing for DOGECOIN Broken

Signs the message, but will not verify with bitcoinjs, or any other libraries. Have tried modifying the magic message and prefix to use both Dogecoin network and Bitcoin network, with same failed results. Please publish how you've gone about signing DOGE.
Message to test with:
Public Address: DPpVqDPfStJq6R4gU82qyCFWpPGDdctjg1
Message: This is an example of a signed message.
Signature: IKCH10PisOuRJmgLvvzgkOVN3pUBTZ6j9z8jNmKynSWDIvNDNedCWsOJrLv+RRkpTaTIMXf5EGAyLH+ggQ50law=

This is was user-error on my side.
Magic Message and prefixes are as-expected.


Creating tables recipe

https://issues.apache.org/jira/browse/ARROW-13715

In the Python cookbook at the moment no subheadings were used, all recipes within a chapter were at the same level. We might consider adding further grouping but I think that should be done for all sections, not just this one, and I fear it might add confusion as we might end up with many subgroups of just two entries each.

I integrated suggestions, so this should now be good to go


Reward account refreshing has a lag

Context
When you send some funds+rewards your UTxO and reward balances don't get updated at the same time.

Information
-

Version
2020.8.3 (git revision: b5b292b5a9f0ddccc0fc5d9e98bf3bf80f1a9d13)

Platform
Linux

Installation
From Hydra

Steps to Reproduce

Do some transaction with withdrawal='self'
Check balance immediately

Expected behavior
UTxO and reward balances have new values.
Actual behavior
UTxO balance has a new value since transaction has been formed, but reward one gets changed only after confirmation.

Thanks for reporting @Fell-x27 :)
This one is known and a duplicate of #1955 (see last-ish comment).


HRQB 41 - Improve task scoping for runs and data cleanup

Purpose and background context
This PR reworks the CLI --task flag into two new ones --include and --exclude.  Using a combination of these, it's now possible to:

load a pipeline, but begin from any task in the pipeline dependencies
exclude tasks from running or having their data cleared between runs

These changes dovetail with some improvements to how pipelines run, including moving the run_pipeline() function to a method on HRQBPipelineTask, where it should only ever run anyhow.
Taken altogether, the output of HRQBClient will remain identical.  All changes are for local development convenience.  When working on a particular task or family of tasks, it is much easier to isolate them for development.
Some areas of interest:

Instead of a default requires() method, HRQBPipelineTasks now expect a default_requires() method (example)

this sets the default root tasks for a pipeline, but can be overridden by --include-tasks CLI flag, by new shared requires() method and helpers

luigi.utils.run_pipeline() moved to method on HRQBPipelineTasks and includes setup + cleanup logic

setup: determine if include/exclude tasks actually exist, clear out memory of previous runs, raise exceptions if tasks are required to run but have been explicitly excluded
cleanup: remove data from run if flagged to do so, skipping excluding tasks

How can a reviewer manually see the effects of these changes?
Some manual testing is possible with these changes, as we can use test fixtures.
  By using --include-tasks and --exclude-tasks we can pinpoint this task, remove it's data each time, but keep data from other required tasks.
The following shows that --include limits scope to MultiplyNumbers and its required tasks:
pipenv run hrqb --verbose pipeline \
--pipeline-module=tests.fixtures.full_annotated_pipeline \
--pipeline=AlphaNumeric \
--include=MultiplyNumbers \
--exclude=GenerateNumbers,GenerateLetters \
status

Given the includes/excludes, we can remove all data while keeping data from desired tasks:
pipenv run hrqb --verbose pipeline \
--pipeline-module=tests.fixtures.full_annotated_pipeline \
--pipeline=AlphaNumeric \
--include=MultiplyNumbers \
--exclude=GenerateNumbers,GenerateLetters \
remove-data

In our development loop for making and testing changes to MultiplyNumbers, we can run this command repeatedly knowing that it will only run and cleanup data we expect:
pipenv run hrqb --verbose pipeline \
--pipeline-module=tests.fixtures.full_annotated_pipeline \
--pipeline=AlphaNumeric \
--include=MultiplyNumbers \
--exclude=GenerateNumbers,GenerateLetters \
run --cleanup

The output from this indicates as much, showing fully complete and successful, then data removed:
2024-07-16 09:52:26,824 INFO hrqb.cli.run() line 183: Pipeline run result: SUCCESS
2024-07-16 09:52:26,825 INFO hrqb.cli.run() line 184:
├── COMPLETE: AlphaNumeric()
   ├── COMPLETE: MultiplyNumbers(table_name=, pipeline=AlphaNumeric, stage=Transform)
      ├── COMPLETE: GenerateNumbers(table_name=, pipeline=AlphaNumeric, stage=Extract)
2024-07-16 09:52:26,826 INFO hrqb.cli.remove_data() line 163: Successfully removed target data(s).
2024-07-16 09:52:26,826 INFO hrqb.cli.remove_data() line 164: Updated status after data cleanup:
2024-07-16 09:52:26,827 INFO hrqb.cli.remove_data() line 165:
├── INCOMPLETE: AlphaNumeric()
   ├── INCOMPLETE: MultiplyNumbers(table_name=, pipeline=AlphaNumeric, stage=Transform)
      ├── COMPLETE: GenerateNumbers(table_name=, pipeline=AlphaNumeric, stage=Extract)

Includes new or updated dependencies?
NO
Changes expectations for external applications?
NO
What are the relevant tickets?

https://mitlibraries.atlassian.net/browse/HRQB-41

Developer

[X] All new ENV is documented in README
[X] All new ENV has been added to staging and production environments
[X] All related Jira tickets are linked in commit message(s)
[X] Stakeholder approval has been confirmed (or is not needed)

Code Reviewer(s)

[ ] The commit message is clear and follows our guidelines (not just this PR message)
[ ] There are appropriate tests covering any new functionality
[ ] The provided documentation is sufficient for understanding any new functionality introduced
[ ] Any manual tests have been performed or provided examples verified
[ ] New dependencies are appropriate or there were no changes

Opting to close out this PR.  After some good discussions with @ehanson8, came to the conclusion that the conveniences these CLI flags add for development work do not outweigh the complexity this adds to the task running framework.
In fact, it kind of entrenches a bad pattern: running lots of tasks you don't need to, and keep the outputs (targets) from those tasks on disk.
An approach that might be more inline with the sensitive data requirements of the project, and the ergonomics of working on a particular task could be:

for local development, all data is deleted after each run by default
identify a task to run and only that task will run; no dependencies
if data is kept between runs, there is friction to do so, and consider even encryption


[QUERY] Proper Behavior of baseBlobClient.Exists() when Container or Blob Does Not Exist

Library name and version
Azure.Storage.Blobs 12.14.0
Query/Question
What is the proper behavior of BaseBlobClient.Exists() when the container or blob does not exist? In the API, it says that it should return a boolean and that if a RequestFailedException is raised, then the exception should be thrown.
In the case where a container or blob does not exist and a 404 error is caught, should the Exists() call result in a ResultFailedException getting thrown or should it return false and not throw the exception?
With Azure Storage Emulator, we are seeing Exists() throw a 404 error in this scenario but with Azurite, it is returning false and not throwing the 404 error.
Environment

.NET Core SDK Version: 3.1.420
Visual Studio 17.4.4

@danieloh1 We are looking into it and get back to you for any additional information.

@danieloh1 I have tried using Azure Emulator but unable to reproduce this issue and the code doesn't throw 404.  Please find below the sample code snippet and the response.

We are

We are creating a BlobServiceClient with the "UseDevelopmentStorage=true"
private static readonly BlobServiceClient client = new BlobServiceClient(StorageEmulator.ConnectionString);
We then check the following and it throws 404 with Azure Storage Emulator.

We are noticing with Azurite that this behavior is different and instead returning false. Is the Azurite behavior the correct one?

Any updates on whether this behavior is expected or not? In Azure Storage Emulator, this was throwing a 404 error instead. If this behavior was changed for Azurite to more closely align with Azure Storage, then we can make the appropriate changes.

I believe Azure Storage Emulator was deprecated. Even so we don't take Azure Storage Emulator issues here either.
You should be following what behavior Azurite does. Which is if a 404 (BlobNotFound) is seen, then the Exists will return false. If a different exception other than that occurs, an exception will be thrown.


Feature/add docusaurus search local

Implementation has to be delayed due to a significant amount of new security vulnerabilities in the local-search extension


too slow training speed

too slow training speed
my current env is
win7 x64 System
Nvidia Geforce GTX 1080 (8G)
CUDA9.0
cuDNN7.0.5
tensorflow-gpu(1.6.0)
tensorpack (0.8.0)
gym now use(0.12.1)
and i used examples data for training
\tensorpack-medical\examples\LandmarkDetection\DQN\data\filenames\image_files
\tensorpack-medical\examples\LandmarkDetection\DQN\data\filenames\landmark_files
for gpu memory limit, and i used parameters:
BATCH_SIZE = 24
and GPU and CPU setting:
mem_fraction = 0.8
# conf = tf.ConfigProto(log_device_placement=True)
conf = tf.ConfigProto()
# conf.allow_soft_placement = True
conf.intra_op_parallelism_threads = 6
conf.inter_op_parallelism_threads = 6
conf.gpu_options.per_process_gpu_memory_fraction = mem_fraction
conf.gpu_options.allow_growth = True
and exclude Data Load's effect.i used FakeData
dataflow = FakeData([[BATCH_SIZE,45,45,45,5],[BATCH_SIZE],[BATCH_SIZE],[BATCH_SIZE]],size=1000,random=False, dtype=['uint8','float32','int8','bool'])
and minimal training setting:
return TrainConfig(
data=QueueInput(dataflow),
model=Model(),
callbacks=[],
# steps_per_epoch=10,
steps_per_epoch=10,
max_epoch=1000,
session_config= conf,
)
the training speed is 28 seconds per iter.
even i reduce the model complexness (by commented Conv3D and Pool3D ):
with argscope(Conv3D, nl=PReLU.symbolic_function, use_bias=True):
# core layers of the network
conv = (LinearWrap(image)
.Conv3D('conv0', out_channel=32,
kernel_shape=[5,5,5], stride=[1,1,1])
.MaxPooling3D('pool0',16)
#  .Conv3D('conv1', out_channel=32,
#          kernel_shape=[5,5,5], stride=[1,1,1])
#  .MaxPooling3D('pool1',2)
#  .Conv3D('conv2', out_channel=64,
#          kernel_shape=[4,4,4], stride=[1,1,1])
#  .MaxPooling3D('pool2',2)
#  .Conv3D('conv3', out_channel=64,
#          kernel_shape=[3,3,3], stride=[1,1,1])
)
the training speed is 22 seconds per iter.
it is 100x slow by comparison with  your training speed
{around ~3-4 it/sec using the default big architecture on a GTX 1080}
I want to know why and
please give me some suggestions about reduce the training time.

why placement  Conv3D in CPU ? and i how to placement to GPU device?

@layer_register(log_shape=True)
@convert_to_tflayer_args(
args_names=['filters', 'kernel_size'],
name_mapping={
'out_channel': 'filters',
'kernel_shape': 'kernel_size',
'stride': 'strides',
})
def Conv3D()
And
@layer_register(log_shape=True)
def Conv3D()
have what differences?
i has errrors when using first Conv3D()
File "C:\Anaconda3\envs\tf1.6\lib\site-packages\tensorpack\tfutils\argscope.py", line 45, in argscope
_check_args_exist(l.symbolic_function)
File "C:\Anaconda3\envs\tf1.6\lib\site-packages\tensorpack\tfutils\argscope.py", line 41, in _check_args_exist
assert k in args, "No argument {} in {}".format(k, l.name)
AssertionError: No argument nl in Conv3D
@amiralansary @crypdick @thanosvlo @thanosvlo
please help me!

why target/conv0/Conv3D can placement at GPU ,but conv0/Conv3D  cannot placement at GPU?
{
"cat": "Tensor",
"id": 22,
"ph": "O",
"tid": 0,
"name": "conv0/Conv3D",
"args": {
"snapshot": {
"tensor_description": "dtype: DT_FLOAT\nshape {\n  dim {\n    size: 24\n  }\n  dim {\n    size: 45\n  }\n  dim {\n    size: 45\n  }\n  dim {\n    size: 45\n  }\n  dim {\n    size: 32\n  }\n}\nallocation_description {\n  requested_bytes: 279936000\n  allocated_bytes: 279936000\n  allocator_name: cpu\n  allocation_id: 1\n  has_single_reference: true\n  ptr:<PHONE_NUMBER>\n}\n"
}
},
"pid": 2,
"ts":<PHONE_NUMBER>004270
},
{
"cat": "Tensor",
"id": 382,
"ph": "O",
"tid": 13,
"name": "target/conv0/Conv3D",
"args": {
"snapshot": {
"tensor_description": "dtype: DT_FLOAT\nshape {\n  dim {\n    size: 24\n  }\n  dim {\n    size: 45\n  }\n  dim {\n    size: 45\n  }\n  dim {\n    size: 45\n  }\n  dim {\n    size: 32\n  }\n}\nallocation_description {\n  requested_bytes: 279936000\n  allocated_bytes: 279936000\n  allocator_name: GPU_0_bfc\n  allocation_id: 430\n  has_single_reference: true\n  ptr:<PHONE_NUMBER>24\n}\n"
}
},
"pid": 4,
"ts":<PHONE_NUMBER>607019
},

please see https://github.com/tensorpack/tensorpack/issues/1259

@courins The slow down is obviously a result of running the code on the cpu and not the gpu. The reason of that is not clear to me, either your envs or tensorpack.
It seems that you have solved this issue using a newer version of tensorpack here tensorpack/tensorpack#1259 , by upgrading to the current tensorpack master and TF 1.13, and modify the code accodring to this
I close this issue and refer to the upgrade open issue #9


Select required error in google chrome

Required selects, only works fine in Firefox.
In Google Chrome we are having the following message:
"An invalid form control with name='name_of_select' is not focusable."

Right now I'm not having any problems in chrome, could you confirm is the problem is solved?
Closed due inactivity, feel free to reopen it if it is still necessary.
Thanks


when using removeStep(index) the next button was gone

I'm trying to use dynamic removing feature on steps by removeStep(index) and it works and the step was removed but it affects the next button on each step which was gone after using it.

Please fix this when possible, thanks :)

Sorry about the delay, and thanks for letting me know! I am hoping to publish a new version with a fix soon.

Fixed in the release 2.6.0.


LargeVis

LargeVis algorithm for k-nearest neighbor graph construction.

Looks great, I have dropped some comments, but I think they are minor.
Btw, it would be really cool if we had a Livebook comparing our different solutions here, similar to this image:

Source.

Looks great, I have dropped some comments, but I think they are minor.
Btw, it would be really cool if we had a Livebook comparing our different solutions here, similar to this image:

That would be the goal. It remains to be seen if the current implementation of k-NN graph construction will work for dimensionality reduction methods. One issue might be the memory cost of predict/2 in RandomProjectionForest. Currently, it uses $O(T \times N \times L)$ memory, for $T$ trees, sample size $N$ and leaf size $L$. This can be reduced to $O(N \times L)$ by looping over trees, but then I am afraid of #230 coming back to haunt me. :)

This can be reduced to by looping over trees, but then I am afraid of https://github.com/elixir-nx/scholar/issues/230 coming back to haunt me. :)

Probably worth giving it a shot anyway.

This can be reduced to by looping over trees, but then I am afraid of https://github.com/elixir-nx/scholar/issues/230 coming back to haunt me. :)

Probably worth giving it a shot anyway, especially as we allow precision to be given as parameter.

After all the feedback, this looks good to go. Just docs missing and I will merge it!

:green_heart: :blue_heart: :purple_heart: :yellow_heart: :heart:


conditional yLabel as Proportion, switching string to date variable bug

This addresses two issues: a) Update Y-axis labels for line plots #1050 and Viz: Error message in line plots when switching x-axis variable #1061.

For a): the Y-axis label is conditionally set to be Proportion in the line plot viz when Proportion mode is used. A example screenshot is shown in the following (SCORE S. mansoni Cluster Randomized Trial)

For b): this is similar situation to switching from number to date variable on scatterplot and lineplot y-axis causes an exception #1003. For the line plot viz, categorical and/or ordinal have string type and its value is string. For a date variable, its type is date but its value is also string: more exactly, date string. So when switching string type to date type, old values of sting from string type is used as a date string, which causes the error when computing axis range margin. To resolve this, a condition is set to check whether it is date type as well as date string value using Date.parse().

screenshot of Y-axis label as Proportion

Hi @moontrip - I don't see this bit implemented
For mean and median modes, add "Mean: " or "Median: "
For example, the label should look like "Mean: Age (years)"

@bobular Thank you for your suggestion. I will add them 👍

@bobular Yep as I replied at slack, I added the additional prefix for Mean and Median options. Here are screenshots. Are they alright for you?

Bob liked and approved my changes at slack so I will merge this PR


Website down

Not a technical issue, but rather having to do with the github repo info itself.
The website: http://martini.codegangsta.io/
referenced on: https://github.com/go-martini/martini
Seems to be down. Not sure if you are aware or if it has been moved. Wasn't sure where else to advise you of this.
Thanks.

That's something @codegangsta should look into.
Anyway, the framework is no longer maintained so I don't think it's worth it to put it back online.


Implement curryTwice

Closes: #16

A case pushing me to implement that functions is using common function to combine two asynchronous results like that:
const af1 = api.common.curry(fs.readFile, 'file1.txt');
const af2 = api.common.curry(fs.readFile, 'file2.txt');
const combine = (file1, file2) => (...);
af2.ap(af1.fmap(twiceCurry(combine)))((err, res) => (
...
));

But fullCurry got this case. fullCurry doesn't cover all cases of twiceCurry but I don't know real case for implement it after fullyCurry proposal.

@DzyubSpirit need more cases, because above can be easily solved with asynchronous composition.

Landed in 2fea99c


WSI instead of WIS?

Since we already have MSI for message-signaled interrupt, wouldn't it be more consistent to use WSI for wire-signaled interrupt, and not WIS for wired-interrupt signaling?

Approved by IOMMU TG - 9/29/2022

Addressed by PR #56


Develop

This pull request is about issue #16  . I added some description to the README page.
I also outlined some more guidelines on how to contribute to the project  in the CONTRIBUTIONS.md file

LGTM :white_check_mark:


chore: remove ember-cli-tslint and add tslint

Purpose
Speed up builds by removing tslint on build.
Summary of Changes
yarn remove ember-cli-tslint
yarn add -dev tslint
Side Effects
No more tslinting on build.
Feature Flags
n/a
QA Notes
n/a
Ticket
n/a
Reviewer Checklist

[ ] meets requirements
[ ] easy to understand
[ ] DRY
[ ] testable and includes test(s)
[ ] changes described in CHANGELOG.md

Pull Request Test Coverage Report for Build 2185

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

Totals

Change from base Build 2184:
0.0%

Covered Lines:
4869

Relevant Lines:
6382

💛  - Coveralls


[Bug/Feature]: When export on change is set, BetterBibTeX will interrupt modification of reference information

Debug log ID
NUJ43GJ7-apse/<IP_ADDRESS>41.5052-6
What happened?
A bug happened!
Hello,
Set export on change, and then import a new reference into the Zotero. As BetterBibTeX is preparing to export the library, we modify some information of the refence, then our operation for the reference will be broken.
Thanks.

What operation for the reference will be broken? I don't see any errors in the log, so I don't know what's going wrong or what you are doing.

What operation for the reference will be broken? I don't see any errors in the log, so I don't know what's going wrong or what you are doing.

Weird. I can't duplicate the bug. May be  the night I will meet it again.

Can you also first upgrade to 118?

Can you also first upgrade to 118?

Yes!

If you find the bug again and comment here, the issue will automatically reopen.


macro DRC problems

download the latest version from this repository today, and copy these files, however, the macros have a lot of DRC errors. (67 error DRC)

@juan2175606 Fixed in latest version

I downloaded the latest version from the repository and the problems continue

@juan2175606 Please send details of Pcell you have used to check


'Close Account' Facade Functions

See "Facade Tickets" in GH-1580

SPA PR is here: https://github.com/SAP/cloud-commerce-spartacus-storefront/pull/1956


MediatR one notification handler

So currently I have this scenario, where each handler will have 1:1 the same logic, so if i have 10 notifications / events that will mean I will also have 10 handler methods...
Smth like this
public Task Handle(TestResultTestParameterAdded notification, CancellationToken cancellationToken)
    => HandleTestResultEvent(notification, cancellationToken);

public Task Handle(TestResultTestParameterRemoved notification, CancellationToken cancellationToken)
    => HandleTestResultEvent(notification, cancellationToken);

public Task Handle(TestResultTestParameterValueChanged notification, CancellationToken cancellationToken)
    => HandleTestResultEvent(notification, cancellationToken);

public Task Handle(TestResultTestRemoved notification, CancellationToken cancellationToken)
    => HandleTestResultEvent(notification, cancellationToken);

public Task Handle(TestResultUnlinkedFromRequirement notification, CancellationToken cancellationToken)
    => HandleTestResultEvent(notification, cancellationToken);

public Task Handle(TestResultCustomExpectedResultCommentChanged notification, CancellationToken cancellationToken)
    => HandleTestResultEvent(notification, cancellationToken);

    public interface ITestResultContextSubscriber :
          INotificationHandler<TestResultRenamed>,
          INotificationHandler<TestResultTestCreated>,
          INotificationHandler<TestResultTestCommentChanged>,
          INotificationHandler<TestResultTestConceptChanged>,
          INotificationHandler<TestResultTestParameterAdded>,
          INotificationHandler<TestResultTestParameterRemoved>,
          INotificationHandler<TestResultTestParameterValueChanged>,
          INotificationHandler<TestResultTestRemoved>,
          INotificationHandler<TestResultUnlinkedFromRequirement>
          {
          }

Is it possible to just have one Handler, and maybe 1 event that has it's own unqiue handler ??
TestResultRenamed and all the other events listed inherit from TestResultEvent, which, in turn, inherits from DomainEvent, and DomainEvent implements INotification from MediatR.
I have tried the follwing but the handle never gets called.
services.AddScoped<ITestResultContextSubscriber<TestResultEvent>, TestResultContextSubscriber>();

public interface ITestResultContextSubscriber<in T> :
    INotificationHandler<T> where T : TestResultEvent
{
}

public class TestResultContextSubscriber : ITestResultContextSubscriber<TestResultEvent>{

      public Task Handle(TestResultEvent notification, CancellationToken cancellationToken) =>
           HandleTestResultEvent(notification, cancellationToken);
}

Thanks in advance !

Hello I may be misunderstanding your question but for me this code (as a program.cs) where I just create a non-generic INotificationHandler which implements INotificationHandler (which is my base event) seems to get called fine
using MediatR;
using Microsoft.Extensions.DependencyInjection;
using mytest;

// Create a new service collection and add mediatr
var services = new ServiceCollection();
services.AddMediatR(cfg => cfg.RegisterServicesFromAssembly(typeof(Program).Assembly));

// Build the service provider
var serviceProvider = services.BuildServiceProvider();

using var scope = serviceProvider.CreateScope();
var mediatr = scope.ServiceProvider.GetRequiredService<IMediator>();
mediatr.Publish(new Notification1());
mediatr.Publish(new Notification2());

internal record NotificationBase : INotification
{
}

internal record Notification1 : NotificationBase
{
}
internal record Notification2 : NotificationBase
{
}

internal class NotificationHandler1 : INotificationHandler<Notification1>
{
    public Task Handle(Notification1 notification, CancellationToken cancellationToken)
    {
        Console.WriteLine("NotificationHandler1");
        return Task.CompletedTask;
    }
}
internal class NotificationHandler2 : INotificationHandler<Notification2>
{
    public Task Handle(Notification2 notification, CancellationToken cancellationToken)
    {
        Console.WriteLine("NotificationHandler2");
        return Task.CompletedTask;
    }
}

internal class NotificationHandlerBase : INotificationHandler<NotificationBase>
{
    public Task Handle(NotificationBase notification, CancellationToken cancellationToken)
    {
        Console.WriteLine($"NotificationHandlerBase with {notification.ToString()}");
        return Task.CompletedTask;
    }
}

Works and I get the output below
NotificationHandler1
NotificationHandlerBase with Notification1 { }
NotificationHandler2
NotificationHandlerBase with Notification2 { }

You're not awaiting anything. Publish returns a Task and needs to be awaited. It works in a console because of the disposing at the end of the execution.

I've added the awaits to my answer.

Yes, I have the same problem. Notification solves 1-to-many handler problem, but the problem I have now is many events-to-1 handler. I've tried with playing around the event class' hierarchies, but some of my event just don't make sense from hierarchy point of view.
Any idea how to solve this?

Events to one handler? Why not create separate handlers that depend on a common class?


Remove restriction for header-only fmt

Conan's fmt is statically compiled by default, glaze should support this configuration as well.

Done, glaze now only relies on the macro FMT_HEADER_ONLY being provided externally, so it should work in either configuration.


tarhelper: Expand ignore capabilities

Expands the capabilities to specify files in a given tar:

Allows for subsequent regexps to reinclude files or directories
Allows for specifying that the file must also be a directory to qualify as a match

@zquestz @variadico @y0ssar1an

LGTM, only nits

OK to merge


Job not started after server reboot

Hello,
I created a service for my app like this :
$forever-service install ledcom --script ledcom_app.js
All work fine with start and stop but if i reboot the server the service do no start automatically.
Can you help me ?
Edit : command (update-rc.d ledcom defaults) give me this output update-rc.d: /etc/init.d/ledcom : file does not exist
Thanks

What OS you are using, have you checked logs in /var/log/ledcom.log file and analyzed if your service get executed and then stops for some reason?

My OS is Ubuntu ... I'm so much testing i must reinstall the server. I will communicate my ledcom.log file as soon as the system recovery is complete.

Here is my log
error: Forever detected script exited with code: 143
error: Script restart attempt #1
Ledcom Server Running at http://xx.xx.xx.xx:3001
error: Forever detected script exited with code: null
error: Script restart attempt #2
Ledcom Server Running at http://xx.xx.xx.xx:3001
I get this after reboot (fresh install)
If i type : status ledcom : ledcom stop/waiting
Thanks

Hello,
I tested to start a node app with and without express framework.
Result : i have the same error
error: Forever detected script exited with code: 143
error: Script restart attempt #1
error: Forever detected script exited with code: null
I really need a solution.
Thanks

Can you check the log file in /var/log/upstart also and send me the logs. Looks like the forever is attempting to start the service but failing for some reason.
Have you checked sudo forever list and see if it is showing your service as running?

One more thing to check, is your service dependent upon some environment variables? and may be during system restart it is not able to find those?

Here is my environment variables
root:~# printenv
TERM=xterm-256color
SHELL=/bin/bash
SSH_CLIENT=<IP_ADDRESS> 49924 22
SSH_TTY=/dev/pts/0
USER=root
LS_COLORS=rs=0:di=01;34:ln=01;36:mh=00:pi=40;33:so=01;35:do=01;35:bd=40;33;01:cd=40;33;01:or=40;31;01:su=37;41:sg=30;43:ca=30;41:tw=30;42:ow=34;42:st=37;44:ex=01;32:*.tar=01;31:*.tgz=01;31:*.arj=01;31:*.taz=01;31:*.lzh=01;31:*.lzma=01;31:*.tlz=01;31:*.txz=01;31:*.zip=01;31:*.z=01;31:*.Z=01;31:*.dz=01;31:*.gz=01;31:*.lz=01;31:*.xz=01;31:*.bz2=01;31:*.bz=01;31:*.tbz=01;31:*.tbz2=01;31:*.tz=01;31:*.deb=01;31:*.rpm=01;31:*.jar=01;31:*.war=01;31:*.ear=01;31:*.sar=01;31:*.rar=01;31:*.ace=01;31:*.zoo=01;31:*.cpio=01;31:*.7z=01;31:*.rz=01;31:*.jpg=01;35:*.jpeg=01;35:*.gif=01;35:*.bmp=01;35:*.pbm=01;35:*.pgm=01;35:*.ppm=01;35:*.tga=01;35:*.xbm=01;35:*.xpm=01;35:*.tif=01;35:*.tiff=01;35:*.png=01;35:*.svg=01;35:*.svgz=01;35:*.mng=01;35:*.pcx=01;35:*.mov=01;35:*.mpg=01;35:*.mpeg=01;35:*.m2v=01;35:*.mkv=01;35:*.webm=01;35:*.ogm=01;35:*.mp4=01;35:*.m4v=01;35:*.mp4v=01;35:*.vob=01;35:*.qt=01;35:*.nuv=01;35:*.wmv=01;35:*.asf=01;35:*.rm=01;35:*.rmvb=01;35:*.flc=01;35:*.avi=01;35:*.fli=01;35:*.flv=01;35:*.gl=01;35:*.dl=01;35:*.xcf=01;35:*.xwd=01;35:*.yuv=01;35:*.cgm=01;35:*.emf=01;35:*.axv=01;35:*.anx=01;35:*.ogv=01;35:*.ogx=01;35:*.aac=00;36:*.au=00;36:*.flac=00;36:*.mid=00;36:*.midi=00;36:*.mka=00;36:*.mp3=00;36:*.mpc=00;36:*.ogg=00;36:*.ra=00;36:*.wav=00;36:*.axa=00;36:*.oga=00;36:*.spx=00;36:*.xspf=00;36:
MAIL=/var/mail/root
PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games
PWD=/root
LANG=fr_FR.UTF-8
SHLVL=1
HOME=/root
LOGNAME=root
SSH_CONNECTION=<IP_ADDRESS> 49924 <IP_ADDRESS> 22
LESSOPEN=| /usr/bin/lesspipe %s
LESSCLOSE=/usr/bin/lesspipe %s %s
_=/usr/bin/printenv

Here is my log form UPSTART
^[[32minfo^[[39m:    Forever processing file: ^[[90mledcom_app.js^[[39m
ledcom started
^[[32minfo^[[39m:    Forever processing file: ^[[90mledcom_app.js^[[39m
ledcom started
Shutting down ledcom:
^[[32minfo^[[39m:    Forever stopped process:
^[[37m   ^[[39m ^[[37muid^[[39m    ^[[90mcommand^[[39m         ^[[90mscript^[[3$
[0] ledcom ^[[90m/usr/bin/nodejs^[[39m ^[[90mledcom_app.js^[[39m 1054    1058  $
ledcom shutdown
^[[32minfo^[[39m:    Forever processing file: ^[[90mledcom_app.js^[[39m
ledcom started
^[[32minfo^[[39m:    Forever processing file: ^[[90mledcom_app.js^[[39m
ledcom started

If i start manually my app (start ledcom) it work fine

Looks like the service start attempt is made by forever, you will have to debug your service to figure out what is going wrong.
What i suspect is some environment mismatch during automated service startup.
Are you using any environment variables in your script or dependent upon HOME env variable?
You can console the process.env.HOME variable from your service and check in logs if that is consistent between automated start as well as manual start. (Do this for any other Env variable you might be using).
Also paste your /etc/init/ledcom.conf file to see if anything problematic is present.

Here is my basic app
var express = require('express');
var app = express();
app.use('/assets', express.static(__dirname + "/assets"));

app.get('/', function(req, res) {

	var data = {
		title : 'Hello World !',
		description : 'home page'
	};

	res.render('index.jade', {data:data});
});

console.log('Ledcom Server Running');
app.listen(3001);

Here is ledcom.conf
#!upstart
#       /etc/init/ledcom.conf
#
#       ledcom - Provisioned using forever-service
#
#       CLI node /usr/local/bin/forever-service install ledcom --script ledcom_$
#       Working Directory /root
#

description "forever-service startup script for node script based service ledco$

env FOREVER_ROOT=/root/.forever

start on (local-filesystems and net-device-up IFACE=eth0)
stop on shutdown

expect fork

# Check if any of $pid (could be plural) are running
env LOGFILE="/var/log/ledcom.log"

# introduce some gaps between restarts and throttle continous restarts
env MIN_UPTIME="5000"
env SPIN_SLEEP_TIME="2000"

# kill signal: Since default needs to be SIGTERM, it is important that services$
# specially if they are doing transactions or other work which should not be in$
# for exceptional situation where you dont care about abrupt shutdown, SIGKILL $
env KILL_SIGNAL="SIGTERM"

# Wait time afer with SIGKILL will be sent to the process, in case SIGTERM is n$
# This is required since when we use SIGTERM, some times if there is problem in$
# or process may never exit, in such siutation we have to forcebly kill it so t$
# this wait time is in millisecond
env KILLWAITTIME=5000

chdir /root

exec  forever -a -l $LOGFILE --minUptime $MIN_UPTIME --spinSleepTime $SPIN_SLEE$

post-start script
        echo "ledcom started"
end script

pre-stop script

 echo "Shutting down ledcom: "

        STATUS=$(forever --plain list | sed 's/data:\(\s*\[[0-9]*\]\s*\(ledcom\$
        if [ -z "$STATUS" ]; then
                echo "Not running"
                return 0
        fi

        # PID=$(<$PIDFILE) - Changed to detection based on actual PID from fore$
        PID=$(forever --plain list | sed -n -e '/data:\s*\[[0-9]*\]\s\(ledcom\)$
        if [ -z "$PID" ]; then
                echo "Could not get pid"
                return 0
        fi

        forever stop ledcom &

        killtree() {
            local _pid=$1
local _sig=${2:--TERM}
            kill -stop ${_pid} # needed to stop quickly forking parent from pro$
            for _child in $(ps -o pid --no-headers --ppid ${_pid}); do
                killtree ${_child} ${_sig}
            done
            kill -${_sig} ${_pid}
        }

        CURRENTWAITTIME=$KILLWAITTIME
        # wait for some time before forcefully killing the process

# wait for some time before forcefully killing the process
        while [ $CURRENTWAITTIME -gt 0 ]; do
                #check if the process is still running
                if [ ! -d "/proc/$PID" ]; then
                        echo "ledcom shutdown"
                        # if not running we can break, since no more wait is ne$
                        break
                fi
                sleep 1
                CURRENTWAITTIME=$(( $CURRENTWAITTIME - 1000))
        done

if [ -d "/proc/$PID" ]; then
                killtree $PID 9
                echo "ledcom Forced shutdown"
        fi

end script

Now if i make my own ledcom.conf in etc/init and in attempt to start manually i get a error  : Unable to load "index.jade" in views (or something like this : i not remember exactly the message). But if i start manually with forever command (forever start ledcom_app.js) all work fine.

Can you also send me output of sudo forever list after the reboot.

After reboot and sudo forever list i get :
info:    No forever processes running

Please send me following information after ensuring you are running latest version of forever-service and forever, also give your exact version number of both forever and forever-service

Ensure ledcom service is not running, by checking in sudo forever list
If it is running stop it,
Delete /var/log/ledcom.log
Delete /var/log/upstart/ledcom.log
Reboot the machine, and after reboot is complete
Take output of the following /var/log/ledcom.log
/var/log/upstart/ledcom.log
sudo forever list

Ok guy, thanks for your support.

Thank you :-) you can close this post :-)


Clean output

Addresses https://github.com/bskinn/sphobjinv/issues/131#issuecomment-716789838
This results in messages being shown without clobbering stdout and reduces duplicated code.
See here: the first lines are stderr, the formatted JSON is the stdout after going though jq
$ python -m sphobjinv convert json -u https://rpy2.github.io/doc/latest/html/ - | jq

No inventory at provided URL.
Attempting "https://rpy2.github.io/doc/latest/html/objects.inv" ...
Remote inventory found.

{
  "project": "rpy2",
  "version": "3.3.1",
  "count": 294,
  "0": {
  ...
}

Codecov Report

Merging #136 into master will decrease coverage by 1.96%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master     #136      +/-   ##
==========================================
- Coverage   96.92%   94.95%   -1.97%
==========================================
  Files          12       12
  Lines         617      614       -3
==========================================
- Hits          598      583      -15
- Misses         19       31      +12

Impacted Files
Coverage Δ

src/sphobjinv/cmdline.py
88.97% <66.66%> (-3.64%)
:arrow_down:

src/sphobjinv/fileops.py
86.36% <0.00%> (-13.64%)
:arrow_down:

Continue to review full report at Codecov.

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

I do like how this revision to the stream writing tightens up the code. However, it totally obliterated the test suite, so I'd rather not merge it in the short term. Odds are my hacky stdio management in the test suite also really needs to be cleaned up, and would
I'll fix selective_print() and cut another alpha soon.

Yeah, the selective_print thing is just a one line fix anyway!
One could also thing about e.g. adding the final url to the JSON output instead of (just) logging it. That way it would be available programmatically.

Oooh, I like that. I even already have a spot in the JSON schema suitable for metadata like that. Will add!

v2.1a2 is up on PyPI -- the stdout content should be clean, and for JSON output (stdout and written file) that was drawn from a URL, the URL where the inventory was found is stored in result.metadata.url.
Let me know how it does for you!

Nodnod, I'm really glad I found it. Note also the .. program-output:: directive, which doesn't echo the command.
There are a number of useful options; see docs.


Add project for custom DTM with Intel JTAG

Hi there @stnolting
As we discussed, this pr adds my setup on how to use Intel specific JTAG as DTM. 🚂
I'm using a rather exotic (but nonetheless very cool!) FPGA board 🦎 from my university as I only have that for now. If you or someone else can test it on a more normal board I'd be happy to change the pin mappings and settings accordingly.
Let me know if I need to change anything (for example licence, copyright notice etc.). 😊

Hey Nik!
This looks really good - especially the documentation you've added! Great work! :+1:
Could you add your new setup to the font page's README as well?

Btw, I am using a DE0-nano FPGA board over here that also features Cyclone 4 FPGA. I will try to test your setup on my board.

Thanks for contributing!


82f5faa5f68215564ee9ae7dfd0d26bc6dd58325  Cannot build: ConfigError

Steps to reproduce
In Google Colab I tried to run npm install and got "Config Error".
https://colab.research.google.com/drive/10x_3XJIYKvjK9hxD3yMRmtSDuQXdnLZ_#scrollTo=09dUXTAr1D44
Failure Logs
ConfigError: Config (unnamed): Key "rules": Key "constructor-super": structuredClone is not defined
    at rethrowConfigError (/content/BgUtils/node_modules/@eslint/config-array/dist/cjs/index.cjs:303:8)
    at /content/BgUtils/node_modules/@eslint/config-array/dist/cjs/index.cjs:1098:5
    at Array.reduce (<anonymous>)
    at FlatConfigArray.getConfigWithStatus (/content/BgUtils/node_modules/@eslint/config-array/dist/cjs/index.cjs:1091:43)
    at FlatConfigArray.getConfig (/content/BgUtils/node_modules/@eslint/config-array/dist/cjs/index.cjs:1120:15)
    at /content/BgUtils/node_modules/eslint/lib/eslint/eslint.js:960:40
    at Array.map (<anonymous>)
    at ESLint.lintFiles (/content/BgUtils/node_modules/eslint/lib/eslint/eslint.js:958:23)
    at async Object.execute (/content/BgUtils/node_modules/eslint/lib/cli.js:502:23)
    at async main (/content/BgUtils/node_modules/eslint/bin/eslint.js:153:22)

Expected behavior
Not sure. I wanted to run the examples, instructions said to "build first", so I gave this a try. I don't know much about npm, javascript etc. I suppose I wanted to build it, then run the node example.
Current behavior
Build doesn't work, possibly my fault? Or Colab's?
Version
Default
Anything else?
No response
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.

Looks like the linter failed. Try npm run build:esm && npm run bundle:node.
I don't use Colab's much so I can't say for sure why it won't compile there.


adding comment re: temps < 1 and fft

adding a comment to explain why we drop temperatures < 1 when calculating the amplitude ratio and delta phi with the linear regression and set temperatures < 1 to 1 when calculating with fft

@SimonTopp ,
Thanks! I don't have that script on github. I could clean it up and add it. Right now it uses a prepped datafile (prepped_withGW.npz) so it wouldn't be easily runnable by someone else unless they ran the two prep steps first. Still worth adding it? I could also put the script and the output graphs in a word doc?

Not sure it really belongs here in river-dl proper, but I think if it can't be accessed it might not be worth mentioning the file name.  Instead I'd just qualitatively say we experimented and this was the best approach.


Use html tag / content in a stream template

In my app I have pages that use BigPipe and pages that don't.
I don't want to duplicate basic template tag in both format. e.g. analytics.scala.stream / analytics.scala.html or footer.scala.stream / footer.scala.html
In a classic html template I use the standard syntaxe :

in index.scala.html : @footer()
in build.sbt : TwirlKeys.templateImports ++= Seq("tags._")

If I do the same in detail.scala.stream I get the following error :

not found: value footer

I can use something like that
@{
  HtmlStream.fromHtml(views.html.tags.footer())
}

but it's a bit ugly and I have the feeling we can do better.
Next level to the question : I have a template with header/menu/footer. I want to use it for stream and html pages.
@(bigPipe   : BigPipe
, enterprise: Pagelet
, investors : Pagelet
, comments  : Pagelet
, documents : Pagelet
)(implicit request: RequestHeader, lang:Lang)

@scripts = {

}

@main("meta.application.index.title"
    , "meta.application.index.description"
    , "meta.application.index.keyword"
    , scripts) {

@{m("meta.project.detail.title")}

        @bigPipe.render { pagelets =>

              @pagelets(enterprise.id)
              @pagelets(investors.id)

              @pagelets(comments.id)
              @pagelets(documents.id)

        }
}

The above doesn't work for the same reason : not found: value main
If I try to use the following I get : expected start of definition
@HtmlStream.fromHtml(views.html.main("meta.application.index.title"
    , "meta.application.index.description"
    , "meta.application.index.keyword"
    , scripts) {

@{m("meta.project.detail.title")}

        @bigPipe.render { pagelets =>

              @pagelets(enterprise.id)
              @pagelets(investors.id)

              @pagelets(comments.id)
              @pagelets(documents.id)

        }
})

Let's start with the simpler case: @footer(). My guess at why you're getting a "not found" error is because you are missing the import. The .scala.html templates get compiled into the view.html package, so other classes in that package see them automatically. However, .scala.stream templates get compiled into the view.stream package, so you'd need to explicitly import the view.html package.
Try the following in a streaming template:
@import views.html.tags._

@footer()

The import works for the "not found" issue.
however when I just use @footer() I get the content escaped
<!-- FOOTER -->
<footer>
    <!-- footer content -->
    <div class="footer-content">

instead of

I can use

@HtmlStream.fromHtml(footer())
which is better. I also try an implicit converter from Html to HtmlStream without success

@implicitHtmlToStream(html:Html) = @{
  HtmlStream.fromHtml(html)
}

// this works
@implicitHtmlToStream(footer())

// this doesn't work
@footer()

OK, glad to hear the import thing works. You may want to add those imports to TwirlKeys.templateImports in your build.sbt file.
The escaping issue is something that would be good to fix. I suspect your approach with implicits should work. Let me experiment a bit.

Ah, actually, Scala won't bother to look for an implicit in this case. That's because the template gets compiled into a series of calls to a _display_ method that takes Any as an argument and returns an HtmlStream. We need some other way to inject this behavior...

Unfortunately, I don't see any way to fix this without a change to Play's Twirl template compiler. The core of the issue is that templates get compiled into a sequence of calls to the _display_ method in BaseScalaTemplate:
  def _display_(o: Any)(implicit m: Manifest[T]): T = {
    o match {
      case escaped if escaped != null && escaped.getClass == m.runtimeClass => escaped.asInstanceOf[T]
      case () => format.empty
      case None => format.empty
      case Some(v) => _display_(v)
      case xml: scala.xml.NodeSeq => format.raw(xml.toString())
      case escapeds: immutable.Seq[_] => format.fill(escapeds.map(_display_))
      case escapeds: TraversableOnce[_] => format.fill(escapeds.map(_display_).to[immutable.Seq])
      case escapeds: Array[_] => format.fill(escapeds.view.map(_display_).to[immutable.Seq])
      case string: String => format.escape(string)
      case v if v != null => format.escape(v.toString)
      case _ => format.empty
    }
  }

If the type of the item in your template is HtmlStream, it gets passed through unchanged. If it's Html, it skips down to the before-last case statement and calls format.escape(v.toString) on it, which is what leads to your HTML being escaped.
I can't find a way to modify this behavior. Implicits won't do it because the compiler has no reason to look for them. I can't override the _display_ method because I don't directly control the class that gets generated for my template. The twirl compiler handles that and, as far as I can see, offers no way to override methods or specify the base class.
The closest I've been able to come is to override _display_ directly in the body of my template:
@_display_(x: Any) = @{
  x match {
    case html: Html => HtmlStream.fromHtml(html)
    case _ => super._display_(x)
  }
}

@footer() // This will no longer be escaped

Defining this at the top of every template is a chore, but you can't import such a method from another class, or you get a "reference to display is ambiguous" error.
If anyone has any suggestions, please let me know. In the meantime, the workaround is to wrap calls to .scala.html templates (e.g. @footer()) with HtmlStream.fromHtml() (e.g. @HtmlStream.fromHtml(footer())).

Thanks for the quick answer. It looks like we can't do really better than @HtmlStream.fromHtml(footer()) so I'll stick we it
I try to reuse an html template (aka menu/footer) in a stream page and can't figure it out.
//detail.scala.stream
@(bigPipe   : BigPipe
, enterprise: Pagelet
, investors : Pagelet
, comments  : Pagelet
, documents : Pagelet
)(implicit request: RequestHeader, lang:Lang)

@main("meta.application.index.title"
    , "meta.application.index.description"
    , "meta.application.index.keyword") {

Coucou

    //I try also @Html("
coucou
") without success
}

//main.scala.html
@(title      : String
, description: String = ""
, keywords   : String = ""
, scripts    : Html   = Html("")
, styles     : Html   = Html("")
)(content: Html)(implicit request: RequestHeader, lang:Lang)

<!DOCTYPE html>

    <title>Title</title>

@content

I always get

type mismatch; found : com.ybrikman.ping.scalaapi.bigpipe.HtmlStreamFormat.Appendable (which expands to) com.ybrikman.ping.scalaapi.bigpipe.HtmlStream required: play.twirl.api.Html

In a .scala.stream template, all markup will be wrapped in an HtmlStream. For example, the following template:
@main(...) {

Some markup

}

Gets roughly compiled to:
main(...)(format.raw("""
Some markup
""")

The format is defined by the template type, and in this case, it's HtmlStreamFormat, who's raw method returns an HtmlStream. Since your main template is looking for Html, this leads to a compile error.
There are two possibilities:

The content you want to pass into the main template is contains streaming content (e.g., pagelets). In this case, there is no way to reuse the main template directly, since it returns Html and the only way to use streaming is to return HtmlStream. Create a new streaming main template. If you want to reuse parts of the non-streaming one, break those into separate helper templates such as head.scala.html, nav.scala.html, footer.scala.html, etc (this is a good practice anyway).
The content you want to pass into the main template is completely static HTML. In that case, put it into a separate .scala.html template and pass it to main:

Some markup

@main(...)(staticMarkupForMain())

Thanks for your response. I think I understand way better how template works now
I try to reverse the process : define template in stream format and use it from html page
@content() = {

coucou

}
@views.stream.main("...")(HtmlStream.fromHtml(content()))

But it only display a toString version of the HtmlStream

com.ybrikman.ping.scalaapi.bigpipe.HtmlStream@718e5a8a

With the complexity of mixing 2 types of templates, is going full stream template a good option ? Do you see any drawbacks ?
The first I can see is when using external libs (e.g. play form helper) we need to wrap it.

A .scala.stream tempate will return an HtmlStream object, which is a wrapper for an Enumerator[Html]. There is no way to include those inside of a .scala.html template, since those return an Html object, which is a wrapper for a plain old String.
In other words, if the base template you're rendering is a .scala.stream template, you can include .scala.html templates within it by wrapping them in an HtmlStream.fromHtml(...) call:
// Controller
def index = Action {
  Ok.chunked(views.stream.main(...))
}

  @someStreamingTemplate()

  @HtmlStream.fromHtml(someHtmlTemplate())

If the base template you're rendering is a .scala.html template, you can only include other .scala.html templates within it:
// Controller
def index = Action {
  Ok(views.html.main(...))
}

  someHtmlTemplate()

Therefore, it's more flexible to use .scala.stream as the base template in your app. I would still build static HTML blocks using .scala.html templates, as it communicates the intent better and makes them more reusable elsewhere (e.g. in tests). And you can reuse static HTML templates from external libs by wrapping them in HtmlStream.fromHtml(...) calls.


base model for asian-bart-ecjk

Which model was asian-bart-ecjk fine-tuned from? bart-large? Which data was used for fine-tuning? I can't find these information in the model card page of huggingface. https://huggingface.co/hyunwoongko/asian-bart-ecjk

I was trying to find answers for the same questions but the author of asian-bart-ecjk doesn't reply back.
Ref: https://huggingface.co/hyunwoongko/asian-bart-ecjk/discussions/1


ASCII Value

Title of programme
Write a JavaScript programme to print ASCII Value of a character.
Additional information
Input  : A
Output : 65

Input  : a
Output : 97

Code of Conduct

[X] I have read and agree to the Codinasion project's Code of Conduct

Can you assign this to me? I have a solution


libfprint-focaltech-2808-a658: init at 1.94.4

Added fingerprint sensor support for focaltech fingerprint scanner 2808:a658.
Discourse discussion: https://discourse.nixos.org/t/request-for-libfprint-port-for-2808-a658/55474/30?u=imsick
Discussion on source: https://gitlab.freedesktop.org/libfprint/libfprint/-/merge_requests/413
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Added fingerprint support for Realtek USB2.0 Finger Print Bridge FocalTech Fingerprint Device for x86_64 architecture.

Vendor ID
Product ID

0x2808
0xa658

This PR also closes

#353860

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/request-for-libfprint-port-for-2808-a658/55474/30

Update: Moved pname and version inside the derivation body instead of defining and inheriting right below.

Could you once again verify that it works with exactly this revision of Nixpkgs?

Yes, verified. Works with no errors.


.pth files generated by buildPythonPackage do not include transitive dependencies

Let's say I have an expression like so:
with import <nixpkgs> {};
rec {
  foo = buildPythonPackage {
    name = "foo";
    src = /path/to/foo;
  };
  bar = buildPythonPackage {
    name = "bar";
    src = /path/to/bar;
    propagatedBuildInputs = [foo];
  };
  baz = buildPythonPackage {
    name = "baz";
    src = /path/to/baz;
    propagatedBuildInputs = [bar];
  };
}

Now, if I do nix-env -iA bar, then of course it will first build foo and then bar, and in my ~/.nix-profile/lib/python2.7/site-packages folder, I will see a bar-propagated-inputs.pth (more or less) containing something like /nix/store/23984vhdbji34589unsdo67234y-python2.7-foo/lib/python2.7/site-packages.
But now let's say that without installing bar, I instead did nix-env -iA baz. It will properly build foo and bar, but now in ~/.nix-profile/lib/python2.7/site-packages I will see a baz-propagated-inputs.pth file, which references /nix/store/23o48hdfio87245h96unr6-python2.7-bar/lib/python2.7/site-packages, but does not reference the path to foo's site-packages. This means that, if one is using python's site package to automagically load paths, only the paths for bar and baz will appear in sys.path, and not foo, which almost certainly will mean that using either of those modules will result in a runtime error.
TL;DR: Either:

the .pth file generated by a python package should contain not only its own dependencies, but those of its dependencies as well, or
Doing an installation of a python package should add to the nix profile not only that package's .pth file, but those of its dependencies as well.

.pth files are now gone in #11086

@domenkozar I guess we should close this issue?

@adnelson I recommend usage of nix-shell or python.buildEnv. Let me know if you have some questions


CMake build broken with 9b7aa03

9b7aa03 renamed mir-run and related files to mir-bin-run. While this change was reflected in GNUmakefile, CMakeLists.txt still refers to the now missing mir-run.c. As a result, mir can no longer be built with CMake.

Thank you for reporting this.  Fixed by 927e9d0f2012b9843548f1da83f9c1535f857b74.

Can confirm, problem is fixed


Adding zone balancing support to template

Waiting to merge this until module is merged, so I can update the ref


Draft: Add datadog support

Sending it out so that I could get feedback/suggestions/thoughts as I work through the remaining items.
TODO:

Add tests
Test for backward compatibility

I have rebased my changes and fixed the test cases.
TODO: add datadog reporter tests

@KevinJMao ok, new tests added. Can you help review the change? thanks!


Notify account access by email.

The following information will be passed:
IP
Device
Parents
Date and time

Developing


How do I know my version?

Hi all,
How do I know my version? I started to work in a new company and they already have it but I do not know what version they have.
Kind regards,
Alexander

Hi @acanton5,
Thank you for raising the query, you can find the version installed in the about tag of the application, screenshot below for reference,

Thank you a lot :)


Slash command doesn't implement signing_key verification

https://github.com/nlopes/slack/blob/master/security.go
https://github.com/nlopes/slack/blob/master/slash.go
I'm trying to write an implementation of this, but I can't figure out how to run ParseForm and read the body (in order to do the verification), but it seems I can't do both?

what do you need to do the verification that requires reading the body of the request? but you can likely copy the request and read the body from one, form from the other.
since I havent looked at the signing key verification stuff i'm lacking context =)

I might have figured it out.. maybe. The problem is that the signing key verification needs access to the body, but so does the ParseForm function on the request. I think I figured out that once I'm done reading the body for verification, I can restore it before calling ParseForm

I can submit a PR to have slash commands perform the signing request validation, but likely there will be a different API to support that, as it requires the signing request secret to be passed in.

so ghost I'm just going to need to see code before I can comment on anything for this really. feel free to submit a PR.

@ghostsquad are you thinking of something like this?
https://github.com/alyosha/slack-utils/blob/master/verify.go#L40

Yes

closing out, resolved.

@james-lawrence where's the verification? I'm not sure how this is fixed.

ah, I misunderstood the link thought it was to a method in this repo. I'd be happy to add a similar function to slackutilx.

This seems related to a pull request I submit yesterday but this seems like it might be the preferred way to handle this situation by having it run the verification and parse the command in one step?
https://github.com/nlopes/slack/pull/541


release-cross.nix: test cross compilation to x86_64-netbsd

Description of changes
PR #138711 added NetBSD. FreeBSD has come a long ways since the last release. It would be good to have this cached for 23.05.
Things done

Built on platform(s)

[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.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

[ ] Fits CONTRIBUTING.md.

release-cross.nix: test cross compilation to x86_64-netbsd

I assume you meant x86_64-freebsd?

oh yeah, whoops.


IXtermTerminal._target isn't used

There is this interface which doesn't do anything.
https://github.com/microsoft/vscode/blob/61c195538be4ebacb3640f9de7fe56695fa0c275/src/vs/workbench/contrib/terminal/browser/terminal.ts#L1000-L1003
It's called by TerminalInstance which does it's own caching of the target value anyway:
https://github.com/microsoft/vscode/blob/61c195538be4ebacb3640f9de7fe56695fa0c275/src/vs/workbench/contrib/terminal/browser/terminal.ts#L1000-L1003

It is actually used but it's still a code smell how there's 2 places the location is stored. I don't think XtermTerminal shouldn't know about vscode views.


Contracts.SeverityLevel should not be an enum

ApplicationInsights library code uses an enum here:
node_modules/applicationinsights/out/Declarations/Contracts/Generated/SeverityLevel.d.t
/**
 * Defines the level of severity for the event.
 */
declare enum SeverityLevel {
    Verbose = 0,
    Information = 1,
    Warning = 2,
    Error = 3,
    Critical = 4
}
export = SeverityLevel;

I don't know if there's a particularly good reason to use an enum here, I'd prefer this to be a const.
Here's my (wip) "user-land" code
type SeverityLevel =
  import('applicationinsights').Contracts.SeverityLevel

// IDEALLY: derive this union from import('applicationinsights').Contracts.SeverityLevel
// ...Appears to not be possible: https://stackoverflow.com/questions/63758383/how-do-i-a-union-type-of-string-values-from-an-enum
type SimpleStrictAiSeverityLevelUnion =
  | 0
  | 1
  | 2
  | 3
  | 4

let a: SeverityLevel = 999999

https://stackoverflow.com/questions/63758383/how-do-i-a-union-type-of-string-values-from-an-enum
Q: Is there a way to use this enum on my last line, let a = 9999, which will cause this to be a type error?
There is a similar issue, but it seems to be focused on getting the enum values at runtime: https://github.com/microsoft/ApplicationInsights-node.js/issues/55

Probably we should just move away from using enums... this is not even a type-error:

SL has type string

This is what I'm going with for now:
// Ideally: SeverityLevel is not an enum, see: https://github.com/microsoft/ApplicationInsights-node.js/issues/942
export type strictAiSeverityLevel =
  | 0
  | 1
  | 2
  | 3
  | 4


Can not remove missing node status

Describe the bug
After a node is marked missing increasing the missing nodes threshold to include that node does not remove the missing status.
To Reproduce

Ingest a node that has an end time around two days ago.
Go to the Node Lifecycle https://a2-dev.test/settings/node-lifecycle and update the "Missing Nodes" to 24 hours.
Go to the client runs page https://a2-dev.test/infrastructure/client-runs and ensure the node is missing.
Go back to the Node Lifecycle page and update the "Missing Nodes" to 7 days.
Go back to the client runs page and ensure the node is still missing.

Expected behavior
The expected behavior is that when a user updates the missing node threshold to included an already marked missing node to convert that node to not missing.
Versions (please complete the following information):

OS: docker
Browser chrome
Automate Build Number<PHONE_NUMBER>1643

Additional context
The only way to remove the missing node status is by ingesting a new Chef Client Run.

let's make sure product agrees this is a bug/problem?


Does it not work on Raspberry Pi 3 B+?

I have a Raspberry Pi 3 B+.
I downloaded the image file, installed it on the Raspberry Pi, and booted it, but there was no response when connected to the car.
So, I looked it up and found that the device in question does not have OTG mode. Is this correct information?
Is there any solution?

You're correct, Raspberry Pi 3 B+ does not support OTG or USB gadget mode. I don't think there is anything much we can do about that. You will need to use one of the other supported devices.


Fix #17: NewReader can't process data bigger than 8K

This shoud also fix issue #25: When the amount of data will be truncated
iconv() will return EINVAL when An incomplete multibyte sequence is
encountered in the input, and the input byte sequence terminates after
it. So if the input is larger than the internal buffer of Reader and
the end of the buffer conatins partial multi-byte chars, then Reader
will failed with EINVAL.
So when iconv() return EINVAL, we checks whether there are more data to
process, if so, we continue without report an error to user.

Assume an input stream which return a single byte in fillBuffer(), in this case, Read() will return an error, but actually this is not an error, so we must try to pull more data and return
error only if there is no more new data.
Moreover, in case of EINVAL, there is remaining data in buffer and fillBuffer() will not be then we will loop forever.
IMHO, the simplest way to deal with EINVAL is just try to read more data to determine whether this is an actual error.

@MoZhonghua sorry,I aslo can't  process data bigger than 8K,can you give me a demo?

@alphayan see #28


"//go:generate" comment moved to GOOS independent file

To avoid cross platform go generate issue
See https://github.com/golang/go/issues/16368

I never tried to run mksyscall from non-windows OS, but it seems to be working well, very handy.
Thanks for your contribution!
PS: It seems that this pull request got some more unrelated commits, BTW I've merged just 378aafb into mainstream.


Data split, prompt, and model predictions for CLUTRR

Hi authors, thanks for the great work! Would it be possible to share more details about the CLUTRR dataset, e.g. the data split, the prompt you used, and the model predictions?
Thanks!

Hi @veronica320 , I've just added this page https://github.com/microsoft/CodeT/blob/main/DIVERSE/data/clutrr.md
Let me know whether it works for you or not.


macos support

[23:45]	Moosems	Akuli Could you add mac support? (my formal request lol)

Jou installs great and the hello world works :).

Some docs on the language would be nice 😄.


CONTRIBUTING.md is missing Rust and Typescript information

The contribution guide is missing information detailing Rust and Typescript.

@desa you want to take a crack at the Typescript stuff?

The query team needs to update the README and/or CONTRIBUTING with Rust info, so I filed an issue for it: https://github.com/influxdata/influxdb/issues/16372

Rust is added, but not typescript.

Ui has moved out of this repo so i'm closing this for now.


Adding support for keys that are not in dot notation

Just a simple argument, then it will skip the dot notation step and do everything else the same!

Hi @Spittal!
Sounds good!
Could you please solve this conflict?
It would help :)
Thanks!

Conflicts fixed!

Thank you very much!


Add Web API template support

I have implemented add web api template support option for express generator for quickly create simple web api without view.

Could you please help trigger rebuild for travis-ci? I did tried build in my side it passed.

Anyone review this?

I want to have this!!!

I want this too :D

Currently developers use their own structure / architecture when developing APPs or APIs using the ExpressJS this might leading to bit confusing for other developers when working on someone else developed APPs. It would be great if you can come up with a new boilerplate for both APPs and APIs separately or together.
This will help stick in one path for the whole community together. TIA.

When will this be merged?

Please merge this.

Hey Everyone!  Sorry that this has sat with no response for so long.  I will take a look this weekend, but I see no reason this at the surface that this should be turned down.
That said, I have been working a bit behind the scenes to build a new generator for the project which will hopefully replace this.  There are many reasons which I have not well documented, and some of which might be contentious, but I promise I will document them.  This re-write is one of the reasons I have not prioritized following up on these issue (which I think was a mistake).
If anyone else wants to help with the process of triaging and getting these lined up for a release on this major version line, the help would be greatly appreciated.  The best way to get started being involved is to join our Triage team (cc @expressjs/triagers).  You can read more about that here: https://github.com/expressjs/express/pull/4055

Ultimately I think we just need to get the new generator going and just not land these types of enhancements to this generator at all.

@dougwilson while I do agree that is the priority, I am sad that this basically means that it is unsupported until I have free time to finish the new one.  If someone can step up to triage and get these all sorted, I am fine managing a few point and patch releases so that things do not continue to stall like they have been.

Anyway, there is my long, overdue review :)

I am sad that this basically means that it is unsupported until I have free time to finish the new one.

I'm not sure what exactly is unsupported? I am supporting many of the repos like this... in my spare time just like everyone :D


Shapely 1.7.a2 and newer versions slow down

Hi,
I'm working on a project using Shapely (currently 1.6.4.post2), and I tried to update my version of Shapely. However, I noticed a huge performance decrease using the most recent release.
After a quick investigation, I noticed this problem appears for all version of Shapely >= 1.7a2.
Steps to reproduce the problem.
Example :
from shapely.geometry import Polygon
import shapely
import time

p1 = Polygon([[0,0],[5,0],[5,5],[0,5]])
p2 = Polygon([[0,0],[5,0],[5,5],[0,5]])

t1 = time.time()

for i in range(10000):
	p1.difference(p2)

print('Execution time :',time.time()-t1,'seconds')

This small piece of code runs in ~0.6 second on my computer using older versions (<=1.7a1).
Using 1.7a2 and newer versions of Shapely, this same code runs in 2.6 seconds.
Operating system
Fedora 30
Shapely version and provenance
Versions >= 1.6.4.post2 installed with pip.
I'm using python 3.7.5.
This might be related to #834

@ELIONET thank you for the report and the script. That method of timing isn't reliable. I recommend the timeit module. Below I'm using it in two different Python environments on my macbook.
1.6.4.post2
$ python -m timeit -s "from shapely.geometry import Polygon" -s "p1 = Polygon([[0,0],[5,0],[5,5],[0,5]])" -s"p2 = Polygon([[0,0],[5,0],[5,5],[0,5]])" "p1.difference(p2)"
10000 loops, best of 3: 106 usec per loop
$ pip list | grep Shapely
Shapely            1.6.4.post2

1.7.0
$ python -m timeit -n 10000 -r 3 -s "from shapely.geometry import Polygon" -s "p1 = Polygon([[0,0],[5,0],[5,5],[0,5]])" -s"p2 = Polygon([[0,0],[5,0],[5,5],[0,5]])" "p1.difference(p2)"
10000 loops, best of 3: 157 usec per loop
$ pip list | grep Shapely
Shapely       1.7.0

I see a 50% slowdown. The wheels for 1.6.4.post2 include GEOS 3.6.2. The wheels for 1.7.0 include GEOS 3.8.0. It is possible that the newer GEOS is slower or I am building it in an unoptimized way, and I will check into both of these possibilities.

In #834 I see signs that this is caused by GEOS 3.8.0, not by anything in Shapely.

For what it's worth, I've done the same timeit tests on a Windows computer with Anaconda, and here's a summary of my data:
conda create --no-default-packages -n geos-env python
conda activate geos-env
conda config --add channels conda-forge
conda install --yes shapely==1.6.4 geos==3.7.2
# 2000 loops, best of 5: 83.3 usec per loop
conda install --yes shapely==1.6.4 geos==3.8.0
# 5000 loops, best of 5: 62.7 usec per loop
conda install --yes shapely==1.7.0 geos==3.8.0
# 5000 loops, best of 5: 48.6 usec per loop

so I'm only seeing good news with newer versions. But this all depends on how the conda-forge folks have prepared the packages.

Thanks @mwtoews. How curious.

I also see a speed up in my environments (conda forge on linux):
Shapely 1.6.4, GEOS 3.7.2:
In [3]: %timeit p1.difference(p2)
42.1 µs ± 366 ns per loop (mean ± std. dev. of 7 runs, 10000 loops each)

Shapely 1.7.0, GEOS 3.8.0
In [3]: %timeit p1.difference(p2)
28.3 µs ± 322 ns per loop (mean ± std. dev. of 7 runs, 10000 loops each)

Haven't seen other reports of poor performance in GEOS 3.8, so a bit puzzled by this. GEOS CMake even defaults to a Release (optimized) build now, so I'm not sure how you could end up with an un-optimized build.

That said, the optimization focus is usually towards improved performance of expensive operations. Possible that newer versions are slower when taking the difference of two squares.

I'm going to test whether the lack of these lines https://github.com/rasterio/rasterio-wheels/blob/master/config.sh#L6-L7 in the shapely-wheels builds is involved. Will report here soon.

@dbaston could you have a look at the example in https://github.com/Toblerity/Shapely/issues/834#issuecomment-589570875 ?
Because that is one that I actually could reproduce (in contrast to the example above), and it's a buffer of a complex polygon that is much slower in GEOS 3.8

I've tested some new wheels that incorporate these changes to the GEOS build https://github.com/shapely/shapely-wheels/commit/df1f48c8eaf677d7a5af5a47a414e1d71bb44eec and see this:
$ python -m timeit -n 10000 -r 3 -s "from shapely.geometry import Polygon" -s "p1 = Polygon([[0,0],[5,0],[5,5],[0,5]])" -s"p2 = Polygon([[0,0],[5,0],[5,5],[0,5]])" "p1.difference(p2)"
10000 loops, best of 3: 61.8 usec per loop

I'm going to close this issue. There's no performance regression in Shapely that I can see. If there is one in GEOS related to simple polygon differences, it only seems to occur in unoptimized builds.

To repeat again: it's #834 that has an actual slowdown example (but that issue was closed in favor of this one)

@jorisvandenbossche that seems worth looking into on the GEOS side, I just don't have time to do so at the moment.

I suppose it is then worth opening an issue on the GEOS side (can I open an issue on github, or is trac preferred?)

Sure, either is fine

@jorisvandenbossche Actually went ahead and tried this in GEOS directly. Not seeing any slowdown in the buffer...57 usec in master and 70 usec in 3.7.

@dbaston thanks, that's interesting. I tried with another python binding (pygeos), and also there seeing the slowdown (with the only difference being the GEOS version).
The main difference with your example is that Shapely/PyGEOS are using GEOSBufferWithParams/GEOSBufferWithStyle, so it might depend on one of those defaults.

Maybe conda builds are also unoptimized. See https://github.com/conda-forge/geos-feedstock/pull/43

Indeed, I built GEOS locally, and then my equivalent PyGEOS example is also much faster:
t = np.arange(1, 10000)
arr = pygeos.points(100000*np.sin(t), 100000*np.sin(4*t))
arr2 = pygeos.buffer(arr, radius=200, quadsegs=32)
union = pygeos.union_all(arr2)

%timeit pygeos.buffer(union, radius=1)

3+ seconds with GEOS 3.8 from conda-forge, ~60ms with GEOS master build from source (using the release build type)

this might be interesting:  http://blog.light42.com/wordpress/?p=3634


Modify Wording on memberSHPE page

The wording should just be pay for the t-shirt.

https://github.com/TAMUSHPE/MobileApp/pull/463/files


Rename validateEmail options to align with detectBot and sensitiveInfo options

The validateEmail currently uses block to indicate which email types to deny. We should change this to deny like detectBot and sensitiveInfo now use.
Also, should we add an allow variant here? cc @davidmytton

Yes I think we should match the API of the others, including allow.


Remove unused variational::base_family class

The base class variational::base_family has no virtual methods and the one function it does define uses no member variables or functions, so can be made standalone outside of the class.  Therefore, base_family should be removed.

[ ] remove src/stan/variational/base_family.hpp
[ ] move function write_error_msg_() into its own file and rename without suffix _
[ ] patch families/normal_meanfield.hpp and families/normal_fullrank.hpp:

[ ] remove references to base_family
[ ] change calls to write_error_msg_ to include and call new function

If the intent was to somehow pull down some of the functionality from the subclasses, then this issue should be repurposed to do that.

Given recent work done in #2618 that has added some virtual methods to variational::base_family, perhaps this issue should be repurposed to move shared functionality (there is a little) to the base class.
@avehtari Thoughts?

See specifically this comment https://github.com/stan-dev/stan/pull/2618#issuecomment-473595318 which contains the justification for the changes I made and why certain things are not possible.


build & store statistics pages

@asmuth  ZJONSSON's fork has statistics implemented. parquetjs-lite https://github.com/ZJONSSON/parquetjs
I realize this post is quite old, but might be useful for others that stumble on it.


Portal CMS backend

[ ] Make proof-of-concept back-end
[ ] Apply DB schema
[ ] Implement authentication (mithril integration)
[ ] Implement file handling (ael integration)

not relevant anymore


Front End Teardown Success Bug Fix

Fixes a bug where the terraform destroy command will always return a 0 exit status code because of the successful pipe to tfdestroy.log

/gcbrun


Anchor Test Failed after generating project with dashed name

I had a minor error when working through the minimal example.

Create project with dash name: anchor init fancy-project
Run anchor test
See error: TypeError: Cannot read property 'state' of undefined

Cause:
Looking at the compiled version workspace.js#attachWorkspace. I added logs for clarification.
function attachWorkspaceOverride(workspaceCache, overrideConfig, idlMap) {
    Object.keys(overrideConfig).forEach((programName) => {
        const wsProgramName = camelcase_1.default(programName, { pascalCase: true });
        const entry = overrideConfig[programName];
        const overrideAddress = new web3_js_1.PublicKey(typeof entry === "string" ? entry : entry.address);

        let idl = idlMap.get(programName);
        console.log('programName', idlMap.keys());
        if (typeof entry !== "string" && entry.idl) {
            idl = JSON.parse(require("fs").readFileSync(entry.idl, "utf-8"));
        }
        workspaceCache[wsProgramName] = new program_1.Program(idl, overrideAddress);
    });
}

The output was as follows:
fancy-project
programName [Map Iterator] { 'fancy_project' }

You can see that the requested program name does not match the entry in the IDL map.
One uses hyphens, the other uses underscores.

Same goes for capitalized names like "MyAppName" it will be built as "my_app_name" in the IDL folder.

A quick workaround here is to change the name in the Anchor.toml to be snake case.

Thanks Armani, I suppose the best course of action would be to normalize
the names when the project is generated?
On Sat, Sep 11, 2021 at 13:32 Armani Ferrante @.***>
wrote:

A quick workaround here is to change the name in the Anchor.toml to be
snake case.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/project-serum/anchor/issues/706#issuecomment-917453051,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAELMHT7WAZ4S2IX52DAFXLUBOOEZANCNFSM5D2MISNA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

That would be a good fix for now.

Addressed by https://github.com/project-serum/anchor/pull/711.

projects with dashes in their name seem to work fine now, so im closing this