doc: RFC for handling prerelease strings.  ref #510

Per this comment in #510, creating an RFC doc for specifying how prerelease versions should be handled.

@chingor13 Thanks for the feedback.  You ask good questions.  If you don't mind, I'd like to reply here and get your thoughts before updating the PR...

Additional considerations:

which commit types influence each version bump

This is a tricky question.
For starters, the Conventional Commits spec doesn't address prerelease fields at all.  There's simply no mention of "prerelease" in the spec.  I suspect this is because the authors don't consider the prerelease field to be versionable in the way "primary fields" (major / minor / patch) are.
Yet the SemVer spec does devote a fair bit of verbiage to the prerelease field.  Notably, it says "Pre-release versions have a lower precedence than the associated primary version".  But, unlike the minor and patch fields, no guidance is given on how it should behave when higher-precedence fields are bumped.  E.g. The patch field "MUST" be set to zero if either the major or minor fields is incremented.  But it says nothing about the prerelease field.
Furthermore, in practice the prerelease string is most-often used to track "significant" branches of development off the main branch.  For example it's not uncommon for a project that has a 1.x release in production to push out a 2.x beta release with semvers like "2.0.0-beta-1".  The semantic being, "this is a candidate for the 2.0.0 release".
But what happens if someone adds a "BREAKING CHANGE" commit to that branch?  Does the version get bumped to "3.0.0-beta-1"?  That's what Conventional Commits would ordinarily mandate.  And if I'm reading the prelease versioning strategy PR you link to properly, that's what would happen there, right?
... but that doesn't really make sense.  In this [common] scenario, the breaking change is redundant with the fact this is a "2.x-beta" candidate.
In other words ...

If a semver has a prerelease field, then I think the assumption needs to be that the primary fields reflect the intended version, once the prerelease tag is removed.  As such, if a prerelease field is present I don't think the commit type should influence primary fields.

how (or if) the primary semver numbers will change

See above.  tl;dr: They shouldn't.

how (or if) the release becomes non-prerelease

See above.  I think the conclusion is that the prerelease field is removed, with no change to the primary semver numbers.

Additional note: we added a prerelease versioning strategy that follows our googleapis/google-cloud-dotnet libraries handle pre-release versioning in #1981. If that one does not suit your needs, it could be used as a blueprint for a similar new versioning strategy.

Interesting.  I wasn't aware of this work.  My initial reaction is that this seems complicated and problematic.  Again, if a semver has a prerelease field, I think the semantic needs to be that the primary fields are the intended version, upon actual release.  And, as such, they shouldn't change.
Thoughts?

In thinking about this further I find myself a bit conflicted.  The semantic of prerereleases is pretty clear: They are candidates for the version mentioned in the primary fields.  E.g. "2.0.0-beta.1" is a candidate for the 2.0.0 release.
As such, my argument for not touching the primary fields is sound... but I'm realizing that's only if it's a major-release candidate(!)
Things get tricky when we start talking about minor or patch release candidates.  For example, "2.3_beta.1" would mean be a candidate for the "2.3" release. (Implying that the current release is 2.2.x.   But what happens if there's a BREAKING CHANGE commit?(??)
(... which I suspect @chingor13 's point to begin with. 😄 )
Three options come to mind:
plays)

Apply the appropriate change to the primary fields and reset the prerelease field.  E.g. the version might become "3.0.0_beta.0"

So... I may need to backpeddle on all of this.  I was only considering major-prerelease versions.  I wasn't thinking about lessor versions like "2.3_beta.1".  😬
When you start thinking through the lessor-version scenarios, I think you kind of inevitably land on a prerelease strategy like the one you've implemented for googleapis/google-cloud-dotnet.
I need to think through this further but, alas, don't have time at the moment.  Before I go, though, I'll note that the semver module has a bunch of test cases for bumping prereleases.    Do you happen to know if the prerelease strategy you've implemented aligns with those?

If you don't mind I would like to add my two cents here, even thought I think it should be covered in the test cases aforementioned.
In general, coming from for example semantic-release module, I would expect that the pre-release would be just a case to be bumped based on the build itself (of course considering changes).
For instance, if in my first commit to introduce this new release would be simply some new feature, we should just bump the minor version, but if for some reason we have to introduce a BREAKING CHANGE it should simply bump major and reset the  pre-release version(as per proposed being .0 again).
If after first Breaking change other breaking changes or features get added, it shouldn't change the target version for example 1.0.0.rc-0++(++ in here is just to refer as incrementing this version) same as other bumps if they happen, the most important is bumped and pre-release restart the increments only if version changes.
This is very useful feature for example in cases were we have this PR releases as things are there to be released in a trunk(for sprint wise for instance) but we would like still to be able to pre-release to some environment for some QA tests for instance and having the pre-release there would be nice for visibility of the versions being tested and linked to the PR as well.


Naming convention and consistency

Hi @0xAtum. I was asked to review this repo and provided some feedback on a private Discord group. I mentioned it'd be more helpful if I share the feedback on the repo. I'll be doing so by opening some issues so that we can discuss, settle each of them and move this forward.
My first issue has to do with agreeing on naming conventions and improving consistency.
Staking or locking?
I question whether "staking" is the right name to use for what we're trying to do. I think we should be talking about "locking" instead. From my Discord feedback:

In my mind the key difference between "locking" and "staking" is not whether you're getting some type of reward or not (perks or APY), it's whether you're risking the principal or not. I.e. with locking you can absolutely expect to get your tokens back when the schedule terminates (barring bug/hack), but with staking you're putting your tokens on the line in some way (slashing, shortfall or whatever).

I think we should settle this discussion first, as it'll influence the nouns and verbs we should use everywhere else in the codebase.
Entity naming consistency
Whatever naming convention we settle on, we should strive for naming consistency. Here's a list of examples of naming inconsistencies we should address:

owner vs wallet vs receiver. Maybe a better name for all these would be "staker" if we stick with the staking theme, maybe "locker" but that's somewhat ambiguous, "recipient" or "beneficiary". Open to discuss.
stakeId vs scheduleId vs cachedTotalVesting. Likely id would be best whenever the context determines the rest, scheduleId whenever it's not clear.
amount vs stakedAmount. amount is likely good enough in all places, as I don't think we ever refer to anything other than the staking/locking amount. Possibly balance could also be an option, but maybe that's not as good or flexible.
lockSchedule (notice the use of the term "lock" and not "stake"!) vs scheduleDuration vs duration. duration is the winner in my mind, for the same reason as amount.
allStakings being indexed by scheduleId. Indexing by id is clear enough and would be more consistent with other changes suggested in this issue.

@0xAtum, I suggest you open a PR to start addressing these issues and we can continue the discussion over there, what do you think. Let me know if you disagree with any of my suggestions or if there's anything to clarify.

Please inform me if there are any issues with the codebase, such as bugs, important optimization, missing features within the scope or required utility functions.
Any additional matters will not be addressed.
You are welcome to modify the codebase to adhere to a specific naming conventions.
Thank you for your cooperation.

Inconsistent naming is a code quality issue, i.e. "an issue with the codebase".
If you don't want to address these issues that's fine as far as I'm concerned, but please don't close issues, as we'd like to use these to keep track of pending work.

I allocated some of my free time to address this issue.
FYI: I fully agree with you regarding code quality and naming conventions. However, I need to distinguish between what is "critical" or "essential" versus what is "minor improvement". Unlike high-maintenance services, Solidity smart contracts are frequently not revisited or modified after deployment, thereby reducing the importance of naming conventions relative to other software standards.
I hope my PR will resolve the naming,
cheers


:bug: WIP test k8s

WIP test pinned k8s versions

/test-ubuntu-e2e-release-0-5
/test-centos-e2e-release-0-5

/cc @furkatgofurov7

This was the test PR for https://github.com/metal3-io/metal3-dev-env/pull/1047 and can be closed now.
/close


ci/coverage: Remove more cruft in diskspace hack

like the hack itself this is less than ideal as it takes real time to remove the cruft
also not entirely clear that by removing more we will free up enough disk space
the other option is to boost the type of worker (ie to a non-free one) which has more space anyway - we might decide that is a good idea for coverage as it takes quite a bit of time to run local tasks - but that comes with a significant cost and for the most part leaves an expensive vm hanging while the rbe tasks complete
cc @asedeno

/retest rbe/cache UNAVAILABLE


wrong call: $entityManager->save()

should be $entityManager->insert() in RestCrudController in Line 120

Thank you. Fixed in current master


Find-DbaTable, filter capabilities

name: Feature request/idea :bomb:
about: Suggest a new feature or enhancement

Summary of new feature
Get-Table is a nice feature to get quickly information table, but misses filters like wildcards (e.g. all tables that begin with tt_). Let's create a Find-DbaTable for that.
Thanks!
Proposed technical details (if applicable)
Latest version of dbatools as of writing
0.9.190

Hi niphlod,
I didn't find out the find-dbatable neither in the website's command index nor from the follwing command
get-command -Module dbatools | Where-Object { $_.Name -like "Find*" } | Sort-Object
My dbatools version = 0.9.190

nothing really. Just saying that the feature request for filtering should be "fullfilled" in another function. All the other info I'm really not sure of.....get-dbatable returns the SMO object and should have all properties related to the table already. Did you already tried using Get-DbaTable ..... | select * ?

Ok. I got it.
Yes, using the following command meets my need:
Get-DbaTable -SqlInstance SRV -Database DB | Sort-Object -Property DataSpaceUsed -Descending | Select-Object Name, DataSpaceUsed -First 10

no probl, I'll go ahead and edit the request so the filtering enhancement remains logged properly

not sure this can be speeded up by some internals. Get-DbaTable does the best job it can already...


Failed to release

Hi,
just discovered a lot of messages like
2020-04-10T15:16:39.792  log_level=info log_ref=P0039 db_id=28 pid=<0.1559.0> Failed to release pid=r2d_fail leaving SnapshotCount=2 and MinSQN=21715
2020-04-10T15:16:39.830  log_level=info log_ref=P0039 db_id=28 pid=<0.1559.0> Failed to release pid=r2d_fail leaving SnapshotCount=2 and MinSQN=21715
2020-04-10T15:16:39.830  log_level=info log_ref=P0039 db_id=28 pid=<0.1559.0> Failed to release pid=r2d_fail leaving SnapshotCount=2 and MinSQN=21715
2020-04-10T15:16:39.839  log_level=info log_ref=P0039 db_id=28 pid=<0.1559.0> Failed to release pid=r2d_fail leaving SnapshotCount=2 and MinSQN=21715
2020-04-10T15:16:39.841  log_level=info log_ref=P0039 db_id=28 pid=<0.1559.0> Failed to release pid=r2d_fail leaving SnapshotCount=2 and MinSQN=21715

What does this message actually mean?

It shouldn't be a cause for immediate concern.  Folds against leveled take a read-only snapshot of the ledger to give a consistent view of the state of the store at the time the fold commenced.
When the fold completes, they should tidy up and de-register themselves (which allows for ledger file deletes blocked due to the snapshot to complete).  If a fold fails to de-register within a timeout it will be killed to free-up the store again.
When a file in the ledger is ready to delete (this will happen quite often) it checks there are no outstanding snapshots which require it to be present.  If not, it can erase itself, otherwise it runs a check to see if any of the snapshots which were blocking the delete have timed out - and this check prompts the log P0036 with pid=r2d_fail.
This can be a bit noisy in the logs, as often many files will be ready to delete at the same time (after a big merge).  If there is a snapshot running, then they will all prompt the log.


fix: Rename home articles section to market news

The type of this PR is: Enhancement

Description
This is a follow up PR for https://github.com/artsy/eigen/pull/4901. As discussed in  CX-1550 this PR aligns the home articles rail to the new design specs and renames the section to "Market News".

Note: I didn't add a changelog because https://github.com/artsy/eigen/pull/4901 already added a changelog about the articles rail.

PR Checklist (tick all before merging)

[x] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[x] I have tested my changes on iOS and Android.
[x] I have added tests for my changes, or my changes don't require testing, or I have included a link to a separate Jira ticket covering the tests.
[x] I have added a feature flag, or my changes don't require a feature flag. (How do I add one?)
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added an app state migration, or my changes do not require one. (What are migrations?)
[x] I have added a CHANGELOG.yml entry or my changes do not require one.

🗞


Make ResourceSelector label prop required

Makes the ResourceSelector's label prop required.
This was split out as a piece of work from https://github.com/grafana/grafana-aws-sdk-react/pull/33/files#r1044841576.

ResourceSelector usages https://github.com/search?q=org%3Agrafana+ResourceSelector+language%3ATSX&type=code&l=TSX
ConfigSelector usages https://github.com/search?q=org%3Agrafana+ConfigSelect+language%3ATSX&type=code&l=TSX (ConfigSelector uses ResourceSelector).
Note these components are only used by Athena, Redshift, and Timestream. But I ran yarn build on all of the AWS data sources to check that there are no type errors.

Actually going to close this. The the label prop is passed through to the InlineField component which accepts an optional label so will just leave this as is.


New version: BinaryFortress.WindowInspector version <IP_ADDRESS>

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[X] Does your manifest conform to the 1.0 schema?

Microsoft Reviewers: Open in CodeFlow

2022-03-18 22:32:50.188   Exception message:
2022-03-18 22:32:50.188   Message box (OK):
                          Unable to execute file:
                          C:\Program Files\Window Inspector\Window Inspector.exe

                          ShellExecuteEx failed; code 216.
                          This version of %1 is not compatible with the version of Windows you're running. Check your computer's system information and then contact the software publisher.


No idea of the fix

Terminal error
"
ERROR:root:!!! Exception during processing !!!
ERROR:root:Traceback (most recent call last):
File "F:\Stable-Diffusion\ComfyUI\ComfyUI_windows_portable\ComfyUI\execution.py", line 153, in recursive_execute
output_data, output_ui = get_output_data(obj, input_data_all)
File "F:\Stable-Diffusion\ComfyUI\ComfyUI_windows_portable\ComfyUI\execution.py", line 83, in get_output_data
return_values = map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True)
File "F:\Stable-Diffusion\ComfyUI\ComfyUI_windows_portable\ComfyUI\execution.py", line 76, in map_node_over_list
results.append(getattr(obj, func)(**slice_dict(input_data_all, i)))
File "F:\Stable-Diffusion\ComfyUI\ComfyUI_windows_portable\ComfyUI\custom_nodes\ComfyUI_FizzNodes\ScheduledNodes.py", line 77, in animate
animation_prompts = json.loads(inputText.strip())
File "json_init_.py", line 346, in loads
File "json\decoder.py", line 340, in decode
json.decoder.JSONDecodeError: Extra data: line 1 column 35 (char 34)
"

