Reading direct continuous

Hello ,
I have problem using direct real time , so now can I read direct messages with creating a loop and requesting to get all inbox messages ? or this will raise error?
thanks

See #845


Update various dependencies

Deliberately not yet bumping Gradle to the just released version 8, but only to the most current 7.x version (7.6.1).
Also adding .gitattributes with EOL settings for Windows cmd and bat files.

Thankyou so much! This looks great


Adding background knowledge using FCI

Hi. I am using fci using the background knowledge. I have a dataframe with 287 features.
I'm doing this
cg_without_background_knowledge = fci(merged_data1.to_numpy(), node_names=merged_data1.columns).
The output of this is a tuple of two which has graphs and edges.
I then get the nodes in this way:
nodes = cg_without_background_knowledge[0].get_nodes()
I print the node names as:
for node in cg_without_background_knowledge[0].nodes:
    print(node.get_name())

The output of this is X1, X2,....., X287. I want to add background knowledge. I tried the following methods
##1)
bk = BackgroundKnowledge()
for i in range(len(node_pairs)):
     bk.add_forbidden_by_node(GraphNode(node_pairs[i][0]), GraphNode(node_pairs[i][1]))

##2)
bk = BackgroundKnowledge()
for i in range(len(nodes_forbidden)):
    bk.add_forbidden_by_node(**nodes[nodes_forbidden[i][0]], nodes[nodes_forbidden[i][1]]**)

where node_pairs=[('feature_1', 'feature_2') ,('feature_x', 'feature_y' ).....] &
nodes_forbidden=[0, 7), (0, 14), (0, 21),(0, 28),....].
When I rerun the fci algorithm as
G, edges = fci(merged_data1.to_numpy(), background_knowledge=bk, node_names=merged_data1.columns)
and when I check the G and I can see that there is still a connection between nodes[0], nodes[7] which I try to forbid in the background knowledge. If there is a link between nodes 0 & 7, I do not want it to be like nodes[0] -> nodes[7]. However, the other way is fine but I do not see the case. I still have connection as nodes[0] -> nodes[7].
My question to you is that how can I know what X1, X2, ....X287 maps to according to my dataset? I am not getting the names of the node as per my data in spite of passing the parameter node_names =  merged_data1.columns.
Please help!

I further investigated in the final graph G that I got from FCI.
a=[]
for i in range(len(nodes_forbidden)):
    if (G.get_directed_edge(nodes[nodes_forbidden[i][0]], nodes[nodes_forbidden[i][1]])) is not None:
        print(nodes_forbidden[i])
        a.append(nodes_forbidden[i])

Length of the forbidden node pairs provided in nodes_forbidden is 40180. And from the above code in this comment len(a) =418, which means there are 418 edges that I prohibited using bk.add_forbidden_by_node method still exists. Could you please tell me why this is happening and how can I resolve it?
I am not understanding the cause of the problem at the first place.

Thanks for reporting. If you would like to, could you please send us a minimal reproducing example of this issue (perhaps via email: yujiazh@cmu.edu)?


Parsing "IEEE Std 802.5r and IEEE 802.5j, 1998 Edition (ISO/IEC 8802-5:1998/Amd.1)"

@ronaldtse I'm trying to parse this identifiers, for me it looks like identifier with 2 dual-PubIDs, but I believe I could be wrong.
"IEEE Std 802.5r and IEEE 802.5j, 1998 Edition (ISO/IEC 8802-5:1998/Amd.1)"
What is strange here it represend 2 dual-PubIDs with different format (first using " and " and second one use identifier inside brackets).
What is "IEEE Std 802.5r and IEEE 802.5j" here? (maybe this could help https://ieeexplore.ieee.org/document/827772)
Originally posted by @mico in https://github.com/metanorma/pubid-ieee/issues/25#issuecomment-1091114588

According to this page:
http://gwise.itwelzel.biz/CDROMs/NASK4/Documents/IEEEStandards/IEEE802.5.html

"IEEE Std 802.5r and IEEE 802.5j" were combined and published as "ISO/IEC 8802-5:1998/Amd.1".
IEEE Std 802.5r and IEEE 802.5j are two separate IEEE Amendments. As we recall, IEEE Amendments use a lowercase letter after the number (802.5r means Amendment "r" of 802.5).
UPDATE: I found http://<IP_ADDRESS>:83/E%3A/share/7.206/共享资源区/学习资料/netword/网络/802/802.5.pdf , which is the actual copy of this document, which has already disappeared from the IEEE site.

It contained different content from its different PubIDs:

According to the page header, this is clearly a dual PubID document:

IEEE Std 802.5r and IEEE 802.5j are two separate IEEE Amendments. As we recall, IEEE Amendments use a lowercase letter after the number (802.5r means Amendment "r" of 802.5).

So, I should parse "IEEE Std 802.5r" and "IEEE 802.5j" as separate amendments? Still not clear for me 😭


fixes issue #27

fix #27
I honestly don't know why this works, it was a bit trial and error for me. Maybe this gives other problems, but at least it solves issue #27 for me.

Unfortunately this will cause pretty bad performance regressions most likely :(. It is good to know though that the resolvers are where the deadlock is coming from.

Actually, on second thought I think its probably worth submitting this for now and accept any possible performance hit since it at least gets things working. I will leave the other bug open though, as we should really try to find the root cause here.


Replace highlight_exists() with hlexists()

Hi @thiagoalessio!
I really liked your plugin and out of curiosity I browsed through the code and noticed that you use highlight_exists() instead of its newer version hlexists() in rainbow_levels#on(). So I thought that maybe you'd like to know.
According to :h hlexists(), the function name highlight_exists is now obsolete, and it has been since Vim 5.2 which was released around the year 2000.
I've been using rainbow_level with this patch for a few days and I didn't notice any issue, also the tests are all validated. Its not a major change but I hope you'll find this PR useful.
Cheers,

Hello @statox thank you very much, your contribution is very welcomed! 💙 ❤


Release Google.Cloud.Batch.V1Alpha version 1.0.0-alpha03

Changes in this release:
Bug fixes

Mark service_account_email as deprecated (commit 28348ae)

Documentation improvements

Removing comment from a deprecated field (commit 28348ae)

Created release for Google.Cloud.Batch.V1Alpha-1.0.0-alpha03


session.login() returns True but subsequent calls fail if the server's session cookie name is not 'RETS-Session-ID'

It would be nice if login() threw an error in this case.
I am working against a RETS server that does not use the standard session cookie name 'RETS-Session-ID', so while the login is "successful", the session_id is never set, so subsequent calls to other endpoints fail because the RETS-UA-Authorization header is incorrect due to the session_id not being included in the hash.
Following up with a PR to allow nonstandard cookie names, and punting on the error throwing from login().

Thanks for PR, address in release 0.4.4


ENHO: Fix diff in generated comments of class enhancements

Depending on release, the generated comments in class enhancements might contain "space-dot" instead of "dot" at the end of method signatures:

The fix replaces the space-dot with just dot.
Test repo: https://github.com/abapGit-tests/CLAS_with_ENHO
Note: This might impact existing repos but makes them cross-release compatible.

hah, thank you SAP


Add liveness to PETSc Array

Minor comments looks fine to me

Just checking, I do not see any minor comments? Is it fine to merge?


Quick question: where do the pretrained model's phoneme dictionaries come from?

Hi!
Great work you're doing here.
I've been testing your tool, it's easy to use and gives fine results.
Since I'm looking for a tool to generate a phonemized imput for the VITS model (in onnx format), I need to use the same tokenizer (phonemizer) that model espects. I've found that your pretrained models already have the dictionary embedded in them. Can I ask where did those dictionaries come from? In your colab training example you use CUNY-CL/wikipron's ones, but I was wondering if those are the ones you used originally or just in the example.
Thanks.

Hi,
Have you found a way to convert the deep phonemizer module to onnx format and use it?

I'm afraid I didn't. I mean, nobody answered my question, so there was nowhere else for me to go. I just let down VITS.

@ADD-eNavarro
https://github.com/as-ideas/DeepPhonemizer/issues/41#issuecomment-2116241220


still awesome

we should be able to share this on social media so more people are aware of this

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

ID: d3757356-1078-e915-8998-2868858792c5
Version Independent ID: 28cf37b6-6a21-9594-392d-29bef64912bb
Content: Build flows - Microsoft Flow
Content Source: flow-docs/guided-learning/build-flows.yml
Service: flow
GitHub Login: @MSFTMan
Microsoft Alias: deonhe

@adebayoyeleye  Thanks for using Microsoft Flow. I'll pass this suggestion on to the team. - Deon

@JLevesque84 @stepsic-microsoft-com @KumarVivek FYI, customer loves GL so much, he want to share it on social media easily. I like the idea of have a Share button. Thoughts?

@MargoC  Is there any plan to have a "Share" button on our Microsoft Learn content so that users can share the joy of Microsoft Learn via social media?

@adebayoyeleye  Thanks for sharing your thoughts. We've begun moving much of our training material to Microsoft Learn (https://docs.microsoft.com/learn/). On Microsoft Learn, there's a "share" button at the end of each module. Per your feedback, the team is also considering adding a "share" button even before the end of the module.


[popper] when hover - does not move focus

Description
Changed behaviour - now when hovering over a tooltip, the focus doesn't move to the tooltip.

Better resolved in https://github.com/semrush/intergalactic/pull/509


CustomResourceDefinition.apiextensions.k8s.io "runners.actions.summerwind.dev" is invalid: metadata.annotations: Too long

Describe the bug
After updating to the 0.15.3 Chart I am getting the following error when sync'ing the Chart via ArgoCD:
Failed sync attempt to f4b6f0a64ab13ed7478264478306ca231b22ff44: one or more objects failed to apply, reason: CustomResourceDefinition.apiextensions.k8s.io "runners.actions.summerwind.dev" is invalid: metadata.annotations: Too long: must have at most 262144 bytes,unable to recognize "/dev/shm/471866979": no matches for kind "RunnerDeployment" in version "actions.summerwind.dev/v1alpha1",CustomResourceDefinition.apiextensions.k8s.io "runnerdeployments.actions.summerwind.dev" is invalid: metadata.annotations: Too long: must have at most 262144 bytes,CustomResourceDefinition.apiextensions.k8s.io "runnerreplicasets.actions.summerwind.dev" is invalid: metadata.annotations: Too long: must have at most 262144 bytes (retried 5 times).

Checks

[x] My actions-runner-controller version (v0.x.y) does support the feature
[ ] I'm using an unreleased version of the controller I built from HEAD of the default branch

To Reproduce
Steps to reproduce the behavior:

Update ArgoCD Application with the 0.15.3 Chart version
After a failed sync, the error message above presented
Controller fails to start with no matches for kind \"RunnerReplicaSet\"

Expected behavior
ArgoCD Application syncs successfully and controller is running.
Environment (please complete the following information):

Controller Version: 0.20.4
Deployment Method: Helm via ArgoCD Application
Helm Chart Version: 0.15.3

Additional context
This is a similar/same issue as faced in https://github.com/prometheus-community/helm-charts/issues/1500

@donovanmuller This is the limitation of K8s. You need to use kubectl replace instead of kubectl apply to upgrade CRDs.

@mumoshu agreed, however, using the Helm chart and then especially when using Argo CD makes this more challenging than it should be.
I've worked around it by applying the advice from the Prometheus thread and using a Helm + Kustomize setup to add the argocd.argoproj.io/sync-options: Replace=true annotation to the affected CRD's.
If there's nothing to action, happy to close.

@donovanmuller Hey, thanks for sharing your work-around!
JFYI, having two ArgoCD apps, one for skipCRDs=true and another for Replace=true is said to work in recent RC version of ArgoCD. Perhaps that's a cleaner solution once released.
https://github.com/prometheus-operator/prometheus-operator/issues/4439#issuecomment-1030198014

@donovanmuller , would you mind sharing your solution? How did you include the CRDs into your kustomiziation folder? Do they need to be added manually or can they be referenced in a smarter way? Might be helpful for everyone who is using ArgoCD and the actions runner controller :)

advice

I'm following solution at https://github.com/prometheus-operator/prometheus-operator/issues/4439#issuecomment-1030198014 at get this problem solved.
    syncOptions:
    - Replace=true

@donovanmuller , would you mind sharing your solution? How did you include the CRDs into your kustomiziation folder? Do they need to be added manually or can they be referenced in a smarter way? Might be helpful for everyone who is using ArgoCD and the actions runner controller :)

