Rails 4.x compatibility

I just had the issue that on windows there only is threading and in lib/spawnling.rb in sel.frun there is a check for "allow_concurrency?" that fails and then the line
@@logger.error("spawn(:method=>:thread) only allowed when allow_concurrency=true")
fails because @@logger isn't there. So all i got was:
D:/Informer/ruby/lib/ruby/gems/2.0.0/gems/spawnling-2.1.3/lib/spawnling.rb:112:in run': undefined methoderror' for nil:NilClass (NoMethodError)
Then clean_connections failed at "verify_active_conenctions!" strangely so because i'd have expected the "respond_to?" check catching that.
I quickly monkey patched it this way:
class Spawnling
def self.allow_concurrency?
true
end
def self.clean_connections
ActiveRecord::Base.clear_active_connections!
end
end
But i guess you should check and set allow_concurreny true if rails >= 4.0.0.
And the clean_conenctions should somehow call the ConnecteionPool Reaper but i haven't yet figured out myself how to do that.

I was looking at this repo in consideration for a Rails 4 app as well.  Is this something that might be fixed?

@neongrau @mhuggins We currently use this on a rails 3 app, so we haven't gotten around to fixing it ourselves. I welcome any PR that passes the tests (and adds rails4 to the gemfiles folder and .travisci.yml).

Spawnling doesn't work with Rails 4, I get errors like this in the code after the fork:
ActiveRecord::StatementInvalid: PG::ConnectionBad: PQconsumeInput() SSL connection has been closed unexpectedly
I've done a lot of reading and experimenting to try and fix Spawnling and the only thing that I've tried that's worked so far is something like this:
ActiveRecord::Base.connection.disconnect! if defined?(ActiveRecord)
fork do
  ActiveRecord::Base.establish_connection      # in patches.rb
  yield      # run code inside Spawnling.new{}
end
ActiveRecord::Base.establish_connection if defined?(ActiveRecord)

This lets both the forked code and original code keep using Active Record without error, but I can't tell if it's leaking connections. Does anyone have a test for that?
ActiveRecord::Base.connection_pool.send(:current_connection_id) stays the same before, after and during the fork.
Much of what I read seems to be false with regards to using ConnectionPool. For example, using ActiveRecord::Base.connection_pool.with_connection { yield } seems to keep using the original connection - because code after the fork gives the PG error again. I am trying to fork, not Thread.new, so that might be an issue.
In any case I am going to move to a different queueing solution now, but I'm interested to know how all this is supposed to be done.

i encounter a similar error but not in spawnling, but in delayed_job and
sometimes in passenger, the only clue i get is that the cause could be from
the pg gem version used, because i don't encounter this issue on my other
rails app, also worth mentioning is that this only happens on my
development laptop not on production
On Tue, Feb 9, 2016 at 10:27 AM, Leslie Viljoen<EMAIL_ADDRESS>wrote:

Spawnling doesn't work with Rails 4, I get errors like this in the code
after the fork:
ActiveRecord::StatementInvalid: PG::ConnectionBad: PQconsumeInput() SSL
connection has been closed unexpectedly
I've done a lot of reading and experimenting to try and fix Spawnling and
the only thing that I've tried that's worked so far is something like this:
ActiveRecord::Base.connection.disconnect! if defined?(ActiveRecord)
fork do
ActiveRecord::Base.establish_connection      # in patches.rb
yield      # run code inside Spawnling.new{}
end
ActiveRecord::Base.establish_connection if defined?(ActiveRecord)
This lets both the forked code and original code keep using Active Record
without error, but I can't tell if it's leaking connections. Does anyone
have a test for that?
ActiveRecord::Base.connection_pool.send(:current_connection_id) stays the
same before, after and during the fork.
Much of what I read seems to be false with regards to using
ConnectionPool. For example, using ActiveRecord::Base.connection_pool.with_connection
{ yield } seems to keep using the original connection - because code
after the fork gives the PG error again. I am trying to fork, not
Thread.new, so that might be an issue.
In any case I am going to move to a different queueing solution now, but
I'm interested to know how all this is supposed to be done.
—
Reply to this email directly or view it on GitHub
https://github.com/tra/spawnling/issues/53#issuecomment-181673512.

--
“I know that you believe you understand what you think I said, but I'm not
sure you realize that what you heard is not what I meant.” –Richard Nixon

Since i've upgraded from Rails 3 to 4.x i'm too having unexplainable DB issues (disconnects, hangs due to no more connections available). When cranking up the the pool size in database.yml this gets fairly rare. But still occurs way too often for my taste (in production mainly).
I've tried wrapping ActiveRecord::Base.connection_pool.with_connection { yield } blocks around anything where spawnling is used but that never really helped to get rid of the problem.

Hi guys, we no longer use spawnling in our work, we stopped using it during a rails 3 app, so we honestly won't be maintaining it anymore, BUT I'm happy for some of you to take on the role of maintaining the gem (it's hardly a big complex gem like many others to maintain).
I will continue to review and merge PR's as they come in and I look forward to someone solving this rails 4 issue in a PR.

Closing as this issue is stale. I note the code does pass in the rails 4 branch, but that the tests (at least at the time of this issue) were not covering this issue.
I'm still happy to accept PR's to address any outstanding issues.
Thanks


[NEW] Add forgot password

See: https://github.com/RocketChat/Rocket.Chat.Kotlin.SDK/issues/137

(Shouldn't be displayed if user doesn't have the option to reset password. :) )


Resin-cli doesn't support new ResinOS version format: 2.0.0.rev1

The new ResinOS version format is not pure semver. 2.0.0.rev1 has just been released, and cli OS downloads are currently broken.

This is now fixed in the latest resin-cli release: 5.7.1


fix offset and stratum checks to be able to send warn

Offset and stratum were sending critical messages  even when only warning
thresholds were crossed.
Also fix offset warn msg to use lower warn threshold instead of
lower critical threshold.
Pull Request Checklist
Is this in reference to an existing issue?
General

[ ] Update Changelog following the conventions laid out here

[ ] Update README with any necessary configuration snippets

[ ] Binstubs are created if needed

[ ] RuboCop passes

[ ] Existing tests pass

New Plugins

[ ] Tests

[ ] Add the plugin to the README

[ ] Does it have a complete header as outlined here

Purpose
Known Compatibility Issues

Superseded by #20


[FEATURE] Builder sections are not translatable

Is your feature request related to a problem? Please describe.
While reworking the German translation, I noticed that the default sections in the builder are not translatable as they're hard-coded in https://github.com/AmruthPillai/Reactive-Resume/blob/a687062866aa6741729369843c17520e8d5af4fc/server/src/resume/data/defaultState.ts?plain=1#L40
Examples:

Work Experience
Education
Awards
Certifications
Publications
Skills
Languages

Applies to section headers, add new "section header name" and the entries shown on the CV itself
Describe the solution you'd like
Make default sections translatable. 😄

Would be great if these section could be translatable (both in the sections and in the pdf itself, since it's the sections' default name).
Thanks!

Would be great if these section could be translatable (both in the sections and in the pdf itself, since it's the sections' default name).
Thanks!

The reason they're not translatable is because the section names can be edited by the user themselves to allow for maximum flexibility.

The reason they're not translatable is because the section names can be edited by the user themselves to allow for maximum flexibility.

True, but for a better user experience it would be great if they were pre-translated already.


RE3 Crashes

Version
3.2.0 (c468b41)
Seed
R730-9JB3-XNZH40Q5Z9KV007PRZZVPZZEZZZZZZZZZZZZZZZWZHZZZZZZZ
Scenario
RE 3 (Jill)
Player Character
Jill
What happened?
I don't know why but most of the seeds i've been trying are all crashing when i get to either the basement where we find the lighter oil  or when i get to any single cutscene part.
I have the biorand Package version of RE3 and

It might help

Can you send me your log file, you can get it by clicking Jill Log.

Can you send me your log file, you can get it by clicking Jill Log.

jILL LOG.txt
Of course.
Thank you for the help sir.
Wish you a happy new year.
I've tried to change a few seeds and even reverted to a previous version but still it keeps crashing. I'm not very tech savy but i don't really understand why this is happening.
Wish you a good night
Best Regards

Sorry I was unable to reproduce, do you have a saved game just before the crash?

Sorry I was unable to reproduce, do you have a saved game just before the crash?

I do not.
The game would not even start for me.
I don't know if it's something from my computer or whatever it is.
It seems 3.1.8 is more stable and it's the one im using atm.
Hopefully it will be ok.
Thank you for the help.
And honestly thank you for all you do.
Wish you a good day.


How is the subscription in connectivity insight explicit?

Problem description
The current subscription is designed in an explicit subscription style however as far as I can see it, this is a classic implicit style subscription.

An instance-based subscription is a subscription indirectly created, additionally to another resource creation. The subscription is not an autonomous entity and its lifecycle is linked to the managed entity

(which I believe in the case of this subproject is the network monitoring policy).
Or did I understand this incorrectly?
Expected behavior

If the above interpretation is correct, change explicit subscription into an implicit one.

Alternative solution

Additional context

The subscription in this case is not short lived and does not get terminated with the managed entity.
Policy threshold definition identified by a Policy Id is an independent entity which can be managed.
And multiple network monitoring subscriptions can be created based on 1 Policy Id against different devices. Lifecycle for each subscription created is independent from each other. real world application perspective expectation is that policy definitions will be long lasting and might not require changes to be done often. where as network monitoring subscriptions will be heavily used.
I think the usecase fits well to use explicit subscription.
Hope my explanation makes sense. Let me know if i need to clarify on any specific points.

@maheshc01 : I am not sure if I fully understand this --The subscription in this case is not short lived and does not get terminated with the managed entity.
So, if the managed entity which is a policy in this case, is deleted, the subscription which enables notifications when the conditions (threshold definitions) specified in that policy are either met or not-met will continue to exist?

you called out a good point which is not captured as part of the specification but more for implementation details. The implementation should infact have checks in place for delete of policy such that is a policy id is being using by 1 or more subscriptions then the policy cannot be allowed to be deleted.


Rename DataBasedSwitch to DataBasedSwitchState and EventBasedSwitch to EventBasedSwitchState

What would you like to be added:
Rename DataBasedSwitch to DataBasedSwitchState and EventBasedSwitch to EventBasedSwitchState
Why is this needed:
It is only about naming "convention" as I see it.
Whether I am reading the documentation or coding I can see that the rest of the states end with the word State, so I am expecting these states to follow the same convention.

