Ignite cluster (both client and server) breaking intermittently with 401 http code in AKS

I am able to deploy ignite cluster in AKS successfully
Note: 1. service account, role and role binding are created already and have also mentioned service account in deployment.yaml
2. service name and namespace are also mentioned in ignite configuration
3. 3. We have configured thin clients in cluster
Client and server are able to join and communicate initially but after below intermittent exception, both are unable to reconnect to cluster and unfortunately, we have to redeploy entire helm chart,
org.apache.ignite.logger.java.JavaLogger error
SEVERE: Failed to clean IP finder up
class org.apache.ignite.spi.IgniteSpiException: Failed to retrieve Ignite pods IP addresses.
at org.apache.ignite.spi.discovery.tcp.ipfinder.kubernetes.TcpDiscoveryKubernetesIpFinder.getRegisteredAddresses(TcpDiscoveryKubernetesIpFinder.java:81)
at org.apache.ignite.spi.discovery.tcp.TcpDiscoverySpi.registeredAddresses(TcpDiscoverySpi.java:2057)
at org.apache.ignite.spi.discovery.tcp.ServerImpl$IpFinderCleaner.cleanIpFinder(ServerImpl.java:2279)
Caused by: class org.apache.ignite.IgniteException: Failed to retrieve Ignite pods IP addresses.
at org.apache.ignite.internal.kubernetes.connection.KubernetesServiceAddressResolver.getServiceAddresses(KubernetesServiceAddressResolver.java:123)
Caused by: java.io.IOException: Server returned HTTP response code: 401 for URL: https://kubernetes.default.svc.cluster.local:443/api/v1/namespaces/ignite/endpoints/cache-service
at java.base/sun.net.www.protocol.http.HttpURLConnection.getInputStream0(HttpURLConnection.java:2000)
at java.base/sun.net.www.protocol.http.HttpURLConnection.getInputStream(HttpURLConnection.java:1589)
at java.base/sun.net.www.protocol.https.HttpsURLConnectionImpl.getInputStream(HttpsURLConnectionImpl.java:224)
at org.apache.ignite.internal.kubernetes.connection.KubernetesServiceAddressResolver.getServiceAddresses(KubernetesServiceAddressResolver.java:111)

Hi, 401 is Unauthorized exception. By default Ignite uses account token located in "/var/run/secrets/kubernetes.io/serviceaccount/token". Custom can be set with KubernetesConnectionConfiguration#setAccountToken. Please check your accounts configuration.


Multithreading / Multiprocessing queries for visualizations

Looking at the logs that we get when our app queries it's backing DB, I see that the queries run sequentially. For the long run, it would be interesting to see if we could multithread this task so that the graphs are able to load in asynchronously w.r.t each other. Instead of a sum maximum load-time, the amount of time for the app to load all graphs would only be the max time to load.

done by worker queue


Added '--config' to yaml example in readme

Updated the example in the readme to include the '--config' flag. Without it skeletor interprets   config.yaml as the experiment name.

Thanks for catching this!


Update 0500-12-06-remote-mapping_en.md

Since in the next paragraph the JOSM hyperlink is pointing to the download/developers site it is consistent to have the ID hyperlink doing the same.

If that website contained something like a user guide I would concur. But it only provides information how to take part in development and how to set your own clone up. This is not what people interested in remote mapping will find useful. As long as the iD developers do not publish a user guide I would like to leave our own guide as reference here.


APT whitelist request for qtbase5-dev

Ran tests and found setuid bits. See https://travis-ci.org/travis-ci/apt-whitelist-checker/builds/72285025.

False positiive, there's a getter & setter to check if setuid is allowed.


Nested emphases

Fixes #303.
Would love feedback on naming and API in general.

[ ] add example to readme probably?

I think Emphasis is pretty good! Falls apart for things like .tightLineSpacing though. 🤔

I could separate that into a distinct FontLineSpacingVariant enum. Makes the API cleaner, but harder to find things if you're coming from knowledge of NSAttributedString.

Gong to try simplifying this to cover just bold and italics for now, since they are by far the most common uses of this feature. New items can be added later with a different API.

Also, that makes the naming much easier: if it's just .bold and .italic, then Emphasis makes more sense.


Memory sizing not optimal on large servers

I'm currently trying to setup the Oracle Database SingleInstance 12.1.x image for a CI/CD server.
It is a CentOS server with 32 CPUs and 250 GB of memory.
We need multiple database instances for each branch.
The image creation is ok and I can use the database. But as soon as I try to limit the memory (to 5GB) of the docker container I run into issues with same kind of error than:
https://github.com/oracle/docker-images/issues/307
The database only works when no memory limitation is set.
I found a workaround by adding an ENV ORACLE_MEMORY variable and removing the code from createDB.sh:
if [ `nproc` -gt 8 ]; then
    sed -i -e 's|TOTALMEMORY = "2048"||g' $ORACLE_BASE/dbca.rsp
fi;

This code is removing the 2GB memory limit on servers with more than 8 cpus (my case). Without this memory limit, my instance is consuming around 80 GB !! of memory just for a single container...
When setting, the dbca.rsp memory limit to 5GB for example, everything works fine with memory limtation on the container.
Would it be possible to add a parameter allowing to select the memory strategy ?

Hi @samuelvetsch, I'm confuse about the 5GB. At the top you say 5GB made you run into similar issues at #307 but at the bottom you say that 5GB worked fine?
You can override the memory requirements by editing the dbca.rsp.tmpl file. The sed command will only replace the TOTALMEMORY = "2048" line but if the line happens to be `TOTALMEMORY=" 5120" it won't take any affect.
Also note that this is only used for database creation time. Once the database is up and running , you can and will have to set the database parameters accordingly SGA_MAX_SIZE, SGA_TARGET, PGA_AGGREGATE_LIMIT, PGA_AGGREGATE_TARGET.

Hi, the trigger is not around 5GB, that was just my target memory size example. The trigger was about the number of CPU.
In my tests, once db created and started with full memory (250GB), I was not able to restart the DB with less than 15GB using docker memory limits. But I did not play with the SGA_MAX_SIZE, SGA_TARGET, PGA_AGGREGATE_LIMIT, PGA_AGGREGATE_TARGET.
I would just suggest to make this TOTALMEMORY an ENV or ARG parameter in order to let user define manually this size.

Once the database is created, passing on an environment variable won't help either anymore as the memory requirements are specified as a database parameter. So let's say you create a new database container, store the database in a volume (as one should), throw the container away and recreate a new container, the TOTALMEMORY won't have any affect anymore as the database is already created. At that point the user has to configure SGA_TARGET and PGA_AGGREGATE_TARGET accordingly for what he wants.
If totalmemory is not specified, the installer itself will take 40% of the total memory available. So if you were to create a new container with Docker memory limits in place, the memory consumption is already adjusted accordingly by the installer itself if you happen to run on 8 CPUs and more.
Otherwise, it will run with as little memory as Oracle recommends, which happens to be 2GB. One can increase that after database setup as well, if wanted.


add dev-mode

Change Summary
Adds a semi-hidden dev mode onto the demo site with more technical, less sales-y options available. Accessible through clicking the home icon 5 times.
This change allows us to fully deprecate the Dogfood Retool app and rely on the demo site for testing. As we add more options we can continue down this route of revealing it through the demo site, which merchant devs will find useful as well.
In addition to the dev mode, add rejection reason test shortcut emails.
Dev mode off (initial load):

Dev mode on:

@alvinsng Hmm, I'm not seeing the complexity other than a frontend boolean - what am I missing?
I chose to have the dev mode and additional options versus in a separate dropdown because the additional options override Prequalified and New, and work in conjunction with Pay Now & Later|Pay Now|Pay Later. ie, a user can select "X - Order Min" and "Pay Now & Later". So it wouldn't quite make sense for the additional options to not show up next to Prequalified|New.


.editorconfig

Would be great if you could put a .editorconfig

👍 did you want to make a pull request for that?

https://github.com/johnbywater/eventsourcing/blob/develop/.editorconfig ;-)


CacheTool: Cleanup TS_INLINE.

inline is part of the language now.

[approve ci]

[approve ci]

This doesn't look like Cache Tool specific issue. Why didn't you remove / replace TS_INLINE from other places?

Because I only need this cleaned up in CacheTool for the YAML work, and because this is a dependent PR rather than stand alone, I wanted to keep it at a minimal scope.


Programmatically focus a Select

It is possible to focus Textfields, buttons with a ref. This isn't working with select.
I do not mean autoFocus. I want to focus on a select field when a user clicks on a button.
My code:
<FormControl  		className="select100"> 		<Select 		        ref={(formControll) => { this.formControll = formControll; }}								   			native 			value={aValue} 			input={<Input id="integer"   />} 		> 			{possibleOptions.map((item, key) => { 			  return (<option value={item} key={key}>{item}</option>) 			})} 		</Select> 	</FormControl>
If I now try to set the focus on the select with:
this.formControll.focus();
it isn't working.

Tech
Version

Material-UI
1.0.0-beta.24

React
16.2.0

browser
Chrome

Please reserve issues for issues with material ui.  If you have questions, please try StackOverflow.
That said, you need to use the inputRef property of the Input component.  See this codesandbox:
        <FormControl className="select100">
          <Select
            native
            value={aValue}
            input={
              <Input
                id="integer"
                inputRef={formControll => {
                  this.formControll = formControll;
                }}
              />
            }
          >
            {possibleOptions.map((item, key) => {
              return (
                <option value={item} key={key}>
                  {item}
                </option>
              );
            })}
          </Select>
        </FormControl>

Thank you. Normally I write on Stackoverflow, but because no one could help (Stackoverflow question), I thought it is a material-ui problem, because I import FormControl and Select from material-ui. I appreciate your help, but also your code isn`t working with inputRef. (Maybe you have written down more code on codesandbox, but sadly the link to codesandbox isn't working)

I just took your code, hardcoded aValue and possibleOptions, and added componentDidMount:
componentDidMount(){
    if (this.formControll) {
      this.formControll.focus();
    }
  }

I tried above solution for MUI 3.4.0
But it is not working

@sedi0861 Try: https://codesandbox.io/s/505nl3x85p

@joshwooding I need the focus when the component loading.
I tried adding
componentDidMount() {     this.handleFocus();   }
But it is not working.

@sedi0861 Sorry, check https://codesandbox.io/s/y79j9x8421
For your case I would use autoFocus

@joshwooding I tried autoFocus but it is not working for me.
And I tried the method in https://reactjs.org/docs/refs-and-the-dom.html#callback-refs
It doesn't work either.

@sedi0861  https://codesandbox.io/s/8nj6jz8n38 If I change handleFocus to use your textInput ref it works

@joshwooding  Oh wow. great! Thanks. How could that happen?  handleFocus did not have any effect on componentDidMount?

it didn't work for me, I used a controlled component https://codesandbox.io/s/inspiring-engelbart-osjok?file=/src/App.js


I need help

when i start a new world the game crashes.
Here is the log:
crash-2023-10-21_00.55.47-client.txt

https://github.com/ThinkingStudios/LambDynamicLights-Forge/issues/4


Default configuration results in The organization you were looking for was not found.

And I have no idea, where to look yet :-(

My mistake, I did NOT follow the readme properly,
I filled the GITLAB_BASE_DOMAIN with the base domain of sentry itself. doh.

:smile:  let me know if you have more questions

Thanks!
Got it working finally :-)


Found a small typo.

https://github.com/google/licenseclassifier/blob/bb04aff29e72e636ba260ec61150c6e15f111d7e/v2/classifier_test.go#L127
I think  at line 127 it should be          name: "overlap at start"

@utkarshshendge this ticket can be closed now.


Oxigraph server Compatibility with SPARQLWrapper

I've setup a entrypoint with the webserver and tried to connect via SPARQLWrapper as follows:

from SPARQLWrapper import SPARQLWrapper, JSON
endpoint = 'http://localhost:7878/query'
sparql = SPARQLWrapper(endpoint)
sparql.setTimeout(300)
#sparql.setReturnFormat(JSON)
sparql.setQuery('SELECT * WHERE { ?s ?p ?o} LIMIT 2')
results = sparql.query()
print(results)

But the server returns an error:
QueryBadFormed: QueryBadFormed: a bad request has been sent to the endpoint, probably the sparql query is bad formed.

Response:
b'Unexpected parameter: format'

Thank you for the feedback!
Oxigraph server is currently quite pedantic about the SPARQL request and returns an error when unknown query parameters are passed to the system. I just pushed e92c36f115bd536ec00973b8dcb7aadd3690db72 that allows unknown query parameters.

Nice, thanks! Once this is deployed in the docker I will test again.

I just released v0.2.2. The new image is in Docker hub.
Feel free to open the issue again if it does not work properly.

Seems to work!  Thanks a lot


Please build a static version

I'm using your release version 4.0d but it requires the visual studio runtime dll's.
Can you go to Project settings -> C/C++ -> Code Generation -> Runtime library and set to either (MTd or MT).
Can't run this on machines without the runtime library and I can't get the source to compile or else I would do it myself.
Thank you.

Hoping to use this miner version in a project of mine since this supposedly works on AMD R7's correct?
However installing the runtime library isn't an option for me.
Tried adding MSVCP140.dll and vcruntime.dll's to the folder where the miner is but that caused it to crash.
Please statically link these runtime dlls by using the settings I provided above and post a release version. I would greatly appreciate that.

sure, i can do that.

You're the best man!

ok, try https://github.com/maztheman/nheqminer/releases/download/v0.5a/v0.5a_AMD_Static.7z
let me know if that works for you.

Doesn't seem to work.
Says MSVCR100.dll is missing.
Did you make sure to in your Visual Studio, open Project properties -> Configuration properties -> C/C++ -> Code generation and change option Runtime Library to /MT instead of /MD for the specific build (release/debug) and version (x64/x86) of the project?

yes there is about 6 projects that are all /MT
On Thu, Dec 22, 2016 at 10:47 AM, basicsk8r13<EMAIL_ADDRESS>wrote:

Doesn't seem to work.
Says MSVCR100.dll is missing.
Did you make sure to in your Visual Studio, open Project properties ->
Configuration properties -> C/C++ -> Code generation and change option
Runtime Library to /MT instead of /MD for the specific build
(release/debug) and version (x64/x86) of the project?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/maztheman/nheqminer/issues/9#issuecomment-268852606,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABZS8OuS-7Wn6zUfV6j4vIMjC6cr-GNeks5rKre8gaJpZM4LUIQ1
.

That's very odd. I didn't have this problem ever with the original nheqminer that nice hash posted.
Must be some setting being over looked :/

Possible to get a 0.4d build statically?
I'm looking for an all in one solution for zcash mining on all platforms and I believe your version works on AMD R7's correct?
Hoping to use just one binary instead of one for each platform and I believe 0.4d is that solution?


Wrench cannot pick up energy cubes

I haven't played for about a month and just got back to the game. I started a new world, built some machines, then whipped up a Yeta Wrench to rearrange the temporary machine setup.
The Yeta Wrench works on everything EXCEPT Energy Cubes. I can switch the orientation of the output, but it will not allow me to pick up the block. I tried a few other mod wrenches to no avail. I tried to break it manually, which just ended up destroying it. So basically, in survival, once I place this block it is stuck there.
Everything was working before I came back to the game. The only thing that might have changed was the Java version (currently Version 8 Update 65, build 1.8.0-65-b17).
I did had an older version of Mekanism (Mekanism-1.7.10-<IP_ADDRESS>) so I updated that. No change.
Currently running: Mekanism-1.7.10-<IP_ADDRESS> on 1.7.10-Forge<IP_ADDRESS>8-1.7.10
Any clue as to why this issue is occuring? Anything else I could try? I didn't see any other conflicts. Thanks.
edit - I did try picking up this block with a pickaxe which is a workaround, but as far as I remember, I never needed it as wrenches (Yeta Wrench in my case) did the job. The wiki confirms that the wrench will pick up the block when shift-right clicking.

Can't reproduce this- I'm able to pick up energy cubes just fine.


Fix typos

Fix some typos

@swift-ci Please test.

@amartini51 That was quick! :D Yes sorry, I found some more typos and wanted to push those.... Maybe force pushing is frowned upon? But I wanted to avoid many commits with same message "fix typo".
I will mark it as ready when I'm done!

@amartini51 ready! I think I found all typos - or rather Xcode Check Spelling Now did, but one has to go through each file individually - an improvement opportunity for Xcode! :)