@jimmyh85 Using the Helm + Kustomize approach, we just use patchesStrategicMerge to add just the annotation to the CRD's rendered from the Helm Chart.
E.g.
kustomization.yaml
apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization

resources:
  - ../../helm

patchesStrategicMerge:
  - patch/runners-crd.yaml

patch/runners-crd.yaml
apiVersion: apiextensions.k8s.io/v1
kind: CustomResourceDefinition
metadata:
  annotations:
    argocd.argoproj.io/sync-options: Replace=true
  name: runners.actions.summerwind.dev

advice

I'm following solution at prometheus-operator/prometheus-operator#4439 (comment) at get this problem solved.
    syncOptions:
    - Replace=true

Thanks, can achieve the same result by setting in web dashboard:

@norman-zon, I also had some issues in setting it up via kustomized-helm, as the helm template and solved it by using below plugin config
  configManagementPlugins: |     - name: kustomized-helm       init:         command: ["/bin/sh", "-c"]         args: ["helm dependency build"]       generate:         command: [sh, -c]         args: ["helm template . --name-template $ARGOCD_APP_NAME --namespace $ARGOCD_APP_NAMESPACE --include-crds > all.yaml && kustomize build"]
Main problem was that the example of helm-kustomized did not consider the namespace. Hope this helps, but the approach to use the sync option replace=true might be easier

@donovanmuller, what does your kustomized-helm plugin config look like? If I use the default config CRDs are not rendered. Do you just add --include-crds ?

@norman-zon 👍
configManagementPlugins: |
  - name: kustomized-helm
    init:
      command: ["/bin/sh", "-c"]
      args: ["helm dependency build ../../helm || true"]
    generate:
      command: ["/bin/sh", "-c"]
      args: ["echo \"$HELM_VALUES\" | helm template ../../helm --name-template $ARGOCD_APP_NAME --namespace $ARGOCD_APP_NAMESPACE $HELM_ARGS -f - --include-crds > ../../helm/all.yaml && kustomize build"]

Is this version release?

Is this version released?

Nothing was changed


Recently released Emacs 28.1 uses C-x x for buffer actions, thus conflicting with the prefix for perspective-el

Hi,
As the title says, recently released Emacs 28.1 uses C-x x for buffer actions, thus conflicting with the prefix for perspective-el.
The NEWS file says:

A new keymap for buffer actions has been added.
The 'C-x x' keymap now holds keystrokes for various buffer-oriented
commands.  The new keystrokes are 'C-x x g' ('revert-buffer-quick'),
'C-x x r' ('rename-buffer'), 'C-x x u' ('rename-uniquely'), 'C-x x n'
('clone-buffer'), 'C-x x i' ('insert-buffer'), 'C-x x t'
('toggle-truncate-lines') and 'C-x x f' ('font-lock-update').

Should perpesctive-el switch to a new prefix? Is there a suggestion for another prefix we should use?

I have always considered C-x x an unfortunate default, and use C-c C-p. I don’t think it conflicts with anything popular..?
That said, it will be a breaking change in Perspective, something I try very hard to avoid. I suppose I can reduce its impact on existing users’ muscle memory by only changing the prefix for Emacs 28 and later.

@zenspider: Do you have an opinion on this? It's a helluva change to inflict on long-standing users of Perspective who never changed the prefix key. As far as I can tell, we can:

Ignore the new Emacs 28 bindings for C-x x and so shadow them.
Change the Perspective prefix, breaking users' muscle memory. Document in the changelog and hope that's enough.
Option 2, but only for Emacs 28.

