All heading levels have same font size

The latest docs-resources have made all heading levels to have same font size. Visually this does not helps in differentiating between the sections and subsections.
Atleast the h1, h2, and h3 can have 2pt difference among each other.

Also the generated pdf has bigger code formatted text then the normal text.

Currently we are relying on the default docs_resources provided theme , but we are willing do change if required. Please let me know

This is the WIP specification which i am working on https://github.com/riscv-non-isa/riscv-rpmi

@wmat

Sorry, just getting back at it after vacation.  So interestingly, we used to have the headings font sizes getting incrementally smaller, however someone complained about that and therefore we set them all to 11.5, except for H1, which is 11.5 * 2.8.
Do you mind if we move this discussion to the Docs SIG mailing list, as I'd like to get additional opinions.

Not sure how heading levels can be issue, if anyone dont want different heading sizes then they can use the same heading level.
Sure lets discuss this in Docs SIG

I'm just doing some testing with the heading levels and fonts on your spec.  Once I get it looking nice I'll update the docs-resources theme.  I decided not to open a discussion yet.  Once thing I noticed with your spec is when using a literal block (beginning a line with a space) in a table, you'll want to make sure that cell is formatted as asciidoc.  You can do this by putting an a before the cell separator:
! [31]		a!

	0b0: Custom resume address not supported.
	0b1: Custom resume address supported.
! [30]		a!

	0b0: Suspend not supported
	0b1: Suspend supported

This ensures it uses the code block category keys in the theme.


Fix uninitialized Rocksdb.level0_stop_writes_trigger

Hi, i add the support to config level0_stop_writes_trigger, because we have this variable in config.h, moreover, we shouldn't  bind them together, they are changed respectively for tuning.

@Mergifyio backport 1.3


Require ruamel namespace package

Rebuilds the package with the ruamel namespace package to avoid clobbering between different packages using the namespace.
cc @jjhelmus @mariusvniekerk

Checklist

[x] Used a fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged. ( please see PR https://github.com/conda-forge/ruamel.ordereddict-feedstock/pull/12 )

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Have gone ahead and merged to move things forward. Please let me know if there are any issues.


Automated Package Upgrade

:tada: This PR is included in version 5.0.284 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Market: Clicking on account balance vs Lowest Ask

This is strange. I don't know what to make of it, but it could be a javascript floating point issue:
When I click on the underlined balance next to billbutler-dev, the Pay field contains 322.78844

When I click on the underlined balance next to Lowest ask, the Pay field contains 322.78049

Pretty sure this is an easy fix and I'd place it as high priority since this is a confidence killer.

This is not actually a bug but a consequence of how the blockchain assets work, specifically the limited and different precisions.
Say asset A has precision 2 (max 2 decimals), asset B has precision 5.
I have 5.12555 of asset B that I want to sell for 2 A/B.
With full precision I would receive 2 * 5.12555 = 10.2511 of asset A. However, that is not possible since the precision of A is 2, so I would need to sell slightly less B in order to receive a maximum of 10.25 A. The max amount I could sell would be 10.25 / 2 = 5.125 B.
So there's a remainder of 0.00055 B that I cannot sell.
What I can do is add a warning in this case, making it more explicit that it is not possible to sell the entire balance at the chosen price.

So, since this is a shortcut means to enter an order, can you just run the calculation to deal with the exception rather than throwing a warning? I can't thing of a case where a user wants to be presented with an error.

Yes, I can calculate the max amount one can sell at the given price, and use that as the total instead of the actual balance. I suspect users will still find it confusing however that upon clicking their balance, the whole amount is not used in the transaction.

This leads to the issue of needing to deduct out fees right? I think that in order to make this flow properly, you should have a line item for the fee and extract the fee as well. Would this provide a clear consistent feedback loop?

Yes the fee should be taken into account as well, I've been meaning to add the fee anyway. Mind making a mockup of how the fee should be presented? Below the total? Below? Line separators? etc

Yes, I'll mock this up and move to 1118

Moving this to 1125 and UXing it soon.

OK, @svk31 here is the UX for the fee.

I'd appreciate some testing of this, it's pretty complicated handling different edge cases here so I might have missed something.. I'm using a notification to tell the user he doesn't have enough funds to pay the fees since it was the fastest way, I can go back later if we really want it in the Fee field.

Ultimately, we will want it in the fee field, but this is a major improvement.


Very interesting work!

What do you think might be the cause of LLM drift?
I had a suspicion that this was the case, because GPT4 seemed to be learning in real time (or something similar), or OpenAI stealth updating their model called "GPT4", waiting for a certain large number of new parameters before calling it "GPT5". But these are just speculations, I don't know the cause.
If it's one of these 2 though, I think the new feedback that GPT4 is getting might be more inaccurate that accurate, and there might also be malicious feedback being incorporated into the model.
I think your paper also shows that "freezing" LLMs and keeping their different versions will have more utility in certain applications where replicability and stability is needed at the expense of making it acquire new knowledge.

I agree. Freezing model will be an important step so will creating a comprehensive evaluation task.  I also think that updates rolled out should be named versions. For these models to mimic reproducibility, we first need to ensure that for every minor change in the model, they name the release.


Add G-Cloud 10 management to ccs-sourcing

Summary
CCS Sourcing need to be able to manage and edit declarations for G-Cloud
10. Until we have an opportunity to re-design this page/journey, we will
simply add G-Cloud 10 to the existing page. It does make things a little
tight and the design clearly doesn't hold up, but we have a ticket to
prioritise this work in the next sprint or so.
Ticket
https://trello.com/c/Voz6ZjDV/216

Yep, just given we expect to be removing this page soon, I went for MVP.


Lookup resources to be installed in a release

I am preparing helm helper functions to ultimately provide a consistent single chart for all application development, one of the issues I had is that Helm does not provide the ability to lookup objects that would be part of the pending release. I am still reading through the source code so I have to see in what Order things are read, but there can always be post-processing stage for some function execution.
Anyway i found a work around to the issue as follows
helm template  --name-template ha-backend-stage --output-dir validation .
for f in `find validation -name \*.yml -print`
 do
  PV_VARS=$(yq '. | select(.kind == "PersistentVolume") | .metadata.name' $f)
done

# then feed the values in during the install phrase using --set to be able to configure the volume and mounts in the deployment description

Typically you would use named templates for that, and template the name so you can include it for both the PV name, and any place you need to reference that PV.


How to get UniquePtr on the Rust side?

I have the following struct on Rust:

#[cxx::bridge]
pub(crate) mod ffi {
    enum SizeType {
        BYTE,
        WORD,
        DWORD,
        QWORD,
    }
     unsafe extern "C++" {
         //...
     }
}

which is also mapped on C++. How do I get UniquePtr<SizeType> on Rust? Do I have to write a C++ function to get it? If I do, there's no point in having the struct on the Rust side.
I tried
let byte_ptr = UniquePtr::new(SizeType::BYTE);
but it does not work:
the trait bound bridge::ffi::SizeType: cxx::memory::UniquePtrTarget is not satisfied
the trait cxx::memory::UniquePtrTarget is not implemented for bridge::ffi::SizeType

Closing in favor of https://users.rust-lang.org/t/how-to-get-uniqueptr-enummember-on-the-rust-side/70954?u=dtolnay.


Add Language Support to Backend

Double check that survey questions/answers support unicode and not just ASCII (for supporting accented Spanish characters)
Add language to the SurveyResponse
Add a new text column and response_options column per language to SurveyQuestion

@mzagaja It sounded like this was on hold based on concerns about recording the language of the survey response. Should this be closed as won't fix?

Let's circle back with Christopher on this. I don't recall the final decision on this.


Max line length fix

This PR corrects the line to fit in 120 char limit. Issue #1041 .

Automated message from Dropbox CLA bot
@staranjeet, it looks like you've already signed the Dropbox CLA. Thanks!

@timabbott : please review.

For all the def lines, the typical Python style solution to wrapping something like this:
-def api_circleci_webhook(request, user_profile, client, payload=REQ(argument_type='body'), stream=REQ(default='circleci')):

is one of these:
+def api_circleci_webhook(request, user_profile, client, payload=REQ(argument_type='body'),
+                         stream=REQ(default='circleci')):
+def api_circleci_webhook(request, user_profile, client,
+                         payload=REQ(argument_type='body'),
+                         stream=REQ(default='circleci')):

where the variables are wrapped aligned with the open-paren of the def statement.
not
+def api_circleci_webhook(request, user_profile, client, payload=REQ(argument_type='body'),
+        stream=REQ(default='circleci')):

Can you change those to work that way?
Additionally, I think in a lot of places you should be doing a more extensive change to the code to make it read reasonably.  E.g. for
-                request.META['HTTP_AUTHORIZATION'] = "Basic " + force_str(base64.b64encode(credentials.encode('utf-8')))
+                request.META['HTTP_AUTHORIZATION'] = "Basic " + force_str(
+                    base64.b64encode(credentials.encode('utf-8')))

I think much clearer code would be to do something like this:
encoded_credentials = force_str(base64.b64encode(credentials.encode('utf-8')))
request.META['HTTP_AUTHORIZATION'] = "Basic " + encoded_credentials

I think most of the commits would be better if one of those two types of changes were made to the code; can you review this branch for issues of that form and clean this up to match our standard style for how to do line-wrapping?  Also, if my above thoughts weren't clear in the documentation, perhaps we should add some text to http://zulip.readthedocs.io/en/latest/code-style.html discussing how we prefer to line-wrap code (we'll want to link to those docs from the linter).
Thanks for working on this @staranjeet!

Ok, I will update it.

@timabbott : I think you can review now.

Thanks for working on htis @staranjeet!  I merged about 2/3 of the commits, mostly unchanged (I redid zerver/views/report.py in what I think is a better way).  The rest don't match the Zulip style for wrapping (there's various issues, am still looking).

OK with some tweaks I merged everything other than the bitbucket one.  For that, I think a better fix for many of them would be to factor out the User {display_name}(login: {username}) part into a variable and use that in each of the templates.

@TroJan can you update the remaining commit so we can close out this pull request?

Closing this since you did the remaining piece in #1384.  FYI, you didn't need to create a new pull request for that -- you could have just force-pushed to the branch this PR was made from.  Thanks @TroJan!


Live Preview Not Working

When I go to click Live Preview on Brackets, it opens Chrome and then says:
This webpage is not available
ERR_CONNECTION_REFUSED
Google Chrome's connection attempt to localhost was rejected. The website may be down, or your network may not be properly configured.
I've already tried uninstalling extensions and re-installing Brackets, but nothing works. My guess that is that something is wrong in the Project Settings. I went there once and may have changed the URL, and now it currently is set at going to http://localhost:8000/.
If anyone is able to help me figure out how to fix this, that would be much appreciated. Thank you!

