Adding colab notebook demo and an interface

The updated code includes:

adding colab notebook demo for new users to run the code without problems.
adding an interface to run either training or testing with specific parameters.
the notebook downloads the code and the dataset from the forked repository and run the training
fix some minor bugs.

Future to do:

improve the code
add testing example
add more explaination to the code and the repository as a tutorial
use tensorboard to draw the training and the testing losses

Dear @cwmok thanks for sharing. I noticed when running the notebook that the training loss is a large negative value e.g.
    Training lvl1...
     step "869" -> training loss "-10781668352.0000" - sim_NCC "-10781668352.000000" - Jdet "136.1196746826" -smo "72.8813"one epoch pass
     step "1396" -> training loss "-118240641024.0000" - sim_NCC "-118240641024.000000" - Jdet "133.8394470215" -smo "72.2927"

Is this normal?

Hi idhamari, This is not normal. I think the problem could be related to your dataset/preprocessing pipeline. Did you normalize the intensity of your data, i.e.: [0, 1]? Please also check out the related issue at #2 .

Hi @cwmok thanks for your quick reply. I am using your code without modification and one of the original challenge public datasets. I think there is already normalization process in the generator class

Thanks for spotting it out. The zero-mean 1-std normalization is not working with the normalized cross-correlation function (the background intensity of the image should be 0). During my experiment, I preprocess the training data within [0,1] and turn the "norm=False". I will update the code very soon. I apologize for the confusion.

@cwmok thanks again for the clarification. I modified the code as you suggested and also added a normalisation code. However, I still get a negative loss value (it is smaller now). I also get zero Jacobian determinant.  e.g.
      Training lvl1...
      step "113" -> training loss "-0.1964" - sim_NCC "-0.198268" - Jdet "0.0000000000" -smo "0.0019"

Maybe there is still there is something missing?

Hi @idhamari, I found that the "normalization code" is not executed as I set Dataset_epoch(names, norm=False) in the training_scripts (Train_LapIRN_disp.py and Train_LapIRN_diff.py) and testing script as well. Therefore, you might either set norm=True in the training script and implement the [0, 1] normalization, or simply normalize the dataset first before you run this code. Remember your setting should be consistent in both the training script and testing script to avoid domain shift issue.

Now the loss function looks good. In our paper, we use negative NCC and therefore the sim-NCC range from -1 to 0.

@cwmok Thanks for your comment.

I found that the "normalization code" is not executed as I set Dataset_epoch(names, norm=False) in the training_scripts (Train_LapIRN_disp.py and Train_LapIRN_diff.py) and testing script as well.
Yes, it is clear now.

Therefore, you might either set norm=True in the training script and implement the [0, 1] normalization, or simply normalize the dataset first before you run this code. Remember your setting should be consistent in both the training script and testing script to avoid domain shift issue.

I am confused, isn't this conflict with:

The zero-mean 1-std normalization is not working with the normalized cross-correlation function (the background intensity of the image should be 0). During my experiment, I preprocess the training data within [0,1] and turn the "norm=False"

more questions :)

Now the loss function looks good. In our paper, we use negative NCC and therefore the value of sim_NCC range from -1 to 0.

so it is normal that Jdet = 0 ?
I also noticed that the training loss is going up down, is this also expected?
  Training lvl1...
  step "869" -> training loss "-0.2139" - sim_NCC "-0.214692" - Jdet "0.0000000000" -smo "0.0008"one epoch pass
  step "1739" -> training loss "-0.2048" - sim_NCC "-0.208969" - Jdet "0.0000000000" -smo "0.0041"one epoch pass
  step "2609" -> training loss "-0.0881" - sim_NCC "-0.197564" - Jdet "0.0094617307" -smo "0.1095"one epoch pass
  step "3000" -> training loss "-0.1529" - sim_NCC "-0.217951" - Jdet "0.0024806887" -smo "0.0650"one epoch pass

I have fixed the normalization method. Here is the sample training loss for your reference:

Training lvl1...
step "20" -> training loss "-0.2638" - sim_NCC "-0.265795" - Jdet "0.0000000000" -smo "0.0020"

I am confused, isn't this conflict with:
Ans: Just remember do not use zero-mean 1-std normalization with NCC function. Please use min-max normalization instead (i.e.: [0, 1].

so it is normal that Jdet = 0 ?
Ans: Yes, it is normal at the lower level. At the higher level, the resulting deformation field will be more complex and the Jdet will not = 0.

I also noticed that the training loss is going up down, is this also expected?
Ans: Yes, it is expected. Similar (less misalignment) input pair seems to yield a better loss value because it is easier to register. On the contrary, images pair with large appearance difference tends to yield higher loss value. To visualize the training loss, visualizing the loss value for each Epoach will help.

@cwmok  thanks for your patience and support.

Just remember do not use zero-mean 1-std normalization with NCC function. Please use min-max normalization instead (i.e.: [0, 1].

This is what I am doing. As I understood, everything now is as expected. FOrgive me if I have many questions, since your paper is a good paper, my goal is to write a tutorial about it providing a colab notebook with some working cases.

To visualize the training loss, visualizing the loss value for each Epoach will help.

As I understood, since the training loss is going up and down, we can not judge the performance until we test the model, right? More questions :)

Did you use augmentation? if yes, how many total images did you use in your training?
Why there is no testing loss provided?

@cwmok thanks for your patience and support. This is what I am doing. As I understood, everything now is as expected. FOrgive me if I have many questions, since your paper is a good paper, my goal is to write a tutorial about it providing a colab notebook with some working cases.

No problem. And thanks a lot for your interest in our work. :)

As I understood, since the training loss is going up and down, we can not judge the performance until we test the model, right? More questions :)

Yes, like other deep learning applications, you need to implement your own validation code using validation data during training. In my paper, we use Dice score of the segmentation map to search for the best model during training (code is not provided as it is too customized).

Did you use augmentation? if yes, how many total images did you use in your training?

In the paper, we didn't use any augmentation, which provides a fair comparison to other state-of-the-art methods. However, in the Learn2Reg challenge, we did use augmentation (i.e. affine augmentation and random horizontal flipping). Empirically, 3-5% gains in Dice score when augmentation is applied in a small dataset.

@cwmok
These are the current training results, I noticed the following:

the similarity measure is not stable.

the training loss is inf after some iterations in level 3 e.g.:
    step "829" -> training loss "1475.7574" - sim_NCC "-0.215631" - Jdet "28.3991413116" -smo "1475.9730"
    step "830" -> training loss "1476.0317" - sim_NCC "-0.288371" - Jdet "28.3438167572" -smo "1476.3201"
    step "831" -> training loss "1476.0468" - sim_NCC "-0.198550" - Jdet "28.3512592316" -smo "1476.2454"
    step "832" -> training loss "1475.9606" - sim_NCC "-0.195672" - Jdet "28.3945274353" -smo "1476.1562"
    step "833" -> training loss "-inf" - sim_NCC "-inf" - Jdet "28.4055004120" -smo "1476.1052"
    step "834" -> training loss "nan" - sim_NCC "0.000000" - Jdet "nan" -smo "nan"
    step "835" -> training loss "nan" - sim_NCC "-0.121906" - Jdet "nan" -smo "nan"
    step "836" -> training loss "nan" - sim_NCC "-0.145650" - Jdet "nan" -smo "nan"

Since you already worked with the same dataset, could you please give some feedback to re-produce your result and explain why I am getting these results?
level 1 training
logLvl1.txt

level 2 training
logLvl2.txt

level 3 training
logLvl3.txt

@cwmok
Thanks a lot for your feedback, I really appreciate it.
This step is not clear:  apply the windowing with lower and upper bound set to -500 and 800"
Shall I map normalize the image to [0,1] then map the values to [-500,800] range? this will conflict with "Making sure that the intensity value of the image's background is zero". I checked the paper in your link, it does not explain this step or provide a reference to explain it.

@idhamari

Shall I map normalize the image to [0,1] then map the values to [-500,800] range? this will conflict with "Making sure that the intensity value of the image's background is zero". I checked the paper in your link, it does not explain this step or provide a reference to explain it.

No, windowing also known as grey-level mapping, contrast stretching, histogram modification or contrast enhancement. It is common in CT preprocessing pipeline as to locate the target anatomical structure for subsequence analysis. For more detail, you may refer to this website.
To achieve this, one way is to apply the numpy clip function to the raw image and set the min and max to your desired HU, i.e.: [-500, 800] in this case, so that all the anatomical structure are clear for observation. At the last step, you have to normalize the preprocessed image to [0, 1] such that the background's intensity for each image is zero.

@cwmok
I am sorry.I read the dialogue above. But why does this error occur when I simply train with oasis dataset

?


Disable drag only on selected items

I need to disable drag only on selected items. Example
Item_0
Item_1
Item_2
i need item_1 to be draggable and the other not.
I want to make Item_1 totally not moveble while the others not.
How i can achieve that ?

@dmanrj there is a no-drag attribute prevents dragging on elements , but i never tried applying this at runtime. just see if this works for u.

Did it using ng-if but this is too verbose imho. I wanted something like no-drag="x.isDraggable".