I don't have an opinion... I immediately changed my prefix (^z ftw!) and suspect many people do.
Unfortunately we want to keep this around, otherwise we could lean into declare w/ obsolete. As such, I suspect option 3 is probably the most gentle, but option 2 could be fine as well and prepare people BEFORE they move over (so much stuff breaks on a big new release that it'd be nice not to be a part of that).

I just realized that C-c C-p conflicts with a bunch of stuff in Org, and also markdown-mode. I'm open to other suggestions.

What about C-c M-p?

What about nil? Maybe don't set it and ask users to find something that works for them? I know I have mine set and thus won't even notice a version bump (nor will anyone else who already customized). Anyone who's using the default can have a message pop up or something.

Even though, it was I, who raised the issue, I'm keeping C-x x for perspective.el rather than the new buffer actions, as most of those buffer actions are in my particular view not so useful (except maybe for rename-buffer, but then again we have crux and so many hacks people made to rename a file along with its buffer). It is very seldom I'll want to perform any of the tasks listed in there.

@zenspider: Yes. Let's get out of the business of deciding on key prefixes for users. I looked at Projectile for inspiration, and it does not provide a default prefix.
@paulophagula: I agree with your reasoning about C-x x. It makes sense for users with muscle memory to keep it. Still, I think it's best for Perspective to just get out of the way and let users decide what to do.
I still don't want to kill it outright for everyone, so I kept C-x x or Emacs 27 and earlier, and blanked it for 28. There's a warning message for people who do not have the prefix set, and a way to disable the warning if someone wants the prefix off.
So overall, still a breaking change, but hopefully one easy to get past. I'd appreciate it if you two would give the change a quick look (see 7ec5a6ea0fc8ee8eade5731c3bfb20c52886d82d) and I'll merge it in a couple of days.

@gcv thank you. I think this settles it, so I'll be closing the issue.


feat: Add Fenced code and LaTeX copy button.

Feature:

Add Fenced Code Block copy button.
Add LaTeX Block copy button.

Fix:

Add some space between title and description text inside Plugin Settings UI.
Fix the issue that the font color is incorrect when using with QQNT Telegram Theme. (#26, #51)
Add try-catch block to ensure onLoad() method will never throw error which may break QQNT / QQNT LiteLoader.

Now you got this fancy button lol


Ability to customise reply message above text field

Is your feature request related to a problem? Please describe.
Currently I'm trying to customise the padding of selected messaging for replying. But couldn't find one except sendMessageBuilder which will replace the text field with icons and features on it.
As you can see in the image below, the padding is a little bit not balance.

Describe the solution you'd like

A field/property for customising the UI of selected message for replying

Thanks.

Hi @dongnqda, From the provided image, could you please let us know what you exactly want? Do you want to change Replying to user or just need a field to customise the reply message? In this case, it is Testing 123 or the whole reply view.

@apurva010 Hi apurva, thank for your reply. What I meant is the padding of selected message appear above textfield was not balance. So is there anyway to make it balance or customise the whole UI of selected message without losing the TextField ?


Web App: Look into possible memory leak

(node:1) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 connect listeners added to [FileStore]. Use emitter.setMaxListeners() to increase limit

@jdharvey-ibm did some investigation on this and conclusion is we can safely ignore this warning, it doesn't mean there's an actual memory leak happening, it's just looking into already closed connections.
Closing issue


Different way for saving states.

Instead of clicking two save and load state buttons, I could use the game instead. All of the games have a built in save option like OOT or SM64. How would you go by saving that way. I feel like it could help with accidentally clicking save state when loading a game so your progress is not erased.

It already does that. Whenever you see the emulator saying "Saving Sram..." it is saving the Save File into the browser. Later if you come back and use the same rom file it will load that Save.

Sorry for my part. I am still a little new to coding.

Another question, I'm having a hard time finding where the canvas is given the width and height, I would like to have the canvas fill up the screen after you selected the rom. I don't like full screen, mainly on mobile, because when you go in full screen, the keyboard just does not work? It's a problem with safari I'm pretty sure.

there's two places, if you are on mobile the it gets it from the window.innerWidth via the setupMobileMode() method in script.js

Then it calls resizeCanvas which actually sets the size.

in desktop mode it's based on the zoom setting. basically just set this.canvasSize to whatever you want and then call this.resizeCanvas()

Thanks, but when i tried that out it will just fill the whole screen, but since its N64, the screen is more square. is there a way to
unlock it so it can stretch based off screen width?

that's what it already does it stretches based on screen width. the function resizeCanvas() maintains the correct aspect ratio. what's probably happening is that when the phone is in landscape the square is past the bottom of youre screen.

Well how do I unlock aspect ratio. Another person already did it, https://lrusso.github.io/Nintendo64/Nintendo64.htm , but when you try to import the game, it only allows .n64 types. So changing the file extension might work.


Add succeed and fail factory methods

Codecov Report

Merging #11 into master will increase coverage by 3.75%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #11      +/-   ##
==========================================
+ Coverage   82.35%   86.11%   +3.75%
==========================================
  Files           1        1
  Lines          34       36       +2
==========================================
+ Hits           28       31       +3
+ Misses          6        5       -1

Impacted Files
Coverage Δ

core/src/main/scala/zio/saga/Saga.scala
86.11% <100%> (+3.75%)
:arrow_up:

Continue to review full report at Codecov.

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


[docs][field] Fix description to fit in Google

A quick iteration on #612. Will make the issue easier to solve.

Closing this to avoid this situation where multiple people on the team are spending time reviewing and suggesting edits on such a tiny task. We just begun a 5-week pass on the docs where we aim to fix hundreds of issues like this. No need to document each tiny one individually.


[Cybergame.tv] show proper channel name

Not sure if there's any api at the moment that can help me.

http://api.cybergame.tv/p/statusv2/?channel=scatman use api to get id, store channel name

also implement #193 (cybergame)


Unmanaged objects cannot be thawed

How frequently does the bug occur?
Sometimes
Description
There are two tabs at the bottom of my app, the first tab has the operation of the database, the second does not, normal is no problem, but I click the switch operation in the two tabs, it is possible to crash to report this error。
When I click the first tab, I execute the following code：
        let items = DbUtils.getRealm().objects(ItemNoteBean.self)
        for (index,item) in items.enumerated().reversed(){
            if (item.title.replacingOccurrences(of: " ", with: "").isEmpty){
                if let attributedString = try? NSKeyedUnarchiver.unarchiveTopLevelObjectWithData(item.contentData) as? NSAttributedString {
                    if !attributedString.string.replacingOccurrences(of: " ", with: "").isEmpty{
                    } else {
                        try! DbUtils.getRealm().write{
                            DbUtils.getRealm().delete(item)
                        }
                    }
                }else {
                    try! DbUtils.getRealm().write{
                        DbUtils.getRealm().delete(item)
                    }
                }
            }
        }

class DbUtils {
    static var realm: Realm?

    static func getRealm() -> Realm {
        if realm == nil {

            do {
                realm = try Realm()
            } catch {
                fatalError("error: \(error)")
            }
        } else {

            print("has initdb")

        }
        return realm!
    }
}

Stacktrace & log output
No response
Can you reproduce the bug?
Always
Reproduction Steps
No response
Version
10.39.1
What Atlas Services are you using?
Local Database only
Are you using encryption?
No
Platform OS and version(s)
iphone 16.6
Build environment
Xcode version: ...
Dependency manager and version: ...

Sometimes this error is also reported here：
@StateRealmObject var item: ItemNoteBean
try! DbUtils.getRealm().write {
item.thaw()?.time = Date().timeIntervalSince1970
}
The above code this sentence report error:  try! DbUtils.getRealm().write

Per the error message, it looks like you're trying to thaw an object that is unmanaged (i.e. not persisted in the database). This is by design, as those objects cannot be frozen or thawed. From the snippets you've provided, it's not clear where the source of the issue originates from, but if you attach a debugger and walk down the stacktrace, you should be able to identify where these unmanaged objects are coming from and restructure your app not to call thaw on them.


Spike adjustments to typography scale

What
Spike some adjustments to the typography scale to assess what the changes would look like (and any knock-on effects) and to get an idea of the impact on users.
We want to spike:

making the govuk-body class 19px across breakpoints. We need to think about how this impacts any govuk-heading classes as we could end up in a situation where heading text is smaller than body text on mobile
making the govuk-body-s class 16px across breakpoints
possibly deprecating govuk-body-xs
making a decision on 24px vs 27px (we think we should remove 27px, but it's used in a heading caption - need to check if it's used anywhere else)

Why
Some of the font-sizes and associated classes in the Design System have been carried over from GOV.UK Elements and probably haven't been revisited for a while. There are some inconsistencies (e.g: 24px and 27px - maybe we should decide on one?) and also some font sizes which we may not recommend using any more (anything below 16px) for accessibility and readability reasons.
Image showing font size and line height at different breakpoints:

Who needs to work on this
Designer; Developer
@christopherthomasdesign and @vanitabarrett were involved in the original discussions with GOV.UK (Mia and Nikin)
Done when

[ ] Created a branch with the changes above
[ ] We have a better idea of the impact of these changes on our components and typography scales
[ ] We have a pre-release we can share with GOV.UK for their own testing
[ ] We have decided next steps

I asked the community to share their thoughts on Slack. Here's a summary of the comments:

Some said that because most users hold their phones closer to their face, the smaller text appears larger
A couple of people tested this quickly for themselves and said when viewing on a phone at their normal reading distance, the 16px text is perceptually bigger than the same text in 19px on a desktop screen further away
However, others flagged that there may be a greater need for larger text because people often use mobiles on the move, or when doing something else at the same time
One person noted that anecdotally they've seen that lots of people tend to increase the system-wide font size on their phones – there was also speculation that the trend towards bigger phones means there's less of a need for smaller fonts
Two well-designed accessible sites (Scope and RNIB) both use the same large font size regardless of mobile or desktop
In terms of impact on services, the main concern was with tables – it could affect their layout, scannability, or how much data you can fit on the screen

A bit of exploration into govuk-xs to see why it was included and how it’s used now.
It was a consolidated from Frontend Toolkit when the Design System was created (the Frontend Toolkit included a 14px/12px mixin). We did this to make it easy to migrate from the old stuff, and not to clash with any existing designs.
I haven’t found loads of examples of where teams have used the 14px/12px size – one example is the ‘Popular on GOV.UK’ text that was on the old GOV.UK homepage.
From searching GitHub for govuk-xs, I’ve found it’s mainly used in old prototypes and demos. However, there are some repos that appear to be live services.
In the more relevant repos, it tends to be used for:

small labels with metadata – for example ‘PDF, 1000KB, 100 pages’, or ‘Last updated: Mon Mar 09 2020 13:36:22’
tabular data
confirming an action – for example ‘Changes saved’

It also pops up a lot in adapted copies of GOV.UK Frontend – i.e. other organisations’ websites, services and design systems.

We're closing this issue in favour of putting together a broader epic to update the font scale which we will work on in the run up to 5.0. Summary of findings from this spike:

14 on the font scale is a carryover from Frontend Toolkit. We hypothesise that it made sense at the time to use this font size for certain smaller elements but with the change in screen sizes since then, we are confident in removing it.
The use of 14 as a primary font size isn't wide spread and we don't believe that there is a legitimate use for it. See Chris's comment above for more details https://github.com/alphagov/govuk-frontend/issues/2418#issuecomment-964079947
Whilst the use of 14 isn't wide spread, 16 is much more common. We'll need to provide detailed comms so that teams, including us, have time to audit their components which use 16 and the impact of the font not moving from 16 to 14 on mobile.
This work can be tied into a general review of our font scale. This includes aligning our "t-shirt" size classes eg: govuk-body-s across all our different typography styles so that xs becomes 16 on the font scale and every proceeding t-shirt size up lines up with it's associated font scale size up eg: s maps to 16, m maps to 19 etc.


feat(provider): add libretranslate provider

adds a new provider package enabling translation via LibreTranslate
fix #87

I think that is good enough for a first version, it is missing some of the fancy features the deepl api package provides, such as retries of failed requests, but I think that is ok for now.
Not sure if this PR should target main or next @sargreal


MiddelTier throws ModelMapper required error

Xpand 19.1.602.5 installed via
pwsh Set-ExecutionPolicy Bypass -Scope Process -Force;iex "$(([System.Net.WebClient]::new()).DownloadString('http://install.expandframework.com'));Install-Xpand -Assets @('Assemblies','Nuget','VSIX','Source')  -Version '19.1.602.5' #-SkipGac -InstallationPath 'YOURPATH'"
and project created via XPand 19.1 solution wizard.
When running application server it gets:
System.NullReferenceException: '_xafWinAssembly check that ModelMapperModule is added to the RequiredModuleTypes collection'
repro project: https://www.dropbox.com/s/xzpkn1pamaj57f4/xpandTest191.Win.zip?dl=0

I think this is related to my issue: If you add the ModelMapper module, another exception is thrown

The DevExpress.XAF repository includes commits that relate to this task:

ModelMapper: Platform detection fails when MiddleTier #542 #551

Please update the related Nuget packages and test if issues is addressed. These are nightly nuget packages available only from our NugetServer.
If you do not use the Xpand.XAF.Modules directly but through a module of the main eXpandFramework project, please wait for the bot to notify you again when integration is finished or update the related packages manually.
Thanks a lot for your contribution.

Closing issue for age. Feel free to reopen it at any time.
.Thank you for your contribution.


文件多选问题

VSCode Version: Code 1.17.1 (1e9d36539b0ae51ac09b9d4673ebea4e447e5353, 2017-10-10T14:24:41.632Z)
OS Version: Windows_NT ia32 10.0.15063
Extensions:

Extension
Author (truncated)
Version

vscode-eslint
dba
1.3.2

beautify
Hoo
1.1.1

cpptools
ms-
0.13.1

debugger-for-chrome
msj
3.3.1

java
red
0.11.0

JavaScriptSnippets
xab
1.5.0

为什么目录树里面的文件不能按住shift进行多选 这个功能很需要啊

This is a feature request for shift-key/shift-click support in the directory/file tree. I agree, this and cntrl-key support (replacing what cntrl-click currently does) would be great.

Hopefully the next version will be available
陈小龙
在2017年10月12日 12:47 ，Caelan<EMAIL_ADDRESS>This is a feature request for shift-key/shift-click support in the directory/file tree. I agree, this and cntrl-key support (replacing what cntrl-click currently does) would be great.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/vscode/issues/36103#issuecomment-336019577, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ALKglgsIm6vhoKAV7ITqUOS2UuZwSfTvks5srZnZgaJpZM4P2Wm2.

@caesay just to clarify by shift-click you mean support for multiple selection? This would be a duplicate of #1023.

Yes, I was just clarifying what the post was about (multi selection, shift
click). It is indeed a duplicate.
On Oct 13, 2017 12:09 AM, "Erich Gamma"<EMAIL_ADDRESS>wrote:

@caesay https://github.com/caesay just to clarify by shift-click you
mean support for multiple selection? This would be a duplicate of #1023
https://github.com/Microsoft/vscode/issues/1023.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/vscode/issues/36103#issuecomment-336129773,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABOkf1gCjQ6BEr8rDFt-iX8fsvSgq3_Rks5srg-agaJpZM4P2Wm2
.

@caesay thanks, then I'm closing this one as a duplicate of #1023


Provide optional "description" argument to consumes and produces decorator

In team we have the rule to document code basis in Russian, but all the docs we provide must be in English.
Autogeneration of docs is using doc attribute of function to generate description.
Current way to achieve the goal:
@producers_router.produces(topic="transaction.response.order_status")
async def exchange_orders_status_producer(message: OrderStatus) -> OrderStatus:
    """Продюсер по топику transaction.response.order_status."""
    return message

exchange_orders_status_producer.__doc__ = "Publish exchange_order's status"

Preferable solution:
@producers_router.produces(topic="transaction.response.order_status", description="Publish order status")
async def exchange_orders_status_producer(message: OrderStatus) -> OrderStatus:
    """Продюсер по топику transaction.response.order_status."""
    return message

Thanks for response =)

Works well for consumers, and does not for producers.

I'll reopen the issue, see what went wrong and we'll do a patch for this in the next couple of days.
Thanks for the quick feedback :)