@antmendoza there is only a single switch state. In the json schema it can be one or the other type:
https://github.com/serverlessworkflow/specification/blob/main/schema/workflow.json#L1118
do you think this can be improved?

yes, I understand that there are two implementations DataBasedSwitch and EventBasedSwitch, in the same way that Eventcondition has Transitioneventcondition and Enddeventcondition.
for example, in the typescript sdk the builder name is type+Builder, so when we are creating a workflow the code looks like this:

 .states([
        databasedswitchBuilder()
          .name('CheckApplication')
...
        operationstateBuilder()

..
        xxxxstateBuilder()

it is weird to me don't see the word state in the databasedswitchBuilder, again there is nothing wrong, just my point of view.

good point, understand now. yeah lets fix this

I can create a PR

thanks!!


Build Tracy profiler on the CI

Just as we are building swiftshader it'd be nice to build and cache tracy binaries so that we can easily point people at host executables.

Windows builds of the tracy profiler are available from the main repo, though on a big release delay: https://github.com/wolfpld/tracy/releases

We should build this as a vcpkg feature (iree[tracy-profiler]) so that we always have a synced version available.

Doesn't seem to me like we'd really need to own this? If Tracy's releases are infrequent or are missing platforms, wouldn't that be better solved on the main repo? I do see the benefit of a vcpkg feature though...

Depends on whether we are always pinned to a major release - if we bump forwards then users need to build themselves. I'm fine with that but I've been asked for the binary a lot. We haven't been at anything but a major release in awhile so it may be fine to have temporary periods of "build it yourself" if it means less work for us.

This got done in #7382 and #9596 has updated docs for using it.


237 UI header icon

@heatherlogan-scottlogic I've 'fixed' this by some slightly better css formatting, and not wrapping the app name. It's not ideal but it'll look better when we come up with a (hopefully) shorter app name.


No document that caught errors are any-typed

Hi,
I've been using Flow for over a year but only today noticed that variables introduced by catch block are essentially any-typed. Whilst this makes sense, given essentially anything can be thrown in JS from any point in the call tree, I don't think this is at all obvious and some documentation highlighting this would no doubt prevent substantial confusion. Even better yet, a linter warning would be awesome help raise this a potential foot-gun.
An example of what I mean is demonstrated in this Try-Flow.
Cheers!

Thanks!
Thinking about this some more, is this something that could actually be improved by having the type checker consider the introduced error variable as having the type mixed so that any code in the catch block first has to check that it has something of the right type?
Introducing such a change would no doubt be quite burdensome on all error handling code, so I can understand not wanting to type it that way for that reason, but it would improve the correctness of the type checker, wouldn't it? As it stands, it is quite easy to assume you have an instance of Error but actually at runtime have a string or something else entirely.

That's definitely something we've considered, and will keep thinking about for the future--possibly gated under a flowconfig option or something. The main blocker is the enormous number of errors that would arise in repos if we made such a change, but it is definitely an appealing soundness improvement .

You can explicitly annotate a catch parameter as mixed:
try {
} catch (e: mixed) {
}

though we do not yet have an option to make this behavior the default


Remove LevelControl cluster for TRADFRI plug

Proposed change

This removes the LevelControl cluster for the IKEA TRADFRI plug to not create the on_level and other settings entities in ZHA, as that can cause confusion about whether or not the on/off plug is a dimmer.
Additional information

The original quirk for older firmware already removed the LevelControl cluster. This PR also does it for newer firmware versions.
See https://github.com/home-assistant/core/issues/100454 for context
Checklist

[ ] The changes are tested and work correctly
[X] pre-commit checks pass / the code has been formatted using Black
[ ] Tests have been added to verify that the new code works

This will mark all LevelControl configuration entities for existing devices as "unavailable/no longer provided".
I think that should be fine though(?), as they were useless before.

I dont need it but some users is not liking not working GUI entries and its looks working OK after deleting and adding the plug new (so ZHA is deleting the old cluster in the DB).

Thanks J


Can't save changes to nested items

There's an issue in detecting these fields are dirty.

Visit example
Expand the first "template_multicolumn_simple"
Expand the "header"
Expand the "buttons"
Try to change the button

Observe nothing happens

This looks fixed now with the latest changes. :) Thank you - will leave for you to close out.

@Zoramite - I think the issue cropped up again. Steps to reproduce:

Visit example
Expand template_hero_simple -> Header
Try to change the body field

Form isn't marked as dirty, so I can't save it


update to docker 1.8.1

/cc @branden

Thanks @mattrobenolt !  When I saw that 1.8.1 had been released I figured I'd checkout the Disqus fork to see if you'd made this change yet...

haha, we're going to just switch back to your version since you merged in all of our changes. :)

@tehranian is this repo set up to autobuild on https://hub.docker.com/r/tehranian/dind-jenkins-slave/ ?

@mattrobenolt re: autobuilds - Yes. I see the new build at the top of https://hub.docker.com/r/tehranian/dind-jenkins-slave/builds/

Yup, saw it. :) Just took a while. Thanks!
On a side note, do you have an opposition to adding myself and @branden as maintainers of this repo? We rely on this pretty heavily now at Disqus and don't like maintaining forks if we don't need to.

@mattrobenolt @branden Added you two as collaborators on the repo.

thx :) We'll try not to break things!


fix(infra/staging): Add INFLUXDB_BUCKET to task def

Thanks for submitting a PR! Please check the boxes below:

[x] I have run pre-commit to check linting
[ ] I have added information to docs/ if required so people know about the feature!
[x] I have filled in the "Changes" section below?
[x] I have filled in the "How did you test this code" section below?
[x] I have used a Conventional Commit title for this Pull Request

Changes
Remove runtime check to configure influx in staging correctly(because of SSE tracking)
Add INFLUXDB_BUCKET environment variable to staging task processor task definition
How did you test this code?

Covered by unit tests

Codecov Report
Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (a14eb23) 96.02% compared to head (c140bda) 96.02%.
 Consider uploading reports for the commit 4c541ac to get more accurate results

Files
Patch %
Lines

api/app/settings/common.py
50.00%
1 Missing :warning:

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3199   +/-   ##
=======================================
  Coverage   96.02%   96.02%
=======================================
  Files        1062     1062
  Lines       32219    32217    -2
=======================================
  Hits        30937    30937
+ Misses       1282     1280    -2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Document that werkzeug.server.shutdown is deprecated

https://github.com/pallets/werkzeug/pull/1873 adds a runtime notice that the current server shutdown method is deprecated, but does not also add a deprecation note in the 'Shutting Down The Server' section of docs/serving.rst.
Noting what the approved method going forward is would be nice, but it's not clear from https://github.com/pallets/werkzeug/issues/1752 that there's agreement.
This is an odd issue, since it comes up by way of Flask. I have several local tools that spin up web interfaces to monitor some temporary process, and have found a /quit route to be quite convenient. Happy to submit an Issue in the flask project, if that's appropriate here.

See #1752 for deprecation.


[azopenai] Whisper support

Just running into some API generation complications, want to track the work.
The basics are that we end up with two new functions - GetAudioTranscription and GetAudioTranslation, which are very similar. Both of them accept a set of inputs like the prompt, the model and the actual audio data to be translated/transcribed.
So far, the default generation is working "ok" but it looks like some fidelity is being lost since we actually do have multipart support in our autorest.go. So:

[x] The requests should be made to be multipart/form-data instead of defaulting to application/json
[ ] The return values are either plain text or a structured JSON body. The generated code is not recognizing that.

Still haven't gotten it all the way through, but these are some of the immediate things.

Fixed with #21599


Extraneous README info

As referenced in #12, README really does not need a lot of that text. It'd be much more succinct and just overall better to remove a lot of it.
Some parts that could be removed:
Right now, it tracks updates for a game called CS:GO, and tracks YouTube uploads for a YouTuber named EthosLab. Very random things, but they're what I want at the moment!

Not very configurable at the moment!!! I'm working on it next :D

^ Needs to be updated with a feature list, and the final paragraph is outdated.
Also, some parts should be rewritten in my opinion. Cut down on exclamation points perhaps?

With the acceptance of #15, this is mitigated somewhat. The updated feature list should be from another issue, #12, so this issue will remain open until that issue is closed.


Do not auto-close when ending with open autoClosingPair

VSCode Version: 1.25.1
OS Version: Mac OS X 10.13.6 (17G65)

Steps to Reproduce:

Open new file
Set language mode to Python
type "marry had a little lamb" (w/o quote)
insert cursor anywhere in the string and type " (quote character)

Expected results:

A quote character will be added

Actual result:

Two quote characters are added

Does this issue occur when all extensions are disabled?: Yes

This bug happens with any line that ends in a character that starts an open autoClosingPair. It's especially prevalent in the Python language mode because Python has so many open autoClosingPairs that begin with a letter, but the bug will present itself in any language mode that has open autoClosingPairs similar to Python's.

Can't reproduce with a vanilla VS Code installation.

/needsMoreInfo

I'm happy to provide a gif of the issue:

I used version 1.26.0-insider in the gif:

@MattHardcastle this is what I get in today's insider without any Python related extension.

Based on your gif I assume you have a python extension installed. Please file the issue against the extension you use. Thanks a ton!

@kieferrm I'm reproducing the insider version of VSCode w/o any extensions installed. I believe you're failing to replicate the issue because you're not ending the string in the correct character. The bug will only present itself when a line ends in in r, u, f, or b.
The issue is caused by a bug in the way electricCharacters are recognized. It presents itself in the native Python extension because python has multi-character opening autoClosingPairs. For example r" opens a raw string in Python. VSCode is configured to recognize that and closes raw strings with a matching quote. The bug is that the opening character match is checking the end of a string instead of checking the cursor character backward for the open autoClosingPair.
I investigated this over the weekend and got a working fix in PR #55315.

@MattHardcastle got it now. Thanks for explaining.

No problem, @kieferrm. Thanks for reconsidering the issue.


Feature/error fix after node

This pull request fixes the missing error messages introduced with the swagger codegen upgrade.  The new codegen uses the new Angular HTTPClientModule API which has bugs in our current Angular version apparently.

Upgrade the Angular version fixes it.

The Angular dependencies are upgraded based on what Angular-CLI would newly generate for a new project.  However, Angular-datatables does like it.

Upgrading angular-datatables to match the new Angular version too.
~Also upgrading ts-node to one of the latest version otherwise npm run-script prebuild would not work.~