Only use Base URL if you are using your own local server. If so, make sure your server is started, otherwise delete Base URL (i.e. set it to [blank]).
If it's still not working, read the Live Preview section of the How To Use Brackets doc and then the Live Preview Isn't Working section of the Brackets Troubleshooting guide.

Oh wow, haha. I don't know why I didn't think of that. Sometimes, I tend to over think these things. It worked! Thank you so much.

Closing as fixed then.


where is the '~' symbol resolve rule

Do you want to request a feature or report a bug?
Yes, sorry for the question.
In the .css file, we can import file use ('~moduleName'),but i can't find why?
I search the source code, find in stackoverflow and issue,but there is nothing about the ~ symbol resolve rule.
Could you tell me why the webpack will resolve as module if path begin with ~ sybmol?
Where is the source code?
Thank you so much.

What is the current behavior?
If the current behavior is a bug, please provide the steps to reproduce.

What is the expected behavior?
If this is a feature request, what is motivation or use case for changing the behavior?
Please mention other relevant information such as the browser version, Node.js version, webpack version, and Operating System.
mac
webpack@3
node@8

https://github.com/webpack/loader-utils/blob/67499ff3d13c13ef455ec0fede90b714fc16d787/lib/urlToRequest.js#L23-L29


[Tech]: Set up Moonbase Alpha for HRMP channel with Moonbeam

Problem statement
We want to open an HRMP channel with the Moonbeam parachain on Kusama. The Moonbeam team requires that we start with a Moonbase Alpha to ensure smooth operations while opening channels between our parachains.
Description
Guide: https://docs.moonbeam.network/builders/interoperability/xcm/xc-registration/xc-integration/#moonbase-alpha-xcm
Definition of Done

[ ] Environment set up
[ ] Sync with Moonbeam team to sync on proceeding further

Requirements
No response

Created chainspec and prepared runtime code, waiting for deployment process


delete installation now requires force flag

