a smart trick to get around those issues

as we do know the powerpak only seems to have 8KB of srm ram support while the super everdrive seems to support 128KB of srm file,so if i will play a nes game in wich nested has to learn alooot of actions in it such as in mario 3,that means that the game will crash early on on the powerpak but later on the everdrive or it don't crash at all on the everdrive,(depending how much of kilobyte of data nested has to learn,so for instance if nested only has to learn 12KB of data from a certain game,sure it will still crash on the powerpak,BUT if i will patch the game with that 8KB of learned data,then the next time i play it,nested only has to learn 4KB of data from that game ,so it will no longer crash on the powerpak,HOWEVER if nested has to learn 16KB of data from a certain game, then the game will obviously still crash soon or later after patching it with 8KB of learned data,so what i did to get around it,i played mario 3 on the everdrive with 128KB worth of data,then i patched it and now it runs fine on both powerpak and the everdrive(aside from the BRK massage in level 7 of world 5 and the roulette and memory game off course) but still this is great,i might use this trick on other games,however other games may only require 4KB of data to be learned,but if nested does deemed that as too little,then it is no wonder why i can't get any calls from adventure island 2, once i tryed patching it with a srm file,so it really depends on the game,now as far as i do know you cannot combine many srm files together and you also cannot add a srm file to a nes nes rom you already patched into snes format as of bow yet,that would be handy to prevent me to having to do the conversion all over again,but still it's worth doing things TWiCE in cases a game crashes alooot,but i also could create srm files and upload them online to share them with others to save their time,so they can patch nes games straight with srm files sothat they don't have to play the whole game till crashing points to create their srm files,in order to save their time,am mean who ever wants to do things twice to make it perfect?
final note,while  the everdrive create save files on it's own from supported roms,on the powerpak however i had to create a 128KB srm file,then after playing i had to hold reset for 5 seconds,then i had to choose the 128KB srm file to save the data on it to later patch that nes game with it,so it works,thus delaying the crash (since the powerpak only could seemingly handle 8KB of srm data) alltrough it seemingly could not work with 8KB or 32KB of srm files for nested (wich i tryed),yeah it's weird but it is what it is,but it’s cool to get many nes roms working that way on both type flashcards sothat i can use the powerpak for the snes and the everdrive for the supaboy,
But am still scratching my head why i can’t metroid to work on the powerpak,even not after patching it with a loaded srm file and since only nested v1.3 works with metroid ,i cannot change the size from 8 to 7 etc,,, to give it more tries, i really want to get metroid working on the powerpak aside from on the everdrive.

Don't distribute srm files, they contain some original code since v1.4.
Load them into a profile and distribute the profile. Yes, you can load multiple srm of the same game into a profile. Profiles are forward and backward compatible, srm files are not.


bug: Input fields not working

https://github.com/Kajol-Kumari/InfluMart/assets/91741581/11a9f926-2bda-440b-9667-591e331da77e
Whenever I try to click an input field, it doesn't display keyboard.

Expected Behaviour:
https://github.com/Kajol-Kumari/InfluMart/assets/91741581/dfaf112e-ae27-4c4b-b6a3-ba7f51fb8c48

@Vidip-Ghosh can you please fix this under https://github.com/Kajol-Kumari/InfluMart/issues/24 as we already have the respective pages issue opened now to get it fixed and responsive.
P.S - regarding the level of the issue, we might upgrade ut based on the complexity seen during development

@Vidip-Ghosh can you please fix this under #24 as we already have the respective pages issue opened now to get it fixed and responsive.
P.S - regarding the level of the issue, we might upgrade ut based on the complexity seen during development

Okay sure. I'll


University of West Hungary changed its name

University of West Hungary changed its name to University of Sopron

@sorcika Pull request merged. Thank you!


JSON reporting

JSON reporting will be done with the pytest-json-report package. This produces a pretty usable JSON and makes it easy to extend it with custom metadata at test run time.
The idea will be to take the JSON generated by pytest-json-report and translate it into something more usable by a reporting tool. To start with, I will focus on test_has_names, which has been reintroduced here, to list which spec names are or are not defined in the given package.

@asmeurer nice! For review/discussion it would be useful if you added an example of what the JSON output looks like (maybe in a gist?) if you have one.

This is ready to be merged. I've extracted a lot of custom metadata from the tests in to the report. For hypothesis errors, for now it only extracts the error message (string). I could look into trying to extract the actual data itself and JSON serializing it, but I think that will be harder, so for now I'm going to wait until it looks like we actually need that.
I have uploaded an example report for numpy.array_api here https://gist.github.com/asmeurer/70d9b05e91a67d6b035b9b5305e427f0. Note that the numpy.array_api report is 11 MB uncompressed.  I also ran the tests with plain numpy, and the report size is 595 MB (I did not upload it). In general, many failures will balloon the file size because the full traceback is stored for each failure. However, they compress quite well (I'm not sure what the best compression to use for the web, but a basic zip compression takes the numpy.array_api report down to 375 KB and the plain numpy report down to 3.4 MB).
If there is any other metadata you can think of that should be included, let me know.

Something to keep in mind when doing any further developments on the test suite is to make sure that any inputs to test functions are JSON-serializable. If they aren't, the to_json_serializable function in reporting.py should be updated. I've made it so that dataclasses and namedtuples are serialized as objects (dicts), and functions are serialized to a string of their name, but anything that is unrecognized will just be serialized as its repr.

In general, many failures will balloon the file size because the full traceback is stored for each failure.

Maybe using --tb=native is helpful? Those long tracebacks are usually unhelpful, and pytest has a terrible default there.

OK, it turns out I was actually wrong here. The tracebacks do contribute a some amount of data in general, but it's not actually the reason the plain numpy report is over half a gigabyte. That's apparently because of the warnings. There are over a million warnings emitted in the run, and they account for 98% of the JSON file size. Of these 98% are the warning "np.bool is a deprecated alias for the builtin bool. To silence this warning, use bool by itself. Doing this will not modify any behavior and is safe. If you specifically wanted the numpy scalar type, use np.bool_ here." (I guess it is getting emitted for every single hypothesis run).
Here's the full warning metadata that's repeated so many times:
{
    "message": "`np.bool` is a deprecated alias for the builtin `bool`. To silence this warning, use `bool` by itself. Doing this will not modify any behavior and is safe. If you specifically wanted the numpy scalar type, use `np.bool_` here.\nDeprecated in NumPy 1.20; for more details and guidance: https://numpy.org/devdocs/release/1.20.0-notes.html#deprecations",
    "category": "DeprecationWarning",
    "when": "runtest",
    "filename": "/Users/aaronmeurer/anaconda3/envs/array-apis/lib/python3.9/site-packages/hypothesis/extra/array_api.py",
    "lineno": 119
}

@honno, any idea why this warning would be omitted every single time hypothesis calls np.bool, instead of just once?
I did try switching from the default traceback type to native, but the savings are actually pretty small, so I'm going to revert it because the default type has more information, and because it keeps the code simpler.

@honno, any idea why this warning would be omitted every single time hypothesis calls np.bool, instead of just once?