I tried to use the accept callback for this but it doesn't disable dragging, it disabled dropping when you return false.
Docs say that you can use it to allow "drag" or not.
$scope.dragControlListeners = {     accept: function (sourceItemHandleScope, destSortableScope) {return boolean}//override to determine drag is allowed or not. default is true.     itemMoved: function (event) {//Do what you want},     orderChanged: function(event) {//Do what you want},     containment: '#board'//optional param.     clone: true //optional param for clone feature.     allowDuplicates: false //optional param allows duplicates to be dropped. };


touch events on non buttons (touchable opacity )

https://github.com/dmikey/syr/blob/master/lib/touchable.js
Done for iOS
https://github.com/dmikey/syr/blob/master/ios/SyrNative/SyrNative/SyrTouchableOpacity.m

Done for android and web.


[main] Upgrade to latest dependencies

Cron -knative-prow-robot
/cc knative-sandbox/eventing-writers
/assign knative-sandbox/eventing-writers
Produced by: knative-sandbox/knobots/actions/update-deps

Needs manual fix for linter issues

/lgtm
/approve

/close
see https://github.com/knative-sandbox/eventing-kafka/pull/1269


Feature/switch to spiral errors

Update CI (codecov)
Switch to the spiral/errors everywhere
Small fixes
resolves #382

linked #368


Support for CHAR data type

pseudo does not support the CHAR data type yet. This represents a single character.

A CHAR is a STRING of length 1.
CHARs are represented in pseudocode with single quotes i.e. 'A'


Update to latest egui and glfw

Made an initial attempt in #1 , but wasn't quite able to get all the way. A lot has changed but I think I'm close.
See the following implementations for up-to-date backends:

miniquad: https://github.com/not-fl3/egui-miniquad/blob/master/src/painter.rs
glium: https://github.com/emilk/egui/blob/master/crates/egui_glium/src/painter.rs
glow: https://github.com/emilk/egui/blob/master/crates/egui_glow/src/painter.rs

Resolved in #1


Migrate go dep to go mod as dependencies

Run export GO111MODULE=on.
Run go mod init knative.dev/operator.
Create tools.go under hack.
Run go mod tidy mutiple times, adding one line in replace section in go.mod file each
time till we find them all.
Run go mod vendor.
Modify the file update-deps.sh under hack.
Run ./hack/update-deps.sh.
Remove Gopkg.lock and Gopkg.toml.

Fixes #
Proposed Changes

Release Note

/test pull-knative-sandbox-operator-integration-tests

/test pull-knative-sandbox-operator-build-tests

@houshengbo now that https://github.com/knative/pkg/pull/1179 is in, could you please update pkg and rebase this PR?


Create a Python Flask Cloud Foundry application in IBM Cloud

As a Developer
I need Python Flask Cloud Foundry application
So that I can deploy the changes successfully
Details & Assumptions:

The developer already has IBM Cloud account set up

Acceptance Criteria:
Given IBM Cloud account, the Python Flask Cloud Foundry application should be added

Created nyu-product-service-fall2103 cloud foundry, created access group (added permissions), added team members in access group. Deployed flask app to cloud


isFetching returns wrong value when component is mounted with previously fetched key

Describe the bug
We are using useQuery (with refetchOnMount: false) and the same query key is used for both parent and child components.
The parent has logic where only when isFetching: true  the child is rendered.
First render (with key "a") and first key change (to key "b") are working as expected, however when we change the key back to previous value (to "a"): for a brief moment the parent gets isFetching: true while child get isFetching: false
Your minimal, reproducible example
https://codesandbox.io/s/is-fetching-bug-hqv1v?file=/src/App.js
Steps to reproduce
On first render the console log looks this way:
parent isFetching=true
child isFetching=true
Starting fetch
parent isFetching=false

First time clicking "Change key" button we get:
parent isFetching=true
child isFetching=true
Starting fetch
parent isFetching=false

Second time clicking "Change key" shows unexpected behavior:
parent isFetching=true
child isFetching=false
Starting fetch
parent isFetching=true
child isFetching=true
parent isFetching=false

Expected behavior
When clicking the "Change key" for the second time, we expect the same result as the first time it was clicked:
parent isFetching=true
child isFetching=true
Starting fetch
parent isFetching=false

However for the parent and child get different values for isFetching
How often does this bug happen?
Every time
Screenshots or Videos
No response
Platform

Chrome: Version 95

react-query version
v3.34.12
TypeScript version
No response
Additional context
No response

The codesandbox link shows an empty app. Have you saved it?

@TkDodo Sorry, I've updated the codesandbox link

I can reproduce it, even on later versions and with react18, where batching has improved. So it's not that. Also you can turn refetchOnMount back on and the issue persists.
It's seems to be a weird case that has to do with the conditional mounting + having cached data available already. I don't have time to look into this right now, so any help is appreciated

It seems that it works correctly with refetchOnMount: true. It's possible that there's a race condition somewhere. Although I agree with @TkDodo that this is very unusual case. I haven't found any cause of that yet, but I'll poke around.

So far I was able to find that queryCache object contains false value for isFetching. When refetchOnMount is false this piece of a code is not executed:
      if (fetchOnMount || fetchOptionally) {
        isFetching = true

so isFetching keeps false value. For me it seems like there's some issue with updating queryCache. I need to investigate it more.

I was able to narrow it down to updateResult function. I believe the issue is with this part from src/core/queryObserver.ts:
  updateResult(notifyOptions?: NotifyOptions): void {
    const prevResult = this.currentResult as
      | QueryObserverResult<TData, TError>
      | undefined

    this.currentResult = this.createResult(this.currentQuery, this.options)
    this.currentResultState = this.currentQuery.state

this.currentResult.isFetching holds true value, but this.currentQuery.state.isFetching is still holding false value.
It seems to me that there are two ways of fixing that problem:

Use this.currentResult values instead of this.currentQuery.state to update this.currentResultState.
When setting isFetching manually while doing:

      if (fetchOnMount || fetchOptionally) {
        isFetching = true

add following line:
this.currentQuery.state.isFetching = isFetching

Solution number 2 seems to do the job, although it seems a bit hacky, on the other hand we already patching isFetching flag here, so it make sense to update it inside a query.

hm, the code in question is in createResult on the observer level when optimistically creating the result (which happens for the first fetch, before the first fetch so that we instantly get a loading state even though we haven't started fetching yet). There, we really shouldn't modify the state of the currentQuery because a) there can be multiple observers, some going optimistically into fetching state and some don't and b) this is called during render, so we should no have side-effects there that change the value in the cache.

Got it, make sense. I'm still learning the codebase, but for sure we need to set that loading state somewhere. The issue reported here is because second query is pulling query state from cache (and doesn't call isFetching = true, due to the fact that fetchOnMount is false) where isFetching is still false. So it seems like we should make a fix inside updateResult?

The issue reported here is because second query is pulling query state from cache (and doesn't call isFetching = true, due to the fact that fetchOnMount is false) where isFetching is still false

Ah got it, I understand now. I have to think about it a bit. Maybe we need to store the optimisticResult itself on the cache or so. This is something that we might need to do anyways for react18 where subscriptions go to currentResult, but we actually also return the optimisticResult, so we technically should subscribe to that.
I'm trying to think of situations where two observers can get different optimistic results though, and I'm sure there are some. Like, if the second observer mounts as enabled: false maybe or so ...

Like:

when we read optimistically and fetchOnMount || fetchOptionally, we store the crated optimistic result
if other observers mount, we read that saved value
we clear the optimistic result after the fetch

Not sure if that's a good idea - we would still perform the side-effect of writing something to the cache during render with that solution :/

But actually, it is interesting that we have one observer where we say: yes, this will be going into fetching state, so we return that optimistically, and then we have another observer that does not get that state because the real fetching hasn't started yet and it uses different options. That is a weird case to solve...

If I'm getting this right, it seems like there should be a decision about weather optimistic update of some query should be propagated to cache (so it's state can be accessed via other queries)?
Maybe a solution would be to use optimistic query values (stored somewhere in some way) for all queries expect ones that would explicitly resign from optimistic updates?

If I'm getting this right, it seems like there should be a decision about weather optimistic update of some query should be propagated to cache (so it's state can be accessed via other queries)?

I currently don't see how that's possible. optimistic result calculation happens during rendering. If we write that result into some cache, we will have a side-effect during render. with react18, that will likely bite us with concurrent features. A render can start and be interrupted at any point, which means we might start a render, that writes to the cache, but never completes, which makes everything non-deterministic.
question: would we expect shouldFetchOnMount or shouldFetchOptionally to be true here? And what about is the value of const mounted ?

In case of second query (the problematic one) in the beginning mounted has false value and shouldFetchOnMount, shouldFetchOptionally are false.
After a couple of renders and mounted is true, but shouldFetchOnMount and shouldFetchOptionally return false.
For that case I guess shouldFetchOnMount should be false, but shouldFetchOptionally should be true? If so should that be based on another query that is run in parent component?
I think I might have been confusing things, because my mental model was that if query in parent component is called and same query is run in child it should just return statuses of the query from the parent, but I think I get it now.
I would see two paths now:

We accept it as expected behavior (since query shouldn't been run on mount it has to wait until isFetching is set in query cache by another query.
We add some condition to render logic for such case, but we would have to somehow pull that information from query in parent component - it seems a bit problematic with current approach.

If mounted is true that means we have another observer already - likely the parent component. So maybe it makes sense to check here if any other observer has fetchOnMount or fetchOptionally set to true, and if they do, we optimistically put the child observer into fetching as well...

That make sense. I'll investigate that.

Inside createResult I haven't found any way to access isFetching value from parent component query in any way (where the value would be true). Even listeners don't provide such info.
To fix that issue we have to find some mechanism to enable sharing query state different than using cache. I'm searching and debugging function calls before createResult, although I wasn't able to find anything in there which would allow pass a data to createResult in order to pull isFetching value from the parent component query.

I made some more debugging and I think I made a mistake. mounted is true, but unfortunately not instantly. So issue will still persist. Moreover the only observer was the query itself. I think this bug might be more sophisticated than just updating queryObserver "if" condition. We would have to find a way to communicate that isFetching state that is set during a rendering:
        isFetching = true

I think I've found something. While debugging I noticed that listener (for parent query) is added after useQuery is called for a children in useEffect (file: useBaseQuery.ts)
  React.useEffect(() => {
    mountedRef.current = true

    errorResetBoundary.clearReset()

    const unsubscribe = observer.subscribe(
      notifyManager.batchCalls(() => {
        if (mountedRef.current) {
          forceUpdate(x => x + 1)
        }
      })
    )

    // Update result to make sure we did not miss any query updates
    // between creating the observer and subscribing to it.
    observer.updateResult()

    return () => {
      mountedRef.current = false
      unsubscribe()
    }
  }, [errorResetBoundary, observer])

As a test I added listener when creating it inside useState initialisation
  const [observer] = React.useState(() => {
      const _observer = new Observer<TQueryFnData, TError, TData, TQueryData, TQueryKey>(
        queryClient,
        defaultedOptions
      )
    mountedRef.current = true

    errorResetBoundary.clearReset()
    debugger;
    _observer.subscribe(
      notifyManager.batchCalls(() => {
        if (mountedRef.current) {
          forceUpdate(x => x + 1)
        }
      })
    )
      return _observer;
    }
  )

We would have to somehow handle unsubscribe, but doing this seems to making a test pass. I haven't followed all possible consequences of this, but from investigation it seems the problem is that we have a delay in adding a listener.
After that change query in child component is receiving isFetching == true.
@TkDodo does that sound somehow reasonable?

I'm afraid not. creating a subscription might trigger a fetch, so we have to do this in an effect. Also, with react18, the whole effect becomes one call to useSyncExternalStore, which we have to do to be compatible with concurrent features.

so I don't really see any way forward with this. It looks like its an edge case that we kinda have to accept for now due to how we set the value of isFetching optimistically.
One workaround I found to make this work consistently is to determine isFetching with useIsFetching instead:
const isFetching = useIsFetching(key) > 0;

here is a sandbox reproduction of your issue, yielding consistent results: https://codesandbox.io/s/is-fetching-bug-forked-ocunx0?file=/src/App.js
note that there are some more re-renders, but this gets better on react 18, which has automatic batching. This sandbox, using react 18 and our v4 beta shows the following output, consistently:
parent isFetching=false
Starting fetch
parent isFetching=true
child isFetching=true
parent isFetching=false

https://codesandbox.io/s/is-fetching-bug-forked-unm5cy?file=/src/App.js
As you can see, there is an additional parent isFetching=false output at the beginning, which is to be expected, because useIsFetching does not set the fetching state to true optimistically - which is what would cause the mentioned problem.
I hope this is an acceptable workaround. Closing for now, but feel free to keep the discussion going if you have some new input.


Update minitest to 2.9.2

Updates io.monix:minitest from 2.8.2 to 2.9.2.
GitHub Release Notes - Changelog - Version Diff
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 me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
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 = "io.monix", artifactId = "minitest" } ]

labels: test-library-update, semver-minor

Superseded by #65.


Added ability to export interfaces

This PR includes interfaces as one of the types that can be exported and renamed. I'm assuming this wasn't possible because interfaces used to be unique, but that's changed.

Just a note. The checks are failing due to an unrelated issue in the Apex CLI (that needs investigation)


Cluster ORG for Auto-cluster apps is not tagged to original org

When an app is created in auto-cluster mode, its cluster name is reservable_, and it's assigned to edgecloudorg Organization on the clusters page.
This is expected to be assigned to the original org which created the app.
Without this tagging, it's hard to track reservable cluster VMs and track usage at cluster level.

Reservable ClusterInsts are owned by system, because they can exist outside of any organization. After a user is done with the reservable ClusterInst, it remains for some time period to be used again. During that time, the organization that triggered the create may be deleted. If the reservable ClusterInst remains unused for a configurable time, the system will automatically delete it.
If the reservable ClusterInst is currently being used, then ClusterInst.ReservedBy will be set to the Organization that is currently using it.

Just FYI, in the UI, the right-most icon near the search bar above the table can be used to hide/show additional columns, including the "ReservedBy" field.

In the metrics for usage we use this ReservedBy to attribute the cluster resources to the correct org for the time it's used by that org's applications

thanks for your inputs, this can be closed.


Update Docker Registry docs to v2.6

Problem description
Docker Registry 2.6 was released on 2017-01-18, but the docs are still not available, can you please update them?
Problem location

I couldn't find the information I wanted. I expected to find it near the following URL: https://github.com/docker/docker.github.io/tree/master/registry

Project version(s) affected
Docker Registry 2.6

I don't think Registry 2.6 has shipped in a Docker Engine version yet. @stevvooe can you confirm?

I don't think Registry 2.6 has shipped in a Docker Engine version yet.

The registry ships independently, as far as I know. Looks like the release was tagged in https://github.com/docker/distribution/releases. We've shipped binaries here: https://hub.docker.com/_/registry/.
@dmp42 @andrewhsu What is the process for this going forward?

If the Registry files used by the master branch in the docs need to be pulled from a different branch, someone in that team should either raise a PR against our Dockerfile or raise a GH issue for us to do it. This time, this issue will suffice if someone can confirm that version 26 is correct.


Create Trace_of_matrix.c

Trace of the matrix is called sum of the elements in a principle diagonal of the square matrix.

Try something better !


Update appInsights, common and vstest to place output correctly.

appInsights output is in a different location than where source-build was looking.  common and vstest don't implement outputPlacementRepoApi, so turn that off.
Fixes dotnet/source-build#318

@dotnet-bot test OSX10.12 Release please

OSX issues are unrelated.  Good to merge.


rasa data validation error

Rasa version: 2.2.4
Python version: 3.7.9
Operating system (windows, osx, ...): Windows Server 2019
Issue:

The file generated by "rasa init" contains error
"c:\programdata\anaconda3\envs\rasa\lib\site-packages\rasa\shared\utils\io.py:93: UserWarning: The example 'good afternoon' was found labeled with multiple different intents in the training data. Each annotated message should only appear with one intent. You should fix that conflict The example is labeled with: goodbye, greet."
This was found by using "rasa data validate"

Problem with "rasa data validate"
After deleting example "good afternoon" for intent "goodbye" to resolve the conflict above. It still return an error (which does not make sense as there should not be any error left.

Error (including full traceback):
Traceback (most recent call last):
  File "c:\programdata\anaconda3\envs\rasa\lib\runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "c:\programdata\anaconda3\envs\rasa\lib\runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "C:\ProgramData\Anaconda3\envs\rasa\Scripts\rasa.exe\__main__.py", line 7, in <module>
  File "c:\programdata\anaconda3\envs\rasa\lib\site-packages\rasa\__main__.py", line 116, in main
    cmdline_arguments.func(cmdline_arguments)
  File "c:\programdata\anaconda3\envs\rasa\lib\site-packages\rasa\cli\data.py", line 262, in validate_files
    and _validate_story_structure(validator, args)
  File "c:\programdata\anaconda3\envs\rasa\lib\site-packages\rasa\cli\data.py", line 297, in _validate_story_structure
    not args.fail_on_warnings, max_history=args.max_history
  File "c:\programdata\anaconda3\envs\rasa\lib\site-packages\rasa\validator.py", line 249, in verify_story_structure
    trackers, self.domain, max_history, self.nlu_config
  File "c:\programdata\anaconda3\envs\rasa\lib\site-packages\rasa\core\training\story_conflict.py", line 185, in find_story_conflicts
    trackers, domain, max_history, tokenizer
  File "c:\programdata\anaconda3\envs\rasa\lib\site-packages\rasa\core\training\story_conflict.py", line 245, in _find_conflicting_states
    for element in _sliced_states_iterator(trackers, domain, max_history, tokenizer):
  File "c:\programdata\anaconda3\envs\rasa\lib\site-packages\rasa\core\training\story_conflict.py", line 333, in _sliced_states_iterator
    _apply_tokenizer_to_states(tokenizer, sliced_states)
  File "c:\programdata\anaconda3\envs\rasa\lib\site-packages\rasa\core\training\story_conflict.py", line 351, in _apply_tokenizer_to_states
    Message({TEXT: state[USER][TEXT]}), TEXT
KeyError: 'text'

Command or request that led to error:
rasa init
rasa data validate

Content of training data file (nlu.yml) :
version: "2.0"

nlu:
- intent: greet
  examples: |
    - hey
    - hello
    - hi
    - hello there
    - good morning
    - good evening
    - moin
    - hey there
    - let's go
    - hey dude
    - goodmorning
    - goodevening
    - good afternoon

- intent: goodbye
  examples: |
    - cu
    - good by
    - cee you later
    - good night
    - bye
    - goodbye
    - have a nice day
    - see you around
    - bye bye
    - see you later

- intent: affirm
  examples: |
    - yes
    - y
    - indeed
    - of course
    - that sounds good
    - correct

- intent: deny
  examples: |
    - no
    - n
    - never
    - I don't think so
    - don't like that
    - no way
    - not really

- intent: mood_great
  examples: |
    - perfect
    - great
    - amazing
    - feeling like a king
    - wonderful
    - I am feeling very good
    - I am great
    - I am amazing
    - I am going to save the world
    - super stoked
    - extremely good
    - so so perfect
    - so good
    - so perfect

- intent: mood_unhappy
  examples: |
    - my day was horrible
    - I am sad
    - I don't feel very well
    - I am disappointed
    - super sad
    - I'm so sad
    - sad
    - very sad
    - unhappy
    - not good
    - not very good
    - extremly sad
    - so saad
    - so sad

- intent: bot_challenge
  examples: |
    - are you a bot?
    - are you a human?
    - am I talking to a bot?
    - am I talking to a human?

Thanks for the issue, @yennycheung will get back to you about it soon!
You may find help in the docs and the forum, too 🤗

Thanks for the issue, @yennycheung will get back to you about it soon!
You may find help in the docs and the forum, too 🤗

duplicate of #7603

duplicate of #7603


Entity Id type

Hello,
When I created a new:domain the resulting project contained Entities that contained an ID field of type GUID that was autogenerated by craftsman. I am wondering if there is a way to define the data type for the ID? Right now if I add an Id as a Property to my Entity then two Id fields are created resulting in build errors. The use case I have is that the Id field needs to be an INT because the legacy database we are using uses autoincremented Ids.

Thank you!


[core] Fix bundle size upload to S3 job

Tested with the PR conditional disable and managed to upload to S3 in https://app.circleci.com/pipelines/github/mui/material-ui/108396/workflows/eb202997-db81-409b-81ba-8c6b81160b1f/jobs/580529/parallel-runs/0/steps/0-113 and in https://app.circleci.com/pipelines/github/mui/material-ui/108396/workflows/eb202997-db81-409b-81ba-8c6b81160b1f/jobs/580529/parallel-runs/0/steps/0-120

It seems to work, great, thanks 👌


Bluetooth: Controller: Deprecate tinycrypt part2 (testing)

Deprecate tinycrypt part2

Closing, not needed now, will reopen if needed to test integrating controller fixes if needed with the tinycrypt deprecation PR.


Request: offer as a library for Android, including de-obfuscation

Currently this repository became very old, and we can use most of it on Android Studio anyway.
Please update it, and offer to to get de-obfuscation of APK files, including ability to run it even on Android itself.

Thanks for the request

If Android Studio addresses your needs, we are more than happy
Deobfuscation - you can use Android Studio (more details)
Ability to run on Android - this is out of our scope as a desktop tool

Thanks for the request

If Android Studio addresses your needs, we are more than happy
Deobfuscation - you can use Android Studio (more details)
Ability to run on Android - this is out of our scope as a desktop tool

@borisf Is there anything this tool offers that isn't available on Android Studio?
Even online, I can get more, no?
Why can't it offer de-obfuscation?
If the tool is so outdated, please consider making it better, and supporting Android too, or at least make it a bit easier.

@borisf Is there anything this tool offers that isn't available on Android Studio?
Even online, I can get more, no?
Why can't it offer de-obfuscation?
If the tool is so outdated, please consider making it better, and supporting Android too, or at least make it a bit easier.

At the moment, we don't have any plans to update ClassyShark, but we are open to pull requests :)

At the moment, we don't have any plans to update ClassyShark, but we are open to pull requests :)

@borisf Does Google have any good alternative library to parse APK files (showing classes, resources, code...), which can run on Android?

@borisf Does Google have any good alternative library to parse APK files (showing classes, resources, code...), which can run on Android?

I am not familiar

I am not familiar

@borisf So Google made Android itself, including the APK standard, and both are open sourced, but there is no nice official library to parse them ...
:(

@borisf So Google made Android itself, including the APK standard, and both are open sourced, but there is no nice official library to parse them ...
:(


Eliminate annoying frontend race conditions involving i18next

Currently it just: i18n: Pass translation data as context to index.html.

There are more steps to accomplish as mentioned in #9087.

Cool, I think based on https://www.i18next.com/how-to/add-or-load-translations#add-on-init, we should just be able to pass the translation_data object into the i18next initialization...

@timabbott I don't think we need any of these plugins after the changes:
-import XHR from 'i18next-xhr-backend';
-import LngDetector from 'i18next-browser-languagedetector';
-import Cache from 'i18next-localstorage-cache';
-import localstorage from './localstorage';

, so I removed all the logic for these plugins and what's is left is just simple logic for loading translation data.
But it's not working at all, maybe I'm missing something.

Also, just realized that I've done both the steps

We hack up i18next, somehow, to read its translation data from the translation_data structure.

Remove all of our logic around caching i18next data in browser local storage.

mentioned in issue description in single step, I'll split the commits properly once this gets working.

Yeayy, made this working, finalizing PR soon,
GIFs:

It looks like CI is failing because it can't find the English translations.json file; we certainly don't need one (they'll all be foo: foo values), so we should just handle the en language specially in the backend code (and give the value as an empty dict or whatever).

@timabbott Updated for final review. Also, I've manually tested the changes by changing all the languages and it works fine for all(no error log).

I can reproduce the casper failure locally with test-js-with-casper 01.  No screenshot, which is annoying.  and it looks like webpack is giving a source-mapped / useless traceback.  you can print-debug by doing console.log inside translations.js.
E.g. I just got this:
2018-05-29 20:32:47.008 INFO [zr] <IP_ADDRESS>       POST    302  14ms (db: 1ms/5q) (+start: 7ms) /accounts/logout/ (unauth via ?)
2018-05-29 20:32:47.008 INFO [django.server] "POST /accounts/logout/ HTTP/1.1" 302 0
2018-05-29 20:32:47.014 INFO [zr] <IP_ADDRESS>       GET     301   0ms /accounts/login (unauth via ?)
2018-05-29 20:32:47.014 INFO [django.server] "GET /accounts/login HTTP/1.1" 301 0
2018-05-29 20:32:47.047 INFO [zr] <IP_ADDRESS>       GET     200  22ms (db: 6ms/6q) (+start: 7ms) /accounts/login/ (unauth via ?)
app console: START
app console: undefined
TypeError: undefined is not an object (evaluating 'i.indexOf')
Traceback:
  http://zulip.zulipdev.com:9981/static/webpack-bundles/translations-d7dd78dfb71dc3eb702a.js:1
  http://zulip.zulipdev.com:9981/static/webpack-bundles/translations-d7dd78dfb71dc3eb702a.js:1
  http://zulip.zulipdev.com:9981/static/webpack-bundles/translations-d7dd78dfb71dc3eb702a.js:1

Oops, the frontend tests failed. Tips for debugging:
 * Check the screenshots of failed tests at var/casper/casper-failure*.png
 * Try remote debugging the test web browser as described in docs/testing/testing-with-casper.md

from
(zulip-py3-venv) tabbott@zaset:~/zulip$ git diff
diff --git a/static/js/translations.js b/static/js/translations.js
index fe8a074..8dd529b 100644
--- a/static/js/translations.js
+++ b/static/js/translations.js
@@ -1,11 +1,14 @@
 // commonjs code goes here
 import i18next from 'i18next';

+console.log("START");
 window.i18n = i18next;
 var language = page_params.default_language;
+console.log(language);
 if (language.indexOf('-') >= 0) {
     language = language.replace('-', '_');  // Change zh-Hans to zh_Hans.
 }
+console.log("HERE");

 i18next
     .init({

which is a lead for what's up.

@timabbott Tests are passing now.
There were some other bugs which are also fixed now, I think, deploying at CZO first can be safe.

I did a further thinking about the language string conversion logic in js file and found that it is redundant now as we did all the work in the backend so just used lang as a placeholder for default language.

Another GIF for specifying the language in URL:

Cool; I'm test-deploying this on chat.zulip.org.  @umairwaheed can you give this a review?

After a bit more testing and adding eea26b6cce99370da805c1795f031f2bcb1abacd to clean up the logic in zerver/views/home.py to be clear and make sense, I've merged this, thanks @shubhamdhama!
I'm going to coordinate with @showell the work of disentangling/removing ensure_i18n.

LGTM.


Roll engine bf15bd080314..669ac3848d0f (6 commits)

https://github.com/flutter/engine/compare/bf15bd080314...669ac3848d0f
git log bf15bd0803142ffc3500f8e5ad4d37aca34246ff..669ac3848d0f52d0d4db28af1ed5e73d8c3a86ef --no-merges --oneline
669ac3848 roll dart and update libraries files (flutter/engine#9201)
a6451caff Roll src/third_party/skia 89fc1db4e809..7e2c3cdbb238 (2 commits) (flutter/engine#9195)
178e7e320 Revert "Roll src/third_party/dart b080e7c97a..9dcb026b26 (30 commits)" (flutter/engine#9200)
369ba9ee5 Roll src/third_party/dart b080e7c97a..9dcb026b26 (30 commits)
44b1cedff Allow the task queues to be swapped in MessageLoops (flutter/engine#9189)
89c0c1b8f Roll src/third_party/skia 346f82c1c3e0..89fc1db4e809 (2 commits) (flutter/engine#9191)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (bmparr@google.com), and stop
the roller if necessary.

Auto-roller completed checks. About to merge.


doc: add doc for FileMap

I'm not sure if Pr's only adding documentation, like this one, are generally well received. If you don't have the time/interest to review such Pr's please just close it!

Thanks for the PR!


No license specified

I don't see a LICENSE file.  Can you add one with the proper license, please?

@jj51726-jd  Thanks. I have a PR to address this.


[Feature Request] - Using jimmer to manage transactions in Ktor

Reason
I found that I used the jimmer framework without using Spring, and it was very difficult to handle transactions.

Description
I hope to provide a usage method similar to sqlClient.useTransaction {}
Existing solutions
No response

Look this
https://github.com/babyfish-ct/jimmer/issues/771#issuecomment-2492717778

It is design for JDBC, and hope java virtual thread can be optimized as soon as possible.
For R2DBC, it is huge project(save command is super complex), so it will not be implement unless java virtual thread still disappoint everyone after 1~2 years.
If you programming mode is JDBC, please view https://github.com/babyfish-ct/jimmer/blob/main/project/jimmer-spring-boot-starter/src/main/java/org/babyfish/jimmer/spring/cfg/support/SpringConnectionManager.java.
Don't simply open and close connections. You need to understand what data structures your technology stack uses to manage connections, and use them to borrow and return connections.

Close it because it duplicated with #771


negative temperature - THGR810

Hello,
I've done some test with thgr810 (humidity and temperature sensor using oregon v3 protocol) that i've put in my freezer (in order to reach the low battery level).
With this sensor the nyb(12) is changed at 8 and not 1 when temperature goes below 0.
may be it depends on the kind of sensor.
Best regards,
Micg77

How did you even get this working? I get nothing with this Sensor (only one I've tested).


Output is written twice and incorrectly to StdErr

Terragrunt output is being rendered twice, once normally and then to StdErr
Example:
terragrunt version v0.45.10
############################################################
RUNNING 'terragrunt run-all plan --terragrunt-non-interactive'
############################################################
**time=2023-05-10T11:17:37Z level=info msg=The stack at /azp/_work/1/s/infrastructure/Azure/terraform/environments/QA/mono will be processed in the following order for command** plan:
Group 1
- Module /azp/_work/1/s/infrastructure/Azure/terraform/environments/QA/mono

Initializing the backend...

Successfully configured the backend "azurerm"! Terraform will automatically
use this backend unless the backend configuration changes.
Initializing modules...

[REMOVED]

Initializing provider plugins...
- Finding hashicorp/azuread versions matching "~> 2.36.0"...
- Finding hashicorp/random versions matching "~> 3.1"...
- Finding latest version of hashicorp/time...
- Finding latest version of hashicorp/helm...
- Finding latest version of hashicorp/kubernetes...
- Finding hashicorp/azurerm versions matching "~> 3.48.0"...
- Installing hashicorp/time v0.9.1...
- Installed hashicorp/time v0.9.1 (signed by HashiCorp)
- Installing hashicorp/helm v2.9.0...
- Installed hashicorp/helm v2.9.0 (signed by HashiCorp)
- Installing hashicorp/kubernetes v2.20.0...
- Installed hashicorp/kubernetes v2.20.0 (signed by HashiCorp)
- Installing hashicorp/azurerm v3.48.0...
- Installed hashicorp/azurerm v3.48.0 (signed by HashiCorp)
- Installing hashicorp/azuread v2.36.0...
- Installed hashicorp/azuread v2.36.0 (signed by HashiCorp)
- Installing hashicorp/random v3.5.1...
- Installed hashicorp/random v3.5.1 (signed by HashiCorp)

Terraform has created a lock file .terraform.lock.hcl to record the provider
selections it made above. Include this file in your version control repository
so that Terraform can guarantee to make the same selections by default when
you run "terraform init" in the future.

Terraform has been successfully initialized!

You may now begin working with Terraform. Try running "terraform plan" to see
any changes that are required for your infrastructure. All Terraform commands
should now work.

If you ever set or change modules or backend configuration for Terraform,
rerun this command to reinitialize your working directory. If you forget, other
commands will detect it and remind you to do so if necessary.

[REMOVED]

No changes. Your infrastructure matches the configuration.

Terraform has compared your real infrastructure against your configuration
and found no differences, so no changes are needed.
##[error]time=2023-05-10T11:17:37Z level=info msg=The stack at /azp/_work/1/s/infrastructure/Azure/terraform/environments/QA/mono will be processed in the following order for command plan:
Group 1
- Module /azp/_work/1/s/infrastructure/Azure/terraform/environments/QA/mono

##[error]Script has output to stderr. Failing as failOnStdErr is set to true.

running
terragrunt run-all plan

should reproduce the issue. It appears to be linked to run-all However, this will also result in incorrect output written to StdErr
terragrunt plan --terragrunt-log-level debug

Expected behavior
This should not write to StdErr when there are no errors. Running in Azure DevOps results in the build failing.
Versions

Terragrunt version: v0.45.10
Terraform version: 1.4.6
Environment details Ubuntu 20.04 (agent), Windows 11

Potentially related....
Adding a hook to run tfsec.
terraform {
  before_hook "before_hook" {
    commands     = ["apply", "plan"]
    execute      = ["tfsec", "--soft-fail" ]
  }
}

If there are issues reported, even though tfsec is configured to fail silently terragrunt fails with StdErr. Given tfsec doesn't fail I would expect terragrunt to succeed. If --soft-fail is remove terragrunt doesn't not proceed past the hook which you'd expect.

@thisispaulsmith, in addition to the log, any shell command returns an exit code, and If the exit code is anything other than 0 this indicates failure. before_hook supports run_on_error argument, see docs.
terraform {
  before_hook "before_hook" {
    ... omitted
    run_on_error = true
  }
}

@levkoburburas  This is happening regardless of exit code. Example running tfsec with with no error.
terragrunt version v0.45.10
############################################################
RUNNING 'terragrunt plan --terragrunt-non-interactive'
############################################################

Initializing the backend...

Successfully configured the backend "azurerm"! Terraform will automatically
use this backend unless the backend configuration changes.
Initializing modules...
Downloading registry.terraform.io/Azure/naming/azurerm 0.2.0 for RESOURCE_GROUP_NAMING...
- RESOURCE_GROUP_NAMING in .terraform/modules/RESOURCE_GROUP_NAMING
Downloading registry.terraform.io/Azure/naming/azurerm 0.2.0 for RESOURCE_NAMING...
- RESOURCE_NAMING in .terraform/modules/RESOURCE_NAMING

Initializing provider plugins...
- Finding hashicorp/azurerm versions matching "~> 3.48.0"...
- Finding hashicorp/random versions matching ">= 3.3.2"...
- Installing hashicorp/azurerm v3.48.0...
- Installed hashicorp/azurerm v3.48.0 (signed by HashiCorp)
- Installing hashicorp/random v3.5.1...
- Installed hashicorp/random v3.5.1 (signed by HashiCorp)

Terraform has created a lock file .terraform.lock.hcl to record the provider
selections it made above. Include this file in your version control repository
so that Terraform can guarantee to make the same selections by default when
you run "terraform init" in the future.

Terraform has been successfully initialized!

You may now begin working with Terraform. Try running "terraform plan" to see
any changes that are required for your infrastructure. All Terraform commands
should now work.

If you ever set or change modules or backend configuration for Terraform,
rerun this command to reinitialize your working directory. If you forget, other
commands will detect it and remind you to do so if necessary.
time=2023-05-19T11:28:39Z level=info msg=Executing hook: before_hook prefix=[/azp/_work/1/s/infrastructure/Azure/terraform/environments/QA/UK/Shared/keyvault]
  timings
  ──────────────────────────────────────────
  disk i/o             3.402109ms
  parsing              1.259873193s
  adaptation           227.7µs
  checks               40.49409ms
  total                1.303997092s

  counts
  ──────────────────────────────────────────
  modules downloaded   0
  modules processed    3
  blocks processed     491
  files read           12

  results
  ──────────────────────────────────────────
  passed               2
  ignored              0
  critical             0
  high                 0
  medium               0
  low                  0

No problems detected!

data.azurerm_client_config.CURRENT: Reading...
data.azurerm_client_config.CURRENT: Read complete after 0s [id=Y2xpZW50Q29uZmlncy9jbGllbnRJZD05MDA2MjNlMC1iNjdkLTRlYjAtYjdlMi0zNGRlOGU4YWUyYzM7b2JqZWN0SWQ9ZTQyZGE2YWYtY2JkYy00YWFhLWFiZTItMDYzMTdlYzE5OTMzO3N1YnNjcmlwdGlvbklkPWNjYTMyNzE4LTVhZDAtNGYxYS04OTg4LWM5ZjQ2M2UyZDQyNjt0ZW5hbnRJZD0zZWRjNDQ4Ni01ODc4LTQ2YTktYTk3NC0yYTc3ODc5MjZmMzU=]

Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
  + create

Terraform will perform the following actions:

[REMOVED]

Plan: 11 to add, 0 to change, 0 to destroy.

Changes to Outputs:

[REMOVED[

─────────────────────────────────────────────────────────────────────────────

Note: You didn't use the -out option to save this plan, so Terraform can't
guarantee to take exactly these actions if you run "terraform apply" now.
##[error]time=2023-05-19T11:28:39Z level=info msg=Executing hook: before_hook prefix=[/azp/_work/1/s/infrastructure/Azure/terraform/environments/QA/UK/Shared/keyvault]
  timings
  ──────────────────────────────────────────
  disk i/o             3.402109ms
  parsing              1.259873193s
  adaptation           227.7µs
  checks               40.49409ms
  total                1.303997092s

  counts
##[error]Script has output to stderr. Failing as failOnStdErr is set to true.

@levkoburburas I should add this this only happens when running in a linux agent. Running in windows locally is fine. No StdErr.

@levkohimins Do you mean tfsec? terraform works fine. It's only when adding the hook to tfsec that this fails above

@thisispaulsmith Oh now I understand what you meant, thanks for the explanation. This is a bug, we'll fix it.

Issue resolved in v0.46.0 release

I just bumped into this issue running on a Ubuntu 22 Azure DevOps agent. The command I'm using is
terragrunt run-all plan --terragrunt-no-auto-init
Any recommendations?

@celsocoutinho-tangany
Either don't use run-all (it works fine in that case) or wrap the return code with an error and don't fail on StdErr

I am running a few terraform projects which are related, and I decided to use terragrunt to manage the dependencies on top of these projects. If I use terragrunt plan instead of terragrunt run-all plan, it is my understanding that I will have to run the command for each of the terraform projects, which will beat the purpose of using terragrunt in the first place, I guess!
If I wrap the return code with an error, and prevent it to fail to stderr, I guess I will never be able to distinguish between a successful terragrunt execution (which is actually the case right now), and a failed one.
If my assumptions are right, I hope there is a plan C 😄

You can check the return code on the terragrunt command.
Using run-all isn't ideal as you'll get a mass of output that's hard to distinguish. You could create a template pipeline and loop over your individual components.

@celsocoutinho-tangany, there is a pretty simple solution, it's to lower the logging level --terragrunt-log-level error. In this case, only messages with the error level will be output to stderr.

Hi @levkohimins , thanks for your reply! I had known that solution before, but it unfortunately reduces the amount of information that is output from terragrunt, which we rely during our CI/CD processes.
In the meantime, the solution I have presented has been working fine, so I believe it is a more comprehensive solution for our use case


Add split_along_batch/seq

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (556ba42) 100.00% compared to head (4a09c51) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #14   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            5         5
  Lines           42        46    +4
=========================================
+ Hits            42        46    +4

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


/v1/provide-mac-table-of-specific-device pagination to be removed

The number of entries in the MAC tables is in the lower 10th.
Pagination is not required and shall be removed again.

Done during preparing v1.0.1


Docker hub image deleted?

Platform

Docker

Describe the bug
The offical docker image seems to no longer exists...
To Reproduce
Steps to reproduce the behavior:
docker pull wappalyzer/cli
Error response from daemon: pull access denied for wappalyzer/cli, repository does not exist or may require 'docker login': denied: requested access to the resource is denied

Expected behavior
It should work no ?

Any information about that please ?

+1

@AliasIO
hope you are all well !
the same for me, what happened to them ?
can you advise please ?

Sorry about that, the image has been restored. It hasn't been maintained for some time and is now deprecated. I recommend using the NPM package or building your own image if you want to use the latest.


Failed Translation from Chinese to Spanish with Missing Content

Issue: Failed Translation from Chinese to Spanish with Missing Content
When attempting to translate markdown documents from Chinese to Spanish, the translation fails entirely. In some cases, not even the disclaimer is shown. This seems to be happening due to the system using only the font of the target language, without processing the content correctly.
Example of original content in Chinese:
# 变更日志

所有 "phi3ext" 扩展的显著更改都将记录在此文件中。

请查看 [保持变更日志](http://keepachangelog.com/) 以获取有关如何构建此文件的建议。

## [未发布]

- 初始版本

Expected translation (into Spanish) should have more than just the disclaimer at the end, but sometimes the output contains only the disclaimer text, such as:

```markdown
Aviso legal: La traducción fue realizada a partir de su original por un modelo de IA y puede no ser perfecta.
Por favor, revise el resultado y haga las correcciones necesarias.

Solution:

Add a feature: Scan the files after translation to detect if the file is either empty or contains only the disclaimer text. If this happens, flag the file as "translation failed" and mark it in the progress bar for retrying later.
Handle translation errors: If Azure OpenAI (AOAI) provides an invalid response or the translation fails, ensure that the translation process does not save an empty or disclaimer-only file, and instead, report the failure in the logs or UI for further action.

Closing issue as we have migrated to a Microsoft Repo


Documentation links broken

Many links in the docs (README and blog posts) forward to https://daqri.com. :(
F.e.: https://artoolkit.org/documentation/doku.php?id=3_Marker_Training:marker_training

Yes, there is a PR merged on dev for this.
As soon as I can I will move those commits also on master.

done, new readme available on master :)

Many are still broken
https://jeromeetienne.github.io/AR.js-docs/misc/EXAMPLES.html

ar.js-docs is deprecated, please refer to readme of ar.js


[Feature Request] Mouse scrolling the pannel triggers workspace switch

Description of the new feature / enhancement
Scrolling the mouse wheel when hovering the pointer over the pannel switches workspace accordingly to the wheel spin direction

Spin down goes forward a workspace
Spin up goes backward a workspace

Scenario when this would be used?
This is useful for keyboardless navigation in the system
Current workaround:

Click Workspace icon in the taskbar
Select desired workspace

Supporting information
I can implement this given some guidance as I'm not familiar with Windows internals

Second workaround: Win+Ctrl+left or Win+Ctrl+right

Great feedback but for this type of feature, best spot is Windows Feedback Hub.
If you're already using the mouse heavily, just wondering why not just use the pointer however.

Not sure, but maybe there is something AutoHotkey could do? I'm thinking a trigger on the scroll, then a check on coordinates or something..?


Add udev rules for non-root usage

With these rules installed you can run the tool without sudo.

Awesome, thanks :exclamation:


Doc: Add documentation for design system

#89  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @ksharma-xyz and the rest of your teammates on  Graphite


pkg(samarium-0.3.1): Added samarium 0.3.1

Closes #522

Adds Samarium as a supported language

@HexF Sorry for ping. But I believe I fixed the issue.
The package from pip was not being installed which led to the pkg.tar.gz not being built. Since there is no package to test, the test fails.

/usr/bin/python3: No module named pip3
So close, just need to update the build script to use pip and not pip3 when calling with python3 -m


Violations are not reported for org IAM rules

Forseti version: 2.25.1
Having trouble with IAM scanner
Forseti deployed via terraform

Not sure if this is a bug or my configuration is incorrect. I am trying to set up a scanner rule so that only service accounts or users/groups of a particular domain can be added to anything within an organization. My current rule looks like this:
rules:
  - name: Allow only my domain accounts and service accounts access
    mode: whitelist
    resource:
      - type: organization
        applies_to: self_and_children
        resource_ids:
          - '*'
    inherit_from_parents: true
    bindings:
      - role: *
        members:
          -<EMAIL_ADDRESS>          -<EMAIL_ADDRESS>          -<EMAIL_ADDRESS>
However no violations are appearing within the bucket; whilst I expect there would be since there are users/groups added to resources that are not part of my domain

Closing this; it was a misconfiguration on my end


Fixes for srk_example.ipynb

This is a small PR fixing two issues in devdocs/srk_example.ipynb. The second commit just fixes the name of SPaRK.
The first commit improves the issue that @lockwo mentioned in #420. The problem was that the convergence order "eventually" approaches 1.5, but at the given step size it didn't quite get into that regime yet. So, by moving a bit to the right on the x-axis (i.e. increasing the step sizes) the order increased from 1.2 to 1.33. I imagine if we increased the step sizes even further, we'd get closer to 1.5 (assuming no other numerical errors).
I also increased the number of samples from 100 to 1000, so the lines have less variance.
The reason this issue arose in the first place is that the SDE being simulated there is 7 dimensional and very complicated. The drift vector field is a 5-layer MLP (taking as input both time and position) and the diffusion VF is some weird function of time. This was needed because with simpler SDEs, the order of the solvers was actually significantly higher than 1.5, so the tests kept failing.
As to what exactly makes the SDE "complicated enough" to not converge too fast and how small a step size you need to get to the correct order, I unfortunately don't have any good answers.

LGTM! Thank you for the quick fix :)


Please add absolute_import at the beginning of every file

I have an import warning:
site-packages/requests/utils.py:17: ImportWarning: Not importing directory '/home/x/platform'
 missing __init__.py

  import platform

so my directory: platform hided system module platform.py
This can only be fixed by adding
from __future__ import absolute_import at the first line of utils.py
Also it would be nice to have this for every module.
This should do no harm to python 3.

This is not a totally unreasonable thing to do, though it's totally tragic that we need it: if we didn't support Python 2.6 this wouldn't be a problem. @sigmavirus24?

So I have done this before to debug other problems. The major issue is that it doesn't protect urllib3 so if someone shadows another standard library module locally, they will still have this problem. I'm +0 on the change (and to be clear this is only not an issue on Python 3+) but I don't think it will help people too much who already aren't following best practices for module naming, i.e., not shadowing standard library modules.

I'm working in a company that we have python and c++ code site together,
and the c++ guys definitely don't care or know about standard python
package name.
Shadowing another standard library is another problem.
On Sat, Sep 12, 2015 at 10:07 AM, Ian Cordasco<EMAIL_ADDRESS>wrote:

So I have done this before to debug other problems. The major issue is
that it doesn't protect urllib3 so if someone shadows another standard
library module locally, they will still have this problem. I'm +0 on the
change (and to be clear this is only not an issue on Python 3+) but I don't
think it will help people too much who already aren't following best
practices for module naming, i.e., not shadowing standard library modules.
—
Reply to this email directly or view it on GitHub
https://github.com/kennethreitz/requests/issues/2771#issuecomment-139774010
.

I'm working in a company that we have python and c++ code site together, and the c++ guys definitely don't care or know about standard python package name.

If the C++ code is used in Python then perhaps your company needs a naming convention such that this doesn't happen. If your C++ developers don't care about not causing problems in the languages they're being consumed from then I'd say find new C++ developers because the ones your company has hired have some serious attitude problems.

Shadowing another standard library is another problem.

For example, if your company starts shadowing socket, httplib, http.client, ssl, OpenSSL, threading, collections or countless others, the protections in requests would only go so far because urllib3 imports those and doesn't also use this. Getting every upstream dependency of yours to use this will likely be more work than just fixing your corporate development process. Also, note that we're not against this change, I'm just being realistic about exactly how helpful it will be for people who carelessly shadow standard library modules.

The C++ developer is our CEO and I like this company.
I agree with you that fixing every upstream dependency would be not
possible, and this seemed to be a python 2 problem that cannot be fixed.
It's still unrealistic to check every standard library modules' name before
make a directory.
Regards.
On Mon, Sep 14, 2015 at 9:40 AM, Ian Cordasco<EMAIL_ADDRESS>wrote:

I'm working in a company that we have python and c++ code site together,
and the c++ guys definitely don't care or know about standard python
package name.
If the C++ code is used in Python then perhaps your company needs a naming
convention such that this doesn't happen. If your C++ developers don't care
about not causing problems in the languages they're being consumed from
then I'd say find new C++ developers because the ones your company has
hired have some serious attitude problems.
Shadowing another standard library is another problem.
For example, if your company starts shadowing socket, httplib, http.client,
ssl, OpenSSL, threading, collections or countless others, the protections
in requests would only go so far because urllib3 imports those and doesn't
also use this. Getting every upstream dependency of yours to use this will
likely be more work than just fixing your corporate development process.
Also, note that we're not against this change, I'm just being realistic
about exactly how helpful it will be for people who carelessly shadow
standard library modules.
—
Reply to this email directly or view it on GitHub
https://github.com/kennethreitz/requests/issues/2771#issuecomment-140078395
.

@shazow If I open a PR that does this for urllib3, would you merge it?

No strong feelings about this, I'd merge it if you think it's a good idea. :)

I think it's probably a good idea. It runs the risk of breaking people who are doing really sketchy crap on Python 2, like attempting to alias stdlib modules by having specially named modules on the import path, but...I feel like that's ok?

Worth a shot, we can always revert once people inevitably complain. :P

@Lukasa it's getting late by you, want me to tackle this tonight?

There's no rush, I'm happy to do it tomorrow, but if you're looking for something to occupy you then feel free to pick it up. =)

I had already done this and stashed it previously, so it was a matter of popping that and committing it. See also: https://github.com/shazow/urllib3/pull/735

This seems like an already solved issue too, so perhaps it's worth closing?


Typescript: TS7016: Could not find a declaration file for module 'numfmt'

Adding a types export fixes the issue for me:
--- a/node_modules/numfmt/package.json
+++ b/node_modules/numfmt/package.json
@@ -19,6 +19,7 @@
   "exports": {
     ".": {
       "require": "./dist/numfmt.js",
+      "types": "./dist/numfmt.d.ts",
       "default": "./lib/index.js"
     }
   },

Thanks for reporting, 3.1.1 has been published. 🙂


MSSQL DB session doesn't work

See https://github.com/yiisoft/yii2/issues/12599#issuecomment-274663294
Additional info

Q
A

Yii version
2.0.11

PHP version
any

Operating system
any

Hi,
I just modified the field 'data' to type varchar(max) and works

PHP 7.1.10
Yii 2.0.12
SQL Server 2017
DbSession
having problems decoding session from DbSession on MS SQL with [data] column as VARBINARY(MAX).
session decode is Failing. changed the datatype to nvarchar(max) and it seems to be working..

:checkered_flag:
Happy Halloween!
:jack_o_lantern:


Capital letters not converted.

Add images for capital letters as well and the code ignores capital letters. Hoping this will be added

Will fix it.
On Sun, Apr 5, 2020, 9:29 PM Pradeep Kumar R<EMAIL_ADDRESS>wrote:

Add images for capital letters as well and the code ignores capital
letters. Hoping this will be added
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/SaiTeja69/Digital_Text_To_Handwriting/issues/2, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ANBSTMBGP25JZYLTF5O7NRLRLCTF7ANCNFSM4MBBWQOA
.


feat(react): migrate to useSyncExternalStore

Migrate the React subscription logic to useSyncExternalStore.
The main advantage of adopting useSyncExternalStore is to follow "The React Way" to subscribe to external stores, which should give us better compatibility and hopefully less surprises with the latest React features.
useSyncExternalStore also comes with a dedicated getter which should hopefully help with the SSR integration.
TODO:

[x] Check how references are managed within Jazz
[ ] Look for ways to optimize state management and how the React Compiler may help on simplifying the best pratices
[ ] Do some resarch on how we could use Suspense (would probably be very useful for SSR)

Tested all the example apps!
Only the PasswordManager required a fix because the changes on how we manage references now is actually a breaking change.
Effects with a coValue as dependency are now triggered every time they get an update.
We will probably need to communicate that 🤔
On the other side this should make us more compatible with the React Compiler, but that's something I will confirm with more tests.
Next-up, unit tests!


build(packaging): unify shared dependencies for integration tests

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/NG-ZORRO/ng-zorro-antd/blob/master/CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[x] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Application (the showcase website) / infrastructure changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: N/A
What is the new behavior?
Integration tests would reuse the dependencies in repo itself.
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information
Internal decision.

Coverage remained the same at 41.233% when pulling 7e18fe5c608304bfc70dfd983e63289bf22e0e48 on trotyl:unify-integration-deps into 4efd6f7f19fab6d35520e4d162d5c16e767a6b06 on NG-ZORRO:master.

Coverage remained the same at 41.233% when pulling 7e18fe5c608304bfc70dfd983e63289bf22e0e48 on trotyl:unify-integration-deps into 4efd6f7f19fab6d35520e4d162d5c16e767a6b06 on NG-ZORRO:master.

Coverage remained the same at 43.85% when pulling c43810051f52a3f3c7abc969fa86613b5c3db336 on trotyl:unify-integration-deps into 3289e7fe13c4d92fa2457f253bad6142839c718b on NG-ZORRO:master.


When targeting API 26 (Android O) logcat is full of warnings.

W/View: requestLayout() improperly called by android.support.v7.widget.AppCompatTextView{9cb9cc0 I.ED..... ......I. 3,3-171,171} during layout: running second layout pass
These went away when I removed com.futuremind.recyclerfastscroll:fastscroll:0.2.5 and everything that made use of it in my code.
I'm getting a ton of these.
I have tabbed fragments which contain a RecyclerView which makes use of fastscroll. When the app starts, only two fragments are created. Whenever something needs to redraw the fragmets, that warning comes up (including each animation frame, so really a lot), one for each fragment. It really floods logcat. When I scroll a fragment to the left, so that a new one is created because a new tab is getting prepared by the system, then not only three different addresses (like the one in the sample 9cb9cc0) exist, but somehow a loop is created so that I get a constant stream of this warnings without even doing anything.
Also, when scrolling the adapter which is under the influence of fastscroll (scrolling it normally), it kind of looks as if everything is getting redrawn twice, it no longer looks smooth like when fastscroll is removed.

Same for me. The performance gets really dragged down by this otherwise fantastic library and the logcat gets flooded by the warnings posted by danielfaust.

Sorry guys, this lib is quite old and I don't have time to support it anymore. If you found this error and be kind enough to propose a PR, it would be more than welcome.

you need:
https://github.com/FutureMind/recycler-fast-scroll/blob/master/fastscroll/src/main/java/com/futuremind/recyclerviewfastscroll/FastScroller.java#L158
@Override
protected void onLayout(boolean changed, int l, int t, int r, int b) {
    super.onLayout(changed, l, t, r, b);

   if (changed) {
      initHandleMovement();
      bubbleOffset = viewProvider.getBubbleOffset();
      ...

   }

also updateHandlePosition can be optimized:
private float previousPos  = 0;
void updateHandlePosition(RecyclerView rv) {
if (previousPos!=relativePos) {
scroller.setScrollerPosition(relativePos);
....


Copy grpc's "error_details" functions

Internal: b/170952273
TL;DR: Both the grpc/grpc repo and the googleapis/googleapis repo have a status.proto file. gRPC compiles theirs into the libgrpc++_error_details.so library, and google-cloud-cpp compiles it into libgoogle_cloud_common.so. Any user who links in both will get an ODR error.
To help avoid this, we're going to declare that the status.proto in googleapis (https://github.com/googleapis/googleapis/blob/master/google/rpc/status.proto) is preferred, and any user needing both grpc and google-cloud-cpp should NOT link libgrpc++_error_details.so.
So step 1 is that we need to copy the functions from https://github.com/grpc/grpc/blob/master/src/cpp/util/error_details.cc (modulo namespace changing and possible function renaming is fine).

We no longer need this due to https://github.com/grpc/grpc/pull/25196

We no longer need this due to https://github.com/grpc/grpc/pull/25196


Not working on LibreWolf

Hello 👋.
It does work on Firefox, didn't tried my Cromium browser yet. But LibreWolf is my daily driver ☹
First I logged in on the web GUI the I tried to sync the extension with the raindrop.io web but didn't let me.
Basic troubleshooting I did: close browser, restart PC

Then isn't the problem caused by LibreWolf? If the upstream (and sole) extension target, firefox, is supported, this is caused by LibreWolf.

@katastrophe92, if you closed the issue because it was not reproducible on Firefox, then close the issue as invalid rather than https://github.com/raindropio/app/issues?q=is%3Aissue+is%3Aclosed+reason%3Acompleted. Additionally, you should open an issue at https://gitlab.com/groups/librewolf-community/-/issues.
If instead the issue was fixed, please clarify.


monitor: Export trace observation point via API

This change is

Coverage decreased (-0.003%) to 44.004% when pulling 2264c3f2f65471beb576f6e6fc5ef49d50a94b50 on pr/tgraf/export-trace-observation-point into a3f76386742b8f5b619413d0c12243e42defcf58 on master.

test-me-please


Settings-Tunables edit button opens wrong setting

[x] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[x] I have searched the existing issues and I am convinced that mine is new.

When going to Settings-Tunables and trying to edit a value, a different value is opened for editing.
Steps to reproduce the behavior:

Go to System-Settings->Tunables
Click on edit button for 'hw.ibrs_disable'
See edit for 'vfs.read_max' is opened

Seems to be wrong for most of the edit buttons.
Expected behavior
The correct value should be opened for editing
Describe alternatives you considered
As a workaround I manipulate the url parameter for "id" until I found the correct id for editing the value.
Environment
OPNsense OPNsense 21.1.1-amd64    (updateted from 20.7 to 21.1 to 21.1.1)
Browser Chrome & Firefox

ugh, php is horrible...added sorting to the page in 21.1.1 ... I'll fix it

Wow, that was a fast response :D

Sorry, lunch got in the way ;)
# opnsense-patch 10646e04c6

That should fix it.
Cheers,
Franco

Yes. Patch seems to work for me.

Thanks, will be in 21.1.2 ❤


Add an SDK section

The documentation needs list all the available SDKs so users are able to quickly see what's supported. Supabase has a nice frameworks section that could be used as inspiration.
https://supabase.com/docs/#start-with-a-framework
Dyte also has a page that's similar to what we're looking for.
https://docs.dyte.io/

https://devicon.dev/ may be a nice resource for technology logos.


 INDIAN Call☂Girls  Service In {Dehradun service}♥  Dehradun 💫

NOtify Me +93I579IO9O╭──╯ . . . . . INDIAN Call☂Girls  Service In {Dehradun service}♥  Dehradun 💫 ,  NOtify Me +93I579IO9O╭──╯ . . . . . INDIAN Call☂Girls  Service In {Dehradun service}♥  Dehradun 💫prem nagar call girls in dehradun escorts
NOtify Me +93I579IO9O╭──╯ . . . . . INDIAN Call☂Girls  Service In {Dehradun service}♥  Dehradun 💫 ,  NOtify Me +93I579IO9O╭──╯ . . . . . INDIAN Call☂Girls  Service In {Dehradun service}♥  Dehradun 💫prem nagar call girls in dehradun escorts

NOtify Me +93I579IO9O╭──╯ . . . . . INDIAN Call☂Girls  Service In {Dehradun service}♥  Dehradun 💫 ,  NOtify Me +93I579IO9O╭──╯ . . . . . INDIAN Call☂Girls  Service In {Dehradun service}♥  Dehradun 💫prem nagar call girls in dehradun escorts

NOtify Me +9671536526 ╭──╯ . . . . . INDIAN Call☂Girls Service In {Dehradun service}♥ Dehradun 💫NOtify Me +9671536526 ╭──╯ . . . . . INDIAN Call☂Girls Service In {Dehradun service}♥ Dehradun 💫NOtify Me +9671536526 ╭──╯ . . . . . INDIAN Call☂Girls Service In {Dehradun service}♥ Dehradun 💫NOtify Me +9671536526 ╭──╯ . . . . . INDIAN Call☂Girls Service In {Dehradun service}♥ Dehradun 💫NOtify Me +9671536526 ╭──╯ . . . . . INDIAN Call☂Girls Service In {Dehradun service}♥ Dehradun 💫


Kestrel not working on AWS ECS with dynamic ports

I am working on getting a Kestrel/Linux based web service running on AWS ECS, from within a Linux container using the default AMI. One of the features of ECS is that you can define the container port, but leave the host port undefined. ECS will then find an open port on the host, and hook up the application load balancer to that port.
When I attempt to get this working, I get a "connection reset by peer" error when attempting to connect to the container, both from the load balancer, and from the container host. If I don't use the dynamic port assignment, and instead specify the host port to same port used in the Dockerfile and for the ASPNETCORE_URLS environment variable, it works perfectly. I understand that this change was made to Kestrel in the RTM timeframe. The change makes sense, as Kestrel should be bound to a specific port, but it breaks this scenario.
I am still researching if it's possible to get the host port from within the ECS container. In the meantime, is there a way to override this behavior, and have Kestrel respond on all ports, or even better, a range of ports? If not, this would be an extremely valuable feature, as right now it limits the ability to run on ECS.

is there a way to override this behavior, and have Kestrel respond on all ports, or even better, a range of ports?

At the moment, Kestrel only supports binding to specific ports (or to port 0, which will dynamically assign a port).

When binding to port 0, how does the dynamic assignment work? Does Kestrel attempt to find an open port? I'm wondering if I can leverage this somehow.

The operation system will choose one for you.
IIRC, you can can configure what ports the system will choose using
cat /proc/sys/net/ipv4/ip_local_port_range
*Hibernating Rhinos Ltd  *
Oren Eini* l CEO l *Mobile: + 972-52-548-6969
Office: +972-4-622-7811 *l *Fax: +972-153-4-622-7811
On Mon, Jan 30, 2017 at 10:55 PM, erick-thompson<EMAIL_ADDRESS>wrote:

When binding to port 0, how does the dynamic assignment work? Does Kestrel
attempt to find an open port? I'm wondering if I can leverage this somehow.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/aspnet/KestrelHttpServer/issues/1332#issuecomment-276187484,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAHIs1uRjnSiQprZpG_DL3UB4o4YaiuWks5rXk4ygaJpZM4Lx3k8
.

Thanks!
I'm thinking that I could use this to have Kestrel listen to that range of ports (all 30000 of them). I'm not sure what the performance overhead would be of doing this. I can't seem to find the code that checks the port against ASPNETCORE_URLS to see if I have any options for using an array or range.

Find what code? You'd have to write it yourself. That's not built in.

Sorry I wasn't more clear.
I was trying to track down the code that uses the ASPNETCORE_URLS environment variable. I got as far as WebHost.EnsureServer(), where the value ends up in IServerAddressesFeature.Addresses, but I can't find where the addresses are checked against the current request.
The reason why I want to find it is that I was thinking about something like the following:

// get port range from cat /proc/sys/net/ipv4/ip_local_port_range
var serverUrls = Enumerable
    .Range(32768, 60999)
    .Select(p => $"http://<IP_ADDRESS>:{p}")
    .ToArray();

var host = new WebHostBuilder()
    .UseKestrel()
    .UseContentRoot(Directory.GetCurrentDirectory())
    .UseStartup<Startup>()
    .UseUrls(serverUrls)
    .Build();

host.Run();

But before I do that, I want to make sure I'm not going to be doing a scan of a 30k item array for every request. Or if I am, are there are other options that could make it more efficient.
Thanks,
Erick
https://github.com/aspnet/Hosting/blob/master/src/Microsoft.AspNetCore.Hosting/Internal/WebHost.cs

I can't find where the addresses are checked against the current request

That doesn't happen.

When running in a container on AWS ECS, if you don't specify the correct port you get a connection reset by peer. What is doing the reset? It's not the OS, as the behavior changed between RC1 and RTM.

No idea, maybe it's a different problem altogether. Did you try turning on verbose logging to see what the logs say?

@erick-thompson I second the request for the logs, but it just occurred to me what might be going on.
Can you still make requests from inside the container? If so, I think the difference between RC1 and RTM is that by default in RC1, Kestrel would bind to "http://*:5000". By RTM (and later RCs), Kestrel's default changed to "http://localhost:5000".
In other words, Kestrel now binds to IPAddress.Loopback instead of IPAddress.Any by default, and will not respond to external requests. This was to make Kestrel's default configuration more secure, particularly for the case when Kestrel is running behind a reverse proxy as we recommend.
Not specifying any URL doesn't instruct Kestrel to use dynamic port assignment. As mentioned above, specifying port 0 does that instead (e.g. "http://*:0") though this seems to be a pretty rarely used feature which requires parsing the IServerAddressesFeature to programmatically determine what port was bound to. Without any other configuration, ASP.NET Core will always bind to port 5000.

I must have missed the ability to specify 0 as a port. I'm planning to try this and see if I can't get dynamic ports working in ECS. I'll update the thread once I test it out.

We are closing this issue because no further action is planned for this issue. If you still have any issues or questions, please log a new issue with any additional details that you have.


Use /usr/bin/env to locate python (wasn't working on eg Ubuntu)

Thanks for reporting this problem and for providing the solution. I commited your fix.


Implement password change UI for Developer portal

Describe your problem(s)
Depends on : https://github.com/wso2/product-apim/issues/8181
Describe your solution

How will you implement it

Optional Fields
Related Issues:

Suggested Labels:

Suggested Assignees:

Fixed with https://github.com/wso2/carbon-apimgt/pull/8780


Ensure task read to EOF

We should use NSFileHandleReadToEndOfFileCompletionNotification instead of the ReadCompletion one. This does a async read until EOF. I tested this and will not cause the pipe buffer to fill (the cause the of the previous installer hang).
Also, I did a bunch of testing on VMs and it is possible for the process to terminate before the EOF notifications, so, I changed the termination handler to wait until EOF to complete the task.

👍


[Doc] round_robin_gradients

Document ZeRO-2 offload optimization: round_robin_gradients.

Looks good - thank you for adding it, @tjruwase!
Did you want to add why is it off by default - i.e. may fail at times? Or is it just a matter of time before it's always working?

We plan to make it work always. I don't see any fundamental reason why it shouldn't, it just needs testing in more scenarios.


Route53 service programs AWS before stack metadata is set for it

https://forums.rancher.com/t/rancher-just-wiped-my-route-53-zone/1448
Caused by bug in go-rancher-metadata. Route53 service accesses /self/stack information to get the environment name. And go-rancher-metadata doesn't validate response code when accessing metadata endpoints, so empty stack data is returned to route53 service which results in environmentName part being empty in FQDN.
Things to fix:

go-rancher-metadata to return error when http response !=200
in external-dns code, wait for the self/stack information to be set before proceeding further.

This bug is not very easy to reproduce. The steps user has taken:

Deploy route53 service, let it run for a day or two.
Make sure all the records for your services are present on Route52 at the end of the second day

Tested with latest build from master:

Deployed route53 service and let the server run for couple of days.
Made sure all the records for your services are present on Route52 at the end of the second day


Release 0.5.0-beta.3

Summary
A few cleanups before 0.5.0-beta.3 can be released.
Checklist

[x] cargo fmt
[x] cargo clippy
[x] cargo test
[x] updated CHANGES.md

Don't have time for a full review right now, but one thing I meant to get in before the next beta: adding Controller to the prelude.

I've added Controller to the prelude. I think worker related types are not used that often and are better imported using use relm4::worker::*
Also, I hope that https://github.com/rust-lang/rust/issues/102352 will be resolved soon, so our CI doesn't fail constantly...


Fix immersive reader in electron

Api requests in electron expect the local token for authentication, where auth.apiAsync attaches the user's token for authentication - so typically we wanna use Cloud.apiRequestWithCdnAsync. In this case we always want to hit the live endpoint too (makecode.com/api/immreader instead of localhost/api/immreader), so this fixes in electron. My understanding is that we want these to hit through the cdn anyways? e.g. https://pxt.azureedge.net/api/immreader?cdn=20210402 (I believethe cdn= changes every 12 hours so that should be good with the refreshes, though I'll double check -- if it's 24 hours will have to fix that or do a different fix.)
test https://arcade.makecode.com/app/d32c7f199fbdc6a758b45eb7255cda3b1caacada-7050ed1730

Also I think small side fix in https://github.com/microsoft/pxt/commit/61eba0bed2cda340551e41292c19664b337f35b4 @livcheerful, let me know if it was intentional but the removeLocal wasn't removing the token as the key didn't match

Ahh looks like the cdn= is only changed every 24 hours and not 12 hours like I though, will see how else to fix. (I guess could just add an optional param to cdnApiUrl that would make it refresh hourly? Seems reasonable to me at least)

Okay updated to make it hit the cdn with an hourly cache instead of a 24hour cache - e.g. https://pxt.azureedge.net/api/immreader?cdn=2021040223. All other requests still hit daily, it's toggled by an option to Cloud.apiRequestWithCdnAsync Here's an updated build:
https://arcade.makecode.com/app/b3170605f9310b1258308471a6e9c2caa016b2bd-6aec3e2b8f

Okay updated to make it hit the cdn with an hourly

Why does it need to hit cdn at all? Could this make the immersive reader fail for up to an hour if the token expired in the meantime?

The backend actually provides a new token every 12 hours, and each token is valid for 24 hours so.....I think it will be fine?

The backend actually provides a new token every 12 hours, and each token is valid for 24 hours so.....I think it will be fine?

Oh that makes sense!

For now you could just disable immersive reader for offline app. We can take our time and figure out.

Skip cdn?


Import 304bec76d5: Updated OpenVR to v1.10.30 (by Maki)

This is an automated proposal to look at a commit made by Maki and import it into Overte

Commit:
304bec76d529928785d9968998ac8fa73f105f6c

Author:
Maki

Date:
Tue, 31 Mar 2020 19:16

Updated OpenVR to v1.10.30

Stats:

Filename
Stats
Lines
Added
Removed
Lines in blame

cmake/ports/openvr/CONTROL

3
2
1
⚠ Path not in HEAD

cmake/ports/openvr/portfile.cmake

52
24
28
⚠ 0

2 files
-
55
26
29
0

To work on this, please assign the issue to yourself, then look at the commit and decide whether this would be a good addition to Overte.

If the commit is useful, tag it with "Tivoli: Keep", and keep it open until it's merged.
If the commit is not useful, tag it with "Tivoli: Discard", and close it.
If the commit is not useful right now, but might be later, tag it with "Tivoli: Maybe later", and close it.
If it's hard to decide, tag it with "Tivoli: Discuss", and keep it open.

Useful commits should be submitted as a PR against Overte. Tag this issue in PR, so that it's automatically closed once the PR is merged.
You can cherry-pick this issue with this command:
git cherry-pick 304bec76d529928785d9968998ac8fa73f105f6c

reverted later


ERROR: Could not dump memory map

Hello, after I load the plug-in, the plug-in does not run properly. And the process list cannot read the process. Please ask, what is the problem? I am a newbie in c++。
ce version 7.5

My computer's CPU is i7 and my computer system is win11. Is it related to the computer system?

do you have a FPGA DMA device?


Travis CI failing for Python v3.6

Pytests failing on tests/callgrind.py. I believe it is coming from the re.match in gprof_dot_reader.py.

I added restrictions on master so that pull requests are required to commit there, and tests are required to pass before merging (that includes @bhatele).  This should keep master clean so we do not have to fix other peoples' bugs.

E   KeyError: 'node'

pandas/_libs/hashtable_class_helper.pxi:1608: KeyError
_________________________________________________________________________ test_read_calc_pi_database __________________________________________________________________________

calc_pi_callgrind_dot = '/private/var/folders/0s/q_y0zhfj6xdd5n7rn780zz6h001qr7/T/pytest-of-gamblin2/pytest-10/test_read_calc_pi_database1/callgrind.dot.64042.0.1'

    def test_read_calc_pi_database(calc_pi_callgrind_dot):
        """Sanity check the GprofDot reader by examining a known input."""
        reader = GprofDotReader(str(calc_pi_callgrind_dot))

        list_roots = reader.create_graph()
        root_names = []
        for root in list_roots:
            root_names.append(root.callpath[-1])

>       assert all(rt in root_names for rt in roots)
E       assert False
E        +  where False = all(<generator object test_read_calc_pi_database.<locals>.<genexpr> at 0x1194aab10>)

tests/callgrind.py:44: AssertionError

Some of the pandas errors are a bit hard to interpret because it's written partially in cython, and I guess there's no context for the cython frames.
I'm not sure when the python 3 tests last worked, though -- @bhatele?

Thanks @tgamblin!
I started working on a fix for this yesterday and just pushed up changes to my branch here: https://github.com/slabasan/hatchet/tree/fix-travis-ci. Using regular expressions containing hexadecimals (e.g., \xc3\x97) in Python2 and Python3 have different behaviors. I could not find an elegant way of handling both cases with a single call. The branch I have checks for the Python version and handles them accordingly -- that is, using byte representation if Python3. Open to ideas on a more elegant solution.

Hey @slabasan: that's a nice catch.  I see what you're talking about now with the hex literals.  I think we should figure out a way to simplify that code, though. Are you around tomorrow?
As it stands, the file looks pretty unmaintainable.  In particular, I think the real issue here is that we're parsing dot lines with a really complicated regex -- there are simpler (and more correct) ways to do this.
I'm curious why @bhatele isn't parsing using pydot or another library before reading the data in.

I am switching the gprof reader to use pydot here: https://github.com/slabasan/hatchet/tree/pydot-for-dot-parsing. Two challenges: (1) pydot imports the beginning keyword lines as nodes, and is included in the list of roots. The keywords are not rows in the dataframe, so I get an error KeyError: 'the label [graph] is not in the [index]'; (2) this code works for Python3, pydot v1.4.1, pyparsing v2.3.0, and setuptools v40.6.3. Using python2 has the following error UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 305: ordinal not in range(128). I'm not sure trying to specify different version dependencies for pydot, pyparsing, and setuptools` depending on the Python version a good path forward.

This should be fixed with #20

Fixed by #25


Perl 6 runtime error.

The problem I am having is that when I run a script, like this:
#!/usr/bin/perl
use v6;
say "Hello World!";
It prints out this error:
Perl v6.0.0 required--this is only v5.18.2
Please help, I really would like to use this to run Perl 6.

On *nix,
#!/usr/bin/env perl6
use v6;
say "Hello world";

On windows,
use v6;
say "Hello world";


improve a11y

[x] make the custom "tooltips" show up on keyboard focus
[x] popup for mobile saying mobile isn't fully supported
[x] explicitly integrate a11y tooling into this repo with lighthouse/@lhci/cli/lhci or axe-cli
[ ] use _2dnote) when using the mouse to drag items around
[ ] replace aria-label with aria-labelledby and text nodes, which can leverage translation services, as per my reading of the "Inclusive Components" book
[ ] ~default to announcing images with filename (requires saving file name too)?~
[ ] consider enabling positioning text for keyboard-only users (maybe leverage _2dnote)?

(continuation of #14)

moving remaining unchecked items into separate tickets


WOR-649 spring 3.0, etc.

https://broadworkbench.atlassian.net/browse/WOR-649
upgrade to spring boot 3 and jakarta
switch from opencensus to opentelemetry

Can you also remove the override in dependabot  configuration? https://github.com/DataBiosphere/terra-billing-profile-manager/pull/95/files

@blakery I can't figure out how to get to the artifact that explains which pact tests failed. Do you know?

@dvoet this is looking good. Would you mind tossing this branch in a bee so that we can do a sanity check that events are getting sent to the Prometheus endpoint?


Phoenix only autocompletes for wrong quote type

Describe the bug:
Somewhat related to #2146 and #2147, when autocompleting for Phoenix, which uses double-quotes to quote identifiers such as schema or table names, the autocomplete returns results after a backtick (which isn't valid) but does not return results after a double-quote (which is valid).
Context: I'm hoping that one of your pre-existing parsers will work well for Postgres SQL for a project I'm working on 😄 . I thought the Phoenix one would be a good fit and ran into this issue when trying it out.
Steps to reproduce it?

Type SELECT * FROM  in an editor for the phoenix dialect.
Type a backtick and trigger autocomplete - schema and table names show up even though they shouldn't.
Instead, type a double-quote and trigger autocomplete - nothing shows up even though it should.

Demo

Hue version or source? Latest version of https://demo.gethue.com/hue/editor/?type=2

It's a rather big change as the editor was originally built with only taking backtick quotes into account for identifiers. Over time we've added some support for different quote types but not throughout. There are for instance still utils that help with deciding whether to backtick a value or not that'd need to be adjusted.
On the parser side of it... A parser for a specific dialect is normally built with a mix of grammar "modules" from other dialects and I don't think it'd be wise to make a split there. An option to explore would be to accept any type of quotes for identifiers for all parsers, and move the decision to use a specific quote up to the consumer of the parse result. Alternatively, we could pick the quote type to use per dialect if we move to a dedicated lexer per parser, semantically that might make more sense.
I agree that this should be fixed as we're now supporting more and more dialects for the editor, however, it's difficult to say when we can see some improvements here at this point of time.

Thanks for the summary, makes sense. I saw this PR which seems to add some support for double-quotes in the grammars, and that the backtickIfNecessary logic has some support for different quote types, but also found some hardcoded things like you alluded to.
I don't think I'm well informed enough to say which of your ideas makes the most sense, but appreciate the informative response!