Can you please explain what is failing for the producers, with code examples if possible.
For a simple app example:
app = FastKafka(kafka_brokers=kafka_brokers)

@app.produces(description="Producer description in documentation")
async def to_hello_world(msg: str) -> HelloWorld:
    """This description should not show"""
    return HelloWorld(msg=msg)

The description "Producer description in documentation" is shown in the async docs for the producer.


Added mci transformation schema

We obviously don't trust @MIslam92 reviews any more. Quite right, they were never any good.


Allow anonymous user to create station and add song.

Map the anonymous station back to user when they register or logging
using social network on that machine.
Clear local data after success register or new social account was created.
Only that anonymous user can add song to the anonymous stations that created by him.

verified!!


FutureWarning: tmpfile is deprecated

What happened:
Plain import produces multi-line FutureWarning with newest dask and dask_jobqueue version. Multiple calls (e.g. for unit testing) pollute the terminal output with many of those.
What you expected to happen:
No FutureWarning
Minimal Complete Verifiable Example:
import dask_jobqueue
>>> lib/python3.8/site-packages/dask_jobqueue/core.py:20: FutureWarning: tmpfile is deprecated and will be removed in a future release. Please use dask.utils.tmpfile instead.
  from distributed.utils import tmpfile

Anything else we need to know?:
Happens on all tested platforms (various *nix, Mac OS)
Environment:

Dask version: 2022.5.0
Python version: 3.8.13
Operating System: Mac OS 12.2 / Ubuntu 20.04
Install method (conda, pip, source): conda

Thanks for opening up an issue @tensionhead. It looks like this was already fixed in https://github.com/dask/dask-jobqueue/pull/529, but hasn't been released yet. There's a discussion around releasing over in https://github.com/dask/dask-jobqueue/issues/555. Closing as the appropriate changes have already been made in main


Translator doesn't hook

Hello. There is chinese game called 九霄缳神记 JIU XIAO, which has been released recently. It seems the game is using some kind of modified text module - and thus translator doesn't work (it doesn't even create config file).
Could you please check it and tell, if it's possible to add support for the game or it's not worth it. You can get whole "managed" folder here. Also, text files are probably loaded from "StreamingAssets\AssetBundles\Assets\resources bundle\csv" path (example). Thanks in advance.

If the config file isn't even created then the plugin has never gotten a chance to initialize.
Are you using BepInEx or the standalone ReiPatcher-based installer?

I've tried both - latest ReiPatcher and latest stable/bleeding edge BepInEx versions. ReiPatcher installs without any errors and BepInEx produces this log in game folder: preloader.log
Thanks for explanation. I thought that the translator is using some default settings if config is not present (and because ReiPatcher patched files without noticable errors). It seems it's not translator's fault, but injector's. Should I close the issue then and open new one in BepInEx section?

This log would suggest that BepInEx has trouble initializing. Can't really see why, though.
Do note that if you have tried both: The ReiPatcher-installer modifies and adds files to the Managed directory that must be removed for BepInEx to work properly.
Even so, I would still expect the ReiPatcher-installer to work (if used without BepInEx). If you installed it through ReiPatcher does it create the config file and/or can you show the UI (through ALT+0)?

I'm aware of ReiPatcher modifying files, so I used BepInEx version with original unmodified files. And while ReiPatcher is seemingly patching files without errors in console, it doesn't create AutoTranslator folder or any other configs upon patching via "Patch and Run" shortcut (like it should). And in-game none of translator's hotkeys works (including UI), so I assume it doesn't hook properly.
The game doesn't have any protection aside from usual steam check, but maybe it's checking integrity of some engine files? Though I can run the game with patched files just fine. Maybe there is a way to activate debug logs or something for ReiPatcher/BepInEx to narrow down the problem?

I have taken a closer look at the managed directory. I think this may be a case of the developers having trimmed unused code away before publishing, which means that code BepInEx and this plugin relies on simply don't exist in the distributed runtime.
This is just a guess. I cannot test it myself, but when I compare the distributed framework assemblies (mscorlib, netstandard, System.Xxx) to distribution with other games targeting netstandard, I find that each one of them is much smaller and has less programming structures when inspected.
Maybe it can be resolved by simply taking the assemblies from another game which also targets that same platform, but that has not been trimmed.
I had two games lying around that used netstandard. A Unity 2018.3 and Unity 2019.1 game.
As I cannot test it myself, perhaps you can try by overriding these files in the managed directory.
Not sure if other files needs to be replaced, but could be. Don't really know that much about this kinda of stuff.
Managed-netstandard-2018.3.zip
Managed-netstandard-2019.1.zip
Even if this makes it possible to initialize the plugin I still fear it might not work if specific unity constructs are referenced in the plugin that have been trimmed. For instance IMGUI constructs when it comes to the translator plugin.

Thanks, it actually works, at least partially. Now AutoTranslator folder with config and all is properly created, but the translator doesn't work in-game (not even UI). I think you are right about other files, probably some UnityEngine.* should be replaced as well. I'll try to find a game with similar version, or maybe it'll be better to just create empty unity project with the same engine version (2018.4.5) and try to re-use some modules from it (especially IMGUI and TextCore).
I'll report back when I find something out and test stuff. Thanks again for your help and all those updates for the translator.

I suspect very much it work work if you did that. Make a project with correct version, target netstandard, include the required modules (not entirely sure how to do that), publish/build it and simply copy the DLLs in the Managed directory and override:

Mono.Security.dll
mscorlib.dll
netstandard.dll
System.*.dll
UnityEngine.*.dll

Doubt you would need to copy specialized packages such as Unity.*.dll (which are installed through their package manager)

Yes, it worked. Translator now hooks and works as expected. Btw, default sample project in unity includes all required modules, so it was easier than I thought. Here are those compiled libraries, maybe you or someone else will need them (e.g. for references) - link.
I really appreciate your help. If you don't have any further questions about this issue, you can close it, otherwise I'll close it by myself in a day.


[Bug] Using vuetify components in Vuetify Material Dashboard

Version
It is reproducible with the latest stable version
Reproduction link
https://jsfiddle.net/
Operating System
Windows 10
Device
Microsot Surface Book
Browser & Version
Chrome Version 75.0.3770.100 (Official Build) (64-bit)
Steps to reproduce

Add a vuetify button to Tim's Vuetify-Material-Dashboard:

button1

Add "fab" to it to make it round.

button1

The "fab" feature doesn't work.

What is expected?
The button to become round
What is actually happening?
The button stays a rectangle

Solution
Additional comments
I want to be able to use vuetify components and all of their features in Vuetify-Material-Dashboard but some of them like "Fab" feature don't work when you use vuetify components in Tim's Vuetify-Material-Dashboard. I believe it has to do with material design preventing certain vuetify features and components to load properly. Is there a way to do something that allows me to use vuetify in Vuetify-Material-Dashboard?

It's because of the CSS in _buttons.scss. You may want to remove the button styling or overwrite some of the CSS in your component scope to make the Fab work.


Distributed app looks to have incorrect compilation