Let me know if the design around this does not work! I think the aesthetics are ok.
Gentle reminder, as I always forget this, change is backwards incompatible and requires a major revision of om when released (as we've changed the way the delete-installation command works by default).
fixes #341

cc @jtarchie @kcboyle 👯♂

Hi, @zachgersh ! Thank you for the PR! We are working on a couple 2.6 support items currently, but have this PR scheduled for shortly after that work is done (about 5 stories), just to give you a general time frame around when we will be able to pull it in.

@zachgersh merged! Thanks so much for the contribution.


Reconcile formulas for apparent resistivity with units checking

in TTFZ we compute:
rxy = T * (abs(Zxy) ** 2) / 5
ryx = T * (abs(Zyx) ** 2) / 5.
But I thought it was:
rxy = 2e-7 * T * (abs(Zxy) ** 2)
ryx = 2e-7*  T * (abs(Zyx) ** 2)
The upper expression is correct when units are mV/km and nT, the lower is correct in SI units.  If we keep the units with the data then we can choose the appropriate form.
The formulas above are correct, but the error bars are not when we use SI data.  We will have to modify the formula:
'''
correct for MT units, but not for SI units
rxy_se = 2 * np.sqrt(self.periods * rxy / 5) * Zxy_se
ryx_se = 2 * np.sqrt(self.periods * ryx / 5) * Zyx_se
'''

Zxy_se needs to be scaled by 1e-3.
This is because in SI we have:
rxy = 2e-7 * self.periods * (abs(Zxy) ** 2)

and for mv/km and nT we have:
rxy = 2e-1 * self.periods * (abs(Zxy) ** 2)

Clearly the SI requires a scale factor of 1e-6 applied to the square of Zxy.
Assuming Z_xy and the standard error of Z_xy have the same units, then we would scale Zxy by 1e-3 to make the conversion.
i.e. we could have written the SI experssion as:
This is because in SI we have:
rxy = 2e-1 * self.periods * (abs(1e-3*Zxy) ** 2)

and then we see that we just substitute Zxy *= 1e-3 when working in SI and the rest of the formulas are the same.

The error bars  now appear to be correct when SI units are used.  There is still some work to be done tracking units, but that is a separate issue (#36 )


Andriy's changed

I'm not sure why all of the files are popping up but with this one I only changed the names of turtlebot and roomba to reflect what Jake wanted for name changes.

All the files are being listed because this PR includes a merge with develop, which probably could have been done before creating the branch in the first place. The way to avoid this in future is by rebasing the branch on top of latest develop. No worries now though.


Give users more help in solving contravariance issues

See https://github.com/dart-lang/sdk/issues/33119#issuecomment-401203174 for background.
DDC throws with:
TypeError: Closure 'Manager_getResponse_closure': type '(SubContext) => Response<SubContext>' is not a subtype of type '(Context) => Response<Context>'

Unfortunately this doesn't really explain anything, and because most of Dart is based on covariance, I (incorrectly) thought "WTF, its a Response, that's even BETTER than Response". Of course, that is not the case...

Does our language tour or Dart 2 FAQ have anything to help here?
Can the error message call out "contravariance" or something to help the user?

In https://github.com/dart-lang/sdk/issues/33119#issuecomment-401203174 specifically, we are statically asking for a Callback<Response<Context>>, but at runtime getting Callback<Response<SubContext>>, which of course is contravariantly invalid.

Hmm, in general, cases where a type parameter T is used contravariantly:
class Foo<T> {
  void Function(T) callback;
}

is a nasty foot-gun.  If we want to avoid the strange runtime error, options appear to be:
(1) Disallow the contra-variant T (in getters / fields / return types).  This would break a lot of code.
(2) Treat the contra-variant T as Object in the type system.  I.e., anything assigned to callback has to actually be (Object) => void.
(3) Use this as a signal that T must be invariant in Foo.  I.e., Foo<Object> f = new Foo<String>() becomes a static error.
In Matan's example, (3) would mean a static error here:
Response<Context> response = manager.getResponse();

Thanks both @jmesserly and @vsmenon for following up!
I'd love to see how much work it would be to do (3), and what the impact on google3 looks like.

(1) is too breaking.
(2) seems essentially the same as 1? Like we're still disallowing contravariant T, but instead of an error, we're silently making it Object for you ... that seems like it would be really confusing. I have a field Callback<T> callback but I can't set a function that takes a parameter T?
(3) seems very breaking as well. We'd need to enforce invariance on those types for runtime casts. It will break because Angular constructs components with <dynamic>, but the models usually have some other <T> on there (if I'm understanding Angular's behavior from threads). I totally want invariance but I don't think we're ready for it (if we were, we shouldn't be hitting these errors, and they wouldn't be so hard to fix).
I haven't had a chance to chat with Leaf yet, but I think we were thinking something more along the lines of resurrecting a fuzzy arrow type approach:
class C<T> {
  final Function(T) callback;
  C(this.callback);
}
main() {
  C<Object> c = new C<int>((int x) => x.abs());
  var f = c.callback; // previously fails, now works
  // f has type `(?) -> dynamic`
  // `?` means we don't know the param type, need to check it before assuming.
  f(42); // works
  f('hello'); // fails runtime param type check
  Function(Object) g = f; // fails runtime cast
  Function(int) h = f; // works
  c.callback = f; // works
}

This would fix most of the examples I think, with no code changes. We could provide a way for users to write these types as well, depending on whether this is a short term hack or not.
Long term yes sound variance please. Need to make Angular understand generics :)

We're working on the generics issue. FWIW, the work-around has been to loosten the expectations to dynamic anyway, so I'm not sure the invariant type will be strictly speaking breaking (or not any more breaking than the current behavior).
For example, we advise users to rewrite:
class ParentComponent {
  String renderName(Animal a) => a.name
}

// <T> is always _dynamic_ in AngularDart today.
class ChildComponent<T> {
  @Input()
  String Function(T) itemRenderer;
}

... to ..
class ParentComponent {
  String renderName(Object a) => (a as Animal).name;
}

Note also an option (4), which is the one that I've recommended, which says that members whose type contains a type variable from the enclosing class in a contravariant or invariant position should be given a static type which is sound (that is, a common supertype of all the possible actual types):
Foo<T>.member will have static type [Null/T]S when S is the declared type of member and T occurs contravariantly in S; and static type Function when T occurs invariantly in S and S is a function type; and static type Object(*) when T occurs invariantly in S and S is a generic instantiation of a class; and the static type S when T occurs covariantly or does not occur at all in S.
The (*) is there because we might instead want to use a superinterface where T is not invariant and not contravariant, e.g., because it doesn't depend on T at all.
For option (3), I'd recommend that just support invariance as a separate feature. With that, developers would be able to get a safe and informative type for a "contravariant member" when they are ready to give up on covariance (which is perfectly fine for all the members whose type is actually covariant).

with (4) you wouldn't be able to call the function, though? Unless you pass null.

Yes, because we should check that actual arguments satisfy the actual requirements. Relying on a check which guarantees that relationship statically would prevent us from ever putting anything else than null into a list (because there is no other statically expressible type than Null which is guaranteed to be less-equal than the actual type argument of any given list).
I do realize that we get that for free as long as we choose to check the type of an actual argument whose type is covariant (from class or via the covariant modifier) in the body of the callee, and it's more difficult to perform the same check from the caller side, but we can at least start "doing the right thing" in this respect in the case where the statically known parameter type contains a Null because it's in the signature one of these "contravariant members".
Another case where it would be very useful to check against the actual parameter type rather than a statically known Null is with callbacks where we basically just want to specify the arity and somehow have a reason to trust that the call will be ok: int foo(int Function(Null) f) { f(something); }.

Bump.  @davidmorgan hit this again in built_value.
void main() {
  var i = new FunctionHolder<int>()..function = (int x) {};
  var j = i as FunctionHolder<dynamic>;
  print(i.function == j.function);
}

class FunctionHolder<T> {
  Function(T) function;
}

A runtime error occurs when we access j.function as the actual function isn't consistent with the static type of j.

@jmesserly - as an initial step, perhaps we can improve the runtime error on j.function?

In this particular case I think @davidmorgan hit it in the VM, so DDC would not be enough.

Good point.  Are injecting the cast in kernel or the backends?  If the former, we should probably mark it specially.

Good point. Are injecting the cast in kernel or the backends? If the former, we should probably mark it specially.

The front ends inject the cast (CFE/Analyzer). You can probably tell it's one of these casts already, because it will be an implicit cast to/from the same function type. Marking them would be cleaner though. I don't recall if Kernel does that or not.


How-To: Control concurrency and rate limit applications

It's unclear if the concurrency setting is per pod, or if it can be  used as a distributed semaphore. Should probably be clarified.

Resolved. @trondhindenes  please take a look at the new docs with the PR


Using $merge with Fastify and shared schemas

I'm trying to put this together with fastify. I added ajv-merge-path as described in the docs into the validation (https://github.com/fastify/fastify/blob/master/lib/validation.js#L157)
example case:
schemas/defs.json
{
  "$id": "/defs.json",
  "type": "object",
  "definitions": {
    "test": {
      "type": "object",
      "properties": {
        "name": { "type": "string" }
      }
    }
  }
}

schemas/foo.json
{
  "$id": "/foo.json",
  "type": "object",
  "definitions": {
    "bar": {
      "type": "object",
      "$merge": {
        "source": {
          "$ref": "/defs.json#/definitions/test"
        },
        "with": {
          "properties": {
            "id": { "type": "string" }
          }
        }
      }
    }
  }
}

fastify.get('/defs.json', (request, reply) => {
  reply.send(require('./schemas/defs'))
})
fastify.get('/foo.json', (request, reply) => {
  reply.send(require('./schemas/foo'))
})
fastify.post(
  '/',
  {
    schema: {
      body: { $ref: '/foo.json#/definitions/bar' }
    }
  },
  function(request, reply) {
    reply.send(fastify.getSchemas())
  }
)

I'm not sure why I need to add the requests for the json files, but Swagger seems to need them
The body in Swagger is empty. If I add properties directly to foo, everything works as expected, so the $merge doesn't work.

Any ideas how I can make this work?

I think it should be submitted to fastify rather than here - most likely it simply doesn't support $merge. Also, fastify seems to use its own instance of ajv that doesn't have $merge added to it. Either fastify needs to add $merge to the instance or you need to pass your own instance (and it still doesn't mean $merge would work for anything but simple validation).

Thanks, I'll check there.


Optimize Frontend Docker Image

Optimized the Frontend Docker Image as follows

900 MB Image reduced to ~ 30 MB
Multi-Stage Build
Different images for Development and normal use. (Steps to use images remain unchanged)

Awesome 👍  Great work!


Release 2<IP_ADDRESS> GA

2<IP_ADDRESS> GA will be released around 00.30 UTC 18th November / 4.30 PT 17th November
https://engineering.helium.com/2021/11/17/blockchain-release-resuming-the-blockchain.html

@shawaj Are we considering this as standard emergency? (requiring 6 hours of testing)
Or as emergency+ and we should get it to production asap?

@shawaj Are we considering this as standard emergency? (requiring 6 hours of testing) Or as emergency+ and we should get it to production asap?

I guess ASAP @KevinWassermann94 given it says it's critical. What do you think?
I'm inclined to say ASAP since it can't break anymore than it already is IMO

@shawaj Absolutetly. This should be critical. First units seem fine on Testnet

Closed by #232


Update van_de_Schoot_2018 (PTSS) dataset

The Synergy version of the PTSS dataset has been improved. We comprehensively reviewed and re-applied inclusion criteria, incorporating updates such as stricter definitions of PTSS, acceptance of treatment studies, inclusion of PTSD cluster subscores, allowance for overlapping datasets, and correcting a few labeling errors. The detailed screening logbook is available at OSF, and the re-labeled dataset is available at DataverseNL, including full methodological details.
We have also updated the review as part of a grant (see the pre-registration) and this updated version will become available soon.
How do we want to process the PTSS re-labeled dataset and the updated dataset? Do we replace the existing version with the re-labeled one and add the update as a second dataset? Or, do we combine the two datasets into one bigger dataset and add this one to the Synergy collection??

In the next major release, we can replace van_de_Schoot_2018 with the updated dataset. As all our datasets are peer-reviewed systematic reviews with OpenAlex ID, having a published paper would be nice (or essential).


Fail to remove reference if stored node contains reference meta-pointer with empty targets

Assume the repository contains Disk_A.json.
When we store Disk-move-child-partition.json, we expect node ID-4 to have no references. It actually keeps the reference to ID-10.

Hi, I tried this on main branch, but I cannot reproduce the problem.

Looked at it with Niko, cannot reproduce it together either, so closing this.

I think this issue still exists.
Use branch lw-java/bulk-api and execute test TestRemoveReference.updateSingleNode()
Expected:
SerializedClassifierInstance{id='ID-4', parentNodeID='ID-3',
...
 references=SerializedReferenceValue{metaPointer=MetaPointer{key='Folder-linked-key', version='2023.1', language='-default-key-FileSystem'}, value=[Entry{resolveInfo='null', reference='ID-10'}, Entry{resolveInfo='null', reference='ID-8'}]},
...

actual:
SerializedClassifierInstance{id='ID-4', parentNodeID='ID-3',
...
 references=SerializedReferenceValue{metaPointer=MetaPointer{key='Folder-linked-key', version='2023.1', language='-default-key-FileSystem'}, value=[]},
...

Somehow opposite in TestChangePropertyValue.updateNode3():
expected:
SerializedClassifierInstance{id='ID-4', parentNodeID='ID-3',
...
 references=SerializedReferenceValue{metaPointer=MetaPointer{key='Folder-linked-key', version='2023.1', language='-default-key-FileSystem'}, value=[]},
...

actual:
SerializedClassifierInstance{id='ID-4', parentNodeID='ID-3',
...
 references=SerializedReferenceValue{metaPointer=MetaPointer{key='Folder-linked-key', version='2023.1', language='-default-key-FileSystem'}, value=[Entry{resolveInfo='null', reference='ID-10'}, Entry{resolveInfo='null', reference='ID-8'}]},
...

I cannot run the Java code, building the project fails.  What I do see is that the file Disk-remove-reference-partition.json you are using is actually the same as Disk_A.json, the references are still in the file. That is not how it was intended, but the expected result might then be different.

I copied the json files from lw-repo/main. Now TestRemoveReference works, but still one issue with TestChangePropertyValue.updateNode3():
expected:
SerializedClassifierInstance{id='ID-4', parentNodeID='ID-3',
...
 references=,
...

actual:
SerializedClassifierInstance{id='ID-4', parentNodeID='ID-3',
...
 references=SerializedReferenceValue{metaPointer=MetaPointer{key='Folder-linked-key', version='2023.1', language='-default-key-FileSystem'}, value=[Entry{resolveInfo='null', reference='ID-10'}, Entry{resolveInfo='null', reference='ID-8'}]},
...

This seens to be correct, because the two references are in DIsk_A , and applying Disk_Property_value_changed-single-node.json does not remove them. Theese two references are missing in the Disk_Property_value_changed-partition json.

Did I introduce this issue during translation to Java or is the TS test missing the discrepancy?

In general the TS testcases check whether the retrieve is identical to the partition.json in the same folder, so this kind of discrepancy will not be found.
In this specific case the references are in the partition.json, so they have probably been changed after you copied them.
In the @lionweb/typescript repo we use the testcases from https://github.com/LionWeb-io/lionweb-integration-testing . These tests are not copied, but are checked out in a setup script. Although they are not automatically updated, it is easy to do a git clean + project setup to update them. Maybe such a setup would make it easier to keep things synchronized.

Closing because cannot reproduce it


Fixed #484 by correcting templates

Fixed #484 - worked for me locally (rebuilding docker image)
Another local test would be appriciated

Not sure what "now" is.

Confirmed that this works for me, you can test it live at https://httpbin.e.ip.saba.us/

✨🍰✨


JSONDecodeError : Expecting value: line 1 column 1 (char 0)

Error message
JSONDecodeError                           Traceback (most recent call last)
 in
----> 1 results = sponsored_products.Campaigns(credentials=my_credentials).list_campaigns()
/opt/anaconda3/lib/python3.7/site-packages/ad_api/base/helpers.py in wrapper(*args, **kwargs)
19         def wrapper(*args, **kwargs):
20             kwargs.update({"path": path, "method": method})
---> 21             return function(*args, **kwargs)
22         wrapper.doc = function.doc
23         return wrapper
/opt/anaconda3/lib/python3.7/site-packages/ad_api/api/sp/campaigns.py in list_campaigns(self, **kwargs)
334
335         """
--> 336         return self._request(kwargs.pop('path'),  params=kwargs)
337
338     @sp_endpoint('/v2/sp/campaigns/{}', method='GET')
/opt/anaconda3/lib/python3.7/site-packages/ad_api/base/client.py in _request(self, path, data, params, headers)
269                 logging.info(data)
270
--> 271         return self._check_response(res)
272
273     # @staticmethod
/opt/anaconda3/lib/python3.7/site-packages/ad_api/base/client.py in _check_response(self, res)
291             raise exception(dictionary)
292
--> 293         data = json.loads(str_content)
294
295         if type(data) is dict and data.get('code') == 'UNAUTHORIZED':
/opt/anaconda3/lib/python3.7/json/init.py in loads(s, encoding, cls, object_hook, parse_float, parse_int, parse_constant, object_pairs_hook, **kw)
346             parse_int is None and parse_float is None and
347             parse_constant is None and object_pairs_hook is None and not kw):
--> 348         return _default_decoder.decode(s)
349     if cls is None:
350         cls = JSONDecoder
/opt/anaconda3/lib/python3.7/json/decoder.py in decode(self, s, _w)
335
336         """
--> 337         obj, end = self.raw_decode(s, idx=_w(s, 0).end())
338         end = _w(s, end).end()
339         if end != len(s):
/opt/anaconda3/lib/python3.7/json/decoder.py in raw_decode(self, s, idx)
353             obj, end = self.scan_once(s, idx)
354         except StopIteration as err:
--> 355             raise JSONDecodeError("Expecting value", s, err.value) from None
356         return obj, end
JSONDecodeError: Expecting value: line 1 column 1 (char 0)
To Reproduce
Steps to reproduce the behavior:

from ad_api.api import sponsored_products
my_credentials = dict(
refresh_token = "xxxx"
client_id = "xxxx"
client_secret = "xxxx"
profile_id = "xxxxx"
)
results = sponsored_products.Campaigns(credentials=my_credentials).list_campaigns()

Hi Daniel,
I didn't use the commas in the issue but I used them in the code, my bad.
Anyway, I tried with the new credentials dict but I got the same error, here's a snippet of my code, am I doing anything wrong?
Is it possible I have this error because there are no active campaigns for these credentials?
`from ad_api.api import sponsored_products
from ad_api.base import AdvertisingApiException
my_credentials = dict(
refresh_token=REFRESH_TOKEN,
client_id=CLIENT_ID,
client_secret=CLIENT_SECRET,
profile_id=PROFILE_ID
)
try:
results = sponsored_products.Campaigns(credentials=my_credentials).list_campaigns()
except AdvertisingApiException as error:
print(error)`

Hi,
I am experiencing the same issue and it seems not to be related with the credentials.
How did you manage to solve it?

Iam experiencing the same issue


Decompiled jar is corrupted

Issue
Decompiled jar fails to extract using zip tools. Same error using 7zip, WinRar, python zipfile module, jar xf.
Not a permission error. Tried with elevated terminal or running as administrator
The output produced by the command output.txt.
The error from 7zip which I get when trying to extract image
The jar's decompiled using fernflower extract perfectly fine, therefore I believe it's a quiltflower issue on my machine.
Running a windows 10 machine
JDK-16
How to reproduce

run command java -jar quiltflower.jar -dgs=true src.jar dest
Extract it with any of the zip tools mentioned

Expected

Jar is successfully extracted.

This is probably fixed by 65d2a843e257e5adeda4dea231b2c2a7b56ed59a , can you test with a dev build?

Tested build from develop/1.6.0 branch and it seems to have fixed the issue. Yesterday had the issue with compiling the development build because gradle build task was conflicting with JRE 8 installation. Had to remove JRE 8 for it to work.
Thanks sky


fix PixelTestWithICCProfileLossy

caused by #2503, this fixes a problem when cross-compiling

yes it should, let me push a fix for that


Cannot connect to External Data Source WWI Blob Storage

When I try to create the external tables I get the following error, can anyone help?
EXTERNAL TABLE access failed due to internal error: 'Error occurred while accessing HDFS: Java exception raised on call to HdfsBridge_Connect. Java exception message:
Parameters provided to connect to the Azure storage account are not valid.'

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

ID: d26e70c3-2769-5b47-2bda-8a4a7f0239d0
Version Independent ID: 558a36bd-e151-1e4c-4427-fc52eb2d91e3
Content: Tutorial: Load data to Azure SQL Data Warehouse
Content Source: articles/sql-data-warehouse/load-data-wideworldimportersdw.md
Service: sql-data-warehouse
GitHub Login: @Ckarst
Microsoft Alias: cakarst

@griff182uk Thanks for the feedback. We are actively investigating and will get back to you soon.

@griff182uk Are you following this tutorial verbatim or are you adding additional steps that are specific to your solution? This tutorial has no issue when every step is followed per the documentation. So, I wish to identify which steps deviated from the documentation, or if there is some steps that were missed, etc.
Regards,
Mike

Thanks Mike, followed the tutorial (I hope) and the only thing I can see different is the region (I have to use West Europe for my MSDN sub), and just slightly different server name for uniqueness.
I've attached screenshot of code ran and architecture in case something obvious jumps out at you. Cheers

@griff182uk I am able to reproduce this. I worked that last time I completed this tutorial. I have assigned the issue to the content author to evaluate and update as appropriate.

Msg 105019, Level 16, State 1, Line 1
EXTERNAL TABLE access failed due to internal error: 'Error occurred while accessing HDFS: Java exception raised on call to HdfsBridge_Connect. Java exception message:
Parameters provided to connect to the Azure storage account are not valid.'

Same.

hello, this appears to still be an issue - any update? thanks

@griff182uk I am unable to reproduce this issue if I follow the documentation exactly as detailed.
We will now proceed to close this thread. If there are further questions regarding this matter, please comment and we will gladly continue the discussion. Please make sure you are running all loading T-SQL as LoaderRC60.

Hi,
was following the tutorial and literally just hit the same error.

Couldn't get the powershell script to run at the start due to restricions in provisioning resources in our environment, so manually created them, manually created the storage accounts and blobs/containers. Used storage explorer to copy from the public storage account to my storage account. And manually imported the database to the the SQL DB using the bacpac file.
Everything has been going smoothly until the step for "create external table" where I get the same error others do.

@Hassan0101 There are two possible issues:

The subscription/DWH environment you are using doesn't have the necessary permissions to execute the tutorial verbatim.

or

In the following section, a step was missed or not executed correctly: Create a user for loading data, steps 2 and 5 need to be executed as the SQL Admin:

CREATE LOGIN LoaderRC60 WITH PASSWORD = 'a123STRONGpassword!';
CREATE USER LoaderRC60 FOR LOGIN LoaderRC60;

GRANT CONTROL ON DATABASE::[SampleDW] to LoaderRC60;
EXEC sp_addrolemember 'staticrc60', 'LoaderRC60';

Note: The CREATE USER LoaderRC60 FOR LOGIN LoaderRC60; T-SQL is duplicated between steps 2 and 5, so only the GRANT CONTROL statement needs to be executed.
At this point, you need to Connect to the server as the loading user and run the remaining steps, including the CREATE MASTER KEY; T-SQL statement as the loading user.
So, in all fairness, the tutorial could use an update to remove to duplicate T-SQL for CREATE USER @mlee3gsd
The only other possibility is that the CREATE MASTER KEY was run previously in another tutorial and there is a conflict here (thinking out load).

@Mike-Ubezzi-MSFT
I'm followiing the version of the tutorial on microsoft learn
I executed:
Create Login TestUser1 with password = 'abcd123Password!'  on master db
Create user TestUser2 from login TestUser1 on the SQL DW and SQL db (retaildb)
I added db owner, db reader, and staticrc60 using EXEC sp_addrolemember command to testuser2 which worked successfully. tutorial doesn't mention grant control in microsoft learn version.
I was able to use ADF to copy data from blob and sql db to Sql dw using testuser1.
Any advice? Only thing I can't do is provision resources without asking our Azure Admins to do it for me.

Thank you very much for the additional feedback. I see the issue. This GitHub issues repo is specifically intended to support the related document. If you follow this document verbatim (except for the duplicate CREATE USER T-SQL as mentioned) there are no issues with the tutorial.
I see Microsoft Learn - Lab 7 Load Data using Polybase is also an Azure Databricks focused lab. There is a Report an Issue link at the bottom of the lab that you should be using to report issues specific to the labs themselves. If you do not report issues, then they don't get fixed.

@Hassan0101 In addition to the above information, you could request that the labs detail the minimum necessary permissions for those using a shared/restricted environment.


[OS X] Unable to type letter 'Ż' in editor

On keyboard mapping for Polish language on OS X, the letter Ż is typed with alt-shift-z. Ghost's editor captures this combination for some reason.
The bug occurs in text fields in Post Settings. It does not occur in the tab editor.
Behavior tried in Firefox, Chrome and Safari.

@tomob if you are using ghost 0.7.2, this is a known regression and a fix will be included in the next release.

Actually I use 0.7.1. If the fix is on the way, I'll close the issue, since a simple workaround is possible (copy-paste).
Thank you.


Add configurable randomizer

[x] Random list n times
[ ] Save history of every random step
[ ] Add Slack button to view result of every step

Also it closes #19


support non routable nodes

Motivation
Closes #1697
Changes
uses src port tuple to return udp messages, uses connect syscall for udp abstraction
Test Plan
added unit tests
TODO

[x] Explain motivation or link existing issue(s)
[x] Test changes and document test plan
[x] Update documentation as needed

bors try

I addressed all your comments.

bors merge

bors merge


definition not showing up

I don't know if I'm doing something wrong but I don't see the definition showing up at all as it does in the screenshot of the ReadMe.
All I see is the word.

Mine is working fine. Don't know if it makes a difference but I'm on OS X 10.9.5.

please check the Show all subtitles in LaunchBar preferences
LaunchBar Preferences -> Appearance -> ✓ Show all subtitles
P.S. As of LaunchBar 6.1.2 this issue could be avoided. I will update the action to prevent this issue, even If you don't want to show all subtitles

Thanks for the great LaunchBar action, Nima, hope we can see this small addition soon.

please check the Show all subtitles in LaunchBar preferences

This worked for me, and actually I wasn't even aware of this preference. Could be useful!


[Automation] Generate Fluent Lite from Swagger specification/containerservice/resource-manager/Microsoft.ContainerService/fleet/readme.md#package-2024-04

[Automation] Generate Fluent Lite from Swagger specification/containerservice/resource-manager/Microsoft.ContainerService/fleet/readme.md#package-2024-04

API change check
APIView has identified API level changes in this PR and created following API reviews.
com.azure.resourcemanager:azure-resourcemanager-containerservicefleet


Clarification of square icons sizes

Updating the square icons after #1755.
Since I was checking many of the original icons, some icons had to be updated because the original was different.

Before merging this request, #1765 needs to be merged. If the requirements change, I'll make changes here as well.
Almost all the square icons have been redone. I'll change a couple more later as part of Quality update 12.


labels in Acquisition geometry

from ccpi.framework import ImageGeometry, ImageData, AcquisitionGeometry, AcquisitionData
import numpy as np

angles = np.linspace(-np.pi, np.pi, 10)
channel = 5
horiz  = 3
ag2D = AcquisitionGeometry('cone',
                         '2D',
                         angles,
                         pixel_num_h=horiz,
                         pixel_size_h=0.25,
                         dist_source_center=233.0,
                         dist_center_detector=245.0,
                         channels=channel, dimension_labels = ['channel', 'horizontal', 'angle'])

X = np.random.randint(100, size=(channel, len(angles), horiz))

data1 = AcquisitionData(X, ag2D)
data2 = AcquisitionData(X, ag2D, dimension_labels = ['channel','horizontal','angle'])

print(data1.dimension_labels)
print(data2.dimension_labels)

since we define dimension labels in ag2D, then we should not define again in the AcquisitionData

in #376 the initialisation of AcquisitionData as in the code is deprecated.
Users shouldn't use it.


Figure Out Configurations On Runner Dockerfile

I copied the arg values from the builder container into the runner container.
https://github.com/Gamma169/go-service-template/blob/09f3449fad2292d4ac0e1cf4038d8fffb1ada9de/Dockerfile#L34
This generally isn't good since we would need to update configurations in two spots.  Should see if we can use the same configurations for both.
Also, the CMD is hardcoded.   Need to figure out a way to get the app name configurable into the CMD.

This is sort of a "feature"; it's expected behavior of the multi-stage Docker build. This is from the documentation:
"An ARG instruction goes out of scope at the end of the build stage where it was defined. To use an arg in multiple stages, each stage must include the ARG instruction." (See here.)

Also, the CMD is hardcoded. Need to figure out a way to get the app name configurable into the CMD.

https://github.com/Gamma169/go-service-template/blob/75d3e395f3defd78a7758ad8b54188abe17c2c57/Dockerfile#L34
^ Adding this line should fix this. The reason is b/c ARGs are only available at build time (i.e., ARGs don't persist after the image is built). When you're executing a CMD that's at run time so the ARG isn't available anymore. Making the ARG an environment variable should do the trick, since ENV does persist into runtime.


Compatibility fixes and tweaks

This commit will fix https://github.com/rainlab/builder-plugin/issues/295 and https://github.com/rainlab/builder-plugin/issues/328.
Commit f1f0f4523d5d084779a18d481bfe3a615e4ff1f9 brought in changes that allowed null default values to be specified as null (without quotes) to be marked as nulled by default, with a quoted null default value to be a string value of "null". It was reported by several people on Slack as well as #295 that this did not work. This rolls back the changes for null, while keeping the changes for the boolean default values. Doctrine, by default, will mark any column that is nullable as null by default, unless a specific default value is given.
This also brings in the https://github.com/marktopper/doctrine-dbal-timestamp-type library to provide support for timestamp columns.
Finally, some sensible widths were put in to columns of the database table datatable widget, to make it a bit more uniform.
As this is technically a breaking change, it will be released as a "critical" update.

Leaving this open to review prior change

Not when you're throwing shade on the commit messages, m8.


No lyrics found for certain of songs

I tried a couple new songs recently, and this thing showed, similliar to old songs aswell no lyrics found. i think the music library not updated? please help this issue

It works for me:

well, have you tried it on discord? idk i'm not good at all at programing or coding

Can you point me to how to test this out on discord? How do I add this bot? Are you sure the bot uses this particular npm package to fetch lyrics?

The problem is because lyrics-finder will find the lyrics from song title in Google and if the title contains text more that artist and song title, it will return nothing.
so to solve that in discord, you can write the code to delete texts in song args that contain official, music, video or s.th sort of that.
Example: song: Justin Bieber & benny blanco - Lonely (Official Music Video), so you should remove (Official Music Video) out
sorry for my grammar


feat: separate conn and io timeout (ADN-372)

Now that we are using TLS, connection establishment can take longer. It might make sense to separate out the timeout config for io vs connection establishment.

It might, but that's configurable to folks now.


chore(deps): update dependency appsflyerframework to v6.14.0

This PR contains the following updates:

Package
Update
Change

AppsFlyerFramework
minor
6.13.2 -> 6.14.0

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Renovate Ignore Notification
Because you closed this PR without merging, Renovate will ignore this update (6.14.0). You will get a PR once a newer version is released. To ignore this dependency forever, add it to the ignoreDeps array of your Renovate config.
If you accidentally closed this PR, or if you changed your mind: rename this PR to get a fresh replacement PR.


cilium: encryption+chaining, use FIB lookup to rewrite src/dst mac addresses when available

When bpf_fib_lookup() helper is available use the call to lookup correct src/dst MAC addresses and write them into packet.

This change is

test-me-please

Coverage decreased (-0.02%) to 41.973% when pulling 781d565725f7c8c1374015fd584a91d121e58adb on fib into a78df6fcfde4be33433760142e57c3302316653c on master.

test-me-please

Forgot to commit updated daemon.sha after changing probes which caused build to fail. retesting..

test-me-please


Save and continue with defaults, remove disabled attr on button, show errors instead

What I Did
Made it so that a person can click through the config screen with just the default values.
How I Did it
Removed unused code from ConfigOnly.jsx and and set the initial form item data in state in the same component. Passed getData method down to ConfigRender.jsx instead of defining it there. Save using both buttons but only continue if nextStep = true.
How to verify it
Run ship init with an app that has a ship.yaml file, probably one from our test repo.
Description for the Changelog
Save config with default values instead of needing to edit inputs
Picture of a Boat (not required but encouraged)
⛵

Would prefer @GraysonNull to look at this since I'm commenting out some code. Some of the config render stuff was a little confusing, want to make sure I didn't mess anything up.


Autopublish plugin

We should re-implement the autopublisher. Should listen for messages in #announcements. If they match a youtube video regex, it should automatically publish them.

What does it publish it to? Im very confused lol.

To whoever is following announcements.


TestScheduler WithAsync swallows all exceptions making all tests pass

Bug
In Unit testing, when doing TestScheduler().WithAsync any expection doesn't fail the test.
I did find this stackoverflow question, which is super close to the same senario but not fully
Which library version?
6.0.0
What are the platform(s), environment(s) and related component version(s)?
.NET 6
What is the use case or problem?
All tests passing, even though they are failing by throwing exceptions.
What is the expected outcome?
Tests that throw exceptions, fail
What is the actual outcome?
Tests that throw exceptions, pass
Do you have a code snippet or project that reproduces the problem?
[Test]
public void Test() => new TestScheduler().WithAsync(async scheduler =>
{
    await Task.Delay(1);
    throw new Exception("This should fail the test");
});

What's this WithAsync method?
I've searched the whole Rx.NET source repo, and we do not define any such method.
Right now if you search: https://github.com/search?q=repo%3Adotnet%2Freactive WithAsync&type=code the only result for WithAsync is this issue that you opened.
I'm going to hazard a guess that you're using ReactiveUI (which uses Rx, but is a separate thing from Rx).
If that's correct, there's an obvious problem with your test: ReactiveUI's WithAsync method returns a Task, but your test returns void. That means that you've given the unit test framework no way of knowing when your test actually finished.
Since WithAsync more or less immediately returns a Task, which you then fail to return from your test, so the test runner thinks your test is done.
If we simply change the return type of your test from void to Task:
[Test]
public Task Test() => new TestScheduler().WithAsync(async scheduler =>
{
    await Task.Delay(1);
    throw new Exception("This should fail the test");
});

then the test fails in exactly the way you want.
This works because by returning the Task, you've made it possible for the test runner to wait for your test to finish, and to discover whether it threw any exceptions. By returning void you were making it impossible for the test runner to find out what the outcome of your code was.

doh, aplogies. Looking back on this and the code, yes it is from the namespace ReactiveUI.Testing.
Thank you for explaining the solution as well in great detail! That makes sense now


"View in Marketplace" makes the hover heavy

Do we really need the "View in Marketplace" part in the hover. My concern is that it makes the hover look heavy.
My suggestion is to cut that. And instead, clicking on the second row in the hover would take the user to the extension view.
If you really want to keep the "View in Marketplace" experience, I suggest:

Use the same background as rest of hover
Call it "View Extension" instead, since you do not actually take the user to the Marketplace

I'm fine getting rid of it. It was a quick design proposal and linking to the extension page seemed like a useful thing to include. Having the extension name be a link could work as an alternative, yeah.

I'm going to leave it for now, it's there, the status bar actions are used for other hovers too, and I haven't heard any other complaints


inheritance - object oriented machines

I've noticed problems while trying to develop machines using automat that inherit from base machine classes.

@output's defined in base methods misinterpret the fact that the super() is called in the following way:
File ".../automat/_methodical.py", line 274, in __get__method=self.method.__name__ AttributeError: GiveFeedback._initialize is a state-machine output method; to produce this output, call an input method instead.
It is not obvious how to define transitions using states and outputs defined in base classes. It appears that states are attached to the base class before instantiation as methodicStates but how do you reference them? Outputs seem to be instantiated upon object creation.

Hi @bitfinity! Thanks for using automat.
In my personal opinion, inheritance is a bad idea generally, and you should favor composition. This is particularly true in Automat, where composition allows for a combinatorially smaller state-space than derived machine structures.
That said, my personal feelings are somewhat secondary to how it might fit into the design. Either way there is an issue here: the documentation should explicitly call out that inheritance is not supported and the error message should make that clear (not effectively failing by accident, which is what it does now), OR if there’s a compelling use case we should figure out how this should work.
So: what is your use-case, specifically, for combining inheritance and automat?
Thanks again!

@glyph Hi Glyph, I think the rationale was there were a lot of actions that were in common between my state machines. I ended up using another library where the actions could be sent directly to another library, and hence, shared. I would have a "runner" or "responder" passed into the machine, and I would make the action be machine.responder.action. I think I remember that yours required a decorator within the machine, which meant that they had to be in the same class as the machine. So I would end up having to write a bunch of methods that just called the base level actions.


NixOS 24.11 Plasma6 ISO is broken

Describe the bug
ISO not loading on VM and barely loading on bare metal machine.
The installation failed 7 times yesterday and at 8th time it succeed but half of the system is broken and things like mounts barely work too, crashing the system to emergency mode.
Steps To Reproduce
Steps to reproduce the behavior:

Download the latest ISO
trying to install it - it'll stuck on black screen for 5 minutes
Installation will crash

Expected behavior
Clean install stable OS like the old one was version 23.11/23.05
Screenshots
Example on VM.

stuck like this for more than 5 minutes.
Additional context
OS unstable currently I somehow succeed to install it on my main system but I'll have to format again since nothing works.
The name of the ISO says plasma6 while the installation itself installs plasma5.
Metadata

Notify maintainers

Note for maintainers: Please tag this issue in your PR.

Add a :+1: reaction to issues you find important.

@InSelfControll  Please be more specific while you write bug report. Link the ISO file you got and write steps to reproduce properly. Without any logs, we are not able to help you or reproduce the same.
Regardless, you should be fine working with any ISO file. You can alternatively make yourself an ISO file if that fits your need as some hardware like WiFi cards might need some extra kernel module such as wl. But it should not cause whatever is happening in your case. Please make sure the file you downloaded SHA matches as well.

I cannot share logs from the installation.
The steps I share are the steps I did and encountered the issue.
I had to downgrade back to 24.05 because almost all the apps I use barely worked even Nvidia drivers and basic mounts of brtfs / ext4 drives.
I tried the last two isos of version 24.11 and both of them stuck on black screen like in the pic I shared.
On my main computer I successfully loaded the iso after 7 tries that it was stuck on black screen and after that I successfully got to the install screen and it also failed 7 times on the 8th time after I disabled ipv6 and replaced my DNS servers and reformatted the drive 7 times it decided to work but still ended with broken libs that were actually missing when the system booted so I couldn't stay on it and decided to downgrade.
It was a clean install all the times.
On vm the latest iso was stuck on black screen for over 30 minutes, when I tried with version 24.05 it booted in light speed to the installation and then into the OS.

I verified that the default graphical ISO (which is based on GNOME) works (in virtual-manager), but I see now that you link to the plasma6 version (which I haven't tested).

Just tested the plasma 24.11 iso you linked in virt-manager QEMU. Did not boot first time for me.
Got the boot menu, initial stage 1 outputs then stuck on black screen indefinitely.
Stopped the VM then started it again with no changes made and it booted completely fine. And every time after that too.
Note i did not give the VM any persistent storage, just the read only ISO.
This was the last output of the failed boot, however the exact same messages also appear on the successful boot so likely not related.

Yea but it doesn't suppose to be like that...
I installed it on my PC and it was stuck like it for 10 minutes each time I tried.
I tried 7 times and at the 8th it booted and then the installation itself failed 7 times and succeed only at the 8th time with a crash at the end so clearly it doesn't install all the necessary OS components correctly.

https://releases.nixos.org/nixos/24.11/nixos-24.11.710486.4dc2fc4e62db/nixos-plasma6-24.11.710486.4dc2fc4e62db-x86_64-linux.iso

I also get black screen with that plasma6 ISO in virt-manager (with default settings) after that stage 1 output posted above. Booted the VM twice.

CC @K900, plasma maintainer

I can't reproduce here.

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/brand-new-to-nixos-wont-boot-to-desktop-environment/56292/6

Do you have configured some video memory for the VM?

Do you have configured some video memory for the VM?

I'm talking about my main PC not the VM.
On the VM when I tested it I haven't configured anything special, base OS.

Can you get any sort of log output from your PC?

Can you get any sort of log output from your PC?

Already downgraded to version 24.05 can't send a log.

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/runing-a-qemu-nix-guest-in-a-fedora-host/57644/2

This is my first time to try NixOS Plasma 6 from the Download (Plasma Desktop, 64-bit Intel/AMD) button 3 days ago, I'm looking around to fix my graphic driver, I'm not sure until I found this.
Downloaded this name: nixos-plasma6-24.11.711349.394571358ce8-x86_64-linux.iso
Flash in USB thumbdrive, boot it on an old PC, select "Installer", receive error message like above and then just blank screen with "_" for long time.
CTRL+ALT+F1 to view logs. This are some errors I saw

Firmware Bug which might not related
nouveau error with some memory addresses
spice vdagentd with no data available

Is this Nvidia? Which GPU model?

Is this Nvidia? Which GPU model?

I'm using an Nvidia GPU.

Which exact model?

Which exact model?

Rtx 2070 super but it was also in VM without GPU.

Those are likely separate issues. Does booting in nomodeset mode work?

Those are likely separate issues. Does booting in nomodeset mode work?

No, the Nvidia driver was also broken on 24.11 it couldn't detect the GPU correctly so always back to noveau.
I had to downgrade the OS to make it work, only after downgrading and disabling sddm my PC became more stable.

Mine one nvidia gtx 1060.
Booting from nomodeset also the same, it's blank if I switch to ctrl+alt+f2. It always hang for a moment when switching to there.
Using nomodeset, if I restart the desktop-manager svc, I can see login screen (with the word wayland at the bottom) but if I enter, it become black again with "x" at the center.
Reproduce for a couple of times, it is consistent.

Hi together,
i have the same problem and i got it fixed.
Yesterday i tried to install NixOS with Plasma on a HP ProDesk 400 with an AMD Ryzen 3400G, but the ISO bootet only to the Blank screen with the blinking underscore.
So i decided to boot the Gnome Image which worked fine.
After the installation of NixOS with Plasma 6 i had the same again.
When Plasma should start i got redirectet to the shell Login screen (STRG + ALT + F1)
Switching to the graphical session (STRG + ALT + F2) showed only the blinking underscore.
after a few tries i found the solution.
i made the following change in the nixos configuration
old
services.displayManager.sddm.enable = true;
new
services.displayManager.sddm.wayland.enable = true;
tldr:
setting up sddm for wayland fixed the problem.
services.displayManager.sddm.wayland.enable = true;

tldr: setting up sddm for wayland fixed the problem.

services.displayManager.sddm.wayland.enable = true;

Thanks for discovering and reporting this! It fixes, or at least works around, #360678 for me.
For reference: I'm running with an nvidia RTX 2060 & Intel i9-9900K and this config snippet:
  services.xserver = {
    enable = true;
    videoDrivers = [ "nvidia" ];

    xkb = {
      layout = "us";
      model = "pc104";
      options = "eurosign:5,lv3:ralt_switch";
    };
  };

  services.desktopManager.plasma6.enable = true;
  services.displayManager.sddm.enable = true;
  services.displayManager.sddm.wayland.enable = true; # https://github.com/NixOS/nixpkgs/issues/363797#issuecomment-2558384445

  hardware.graphics.enable = true;

  hardware.nvidia = {
    open = true;
    modesetting.enable = true;
    nvidiaSettings = true;
    package = config.boot.kernelPackages.nvidiaPackages.stable;
  };

SDDM on Wayland is still experimental, but we may want to enable it by default if it works better for people...

I want to confirm that I had the same problem (amd gpu 6700XT) and booting from the GNOME iso and then changing to sddm in the config did fix the problem and allowed me to use KDE.


register one device from cli

i saw there is the register_devices and it takes a file or a dict. is there an easy way to add one device using the cli? like fastlane run register_device name:"bla's iphone" udid:"123"?

Sorry, currently no, however I could see that being very useful. If you feel like it, feel free to update register_devices.rb to access those parameters, I'm sure many people would love that feature 👍

alright. should i make it into a different action? register_device? otherwise we might have weird cases if you have a name and udid plus a devices file etc.

Yeah, that sounds like a great idea. And the register_devices action could call the register_device action 👍
Let me know if you need any help. Thanks for contributing to fastlane ❤

i made a pr but i didnt make register_devices call register_device, fistly because i dont know, and secondly because then it has to check all the time for credentials, and i dont know if that will be too slow.. what do you think?

The PR was merged so I'm going to close out this issue 🎉


App not showing up in permissions

The permissions popups seem to be working okay now, but my app seems to no longer be a choice in setting to give the permissions to. Is anyone else experiencing this?

I restarted my computer and it seems to work now. Hopefully it was just a fluke.


[CP] iOS screen flickering speculative fix and logging

issue_link
https://github.com/flutter/flutter/issues/100522
Commit Hash
n/a
Target
stable
pr_link
n/a
Impacted Users
All iOS apps
Impact Description
This CP request is a speculative fix (and additional logging) for the iOS screen flickering issue linked above. One possibility is that the root cause of that issue is a race in Metal command buffer submission when an app is going into the background. The two commits below have been on beta and master channel for a good while. Since the next stable is still a way off, we'd like to alleviate some pain in the meantime in case these changes help fix or diagnose the issue.
https://skia-review.googlesource.com/c/skia/+/575667 <- We suspect that this may fix a race in command buffer submission when going into the background.
https://skia-review.googlesource.com/c/skia/+/566883 <- If that change does not fix the flickering issue, then this change will give more verbose logs when command buffers fail.
The 3.3 release used Skia at https://github.com/google/skia/commit/e2ba2f5b20e0d188dbfb5d3df64aa9787ebb3f37. We'll need to CP the above patches onto a new Skia branch based on that hash.
Workaround
There is no workaround
Risk
medium
Test Coverage
yes
Validation Steps
We don't have a local reproduction for this issue, so the fix is speculative.

Thank you @christopherfujino for the background on skia CPs. The release team doesn't have a playbook on this, so I'm going to assign to me to figure out and create a playbook entry.
I'm currently blocked from creating new skia branches. I've gone ahead and filed https://bugs.chromium.org/p/skia/issues/detail?id=13865 to request access for the release team.

https://skia.googlesource.com/skia/+/refs/heads/flutter/flutter-3.2-candidate.5 is created

I sent the CP CLs:
https://skia-review.googlesource.com/c/skia/+/597196
https://skia-review.googlesource.com/c/skia/+/597197

I don't have +2 in the Skia repo, so we'll need a review from folks on the Skia team on those CLs.

Marking approved on behalf of @chinmaygarde (based on the lgtms in the linked CLs)

This was released in 3.3.7

My app has a lot of animations, so it is easy to switch back and forth to the background many times. The current test 3.3.7 still hasn't fixed it.
It work well in debug mode, but it will flickering work in release mode.
@CaseyHillers @zanderso
2022-11-07 14:36:36.294111+0800 Runner[4777:2920461] Metal GPU Frame Capture Enabled
2022-11-07 14:36:36.294240+0800 Runner[4777:2920461] Metal API Validation Enabled
[2022-11-07 14:36:36.439] [0] [debug] /tmp/jenkins/IRIS-SDK/base/internal/log/iris_logger.cc:35 InitLogger /var/mobile/Containers/Data/Application/127F47F5-61A9-4499-AC94-3FFAA77AF131/Library/Caches/
2022-11-07 14:36:36.481449+0800 Runner[4777:2920461] Unable to upload statistics，appkey or stats is nil
2022-11-07 14:36:36.481471+0800 Runner[4777:2920461] device token changed, reconnect
2022-11-07 14:36:36.954976+0800 Runner[4777:2920461] [RC-Flutter-IM] iOS setReconnectKickEnable, start param:1
2022-11-07 14:36:36.956640+0800 Runner[4777:2920461] RCFwLog, LogMode = 0, monitorLevel = 5, monitorType = 268434431
2022-11-07 14:36:37.073836+0800 Runner[4777:2920800] flutter: 2 has been spawned
2022-11-07 14:36:37.075708+0800 Runner[4777:2920800] flutter: initialized
2022-11-07 14:36:37.397608+0800 Runner[4777:2920461] [RC-Flutter-IM] iOS onConnectionStatusChanged,status:10
2022-11-07 14:36:37.410222+0800 Runner[4777:2920461] clientIp:
[000:000] [83203] (messagequeue.cc:523): Message took 78ms to dispatch. Posted from: Connect@/Users/jenkins/slave/workspace/iOS-SDK-Release/protocol-cpp/rcps/rcps_client_impl.cpp:546
2022-11-07 14:36:37.488674+0800 Runner[4777:2920793] [RC-Flutter-IM] iOS connect, dbOpened，code: 0

2022-11-07 14:36:37.500244+0800 Runner[4777:2920800] [VERBOSE-2:dart_vm_initializer.cc(41)] Unhandled Exception: MissingPluginException(No implementation found for method getRCNotification on channel com.push.msg.method)
#0      MethodChannel._invokeMethod (package:flutter/src/services/platform_channel.dart:294)
<asynchronous suspension>
#1      _IndexState.initNotification (package:xxxx/pages/index.dart:171)
<asynchronous suspension>
2022-11-07 14:36:37.515019+0800 Runner[4777:2920785] L-collection-S,host:<IP_ADDRESS>,port:443,cost:26,nettype:1,error:0
2022-11-07 14:36:37.515146+0800 Runner[4777:2920793] L-collection-S,host:<IP_ADDRESS>,port:8000,cost:0,nettype:1,error:-1
2022-11-07 14:36:37.515326+0800 Runner[4777:2920790] L-collection-S,host:t-tcpproxy-cn.ronghub.com,port:443,cost:0,nettype:1,error:-1
2022-11-07 14:36:37.515342+0800 Runner[4777:2920785] L-collection-S,host:t-tcpproxy-cn.ronghub.com,port:8085,cost:0,nettype:1,error:-1
2022-11-07 14:36:37.515615+0800 Runner[4777:2920793] [RC:IM-stats_cmp-S]data:{"hos":"http:\/\/<IP_ADDRESS>:443","fg":1,"spr":0,"dt":0,"pr":0,"bid":"xxxxxx.music","idx":0,"net":"wifi","rct":0,"dip":"","cpd":"62BD596F2C9857C993DBCC2F2DDBE47D","ddu":0,"dh":["<IP_ADDRESS>"],"ptc":2,"cod":0,"cr":[],"dur":26}
[000:064] [83203] (rcps_pull_message.cpp:55): PullMessageRequest: P-reason-C;;;recv;;;1667561924461, send:<PHONE_NUMBER>149, broadcast: 0
2022-11-07 14:36:37.553327+0800 Runner[4777:2920461] [RC-Flutter-IM] iOS onConnectionStatusChanged,status:0
2022-11-07 14:36:37.553685+0800 Runner[4777:2920790] [RC-Flutter-IM] iOS connect, success
[000:173] [83203] (rcps_pull_message.cpp:98): DealWithResponse: P-reason-C;;;sync;;;1667750400000,broadcast: 0,finished: 1,size: 0
2022-11-07 14:36:38.306210+0800 Runner[4777:2920790] [NSURLSessionTaskMetrics] 0.411221
2022-11-07 14:36:38.307846+0800 Runner[4777:2920795] [RC:IM-stats_navi-S]data:{"hos":"https:\/\/sirb.cn.rongnav.com\/navi.json","fg":1,"spr":0,"dt":1,"pr":0,"bid":"xxxxxx.music","net":"wifi","dip":"<IP_ADDRESS>","ddu":0,"dh":["<IP_ADDRESS>"],"ptc":1,"cod":200,"cr":[],"dur":411}

[180:066] [83203] (rcps_pull_message.cpp:55): PullMessageRequest: P-reason-C;;;recv;;;1667561924461, send:<PHONE_NUMBER>149, broadcast: 0
[180:535] [83203] (rcps_pull_message.cpp:98): DealWithResponse: P-reason-C;;;sync;;;1667750400000,broadcast: 0,finished: 1,size: 0


adsense-invalid-click-protector for Auto ADS

its juts a Request an Feature.  invalid click protector for Adsense Auto Ads.  please anyone can make for Adsense auto ads invalid click protector.

You have been given a suggestion here:
https://forums.classicpress.net/t/ad-invalid-click-protector-aicp/4662

So they have to just add in the function to check what CMS is the plugin installed on and provide in the plugin the code to work with CP and the code for blocks to be used when installed on WP. That way they can also make sure that the plugin is up-to-date security-wise.

If I'm not mistaken there aren't any Wordpress features >4.9 which are used. Correct me if I'm wrong. So just fixing Requires at least:  in readme.txt should solve the issue (make it 4.9).  Thanks!


OIDC-14 Add retry capability

OIDC-14 Provide the retry-decode function to re-enqueue the interceptor. This is useful if you've refreshed a cache or some such and would like to re-attempt the unsigning of the token.

This functionality is superseded by https://github.com/yetanalytics/pedestal-oidc/pull/10


[stable/kong] Improving docs to help when we set admin tls to false

Improves a bit the documentation. When we set tls to false and change the ports, the readiness and liveness probes fail because it tries to connect with HTTPS to a HTTP port.
I was struggling with this when trying the new version and I believe it will help.

/assign @unguiculus

cc @shashiranjan84

It seems that in the meantime a new version has appeared #4290, can you increase the version again?

/ok-to-test

/lgtm


chore(CPlusPlus): add search sorted rotated

Searching element in the sorted-rotated array
What kind of change does this PR introduce? (check at least one)

[x] New algorithm

Nice work


Ability to reply in the same thread in email component

Is your feature request related to a problem? Please describe.
Currently there is no way to reply to an email thread via the mailbox component.
Describe the solution you'd like
The "Send a Message" API supports a reply_to and reply_to_message_id fields which can be used along with the compose component, to provide this reply to feature in the mailbox component.

Hi @himankbhalla, thanks for reaching out.  Let's discuss this on our call tomorrow.

Hi, @himankbhalla
Quick update. There's a canary release that includes the reply functionality.
https://unpkg.com/<EMAIL_ADDRESS>https://unpkg.com/<EMAIL_ADDRESS>A demo can be seen here. https://components.nylas.io/components/mailbox/src/index.html


1000 domains limit

contradicting info here https://docs.microsoft.com/en-us/sharepoint/restricted-domains-sharing
is it 1000 or 3000?

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

ID: ab9f7bfd-4962-7f83-b09c-6ee338296ec3
Version Independent ID: d0fc2b43-93e7-a8c0-f541-557a65cc9256
Content: Manage sharing settings - SharePoint Online
Content Source: SharePoint/SharePointOnline/turn-external-sharing-on-or-off.md
Service: sharepoint-online
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

@junterpfe  Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

@junterpfe Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


api: add wrenCopySlot

Simple utility function which copies a value from one slot to another, as originally proposed in #836

Instead add wrenCopySlots and make wrenCopySlot a special case/macro...

I thought about that, but it opens up memcpy esque cans of worms (what do we do if they're overlapping).
Nothing insurmountable but I've never actually needed to move multiple

Well this can be as this for now. The main usage for copying in block, is
when one needs to realign the stack for a reentrant call, or when
performing tail call optimisation (which are not supported yet).. That
said, implementation should always be safest, so it should follow memmove
semantic.


app manage url

I think the original reason I chose that string was it gives more exposure to the platform and shows the user first the homepage which they can learn to navigate from (+it's a shorter string, fits into a line easier).

Yeah, that is true.  What about a line:
Manage all Cloud Run services:
https://blah
Manage this service:
https://asdf

Too much?

I think the patch is fine as is, lets try this.


[Format] Add Opaque canonical extension type

Describe the enhancement requested
As proposed on the mailing list: https://lists.apache.org/thread/4pykofrzvkl7dwsnzys8rwnq2owfnt43

[ ] #43454
[ ] #43455
[ ] #43456

Component(s)
Format

Issue resolved by pull request 43457
https://github.com/apache/arrow/pull/43457


Implement tagging

Setup tagging for articles and other types of content. The article api controller needs to save and/or create the tags when it successfully saves.

Resolved.


Release 4.3.0

We’re depending on the new forceWatch feature in our build, and we’re having some headaches because of the need to depend on the master version of this module, rather than the latest release.
Could we get a 4.3.0 release pushed to NPM so we can switch back to an official release version?

I'm not sure what headaches that will give you, but yeah, I've released it now.

We’ve been tripping over some apparent bugs in Yarn to do with Git dependencies. :P
Thanks so much for the release!


Error Handling

We should use this thread to further discuss error handling.
Currently, we return an error like so:
if(auto err = some_function()) {
  // handle error
}

@asuessenbach would you be able to make a PR showing how we can use exceptions?

I have ported the filesystem files from StackErrorPtr to exceptions, by essentially just replacing them with a std::runtime_error and adjust their usage.
One could argue, though, whether, for example, trying to read a chunk of data from a non-existent file, or trying to read beyond the file size actually are runtime errors or in fact programmer's errors. (I think, we can assume the files used here are constant!)
But in either case, it should show the principal usage.

This issue is left open to track the remaining error handling comments.
We would like to use a macro to help simplify throwing errors. @gpx1000 suggestion.

Does it make sense to make this a macro and use something like:
#define ERROR_STR(...) (( FILE ":" LINE " " VA_ARGS))

We would also like to remove the usage of hard coded file paths. Preferably cutting down the file path length generated by __FILE__ as much as possible. @gary-sweet suggestion

Resolved in future framework v2.0 changes


[Snyk] Upgrade keycloak-js from 21.0.1 to 21.1.2

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade keycloak-js from 21.0.1 to 21.1.2.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 5 versions ahead of your current version.
The recommended version was released 3 months ago, on 2023-06-28.

Release notes

Package name: keycloak-js

21.1.2 - 2023-06-28No content.

21.1.1 - 2023-04-26No content.

21.1.0 - 2023-04-19No content.

21.1.0-1 - 2023-04-19

21.0.2 - 2023-03-30No content.

21.0.1 - 2023-03-01No content.

    from keycloak-js GitHub release notes

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Checkmarx One – Scan Summary & Details – ac385e1e-acd7-445c-be2d-540231d0b376
No New Or Fixed Issues Found


Command Line Support

Hello,
I have installed MoboDevelop.StyleCop <IP_ADDRESS> with Visual Studio For Mac 7.1.
I'm interested in integrating StyleCop as part of my automated build in Jenkins but it is unclear how I can run StyleCop from command line.
Any help would be appreciated.
Thank you.

Hi,
I'm sorry but this won't be possible with this MonoDevelop plugin but there is a NuGet package that might help you out. Checkout this https://www.nuget.org/packages/StyleCop.MSBuild
Hope this will help you.

Thank you for your answer.
I was able to use https://www.nuget.org/packages/StyleCop.MSBuild that integrates StyleCop in the build process and outputs an .xml file with the warnings. I've integrated that in Jenkins, only that unfortunately the Violations plugin that has StyleCop support also has a bug that doesn't really allow  navigation to the source file (since it hasn't been maintained since 2015).
Still, it does the job so thank you for your help!

If you're talking about the Jenkins plugin you might checkout this one:
https://github.com/jenkinsci/warnings-plugin
Not sure how far this one is or if it's any better, just found a reference to this in a issue where they discussed the future of the violations-plugin and someone referenced this plugin: https://github.com/jenkinsci/violations-plugin/tree/violations-lib but in Jenkins wiki it says that this one isn't maintained anymore and a link to the above warnings-plugin.
I didn't try any of these plugins so I don't know if they work better or not but at least they should support StyleCop violations according to the wikis..

It's funny that I've been using https://github.com/jenkinsci/warnings-plugin on a separate Android native project for a long time, and it never occurred to me that it might have support for StyleCop as well.
When I performed a quick Google search it didn't pop up as it would support it. I'll have look, it might just work as expected.
I'll have a look and get back to you. Thank you!

It works exactly as I expected, it's exactly what I need. Thank you for suggesting using it.

You're welcome, I'm glad that I could help and that the suggestion worked!


there is no gen_summary_ops

when i
from tensorflow.contrib.summary import summary_ops
it raises an error
---> 29 from tensorflow.contrib.summary import gen_summary_ops
30 from tensorflow.core.framework import graph_pb2
31 from tensorflow.python.eager import context
ImportError: cannot import name 'gen_summary_ops'

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow installed from
TensorFlow version
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce

Thank you for your reply.
In fact , I meet this problem when I tried to run the sample of multimodel in the tensor2tensor，the official sample.And it told me that there is no module tensorflow.contrib.eager. So I checked my tensorflow lib and found there was no eager module in the directory contrib. Then I downloaded the whole directory contrib from /tensorflow/tensorflow/contrib in the github and replaced the local one.Then the problem appered.
My tensorflow version is tensorflow-gpu 1.4.1. And I find that it is the latest version on the pypi .And all the other relevant libs work well.Acturally, it works well except running the scripts from tensor2tensor.
I guessed that maybe your codes in the contrib are not up-to-date

@wqh17101 The "gen" part of gen_summary_ops is there because the code for these operations is generated during the build process. If you only take the raw code from github and don't build, you will end up with no generated classes.

So,how can I get the module correctly?

I have no experience with the GPU version of the Python libraries - so don't close this issue as it appears from what you've reported that there is some problem with the 1.4.1 GPU release.
To address your situation now, perhaps you could build your own Python wheel: https://www.tensorflow.org/install/install_sources

Nagging Awaiting Response: It has been 14 days with no activityand the awaiting response label was assigned. Is this still an issue?

Nagging Awaiting Response: It has been 14 days with no activityand the awaiting response label was assigned. Is this still an issue?

Nagging Awaiting Response: It has been 14 days with no activityand the awaiting response label was assigned. Is this still an issue?

It has been 14 days with no activity and the awaiting response label was assigned. Is this still an issue?

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


Jana/responsive divider

disappearing border and appearing horizontal divider. need to fix style changes for horizontal divider because dashed line looks a little weird

fixed a formatting thing w/hardware village description text area, so now the width should stay consistent w/other event description text boxes


Some updates

Regarding the setup instructions for StandardFirmataEthernet, the user now needs to uncomment a couple of includes for their particular ethernet hardware. See this section: https://github.com/firmata/arduino/blob/ethernet/examples/StandardFirmataEthernet/StandardFirmataEthernet.ino#L67-L106
Also, the second part of step 11 should read:
#define local_ip IPAddress(10, 0, 0, 15)

Another thing to note is that if the user uploads ConfigurableFirmata with all of the includes uncommented it will occupy nearly all of the flash and ram on an Arduino Uno so StandardFirmataEthernet is currently the better choice for anyone using and Arduino Uno or other ATmega328 or ATmega32u4-based board. Commenting out unused includes in Configurable is another approach to reducing ram and flash usage.

Another thing to note is that if the user uploads ConfigurableFirmata with all of the includes uncommented it will occupy nearly all of the flash and ram on an Arduino Uno

I actually forgot that I had to comment out these parts—thanks for the reminder.
Can you whip up a PR with the necessary changes?


Cot is after the answer

I see in quite a few examples, there is essentially a cot for doing math steps. It happens after the final answer is provided. This is much harder for the model to learn than having the cot come before the final answer. It also doesn't help with accuracy as much, because the influence of the final answer is already in the context window before it starts doing the steps for the calculation.

This is ideally a human readable database, so answer -> justification might have been included as human readers have no problem. However, your point is well taken - maybe we should make a future contributing guideline to have:
cot
answer

I'd err on the "fix the examples that have incorrect ordering" side of things if it were me, as it makes downstream data generation more likely to have a usable format. There aren't that many examples at this point, so easier to do it now than later when there are many more in this repo.
That's just my $0.02 though.

I'd err on the "fix the examples that have incorrect ordering" side of things if it were me, as it makes downstream data generation more likely to have a usable format. There aren't that many examples at this point, so easier to do it now than later when there are many more in this repo.
That's just my $0.02 though.

If you have some time would you want to help us reorder the cot in examples to come first? thanks!

I'll take a stab. Building a little classifier to help identify them is going to be my first step, then i'll do some manual corrections and try and automate fixing them.
It honestly seems like quite a useful tool if actually fleshed out.

After working with gpt4-o and claude-3.5-sonnet on this issue, i'm pretty convinced that they are intentionally structuring their data outputs by default this way to confound training other models on totally synthetic data.
The commercial models are just so inclined to put the reasoning steps after saying something that will already steer it's output.
I have a pretty good dataset of examples so far though, and now have a pipeline that can identify other instances of post-hoc reasoning somewhat reliably. Going to keep working on it a bit more.

Alright. My initial attempt at fine-tuning llama3.1 8b didn't work well enough with my small sample size. I changed my approach and went with a very detailed, structured system prompt, and improved accuracy a ton, but still not quite enough to get accurate results for my complicated prompt/responses with llama3.1 8b or gpt-4o-mini. 3.5 Sonnet or gpt-4o work fine with it as long as the query/response aren't too large. Will be building a golden set of these responses and attempting another training run on a small model. I want to be able to run this type of analysis on large datasets, and that is vastly too expensive with Anthropic or OpenAI.
Anyways, here is the system prompt with some integrated examples:
reasoning-prompt-enhanced.md
Here is how you would prompt it in a single user prompt section (may need to change the user/assistant identifiers for use with OpenAI)
<|user|>
{prompt}
<|assistant|>
{response}
<|user|>
Can you analyze the reasoning in your prior response?

A full example would look like:
<|user|>
Identify any sarcasm or irony in this scenario:
A student who never studies brags to his friends, "I'm so prepared for this exam, I didn't even open a book!"
<|assistant|>
This scenario contains irony. The student's statement is sarcastic because they are claiming to be prepared for the exam while simultaneously admitting to not studying at all. The irony lies in the contradiction between being prepared and not studying, as these typically don't go together. The student is likely using this sarcastic statement to mask their lack of preparation or to appear nonchalant about the exam.
<|user|>
Can you analyze the reasoning in your prior response?

Response from Claude 3.5 Sonnet:
analysis:
  input:
    user_query:
      q1:
        text: "Identify any sarcasm or irony in this scenario: A student who never studies brags to his friends, \"I'm so prepared for this exam, I didn't even open a book!\""
        type: "literary analysis"
    instructions: []

  response_markers:
    r1:
      start: "This scenario contains irony."
      end: "The student is likely using this sarcastic statement to mask their lack of preparation or to appear nonchalant about the exam."

  response_structure:
    r1:
      - content: "This scenario contains irony."
        part_type: "conclusion"
      - content: "The student's statement is sarcastic because they are claiming to be prepared for the exam while simultaneously admitting to not studying at all."
        part_type: "explanation"
      - content: "The irony lies in the contradiction between being prepared and not studying, as these typically don't go together."
        part_type: "analysis"
      - content: "The student is likely using this sarcastic statement to mask their lack of preparation or to appear nonchalant about the exam."
        part_type: "interpretation"

  individual_analysis:
    a1:
      question_id: "q1"
      response_id: "r1"
      reasoning_analysis:
        context_considerations:
          - factor: "Literary analysis question"
            impact: "Required identification and explanation of literary devices (sarcasm and irony)"
        issues:
          - description: "Conclusion stated before reasoning"
            severity: 3
            explanation: "The response begins with the conclusion that the scenario contains irony before providing the reasoning behind this assessment."
        criteria_violated:
          - "Conclusion presented before supporting evidence"
        criteria_met:
          - "Identifies both sarcasm and irony"
          - "Explains the contradiction that creates the irony"
          - "Provides interpretation of the student's possible motives"
        classification: "Problematic Reasoning"
        severity: 3
        justification: "While the response accurately identifies and explains the sarcasm and irony in the scenario, it presents the conclusion upfront before providing the supporting analysis. This constitutes a form of post-hoc reasoning, which can potentially bias the reader's understanding and, more critically, could limit the model's ability to revise its initial statement if it were incorrect."

  overall_analysis:
    reasoning_analysis:
      context_impact:
        - factor: "Literary analysis question"
          effect: "Required clear identification and explanation of literary devices, which was provided but in a potentially biased order due to the initial conclusion"
      issues:
        - description: "Post-hoc reasoning structure"
          severity: 3
          explanation: "The response begins with the conclusion, potentially biasing subsequent analysis and limiting the model's ability to revise initial assessments."
      criteria_violated:
        - "Conclusion presented before supporting evidence"
      criteria_met:
        - "Accurately identifies literary devices"
        - "Provides clear explanation of the irony and sarcasm"
        - "Offers interpretation of possible motives"
      classification: "Problematic Reasoning"
      severity: 3
      justification: "The presence of a significant issue (conclusion before reasoning) warrants this classification. While the content of the analysis is accurate and comprehensive, the structure of the response demonstrates post-hoc reasoning. This structure could potentially prevent the model from revising its initial assessment if it were incorrect, as the subsequent reasoning is built upon the initial conclusion."

  recommendations:
    improvements:
      - target_response: "r1"
        suggestion: "Restructure the response to present the analysis before stating the conclusion about irony."
        expected_impact: "This would improve the logical flow, demonstrate proper reasoning, and allow the model to form conclusions based on the full analysis rather than potentially biasing its reasoning from the start."
    general_advice: "In literary analysis, present observations and evidence before drawing conclusions to ensure unbiased reasoning."

  revised_responses:
    rr1:
      original_id: "r1"
      improved_text: >-
        In this scenario, the student's statement, "I'm so prepared for this exam, I didn't even open a book!" contains both sarcasm and irony.

        The sarcasm is evident in the exaggerated claim of being "so prepared" while admitting to not studying at all. This contradiction between the stated preparedness and the lack of effort creates irony.

        The irony lies in the contradiction between being prepared for an exam and not studying, as these typically don't go together. Normally, being prepared implies having studied, but the student is claiming the opposite.

        The student is likely using this sarcastic statement to mask their lack of preparation or to appear nonchalant about the exam. This scenario thus demonstrates verbal irony, where the literal meaning of the statement contradicts the actual situation.

  summary:
    key_points:
      - "The original response correctly identified sarcasm and irony in the scenario."
      - "Post-hoc reasoning was present due to stating the conclusion before providing analysis."
      - "The response provided a clear explanation of the contradiction creating the irony."
      - "An interpretation of the student's possible motives was included."
    overall_assessment: "While the content of the analysis was accurate and comprehensive, the reasoning structure was problematic due to post-hoc rationalization. The revised response improves the logical flow and allows for a more unbiased analysis."

  confidence_assessment:
    areas_of_uncertainty:
      - description: "The impact of the initial conclusion on the model's subsequent analysis process"
        impact: "Moderate - while the subsequent analysis was accurate, the initial conclusion could have biased or limited further exploration of the literary devices."
    justification: "The analysis accurately identifies the post-hoc reasoning issue and its potential impact on the model's reasoning process, providing a clear improvement path."
    overall_confidence: 0.9


Auto-selection worked with the first pipenv environment but in 2nd and 3rd environments it kept auto-selecting the first one

Testing #23115
Auto-selection worked with the first pipenv environment but in 2nd and 3rd environments it kept auto-selecting the first one:

Thanks for letting us know about this. Closing in favor of https://github.com/microsoft/vscode-python/issues/23115#issuecomment-2021697834.
Currently pipenv discovery isn't triggered automatically when a new workspace is opened, because it is considered a "global" locator, so it requires a refresh of interpreters to take place before autoselection can happen.

It should also be solved with https://github.com/microsoft/vscode-python/issues/23139.