I also tried without the math function but it also doesn't works

The batch string schedule puts out a raw list of the scheduled prompts but not the dictionary values. so your prompt example would be sending:
(neutral:weight calc at frame #), (neutral:weight calc at frame #), (neutral:weight calc at frame #), (neutral:weight calc at frame #), (neutral:weight calc at frame #), (hint_of_smile:weight calc at frame #), (hint_of_smile:weight calc at frame #), (hint_of_smile:weight calc at frame #), etc.
some repos use lists of prompts for their inputs but the prompt schedules in this repo need to be formatted to the dictionary format like in the examples. the regular string schedule is the same but only outputs the one prompt at the current frame, not the dictionary value. to do what I think you want to do, use a primitive as the input. the values are still calculated by the regular prompt schedule nodes. I hope this helps!


Does client send .tei.xml files preferentially?

Looking at https://github.com/softcite/tutorials/blob/master/process_all_of_plos.md seems that one wants to convert to tei.xml files first, before sending the file to the server.
Looking at the code, though, client.py:197 says "only PDF for the moment" ... is that still true?  Reading further it seems that the client prefers to send pdf over the tei.xml?  But the annotate method prefers the tei.xml?  In the tutorial linked above you use repo-in and I think pass a folder that only has tei files in it?
Context of the question is where I'm up to in the https://github.com/howisonlab/screenit-softcite repo :)

Hi @jameshowison !
I didn't updated the comments in the code sorry. The client supports PDF and XML as input. The client has no preference... if it finds a PDF in the input repository it calls the PDF processing endpoint, if it finds a TEI (just checking the extension .tei.xml, nothing more sophisticated), it calls the TEI endpoint, and for other XML it calls the XML endpoint that will use Pub2TEI to identify and transform the right XML formats (among all the supported one).
It's possible to pass a repository with a mixture of PDF, TEI XML and other XML formats, the client will make the right calls to the server accordingly. Fastest will be TEI, then PDF and other XML will be slower because they require a one shot application of the XSLT from Pub2TEI - which take 2-3 seconds just to load. This is why it's largely preferable to transform all the XML in batch with Pub2TEI before software mention extraction, because the XSLT will be loaded just one time for all.

I updated the comments/readme with 7bec98952f7e0a20e64c057320ea6fed1426fe55
I forget to add the following to answer your question: the client processes all the PDF and an XML it finds, which means that for the same document, at the same place in the input repository, if there are at the same time a PDF and an XML fulltext, it will process both in parallel, and the last one to update the json software annotation file wins. I have only really considered so far the scenario where we only have one full text file for a document.
So probably the best would be to process only one version of the full text, preferably in this order for quality reasons: tei, xml, pdf.
We could also add a parameter to manage the preferences.

Ah, got it. That was my read of the code as well. Yeah, by default
preferentially doing tei.xml makes sense :) flag to manually choose between
Btw I got things working on my laptop (without GPU, M1 Mac troubles) and it
was super fast.
On Sat, Jul 22, 2023 at 05:14 Patrice Lopez @.***>
wrote:

I updated the comments/readme with 7bec989
https://github.com/softcite/software_mentions_client/commit/7bec98952f7e0a20e64c057320ea6fed1426fe55
I forget to add the following to answer your question: the client
processes all the PDF and an XML it finds, which means that for the same
document, at the same place in the input repository, if there are at the
same time a PDF and an XML fulltext, it will process both in parallel, and
the last one to update the json software annotation file wins. I have only
really considered so far the scenario where we only have one full text file
for a document.
So probably the best would be to process only one version of the full
text, preferably in this order for quality reasons: tei, xml, pdf.
We could also add a parameter to manage the preferences.
—
Reply to this email directly, view it on GitHub
https://github.com/softcite/software_mentions_client/issues/4#issuecomment-1646549883,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAWOUXJ76J5C3IWFNHK32DXRORWTANCNFSM6AAAAAA2TJOANQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Btw I got things working on my laptop (without GPU, M1 Mac troubles) and it
was super fast.

Imagine what it would be with a GPU :D

I have updated software_mentions_client to version 0.1.9 on PyPi to prefer TEI XML files over PDF when both are available.


Constructor methods (js interop)

I wanted to give hacking on the compiler a shot, and I have to say it was surprisingly easy (at least this far). Very cool!
This is something I have been wanting for JS interop for a while, given that it's a prevalent pattern in the JS world, and the transformation to classes is not always trivial.
I'm sure there is a lot that is not optimal here, I just wanted to hear if you think this is worthwhile before spending more time. WDYT?

Can one of the admins verify this patch?

Dear Jenkins, test this please

@oyvindberg Given that this is a language change, it must go through a Proposal before we discuss the code, in the form of an Issue. The proposal should clearly state the problem it addresses, the proposed solution, and ideally evaluate pros and cons with alternatives.
Changing the language is not something we do lightly, even ourselves, because we keep supporting everything forever.
For some good examples of Proposals that we had before, see:

Accepted: #1795, #2009, #2702, #2734 (the latter is shorter)
Rejected: #2720

Also, for us to be able to look at your PR, we need you to sign the Scala CLA.

Ok, good, thanks. I'll have a look at that tomorrow :)

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/3902/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/5159/
Test PASSed.