Hello.
Thanks for this great project.
I noticed that the compiled output of this package has a problem on it. Despite it is using the requireInterop helper, it still tries to access the default exported property, which is not required:
// getfieldNames.js
function addFieldNames(type) {
  var fieldNames = _astTypes["default"].getFieldNames({
    type: type
  });

That throws an exception because prop default does not exist.
I tested on node 14.
Changing it to this makes it work:
// getfieldNames.js
function addFieldNames(type) {
  var fieldNames = _astTypes.getFieldNames({
    type: type
  });

Huh that's odd, I couldn't reproduce it, as interopRequireDefault creates an object with a default property as far as I can tell.  But, I had an old version of @types/jscodeshift that was pulling in a different version of ast-types, so I don't doubt there was some issue...I went ahead and changed all those statements to import * as t from 'ast-types'.  There should be a release once CI passes!

:tada: This issue has been resolved in version 1.2.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


nvcc is present in AMD64 ros-torch:noetic-desktop-full-torch2.1.0 image, but not in ARM one

BACKGROUND
I'm currently using the ros-torch:noetic-desktop-full-torch2.1.0 image because it's has ROS+CUDA preinstalled as well as being a multi-architecture image. So far I've only had it running successfully on AMD64 (Intel) hardware, and am trying to use it on ARM hardware. On the ARM image, as part of my build, cmake is throwing errors about not being able to find CUDA. So I tried to run nvcc --version on the ARM build and found it wasn't there.
STEPS

On an AMD64 machine, rundocker image pull rwthika/ros-torch:noetic-desktop-full-torch2.1.0 to get the latest image
Start up a container: docker run -it --rm rwthika/ros-torch:noetic-desktop-full-torch2.1.0
Run nvcc --version - you get valid output:

root@b50b900aea99:/docker-ros/ws# nvcc --version
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2021 NVIDIA Corporation
Built on Wed_Jul_14_19:41:19_PDT_2021
Cuda compilation tools, release 11.4, V11.4.100
Build cuda_11.4.r11.4/compiler.30188945_0

which nvcc shows: /usr/local/cuda/bin/nvcc

Now repeat steps 1-3 on an ARM64 host.

RESULT
On the ARM host, I get:
bash: nvcc: command not found

EXPECTED
I would expect both images to have similar contents.
NOTES
I noticed on the Docker hub page that there was a big difference in image size: https://hub.docker.com/layers/rwthika/ros-torch/noetic-desktop-full-torch2.1.0/images/sha256-e98452925fa557ee164e5511459629f3d8e1160f06b9b39b65c84486d6ab1fbf
linux/amd64 -> 9.19GB
linux/arm64 -> 3.2GB

For the ros-torch:noetic images, we are using these two images as base images:

amd64: nvcr.io/nvidia/tensorrt:21.08-py3 AS base-tensorrt-ubuntu20.04-amd64
arm64: nvcr.io/nvidia/l4t-tensorrt:r8.5.2-runtime AS base-tensorrt-ubuntu20.04-arm64

Note that the arm64 variant (optimized for L4T NVIDIA Jetson devices) is explicitly set to runtime, i.e., it does not include developer tools such as the CUDA compiler nvcc. For amd64, NVIDIA does not offer devel and runtime versions, that's why the amd64 image is larger and does include nvcc.
Nevertheless, CUDA is indeed installed in both amd64 and arm64 images. I just checked again, you can find CUDA at /usr/local/cuda-11.4 in the arm64 image. If you only need to use CUDA (e.g., when just using PyTorch to run inference), then you should be good to go. If you require nvcc, you need to manually install the CUDA developer tools.
That being said, note that we are not going to maintain these images much longer, as ROS 1 Noetic is set to go end-of-life in May 2025. You should consider switching to ROS 2.

Thanks for the explanation.
For anyone else running into this, I was able to install the CUDA developer tools in my Dockerfile by adding:
ARG BUILD_ARCH
RUN if [ $BUILD_ARCH = "arm64v8" ]; then \
        apt-get update && \
        apt-get install -y cuda-toolkit-11-4 && \
        rm -rf /var/lib/apt/lists/* \
    ; fi

... and running docker build with a --build-arg BUILD_ARCH= arm64v8.
Also, I understand that ROS 1 is older - it's just what we're on currently.


Chat does not work

I have followed all steps and managed to do all steps with no errors. When i do npm start it works fine and opens the browser window, however it does not return any conversation. Where can i check logs for an error?

Thanks for submitting the issue. You can check the logs in the terminal to see if something is failing? Can you please share what message you typed in the chat box? and a screenshot of the browser so that we can look into the issue.
Thanks!

Hi Mamoon
What I ended up doing was add a line in script that points to my folder
where all files are contained for the page and it worked from there.


How to point a tab to a subdomain and in a new tab?

Checklist

[x] I have read the newlest tutorials and know the correct effect of the functional design.
[x] There is no similar question on existing issues (including closed ones).
[x] I have tried to find the answer on Jekyll Forum and StackOverflow.
[x] My question is based on the latest code of the master branch.

Description
Hello, I've come to bother you again, hoping to have done enough research beforehand.
I have added a page named DRAW under HOME, I would like to make it point to a subdomain and in a new tab but after trying several different things I can't manage to do it.

I tried different tags in my _tabs/draw.md but nothing worked, I think I'm not looking at the problem from the right angle and I'm missing some information.
Is it easier to integrate the desired html rather than redirecting to a subdomain?
Should I link the DRAW directly with a page in html instead of md?
Thanks in advance!

Following codes make your tabs point to another page under the same domain.
https://github.com/cotes2020/jekyll-theme-chirpy/blob/41ed33145639415148aec8e85edc7a6fd0de0ca3/_includes/sidebar.html#L41-L50
Here is my suggestion:

      your_tab_name

 {% for tab in site.tabs %}

     {% capture tab_name %}{{ tab.url | split: '/' }}{% endcapture %}

     {{ site.data.locales[site.lang].tabs.[tab_name] | default: tab.title | upcase }}

 {% endfor %}

Also change $tab-count to {{ site.tabs | size | plus: 2 }} here.
https://github.com/cotes2020/jekyll-theme-chirpy/blob/41ed33145639415148aec8e85edc7a6fd0de0ca3/assets/css/style.scss#L8

Thanks for your answer @NichtsHsu , I understand the idea, however I use the starter and I don't have a _includes/sidebar.html folder.
So should I include the files in the root of my repository?

So should I include the files in the root of my repository?

Yes, according to the jekyll doc overriding theme defaults, you can cover theme's files by creating your own under the same directory.

It's great, thank you @NichtsHsu !
Sorry I'm in the middle of discovering and learning all this!

I would suggest the easy way with redirect
https://github.com/ser-mk/ser-mk.github.io/commit/a14a03c0aa6582d96ca0bb5df5af84b3a8a36933
just define redirect layout and then set redirect link


Bump star container image to 2.7.5b--0

Coverage remained the same at 0.0% when pulling 14b71dc90f76a566b1c02d50ed061fed6d93f659 on heuermh:star-2.7.5b into 0229eb26fbf39a720512b3d3f6685f7611a35054 on bigdatagenomics:master.

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


module openvpn: Document @-symbol usage in ExecStart

Issue description
Module openvpn contains:
serviceConfig.ExecStart = "@${openvpn}/sbin/openvpn openvpn --suppress-timestamps --config ${configFile}";

, which certainly looks wrong, although it probably isn't. It can use a comment explaining what is happening.
Possibly documented here: https://www.freedesktop.org/software/systemd/man/systemd.unit.html#Description
Technical details
master and 18.03.

See ExecStart= in systemd.service(5): https://www.freedesktop.org/software/systemd/man/systemd.service.html#ExecStart=


Allow user-specified identifiers at export tasks

Once a user submits an export task (i.e., transforming an RDF resource into CSV or SHAPEFILE format), it is not possible to utilize the same resource in another export, e.g., in case this task has failed or the user wishes to get the data in another format. Currently, the application issues an error like "Workflow already exists".

Currently, we have an application-level constraint that every pair of (process-name, user-id) is unique.
Since the process-name (in the case of an export) is based only on the resource name (here), an attempt to create another process will fail.
Maybe we should name the exporting process to something based on the pair (resource-name, export-timestamp). It is kind of heavy in terms of performance (since the same resource will always provide the same export), but is acceptable for now.


Build fails

Make ends up in following error:
/home/osboxes/shared/Ohjelmointi/stlink-master/src/usb.c: In function ‘_stlink_usb_version’:
/home/osboxes/shared/Ohjelmointi/stlink-master/src/usb.c:192:18: error: comparison of integer expressions of different signedness: ‘ssize_t’ {aka ‘int’} and ‘uint32_t’ {aka ‘unsigned int’} [-Werror=sign-compare]
if (size != rep_len) {
^~
cc1: all warnings being treated as errors
make[3]: *** [CMakeFiles/stlink-static.dir/build.make:89: CMakeFiles/stlink-static.dir/src/usb.c.o] Error 1
make[2]: *** [CMakeFiles/Makefile2:73: CMakeFiles/stlink-static.dir/all] Error 2
make[1]: *** [Makefile:152: all] Error 2
make: *** [Makefile:28: release] Error 2

Hello @mmikkone. Look like this issue was already fixed in develop branch.

Thank you. I was able to build it from develop branch.

Fixed with PR #872.


Add ARM to make cross.

This adds a linux/arm build to Makefile which will be included in the
release.sh script.

ping @surajnarwade and @kadel
Would need a quick review of this before I release 0.5.0 today :)

#541

Hey @surajnarwade and @kadel
I'm going to go ahead and merge this in so I can do the 0.5.0 release today!

LGTM :)


Release nwbview 0.2.4

The release solves #33.
I was wrong about the include directive of cargo.toml. It is not needed when I compile it on the project directory but the cargo publish command compiles it in a different directory and the static/icon.png needs to be copied there to be present at compile time.
@@ -14,7 +14,8 @@ include = [
  "Cargo.lock",
  "Cargo.toml",
  "LICENSE",
  "src/**/*"
  "src/**/*",
  "static/icon.png",
]

Ah :sweat_smile:

And did you see https://github.com/brainhack-ch/nwbview/pull/34#discussion_r1135787300 ?

Oh, I didn't. :) Now I see. Reading...

Thanks it is simpler now :)

You're welcome :)
Now it looks good to me!


[chip] sleep_pin_retention test: SW/DV handshake + GPIO check

This PR includes #15338
Add GPIO/ PINMUX configuration
In this commit, the sleep_pin_retention test selects IoA[7:0] PADs for
GPIO[7:0] and drives the PADs (via GPIO) to all zero befor the test
begins.
Random choice of GPIO value
In this commit, the chip_sw_sleep_pin_retention test choses GPIO[7:0]
value randomly per round and drive the PADs.
SV side receives the value and check the PADs

As #15421 was merged, let me close this PR. Thanks for the review @cindychip


Please Add Unifeso

Official site of Unifeso University
https://www.unifeso.edu.br/
Computer Sciente course page on Unifeso's site
https://www.unifeso.edu.br/cursos/graduacao/ciencia-da-computacao
Students panel:
Please, note (marked in the image below) that the email address provided from the university to their students has the 'unifeso.com.vc' suffix. Therefore, consider that both domains belongs to Unifeso University.
https://portalint.unifeso.edu.br/FrameHTML/web/app/edu/PortalEducacional/login/

@hugonorte First stage of verification passed, official domain for UNIFESO - Centro Universitário Serra dos Órgãos is unifeso.edu.br as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

Thank you for the feedback. Please, note (marked in the image above) that the email address provided from the university to their students has the 'unifeso.com.vc' suffix. Therefore, consider that both domains (unifeso.edu.br and unifeso.com.vc) belongs to Unifeso University.

@hugonorte Thank you for your patience while waiting for a review and processing your request. Pull request merged.
Please start requesting the licenses in about an hour, to let the changes to propagate through our system.


Design: AMI image builder image sharing

User Story
As a Modernisation Platform Engineer
I want to create EC2 AMIs using Image Builder that are held centrally within the shared services account but are made available to consuming member accounts
So that that Modernisation Platform consumers can use them as parent image sources
As a consumer of the Modernisation Platform
I want to be able to build AMIs based on core Modernisation Platform AMIs and deploy EC2 instances based on those AMIs
So that I can benefit from managed AMIs through Image Builder
Value

Centralise AMIs built using ImageBuilder
Share them securely without copying them across accounts
Share common, hardened images to improve the security posture across EC2 instances

Questions / Assumptions
Assume we can effectively use Resource Access Manager to share ImageBuilder images to accounts / organisational units
Definition of done

[ ] Clear, capture design showing how AMIs can be shared (added to Network Architecture drawio file)
[ ] Design effectively informs implementation
[ ] Run through design with team to gain acceptance

Reference
How to write good user stories
Sharing Image Builder images

AMI diagram updated with the design around kms key, launch permission and use of environment management secret.
Closing this issue.


Add secondary feature to allow exeute ad-hoc queries

Implementation should be simple

The purpose of spwrap to call stored procedures, only!
Out there, there's a bunch of very useful jdbc wrappers to call sql queries (ollin, JDBI, Spring JdbcTemplate, jdbc.jcabi, fluentJdbc and more) see https://github.com/search?l=Java&q=jdbc&type=Repositories&utf8=✓


Incorrect Discovery URL?

Within the documentation, you reference this URL:
(Step 5 under verify Auto Enrollment)
Verify that the MDM discovery URL during auto-enrollment is https://enrollment.manage.microsoft.com/enrollmentserver/discovery.
The URL within the console is the following: https://enrollment.manage.microsoft.com/enrollmentserver/discovery.svc
Notice no 'SVC' extension in documentation.
Which is correct?

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

ID: ad012591-5c74-f1cd-d3a2-443768e8ceee
Version Independent ID: 918ea4bc-0849-3745-5d9d-a21a973bcd3b
Content: Enroll a Windows 10 device automatically using Group Policy
Content Source: windows/client-management/mdm/enroll-a-windows-10-device-automatically-using-group-policy.md
Product: w10
Technology: windows
GitHub Login: @ManikaDhiman
Microsoft Alias: dansimp

@learningdaily - Thank you for submitting feedback.
I will get this issue over to the Win10 ITPro writing team for investigation.
Thank you for reporting and making the docs better. Much appreciated.
I made a note to request the team to update this when the work is complete.

@ManikaDhiman - Can you please share your insights on this issue?
Thank you.

Hi @mypil, this issue is fixed via PR #4923. Thanks!

