Multicasskop fixes

Fix crd name and drop files that are not used.
Restore dropped PROJECT in main directory.

Q
A

Bug fix?
[x]

New feature?
[]

API breaks?
[]

Deprecations?
[]

Related tickets
fixes #X, partially #Y, mentioned in #Z

License
Apache 2.0

What's in this PR?
Fix issues that were introduced with operator-sdk bump, see
https://github.com/Orange-OpenSource/casskop/pull/376#issuecomment-1074669815
Checklist

[ ] Implementation tested
[ ] Logging code meets the guideline
[ ] User guide and development docs updated (if needed)
[ ] Append changelog

@cscetbon

A PROJECT configuration file to store information about GVKs, plugins, and help the CLI make decisions.
https://sdk.operatorframework.io/docs/building-operators/golang/migration/#what-is-new
If you try to run operator-sdk cli commands without existing PROJECT file(in case you want create new api, webhook) you will get an error.
FATA[0000] failed to create API: unable to load configuration file: unable to load the configuration: unable to read "PROJECT" file: open PROJECT: no such file or directory

If you try to run operator-sdk cli commands without existing PROJECT file(in case you want create new api, webhook) you will get an error. FATA[0000] failed to create API: unable to load configuration file: unable to load the configuration: unable to read "PROJECT" file: open PROJECT: no such file or directory

Can you tell me more about the commands that we need to run now ? Cause I then need to update https://github.com/cscetbon/casskop/blob/master/.github/workflows/tests-and-docker-images.yml#L20-L58. This test didn't catch the missing PROJECT so I imagine it doesn't really do anything with the new operator-sdk

@cscetbon
make generate uses controller-gen for generating crds and k8s projects as expected.
Error with missing PROJECT would appear if we wanted to create new API, webhook, etc using operator-sdk cli https://sdk.operatorframework.io/docs/cli/
So, having PROJECT is mostly meaningful in turns of development new things using operator-sdk functionality.
There is a feature named operator-sdk bundle https://sdk.operatorframework.io/docs/cli/operator-sdk_bundle/
I didn't adopt it for casskop with bump of operator-sdk. I guess operator-sdk bundle validate can be used as a test.
If you think it is a valid thing to do I can add PR for that.

There is a feature named operator-sdk bundle https://sdk.operatorframework.io/docs/cli/operator-sdk_bundle/
I didn't adopt it for casskop with bump of operator-sdk. I guess operator-sdk bundle validate can be used as a test.
If you think it is a valid thing to do I can add PR for that.

I saw you've added that command in multi-casskop but not in Casskop. So yes let's do it and use it to add validation
Thanks


[FIX] babs-merge: bugs when n>5k

Our lab's current (bash) scripts have bugs when handling data with huge number of subjects (e.g., > ~5k).

Matt 12/23/22: we have 2 versions of merging, one of them failed with 1000+ jobs…
Me: coarse test of babs-merge with smaller dataset; then test with HBN 2000+ jobs!

Matt 4/5/23: Those bugs are in the bash script (e.g., this one which we have been used for ~2yr: merge_outputs_postscript.sh) and are hidden as bash commands are not easy to read. They should be gone after translate the bash script into python version of babs-merge.


Is there any movement towards the adoption of flutter for TVs? (WebOS and/or Tizen)

Is there any movement towards the adoption of flutter for TVs? (WebOS and/or Tizen)
Would be great to have a Discussions here, not?

@wilsonneto-dev
this seems more a topic for reddit than for flutter gihub repo
https://www.reddit.com/r/FlutterDev/comments/hcv1mg/can_flutter_build_smart_tv_apps/

Great, thanks so much!


Add region to skr tests

Description
Changes proposed in this pull request:

Add region parameter to provisioning calls for non trial plans
Remove not used subAccountRegion from provisioning url request

Related issue(s)

See also https://github.com/kyma-project/control-plane/issues/3006

/test pre-kyma-environment-broker-presubmit-scanner

/test pre-kyma-environment-broker

/hold

/unhold

/test pre-kyma-environment-broker


back merge with antd

First of all, thank you for your contribution! :-)
Please makes sure that these checkboxes are checked before submitting your pull request, thank you!

[ ] Make sure that you propose pull request to right branch: bugfix for master, feature for branch feature.
[ ] Make sure that you follow antd's code convention.
[ ] Run npm run lint and fix those errors before submitting in order to keep consistent code style.
[ ] Rebase before creating a pull request to keep commit history clear.
[ ] Add some descriptions and refer relative issues for you pull request.

Extra checklist:
if isBugFix :

[ ] Make sure that you add at least one unit test for the bug which you had fixed.

elif isNewFeature :

[ ] Update API docs for the component.
[ ] Update/Add demo to demonstrate new feature.
[ ] Update TypeScript definition for the component.
[ ] Add unit tests for the feature.


关于rich中预定义样式的优化

What problem does this feature solve?
formatter中使用rich，只能先在rich中定义好富文本对象，再在formatter中去使用（比如'{a|这段文本能使用a样式}'）。
希望能在formatter中使用一个计算出来的样式对象，比如 '{a|这段文本能使用a样式}' 这个a不再是预定义在rich中的对象，而是可以在formatter函数中定义的一个对象。这样在使用一些动态的字体颜色、背景色等会方便很多
What does the proposed API look like?

这是我想象中的样子。当formatter中没有定义a时，就使用rich中的a

Reopen please
On Mon, May 9, 2022, 12:32 AM echarts-bot[bot] @.***>
wrote:

I'm sorry to close this issue for it lacks the necessary title. Please
provide a descriptive and as concise as possible title to describe your
problems or requests and then the maintainers or I will reopen this
issue.
Every good bug report or feature request starts with a title. Your issue
title is a critical element as it's the first thing maintainers see.
A good issue title makes it easier for maintainers to understand what the
issue is, easily locate it, and know what steps they'll need to take to fix
it.
Moreover, it's better to include keywords, as this makes it easier to find
the issue self and similar issues in searches.
—
Reply to this email directly, view it on GitHub
https://github.com/apache/echarts/issues/16997#issuecomment-1120695389,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQXZRONAKDBO43IUQJ3TQN3VJCWO7ANCNFSM5VNGNKWQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

In most cases, it should be sufficient to use current implementation with several styles and set the name (like "a") dynamically.

是的，大部分场景使用预定义的样式是够用了。但是能让它更好用不是更好吗

动态的话
formatter(value){
     let map = {
          valueA:richA,
          valueB:richB,
          ...
     }
     return `{${map[valueA]}|${valueA}}`
}

rich:{
    richA:{},
    richB:{}
}

这样就可以吧

@Mr-Lixiaoqaing 你参考下楼上的做法，如果不满足的话，可以提一个 Pull Request 试试~


Feature: Python SDK to support SurrealDB 2.x

Is your feature request related to a problem?
Current surrealdb.py does not support SurrealDB 2.x
Even mentioned on official site: https://surrealdb.com/docs/sdk/python
Describe the solution
Update Python SDK to support for SurrealDB 2.x
Alternative methods
n/a we use python on backend
SurrealDB version
2.0.4
surrealdb.py version
0.3.2
Contact Details
No response
Is there an existing issue for this?

[X] I have searched the existing issues

Code of Conduct

[X] I agree to follow this project's Code of Conduct

It does support 2.x, but in a limited way. For example: only Root-Authentication seems to be supported and leaves Namespace and Database-Authentication behind. For now you could use Maxwell Flitton's surreal-lite.py which does support these methods.

Is there any estimate on when this is coming?

Yeah, it will be great to get the full functionality of v2

Is there any estimate on when this is coming?

no sadly there is no estimate

I don't get it. Python is the largest, strongest AI community. This is literally the folks that would be excited about an AI-friendly database and we don't get even an estimate on support for a version that has been released for months?

Hi @lfnovo,  @Deutscher775, @vadim-su, @martinholovsky 👋  the latest version of the Python SDK has been completely re-written and now supports SurrealDB 2.0. Version 0.4.1 available on PyPI here now makes use of a native Python with binary communication with remote databases, and support for embedded database usage. The native Python library now has support for custom types, including datetimes, durations, decimals, geojson types, Record IDs, ranges, and tables 🎉.
We're working on improved documentation at the moment, and will have an update soon!

@tobiemh Alright man! Now we are talking!! I am going to pretend I am important and my wish got granted after 12 hours :)


Improve contextual ranking

Pagefind ranks results based on term-frequency in the document. This should be improved by ranking the distance of search terms, so that cat hat returns the cat in the hat above the cat wasn't particularly related to the hat.
(i.e. cat . . hat > cat . . . . . hat)

Additionally our term frequency should move to tf-idf, which will be fairly trivial to do.

I was wondering how Pagefind ranks the results. Thanks for contemplating the addition of this!

To expand on the above, the current ranking is a combination of term frequency and the distance from your search term to the word. So if you search for be, a result containing beans should rank higher than a result containing begrudgingly (if both results had the same term frequency).
There's a few things on the roadmap for ranking. One if the contextual ranking mentioned above. Another is TF-IDF or BM25. And probably most imminently is a way to mark regions of the page with a higher weight, so that things like titles can play more of a role in the ranking.
Keen to hear if there's anything specific you'd like to see as part of this!

Thanks for the in-depth explanation! Right now, nothing in particular. I'm looking forward to see how this develops and I'm happy that this exists!

Pulling this back from 1.0 for now, but still interested in exploring it! I'd like to get a good set of benchmarking into the codebase first, so that this can be developed without impacting performance.

And probably most imminently is a way to mark regions of the page with a higher weight, so that things like titles can play more of a role in the ranking.

This is part of the 1.0 release, so there are some improvements coming now at least 🙂

While contextual ranking didn't make it in...

Additionally our term frequency should move to tf-idf, which will be fairly trivial to do.