Notes: Forgot to upgrade tslint (will do so at another more convenient time)

Codecov Report

Merging #191 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #191   +/-   ##
========================================
  Coverage    63.06%   63.06%
========================================
  Files           95       95
  Lines         2794     2794
  Branches       364      364
========================================
  Hits          1762     1762
  Misses         838      838
  Partials       194      194

Impacted Files
Coverage Δ

src/app/shared/error.service.ts
90% <ø> (ø)
:arrow_up:

...-workflow-modal/register-workflow-modal.service.ts
85.71% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


refactor(data/finset): change structure to subtype

This PR redefines finset as a subtype of multiset. In terms of an immediate benefit, there may not be much. It does have a bit of elegance in reusing subtype instead of defining one as a structure. It does allow the reuse of some subtype theorems. Some finset theorems might even be better restated using subtype as the underlying type, though I didn't do more of that than necessary. The real benefit may be in sharing structure with a future finmap, which is also a subtype of multiset (see #174 for discussion). In particular, there are a number of things that do not depend on the subtype property, which can be shared.
I did run into one strange problem with this change:
home/travis/build/spl/mathlib/algebra/big_operators.lean:252:11:
state:
α : Type u,
β : Type v,
_inst_1 : decidable_eq α,
_inst_2 : comm_semiring β,
δ : α → Type u_1,
_inst_3 : Π (a : α), decidable_eq (δ a),
t : Π (a : α), finset (δ a),
f : Π (a : α), δ a → β,
s : finset α
⊢ decidable_eq (multiset α)

After trying an explicit instance for decidable_eq (multiset α), I learned that
term has has type
  decidable_eq.{u+1} (multiset.{u} α) : Type u
but is expected to have type
  decidable_eq.{u+2} (multiset.{u} α) : Type (u+1)

This occurred at the use of induction s using finset.induction. When I changed that to finset.induction_on s, the problem went away. I believe this was the only place using induction s using finset.induction.

As fas as I see it the structure approach makes the type class inference more stable. Especially, when we assign type class instances different from the original type.I think it can happen that the elaborator definitionally unfolds our type before the type class search runs. In this case the wrong instances may apply.
I don't see a big advantage of directly using subtype, but I think the structure approach might be more stable.

I am inclined to reject this change. As for connecting with finmap, you can prove that finmap has an injective map to finset and then it won't matter how the definitions are made.

Makes sense. Thanks for the feedback.


[Telemetry Compliance] In settings, data collection "learn more" link is broken

Version
Emulator version: 4.3.2
Describe the bug
In the settings page, under "Data Collection", the "Learn more" link is broken. It looks like it tries to go to https://aka.ms/bot-framework-emulator-data-collection.

[bug]

Hey @navzam ,
This is a known issue. It was supposed to link to information about the data collection policy, but that hasn't been finalized yet. Data collection is going to be disabled in 4.3.3 which should be released later today.

The link has been changed to direct to this repo for now.

Needs to be updated along with #1236

This was fixed in #1236. Link now says "Privacy statement" and links to the official Microsoft privacy statement.


openafs_1_8: Init at 1.8.0 (was: openafs: <IP_ADDRESS> -> 1.8.0)

Introduce new "server" output holding the server binaries
Adapt tsmbac.patch to new build environment
Adapt openafs server nixos module accordingly
Update upstream CellServDB: 2017-03-14 -> 2018-05-14

Motivation for this change
Update to new stable release.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[x] other Linux distributions

[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg  build linuxPackages.openafs

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: linuxPackages.openafs
Partial log (click to expand)
while evaluating the attribute 'linux_4_14' at /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/top-level/all-packages.nix:13582:3:
while evaluating 'callPackageWith' at /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/lib/customisation.nix:113:35, called from /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/top-level/all-packages.nix:13582:16:
while evaluating 'makeOverridable' at /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/lib/customisation.nix:72:24, called from /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/lib/customisation.nix:117:8:
while evaluating anonymous function at /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/os-specific/linux/kernel/linux-4.14.nix:1:1, called from /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/lib/customisation.nix:74:12:
while evaluating 'buildLinux' at /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/top-level/all-packages.nix:13885:16, called from /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/os-specific/linux/kernel/linux-4.14.nix:5:1:
while evaluating 'callPackageWith' at /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/lib/customisation.nix:113:35, called from /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/top-level/all-packages.nix:13885:23:
while evaluating 'makeOverridable' at /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/lib/customisation.nix:72:24, called from /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/lib/customisation.nix:117:8:
while evaluating anonymous function at /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/os-specific/linux/kernel/generic.nix:1:1, called from /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/lib/customisation.nix:74:12:
assertion failed at /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/os-specific/linux/kernel/generic.nix:48:1

Failure on aarch64-linux (full log)
Attempted: linuxPackages.openafs
Partial log (click to expand)
checking for ranlib... (cached) ranlib
checking for bison... bison -y
checking if lex is flex... yes
checking whether byte order is known at compile time... yes
checking whether byte ordering is bigendian... no
checking whether printf understands the %z length modifier... yes
checking your OS... linux
checking your AFS sysname... configure: error: An AFS sysname is required
builder for '/nix/store/qfndd5zfy580gnnczq9np3xv7zh0hdqn-openafs-1.8.0-4.14.49.drv' failed with exit code 1
error: build of '/nix/store/qfndd5zfy580gnnczq9np3xv7zh0hdqn-openafs-1.8.0-4.14.49.drv' failed

Success on x86_64-linux (full log)
Attempted: linuxPackages.openafs
Partial log (click to expand)
make[2]: Leaving directory '/build/openafs-1.8.0'
make[1]: Leaving directory '/build/openafs-1.8.0'
installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/wrldnqjfx7va11pwfj1vk7vzxzwavsjl-openafs-1.8.0-4.14.49
strip is /nix/store/21ymadblbmsbb2bk4q7gl4kjasp8zmgd-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/wrldnqjfx7va11pwfj1vk7vzxzwavsjl-openafs-1.8.0-4.14.49/lib
patching script interpreter paths in /nix/store/wrldnqjfx7va11pwfj1vk7vzxzwavsjl-openafs-1.8.0-4.14.49
checking for references to /build in /nix/store/wrldnqjfx7va11pwfj1vk7vzxzwavsjl-openafs-1.8.0-4.14.49...
/nix/store/wrldnqjfx7va11pwfj1vk7vzxzwavsjl-openafs-1.8.0-4.14.49

According to Hydra logs this never built on aarch64, although upstream seems to support ARM.
Please mark the package as broken on aarch64: meta.broken = stdenv.isAarch64;

@GrahamcOfBorg build linuxPackages.openafs

@GrahamcOfBorg build linuxPackages.openafs

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: linuxPackages.openafs
Partial log (click to expand)
while evaluating the attribute 'linux_4_14' at /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/top-level/all-packages.nix:13588:3:
while evaluating 'callPackageWith' at /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/lib/customisation.nix:113:35, called from /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/top-level/all-packages.nix:13588:16:
while evaluating 'makeOverridable' at /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/lib/customisation.nix:72:24, called from /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/lib/customisation.nix:117:8:
while evaluating anonymous function at /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/os-specific/linux/kernel/linux-4.14.nix:1:1, called from /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/lib/customisation.nix:74:12:
while evaluating 'buildLinux' at /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/top-level/all-packages.nix:13891:16, called from /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/os-specific/linux/kernel/linux-4.14.nix:5:1:
while evaluating 'callPackageWith' at /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/lib/customisation.nix:113:35, called from /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/top-level/all-packages.nix:13891:23:
while evaluating 'makeOverridable' at /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/lib/customisation.nix:72:24, called from /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/lib/customisation.nix:117:8:
while evaluating anonymous function at /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/os-specific/linux/kernel/generic.nix:1:1, called from /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/lib/customisation.nix:74:12:
assertion failed at /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/os-specific/linux/kernel/generic.nix:48:1

Failure on aarch64-linux (full log)
Attempted: linuxPackages.openafs
Partial log (click to expand)
checking for ranlib... (cached) ranlib
checking for bison... bison -y
checking if lex is flex... yes
checking whether byte order is known at compile time... yes
checking whether byte ordering is bigendian... no
checking whether printf understands the %z length modifier... yes
checking your OS... linux
checking your AFS sysname... configure: error: An AFS sysname is required
builder for '/nix/store/y4v8vky78iynmd7wphdhhx4rgw60m1qv-openafs-1.8.0-4.14.50.drv' failed with exit code 1
error: build of '/nix/store/y4v8vky78iynmd7wphdhhx4rgw60m1qv-openafs-1.8.0-4.14.50.drv' failed

Success on x86_64-linux (full log)
Attempted: linuxPackages.openafs
Partial log (click to expand)
make[2]: Leaving directory '/build/openafs-1.8.0'
make[1]: Leaving directory '/build/openafs-1.8.0'
installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/m7nv7x2l7frz6vnlc2cffi6bgjxxzijq-openafs-1.8.0-4.14.50
strip is /nix/store/4mf2xm9p32lzrim927yk92xhx35yaz62-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/m7nv7x2l7frz6vnlc2cffi6bgjxxzijq-openafs-1.8.0-4.14.50/lib
patching script interpreter paths in /nix/store/m7nv7x2l7frz6vnlc2cffi6bgjxxzijq-openafs-1.8.0-4.14.50
checking for references to /build in /nix/store/m7nv7x2l7frz6vnlc2cffi6bgjxxzijq-openafs-1.8.0-4.14.50...
/nix/store/m7nv7x2l7frz6vnlc2cffi6bgjxxzijq-openafs-1.8.0-4.14.50

still broken on aarch64

I changed the direction of this PR. The packages openafs and openafs_1_8 are both maintained for a while to allow for a smooth transition (especially for servers) from OpenAFS 1.6 to 1.8.
I introduced nixos options for package selection to further smoothen the transition.

I just noticed a bug in the new openafs release. Please don't merge, yet.

The bug is fixed. PR is ready to be merged.

@GrahamcOfBorg  build linuxPackages.openafs_1_8

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: linuxPackages.openafs_1_8
Partial log (click to expand)
while evaluating the attribute 'linux_4_14' at /private/var/lib/ofborg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/lnl7-mac/pkgs/top-level/all-packages.nix:13688:3:
while evaluating 'callPackageWith' at /private/var/lib/ofborg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/lnl7-mac/lib/customisation.nix:113:35, called from /private/var/lib/ofborg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/lnl7-mac/pkgs/top-level/all-packages.nix:13688:16:
while evaluating 'makeOverridable' at /private/var/lib/ofborg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/lnl7-mac/lib/customisation.nix:72:24, called from /private/var/lib/ofborg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/lnl7-mac/lib/customisation.nix:117:8:
while evaluating anonymous function at /private/var/lib/ofborg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/lnl7-mac/pkgs/os-specific/linux/kernel/linux-4.14.nix:1:1, called from /private/var/lib/ofborg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/lnl7-mac/lib/customisation.nix:74:12:
while evaluating 'buildLinux' at /private/var/lib/ofborg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/lnl7-mac/pkgs/top-level/all-packages.nix:14004:16, called from /private/var/lib/ofborg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/lnl7-mac/pkgs/os-specific/linux/kernel/linux-4.14.nix:5:1:
while evaluating 'callPackageWith' at /private/var/lib/ofborg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/lnl7-mac/lib/customisation.nix:113:35, called from /private/var/lib/ofborg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/lnl7-mac/pkgs/top-level/all-packages.nix:14004:23:
while evaluating 'makeOverridable' at /private/var/lib/ofborg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/lnl7-mac/lib/customisation.nix:72:24, called from /private/var/lib/ofborg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/lnl7-mac/lib/customisation.nix:117:8:
while evaluating anonymous function at /private/var/lib/ofborg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/lnl7-mac/pkgs/os-specific/linux/kernel/generic.nix:1:1, called from /private/var/lib/ofborg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/lnl7-mac/lib/customisation.nix:74:12:
assertion failed at /private/var/lib/ofborg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/lnl7-mac/pkgs/os-specific/linux/kernel/generic.nix:48:1

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: linuxPackages.openafs_1_8
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowBroken = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowBroken = true; }
to ~/.config/nixpkgs/config.nix.