I realized that this feature is trying to address #2398, in a way. It's providing some Scala syntax for the JavaScript syntax new x.Foo(a, b, c). The Scala syntax it offers is x.Foo(a, b, c). It would be so much better to offer the syntax new x.Foo(a, b, c) in Scala too!
That would be provided by allowing (and supporting through the pipeline) nested JavaScript classes. The snippet from the test case in the PR would be written as:
  @js.native
  @JSGlobal("ConstructorMethodTest")
  object ConstructorMethodTest extends js.Any {
    @js.native
    class Class(s: String) extends js.Object {
      val field: Int = js.native
      def method(): String = js.native
      def getConstructorParam(): String = js.native
    }
  }

without any ceremony.
Supporting nested JavaScript classes is technically much more involved than your @JSConstructorMethod, but it is much better from a language design point of view, and provides an even more intuitive and boilerplate-free syntax to users. It is something we vaguely have in mind for 1.x, together with #2805 (they don't look similar on the surface, but in fact they are underneath).
Therefore, I don't think your @JSConstructorMethod proposal would make it, when contrasted to the larger scheme.

Alright, sorry for the late answer, I should have explained myself right away.
It is #2398 , yes. I had a look through some issues, but failed to notice that one.
I modelled this after Typescript's constructor methods. In Typescript, it seems facade classes are sugar for a var with constructor methods (the constructor type) plus an interface with the class members, so constructor methods are more general than classes there.
Given that, it is sometimes useful to talk about the constructor type, when Javascript people use the fact that constructor type instances are first-class values, and are returned from functions and so on. For example from the stdlib:
interface RegExpConstructor {
    new (pattern: RegExp, flags?: string): RegExp;
    //...
    [Symbol.species](): RegExpConstructor;
}

Anyhow, I'm sure inner classes cover most important use-cases. It was fun to have a go at the compiler anyway :)

I realize I forgot to answer here, sorry.
I agree that it it would sometimes be useful to talk about the constructor type. So far our best type for those are js.Dynamic that can be given to js.Dynamic.newInstance(ctor)(...). That's for example the result type of js.constructorOf. In theory, better types should exist, but that would require a different approach, I think (e.g., well-known types js.Constructor[Ts..., C] or something like that, like we have js.FunctionNs).
In practice, when we really do want something more precise, we can use a user-defined value class wrapping the js.Dynamic constructor. The solution in this PR does not really provide a less verbose or safer solution than such a wrapper.
In the present situation, I would at least wait until we have addressed the inner class issue before proceeding in this direction. Better support for inner classes should be uncontroversial, and then we can see whether more is still needed.
I'm going to close this PR for now.


ARROW-3369: [Packaging] Wheel builds are failing due to wheel 0.32 release

@xhochy sorry, it's WIP


python312Packages.pyclipper: 1.3.0.post5 -> 1.3.0.post6

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python312Packages.pyclipper is: Cython wrapper for clipper library
meta.homepage for python312Packages.pyclipper is: https://github.com/fonttools/pyclipper
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.3.0.post6 with grep in /nix/store/vipsq5mack1jm2qzpm86wg4a378ah100-python3.12-pyclipper-1.3.0.post6
found 1.3.0.post6 in filename of file in /nix/store/vipsq5mack1jm2qzpm86wg4a378ah100-python3.12-pyclipper-1.3.0.post6

Rebuild report (if merged into master) (click to expand)

75 total rebuild path(s)

75 package rebuild(s)

First fifty rebuilds by attrpath

deltachat-desktop
easyocr
fira-math
fontbakery
k40-whisperer
mokuro
nanoemoji
nomnatong
noto-fonts-color-emoji
open-webui
openmoji-black
openmoji-color
python311Packages.afdko
python311Packages.babelfont
python311Packages.beziers
python311Packages.booleanoperations
python311Packages.bumpfontversion
python311Packages.cffsubr
python311Packages.collidoscope
python311Packages.diffenator2
python311Packages.easyocr
python311Packages.fontbakery
python311Packages.fontfeatures
python311Packages.fontmake
python311Packages.fontparts
python311Packages.gftools
python311Packages.glyphsets
python311Packages.glyphslib
python311Packages.glyphtools
python311Packages.nanoemoji
python311Packages.notobuilder
python311Packages.nototools
python311Packages.pyclipper
python311Packages.rapidocr-onnxruntime
python311Packages.shaperglot
python311Packages.statmake
python311Packages.ufo2ft
python311Packages.ufomerge
python311Packages.ufoprocessor
python311Packages.vttlib
python312Packages.afdko
python312Packages.babelfont
python312Packages.beziers
python312Packages.booleanoperations
python312Packages.bumpfontversion
python312Packages.cffsubr
python312Packages.collidoscope
python312Packages.diffenator2
python312Packages.easyocr

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/vipsq5mack1jm2qzpm86wg4a378ah100-python3.12-pyclipper-1.3.0.post6 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python312Packages.pyclipper https://github.com/r-ryantm/nixpkgs/archive/af9a4001993091f54a541b4bd86c4701cf18868d.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/af9a4001993091f54a541b4bd86c4701cf18868d#python312Packages.pyclipper

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/vipsq5mack1jm2qzpm86wg4a378ah100-python3.12-pyclipper-1.3.0.post6
ls -la /nix/store/vipsq5mack1jm2qzpm86wg4a378ah100-python3.12-pyclipper-1.3.0.post6/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

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

Result of nixpkgs-review pr 349577 at af9a4001 run on x86_64-linux 1

:x: 1 package failed to build:

open-webui

:white_check_mark: 69 packages built:

deltachat-desktop
easyocr (python312Packages.easyocr)
fira-math
fontbakery (python312Packages.fontbakery)
k40-whisperer
mokuro
nanoemoji (python312Packages.nanoemoji)
nomnatong
noto-fonts-color-emoji
openmoji-black
openmoji-color
python311Packages.afdko
python311Packages.babelfont
python311Packages.beziers
python311Packages.booleanoperations
python311Packages.bumpfontversion
python311Packages.cffsubr
python311Packages.collidoscope
python311Packages.diffenator2
python311Packages.easyocr
python311Packages.fontbakery
python311Packages.fontfeatures
python311Packages.fontmake
python311Packages.fontparts
python311Packages.gftools
python311Packages.glyphsets
python311Packages.glyphslib
python311Packages.glyphtools
python311Packages.nanoemoji
python311Packages.notobuilder
python311Packages.nototools
python311Packages.pyclipper
python311Packages.rapidocr-onnxruntime
python311Packages.shaperglot
python311Packages.statmake
python311Packages.ufo2ft
python311Packages.ufomerge
python311Packages.ufoprocessor
python311Packages.vttlib
python312Packages.afdko
python312Packages.babelfont
python312Packages.beziers
python312Packages.booleanoperations
python312Packages.bumpfontversion
python312Packages.cffsubr
python312Packages.collidoscope
python312Packages.diffenator2
python312Packages.fontfeatures
python312Packages.fontmake
python312Packages.fontparts
python312Packages.gftools
python312Packages.glyphsets
python312Packages.glyphslib
python312Packages.glyphtools
python312Packages.notobuilder
python312Packages.nototools
python312Packages.pyclipper
python312Packages.rapidocr-onnxruntime
shaperglot (python312Packages.shaperglot)
python312Packages.statmake
python312Packages.ufo2ft
python312Packages.ufomerge
python312Packages.ufoprocessor
python312Packages.vttlib
signal-desktop
signal-desktop-beta
smc-chilanka
twitter-color-emoji
whatsapp-emoji-font


keras.model_to_estimator feature request

Really does not depend on system information, just a completely new feature.
Using the:
tf.keras.estimator.model_to_estimator(keras_model=model,
                                                    model_dir=model_dir)

does not allow us to log images in tensor board while training the model. I believe this is because the keras model is serialized and then rebuilt in this function and it does not capture the summaries that you put in the original graph.
I am able to get around this by using a Lambda layer that looks like this:
def viz_layer(x):
    from tensorflow.summary import histogram
    histogram('hist', x)
    return x

model.add(layers.Lambda(viz_layer))

So two problems with this:

It's pretty hacky to have to add identity layers in to the network to visualize
I can't get the lambda layer to work without importing tensor flow inside the layer which is pretty bad

It would be great if the original summaries were captured or if there were a more streamlined way to add summaries.

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

Is it related to https://github.com/tensorflow/tensorflow/issues/14879#issuecomment-351996902?

Yes exactly!

There are many users in sparse tickets that are asking full docs coverage of high level api. See also https://github.com/tensorflow/tensorflow/issues/14504#issuecomment-371195181

Yeah, multi-gpu was literally the next question I was going to have. I'm not sure if I'd call this just docs coverage. I really don't think there is an effective way to add summary visualizations to keras models going into estimators. Seems to be a problem with: keras.models.clone_model

You can use SessionRunHook but I don't know if it is the official solution.

Is there an example doing this to modify a keras graph?

No but you can take variables from the graph and add summary with standard TF calls inside this hook.

This seems very funky. I can only use the hook in a train or evaluate. So for each evaluate I'll need to add the summaries to the graph? Do I need to do it just once?

Was partially related to https://github.com/tensorflow/tensorflow/issues/15332

Just a small followup here. I have gone ahead and added image summaries to the model, however I am unable to output image summaries to TB with the evaluation hook:
    est = tf.keras.estimator.model_to_estimator(
        keras_model=model, model_dir=model_dir)

    eval_summary_hook = tf.train.SummarySaverHook(save_steps=validation_steps,
                                                  output_dir=os.path.join(
                                                      model_dir, 'test_images'),
                                                  summary_op=tf.summary.merge_all())

    for _ in range(epochs):
        est.train(lambda: make_train_iter(
            'image_input'), steps=steps_per_epoch)
        est.evaluate(lambda: make_test_iter('image_input'),
                     steps=validation_steps, name='test', hooks=[eval_summary_hook])

The error I get is:
ValueError: Fetch argument <tf.Tensor 'Merge/MergeSummary:0' shape=() dtype=string> cannot be interpreted as a Tensor. (Tensor Tensor("Merge/MergeSummary:0", shape=(), dtype=string) is not an element of this graph.)
This seems to imply the summaries that I merged from the keras estimator originally are forgotten when keras clones the model, so I only reference the old summaries. Super related to: #15332