@learningdaily : The documentation will be updated with the .svc ending, we just need to wait a few days for the migration from Github to docs.microsoft.com to take place.
Your issue was already reported in issue ticket #4841 .

@learningdaily - From our understanding, the issue has been resolved. If you feel it hasn't been resolved, please re-open this issue.
Thank you for your contribution to make the docs better! Much appreciated!


AMA session #4 Dutch translation

Added the Dutch translation and added a link to it in the readme file.

LGTM. Thanks so much for the translation!


filenames with special keywords result in unusable imports

Describe the bug
My issue:
I have a file called delete.ts, when fetched by auto-import results in const delete which of course generates a runtime issue when used.
Perhaps we could allow to add a prefix to the exports? This way the problem would be solved
Reproduction
Create a file called delete.ts
System Info
System:
    OS: macOS 13.0.1
    CPU: (10) arm64 Apple M1 Pro
    Memory: 319.97 MB / 16.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 16.14.2 - ~/.nvm/versions/node/v16.14.2/bin/node
    Yarn: 3.3.1 - ~/.nvm/versions/node/v16.14.2/bin/yarn
    npm: 8.17.0 - ~/.nvm/versions/node/v16.14.2/bin/npm
  Browsers:
    Chrome: 108.0.5359.124
    Firefox: 108.0.2
    Safari: 16.1

Used Package Manager
yarn
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A, please open a GitHub Discussion instead.
[X] The provided reproduction is a minimal reproducible of the bug.

I'd consider this out-of-scope. In general, I think you should avoid using keywords for auto-import.

what do you mean by avoiding keywords? That files shouldn't have the names of keywords? I don't see how this can be out of scope, since it doesn't seem smart to have a software library that generates un-runnable files without any warnings...


Refactor Stream Inheritance

In the current state, the base object is MassComposition, while subclasses are: Sample, IntervalSample, BlockModel and Stream.
The Stream object relates to an edge of a Flowsheet since it has knowledge of node_in and node_out.
The reality is that Sample, IntervalSample and BlockModel can also be a Stream object.
This task is to refactor the inheritance so that Stream is not a direct subclass of MassComposition but rather a subclass of Sample, IntervalSample and BlockModel .  Math operations performed on these objects capture the relationships and so will return Stream objects, that are also of the superclass type such as Sample.

The intent has been delivered, however Sample is still a subclass of MassComposition.  The _nodes property has been removed from the MassComposition base class and instead is replaced with the nodes property of the Stream class.  When math operations are called, or when a class method is called directly, any MassComposition derivative object becomes a Stream object, but also retains it's original type.


fix typos

fix typos.
Signed-off-by: allencloud<EMAIL_ADDRESS>