@GrahamcOfBorg build openafs_1_8

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: openafs_1_8
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: openafs_1_8
Partial log (click to expand)
shrinking /nix/store/4lczh0rw68wh8f8ipj30y96ar2zc88yw-openafs-1.8.0-server/libexec/openafs/dasalvager
shrinking /nix/store/4lczh0rw68wh8f8ipj30y96ar2zc88yw-openafs-1.8.0-server/libexec/openafs/salvageserver
shrinking /nix/store/4lczh0rw68wh8f8ipj30y96ar2zc88yw-openafs-1.8.0-server/libexec/openafs/dafileserver
shrinking /nix/store/4lczh0rw68wh8f8ipj30y96ar2zc88yw-openafs-1.8.0-server/libexec/openafs/davolserver
shrinking /nix/store/4lczh0rw68wh8f8ipj30y96ar2zc88yw-openafs-1.8.0-server/libexec/openafs/upserver
shrinking /nix/store/4lczh0rw68wh8f8ipj30y96ar2zc88yw-openafs-1.8.0-server/libexec/openafs/upclient
strip is /nix/store/4qvrxzxa535y8304mk195x50b6p9607d-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/4lczh0rw68wh8f8ipj30y96ar2zc88yw-openafs-1.8.0-server/libexec
patching script interpreter paths in /nix/store/4lczh0rw68wh8f8ipj30y96ar2zc88yw-openafs-1.8.0-server
checking for references to /build in /nix/store/4lczh0rw68wh8f8ipj30y96ar2zc88yw-openafs-1.8.0-server...


CAMEL-18292: Addresses ujson.IncompleteParseException: exhausted input error when …

…body is null or empty

DataSonnet does accept null, you can try this:
System.out.println(new Mapper("payload").transform(new DefaultDocument<>(null)).getContent());
// the output is the String "null"

Further, changing a null to empty string is a potentially breaking change for anyone that's checking for null payload.
I propose we roll this change back and figure out the root cause of the ujson exception. Perhaps checking for null body and if so setting the DefaulDocument.NULL_INSTANCE
@javaduke @davsclaus

Actually, I agree, upon further investigation I think it should be addressed in datasonnet, I'm on it.


Startup script does not look right

When the PennMUSH container starts to copy the files to the Volume I have mounted to /mush/game it says
cp: cannot create regular file '/mush/game/./CONNLOG.md': Permission denied
,cp: cannot create regular file '/mush/game/./README': Permission denied
,cp: cannot create regular file '/mush/game/./access.README': Permission denied
,cp: cannot create regular file '/mush/game/./alias.cnf': Permission denied
,cp: cannot create regular file '/mush/game/./aliascnf.dst': Permission denied
,cp: cannot create directory '/mush/game/./data': Permission denied
,cp: cannot create regular file '/mush/game/./getdate.README': Permission denied
,cp: cannot create regular file '/mush/game/./getdate.template': Permission denied
,cp: cannot create symbolic link '/mush/game/./info_slave': Permission denied
,cp: cannot create directory '/mush/game/./log': Permission denied
,cp: cannot create regular file '/mush/game/./mush.cnf': Permission denied
,cp: cannot create regular file '/mush/game/./mushcnf.dst': Permission denied
,cp: cannot create regular file '/mush/game/./names.cnf': Permission denied
,cp: cannot create regular file '/mush/game/./namescnf.dst': Permission denied
,cp: cannot create symbolic link '/mush/game/./netmush': Permission denied
,cp: cannot create regular file '/mush/game/./restart': Permission denied
,cp: cannot create regular file '/mush/game/./restart.dst': Permission denied
,cp: cannot create regular file '/mush/game/./restrict.cnf': Permission denied
,cp: cannot create regular file '/mush/game/./restrictcnf.dst': Permission denied
,cp: cannot create directory '/mush/game/./save': Permission denied
,cp: cannot create symbolic link '/mush/game/./ssl_slave': Permission denied
,cp: cannot create directory '/mush/game/./txt': Permission denied
,cp: preserving times for '/mush/game/.': Operation not permitted
,
Notice that the files are trying to be copied into the "." dot folder. That does not appear to be a valid operation. I have mounted the Volume into another Docker container and verified that I have write access to it. I created a file in the volume.

I resolved this issue by directly pointing to /usr/bin/entrypoint.sh as my startup command.

Glad you were able to resolve it. I'm curious what the full command was that you were trying to run. You shouldn't have to point directly to /usr/bin/entrypoint.sh. It should already do that for you.


mockotlpserver: compare JSON serialization form to OTel Java's "Logging OTLP JSON exporter"

https://opentelemetry.io/docs/languages/java/configuration/#logging-otlp-json-exporter
In mockotlpserver, some serialization decisions were made the object form of an OTLP http/json service request. That form is used in our tests, and also in some OTel JS contrib repo tests via the runTestFixture facility we added in the contrib-test-utils package. I'm curious if/how this form differs from the serialization form used by the OTel Java otlp-json exporter -- an experimental exporter for dev/debugging.  If there are differences, it might be good to align.
This is low prio.

https://github.com/open-telemetry/opentelemetry-java/tree/main/exporters/logging-otlp/src/main/java/io/opentelemetry/exporter/logging/otlp


[Bug]: clicking on button in the table should not trigger onRowSelected event of the table

Is there an existing issue for this?

[X] I have searched the existing issues

Description
The onRowSelected of the table gets triggered if the user clicks on the button may have in one of the columns

Steps To Reproduce

Create a table with button & add an onClick action
Add onRowSelected action for the table
Click on the button, you'll see that the onRowSelected also gets triggered.

Public Sample App
No response
Version
Cloud

@pranavkanade already tracking this here - https://github.com/appsmithorg/appsmith/issues/11403 🥲


also support .tar and .tar.gz (fixes #2)

Support is already there, just need to add selector globs

Awesome! thanks for the contribution!
Looks like this PR is pointing your master branch at the repo tho, so it's picking up all the other PR changes too

opened new PR off feature branch instead


Skip duplicate players

Prerequisites

[X] I have searched for existing issues that already include this feature request, without success.

Describe the Feature Request
If player is on club "duplicate" skip bidding/buying
Describe the Use Case
.
Describe Preferred Solution
If player is on club "duplicate" skip bidding/buying
Describe Alternatives
No response
Additional Information
No response

Duplicate of #527


MRG: Add minimum_phase filter function

Adds a function for converting a linear-phase FIR filters to minimum phase. Currently uses one two of four possible methods (Hilbert and homomorphic), and probably not the most general one, at least not based on my implementation. I plan to also implement the homomorphic method.
Includes tests and an example, so people can feel free to look and comment if they have time.

Ready for review/merge from my end. CIs are happy and rendered docs are here:
https://800-1460385-gh.circle-artifacts.com/0//home/ubuntu/scipy/doc/build/html-scipyorg/generated/scipy.signal.minimum_phase.html#scipy.signal.minimum_phase
@WarrenWeckesser it looks like you did a lot of the FIR filter design stuff (over 5 years ago!) so you might be interested in this.

I'm interested, but I might not get to a review for several days (and in Warren time, that can be a lot longer).  So if any of the other active signal folks get involved, don't wait for me.

There is no big rush, would be good to get for 0.19 though.

I take it Warren time has taken over @WarrenWeckesser... :)
@e-q @endolith any interest in reviewing filter design stuff like this?

Comments addressed, thanks @endolith

Thanks @e-q comments addressed.

Everyone seems happy, I'll pull the trigger in a few days if nothing else crops up.


960x540

It seems that
./mandelbrot WIDTH HEIGHT FILENAME
ignores WIDTH and HEIGHT.
It always generates an image 960x540.

Thaks, the README was missing the -g flag.


[PE-D][Tester D] No error shown when JSON data file is corrupted.

To reproduce:

Open the addressbook.json file in ./data folder
Change one of the key values to an empty string.

Launch Hirehub
Observe that application fails to load data without any error information.

Expected:
Would it be better to include form of error on the GUI when the data file is corrupted/not readable by the application? Although there might be some form of logging in the CLI, I think users would generally not check them.
Actual:
Application loads without any data, or error message.

Labels: type.FunctionalityBug severity.Medium
original: ruijietay/ped#10

The error should be seen in your terminal. Nonetheless, this degree of support for error handling is sufficient, per the TP constraints. Thank you for the feedback.
https://github.com/nus-cs2103-AY2324S2/forum/issues/618


fix: skip packages with version 0.0.0

When adding a new package to a monorepo, this script will fail unless the new package is included in the next release. It sees the initial version of 0.0.0 as "new" because it doesn't match the version on npm (which does not exist).
The script has been updated to skip packages with the version 0.0.0 to better accommodate this scenario.
To test this:

Navigate to the core monorepo
Checkout the commit for v79
Run the get-release-packages.sh script
See that the polling controller is not included in the final printed output (ignoring the GITHUB_OUTPUT unbound variable error)

This depends on #91 to fix CI

Verified. I get the following output. No polling controller
+ RELEASE_PACKAGES='{"packages":{"@metamask/accounts-controller":{"name":"@metamask/accounts-controller","path":"packages/accounts-controller","version":"2.0.1"},"@metamask/address-book-controller":{"name":"@metamask/address-book-controller","path":"packages/address-book-controller","version":"3.1.3"},"@metamask/announcement-controller":{"name":"@metamask/announcement-controller","path":"packages/announcement-controller","version":"4.0.2"},"@metamask/approval-controller":{"name":"@metamask/approval-controller","path":"packages/approval-controller","version":"3.5.2"},"@metamask/assets-controllers":{"name":"@metamask/assets-controllers","path":"packages/assets-controllers","version":"14.0.0"},"@metamask/base-controller":{"name":"@metamask/base-controller","path":"packages/base-controller","version":"3.2.2"},"@metamask/composable-controller":{"name":"@metamask/composable-controller","path":"packages/composable-controller","version":"3.0.2"},"@metamask/controller-utils":{"name":"@metamask/controller-utils","path":"packages/controller-utils","version":"5.0.1"},"@metamask/ens-controller":{"name":"@metamask/ens-controller","path":"packages/ens-controller","version":"5.0.1"},"@metamask/gas-fee-controller":{"name":"@metamask/gas-fee-controller","path":"packages/gas-fee-controller","version":"7.0.1"},"@metamask/keyring-controller":{"name":"@metamask/keyring-controller","path":"packages/keyring-controller","version":"8.0.1"},"@metamask/logging-controller":{"name":"@metamask/logging-controller","path":"packages/logging-controller","version":"1.0.3"},"@metamask/message-manager":{"name":"@metamask/message-manager","path":"packages/message-manager","version":"7.3.4"},"@metamask/name-controller":{"name":"@metamask/name-controller","path":"packages/name-controller","version":"2.0.0"},"@metamask/network-controller":{"name":"@metamask/network-controller","path":"packages/network-controller","version":"13.0.1"},"@metamask/notification-controller":{"name":"@metamask/notification-controller","path":"packages/notification-controller","version":"3.1.2"},"@metamask/permission-controller":{"name":"@metamask/permission-controller","path":"packages/permission-controller","version":"4.1.2"},"@metamask/phishing-controller":{"name":"@metamask/phishing-controller","path":"packages/phishing-controller","version":"7.0.0"},"@metamask/preferences-controller":{"name":"@metamask/preferences-controller","path":"packages/preferences-controller","version":"4.4.2"},"@metamask/rate-limit-controller":{"name":"@metamask/rate-limit-controller","path":"packages/rate-limit-controller","version":"3.0.2"},"@metamask/selected-network-controller":{"name":"@metamask/selected-network-controller","path":"packages/selected-network-controller","version":"2.0.1"},"@metamask/signature-controller":{"name":"@metamask/signature-controller","path":"packages/signature-controller","version":"6.1.1"},"@metamask/transaction-controller":{"name":"@metamask/transaction-controller","path":"packages/transaction-controller","version":"13.0.0"},}}'
+ echo 'RELEASE_PACKAGES={"packages":{"@metamask/accounts-controller":{"name":"@metamask/accounts-controller","path":"packages/accounts-controller","version":"2.0.1"},"@metamask/address-book-controller":{"name":"@metamask/address-book-controller","path":"packages/address-book-controller","version":"3.1.3"},"@metamask/announcement-controller":{"name":"@metamask/announcement-controller","path":"packages/announcement-controller","version":"4.0.2"},"@metamask/approval-controller":{"name":"@metamask/approval-controller","path":"packages/approval-controller","version":"3.5.2"},"@metamask/assets-controllers":{"name":"@metamask/assets-controllers","path":"packages/assets-controllers","version":"14.0.0"},"@metamask/base-controller":{"name":"@metamask/base-controller","path":"packages/base-controller","version":"3.2.2"},"@metamask/composable-controller":{"name":"@metamask/composable-controller","path":"packages/composable-controller","version":"3.0.2"},"@metamask/controller-utils":{"name":"@metamask/controller-utils","path":"packages/controller-utils","version":"5.0.1"},"@metamask/ens-controller":{"name":"@metamask/ens-controller","path":"packages/ens-controller","version":"5.0.1"},"@metamask/gas-fee-controller":{"name":"@metamask/gas-fee-controller","path":"packages/gas-fee-controller","version":"7.0.1"},"@metamask/keyring-controller":{"name":"@metamask/keyring-controller","path":"packages/keyring-controller","version":"8.0.1"},"@metamask/logging-controller":{"name":"@metamask/logging-controller","path":"packages/logging-controller","version":"1.0.3"},"@metamask/message-manager":{"name":"@metamask/message-manager","path":"packages/message-manager","version":"7.3.4"},"@metamask/name-controller":{"name":"@metamask/name-controller","path":"packages/name-controller","version":"2.0.0"},"@metamask/network-controller":{"name":"@metamask/network-controller","path":"packages/network-controller","version":"13.0.1"},"@metamask/notification-controller":{"name":"@metamask/notification-controller","path":"packages/notification-controller","version":"3.1.2"},"@metamask/permission-controller":{"name":"@metamask/permission-controller","path":"packages/permission-controller","version":"4.1.2"},"@metamask/phishing-controller":{"name":"@metamask/phishing-controller","path":"packages/phishing-controller","version":"7.0.0"},"@metamask/preferences-controller":{"name":"@metamask/preferences-controller","path":"packages/preferences-controller","version":"4.4.2"},"@metamask/rate-limit-controller":{"name":"@metamask/rate-limit-controller","path":"packages/rate-limit-controller","version":"3.0.2"},"@metamask/selected-network-controller":{"name":"@metamask/selected-network-controller","path":"packages/selected-network-controller","version":"2.0.1"},"@metamask/signature-controller":{"name":"@metamask/signature-controller","path":"packages/signature-controller","version":"6.1.1"},"@metamask/transaction-controller":{"name":"@metamask/transaction-controller","path":"packages/transaction-controller","version":"13.0.0"},}}'


Reduce transfer size of checkout mode transfers

By only transferring the .git directory, we can save data in transfers
An example repo at my organisation is 27MiB when cloned with --no-checkout --depth 1, but is 135MiB when cloned with only --depth 1
Given a repo can be "re-hydrated" from .git alone, lets do that!

I also improved the test coverage in this PR
Permissions retention will be a separate PR and minor version


Duniverse mode

This PR adds a --duniverse-mode flag to mdx rule to generate rules that works within a dune-only context and therefore with a duniverse.
This is done by parsing the #require statements within OCaml toplevel code blocks, adding (package ...) dependencies to the mdx test rule for any package found that way and by setting findlib's config so that it looks in the _build/install directory instead.
This is more of an hack rather than a long term solution but it gives us something to iterate on and it solves some issues for realworldocaml/book.
Let me know what you think!

cc @Julow

Small fix that we need here: not to add a package dependency to compiler-libs!