Nagging Assignee @fchollet: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @fchollet: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @fchollet: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @fchollet: It has been 59 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @fchollet: It has been 74 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @fchollet: It has been 91 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @fchollet: It has been 106 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @fchollet: It has been 121 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @fchollet: It has been 136 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @fchollet: It has been 151 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @fchollet: It has been 167 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

This may be a case where using a model_fn that wraps your Keras model is the best strategy-- see option 2 here. That would allow you to add summaries without trying to hack them into the graph, as the current expectation is that summaries are separate from the graph, controlled by callbacks.
CC @omalleyt12 -- this is a good example of a use-case where users expect to have summaries in the graph. Do your planned updates for TB offer an easier solution here?
@knathanieltucker , in the meantime, does the model_fn route work for you?

That certainly works. 👍

Great. Closing for now-- please reopen a new issue if you encounter any difficulties.


only re-train the core or nlu that was updated in config

If you change only one part of the config (in this case I added a policy), you get
2019-05-05 14:55:17 INFO     rasa.model  - Data (config) for dialogue model changed.
2019-05-05 14:55:17 INFO     rasa.model  - Data (config) for NLU model changed.
it seems to check the file/overall dict instead of the contents of each sub-dictionary. It would be great if in this case it noticed that I didn't actually change the config for the NLU model and it didn't go through re-training.

@tabergma I think you already merged that in right?

yes, this is merged


Update setup.py to install correct pynwb version

Currently, using pip will install the incorrect version of pynwb causing ecogVIS to crash when loading an NWB file.
Resolves Issue #118

Hey @bendichter, why did you close this pull request?

I'm not sure. Seems like it should be fine


Update profileAvatar

var imgurl = '';

socket.on('imgUrl', function(url) {
  imgurl = url;
  console.log('setting image to:', url);
});

 <Widget
        profileAvatar={imgurl}
        handleNewUserMessage={this.handleNewUserMessage}
/>

Is this field only settable once? I have multiple users and would like to change this as messages from different users come in but it seems once it's set I can only use the initial image for all incoming messages.

Try sending a state and change it when you receive a new URL from the socket.
Problem: idk if it will change only one message's avatar or everyone

I ended up trying that but it replaces every response image, I had to end up writing a hacky way to find the last avatar with a certain class name and replace it with the image of the incoming user


Getting undefined for port when using receive

Hi @raimonomiar
which version you’re using?
it should works fine in v1.2.2
by the way a temporary fix is specify a port for receiving use -q option, it would be a temporary solution

@parvardegr the problem is only for the message that shows undefined. The QR code works fine and made a pr for the fix #32

you can just remove that console.log line, it wasn't suppose to be there, just a silly bug.

merged


Bower broken font path

The font path in the css files begins with a ~. For example ~grommet/scss/hpinc/font/hps-me-w27-regular-woff.woff.

Thanks for reporting that, we have it fixed in the master now, it should be part of the next version. In the mean time you can use the fonts CDN paths:
Metric font: http://hpefonts.s3.amazonaws.com/MetricWeb-Regular.woff
HPSimplified font: http://hpincfonts.s3.amazonaws.com/hps-me-w27-regular-woff.woff

I'm a sucker for https. It might be worth updating those URLs for the default. Especially if an app is going to be used behind https. Just thinking of a sane default.

Fixed by #76.


Improve position of stretchy accents.  #1735

Only bubble up italic correction or skew for nodes with only one child.
Resolves issue #1735

I've been playing around with the example from the issue and observe some very bizarre behaviour:

in HTML-CSS no accent is displayed
in CommonHTML the enriched math becomes huge
in SVG it looks good
Maybe we can discuss this tomorrow.

Both issues seem to be related to the mathsize="36pt" specification, and have nothing really to do with the issue being addressed here.  I think they should be added as a new issue instead of holding up this one.  (Note that this issue is about horizontal spacing within CHTML only.)

The HTML-CSS issue is due to the way HTML-CSS positions the accents, which fails for large mathsize settings.  I've opened #1798 for this.

Sounds good.

CommonHTML problem could be an maction issue. It's independent of accents. I'll file an issue.

I'm working on it already.  It is more than just mactions, it is the collapsible extension itself, somehow.  Just starting to look.

I've filed it in #1800. Feel free to close or move to a11y.

Thanks.  I'm working on it and will figure out if it is a CHTML or an a11y-based problem (or both).  It does seem to only affect CHTML, but that might be just a difference in how CHTML handles mathsize.


latest kha compiles .wav as ogg & .mp4

dont know if this is a bug or whatever, but noticed more recently with latest kha is that .wav files when compiled from a project to html5, convert to both .ogg & .mp4
never seen that before

Normal behavior. Some browsers support ogg, some support mp4, so we output both. Has been like that forever but only if you have the ffmpeg option set, which is required for mp4 - if it is not set, Kha uses software ogg decoding where necessary.


A model connecting host galaxy properties with multiple supernova populations

Here's an excerpt from Phil and my notes:
Several studies in recent years have pointed out correlations between type Ia SNe Hubble diagram residuals and the properties of their host galaxies, including stellar mass and morphological type at cosmological redshifts, and local star formation rates at low redshift.
At redshifts 0.1 - 1.0, the available information on the host galaxy is limited, but we can expect to be given posterior samples in M*, SFR, and other SED parameters, and also in structural parameters such as size, ellipticity and bulge/disk ratio. Taking these cartoon models seriously, we may be able to make rough approximations to local host properties, by using the model to interpolate the colors and brightnesses to the SN position.
There is some evidence for multiple sub-populations of supernova, whose intrinsic properties we might expect to depend differently on host galaxy parameters. The relative abundances of the populations will evolve over cosmic time, if they are coupled to the host properties.
So, here's what we're planning to do:

[ ] 1. Write down a generic PGM for the above

[ ] 2. Focus it on a possible simple implementation, anticipating having access to posterior samples in both SN and host parameters.

[ ] 3. Write out the probability theory corresponding to this model, and determine good sampling options to pursue.

[ ] 4. Design a small simulated dataset and analyze it using this model.

Comments welcome!

Hey @wmwv, perhaps the Bildsten and Scannapieco A/B rates idea might have
some legs in guiding this particular model, what do you think?
On Fri, Jun 19, 2015 at 2:25 PM, Kara Ponder<EMAIL_ADDRESS>wrote:

Here's an excerpt from Phil and my notes:
Several studies in recent years have pointed out correlations between type
Ia SNe Hubble diagram residuals and the properties of their host galaxies,
including stellar mass and morphological type at cosmological redshifts,
and local star formation rates at low redshift.
At redshifts 0.1 - 1.0, the available information on the host galaxy is
limited, but we can expect to be given posterior samples in M*, SFR, and
other SED parameters, and also in structural parameters such as size,
ellipticity and bulge/disk ratio. Taking these cartoon models seriously, we
may be able to make rough approximations to local host properties, by using
the model to interpolate the colors and brightnesses to the SN position.
There is some evidence for multiple sub-populations of supernova, whose
intrinsic properties we might expect to depend differently on host galaxy
parameters. The relative abundances of the populations will evolve over
cosmic time, if they are coupled to the host properties.
So, here's what we're planning to do:

Write down a generic PGM for the above

Focus it on a possible simple implementation, anticipating having
access to posterior samples in both SN and host parameters.

Write out the probability theory corresponding to this model, and
determine good sampling options to pursue.

Design a small simulated dataset and analyze it using this model.

Comments welcome!
—
Reply to this email directly or view it on GitHub
https://github.com/kbarbary/snpgm/issues/13.

Hi @kponder,

we can expect to be given posterior samples in M*, SFR, and other SED parameters, and also in structural parameters such as size, ellipticity and bulge/disk ratio.

I am not sure I am understanding you correctly but I interpret your statements in the following way:
P(SN probability / volume /time  \vert z ) = P(SN probability  / time / volume | Host galaxy parameters) P( Host galaxy parameters |z ) number density of galaxies (z) * Volume (z)
Of course you will need the extra piece P(SN parameters \vert Host Galaxy parameters)  with this.
It would be interesting to talk about this, as I think I am trying to do part of the simulation step which may be of help to you? The way I am getting P( Host galaxy parameters \vert z) is through  the LSST catsim database. Of course, this is not real data, and will need improvements, but I am hoping that this is a product that will improve and later come to represent our best understanding. That is my logic for using this framework.

Yes, but ideally with local measures of star formation (Rigault+13).  Restframe NUV is substantially more informative than restframe u (Rigault+15, Kelly+15).
The obviously tempting story is that there are different populations from short delay time and long delay time SNeIa.  This maps strongly to local projected measures of star formation, and more loosely to global star formation measures.
The Bildsten+Scannapieco model is consistent with such a story.
It doesn't, however, provide a physically-motivated prediction from first principles.  Rather, it provides a transfer function from a given star-formation history (stellar age distribution) to an expected rate.
This transfer function does, however, encode the information about the relative efficiencies of each channel.  Together with the stellar age distribution, this should then result in a prediction for n_A(z) and n_B(z).

Michael

On Jun 19, 2015, at 14:35 , Phil Marshall<EMAIL_ADDRESS>wrote:
Hey @wmwv, perhaps the Bildsten and Scannapieco A/B rates idea might have
some legs in guiding this particular model, what do you think?
On Fri, Jun 19, 2015 at 2:25 PM, Kara Ponder<EMAIL_ADDRESS>wrote:

Here's an excerpt from Phil and my notes:
Several studies in recent years have pointed out correlations between type
Ia SNe Hubble diagram residuals and the properties of their host galaxies,
including stellar mass and morphological type at cosmological redshifts,
and local star formation rates at low redshift.
At redshifts 0.1 - 1.0, the available information on the host galaxy is
limited, but we can expect to be given posterior samples in M*, SFR, and
other SED parameters, and also in structural parameters such as size,
ellipticity and bulge/disk ratio. Taking these cartoon models seriously, we
may be able to make rough approximations to local host properties, by using
the model to interpolate the colors and brightnesses to the SN position.
There is some evidence for multiple sub-populations of supernova, whose
intrinsic properties we might expect to depend differently on host galaxy
parameters. The relative abundances of the populations will evolve over
cosmic time, if they are coupled to the host properties.
So, here's what we're planning to do:

Write down a generic PGM for the above

Focus it on a possible simple implementation, anticipating having
access to posterior samples in both SN and host parameters.

Write out the probability theory corresponding to this model, and
determine good sampling options to pursue.

Design a small simulated dataset and analyze it using this model.

Comments welcome!
—
Reply to this email directly or view it on GitHub
https://github.com/kbarbary/snpgm/issues/13.

—
Reply to this email directly or view it on GitHub.

Kara's plan sounds good to me.
Regarding rate models, as @wmwv said, the A+B model captures the rough behavior (younger stars = higher SN rate) but isn't really physical. A better model is sn_rate(t) ~ t^-1 for t > 40 Myr stellar age. This is physically motivated (for the double degenerate model) and current measurements are consistent with it. In rates at least, there's not much evidence for "two populations" rather than a continuum.

That's interesting. Sounds like continuing to work on model comparison
statistics should be part of our plan.
With regard to the local vs global measurements: what's a good dataset in
which to investigate "semi-local" measurements, like the Sersic models we
expect from LSST? Has anyone been thinking about experiments like taking
nearby galaxies with local measurements and "observing" them at higher
redshift with PhoSim or some other simulation tool? Or do we already know
how broadband surface brightness at the SN position relates to local SFR?
On Tue, Jun 23, 2015 at 7:09 AM, Kyle Barbary<EMAIL_ADDRESS>wrote:

Kara's plan sounds good to me.
Regarding rate models, as @wmwv https://github.com/wmwv said, the A+B
model captures the rough behavior (younger stars = higher SN rate) but
isn't really physical. A better model is sn_rate(t) ~ t^-1 for t > 40 Myr
stellar age. This is physically motivated (for the double degenerate model)
and current measurements are consistent with it. In rates at least, there's
not much evidence for "two populations" rather than a continuum.
—
Reply to this email directly or view it on GitHub
https://github.com/kbarbary/snpgm/issues/13#issuecomment-114518458.


Compatible with github API error message

To solve this issue #130

Great. Should be merged IMHO.


If plot ranges cannot be determined use mpl autoscale view

This ensures that the xlim and ylim on a plot correspond to the actual data limits at all times, allowing calculations based on these values to be performed.

Apparently some Spikes plots are affected by this, will have to investigate that before this can be merged.

Ready to merge.

Not sure why the reference data is complaining but this is still ready to merge.

With the pr build passing, I'll merge it now.


Add some joystick functions to input. Enables manipulation of mapping…

…s at runtime
The following functions were added to Input:
add_joy_mapping(String mapping, bool update_existing) and remove_joy_mapping(String guid) for manipulation of gamepad mappings.
bool is_joy_known(int device) defaults to true on all platforms that don't use the remapping code. Otherwise, checks whether the device is mapped.
get_joy_guid(int device) returns the sdl2 device guid if using remapping, defaults to "Default Gamepad"
I also documented these and the other joystick related functions. doctool updated quite a bit of things unrelated to this, I hope that's not a problem.
Also I hope the implementation is ok. I added is_joy_known and get_joy_guid to OS for the default variants. Platforms that use the remapping then override these to call back to InputDefault

You can get my remapping project to get gamepad mappings without having to use external tools ;)
I didn't include it as the code is a bit ugly right now..
Also I don't know if that really fits the purpose of a demo, it's more of an utility.

uh was this closed before? I closed it by accident

No, it was not closed before that commit.


Not rending latex modifiers

Before reporting a bug

[x] Check common issues.
[ ] Check the bug is reproducible in the demo. If not, check KaTeX is up-to-date and installed correctly.
[x] Search for existing issues.

Describe the bug:
There is rendering issue when using katex and latex modifiers
(La)TeX code:
The code of (La)TeX you tried to render:
${\displaystyle |\psi \rangle =\alpha |0\rangle +\beta |1\rangle }$

Expected behavior:

from: latex.codecogs.com
Screenshots:

le of displaystyle is rendered :man_facepalming:
Environment (please complete the following information):

KaTeX Version: ^0.10.1 with showdown-katex
Device: Desktop
OS: Arch Linux
Browser: Any
Version: Any

showdown-katex can render either KaTeX or AsciiMath.  I can reproduce the faulty rendering on the showdown-katex demo page by submitting the string to an AsciiMath field instead of to a KaTeX field.

The same imput string, when submitted to a KaTeX field on the demo page, renders correctly.
I would speculate that your installation is set so that the $...$ delimiters are programmed to invoke AsciiMath instead of KaTeX.

So how do i fix it
the showdown-katex is using 0.10 version of katex

As @ronkok wrote above, the fix is likely to change the configuration in showdown-katex to use LaTeX mode instead of ASCII math mode. If it's not clear from the documentation, perhaps you should open an issue for showdown-katex. We know KaTeX but unfortunately not as much about showdown-katex.

@ronkok
    {
      displayMode: true,
      throwOnError: false, // allows katex to fail silently
      errorColor: '#ff0000',
      delimiters: [
        { left: "$", right: "$", display: false }, // katex default
        { left: '~', right: '~', display: false, asciimath: true },
      ],
    }

This is my configuration

@tbhaxor You need to raise this issue in a showdown-katex forum, not here. We don't know showdown-katex here and we can't tell if such a configuration is well formed or not.
If you get to the point where you have successfully invoked KaTeX and not AsciiMath, and then KaTeX is still acting improperly, then please come back and raise a new KaTeX issue here.


State file needs a real database

Currently the entire state history is stored in memory. Thus, bosun's memory usage is theoretically unbounded. To fix this, we must begin storing the state file in some sort of go-based DB.

We moved to BoltDB so this is "fixed" although long term we are considering something like ledisDB so we can use redis and get replication. But we did move to an actual go based DB.


.Net SDK release v3.0.0

This is a tracking issue for the changes going into the next major version release v3.0.0:
1.  Fixed a typo in Reservoir [PR #62]:
Changes the class and methods from Reservior to Reservoir.
2. Update .net framework version to 4.5.2 [PR #101]:
Current SDK targets net45 as the TargetFramework for its projects. Since, the support for .Net Framework 4, 4.5 and 4.5.1 has officially ended, we plan to target the net452 version. This would break customers who are using the SDK with applications targeting net45.
3. Drop dependency on AWS SDK for .Net:
Current SDK depends on AWSSDK.Core and AWSSDK.XRay to fetch sampling configuration from service. This adds a fair amount of overhead to the SDK. We aim to make the X-Ray SDK lightweight with minimal footprint on customer applications. For the v3.0, the plan is to replace AWS SDK with HTTP calls as the mechanism to fetch sampling configurations.
4. Update IPlugin interface to use async methods:
Please refer to this PR for more details.

How about .NET Core 3.1 (LTS) support?
Microsoft is soon to abandon .NET Core 3.0.

Re: Item 4: Update IPlugin interface to use async methods.
I don't see how this will be possible.  I took a look into what it would take, and it leads down to AWSXRayRecorderBuilder().Build(), which is a core piece of initialization, and would require making much of the library async, including places where it is probably not intended.  I see two possible paths, either A) Require initialization to be performed async (don't use a static initializer), or B) Keep NetCore3.1 and lower using the WebRequest route and net5.0 and higher can utilize the sync method on HttpClient.  Any thoughts?

Re: Item 3: Drop dependency for AWS SDK for .Net.
I'm also not seeing how this will be possible for several reasons:

Extensive use of AWS SDK's Internal Logger class. (Our own logger could be implemented to decouple us if we strongly desired to remove the sdk)
RegisterXRayForAllServices and XRayPipelineCustomizer utilize AmazonServiceClient to register services from the AWS SDK.  This functionality is, in my opinion a very desirable feature of this library.  See #241 for an alternative change that would strongly type the constraint for RegisterXRay to enforce that only Amazon Service Client Interfaces are passed.


Use slices instead of u32 lengths and pointers?

Are there any plans in the future to support this? as to improve the safety and the ergonomics of the API just like how ash does it.

There are two aspects to this feature:

slices in function prototypes
slices in structs

The former is already done in vulkan-zig and works fine. The latter requires re-writing structs, and while the general performance overhead of that would probably be not too bad (though youd be copying a bunch of bytes every time and probably better to avoid that), the real problem is for pNext chains. That either requires dynamic allocation or figuring out the max required static size by allocating all possible extension chains on the stack. It is possible for sure, but originally I decided against it because of the points above. Note that a wrapper type is not possible here due to abi layout constraints.

Do you have any idea on how rust binding ash manages that?

That wouldn't be a fully general solution, though, as sometimes two fields are specified by one length, or a length is nested somewhere in another struct rather than being adjacent to the field.

What if we just also generated methods named after the ptr + length pair, which returns a slice?

That wouldn't be a fully general solution, though, as sometimes two fields are specified by one length, or a length is nested somewhere in another struct rather than being adjacent to the field.

most of the related ptr/len pairs have the same name, if we strip out the prefix and the suffix. for the other edge cases, we can either leave them unchanged or we can manually work around those.

What if we just also generated methods named after the ptr + length pair, which returns a slice?

That would work, but how often do you read from a parameter struct?

Maybe instead of using zig's built in pointers (and therefore having to copy to convert) there could be a wrapper struct of some kind ie.

Im not sure that this will abi compatible.

Do you have any idea on how rust binding ash manages that?

It doesnt. Ash uses builders to construct a struct and perform the conversion but id like to avoid that. See for example InstanceCreateInfoBuilder#enabled_layer_names.

This has been quiet for a while. I'm gonna make the decision to not do this for now, to avoid rewriting the struct.


Added 9 Icons

Closes https://github.com/tandpfun/skill-icons/issues/152
Closes https://github.com/tandpfun/skill-icons/issues/252
Closes https://github.com/tandpfun/skill-icons/issues/471
Closes https://github.com/tandpfun/skill-icons/issues/487

@Juan-Carlos-Estevez-Vargas What do you use to make these? Figma?

@Juan-Carlos-Estevez-Vargas What do you use to make these? Figma?

No, I used Vect

Hello, this PR will not be accepted, right?

Can this PR be merged? I would like to use SQL Server icon


