Holding and dropping items

Copied from Urist's design doc:
Held items
A mob has a container with X hand slots, usually 2. He can switch between them freely (using the mouse wheel? hmm). A hand slot can be empty or it can contain a held item. The item (or emptiness) in the currently selected hand is used as the “currently equipped” value for PC interaction with the environment.
The default command when using primary interaction mode with an empty hand on an item that can be held is “pick up”, and this command is only available in these circumstances. Picking up an item places it into the current hand slot.
Some items may occupy multiple slots. In that case, picking up that item will push around your inventory so the items can be in two consecutive hands.
All held items have a “drop” command in secondary interaction mode, always in the same place on the wheel (probably left part). It drops the item under the character’s feet, leaving that hand empty.
All held items also have a type tag that can be “default”, “none” or one of the specific other types. This determines in what kind of toolbelt slots it can go. “Default” can go only to non-specialised toolbelt slots, “none” can’t go to toolbelt slots and specific types can go to toolbelt slots of that type or to non-specialised slots.

We have this functionality


Fix MD link checking, simplify redirect and ignore docs.github.com 403

See commit messages for details.

This will need to be done for other repos, but I wanted one instance of this change to go through review before sending more PRs.


update dependencies

Overview
Updates the dependencies
Note

The PR will probably fail, until #440 is merged in.

Yessir There is no rush to this PR. I wanted to show what npm outdated is showing. after viewing the data from npm outdated and doing a little digging. I use npm-check-updates https://www.npmjs.com/package/npm-check-updates.
I'll double back and QA and check on the major version changes.

I didn't see any glaring issues. I did a quick pass.

Here is one big plus for this update (Gonna wait till Monday to merge) https://blog.bitsrc.io/automatic-batching-in-react-18-what-you-should-know-d50141dc096e

Can you give a tldr lol, do we have to do anything special to receive those benefits?

TL:DR

as you web application grows, the number of nested components will increase. Therefore, if a parent component executes an un-batched state update, the entire component tree will be re-rendered per state update.

This behavior can cause severe performance issues related to application speed

React v18 ensures that state updates invoked from any location will be batched by default. This will batch state updates, including native event handlers, asynchronous operations, timeouts, and intervals.

There is a little more to this but basically say you have an a few different useState()
const [loading, setLoading] = useState(true);
const [data, setData] = useState({});