(I assume you're testing the top-level numpy namespace)
If you meant emitted, I'd guess it's to do with the pytest reporting steps. Hypothesis might "merge" warnings together as it does errors in the reporting stage, so pytest would only show one warning, but however we capturing warnings here ignores that. Generally it makes sense that warning is emitted many times, as np.bool is accessed many times due to the vast number of Hypothesis examples generated in a run of the whole test suite.

Note flaky CI failure is due to https://github.com/data-apis/array-api-tests/issues/125

personally I'd prefer test_has_names.py:: test_has_names -> test_has_attributes.py:: test_has_attributes

Your suggestion is to have two separate files for this? I don't really see the value in that, especially since the file is so short and uses the same logic for names and attributes.

My point with the warnings is that by default, warnings are only omitted once per warning:
>>> import numpy as np
>>> np.bool
<stdin>:1: DeprecationWarning: `np.bool` is a deprecated alias for the builtin `bool`. To silence this warning, use `bool` by itself. Doing this will not modify any behavior and is safe. If you specifically wanted the numpy scalar type, use `np.bool_` here.
Deprecated in NumPy 1.20; for more details and guidance: https://numpy.org/devdocs/release/1.20.0-notes.html#deprecations
<class 'bool'>
>>> np.bool
<class 'bool'>

But this exact same warning is emitted every single time. So somewhere something is changing the warnings filter to enable them always. I did see some warnings code in hypothesis but I wasn't sure if it was what was causing this. I guess it could also be pytest doing it somehow.

personally I'd prefer test_has_names.py:: test_has_names -> test_has_attributes.py:: test_has_attributes

Your suggestion is to have two separate files for this? I don't really see the value in that, especially since the file is so short and uses the same logic for names and attributes.

Ah I was being lazy heh, I meant rename test_has_names.py to test_has_attributes.py, and rename the test case test_has_names to test_has_attributes.

Ooo, this feels like Numpy is doing some magic which pytest/Hypothesis isn't equipped for?

Ooo, this feels like Numpy is doing some magic which pytest/Hypothesis isn't equipped for? Probably requires some hacking around to fix.

It shouldn't do anything fancy; the one thing is filterwarnings = error in pytest.ini.

NumPy's pytest.ini is irrelevant here. It looks like the pytest defaults are to store each warning invocation separately. Since the warning metadata doesn't even contain enough information to distinguish the different warnings, I think the simplest solution here is to post-process the JSON to deduplicate the warnings. (probably a lot of the stuff I'm doing here could be upstreamed to the pytest-json-report library)

OK, I deduplicated the warnings output. The plain numpy report is now 11 MB and numpy.array_api is 6.1 MB (both uncompressed). I've uploaded examples of each to the gist. I'd like to hear feedback from Athan on whether I need to try to reduce this file size any further. Again, these are uncompressed sizes, and they compress well (both are under 0.5 MB after basic zip compression).

Ooo, this feels like Numpy is doing some magic which pytest/Hypothesis isn't equipped for? Probably requires some hacking around to fix.

It shouldn't do anything fancy; the one thing is filterwarnings = error in pytest.ini.

Ah my confusion here is say the following
# Using ipython
>>> class foopy_factory:
...     @property
...     def bool(self):
...         warn("don't use this", DeprecationWarning)
...         return bool
...
>>> foopy = foopy_factory()
>>> foopy.bool
DeprecationWarning: don't use this
bool
>>> foopy.bool
DeprecationWarning: don't use this  # warning is emitted again
bool

the warning is emitted for each access of foopy.bool, but Aaron's example only show it emit the first time.

>>> import numpy as np
>>> np.bool
<stdin>:1: DeprecationWarning: `np.bool` is a deprecated alias for the builtin `bool`. To silence this warning, use `bool` by itself. Doing this will not modify any behavior and is safe. If you specifically wanted the numpy scalar type, use `np.bool_` here.
Deprecated in NumPy 1.20; for more details and guidance: https://numpy.org/devdocs/release/1.20.0-notes.html#deprecations
<class 'bool'>
>>> np.bool
<class 'bool'>

I see now that the emit-once behaviour is seen using the default python interpreter, but not using say IPython.
# Using python
>>> class foopy_factory:
...     @property
...     def bool(self):
...         warn("don't use this", DeprecationWarning)
...         return bool
...
>>> foopy = foopy_factory()
>>> foopy.bool
DeprecationWarning: don't use this
bool
>>> foopy.bool
bool  # no warning this time

Seems like Aaron's found a solution anywho, but might be worth noting that environment things can affect this behaviour... or I might be missing something, I'm pretty ignorant on this stuff heh.

Need to have this merged so I can start testing it.

@asmeurer Looking through the sample JSON, they look fine to me. No need to reduce further, IMO. The raw files can be parsed and transformed for subsequent presentation on, e.g., a website. So not too concerned regarding initial report file sizes, particularly as we can store them on disk in compressed format.

One thing we should consider, however, is versioning the JSON format. E.g., using JSON schema. This would allow versioning our backend server APIs to handle transformations of different versions of the JSON report.


WebView2 tutorial missing information

The WebView2 tutorial is missing information that will ensure the object will be properly loaded. I described the details of the missing information in the following comment issue:
https://github.com/MicrosoftEdge/WebViewFeedback/issues/228#issuecomment-687873636

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

ID: 27602e44-0300-1528-058c-3066f533b98c
Version Independent ID: f95488e7-ec44-6ced-dc3f-ba8f88c70d31
Content: Microsoft Edge WebView 2 for Windows Forms apps - Microsoft Edge Development
Content Source: microsoft-edge/webview2/gettingstarted/winforms.md
Product: microsoft-edge
Technology: webview
GitHub Login: @MSEdgeTeam
Microsoft Alias: msedgedevrel

It would be good to fix the sizing of images (they all over the place) and code snippets as well.
E.g. the code example in step 4 provides several bad suggestions.

If the form was designed in the desirer, why does user need to wire up a handler manually?
There is no need to wire up event handlers when an implementation can be plumbed via an override - in the latter case there is no allocations for handlers.

public Form1()
{
    InitializeComponent();
    this.Resize += new System.EventHandler(this.Form_Resize);
}

private void Form_Resize(object sender, EventArgs e)
{
    webView.Size = this.ClientSize - new System.Drawing.Size(webView.Location);
    goButton.Left = this.ClientSize.Width - goButton.Width;
    addressBar.Width = goButton.Left - addressBar.Left;
}

should look like this:
public Form1()
{
    InitializeComponent();
}

protected override void OnResize(EventArgs e)
{
    base.OnResize(e);

    webView.Size = this.ClientSize - new System.Drawing.Size(webView.Location);
    goButton.Left = this.ClientSize.Width - goButton.Width;
    addressBar.Width = goButton.Left - addressBar.Left;
}

But the biggest issue with the overall example IMO is that we require a user to perform manual layouts. I suggest to update the layout of the form in a way that handles resize and layout events automatically.
Windows Forms provides Control.Dock and Control.Anchor properties that take care of control placement, as well as TableLayoutPanel and FlowLayoutPanel controls that facilitate complex layouts.
I think it is important to make our examples as good as they can be, as users often copy code and then build on top of it and deploy into production.

Closing this issue because I opened an issue on the engineering team's feedback repo. Thanks.


Null Safety Migration

Dart Null Safety is released and packages should start to be migrated.
Migration guide

I have release 0.4.0 version which supports null safety. Please let me know if you face any issue with this version.


MiqAeEngine instantiate accepts a user object

Changed the specs to pass in the user object to MiqAeEngine.instantiate.
Removed the monkey patch file.

@kbrock @gmcculloug @tinaafitz
Please review

LGTM
I expected you to just modify the monkey patch, allowing a user to optionally be passed in.
I will merge when it turns green.


Rails Master Breaks

After this commit https://github.com/rails/rails/commit/bd2f5c062da011e092c1f122567f24bd5fc6d9b5 Rails master is full with errors.
  1) Error:
test_allows_session_fixation(ActiveRecordStoreTest):
NoMethodError: undefined method `accept' for nil:NilClass
    /Users/arunagw/checkouts/rails/activerecord/lib/active_record/connection_adapters/abstract/database_statements.rb:7:in `to_sql'
    /Users/arunagw/checkouts/rails/activerecord/lib/active_record/connection_adapters/abstract/database_statements.rb:87:in `insert'
    /Users/arunagw/checkouts/rails/activerecord/lib/active_record/connection_adapters/abstract/query_cache.rb:19:in `insert'
    /Users/arunagw/checkouts/rails/actionpack/test/../../activerecord/lib/active_record/relation.rb:70:in `insert'
    /Users/arunagw/checkouts/rails/actionpack/test/../../activerecord/lib/active_record/persistence.rb:316:in `create'
    /Users/arunagw/checkouts/rails/actionpack/test/../../activerecord/lib/active_record/timestamp.rb:57:in `create'
    /Users/arunagw/checkouts/rails/actionpack/test/../../activerecord/lib/active_record/callbacks.rb:268:in `create'
    /Users/arunagw/checkouts/rails/activesupport/lib/active_support/callbacks.rb:392:in `_run_create_callbacks'
    org/jruby/RubyKernel.java:2093:in `send'
    /Users/arunagw/checkouts/rails/activesupport/lib/active_support/callbacks.rb:81:in `run_callbacks'
    /Users/arunagw/checkouts/rails/actionpack/test/../../activerecord/lib/active_record/callbacks.rb:268:in `create'
    /Users/arunagw/checkouts/rails/actionpack/test/../../activerecord/lib/active_record/persistence.rb:297:in `create_or_update'
    /Users/arunagw/checkouts/rails/actionpack/test/../../activerecord/lib/active_record/callbacks.rb:264:in `create_or_update'
    /Users/arunagw/checkouts/rails/activesupport/lib/active_support/callbacks.rb:410:in `_run_save_callbacks'
    org/jruby/RubyKernel.java:2093:in `send'
    /Users/arunagw/checkouts/rails/activesupport/lib/active_support/callbacks.rb:81:in `run_callbacks'
    /Users/arunagw/checkouts/rails/actionpack/test/../../activerecord/lib/active_record/callbacks.rb:264:in `create_or_update'
    /Users/arunagw/checkouts/rails/actionpack/test/../../activerecord/lib/active_record/persistence.rb:37:in `save'
    /Users/arunagw/checkouts/rails/actionpack/test/../../activerecord/lib/active_record/validations.rb:50:in `save'
    /Users/arunagw/checkouts/rails/actionpack/test/../../activerecord/lib/active_record/attribute_methods/dirty.rb:22:in `save'
    /Users/arunagw/checkouts/rails/actionpack/test/../../activerecord/lib/active_record/transactions.rb:241:in `save'
    /Users/arunagw/checkouts/rails/actionpack/test/../../activerecord/lib/active_record/transactions.rb:295:in `with_transaction_returning_status'

attr_accessor :visitor added.

Noting that rails/rails/bd2f5c062da011e092c1f122567f24bd5fc6d9b5 causes an issue.

I have de same problem with current rails 3.2.0.rc2 and the jdbc adapters.
I managed to figure out, that the visitor for the adapter is unset.
As a workaround one can use the following code in an initializer to fix the problem.
class ActiveRecord::ConnectionAdapters::SQLiteAdapter
  def visitor
    @visitor ||= Arel::Visitors::SQLite.new(self)
  end
end

Afaik the initializer should set the visitor. I don't know why it actually is not doing it at the moment.

I'm very new to Ruby and noticed this same issue when running JRuby on Rails (3.2.0 - stable) with MySQL.
Added a similar line to the initializer of ActiveRecord::ConnectionAdapters::MysqlAdapter:
@visitor ||= Arel::Visitors::MySQL.new(self)

Fixed by #156.

I am getting this issue with rails 3.2.  Is the fix in the master?  I am referencing ar-jdbc like this:
gem 'activerecord-jdbc-adapter', :git => "https://github.com/nicksieger/activerecord-jdbc-adapter.git", :branch => 'master'
gem 'activerecord-jdbcsqlite3-adapter'
I still get the error "undefined method `accept' for nil:NilClass".

Adding the initializer mentioned earlier does work as a workaround.


How can I implement SSO in yii2

Is there a easy way to config the SSO (single sign-on) in yii2?

Please use forum for questions. GitHub is for issue reports.


Quill Delta object as content

Hi here!
What about delta data, mentioned in https://quilljs.com/docs/api/#setcontents
Its is possible in vue-quill-editor component ?
Tnx!

https://github.com/surmon-china/vue-quill-editor#use-in-spa
You can get the quill instance object by attr ref to do any somethinng!


Prevent overriding @final methods in super classes

Lint for overrides of /** @final */ methods. Similar to deprecation lint rule (#2395)

Note: per #4534, this issue will be closed in less than a month if no PR is sent to add the rule. If you really need the rule, custom rules are always an option and can be maintained outside this repo!

💀 It's time! 💀
TSLint is being deprecated and no longer accepting pull requests for new rules. See #4534. 😱
If you'd like to see this rule implemented, you have two choices:

Recommended: Use typescript-eslint to lint your TypeScript code with ESLint ✅
Legacy: Write it as a custom TSLint rule 🤷♂

👋 It was a pleasure open sourcing with you!
If you believe this message was posted here in error, please comment so we can re-open the issue!

🤖 Beep boop! 👉  TSLint is deprecated 👈 (#4534)  and you should switch to typescript-eslint! 🤖
🔒 This issue is being locked to prevent further unnecessary discussions. Thank you! 👋


s20 switch support

Support for Orvibo's S20 WiFI Smart Switch. At ~$20, with no hub dependencies, its a great bargain.

Could you rename this platform to orvibo ?

I was adding comments before the rename, make sure to check out my comments on the outdated diffs.


WIP: add BanAwareCachePolicy for http caching

Add policy for scrapy's http caching that will refuse caching of banned responses as per https://github.com/TeamHG-Memex/scrapy-rotating-proxies/issues/20

I'm not sure how should we cover multiple policies? Should we just wrap the two default policies that scrapy has (RFC2616Policy and DummyPolicy) or maybe we should extend HttpCacheMiddleware this way all policies would be supported?


can't select by telecontroller on android Tv device

can't select by telecontroller on android Tv device?

Haven't tried the component on Android TV. Feel free to open a PR to support this feature.


JTAppleCalendarViewDelegate

(Required) Version Number:
Description
Steps To Reproduce
Expected Behavior
Additional Context

JTAppleCalendarViewDelegate. is not working not showing

this issue did not followthe guidelines. Closing until re-written


Wrong calculations with recursion to Nat ->: Nat ->: Nat

I tried to translate this code:
fib :: Int -> Int
fib n = fib_aux n 0 1
  where
    fib_aux :: Int -> Int -> Int -> Int
    fib_aux 0 m1 m2 = m1
    fib_aux n m1 m2 = fib_aux (n-1) m2 (m1+m2)

I wrote
import TLImplicits._
import shapeless._
val Nat = "Nat" :: Type
val NatInd = ("0" ::: Nat) |: ("succ" ::: Nat -->>: Nat) =: Nat
val zero :: succ :: HNil = NatInd.intros
val one = succ(zero)
val two = succ(one)
val three = succ(two)
val four = succ(three)
val five = succ(four)
val six = succ(five)
val n = "n" :: Nat
val m = "m" :: Nat
val m1 = "m1" :: Nat
val m2 = "m2" :: Nat
val recNNN = NatInd.rec(Nat ->: Nat)
val addn = "add(n)" :: Nat ->: Nat
val add = recNNN(m :-> m)(n :-> (addn :-> (m :-> succ(addn(m)) )))
val recNNNN = NatInd.rec(Nat ->: Nat ->: Nat)
val fibn = "fib_aux(n,_,_)" :: Nat ->: Nat ->: Nat
val fib_aux = recNNNN(m1 :-> (m2 :-> m1))(n :-> (fibn :-> (m1 :-> (m2 :-> fibn(m2)(add(m1)(m2)) ))))
val fib = n :-> fib_aux(n)(zero)(one)

But this produces powers of 2 instead of fibonacci.
@ fib(zero)
res48: Term = 0 : (Nat : 𝒰 _0)
@ fib(one).fansi
res49: String = "succ(0)"
@ fib(two).fansi
res50: String = "succ(0)"
@ fib(three).fansi
res51: String = "succ(succ(0))"
@ fib(four).fansi
res52: String = "succ(succ(succ(succ(0))))"
@ fib(five).fansi
res53: String = "succ(succ(succ(succ(succ(succ(succ(succ(0))))))))"
@ fib(six).fansi
res54: String = "succ(succ(succ(succ(succ(succ(succ(succ(succ(succ(succ(succ(succ(succ(succ(succ(0))))))))))))))))"

The function fib_aux is giving the cumulative sum starting with m1 and m2, instead of the sum of the two previous entries.

But Haskell's fib_aux
fib :: Int -> Int
fib n = fib_aux n 0 1
  where
    fib_aux :: Int -> Int -> Int -> Int
    fib_aux 0 m1 m2 = m1
    fib_aux n m1 m2 = fib_aux (n-1) m2 (m1+m2)

calculated correctly
GHCi, version 8.0.1: http://www.haskell.org/ghc/  :? for help
Prelude> :l 1.hs
[1 of 1] Compiling Main             ( 1.hs, interpreted )
Ok, modules loaded: Main.
*Main> fib 0
0
*Main> fib 1
1
*Main> fib 2
1
*Main> fib 3
2
*Main> fib 4
3
*Main> fib 5
5
*Main> fib 6
8
*Main> fib 7
13

Another specific wrong calculation
@ fib_aux(three)(m1)(m2) == add(add(m1)(m2))(add(m1)(m2))
res40: Boolean = true

As expected:
fib_aux(succ(n))(m1)(m2) ==  fib_aux(n)(m2)(add(m1)(m2))

But we have
@ fib_aux(three)(m1)(m2) ==  fib_aux(two)(m2)(add(m1)(m2))
res58: Boolean = false

And this persists even if we plug in succ(two) in place of three.
This is clearly a bug. We also see
@ fib_aux(succ(n))(one)(one).replace(n, two).fansi
res63: String = "succ(succ(succ(0)))"
@ fib_aux(three)(one)(one).fansi
res64: String = "succ(succ(succ(succ(0))))"
@ fib_aux(succ(two))(one)(one).fansi
res65: String = "succ(succ(succ(succ(0))))"

the first is correct but the second  wrong.

An earlier point where this breaks:
@ def fn(t: Term) = FormalAppln.unapply(t).get._1
defined function fn
@ fn(fib_aux(n)(m1)(m2))
res90: Term = ((<function1>) (n : (Nat : 𝒰 _0)) : ((Nat : 𝒰 _0) → ((Nat : 𝒰 _0) → (Nat : 𝒰 _0)))) (m1 : (Nat : 𝒰 _0)) : ((Nat : 𝒰 _0) → (Nat : 𝒰 _0))
@ fn(fn(fib_aux(n)(m1)(m2)))
res91: Term = (<function1>) (n : (Nat : 𝒰 _0)) : ((Nat : 𝒰 _0) → ((Nat : 𝒰 _0) → (Nat : 𝒰 _0)))
@ fn(fn(fn(fib_aux(n)(m1)(m2))))
res92: Term = <function1>
@ fn(fn(fn(fib_aux(n)(m1)(m2)))) == fib_aux
res93: Boolean = true
@ fn(fn(fn(fib_aux(succ(n))(m1)(m2)))) == fib_aux
res94: Boolean = true
@ fn(fn(fn(fib_aux(succ(succ(n)))(m1)(m2)))) == fib_aux
res95: Boolean = false

This means that we do have a formal application, but by a different function.

A substitution that should change nothing (on branch debugtrace) is giving a different but still wrong answer. Problem seems to be with substitution of inductive type definitions, hence for nested inductive types. The change in code is
-  var crash = false
-
   case class DataCons[
       H <: Term with Subs[H], C <: Term with Subs[C], D <: Term with Subs[D]](
       data: D,
@@ -53,11 +51,7 @@ object RecursiveDefinition {
     def newobj = DataCons(data.newobj, defn, tail)

     def subs(x: Term, y: Term) =
-      {val result =  DataCons(data.replace(x, y), defn, tail.subs(x, y))
-        if (crash && data.replace(x, y) != data)
-          throw new IllegalArgumentException(s"Attempting to substitute $x by $y in $data getting ${data.replace(x, y)}")
-        result
-      }
+      DataCons(data.replace(x, y), defn, tail.subs(x, y))

the result depends on whether crash is true or false, and with behaviour repeated when making switches.

Diagnosis (should be confirmed by fixing).

This is a subtle error caused by bound variables escaping scope, or more accurately, the same bound variable being used in two lambdas, one nested in the other.
This normally does not happen as new variables are created and used when defining a lambda.
However, when we compute fib_aux(succ(n)),

in specifying the data defining fib_aux, variables m1 and m2 are generated.
the result is a function, which is a lambda of the form m1 :-> m2 :-> y.
but the y itself involves a formal application of fib_aux, and the definition of fib_aux uses the data which has the same bound variables m1 and m2.
this means that when we apply the function fib_aux(succ(n)), we substitute arguments for m1 and m2. But these substitutions also have an effect on the definition of fib_aux as this uses the same variables.

It is clear that  such substitutions are happening, and they should not. Of course one has to test after correcting whether this is the only source of error.
A solution is to sanitize the data in recursive definitions before using in a formal application, i.e., with the data not changed but using freshly generated variables for lambdas. This can be done by using translation from Term to Term, but will first try a simpler implementation.

The above was done (crudely) and seems to have partially worked: the bug #120 seems to be fixed, and manually applying the recursion definition in this case works okay. But the main bug persists.

With another change, fib seems to work. But:

the change should also be made for indexed types.
a better rebuild should be used, including for example products.
computations have slowed down, may need an opt-out - for example to use a specified rebuilder, perhaps one that is just the identity.


Bug:

Description of the bug

Using Mock-Component to mock any component is causing other unrelated components that are explicitly declared in test modules configuration to be mocked as well, this is unexpected behavior because the component was explicitly declared in the test module and we are not expecting it to be mocked in any context within the test suit.
More info can be found in this SO Question
An example of the bug
You can find the example here
Expected vs actual behavior
in the provided example, we have app component, and wrapper component, wrapper component's template includes app component instance.
Expected: when we create an instance of wrapper component, a real instance of app component should be instantiated since it's part of wrapper component and it was explicitly declared within the test bed module.
Actual:  when we create an instance of wrapper component, an empty mocked instance of app component is created.
The issue is linked to MockComponent usage, the issue wasn't present on older version of ng mock (13.0)
I hope that you find this info helpful.
Best regards.

Hi @rezres,
the issue here is that test-helpers.ts is a partial copy-paste of ng-mocks sources, because of that ng-mocks detects wrapper as a mock of AppComponent.
To fix this, delete the line 7 in test-helpers.ts:
mockOf: { value: mockClass },


Fix FacilityIndex deletion behavior

Overview
FacilityIndex items aren't being deleted when facilities are deleted.
Expected Behavior
When a facility is deleted, its associated FacilityIndex object is deleted.
Actual Behavior
The FacilityIndex row isn't deleted.
Steps to Reproduce

On production, run the following query:

SELECT count(*) from api_facilityindex i WHERE NOT EXISTS (SELECT FROM api_facility f WHERE i.id = f.id;

This will grab the count of facility index rows that are linked to nonexistent facilities, (over 2000), demonstrating that facilities aren't being removed from the index when needed.

Locally, delete a facility through the admin dashboard, then search for it by id in the facility index. It will be found in the index.

SELECT * FROM api_facilityindex WHERE id = '{id}';

I think post_delete will likely hep us here. I would be comfortable calling this a 1⃣ but would also not object to a 2⃣ .


Added Advice from Previous students page

I have created a possible design of the advice page per #595. It is mostly empty at the moment. I have left obvioius space for people to add comments in the hope that they do. I have added advice for Maths for CS and Computer Networking.
Before we merge it, do you have any questions about the format?
A few problems:

I kept the table formatting from the main readme. It is a pain to do multi-line comments inside one cell in a table. If it is a pain to update, people may not want to add their advice.

It duplicates a lot of info from the main readme, so if the main readme changes, this page will have to change too.

@xxylem I think your point #2 (If the main readme changes, this page will have to change too.) is right on the nose. Maintainability is of chief concern to any project, and we don't want to structure our projects to need to be changed in two places.
What I think would be better is to simply have a page that has entries only for courses that have student advice already. Each course entry would just have:

The course title
An anchor tag on that course title
A bulleted list of advice for that course

Then on the main page of the repo, if a course has advice that's listed, we can put a link directly to that advice.
This way, the course advice page can remain unordered. And the only time we would need to change it based on main page changes is if a course is removed altogether.
Is that a clear enough description for you to refactor this on your own? If so, do so and I'll merge it in. Before you make the pull request again, it will be good to do a squash of the commits to make the git history clearer. You can see a video on that here:
https://www.youtube.com/watch?v=V5KrD7CmO4o
(Heads up, you'll need to force push the branch, as this step changes your commit history)
Also, I think your comments on Math for CS are important enough to be part of the main repo. Can you make a pull request against the main page for that? The section for Core Math can be altered to have the same structure as Core Systems and the recommendations for readings, recitation and problem sets can go in the Additional Text / Assignments section.
So two commits:

Main repo only, changing the Core Math: Math for CS section to include more direction.
Addition of advice.md, with one entry about Computer Networks. Main repo: Core Systems: Computer Networks is altered to include a link to that advice entry.


Add Phoummala Schmitt Profile Page to Website Listing

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

advocates/media/profiles/Phoummala-Schmitt.png
:white_check_mark:Succeeded
View

advocates/phoummala-schmitt.yaml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Correct mistake in rooms/04-create-or-join-room.ts

Fixed console.log functions in 'onInit' and 'onJoin'

Thanks a lot @4eezer!


Tokio 1.0 support

Support for Tokio 1.0.
Some things were moved from tokio crate to tokio-util crate - in this project's case it's the DelayQueue. Enabled only the features necessary. See https://docs.rs/tokio-util/0.6.0/tokio_util/ .
Allowed myself to add .idea/ folder into gitignore :wink: .

Hey, appreciate this, but there's already one in progress for this in #363.
I think this as it stands has one problem that we are ultimately pulling in an older tokio via the older reqwest, so need to use the patched branch to avoid that.

Hey, appreciate this, but there's already one in progress for this in #363.
I think this as it stands has one problem that we are ultimately pulling in an older tokio via the older reqwest, so need to use the patched branch to avoid that.


CVEs

There are a lot of vulnerabilities in the image: https://quay.io/repository/konveyor/oadp-operator/manifest/sha256:3e6d24020434f923a8ec0de43e778a1c0c23db5ac99b8d9ab6fd8acec13e54d9?tab=vulnerabilities - can you update the packages?

@davidkarlsen Thank you for the heads up. Our previous releases were using a version of ansible operator with some CVEs and we have since updated our images for our work to move to a golang-based release. Our latest images are passing security scans without issues. We have no intention of updating the 0.2.x release to fix these vulnerabilities but will ensure 0.3.x does not have any CVEs.

@davidkarlsen Currently targeting 9/29. Will be version 0.3.0.

@dymurray Maybe the readme should be updated with upgrade nodes and what velero version it is?


[SR-14493] System library targets don't look outside the SDK on macOS

Previous ID
SR-14493

Radar
rdar://problem/76704075

Original Reporter
@Lukasa

Type
Bug

Status
Reopened

Resolution

Additional Detail from JIRA

Votes
0

Component/s
Package Manager

Labels
Bug

Assignee
@neonichu

Priority
Medium

md5: 304abf79948934d5fe4888d4af1bc861

Issue Description:
When using system library targets on macOS, builds will consistently fail to link or include any library that is present on the system but outside the SDK. This makes it very hard to actually use libraries of this form, as they invariably force users to use unsafeFlags to actually get the linker to work.

This is still an issue, in my case hitting it with vulkan


Season Number and Swapping Episode Title with Number

Does this script just put the episode title and name in the filename or does it also include the season number?
Also, if you have seasons of shows where you have the season and episode number after the episode title, could this script be used to reverse that so the filename would have the title after the season and episode number? For example, to turn "Mow Better Blues - S01E02.mp4" into "S01E02 - Mow Better Blues"?

The script only converts as episode as "Episode Number - Episode title"
(e.g. 1- Pilot).
The episodes are needed to be in it's respective season folder, for the
script to work properly. Since, they are already sorted season-wise, I
thought it was redundant to include the season number in the episode name.
On Sat, Sep 8, 2018, 7:09 PM RedSoxFan04<EMAIL_ADDRESS>wrote:

Does this script just put the episode title and name in the filename or
does it also include the season number?
Also, if you have seasons of shows where you have the season and episode
number after the episode title, could this script be used to reverse that
so the filename would have the title after the season and episode number?
For example, to turn "Mow Better Blues - S01E02.mp4" into "S01E02 - Mow
Better Blues"?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bhatushar/TV-Show-Rename/issues/2, or mute the thread
https://github.com/notifications/unsubscribe-auth/AUPLnW0C6vC9aFnhJQ-kGLDNHKro2Mhyks5uY8iggaJpZM4Wf3_u
.


Create List DID Page

The page misses a dedicated List DID Page.
Note that while many aspects of previously created tanstack tables (ie DIDSelectTable) can be reused, they will need to be adapted and placed into separate components. This keeps the components specialised, the code bottom-heavy and stays in line with the tailwind philosophy.
The entire task can be summarised as follows:

[x] Write storybook page
[x] Move storybook page into proper nextjs page
[x] Write mock API endpoint on mock flask server
[ ] Connect to API endpoint with simple tanstack table note the metadata does not necessarily have to be shown in a tanstack table, this would be overkill for a single row of data (it also works against the framework).
[x] Create subpages to show information for individual DIDs => as per my conversation with @bari12 , this information will not be accessible on the List DID page, but rather on a page dedicated to the DID.
[x] Add dark mode
[x] Redo responsiveness
[x] Add tests => for a different issue

Completed as per #189
Tests as per #208


ADFS as IdP will break in sandboxes that are updtaed for enhanced domains.

[Enter feedback here]
The URL for sandboxes that are upgraded to use enhanced domains will have a change in the base URL
MyDomainName--SandboxName.my.salesforce.com will become
MyDomainName--SandboxName.sandbox.my.salesforce.com
(See: ([https://help.salesforce.com/s/articleView?id=sf.domain_name_enhanced_considerations.htm&type=5]))
This is apparently enough of an issue that Salesforce has postponed enforcement on sandbox orgs with SSO enabled until Spring '23.
Before you deploy enhanced domains, make sure you have at least one admin user with the ability to log in with local credentials and an MFA method or you will lock yourself out.

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

ID: c206e955-bb5b-0707-5cf5-7b29cdf0ff6e
Version Independent ID: f00eff35-9f21-ea45-45a6-eba4be4fda04
Content: Tutorial: Azure Active Directory integration with Salesforce Sandbox - Microsoft Entra
Content Source: articles/active-directory/saas-apps/salesforce-sandbox-tutorial.md
Service: active-directory
Sub-service: saas-app-tutorial
GitHub Login: @jeevansd
Microsoft Alias: jeedes

@bengelj
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

Updating the document.
#please-close


Can just load .env from sub-directory?

Here is my directory structure:
root
├── Justfile
├── project1
│  └── .env
└── project2

I execute the recipe in the project1 directory with .env file variables, but just do not load the .env file in project 1. Is there any way to load it?

This currently isn't possible. I think this is tracked by something like #945, which would allow specifying a custom .env path, e.g. project/.env.


Help: Is it possible to customize the input of the engine?

What do you need help for ?
Please note that the documentation is still under construction.
The input for me feels really bad. Better than vanilla but pretty bad. Is it possible to modify the input

ill take this personally but
you can customize the input by the
function onKeyShit(pressedArray:Array<Bool>, justPressedArray:Array<Bool>, justReleasedArray:Array<Bool>) {
   // do your custom input code
   for(e in [pressedArray, justPressedArray, justReleasedArray]) {
      for(i in 0...e.length) e[i] = false; // disable og input
   }
}```


⚡ Feature | Drag and drop images in text container

Users need to be able to drag and drop images (jpg/png/gif..) from their desktop to the editor.
When dragging an image to the editor, we'll display a thin grey (7) line of 2px between the paragraphs so the user can clearly see where he's placing the image:

releasing the left mouse click will drop the image in the wanted paragraph and will load like shown
Use the spinner icon (white) we are using on the login page
Use a color 12 overlay 70% opacity to cover the image while loading:

FIGMA FILE: https://www.figma.com/file/elxKahbV8rj85IWOEyZeVu/Product-design?node-id=4648%3A35347

@gregogun to handle the dropzone part, we already have the react-dropzone npm package so please use this one :)


Updated with READMe with New Instructions

This pull request updates the README file to improve the setup instructions for new contributors to the Open Source Design website. The updates include:

Detailed steps for managing Ruby versions using rbenv, aimed at helping new contributors set up their development environments more efficiently.
An expanded prerequisites section that provides step-by-step guidance on installing necessary tools and dependencies. This aims to reduce setup time and potential configuration errors.
Additional context to help new contributors understand the importance of each step and how they fit into the overall development process.

did it work?

done!

@subz3r0o0  unfortunately there is this file that I wrote about above. Sync with the main branch, I added exceptions https://github.com/opensourcedesign/opensourcedesign.github.io/pull/450
It should help.

did it work?

https://github.com/opensourcedesign/opensourcedesign.github.io/pull/447/files
image folder
I think that since it is already in the history of the git, it needs to be done manually.

sorry just to clarify. what file do i need to remove manually? the ds file ?

You can delete after the merge :)

sorry just to clarify. what file do i need to remove manually? the ds file ?

Also you can delete from the link that arhell gave


Auto Login

Description of the new feature / enhancement
Please provide auto login and Image BG remover.
Scenario when this would be used?
Please provide auto login and Image BG remover.
Supporting information
Please provide auto login and Image BG remover.

You can read here on how to activate autologin in Windows: https://www.alphr.com/how-to-enable-auto-login-in-windows-10/


WRKLDS-1492: Update k8s dependencies to 1.31.1

This PR updates k8s dependencies to 1.31.1 in addition to the required OCP libraries.

/hold
I'd like to check pod logs first to see everything is ok

After checking the CI logs, cluster-openshift-controller-manager-operator seems to be healthy and no issue
/hold cancel

/approve
/lgtm


RDP Wrapper and printer on Windows 7 Home Basic

Hello!
I've install RDP Wrapper on Windows 7 Home Basic.
Remote desktop work fine, but I have a problem with local
resources redirect (printers and disks).
Is there a way to solve the problem?
Many thanks for you answers!

Remote Easy Print doesn't work with rdpwrap

May I disable Remote Easy Print on Windows 7 Home Basic?

To get printer redirect to work you have to edit the printer ports and share them as per this Link. I had to use LPT4 to get mine to work.
https://www.farmhousenetworking.com/networking/remote-access/remote-desktop-network-printer-redirection/

Thanks for your advice! But the attempt was unsuccessful.
On remote computer i haven't ports TS001, TS002, TS003 etc.
I'm trying to create local ports TS001 and then TS003.
On remote computer print jobs work fine - on local printer
do nothing.
May be another decision?


Missing -MD -MF

The autogenerated toolchain definiton for both gcc and clang on Linux (Ubuntu 20.04) produces
-MD -MF bazel-out/k8-fastbuild/bin/_objs/main/main.pic.d

as part of the command when building
cc_binary(
    name = "main",
    srcs = ["main.cpp"],
)

but this is not reflected in the compilation database.
Relevant section of $(bazel info execution_root)/external/local_config_cc/cc_toolchain_config.bzl:
dependency_file_feature = feature(
    name = "dependency_file",
    enabled = True,
    flag_sets = [
        flag_set(
            actions = [
                ACTION_NAMES.assemble,
                ACTION_NAMES.preprocess_assemble,
                ACTION_NAMES.c_compile,
                ACTION_NAMES.cpp_compile,
                ACTION_NAMES.cpp_module_compile,
                ACTION_NAMES.objc_compile,
                ACTION_NAMES.objcpp_compile,
                ACTION_NAMES.cpp_header_parsing,
                ACTION_NAMES.clif_match,
            ],
            flag_groups = [
                flag_group(
                    flags = ["-MD", "-MF", "%{dependency_file}"],
                    expand_if_available = "dependency_file",
                ),
            ],
        ),
    ],
)

I’m not sure if there is a use case for which this is actually important, but I thought I’d report it since it’s a discrepancy between what was actually executed and what the compilation database claims the command was.

Thanks for the report. There have been many changes to the C++ toolchain since this project was initially written, and I have mainly relied on community support. So thank you for your efforts.
I am actually looking to transfer the ownership of this project to someone who is interested. Will you be willing to take over?

I’m not sure I can commit to full ownership but if you want to promote me to collaborator I can try to help out.

Thanks. I just sent you an invite for Write access.

I think it's perhaps desirable to skip these flags. The -MD and -MF flags tell the compiler to effectively write a Makefile for this compile action at the location given by -MF. Even if we do include this flag, we would want to keep a different temporary location for -MF.
Let's close the issue until a valid use case comes up which can guide us on how to do this properly.


Rename live test matrix leg name to be more generic/general purpose rather than "with_iot_samples"

Right now the leg name has iot_samples in the name. Maybe it should be called with_service_samples or just simply with_samples.
https://github.com/Azure/azure-sdk-for-c/blob/9f5a6334863ebbf214c6e2fbd5d6e3fce52fe0de/eng/pipelines/templates/jobs/archetype-sdk-tests.yml#L10
With storage support being added (granted it is currently in a feature branch), it is a bit confusing to see those samples under iot related name.
https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1187023&view=logs&j=437eda56-6549-565d-5141-89d54d1c1c74&t=be8930a4-c8ca-5981-39a3-9e0e1a7cda3f

https://github.com/Azure/azure-sdk-for-c/blob/bfca1f1d9ffe320f7d9d5ea448b4788f9d974347/eng/pipelines/templates/stages/archetype-sdk-client.yml#L24-L26
cc @antkmsft

I'd drop "with_samples" - one test may chose to do live testing via samples, other one may not. The goal is to live test, so if some sample is not able to be run as a live test, I think it will ultimately be excluded, proving that the main intent is to "run live test", but whether "with samples" or not, is a secondary detail.

cc @danewalton since this rename happened as part of: https://github.com/Azure/azure-sdk-for-c/pull/1608

Seems pretty low priority.  EngSys team will not get to this.  If this is important to the C team, feel free to make these changes.


Re: Wanted to show you project I made with your library

Hi @adrielcafe, I wanted to show you my sample app I made with Voyager: https://github.com/JohnBuhanan/MVI-Public
I studied a dozen github repos and felt like yours was closest to what I wanted.
Three things I thought you might find interesting:

I wrapped your Navigator so that I can do routing from ViewModels.
Each feature contains an api gradle module and an impl gradle module.  The impls can be commented out from settings.gradle and app.gradle to unload them from Android Studio indexing and gradle builds.  This allows scaling for massively multi-module apps.
I used reflection to get access to the "internal factories" of ScreenRegistry.

Biggest thing I am thinking about now is best way to "navigate for result".

Hi @JohnBuhanan
Do you know why it is not working without reflection of ScreenRegistry?

@ynsok
I think there is type-erasure happening with the way I do Routing :(
Maybe there is a better way to do routing from ViewModel.

Generally, I think it should be enough, and solve our problem. What do you think?
@adrielcafe  @DevSrSouza  @JohnBuhanan
     fun get(provider: ScreenProvider): Screen {
        val factory =
            factories[provider::class] ?: error("ScreenProvider not registered: ${provider::class.qualifiedName}")
        return factory(provider)
    }

I've implemented the @ynsok suggestion in 1.0.0-beta16, please reopen the issue if that was not enough.
About navigating from ViewModel/ScreenModel or any other class, I like the idea, some times that kind of logic is too complex and should not be in the UI layer. Feel free to open another issue about that, I'll start to think about an implementation.


I only want to show one picture

I have an album with only one picture in it I want displayed. I open the app, select that album, which shows only the one picture. after a time, it opens all other albums in my google account. I only selected the ONE album, and that is the ONLY album I am trying to display - with only ONE picture in it, yet it keeps changing to other albums/pictures.
Is there a way to force it to not shuffle or change albums? I set the timer to 999 in hopes that it will work, but I am still waiting... after 999 seconds it opened a different album, so that wasn't the solution.
Thanks in advance!

Is this during screensaver playback or within the app itself? Showing only 1 image is troublesome because Roku forbids applications from disabling the screensaver if nothing on the screen is changing. It's a requirement from Roku.
To get around this, you could create an album with 1 photo, select only that album for screensaver playback. See #2 here -> https://www.photoviewapp.com/faq/
OR use that 1 album within the app and disable your screensaver.

I have used this work around to display 1 image.  It works with one
potential drawback.  The roku screensaver times out after 12 hours and
rolls to a "screen saver timed out" screen.  At this point, you need to
take roku back to home screen and wait for screensaver to reopen (I set
mine to 1 minute delay for screensaver)
On Wed, Oct 28, 2020 at 12:06 PM Chris Taylor<EMAIL_ADDRESS>wrote:

Is this during screensaver playback or within the app itself? Showing only
1 image is troublesome because Roku forbids applications from disabling the
screensaver if nothing on the screen is changing. It's a requirement from
Roku.
To get around this, you could create an album with 1 photo, select only
that album for screensaver playback. See #2
https://github.com/chtaylo2/Roku-GooglePhotos/pull/2 here ->
https://www.photoviewapp.com/faq/
OR use that 1 album within the app and disable your screensaver.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/chtaylo2/Roku-GooglePhotos/issues/338#issuecomment-718037270,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APD7SGGKQFWWIJSSRVSA6YDSNA6PDANCNFSM4Q3ET4YQ
.


[CELEBORN-1810] Using Operation description instead of ApiResponse description for RESTful APIs

What changes were proposed in this pull request?
Using Operation description instead of ApiResponse description for RESTful APIs.
Why are the changes needed?
Make the API description in correct place.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Before:

After:

Will change the description for exclude worker after https://github.com/apache/celeborn/pull/3023 merged.


Newer release available antlr4_runtimes: 4.9.2 (current: 4.7.2)

Package Name<EMAIL_ADDRESS>Current Version: 4.7.2@2018-12-18
Available Version: 4.9.3@2021-11-06
Upstream releases: https://github.com/antlr/antlr4/releases

New version is available for this package
New Version: 4.9.3@2021-11-06
Upstream releases: https://github.com/antlr/antlr4/releases
New Issue Link: https://github.com/phlax/envoy/issues/44

New version is available for this package
New Version: 4.9.3@2021-11-06
Upstream releases: https://github.com/antlr/antlr4/releases
New Issue Link: https://github.com/phlax/envoy/issues/45

New version is available for this package
New Version: 4.9.3@2021-11-06
Upstream releases: https://github.com/antlr/antlr4/releases
New Issue Link: https://github.com/phlax/envoy/issues/46


chore(main): release 1.0.0

:robot: I have created a release beep boop
1.0.0 (2024-01-28)
Features

docker support (027e214)
format Markdown files with Prettier (3fc8d14)
initial GitHub Actions support (#5) (1385de8)
initial import (16b6820)
only build dev container when updated (#4) (1891bc5)
optional use of base dev container image (#3) (799dae2)
pre-build dev container workflow (#1) (1fceb55)
prettier config file (2d42cdd)
set VSCode file associations for config files (60638dc)
support for Release Please (#6) (544d373)

Bug Fixes

fix file pattern match for trigging dev container (057a6f6)
move GitHub Actions workflow directory (#2) (fd9909c)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/jwbennet/project-base/releases/tag/v1.0.0 :sunflower:


RSS feed not updating

👋. I was wondering why I hadn't seen updates for a couple of days in Feedbin (especially with all the LUNA shenanigans!) and noticed that the RSS feed has stopped updating.
Hopefully it's not something that's too terminal!

Agh, thanks, not sure why the alert I set up didn't fire. Should be ok now!

Thanks a bunch for the super quick fix (on the weekend!). Love following the site and the madness!


Add documentation for 7009 errors

For what it's worth, this error shows up in all sorts of completely unrelated situations.
There's probably only so much targeted help we can give people for this error. Some suggestions (any/all of):

make the doc more clear that there are lots of ways this error can happen
figure work out more cases where this error can happen, add more examples
split the error up in to more than one code, so it's possible to give better help / make the message better

Looks like we have some rebasing required @Morriar ?

@froydnj rebased, thanks! 👍


Errors not shown, codeactions called all the time when I have all code analysis turned off.

##Issue Description ##
All code analysis is turned off and /codeactions continually is called.
Actual errors in my code are not reported until you actually open the file with the error in it.
Reinstall and it works for anywhere from a few minutes to a few days until it breaks again.
Steps to Reproduce
Wish I could, but this has happened for the last year happens on multiple machines we use
We have a solution setup properly and omni is using the solution.
Expected Behavior
Don't call /codeactions constantly all the time
Show me the correct set of errors in the problems tab always without having to browse to files.
Actual Behavior
/codeactions called all the time
Logs
OmniSharp log
    ************  Response ************

{
"Request_seq": 2072,
"Command": "/v2/getcodeactions",
"Running": true,
"Success": true,
"Message": null,
"Body": {
"CodeActions": [
{
"Identifier": "Wrap every parameter -> Align wrapped parameters",
"Name": "Wrap every parameter -> Align wrapped parameters"
},
{
"Identifier": "Wrap every parameter -> Indent all parameters",
"Name": "Wrap every parameter -> Indent all parameters"
},
{
"Identifier": "Wrap every parameter -> Indent wrapped parameters",
"Name": "Wrap every parameter -> Indent wrapped parameters"
},
{
"Identifier": "Unwrap and indent all parameters",
"Name": "Unwrap and indent all parameters"
},
{
"Identifier": "Pull 'Validate(string moleculePrimaryKeyID, T t)' up -> Pull 'Validate' up to 'IValidationRuleDefinition'",
"Name": "Pull 'Validate(string moleculePrimaryKeyID, T t)' up -> Pull 'Validate' up to 'IValidationRuleDefinition'"
}
]
},
"Seq": 8437,
"Type": "response"
}
C# log
Post the output from Output-->C# here
Environment information
VSCode version: 1.43.0
C# Extension: 1.21.15
Dotnet Information
.NET Core SDK (reflecting any global.json):
 Version:   3.1.102
 Commit:    573d158fea
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.18362
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\3.1.102\
Host (useful for support):
Version: 3.1.2
Commit:  916b5cba26
.NET Core SDKs installed:
3.0.100 [C:\Program Files\dotnet\sdk]
3.1.102 [C:\Program Files\dotnet\sdk]
.NET Core runtimes installed:
Microsoft.AspNetCore.App 3.0.0 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 3.1.2 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 3.0.0 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 3.1.2 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.WindowsDesktop.App 3.0.0 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 3.1.2 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
To install additional .NET Core runtimes or SDKs:
https://aka.ms/dotnet-download

Visual Studio Code Extensions

Extension
Author
Version

csharp
ms-dotnettools
1.21.15

rest-client
humao
0.23.1

vscode-guid
heaths
1.4.19

vscode-icon-theme
jtlowe
1.6.5

vsliveshare
ms-vsliveshare
1.0.1711

 clipboard

Thanks for filing the issue @numbelvis. This is arguably not a well-design option, of which there isn't really an analogue in VS either. So adjusting this would imply a more holistic design, since there's different levels of suggestions people may want to see.

Ahh ok, I thought may be that was the case.  Thanks for taking time!


add(front):add watchlist (fav) page

Add a fav page with watchlist crypto

Page watchingList working need to add skeletton and null value


2.0 preview error

Testing 2.0 using preview packages 2.0.0-preview-18334, I got an error here :
https://github.com/agriffard/TheBootstrapTheme/blob/9993e862945489e29170402aeaf6757c72b70ecc/src/TheBootstrapTheme/Views/Layout.cshtml#L10
RuntimeBinderException: Cannot implicitly convert type 'System.Text.Json.Dynamic.JsonDynamicValue' to 'bool'. An explicit conversion exists (are you missing a cast?)

We should also change the views that do this kind of things. We have a type and tools to bind json to settings, let's use it.


Add k_board 1.3.0 + CI/CD + clippy + fmt

Hello
Yesterday k_board 1.3.0 was uploaded. I recommend this version for the project.
I add some CI/CD for continuous integration.
Cheers!

Hi,
Wow, thanks a lot for the changes! Honestly, it really caught me by surprise (in the best way).
Also, I just want to say that your k_board crate is awesome! I really appreciate all the work you’ve put into it. Thanks for creating something so cool!
Cheers.


chore: reorg monorepo s01

Description

Integration tests
Adapter to test:

postgres

Python version to test:

3.8

Looks great @drochetti! Great idea to reorganize the monorepo.
The adapter integration tests are failing because the package resolver is getting confused with the new paths, I think. The place to start resolving this would be projects/adapter/src/dbt/adapters/fal_experimental/utils/environments.py, specifically the _get_dbt_packages function.

Nice! Thanks @mederka
It seems like most of the paths used by the file you mentioned and all the other cucumber tests are defined here: projects/adapter/integration_tests/features/steps/fal_adapter_steps.py, is that correct?


Link in calender not click-able

The link in the calenderbox is not click-able, neither in the description nor in the location field.

Reminds me about  my PR a while ago: https://github.com/fsi-hska/iwi-website/pull/27
Maybe someone with the Calendar API set up can test it with the <Link/> fix ;-)

I will have a look at this, we need to be careful though as we could get problems with XSS if this is not implemented carefully.
Whatever solution we come up with, it should also fix the problem of overflowing long links (as mentioned in #59 )


F-Droid can't build

ref: https://monitor.f-droid.org/builds/log/openfoodfacts.github.scrachx.openfood/1258#site-footer
It's a typo in https://github.com/openfoodfacts/smooth-app/blob/v4.13.0/packages/smooth_app/android/app/src/main/res/xml/backup_scheme.xml#L13 I guess
/LE: fyi  https://gitlab.com/fdroid/fdroiddata/-/commit/67c4c9149c00c8a612c5d3acab8582f26a626625

https://github.com/openfoodfacts/smooth-app/issues/5102 It still fails...

Is that still the case, there seems to be a version from the 9th march


Limit subsquid requests to comply with dos protection

https://docs.subsquid.io/graphql-api/dos-protection/
--max-root-fields=50 --max-response-size=5000

no restricted queries
requests of more than 500 items need to paginated


Manage tokens mode: UI for creating database-backed tokens with restricted permissions

I need this for Datasette Cloud. This would be a database-backed alternative to the new default signed token UI added in Datasette 1.0:

Here's the most ambitious plan for this feature:

Users can create tokens using a UI similar to the default one, but those tokens are stored in a database table
Token permissions are stored in the database, probably as JSON to match the format shown here: https://docs.datasette.io/en/1.0a2/authentication.html#restricting-the-actions-that-a-token-can-perform
This means tokens can be revoked and audited
Auditing: tokens store a created date and a last-used date. The last-used date is only accurate to the nearest 60s (or 5m perhaps), to avoid having to write to the database every time a token is authenticated (we only write if it hasn't had a write in the last 60s). This means we can see which tokens are actively being used.
Token permissions can be edited after the token has been created. This means users can issue broad-scoped tokens and then reduce permissions on them once they figure out what they need.

The most ambitious version:

Token permission checks are also logged on a short-term basis (maybe just 10m or so) such that the UI can reflect which permissions a token actually needs to get its job done.

That last feature is my dream feature for API tokens, but I don't know how feasible it is to implement.

I'm going to turn this mode on with a plugin configuration option:
plugins:
  datasette-auth-tokens:
    manage: true

Where manage = "you should manage the tokens".

Updated schema:
CREATE TABLE _datasette_auth_tokens (
   id INTEGER PRIMARY KEY,
   secret TEXT,
   description TEXT,
   permissions TEXT,
   actor_id TEXT,
   created_timestamp INTEGER,
   last_used_timestamp INTEGER,
   expires_after_seconds INTEGER
);

Now has expires_after_seconds for storing e.g. 3600 to have the token expire after an hour.
And description for an optional note the user can specify describing what the token is being used for - useful for auditing.

Moving the work on this to a PR.

Schema update:
CREATE TABLE _datasette_auth_tokens (
    id INTEGER PRIMARY KEY,
    token_status TEXT DEFAULT 'L', -- [L]ive, [R]evoked, [E]xpired
    description TEXT,
    actor_id TEXT,
    permissions TEXT,
    created_timestamp INTEGER,
    last_used_timestamp INTEGER,
    expires_after_seconds INTEGER,
    secret_version INTEGER DEFAULT 0
);

The token_status column is new - it shows if a token is live, revoked or expired. This exists for two reasons:

To enable faceting by those statuses on a regular table view
To provide a column that a render_cell() hook can use to show a bit of custom UI, screenshot below

The secret column is gone. I decided that being able to publicly (at least to registered users) display the API keys table would save a bunch of time on implementing features for listing tokens, but preventing authorized users from select secret from _datasette_auth_tokens was too much hassle. So I'm switching to signed tokens instead, which means the tokens table doesn't need to store any secrets.
That's what secret_version is for - it allows for secret rotation without invalidating all existing API tokens. A value of 0 indicates that the DATASETTE_SECRET for the instance is being used. I'll add a mechanism later whereby you can instead configure 0 to be a fixed secret, then 1 to be another secret etc - so you can roll out new secrets without disabling existing API keys.
I moved actor_id  to be next to description because it's a nicer order.
Table screenshot:

I started work on a view for this before deciding that it would be better to use an existing table implementation:

async def list_tokens(request, datasette):
    _check_permission(datasette, request)
    db = datasette.get_database()
    tokens = []
    for row in (
        await db.execute(
            "select id, description, permissions, created_timestamp, expires_after_seconds "
            "from _datasette_auth_tokens where actor_id = :actor_id order by id desc",
            {"actor_id": request.actor["id"]},
        )
    ).rows:
        tokens.append(
            {
                "id": row[0],
                "description": row[1],
                "permissions": json.loads(row[2]),
                "created_timestamp": row[3],
                "expires_after_seconds": row[4],
            }
        )
    context = await _shared(datasette, request)
    context.update({"tokens": tokens, "datasette": datasette})
    return Response.html(
        await datasette.render_template(
            "list_api_tokens.html", context, request=request
        )
    )

System should show a custom "token is revoked" or "token has expired" error.

I wanted to raise Forbidden("Token has expired") in actor_from_request() so that the user would get an error that their token had expired (or been revoked) - but it turns out that doesn't work cleanly in Datasette, exceptions raised from that hook turn into 500 errors at the moment.

Spotted a problem while working on this: if you grant a token access to view table for a specific table but don't also grant view database and view instance permissions, that token is useless.
This was a deliberate design decision in Datasette - it's documented on https://docs.datasette.io/en/1.0a2/authentication.html#access-permissions-in-metadata

If a user cannot access a specific database, they will not be able to access tables, views or queries within that database. If a user cannot access the instance they will not be able to access any of the databases, tables, views or queries.

I'm now second-guessing if this was a good decision.

I wanted to raise Forbidden("Token has expired") in actor_from_request() so that the user would get an error that their token had expired (or been revoked) - but it turns out that doesn't work cleanly in Datasette, exceptions raised from that hook turn into 500 errors at the moment.

Potential workaround: I could set the actor to {"token_error": "expired"} and then have a check permissions plugin hook that returns False if it sees that - though that might not solve the problem of wanting to display a custom error message. Maybe that method could raise Forbidden though.


Kivy3 app on  Mojave 10.14.6 doesn't run

Software Versions

Python:
OS:
Kivy:
Kivy installation method:

Describe the bug
A clear and concise description of what the bug is.
The issue tracker is a tool to address bugs only (search known bugs: https://git.io/vM1iE).
Please use the #support Discord channel at https://chat.kivy.org/ or Stack Overflow for
support questions, more information at https://git.io/vM1yQ.
Expected behavior
A clear and concise description of what you expected to happen.
To Reproduce
A short, runnable example that reproduces the issue with latest kivy master.
Code and Logs and screenshots
// REPLACE ME: Paste your code and logs here, a template can be found at https://git.io/vM1id.

Additional context
Add any other context about the problem here.

I believe the Kivy app has its own bundled python
OS 10.14.6
Kivy3 downloaded today
The app just doesn't start. I tried the suggestion
sudo ln -s /Applications/Kivy.app/Contents/Resources/script /usr/local/kivy
Which runs OK but still the app does nothing?
Any help to a beginner would be very much appreciated


Deprecation warning on installation

Warning text:
DEPRECATION: Legacy editable install of openawsem==1.1.0+121.gd60f7ec from file:///home/fc36/openawsem (setup.py develop) is deprecated. pip 25.0 will enforce this behaviour change. A possible replacement is to add a pyproject.toml or enable --use-pep517, and use setuptools >= 64. If the resulting installation is not behaving as expected, try using --config-settings editable_mode=compat. Please consult the setuptools documentation for more information. Discussion can be found at https://github.com/pypa/pip/issues/11457

We should change setup.py to pyproject.toml


Style: Improve Responsiveness of Intro and FAQ Page

@dnsos should be ready for review and merge now. I did a lot of restyling and content change stuff...

Before merging, I'll need to fix some Typescript issues actually


fixed issue #3

Thanks for fixing this!
Something was off with my Github notifications, hence I'm only seeing this now. (And I already fixed it in the mean while) Sorry about this.


Clear text colors before appending timestamps to messages

mIRC colors and formatting affect appended timestamps on buffer playbacks.
This patch fixes the issue by appending a ^O control char to clear all formatting before adding the timestamp.

Why the space after that byte?

Maybe because the line replaced by the commit was adding a space to the string.

Yeah, didn't notice it...


[💡 FEATURE REQUEST]: Chinese translation

Plugin
None
I have an idea!
I want to make a Chinese version of the document in order to better promote the project, How to start

Hey @shellphy 👋
This would be a nice feature to add. However, without native speakers of the language, that would be impossible to implement.
If you can help with that, basically, all the docs are here in md's: https://github.com/roadrunner-server/docs. I guess that they can be translated via things like ChatGPT and then corrected by the native speaker.

Hey @shellphy 👋 This would be a nice feature to add. However, without native speakers of the language, that would be impossible to implement. If you can help with that, basically, all the docs are here in MD's: https://github.com/roadrunner-server/docs. I guess that they can be translated via things like ChatGPT and then corrected by the native speaker.
After that, you may attach the archive here, to the ticket and ping me or join our Discord server and send directly to me.

As a Chinese, I think such a good project as roadrunner is not very popular in China, which is largely because there is no Chinese version of the document.

Yeah, agree. I would greatly appreciate it if you could help to translate into Chinese.


Optional (depopts) dependencies are forced

For example, the following opam file:
opam-version: "2.0"
depends: [
  "logs"
]

Produces:
opam-version: "2.0"
synopsis: "opam-monorepo generated lockfile"
maintainer: "opam-monorepo"
depends: [
  "base-bigarray" {= "base"}
  "base-bytes" {= "base"}
  "base-threads" {= "base"}
  "base-unix" {= "base"}
  "cmdliner" {= "1.0.4+dune"}
  "cppo" {= "1.6.8"}
  "csexp" {= "1.5.1"}
  "dune" {= "2.9.1"}
  "dune-configurator" {= "2.9.1"}
  "findlib" {= "1.8.1+dune"}
  "fmt" {= "0.8.9+dune"}
  "logs" {= "0.7.0+dune"}
  "lwt" {= "5.4.2"}
  "mmap" {= "1.1.0"}
  "ocaml" {= "4.13.1"}
  "ocaml-base-compiler" {= "4.13.1"}
  "ocaml-config" {= "2"}
  "ocaml-options-vanilla" {= "1"}
  "ocamlfind" {= "1.8.1+dune"}
  "ocplib-endian" {= "1.1"}
  "result" {= "1.5"}
  "seq" {= "base+dune"}
  "stdlib-shims" {= "0.3.0"}
]

Most of which are optional.

opam-monorepo ignores depopts. This happens because the dune port of logs declares some of them as regular dependencies (see here).

Ah, I see. I hadn't considered that the logs-dune package might have a different opam file from the official one.

If you have a need for a minimal logs installation in an opam monorepo setup please let me know so we can try to improve the port.

That would be nice :-)

I'm closing this as it is unrelated to opam-monorepo itself. I'll re-open an issue on dune-universe/opam-overlays as this is where it should be fixed.
If I'm late to do so, please feel free to re-open it there yourself!


v0.9.0 is non idempotent

Description
In prior versions of the compiler, repeated compilations produced byte-for-byte identical results. With v0.9.0, there seems to be some behavior that causes binary diffs in compiler output for identical inputs across multiple runs. In particular, I'm seeing diffs in the autogenerated IDs contained at the root of the document.
Setting document(date: none) does not resolve the issue.
Reproduction URL
The following document produces different compiler output on repeated runs. (typst v0.9.0, macOS)
#set document(author: "", title: "", date: none)
= test

Operating system
macOS
Typst version

[X] I am using the latest version of Typst

This is intended behaviour as IDs are expected by many PDF viewers. Writing the creation date is also good practice. As you've noticed, setting the date to none reverts back to the previous behaviour.

@laurmaedje But shouldn't the generated ID be the same across repeated runs?
Byte-for-byte reproducibility makes tracking pdfs alongside source files in git much easier.

The generated ID should be the same if the date is disabled. It consists of a hash of the file path and a hash of the document content. Are you running clean typst compile every time?

The ID consists of two parts: A stable document ID (file path hash) and an instance ID (document hash) that changes through modifications. Can you confirm which of these two changes by opening the PDF in a text editor and searching for the /ID field (it's close to the end of the file)?

Searching for the /ID field at the end of the document reveals that the second part of the ID is what is changing between repeated clean runs of typst compile

Ok, I have an idea what's going on. The document hash is probably not stable across program invocations because of ASLR (address space layout randomization) or something similar. We're sometimes hashing pointers instead of data for better performance. I think a good fix would be to not hash the document struct itself, but hash the generated PDF content itself (as far it is generated when we write the ID). That should be stable and also more performant anyway.


How to increase update frequency?

I ran the container as follows:
docker run \
  -e OVERPASS_MODE=clone \
  -e OVERPASS_META=attic \
  -e OVERPASS_DIFF_URL=https://planet.openstreetmap.org/replication/minute/ \
  -e OVERPASS_RULES_LOAD=20 \
  -e OVERPASS_UPDATE_SLEEP=15 \
  -v /big/docker/overpass_clone_db/:/db \
  -p <IP_ADDRESS>:12346:80 \
  -i -t \
  --name overpass_world \
  --restart=always \
  wiktorn/overpass-api

It is running and updating just fine, but it seems to only update approximately every third minute and is always at least 2 mins behind the overpass-api.de server (and sometimes up to 5 mins behind). For example, responses to /api/augmented_diff_status would look something like this:

minute
0
1
2
3
4
5
6

me
4463225
4463225
4463225
4463227
4463227
4463227
4463230

overpass-api.de
4463227
4463228
4463229
4463230
4463231
4463232
4463233

I would think that with OVERPASS_UPDATE_SLEEP=15 it's checking ~4 times a minute for updates, but maybe I'm misunderstanding something else? I can't say I fully understand what OVERPASS_TIME actually does, so maybe that's involved? Ultimately I am trying to get it to stay up to the minute and essentially be a clone of overpass-api.de where I am not rate limited.

Thanks for taking the time to look into this! Since there's nothing in my config that was obviously wrong, I'm going to close the ticket-- I'll poke around a bit regarding the possibility of a different replication server and report back if I find anything of note.


Regression in X509_verify_cert

OpenSSL 1.1.1i caused the regression in FreeIPA setups having enabled Anonymous PKINIT (https://pagure.io/freeipa/issue/8632).
The change https://github.com/openssl/openssl/commit/315c47e00bb953abe8892a3c1272289330b29d23 introduced new condition if (issuer != x which always fails in such setups.
FreeIPA uses kinit tool provided by MIT Kerberos https://github.com/krb5/krb5
(gdb) bt
#0  find_issuer (ctx=0x437a30, sk=0x43e680, x=0x43e4a0) at crypto/x509/x509_vfy.c:341
#1  0x00007ffff78e2980 in get_issuer_sk (issuer=0x7fffffffcbd0, ctx=<optimized out>, x=<optimized out>)
    at crypto/x509/x509_vfy.c:361
#2  0x00007ffff78e3e90 in get_issuer (cert=0x43e4a0, ctx=0x437a30, issuer=0x7fffffffcbd0)
    at crypto/x509/x509_vfy.c:2920
#3  build_chain (ctx=<optimized out>) at crypto/x509/x509_vfy.c:3059
#4  verify_chain (ctx=0x437a30) at crypto/x509/x509_vfy.c:217
#5  0x00007ffff78e50bc in X509_verify_cert (ctx=ctx@entry=0x437a30) at crypto/x509/x509_vfy.c:303
#6  0x00007ffff79e4913 in cms_signeddata_verify (context=context@entry=0x40dc10, plgctx=0x4357d0,
    reqctx=0x436720, idctx=0x4368b0, cms_msg_type=cms_msg_type@entry=2,
    require_crl_checking=<optimized out>,
    signed_data=0x443330 "0\202\002\364\006\t*\206H\206\367\r\001\a\002\240\202\002\345\060\202\002\341\002\001\003\061\v0\t\006\005+\016\003\002\032\005", signed_data_len=760, data=0x7fffffffcef8,
    data_len=0x7fffffffcef4, authz_data=0x0, authz_data_len=0x0, is_signed=0x0)
    at pkinit_crypto_openssl.c:1658
#7  0x00007ffff79da837 in pkinit_as_rep_parse (context=context@entry=0x40dc10,
    plgctx=plgctx@entry=0x412140, reqctx=reqctx@entry=0x4366a0, pa_type=17,
    request=request@entry=0x436550, as_rep=as_rep@entry=0x7fffffffcfa0, key_block=0x7fffffffd010,
    etype=18, encoded_request=0x438b30) at pkinit_clnt.c:657
#8  0x00007ffff79dade0 in pa_pkinit_parse_rep (context=context@entry=0x40dc10,
    plgctx=plgctx@entry=0x412140, reqctx=reqctx@entry=0x4366a0, request=request@entry=0x436550,
    in_padata=in_padata@entry=0x43dc50, etype=<optimized out>, as_key=0x7fffffffd010,
    encoded_request=0x438b30) at pkinit_clnt.c:446

...

(gdb) l
336             issuer = sk_X509_value(sk, i);
337             /*
338              * Below check 'issuer != x' is an optimization and safety precaution:
339              * Candidate issuer cert cannot be the same as the subject cert 'x'.
340              */
341             if (issuer != x && ctx->check_issued(ctx, x, issuer)
342                 && (((x->ex_flags & EXFLAG_SI) != 0 && sk_X509_num(ctx->chain) == 1)
343                     || !sk_X509_contains(ctx->chain, issuer))) {
344                 rv = issuer;
345                 if (x509_check_cert_time(ctx, rv, -1))
(gdb) p issuer == x
$15 = 1

Similar reproducer is attached:
x509_verify.c.txt
kdc.crt.gz

Ping @DDvO
Sent with GitHawk

So it just seems to be a self-signed certificate in the trust store. I can't reproduce the error using openssl verify -CAfile kdc.crt kdc.crt, but can with the test program. I currently don't see why they should behave differently.

Thanks @stanislavlevin for reporting this including the code and cert to reproduce the issue.
(BTW, the example code has a double-free on x at the end, but this is not the point here.)
Interestingly, this happens with the 1.1.1 branch but not with the 3.0.0 master.
I've started investigating why...

So it just seems to be a self-signed certificate in the trust store. I can't reproduce the error using openssl verify -CAfile kdc.crt kdc.crt, but can with the test program. I currently don't see why they should behave differently.

I've meanwhile found why this discrepancy exists: in the given test program, x and the cert in the trust store are the same object,
while with  openssl verify they are two instances of the same cert (i.e., non-equal, but equivalent cert objects).

The problem is just a symptom of a more general flaw of build_chain() that I've been pondering on recently:
When it hits a self-issued cert it makes little sense to try and find an issuer of it -
the algorithm should instead just look and see if the cert can be found in the trust store.
Then this and other related issues I've been fighting with for about a year will be gone.

It's not because it's self-issued that it's self-signed. We check that it's the trust store if it's self-signed or partial chain is enabled. So in case partial chain is not enabled and it's self-issued but not self-signed, it should continue to look for the issuer.

Yes, self-issued does not imply self-signed.
I meanwhile think that my above statement can be generalized to:
The chain building should stop whenever the current cert is in the trust store (and its trust is not explicitly rejected).

But for some reason, we have the "partial chain" option for that. I've never really understood why we have that option.

But that all seems to be unrelated to this issue

Yeah, right, the partial_chain business is unrelated here, and am also not convinced about it.
Well, and as long as we respect that option, my statement should have read:
The chain building should stop whenever the current cert

is in the trust store
its trust is not explicitly rejected for the current purpose,
and it is self-signed or the partial_chain option is set.

Meanwhile I've found a good solution and set up a respective PR in #13749 for v1.1.1,
and a related PR #13748 for the 3.0.0 master which also add some more improvements.

The fix in  #13749 has been merged today, so closing this.

The fix in  #13749 has been merged today, so closing this.

I've verified the fix, it works as expected. Thank you!

I've verified the fix, it works as expected. Thank you!


Personal menu

Removing local time on the personal menu. Local time can still be found in the user card.
Fixes:
https://github.com/zulip/zulip/issues/30044

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Thanks! Please go through our guide on submitting a pull request to learn how to present your proposed changes to Zulip, so that your work can be reviewed.

Closing this PR as it does not meet our guidelines. Thanks for giving it a try @adath686 !


How to deal with paste image from HTML

I am trying to copy / paste part HTML page with an image into DRAFT. For image,  I see that DRAFT has created atomic block but no entity was attached to it,  image is not displayed in DRAFT.
I have a decorator already which looks for IMAGE type entities and renders them appropriately.
How to handle copy/paste events and insert custom render behaviour?

@ravuribabu - my apologies for such a late response. I think this should be handled by #832, can you see if this solves what you were looking for?

I think this was included in the 0.10.1 release, from https://github.com/facebook/draft-js/commit/8ac72f723fb2d9102db833a9b060dfd66df65652
If any questions remain please open a new issue, thanks!


Explicitly mark objects of structured type with base class data

Base class data is vended as "just another child", which is mostly OK, except sometimes it's useful to know when the base class is being parsed vs. a member variable
Relying on the name "super" isn't actually correct, but at the Mirror level we actually know base class vs. ivar
This change propagates that knowledge via adding an extra byte to the payload for structured data in the logger format, such that the logger can tell consumers whether the 0th child is the base class or the first member variable
rdar://problem/19072746

The Swift project moved the default branch to main and deleted master branch, so GitHub automatically closed the PR. Please re-create the pull request with main branch.
More detail about the branch update - https://forums.swift.org/t/updating-branch-names/40412


Package a RisingWave image that supports HDFS after each release

Currently we support HDFS as storage backend but it's off by default, thus we should package a RisingWave image that supports HDFS after each release. We need to package manually on EC2 instead of using buildkite, I have write a guidance to explain how to package it.
https://www.notion.so/risingwave-labs/Package-RisingWave-with-HDFS-6ea9e37568bd4e1882b9217efdc5981b?pvs=4
Note that libhdfs is always upwardly compatible, so we only need to package a lower version of HDFS. Let us use HDFS 2.7.3 as the default (the same with Databend).
I have package one for v1.2 here, and we need  need to do this every time we release from now on.

seems like the guidance to package is a internal doc, looking forward this feature or any suggestion to mannual package with 1.5.x?

Hi @sheltonsuen , indeed it's an internal doc, and we can provide a 1.5.x image that supports HDFS if you need. Could you please let us know your hdfs version?

thanks, It's fine, after using the 1.2 image, working as expected

You mean you are using RisingWave 1.2 on HDFS, and everything works well?
Now you want to use RisingWave 1.5.x so that requires a new RisingWave version, right?

sorry about confusing, I mean this image: RW_v1.2_HDFS_v2.7-x86_64

I see. We will deliver a new version later, and will let you know here~


Gradients in the headers fail to work

For some reason, the gradients in the headers (headerView in most of the app's View Controllers) do not appear. Instead, only the background colour (of which was hardcoded) was shown.
I'll need to investigate a bit and figure out why this fails to work.

Found the answer here!
It seems like the issue was that headerGradientLayer.frame = headerView.bounds was called in viewDidLoad, which results in the gradient failing, I assume. By moving the line of code over to viewDidLayoutSubviews, the frame is set when Auto Layout runs, meaning that the gradient appear. TIL!

Found the answer here!
It seems like the issue was that headerGradientLayer.frame = headerView.bounds was called in viewDidLoad, which results in the gradient failing, I assume. By moving the line of code over to viewDidLayoutSubviews, the frame is set when Auto Layout runs, meaning that the gradient appear. TIL!

Wait a second...
Copy-pasting the code from HomeViewController (where the gradient works perfectly fine) to WelcomeViewController causes something to break; suddenly, the same lines of code produced different results. What's happening here? 😵

Wait a second...
Copy-pasting the code from HomeViewController (where the gradient works perfectly fine) to WelcomeViewController causes something to break; suddenly, the same lines of code produced different results. What's happening here? 😵

A helpful answer from Stack Overflow provided the solution to this issue.
Thank you to Tyler for the answer!

A helpful answer from Stack Overflow provided the solution to this issue.
Thank you to Tyler for the answer!


errors,version(ticdc): refine ErrCheckClusterVersionFromPD error

What problem does this PR solve?

Issue Number: close #4341
What is changed and how it works?

Add actual error to error message
Add retry hint.

Release note
None

Codecov Report

Merging #4435 (f294726) into master (ab62e6b) will decrease coverage by 0.0878%.
The diff coverage is 46.6303%.

Flag
Coverage Δ

cdc
59.4195% <55.1105%> (-0.1964%)
:arrow_down:

dm
52.0856% <41.8856%> (+0.0200%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.
@@               Coverage Diff                @@
##             master      #4435        +/-   ##
================================================
- Coverage   55.5183%   55.4305%   -0.0879%
================================================
  Files           495        495
  Lines         61151      61044       -107
================================================
- Hits          33950      33837       -113
- Misses        23789      23803        +14
+ Partials       3412       3404         -8

/run-all-tests

/merge

/run-kafka-integration-test
/run-dm-integration-test


Error in WSL font highlight after resizing window

Lastest version of MS terminal preview
The highlight of exec files will behave strangely after resizing the window

/dup #32

Sorry for the duplicating, that one did not show up during my keyword search


Update comment from 'iff' to 'if'

"Iff" means "if and only if". See https://en.wikipedia.org/wiki/If_and_only_if or https://www.merriam-webster.com/dictionary/iff

"Iff" means "if and only if". See https://en.wikipedia.org/wiki/If_and_only_if or https://www.merriam-webster.com/dictionary/iff

Thx, sorry for my misunderstanding.


Fix ThrottleGatewayFilter obtain TokenBucket problem

TokenBucket should not create every time

@JoeCqupt Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@JoeCqupt Thank you for signing the Contributor License Agreement!

@spencergibb  PTAL


Clone  to specified folder crashes

I am cloning a new empty repo into a specified folder. The clone command fails with "The given key was not present in the dictionary"

Last part of the long line:

You said you're cloning a "new empty repo" which sounds like you're hitting https://github.com/Microsoft/VFSForGit/issues/58. Please reactivate if I misunderstood that.

That's right, it is the same issue.  Understand error messages are not high prio right now, but over some time I hope you get out a more meaningful error here.  :-)


netperf aggregate throughput stat

Add up all the throughputs. TODO evaluate whether we ramp ups and ramp downs are distorting this at all. Just wanted to get this in before 1.8.0.

LGTM

Added the assert you guys wanted


CSL dataset Trained checkpoint

hi, me again, may you upload  CSL dataset Trained checkpoint, thanks

As there are some tricks to handle this dataset, we leave it for future plan. You can refer to this issue here.

And how to generate attention heat map in SEN? Thanks

We follow Grad-CAM to generate the class activation maps of the last layer of SEN as attention heat maps.

If I'm understanding correctly, the way data is handled shouldn't be a topic so difficult to discuss publicly that it might as well be written in Martian language!

Sorry, i can't fully understand your question. What does 'the way data is handled' refer to?

We follow Grad-CAM to generate the class activation maps of the last layer of SEN as attention heat maps. I just upload the 'generate_cam.py' and you may use it to generate the attention maps.

Sorry for my relpying at the wrong place


Mine

Describe the feature you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Mine!

craft


wt command should by default background itself (detach) when opening a new window

wt command should by default background itself (detach) when opening a new window
When the user types into a window "wt -- ssh user@host" it will open up a new window. In the current implementation, it will also leave the original window blocked until the child window is closed.
Proposed technical implementation details (optional)
A default action of backgrounding (or simply forking and exiting the parent process) would leave the parent shell available for use. Not backgrounding/forking blocks the shell and is (unless you are waiting for the exit code of the child) both an abuse of resources and clutter/messy.
IFF the user explicitly wants to wait for the exit code, there should be a command-line switch for enabling that.

Not backgrounding/forking blocks the shell and is both an abuse of resources and clutter/messy.

It is also a bug, albeit one in powershell: https://github.com/PowerShell/PowerShell/issues/9970


Incorrect installation command for Arch linux

The installation command in the README for Arch linux is pacman -U jless, but jless is not (yet!) in the main arch repositories. The link to the AUR is correct, but the pacman command cannot install from the AUR.
Since using tools like yay/aurman etc. which do install from the AUR are not officially supported by arch, and arch users presumably know how to install from the AUR anyway, I'd recommend just keeping the link, and not giving an installation command, just a message like "Available in the AUR".
Love the tool!

Thanks for the heads up: Fixed in 8cf15ea0a1e2ab2e4151a183a1125fc596c8371c (README) and 8210adfcc9232aa79003635151f2596b5afd6c92 (website)

@PaulJuliusMartinez the link https://aur.archlinux.org/packages/jless is a 404. Did you remove the package?

In addition, the command is not quiet right to install directly from Arch Linux repositories.
pacman -U installs a package from a file path or an URL.
To install directly from the repositories, pacman -S is being used.


Analyze why ErrorTolerantWalker is needed

Uncomment the exception handling in ErrorTolerantWalker and run against the project.
There are times where calling the TSLint API results in an exception.
For all exceptions:

ask TSLint project if the error is on our side or is an error in their project
create the relevant issue in GitHub

We might be instantiating the typechecker and language services wrong:
Try an apply method like this:
public apply(sourceFile: ts.SourceFile): Lint.RuleFailure[] {
    const program = ts.createProgram([sourceFile.fileName], Lint.createCompilerOptions());
    const checker = program.getTypeChecker();
    return this.applyWithWalker(new BooleanTriviaWalker(checker, program.getSourceFile(sourceFile.fileName), this.getOptions()));
}

The error walker is needed because tslint does not configure the language services. We can file a bug there, but there is not much we can do on our side.


[ACM-10272] Added warning logs for deprecated spec field usage

Description
Added a warning log to indicate that a deprecated field is being used with the MCE CR spec.
Related Issue
https://issues.redhat.com/browse/ACM-10272
Changes Made
Provide a clear and concise overview of the changes made in this pull request.
Screenshots (if applicable)
Added additional logging to check if deprecated fields are being used within MCE.
Checklist

[ ] I have tested the changes locally and they are functioning as expected.
[ ] I have updated the documentation (if necessary) to reflect the changes.
[ ] I have added/updated relevant unit tests (if applicable).
[ ] I have ensured that my code follows the project's coding standards.
[ ] I have checked for any potential security issues and addressed them.
[ ] I have added necessary comments to the code, especially in complex or unclear sections.
[ ] I have rebased my branch on top of the latest main/master branch.

Additional Notes
Add any additional notes, context, or information that might be helpful for reviewers.
Reviewers
Tag the appropriate reviewers who should review this pull request. To add reviewers, please add the following line: /cc @reviewer1 @reviewer2
/cc @cameronmwall
Definition of Done

[ ] Code is reviewed.
[ ] Code is tested.
[ ] Documentation is updated.
[ ] All checks and tests pass.
[ ] Approved by at least one reviewer.
[ ] Merged into the main/master branch.

/lgtm