Ferdi (latest 5.5.0) on Mac Mini m1 with Big Sur > MS Teams settings missing

Steps to reproduce
Add Teams to Ferdi
Accept Microphone / Video change for Ferdi
Make video/audio call
Expected behavior
Normally you can change the webcam and/or mic settings. Also you can enable webcam filters (like blurred or diff. background)
Actual behavior
There are NO settings (no devices tab) what so ever for cam/mic and thus also no Webcamfilters...
If you open the regular teams app (next to Ferdi.. there you have everything right away..
ENVIRONMENT
Ferdi 5.5.0
Machine: Mac Mini M1 with Big Sur 11.4
WHAT WE SHOULD SEE:

IN FERDI:

@SeBr28 - Ferdi 5.5 is really old. Please try running the latest nightlies from here.

Thank you for the quick replay.. however.. hmm..
i see in the nightly builds.. Three versions for Mac.. As it is a M1 chip i took the Arm64 - Mac but also the other just for MAC won't open. It says I have no permission to open the App and have to ask for permission..
Sorry if I seem noob..

@SeBr28 - please see the other nightlies where I have linked to a blog post on the security that has been implemented by apple in catalina and newer OSes.
If you are on a Mac M1 - you need the Mac M1 builds

I have added the link to the article in this release as well.

It was a bit of a search (pretty new at Apple) but found it, got it working.. and now on the Arm64 version.
The issue remains however for the teams settings...

If the teams settings dont appear - then its a limitation of MS Teams - in how they expose settings for a browser-based usage of their service. Ferdi will not be able to do much in this case im afraid.
The Franz TODOs page is WIP right now to rebrand to Ferdi.

Thnx for the very quick replies... will see what comes... :) thank you


Download links and nextpush and new instructions

Add NextPush distributor page
Change Links to badges

Closes #39 Closes #40

I also made changes that I hope will be slightly more helpful to explain UP to new users. Comments are welcome :)

Also, @p1gp1g do you want to mark NextPush WIP here as it is on the repos? I can change that in the PR if so, or if not, no changes are needed for that

Also, @p1gp1g do you want to mark NextPush WIP here as it is on the repos? I can change that in the PR if so, or if not, no changes are needed for that

I've marked them beta. We can tell it is in beta on the doc and removing it when it will be tested enough

@p1gp1g it should be ready to merge now


Name of input file missing in constructed pandoc command

the outputted and constructed command from spandoc (in the console) should output a working pandoc command (for executing at the command-line-interface), this is not the case for windows nor for linux

fixed with 4ca47ac


Replace go get, further fix up error handling/reporting

Fixes another issue reported in #186 where an entire package is missing.
Don't bother restoring/downloading if already done.

@ironcladlou This should solve the aws-sdk issue .... but there are some consistency issues with github.com/openshift/origin Godeps.json file.
Some of them are duplicate shas for the same repo, one for package A another for package B in the same repo:

https://github.com/openshift/origin/blob/0d4de0b1801e8c58a28c8bcd9d12b5d0c46de1cb/Godeps/Godeps.json#L744-L753
https://github.com/openshift/origin/blob/0d4de0b1801e8c58a28c8bcd9d12b5d0c46de1cb/Godeps/Godeps.json#L876-L895

Some appear to be shas from forks of the repos that are not on the canonical master (see #366).
There is at least once case where a recorded package is missing (looks to be in a PR) but is also recorded twice with 2 different shas.

/cc @eparis

@eparis confirmed that this solves the docker/timeutils issues for you as well.

https://github.com/docker/docker/pull/18685 for reference

@freeformz Thanks for the analysis- I'm still working my way through the other problems, but was focused just on the sub package import issue.
I just verified that this commit fixes #186. Thanks again!

I can confirm this fixed my issue!

This patch cause an issue for me.
My build script is like this:
godep restore
go get
go build

example Godeps/Godeps.json:
{
        "GoVersion": "go1.5",
        "Deps": [
                {
                        "ImportPath": "google.golang.org/appengine",
                        "Rev": "58c0e2a2044a8d1abd8dd1d97939cd74497d0806"
                }
        ]
}

Then run build script in docker golang:1.5 (clean environment for GOPATH)
Error message:
godep: Dep (google.golang.org/appengine) restored, but was unable to load it with error:
    Unable to find dependent package github.com/golang/protobuf/proto in context of /go/src/google.golang.org/appengine
godep:  This may be because the dependencies were saved with an older version of godep (< v33).
godep:  Try `go get github.com/golang/protobuf/proto`. Then `godep save` to update deps.
godep: Error checking some deps.

But it worked well on v37 tools/godep@03595abb97a99e98133f9390b9c11e0aecf5f2e8
PS:
Why I don't list all dependence in Godeps.json ?
When I use some package, I don't care what it depend on.
I just use the top level API.
So if I write to Godeps.json, I should update often.
And all dependence in Godeps.json will cause it is hard to understand which package is necessary.

@tsaikd What version of godep are you using? and if it's not the latest, is it still an issue with the latest version?

@freeformz I test from v45 to v38, all failed.
But v37 work.
I write a test Dockerfile if you want to verify
https://gist.github.com/tsaikd/710fa81c9ca409e7a62e
$ godep version && godep restore
godep v45 (darwin/amd64/go1.5.2)
godep: Dep (google.golang.org/appengine) restored, but was unable to load it with error:
	Package (github.com/golang/protobuf/proto) not found
godep: Error checking some deps.

$ godep version && godep restore
godep v44 (darwin/amd64/go1.5.2)
godep: Dep (google.golang.org/appengine) restored, but was unable to load it with error:
	Package (github.com/golang/protobuf/proto) not found
godep: Error checking some deps.

$ godep version && godep restore
godep v43 (darwin/amd64/go1.5.2)
godep: Dep (google.golang.org/appengine) restored, but was unable to load it with error:
	Package (github.com/golang/protobuf/proto) not found
godep: Error checking some deps.

$ godep version && godep restore
godep v42 (darwin/amd64/go1.5.2)
godep: Dep (google.golang.org/appengine) restored, but was unable to load it with error:
	Package (github.com/golang/protobuf/proto) not found
godep: Error checking some deps.

$ godep version && godep restore
godep v41 (darwin/amd64/go1.5.2)
godep: Dep (google.golang.org/appengine) restored, but was unable to load it with error:
	Unable to find dependent package github.com/golang/protobuf/proto in context of /Users/tsaikd/project/go/src/google.golang.org/appengine
godep: 	This may be because the dependencies were saved with an older version of godep (< v33).
godep: 	Try `go get github.com/golang/protobuf/proto`. Then `godep save` to update deps.
godep: Error checking some deps.

$ godep version && godep restore
godep v40 (darwin/amd64/go1.5.2)
godep: Dep (google.golang.org/appengine) restored, but was unable to load it with error:
	Unable to find dependent package github.com/golang/protobuf/proto in context of /Users/tsaikd/project/go/src/google.golang.org/appengine
godep: 	This may be because the dependencies were saved with an older version of godep (< v33).
godep: 	Try `go get github.com/golang/protobuf/proto`. Then `godep save` to update deps.
godep: Error checking some deps.

$ godep version && godep restore
godep v39 (darwin/amd64/go1.5.2)
godep: Dep (google.golang.org/appengine) restored, but was unable to load it with error:
	Unable to find dependent package github.com/golang/protobuf/proto in context of /Users/tsaikd/project/go/src/google.golang.org/appengine
godep: 	This may be because the dependencies were saved with an older version of godep (< v33).
godep: 	Try `go get github.com/golang/protobuf/proto`. Then `godep save` to update deps.
godep: Error checking some deps.

$ godep version && godep restore
godep v38 (darwin/amd64/go1.5.2)
godep: Dep (google.golang.org/appengine) restored, but was unable to load it with error:
	Unable to find dependent package github.com/golang/protobuf/proto in context of /Users/tsaikd/project/go/src/google.golang.org/appengine
godep: 	This may be because the dependencies were saved with an older version of godep (< v33).
godep: 	Try `go get github.com/golang/protobuf/proto`. Then `godep save` to update deps.
godep: Error checking some deps.

$ godep version && godep restore
godep v37 (darwin/amd64/go1.5.2)

AFAICT your Godeps.json file is incomplete?
If I make a simple app that uses the appengine package my generated Godeps.json file looks like this:
$ cat Godeps/Godeps.json
{
	"ImportPath": "github.com/freeformz/t365",
	"GoVersion": "go1.5",
	"Deps": [
		{
			"ImportPath": "github.com/golang/protobuf/proto",
			"Rev": "68415e7123da32b07eab49c96d2c4d6158360e9b"
		},
		{
			"ImportPath": "golang.org/x/net/context",
			"Rev": "0ab009005dc16437045369e823a35d1af6232b69"
		},
		{
			"ImportPath": "google.golang.org/appengine",
			"Rev": "54bf9150c922186bfc45a00bf9dfcb91a5063275"
		}
	]
}

Also: You do not need to do a go get after a godep restore as long as all of your deps are recorded (which godep save does).

For example, when I write a nodejs app, I just record necessary dependent packages that I used. Npm will fetch dependencies recursively. In appengine case, why I need to record protobuf? I don't use it. It's the second level dependency. About go get tool, it even forbid to import unused packages.
If my use case does not fit the philosophy, just igore it. I can choice the v37 in my project. It works well.

Go is not node. The google.golang.org/appengine package uses github.com/golang/protobuf/proto and w/o it the code won't build. Godeps is about making builds repeatable. W/o that code vendored there may be times where I can build your code (I've seen npm go down and cause all sorts of problems where users could no longer deploy).

Still appears to be failing with godep version 51.
$ godep version
godep v51 (linux/amd64/go1.5.2)

$ godep restore
godep: error restoring dep (github.com/prometheus/client_golang/model): cannot find package "github.com/prometheus/client_golang/model" in any of:
	/usr/local/go/src/github.com/prometheus/client_golang/model (from $GOROOT)
	/highland/gocode/src/github.com/prometheus/client_golang/model (from $GOPATH)
godep: Error restoring some deps. Aborting check.