@swift-ci Please test.


Fix edge case where sign_bit_set is symbolic

This pullrequest adds more accurate handling of the signextend instruction modelling.
In particular, it accepts situations where the sign bit is symbolic.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Show dataPointLabelComponent in left/right side if trip, but it should be in visible zone of chart (cropped)

Thank you for your wonderful library, it helps me a lot
I see the datappointLabel is in the middle of trip now, I want it to be on the left or right of the trip.

I can use dataPointLabelShiftX to do that, but it cropped over the chart, and it does not fit in the visible chart

Do you have any way to make datapointLabel dynamically positioned way, it should not be cropped in the chart and in the left/right of trip?

Hi @dumihi313
You can dynamically position the datappointLabelComponent with the help of the third parameter in the callback function. The third parameter is the index of the current pointer or pointerIndex.
See https://stackoverflow.com/a/78814198/7533126

will this be applicable to focusedDataPointLabelComponent too?


mac desktop opens larger than my screen

not sure why...

Oh, oh. I am really sorry for this.
Thanks for reporting it.
Will look into it and let you know :)


As usual, not an open license

In the best traditions of Facebook, you're offering this model under a license that makes it impossible to use it for commercial projects. Which, in turn, means that nobody will have the scale to turn it into something useful.

In addition to your non-existent customer support for users and advertisers.
I really hope that one day your company goes bankrupt, so you give space to somebody who truly cares. The culture at Facebook is rotten.

Is anyone's company owned research lab really a good actor? Ah yes misusing the word "open" again. EleutherAI said they'd stop making bigger LLMs. I guess they'll be back at it... good thing for Huggingface. Not hating on anyone but yeah this is what hacker collectives are for.

Please refer to our whitepaper for extensive discussion of why we believe this technology is not ready for commercial applications.

I think that's pretty fair we don't mean to harass your team. Respect to you! :)) have a great rest of your day.


[feat] add "Kubernetes" to the data model

Added the kubernetes to the data model

@icepuma First technology PR and I'm thinking we should evolve the tehcnology HCL
technology "Kubernetes" {
  draft = false

  description = "Kubernetes, also known as K8s, is an open-source system for automating deployment, scaling, and management of containerized applications"

  website = "https://kubernetes.io/"
  documentation = "https://kubernetes.io/docs/home/"

  open_source {
    repository = "https://github.com/kubernetes/kubernetes"
  }
}

[ ] I think we can make the open_source and repository a block

If no block, open_source = false
If a block, open_source = true and repository is mandatory

[ ] Description may become over loaded here, so I'm thinking

Description should be used for SEO. It should be 300-500 characters (AutoGenerated from GPT and/or Wikipedia?)
Add a "Tagline" which for Kubernetes would be the current description: "Kubernetesis an open-source system for automating deployment, scaling, and management of containerized applications"

[ ] Add HashTags/Tags list
[ ] Add Aliases list (K8s)

Anything else that would be useful?
Feedback from you @JOSHUAJEBARAJ would be great also

ChatGPT generated this from my Kubernetes example :)
technology "prometheus" {
  draft = false

  name = "Prometheus"
  aliases = [
    "Prometheus Monitoring"
  ]

  tagline = "An open-source monitoring and alerting platform for cloud native environments"
  description = <<-END
    Prometheus is an open-source monitoring and alerting platform designed for cloud native environments. It features a multi-dimensional data model, a flexible query language, and a variety of powerful visualization and alerting tools.

    Prometheus is designed to be easy to deploy and operate, and can be integrated with a variety of other tools and platforms. It is highly scalable and can handle large amounts of data and a high volume of requests.

    Some key features of Prometheus include a powerful query language, support for metric recording and alerting, support for distributed architectures, and integrations with a variety of tools and platforms.
  END

  website = "https://prometheus.io/"
  documentation = "https://prometheus.io/docs/"

  tags = [
    "Monitoring",
    "Alerting",
    "Cloud Native",
    "Data Modeling",
    "Visualization",
    "Scalability"
  ]

  open_source {
    repository = "https://github.com/prometheus/prometheus"
  }
}

Will think about it after the hackathon call

@JOSHUAJEBARAJ Can you update to the new format? The changes to support the extra fields has been merged.

Hey @rawkode  Sure will do Just want to make sure we are using this format right ?
https://raw.githubusercontent.com/RawkodeAcademy/RawkodeAcademy/main/projects/rawkode.studio/content-management/data/technologies/rust.hcl

Yes. All the fields in the examples above are valid

Closing this PR and creating a new PR


Improve support for Secure PSoC64 family of devices

Depending on the configuration of the target, Debug registers may not be accessible immediately after acquisition. Catch all errors and repeat attempts to halt the CPU until it succeeds (with timeout).

/morph test


OSF download

Expands section about data, including more details about how to use OSF for hosting external data, with example functions for downloading and extracting them.

That's awesome! I will use this soon hopefully.
consider creating an example in the gallery that downloads a  relatively small dataset on OSF (a few MBs), and reference that one as a tutorial.
An idea could be moving your description of the python/command line commands to download the data to the example, and reference the example in the note, so that people can actively run the python version of the data pipeline.
Let me know if you like the suggestion!

@BalzaniEdoardo I think that's a good idea, went ahead and added an example. Let me know what you think! Two notes:

I named it plot_osf.py because of the issue you mentioned in #37, where, by default, only files starting with "plot" are run. It's not actually plotting anything, so it's confusing here. Where does the configuration live?
I'm not sure how much belongs in the note now, as opposed to the example. Double-check how I split them up and let me know what you think.

@BalzaniEdoardo I think that's a good idea, went ahead and added an example. Let me know what you think! Two notes:
* I named it `plot_osf.py` because of the issue you mentioned in [mkdocs-gallery sort tutorials & configs #37](https://github.com/flatironinstitute/ccn-template/issues/37), where, by default, only files starting with "plot" are run. It's not actually plotting anything, so it's confusing here. Where does the configuration live?

* I'm not sure how much belongs in the note now, as opposed to the example. Double-check how I split them up and let me know what you think.

For the configuration, that will be my second-to-next contribution to the template. Take a look at the issue #37, If I have to bet that's where you should set the config

Coming back to this much later: instead of using custom functions to download from OSF, use pooch. I'm changing plenoptic to use pooch as part of this PR and it's very effective.


Handshake Improvements.

Change acceptableVersion so that it also returns the "acceptable" data
agreed upon by the client and server.
Update the client so that it checks that response is acceptable.
Change runApplication to take the result of the version negotiation
instead of a local and remote version of protocol options.

bors r+


Add support to System Manager and Key Management Service

Hi all,
serverless framework supports those two systems in order to include secrets into your serverless application via env variables. It would be cool to have support for them into the configuration!

The following should be enough to read and decrypt SSM parameters
- Effect: Allow
  Action:
    - ssm:DescribeParameters
    - ssm:GetParameter
    - ssm:GetParameters
    - ssm:GetParametersByPath
  Resource: '*'
- Effect: Allow
  Action:
    - kms:Decrypt
  Resource: '*'

Hi @StErMi / @rvanlaarhoven ,
I submitted a PR with suggestions please take a look at #16
Thank you

Close the issue with PR merge


Add locations table

Adds a table to list the locations for each schema. Adjusts table size based on how many identifiers a schema has.

Codecov Report

Merging #64 (e0b5889) into develop (d0c491b) will decrease coverage by 0.28%.
 Consider uploading reports for the commit c1e069a to get more accurate results

@@             Coverage Diff             @@
##           develop      #64      +/-   ##
===========================================
- Coverage    83.53%   83.25%   -0.28%
===========================================
  Files           55       55
  Lines         3741     3757      +16
===========================================
+ Hits          3125     3128       +3
- Misses         616      629      +13

Impacted Files
Coverage Δ

dtbase/webapp/app/locations/routes.py
27.27% <18.75%> (-1.65%)
:arrow_down:


♻ Change methods not using its bound instance to staticmethods

The method doesn't use its bound instance. Decorate this method with @staticmethod decorator, so that Python does not have to instantiate a bound method for every instance of this class thereby saving memory and computation. Read more about staticmethods here.

📝 Docs preview for commit d3071952b6ee4105d3e6a3d8f9f066d6d3aa0c2f at: https://9cb8dfe1.fastapitiangolo.pages.dev

📝 Docs preview for commit 81541735ab657ce636b88ffd6e93fa97c2eea6aa at: https://77c3a885.fastapitiangolo.pages.dev

Thanks for the interest. The intention of these docs is to explain the concepts, only about dependencies. This would be an optimization specific to how to handle classes that requires its own long explanation and wouldn't fit in this section, so I'll pass on this one. :coffee:


Added width prop in readme

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[ ] Feature
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes
[ ] No

If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:

[ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[ ] All tests are passing
[ ] New/updated tests are included

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:

I have already have same change in other pull request. Added 'width' and 'show-alpha' prop and some default-colors style fix.


Namespace lib directory and constants

Right now, all the Ruby files currently live in the lib directory. This can be a problem because the "require namespace" is flat, which can cause problems when there's overlap. It would be better to have them in lib/langchain
Similarly, all the constants are in the global namespace, which is shared. It'd be better to have them under Langchain.
So I propose:

move everything in lib into lib/langchain  (except lib/langchain.rb)
update everything to be under Langchain

I can make the change, but it is very prone to getting out of date and causing merge conflicts, so wanted to get feedback before attempting.

I mentioned this in https://github.com/andreibondarev/langchainrb/pull/45#issuecomment-1559594160 and agree. We'll need to namespace to avoid namespace collisions. Let's wait for @andreibondarev to weigh in. We have a quick meeting scheduled tomorrow where I'll bring this up if Andrei hasn't weighed in beforehand.
Thanks for raising this @technicalpickles!

@technicalpickles @rickychilcott Good call! I was trying to originally avoid the Langchain::Foo::Bar::Zip::Zap-madness but the risk of name conflicts is too large indeed.

@technicalpickles I went ahead and did these sets of changes. Just wanted to avoid dragging this out through the week! Thank you for your input here though!
(I tried to cherry-pick your commits to give you credit for the work but they were all in conflicts with the current version.)

@andreibondarev thanks! That makes a lot more sense than me trying to fix, and finding it out it's out of date 😅


Orcas Gen5 on Central US and Gen4 on West Europe are disabled.

Orcas Gen5 on Central US and Gen4 on West Europe are disabled.

@yukozh : Thanks for your contribution! The author, @jan-eng, has been notified to review your proposed change.

@yukozh Thanks for noticing the issue, and for being proactive to update the docs.
@ajlam @rachel-msft Is this an expected doc change, or an issue with the Azure portal somewhere where the generation 4 and 5 isn't showing up in certain regions. Didn't want to fix the docs if the portal needs adjustment.
Thanks, Jason

@yukozh, I already made this change last week, which has now been published:

https://docs.microsoft.com/en-us/azure/mysql/concepts-pricing-tiers
https://docs.microsoft.com/en-us/azure/postgresql/concepts-pricing-tiers

@JasonWHowell, this is related to portal.
#please-close


Hover Menu can't click and move after I rotate a screen

Like the title, I can not do how to be able to click and move it after I rotate the phone screen. I am looking forward to you can fix this, thank you very much!
Video description: https://www.youtube.com/watch?v=FeowYjSBv_M
(Sorry about my English)

There is another problem, I hope this feature you will add in the future

When in landscape mode, it seems that the content display is relatively small, when the keyboard is displayed, it will cover the content. So, I'm really looking forward to you doing the content display like Messenger, thank you very much.

1. Hover Menu

2. Messenger

The issue of orientation change has been fixed on the develop branch and will be available in the next release.
As for your request about a different layout in landscape, please file a separate ticket for that so it can be tracked.


Support mapping as input for most tensor ops

Should resolve #447.
Only needed it for rearrange and crop to work with my input/output layer set, though.

Very interesting typing problem...
Not sure if we can use overload on a decorator like this to overload the decorated function. Maybe it's more proper to use type variable here to link returned function's input and output type?

Have you ever seen a similar pattern elsewhere? It looks a bit clumsy for something that the user could do in a one-liner like {k: rearrange(v, ...) for k, v in layers.items()} - just complicated with the cue specs.

Maybe it's more proper to use type variable here to link returned function's input and output type?

I tried that at first, but you have functions (to_np) that are meant to change the type.

Think that last commit should fix the remaining issues (Unexpected keyword argument "data" for "interpolate"). That is a bit annoying, since type hints shouldn't require changes to function calls...

Oh, now I think I understand what you mean. support_dict always only takes a Callable that expects a TensorTypeVar as input, and the wrapper function inside is the overloaded one that returns a Callable whose output depends on whether the input is a mapping or not... I can let mypy do the painful work of calculating the return type... but I don't think we can actually annotate anything as Overload yet(?):
def supports_dict(func: Callable[Concatenate[TensorTypeVar, P], ReturnTensorTypeVar]):
    @overload
    def wrapper(
        data: Mapping[HashableTypeVar, TensorTypeVar], *args: P.args, **kwargs: P.kwargs
    ) -> dict[HashableTypeVar, ReturnTensorTypeVar]:
        ...

    @overload
    def wrapper(data: TensorTypeVar, *args: P.args, **kwargs: P.kwargs) -> ReturnTensorTypeVar:
        ...

    def wrapper(data, *args, **kwargs):

        if isinstance(data, Mapping):
            return {k: func(v, *args, **kwargs) for k, v in data.items()}
        else:
            return func(data, *args, **kwargs)

    reveal_type(wrapper)
    return wrapper

Revealed type is "Overload(def [HashableTypeVar <: typing.Hashable] (data: typing.Mapping[HashableTypeVar`-4, TensorTypeVar`-1], *P.args, **P.kwargs) -> builtins.dict[HashableTypeVar`-4, ReturnTensorTypeVar`-3], def (data: TensorTypeVar`-1, *P.args, **P.kwargs) -> ReturnTensorTypeVar`-3)"

How about just using TypeVar, and not using the decorator for to_np, to_torch, as_type?

How about just using TypeVar, and not using the decorator for to_np, to_torch, as_type?
Fine with me, but it doesn't seem to cause harm.

Bigger issue now is that the typecheckers are completely happy with convert.to_float32("Hello") and I don't understand why.

Does that happen if you remove the overlaod form the decorator?

Good idea. Still happens when I remove the overloads from wrapper. But works as expected if I additionally set the return type of supports_dict...
So likely mypy needs the return type of the outer function... which I don't know how to provide with overload

I didn't mean your new solution with wrapper, I mean the original decorator supports_dict -- fully annotate it with a type variable that can take on either Torch | NDArray or dict of them, and returns the same type

Perhaps we should just meet tomorrow to discuss?

@nkemnitz https://github.com/python/mypy/issues/16137 doesn't seem to be getting any attention, so I extended our mypy plugin to handle your supports_dict function. I think we're good to merge now. Sorry this took so long -- typing system bugs are always extra painful


Cannot register and deregister same stake credential

Description
This addresses issue #512

Checklist

[x] Commit sequence broadly makes sense and commits have useful messages
[x] Code is formatted according to CONTRIBUTING.md
[x] Self-reviewed the diff

I think all the changes in Ledger.Certs and Ledger.Epoch are just reverting formatting/cleanup that I've done over a month ago. Can you revert those two files?

I made (most of) those changes to try to ensure that the vertical vector formatting doesn't get confused by the line breaks.  (The insertion of the module with no parameters was a mistake.)  In any case, I'll revert the changes and check if the pdf looks okay.  But I may need to reformat the vectors as long single lines so they look okay in the pdf.

Ah, sorry, I spoke too soon. I wrote

Check the correctness of refunds in UTXO

in the issue, but what I meant was to check it there instead of in CERTS. So it's also necessary to remove those checks from CERTS, because that's what actually prevents us from registering and deregistering in the same transaction. I think the best way to do that is to remove deposits from CertEnv and all the other environments in the file and then follow all the errors.
Note that this also means that ValidCertDeposits needs to be slightly adjusted, so that all the removed checks are done there. In particular, it also needs to check deposit amounts, so it needs to have a Deposits argument rather than ℙ DepositPurpose. I don't think it should affect much though.

@UlfNorell do you have time to work on these changes?  I'd like to get back to finishing the Ledger pov PR and work on issue #578?
Btw, I merged your branch into the one on which this PR is based and fill the remaining holes and rewrote some of the proofs, so it now type-checks and passes the CI.  I suppose you will want to make a new branch off this one like you did last time (assuming you have time to work on this).  Thanks!!

@williamdemeo I'll try to make some progress today. If it's ok with you I'll just push to this branch. Last time was a major change to the way things were done, but now it's just continuing work on the PR.

Sure, just push to this branch. 👍
On Thu, Oct 10, 2024, 23:08 Ulf Norell @.***> wrote:

@williamdemeo https://github.com/williamdemeo I'll try to make some
progress today. If it's ok with you I'll just push to this branch. Last
time was a major change to the way things were done, but now it's just
continuing work on the PR.
—
Reply to this email directly, view it on GitHub
https://github.com/IntersectMBO/formal-ledger-specifications/pull/547#issuecomment-2406563867,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA25MJA6JMKSQ7WCI7QQIMDZ25MMBAVCNFSM6AAAAABMTKIIT6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMBWGU3DGOBWG4
.
You are receiving this because you were mentioned.Message ID:
@.***
com>

But I may need to reformat the vectors as long single lines so they look okay in the pdf.

Did something change that affects the vectors? I remember being able to strategically insert newlines in certain places without them affecting how the PDF renders. Also, if I look at CERTBASE and certState' in EPOCH, they look fine in the PDF right now, I don't see what needs to be fixed.

Ah, sorry, I didn't realize the script output could be manipulated by strategically inserting newlines.  I guess nothing needs to be fixed then. 👍🏼

@UlfNorell I just merged the latest master, so I suggest a git pull before doing more work on this.

I think I managed moving the checks from CERT to ValidCertDeposits. There were just one small question about whether the check in GOVCERT-deregdrep was wrong before.

@WhatisRT I finished fixing the pdf.  I think this PR is good to go now.


Unhighlight search when close minimap

Feature request.
After search in neo-minimap window (with /), the found item is highlighted. It left highlighted after it selected and minimap closed.
Please add option (or hard-coded) to hide the highlight. Something like this:
	vim.keymap.set("n", "<CR>", function()
		jump_and_zz(line_data, opts)
+		vim.cmd([[nohlsearch]], false)
		vim.api.nvim_win_close(win, true)

Hi @damanis , this feature is implemented at eba4b2e39ce3af584b646dfb7246031a37db880c, please update your plugin to receive the change.


How to scale cells?

Hey!
I'm not good at flow layout, please help me with my task.
I need something like that:

So I need to scale cells where user scroll the collectionView. How can I do that, through edit your layout?
Thanks!

Hi, you can use this solution. Make different size of cell if it selected. In UICollectionViewCell class make override var isSelected, to make different size of it.
override var isSelected: Bool {
    didSet {
        if isSelected {
            UIView.animate(withDuration: 0.2, delay: 0, options: .curveEaseOut, animations: {
                self.transform = CGAffineTransform(scaleX: 1, y: 1)
            }, completion: nil)
        } else {
            UIView.animate(withDuration: 0.2, delay: 0, options: .curveEaseOut, animations: {
                self.transform = CGAffineTransform(scaleX: 0.85, y: 0.85)
            }, completion: nil)
        }
    }

@deafjam
I am not sure, if your solution is correct. @Banck I believe he wanted the cells to scale from 0.85 to 1.0 when they are centered.


Fleet SDK Upgrade

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

Thank you for your contribution! We will review the pull request and get back to you soon.

@Ealianis, is this PR ready for review?

@Ealianis, is this PR ready for review?

@kairu-ms , yes this now ready for review.

@jsntcy , @kairu-ms , please can you help us get this merged ASAP?  Thank-you for your help.


is it supposed to prioritize the second image on my readme?

I have a few images on my readme, and on dotfyle it chooses shows the second image as the default image and it doesn't look very good due to the aspect ratio. Is there a way plugin owners can set which image they want to display for there plugin preivew...
ideally I would like it to show the first image on my readme

this is what dotfyle shows

this is what my readme looks like

This is actually a duplicate issue of #128

actually  I think what is happening is that it shows the first Media image related to the repo that was inserted
and its actual showing an image that is no longer being referenced on my readme, the image does however still exsits
I think this is simular to the issue #128 where old media that doesn't exist on the readme isn't being removed


Generate small database

closes #130

To what extend is the small database used?
For example, the Home Manager module still seems to use the full database:
https://github.com/nix-community/nix-index-database/blob/7e3246f6ad43b44bc1c16d580d7bf6467f971530/home-manager-module.nix#L13
Maybe we should provide an option, like programs.nix-index-database.full = lib.mkDefault false;, to use the full or small database globally.

only for comma.

@trueNAHO it is also exposed in the flake, so one can use it externally as well. Here's how I use it: https://github.com/gvolpe/nix-config/blob/47f0b31bae674d79e410cc2f6b938b2eb624b7bc/home/shared/programs.nix#L41
A bit of documentation on what my fork offers: https://github.com/gvolpe/nix-index/pull/1


Certificates still broken

Full discussion with patch. This only recently became an issue for me, I guess Dreamhost changed something. Possibly this was fixed but became unfixed.
https://code.google.com/archive/p/k9mail/issues/3976

Either it's a bug in BouncyCastle (unlikely) or it's a bug in Dreamhost's certificate deployment (plausible).
Have you tried contacting Dreamhost. What's the behaviour on other platforms?

The linked issue is outdated. If you want to report a new bug please fill out the issue template and include enough information to reproduce the issue, at the minimum the server settings.

The linked issue isn't outdated and clearly shows what's wrong, including a patch. Has that patch been merged into K9?
As the linked issue clearly shows there are nothing wrong with android or dreamhost.
The behavior on other platforms (thunderbird evolution etc).

The linked issue isn't outdated and clearly shows what's wrong, including a patch. Has that patch been merged into K9?
As the linked issue clearly shows there are nothing wrong with android or dreamhost.
The behavior on other platforms (thunderbird evolution etc) is that those asks me if I want to accept the self-signed certificate. As they should. And then when I have answered yes it works.

The patched code (#136) relies on tests that will presumably fail in the future when the correctly created base-64 encoded plain-text certification in the test expires. That's not particularly useful from a testing perspective.
It's not been merged because it needs re-assessing following more recent changes.
From S/O:

The discussion seems to imply that the error is caused by certificate chains that have a self-signed cert in the middle of the chain, and OpenSSL is more tolerant of this than Android's SSL implementation (which might be why Thunderbird works).

I'm not entirely sure what S/O mean by this. Either it's a valid certificate chain or it's not. A self-signed certificate can be fine only if it's the root cert. You can't pretend you're signed by a parent cert when you're not. That would be fraud.
Specifically, I worry that we might be falling into viewing the chain as a simple tree: https://www.blackhat.com/presentations/bh-usa-09/MARLINSPIKE/BHUSA09-Marlinspike-DefeatSSL-SLIDES.pdf
At minimum it would need to be re-based on-top of the "recent changes to certificate validation"
I might get time to look at it. But I still don't like the impossible to maintain unit test - there'd be no way to fix the test when it breaks - you'd have to start from scratch. It might be possible to mock the response and test it that way, rather than relying on actual certs.
I have an HTC M7. I don't know if this device can reproduce the issue or not?

I doubt that the fraud argument could be reasonable with dreamhost. Neglect possibly but that's different.
Other mail applications like thunderbird and evince warns about the problems and asks the user if they accept the certificate. Not very user friendly but it works.


Interaction type handling

Changes
Commands
The whole command and event structure changed, now to create a command you create a file in any folder inside the commands folder and follow the following structure
export default new SlashCommand({
    builder: new SlashCommandBuilder()
        .setName("test")
        .setDescription("this is a test command!"),

    async handler(): Promise<void> {
        await this.context.reply("This is a test!");
    }
});

If you want a user context menu command, simply create a new UserCommand instead.
Events
The events system works the same, simply create an event and export a new Event inside, Explicitly import import {Event} from "../modules/handlers/HandlerBuilders.js";, otherwise it won't work.
export default new Event({
    event: Events.ClientReady,

    handler(): void {
        logger.info("Client ready!");
    }
});

Warning!
You must invoke the correct type for each type of command, otherwise the event will ignore the command

Very good code, change the commit prefix to comply with the standard and that's all.


Homepage content lengths vary

The content on the homepage for the three sections are not of equal length. This causes the buttons to not line up on the same horizontal plane.

Fixed with the homepage alignment issue.


Updated some incomplete docs

#5739
Updated the following incomplete feature guides:

docs/guide/expanding.md
docs/guide/fuzzy-filtering.md
docs/guide/global-faceting.md
docs/guide/global-filtering.md
docs/guide/grouping.md

Content is based on my current understanding of the TanStack table library. It may not be full and comprehensive but at least include the essentials.

Impressive following through. Feel free to open up your GitHub sponsors if you live somewhere that allows that, especially if you plan to contribute more.

Impressive following through. Feel free to open up your GitHub sponsors if you live somewhere that allows that, especially if you plan to contribute more.

Thanks, Kevin. Sponsorships are not available in my region. I'm certainly eager to contribute more whenever possible.

It's ok to use AI to help write the docs, but we need to make sure that we don't leave any hallucinations in it.

It's ok to use AI to help write the docs, but we need to make sure that we don't leave any hallucinations in it.

Thank you for the review. I use AI for reviewing English language and it appears there was a confusion between the table.options and the table APIs, leading to some auto-corrections in the code. My Bad.


Table rebuilds when changing column type from VARCHAR(MAX)

SqlPackage or DacFx Version: <IP_ADDRESS>
.NET Framework (Windows-only) or .NET Core: .NET Core
Environment (local platform and source/target platforms): Microsoft SQL Server 2019 (RTM-CU12) (KB5004524) - 15.0.4153.1 (X64)  	Jul 19 2021 15:37:34  	Copyright (C) 2019 Microsoft Corporation 	Express Edition (64-bit) on Windows 10 Enterprise 10.0 <X64> (Build 22621: ) (Hypervisor)

Steps to Reproduce:

Create a table with a VARCHAR(MAX) column
Deploy the table
Change the column from VARCHAR(MAX) to VARCHAR(250)
Expected: sqlpackage emits an ALTER Column statement
Actual: sqlpackage emits a table rebuild

Change
CREATE TABLE TestTable(Col VARCHAR(MAX))

To
CREATE TABLE TestTable(Col VARCHAR(250))

Results in:
CREATE TABLE [dbo].[tmp_ms_xx_TestTable] (
    [Col] VARCHAR (250) NULL
);
IF EXISTS (SELECT TOP 1 1
           FROM   [dbo].[TestTable])
    BEGIN
        INSERT INTO [dbo].[tmp_ms_xx_TestTable] ([Col])
        SELECT [Col]
        FROM   [dbo].[TestTable];
    END
DROP TABLE [dbo].[TestTable];
EXECUTE sp_rename N'[dbo].[tmp_ms_xx_TestTable]', N'TestTable';

Why is this a problem? Table rebuilds are bad and often impossible to do on extremely large tables with high usage. For simple column table data changes, I expect sqlpackage to script simple ALTER statements.
Did this occur in prior versions?  If not - which version(s) did it work in?
(DacFx/SqlPackage/SSMS/Azure Data Studio)

Thanks for reaching out to us.
Yes, there are cases when table gets rebuild because of small, sometimes unnecessary changes. This is an existing problem and Sqlpackage team is working on it.
Meanwhile you can use a publish option, /p: AllowTableRecreation, which specifies whether to allow table recreation during deployment if required to perform the schema change. If you set it to False, then the deployment will halt and let you know which tables are getting recreated.


test: Run with host firewall

test-me-please

Coverage increased (+0.01%) to 37.197% when pulling 2572a2a1104f3f6ef2b5dc40007bebeede66c7de on pr/pchaigno/test-host-firewall into 0a173dc6387b56279be1e82731d994aacacdc3a1 on master.

test-me-please


[Windows] Unable to parse multiple java paths returned.

When locating the java installation in the code, in some cases multiple paths are returned. It appears that the below code is unable to parse the path in cases where multiple are returned.
https://github.com/grvlle/constellation_wallet/blob/aaf4f19a6a5487dfa49edb3d30cf124925447446/utils.go#L57

This is resolved by setting the env var PATH on windows to not include the system32 directory. Contemplating whether or not to account for multiple java paths or providing suggestions in the error message.

Fixed as per PR #51


Fn:If argument count not checked

cfn-lint version:
0.25.3
Description of issue.
cfn-lint does not currently appear to check the number of arguments passed to Fn::If, while there may be cases where this is not practical to check, eg where an existing list is passed.
I believe it would probably be possible to detect the following case, where a list of two elements is passed explicitly.  I would guess that this may also be true for various other intrinsic functions, but have not tested that.
Description: "Test - If argument count"
Parameters:
    TestParam:
        Type: String
        Default: Value

Conditions:
    HasParam: !Not [ !Equals [ !Ref TestParam, '' ] ]

Resources:
    SSM:
        Type: AWS::SSM::Parameter
        Condition: HasParam
        Properties:
            Type: String
            Description: "Just a test"
            Name: "/This-Is-Just-A-Test"
            Value: !If
              - HasParam
              - ValueIfTrue
```

Agreed.  Should be an easy thing to add.


Unrecognized Windows Sockets error

Since the author of issue #257 deleted his account and I'm with the same issue. I created other issue and the previous issue should be closed.
Note that all code is the same of this wiki
Setup:
PS C:\Users\petit> gradle -v
------------------------------------------------------------
Gradle 4.0
------------------------------------------------------------

Build time:   2017-06-14 15:11:08 UTC
Revision:     316546a5fcb4e2dfe1d6aa0b73a4e09e8cecb5a5

Groovy:       2.4.11
Ant:          Apache Ant(TM) version 1.9.6 compiled on June 29 2015
JVM:          1.8.0_131 (Oracle Corporation 25.131-b11)
OS:           Windows 10 10.0 amd64

build.gradle:
apply plugin: 'java'
apply plugin: 'application'

repositories {
    jcenter()
}

dependencies {
    compile "cloud.orbit:orbit-runtime:1.1.0"
}

mainClassName = "Main"

Output:
PS C:\Users\petit\Documents\Code\orbit-test> ./gradlew run

SLF4J: Failed to load class "org.slf4j.impl.StaticLoggerBinder".
SLF4J: Defaulting to no-operation (NOP) logger implementation
SLF4J: See http://www.slf4j.org/codes.html#StaticLoggerBinder for further details.
Jul 05, 2017 7:43:12 PM org.infinispan.remoting.transport.jgroups.JGroupsTransport start
INFO: ISPN000078: Starting JGroups channel orbit-helloworld-cluster
Jul 05, 2017 7:43:12 PM org.jgroups.stack.DiagnosticsHandler bindToInterfaces
**WARNING: failed to join /<IP_ADDRESS>:7500 on net0: java.net.SocketException: Unrecognized Windows Sockets error: 0: no I
net4Address associated with interface**

-------------------------------------------------------------------
GMS: address=orbit-helloworld-cluster, cluster=orbit-helloworld-cluster, physical address=<IP_ADDRESS>:57133
-------------------------------------------------------------------
Jul 05, 2017 7:43:14 PM org.infinispan.remoting.transport.jgroups.JGroupsTransport viewAccepted
INFO: ISPN000094: Received new cluster view for channel orbit-helloworld-cluster: [orbit-helloworld-cluster|0] (1) [orbi
t-helloworld-cluster]
Jul 05, 2017 7:43:14 PM org.infinispan.remoting.transport.jgroups.JGroupsTransport startJGroupsChannelIfNeeded
INFO: ISPN000079: Channel orbit-helloworld-cluster local address is orbit-helloworld-cluster, physical addresses are [19
<IP_ADDRESS>:57133]
Jul 05, 2017 7:43:14 PM org.infinispan.factories.GlobalComponentRegistry start
INFO: ISPN000128: Infinispan version: Infinispan 'Mahou' 8.1.4.Final
Here: Welcome to orbit
You said: 'Welcome to orbit', I say: Hello from<PHONE_NUMBER> !
Jul 05, 2017 7:43:15 PM org.infinispan.remoting.transport.jgroups.JGroupsTransport stop
INFO: ISPN000080: Disconnecting JGroups channel orbit-helloworld-cluster
Jul 05, 2017 7:43:15 PM org.infinispan.remoting.transport.jgroups.JGroupsTransport stop
INFO: ISPN000082: Stopping the RpcDispatcher for channel orbit-helloworld-cluster

BUILD SUCCESSFUL in 5s
2 actionable tasks: 1 executed, 1 up-to-date

From your log, it looks like it actually ran correctly so the warning is not affecting the correct functioning of the application.
Here: Welcome to orbit
You said: 'Welcome to orbit', I say: Hello from<PHONE_NUMBER> !

I am not 100% sure what would cause the warning you're seeing and unfortunately I don't have access to a windows machine right now. You may need to set java.net.preferIPv4Stack=true or something similar.
 If you include an slf4j compatible logging library such as sl4j-simple you will see the Orbit output logs as well.
Please let me know if you have any other questions/issues and I'm happy to help.

I believe @johnou develops on Windows and uses the JGroups/Infinispan module so he may have some additional insight.

System.setProperty("java.net.preferIPv4Stack" , "true");
Solved the issue.
But I don't believe this is the best method.

@lai0n if your machine has multiple interfaces your jgroups configuration should bind to a specific address or interface. Here are some examples below, the Jgroups documentation contains more snippets for use cases like Docker.
<TCP bind_addr="match-interface:eth0"
  ...
 />

<TCP bind_addr="${jgroups.bind_addr:<IP_ADDRESS>}" bind_port="7800"
  ...
 />

@johnou I'm not really a jgroups expert
I tried
<config xmlns="urn:org:jgroups"
        xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
        xsi:schemaLocation="urn:org:jgroups http://www.jgroups.org/schema/jgroups.xsd">
    <TCP bind_addr="match-interface:eth0"
         bind_port="7800" />
    <TCPPING port_range="1"/>
    <VERIFY_SUSPECT timeout="1500"  />
    <pbcast.NAKACK2 use_mcast_xmit="false"
                    discard_delivered_msgs="true"/>
    <UNICAST3/>
    <pbcast.STABLE stability_delay="1000" desired_avg_gossip="50000"
                   max_bytes="400000"/>
    <pbcast.GMS print_local_addr="true" join_timeout="2000"
                view_bundling="true"/>
</config>

but no success. Can you give me a example?

@lai0n it would be helpful to know more about your environment, do you want a simple local host configuration? Multiple machines? Static ips / host names? Your jgroups tcpping tag is missing initial_hosts. You might want to keep an eye on https://github.com/orbit/orbit/pull/262 I am on holiday at the moment so apologies for the satellite delay.

@johnou a simple local host configuration, one machine, static IP

Try something like this..
<config xmlns="urn:org:jgroups"
        xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
        xsi:schemaLocation="urn:org:jgroups http://www.jgroups.org/schema/JGroups-3.6.xsd">
   <TCP bind_addr="${jgroups.tcp.address:<IP_ADDRESS>}"
        bind_port="${jgroups.tcp.port:7800}"
        enable_diagnostics="false"
        thread_naming_pattern="pl"
        send_buf_size="640k"
        sock_conn_timeout="300"
        bundler_type="sender-sends-with-timer"

        thread_pool.min_threads="${jgroups.thread_pool.min_threads:2}"
        thread_pool.max_threads="${jgroups.thread_pool.max_threads:30}"
        thread_pool.keep_alive_time="60000"
        thread_pool.queue_enabled="false"

        internal_thread_pool.min_threads="${jgroups.internal_thread_pool.min_threads:5}"
        internal_thread_pool.max_threads="${jgroups.internal_thread_pool.max_threads:20}"
        internal_thread_pool.keep_alive_time="60000"
        internal_thread_pool.queue_enabled="true"
        internal_thread_pool.queue_max_size="500"

        oob_thread_pool.min_threads="${jgroups.oob_thread_pool.min_threads:20}"
        oob_thread_pool.max_threads="${jgroups.oob_thread_pool.max_threads:200}"
        oob_thread_pool.keep_alive_time="60000"
        oob_thread_pool.queue_enabled="false"
   />
    <TCPPING async_discovery="true"
        initial_hosts="${jgroups.tcpping.initial_hosts:<IP_ADDRESS>[7800]}"
             port_range="10"
             send_cache_on_join="true"
             return_entire_cache="true" />
   <MERGE3 min_interval="10000"
           max_interval="30000"
   />
   <FD_SOCK />
   <FD_ALL timeout="60000"
           interval="15000"
           timeout_check_interval="5000"
   />
   <VERIFY_SUSPECT timeout="5000" />
   <pbcast.NAKACK2 use_mcast_xmit="false"
                   xmit_interval="1000"
                   xmit_table_num_rows="50"
                   xmit_table_msgs_per_row="1024"
                   xmit_table_max_compaction_time="30000"
                   max_msg_batch_size="100"
                   resend_last_seqno="true"
   />
   <UNICAST3 xmit_interval="500"
             xmit_table_num_rows="50"
             xmit_table_msgs_per_row="1024"
             xmit_table_max_compaction_time="30000"
             max_msg_batch_size="100"
             conn_expiry_timeout="0"
   />
   <pbcast.STABLE stability_delay="500"
                  desired_avg_gossip="5000"
                  max_bytes="1M"
   />
   <pbcast.GMS print_local_addr="false"
               join_timeout="${jgroups.join_timeout:5000}"
   />
   <MFC max_credits="2m"
        min_threshold="0.40"
   />
   <FRAG2 />
</config>

Solved.
Log:
Jul 11, 2017 9:26:48 PM org.infinispan.remoting.transport.jgroups.JGroupsTransport start
INFO: ISPN000078: Starting JGroups channel orbit-helloworld-cluster
Jul 11, 2017 9:26:54 PM org.infinispan.remoting.transport.jgroups.JGroupsTransport viewAccepted
INFO: ISPN000094: Received new cluster view for channel orbit-helloworld-cluster: [orbit-helloworld-cluster|0] (1) [orbi
t-helloworld-cluster]
Jul 11, 2017 9:26:54 PM org.infinispan.remoting.transport.jgroups.JGroupsTransport startJGroupsChannelIfNeeded
INFO: ISPN000079: Channel orbit-helloworld-cluster local address is orbit-helloworld-cluster, physical addresses are [12
<IP_ADDRESS>:7800]
Jul 11, 2017 9:26:54 PM org.infinispan.factories.GlobalComponentRegistry start
INFO: ISPN000128: Infinispan version: Infinispan 'Mahou' 8.1.4.Final
Here: Welcome to orbit
You said: 'Welcome to orbit', I say: Hello from 691657013 !
Jul 11, 2017 9:26:55 PM org.infinispan.remoting.transport.jgroups.JGroupsTransport stop
INFO: ISPN000080: Disconnecting JGroups channel orbit-helloworld-cluster
Jul 11, 2017 9:26:55 PM org.infinispan.remoting.transport.jgroups.JGroupsTransport stop
INFO: ISPN000082: Stopping the RpcDispatcher for channel orbit-helloworld-cluster

Glad to hear you resolved the issue!


[284] Unclear Feedback Received

Title:
Unclear Feedback Received
Description:
nice
Summary:
The customer provided a single word ('nice') as feedback, which is unclear and lacks context. Follow up with the customer to gather more details or clarify their intent.
Info
Reported by: Guest<EMAIL_ADDRESS>- Open in Gleap
Location: SG
Type: 🚨 BUG
🔗 Open share link
🌍 Open ticket in Gleap
Metadata
browserName: Chrome(<IP_ADDRESS>)
userAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
browser: Chrome
systemName: Windows
sessionDuration: 42
devicePixelRatio: 1.5
screenWidth: 1707
screenHeight: 1067
innerWidth: 1181
innerHeight: 626
currentUrl: https://docs.near.org/build/smart-contracts/quickstart
language: en-SG
sdkVersion: 13.7.3
sdkType: javascript
environment: prod

Internal note 📝: The customer provided a single word ('nice') as feedback, which is unclear and lacks context. Follow up with the customer to gather more details or clarify their intent.


Update logo

I'm submitting a ...
[ x] bug report
[ ] feature request
[ ] question about the decisions made in the repository
[ ] question about how to use this project
Summary
I would like to update my token logo with the following;
PR #6626
Other information (e.g. detailed explanation, stack traces, related issues, suggestions how to fix, links for us to have context, eg. StackOverflow, personal fork, etc.)

merged


It is impossible to add a new tag if there is already a tag in the options with the same root or a part that matches the text of the new tag in taglist field

Reproduction
https://formkit.link/2146cfa3c26694a88166e31aa231f8ee
Describe the bug
It is impossible to add a new tag if there is already a tag in the options with the same root or a part that matches the text of the new tag. For example in the options list, I already have the 'networking' option, but I want to add the 'network', in the current implementation it's not possible.
Environment
• OS: Win 11
• Browser chrome
• Version 1.6.5

This would probably be best by solved by adding a "Add new value" listitem to the top of the listbox. That way users can click that to add "network" in your case without having to prevent previously selected list items from being shown during search.

@sashamilenkovic  Could you please give a hint on the solution?


Supermarket incorrectly redirects Berkshelf clients to http://servername:443

Using Supermarket 1.5.0 alpha (via Omnibus),  knife supermarket download works just fine but berks install fails.
With a Berksfile that looks like:
source "https://default-centos-66.vagrantup.com/"
cookbook 'audit'

It successfully contacts the /universe endpoint,  but then talks HTTP to port 443 to get the /api/v1/cookbooks endpoint:
2015/01/14 19:31:47 [info] 23679#0: *3 client sent plain HTTP request to HTTPS port while reading client request headers, client: <IP_ADDRESS>, server: default-centos-66.vagrantup.com, request: "GET /api/v1/cookbooks/audit/versions/0_1_3 HTTP/1.1", host: "default-centos-66.vagrantup.com:443"

Berkshelf debug output shows:
I, [2015-01-14T11:31:46.913372 #15416]  INFO -- : Installing audit (0.1.3)
D, [2015-01-14T11:31:46.913633 #15416] DEBUG -- :   Downloading audit (0.1.3) from https://default-centos-66.vagrantup.com/
D, [2015-01-14T11:31:46.913817 #15416] DEBUG -- :     => #<Berkshelf::APIClient::RemoteCookbook:0x007f9b60a51290 @name="audit", @version="0.1.3", @attributes=#<Hashie::Mash dependencies=#<Hashie::Mash> download_url="http://default-centos-66.vagrantup.com:443/api/v1/cookbooks/audit/versions/0.1.3/download" location_path="http://default-centos-66.vagrantup.com:443/api/v1" location_type="opscode">>
Installing audit (0.1.3) from https://default-centos-66.vagrantup.com/ ([opscode] http://default-centos-66.vagrantup.com:443/api/v1)
D, [2015-01-14T11:31:46.916601 #15416] DEBUG -- : ==> Chef response did not contain a JSON body
E, [2015-01-14T11:31:46.916778 #15416] ERROR -- : Actor crashed!
Berkshelf::CommunitySiteError: An unexpected error occurred retrieving 'audit' (0.1.3) from the cookbook site at ''.
	/opt/chefdk/embedded/apps/berkshelf/lib/berkshelf/community_rest.rb:127:in `find'
	/opt/chefdk/embedded/apps/berkshelf/lib/berkshelf/community_rest.rb:103:in `download'
	/opt/chefdk/embedded/apps/berkshelf/lib/berkshelf/downloader.rb:62:in `try_download'
	/opt/chefdk/embedded/apps/berkshelf/lib/berkshelf/downloader.rb:36:in `block in download'
	/opt/chefdk/embedded/apps/berkshelf/lib/berkshelf/downloader.rb:35:in `each'
	/opt/chefdk/embedded/apps/berkshelf/lib/berkshelf/downloader.rb:35:in `download'
	/opt/chefdk/embedded/apps/berkshelf/lib/berkshelf/installer.rb:105:in `install'
	/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/celluloid-0.16.0/lib/celluloid/calls.rb:26:in `public_send'
	/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/celluloid-0.16.0/lib/celluloid/calls.rb:26:in `dispatch'

Confirmed on my install, universe.json is being generated with http:...:443 URLs.  I'll see if I can figure out a fix since I'm hoping to launch the internal site by Friday...

Couple things seem to be going on here.
For starters, Universe/UniverseCache key off of the incoming protocol and use it to build and cache the universe.json response.  Public Supermarket seems to do the right thing (http visitors get the version that has http URLs, https visitors get the version with https URLs).  Privatized behind-a-load-balancer Supermarkets, which do all their talking in HTTP but are supposed to serve nothing-but-HTTPS URLs, will always return the 'http' universe.
Other URLs in the API also appear broken.  It looks like they're pulling from the Rails environment inherited by ActiveController itself.  I'm not sure if there's a more proper way to do it, but adding a stanza to config/application.rb pointing to action_controller instead of action_mailer and setting the protocol/host/port info seems to have fixed the URLs returned by the API.
Removing UniverseCache ( ... yeah, not best practice!) from the call to /universe and forcing the protocol to be generated as 'https' seems to be generating proper universe URLs.
I've also brutalized in an /api/v1/users endpoint just so I can occasionally query it and see who's signed up.  I was hoping to launch with a pretty user directory but for some reason I can't get UsersController#index to render (even though it's routing ... maybe I should stick to Sinatra apps? :D ).
Things look good for me, though I do have one user complaining that he's somehow getting an http-to-https redirect error from Berkshelf when he adds our internal Supermarket to his source list.  It's a big head scratcher for me because I know the app isn't serving any more HTTP URLs.


Allow mailer methods to receive a block

Since the internal method devise_mail takes a block and ActionMailer's mail takes a block, I thought that it made sense for the devise mailer methods to also receive a block to make format overrides a bit easier.

ready


Adding additional interface for Object Detection

Hi @MinuraPunchihewa!
First off I  want to thank you for your contributions. I've been having a blast playing with the functionality.
I added this for something I'm testing but I'm hoping you / others can benefit.
Change: Adding additional interface for Object Detection in addition to error catching API calls to return JSON when no image / object labels get returned

Using https://github.com/MichaelLantz/mindsdb/tree/ml_integration/hf_inference_api_playground these are some tests I ran successfully:
CREATE MODEL mindsdb.hf_object_detection PREDICT detection USING   task = 'object-detection',   engine = 'hf_api_engine',   api_key = 'hf_...',   column = 'image'
followed by:
SELECT image, detection FROM mindsdb.hf_object_detection WHERE image = 'https://raw.githubusercontent.com/MinuraPunchihewa/hugging-py-face/main/tests/resources/dogs.jpeg';

@MichaelLantz A couple of things,

I made a few changes recently, so please pull from the main branch, if you don't mind.
I have created an issue for the task that you are working on and I can assign that to you if you like. If you want me to assign it, could you please comment on the issue,
https://github.com/MinuraPunchihewa/hugging-py-face/issues/11


Use tee with sudo

When trying to redirect to a file, the script will complain about permissions
because the sudo privileges don't apply to the redirect. To ensure files get
updated, use tee running with sudo. This way elevated will be used to update the
file.

Thanks dear!
I wonder, such privilege issues would occur at all places involving stream redirection, like Line Number 96. If yes, then please provide a fix.

Hi @himanshub16,
Looking at the code, I think the entire line 96 should be removed. The question posed to the user is to modify not replace. This line could cause a user major headache if the environments file is replaced. I would suggest completely removing the conditional else block... Or at the very minimum backup the environments file.

Ok. Thanks @yasn77
I will look over it.


Resolve error when using fill and gradient tool with node graph frame

Resolve the type error showing when the user tries to use the gradient or fill tool on the node graph frame layer.

"node graph frame" in this case means a bitmap output?


How to animate my website

Hi!
I have been using wowchemy academic theme and built my website with some initial help tips from @gcushen and @rodrigoalcarazdelaosa. I want to animate menu icons (e.g. 'recent and upcoming') on my home page. Using the support document, I feel such changes in home page requires editing themes/academic/layouts/partials/ to layouts/partials. Unfortunately, I am unable to locate the 'themes/academic' folder in my repository.
Need expert advice on this.

If you need source files you have to get them from this repo

Thank you @rodrigoalcarazdelaosa! Sorry, I am a beginner. I will appreciate specific location folders/files in the suggested repo that needs to be copied/edited into the specific location of my repo. Further, I need to know how to animate. An example will be helpful.
Thanks in advance.

Closing this as it's not a bug. For animation ideas check out similar discussions on https://discourse.gohugo.io/ etc. and add them to your custom CSS https://wowchemy.com/docs/customization/#customize-style-css


Factor out a base class of ManifestParser

Create a Parser base class that holds parser functionality not specific
to the build manifest file format.  This will allow it to be re-used for
other parsers later.

This looks find, but there's no need to merge this without any alternative parsers. Can you include this commit in the PR you're working towards?

Sure, but the work is pretty extensive and this prepares it with incremental PRs.

That's not a problem, as we can look at the individual commits in a PR.


connect weather API

connect and test weatherstack API

Used Tomorrow.io API instead, connected and accepting Zipcode input. Returning correct data.


Add chat via gitter or slack

I've found that gitter and slack via slackin are great resources for helping people with hard to describe issues (e.g. almost all the troubleshooting requests in the issues queue). Implementing this for protractor would make it easier to shift issues to SO or Gitter/Slack and close them immediately; it would also hopefully reduce the clutter on issue threads while we are trying to get some basic information on the issue.
I'd be happy to help moderate the channel and assist someone with contributor access in setting it up.

It looks like angular.js is using Gitter, so it would be great to follow precedent there and set one up for Protractor as well. Let me look into it, check on access stuff and get back to you.

+1 for gitter. If you have a github account (we participate in github, so yeah...) it is as easy as 'log in using your github account'. Slack needs you to register for every channel you want to enter. Also gitter is more user friendly, just my opinion.

OK, made it so: https://gitter.im/angular/protractor
Go wild!

I'll be adding links from the FAQ and CONTRIBUTORS shortly.

Done with 546f16335714309709566b828f46a60eb9fe6ee3
Thanks for the suggestion!


Doh! Forgot the viewport tag.  The site should now work on phones.

Coverage remained the same at 83.87% when pulling cebb6837bb5492c7c51e432705485332f6b70f6d on raysuelzer:ror4 into 39a7223921191f4ab2e6d06d90b339e2565fa54d on MushroomObserver:ror4.


Remove support for machine health check flags

Issue #, if available:
Description of changes:
Remove support for machine health check flags
Testing (if applicable):
Documentation added/planned (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm
/approve

/cherrypick release-0.19

/lgtm
/approve


Leshan request timeout of 5 s too short for large files / representations

Report from interop event: For some LWM2M Clients Leshan's default request timeout is too short, in particular when sending large files (e.g., for firmware upgrade).
A way to configure the timeout or a good description where to change it would help.

We clearly need a way to configure timeout.
For now, this is just not possible.
I think this should be something we must address at short term.

hi @sbernard31
i noticed that, for bootstrapped simulator with security communication, 10 min after simulator sent the reg update, the request sent by the server cannot reach the simulator anymore. hence, i got coap timeout error. (note: lifetime is set to 15 min and request from server can still reach right after the next reg update.)
however, for non-bootstrapped simulator with no security communication, this won't happen.
i think this is related to DTLS.
i feel the data in NAT is removed after a while after reg up.
Do you know how this can be fixed ?
Thanks :)

This seems not linked at all to this issue ... So I would prefer if we talk about that in a new issue, so I created #585.

Can you point us in the right direction on how to fix this?  Either a clumsy fix ( where can we just increase the timeout to be longer ) or would it be simple to reset the timeout on each packet received?

In ClientServlet of leshan-server-demo, there is call of severals server.send(...).
This is where you can choose the timeout.
Look at LwM2mServer javadoc to have more details about the API.
You can also look at https://github.com/eclipse/leshan/wiki/Request-Timeout if you want to better understand timeout.

After only ~5 years, a PR for this issue : #808 ! :confetti_ball:

I think this should be something we must address at short term.

Never trust me ... :confounded:


[5.1] Exists property is set to false on soft deleted eloquent model

If you soft delete a model the property exists is set to false which may lead to problems.
Try the following code as a test:
$model->delete();
$model->forceDelete();

Surprisingly the model won't be hard deleted because the first delete call will set the exists property to false thus the second call of forceDelete won't perform anything and the model remains in the DB.
Soft deleting a model should prevent setting the exists property to false because otherwise the model object is in an inconsistent state and any logic relying on this property to check if a model was deleted will fail.

Wow, I really should call it a day - even the automatic code formatting failed on me thus the trailing spaces.
Happy weekend everybody!

Could you rebase to resolve the merge conflcits please.

Could you rebase to resolve the merge conflcits please.
Done.

@GrahamCampbell I found some other bugs related to the model deletion and the exists property:

the return value documentation of some deletion methods is wrong
the return value of some deletion methods is inconsistent (forceDelete should have the same return value types as delete)
the deleted event is fired even if a deletion fails
deleting a model that doesn't exist anymore in the DB doesn't fail and fires the deleted event (see above)
calling modification methods (update ...) on a model object for which a deletion failed will operate on the model as if it doesn't exist yet / anymore

Should I add fixes for those to this pull request?

The return type of forceDelete has already been discussed. #9624

The return type of forceDelete has already been discussed. #9624

Thanks for the cross reference! Even though I would discuss this from a different point of view.
The forceDelete method (just as the delete method) may fail for whatever reason and without throwing an exception. As a developer you have to know whether this implied 'strong' operation has failed or not, you need the model to be deleted and you need to know if the deletion was successful or not and if you should take any additional action.
You could of course make a separate call after each forceDelete to check if the model still exists but this would be really bad design. Especially since the forceDelete does / should always know if the operation was successful or not.
That being said you may still return void in forceDelete if that's a choice by design but the meaning of this should be very clear and documented. I.e. a void return would mean 'I don't know what happened, check for yourself' while a bool value would indicate very clearly the result of the operation on which the developer can rely. 'true' means the deletion operation was successful while 'false' means the operation did nothing due to some kind of error which should be handled by the caller.

Btw. there's no such thing as a 'real' void return in PHP. Not calling return is the same as returning NULL

Yes, but not the same as void. Returning a string is allowed if the return annotation says void for example. void just means, don't use the return value, for whatever reason.

void just means, don't use the return value, for whatever reason.

True, that's at least the definition of phpdoc.

void, this type is commonly only used when defining the return type of a method or function. The basic definition is that the element indicated with this type does not contain a value and the user should not rely on any retrieved value.

I don't wanna get offtopic but I guess the point is that you shouldn't return anything if you define the return value as void. Defining a return value as undefined and telling the users to not use it but on the other hand actually returning meaningful and well defined data is unexpected and inconsistent. Why return something if it can't and shouldn't be used anyways? That's crying for wrong usages, misunderstandings and unwanted side effects.
Which brings me back to the mentioned methods delete and forceDelete. The delete method of the Eloquent Model class should currently explicitly return null instead of omitting the return statement (which would semantically be a @return void) even though PHP internally returns null.
Question regarding this: what's the current definition and interpretation of the possible return values of the delete method? true for success, false for failure and null if the method didn't do anything?
forceDelete on the other hand defines the return value as void but actually returns the result of the delete call (which isn't void). But if you look at the forceDelete implementation of the SoftDeletes trait then you're gonna see that this implementation doesn't use a return statement which would comply to the phpdoc definition.
My suggestion regarding this particular point for a clear and consistent definition and implementation (note that I haven't added those to the pull request yet):
both delete and forceDelete should have the exact same return type definition and should both return true on success and false on failure (tbd if null remains for NOPs). Which would give the caller the opportunity to actually know that sth. went wrong during the deletion process.
Please note that the other previously listed bugs (3, 4 and 5) not necessarily have anything to do with this. Those remain no matter how the definition of the return values look like.
Sorry for the TLDR block.

Should this be considered a breaking change?

Yeh, this probably needs to go to 5.2.

The pull request in its current form is not a breaking change but rather a bugfix. I wouldn't recommend to postpone it.
Changing the return value behavior of forceDelete to match the one of delete shouldn't be either because it was meant to be void and would become bool or null just like the one of delete. I.e. sth. is added that couldn't be and shouldn't have been used before. forceDelete of the Model class currently returns the result of delete anyways, only soft-deleted models would be affected.
Returning false upon deletion failure and not firing deleted however could be a breaking change if developers did sth. strange like expect the delete to return true on failure or expect this event to fire no matter of the result of the deletion. Which seems odd to me but there might be a slight possibility.
From my point of view the current deletion behavior (maybe except the deleted event firing) should be fixed since 5.1 is meant to be a LTS version which might be used for several years. But that's of course a decision the software architect has to make ;-)
How about I open a new separate pull request later for the change of the return value thing so you're actually able to see what would change in code?

Ping @HolgerW1.

@HolgerW1 Just to understand... Why you closed this PR?

Why you closed this PR?

Because we're not accepting it on 5.1. We're waiting on a PR to 5.2.

@GrahamCampbell It isn't considered like a bugfix? Users on 5.1 will be ever affected by this bug on LTS? I don't think that it is a breaking-change, but an expected behaviour when you call forceDelete() (you should literally force delete the row, if it don't delete, it fails).
Imagine that you have a system with Administrators and Moderators. If some Moderator delete something, they will do a soft delete. Administrators can check if it will be hard deleted or restored (returning to Moderators) or just keep as soft delete.
To hard delete in this case, you should use a workaround (nothing good) by restoring first and then forceDeleting.
Ok, it's just a example. In this case I should consider create another system like "tag as invalid" to Administrator check if before delete. But is possible you expect that by terms.

Just to understand... Why you closed this PR?

As Graham said, they consider it a breaking change and it seems there was no chance for it to get it in for 5.1. Furthermore even though this PR would fix this specific issue the whole delete and forceDelete functions could need some major refactoring. I.e. it makes more sense to not just fix this specific issue but rather adapt the whole deletion functionality.

We're waiting on a PR to 5.2.

Sorry, but I currently don't have the time to create a PR for the master branch. Plus I think it would make more sense if someone would review the current deletion implementations. Especially the inconsistent return behavior and the setting of the exists property.

This is fixed in 5.5.


JsConvertValueToString failed if the input value is taggedInt

We are incorrectly treating taggedInt as JavascriptNumber.
Same as OS 6980708
Details from the bug above:
I was trying to convert something to a string in an external function.  Instead of calling JsConvertValueToString, I tried to get the "toString" method and call it.
MyExternalFunction(..., JsValueRef *pArguments, ...)
{
    ...
    JsGetProperty(pArguments[...], propIdToString, &jsToString);
    ...
}

fails with JsErrorArgumentNotObject when I call it on TaggedInt.
I can do this for string conversion:
host.externalFunction(<some object>)
I cannot do this:
host.externalFunction(42)

Please add a test case here.

@Yongqu What is milestone ao?

should be 1.2

Added additional details from the bug linked above. I believe they are the same bug. I am investigating.

@dilijev Any update on this bug? Should this be 1.2?

I'm trying to figure out whether this will have any impact on 1.2 or whether this should be a future work issue. I believe this is specific to ch.exe and maybe the Jsrt API (which IIRC is not necessarily a priority for the 1.2 milestone).
/cc @liminzhu @Yongqu

@yongqu confirmed this Jsrt API is used in our hosts and this issue affects milestone 1.2

Delaying work on this pending additional details from @Yongqu

Assigning this back to @Yongqu while we collect more info.

We should allow value ref for APIs like JsGetProperty, in the effect that JsGetProperty(_valueRef_for_42) should work by checking up the prototype chain of number. Doesn't need to be in release 1.2 though


Predator fix

Fixes issue #48 by adding back the texture tags. This only works in Gazebo 9. Issue #97 has been created to address Gazebo 11 compatibility. This also does not fix inertia values (see issue #98). All it fixes is the texture.
To test:
roslaunch uuv_dave uuv_predator.launch

Verify that the predator has the correct (black texture) and is not the light gray material that appears when the texture isn't loading. That is, it should look like this:

Not this:

For thoroughness, one might also check that the controls work:
Right Joystick

Key
Action

Down
Shoulder joint lowers

Up
Shoulder joint raises

Left
Shoulder joint swivel left

Right
Shoulder joint swivel right

Left Joystick

Key
Action

Down
Elbow joint lowers

Up
Elbow joint raises

Left
Wrist joint swivel left

Right
Wrist joint swivel right

4-way keypad

Key
Action

Down
Wrist joint lowers

Up
Wrist joint raises

XYAB Buttons

Key
Action

B
Claw opens

X
Claw closes

@crvogt Have you had a chance to look at this?

@crvogt Have you had a chance to look at this?

Just writing the review!


feat: only re-highlight changed ranges

Requires https://github.com/euclidianAce/ltreesitter/pull/12.

actually it seems like this introduces odd artifacts:

Have a fix, will push later

Resolved the issue with the artefacts, and rebased onto master.


Number of Parallel Executions

Hi,
Wanted to understand what will be the server/VM resource that will be required to run 3 parallel windows container?.
Thanks,
Arun

Approx 2 cpu 4Gb  per container.
вт, 1 июн. 2021 г., 19:00 arunganesh-a @.***>:

Hi,
Wanted to understand what will be the server/VM resource that will be
required to run 3 parallel windows container?.
Thanks,
Arun
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/aerokube/windows-images/issues/39, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKY23KUNXSN5ORFBF6NWLTTQT72ZANCNFSM4547EJWA
.

Thanks, I can see while building the image we are specifying 2GB ram.. should I change it to 4 GB?

You can tune memory and cpu here:
https://github.com/aerokube/windows-images/blob/master/image/entrypoint.sh#L21
сб, 5 июн. 2021 г., 21:32 ArunGanesh88 @.***>:

Thanks, I can see while building the image we are specifying 2GB ram..
should I change it to 4 GB?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/aerokube/windows-images/issues/39#issuecomment-855278379,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKY23KNF36CVKGHGJW5YYTTRJUSZANCNFSM4547EJWA
.

Thanks.. now I am able to get the container up in less than a min around 40 seconds.. but still the automation script execution speed is slow... is there anything else that I should do to improve the speed?. I am using ubuntu20.04 docker 20.10.

If you are running on vm there is no way to ged it faster, on bare metal
server container starts in 10-15 sec...
сб, 12 июн. 2021 г., 14:29 arunganesh-a @.***>:

Thanks.. now I am able to get the container up in less than a min around
40 seconds.. but still the automation script execution speed is slow... is
there anything else that I should do to improve the speed?. I am using
ubuntu20.04 docker 20.10.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/aerokube/windows-images/issues/39#issuecomment-860040386,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKY23LNGQ5HBPGI6AZ6M4TTSNAIHANCNFSM4547EJWA
.


Win7: CPU usage at 100% when opening huge files

VSCode Version: 1.5.3 (1.5.3)
OS Version: Win7 64 Enterprise

Steps to Reproduce:

I opened a file rollup.js has transpiled just now. The file was about 16000+ LoC. Syntax coloring was active.
Then VScode became very unresponsive. It took me 2 minutes to close the file. I don't saw the typical 'keep waiting or close' message.
In Process Explorer saw that a VScode process consumed 100% of a single CPU core.

@fibric Can you please share with us the contents of the file that produces this problem or perhaps a very similar file that also produces the same freeze?

Sorry for the late answer.
I experimented with some Rollup.js plugins, and I wanted to know why some native modules such as fs, http, etc. have gotten baked into the bundle even though I excluded them.
On opening the 782KB file, VScode became super unresponsive.
bundle.cjs.zip
I tried the newest Insiders build today, and this time, everything hangs except the menu bar.

It has become worse. Maybe VScode should warn on open big files until VScode can handle big files.

I can open that file without any problems in the latest insiders release. Let's try to figure out what makes your setup different.

Do you have any extensions installed? If so, can you reproduce using --disable-extensions?
Do you open the file alone in VS Code, or as part of opening a folder?
Does VS Code still work, but just very unresponsive? If so, can you send us a CPU profile?

--
To create a CPU profile using developer tools in VS Code:

select Help > Toggle Developer Tools
click on Profiles
click the start button
do whatever you need to do to get the CPU spin high
wait for some time (10-20 seconds should be enough)
stop the profiling
click the save button and attach the output file to this issue

With enabled extensions, I see these two messages in the developer console.

Handling of 'mousewheel' input event was delayed for 392 ms due to main thread being busy. Consider marking event handler as 'passive' to make the page more responive.

Failed to clear temp storage: It was determined that certain files are unsafe for access within a Web application, or that too many calls are being made on file resources. SecurityError

When I save the file, it's about 0 bytes. But in the developer tools, I see a huge list of captured data.

With disabled extensions, I see only this message in the developer console.

Failed to clear temp storage: It was determined that certain files are unsafe for access within a Web application, or that too many calls are being made on file resources. SecurityError

And again, the saved file is about 0 bytes.

And here is what I've captured in VSCode Insiders.
With extensions enabled

Without extensions

Again the saved file is about 0 bytes.

I also took some screenshots about the CPU consuming threads when the huge file is being opened.

The screenshots are good, but it would be so much more helpful if you could export the CPU profile and attach that to the GitHub issue. Thanks!

@fibric
Thank you for the information, here is my interpretation of the screenshots
Stable
Screenshot 1 and 2 - Stable with extensions enabled
Total time spent tokenizing (_updateTokensUntilLine): 6.8 seconds
Time spent with decorations: ~40 seconds
You most likely have an extension that operates on .js files and decides to add editor decorations (as in hundreds of thousands of decorations).
Screenshot 3 and 4 - Stable with extensions disabled
Total time spent tokenizing (_updateTokensUntilLine): 3.8 seconds
Time spent with decorations: ~76ms
I cannot really explain why tokenization would be faster with disabled extensions (it should be the same), unless perhaps you have an extension that contributes a different grammar for JavaScript and by disabling extensions we end up using the default grammar we ship with.

Note: It is ok for tokenization to take 7 seconds. Tokenization yields every 20ms and will not make VSCode unresponsive (i.e. you can scroll and type while tokenization runs).

Insiders
Screenshot 5 and 6 - Insiders with extensions enabled
Total time spent tokenizing  (_updateTokensUntilLine): 8.5 seconds
Time spent with decorations: ~ 3-4seconds
Again there is someone adding a lot of extensions
Screenshot 7 and 8 - Insiders with extensions disabled
Total time spent tokenizing  (_updateTokensUntilLine): 4.4 seconds
Time spent with decorations: I don't event see it
More or less the exact same conclusions as the Stable.
CPU screenshots
In the first screenshot there is the renderer thread consuming high CPU -- this is captured in detail in the profiles you've taken and I explained the reasons for the usage.
The other process that consumes high CPU looks to be like some sort of language server, most likely tsserver (that gives language smarts such as suggestions, formatting, etc.). There is some way through jsconfig.json I believe to disable the tsserver smarts for specific .js files (i.e. give up completions, squiggles, etc.). That will cause this process to not consume CPU.
Conclusions
All in all, there is nothing actionable for me here. There is an extension that adds lots of API and it does so on a somewhat-large file. The JavaScript language server consumes CPU (I'm guessing it is parsing the file to build an AST). There is additional CPU consumption if you are on a system with a touchscreen (#12036)
p.s. @joaomoreno at least on Windows, saving cpu profiles no longer works since updating Electron.


fixed the missing period

https://tigera.atlassian.net/browse/DOCS-1136?atlOrigin=eyJpIjoiOTI1ZTMxZWU5ZDFkNGZkY2FhMTM1MDNiOGFhMzY2MjciLCJwIjoiaiJ9

@ctauchen Can you review this one? I do not seem to have auto-merge right. Approver is required.

Fixes DOCS-1137 as well.


IoT Hub Device SDK

Hi, I am currently using Rust to connect and send messages to Azure IoT Hub. I noticed this functionality is missing here which I think is a shame since Rust gains some popularity in the embedded world.
I would like to contribute but first I want to get an OK from maintainers that this is something they would like to see here.

I’m not a maintainer on this repo, but I’d like to see it and would be willing to help with code reviews.

So you are looking for something similar to the Azure IoT device SDK for C. Do any of these generated crates help?

https://crates.io/crates/azure_mgmt_iothub
https://crates.io/crates/azure_mgmt_iotcentral
https://crates.io/crates/azure_svc_iotcentral
https://crates.io/crates/azure_svc_deviceupdate, see #733

@cataggar I am looking for a subpackage in the C SDK to connect to IoT Hub as a device (https://github.com/Azure/azure-iot-sdk-c/tree/main/iothub_client)
The first two crates you have posted are management APIs to create or manage actual Azure resources. The other two are cloud-side data plane operations on objects in the IoT Hub/Central, but it is still cloud-side management of the devices.
I am talking about a library created for devices so that they are able to connect to IoT Hub and send messages through MQTT.

I think IoT support in Rust is a great use case. To help build the business case, please fill out the survey that I linked in #738.

I have personal experience only with rumqttc but ntex seems intreguing. On the first look there may be an issue with TLS because it seems to use only rustls which cannot be used to connect to Azure IoT Hub (there are no shared cipher suites without enabling preview options on the IoT  Hub). For the record rumqttc currently does not support native-tls but there is an open PR for that.

An MQTT v3.1.1 Client implementation in Rust
https://github.com/Azure/iotedge/tree/main/mqtt/mqtt3

cc @onalante-msft

I would also need this feature. Any development on this?

This would be awesome to have, but needs to be funded. It is not planned for now.