This made it in at the eleventh hour! In my large test sites, ranking is already a lot better 🙂
(demo @ https://mdn.pagefind.app/ )


USB Type-C to HDMI/DP doesn't seem to work

I haven't been able to get this to work since Big Sur, it stopped working in Monterey.
I use a Cable Matters USB C Multiport Adapter since I need to use DisplayPort to get full refresh rate on my 24" Dell retina monitor. I'll give some basic Type-C to HDMI adapters at try to verify the adapter isn't a problem, but I do recall this adapter working in Big Sur.
Has anyone tested this recently on Sonoma?

I recall that my Apple USB-C to HDMI adapter used to work on Big Sur as well. I'm afraid I don't have the hardware anymore and won't be able to test this, sorry.

No worries, I have 4-5 different adapters here, I'll test and report back.

Doesn't work for me either.
I use a HP Thunderbolt Dock

I also have this problem, don't work on my two usbc hubs. And I found the hub usb sometimes using usb2.0 instead if usb3.0.


List all running tasks

Hello
Is there a way to list all running tasks or is it something that can be added as a feature?
It would be helpful to see all running tasks and we can do whatever with them.
Thanks in advance

@raliqala Released in 2.3.0, please give it a try and let me know if it works as expected!


Update README.md

Adding my iOS shortcut within the Community section

Additional commit edit (6aab7f4) to put in alphabetical order

Thanks!


Make tests verbose

Presently the tests return this:
$ go test
PASS
ok  	github.com/esadkrs/ghosts	1.043s

It would be nice if each test reported its execution, one test per line, say.

Just run
$ go test -v

or
$ gotest

Closing.


Allow settings-only mode

There should be a way to only show settings (and even user-defined lists of settings) in HomeKit as a multi-switch accessory.
This would allow voice control of settings such as speech enhancement or night mode without requiring any other visibility of Sonos in HK.
Ideally, I would be able to configure SonosZP to show a single tile that allowed me to turn on or off Night Mode and Speech Enhancement and any subwoofer I had paired to a given zone/surround setup. This would allow me to automate the "evening" Sonos settings along with lights and other HK devices.

You can change the settings from Siri: use Eve or another decent HomeKit app to create a scene with the setting.  Then recall that scene from Siri.
Only exposing a single setting is beyond the scope of Homebridge ZP.  If you don’t like to see the accessories exposed by Homebridge ZP, don’t use that plugin.  You could probably use a plugin like homebridge-commander to expose dummy switches that issue zp commands to change a setting.


Incorrect Container Startup Script Path in Dinky Helm Chart

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
The current Helm chart for Dinky has an incorrect path for the container startup script. The path is set to /dinky/auto.sh startOnPending {{ .Values.spec.extraEnv.flinkVersion}}, but it should be /opt/dinky/auto.sh startOnPending {{ .Values.spec.extraEnv.flinkVersion}}.
What you expected to happen
The current Helm chart for Dinky has an incorrect path for the container startup script. The path is set to /dinky/auto.sh startOnPending {{ .Values.spec.extraEnv.flinkVersion}}, but it should be /opt/dinky/auto.sh startOnPending {{ .Values.spec.extraEnv.flinkVersion}}.
How to reproduce
The current Helm chart for Dinky has an incorrect path for the container startup script. The path is set to /dinky/auto.sh startOnPending {{ .Values.spec.extraEnv.flinkVersion}}, but it should be /opt/dinky/auto.sh startOnPending {{ .Values.spec.extraEnv.flinkVersion}}.
Anything else
No response
Version
1.1.0
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

参考官网进行修改镜像之后，是需要改动的，同时还需要将MySQL的驱动进行挂载


this.translate() with parameter

Hi I'm doing a form validation
and was going through every property of an object and just making a basic validation whether it was empty or not. and wanted to add to a translation the name of the property that corresponds to the name of the field.
export default {
    data: function() {
        return {
            image: undefined,
            info: {
                name: '',
                description: '',
                dish: '',
                phone: '',
                skype: '',
                email: '',
                languages: [],
                payment: [],
                imageData: ""  // we will store base64 format of image in this string
            },
            error: {
                name: undefined,
                description: undefined,
                dish: undefined,
                phone: undefined,
                skype: undefined,
                email: undefined,
                languages: undefined,
                payment: undefined,
                imageData: undefined
            },
        }
    },
    methods: {

        finalizeHost(){
            //Small validation...
            let foundErrors = true;
            for (var prop in this.info) {
                if (this.info.hasOwnProperty(prop)) {
                    if(this.info[prop] === '' || this.info[prop] === undefined)
                    {
                        **_console.log(this.translate('emptyField', prop));
                        this.error[prop] = this.translate('emptyField', prop);_**
                        foundErrors = true;
                    }else{
                        this.error[prop] = undefined;
                    }
                }
            }
            if(foundErrors){
                return;
            }
        }
}

It translates but the {field} doesn't change it displays "{field}, cannot be empty"
if I put in the template the component <p v-lang.fieldEmpty="{name: 'Name'}"></p> it works.
Is this only possible inside computed?

Hello,
your message emptyField in translate js file is like emptyField: 'The {name} field is empty'? With {name} annotation?
If yes, you need to pass an object to translate method e.g this.translate('emptyField', {name: 'age'}), so the message is translated to The age field is empty.
To simplify this, in your case, you can transform your message to emptyField: 'The {0} field is empty', so you can pass only the value to translate method, e.g: this.translate('emptyField', 'age'), and get the same result.


Add an option to send an empty commit in github.dev/vscode.dev

Currently if you try using the vscode git UI, you can only send a commit, if some changes have been made to the code base. It would be nice to have an option of sending an empty commit using the UI.

Here's a scenario where it could be helpful. Say, you have a repository which runs various CI's. But you don't want to change anything in the code base. You only want to send an empty commit, such that the CI(s) are triggered because of the commit. Yes, I can issue the following command from the command line. But, I can't if I am using github.dev (without a command line/codespaces).
git commit --allow-empty -m "This is an empty commit"

An example:
When conda-forge feedstock's are created, sometimes you would need to send a PR with an empty commit, to trigger building the package(s) on conda-forge channel. I find it quite useful to work on the conda-forge PR's from github.dev. So, allowing to send an empty commit with an optional message would be really helpful.

This will be available with the next release of RemoteHub. We ship with the rest of VS Code in ~2 weeks.

Note for team feature verification during endgame--please ping me for a VSIX.

@joyceerhl Woohoo!! 🔥🚀

@joyceerhl Could there be another feature, where you can basically pass on a git command on GitHub.dev using VSCode command pallet?
This would allow people to use any command as they need. And based on the usage of such commands on GitHub.dev, you could keep adding more formalized git commands as new features.

@joyceerhl Looks like the pre-release VSIX I have doesn't have this feature. I think I need a different one :)

Verified. It's a bit weird that I must write the message in the SCM view and then go to the command palette to make the actual commit


Angular.js Development Internship

I am Ramanujam.I have added some important information about Angular.js Development Internship .
I request you to please consider it.
THANK YOU

resolve conflicts

only one pr at time is accepted , max two pr in same repo


WIP: fastapi inference server implementation

Inference server based on fastapi that can be started from the command line or from 3D Slicer's MONAIAuto3DSeg module
https://github.com/user-attachments/assets/394a3c4d-b2e1-4877-8a99-c43de7d61e99
addresses #30

Thanks for the updates. I'm reviewing the changes now. Is there any additional changes that you plan to make before this branch is ready for integration?

I've pushed a few minor suggestions instead of commenting on them.

Some changes were GUI tweaks, those should be trivial.
I've also starting to make WebServer in server.py make an inference web server class (it was just a generic process class). It could either make sense to continue specialization of this class (move as much as possible into this class) or my changes could be reverted and this class could be renamed to something like BackgroundProcess and then it could be used for both the web server and the local MONAIAuto3DSeg inference process (because currently starting/stopping the process, getting the process outputs in the background, etc. are implemented twice: in server.py and in MONAIAuto3DSeg.py. Or maybe keep a generic BackgroundProcess class and a separate LocalInference and InferenceServer classes (using BackgroundProcess as parent).

When the inference is performed locally then there is no progress information in the GUI (segmentation completes succesfully). It should be fixed.
Another small thing: when starting a local segmentation server, before installing all the dependencies, it would be nice to tell the user that dependencies need to be installed that may take several minutes and let the user decide if he is OK with that or would like to cancel instead.

In my opinion, the information displayed when running locally or remotely is insufficient as there is barely any feedback provided about the ongoing process.

@lassoan I will experiement with the suggestions you made RE BackgroundProcess and LocalInference, InferenceServer.

In my opinion, the information displayed when running locally or remotely is insufficient as there is barely any feedback provided about the ongoing process.

It may depend on the hardware and model.
On my computer with a GPU all segmentations complete within 1-2 minutes, with a log message printed every couple of seconds (the longest wait between two messages is maybe 10 seconds).
That said, the messages are good for showing that something is happening, but they do not give a good indication of how much longer the segmentation will take. It could be nice to implement overall progress reporting (tqdm reporting is done during inference, but that usually takes a couple of seconds, so again not much info about the overall progress). I've added a separate issue for tracking this feature request: https://github.com/lassoan/SlicerMONAIAuto3DSeg/issues/81

@lassoan I made the changes you requested. Now using a parent class BackgroundProcess and having a LocalInference and InferenceServer inheriting from it. I also made use of the dataclass decorator to keep track of ProcessInfo / SegmentationProcessInfo.
Let me know what you think and we can go from there.

These changes are great! If the segmentation works well at least locally then we can merge this. Two things to fix before merging this:

When doing local segmentation, there is still no progress information is visible at all. Only a "Processing finished." message in the end. With no feedback at all, after 10-15 seconds, most people would think that the processing failed to start.
Wanted to try segmentation in server mode, so started the server and typed the server address into the local server URL. It fails with this error:

Traceback (most recent call last):
  File "C:\Users\andra\AppData\Local\slicer.org\Slicer 5.7.0-2024-09-21\bin\Python\slicer\util.py", line 3295, in tryWithErrorDisplay
    yield
  File "C:/D/SlicerMONAIAuto3DSeg/MONAIAuto3DSeg/MONAIAuto3DSeg.py", line 591, in onApply
    self._segmentationProcessInfo = self.logic.process(inputNodes, self.ui.outputSegmentationSelector.currentNode(),
  File "C:/D/SlicerMONAIAuto3DSeg/MONAIAuto3DSeg/MONAIAuto3DSeg.py", line 1391, in process
    r.raise_for_status()
  File "C:\Users\andra\AppData\Local\slicer.org\Slicer 5.7.0-2024-09-21\lib\Python\Lib\site-packages\requests\models.py", line 1024, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 500 Server Error: Internal Server Error for url: http://desktop-nm47af2:8891/infer?model_name=abdominal-organs-3mm-v2.0.0

The server does not log anything even though log to console and GUI are enabled.
Minor GUI tweaks:

Collapse "Local segmentation server" section by default (advanced option)
Add a label ("Server:") next to the server address combobox, because without that it is difficult to explain users what to do (now I would need to say "type the server URL to the left from the Connect button"; or above the "Segmentation model" selector). Maybe even adding a checkbox "Local processing" would make things even more clear (if unchecked then the server address combobox and "Connected" button would be hidden)
Would be nice to separate the server output from the segmentation output. If the two modes (run segmentation server, run inference) are mutually exclusive that should be clearly reflected on the GUI, for example by having a selector at the very top that lets user choose between them (and then it would hide the entire GUI of the other function)

@lassoan did you check the server logs when trying to run inference with it? Wondering what happened there since it works for me....

No logs are displayed, even if I click log to GUI. Do you see any logs when using local or server inference?

Not seeing any logs in the UI, but Slicer's error log window. Everything works on Linux.

I will work on the "Log to GUI". Do you want to see the actual log information there or also just some custom statements?

@lassoan I will also try on the Windows server we intended for the SimplifiedViewer.

I just installed everything on the Windows and can start a server which I can connect to from my local MBP. I can retrieve model information, but when trying to run inference, I get the same error after some timeout. No signs of an attempted connection on the server-side. I will try with uvicorn directly.

Let me know if you get stuck and then I can try to debug it.

@lassoan I committed a fix for running the server on Windows. Maybe you can test it locally on your machine and confirm. I am going to work on the progress reporting now.

@lassoan Ready to test. Let me know what you think about the progressbar.
A few more thoughts when running remotely. Sometimes, the files to be sent to the server are huge. It may be good to explore more user feedback for uploading/running inference and not having the main loop blocked for the whole process.

@lassoan I made the requested changes. Improved error reporting on the client side and limiting the number of inference requests to 5/minute.
Let me know what you think.

Thank you, I'll test this and let you know!

Thanks for the updates, it is getting really close to be ready. Just a few small things to fix before integration:

If remote segmentation mode is selected but the user did not connect to the server then the segmentation model selector and Apply button (and maybe also the input selectors) should be disabled.
There is still no progress information when doing inference locally (without server connection). This is all I see in the textbox:

Starting...
In Progress
Importing Results
Processing finished.

We would need to see something like this:
Starting...
In Progress
You are using `torch.load` with `weights_only=False` (the current default value), which uses the default pickle module implicitly. It is possible to construct malicious pickle data which will execute arbitrary code during unpickling (See https://github.com/pytorch/pytorch/blob/main/SECURITY.md#untrusted-models for more details). In a future release, the default value for `weights_only` will be flipped to `True`. This limits the functions that could be executed during unpickling. Arbitrary objects will no longer be allowed to be loaded via this mode unless they are explicitly allowlisted by the user via `torch.serialization.add_safe_globals`. We recommend you start setting `weights_only=True` for any use case where you don't have full control of the loaded file. Please open an issue on GitHub for any issues related to this experimental feature.
`apex.normalization.InstanceNorm3dNVFuser` is not installed properly, use nn.InstanceNorm3d instead.
Model epoch 294 metric 0.9070999026298523
Using crop_foreground
Using resample with resample_resolution [3.0, 3.0, 3.0]
Running Inference ...
`torch.cuda.amp.autocast(args...)` is deprecated. Please use `torch.amp.autocast('cuda', args...)` instead.

0%| | 0/4 [00:00
25%|##5 | 1/4 [00:00
100%|##########| 4/4 [00:00
Logits torch.Size([1, 25, 120, 110, 161])
Converting logits into predictions
preds torch.Size([1, 1, 120, 110, 161])
preds inverted torch.Size([512, 512, 685])
Computation time log:
Loading volumes: 5.59 seconds
Preprocessing: 1.69 seconds
Inference: 1.04 seconds
Logits: 0.37 seconds
Preds: 0.38 seconds
Convert to array: 0.27 seconds
Save: 2.95 seconds
ALL DONE, result saved in C:\Users\andra\AppData\Local\Temp\tmpb3b9bk9t\output-segmentation.nrrd
Importing Results
Processing finished.

When segmenting on remote server, progress information appears in the server - good!, However, we don't see anything on the client side, so users may miss some warning messages or other useful information. It would be nice if progress information could be displayed continuously, but if that was hard then at least the segmentation process output should be sent to the client on completion so that it can display it. Since the server interface is a new feature, lack of this feature would not be a regression, so if you would prefer not to spend time implementing this now then you can just create an issue for it.

I've fixed a path-related bug that broke things on Windows and pushed it to this branch.

I've tested it and progress reporting now works well for local inference.
Only two tasks remain now:

If remote segmentation mode is selected but the user did not connect to the server then the segmentation model selector and Apply button (and maybe also the input selectors) should be disabled. Alternatively, the segmentation model list could be cleared. The point of all these is to make it clear for the user if models come from the server or local; and if the segmentation happens locally or on the server.
Send processing output from server to client (optional, may submit issue instead)

Will work on it in a bit. Helping Nicolas was something right now.

I've implemented persistency for the remote processing button (last state stored in QSettings) + disabling the model selection and Apply button if not connected to server.
I've submitted an issue for sending processing output from the server to the client: https://github.com/lassoan/SlicerMONAIAuto3DSeg/issues/85


Seems a bug in editany_lora.py

the function get_pipeline_embeds will throw an error when the tokens of negtive_prompt greater than prompt.

Thanks for the feedback. This bug should be fixed in the new github version. You can try it now.

Thanks for the feedback. This bug should be fixed in the new github version. You can try it now.

It's not useful by the update, because when  count_prompt < count_negative_prompt, the tokenizer of prompt may still > the tokenizer of negative_prompt. However I pulled a request which has solved this problem.


numpy >= 1.24 support

Issue Type
Bug
Source
pip (model-compression-toolkit)
MCT Version
1.7.1
OS Platform and Distribution
mac
Python version
3.9
Describe the issue
np.float is not supported in numpy 1.24

Expected behaviour
No response
Code to reproduce the issue
running mct with numpy 1.24

Log output
No response

Hi @Chizkiyahu, thanks for using MCT.
In NUMPY v1.24 the dtype changes from np.float to np.float32. (also np.int->np.int32).
MCT v1.7.1 supports NUMPY version below 1.24.
So you can try to downgrade NUMPY to version  below v1.24 or get the nightly version which supports NUMPY v1.24.
Shortly, we about to release new version of MCT which supports NUMPY v1.24 as well.
If you have any other issue don't hesitate.


Remove a bunch of generic params from GpioPin

[ ] The code compiles without errors or warnings. - even the original code had warnings.
[ ] All examples work. - if they compile, they should work.
[x] cargo fmt was run.
[x] Your changes were added to the CHANGELOG.md in the proper section.
[ ] You updated existing examples or added examples (if applicable). - as above.
[ ] Added examples are checked in CI
[x] You add a description of your work to this PR.
[ ] You added proper docs for your newly added features and code. - no new features

PR description
We had a conversation with @MabezDev over at matrix where he mentioned that ideally GPIOs should have fewer generic parameters. Since their capabilities are tied to the hardware, this PR refactors those generic parameters into a companion trait.

Great idea! 👍

I haven't used other peripherals yet other than Spi. There, the DMA channel config could maybe be slightly simplified (though I'm not sure about what the SuitablePeripheralX means just yet).


Use 'isSearchingForReports' to control green circular loading on request money flow Contact Search

Details

We got isSearchingForReports from onyx and used it in MoneyTemporaryForRefactorRequestParticipantsSelector to control green circular loading.
Fixed Issues

$ https://github.com/Expensify/App/issues/37144
PROPOSAL: https://github.com/Expensify/App/issues/37144#issuecomment-1961608043
Tests

Click + and select Request money
Enter amount and click next
Search for any contact in the field
Green circular loading ring is seen like when we search for a contact to chat

[x] Verify that no errors appear in the JS console

Offline tests

N/A
QA Steps

Same as Tests.
PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

https://github.com/Expensify/App/assets/95647348/88fb4cdb-2477-4801-93de-a272f1446710

Android: mWeb Chrome

https://github.com/Expensify/App/assets/95647348/bc9aae9c-3803-46e5-bfba-cf36b317a131

iOS: Native

https://github.com/Expensify/App/assets/95647348/07619730-9dcd-48f7-8c68-7fad99ee70fa

iOS: mWeb Safari

https://github.com/Expensify/App/assets/95647348/6ea5a5fd-6684-42c3-a1c3-76f55d07780e

MacOS: Chrome / Safari

https://github.com/Expensify/App/assets/95647348/6a18b4f0-7504-41c7-87f8-82112c951dc8

MacOS: Desktop

https://github.com/Expensify/App/assets/95647348/d1d590e1-ce25-4648-98e2-d15319edd000

@mollfpr I didn't update MoneyRequestParticipantsSelector to have isLoadingNewOptions as you commented, because I still think it's out of scope, and we'll address that here. But if necessary, tell me and I'll work on it.

@mollfpr I didn't update MoneyRequestParticipantsSelector to have isLoadingNewOptions as you commented, because I still think it's out of scope, and we'll address that here. But if necessary, tell me and I'll work on it.

@mollfpr I didn't update MoneyRequestParticipantsSelector to have isLoadingNewOptions as you commented, because I still think it's out of scope, and we'll address that here. But if necessary, tell me and I'll work on it.


Matching Entities with same ID

Hello,
I'm not sure if I'm blind or just don't understand something, but I want to match the same entity when I add one with a plugin.
Currently, if I add the same person a second time, it just creates another one.
I'm searching for something like the "hasPersistendResultIds" option from the connectors.
Or do I need to check the entire chart if the ID is present, before adding it?
Cheers,
Timon

Hi Timon
This issue might give you some clues
https://github.com/i2group/analyze-connect-node-sdk/issues/16
In that the user was experiencing differences in behaviour between i2 Notebook and  i2 Analyst's Notebook Premium.
Currently we do not have quite the same functionality in this area as we have in i2 Analyst's Notebook Premium but with each release we are bringing that closer.
You should however be able to get to a point where items are matched and 'Fused' on the chart so they display as a single item.
From our November 2023 release i2 Notebook now works with system matching rules to help minimise duplication, but did not do so before that.
How things interact is also dependent on the deployment pattern in use.
Can I ask:
What version of i2 Analyze you are doing this with?
What deployment pattern you are using (e.g. daod-opal, chart-storage-daod or information-store-daod-opal)?
If your deployment is also using an Information Store, and you are on our latest i2 Analyze 4.4.3  have you configured system match rules with source identifier matching documented here ?  (Note: the steps are different depending on the  deployment pattern)


Update eff to 5.5.2

Updates org.atnos:eff from 5.5.1 to 5.5.2.
Release Notes/Changelog
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention @scala-steward in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.atnos", artifactId = "eff" } ]

labels: semver-patch

Codecov Report

Merging #40 into master will increase coverage by 1.85%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #40      +/-   ##
==========================================
+ Coverage   66.82%   68.68%   +1.85%
==========================================
  Files           8        8
  Lines         416      396      -20
  Branches       21       21
==========================================
- Hits          278      272       -6
+ Misses        138      124      -14

Impacted Files
Coverage Δ

src/main/scala/io/github/vigoo/clipp/Parser.scala
91.42% <0%> (-0.17%)
:arrow_down:

.../io/github/vigoo/clipp/usageinfo/debug/debug.scala
0% <0%> (ø)
:arrow_up:

...hub/vigoo/clipp/usageinfo/UsagePrettyPrinter.scala
40.22% <0%> (+0.45%)
:arrow_up:

src/main/scala/io/github/vigoo/clipp/clipp.scala
65% <0%> (+9.44%)
:arrow_up:

.../scala/io/github/vigoo/clipp/usageinfo/graph.scala
100% <0%> (+9.52%)
: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 f1d0c9b...611f250. Read the comment docs.


feat(Keepkey): handle factory new keepkey connect

Description

Add new alerts and download link for users to update their out of the box Keepkey.
Preview:  https://web-bryanmutai.vercel.app/
Notice

[x] Have you followed the guidelines in our Contributing guide?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[x] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

closes #2821
Risk

Lower risk as this changes add UI related improvements that don't affect core state management.
Testing

Engineering

Connect a factory out of the box Keepkey wallet,
Click "Connect Wallet" and select Keepkey option
Click "Pair" and attempt to connect wallet. This should fail.
See error alert and follow download process as shown in screenshots below.

Operations

Follow same steps as engineering.
Screenshots (if applicable)

@BitHighlander could you please have a look?

Sorry for the delay in updating this @bitlander @gomesalexandre , I've set up a separate modal to handle the platform detection and showing the user which platform they are using and acting accordingly or clicking on a link to select manually if the platform isn't correct

Hi @gomesalexandre, I've applied changes based on your feedback. Thank you.

Incase a preview build is needed, can't see them on GH build process anymore: https://web-bryanmutai.vercel.app/

Ive tested the onboarding for an OOB (out of box) device and can confirm that we detect and bring a user into the updater app download modal.
This a large improvement over giving new users a generic error without any information on why they cant connect.
+1 👍
:shipit:

Could you please have another quick look @gomesalexandre


dev-dependencies included in packages

there are many dev modules such as eslint and gulp that have dozens of dependencies. Listing each of them for each lambda in the excludes is not ideal.
An includePatterns would help a lot, as would blob matching as opposed to regexp

As a workaround, I'm using:
      "excludePatterns": [
        "node_modules/(?!jaws-core-js)[a-zA-Z-]+",
        "node_modules/(?!dotenv)[a-zA-Z-]+"
      ]

This is only a problem when turning off browserify.


book: add NIP-05 python examples

Description

Added NIP-05 Python Example
Updated book page for NIP-05
Included minor print statement updates to:

json.py
keys.py
nip44.py
nip59.py

Notes to the reviewers
The minor updates to the existing python examples simply make the just test easier to review and keep consistent indentation with the previous examples.
Please take a look at the content of the book page as I've added a couple of sentences there to summarize the NIP-05 functionality, but this will def need a review.
Checklist

[x] I followed the contribution guidelines
[x] I ran just precommit or just check before committing
[ ] I updated the CHANGELOG (if applicable)

Thanks for the review. Agreed, I should have called out the verify_nip05() and get_nip05_profile() this makes more sense and is cleaner as it separates these examples more clearly from the NIP-01 metadata content.
I kept in the event builder (just for the purpose of testing) and left it out of the code snippets that will form part of the documentation.
On a side note I've tried the verify_nip05() a bunch of different ways, with both your verified NIP-05 identifiers "_" and "yuki" as well as a couple of my own for two different sites. What I found were two things:

At first I had issues with mine because of a trailing "," it seemed not to like that my nostr.json had an extra comma (e.g. names{ "alice" : "aliceslongpublickeyinhex", }. When I removed this it was able to correctly parse the json (something I think clients already work around because that previous nostr.json has been working find for Damus/Primal). Maybe good to have something to gracefully handle these extra commas.
Outside of obvious json parsing issues I found that with a range of known verified NIP-05s we seem to return None across the board. Just wondering if something is going awry on the rust side. Would you mind testing and let me know if you see the same.

Let me know if you have any additional suggestions/comments.


Redirect FAQ to Freshdesk

IRC
Now that theming the Freshdesk site is done...
http://gittip.freshdesk.com/support/solutions/folders/1000078951

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Closing in light of our decision to shut down Gratipay.
Thank you all for a great run, and I'm sorry it didn't work out! 😞  💃


Submenus

I cannot get the submenus to work as I think they should in Hugo, so I am thinking they are not supported in Anatole. Is it so?
Thanks in advance!

Regrettably that is currently not possible.On Dec 24, 2020 6:43 PM, gaztelugatxe<EMAIL_ADDRESS>wrote:
I cannot get the submenus to work as I think they should in Hugo, so I am thinking they are not supported in Anatole. Is it so?
Thanks in advance!
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or unsubscribe.


[PE-D] Help Command allows you to navigate away from User Guide

Currently, the help command opens up a window that shows a webpage version of the User Guide. However, it is possible to navigate away from the User Guide to other pages on GitHub, and the user will not be able to go back to the User Guide, even if they try to close and reopen the help window. The only way to get the User Guide back is to close and reopen the app.
Steps to recreate bug:

Open help window via help, F1, or GUI.
Click the GitHub icon the top right corner.

Then the following page will appear. Although there is a way to get back to the User Guide from this point, it will require the user to have a knowledge of how GitHub works.

Even if the user closes and reopens the help window, it will still be on the above page and not on the User Guide.

Labels: type.FunctionalityBug severity.Low
original: danielsimre/ped#9

Related to #138

Fixed in #234


Enrich EF APIs overwrite an existing ExecutionStrategy by default

When someone calls EnrichSqlServerDbContext after they have configured their DbContext with an execution strategy, for example:
builder.Services.AddDbContext<MyDb1Context>(options =>
{
    options.UseNpgsql(npgsql =>
    {
        npgsql.ExecutionStrategy(c => new MyExecutionStrategy(c));
    });
});
builder.EnrichNpgsqlDbContext<MyDb1Context>();

We are overwriting the ExecutionStrategy by default (since settings.Retry is true by default)
https://github.com/dotnet/aspire/blob/c5f1b95332f4e462b2e608dfc006afca0f629661/src/Components/Aspire.Microsoft.EntityFrameworkCore.SqlServer/AspireSqlServerEFCoreSqlClientExtensions.cs#L111-L119
We should consider if this is the best behavior. Other options I can think of would be to detect if the ExecutionStrategy is already set, and log a warning.
cc @DamianEdwards

@sebastienros - I know we had this conversation during the first PR. But @DamianEdwards hit this and we might want to rethink the behavior here.

If we can detect it was set (we might be able AFAIR) I would suggest to not set it then, and log a warning that we are not configuring retries and users should do it in their code.
We could also have a setting that tells Enrich that the execution strategy was configured on purpose and to not log the warning. Like SkipExecutionStrategyCustomization. Setting this flag could be suggested in the warning. Could even be an exception, like we do when a DbContext is registered but users still call AddSqlServerDbContext.


move document package from internal to pkg

The purpose of this PR is to make it possible to reuse runme's Markdown -> Notebook code.
In particular I'd like to use it to parse ChatGPT output into a notebook so that code blocks can be rendered in RunMe notebooks.
In Foyle, there is currently a bug with parsing out code blocks nested inside lists jlewi/foyle#114. Runme's code properly handles this use case. This will allow us to reuse RunMe's code to fix this problem.

Looks like the test failure was a flake because it passed at 8c1e584


Authenticatable parent for model

PHP doesn't have multiple inheritances, so can't extend both Authenticatable and the package model for Authenticatable models. we need a built-in Authenticatable class on the package.

Solved https://github.com/mshamaseen/laravel-repositories/commit/d769d8850dd658c950b5d4cf1d93ad39b5b33af7


add an example about writing Parquet format file

This pull request demonstrates how to write messages into a Parquet format file.
How to run this example:

sbt avro:generate
sbt assembly
bin/gear app -jar jarfile -output file:///tmp/parquet

+1

This test fails which is unrelated
[info] SubmitApplicationRequestServiceSpec
[info] - should return an AppMasterDataDetail dag that matches SubmitApplicationRequest dag *** FAILED ***
[info]   Request was neither completed nor rejected within 1 second (SubmitApplicationRequestServiceSpec.scala:168)

@huafengw Could you add experiments/parquet/README.md with above notes on how to run?

@kkasravi README.md is added

Will submit a new pr based on Kam's work


Why ignore em/strong/span?

I get ignoring something like pre, but was just curious why span tags are not beautified.

:+1: I'm curious about this as well. For my use case beautifying spans improves the readability of some code examples.
This is great overall though, thanks!

The main reason was that span and strong are both inline elements, and js-beautify seemed to produce unexpected results (often) when those elements weren't ignored.
I believe you can override the defaults by passing an unformatted array on the options.

That makes sense. Thanks @jonschlinkert !

Please continue to ignore inline elements. Don't break my markup like Prettier does. Prettier will change:
text (something here) and stuff
to:
text (
  something here
) and stuff

which changes the rendered output from:
text (something here) and stuff
to:
text ( something here ) and stuff
Making the code more legible = good, altering the rendered output = bad.

I believe you can override the defaults by passing an unformatted array on the options.

Although you can do this, it doesn't seem to have any effect: strong and em are still on the same lines as their parents.
Relevant issue: https://github.com/beautify-web/js-beautify/issues/1900
Here is how you can achieve this:
  // eslint-disable-next-line @typescript-eslint/no-explicit-any
  return pretty($.html(), { unformatted: [], inline: [] } as any);

By the way, there are a lot of options that are undocumented and not in the types. Would it be possible to at what's missing?


Update ImageColumn.php

Remove the default of protected bool $header = false;  in ImageColumn.php to reflect the default state as documented at https://ramonrietdijk.github.io/livewire-tables/usage/columns.html#header

Hi @KenKodz,
Thank you for your contribution!
The documentation states that a header can be disabled. However, the documentation also states that image columns have their header disabled by default.
This change would also mean that projects that are currently using image columns would now need to add extra code which I don't really like.
That's why I'm unfortunately going to close this pull request.

Fair enough. I didn't see it in the image column docs. Thanks!


sampling is uninformative

There are several issues with the information provided by the sampling script:

During burnin, it implies to know how long the sampling takes, but doesn't. It'd be better if it conveyed to the user that it samples until convergence.
There's always a warning about not finding the attribute _random in the ConvenienceSampler. I think this is due to a recent update of the emcee package.
When performing a thermodynamic integration, it only provides the value of the beta parameter. But also providing acceptance rates or maybe even mean and standard deviation of the sampled parameters would also be super helpful.

I think this has been largely fixed with a series of commits up to and including 2b836d59b6b1aa061900202efbd5d6c3eb50fee2.


When entering specific values into a polygon collision shape in a tilemap, Godot crashes.

Godot version
4.0.dev (036b7a0)
System information
Arch Linux, Vulkan, Official NVIDIA
Issue description
I'm having a very strange issue. I'm experimenting with tilemaps, and trying to set collision shapes for specific tiles. After figuring out how to add collisions (the master documentation on tilemaps needs to be updated), I tried to create a 4-vertex collision polygon with vertices at { (0,0), (0,3), (3,3), (3,0) } relative to the selected area. Now, all of these happened from the same commit.
When I built Godot without builtin Bullet, Opus, LibPNG, FreeType, and a few others, I got a segmentation fault and the program became broken. I used ThinLTO and Clang/LLD.
When I rebuilt Godot with the same options, but without ThinLTO, the build broke with errors on the final linking.
When I rebuilt Godot with the same compiler, but without builtin options or ThinLTO, the same thing happened as when I used the builtin options without ThinLTO.
When I deleted the build directory and recloned/rebuilt using Clang/LLD, no builtin library options, and no ThinLTO, the build was successful. Note that this was the same scenario and commit as the third scenario, down to the build script used. When I tried to change those specific values I was talking about, Godot crashed with a large memory dump (but no segfault message).
Note that each time I used dynamic C++ linking, because Clang is broken without it (something to do with libatomic).
Steps to reproduce
This is probably a non-reproducible bug, but here is exactly what I did on the last attempt.

Built Godot 4.0.dev 036b7a0 with use_llvm, use_lld, target=release_debug, -j12 and use_static_cpp=false
Opened a previous project where the tilemap I created already had the physics shape, but the vertexes were off.
Tried to change the vertexes of the polygon physics shape for the square ground tile
Once I got to the third vertex, it invariably crashed.

Minimal reproduction project
tilemaps.zip

Duplicate of https://github.com/godotengine/godot/issues/42037 (same cause).


ProcessBuilder: Generic module to build processes within code

Objective of issue: In the repo, Processes are produced on the fly in the SolverProcessBuilder.
That's a feature which should not be hidden in the optimization library, but should be a generic tool in Lava.
In addition, in builder.py, the decorators are currently replaced in this way:
        setattr(solver_model, "implements_process", self.solver_process)         # Get requirements of parent class         super_res = solver_model.required_resources.copy()         # Set new requirements not overwriting parent class requirements.         setattr(solver_model, "required_resources", super_res + requirements)         setattr(solver_model, "implements_protocol", protocol)
That will work with the current decorators; but future pushes to the decorators may break it.
Lava version:

[x] 0.3.0 (feature release)

@phstratmann should this rater be split in a lava issue and a lava-optimization issue?

This issue covers two points:

A feature request for something that can generate Lava Processes on the fly. This should be filed in the lava-nc/lava repository.
An idea to make the lava-optimization code more future-proof, which would be a feature request to be filed in lava-nc/lava-optimization.
@phstratmann Could you please change the stories accordingly?

Replaced by issues #169 and #580


[REVIEW] Astrodevil

GitHub Profile Link
https://github.com/Astrodevil

profile looks good. few tips the profile views badge sometimes gives wrong numbers ( just what I heard from others)
the leetcode badge is not visible in dark mode

Made PR to remove profile views badge and border on stat images: https://github.com/Astrodevil/Astrodevil/pull/4
Might consider using a consistent theme for all stat imgs

@starlightknown @krishguptadev Thanks For The Suggestions🙏🏼. I will make changes.

Looks great, reviewed on live stream...
Some suggestions

broken image under "open source gallery"
suggest removing stats and recent activity, so that blog posts are higher
pinned even number of repos like 2 or 3


Import: Remove io.ReadWriter parameter

This is a breaking change, so we will need to release a 0.2 version
after this.
Signed-off-by: Joe Richey<EMAIL_ADDRESS>

This should be merged after #77 and #75

So it looks like we've already had a breaking change since v0.1.2. When @BenDHillier did #46, we changed:
func CreateImportBlob(ekPub crypto.PublicKey, sensitive []byte) (*tpmpb.ImportBlob, error)

to
func CreateImportBlob(ekPub crypto.PublicKey, sensitive []byte, pcrs *tpmpb.Pcrs) (*tpmpb.ImportBlob, error)

which is backwards-incompatible.
So unless we want to undo that (which I don't), we should just release v0.2.0 after we have make all the backwards-incompatible changes we want to.


lower deployment target even for post-12.0 sys-abi

Follow-up to this comment.
Fixes #157

PTAL @isuruf

Thanks


Full Restart for VS Code

There should be a command to fully restart VS Code. I'm aware of the Reload Window command, but this isn't the same as closing all windows and reopening VS Code. The example of the difference I found is when changing the PATH Environment Variable on Windows, the terminal needs to be restarted to take effect. The Reload Window command is not sufficient to restart the terminal, meaning all open windows have to be manually closed and reopened. As someone who has lots of projects open at once, this is frustrating. Admittedly, this is the only difference I've found so far, so the solution could be that the Reload Window command should ensure the terminal is fully restarted.
Note: This is very similar to Issue #79271, which was closed and locked with an answer which didn't solve the issue.

Version: 1.43.2 (user setup)
Commit: 0ba0ca52957102ca3527cf479571617f0de6ed50
Date: 2020-03-24T07:38:38.248Z
Electron: 7.1.11
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT x64 10.0.18362

There should be a commend and also a option is file Menu to manually Restart


Windows build: Fatal error: pcap.h: No such file or directory

I'm getting the error below when trying to build, am I missing a path or something?  A search shows that the file is located in the following directory:
go\src\github.com\regner\albiondata-client\thirdparty\WpdPack\Include\pcap\pcap.h
$ go build cmd/albiondata-client/albiondata-client.go
# github.com/regner/albiondata-client/vendor/github.com/google/gopacket/pcap
vendor\github.com\google\gopacket\pcap\pcap.go:21:10: fatal error: pcap.h: No such file or directory
 #include <pcap.h>

You need to copy https://github.com/Regner/albiondata-client/tree/master/thirdparty/WpdPack to c:\WpdPack

That did the trick!
Much appreciated.


Lock user account for a period of time after failed login attempt

Algorithm: After X failed attempts, the initial lockout period is 1 minute. After each lockout expires, the next failed attempt triggers the next lockout. The first X lockouts are 1 minute each (technically y^0 minutes). The next X lockouts are y^1 minutes each, the X after that are y^2, and so forth. A successful login resets the counter of failed attempts to 0.
We will the algorithm with x = 10, and y = 10
@maartenba @xavierdecoster @yishaigalatzer @qianjun22 @scottbommarito @ryuyu @shishirx34

Added a bunch of fixes. Please PR again.

LGTM


Drapion image is not rendering

The Drapion image is not getting rendered on the index page.
Root Cause:
Invalid syntax in code

Fixed in PR 558


Rake task to update Pick losses

Game scores will be entered manually. This PR

created a rake task that allowed me to update the Picks table with NflTeam.short_city. This allows me to compare Picks with Games. Before this table update, Picks were identified by NflTeam.id
created a rake task that populates the Game winner column and identifies whether each Pick is a loss or not. Given this, my weekly job will be to enter in each game's scores and then run picks:update_losses task
Small change to hide the 'Make Pick' button on team the user does not own.

Fixes #32

@wonderwoman13

Make sure you grab the full game schedule on to Production before the trial is up

I tried to do this this morning and received ERR_403_DEVELOPER_INACTIVE from SportRadar gem. I think this might have something to do with my API key. Is there somewhere unique I need to place this in production?

Yes, you need to log into Heroku and add the key in the configuration. Look for where your Sendgrid variables are.

On Oct 14, 2016, at 8:46 AM, Melissa Johnson<EMAIL_ADDRESS>wrote:
@wonderwoman13
Make sure you grab the full game schedule on to Production before the trial is up
I tried to do this this morning and received ERR_403_DEVELOPER_INACTIVE from SportRadar gem. I think this might have something to do with my API key. Is there somewhere unique I need to place this in production?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Issue753 defocus

Main changes:

get_crystal_ideal_configuration(defocus=float) implemented for johann, von hamos and pinhole

Issues:

Fixes, in devel, issue #753

Examples:
Playing with tuto_real0.py, we can easily create a focused or a defocused von hamos spectrometer and compare the results.
First a focused von hamos (defocus = 0 in tuto_real0.py)
In [1]: import tofu as tf

In [2]: test = tf.tests.tests09_tutorials.tuto_real0

In [3]: conf, coll = test.main()

In [4]: dax = coll.plot_diagnostic('d02', elements='o', plot_config=conf)

In [5]: dax = coll.plot_diagnostic('d02', elements='o', plot_config=conf, data='lamb')

Now a defocused von hamos (defocus = -1.5 in tuto_real0.py)
In [1]: import tofu as tf

In [2]: test = tf.tests.tests09_tutorials.tuto_real0

In [3]: conf, coll = test.main()

In [4]: dax = coll.plot_diagnostic('d02', elements='o', plot_config=conf)

In [5]: dax = coll.plot_diagnostic('d02', elements='o', plot_config=conf, data='lamb')

It can be seen that defocusing (moving the camera towards the crystal) results in:

the same spectral range
a lower spectral resolution (the spectral range is dispatched on a smaller number of pixels)
a taller image (because the crystal is larger than the camera)

The image distortion (smaller in horizontal direction, larger in vertical direction) globally results in a slightly smaller etendue per pixel
But since the image is always contained on the camera, the total photon flux is conserved, which is visible by integrating the (etendue x dlamb) over the camera.
Since the LOS-approximation is less accurate for computing the etendue, that flux conservation is best checked by using the etendue x dlamb calculated using a vos / reverse ray-tracing approach. It works provided that the reverse ray-tracing is properly converged (i.e.: enough rays are traced).

@cjperks7 , @adriendaros , tagging you for information


Automated cherry pick of #6636: fix: ignore match weights when searching matched policysets

Cherry pick of #6636 on release/2.13.
#6636: fix: ignore match weights when searching matched policysets

Codecov Report

Merging #6657 into release/2.13 will increase coverage by 0.00%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##           release/2.13   #6657   +/-   ##
============================================
  Coverage          6.86%   6.86%
============================================
  Files               536     536
  Lines             90621   90616    -5
============================================
+ Hits               6222    6224    +2
+ Misses            83889   83883    -6
+ Partials            510     509    -1

Flag
Coverage Δ

#aFlag
6.86% <0.00%> (+<0.01%)
:arrow_up:

Impacted Files
Coverage Δ

pkg/cloudcommon/policy/resources.go
0.00% <ø> (ø)

pkg/util/rbacutils/policyset.go
75.00% <0.00%> (+9.78%)
:arrow_up:

pkg/util/rbacutils/rbac.go
55.04% <0.00%> (-0.70%)
:arrow_down:

pkg/cloudcommon/db/lockman/inmemory.go
67.07% <0.00%> (+2.43%)
:arrow_up:

pkg/appsrv/response.go
69.11% <0.00%> (+2.94%)
: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 c53a78a...09611b6. Read the comment docs.


Swarm manager crash

Swarm Version: 0.3.0-rc3
Discovery Service: Consul
Docker Version: 1.6.0
Kernel: 2.6.32-504.el6.x86_64
time="2015-06-18T13:19:28Z" level=info msg="Removed Engine c4t11666.company.com"
time="2015-06-18T13:19:37Z" level=error msg="HTTP error: No such container: 6b3ed3244b8de3c60accae128906155bd23f4ae9edd94e446cf32c0aadb5747a" status=404
time="2015-06-18T13:19:37Z" level=error msg="HTTP error: Container 6b3ed3244b8de3c60accae128906155bd23f4ae9edd94e446cf32c0aadb5747a not found" status=404
time="2015-06-18T13:20:07Z" level=info msg="Registered Engine c4t11666.company.com at c4t11666.company.com:2375"
2015/06/18 13:43:45 http: panic serving <IP_ADDRESS>:19661: runtime error: index out of range
goroutine 14750 [running]:
net/http.func·011()
/usr/src/go/src/pkg/net/http/server.go:1100 +0xb7
runtime.panic(0x963280, 0xcd6a3c)
/usr/src/go/src/pkg/runtime/panic.c:248 +0x18d
github.com/docker/swarm/api.getContainersJSON(0xc208052540, 0x7fa76b4ef800, 0xc20811ee60, 0xc20816e680)
/go/src/github.com/docker/swarm/api/handlers.go:199 +0x16e7
github.com/docker/swarm/api.func·008(0x7fa76b4ef800, 0xc20811ee60, 0xc20816e680)
/go/src/github.com/docker/swarm/api/primary.go:111 +0x2a0
net/http.HandlerFunc.ServeHTTP(0xc2080b72c0, 0x7fa76b4ef800, 0xc20811ee60, 0xc20816e680)
/usr/src/go/src/pkg/net/http/server.go:1235 +0x40
github.com/gorilla/mux.(*Router).ServeHTTP(0xc20801a320, 0x7fa76b4ef800, 0xc20811ee60, 0xc20816e680)
/go/src/github.com/docker/swarm/Godeps/_workspace/src/github.com/gorilla/mux/mux.go:98 +0x292
github.com/docker/swarm/api.(*dispatcher).ServeHTTP(0xc20808a1e0, 0x7fa76b4ef800, 0xc20811ee60, 0xc20816e680)
/go/src/github.com/docker/swarm/api/server.go:32 +0x9d
net/http.serverHandler.ServeHTTP(0xc2081163c0, 0x7fa76b4ef800, 0xc20811ee60, 0xc20816e680)
/usr/src/go/src/pkg/net/http/server.go:1673 +0x19f
net/http.(*conn).serve(0xc2081f0300)
/usr/src/go/src/pkg/net/http/server.go:1174 +0xa7e
created by net/http.(*Server).Serve
/usr/src/go/src/pkg/net/http/server.go:1721 +0x313
2015/06/18 13:44:38 http: panic serving <IP_ADDRESS>:19653: runtime error: index out of range
goroutine 14737 [running]:
net/http.func·011()

+1

Hi @eoghanino and @robtec. I cannot seem to reproduce using swarm v0.3.0 with the latest Consul and kernel 3.13. Would be great if you can provide us with the steps that led to the panic :+1: (or out of curiosity, testing with a more recent kernel and the latest version of Consul)

@abronan kernel version is 2.6.32 rhel 6.3 patched to 6.6

@eoghanino Are you running Swarm inside a container or using the binary? We have known issues running Swarm with rhel, it is not directly related but you might take a look at #781.

@robtec Same issue as well?

@abronan Ok, thanks for the info, its pretty stable just the removal of nodes from the manager when using consul

@eoghanino Ok! #924 will fix the node removal issue with Consul.

@abronan cool thanks!

I'll close the issue ;) feel free to re-open this one or create a new issue if you still have problems running Swarm with Consul


Flutter Web: video_player_web videos are not playing on Iphone Safari browser

I am using video_player: ^2.5.1 and video_player_web 2.0.13 and videos are not playing on iphone 13 mit iOS 16.1 Safari browser, but on all other platforms.
The error is "The request is not allowed by the user agent or the platform in the current context, possibly because the user denied permission"

Hi @razfazz,
Can you please provide your flutter doctor -v and flutter run --verbose logs, and a minimal complete reproducible code sample?
Thank you

i have the same issue. moved to iframe because of it. hopefully this gets fixed soon

I think the issue is close connected to this one https://github.com/flutter/flutter/issues/47030

here is a reproducible example https://github.com/rxlabz/vplayer_safari

Hi @razfazz, the code sample you have linked to is using a very old version of the video_player plugin. You may want to try a new example using the latest version of the video_player plugin.
In any case, this is a duplicate of #47030, as such I'll be closing this in favor of #47030.
Thank you


Ast10x0 clock control

Add clock driver support for Aspeed AST10X0 series SOCs.

So looks good.
Only question is about the fixed frequencies in here.  I think a comment or two would be good.  For example how or why is it that HPLL_FREQ is 1000Mhz?  Why is I2C if not coming from HPLL, 480Mhz, and why UART is fixed 24Mhz/13.

These fixed frequencies are based on the hardware design.
HPLL is the clock source for CPU and BUS usage so it is fixed and unable to changed.
The I3C clock is not used as the SCL directly. Instead, it will be further divided into the other slow clocks in I3C IP according to the configuration of the I3C registers.
The UART clock is not also used as the BAUD. It will be further divided by the UART IP.

Is there anyone who can complete a second review of this?
@nordic-krch


[ npm ERR! ] npm install structor: Refusing to install structor as a dependency of itself.

I tried to follow the install instructions and got the following error while running npm install structor.
npm ERR! Linux 4.12.0-2-amd64
npm ERR! argv "/usr/bin/node" "/usr/bin/npm" "install" "structor"
npm ERR! node v6.11.3
npm ERR! npm  v3.10.10
npm ERR! code ENOSELF

npm ERR! Refusing to install structor as a dependency of itself
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR!     <https://github.com/npm/npm/issues>

npm ERR! Please include the following file with any support request:
npm ERR!     /home/user01/lab/hack/structor/npm-debug.log

try: npm install -g structor


Component/duo support, try catch?

Can you explain the idea behind the try/catch for a module that isn't part of the package in 3eacf6a75eaf247aa34a917906c9714d028d0871? I get that the result is that parent apps may supply their own copy of node-extend by depending on it themselves, but I can't tell why they'd want to do that. I also don't know what "component/duo support" means, and it's hard to work out what's going on just from the diff.
My use case is that I have this module in my dependency tree (via mdast) in a React Native app, and the React Native packager doesn't really love modules that cannot be resolved, so I'm trying to figure out the best way to handle this. It looks like the node-extend module is quite old and the github repo doesn't even exist anymore, so I don't particularly want to add it to my app.
Thanks.

Read more about Component and Duo. Read more about installation methods for mdast in its documentation.
This work-around is needed because the extend npm module is called node-extend on GitHub (the project called node-extend on npm is never used). GitHub is used by non-npm package managers.
I have no experience with “React Native packager”. It doesn’t yet seem to support the browser field, but they seem to be working on it.
I’m closing this because I think supporting the browser field in react-native/packager is the cleanest fix for this.
An alternative is the blacklistRE found in packager

Thanks for the response. Unfortunately I don't think the blacklist option is accessible from the packager CLI. For now I fixed the issue by just adding unified v2.1.0 to the root project's dependencies.


Document architecture and purpose of system

The architecture and purpose of the system are presented in an overview of about 1–1.5 pages; consider using a diagram. Note: If you manage to improve on existing documentation or fill a gap in the project here, please consider making your documentation available to the project; they may be grateful for it!

Done in external document.


React Injection ?

react pe dependencies check ulang masa mo install satu2 dpe dependencies ba nya so depecreted

Okk ka tsmo cek
On Sat, 27 May 2023 at 18.28, Devaliel @.***> wrote:

Assigned #8
https://github.com/RSUD-Dr-Sam-Ratulangi/SistemInformasiPengadaanRsudSamrat/issues/8
to @narivay https://github.com/narivay.
—
Reply to this email directly, view it on GitHub
https://github.com/RSUD-Dr-Sam-Ratulangi/SistemInformasiPengadaanRsudSamrat/issues/8#event-9356792653,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATS4JVFDNGBBVYV26YK4MLTXIHJLXANCNFSM6AAAAAAYRDD5RY
.
You are receiving this because you were assigned.Message ID:
<RSUD-Dr-Sam-Ratulangi/SistemInformasiPengadaanRsudSamrat/issue/8/issue_event/9356792653
@github.com>

npm WARN deprecated<EMAIL_ADDRESS>Modern JS already guarantees Array#sort() is a stable sort, so this library is deprecated. See the compatibility table on MDN: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/sort#browser_compatibility
npm WARN deprecated<EMAIL_ADDRESS>This package has been deprecated and is no longer maintained. Please use @rollup/plugin-terser
npm WARN deprecated<EMAIL_ADDRESS>Please use @jridgewell/sourcemap-codec instead
npm WARN deprecated<EMAIL_ADDRESS>Use your platform's native performance.now() and performance.timeOrigin.
npm WARN deprecated<EMAIL_ADDRESS>This SVGO version is no longer supported. Upgrade to v2.x.x.


Proteome Comparison: links for the features in the circular graph

The links for the features in circular graph do not work on firefox. For example:
https://www.beta.patricbrc.org/view/Feature/fig|1457189.3.peg.432
The link has to be url encoded: https://www.beta.patricbrc.org/view/Feature/fig|1457189.3.peg.432
Please make the correction on the server side for generating the html file.

Fixed in PR:
https://github.com/TheSEED/app_service/pull/58

I am flagging "fixed" per Fangfang's comment. @maovt please verify and close it

It's not deployed yet until Bob @olsonanl is back.

This is merged & deployed now.

From: Fangfang Xia<EMAIL_ADDRESS>Sent: Thursday, December 01, 2016 11:10 AM
To: PATRIC3/patric3_website
Cc: Olson, Robert D.; Mention
Subject: Re: [PATRIC3/patric3_website] Proteome Comparison: links for the features in the circular graph (#1323)
It's not deployed yet until Bob @olsonanlhttps://github.com/olsonanl is back.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/PATRIC3/patric3_website/issues/1323#issuecomment-264232750, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ACDXIsIQizDHC81hHzKL9O-9LsWW_hBEks5rDv-MgaJpZM4K_aau.

Tested and found that a client side fix was still required to handle the url change (for fig id) in Firefox. I fixed the client side code and made a pull request. @hyoo please include this change in the next beta build.

Thanks Bob.

On Dec 2, 2016, at 9:00 PM, olsonanl<EMAIL_ADDRESS>wrote:
This is merged & deployed now.

From: Fangfang Xia<EMAIL_ADDRESS>Sent: Thursday, December 01, 2016 11:10 AM
To: PATRIC3/patric3_website
Cc: Olson, Robert D.; Mention
Subject: Re: [PATRIC3/patric3_website] Proteome Comparison: links for the features in the circular graph (#1323)
It's not deployed yet until Bob @olsonanlhttps://github.com/olsonanl is back.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/PATRIC3/patric3_website/issues/1323#issuecomment-264232750, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ACDXIsIQizDHC81hHzKL9O-9LsWW_hBEks5rDv-MgaJpZM4K_aau.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub https://github.com/PATRIC3/patric3_website/issues/1323#issuecomment-264611622, or mute the thread https://github.com/notifications/unsubscribe-auth/ABAf3fJbgvAmyXzJZMTT7TUf7rmMtiPiks5rENsygaJpZM4K_aau.

Tested on beta and it works now.


fix: combine all batches from bulk api into single stream

I've done some basic testing on my end and am now using this via a fork for my project. Solves #429.

@stomita Would you please review this? CI seems to be failing for unrelated cause. Let me know if there's anything I can do to help - Thanks!

@tejasmanohar Thanks. It seems you use multistream lib  , I'd like to know why you chose this lib ?

@tejasmanohar I checked the stream-concat you mentioned at first, and found that it has a feature of somewhat "lazy-opening" streams. It will save buffer memory instead of parallel fetching. How do you think of this approach ?

@stomita It solves the problem we have- combining streams, and is recommended here by one of the former Node core contributors as an alternative to his module that supports the Node streams2 API. Plus, the implementation is concise & straight-forward, and it appears to be actively maintained. Most importantly, I feel comfortable debugging it in the case of problems. Do you see anything wrong with the library or have another suggestion? I'm open to whatever fits the use case.

Also, I urge that this PR is released as a major change since the caveat of the feature was commented inline in the code (& maybe listed elsewhere? I could not find it in documentation)- plus, it introduces serious functionality changes that were never a part of this library. That said, I do suggest that we add a notice about it to the documentation because the prior functionality was not expected.

Thanks! @stomita


Create view with visual diagram

Allow for download option of metadata or pdf/image

@hazelshapiro We now have the diagram displaying on the main response view. Is this story resolved because of that change?

This is related to the "analyst view" in particular. It would be resolved by updating the permissions so a "respondent" can see the whole response and edit it; whereas, an "analyst" would just see the main page with the visual diagram (and table views, as described in a separate issue). Does that make sense?

It does, thank you!

@hazelshapiro At this time, Analysts can view the diagram, but they also are shown forms to update the diagram when they click on nodes. That should be fixed, but aside from that are there other changes that are needed to consider this complete? Could you rewrite the issue description to indicate those things?


Property:language

Definition
The language or communication system in which the collection desciption is written

Dimension

Existing property
ead:language

Existing class

Existing property identifier
https://www.loc.gov/ead/tglib/elements/language.html

Format

Required

Repeatable

Constraints

Examples

Notes

It's probably more consistent to use dc:language than ead:language, unless there's a particular reason


Documentation update

Updated documentation on using the trust sandbox example to fix an issue where the notary server tag was not found.
Signed-off-by: Matthew Lapworth<EMAIL_ADDRESS>

@dragonndev thanks for the PR... unfortunately, that file is vendored from a dependency of Dockercraft. I'm closing this for now. I you would like to pursue this patch, you'd be better placed opening a PR against https://github.com/docker/docker

Thanks @dave-tucker . A fix in that repo was merged last week. Cheers!


Matching data types of Static_folder & template_folder

template folder is an optional input for the flask object so to maintain similarity in inputs like Static_folder & template_folder, converted the output into string

fixes #

Checklist:

[ ] Add tests that demonstrate the correct behavior of the change. Tests should fail without the change.
[x] Add or update relevant docs, in the docs folder and in code.
[ ] Add an entry in CHANGES.rst summarizing the change and linking to the issue.
[ ] Add .. versionchanged:: entries in any relevant code docs.
[ ] Run pre-commit hooks and fix any issues.
[ ] Run pytest and tox, no tests failed.

No. This is not a astring.


How to generate proxies?

How do I set hubConfig to EnableJavaScriptProxies true?

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

ID: 1387f556-b4b7-70f9-5bd0-f0bccc597689
Version Independent ID: fb016cdc-d281-6b5b-7466-2c718e7274b4
Content: Use hubs in ASP.NET Core SignalR
Content Source: aspnetcore/signalr/hubs.md
Product: aspnet-core
Technology: aspnetcore-signalr
GitHub Login: @bradygaster
Microsoft Alias: bradyg

It was so convenient to use the Hubs with Typescript in AspNet.SignalR with EnableJavaScriptProxies (...or you could have generate the proxies manually with signalr.exe):
// Typescript types:
interface ISignalHub extends SignalR { SignalHub : any; }
$.connection.hub.url = "/signalr";
(<ISignalHub>$.connection).SignalHub.server
   .myServerMethod("some parameter").done(function (data) {
        // it just works!
    });

Compare the AspNetCore version, so much wizzard-walkthrough, and still not showing is it possibe to call a Hub server side method:
https://docs.microsoft.com/en-us/aspnet/core/tutorials/signalr-typescript-webpack?view=aspnetcore-2.1&tabs=visual-studio

There is no equivalent to proxies in ASP.NET Core.
The same code in ASP.NET Core SignalR would look like:
connection.invoke("myServerMethod", "some parameter")
    .then(function (result) {
        // code
    });

The walkthrough shows calling a method on the server in https://docs.microsoft.com/aspnet/core/tutorials/signalr-typescript-webpack?view=aspnetcore-2.1&tabs=visual-studio#enable-client-and-server-communication-1

Closing this per @BrennanConroy's reply.


json alt codes

Looks like special symbols with --json flag converted incorrect. For example © conveted to
"hint":"Parse error: \169"
But according to string description at json.org perhaps should be
"hint":"Parse error: \\169"
At least,
JSON.parse('[{"module":"","decl":"","severity":"Error","hint":"Parse error: \169"...')
generate exception "SyntaxError: Unexpected number in JSON at position 65"

I believe JSON requires only one slash there.
Assuming the JSON.parse fragment is in Javascript, then \169 in a Javascript string is converted by Javascript into the copyright symbol, before it gets passed on to JSON.parse. Does that make sense? If not, can you provide a complete JSON.parse fragment I can paste into my browser and test.

Full Hlint result
[{"module":"","decl":"","severity":"Error","hint":"Parse error: \169","file":"1.hs","startLine":970,"startColumn":1,"endLine":970,"endColumn":1,"from":"  Contact GitHub API Training Shop Blog About\n> \169 2016 GitHub, Inc. Terms Privacy Security Status Help\n","to":null,"note":[],"refactorings":"[]"}]
I am just try parse as is, without any modifications. I got the same exception using online json parser, for example json.parser.online.fr

First thing to note, be careful with json.parser.online.fr - it's a bit broken and doesn't follow the JSON spec. I found the http://json.parser.online.fr/beta/ works better. In particular the former doesn't allow newlines in JSON strings.
Reading the JSON spec, you are indeed right - \169 is not a valid escape code. It should be either the copyright symbol in UTF8, or alternatively \u00a9. I've changed HLint to generate the latter.

Fixed in 1.9.40. Thanks for the bug report!


Suggestion and hint were mixed in cellParams

@salvadord I assigned this PR to you for reviewing as I think it makes more sense for you to review and merge... this is need to make this other PR https://github.com/MetaCell/geppetto-netpyne/pull/15 that I am about to merge works properly


Add tests for missions

The following requests were completed in this pull request:

Add tests for Mission and JoinedMissions components

Great job @dennis-every, time to merge.


Neodigm SodaPop | Create a ModalWarn life cycle event

The use case is that could fire a toast msg explaining that a response is required or show a tooltip pointing to a close button.
Create a wiki recipe showing how it would be used. Update the landing page modal with some interaction.

FYI - I believe this should be an easy task.


TiFlash hang

Bug Report
Please answer these questions before submitting your issue. Thanks!
one of the hang tiflashs process stack is as follows:
bt.txt

1. Minimal reproduce step (Required)

2. What did you expect to see? (Required)
3. What did you see instead (Required)
4. What is your TiFlash version? (Required)
6.2

Thread 910 is blocked by 918
#0  0x0000ffffa420efcc in __lll_lock_wait () from /lib64/libpthread.so.0
#1  0x0000ffffa4209c40 in pthread_mutex_lock () from /lib64/libpthread.so.0
#2  0x0000ffffa7640b38 in std::__1::mutex::lock() () from /tiflash/libc++.so.1
#3  0x000000000663e884 in lock_guard (__m=..., this=<optimized out>) at /usr/local/bin/../include/c++/v1/__mutex_base:90
#4  lock (this=0x62) at /home/jenkins/agent/workspace/build-common/go/src/github.com/pingcap/tics/dbms/src/Storages/Page/V3/BlobStore.h:131
#5  chooseStat (this=0xffff6079a170, buf_size=3768237) at /home/jenkins/agent/workspace/build-common/go/src/github.com/pingcap/tics/dbms/src/Storages/Page/V3/BlobStore.cpp:1416
#6  operator() (this=<optimized out>) at /home/jenkins/agent/workspace/build-common/go/src/github.com/pingcap/tics/dbms/src/Storages/Page/V3/BlobStore.cpp:456
#7  DB::PS::V3::BlobStore::getPosFromStats(unsigned long) (this=0xffff6079a0f0, size=<optimized out>)
    at /home/jenkins/agent/workspace/build-common/go/src/github.com/pingcap/tics/dbms/src/Storages/Page/V3/BlobStore.cpp:444
#8  0x0000000006640fe4 in DB::PS::V3::BlobStore::write(DB::WriteBatch&, std::__1::shared_ptr<DB::WriteLimiter> const&) (this=0xffff6079a0f0, wb=..., write_limiter=...)
    at /home/jenkins/agent/workspace/build-common/go/src/github.com/pingcap/tics/dbms/src/Storages/Page/V3/BlobStore.cpp:294
#9  0x0000000006655860 in DB::PS::V3::PageStorageImpl::writeImpl(DB::WriteBatch&&, std::__1::shared_ptr<DB::WriteLimiter> const&) (this=0xffff60799f18,
    write_batch=<unknown type in /tiflash/tiflash, CU 0x70a489e, DIE 0x70d7fa4>, write_limiter=...)
    at /home/jenkins/agent/workspace/build-common/go/src/github.com/pingcap/tics/dbms/src/Storages/Page/V3/PageStorageImpl.cpp:109
#10 0x0000000006688218 in write (this=0xfff9c09578c8, write_batch=<unknown type in /tiflash/tiflash, CU 0x726b6b3, DIE 0x728c799>, write_limiter=...)
    at /home/jenkins/agent/workspace/build-common/go/src/github.com/pingcap/tics/dbms/src/Storages/Page/PageStorage.h:276
#11 writeIntoV3 (this=<optimized out>, write_batch=<unknown type in /tiflash/tiflash, CU 0x726b6b3, DIE 0x728c748>, write_limiter=...)
    at /home/jenkins/agent/workspace/build-common/go/src/github.com/pingcap/tics/dbms/src/Storages/Page/PageStorage.cpp:484
#12 DB::PageWriter::write(DB::WriteBatch&&, std::__1::shared_ptr<DB::WriteLimiter>) const (this=0xffe8a06c1d58, write_batch=<unknown type in /tiflash/tiflash, CU 0x726b6b3, DIE 0x728c554>,
    write_limiter=...) at /home/jenkins/agent/workspace/build-common/go/src/github.com/pingcap/tics/dbms/src/Storages/Page/PageStorage.cpp:466
#13 0x00000000063b0b94 in DB::DM::WriteBatches::writeLogAndData() (this=0xfffdd20e9f58)
    at /home/jenkins/agent/workspace/build-common/go/src/github.com/pingcap/tics/dbms/src/Storages/DeltaMerge/WriteBatches.h:115
#14 0x00000000064c2bb4 in DB::DM::ColumnFileFlushTask::prepare(DB::DM::WriteBatches&) (this=0xffe4002a4cf8, wbs=...)
    at /home/jenkins/agent/workspace/build-common/go/src/github.com/pingcap/tics/dbms/src/Storages/DeltaMerge/Delta/ColumnFileFlushTask.cpp:58
#15 0x00000000064b4160 in DB::DM::DeltaValueSpace::flush(DB::DM::DMContext&) (this=0xfff7801b1cf8, context=...)
    at /home/jenkins/agent/workspace/build-common/go/src/github.com/pingcap/tics/dbms/src/Storages/DeltaMerge/Delta/DeltaValueSpace.cpp:186
#16 0x00000000063e28a4 in DB::DM::Segment::flushCache(DB::DM::DMContext&) (this=0xfff7801b3c78, dm_context=...)
    at /home/jenkins/agent/workspace/build-common/go/src/github.com/pingcap/tics/dbms/src/Storages/DeltaMerge/Segment.cpp:1357
#17 0x00000000063bb8d8 in DB::DM::DeltaMergeStore::handleBackgroundTask(bool) (this=<optimized out>, heavy=<optimized out>)
    at /home/jenkins/agent/workspace/build-common/go/src/github.com/pingcap/tics/dbms/src/Storages/DeltaMerge/DeltaMergeStore.cpp:1722
#18 0x000000000630e148 in operator() (this=0xffe8a02c0170) at /usr/local/bin/../include/c++/v1/__functional/function.h:498
#19 operator() (this=0xffe8a02c0170) at /usr/local/bin/../include/c++/v1/__functional/function.h:1175

Thread 918 is blocked by 910
#0  0x0000ffffa420efcc in __lll_lock_wait () from /lib64/libpthread.so.0
#1  0x0000ffffa4209c40 in pthread_mutex_lock () from /lib64/libpthread.so.0
#2  0x0000ffffa7640b38 in std::__1::mutex::lock() () from /tiflash/libc++.so.1
#3  0x000000000663f6f4 in lock_guard (__m=..., this=<optimized out>) at /usr/local/bin/../include/c++/v1/__mutex_base:90
#4  lock (this=0xffff6079a170) at /home/jenkins/agent/workspace/build-common/go/src/github.com/pingcap/tics/dbms/src/Storages/Page/V3/BlobStore.cpp:1281
#5  blobIdToPath (this=0xffff6079a170, file_id=107) at /home/jenkins/agent/workspace/build-common/go/src/github.com/pingcap/tics/dbms/src/Storages/Page/V3/BlobStore.cpp:1484
#6  getBlobFileParentPath (this=0xffff6079a0f0, blob_id=107) at /home/jenkins/agent/workspace/build-common/go/src/github.com/pingcap/tics/dbms/src/Storages/Page/V3/BlobStore.cpp:1196
#7  operator() (this=<optimized out>) at /home/jenkins/agent/workspace/build-common/go/src/github.com/pingcap/tics/dbms/src/Storages/Page/V3/BlobStore.cpp:1207
#8  getOrSet<(lambda at /home/jenkins/agent/workspace/build-common/go/src/github.com/pingcap/tics/dbms/src/Storages/Page/V3/BlobStore.cpp:1206:43)> (this=0xffff6079a238, key=@0xfffdd6169c20: 107,
    load_func=<optimized out>) at /home/jenkins/agent/workspace/build-common/go/src/github.com/pingcap/tics/dbms/src/Common/LRUCache.h:127
#9  DB::PS::V3::BlobStore::getBlobFile(unsigned long) (this=0xffff6079a0f0, blob_id=107)
    at /home/jenkins/agent/workspace/build-common/go/src/github.com/pingcap/tics/dbms/src/Storages/Page/V3/BlobStore.cpp:1206
#10 0x00000000066466c0 in DB::PS::V3::BlobStore::getGCStats() (this=0xffff6079a0f0)
    at /home/jenkins/agent/workspace/build-common/go/src/github.com/pingcap/tics/dbms/src/Storages/Page/V3/BlobStore.cpp:1044
#11 0x0000000006657994 in DB::PS::V3::PageStorageImpl::gcImpl(bool, std::__1::shared_ptr<DB::WriteLimiter> const&, std::__1::shared_ptr<DB::ReadLimiter> const&) (this=0xffff60799f18, write_limiter=...,
    read_limiter=...) at /home/jenkins/agent/workspace/build-common/go/src/github.com/pingcap/tics/dbms/src/Storages/Page/V3/PageStorageImpl.cpp:314
#12 0x000000000642cff4 in gc (this=0xffff6079a1e0, not_skip=false, write_limiter=..., read_limiter=...)
    at /home/jenkins/agent/workspace/build-common/go/src/github.com/pingcap/tics/dbms/src/Storages/Page/PageStorage.h:331
#13 DB::DM::GlobalStoragePool::gc(DB::Settings const&, bool, std::__1::chrono::duration<long long, std::__1::ratio<1l, 1l> > const&) (this=0xffff602d32c8, settings=..., immediately=<optimized out>,
    try_gc_period=...) at /home/jenkins/agent/workspace/build-common/go/src/github.com/pingcap/tics/dbms/src/Storages/DeltaMerge/StoragePool.cpp:161
#14 0x000000000630e1b8 in operator() (this=0xffff60880a30) at /usr/local/bin/../include/c++/v1/__functional/function.h:498
#15 operator() (this=0xffff60880a30) at /usr/local/bin/../include/c++/v1/__functional/function.h:1175

In thread 910, an attempt to acquire the following lock in chooseStat.
std::map<String, std::list<BlobStatPtr>> stats_map;

In thread 918, an attempt to acquire the following lock in getBlobFileParentPath.
String BlobStore::getBlobFileParentPath(BlobFileId blob_id)
{
    String parent_path = blob_stats.blobIdToPath(blob_id);
    ...
}


[FR] – Artifact Registry tags should include versions for critical dependencies

in addition to latest/newest and v#.#.#
Artifact registry does not show LABELs from Dockerfile in the image's MANIFEST as GitHub does:

example: https://github.com/gchux/cloud-run-tcpdump/pkgs/container/cloud-run-tcpdump/279660289?tag=v1.1.0-gen1

we should also add a TAG that shows the version of critical dependencies:

tcpdump
libpcap
pcap-cli

Done with https://github.com/gchux/cloud-run-tcpdump/commit/2b22e4cd6b29d0b9e72d9b3507add3fd910458df