{
        "ImportPath": "github.com/GoogleCloudPlatform/heapster",
        "GoVersion": "go1.4.2",
        "Packages": [
                "./..."
        ],
        "Deps": [
                {
                        "ImportPath": "github.com/prometheus/client_golang/model",
                        "Comment": "0.4.0-1-g692492e",
                        "Rev": "692492e54b553a81013254cc1fba4b6dd76fad30"
                },
                {
                        "ImportPath": "github.com/prometheus/client_golang/prometheus",
                        "Comment": "0.4.0-1-g692492e",
                        "Rev": "692492e54b553a81013254cc1fba4b6dd76fad30"
                },
                {
                        "ImportPath": "github.com/prometheus/client_golang/text",
                        "Comment": "0.4.0-1-g692492e",
                        "Rev": "692492e54b553a81013254cc1fba4b6dd76fad30"
                },
                {
                        "ImportPath": "github.com/prometheus/client_model/go",
                        "Comment": "model-0.0.2-12-gfa8ad6f",
                        "Rev": "fa8ad6fec33561be4280a8f0514318c79d7f6cb6"
                },
                {
                        "ImportPath": "github.com/prometheus/procfs",
                        "Rev": "490cc6eb5fa45bf8a8b7b73c8bc82a8160e8531d"
                }
        ]
}

@DreadPirateShawn I was able to reproduce this by synthetically checking out the affected package and running restore. Will look more: godep: error restoring dep (github.com/prometheus/client_golang/model): cannot find package "github.com/prometheus/client_golang/model" in any of:
/usr/local/go/src/github.com/prometheus/client_golang/model (from $GOROOT)
/Users/emuller/go2/src/github.com/prometheus/client_golang/model (from $GOPATH)


[Example] Fix package.json

In your example project, your package.json has a trailing comma for your final dependency listed, this gives an error when running npm install

Thanks!


File format examples use > 200 MB

We currently have lots of file format examples in the repository. I do think we need those for testing purposes and there will likely be more than those in the future.

We could simply ignore and accept that these files need to be in the repository.
We could move them to a separate (non-version controlled) location and download during testing.
We could try to find smaller example files or reduce the number of grid points. (We can only do this for files that we can actually write).

Any suggestions/thoughts?

Where is the size of the repository causing problems ?
a. Are there limitations on github ?
b. Or is it the time for installation ?
For b the most elegant would be an option to exclude directories on pypi.
I speculate that exists for the installation but not for the download.
Solution 2. would then be the simplest alternative. I suggest to put the files in a public github repository in ContactEngineering. We just need to clone it to obtain the test files.

Github has a 10 GB limit I think. As long as we don't hit that we are good.
PyPI is much smaller but I think we are already excluding the file format examples for that.

Hm, solution 2 adds more complexity. We will have to make always ensure that the file repo and the code repo are in sync. One problem is, that we could have different branches on the code side, but only one file repo.
I think this might cause problems in the future. Also we cannot run the tests without being online.
And if we're online, it will always take time and bandwidth to download the files on every test run.
Probably this will get on our nerves..
I prefer solution 3: I think having small files in the repo is no problem.
We should make the simplest example files possible. This also can help for other tests (analysis?) and speeds up testing.
If this is not possible, we should ask people with access to instruments to provide as small sample files.

For solution 3 we need to ask people with access to the instruments for small files.

We could pull file format examples from surfaces published at https://contact.engineering/ - is there an automated way to do that, @mcrot

Yes, e.g.:
import requests
import io
from SurfaceTopography.Container import read_container

def read_published_container(publication_url):
    # First get the page for the publication in order
    # to get the download URL
    publication_response = requests.get(publication_url)
    download_url = publication_response.url + "download/"

    # Then download and read container
    container_response = requests.get(download_url)
    container_file = io.BytesIO(container_response.content)
    return read_container(container_file)

This currently works for containers with binary data files, e.g.:
read_published_container("https://contact.engineering/go/btpax/")

This currently doesn't work if a text file is included in the container, e.g.:
In [86]: read_published_container("https://contact.engineering/go/25t52/")
/home/michael/work/imtek/SurfaceTopography/SurfaceTopography/Container/IO.py:70: YAMLLoadWarning: calling yaml.load() without Loader=... is deprecated, as the default Loader is unsafe. Please read https://msg.pyyaml.org/load for full details.
  meta = yaml.load(z.open('meta.yml'))
---------------------------------------------------------------------------
CannotDetectFileFormat                    Traceback (most recent call last)
<ipython-input-86-a71139bcb657> in <module>
----> 1 read_published_container("https://contact.engineering/go/25t52/")

~/work/imtek/SurfaceTopography/pull_surface_container.py in read_published_container(publication_url)
     17     # return container_file
     18
---> 19     return read_container(container_file)
     20
     21

~/work/imtek/SurfaceTopography/SurfaceTopography/Container/IO.py in read_container(fn, datafile_keys)
     84                 if datafile_key is None:
     85                     raise ValueError('Could not detect data file.')