(Rebased on master and merged https://github.com/realworldocaml/mdx/pull/155 and https://github.com/realworldocaml/mdx/pull/156)

I cleaned the history and rebased on master.

Ok this PR is now ready for a final pass of review, @samoht do you want to have a look at it?

Okay wait, it's not, we messed the rebase and removed the extra package mdx dependency, I'll fix that!

Ok so I further cleaned that up.
It now depends on #184 but I expect we merge it soon.
I fixed:

the compatibility issues with older versions
the lack of tests
the forgotten extra (package mdx) dependency that must be added in duniverse mode

Once #184 is merged I think we can give this a last review round and merge. If you want to see it in action, you can look at: https://github.com/realworldocaml/book/pull/3185

Just rebased this.
Given that the next scheduled release is 2.0.0 (I'm happy to backport bug fixes if needed) I think it's okay to merge this even if it's not yet perfect so that RWO depends on master.
A quick last round of review just to make sure I haven't broken anything by accident should be enough, I'm willing to change this incrementaly in subsequent PRs!

My comments still hold: please add a licence header to new files and a bit of documentation to the new mlis. Also, if you depend on rresult for just one function it's probably better to just redefine that function instead of bringing a new dependency. Other than that, that seems ok.

I just added the license headers, documentation to Prelude and Library and removed the dependency on Rresult!

I think this should be ready to merge!


New-MgGroup example not working

Hi in the example for New-MgGroup it should be 'New-MgGroup @param' instead of 'New-MgGroup -BodyParameters $param'. Splatting worked for me.

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

ID: 2b994ee0-d0b5-715f-bc48-44d8256499d7
Version Independent ID: 6fda2b17-e142-8087-b47f-f589a3cfa1d8
Content: PowerShell V2 examples for managing groups - Microsoft Entra
Content Source: articles/active-directory/enterprise-users/groups-settings-v2-cmdlets.md
Service: active-directory
Sub-service: enterprise-users
GitHub Login: @barclayn
Microsoft Alias: barclayn

@ohlrogge
Thanks for your feedback! We will investigate and update as appropriate.

@ohlrogge
Thanks for reporting this! We have created a PR for this issue and the changes should go live soon.


Edge support of OverscrollBehavior in V18 can't be full support and according to MDN, it supports something missing

Because it lacks support of the none value (Actually supported but incorrectly treaten as contain), please see MDN and Edge 18 Changelog

Thanks!


UB in SockaddrStorage::hash

SockaddrStorage::from_raw allows creating a variable that isn't fully initialized.  But the SockaddrStorage::hash implementation assumes that if the ss_family field is of an unknown type, then the entire structure must be fully initialized.  You could cause it to access uninitialized data by doing something like the following:
let sa = libc::sockaddr {
    sa_len: 8,
    sa_family: 255,  // Not a valid family
    .. unsafe{ mem::zeroed() }
};
let ss = unsafe{ SockaddrStorage::from_raw((&sa).as_ptr(), Some(sa.sa_len)).unwrap() };
let mut s = DefaultHasher::new();
ss.hash(&mut s);

Granted, this requires the use of unsafe.  But we can and should still fix it by always validating sa_family in SockaddrStorage::from_raw.

Actually, this isn't a bug.  When I wrote it up, I assumed that the sockaddr_storage would be not fully initialized.  But actually, that was never true.  It's always been zero-initialized in this case.  See
https://github.com/nix-rust/nix/blob/c1147c62ceb3f3ca392bb307db46fd6a7247a9f3/src/sys/socket/addr.rs#L1270


Bump brotli to the current hawtness

Clippy was complaining about multiple brotli versions in my project.  Bumping this doesn't appear to break anything.

Thank you! I’ll get the release out in a bit.
@all-contributors please add @mpalmer for code

@mpalmer  2.2.3 out now, thanks again!


The RotateTo method invalidates the subsequent mapView.Map.Home method

After Navigator.RotateTo method, the MmapView.Map.Home has no effects.
Steps to reproduce
mapView.Navigator.RotateTo(0);
mapView.Map.Home = n => n.CenterOn(position.ToMapsui());
mapView.MyLocationLayer.UpdateMyLocation(position, true);
** Behavior**
The map remains centered in 0,0 coordinates.
Expected behavior
The map is positioned at the coordinates contained in the "position" variable.
Smartphone:

Device: Samsung A53
OS: Android 13

I can reproduce the problem with the current master with this code:
using Mapsui.Extensions;
using Mapsui.Projections;
using Mapsui.Tiling;
using System.Threading.Tasks;

namespace Mapsui.Samples.Common.Maps.Demo;

public class OsmSample : ISample
{
    public string Name => "1 OpenStreetMap";
    public string Category => "Demo";

    public Task<Map> CreateMapAsync()
    {
        return Task.FromResult(CreateMap());
    }

    public static Map CreateMap()
    {
        var map = new Map();
        map.Layers.Add(OpenStreetMap.CreateTileLayer());

        var centerOfLondonOntario = new MPoint(-81.2497, 42.9837);
        var sphericalMercatorCoordinate = SphericalMercator.FromLonLat(centerOfLondonOntario.X, centerOfLondonOntario.Y).ToMPoint();
        map.Navigator.RotateTo(90);
        map.Home = n => n.CenterOn(sphericalMercatorCoordinate);

        return map;
    }
}

What happens:

The call to map.Navigator.RotateTo(90) results in a call to Map.Navigated
In Map.Navigated Initialized = true; is set
If later CallHomeIfNeeded() is called the Initialized field is already true and it will not call home (it is not needed)

Is this a bug?
The Initialized = true; was explicitly added. It is set nowhere else except after the first call to home. It was probably added intentionally to prevent the call to Home (which has a default set that zooms to the full extent of the map). But there also other solutions for this. For instance set Home = null, or do the Navigate call on the Home action. I think is clearer if Initialized is only set after Home is called (And Maybe Initialized should be renamed to 'HomeIsCalled' instead). So, let`s consider this a bug.
Future considerations.
So, this can be fixed by just removing the Initialized = true line from Navigated. Still the construction with the Home method is not very clear. The reason it exists is that during Map creating the full extent of the Map (or of a specific layer, or specific data you may want to zoom into) is usually not known. Also the ScreenWidth and ScreenHeight may not be known which makes is also needed for some navigation method. Still if you do not have that problem and just know the extent it is so much simpler if you could just set it on the map. Also I wonder if we need Navigated at all or would it be better to use ViewportChanged as alternative. It is similar and it is always triggered when there is a Viewport change.

I've used mapView.Navigator.NavigateTo and now works fine.
Thank you!


Keyboard failure during new chroot install (trusty)

Whether using an in browser chrosh or VT-2 crosh, when installing a new trusty chroot (with no or any targets) the keyboard fails to work on anything but VT-2. Works after rebooting the machine, but its impossible to properly finish the install if not using VT-2.
My chromebook is an Acer cp720.

I found that the easiest way is to use a usb mouse, and then open a screen keyboard

I had the same issue, had to plug in a usb keyboard to complete installation.

Seems #1994 is the first issue where this was reported.

I tried to install crouton with unity-trusty today. And I face the issue of keyboard not responding at the end of the installation when it asks chroot username. Restarted the chrome pixel and then went to crosh's shell and entered sudo enter-chroot command. And now it asked the username and password again. And that made me login successfully. I searched every where on web for the solution. My trail and error solved this. But not sure if there will be any side effects as this is first time I am using chrome book

@SkylabInc You did the right thing, by restarting and using sudo enter-chroot to continue installation. No further side effects will appear.

Thanks.


Add details about uploading brink package in readme

Updating readme - can this be merged directory to master?
Text:
Uploading the brink package
Follow the instructions in the wiki under Development/Python to upload the new brink version.

I don't know what to say. Publishing brink / compat / keycert or any other python package should be common knowledge.
We have a private python repo... and the steps to do it are the same for all packages.
I don't think that we need to replicate this

Yeah but the details are inconsistent.
For example, the README in this repo talks about creating a ticket, branch etc which is common knowledge.
But if someone is reading this and they have not published brink / compat etc for the first time then the info would be useful.
Having inconsistent / incomplete info across various repos is a crappy experience for anyone wanting to contribute to this.

Agree. I tried to update the readme as part of https://github.com/chevah/brink/pull/107 so you can try to push an updated README on that PR.
Thanks!


Router-link by name ":to-name"

What problem does this feature solve?
It would be convenient to have a shortcode for routing by name of the route. This will save the user from constantly passing an object with only one "name" field
What does the proposed API look like?
I propose to create a shortcut:
<router-link to-name="home"> Home page </router-link>

that will be equal to
<router-link :to="{name: 'home'}"> Home page </router-link>

Duplicate of https://github.com/vuejs/vue-router/issues/2818


[REFACTORING] delete PUBLIC_AZURE_BLOB_TOKEN and PUBLIC_AZURE_URL variable from .env

Refactoring Description
It is not good to expose blob token in public. also PUBLIC_AZURE_URL is duplicate with AZURE_STORAGE_ACCOUNT variable.
Refacotring components
These two variables are used in monorepo packages

electricity-dashboard
current-location
style-viewer

Refactoring Implementation
I am not sure how we can remove blob token from electricity dashboard. maybe we use geohub's dataset api to get access token?
Refactoring Impact
No response

Let me reopen this issue since we still need to work on PUBLIC_AZURE_URL variable


Update Composer dependencies (2023-03-22-00-08)

Visual regression test passed!
View the visual regression test report


Make filter active after user types 2nd letter of word

Is it possible to initiate the filter only once the user enters in the second letter as a pose to the first?

Same issue


[usbdev] rx pid err sequence

This PR contains rx pid err sequence. In this we are sending invalid data pid by toggling the pid in call_data_seq task from base_vseq and inresult we are expecting the rx pid interrupt should be raised, indicating pid error.

This looks good to me. Could you rebase, and then we should be good to go.

Okay i will rebase it.

Hmm. The author of this commit has changed and is now @vogelpi. I'm not sure whether this is correct, but I don't want to merge something with the wrong author. Pirmin: would you mind taking a look at this?

Hmm. The author of this commit has changed and is now @vogelpi. I'm not sure whether this is correct, but I don't want to merge something with the wrong author. Pirmin: would you mind taking a look at this?

I don't know how it is changed

I definitely haven't contributed / pushed into this PR. Could it be that this is an artifact from a rebase operation? I've merged a PR into master this morning.

Please fix things here in the following ways:

The copyright header in the repo has changed. The easiest way to get things to work is to copy it from another vseq in the same directory.
The commit author should be you, not me. Similarly, the Signed-off-by line should be your name, not mine. At the moment, you are falsely claiming that I have signed off the change. Please fix this.

Please fix things here in the following ways:

The copyright header in the repo has changed. The easiest way to get things to work is to copy it from another vseq in the same directory.
The commit author should be you, not me. Similarly, the Signed-off-by line should be your name, not mine. At the moment, you are falsely claiming that I have signed off the change. Please fix this.

I have fixed this issue.
Yes, I understand.  I was the author and i wrote that code. I was trying to add myself as an author to the code, but GitHub was not accepting it and was giving an error.


Expected Maintenance - Database Upgrades

On the evening of Tuesday, August 3rd, 2021, we will be performing database updates and server upgrades that will continue to improve the performance of the Autovance applications.
To achieve this, we will have a short downtime between 11 pm MDT and 12 am MDT, where all Autovance applications will be unavailable as we perform the database upgrades.

On the evening of Tuesday, August 3rd, 2021, we will be performing database updates and server upgrades that will continue to improve the performance of the Autovance applications.
To achieve this, we will have a short downtime between 11 pm MDT and 12 am MDT, where all Autovance applications will be unavailable as we perform the database upgrades.

The downtime has been delayed a day, and is now scheduled for August 4th.

We've started the process and will keep this post up to date with progress.

We are working through the upgrades and expect the process to complete on schedule.


fix flaky test that can hang in CI

motivation: stable CI
background:

testSignal can hang sometimes when the child processes it creates do not recieve the expeted signal.
this has been observed when waitForFile timed-out and the test existed before issuing the signal.
this behavior is unexpected and points to potential issue with xctest, but this fix attempts to workaround the underlying issue by making sure the sub-rocesses are forced-killed by the test itself

changes:

regsiter the sub-processes in a process-set and force temrinate them at the end of the test

rdar://74356445

@swift-ci please test


Add image files for eventual doc build

There are two image files (with very poor names) that you need to download that are associated with the Requests presentation build that will also be used for the Response to SRG Appendix in the document build.  They are on the google drive here, and should go in beamer\images
Downloaded?

[ ] Kelli
[ ] Chris
[ ] Andy
[x] Aaron

Would put them in main-figures directory if going to be used in the doc

Please ignore the above tasks and instead use these check boxes.  Add the three png files located on the drive here  to hake-assessment\doc\main-figures in order to properly build Appendix B (response to SRG requests).

[ ] Aaron
[ ] Kelli
[ ] Chris
[ ] Andy


Percents not always matched

Here are a couple examples where the percent signs aren't always colored correctly.
On this one, only the % characters surrounding MouseY are colored correctly.
foo := { "color": %color%, "MouseX": %MouseX%, "MouseY": %MouseY% }
On this one the opening one before color matches the closing one after MouseY and the others are treated like part of a variable name.
SplashTextOn, , , color: %color% x: %MouseX% y: %MouseY%

Can you upload a screenshot of what you're seeing?

I have to apologize - I've edited the file since and am not able to reproduce the issue.  I'm now thinking I must have had a syntax error somewhere earlier on in the file I was editing that threw it off.


Fixes for ckanext-attribution

nameType bugfix
somehow forgot to implement the ability to add ORCIDs etc, so that's added now

Coverage decreased (-1.07%) to 62.921% when pulling 390a68d7330c8a72e04dc403f35cb9bc3c8a1aac on ginger/attribution-fixes into 079a11d43b61f8c58286c043d623aae03af3ea72 on main.


Ability to switch between sub and dub?

Not sure if already has this ability but i would like to switch between sub and dub anime.

Yeah i am working on this feature it will be out in few days

quick reply? love it, noted.

so the people who already have the code, like how would they update it?

so the people who already have the code, like how would they update it?

Well you can fork my repo (ps: i will release the dub anime update this week) currently i added pwa and removed some bloated code to improve performance

got u

I have added this feature clear cache of your browser to see the changes or pull the latest changes in your local machine


PageHeader: Expose PageHeader badge type prop

Summary
The badge style can now be changed through type option of badge prop.
What changed?
Added type prop to PageHeaderBadge that exposes type prop of Badge.
Links

Jira

Checklist

[x] Added unit and Flow Tests
[ ] Added documentation + accessibility tests
[ ] Verified accessibility: keyboard & screen reader interaction
[ ] Checked dark mode, responsiveness, and right-to-left support
[n/a] Checked stakeholder feedback (e.g. Gestalt designers)

Actually in the same Title example can you explicitly code the type badge?

If you fix this,
Let me know and I can merge!


🐛 BUG: Logpoints don’t resolve variable names from source maps (VS Code & Chrome)

Which Cloudflare product(s) does this pertain to?
Wrangler core
What version(s) of the tool(s) are you using?
3.15.0 [Wrangler]
What version of Node are you using?
20.8.1
What operating system are you using?
Linux (Ubuntu 20.04)
Describe the Bug
Start by creating a rudimentary worker with minify = true in your wrangler.toml:
export default {
  fetch: async () => {
    const text = "OK";
    return new Response(text);
  },
};

Add a logpoint on the return line that logs text (in Chrome, just add a plain logpoint with text; in VS Code, add {text}). Start wrangler dev, attach a debugger (Chrome or VS Code), and run the code.
In VS Code, it will output:
'ReferenceError: text is not defined\n' +
  '    at eval (logpoint-c9ec1b7.cdp:3:5)\n' +
  '    at eval (logpoint-c9ec1b7.cdp:7:3)\n' +
  '    at Object.fetch (file:///workspaces/test/package/debugging/.wrangler/tmp/dev-4ny7aX/index.js:3:338)\n' +
  '    at g (file:///workspaces/test/package/debugging/.wrangler/tmp/dev-4ny7aX/index.js:3:1055)\n' +
  '    at _ (file:///workspaces/test/package/debugging/.wrangler/tmp/dev-4ny7aX/index.js:3:202)\n' +
  '    at Object.next (file:///workspaces/test/package/debugging/.wrangler/tmp/dev-4ny7aX/index.js:3:239)\n' +
  '    at b (file:///workspaces/test/package/debugging/.wrangler/tmp/dev-4ny7aX/index.js:3:531)\n' +
  '    at _ (file:///workspaces/test/package/debugging/.wrangler/tmp/dev-4ny7aX/index.js:3:202)\n' +
  '    at m (file:///workspaces/test/package/debugging/.wrangler/tmp/dev-4ny7aX/index.js:3:284)\n' +
  '    at Object.fetch (file:///workspaces/test/package/debugging/.wrangler/tmp/dev-4ny7aX/index.js:3:1493)'

In Chrome, it will output:
 ReferenceError: text is not defined
    at eval (VM11 logpoint:1:38)
    at Object.fetch (index.ts:4:5)
    at g (middleware-loader.entry.ts:46:16)
    at _ (common.ts:53:9)
    at Object.next (common.ts:50:11)
    at b (middleware-miniflare??json-error.ts:22:30)
    at _ (common.ts:53:9)
    at m (common.ts:63:9)
    at Object.fetch (middleware-loader.entry.ts:114:11)

Disabling minify fixes this behaviour. Alternatively, it is possible to inspect the compiled .wrangler/tmp/dev-4ny7aX/index.js file and look for the variable. In my case it was being transpiled to e. Changing the logpoint to log {e} succeeds in both cases, which indicates that the logpoints aren’t following the source maps.
I think minify = false in dev mode is a reasonable workaround. I am not sure if this is a workerd issue or not.
Please provide a link to a minimal reproduction
No response
Please provide any relevant error logs
No response

I am also seeing a related error with minify = true where clicking ‘next’ in either debugger often skips lines. Turning minify off also resolves this. I’m assuming that it’s trying to move to the next symbol in the minified file and is struggling with that; resolving the above would probably resolve the same thing so I won’t split it out into a separate issue but worth noting.

Unfortunately I think this is an issue with the upstream VSCode debugger and Chrome DevTools debugger. Chrome now has an experimental option called "Resolve variable names in expressions using source maps" which fixes this behaviour—hopefully VSCode will add similar support soon.


Permit notification failure log func to be overridden

Some applications use log.SetOutput to enhance behavior of the built-in logger.  Subsequent calls to log.Printf, log.Println, etc. will go through the supplied Writer to write to the log.  This behavior is not desirable in our use case, so I've provided a patch to allow the client to override this behavior.

@RickSnyder thanks for sending PR, but I prefer different API to provide custom logger:
gobrake.SetLogger(log.New(...))

Please use that.


craco build leads to Unable to find a source map

when I use craco to build a production and run source-map-explorer 'build/static/js/*.js in the terminal,
Command line report Unable to find a source map. wrong.

Hi @tamaka365 , I know this is kind of old, but i'm facing the same issue, did you remember what did you found about it ?


feat(extras): add lazygit

Add lazygit extra. Inspired from LazyVim lazygit theme config.
Also fixed a typo in dunst. I hope that's okay with you that it's in the same PR.

Thanks!


Run single experiment by number

Support just running a single experiment, will speed up dev likely

Already have with -c


2.1.4 :sparkling_heart:

Coverage remained the same at 97.733% when pulling 3d2e55fad2a7b3739f9f5bf8eb893548947bb03e on 2.1.4 into 507f6e7b7634f7242ab4d99a06fdc37949f6e634 on devel.

Coverage remained the same at 97.733% when pulling 3d2e55fad2a7b3739f9f5bf8eb893548947bb03e on 2.1.4 into 507f6e7b7634f7242ab4d99a06fdc37949f6e634 on devel.

Coverage remained the same at 97.733% when pulling c89b90663f673339bf6ae00e100152113a1134e8 on 2.1.4 into 507f6e7b7634f7242ab4d99a06fdc37949f6e634 on devel.


filter_design.py - iirfilter

I applied a Butter High Pass filter to my samples, if my sampling frequency was set to a 1000ish the results were wierd. So after some investigation, i think there might be something inconsistent in filter_design.py.
In iirfilter method, if variable the analog is set to false, we are redirected to bilinear_zpk
if not analog:
z, p, k = bilinear_zpk(z, p, k, fs=fs)
but according to bilinear_zpk documentation, we should go there otherwise:
"Return a digital IIR filter from an analog one using a bilinear transform."
Thank you for your time!

That code always designs an analog filter.  Then, if the argument analog is False, it transforms the analog filter to a digital filter using the bilinear transformation.

I applied a Butter High Pass filter to my samples, if my sampling frequency was set to a 1000ish the results were wierd.

If you can post a minimal, reproducible example, we can take a look at it and try to figure out what causes the unexpected results.

No response for a month, so closing.  @RafaelsRamos, if you still believe there is a problem, please let us know.


symbol BuildConfig cannot be resolved in class FtcRobotControllerActivity

Hi, we're using the 2023 season version of the SDK via the RoadRunner distribution.  The project builds successfully.  However, as we looked "under the hood" in the FtcRobotController/java/internal/FtcRobotControllerActivity.java class, an error message appeared: symbol BuildConfig cannot be resolved in lines 82 and 406
82 is the import statement
import com.qualcomm.ftcrobotcontroller.BuildConfig;
406 is the usage.
We did not make any changes to the code; the problem happens on two separate "clean" installations.  The error message does not appear to interfere with the project build.  Just FYI.

That typically means that you need to tell Android Studio to do a Gradle Sync.

RESOLVED.
Thanks.  That was a clue enough to get resolution, as follows:

Upgraded Android Gradle Plugin to 8.1.1.  Synced.  No change.
Executed Build > Make Project.  Error disappeared.

Issue occurred and was resolved for both the RoadRunner distribution and a clean copy of the FIRST-Tech-Challenge distribution.


what is error?

Mobile camera is not supported by browser, or there is no camera detected/connected

Could be this:
https://github.com/kasperkamperman/MobileCameraTemplate/issues/6
Or WebRTC is not supported. Please give me more info on your OS and browser, otherwise I can't say much about this.


Using Partial Labelling (Semi-Supervised UMAP)

![Uploading ~S]4WTV4A4J7D]6V)RB2RXO.png…]()
hello,
I only have some of my data labelled when I use umap. How to glabelset the predicted labels of the unknown label data points？

It is almost certainly the labels that are contributing the clean class separation. Moreover since UMAP works fundamentally with distances and inter-relationships rather than coordinates it is not really possible to extract any meaningful feature weights or contributions. The best approach, if that's what you really need, would be to try using ParametricUMAP (possibly in semi-supervised; see some of the ParametricUMAP notebooks for more details) and then analyse the resulting learned network weights which are an explicit mapping from input coordinates to the resulting space.

Thank you very much for your answer.


[SMALLFIX] Improve efficiency of async path cache

@calvinjia

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/15585/
Test PASSed.

Merged build finished. Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/15586/
Test PASSed.

@gpang One minor comment, looks good otherwise.

@calvinjia Updated. Thanks!

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/15587/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/15588/
Test PASSed.


State not updated

When I have a
state :complete do
  enter lambda { |object|
   #do something
  }
end

The state of the object is not updated I see there is a couple of previous issues. The thing is it enters the state :complete hook but does not update the state of the object

Ignore the issue was something wrong in my code


Azure Messaging Eventgrid (4.6.0) issue with JObject in data property

We have been using Microsoft.Azure.Eventgrid 3.2.x package for quite a while now in our function projects to publish messages.
When publishing, we use a JObject to fill the data property of the EventGridEvent, works like a charm as shown in the following example
var data = new JObject             {                 {"fileId", fileId},                 {"fileName", fileName}             };
and later on this JObject is used as
var eventGridEvent = new EventGridEvent(                 Guid.NewGuid().ToString(),                 subject,                 **someJObject**,                 eventType,                 DateTime.Now,                 "1.0"             );
Last week we saw that this nuget package has been deprecated and we needed to move to the new Azure Messaging Eventgrid 4.6.x nuget package so we did.
When using empty JObjects in some cases (just sending a notification that something is ready for processing), we also use the above code, which breaks (new method signature isn't reflected in above example). In run time, we get the error: The collection type 'Newtonsoft.Json.Linq.JObject' is not supported.
When we do some modifications to use a json string instead, all quotation marks get translated to unicode formatting (even when specifying the formatter to None). The Microsoft Docs example to use objects and serialize them might be an option, but requires quite some code changes.
In the end, this left us in a non working situation on production, where we had to rollback to the old package.
Any idea if JObject support for the data element will be back into support? Is this just some bug that needs to be resolved, or is this by design?
Awaiting your responses.

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

Hi @SahinOzdemir,
The Azure.Messaging.EventGrid library uses System.Text.Json by default for serializing objects to JSON. This is by design as the Azure.* libraries will not depend on NewtonSoft.Json. In order to use NewtonSoft types directly, you would need to use the constructor that accepts a BinaryData instance, and serialize the JObject to BinaryData by using the Microsoft.Azure.Core.NewtonSoftJson package.
var serializer = new NewtonsoftJsonObjectSerializer();
var evt = new EventGridEvent(
  "Subject",
  "Microsoft.MockPublisher.TestEvent",
  "1.0",
  serializer.Serialize(new JObject { {"fileId", fileId}, {"fileName", fileName} }));

Alternatively, you can parse your JSON string into a JsonElement (from System.Text.Json):
var data = JsonDocument.Parse(new JObject { {"fileId", "id1"}, {"fileName", "name1"} }.ToString()).RootElement;
var evt = new EventGridEvent(
  "Subject",
  "Microsoft.MockPublisher.TestEvent",
  "1.0",
  data);

@pakrym pointed out that in .NET 6.0 (which is still in preview), there is a JsonNode type being included in System.Text.Json which should have similar semantics to the JObject so swapping out those types would be a third option.

Hi,
Thanks for your quick responses! I'll check if the provided solution solves our issue.

Above approach seems to work, and is a minor adjustment in code.
Thanks for the support and I'll close the issue


Update Link for Quickstart Example in Document

The link here:
Our quickstart guide can be used to get running quickly
should be:
https://github.com/deepmind/acme/blob/master/examples/quickstart.ipynb
instead of
https://github.com/deepmind/acme/blob/master/examples/quickstart

Also for this line just below the above.
You can also take a look at our tutorial. The original link is wrong.

Fixed! Thanks for flagging :)


Fix for issue #45-- don't try and scroll a position:fixed element into view

Issue https://github.com/iamdustan/smoothscroll/issues/45 shows an issue where if the parent element is position:fixed, smoothscroll tries to scroll the body to scroll it to the top of the viewport.  position:fixed elements never move of course, so in this case, it shouldn't do anything to the scrollpos of the body.

@asselin nothing to be sorry mna, thanks for your contribution. We are going to make this easier and inform when there's a lint issue beforehand.

Thanks! Sadly, this PR and others are entering in conflict (https://github.com/iamdustan/smoothscroll/pull/73/files) so I'm gonna manually pick up the work of each to simplify the release and be equally fair with everyone since someone would need to resolve the conflicts no matter the other I chose.
I'm gonna make a PR and put you as a reviewer so you can check your changes.


Installing pg_duckdb on Windows

Discussed in https://github.com/duckdb/pg_duckdb/discussions/169

Originally posted by enstarbob September 11, 2024
Both PostgreSQL and DuckDB are installable on Windows, but the installation guidance for pg_duckdb doesn't mention Windows.
Is this just a documentation oversight and 'of course you can', or is there some extra limitation currently preventing it running on Windows?
Either way, it would be really useful if it could.

Sorry for the lack of response in the discussion, but this is covered under #154. I don't want an issue for every instance of "please support this platform, OS, etc." 😁
Windows support isn't an oversight in the README; it's not something we currently support, which means we don't test or build on Windows. If someone is very interested in WIndows support, we would welcome PRs to help with Windows compatibility.
Additionally, some features we're working on right now may not be supported on Windows, and we think that's OK. Ideally, anything that can't be supported on Windows would be disabled in that case.


Add "--native-steam-dir" option

For users who don't save account credentials on their computers.
Note: We recommend that such users start Steam and login beforehand. Otherwise they would need to wait 99 seconds.

I still don't know how our check could have failed

I assumed that @TheStami checked "Don't save account credentials on this computer" in Steam configuration but not sure without providing @TheStami's configuration.
I'm using Ubuntu 20.10 without checking it and the autodetection is working.

Instead of skipping everything we should add the path to our list so we can actually wait correctly.

When user checks that, our script can't detect the change of loginusers.vdf (= can't stop waiting even when the user logged in). So I recommend that such users start Steam and login beforehand.

our script can't detect the change of loginusers.vdf

I'm sorry, after checking "Don't save account credentials on this computer", loginusers.vdf was created temporarily. But Steam client might change the behavior in the future.


Upgrading Guzzle

Just wondering if there's a reason this project uses Guzzle 3? Any plans to upgrade to 5 or 6 in the near future? Any interest?
Would be happy to help with the upgrade process if that's interesting to you at all.

Upgrading guzzle would be great indeed. But last time I take a look at Guzzle, the support of service description was missing... I didn't check that's point anymore.
But I heard of Httpplug which looks very nice and allow us to abstract the http layer.
I prefer this idea instead of upgrade Guzzle. WDYT ?

I see. I have used Guzzle quite a bit, but this is my first time with the
Service classes. I see they have a new repo for services, and it looks like
someone's working on making it compatible with Guzzle 6:
https://github.com/guzzle/guzzle-services/issues/89
I'll just keep an eye on that issue and maybe do some reading in the
meantime. Thanks!
On Dec 15, 2015 1:49 AM, "Pierre Tachoire"<EMAIL_ADDRESS>wrote:

Upgrading guzzle would be great indeed. But last time I take a look at
Guzzle, the support of service description was missing... I didn't check
that's point anymore.
But I heard of Httpplug http://docs.httplug.io/en/latest/ which looks
very nice and allow us to abstract the http layer.
I prefer this idea instead of upgrade Guzzle. WDYT ?
—
Reply to this email directly or view it on GitHub
https://github.com/Wisembly/clearbit-php/issues/3#issuecomment-164676170
.

As for Httpplug, I have not used it or looked into it much, but :+1: for anything that uses the PSR-7 standard. That's initially why I was interested in a switch to Guzzle 6, but it could be interesting to try something completely new.
Either way might take some heavy refactoring, but I'm up for helping.

@krichprollsch already lnows my point of view : guzle 3 should just burn. But we are indeed looking at HttpPlug or even trying to migrate to at least Guzzle 4 (for legacy reason on our main application, which has this lib as a dependency).

Haha, fortunately for my app, Guzzle 3 and Guzzle 6 are in completely
different namespaces so I can use both at the moment.
On Dec 15, 2015 10:09 AM, "Baptiste Clavié"<EMAIL_ADDRESS>wrote:

@krichprollsch https://github.com/krichprollsch already lnows my point
of view : guzle 3 should just burn. But we are indeed looking at HttpPlug
or even trying to migrate to at least Guzzle 4 (for legacy reason on our
main application, which has this lib as a dependency).
[image: image]
https://cloud.githubusercontent.com/assets/239685/11815965/88fbceec-a34e-11e5-9e1b-dbe56e83bbe0.png
—
Reply to this email directly or view it on GitHub
https://github.com/Wisembly/clearbit-php/issues/3#issuecomment-164810368
.

That's also why we are also currently using guzzle 3 and guzzle 4 (and guzzle 3 was picked here because @krichprollsch wanted the services, which I think are more problematic than anything and also because it was in our deps :D)

Hey here, I'm currently working on a 2.0.
The idea is to use httplug to abstract http client usage.
I try the lib jane-openapi to keep the generation from a json file.
I ilke the result.
Currently I have implemented only the combined enrichment API.

since the 2.0 is released, this is not relevant anymore.


Updating SWRL.md

What do the "|" symbols mean in your syntax example? Explain in the same paragraph where you define the symbols.
Unify your use of the -> vs unicode symbol arrow.

What do the "|" symbols mean in your syntax example? Explain in the same paragraph where you define the symbols.
Unify your use of the -> vs unicode symbol arrow.

I just meant to put them together as a table, but it turned out to be different from expected. Removed them now.
It was a typo, updated the unicode arrow → everywhere now.


CAMEL-14963: Route Template. WIP

Draft PR with work in progress. There is still a long way.
See the RouteTemplateTest unit test as an example

See the example at
https://github.com/apache/camel-examples/tree/master/examples/camel-example-routetemplate


Use Gradle project's class loader in model verifier

Addresses https://github.com/SpineEventEngine/core-java/issues/739.
Before
Right now, the ModelVerifier uses it's own class loader appended with projects' output directories.
After
In this PR, I make it use the class loader of the current project's buildscript.

@armiol PTAL


adding image uploader web application

Adding image uploader web application
Description
In short, here user can save his/her pics in database and those images can be shown on web page along with the date when the image was uploaded
user can delete any particular image of its own
in depth implementation of CRUD concept
How to run this application

pip install django
cd [to this directory]
py manage.py runserver
open any web browser and fill url with http://<IP_ADDRESS>:8000/

HAVE FUN WITH THIS APPLICATION :)
Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.

@its-puneet
I kindly merge the PR


[SPARK-13399][STREAMING] Fix checkpointsuite type erasure warnings

What changes were proposed in this pull request?
Change the checkpointsuite getting the outputstreams to explicitly be unchecked on the generic type so as to avoid the warnings. This only impacts test code.
Alternatively we could encode the type tag in the TestOutputStreamWithPartitions and filter the type tag as well - but this is unnecessary since multiple testoutputstreams are not registered and the previous code was not actually checking this type.
How was the this patch tested?
unit tests (streaming/testOnly org.apache.spark.streaming.CheckpointSuite)

Test build #51613 has started for PR 11286 at commit 3f2980a.

Test build #51613 has finished for PR 11286 at commit 3f2980a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/51613/
Test PASSed.

Test build #51643 has started for PR 11286 at commit 41c0b90.

Test build #51643 has finished for PR 11286 at commit 41c0b90.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/51643/
Test PASSed.

Merged to master