Thanks @allencloud, one comment, but LGTM otherwise (but I'm not a maintainer in this repo :smile:)

@thaJeztah Thanks for your feedback.😄 Actually I am doing a deep learning of libnetwork.

thanks! LGTM

LGTM


[Feature]: permit number matchers instead of literal values

🚀 Feature Proposal
Matchers like stringMatching can be used in place of literal values in toEqual-like contexts, e.g.
describe('4xxresponse', () => {
  it('string matching', () => {
    const response = callMyApi(payloadCausing4xxResponse);
    const expectedData = ...;
    expect(response).toEqual({
        statusCode: expect.stringMatching(/^4\d\d/$), // for response like `"statusCode": "429"`
        data: expectedData,
    });
  });
});

The proposal is to allow this to work for number-matchers as well, e.g.
describe('4xxresponse', () => {
  it('number matching', () => {
    const response = callMyApi(payloadCausing4xxResponse);
    const expectedData = ...;
    expect(response).toEqual({
      statusCode: expect.toBeGreaterThan(400), // for response like `"statusCode": 429`
      data: expectedData,
  });
});

Specifically the following matchers:

toBeGreaterThan
toBeGreaterThanOrEqual
toBeLessThan
toBeLessThanOrEqual
toBeCloseTo EDIT: Added 2021-11-10

Motivation
Same as the reasons for permitting string matchers in place of literal values;

easier to write, just one expect(recieved).matcher(expected)
easier to read (the test is a description of what we expect the result to look like, rather than a long list of assertions)

Example
Refer to feature proposal 🙂
Pitch
It would be nice to have feature parity with those string matchers that work in place of literal values. Also imo the current situation (where string matchers work instead of literals but number matchers do not) is a bit confusing for a newcomer (e.g. "shoot, which matcher can I use in what place?", speaking from experience).

Hey, I'd be happy to create a PR for this one.

https://github.com/facebook/jest/pull/13669#issuecomment-1369434030


ci: Move MacOS builds to CircleCI

Description of Change

This PR moves our MacOS builds from VSTS to CircleCI as we are discontinuing use of our MacStadium servers on VSTS.
Checklist

[x] PR description included and stakeholders cc'd
[x] PR title follows semantic commit guidelines

/trop run backport-to electron-3-0-x


change min_commit uncommitted indicator

currently an un-committed entry is marked by min_commit = 0
This complicates finding the active entry when reading the branch current state, which includes uncommitted entries. after reading the first row, we need to read all other entries until the last entry of that path. only then we will know if there is an uncommitted entry that overrides the top entry.
Th change is to set min_commit to MaxCommitID when it is uncommitted. now when an entry is uncomitted - it is the first one in the index order. so we immediatly know that the active entry is the first one we get.

Hey @tzahij, I'd like to help in solving this issue. Can you help me with the files I need to change?

Hi @haardikdharma10 assigned this task to tzahij as this change touches a lot of the groundwork he introduced to the project.
I suggest any "good first issue" labeled first, I can help if you have questions now to address the problem and contribute.

as we're changing data model, I'm closing this in the meantime. Please reopen if relevant.


[Regression since beta 9] Can't add a custom class (class_name) via editor, if it extends another custom class.

Godot version:
3.1.11
OS/device including version:
Windows 10 64bit
Issue description:
Cant add a custom class_name node from the inspector if it extends from another class_name class. It also shows no icon in the editor. Adding a normal node and then adding the script manually works. Error code:
 ERROR: instance: Condition ' !ti ' is true. returned: __null At: core/class_db.cpp:512 ERROR: _create: Condition ' !c ' is true. At: editor/scene_tree_dock.cpp:1698
Steps to reproduce:
Try to add "ClassTwo" to the scene via editor
Minimal reproduction project:
Test.zip

same here! Can we not move it to milestone 3.1 ? Since 3.1 introduced the class_name customization might be consistent to make this work, maybe ?
I am not sure it worked before beta11 though, did it, @DavidMannes  ?

Same here. Is there a workaround?

If you save a PackedScene with a Node with a script that extends the another custom node ?

Yeah, workaround would be to make a scene with one node or as i said in OP, its possible to add a normal node and then attach the script to it, like you would do in 3.0 and before.
I tried to locate the error and it seems that 01a3dd389cc84b9a059fe8e45f2d51ce1d69eebc is the cause, before that its fine and after that its bugged.
@reduz

I tried to locate the error and it seems that 01a3dd3 is the cause, before that its fine and after that its bugged.

That's clearly not possible, this code is only rendering in GLES2.

The bug isn't there in 3.1.8, but in 3.1.9 it is, so i looked at https://github.com/godotengine/godot/compare/a32b26dfa26f2a039bf9c84b90d10666bcf785c9...a5370b1b1bed3744941c90b4a05d4516aabd4c67 to find the time when the bug appeared, but apparently that doesnt including every change or they arent sorted by date.

I can confirm that used to work in beta8 but not beta9 forward

Thanks, I'll bisect between a32b26d and a5370b1 to see what caused it.

I double checked on my mac, and it actually doesn't work on beta8. But it does on beta7.
Can someone else confirm ?

I double checked on my mac, and it actually doesn't work on beta8. But it does on beta7.

It works fine for me with both beta7 and beta8 on Linux.

Bug is on Windows since beta9, but i dont have it on Linux with beta9, will try some other versions now.

Regression from f669ebeeaf75a00b4c66dc92f62bd5030cb11660, CC @reduz.
f669ebeeaf75a00b4c66dc92f62bd5030cb11660 is the first bad commit
commit f669ebeeaf75a00b4c66dc92f62bd5030cb11660
Author: Juan Linietsky<EMAIL_ADDRESS>Date:   Tue Feb 26 18:43:37 2019 -0300

    -Properly handle missing ETC support on export
    -Added ability for resource importers to save metadata
    -Added ability for resource importers to validate depending on project settings

:040000 040000 6b119a5ab359ee8bdc27fd1bbf0cad0e5f7c2554 aa674f59d899d59d5e4c05277dc3b5078e4024fd M      core
:040000 040000 f16a25922520502cd77ed8ebe60a26a4ceb60d57 11586734a6ee4efe9c5a464b914fa3a5ca07d1c3 M      editor
:040000<PHONE_NUMBER>53feb6976a0156185448c733a8e5df145f34 74d09318a46a2795dba4f4919f99bdefc0baeaea M      modules
:040000 040000 4f16fe5ff5d2deb0007832a98d37f01a6f1cc8b0 65c7c673685e24765f9e81dbd90c027cd60b6844 M      platform
:040000 040000 ab8d3d3c28578a0f18488c85f2235ff4efb6d767 ec51e3f53a7f3d689cdf9e47af66eeff6fda0d03 M      servers

$ git bisect log
git bisect start
# good: [a32b26dfa26f2a039bf9c84b90d10666bcf785c9] Several fixes to make GLES2 on HTML5 work much better. Changed math class error reporting to be a bit less paranoid.
git bisect good a32b26dfa26f2a039bf9c84b90d10666bcf785c9
# bad: [a5370b1b1bed3744941c90b4a05d4516aabd4c67] -Fix problem of order of import plugins, closes #26340 -Ensure resource previewer does not start until first import is done
git bisect bad a5370b1b1bed3744941c90b4a05d4516aabd4c67
# bad: [464e1142c48da5d24fb9d2462b707b695f9b5adc] Don't crash on previewing an AtlasTexture without a region
git bisect bad 464e1142c48da5d24fb9d2462b707b695f9b5adc
# good: [7f289304af9484ad3aa145adc221d5b12608d539] Fix #26100 by casting to integer.
git bisect good 7f289304af9484ad3aa145adc221d5b12608d539
# bad: [f669ebeeaf75a00b4c66dc92f62bd5030cb11660] -Properly handle missing ETC support on export -Added ability for resource importers to save metadata -Added ability for resource importers to validate depending on project settings
git bisect bad f669ebeeaf75a00b4c66dc92f62bd5030cb11660
# good: [aea5980f012c1094b116a1bcb4c37e63a83e4ecc] Merge pull request #26314 from jahd2602/export-icon-dpi-fix
git bisect good aea5980f012c1094b116a1bcb4c37e63a83e4ecc
# good: [3b0f3016609d4915b430738125b85a16188f1691] Merge pull request #26312 from ericrybick/master
git bisect good 3b0f3016609d4915b430738125b85a16188f1691
# first bad commit: [f669ebeeaf75a00b4c66dc92f62bd5030cb11660] -Properly handle missing ETC support on export -Added ability for resource importers to save metadata -Added ability for resource importers to validate depending on project settings

It turns out that the bug is not caused by f669ebe, but by a later commit (probably bda60bf).
f669ebe just changed the name of the filesystem cache file, and if it has been generated by a commit after bda60bf, the bug appears (it's not re-created when using an older commit).
If you delete the filesystem_cache5 file in %APPDATA%/Godot/projects/<project name>-<hash>/, it should work fine in beta 9 or beta 10, but start failing in beta 11.


Values of some parameters

In the module named "MODELS" :
class GCN(nn.Module):
def init(self, nfeat, nhid, nclass, dropout):
but the values of these paramters are not specified in the code. I am confused how the code gets these values! nfeat, nhid, nclass
would be great helping me out

Please check line 45-48 in "train.py":
model = GCN(nfeat=features.shape[1],
nhid=args.hidden,
nclass=labels.max().item() + 1,
dropout=args.dropout)

Please check line 45-48 in "train.py":
model = GCN(nfeat=features.shape[1],
nhid=args.hidden,
nclass=labels.max().item() + 1,
dropout=args.dropout)

Dear Jaborie, I appreciate your response. I was wondering do you happen to know in layers module:
class GraphConvolution(Module):
def init(self, in_features, out_features, bias=True):
Where the values of these parameters are fed into the system? They have not been mentioned anywhere in the code: in_features, out_features
Thank you in advance


How to connect to the client?

Deploying the client yields a default Tomcat page. What am I missing?

It's possible you're not missing anything - it might be broken :(

@seanknox did you try to add « /mrp » in the path ?

I did not! Will try that.

From: Julien Stroheker<EMAIL_ADDRESS>Sent: Monday, August 13, 2018 3:03 PM
To: Microsoft/PartsUnlimitedMRPmicro
Cc: Sean Knox (KUBERNETES); Mention
Subject: Re: [Microsoft/PartsUnlimitedMRPmicro] How to connect to the client? (#21)
@seanknoxhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fseanknox&data=02|01|Sean.Knox%40microsoft.com|dedf27c75fa846d4a1f808d601689036|72f988bf86f141af91ab2d7cd011db47|1|0|636697945943685170&sdata=6pgaueiiUElPmpgc24xlXRjgxHUnlEMfwiSoDQOzGtE%3D&reserved=0 did you try to add « /mrp » in the path ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2FPartsUnlimitedMRPmicro%2Fissues%2F21%23issuecomment-412680531&data=02|01|Sean.Knox%40microsoft.com|dedf27c75fa846d4a1f808d601689036|72f988bf86f141af91ab2d7cd011db47|1|0|636697945943685170&sdata=XZVLgVY49eC2FRuv2F80jZ3Er7asjapyUk0C%2FelO2kY%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABLLDiAxdMyGR0PRt-C51gB8LHYofI8_ks5uQfeggaJpZM4V5Bwr&data=02|01|Sean.Knox%40microsoft.com|dedf27c75fa846d4a1f808d601689036|72f988bf86f141af91ab2d7cd011db47|1|0|636697945943695178&sdata=%2Fa4q%2Bm6bQyeSIoU4EJOoAqu7OBrrHgTmsf3jB4yBcdo%3D&reserved=0.

@seanknox The endpoint is at /mrp_client/ which is the name of the WAR produced.

Just verified - @lalo is correct.  It is served up on the /mrp_client/ URL.  I'm adding as a note to the readme.


Update MuddySwamp.py to add more Administrator commands

For those unfamiliar, when you run MuddySwamp.py, the script logs all activity, and also accepts input from the person running the script. As the administrator (person running the script), you can enter commands.
Right now we have:
broadcast [message] : send message to all players
players: list players
help: print help menu

Possible commands include:

kick: force a player to disconnect (may require changing the mudserver.py code)
whisper [player] [message]: tell [player] a [message]
move [player] [location name]: forcefully move a player to another location
give [player] [item]: give a [player] an [item]

I'd love to hear any more ideas for Administrator commands!

I will be breaking this up into smaller issues that specifically tackle each desired command.


Register EMSL project prefix on identifiers.org

I recommend registering EMSL.project as a prefix just in case the same 5 digit number is reused for other entities, e.g. metabolites.
It looks like the redirect should be to:
https://search.emsl.pnnl.gov/?project[0]=projects_$LocalID
E.g.
https://search.emsl.pnnl.gov/?project[0]=projects_51283
is this correct @SamuelPurvine
See https://microbiomedata.github.io/nmdc-schema/identifiers/
Note: once registered with identifiers.org it should percolate out to n2t, bioregistry, prefixcommons, ...

Wow, I never saw that this was assigned to me, apologies! I'm fairly certain I do not have anything like the level of paygrade to register anything having to do with EMSL onto an external resource. Certainly I would need to work with @lamccue about this. I know when we discussed having someone at Kitware do this there was pretty strong pushback.

https://bioregistry.io/registry/emsl.project
see also

https://github.com/microbiomedata/nmdc-schema/issues/1139


Fixed char const-ness warning

C++ does not allow to implicitly cast away the const modifier, which
resulted in a warning for term and delim parameters. This patch changes
the types of these parameters and fields to const char *, since they
are not intended to be mutable.

Thanks nythil.


Update linters and gofmt script

What this PR does / why we need it:
Update golangci-lint configuration to explicitly list all enabled linters and add a few more to the existing set. The original change was driven by a desire to  remove gofmt scripts and relying on golangci-lint in combination with goimports to provide formatting and validation.
Second commit fixes all lint failures.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Sorry for PRs like this but as we're working on the codebase more, I would like to ensure we have as good tooling for keeping a consistent codebase as much as possible.

/cc @hectorj2f

@irfanurrehman @RainbowMango do you have any comment regarding this change ? Otherwise I'd say, we can merge it.

Just quickly went through the PR. LGTM. Thanks.
Would you like to take an eye on it? @irfanurrehman

I can imagine it becomes harder to make the robot happy:)  Hope people/contributor won't get crazy with so many linters :)
Maybe we should consider improve the performance of CI job later.

I'd wait for @irfanurrehman before giving it the #/lgtm

@RainbowMango the majority of these linters were already applied by virtue of being on by default. This pr just ecologically specifies then to reduce chance of surprise on upgrade of golangci-lint.

Thanks for doing this @jimmidyson.
I have just one suggestion, pending which the PR looks good to me.

/lgtm


Adding input output and valid feature types to the model schema

In addition to the fields currently tracked in the schema, we should add:

The type of features that this model can be used for.
The expected input and output structure that the model uses.

https://github.com/ludwig-ai/model-hub/pull/10/


Wait to initialize auto update manager until config is loaded

Fixes #17172
This regressed in #16628.

FWIW, Here was my testing strategy:

Disable the Automatically Update setting.
Add the following line to the beginning of AutoUpdateManager.scheduleUpdateCheck:

console.log('scheduleUpdateCheck')

Run atom --dev --foreground.
See the scheduleUpdateCheck message appear. This confirms the bug; that method should not be getting called.

After this change, the message no longer appears.


Cannot interact with certain modal views

Part of the font selection modal view from lxappearance is unselectable.
Steps to reproduce:

Open lxappearance
Click the font selection box
You are now unable to move/resize the window from the bottom and you are unable to click the OK and Cancel buttons

This works as expected inside wlc/example

An update on this: If lxappearance is set to floating, and then you open the font selection you are able to resize of click the buttons

I can reproduce this and have similar issues for many other kinds of windows. I will investigate this soon.

This problem is more clearly exposed with borders enabled:

Thanks. Window borders does reveal a lot about this problem. Pulled in @Cloudef, since I'm betting this is a wlc thing.

It looks like the floating view has an offset of the parent view origin.

But the borders are correctly calculated

Right. I'm thinking that wlc is giving us one geometry for the window and rendering it in a different place. It's probably more subtle than that, though, because interactivity with these windows is weird.

Both parented view's and subsurfaces have special logic. If this is either one of them, I would not be surprised if there was bugs. If input coordinates are wrong, then the surface is not rendered in correct place or vice versa.

@SirCmpwn Did you guys use wlc_get_visible_geometry that was recently added for visual stuff though?

Doing the below will set the floating view to the parents x,y. Not sure how to fix it yet though
diff --git a/sway/container.c b/sway/container.c
index 21a929b..75eb8c1 100644
--- a/sway/container.c
+++ b/sway/container.c
@@ -311,8 +311,8 @@ swayc_t *new_floating_view(wlc_handle handle) {
        wlc_view_get_visible_geometry(handle, &geometry);

        // give it requested geometry, but place in center
-       view->x = (swayc_active_workspace()->width - geometry.size.w) / 2;
-       view->y = (swayc_active_workspace()->height- geometry.size.h) / 2;
+       view->x = 0;
+       view->y = 0;
        view->width = geometry.size.w;
        view->height = geometry.size.h;

It uses sway's internal state for that view, which is calculated by sway and handed to wlc_view_set_geometry.

We only use wlc_view_get_visible_geometry now with the border stuff merged fwiw. (I kind of forgot about this, but it is the right thing to do anyway).
Doesn't solve this problem directly though.

Really? So far as I can tell, that's not the case (after reading through the relevant code).

It was sneaked in here: https://github.com/SirCmpwn/sway/commit/5a13cb0ed136906a4370235214601b0129548c49#diff-5125e6a182d823f6a2ad1c0b7a576892L907 and 2 other places in that commit, to replace wlc_view_get_geometry

Ah. Cool.

Fixed by https://github.com/Cloudef/wlc/issues/145


scopeToImports feature

It could be useful to have a scopeToImports function which when given a scope URL will flatten that scope into "imports" where possible.

With the new model of "imports" first, this no longer makes sense.


Update free-programming-books-langs.md

What does this PR do?
Add resource(s)
For resources
Description
Fundamentals of Database Systems by Elmasri and Navathe
Why is this valuable (or not)?
How do we know it's really free?
It's a pdf format book provided by Central Connecticut State University on their website
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

This appears not to be an authorized distribution. Please see docs/CONTRIBUTING.md guidelines.
If you can find some connection between the authors and CCSU, please reopen the pR - also add authors


basis of record in the IPT: "occurrence" vs "taxon" and "event"

The current accepted terms for basis of record according to TDWG are:  PreservedSpecimen, FossilSpecimen, LivingSpecimen, MaterialSample, Event, HumanObservation, MachineObservation, Taxon, Occurrence.
However, on GBIF.org, not all of them are accepted/interpreted: Taxon, Occurrence and Event are flagged and interpreted as unknown.
For some reason, the IPT considers Occurrence a valid term but not Taxon and Event: https://rs.gbif.org/vocabulary/dwc/basis_of_record.xml
This means that several publishers publish data with an Occurrence basis of record that are interpreted as unknown (see examples here).
I don't think it makes much sense. Since we already excluded Taxon and Event from the list of accepted terms in the IPT, I suggest to remove Occurrence as well (or have everything).
Perhaps for the next IPT version?

@ManonGros https://rs.gbif.org/vocabulary/dwc/basis_of_record.xml says:
A resource describing an instance of the Occurrence class. NOTE: this value is ambiguous and hence should only be used when the resource type is unknown.

Fair enough, perhaps we should just have "unknown" instead of "occurrence"? it would be more explicit

From Tim:
Just bear in mind that https://rs.gbif.org/vocabulary/dwc/basis_of_record.xml is not really ours to change... it's defined in the DwC (https://dwc.tdwg.org/list/#dwc_basisOfRecord) and we just reformat that into the XML flavour for the DwC-A
I think we should leave it as it is


encoding/json: unmarshalling same field into two struct fields

Please answer these questions before submitting your issue. Thanks!

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

What operating system and processor architecture are you using (go env)?
linux, amd64

What did you do?
I'm trying to unmarshal the same JSON field into two struct fields.
http://play.golang.org/p/K86eAWaVyr

What did you expect to see?
Struct fields must have JSON value or json.Unmarshal() method must return an error.

What did you see instead?
Struct fields are empty and no errors returned.

Hopefully this is fixable without compatibility worries.

I took a look at this. I don't have a complete answer for it, but want to document my findings here.
This behavior is caused by the "annihilation" logic shared between encoding and decoding. For encoding, this makes sense: since the two sources might have different value, we might as well not write anything. One might argue that this should be an error. I'm not sure what the rationale is, but the doc on dominantField function ( https://golang.org/src/encoding/json/encode.go#L1139 ) and a test (TestDuplicatedFieldDisappears) indicate that it's a deliberate decision.
Since encode makes a decision to not error, I think we should refrain from returning an error from the decode side. One way to solve this issue is to give decode its own version of dominantField that does "union" instead of "annihilation", setting all "conflicted" fields to the same value.
Of course, another way out is to document this behavior and keep the implementation as is. What do you think?
(Please let me know if golang-dev is a better place for this discussion.)

ping @bradfitz . Is this too late for 1.7 freeze?

Things aren't frozen yet, but this would need to have a decision and an implementation soon. I'm not familiar with any past decisions here.
/cc @rsc

I thought a little more about this. Currently dominantField chooses tag over field name. If we make a change along the way I earlier described ("union" instead of "annihilation") we now have to answer more questions like: If we have tagged fields, do we set the fields that are untagged anyway?
Furthermore, we will probably want to document this behavior to prevent confusion in the future. It would be confusing if Marshal and Unmarshal follow different rules. Rule for Marshal is already documented, so we might as well follow it.
I am leaning towards retaining the old behavior. Will this need to wait for 1.8? I'd appreciate your thoughts. @bradfitz @rsc

The annihilation logic is intentional. I don't think we can change this. This use case is already defined to ignore both. I apologize.


Adds Hill Climbing Search diagram for chapter 4

This commit

Refactors previous code a little to reuse code for the 2nd diagram
Adds Hill Climbing Search Visualization where user can click to decide the position of the robot and watch as it climbs the hill using Hill Climbing Search
Beautifies index.html for Chapter 4

Cool, generators!
Color suggestions:

For larger areas, use less bold colors (either lower 's', or 'l' away from 50%)
Use bold colors to draw attention to something small and important (higher 's', 'l' near 50%)

Specifically, GRMA and maxima in the new diagram: GRMA is big so it should be less bold; the maxima are small and important so they could be more bold. Maybe try
var colors = {
  hill: 'hsl(217,30%,70%)',
  maxima: 'hsl(110,100%,75%)',
  unvisitedMaxima: 'hsl(110,100%,50%)',
  GMRA: 'hsl(110,25%,60%)'
};

Updated as per suggestions.


Resolves #23

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@30c72f3). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             master       #90   +/-   ##
==========================================
  Coverage          ?   100.00%
==========================================
  Files             ?         8
  Lines             ?       631
  Branches          ?         0
==========================================
  Hits              ?       631
  Misses            ?         0
  Partials          ?         0

Continue to review full report at Codecov.

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

LGTM


community: Added Lantern as VectorStore

Support Lantern as a new VectorStore type.

Added Lantern as VectorStore.
It will support 3 distance functions l2 squared, cosine and hamming and will use HNSW index.
Added tests
Added example notebook

Overall works for me. I need to check on the integration tests (whether we need any changes there). Would be helpful to include more useful docstrings

Thank you for detailed review!
I did the requested changes feel free to recheck.


Typely.Core v1.1.0-alpha

Codecov Report
Patch coverage has no change and project coverage change: +2.78 :tada:

Comparison is base (3ea670b) 94.87% compared to head (75aa396) 97.66%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main       #8      +/-   ##
==========================================
+ Coverage   94.87%   97.66%   +2.78%
==========================================
  Files          28       21       -7
  Lines         859      727     -132
==========================================
- Hits          815      710     -105
+ Misses         44       17      -27

see 7 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Add "Create Issue" button to repo view

Once native issue creation is supported, we should replace the "Report a Bug" button in Settings with a native UI.

YESSSSS
Also we can reuse the same thing for bug reporting

@rnystrom Gonna tackle this tomorrow, I’m thinking of just creating a UIScrollView with a couple fields and labels - obviously with a bit of finessing was this across the lines you were thinking?
Don’t wanna go completely down different route

@Sherlouk I guess all we need is:

Title UITextField
Body UITextView

I'd love to be able to preview the body like we do w/ issue/PR comments. Maybe still have the input accessory controls that IssueViewController has?

Hmm just trying to find a nice way to lay it all out - Sensing a really complicated view controller with all the fields with validation, previewing, markdown, etc.

Ya, it could balloon really fast. Maybe check out some open source form things? Also I browse through dribbble a lot when looking for inspiration.

Had a gander at the major open-source form providers, but non of them give us markdown/preview functionality. Though Eureka at least does allow "custom cells".
It may at least provide a nice wrapper around all the validation etc? Not sure if it's worth it

Ya probably more trouble than its worth. We can start super simple maybe. How about not even using UIScrollView. Just have a title UITextField at the top and a scrollable UITextView beneath it? Size it so it responds to the keyboard. Then put the input controls as the accessory view, and preview button as a nav item bar button item?

Got a little something in progress now, will ping some screenshots soon!

Slightly bigger issue, there's no GraphQL API for new issues 🙄

Lol welcome to my hell. GitHawk atm uses a mixture of v3 and v4 APIs to work. In v4 (GQL) you can add comments and reactions, but you can't add labels or close issues... 🤷♀
We'll have to use this API to create issues.
Here's how I mix v3 and v4 in Issues.


Fix rat_excludes broken paths

What does this PR do?
Fix rat_excludes broken paths
Which TC components are affected by this PR?

[ ] Documentation
[ ] Grove
[ ] Traffic Analytics
[ ] Traffic Monitor
[ ] Traffic Ops
[ ] Traffic Ops ORT
[ ] Traffic Portal
[ ] Traffic Router
[ ] Traffic Stats
[ ] Traffic Vault
[ ] Other _________

What is the best way to verify this PR?
Check all that apply

[ ] This PR includes tests
[ ] This PR includes documentation updates
[ ] This PR includes an update to CHANGELOG.md
[ ] This PR includes all required license headers
[ ] This PR includes a database migration (ensure that migration sequence is correct)
[ ] This PR fixes a serious security flaw. Read more: www.apache.org/security

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/2837/
Test FAILed.


[FLINK-23352][streaming-java] Add configuration for batch size and socket timeout of collect sink

What is the purpose of the change
Collect sink has a batch size which is currently not configurable. By the user request in FLINK-23352 we're making it configurable in this PR to support user with super large row size.
Note that these configuration are only available for table APIs. As we cannot read user config in data stream API, data stream users will have to write their own Java code to create collect sink explicitly if they want to configure these.
Brief change log

Add configuration for batch size and socket timeout of collect sink

Verifying this change
This change added tests and can be verified by running the added tests.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

Thank you for above comments，but I still hava a question. Is there a maximum limit on this value? If users can configure it as much as possible, there should be a maximum limit, such as not exceeding the size of taskmanager's taskmanager. memory. process. size？
Thank you for your reply！


lower MSRV to 1.31 and test it in CI

make it explicit that MSRV does not cover cargo test-ing this crate
remove unidiomatic edition 2015 syntax from API documentation

We can lower the MSRV if we only claim that the crate can be built with the specified version. When used as a dependency, the dev-dependencies (like quickcheck) won't be built so that ought to be fine (cast has no dependencies, only dev-dependencies).
Also, I'd like to not make the MSRV depend on third party crates because they (or their recursive dependencies) may change their actual (which is not the same as documented) MSRV at any time.
cc @briansmith

As a consumer I am more interested in knowing which version of Rust the test suite has been verified to pass in, but also I admit I seem to have different priorities w.r.t. MSRV than you.


Document how to retrieve unmappable bundles

User Story
As a ReportStream engineer,
I would like the process for retrieving bundles that could not be mapped to a condition,
so that I can easily review them and potentially add the missing mapping.
Description/Use Case
Add an SOP document that describes how to look up and retrieve bundles that could not be mapped to a condition. We should also determine (or punt) the process for triggering the SOP and recording resolved unmappeds (to avoid future reinvestigation).
Risks/Impacts/Considerations

Verify if 60day limit applies to messages we send to ourselves and if the azure container needs to get cleared out regularly

Dev Notes

There is a specific log message for this (see: UnmappableConditionMessage)

Acceptance Criteria

New SOP document created
Process finalized (or record-ably punted)

Hey team! Please add your planning poker estimate with Zenhub @arnejduranovic @jalbinson @JessicaWNava @JFU-NAVA-PBC @jimmyfagan @thetaurean

Rework this one to reflect new approach
@thetaurean

Reworked to reflect new approach


Whitelist file

Please add an option to specify a whitelist file, i.e. a file that lists files (e.g. newline separated) that are expanded to the PATH... command line argument of borg create, i.e. added to the borg_create_args argument of bsrv.

The next release contains a parameter borg_create_args_file. Every line in the supplied file is added to the end of borg_create_args. Therefore this can be used as a whitelist file, but may also contain arguments.

Implemented in v0.0.1 (dcebe8430d0f0fa6ca5d3816d6fbf55b5ec449f1)