---> 86                 t = read_topography(
     87                     z.open(datafiles[datafile_key]),
     88                     physical_sizes=physical_sizes,

~/work/imtek/SurfaceTopography/SurfaceTopography/IO/__init__.py in read_topography(fn, format, communicator, **kwargs)
    240         The object containing the actual topography data.
    241     """
--> 242     with open_topography(fn, format=format,
    243                          communicator=communicator) as reader:
    244         t = reader.topography(**kwargs)

~/work/imtek/SurfaceTopography/SurfaceTopography/IO/__init__.py in open_topography(fobj, format, communicator)
    189                     # has to be set back to the top of the file
    190                     fobj.seek(0)
--> 191         raise CannotDetectFileFormat(msg)
    192     else:
    193         if format not in lookup_reader_by_format.keys():

CannotDetectFileFormat: tried XYZReader:
 Expected two or three columns for topography that is a list of positions and heights.

tried AscReader:
 cannot use a string pattern on a bytes-like object
[...]

Nice... can we stick this function into SurfaceTopography?

Yes, of course .. looks already similar to read_container. We should add tests with both, binary and text data files.
I can add it.


[please test] BYOK with ollama

With the ollama project it's easy to host our own AI models.
You can set up bring-your-own-key (BYOK) to connect to ollama server, and see if you can use StarCoder2 for code completion, llama models for chat.
Does it work at all? What we need to fix to make it better?

@olegklimov I would like to take this up. Can you please some docs/example of how this can done? Do we need to test the integration here or make changes as well to make it work?

Oh, here https://docs.refact.ai/byok/ you can test if we have documentation that is any good :D

Hi @pardeep-singh
Did you had a look over Oleg's approach?


Add Images to ArchiMate Views

You can already do it in the "Canvas" view. Would users like it for ArchiMate Views as well? Some users were asking for additional illustrations.
So far I kept it restricted to the Canvas View in case it didn't work out. But I could easily open up the functionality to ArchiMate views.
Caveats are the same as including an image in a Canvas view:

The *.archimate file will be in an archive format (zipped) to contain all the assets together
Saving and loading is a bit slower Opening a View with images in it is a bit slower
Use reasonable size images, not your whole Flickr family holiday collection.

Understand why image shouldn't be treated as separate case but note should allow images and more robust store say word docs etc.  Actually it goes to reference material used to construct model.

Hi Peter,
You can add images on Archimate views using my specialization plugin.
Best regards
Hervé

I've used Sparx to store rtf docs in the model data base and found it better to add links in the model to content in the corporate Sharepoint. Ie, program and project documentation.
And some of that program and project documentation in Sharepoint contains diagrams and text mastered in an Archimate model. And the content of an Archimate model can be replicated from the tool in HTML, which can be loaded straight into Sharepoint.
Options.
Jon McLeod
Director
McLeod Consultancy Pty Ltd
GPO Box 813, Canberra ACT 2601 Australia
Silver Member, The Open Group

On 14 Nov 2017, at 10:55, Peter Prib<EMAIL_ADDRESS>wrote:
Understand why image shouldn't be treated as separate case but note should allow images and more robust store say word docs etc. Actually it goes to reference material used to construct model.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Hi Hervé
Tried it out.  Doesn't work as described.  Changes icon on models but fails to materialize on view and has odd behavior on properties where by it overrides all tabs.

Hi Peter,
Could you please open an issue on the plugin site, detailing what you do and I will help you make the plugin work.
Best regards
Hervé

I would like to add images into archimate views. Enterprise architecture is communicated with business people and it could help.
To add images to exported diagrams is uncomfortable, every small change in diagram implies manual (=error prone) work. Thanks.

90% of my archi schemes are redrawn by PMO : they need cool pictures etc. When a manager pay $$$$ to Oracle, he wants at least the logo on the IT view.
I think comments with pictures would be a nice first step (not to alter the model).
Or adding a picture on a component, and having a global View setting to show/hide customizations.

Hi,
This is an old feature request... FWIW, I now think that this should be possible and could help in several occasions. I've described my ideas for such feature in this page.
@gitfirenz : in fact you can already do it through @herve91's specialization plugin. You will not be able (to my knowledge) to add images to visual notes, but you will be able to override top right icons on elements, meaning that you can easily show a nice oracle icon on services, nodes or artifacts related to such technology.

In addition, I bet that even with such feature in Archi, you'll still see PMO (or others) redrawing things in PowerPoint (or similar). This is very often motivated by the fact that they want to be able to change the layout and visual to better support their message.
On solution that we are exploring is to create a plugin (or a jArchi script) that would export a view to a Visio or PowerPoint document using real shapes and connectors, not as a single image. Of course this would not be 100% similar to the original ArchiMate view in Archi, but would be similar enough to be usable (and later customized by PMO).

Thanks for you quick reply !
I'll try specialization plugin very soon. I have to check how it mix with collaboration plugin.
Exporting to PowerPoint would be nice, you're right.
There is a big challenge in sharing, publishing, socializing our beloved schemas in enterprise. I'm expecting a lot from the next versions (according to the roadmap) :)

The specialization plugin will not interfere with the collaboration plugin as it does not change the objects seen by the specialization plugin.
It sets properties on elements and relationships and use those properties to determine if icons and labels should be changed.
Hope this helps


update parking_lot

Nope it’s only what I came across aswell

Thanks, I need to setup dependabot to remind me of any dep upgrade.


[libtins] WIP: Fix static build

closes #11904
To do:

[x] Fix Packet.lib linking
[x] Add TINS_STATIC definition
[ ] Fix Version.lib linking
[ ] Packet.lib(Packet32.obj) : error LNK2005: LoadLibrarySafe already defined in wpcap.lib(pcap-tc.obj)
[ ] Packet.lib(Packet32.obj) : error LNK2005: DllMain already defined in wpcap.lib(pcap-win32.obj)

For now, I have no idea how to fix LNK2005. winpcap seems to be designed to use as DLL.
Current output with this PR:
2>Packet.lib(Packet32.obj) : error LNK2005: _DllMain@12 already defined in wpcap.lib(pcap-win32.obj)
2>Packet.lib(Packet32.obj) : error LNK2005: _LoadLibrarySafe already defined in wpcap.lib(pcap-tc.obj)
2>LINK : warning LNK4098: defaultlib 'LIBCMT' conflicts with use of other libs; use /NODEFAULTLIB:library
2>Packet.lib(Packet32.obj) : error LNK2019: unresolved external symbol _GetFileVersionInfoSizeW@8 referenced in function _PacketGetFileVersion
2>Packet.lib(Packet32.obj) : error LNK2019: unresolved external symbol _GetFileVersionInfoW@16 referenced in function _PacketGetFileVersion
2>Packet.lib(Packet32.obj) : error LNK2019: unresolved external symbol _VerQueryValueW@16 referenced in function _PacketGetFileVersion
2>C:\projects\libtins-test\build\Debug\main.exe : fatal error LNK1120: 3 unresolved externals

Thanks for your PR! Can you please correct the Version field in the CONTROL file? You can get more information from maintainer-guide.md.

It might help to add
vcpkg_check_linkage(ONLY_DYNAMIC_LIBRARY)

to winpcap. This will prevent it from being used with the static CRT, but if a user defined a static lib + dynamic CRT triplet, it should work.

Pinging @Pospelove for response, can you address the review suggestions? Thanks!

I am not involved currently, so closing this for now.


Rename minos.networks.rest as minos.networks.http and minos.networks.RestBuilder as minos.networks.HttpHandler

As we decided during the last workshop, Rest is a better word than Http in this case.


Add Mauritius public holidays

🎁 Adds support for Mauritius
🐞 fix Cycle definitions for when they do not contain a validFrom definition. Previously it would just silently fall back to EVERY_YEAR behavior. Now it calculates backwards, if only a validTo is given and throw an exception if neither is given.
However, it is missing translations at the moment, as I did not know how to translate them.
Also as long as there is no support for the Tamil and/or Hindu Calendar (see #282), one has to manually define many holidays, which is very time consuming and error prone.
Additionally as long as there is no support for the Chinese Calendar (see #285), one has to manually define many holidays, which is very time consuming and error prone.
Please merge this PR AFTER https://github.com/focus-shift/jollyday/pull/277.

Good Morning @XSpielinbox, could you rebase your branch against the main? And maybe split the other pull requests #283 and #288 into branches? That would be easier to merge and maintain.

🐞 fix Cycle definitions for when they do not contain a validFrom definition. Previously it would just silently fall back to EVERY_YEAR behavior. Now it calculates backwards, if only a validTo is given and throw an exception if neither is given.

Can you give me an example?
And if both are not given, then it should not throw an exception, because this is legit if there was not starting or ending year of a public holiday.

@XSpielinbox can you translate it with https://www.deepl.com/translator for the other languages?

Can you give me an example?

I added a test case for that, see FixedParserTest.java. A need for that can for example be seen in the Mauritius Holiday Id Al Fitr.
I noticed this shortcoming in the making of the previous holidays already (with that fix, some things in the already merged holidays could be solved more elegantly), but with Mauritius I saw no way around this anymore.
The issue is that in case of a "only every x years" cycle with only a validTo and no validFrom the limited.validFrom() != null would be false, therefore the if statement would not be taken and one would completely fall trough the switch statement hitting the return true; at the end of the method. This then results in the same behavior as if EVERY_YEAR would have been given, thus making it unusable. When I have a holiday that is celebrated since e.g. 2012 and takes place every three years for example I can easily do that, but when it was celebrated at last 2012, but before that only every three years I cannot specify validFrom, because it was celebrated all along before that. But like I can calculate three years cycles starting from 2012 it is trivial to calculate three year cycles ending 2012. But when true is returned, every year before 2012 is regarded as a holiday, which is not what I want.

And if both are not given, then it should not throw an exception, because this is legit if there was not starting or ending year of a public holiday.

Yes, of course it is legitimate to have no start or end year. I did not change that. After all all the other holidays work as before and all the tests still pass. The problem is that for holidays that only occur every x years with x > 1, I don't know a way to determine which cycle is ment without any additional information. If I say every year, it is trivial. If I say every odd or every even year I also don't need a start or end date to say whether this year is in the cycle or not. But when I say every 4 years for example, how do I know whether the years should be ..., 2012, 2016, 2020, ... or ..., 2013, 2017, 2021, ... or 2014, 2018, 2022, ... or 2015, 2019, 2023, ... An additional parameter is needed to determine it fully: I could for example say, every 4 years, including the year 2013. Then it would be clear that the second option must be ment. When I say "every year dividable though 4", it is also exactly determined again: It must be the first of the 4 options, but the schema says every="FOUR_YEARS" and at least I would find it confusing, if it would assume that it assumes every year dividable though 4. To explicitly fail in this case (for which I don't see a use case anyway at the moment) and not silently fall back to something like EVERY_YEAR when I explicitly state FOUR_YEARS, just because I did not give a start or end date, I throw the exception. In case you have some good default for this case or have some amazing way to determine which possible cycle was ment, one can just replace the exception with that. Also if in some future additional support for example every though 7 dividable year would be added (for whatever reason), one could just replace the exception with handling for these additional cases. You can consider the Exception just as a place holder for now to avoid some (in my opinion not sane) default.
If I state EVERY_YEAR, ODD_YEARS or EVEN_YEARS, the code does not check the existence of validFrom or validTo as it did not before. After all it is fully determined, nevertheless and so now Exception will be thrown. Everything will just work as expected and just like before.

Hey @XSpielinbox,
great work! It took me some time today to unterstand the code better. With your information and with the given code I totally understand what you mean and I like your implementation a lot.
I improved the code a little bit to clear things up and added tests for the ValidCycle class.

@XSpielinbox can you translate it with https://www.deepl.com/translator for the other languages?

For sure, I can do that. However, you merged it already. Shall I just make a new PR for that? Also: you already made some changes to the translations - what were your thoughts behind that?

Hey, now I know what I forgot. I translated most of the new holidays. Thanks for your offer. I need to look after the last translations, but that is not a big deal.
There where some issues with the encoding that I just fixed. So now that should be ok for the moment

Hey, now I know what I forgot. I wanted to write you that the translations are done. I translated most of the new holidays. I need to look after the last translations, but that is not a big deal.
There where some issues with the encoding that I just fixed. So now that should be ok for the moment

Ah, ok. Thank you. Yes, the German and Greek translations seem finished. I noticed a typo in the German translation however: It should be "Tag der Unabhängigkeit und der Republik". There is also a typo in the javadoc of the new method in the ValidCycle.java.
The Dutch, Portuguese and Swedish translation are still completely missing, however. Shall I add them or will you do that too?

I will do that. I want to take a deeper look into the translations. And maybe provide some h ader for the utf8 encoding if that is possible


Autorest Regen Preview to 3.0.0-alpha.20230918.5 by Pan Shao from refs/pull/3671/head

Triggered from https://github.com/Azure/autorest.csharp/pull/3671

API change check
API changes are not detected in this pull request.


python3Packages.mne-python: 1.3.1 -> 1.5.1

Description of changes
Update python3Packages.mne-python to 1.5.1 and add missing dependencies:

h5io
pytest-harvest
pymatreader

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

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

8 packages built:

python310Packages.h5io
python310Packages.mne-python
python310Packages.pymatreader
python310Packages.pytest-harvest
python311Packages.h5io
python311Packages.mne-python
python311Packages.pymatreader
python311Packages.pytest-harvest

@ofborg build python310Packages.mne-python python311Packages.mne-python

Probably bumped in python-updates branch.
Thanks.

It seems like the bump broke the build, so we will need to merge these changes. However, there are a couple test errors currently and the tests are also an absolute beast to run, so I suggest potentially disabling them or moving to passthru if possible for now.

Done in https://github.com/NixOS/nixpkgs/commit/fb6ac4804821d30d38629324f9be1e5e3575480f.


Transient problem with bgpstreamlive and ris

Describe the bug
Using ris and bgpstreamlive leads to crashing of the bgpstreamlive process.
Affected Component(s)

[X] Back-End (Database, Microservices, Containers, etc)
[ ] Front-End (Flask, API, etc)
[ ] Docs
[ ] Build System

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
System (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Connected to https://github.com/CAIDA/libbgpstream/issues/166

https://github.com/slowr/bgpstream-redis/pull/2


Could not install pages and numbers, keynote

Also, slack...

It was caused by timeout of login processing to AppStore.


separate the refactoring codes and testing codes into different files

https://github.com/miniatureseal/dsci-310-group-10-GCC/pull/82


Ref missing?

👋
Didn't something change with how we can attach a ref to this package? I noticed in 6.1.0 that I can attach a ref, but in 7.0.3 it can't find the element.

Could you share your code? No idea how do you try to attach it.

Hey,
Thanks for the quick response. I created a quick demo on https://codesandbox.io/s/4r817movz9
If you look in src/index.js I've been able to use __rootDOMNode but it doesn't appear to be available in recent versions. Is there a better way to grab the actual DOM behind this package? Thanks!

You just have to use <Textarea inputRef={el => (this.textarea = el)} />. _rootDOMNode was never part of the public API (which is denoted by _, common convention for private things) and it got renamed to _ref in v7.
Either way, please just use inputRef instead of ref for this component.

Great, thanks!