useEffect(() => {
  (async () => {
    const response = await fetch(someURL);
   setLoading(false);
   setData(response);
   )();

So on a small local level you woudn't really tell much different but on a larger scale eash of the 'setState' here causes a re-render. This can cause unexpected side-effects and re-rerenders. Batching allows for both these to be updated in a call and only re-render once.
There is no updates that we need to do currently for the benefit. They do mention:

Automatic batching is indeed an amazing feature. But, there can be situations where we need to prevent this from happening. For that, React provides a method named flushSync() in react-dom that allows us to trigger a re-render for a specific state update.


Add FilePropertiesStack helper class

This keeps track of the various cascading properties of file
layers at different levels. Allows pushing and popping, but its
true value is calculating and providing the various file permissions
and properties to whatever bit of code is populating a layer.

@mpeddada1 @chanseokoh if someone could take a look and approve?


Make a new demo on Hugging Face

So the old CogVideo space on Hugging Face was removed. I'm not using Replicate because I need a credit card to continue. Please, make a new demo on CogVideo.

Hi, thanks for the suggestion! The old demo was temporally removed because of computational resources issues, and we will recover it as soon as we can

Okay.


mac os x uses glibtoolize not libtoolize

for mac os x hosts "libtoolize" is not a correct command.
this is because mac bundles it's own version of "libtool",
which works differently, and may not even exist.
instead if you use brew to install libtool: brew install libtool
on mac, it writes to: glibtoolize as to leave mac happy with
it's own version of libtool by default.
this commit will properly use glibtoolize if detected, otherwise
will fallback to just using libtoolize.

oh good catch. Thanks for the contribution!


Install 'ncclient' and dependencies in ansible-silo-base image.

Open to feedback on if these changes are being done in the right place in the codebase!

@woneill : not that I have a horse in this race, but it would be helpful to state why you need ncclient installed in the base image.

@erikdw Good point! It got buried in the description of the first commit but 'ncclient' is used by many of the networking ansible modules. In my case, I need it for the Juniper support: http://docs.ansible.com/ansible/latest/junos_facts_module.html#requirements-on-host-that-executes-module

@woneill : ahh, cool, thanks for the info.  I'll bow out of this discussion now.

Closing this for now until I can figure out what's making the TravisCI build unhappy yet it works on my machine.
I'll resubmit once I get it figured out.


Remove references to PHP 5.6

Coverage increased (+0.005%) to 92.16% when pulling 998d61e4cad261a61e36f411c6307d800a7caf23 on kubawerlos:remove_references_to_php_5.6 into 511312867de2fc7c9a59e11f6df6d1f0da2b504d on FriendsOfPHP:3.0.

@keradus what changes?

Thank you @kubawerlos.


Option for always-on screen

Hi,
can you please add an option to keep the screen on as long as the app is open?

Will also add this in the next version!

There's a new update with this option now available in the settings.


Support url form encoded bodies

Motivation
Issue 182 Supporting runtime update for add URL form encoder & decoder
Modifications
Add converter methods for Codable <--> URLEncodedForm.
Note: Trying to encode a primitive collection (e.g. an array of String) form a Codable type fails because the URIEncoder rejects nested object types. I didn't make any changes to the Encoder here, as I think there may be a larger discussion needed about how to approach this. It would be beneficial to support this, as there may be a form which contains multiple values for the same key.
Decoding an exploded key from to a Codable type containing an array works already, but I've not included the test in this PR, as that should be included in the above.
Result
Converter methods for URLEncodedForms are now available for use in generated content.
Test Plan
Update tests for encoders and decoders.

Note: Trying to encode a primitive collection (e.g. an array of String) form a Codable type fails because the URIEncoder rejects nested object types. I didn't make any changes to the Encoder here, as I think there may be a larger discussion needed about how to approach this. It would be beneficial to support this, as there may be a form which contains multiple values for the same key.

@bfrearson, please file a separate issue about that if it's specific to the URI encoder/decoder and not specific to using it for bodies here. If it doesn't work, it'd affect query items as well, which can commonly be arrays.

@czechboy0 All changes complete on both generator & runtime 👍🏻


Add page speed/loading/web-vitals metrics

🐣 Is your feature request related to a problem? Please describe.
It would be nice to track the load times of different pages. This would allow you to see what pages take longer and might need to be improved or how the load times vary by load/usage or after a new version is deployed.
💡 Describe the solution you'd like
Using the ackee-tracker you could get the time it took to load the page like this:
const loadTime = window.performance.timing.domContentLoadedEventEnd- window.performance.timing.navigationStart;

And then just send it along with all the other data to the server.
Ackee could display it similarly to other metrics like pages. Here's a quick mockup:

Or even better show you the loading times of different pages over time, similarly to how durations are displayed.
📋 Additional context
Here's how Kindmetrics displays this. They also show you page speeds grouped by country which I think is very useful too, but as you mentioned in #103 this can't be easily integrated with Ackee.

That's definitely something I will take a look at and there has already been an issue for it which suggests using web-vitals: https://github.com/electerious/Ackee/issues/136
I would like to see this in Ackee by default, but it's possible to use the event API of Ackee v3 (not released, yet) to report web-vitals. I just need to add an event type that shows the average value in charts and lists. It might be a good and easy solution until it's a part of Ackee (I might add the average event types after v3).
Will close the other issue in favour of this one.

I've added new options to the events that allow to show average data (either as a chart or list). Could be worth a try! (It's now on the develop branch)

I've added new options to the events that allow to show average data (either as a chart or list). Could be worth a try! (It's now on the develop branch)


Fix pages variable check

Fix uppercase PAGES variable after Pelican made a breaking change to require lowercase:
http://docs.getpelican.com/en/stable/faq.html#since-i-upgraded-pelican-my-pages-are-no-longer-rendered

Thanks a bunch @tcprst !


Fix CRaC JDK download in docker container

Azul added a JRE to their CRaC java deliverables.
The query wasn't specific enough to say we want a JDK, so we end up pulling the JRE if it comes back first in the response. The JRE doesn't include the JCmd command which we use for snapshotting...
This change adds java_package_type=jdk to the request URL so we just get the JDK.
Done on the 3.7.x branch, to allow fixing Micronaut 3.x

The windows pipeline has some issue with graal 🤔 seems to be looking for a json file that's not there any more 🤔

The workflows on branch 3.7.x are out of sync, which explains the error on Windows. I guess we have to ignore it for now.


Add a cop that detects constant, class, and module declarations in DSL

Those declarations are defined in the global namespace and leak between examples.
Fixes #197
Related to #750
All credit goes to @jonatas, the original creator. I've changed the original cop created for in-house use to comply with the strict coding style of this repository.

Before submitting the PR make sure the following are checked:

[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the changelog if the new code introduces user-observable changes.
[x] The build (bundle exec rake) passes (be sure to run this locally, since it may produce updated documentation that you will need to commit).

If you have created a new cop:

[x] Added the new cop to config/default.yml.
[x] The cop documents examples of good and bad code.
[x] The tests assert both that bad code is reported and that good code is not reported.

@dgollahon Appreciate if you take a look. I'm reading your comments to #381 ATM.

before do
  stub_const('Foo', Class.new)
  class Foo
    # define the class now
  end
end

that you mention can be written differently, by defining the method inside a block passed to Class.new, e.g.
before do
  fake_class = Class.new do
    # define the class now
  end
  stub_const('Foo', fake_class)
end

Do you think that it's a frequent situation when some methods are defined in a block, and at the same time examples/contexts need to add/redefine methods on that class?
cc @backus

@bquorning please take a look.

What to do when I want to instantiate the class inside an example? That is not possible with stub_const, right? (e.g. RuboCop::RSpec::FakeCop)

You can, e.g.:
  before do
    fake_class = Class.new(described_class) do
                   def do_something
                   end
                 end
    stub_const('OtherClass', fake_class)
  end
  let(:instance) { fake_class.new }

The reason this is not possible with FakeCop is different, I've described the reason in the message for "Disable LeakyConstantDeclaration" commit.

@bquorning @Darhazer Do you have something in mind that needs to be made here or is it ready to be merged?

Nope, this is merge-able. Thank you for this cop ❤

@pirj Sorry I didn't get a chance to review this. I haven't been doing much OSS lately and the ping got lost in my notification queue. Glad you pushed through and implemented the cop, I think it's a valuable one :)
I just skimmed my old comments so I don't know if I clearly explained this before or if it has a place in the good/bad examples in the docs (or maybe in the style guide?) is that cases like:
before do
  stub_const('Foo', Class.new)
  class Foo
    # define the class now
  end
end

that could be written to pass the cop like this:
before do
  fake_class = Class.new do
    # define the class now
  end
  stub_const('Foo', fake_class)
end

should, imo, usually be written like this:
let(:foo_class) do
  Class.new do
    # define the class now
  end
end

This way nothing mutates the constant space and you can't possibly run into the normal issues you can with stub_const. Usually when you need a faux class you don't need it to be referenced as a particular constant name. In rare cases where that is needed, I find it's usually overriding some existing constant (occasionally necessary in weird situations), or dynamically setting a constant on a newly generated dynamic class (and in this case i would use my let version on the outermost class, still not polluting the global constant space). My main point is that it's sometimes worth reminding people that classes are just objects and don't have to be assigned to a constant, so this strategy might have a place in our guide or documentation somewhere, but as with most techniques, it should be used judiciously.
As an aside / trivia, if you use this it helps to be aware that anonymous classes have unusual #name behavior.
Class.new.name # => nil

but if you write it this way
Foo = Class.new
Foo.name # => 'Foo'

the constant assignment causes the anonymous class to receive a name (i'm guessing this is some sort of const assignment hook that mutates the anonymous class, but I don't know exactly how it works, mechanically). In practice this makes it seem the same as if you had written your class the standard way
class Foo
end

Anyway, nice work.

Ah, @bquorning also pointed to a good alternate explanation of what I was saying here: https://github.com/rubocop-hq/rubocop-rspec/issues/197#issuecomment-261495168

@dgollahon, love the reasoning for sacrificial classes.
I had a problem similar to what you mention just today:
    let(:dummy_job) do
      Class.new(ActiveJob::Base) do
        def self.name
          'DummyJob'
        end

        def perform
          # ...
        end
      end
    end

I ended up adding:
    before do
      stub_const('DummyJob', dummy_job_class)
    end

simply because ActiveJob is expecting ::DummyJob to be present in integration tests.
So, YMMV.
Also, this case won't work, since parent class's def self.inherited expects to have a class name, the situation you mentioned.
Leaking constants problem is undeservedly missed from the guide.
Are you up to add sacrificial classes guideline to the style guide and update LeakyConstantDeclaration cop's good examples to reflect this?

because ActiveJob is expecting ::DummyJob to be present in integration tests

Really? Wow, that's odd. I haven't interacted with ActiveJob in a long time but that is a weird API.

Also, this case won't work, since parent class's def self.inherited expects to have a class name, the situation you mentioned.

Ah, yeah, interesting case. Any time you have inheritance hooks things get weird.

Are you up to add sacrificial classes guideline to the style guide and update LeakyConstantDeclaration cop's good examples to reflect this?

Maybe, but likely not. I'm a little less keen on maintaining the style guide v.s. rubocop-rspec, but I'm happy to weigh in on and review any changes if someone else would like to update the information. I might change my mind, but there are other things I'd probably rather tackle first.

@dgollahon In response to this https://github.com/rubocop-hq/rubocop-rspec/issues/750#issuecomment-515582100
It's actually a very good point that re-opening the class doesn't by itself lead to constant leakage.
I'm concerned about the possibility of the leakage when stub_const is being removed. Some cases of that are impossible to detect statically, e.g. when stub_const is defined in a shared example residing in another file.
This makes it a really hard task to detect if it's re-opening of a stubbed class or a new class definition and with all the downsides of the latter and existing workaround (Class.new do) for the former I'm not sure if it is worth it to allow re-opening case.
At the same time, I can think of a case when constants need to be defined in that stubbed class, e.g.
let(:klass) do
  Class.new(ClassUnderTest) do
    SOME_CONSTANT = ...
  end
end

and without re-opening, this example may become quite clumsy.
Let's give it another thought if we notice or someone complains that the cop doesn't simplify our lives as it is, what do you think?


youtube doesn't play video and says sign in to confirm you are not a bot.

script version 5.6
it's now popping up as sign in to confirm you are not a bot.
on youtube both chrome and chromium edge

yes same thing happens for me also


Communication with Thingsboard IOT dashboard.

Hello,
I followed your instructions and was able to set up the communication with the coiote IOT dashboard. Now I want to set it up with a local instance of thingsboard with PSK on port 5686 ( DTLS port of thingsboard lwm2m) but I am getting the following error on the esp32.

eager to hear from you.
thanks

Hi!
Looking at the error code it seems that operation timed out: https://tls.mbed.org/api/ssl_8h.html#a2fdb240812860402a1a946eae8264148
If you were able to connect to coiote using same aplication, client has to be working. Please check if there is a connection between client and serwer and it is not blocked by firewalls or other blockers.

Thanks for the reply.
I'm running thingsboard IOT dashboard locally on an RPi and it doesn't have any firewall by default. To test the network I have simulated a client using wakaama git and there is no network issue there.

could this problem be side stepped by creating a client without PSK security? when I try this configuration:

it gives the following error and goes into a boot loop:

Also, could you please tell me how you checked the error code from the website that you linked (https://tls.mbed.org/api/ssl_8h.html#a2fdb240812860402a1a946eae8264148) It would help me a lot in the future to debug
Thanks

If you want to use nosec, you have to change it manually in code here:
https://github.com/AVSystem/Anjay-esp32-client/blob/main/main/main.c#L61
To use ANJAY_SECURITY_NOSEC mode.
It will also timeout if you provide wrong security configuration, which can be a case here
Regarding mbedtls error, you can find it in the log: -26624 which converts to hexadecimal and can be found on mbedtls page

Thanks for the reply.
when i change it to NOSEC it works. After doing some more testing i can say PSK doesn't work with thingsboard because:

Thingsboard requires the PSK key to be in hexadecimal format. is this possible and if so can you please tell me how?
Thingsboard requires the PSK key to be at least 128 bits long. Does the esp32 support uint128_t?

We have not used thingsboard but it seems that it should be just plain text hexadecimal - for example "0123456789ABCDEF0123456789ABCDEF"
128 bits long means at least 4 bytes -> 4 characters, same requirement is with coiote server

Sorry for the late reply,
I'm still facing the same issue with PSK configuration. Any help is very much appreciated.
here is the configuration of the device on thingsboard

here is the client configuration on the esp

I still get the same handshake timeout error in the terminal window

esp-client NOSEC with thingsboard works. wakaama test client (using dtls) with thingsboard works

Hi, I chose the string as:
convertstring32a
and converted it to hex which came out to be
636F6E76657274737472696E67333261
I entered the hex code in the tb dashboard and the plain text key in the esp config. when i build and run it i still get the same handshake timeout error

Just noticed it, was on vacation before, great to hear that!


Improvements to documentation

Looking for feedback.
An overview of the biggest changes:

I've removed the "Customization" section,
I've renamed "Recipes" to "Development recipes", it also receives all the files in "Customization"
I've added a new section, "Addons". It has the old content for addons from the Customization
A new page in Addons, "Best practices" for addons
I've made "Getting started" a section. The old page becomes "Bootstrap Volto"
I've added a new page "Developer roadmap" under "Getting started"
I've also added "Learning resources" under "Getting started", with links to training and Youtube videos

@plone/volto-team It would be nice to get this reviewed and merged, there are quite a few improvements to the docs, there's a lot of interest in the community around Volto, it's important to have nicer docs, if possible.

I square read the PR and this is indeed a great improvement. Just a general recommendation for our technical docs. We should try to write as short as possible and avoid rationalizations and long explanations. We do not need to rationalize the decisions that we made and we can just state things as they are. Most people scan dev documentation instead of reading it from the start to the end. This should be reflected in the structure and writing style of our dev docs. Maybe @svx has to add something here as well. :)

+1 of merging this right away. We can always refine later.
At some point, we should try to invite @svx to a nice place somewhere in the world to give all of us training on how to write good technical docs.

Would be glad to !


Error in BaseBEVBackbone

if len(ups) > 1:
x = torch.cat(ups, dim=1)
for some special case, if input feature map size is [199,199] in basic setting in second, ups[0] and ups[1] may have different size,[199,199] and [200,200] respectively, that may cause error

to reproduce 199, you can just delete the last layer in VoxelBackBone8x, and set voxel size 0.1,0.1


[docs] [material-ui] The "App bar with search field" sample code has problems on small screen devices

Duplicates

[X] I have searched the existing issues

Related page
https://mui.com/material-ui/react-app-bar/#app-bar-with-search-field
Kind of issue
Other
Issue description
I was reading the Material UI documentation (vesion 5.14.19) and tried to apply the sample code from "App bar with responsive menu" from react-app-bar.
But when it runs on a small device, the back part of the search box cannot be clicked and typed, only when focus is moved to the front.
Attached is a screenshot:

Context 🔦
Users may not start clicking from the front part of the search box, and it may be confusing when they click the back part of the search box and nothing happens, so I think this may need to be optimized, after all, many developers will directly use the Sample code for development.

Thanks for the report @instaer, I've opened a PR to fix it 😊


regenerate clients

@andreamlin Do you know if we updated proto dependency or something? A LOT of proto-generated files have the same changes.

This looks like an accidental reversion in a previous run: https://github.com/pongad/google-cloud-java/commit/a02ff76245067fb508d2bb10528bf25d03040c52#diff-018ee9c8f39b25865c8c1fc87966402d
@andreamlin maybe you generated with an old protoc somehow?

Oh that might have been the case. My local version of protoc is 3.4.0; though I used artmanflow which I think uses the Dockerfile that installs protoc 3.5.1? Anyhow if this was generated with protoc 3.5.1 that should be correct and the last batch generation I did was using an older version of protoc.

This is strange. The field has been around since September 2017, and we should be using the same docker image...

@garrettjonesgoogle @andreamlin Does this look good to you? If so I'll merge and start the actual release.


A request directed to a state machine should fault if no row exists

If a request is directed at a state machine, the lack of existence of a row should send a fault back to the requester.

Changed so that the event correlation can configure the behavior on a missing instance.
Commit 20588ac makes this possible, and if a fault is specified, the fault returns to the request as a request fault exception.


djview: fix mozilla plugin path

Motivation for this change
Attempting to fix nixpkgs.config.firefox.enableDjvu = true;
The plugins are in $out/lib/mozilla/plugins.
$out/lib/netscape/plugins does not exist

nixpkgs.config.firefox.enableDjvu = true; forks with 64-bit firefox-esr-52.
It does not work with 32-bit firefox-esr-52 on 64-bit nixos, but it seems to be related to massive problem with 32-bit GUI programs on 64-bit nixos caused by recent GTK refactorings

Oh, still keeping browser plugins alive?  There must be a strong motivation for that, right?  To be clear, I'm interested why the effort, as Mozilla had basically killed them off.

there are devices (IPMI consoles interfeced of the servers) which require browsers with java plugin (some of them require Java not newer than 6), and there are still in production.
there are websites of offline libraries which do not allow to download scans but allow to them in djvu plugins (and it is difficult to sniff where to download the djvu file, because they load it in small  fragments, even smaller than a page, each fragment in own .djvu file), and they are still online

I do not expect it is going to change quickly only because someone at Mozill called it deprecated or even vulnerable. It requires big effort and big money to change.

Yeah, I agree browser world/vendors tend to be "aggressive" in some ways, e.g. wrt. long-term compatibility.

up

Another reason to keep firefox-esr-52 and firefox-esr-60 is the modern firefox (66), as well as chromium, are broken on i686-linux and it is unlikely that someone will fix them

Right, IIRC there's the thing that the final libXUL link needs > 4G memory.  Maybe it would be possible to work around by some setting (using a different linker or whatever), and there's that option to try 64->32 cross-compilation, but hardly anyone wants to put much effort into i686 anymore.


oracle/database:18.4.0-xe doesn't allow connections for the system user

When setting up oracle 18.4.0-xe as
https://github.com/oracle/docker-images/tree/master/OracleDatabase/SingleInstance
when trying to connect from outside the instance I get the exception:
ORA-12528, TNS:listener: all appropriate instances are blocking new connections

or
ORA-12526, TNS:listener: all appropriate instances are in restricted mode.

As a workaround, I connect on the docker container
docker exec -it --user=oracle ${oracle_image_name} bash

and disable the restricted sessions setting
sqlplus /nolog
conn sys as sysdba
password: oracle
alter system disable restricted session;

and after this works.
Could you please enable by default or environment variable the restricted sessions?

I cannot reproduce this issue. After a successful setup, I can connect from within the container as well as outside the container on the same instance with SYSTEM and any other user.
The two error messages above could indicate that the database wasn't fully set up yet when the connections where tried to be established.
Can you confirm that you saw this message in the container log before you tried to connect:
#########################
DATABASE IS READY TO USE!
#########################

I haven't used the docker image since issuing this ticket.
If the connection to the database works on your machine then probably I must have done something wrong while setting up the Oracle docker instance.
Please disregard then this ticket.


serialize support

If I use gson to save and restore an object, I can't save this object directly.
I must clone it and save the new one.
I migrate from Sugar, which works pretty fine with gson.

Sorry. You can paste some codes for more details. As far as I know, LitePal can save a object which parsed from gson directly. Never used clone stuff.


ClientInterface does not have shortcuts get(), post() etc.

The client implementation has the magic methods for get etc but the ClientInterface does not. So if you typehint against the interface, it's impossible to actually use them.
So I would propose to add them to the interface.

By "impossible to actually use them," do you mean that IntelliJ autocomplete does not work as expected?

it means you cannot rely on them being available. otherwise the whole interface makes no sense.

Those methods are really only there to make backwards compatibility easier for people who were already relying on a concrete class. Adding new methods to the interface right now would be a breaking change.
IMO, I don't really think the client needs that many methods. I think you should use other methods like request for creating and sending, and send to create a request that you already created.

That's fine to me as well. But then the documentation needs to be updated to not use those methods. And they should be marked as deprecated, just for BC.


Is Klipperscreen over USB possible without a Pi?

As per the title.
The guide recommends to use a Raspberry Pi to drive a touchscreen, but would it be possible to forego a Raspberry Pi, and use a USB-C cable attached to one of the spare USB-C ports on the Qidi mainboard itself to drive a touchscreen via Klipperscreen installed on the Qidi mainboard instead?

As per the title.
The guide recommends to use a Raspberry Pi to drive a touchscreen, but would it be possible to forego a Raspberry Pi, and use a USB-C cable attached to one of the spare USB-C ports on the Qidi mainboard itself to drive a touchscreen via Klipperscreen installed on the Qidi mainboard instead?

No, this will not work as the display is controlled via HDMI. Unfortunately, the Qidi mainboard does not have an HDMI output

I'm aware that the Qidi main board does not have an HDMI slot.  The Qidi mainboard has a USB-C slot though.  There are USB-C to HDMI adapter out there, also even 5" LCD touch screens that are driven by USB-C.
As one example: https://www.amazon.com/Anker-Aluminum-Pixelbook-Compatible-Thunderbolt/dp/B07THJGZ9Z?th=1

As per the title.
The guide recommends to use a Raspberry Pi to drive a touchscreen, but would it be possible to forego a Raspberry Pi, and use a USB-C cable attached to one of the spare USB-C ports on the Qidi mainboard itself to drive a touchscreen via Klipperscreen installed on the Qidi mainboard instead?

No, this will not work as the display is controlled via HDMI. Unfortunately, the Qidi mainboard does not have an HDMI output

I'm aware that the Qidi main board does not have an HDMI slot. The Qidi mainboard has a USB-C slot though. There are USB-C to HDMI adapters out there, also even 5" LCD touch screens that are driven by USB-C.
As one example: https://www.buydisplay.com/5-inch-usb-capacitive-touch-screen-panel-overlay-for-rasperry-pi
or for an adapter: https://www.amazon.com/Anker-Aluminum-Pixelbook-Compatible-Thunderbolt/dp/B07THJGZ9Z?th=1

Ok, the first link is just an overlay for a display. This is not a real touch display. The second link could be interesting if the two USB-C ports can be used at all. I'll ask Qidi about this. Klipperscreen must also be able to control the output via the USB-C.

Ok, the first link is just an overlay for a display. This is not a real touch display.

Argh.  I missed that it wasn't a full display.  I shouldn't comment before coffee.  :(  Thank you for catching it.

With the adapter (USB-C to HDMI), for it working, the OS needs to support DP Alt mode. So if the main controler (RK3328) and the kernel compiled with the option, it should (may be) works.
But is the USB-C port of the Qidi board useable (M-USB between screen plug and μSD reader) ?

But is the USB-C port of the Qidi board useable (M-USB between screen plug and μSD reader)

I have written to the support team about what the two USB ports to the left and right of the port for the display can be used for. And what the technical data of the two ports are.

According to Qidi, the two USB-C ports were planned for debugging and as an additional option for connecting the display. An answer as to whether the M-USB slot can be used for other purposes is still pending

According to Qidi, the two USB-C ports were planned for debugging and as an additional option for connecting the display. An answer as to whether the M-USB slot can be used for other purposes is still pending

So, by "were planned" meaning that they're functional, or non-functional?

According to Qidi, the two USB-C ports were planned for debugging and as an additional option for connecting the display. An answer as to whether the M-USB slot can be used for other purposes is still pending

So, by "were planned" meaning that they're functional, or non-functional?

I have just received a reply from support that these two ports are not functional and cannot be used for any other purpose.

I have just received a reply from support that these two ports are not functional and cannot be used for any other purpose.

That's seriously disappointing.  I do believe that there's possibly a better alternative here.
https://www.makerbase.store/products/makerbase-mks-skipr-3d-printer-board?variant=40517147000905
It seems that we can use that board as a complete drop in replacement for the Qidi main-board.  It has HDMI and enough ports to drive for a Qidi printer's peripherals, all for what seems to be fairly similar to the price of a separate Pi4B to drive KlipperScreen.  Doing it this way, we can keep the original mainboard aside, and do the mainline klipper solution + screen all one one board.
Your guide is still extremely useful though, as it still applies for setting up the MKS SKIPR board.  We just need to be aware of any pin differences between the two boards and modify the printer.cfg file accordingly.

@stew675 I personally would not go for MKS. They are notoriously cheap and low quality.
I think the BigTreeTech Manta would be the better fit.
I personally used them in two printers already, and I am really tempted to also stick one in the X-Plus because the EMMC Flashing annoys me greatly.

@stew675 I personally would not go for MKS. They are notoriously cheap and low quality. I think the BigTreeTech Manta would be the better fit. I personally used them in two printers already, and I am really tempted to also stick one in the X-Plus because the EMMC Flashing annoys me greatly.

Qidi are already presently using what is essentially a cut-down MKS SKIPR 1.0.
I'd not seen the BTT Manta before.  So am I correct in understanding that the CB1 is basically a 64 bit Raspberry Pi clone ARM CPU that sits on the Manta like a "hat" to give "all in one" functionality?  If so, that's really interesting.
The Qidi print-head seems to connect to the mainboard via a USB-C power cable, labelled as HE-USB on their control board.  This cable is driving all the fans + filament heater in the print head.  My question here is do the Manta boards provide something similar?

My question here is do the Manta boards provide something similar?

I am of the opinion that you would then have to modify the print head, as the USB-HE is not available on the Manta board.

USB-HE is just a totally normal USB-C.
I had it connected to the normal USB-A today over an adapter. Nothing special happening there. The Mantas all have at least one USB-C.

USB-HE is just a totally normal USB-C.

The HE-USB (High Energy?) port is also built like a normal USB port. Only that it is operated here with 24Volt

Like I said. I have a THR already running on an Manta. Will try the QIDI THR too to see if it heats right. 24V is totaly within USB-C PD specs.

https://plugable.com/blogs/news/what-is-240w-usb-extended-power-range-epr
USB PD SPR (the older standard) is only a maximum of 20V (and 100W)
USB PD EPR is up to 48V (in 100mV increments) and 200W
Now it's possible that the Qidi X_4 board is just feeding VBB (the base 24V from the PSU) through the cable without any specific regulation.  Actually that seems like that would be the cheapest way for them to do it, over a fairly standard USB-C cable that's rated for the ~40W or so that the print head is going to pull at peak, which is basically any regular USB-C cable.
So perhaps instead of asking if PD EPR is support, the question may be (and keeping in mind I only have very rudimentary knowledge of all these electrical connector standard) is if the BTT Manta has a USB-C port that supplies VBB, perhaps by setting a jumper on the board or something?

Just tried it. It heats, but slowly.
Yeah seems like QIDI or MKS, as always, cooked their own stuff here.
We stuck with the mkspi.

Just tried it. It heats, but slowly.
Yeah seems like QIDI or MKS, as always, cooked their own stuff here.
We stuck with the mkspi.  Was hoping to get rid of that board.

Yeah.  I suspect that there's no actual USB standard power regulation going on there.  The cable is likely feeding the standard communication 5V signal, and for the power delivery pins, it will also be defaulting to 5V unless the other end starts asking for more.  I suspect that the THR print-head isn't doing any proper USB power negotiation at all (I believe the chips that do that negotiation would add cost), so more than likely it's going to be what I suggested, and that is the HE USB port is using regular USB voltage for data, and simply feeding in unregulated 24V channelled straight from the PSU.
That would be the absolutely cheapest way for them to implement it, but aside from the data pins, there's not going to be any usual USB power standards being followed here.
As @leadustin suggests, we'd either need to replace the print head entirely to suit a BTT Manta or a vanilla MKS-SKIPR

Just tried it. It heats, but slowly.
Yeah seems like QIDI or MKS, as always, cooked their own stuff here.
We stuck with the mkspi.  Was hoping to get rid of that board.

I had a thought.  It may be possible to make up a small circuit board that takes the USB data signals from the Manta, and the power from the PSU, and then have it do power the Qidi way to the print head.  It wouldn't need to be that large.  You'd run a short male to male cable from the Manta into this mini-board, and on there have a USB-C connector that is feeding power from the PSU to the power-delivery pins on the USB cable that runs to the print head.  Might even be able to do it with a bread board setup.

Just tried it. It heats, but slowly.
Yeah seems like QIDI or MKS, as always, cooked their own stuff here.
We stuck with the mkspi.  Was hoping to get rid of that board.

I had a thought. It may be possible to make up a small circuit board that takes the USB data signals from the Manta, and the power from the PSU, and then have it do power the Qidi way to the print head. It wouldn't need to be that large. You'd run a short male to male cable from the Manta into this mini-board, and on there have a USB-C connector that is feeding power from the PSU to the power-delivery pins on the USB cable that runs to the print head. Might even be able to do it with a bread board setup.

You could check whether you could tap 24V+- from the HE ports on the Manta board and connect it to the print head via a custom board.

If you want to use a BTT Manta (or other MB similars), would'nt it be simpler to use the CAN port instead of USB cable  ?
It's well known that the HE-USB connector on the Q4 (Q6 but not Q7 that use an other cable and other card without USB port) cards deliver 24V. You can't use it as a normal USB-C port unless you want to fry your peripheral connected.
Makerbase produce a lot of cards for other brands (Artillery, Two trees, Qiditech, …), their cards are'nt so bad but unlike Bigtreetech or Mellow or Duet, their documentations are not the best 😏

It's well known that the HE-USB connector on the Q4

Just for educational purposes, could you provide some links to where it's documented that this is the case?  Since none of the other people on this discussion appears to have "known", then I would argue that it is not as "well known" as you are claiming, which means that, speaking only for myself, there must be some place where this is commonly discussed frequently for all to know, and I'd like to know where that is.

QidiTech does not provide the electronic diagram of the board despite several requests.
For your information, I have been testing this printer since I received it at the beginning of August 2023. I report my discovery and analysis of this model on a French forum: lesimprimantes3d.fr, in addition a Github repository has also been opened accessible here
@leadustin knows this because he's been on the forum :smirk:
Otherwise, if you have a multimeter, it's not very difficult to test this USB-C plug (HE-USB): the power supply is indeed 24V. Near this HE-USB is a HE-MX connector, regularly used for CAN (GND,D-,D+,VIN), with a multimeter you could probe the pins between HE-USB and HE-MX and see if they match up.
If you've got a USB stick you no longer need, plug it in and see what happens.
:smiley:

For your information, I have been testing this printer since I received it at the beginning of August 2023. I report my discovery and analysis of this model on a French forum:

I believe that there is some confusion here, possibly caused by cross-language translation/interpretations.
We were already aware that it was highly likely that the HE-USB is feeding 24v straight from the PSU after some investigation by @NightHammer1000 .  You claimed that this was a well known fact, citing Q4 and Q6 boards, which I assumed must have been a reference some Makerbase products/boards that I don't know about.  Your statement about it being "well known" came across as rude and contentious.  I don't know what are the Q4/Q6/Q7 items that you initially brought up are referring to.
It would have been better to initially say something like: "I can definitely confirm that the HE-USB port supplies 24V based upon my findings ".  That would be a less antagonistic and a more productive statement to open with.
Moving on past all that though, where we're at now is finding an option to construct (or purchase) an adapter for something like a BTT Manta M5P, or Makerbase SKIPR board that is capable of driving the stock Qidi A-4 print-head board rather than having to replace the Qidi A-4 print-head board.  Neither the SKIPR nor the M5P provide an HE-USB output as standard.
Since you seem to be more familiar with these interfaces are you able to offer any suggestions on how that might best be achieved, such as via some off-the-shelf item?

After some research, it seems what's needed is a pair of USB-C Breakout Boards.
For example:  https://www.adafruit.com/product/4090
A USB-C cable from a control board cable can be plugged into one of these boards.  Let's call that Board 1.
A 2nd USB-C cable can be run from Board 2 to the print head.
The data pins from Board 1 can be wired to the data pins on Board 2.  24V power can be wired to the voltage pins of Board 2.  In this way we can fairly easily fabricate an adapter for under $10 that will convert a standard USB-C port into the HE-USB style port that the Qidi A-4 print-head boards wants.
@NightHammer1000 this may be the solution that you're looking for in order to get rid of the stock Qidi control board. :)
I think that this actually completes the topic of my original question.  I'll close this issue now.


App labels not populating while creating a chaos experiment in Litmus

I am able to see the namespaces available but not app labels. Will it be auto populated? Also if i manually, enter i m getting error as pod not found.

Hi @kavinarvind, are you still facing this issue with the latest version of litmus? If yes, can you provide some more information, as have you installed the agent in cluster or namespace mode?


Add lombok Library and make zerocode more user friendly and reduce no of lines

Hi All,
Requesting a enhancement to zerocode library by adding Lombok library. this will not impact end user/consumer but it will help contributor to see user friendly code,reduced no of lines and add advantages which are provided by Lombok library.
https://projectlombok.org/ - add lombok maven depedency and replace getter/setter/toString/HashCode/Equals method with @Data
Getter Setter with @Getter @Setter annotation.

@authorjapps  @BeTheCodeWithYou @dinesh76in  - do you think this will add benefit to zerocode framework? if yes, then we can pickup this up.

@aghorpade , actually this might not add any value in my opinion. We have to maintain it with unnecessary overhead. Also we need to install  a plugin to work in the IDE to avoid compile-time error.
Even now we don't write the getters/setters, toString, equals/hasCode etc manually, we use IDE to generate it for us.
Let's close this issue for now.

@aghorpade  good suggestion but I think for the zerocode, I don't see much benefits but rather overhead for maintenance.  Yes lombok is very good and I have used it but I think should be used for only private projects.  Debugging becomes difficult if you wish to put debug on equals() or hasCode.. you can't, as they are generated magically :-)
I just did quick scan on google for it's side effects, and this article really a must read before using  anywhere in general
lombok<EMAIL_ADDRESS>@authorjapps  agree with you points.

Cool.. Just for everyone's life to be easy.. lets drop this idea to add Lombok. We can close this ticket


Walk me through building the app and making a first simple contribution to the project

I tried to follow the instructions in the readme, but got stuck. Please help me out and let's improve the instructions for everyone along the way. Note that I'm still on macOS Mojave since one of my most used apps unfortunately isn't compatible with x64 versions of macOS.
So, first I ran this command:
sudo apt install git npm
Here I was prompted for my password. After filling that in, I got this output:
No Java runtime present, requesting install.

Followed by this popup:

So I clicked More Info… and was taken to https://www.oracle.com/java/technologies/downloads/ Here I could choose between different versions of Java. I had no idea which one to pick but picked the one named x64 DMG Installer. After that one was downloaded, I opened it and double-clicked the icon labelled JDK 17.0.1.pkg.

After having walked through the installation of the JDK, I tried running sudo apt install git npm again but this time got this error:
Unable to locate an executable at "/Library/Java/JavaVirtualMachines/jdk-17.0.1.jdk/Contents/Home/bin/apt" (-1)

So right now I'm stuck there. Help me out please.

I will go through the installation instructions tonight when the kids are asleep!

Awesome that you want to contribute!
apt is more of a Linux thing as far as I know. You can install Node (which contains npm) by downloading an installer from Node.js. We are currently using Node v14.15.4 for the app.
You can then install Git using its installer.
If you are more comfortable using the terminal I would recommend installing Homebrew and using it to install git and node, eg. brew install node.
When you got both setup you should hopefully be able to follow our instructions. Or @viktorlarsson might find something else missing tonight.
Welcome, looking forward to that first contribution 😃

I have now updated the developer documentation, could you please follow the instructions for your operating system and the app you would like to run and see if it's understandable?
https://github.com/kolplattformen/skolplattformen#getting-started-with-development

Would also advise you to pull the latest changes from main using git pull origin main


bazel: change .bazelversion to be symlink from envoy

These should likely always match since envoy's build configs could
depend on it.
Signed-off-by: Keith Smiley<EMAIL_ADDRESS>

Thanks @keith! This sounds good to me. We have historically had a couple of stretches where we needed to build on a different version of bazel from the version on Envoy main, but this seems good to have as a default and it's not like we can't change it if we're forced to down the road.


CSS API: Deprecate style invocation in CSS queries

Following code generates exactly the same CSS:

val cls by style {
                (self + hover)  {
                    color(rgb(10, 20, 30))
                }
        }

val cls by style {
                (self + hover)  style {
                    color(rgb(10, 20, 30))
                }
        }

In this issue we are not consider here the selectors API itself (in this case, the self + hover) - the goal of this task would be to deprecate second syntax, with style since it actually doesn't provide any technical or usability benefits compared to the first syntax.

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Repeated TypeError on message received

When a message is received I get this error and clicking OK makes it pop up repeatedly. TypeError: Error processing argument at index 0, conversion failure from... Let me know if logs would help debug this issue and I can upload them.

Had same issue. Exit messenger, renamed folder %appdata%\Android Messages to something else, started messenger, re-enabled it on the phone, exit it again, renamed that folder back to it's original name. That fixed the problem for me without losing any settings. Go figure.


problem to get informations with steamid

Hello,
i like your package, but i have a problem.
My request (for test with my profileid):
$steamid="76561197982861719";
$format="";
$test = Steam::user($steamid)->GetPlayerSummaries()[0];
print_r($test);

The return:
(
    [steamId] =><PHONE_NUMBER>2861719
    [steamIds] => Syntax\SteamApi\Containers\Id Object
        (
            [id32] => STEAM_1:-1:-38280597906391040
            [id64] =><PHONE_NUMBER>
            [id3] => [U:1:-76561195812782081]
            [communityId] => STEAM_1:-1:-38280597906391040
            [steamId] =><PHONE_NUMBER>
        )

Second request (for test with my steam id):
$steamid="STEAM_1:1:11297995";
$format="";
$test = Steam::user($steamid)->GetPlayerSummaries()[0];
print_r($test);

The return:
Undefined offset: 0
I dont understand with i dont get my profil with my steam id, i need help please :)
Thanks

I don't think the call itself allows for a steam ID type other than via the normal STeam ID (the one in your first example)


Bugfix/rdf bugfix

Bug in rdf
If not all optional keys are  defined in the input file, a PQKeyError occurs. But not all optional keys can be set simultaneously.
Suggestion to solve it:

This was solved by include all not defined optional keys in the dictionary and set them at  None  in RDFInputFileReader  before the keys are actually used.

Further, for the parsing also NoneType  in _parse.py the conditions for the InputFileError messages are changed so that None is not throwing an error anymore.

The rdf program works only if a topology is given either with restart_file or (moldescriptor_file and restart_file).
If it is not given, an error occurs during the selection part of the target and reference molecules
Therefore, in RDFInputFileReader an error message was included.

The target_density in RDF  was calculated:

target_density = len(self.target_index_combinations[0])

but this is only defined  if no_intra_molecular=True. So this was changed to:
        if self.no_intra_molecular:
            target_density = len(self.target_index_combinations[0])
        else:
            target_density = len(self.target_indices)

At the moment, no unexpected errors occur.

I really like the idea of introducing the "None" if I get it right. Could you provide a sample inputfile? And you execute it then just with rdf <inpu> or?

I really like the idea of introducing the "None" if I get it right. Could you provide a sample inputfile? And you execute it then just with rdf <input> or?

Yes I just use rdf <input>  and no further errors have been found so far.
The input file looks like:
reference_selection = H
target_selection = H
delta_r = 0.05
out_file = rdf_test.out
restart_file = md-test.rst
traj_files = test.xyz

The tests are blocked most likely due to a numpy function numpy.atleast_1d in Trajectory.__init__(). I have the same issue on mac. Removing it could resolve this one.


Forgot to update version resource

We forgot to update the version resource in the launcher on v1.4.0 release.
(Lines 64, 65, 82, 87)
We probably wanna add that to some release checklist, maybe even do a safety check in CI to make sure values on those lines and release tag match up.

Could this process be automated across the whole project with the build number everywhere being updated from one text file when building?

I'll probably update resource strings to 'MASTER_BRANCH' and have them replaced on the build pipeline

The resource version is set to DEV and 0,0,0,0 on master branch: https://github.com/R2Northstar/NorthstarLauncher/commit/ce369d39889304b5712e9628c7ae084c27a1cce2
I added a github actions step to replace those with the tag that's being built: https://github.com/R2Northstar/Northstar/pull/146/commits/922a4817767c9089a2a8cf9e7570020897b0e8e9

Do we maybe also apply a similar action to the NorthstarMods?
(assuming we change the version to mirror that of release)

Do we maybe also apply a similar action to the NorthstarMods? (assuming we change the version to mirror that of release)

I'll try to do that next


Server Problem on Demo Site?

I'm seeing 503 errors on the demo pages:
For example: https://serviceworke.rs/push-simple_demo.html
https://serviceworke.rs/push-simple/register  503 error
https://serviceworke.rs/push-simple/sendNotification?.... 503 error

I'm not seeing 503 errors.
Does the demo work for you? What browser are you using?
The register request is empty, I don't see how it could fail:
  app.post(route + 'register', function() {
    // A real world application would store the subscription info.
  });

No, the notification wasn't working, it was on Chrome Canary on Windows (my work machine).
I'm on Chrome (standard ) on Mac now and it's working fine here

I do still see the 503 responses but they're obviously not affecting it.
Log; serviceworke.rs-1452805485737.txt

-if you want me to check anything on Canary I can check when I'm at work tomorrow..

Thank you, I think I've found the issue and #197 should fix it.
The errors were spurious though, so they shouldn't affect the notification stuff.
Can you open another issue, after #197 is merged, if you still can't receive notifications on Windows?

Yes,
No problem,
I'll check it, when it's merged..

Issues still exists, don't expect to be resolved until pull #197 is merged


New Char sheet portrait overlaps border

It appears the portrait in the new char sheet overlaps the frame, so portraits can be partially obscured

This is intentional to give a 'full card art' effect for more traditional portrait styles, like the following:

It will not work equally well with all styles of portrait I'm afraid.


Generalise mux over the set of protocol ids

Here for comment initially. Needs to be rebased now that #311 was merged. Should be easy to fix up.
We will need to instantiate the mux with different sets of mini-protocols for the node-to-node vs the node-to-client protocols.
The approach we take is a simple trick using custom bounded enumerations. This makes use of the Ix type, arrays and functions covering the whole range of the enumeration.
This also eliminates some of the Data.Map lookup error conditions.
Also update the Pipe and Socket demos to use this.

All comments should be addressed. Rebased and tests fixed up. Should be ready to go.

LGTM.

bors r+


Make lsd compile with ARM as target

Expected behavior
On small SBC systems, lsd would be extra useful since a lot of interaction happens via ssh and the console on these systems. An ARM target would be helpful to ease the install.
Actual behavior
No ARM target is available, lsd needs to be build from source, probable issues present

Hi @emk2203,
I should be possible to add a target, I'm on it.


Fix Kubelet Configuration apply when host OS config is specified (#8602)

Issue #, if available:
Description of changes:
Fix Kubelet Configuration apply when host OS config is specified
https://github.com/aws/eks-anywhere/issues/8594
Cherrypick from https://github.com/aws/eks-anywhere/pull/8602
Testing (if applicable):

unit test
e2e test

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


release windows 10 under a free software license.

Duplicate of https://github.com/microsoft/vscode/issues/74714, https://github.com/microsoft/calculator/issues/528, https://github.com/microsoft/dotnet/issues/1051, https://github.com/microsoft/terminal/issues/1098 and https://github.com/microsoft/CNTK/issues/3674.


Breakage in recent linux

Hi there, thanks for writing this awesome wrapper package.
I've just recently started having an issue, where I get the following output running the demo candump app
$ go run cmd/candump.go -if vcan0
2018/09/19 03:37:16 invalid argument
exit status 1

I traced the error down to the call to syscall.Bind() in NewReadWriteCloserForInterface, but I'm not familiar enough with the relationship between the golang syscall package and the C end to take that gap any further. It's possible that this is actually a lower level issue in the go stack, but I figured I'd come here first just in case.
The kernel version on the vm it broke for:
$ uname -a
Linux kali 4.17.0-kali3-amd64 #1 SMP Debian 4.17.17-1kali1 (2018-08-21) x86_64 GNU/Linux

(Just for reference, I've confirmed multiple times including with the socketcan C based canplayer/candump executables that my vcan socket is definitely present and up and functional).

Which version of Go are using?

I'll spend some more time looking into it and fire over a PR if I get it
sorted, thanks for responding.
On Fri, 28 Sep 2018, 11:54 Matthias<EMAIL_ADDRESS>wrote:

Maybe we have to change the binding to this
https://github.com/golang/sys/blob/master/unix/syscall_linux.go#L541-L546
.
(Currently I don't have time to look into it.)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/brutella/can/issues/6#issuecomment-425399534, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA2KuZZ_68eOeer3EsHaFn3S3MXkkvPEks5uff_dgaJpZM4WvTyB
.

I've added a possible fix in the fix-linux branch.


Support changing the active stack frame

See https://github.com/nodejs/node/issues/14049

From the original thread:

There's no official/documented way of doing it right now. Some things that work but definitely need some cleanup and an official interface:
debug> bt[2].list() // list source context for frame at index 2
debug> bt[2].loadScopes() // show the scope chain w/ values for the frame #2

Things that do not exist in any form yet: Eval on a specific stack frame or changing the selectedFrame.

Does the original node debug support changing stack frames? If so we should implement it in a way that's close to it.

Afaict the previous debugger didn't support switching frames either (currentFrame couldn't be modified). So the interface is pretty open.


Cant require a library, have tried .require_script and , .require_file, and tried hard coding.

Here is my provided code
C++
#include <sol/sol.hpp>
#include <iostream>
#include <string>

int main()
{
        sol::state lua;
        lua.open_libraries(sol::lib::base, sol::lib::package);
        std::string x = lua["package"]["path"];
        lua["package"]["path"] = x + ";/usr/local/share/lua/5.1/LuaMidi.lua";
        lua.script_file("test.lua");
}

Lua
local lm = require('LuaMidi')
local T = lm.Track
local ne = lm.NoteEvent
local w = lm.Writer

local track = T.new()
local notes = {'C3','D3','E3','F3','G3','A3','B3','C4'}

track:add_events(ne.new({pitch =notes,sequential=true}))
local wr = w.new(track)
wr:save_MIDI('C Major Scale')

the above c++ code is my most recent attempt at running this file. both lua and luajit compile it just fine.
I have tried
lua.require_script("lm","require(\"LuaMidi\")",true);
lua.require_file("lm","/path/to/lib/file",true)

lua.script(R"(
local lm = require('LuaMidi')
local T = lm.Track
local ne = lm.NoteEvent
local w = lm.Writer

local track = T.new()
local notes = {'C3','D3','E3','F3','G3','A3','B3','C4'}

track:add_events(ne.new({pitch =notes,sequential=true}))
local wr = w.new(track)
wr:save_MIDI('C Major Scale')

)")

the error I get is
[sol2] An exception occurred: caught (...) exception
[sol2] An exception occurred: caught (...) exception
[sol2] An exception occurred: caught (...) exception
[sol2] An exception occurred: caught (...) exception
[sol2] An error occurred and panic has been invoked: caught (...) exception
terminate called after throwing an instance of 'sol::error'
  what():  lua: error: caught (...) exception
Aborted (core dumped)

I dont really know what else to do.

package.path should probably have ;/usr/local/share/lua/5.1/?.lua appended to it.
and then you would use require('LuaMidi') where LuaMidi is the file name that replaces the ? in the package.path.
The luajit errors are weird. I get those too. For some reason the real error is lost and we only see caught(...) which is probably not what should be happening.

Am I not appending that? In my c++ code I have the append to the package
std::string x = lua["package"]["path"];
        lua["package"]["path"] = x + ";/usr/local/share/lua/5.1/LuaMidi.lua";

You are appending
;/usr/local/share/lua/5.1/LuaMidi.lua
when I think you should be appending
;/usr/local/share/lua/5.1/?.lua

Tried that ,still throwing the same error.

I guess I should also.emphasize.
The lua compilers run the lua file just fine, without having to add it to the package.path. I dont know if thats relevant or not, but it does compile through Lua and Luajit

require_script and require_file shouldn't have more "requires" in them: it should have the actual script you're attempting to run and place in the "library" global named "lm", in your case.
I am no expert in handling these kinds of errors. It will be a while as I set up a decent test case and try to get this to work. As a side note: where are you storing LuaMidi.lua?

As in: is this in the Working Directory of the application? Is it actually at /usr/local/share/... ?

Yes its actually in that location. Its LuaMidi downloaded through luarocks.
And the scripts provided is the actual scripts

Wait. That is the working directory of the midi library installed through luarocks, not the working directory of the application I am developing.

https://github.com/ThePhD/sol2/blob/72181513d5459c4209f567fa4b36228e51709216/examples/source/require_override_behavior.cpp

Side note: this requires sol3. However, you can mock up the behavior yourself, with examples as seen in #692!


Fix documentation about ListProjectIssues method

Current master has wrong docs(ListIssues instead of ListProjectIssues)

Thanks! LGTM!


Add a way to access whole fetch response on calls

Currently all response info except body is stripped during parsing process. Add a way to access this metadata about queries and requests for monitoring and tracing purposes.

Hey @PSoltes ,
Thanks for the raising issue, you can trace request, response and the error from the SDK by setting the debug to true as follow:
const stack = Contentstack.Stack({ api_key: 'api_key', delivery_token: 'delivery_token', environment: 'environment', fetchOptions: {
   debug: true
}
});

This will enable logs from Contentstack JS SDK.
If you want to track logs you can use logHandler as follow:
const stack = Contentstack.Stack({ api_key: 'api_key', delivery_token: 'delivery_token', environment: 'environment', fetchOptions: {
   debug: true,
   logHandler: (level, data) => {

   }
}
});

In this level is string value for log type and data will contain the details for the log.

Seems to be working as expected. Closing. Thanks again :muscle:


Use a table instead of two arrays to hold pairs

This simplifies configuration, prevents mismatches and simplifies the
code as well.

Thank you

Thank you


LT: test_cose_json / test_cbor_json failure

File: LT/2DCode/raw/2.json
tests/test_ehealth_certs.py:310: in test_cose_json
    assert any([_ordered(hcert) == _ordered(config_env[JSON]),
E   assert False
E    +  where False = any([False, False])

tests/test_ehealth_certs.py:328: in test_cbor_json
    assert any([_ordered(cbor_object) == _ordered(config_env[JSON]),
E   assert False
E    +  where False = any([False, False])

@daniel-eder , This ticket could be closed now. I have fixed test script to ignore microsecond value while comparing COSE , CBOR and JSON payload.


Add additional tracking events for assessment flow v0

Description
This PR adds some additional tracking events for the profile tab. The reason for this is that we're currently unsure if the current "profile screen shown" figure is correct.
On what platform have you tested the change?

[ ] Android - Emulator
[ ] Android - Physical device
[x] iOS - Emulator
[ ] iOS - Physical device

Out of scope and potential follow-up
Screenshots
Checklist (for submission)

[ ] Analytics/tracking events have been added (if relevant)
[ ] Added new screen(s) to src/features/screens.ts and new Redux slice(s) to mocks/mockedInitialState.ts (if relevant)

Checklist (for reviewers)

[ ] Checked against Figma designs (if relevant)
[ ] Checked that data has been successfully saved to the backend (if relevant)

Checklist (for merger)

[ ] Make sure the right merge strategy is chosen, commit merge into master and squash merge into develop.

Analytics changes look good to me.
Just to check, are the changes in src/features/dashboard/DashboardUKScreen.tsx intentional, as they don't appear to be related to analytics?

Analytics changes look good to me.
Just to check, are the changes in src/features/dashboard/DashboardUKScreen.tsx intentional, as they don't appear to be related to analytics?

Thanks for checking - yes it's intentional. Basically noticed a bug where overlay for Android was not working as expected, thus affecting analytics tracking.


"webpack#ajv" not installed after update to webpack 3.1.0

Do you want to request a feature or report a bug?
Report a bug.

What is the current behavior?
yarn check results in the following error:
error "ajv" is wrong version: expected "4.11.7", got "5.2.1"
error "webpack#ajv" not installed
error Found 2 errors.

If the current behavior is a bug, please provide the steps to reproduce.

Update to webpack 3.1.0.
Run yarn check.

What is the expected behavior?
No errors.
If this is a feature request, what is motivation or use case for changing the behavior?
Please mention other relevant information such as the browser version, Node.js version, webpack version and Operating System.
Node.js version: 8.1.0
wepack version: 3.1.0

How should webpack influence this? Isn't this a yarn bug if it incorrectly install ajv?

@sokra I wasn't sure if this is related to yarn or not since this error didn't occur with webpack < 3.0.0. I'm not sure but I think this error is caused because yarn didn't install ajv not the opposite.

It's probably related to the package manager. I have seen the same issue... even with ajv by using<EMAIL_ADDRESS>

The problem here is not really a yarn bug, but rather that webpack's dependency on<EMAIL_ADDRESS>is incompatible with one of webpack's transitive dependencies which depends on<EMAIL_ADDRESS>The only for webpack to attempt to solve this issue would be to downgrade to<EMAIL_ADDRESS>but I don't think that's a very good option (other packages, e.g<EMAIL_ADDRESS>will pull in<EMAIL_ADDRESS>anyway, so this will crop up elsewhere).
The resolved dependency tree for my package has:
<EMAIL_ADDRESS>-><EMAIL_ADDRESS>-><EMAIL_ADDRESS>-><EMAIL_ADDRESS>-><EMAIL_ADDRESS>-><EMAIL_ADDRESS>-><EMAIL_ADDRESS>-><EMAIL_ADDRESS>
Working backwards,

<EMAIL_ADDRESS>bumps to<EMAIL_ADDRESS>request/request@baf9c1f bumps to<EMAIL_ADDRESS><EMAIL_ADDRESS>depends on<EMAIL_ADDRESS>
So all that needs to happen is a minor or patch release of request and all will be well again*
* assuming no other dependencies in your package pull in<EMAIL_ADDRESS>e.g. for me<EMAIL_ADDRESS>is still going to be a problem 😞

The problem here is not really a yarn bug, but rather that webpack's dependency on<EMAIL_ADDRESS>is incompatible with one of webpack's transitive dependencies which depends on<EMAIL_ADDRESS>
The package manager should be able to solve this. I. e. it could create a tree like this:
node_modules
  ajv@5
  webpack
  watchpack
  chokidar
  fsevents
  node-pre-gyp
  request
  har-validator
    node_modules
      ajv@4

Ah, yeah, you're quite right, yarn does actually do that<EMAIL_ADDRESS>gets installed to node_modules/har-validator/node_modules/ajv. My bad.
I guess yarn check is doing the wrong thing here then?

I've raised this with yarn, installing webpack on its own does not cause this error, it seems to require another package to have a transitive dependency on ajv@4; weird!

Great


Source of high resolution test samples

Hi, thanks for your excellent work.
I've read your paper and I'm really impressed by the results in your paper. But I am having some question when trying to test the model. I would appreciate it if you could provide some help.
In the experiment part, it's mentioned that the model is tested in several different resolutions, from 512512 to 8K. However, when I was testing the model, I found that the images in Places2 dataset are at most 512 pixels, and there seems to have no ground truth image whose size is really larger than 512. If the high resolution samples are produced by resizing the 512512 images, it seems that the evaluations on high resolution images are actually equivalent to that on 512*512 because each part of the model is essentially working on the 512 bases (including the ground truth). Am I misunderstanding something or there is another source of private testing samples?
Thanks for any help in advance :).

For our model HiFill, the test results do not vary much by the size of raw input images. However, for other methods, it matters a lot.
Please find more detils here:
https://github.com/duxingren14/Hifill-tensorflow


fix(compiler-sfc): handle keyof operator with index object

fixes https://github.com/vuejs/core/issues/11580

Vue doesn't support the case where the prop key is a symbol, so I didn't bother fixing the current logic which will return String in this case
i.e
const sym = Symbol('uniqueKey');

const props = defineProps<{
  [sym]: string;
}>()

Error: [@vue/compiler-sfc] Unsupported computed key in type referenced by a macro


Improve git.Repo compatibility when run from subdirectories

This improves compatibility of the git repo initialisation to traverse parent directories looking for the .git directory.
Initialising with a getcwd is not necessary as the library automatically assumes cwd.

@BradWhittington Thanks for your contribution. I'll have to fix up the tests workflow for pull requests from outside sources to either skip the CodeCov step altogether or allow the workflow to use the CODECOV_TOKEN secret once it has been approved.
A more complete solution would probably be to extend ls_files with a base directory parameter, we should be able to get the base directory from the pytest-cov plugin (that should match cwd by default, but can be explicitly set using --cov=dir or one of the coverage config files), we can still traverse parent directories looking for the .git folder, but then we should only return the git tree from that subdirectory if it was specified.
That would also improve behavior for operation outside a git repository. If you feel like taking a crack at this that would be great. Otherwise this small patch is probably fine on its own, but doesn't improve behavior in a significant way where I would bother creating a new release, before also implementing the aforementioned improvement.

Unfortunately don't have the time now to make a more complete solution... If I do I'll open as a seperate pull. If you're happy to merge this for now, and then it can go out with whatever the next point release is...
Bumped into the issue because our code is all in /src/ and we run pytest in that dir vs. repo root, for now I'm just installing this patch branch directly.

@BradWhittington Alright that's fair. The workflow run for pull requests should be fixed now. Please merge the changes into your feature branch so the checks can be run again without the codecov related failure.


perfect hashes

Create perfect hashes for some of our lists.
This needs support from some not yet existing external tool to create PH/MPHF for integer keys:

[x] nbperf (in work) https://github.com/rurban/nbperf/issues/3
[ ] gperf (in work) https://gitlab.com/rurban/gperf/-/issues/6
[ ] Postgresql PerfectIntHash.pm (in work) https://github.com/rurban/Perfect-Hash/blob/master/lib/Perfect/PerfectHashInt.pm

my nbperf has now proper intkeys support. doing perf tests now.

It is not better, as it needs too much space for each codepoint in the lists, compared to the sorted ranges


ol.source.TileArcGISRest with custom CRS

It seems that ol.layer.Tile with source new ol.source.TileArcGISRest() ignores "projection" parameter. It uses EPSG:3857 instead. Am I wrong?
My code:
new ol.layer.Tile({
	extent: [306520.146, 5973291.050, 680103.277, 6257813.452],
	source: new ol.source.TileArcGISRest({
		url: "http://www.domain.com/some_url/MapServer/",
		params: {
			FORMAT: "png8",
			DPI: 96
		},
		projection: ol.proj.get("EPSG:3346")
	})
})

By the way, I can't set custom DPI also! Documentation doesn't mention I can't do that!

Tiles will be requested in the view's projection.

DPI parameter will be set automatically so it matches the pixelRatio used internally: https://github.com/openlayers/ol3/blob/master/src/ol/source/tilearcgisrestsource.js#L115
So you would want to set the value of 90 and have it multiplied with the device's pixelRatio?

Thanks! Setting projection in map's view solved the issue!
I just thought it would be possible to set custom DPI like in ArcGIS JS API.

Sure I think that is a valid enhancement request to be able to set the DPI. I'll update the title of this ticket to reflect that.


[Connector] Add file metadata fetch support to msazurestorage connector

Problem
We have introduced download blob support to msazurestorage via [1].
While we are doing the feature, we noticed that we can also retrieve file metadata details.
We need a new connector operation to fetch these metadata details
[1] https://github.com/wso2-extensions/esb-connector-msazurestorage/pull/15
Solution
Add new operation to fetch blob metadata
Affected Component
MI
Version
No response
Implementation
No response
Related Issues
No response
Suggested Labels
No response

Status:
Local test
Thanks

Done. Changes in the code and the documentation sent.
I'll be waiting for feedback on this,
Thanks!


Cannot download images

Im trying to translate manhua by putting this (https://rawkuma.com/) site link but its say cannot download images.plss fix this!
Screenshot_2022-11-01-03-04-04-344_com.wiryaimd.mangatranslator.jpg

Hey, it looks like you're in the wrong place. The manga translator app that you showed in your screenshot is not mine. ☹


Custom Fee Rate in sendToOnChainAddress Method?

Currently, in the sendToOnChainAddress method, it is not possible to specify a custom fee rate when sending Bitcoin.

@BitcoinZavior I will like to takle this.


[Toggle] Unknown props thumbStyleSwitched, trackStyleSwitched on

Getting console error: Unknown props thumbStyleSwitched, trackStyleSwitched on  when setting these props on Toggle component.
<Toggle
     label="Calculated Risk?"
     toggled={props.risk}
     onToggle={(evt, isInputChecked) => props.onChange({risk: isInputChecked}) }
     thumbStyle={{backgroundColor: '#ECEFF1'}}
     trackStyle={{backgroundColor: '#B0BEC4'}}
     thumbStyleSwitched={{backgroundColor: '#B0BEC4'}}
     trackStyleSwitched={{backgroundColor: '#465A64'}}

Versions

Material-UI: 0.18.3
React: 15.5.4
Browser: Mac OSX Chrome Version 58.0.3029.110 (64-bit)

trace:
Warning: Unknown props `thumbStyleSwitched`, `trackStyleSwitched` on <input> tag. Remove these props from the element. For details, see https://fb.me/react-unknown-prop
    in input (created by EnhancedSwitch)
    in div (created by EnhancedSwitch)
    in EnhancedSwitch (created by Toggle)
    in Toggle (at LayerDrawer.js:141)
    in div (created by CardText)
    in CardText (at LayerDrawer.js:140)
    in div (created by Card)
    in div (created by Paper)
    in Paper (created by Card)
    in Card (at LayerDrawer.js:125)
    in div (created by Paper)
    in Paper (created by Drawer)
    in div (created by Drawer)
    in Drawer (at LayerDrawer.js:24)
    in Unknown (at index.js:88)
    in div (at index.js:77)
    in MuiThemeProvider (at index.js:76)
    in Tool (at App.js:25)
    in Route (at App.js:21)
    in div (at App.js:19)
    in Router (created by BrowserRouter)
    in BrowserRouter (at App.js:18)
    in App (at index.js:6)

We have been porting the component on the v1-beta branch. We reimplemented it from the ground-up. While we haven't tested it, I think that the issue is most likely fixed on that branch. Hence, I'm closing it.
Still, we will accept PR fixes until v1-beta takes over the master branch.


Wont Compile

Describe your problem here.
I try to compile it with no edits to the script FunkinLua or any script but it fails to compile I attached a screenshot

Please Help
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
no

haxelib git linc_luajit https://github.com/nebulazorua/linc_luajit

Thanks it works now


fixed Issue #1443

also added endpoint
get-lists-ownerships

Since this is the last PR I want to include for v3.10, I've gone ahead and removed the extraneous changes myself.
Feel free to open separate PRs for them.


DataFlowGraphModel.hpp , DataFlowGraphModel.cpp , UndoCommands.cpp

1:When you add a connection, you make security judgments about the data

1: Added topology connection checks to prevent cyclic connections.
2: When calling addConnection, data consistency must be verified, and it should be checked if a connection already exists (since another workflow might impact synchronization with the current workflow).


Ollama is unable to resume interrupted pulls

What is the issue?
Previously if I interrupted a pull and started it again (Usually after the speed dropped or wouldn't go to completion after 95%), it was able to resume.
However today when I pulled the gemma2:2b model, it was unable to do so and started from scratch. This happened multiple times.
Not sure why this could be the case. I am on Windows 11, with the latest version.
OS
Windows
GPU
Nvidia
CPU
No response
Ollama version
0.3.1

this is fixed by #6108 which will be in the next release


Anyone got a RGBW downlight brand that consistently flashes?

Zemismart was flashing up until recently, now they're not.  I've tried three other brands on Amazon, and they're all blocking the flash.  Anyone have any recommendations at this point?  Tearing these beasts apart requires nearly destroying them, and I'd rather find something that works with tuya-convert.

I was looking too, didn't find any.  I resigned myself to buying the available units and if necessary swapping in ESP8266's.  So, I placed an order and received a half dozen (so far) from China from an Ali vendor named FrankEver.  They arrived with a Beken BK7231T on board, so a swap it will likely be.  I'd love to figure out building firmware and flashing these if I could ever find the time, because they look like a great little chip.  ARM based, 32 bit, fast, lots of features.  The lights themselves are nice, the frosted lens pops right out with a suction cup, and the high voltage side sits under a removable cap on the back.  Very easy to get into.

I was looking too, didn't find any. I resigned myself to buying the available units and if necessary swapping in ESP8266's. So, I placed an order and received a half dozen (so far) from China from an Ali vendor named FrankEver. They arrived with a Beken BK7231T on board, so a swap it will likely be. I'd love to figure out building firmware and flashing these if I could ever find the time, because they look like a great little chip. ARM based, 32 bit, fast, lots of features. The lights themselves are nice, the frosted lens pops right out with a suction cup, and the high voltage side sits under a removable cap on the back. Very easy to get into.

Just going to quickly chime in here! Preface: I work for a wholesale company and we got samples of this product ordered. Not sure if this is on aliexpress.
I've de soldered with a heat gun and flashed this Ssluce light. Probably took 15/20 minutes of time but I was able to successfully convert it with used the Deta  template. If i recall correctly I think only the TX pin was missing hence me having to desolder the TYWE2L chip.
I'm very happy I was able to get a downlight with Tasmota and might look into ordering more in the future. Currently though my a fan and waiting for development on any Wifi/Bluetooth Mesh (light to light) chips that come out, similar to Yeelight with their Bluetooth Gateway + M2 Downlight.
https://cdn.discordapp.com/attachments/530862923363516417/817840106823090216/image0.jpg
https://cdn.discordapp.com/attachments/530862923363516417/816721674413801512/image0.jpg

I pulled the https://github.com/bekencorp/bk7231_freertos_sdk from BekenCorp, but they utilized the commercial REVA RivieraWaves WiFi IP so I can't compile it.  So unfortunately it looks like a dead end for this SoC.
So, I popped the SoC off the board and will strain my eyes to their max stitching in some ESP8266 ESP-12 modules I've got laying around.  I'll be giving up hardware PWM which is a bummer, but they should work for my purposes.


Aimbot on key doesnt work

I toggled on key but it just didnt work. I dont have the knowledge to find out myself how to fix it so I am hoping someone here does have the required knowledge! :D

Bind key and select appropriate mode (hold/toggle).

Do I need to reselect the mode I wanna use? Like I had it on hold before using the bind do I need to switch to toggle back to hold for it to work? (Can’t test it right now)

Bind key and select appropriate mode (hold/toggle).

Hello,maybe move appropriate mode (hold/toggle) next to enable checkbox will make ui better

Bind key and select appropriate mode (hold/toggle).

ok so it only works when aimlock is enabled.. is it supposed to be like this ?

Bind key and select appropriate mode (hold/toggle).

ok so it only works when aimlock is enabled.. is it supposed to be like this ?

Yes, you need to have the checkbox enabled in order to use the feature. Just setup your bind and you are good to go.
kind regards 😃 👍

It works but you must have Max angle delta >0


Git: warning: LF will be replaced by CRLF in .vscode/launch.json.

VSCode Version:
OS Version:

Steps to Reproduce:

Reproduces without extensions: Yes/No

on git  commit error stopping all changes beign commited

Please configure your git not to do that. https://git-scm.com/docs/git-config#git-config-coresafecrlf


Introduce TrajectoryBuilderInterface.

RFC=0004

@wally-the-cartographer merge

Merge requested by authorized user cschuet. Merge queue now has a length of 1.


optimize file based sources

For files manually compiled, consider downloading from source where available (vs manually compiling and storing on s3).
Where manual compilation is still required:

include just the layer of interest in the file
standardize format

Looking at the first candidate (migratory bird sancuaries / national wildlife area), it doesn't seem like ogr can reach the .gdb via vsicurl - curl needs the -L flag to follow redirects and download successfully:
curl -v -L -O "https://data-donnees.az.ec.gc.ca/api/file?path=%2Fspecies%2Fprotectrestore%2Fcanadian-protected-conserved-areas-database%2FDatabases%2FProtectedConservedArea_2022.gdb.zip"


Replace TripleO images by TCIB images

We imported container build tools from TripleO as TCIB[1]. Now we are retiring TripleO and should complete the migration.
[1] https://github.com/openstack-k8s-operators/tcib

@kajinamit I rebased this PR to the webhook work that moved the default ContainerImage to an env variable. Could you please check that this look OK from your perspective?

should we just swap to antelope at this point?
we will need to do that sonner rater then later.
the images already exist and i would prefer to avoid changing these twice in a few weeks
https://quay.io/organization/podified-antelope-centos9

@kajinamit I rebased this PR to the webhook work that moved the default ContainerImage to an env variable. Could you please check that this look OK from your perspective?

Thanks. It looks good to me. I didn't push these replacement patches because kuttl tests fail unless we remove hard-code in install_yamls but it seems placement-operator is not affected because kuttl tests are not yet added so we are good.

should we just swap to antelope at this point? we will need to do that sonner rater then later.
the images already exist and i would prefer to avoid changing these twice in a few weeks https://quay.io/organization/podified-antelope-centos9

I think we can skip zed and wait for antelope. afaik rdo antelope is still being prepared so we probably have wait for another week to get the new images built with antelope packages but I'll double check it.

I'm OK to wait for the proper antelope packages.

I confirmed these podified-antelope-centos9 images are built from RDO antelope packages so we can use these now.

I was able to run the antelope placement now:
$ openstack trait list | grep COMPUTE_SHARE_LOCAL_FS
| COMPUTE_SHARE_LOCAL_FS                |


Allow serialization of literal strings using single quotes

Replace the italic text below with the relevant information to help me understand your request.
Is your feature request related to a problem? Please describe.
Currently it is not possible to serialize say a system path such as C:\Path\To\Some\File.txt into literal TOML string. Instead, the strings are escaped and put in double quotes which makes them hard for humans to read and edit which is countering the main advantage of using TOML in the first place.
Describe the solution you'd like
Make it possible to output raw, unescaped, strings with single quotes.
Describe alternatives you've considered
I don't see any alternative to supporting one of the crucial TOML features.
Additional context
N/A

This is implemented now in master. The new behavior doesn't require any additional work on your part, the library will automatically try to use the 'most literal' form of string it can to emit the output in a round-trippable way. As long as your strings don't contain non-whitespace control characters they should use the literal = 'form'.

CC @bobfang1992 - you may want this!

@marzer Thanks for implementing this. It works just fine for my purpose (storing paths and USB device identifiers):
Camera = '\\?\usb#vid_0603&pid_8122&mi_00#7&1316d950&0&0000#{65e8773d-8f56-11d0-a3b9-00a0c9223196}\global'
CapturePath = 'C:\Camera'


lab needs better error messages for duplicate approvals and unapprovals

[prarit@prarit rhel-7]$ lab mr approve 119
Merge Request !119 approved
[prarit@prarit rhel-7]$ lab mr approve 119
2021/05/07 10:34:35 mr_approve.go:55: POST https://gitlab.com/api/v4/projects/24140364/merge_requests/119/approve: 401 {message: 401 Unauthorized}
[prarit@prarit rhel-7]$ lab mr unapprove 119
Merge Request !119 unapproved
[prarit@prarit rhel-7]$ lab mr unapprove 119
2021/05/07 10:35:01 mr_unapprove.go:55: POST https://gitlab.com/api/v4/projects/24140364/merge_requests/119/unapprove: 404 {message: 404 Not Found}
[prarit@prarit rhel-7]$ lab mr approve 119
Merge Request !119 approved
I'll work on this.  Just opening up the issue so I don't forget about it.

Resolved by bef39e0eea40 ("approve/unapprove: Provide better error messages for repeat actions")


what the sigpanic is used for ?

Here is the code used in Caller() func:
	c.pc = pcs[1]
	if runtime.FuncForPC(pcs[0]) != sigpanic {
		c.pc--
	}

Could you please explain why the program counter is reduced by one ?

Found the answer in git history.
For those who may interested see https://github.com/golang/go/issues/7690
and note that the issue is tackled today by runtime.Frames

In short we must subtract one from the PC in most cases because runtime.Callers returns PCs for where each stack frame would return to rather than where it was called from.
Go 1.7 introduced the runtime.CallersFrames function which takes care of that for us automatically. I would like to update this package to use the new API sometime soon, but it hasn't happened yet.
The need to check for sigpanic is an inner detail of the Go runtime that was not documented until Go 1.4. Those docs were removed again with the improved support of the new API in Go 1.7.
From Go 1.4 until Go 1.7 the documentation for runtime.Callers had more detail about these issues. The older docs can be seen in the commit where the extra detail was added, https://github.com/golang/go/commit/8db71d4ee89a505c375b550eb8fb8cc33bbabc03. The relevant portion of the docs said:

Note that since each slice entry pc[i] is a return program counter, looking up the file and line for pc[i] (for example, using (*Func).FileLine) will return the file and line number of the instruction immediately following the call. To look up the file and line number of the call itself, use pc[i]-1. As an exception to this rule, if pc[i-1] corresponds to the function runtime.sigpanic, then pc[i] is the program counter of a faulting instruction and should be used without any subtraction.

Unfortunately runtime.sigpanic is not exported, so this package had to capture it's *runtime.Func with an ugly hack, as seen here: https://github.com/go-stack/stack/blob/master/stack.go#L208.


Ctrl + p is not searching inside vendor folder in a laravel project in the vscode 1.19.1

VSCode Version: 1.19.1
OS Version: linux Ubuntu 17.04

Steps to Reproduce:

Press Ctrl+p and try do find a file that is inside vendor folder in a laravel project

In the vscode version 1.18.* the Ctrl+p command works normally. But in the version 1.19 it is not working to find files inside vendor folder.
How can I fix it? Are there any configuration that I need todo?

Reproduces without extensions: Yes

"search.useIgnoreFiles": false,// ?

Is it in a gitignore file? If so you can tell vscode to not look at gitignore files by setting "search.useIgnoreFiles": false like @usernamehw says.

That's work for me!
Thanks!!


WIP Status

Related to https://github.com/aurae-runtime/auraed/pull/13 and https://github.com/aurae-runtime/aurae/pull/13


[WIP] Started to port F# List to Fez!! Re-implemented some functions …

I have started to take on the task of https://github.com/kjnilsson/fez/issues/11.
I have also re-implemented some functions to have better performance than what was given from erlang lists module or at least I hope that's what I did 😂 .
I have exported all the functions so far but I will clean up what is exported and what is not.
It is in WIP, and thus needs multiple things before being ready to merge :

Implement all functions left from FSharp list.
Export only necessary functions
Clean Documentation/ Comments
Add a battery of test

Hi Fahd,
I think distinct in f# preserves the order, so can't be sorted.
On my fsharpi:

List.distinct [1;2;4;3;3;4;1];;
val it : int list = [1; 2; 4; 3]

Fantastic. Not sure I follow the reasoning about re-implementing functions present in erlangs lists module but don't really mind. Have you measured the performance?

I have made some performance measures in which I can see some difference of performance.
I use timer:tc function to do so.
I will create a module where you could see the performance of each functions, on a battery of predefined list, and see their counterpart in erlang.

we should also create expecto test project for this.

This is what I was thinking but would it be done for testing the erlang part ?
If so I think that would be neat !! 😄

on fable we compile the F# tests to javascript.
So what we need is a nice erlang test framework to compile to ;-)

can you please add a sample of how we would use it?

I don't know anything about erlang by the way 😂  so I will look for erlang test framework and provide a sample later tonight :slightly_smiling_face:

I'm not convinced we need to add much in terms of testing frameworks atm. The current approach of compiling one or more sample files then testing them using eunit in erlang works and also covers using the output from erlang itself. What I think we need need is an easier way to run the eunit tests from windows. Currently I'm using erlang.mk which would work well from cygwin, git bash, msys2 etc but probably not from PS or cmd.
For testing erlang code directly you can either add tests to the bottom of the module within a
-ifdef(TEST).
% tests go here
-endif.

or just add an `eunit` test module in the tests directory.

http://erlang.org/doc/apps/eunit/chapter.html#Running_EUnit

Naturally if you want to add direct tests of the compiler code we'd need a test runner of some kind.

I've added run-tests.cmd which shows you how to make a basic eunit test runner for the basics_tests test module.

Some functions are left to implement. Implemented so far (60/67). I do not know how to implement what is left though.

I have added comments from Msdn Documentation. I do not know if it is worth the case to do that. @kjnilsson If you think that it is not usefull, then I'll remove them 🙂

I don't mind the comments being there but I am not sure what function they would have as you wouldn't benefit unless you used this module from erlang directly.

Yes You are right they do not make sense in fact 🙂
I thought that having the type signature for the function would be nice for anyone who will want to make some change to these functions in the future, but In fact it doesn't make sense at all as they will be tests for them.

feel free to add type specs though.

Thanks!


Calculation error in maximum spanning tree

When applying your codebase to my works, I discovered a problem within glomap/math/tree.cc. I think there's something wrong with this MaximumSpanningTree (mst) function.
Specifically, I gave this function a rather small input - simply 10 images - and printed all the information about the graph:
edge: 7 9 65
edge: 7 8 233
edge: 9 10 154
edge: 2 4 103
edge: 2 3 134
edge: 3 7 38
edge: 5 6 311
edge: 3 6 45
edge: 3 5 95
edge: 1 5 41
edge: 3 4 268
edge: 1 4 41
edge: 1 3 138
edge: 4 6 79
edge: 1 2 314
edge: 6 7 38
edge: 5 7 93
edge: 6 8 52

These are the edges with their nodes and weight. And the result I get:
Total weight: 1247
9 10
8 6
7 9
6 5
5 7
4 6
3 1
2 1
1 4

I summed these weights up and can tell this is a legal spanning tree with correct total weight. However, I used other code to calculate the mst of the same problem, and after my verification, this is not the mst for the problem, as the following one is better:
Total weight: 1671
9 10
8 7
7 9
6 5
5 7
4 3
3 5
2 1
1 3

After my exploration I suppose that the problem originates from boost library, as its implementation may not support negative edge weights, so I changed the calculation of weights from weights_boost[e] = -image_pair.inliers.size(); to weights_boost[e] = 1e6-image_pair.inliers.size(); to avoid negative weights, then your code can calculate correct results too. Of course this is an easy fix, and I don't know the actual effect of this mathematical bug, just think it should work as it is supposed to do.

I've made a pull request here: https://github.com/colmap/glomap/pull/90#issue-2499002505. In my implementation I find out the biggest weight and add that value to all the weights, so that all weights should be >= 0.

Thanks for the contribution.


Live Query when using express routing

When I update data using express routing as described here, I do not receive these changes with a live query
this is my route in express:
app.get('/Building', remultApi.withRemult, async (req, res) => {
    try {
        const repo = remult.repo(Deliveries)
        await repo.update(req.query.packIdReq , {status: "OK"})
        res.send({message: "success"})
    } catch {
        res.send({message: "error"})
    }
})

remult version: 0.23.4

What did I miss?

Hi @yossef7525,
I've tested and it works - here's a code sandbox that demo's that:
https://codesandbox.io/p/devbox/live-rghmq7

Works for me, my mistake was redeclaring "remultExpress" in both api.ts file, and index.ts file


Updating associated record inline doesn't work

If I have a class Parent that has many instance of Child, and a child is already in the parent.children collection, then you can edit that child inline. The editor opens correctly, but when you hit save, instead of JSON coming back like ra.remote-form.js line 88 expects, HTML (the child form re-rendered--but without the submitted changes) is the response. This causes a JS exception to thrown thrown, and also the child object is not updated in the db.
Note: I verified that this is not caused by my recent patch to not force XHTML in HAML.

Can't reproduce with dummy_app. Can you investigate a bit more?

Can you try latest head?
I did find a race condition in remote-form that affected chrome.

I just tried latest head with the same results. Watching the request in Chrome's web inspector, I see the "Accept" header is set to "text/javascript" but the response from the server contains the re-rendered HTML form (and without the form values updated.)

I don't have assets precompiled in development anyway. My version of jquery-rails is at 2.0.1. And I just updated to Rails 3.2 still have the same issue. HTML is the response rather than JSON.