Posssible mesh-damage bug.

This is caused by something I'm doing regarding order of operations, but I haven't found it yet.
Assets are being loaded, but by new code which may possibly be deleting a Mesh while it is in use. Or something.
The observed behavior is that meshes and textures are being loaded successfully, and then, suddenly, many meshes already loaded into the GPU have their triangle connections trashed. See the before and after pictures below. This happens all at once.
This is repeatable.
Now, I'm doing a different order of mesh, material, and object loading than usual, so that's probably related. But it looks like something trashed the triangle vertex list, something I should not be able to do from safe code. Has this been seen before?

What version is this?

2.0, the released version.

0.2.3 has a patch which fixes an issue with meshes being improperly invalidated, so make sure you have the most recent patch. If it's still a problem with that version, I haven't seen that yet.

Tried rend3 0.2.3, same results.
(rend3-pbr is still 0.2.2; there is no 0.2.3 yet.)
I'm still not clear on what sets this off. It's not material loading; even with everything loaded as plain grey, it still happens. I'm probably replacing an existing object, but I haven't yet located the event which sets this off. When this happens varies from run to run.

Interesting, definitely haven't seen this yet

Working on narrowing it down. Looks like one specific object is causing a problem.

Well, I can reproduce the problem consistently. I can load objects 0..597 with no trouble, and then on object #598, the displayed meshes all turn to junk as shown above. Always the same junk. I can dump object #598, and it has nothing special about it. I'm validating all the meshes (vertex count == normal count == uv count, tri count % 3 == 0, vertices in tri list in range, no dup vertices in a triangle), and all meshes validate OK.
If I quit loading just before object #598, the scene looks fine.
If I just skip object #598, instead of stopping, the problem occurs on some later object.
If I go back to an old version of my program, before I changed some of the order of events, it all works fine.
GPU memory usage is only 9% of 32GB at fail, so this is not an out of memory condition.
I need to look at this more. Any ideas?

I can confirm that I have been having this issue as well.

I think I see what might be happening.

See pictures above. The umbrellas are drawn correctly at first, and then suddenly the whole scene is garbled. Garbled in a very specific way. It looks like the vertices and UVs are fine, but the triangle indices were altered in a regular but wrong way. Such as being shifted by one, resulting in triangles formed by using vertices from adjacent triangles.
This suggests a bug in the memory management of vertex index arrays. As I understand it, it is customary at the Vulkan level to combine multiple objects into arrays. So arrays which grow have to be managed. That means resizing, growing, sometimes recopying when there's no room to grow in place, and possibly compacting after deletions. I would suggest looking for an off by one error in that memory management code. Thanks.

I think I spotted one possible issue -- if a mesh gets deleted, and then the index buffer is resized at all, some objects may get pointed into the wrong indices. I'll try to give this a fix soon. Do you think this could be the problem?

That sounds very likely. Some of my changes mean meshes are being deleted for the first time.

Oops, didn't mean to close this, this should be fixed on trunk, but going to hold this open until we validate it's fixed.

OK. This problem has me stalled, because I was in the middle of a big code reorganization when I discovered it. Should I try to use the current trunk, or wait a bit? If I use trunk, there are major API changes not yet well documented, and I'm not sure if the API has stabilized yet, So I'd been waiting for a new crates.io version. But I have to have a fix for this to go forward. What do you suggest? A crates.io version that just has this mesh damage fix would be nice.

I can try to backport this once I get word that it does indeed fix the problem. 0.3 is very close to release, but you shouldn't be forced to upgrade to get a bug fix like this, so I'm fine with backporting.

Thanks. I'll have to convert to 0.3 eventually, so once that's out I will try to use it.
Right now, I've created a branch in my own code so I can work with the Github version, and am struggling with making that compile.

Appears to be fixed in the version in the repository: https://github.com/BVE-Reborn/rend3.git#f2b7df4775919a0fc766e3a5718d8b4bba38bc28
I've converted over to that, so I don't need a backport.
Here are the diffs to my code, to give a sense of what had to be changed to go from 0.2.3 to pre-0.3.x. This is useful only  as a guide to documentation, showing what had to be touched. I'm not using the 'framework".
diffs.txt

Sounds good.
Thank you for that diff, that is very helpful as I fill out the changelog.


Implement notifications mechanism

Implement a notification system to notify users about new comments, replies, or any other relevant updates. Display notifications in a designated area of the user interface.

I can do this. What database should I use? Sqlite inbuilt in django or something else?

I can do this. What database should I use? Sqlite inbuilt in django or something else?

PostgreSQL, MongoDB or something similar.


Console errors for ModuleGoldenApple.onPrepareItemCraft

nullPointerException
.onPrepareItemCraft(ModuleGoldenApple.java:39)
I am developing a custom items crafting plugin and I am receiving this error from OldCombatMechanics when I include the following code in my plugin. I can perceive no reason why this should happen!
public void onPrepare(PrepareItemCraftEvent e) {
.....Some code which filters to accept only 'my' custom recipes....
if (!this.getConfig().getBoolean(new StringBuilder("EnabledWorlds.").append(w).toString())) {
logger.info("A player tried to craft in an unregistered world called " + w);
e.getInventory().setResult(new ItemStack(Material.AIR, 1));
return;
}
.....Other code to manage recognition of my custom recipes...
None of the rest of my code causes any issue, and I am not crafting anything with Gold or Apples!

@Rayzr522

Heh I'll go take a look. My modules, breaking again :P

@Goldentoenail
Sorry for the slow response, but the only way I could see this happening is if somehow in your code maybe you are setting a recipe's result to null. That's all I can figure out. Obviously I need to put a null check in that code (should've done that anyways), but still, debug all the results for recipes that you're setting. That is the only way I can think of for a NPE to occur.


Deneb not accepting Vega Lite condition in encoding

I am trying to add a condition to Vega Lite in Deneb that dynamically sets the y-axis maximum to 100 when the indicator selected in Power BI is a percentage, else retains the default y-axis maximum behaviour. I have tried various ways of adding a condition to the encoding section, but wherever I place it, it is rejected by Deneb with error message "should NOT have additional property: condition". I can't find a way around this, no matter where I place the condition. Can you advise what I'm doing wrong?

Below is the relevant section of code:
"encoding": {
"x": {
"field": "areaname",
"type": "nominal",
"sort": {"field": "pracbarorder"},
"axis": {
"labels": false,
"ticks": false,
"title": null
}
},
"y": {
"field": "val",
"type": "quantitative",
"axis": {
"ticks": true,
"title": null
},
"scale": {
"condition": {
"test": "data('dataset')[0]['unit']=='%'",
"domainMax": 100
}
}
},
"color": {
"field": "pracareagroup",
"type": "nominal",
"legend": {
"orient": "none",
"legendX": {
"expr": "width-320"
},
"legendY": -100,
"labelFontSize": 13,
"labelLimit": 300,
"symbolOpacity": 0.75,
"title": null
},
"scale": {
"range": {
"field": "legendColor"
}
}
}
},

Hi @david5053, try something like the following. Note, the params section and the transform section is purely for toggling to as a demonstration. The important pieces are in the scale and axis properties under the y channel
{   "$schema": "https://vega.github.io/schema/vega-lite/v5.json",   "width": 200,   "params": [     {       "name": "categoryCValue",       "value": 0.75,       "bind": {         "input": "range",         "min": 0,         "max": 1.5,         "step": 0.1,         "name": "Cat C Value"       }     },     {       "name": "yAxisAsPercentage",       "value": true,       "bind": {"input": "checkbox", "name": "Y-Axis as %"}     }   ],   "data": {     "name": "dataset",     "values": [       {"category": "a", "value": 0.25},       {"category": "b", "value": 0.35},       {"category": "c"}     ]   },   "transform": [     {       "calculate": "datum['category'] === 'c' ? categoryCValue : datum['value']",       "as": "value"     },     {"calculate": "yAxisAsPercentage ? 'percentage' : 'number'", "as": "unit"}   ],   "mark": {"type": "bar"},   "encoding": {     "x": {"type": "nominal", "field": "category", "axis": {"labelAngle": 0}},     "y": {       "type": "quantitative",       "field": "value",       "scale": {         "domainMax": {           "expr": "data('data_0')[0]['unit'] === 'percentage' ? 1 : null"         }       },       "axis": {         "labelExpr": "data('data_0')[0]['unit'] === 'percentage' ? format(datum.label, '.0%') : format(datum.label, '0.2f')",         "tickCount": {           "expr": "data('data_0')[0]['unit'] === 'percentage' ? 11 : null"         }       }     }   } }

Hi @Giammaria, thanks for your helpful suggestion. The problem with setting domainMax to null is that is doesn't restore the default behaviour.
If you remove the scale section of your code, you'll see the default behaviour is for Vega Lite to determine an appropriate value for the maximum and to show it at the top of the y-axis.

When domainMax is set to null, the axis is often cut off without a maximum value being show.

I'd like to create a condition that only changes the default behaviour of Vega Lite if the unit is a percentage, as I don't want the y-axis to be cut off without a maximum value being shown.

I have worked it out!
If setting a domainMax, you also need to explicitly specify "nice": true to restore the default behaviour so that the axis ends on a nice round value. So @Giammaria's code should be modified as below:


Bug: Code breaks after around 405 iterations

This issue has been reported in CEA.
Currently, it needs tests in other machines to verify if the problem is related to memory leak or something else.

Simulation number 2402
Running archetypes-mapper with scenario = C:\CEA\CS2_Data_generator_plugin\test522
Using Daysim binaries from path: c:\users\luis.santos\documents\cityenergyanalyst\dependencies\daysim\bin64
Using Daysim data from path: c:\users\luis.santos\documents\cityenergyanalyst\dependencies\daysim\lib
verifying geometry files
zone: C:\CEA\CS2_Data_generator_plugin\test522\inputs\building-geometry\zone.shp
surroundings: C:\CEA\CS2_Data_generator_plugin\test522\inputs\building-geometry\surroundings.shp
Getting geometry materials
Creating 3D geometry and surfaces
Saving geometry pickle files in: C:\Users\LUIS~1.SAN\AppData\Local\Temp\tmpyj6fxs11\test522_radiation_geometry_pickle
Standardizing coordinate systems
Reading terrain geometry
Creating 3D building surfaces
Calculating terrain intersection of building geometries
Using single process
Calculation of terrain intersection for building 1 completed out of 1
Using single process
Calculation of terrain intersection for building 1 completed out of 8
Calculation of terrain intersection for building 2 completed out of 8
Calculation of terrain intersection for building 3 completed out of 8
Calculation of terrain intersection for building 4 completed out of 8
Calculation of terrain intersection for building 5 completed out of 8
Calculation of terrain intersection for building 6 completed out of 8
Calculation of terrain intersection for building 7 completed out of 8
Calculation of terrain intersection for building 8 completed out of 8
Generating geometry for surrounding buildings
Generating geometry for buildings in the zone of analysis
Using single process
Generating geometry for building 1 completed out of 1
Creating radiance material file
Creating radiance geometry file
Converting files for DAYSIM
Transforming weather files to daysim format
Running command `epw2wea "C:\CEA\CS2_Data_generator_plugin\test522\inputs\weather\weather.epw" "C:\Users\LUIS~1.SAN\AppData\Local\Temp\tmpyj6fxs11\cea_radiation\common_inputs\weather_60min.wea"`
multiprocessing.pool.RemoteTraceback:
"""
Traceback (most recent call last):
  File "C:\Users\luis.santos\Documents\CityEnergyAnalyst\Dependencies\Python\lib\multiprocessing\pool.py", line 121, in worker
    result = (True, func(*args, **kwds))
  File "C:\Users\luis.santos\Documents\CityEnergyAnalyst\Dependencies\Python\lib\multiprocessing\pool.py", line 44, in mapstar
    return list(map(*args))
  File "c:\users\luis.santos\documents\cityenergyanalyst\cityenergyanalyst\cea\utilities\parallel.py", line 131, in __apply_func_with_worker_stream
    result = func(*args)
  File "c:\users\luis.santos\documents\github\stochastic_scenario_generator\scenario_generator\stochastic_scenario_generator.py", line 166, in stochastic_scenario_generator
    radiation_main.main(config)  # runs solar radiation script
  File "c:\users\luis.santos\documents\cityenergyanalyst\cityenergyanalyst\cea\resources\radiation_daysim\radiation_main.py", line 270, in main
    cea_daysim.execute_epw2wea(weather_file)
  File "c:\users\luis.santos\documents\cityenergyanalyst\cityenergyanalyst\cea\resources\radiation_daysim\radiance.py", line 122, in execute_epw2wea
    epw2wea_result = subprocess.run(shlex.split(command), stdout=subprocess.PIPE)
  File "C:\Users\luis.santos\Documents\CityEnergyAnalyst\Dependencies\Python\lib\subprocess.py", line 488, in run
    with Popen(*popenargs, **kwargs) as process:
  File "C:\Users\luis.santos\Documents\CityEnergyAnalyst\Dependencies\Python\lib\subprocess.py", line 800, in __init__
    restore_signals, start_new_session)
  File "C:\Users\luis.santos\Documents\CityEnergyAnalyst\Dependencies\Python\lib\subprocess.py", line 1207, in _execute_child
    startupinfo)
OSError: [WinError 8] Not enough memory resources are available to process this command
"""

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "c:\users\luis.santos\documents\cityenergyanalyst\cityenergyanalyst\cea\worker.py", line 146, in worker
    run_job(config, job, server)
  File "c:\users\luis.santos\documents\cityenergyanalyst\cityenergyanalyst\cea\worker.py", line 108, in run_job
    script(config=config, **parameters)
  File "c:\users\luis.santos\documents\cityenergyanalyst\cityenergyanalyst\cea\api.py", line 59, in __call__
    self._runner.__call__(*args, **kwargs)
  File "c:\users\luis.santos\documents\cityenergyanalyst\cityenergyanalyst\cea\api.py", line 37, in script_runner
    script_module.main(config)
  File "c:\users\luis.santos\documents\github\stochastic_scenario_generator\scenario_generator\stochastic_scenario_generator.py", line 245, in main
    list_of_dataframe_with_instances)
  File "c:\users\luis.santos\documents\cityenergyanalyst\cityenergyanalyst\cea\utilities\parallel.py", line 96, in wrapper
    result = map_result.get()
  File "C:\Users\luis.santos\Documents\CityEnergyAnalyst\Dependencies\Python\lib\multiprocessing\pool.py", line 657, in get
    raise self._value
OSError: [WinError 8] Not enough memory resources are available to process this command


Submit records using Batch API

Currently, the Ably connector submits messages to Ably using a single Realtime connection. This limits throughput and the maximum number of channels that can be used from a single ChannelSinkTask. Given that we support dynamic mapping of SinkRecord to Ably channels, it can be hard for users to control the number of distinct outgoing channels from a single sink task.
Moving to the Batch REST API will improve this, particularly with parallel submissions of batches, at the cost of message ordering. We propose making parallelism configurable, so that users can select to either have very high throughput at the cost of ordering, or maintain ordering for more limited throughput per channel sink task.

Note this is implemented in #116

Implemented in https://github.com/ably/kafka-connect-ably/pull/125


spin animation in styleguide may have useless translateY(0.6em)

Found a bug? Please fill out the sections below. 👍
Issue Summary

I don't get that translate. Is this intended behavior?
And in core.css, not translateY, just rotate.
It's also affecting buttons:

Steps to Reproduce

enable styleguide

I have confirmed that this issue can be reproduced as described on a fresh Wagtail project: (yes / no)
yes

Technical details

Python version: 3.8.1* Django version: Look in your requirements.txt, or run pip show django | grep Version.
Wagtail version: 2.13.2
Browser version: chrome 90

👋 this has been fixed, not sure when. https://static-wagtail-v5-2.netlify.app/admin/styleguide/


Iterate Through an Array with a For Loop

Challenge Iterate Through an Array with a For Loop has an issue.
User Agent is: Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/57.0.2987.133 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

var myArr = [ 2, 3, 4, 5, 6];

// Only change code below this line
var total= 0;

for (var i= 0; i < myArr.length; i++) {
  total += myArr[i];
}

Not a 100% sure if this is a bug, but I was trying to practice getting into the habit of not having the example portion at the top of the page so that I could write the code without some form of "cheat sheet". So I deleted the example portion of it and typed in the correct code and I wasn't able to move forward with the exercise.

Hi,
fcc checks for two for loops, one for the provided example and one for your code.
I guess it should only check for one.
I'll wait for mods to chime in and see if we need to change > to >= on this line .

Hi Retspihaton,
I don't think this is a bug, as the task says to only change the code below the line. This is common behaviour elsewhere, so I don't think it will be changed to use >=, unfortunately. If it does get changed, it might be good to check other places where example code is required and see if the same can be done there (the example code is sometimes redundant, like in this example).


feat(webpack): ignoring mock-data dir when prod

What kind of change does this PR introduce?
feature

What is the current behavior?
mock-data files being copy in prod build

What is the new behavior (if this is a feature change)?
mock-data files not being copy in prod build

I discovered it's very useful to use mock-data files both in development and test, but it's bit annoying when publishing to prod, these files usually unnecessary.

thanks!


as promised, I made the extension to let timestampable work with zenddate type by benjamin eberlei

made compatible with zenddate dateype by benjamin eberlei (see: https://github.com/beberlei/DoctrineExtensions/blob/master/lib/DoctrineExtensions/Types/ZendDateType.php)
hope you like it ;)
Let me know, what you think of it.

so as far as I understand operating simply with zend date does not work?
<?php
class Timestampable
{
    /**
     * @ORM\Column(type="datetime")
     * @Gedmo\Timestampable(on="create")
     */
    private $createdAt;

    function getCreatedAt() {
        return new Zend_Date($this->createdAt->format());
    }
}

This would be in general the same thing, are there any reasons why this does not suit you?

Hi Gediminas,
the reason why I feel uncomfortable with this approach:
First we instantiate a DateTime object and soon afterwards you we
instantiate the Zend_Date object.
The standard DateTime object will never be used and I want to save the
extra costs and I have to write a little more boilerplate code for the
"getCreatedAt" method.
But if you like your approach more, you don't have to use my small
extension, just wanted to ask you. :-)
Thanks for you reply.
Best Regards
Sascha-Oliver Prolic
2011/10/26 Gediminas Morkevicius
<EMAIL_ADDRESS>
so as far as I understand operating simply with zend date does not work?
<?php
class Timestampable
{
   /**
    * @ORM\Column(type="datetime")
    * @Gedmo\Timestampable(on="create")
    */
   private $createdAt;

   function getCreatedAt() {
       return new Zend_Date($this->createdAt->format());
   }
}

This would be in general the same thing, are there any reasons why this does not suit you?
--
Reply to this email directly or view it on GitHub:
https://github.com/l3pp4rd/DoctrineExtensions/pull/179#issuecomment-2532599

its not a problem since you have done it, but in general its a very cheep operation. and for instance having more if statements for checking which type is it may become overused. it might be not the case for this specific case, this part with types might be improved somehow for all extensions maybe. thanks anyway for bringing this up

just another reason:
when I call getCreatedAt() it will instantiate a new Zend_Date object
everytime. A test expecting that the dates are the same objects would
fail, therefore the code should become something like this:
createdAtZendDate instanceof \Zend_Date) {
            return $this->createdAtZendDate;
       }
       return $this->createdAtZendDate = new
Zend_Date($this->createdAt->format());
   }
}

a bit too much code in my opinion (without traits as coming in php 5.4).
Your argument with the if statement is however valid. What do you
think about implementing kind of a state pattern?

Regards

Sascha-Oliver Prolic

2011/10/26 Gediminas Morkevicius
:
> its not a problem since you have done it, but in general its a very cheep operation. and for instance having more if statements for checking which type is it may become overused. it might be not the case for this specific case, this part with types might be improved somehow for all extensions maybe. thanks anyway for bringing this up
>
> --
> Reply to this email directly or view it on GitHub:
> https://github.com/l3pp4rd/DoctrineExtensions/pull/179#issuecomment-2533531
>

--
Sascha-Oliver Prolic


BnP Avro input to string/string output

Is it possible to run a BnP job, where the input is Avro, and the output is a read-only store with both the key and the value as simple strings?
It seems to me that once the input is Avro, that also forces the output to be Avro as well. That creates a ton of problems because it seems that client access to the cluster now needs to be Avro-aware, and my use case doesn't need Avro once data is in Voldemort.
EDIT: Specifically, this is what bothers me:
Key serializer differs:
	BnP config/data has:	SerializerDefinition(name = avro-generic, schema-info = {0={"type":"string","avro.java.string":"String"}}, compression = null)
	Voldemort server has:	SerializerDefinition(name = string, schema-info = {}, compression = null)
Value serializer differs:
	BnP config/data has:	SerializerDefinition(name = avro-generic, schema-info = {0="bytes"}, compression = null)
	Voldemort server has:	SerializerDefinition(name = bytes, schema-info = {}, compression = null)

Avro uses UTF-8 to serialize Strings. Voldemort String does the same. Not
sure whether there are any gotchas in the Voldemort client, but you should
be able to use StoreClient<String,String> on Avro UTF-8. There could be
some inconsistencies as I haven't tried this.
Thanks,
Arun.
On Thu, Jun 23, 2016 at 7:32 AM, Elad Efrat<EMAIL_ADDRESS>wrote:

Is it possible to run a BnP job, where the input is Avro, and the output
is a read-only store with both the key and the value as simple strings?
It seems to me that once the input is Avro, that also forces the output to
be Avro as well. That creates a ton of problems because it seems that
client access to the cluster now needs to be Avro-aware, and my use case
doesn't need Avro once data is in Voldemort.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/voldemort/voldemort/issues/421, or mute the thread
https://github.com/notifications/unsubscribe/AGN-Mw1fJBqh9Zhxyz18z61MxxvBaX-Rks5qOpkTgaJpZM4I8321
.

Take a look at the error I'm getting:
Key serializer differs:
	BnP config/data has:	SerializerDefinition(name = avro-generic, schema-info = {0={"type":"string","avro.java.string":"String"}}, compression = null)
	Voldemort server has:	SerializerDefinition(name = string, schema-info = {}, compression = null)
Value serializer differs:
	BnP config/data has:	SerializerDefinition(name = avro-generic, schema-info = {0="bytes"}, compression = null)
	Voldemort server has:	SerializerDefinition(name = bytes, schema-info = {}, compression = null)

Store definition should say Avro Generic, (not versioned) and String. But
Voldemort Client can be StoreClient<String,String>
Thanks,
Arun.
On Thu, Jun 23, 2016 at 11:33 AM, Elad Efrat<EMAIL_ADDRESS>wrote:

Take a look at the error I'm getting:
Key serializer differs:
BnP config/data has:    SerializerDefinition(name = avro-generic, schema-info = {0={"type":"string","avro.java.string":"String"}}, compression = null)
Voldemort server has:   SerializerDefinition(name = string, schema-info = {}, compression = null)
Value serializer differs:
BnP config/data has:    SerializerDefinition(name = avro-generic, schema-info = {0="bytes"}, compression = null)
Voldemort server has:   SerializerDefinition(name = bytes, schema-info = {}, compression = null)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/voldemort/voldemort/issues/421#issuecomment-228142534,
or mute the thread
https://github.com/notifications/unsubscribe/AGN-My6wq37UDZqTzSTAXnoP3h-Ss7Kpks5qOtFwgaJpZM4I8321
.

@arunthirupathi I'm not sure what you mean by "Voldemort Client can be StoreClient<String,String>" - there's no Java code involved in this process. Just store definitions on the nodes and BnP configuration. Are you saying that the problem is that the schema-info is specified with a version 0?

Sorry for not being clear.
Avro String and Voldemort String serializes to same byte representation and
may be back. But Build and Push Job enforces strict check on the Schema so
it will not match. But if you have a client that reads from Voldemort (
general way to do that is by creating a StoreClient, you will directly get
a String).
In other words, what is the problem if you have to define the Store as
Key/Value Schema as Avro String ?
Thanks,
Arun.
On Thu, Jun 23, 2016 at 11:49 AM, Elad Efrat<EMAIL_ADDRESS>wrote:

@arunthirupathi https://github.com/arunthirupathi I'm not sure what you
mean by "Voldemort Client can be StoreClient" - there's no Java code
involved in this process. Just store definitions on the nodes and BnP
configuration. Are you saying that the problem is that the schema-info is
specified with a version 0?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/voldemort/voldemort/issues/421#issuecomment-228147283,
or mute the thread
https://github.com/notifications/unsubscribe/AGN-M288n652s-SzzAOXV4avsRvklIIpks5qOtUcgaJpZM4I8321
.

@arunthirupathi sorry for the delay - I was on vacation.
The way I see it, Voldemort has a design flaw if the output must be Avro as well once the input is Avro. We use Avro as a format for processing in Hadoop. Once we're done with Hadoop, we need the data accessible in a different format. I don't see why I still have to fuss with Avro when I don't really need it. The problem is that I don't want to add dependencies and code that I don't really need just because the underlying datastore tightly couples input and output format. :/
Voldemort has all the information to do this itself: it knows the input is Avro, it knows which fields it needs to look at for key/value, and it knows the output should be strings, so it can validate those fields are strings as well.
Is it still possible to use a custom serializer? Maybe then I can remove the "Avroness" from the data and keep the format knowledge internally...

It might be possible to use non-Avro serializers in BnP. Some people did mention doing that on the mailing list, but it is not something we support internally at LinkedIn. We don't have much experience with this kind of set up, so we can't offer much guidance.
It might "happen to already work", or it may require a fairly simple patch to polish some of the rough edges. I don't know... I'm afraid you're on your own on this own. You might want to search the mailing list and reply to some of the old threads or start a new one, in order to get some feedback from the rest of the community.

As per the mailing list discussions, Voldemort 1.6.9 used to support String
serialization. But 1.10.* only supports Avro or JSON.
As Felix pointed out, Voldemort RO is only tested always with Avro and JSON.
PS : Community had success with using Voldemort RO setup with zoned stores
as well, but that is not intended as well and 1.10.* errors out on these
conditions.
Thanks,
Arun.
On Sun, Jul 10, 2016 at 8:54 PM, Felix GV<EMAIL_ADDRESS>wrote:

It might be possible to use non-Avro serializers in BnP. Some people did
mention doing that on the mailing list, but it is not something we support
internally at LinkedIn. We don't have much experience with this kind of set
up, so we can't offer much guidance.
It might "happen to already work", or it may require a fairly simple patch
to polish some of the rough edges. I don't know... I'm afraid you're on
your own on this own. You might want to search the mailing list and reply
to some of the old threads or start a new one, in order to get some
feedback from the rest of the community.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/voldemort/voldemort/issues/421#issuecomment-231637054,
or mute the thread
https://github.com/notifications/unsubscribe/AGN-M-m_6ZK7f1ryEXq7i3RP6jMOPGUqks5qUb6GgaJpZM4I8321
.

Fair enough. When the time comes, I'll either write my own "coordinator" that knows how to deal with the Avro store and/or custom serializers to avoid Avro knowledge in the resulting stores.
@arunthirupathi why was this functionality removed in 1.10.*, btw?


Which pin should i connect the signal to on the Seeed esp32 s3?

I am trying to use the above code for the seeed studio esp32 s3. Please tell me which pin can i connect the speaker to on this board for getting audio output?

This is PIN 18 by default, but can be updated by replacing ".data_out_num = PDM_GPIO_NUM" in the code


Added a tempered_sample helper function and no-swap burn-in functionality

This PR adds two main things:

A tempered_sample helper function to allow a user to call directly with tempering related args without having to wrap a sampler. This facilitates more "standard" workflow from model => sampling, where a user may want to try many temperature ladders, swap strategies etc. on the same model. This does not replace or affect the pre-existing workflow of wrapping a sampler using the tempered function (perhaps this should be renamed tempered_sampler. This function made particular sense in context of (2) below..
No-swap burn-in functionality, i.e. a user can specify a set number of steps for the parallel tempered chains to be run without any attempts at swapping amongst them, this is a fairly standard part of the PT algorithm we did not support previously as it is hard to sensibly insert the necessary kwarg into the execution path to facilitate this. This is distinct from discard_initial in AbstractMCMC, as that functionality still works and would discard the initial M samples carried out under the full PT regime where swapping etc. may occur.

The implementation is roughly as follows:
Hi-jack the initial step unique call (where no state is present) to pass through burn-in related kwargs and trigger N_burnin samples to be made prior to starting the main PT regime.

Thanks @HarrisonWilde!


Shooter isn't emptied when the foul occurs

When the pyramid is breached and the level is failed, sometimes the shooter still works. That shouldn't be the case.

The bug is nonexistent in https://github.com/jakubg1/OpenSMCE/commit/e72272f402e176b39a8db54be470ee1dd059ba04 and was probably fixed much earlier. This was probably one of the bugs created by introducing the new list utility.


Missing -f option in 1.2

Describe the bug
The -f option has been removed in v1.2
To Reproduce

Install 1.2

Error Log
root@7a84c5cfeaad:/home/presets/echo# poetry self add poetry-multiproject-plugin==1.1.6

Updating dependencies
Resolving dependencies... (0.6s)

Writing lock file

Package operations: 4 installs, 0 updates, 0 removals

  • Installing mypy-extensions (1.0.0)
  • Installing typing-extensions (4.5.0)
  • Installing mypy (0.991)
  • Installing poetry-multiproject-plugin (1.1.6)
root@7a84c5cfeaad:/home/presets/echo# poetry build-project --help

Description:
  Builds a package, as a tarball and a wheel by default.

Usage:
  build-project [options]

Options:
  -f, --format=FORMAT        Limit the format to either sdist or wheel.
  -h, --help                 Display help for the given command. When no command is given display help for the list command.
  -q, --quiet                Do not output any message.
  -V, --version              Display this application version.
      --ansi                 Force ANSI output.
      --no-ansi              Disable ANSI output.
  -n, --no-interaction       Do not ask any interactive question.
      --no-plugins           Disables plugins.
      --no-cache             Disables Poetry source caches.
  -C, --directory=DIRECTORY  The working directory for the Poetry command (defaults to the current working directory).
  -v|vv|vvv, --verbose       Increase the verbosity of messages: 1 for normal output, 2 for more verbose output and 3 for debug.
root@7a84c5cfeaad:/home/presets/echo# poetry self add poetry-multiproject-plugin~=1.1

Updating dependencies
Resolving dependencies... (0.3s)

Writing lock file

Package operations: 0 installs, 1 update, 0 removals

  • Updating poetry-multiproject-plugin (1.1.6 -> 1.2.0)
root@7a84c5cfeaad:/home/presets/echo# poetry build-project --help

Description:
  Builds a package, as a tarball and a wheel by default.

Usage:
  build-project [options]

Options:
      --with-top-namespace=WITH-TOP-NAMESPACE  To arrange relative includes, and to modify import statements.
  -h, --help                                   Display help for the given command. When no command is given display help for the list command.
  -q, --quiet                                  Do not output any message.
  -V, --version                                Display this application version.
      --ansi                                   Force ANSI output.
      --no-ansi                                Disable ANSI output.
  -n, --no-interaction                         Do not ask any interactive question.
      --no-plugins                             Disables plugins.
      --no-cache                               Disables Poetry source caches.
  -C, --directory=DIRECTORY                    The working directory for the Poetry command (defaults to the current working directory).
  -v|vv|vvv, --verbose                         Increase the verbosity of messages: 1 for normal output, 2 for more verbose output and 3 for debug.
root@7a84c5cfeaad:/home/presets/echo#

Expected behavior
Able to use -f sdist to specify that I only want an sdist to be built.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Linux
Python version:  3.6.9
Poetry version: 1.3.2

Additional context
I think that adding the --with-top-namespace option has replaced the -f option instead of adding a second option.

Thank you for reporting! Yes, it seems I have messed something up with the latest release. I'm having a look at it right now.

I believe this is now fixed and a new version is available to install: v1.2.1. Huge thanks for reporting this! ⭐

Hey David,
Thanks for the speedy action!
I've tested the new version and it fixes the issue for me, thank you!
-- Craig Blaszczyk
El mié, 15 feb 2023 a las 17:29, David Vujic @.***>)
escribió:

I believe this is now fixed and a new version is available to install:
v1.2.1. Huge thanks for reporting this! ⭐
—
Reply to this email directly, view it on GitHub
https://github.com/DavidVujic/poetry-multiproject-plugin/issues/28#issuecomment-1431742734,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGSUSRRJJ457ROIRKU274LWXUG6RANCNFSM6AAAAAAU454PMM
.
You are receiving this because you authored the thread.Message ID:
@.***>


Test if config.py matchs documentation

Hello!
I had to move the test that @trgiangdo sugestted on the other PR to taipy core, since is here where we have the config classes that holds the documentation. Feel free to ping me if there is any questions.

I tested this again and it is not working. I try to change the docstring on taipy-core, and the test still passed.
My hypothesis is that when we _inject_section or assign the method to Config, the .__doc__ is actually updated, but not the intellisense of VSCode or PyCharm.
We need another work-around if possible.

That's weird, because I did test a change in docstring and the test failed. I will try to write a test to validate this case.

See the above image where I changed the docstring of `JobConfig._configure` and running the test again fails as expected.

I've added a expected to fail test where I explicitly change the documentation and compare to the intelicense. I think this works, but if you find something strange please let me know.

After discussion, We believe this is not helping us test whether the content is correct.
I am closing the PR.


Use stencils for clipping

If we can use stencils then we can rid of the clipping inside the fragment shaders, which would allow removal of the discard statement in the fragment shaders. Removal of discard will enable the fixed-function early fragment-test to be enabled again and improve performance.

Issue 1: stencil are 8 bits, so only 255 different clipping area exists.
Issue 2: In between draw calls you need to change the stencil reference parameter.
Issue 3: Difficulty with only stencil-reference to be used for both the comparison and the value to write in the stencil buffer. So you cannot clip a clipping rectangle easy. Maybe possible to color draw into the stencil buffer?

This will never work


Support for block-based format patterns (read: pride flag formatting!)

It'd be kinda cool to be able to do this. Pride flag presets maybe?

Introduced in 1.4.1. Todo documentation.


ImportError: cannot import name 'build_dataset' from 'mmdet.datasets' (/content/mmdetection/mmdet/datasets/init.py)

Thanks for your error report and we appreciate it a lot.
Checklist

I have searched related issues but cannot get the expected help.
I have read the FAQ documentation but cannot get the expected help.
The bug has not been fixed in the latest version.

Describe the bug
A clear and concise description of what the bug is.
Reproduction

What command or script did you run?

A placeholder for the command.

Did you make any modifications on the code or config? Did you understand what you have modified?
What dataset did you use?

Environment

Please run python mmdet/utils/collect_env.py to collect necessary environment information and paste it here.
You may add addition that may be helpful for locating the problem, such as

How you installed PyTorch [e.g., pip, conda, source]
Other environment variables that may be related (such as $PATH, $LD_LIBRARY_PATH, $PYTHONPATH, etc.)

Error traceback
If applicable, paste the error trackback here.
A placeholder for trackback.

Bug fix
If you have already identified the reason, you can provide the information here. If you are willing to create a PR to fix it, please also leave a comment here and that would be much appreciated!

@ShSi1001 That has already been deleted. Please synchronize with the latest code.

一样的问题

I meet the same issue when using demo/MMDet_InstanceSeg_Tutorial.ipynb

I solve this issue by using the demo/MMDet_InstanceSeg_Tutorial.ipynb in branch dev-3.x.
So, for the mmdet developer, could you please update the version from dev-3.x to main branch?

还没改哎


Feature requests

Thanks so much for creating this awesome library.  I just noticed how to use render_with.  Super helpful!
I have two requests:

Allow links/redirects in record actions menu, with a way to pass the current record - In my case, I'd like to link to a preview page and a page with a specialized form that is outside of the scope of an edit page.
Configure index table fields and order  - I'd love to be able to order and hide some.

If I have overlooked existing capabilities, please let me know!
Thanks!
-Mario

Glad to hear you're finding the lib useful!

(Also there is the :hidden_fields option, but I'm assuming you want something specific to list view?)

Is there a way to specify custom rendering of foreign key fields (org_id)?  I noticed that these fields don't seem to go through the render_field function. I was trying to specify how to handle those in :render_with .

I'll need to check to be sure but that sounds like a bug. I think they should go through the same render flow as any other field, just the default should be different.


Add download_on_remote flag

If download_on_remote variable is set and is True, then downloading, unpacking and propagation happens on the remote host. Useful when the connection to the remote host is slow.
There's a "problem" with this PR: it is only compatible with Ansible 2.8>. The reason being that Ansible does not handle recursive copies with remote_src before the 2.8 release. There are workarounds, sure and I can work on it if the project considers that this PR is a sensible use case.
The build fails because the remote machine is not able to download the file, as far as I can see.

it is only compatible with Ansible 2.8

I cannot merge it with such restriction. Hard requirement is current and one previous ansible versions (currently 2.7 and 2.8) and debatable soft requirement is one version before that (currently 2.6).


Add Doc around windows installation

Describe your changes :

fix #8241

Type of change :

[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Documentation

Frontend Preview (Screenshots) :
For frontend related change, please link screenshots of your changes preview! Optional for backend related changes.

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x] All new and existing tests passed.

Reviewers

Thanks @ShilpaVernekar 🙌


[#716] update datastore documentation

Purpose
Complete the datastore connection documentation to match what's finished in the UI.
Changes

updated the old datastore image with the current state
added additional images and docs around datastore management: panel use, testing connections, disabling/deleting datastores

Checklist

[x] Update CHANGELOG.md file

[x] Merge in main so the most recent CHANGELOG.md file is being appended to
[x] Add description within the Unreleased section in an appropriate category. Add a new category from the list at the top of the file if the needed one isn't already there.
[x] Add a link to this PR at the end of the description with the PR number as the text. example: #1

[x] Applicable documentation updated (guides, quickstart, postman collections, tutorial, fidesdemo, database diagram.
If docs updated (select one):

[x] documentation complete, or draft/outline provided (tag docs-team to complete/review on this branch)

Ticket
Fixes #716

@conceptualshark reviewed the above and it looks great! Just a couple of things:

update spelling of "pagenated" to "paginated"
is it "reenabled" or "re-enabled"?
this sentence "For systems with larger collections of datastores, search options are available to filter results." The way I read it implies that search options are only available for larger collections of datastores. Maybe we can reword it a little to imply that search options are available to help them find the datastore they are looking for?
For the testing datastores section, not sure if you purposely left out the "grey" connection option or not but that color implies that the creds haven't been tested yet to confirm if it passed or failed.

@conceptualshark reviewed the above and it looks great! Just a couple of things:
* update spelling of "pagenated" to "paginated"

* is it "reenabled" or "re-enabled"?

* this sentence "For systems with larger collections of datastores, search options are available to filter results." The way I read it implies that search options are only available for larger collections of datastores. Maybe we can reword it a little to imply that search options are available to help them find the datastore they are looking for?

* For the testing datastores section, not sure if you purposely left out the "grey" connection option or not but that color implies that the creds haven't been tested yet to confirm if it passed or failed.

Thank you for the quick review! Should have resolved all of these. 👍


Unable to find License information for this program

I am unable to find License information for this program. Typically a Github repo will have a LICENSE file in it.

Added https://github.com/garrettfoster13/aced/commit/b5d1ad1b8cfb84a6420be22658beec340ef9e396


Eliminate common subfilters when converting it to a CNF

Description
Filters.toCNF() is used in 2 places, 1) to find prefilters when scanning segments that bitmaps are available to evaluate them and 2) to find filters which can be pushed down in a Join query. This function was adopted from Hive, but have some issues with optimization. One of issues is that it didn't eliminate common subfilters, which sometimes resulted in a huge number of subfilters which in turn could cause an OOM error. This PR is to eliminate those common subfilters by changing the type of subfilters of AndFilter and OrFilter from List to Set. This is to detect common subfilters of different orders, otherwise those subfilters should be sorted in some order which seems more complicated than using a Set.
The approach used in this PR can create a filter in a suboptimal CNF. There are at least 2 cases I'm aware of.

Even though x IN (1,2,3) is equivalent to x = 1 OR x = 2 OR x = 3, but this case is not handled yet. We may need to decide which form is more optimal (probably using IN filter is better since it will use less memory).
There are some missing cases which can be further reduced. For example, (A || ~(E)) && (A || ~(F)) && (A || ~(E) || ~(F)) can be reduced as below. This case is not handled yet, but I left a comment about it in FiltersTest.testToCNFWithComplexFilterIncludingNotAndOr().

(A || ~(E)) && (A || ~(F)) && (A || ~(E) || ~(F))
=> ((A && (~(E) || ~(F))) && (A || ~(E) || ~(F)))
=> (A && (~(E) || ~(F))
=> (A || ~(E)) && (A || ~(F))

This PR has:

[x] been self-reviewed.

[ ] using the concurrency checklist (Remove this item if the PR doesn't have any relation to concurrency.)

[ ] added documentation for new or modified features or behaviors.
[ ] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[ ] added or updated version, license, or notice information in licenses.yaml
[x] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[x] added unit tests or modified existing tests to cover new code paths.
[ ] added integration tests.
[ ] been tested in a test Druid cluster.

I'm tagging this PR as 0.18.0 since it could affect to Join performance significantly which is a new feature of 0.18.0.


Change install to one env file & update docs

Checklist

[x] I've given this PR a concise, self-descriptive, and meaningful title
[x] I've linked relevant issues in the PR body
[x] I've applied the relevant labels to this PR
[ ] I've added relevant tests for my contribution
[x] I've updated the documentation and/or added correct docstrings
[ ] I've assigned a reviewer
[x] I've consulted ADS's internal developer documentation to ensure my contribution is in line with any relevant design decisions

Resolves #481, Resolves #482, Resolves #483

@jcohenadad I had to leave activating the conda env in the installation step, as the installation step needs another command within in (pip install -e .). However, I've added a small section under the "Using AxonDeepSeg" section to say that you need to activate the venv if not already activated.

We made the changes to use only the common environment.yml file to install ADS on all OS.
I tested it on my laptop in Ubuntu 20.04 in WSL1 and Windows 10, everything is working as expected.
@vs74, @Stoyan-I-A, @dyt811, @ahill187, @joshuacwnewton, @gisellemartel Could you re-test the installation using this branch? The rendered doc is here. Thank you everyone for your help!

Installed ADS by following the docs. I had no issues with the installation on MacOS Mojave. Tested axondeepseg_test and flseyes too; everything works like a charm.

Installing worked for me on Ubuntu 20.04, as well. :slightly_smiling_face:

The installation worked. Tested on Windows 10.

Installation worked for me on MacOS Catalina 10.15.7!

Installation worked for me on MacOS Catalina 10.15.7! Testing the installation however failed for axondeepseg_test. Here is a log: log.txt

Have you used the Theano backend in the past? It looks like the software is trying to use Theano instead of tensorflow.

Installation worked for me on MacOS Catalina 10.15.7! Testing the installation however failed for axondeepseg_test. Here is a log: log.txt

Have you used the Theano backend in the past? It looks like the software is trying to use Theano instead of tensorflow.

Hmm interesting, could this be a tool that was installed with pip and the --user install? I've encountered instances where those have leaked into my venvs, very annoying and tough to troubleshoot if you're not aware/remember.

I found this, it seems to be an issue when installing Keras through conda-forge, though the post is 3 years old.
The only way I found to switch the backend permanently was to explicitly export the variable in ${CONDA_PREFIX}/etc/conda/activate.d/keras_activate.sh with export KERAS_BACKEND="tensorflow"

I found this, it seems to be an issue when installing Keras through conda-forge, though the post is 3 years old.
The only way I found to switch the backend permanently was to explicitly export the variable in ${CONDA_PREFIX}/etc/conda/activate.d/keras_activate.sh with export KERAS_BACKEND="tensorflow"

@gisellemartel could you try this fix temporarily to see if resolves the issue? If so, @mariehbourget let's just make a quick Notice for it on the ReadTheDocs, as it seems like a rare issue and we're migrating away from Keras when we move to IVADOMED soon.

@gisellemartel, did you get the chance to test the fix describe above?

@gisellemartel, did you get the chance to test the fix describe above?

Thank you @mariehbourget @mathieuboudreau @Stoyan-I-A for your suggestions. I tried the suggested fix and it worked! I do still get some warnings after running the test however
log.txt

Thanks @gisellemartel for the tests.
@mathieuboudreau, I added the warning to the doc. Can you review it and let me know if that is ok with you (I can't assign you officially since you opened the PR). Thanks!

@mariehbourget The message reads well for me; if the tests pass you can approve and merge, which will open the door to merging #479.

Haha fair. Maybe it coyld be in a troubleshooting section, but lets merge it here for now as the other PR is urgent
Téléchargez Outlook pour Androidhttps://aka.ms/ghei36

From: Joshua Newton @.>
Sent: Wednesday, March 24, 2021 3:50:19 PM
To: neuropoly/axondeepseg @.>
Cc: Mathieu Boudreau @.>; Mention @.>
Subject: Re: [neuropoly/axondeepseg] Change install to one env file & update docs (#484)
@joshuacwnewton commented on this pull request.

In docs/source/documentation.rsthttps://github.com/neuropoly/axondeepseg/pull/484#discussion_r600745258:

+.. NOTE :: For some users, the test may fail because Keras is using Theano backend instead of Tensorflow. In that case, you will see the line Using Theano backend. when launching axondeepseg_test. To fix this issue, add the line export KERAS_BACKEND="tensorflow" at the end of the <your_conda_install_location>\envs\<your_environment_name>/etc/conda/activate.d/keras_activate.sh file, then deactivate and reactivate your environment. The test should print Using Tensorflow backend. and pass.

I guess what I'm saying is, could this be an isolated incident that doesn't apply to most users? 😛
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/neuropoly/axondeepseg/pull/484#discussion_r600745258, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAK25ZOZNC2WPMSS4MGIT3TTFIUOHANCNFSM4ZTU5G2A.


Enhance e_ext_san_contains_reserved_ip linter to check for more reserved IP ranges, reject <IP_ADDRESS> IP address

lints/lint_ext_san_contains_reserved_ip.go checked if input IP address != <IP_ADDRESS> before
proceeding with the reserved range checks, while the IP <IP_ADDRESS> is in the reserved range.
Hence when the input IP is <IP_ADDRESS> the linter returns Pass status

util/ip.go:
{net.ParseIP("<IP_ADDRESS>"), net.ParseIP("<IP_ADDRESS>")},
contained a typo.

Additionally, the checks were missing few reserver ranges.

Hi guys. Sorry for the two commits that broke the CI.

Good points.
For starters I have just fixed what was missing in the currently available linters, would be nice to maybe add few additional linters to check for those corner cases.
Also IIRC in the zmap/zcrypto/x509 package, common name is a string, not a slice, so for certificates with multiple CNs the subsequent ones are ignored, or the parser returns error if the sequence has more than one element.
There should  be a lint that examines the raw subject and catches such cases also.

@szank right -- the way I read the code, it takes the last (so, most-specific) CN in the sequence, which by my reading is indeed what the browsers/etc are supposed to use in absence of an SAN, but IIRC at least some browsers do allow multiple CNs.
It seems like the library does expose the raw RDNSequence, so one could walk through that to get all of the CNs, but it would be strange to do that here but nowhere else CNs are consumed.

Hey, I just wanted to make sure that there are no more actions points in this PR that I could address before the merge is approved. Could you guys please give me an update?

@szank I don't see any problems.

With regard to @justinbastress's original comment about repeated CN's, those show up in Subject.ExtraNames, IIRC.
I suspect the proper way to handle repeats is to extract anything that runs checks on the CN to some utility function that accounts for ExtraNames (or to just error on ExtraNames in another lint, which we might do already?). I don't think that blocks this PR.


Use Molecule for testing

This role isn't tested with Molecule like other of my roles. Make it happen. :-)

Molecule has been added to this repo and is executed on every PR.


After a certain time the undo button becomes disable.

@llaske ,
In the Paint activity, the undo button becomes disabled after some time. Additionally, there is no shortcut key available for undo and redo actions. Could you please investigate this issue? If it appears to be a genuine problem, I would like to work on resolving it.
Thank you!
https://github.com/llaske/sugarizer/assets/154233802/bfa047ac-ee81-419c-ae63-9bec6b77a195

Yes the undo is limited for memory reason.
Short cuts is useless because it's too complex for children and most of the time Sugarizer is use on tablets.


Reduce closure size of installer images

Motivation for this change
In a minimal NixOS ISO installer environment I noticed some packages like X11 libraries which I felt were not needed and digged deeper.
The base installer profile includes dar in the system path which depends on gnupg which depends on pinentry, gtk2, openldap… I don't think this is a generally used tool and therefore propose to remove it.
But these were not all X11 dependencies. w3m was included twice, with X11 (from the base installer profile) and without (from services.nixosManual). But w3m-nox depended on imlib2 which again depended on X11 libraries. Therefore a new imlib2-nox was introduced. The minimal installer is now free of X11 libs!
Both the sizes of the kernel and the squashfs for the ISO and netboot images were reduced by about 20-25% by using XZ instead of gzip for compression. Disabling sound support also saved a few megabytes.
Stats for iso_minimal.x86_64-linux

master: 390MB
this PR: 309MB

Discussion & future work

Split linux-firmware-nonfree into categories and only include essential ones (~ 157M total)
Maybe put some non-essential kernel modules into own store paths or outputs? (~ 69M total)

Nice, 13.10 for example was 280MB.


Render badge conditionaly accroding to join mission data

Hi @houdamzari 👋
In this PR

[ ] I added render badge functionality with mission reservation data.
[ ] Fixed the Linter errors.

Issue Reference #6
If you have any questions about this PR feel free to ask, Thank you 👍

Status Approved :heavy_check_mark:
Hi @raihan2bd, great job implementing the requirements,
✔ Followed Gitflow.
✔ No linter errors.
✔ Good commit messages.
✔  Showing reservation badge functionality.
There is nothing left but to merge it!

Cheers and Happy coding!👏👏👏


Embedded labels

Description of Changes

Added a labeler helper
Added labeler to input, router, and metadata operators
Updated documentation for the new labels field
Added event id to windows event entries

Please check that the PR fulfills these requirements

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)
[ ] Add a changelog entry (for non-trivial bug fixes / features)
[ ] CI passes

Log Files
Logs / Second
CPU Average (%)
Memory Average (MB)

1
1000
4.896696
28.11584

1
5000
8.724367
42.411503

1
10000
15.190112
54.435345

1
50000
60.760563
158.89252

1
100000
116.98186
252.49219

10
100
5.6899247
30.174839

10
500
10.189876
40.801186

10
1000
15.586553
53.655846

10
5000
66.50175
182.41905

10
10000
124.65571
294.39374

Log Files
Logs / Second
CPU Average (%)
Memory Average (MB)

1
1000
3.1207683
28.670797

1
5000
8.86223
41.69329

1
10000
14.345097
53.81654

1
50000
61.29461
153.17538

1
100000
120.57498
238.7663

10
100
5.2932453
25.82651

10
500
11.620912
41.91972

10
1000
16.569319
53.59173

10
5000
77.81436
185.97589

10
10000
126.78518
370.35385

Log Files
Logs / Second
CPU Average (%)
Memory Average (MB)

1
1000
3.327655
28.651806

1
5000
8.362249
38.23074

1
10000
14.948666
53.7962

1
50000
60.553974
169.26671

1
100000
112.47245
244.67444

10
100
5.39662
30.109375

10
500
10.172604
44.47993

10
1000
15.465848
52.85143

10
5000
66.811676
176.82556

10
10000
122.495415
294.5097

Log Files
Logs / Second
CPU Average (%)
Memory Average (MB)

1
1000
3.258665
27.268858

1
5000
8.103675
40.683865

1
10000
14.345172
52.893856

1
50000
58.46899
218.29149

1
100000
108.42342
346.2981

10
100
4.224215
30.959455

10
500
9.672694
40.40948

10
1000
16.569426
53.768185

10
5000
62.70569
171.5586

10
10000
136.97307
310.3909


Add support for NPM package manager

This PR adds the necessary configuration files for publishing ppx_deriving_protobuf runtime library to the NPM package manager using BuckleScript for transpiling to JS.
The npm package is available at https://www.npmjs.com/package/bs-ppx_deriving_protobuf. The bs- is intentional so that:

it avoids conflict with ongoing effort to mirror opam package in npm (no JS involved)
it clearly indicates this package works with BuckleScript and require BuckleScript as dependency to work.

Hang on. Are you only including the runtime parts? Or are you using both the runtime parts and the AST transformer part?

Only the runtime part. The BuckleScript compiler is a native executable and takes ppx argument just like the regular ocaml compiler; those ppx can come from opam.

Thanks!


Missing labels

On Linux Mint, the menu bar on the left that has the network icons is still missing the labels.
I know this got fixed earlier.. but somehow it is still missing.

Fixed in 8dd6039


Clippy

I don't know that you actually want this, but did a run of cargo clippy --fix and followed up with a rustfmt...
Now, clippy seems to highlight a few warnings still...
`cel-interpreter` (lib) generated 9 warnings

The Arc one also caught my attention when working on #48 ... Correct me if I'm wrong, but there is no support for multithreading in CEL, is there? So are you planning on making the interpreter mutlithreaded eventually? Or should I take a stab at making all the Arc just Rc?

warning: bound is defined in more than one place
   --> interpreter/src/context.rs:118:37
    |
118 |     pub fn add_function<T: 'static, F: 'static>(&mut self, name: &str, value: F)
    |                                     ^
119 |     where
120 |         F: Handler<T> + 'static,
    |         ^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#multiple_bound_locations
    = note: `#[warn(clippy::multiple_bound_locations)]` on by default

warning: use of deprecated method `chrono::DateTime::<Tz>::timestamp_nanos`: use `timestamp_nanos_opt()` instead
   --> interpreter/src/objects.rs:634:38
    |
634 |             Value::Timestamp(v) => v.timestamp_nanos() > 0,
    |                                      ^^^^^^^^^^^^^^^
    |
    = note: `#[warn(deprecated)]` on by default

warning: field `0` is never read
   --> interpreter/src/magic.rs:201:17
    |
201 | pub struct List(pub Arc<Vec<Value>>);
    |            ---- ^^^^^^^^^^^^^^^^^^^
    |            |
    |            field in this struct
    |
    = note: `List` has a derived impl for the trait `Clone`, but this is intentionally ignored during dead code analysis
    = note: `#[warn(dead_code)]` on by default
help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field
    |
201 | pub struct List(());
    |                 ~~

warning: function `test_script` is never used
 --> interpreter/src/testing.rs:6:15
  |
6 | pub(crate) fn test_script(script: &str, ctx: Option<Context>) -> ResolveResult {
  |               ^^^^^^^^^^^

warning: method `clone` can be confused for the standard trait method `std::clone::Clone::clone`
   --> interpreter/src/context.rs:135:5
    |
135 | /     pub fn clone(&self) -> Context {
136 | |         Context::Child {
137 | |             parent: self,
138 | |             variables: Default::default(),
139 | |         }
140 | |     }
    | |_____^
    |
    = help: consider implementing the trait `std::clone::Clone` or choosing a less ambiguous method name
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#should_implement_trait
    = note: `#[warn(clippy::should_implement_trait)]` on by default

warning: usage of an `Arc` that is not `Send` and `Sync`
   --> interpreter/src/functions.rs:274:25
    |
274 |             Value::List(Arc::new(values))
    |                         ^^^^^^^^^^^^^^^^
    |
    = note: `Arc<Vec<Value>>` is not `Send` and `Sync` as `Vec<Value>` is neither `Send` nor `Sync`
    = help: if the `Arc` will not used be across threads replace it with an `Rc`
    = help: otherwise make `Vec<Value>` `Send` and `Sync` or consider a wrapper type such as `Mutex`
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#arc_with_non_send_sync
    = note: `#[warn(clippy::arc_with_non_send_sync)]` on by default

warning: usage of an `Arc` that is not `Send` and `Sync`
   --> interpreter/src/functions.rs:308:25
    |
308 |             Value::List(Arc::new(values))
    |                         ^^^^^^^^^^^^^^^^
    |
    = note: `Arc<Vec<Value>>` is not `Send` and `Sync` as `Vec<Value>` is neither `Send` nor `Sync`
    = help: if the `Arc` will not used be across threads replace it with an `Rc`
    = help: otherwise make `Vec<Value>` `Send` and `Sync` or consider a wrapper type such as `Mutex`
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#arc_with_non_send_sync

warning: usage of an `Arc` that is not `Send` and `Sync`
   --> interpreter/src/ser.rs:254:24
    |
254 |         Ok(Value::List(Arc::new(self.vec)))
    |                        ^^^^^^^^^^^^^^^^^^
    |
    = note: `Arc<Vec<Value>>` is not `Send` and `Sync` as `Vec<Value>` is neither `Send` nor `Sync`
    = help: if the `Arc` will not used be across threads replace it with an `Rc`
    = help: otherwise make `Vec<Value>` `Send` and `Sync` or consider a wrapper type such as `Mutex`
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#arc_with_non_send_sync

warning: usage of an `Arc` that is not `Send` and `Sync`
   --> interpreter/src/ser.rs:303:51
    |
303 |         let map = HashMap::from_iter([(self.name, Arc::new(self.vec))]);
    |                                                   ^^^^^^^^^^^^^^^^^^
    |
    = note: `Arc<Vec<Value>>` is not `Send` and `Sync` as `Vec<Value>` is neither `Send` nor `Sync`
    = help: if the `Arc` will not used be across threads replace it with an `Rc`
    = help: otherwise make `Vec<Value>` `Send` and `Sync` or consider a wrapper type such as `Mutex`
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#arc_with_non_send_sync

So either these instances would suffice as plain Rcs or clippy is wrong... Should I leave things as they are for these? I can quickly try and see if rustc then complains if they really need to be Sync/Send ...

Added a few clean ups, also misread the clippy issue with these Arc, the problem is the what is put in them that's not Sync/Send. So I guess so either T should be made Send + Sync.
I'll look in the Clone one: warning: method clone can be confused for the standard trait method std::clone::Clone::clone, which I think is the lifetime issue, yes?

Yes I've seen this one. I'd be open to renaming to .fork if you would like. Under the hood the function takes one context and returns a new context that has a reference to the original context, sort of like setting up a tree of contexts. I wrote up and explanation of how and why here if it's helpful. In any case, fork seems like an accurate name for what's happening. Thoughts?

Probably not .clone() indeed. So we're talking about inner scopes here, right? Always lexical ones I think in CEL? So should that just be a .inner_scope() or .add_new_inner_scope() or something? Fork isn't quite right neither, as it stacks rather than fork it... Tho... 🤔
Only two hard things in computer science, eh?! 😉
I'd go with .fork() over .clone() for sure tho...

Always lexical ones I think in CEL?

Correct, yes I'd be good with either suggestion! Take your pick.

Dunno if having rustfmt check & clippy at the github action level is something you'd want, but I took the freedom to add it... Can easily drop the commit if needed.
Finally the Arc issue was because Value wasn't Sync + Send, because Map wasn't as it'd hold an Rc...
Now confused by the test run failures... They reported rustc error don't point to the code as I have it 🤔

Passes here... confused

Rebased 🤦 Sorry for all the noise... hadn't updated my own fork(s)/clone(s) with my own changes...
So this is ready for review/merge, either with or without github actions changes


FriendlyNameMapper::to_string(uint32_t) blocking on linux

Hi!
Blender developer working on integrating Vulkan backend into Blender. Shader compilation performance is really important in our usecase as precompilation is not really possible. However when running more optimizers in parallel the performance degrades very fast due to stdlib implementation.
See the next hotspot where we compile hundreds of shaders using all threads available on the system. It takes almost as long as running on a single thread.

Via hotspot the issue tracks down as stdlib that uses global lock when accessing the locale. This could be evaded by changing the to_string(uint32_t) implementation at https://github.com/KhronosGroup/SPIRV-Tools/blob/main/source/name_mapper.cpp#L34 This part of the code doesn't require any locale but as noted in the comments is done this way for certain android compilers/platforms.
Would it be possible to reevaluate the current implementation so it would not stall when using multithreaded compilation on Linux?
I am running on a regular Ubuntu 24.04 Linux-6.8.0-41-generic-x86_64-with-glibc2.39 64 Bits, X11 UI.

We are experimenting with using C17 std::to_chars See https://projects.blender.org/blender/blender/pulls/127418/files
Perhaps using this when CPP17 is defined it would be an option as many parts of the std uses this global lock.

Doesn't SPIRV-Tools already require C++17? https://github.com/KhronosGroup/SPIRV-Tools/blob/7c9210cc1d82e8bf0ca969a88db069c83d838398/CMakeLists.txt#L38
The comment about std::to_string was added in 2016, and may not be relevant anymore.

I'm discussing this internally. When we started to require C++17 for spirv-tools, we broke the compilers that this workaround was trying to help. I don't think we have the requirement anymore. I'll make the change.

Do you need human-readable names for the instructions ? If not the CLI --raw-id and corresponding SPV_BINARY_TO_TEXT_OPTION_FRIENDLY_NAMES options should skip the string operations entirely.


Adds union merge strategy to changelog

Should prevent some merge conflicts in PRs.

LGTM, mind if I merge this before CI check finishes?

Current coverage is 80.13%

Merging #306 into master will not affect coverage as of 4255f49

@@            master    #306   diff @@
======================================
  Files           13      13
  Stmts          579     579
  Branches         0       0
  Methods
======================================
  Hit            464     464
  Partial          0       0
  Missed         115     115

Review entire Coverage Diff as of 4255f49

Uncovered Suggestions

+8.47% via ...rkLoggerPlugin.swift#28...76
+2.77% via Source/Moya.swift#18...33
+1.56% via ...rkLoggerPlugin.swift#12...20
See 7 more...

Powered by Codecov. Updated on successful CI builds.

Note that I don't think this'll apply to any PRs sent without this commit in their history.


[Suggestion] Option to split rank and user permissions into seperate flatfiles

Not sure how hard it would be but this would be invaluable to our large server networks where we have 40+ servers and editing permissions currently has to be done via a synchronization command which doesn't always work (one of the servers might be mid-restart).
It would be much easier to manually edit the rank flat file and upload it to all the servers at once without having to do surgery on each individual file to retain user ranks.
Thanks for your time

A multi-backend that allows splitting permissions into different data stores (multiple files, file and sql, separate sql, etc) is in development for pex2


Relationship lines draw for invisible TreeItems

Godot version
df9dae2
System information
Windows 10 x64
Issue description

#60061 didn't handle relationship lines properly.
CC @monkeyman192
Steps to reproduce

func _ready() -> void:
	var tree = $Tree as Tree
	tree.create_item().set_text(0, "Root")
	tree.create_item(tree.get_root()).set_text(0, "Child 1")
	tree.create_item(tree.get_root()).set_text(0, "Child 2")
	tree.get_root().get_first_child().get_next().set_visible(false)

Enable theme_override_constants/draw_relationship_lines in $Tree
Run

Minimal reproduction project
No response

Ah whoops! 😅 I didn't even realise this was an option when testing...
I'll open a PR with a fix 😄


Exclude MSLoginSafariViewController from Mac target as it depends on iOS specific SFSafariViewController

Most of the MSLoginSafariViewController references had already been scoped to iOS only, we should fix the remaining to make the project compiled on Mac platform again, even though that implies lack of login support on Mac platform.

@HongruiZhang,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.
It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

@HongruiZhang, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, Microsoft Pull Request Bot

@HongruiZhang Thanks for your pull request. We really appreciate it. 👍

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

@HongruiZhang Could you sign the Microsoft Contributor License Agreement again? We will merge your PR once it is signed. Thanks!

@HongruiZhang We appreciate your contribution. Because Contributor License Agreement is not signed yet, I transferred this changes to #166 to expedite release of v3.4.0 of iOS SDK.


Removing a reference/dependency not working as expected

Using Visual Studio 2015 v14 Update 3 with .Net Core 1.0.0
Regarding the removal of a project dependency from another project in a solution:
Doing so should break any usage of that assembly/namespace in the project, yet in some cases no errors are raised on rebuild. When this happens, we don't know there are build errors until we push a commit and our CI environment tries to build and deploy (at that point it blows up).
I've tried the following with no success:

Removing the reference both manually in project.json, and using the VS right-click context
A Clean and Rebuild after removing
Building on a different configuration after removing
Manually deleting the "bin" and "obj" folders under said project after removing
Running "dotnet restore" and "dotnet build" from CMD after removing

Anyone else run into this?

Please re-activate if this is still the case.
/cc @blackdwarf


Making AuthResponse public so as to use this: "directus.auth.storage.storeLoginData(data)"

This is  to set accesstoken and refresh token manually. Saw a method storeLoginData() for this, but it expects AuthResponse as param, which is a private class.
It would be helpful if it could be made accessible.

There are a couple of things that must happen when you set access token, look at login method.
It would leave library state in incorrect state
Should be something like this:
      await storage.storeLoginData(data);

      tokens = loginDataResponse;
      currentUser = CurrentUser(client: client);
      tfa = Tfa(client: client);

I am open for PR that adds additional method on auth_handler.dart, something like
Future<void> setAuthState(...) {
///
}


rooftop solar prior to march 2018 has gone missing

* NSW goes back to ~october 2015 because there was utility solar in the state.

there have been issues with WA Rooftop Solar on the main Energy graphs but only when the duration = "ALL".
in the past the historical years where there but no data from 1 dec 2022 onwards (IIRC)
Today it's all gone.
I'll log it as seperate bug but it may be related.


Ajax Forms and back button

I always assign an Id to my elements so that I can easily use goTo to access them. This is a problem with ajax forms however. Suppose you have an ajax form with id "formId". Then, suppose the user submits the form but there are some errors, so the same form is returned from the server. If the user decides to cancel and presses the back button, nothing happens because the history tries to goBack to "formId", but formId already has the "currentAttribute". The function goBack should goBack to the previous element with a different Id. If I remove the Id attribute, when the user presses back on the form with incorrect data, it brings the user back to the empty form, and another back returns to the correct place. A single back should be tapped.

Fixed.


densenet memory error

I use your densenet.py file to generate a pre-training network, but the network training process will increase over time, the use of GPU memory is also increasing, until the OOM.Why is this?

Hi @chencodeX I can't see anything obviously wrong in the densenet model, perhaps you are hanging on to tensors in your training loop. Would you be able to post a minimum working example showing this behavior?

it's ok!
def forward(self, x):
x[:, 0] = (x[:, 0] - 0.485) / 0.229
x[:, 1] = (x[:, 1] - 0.456) / 0.224
x[:, 2] = (x[:, 2] - 0.406) / 0.225
features = self.features(x)
# temp_size = features.size(0)
out = F.relu(features, inplace=True)
out = F.avg_pool2d(out, kernel_size=7).view(features.size(0), -1)
out = self.classifier(out)
return out
I made a little change to your code and typed the data for an operation.
Now I already know how to fix this problem, I added a line of code:
x = x.clone()
like this:
def forward(self, x):
x = x.clone()
x[:, 0] = (x[:, 0] - 0.485) / 0.229
x[:, 1] = (x[:, 1] - 0.456) / 0.224
x[:, 2] = (x[:, 2] - 0.406) / 0.225
features = self.features(x)
# temp_size = features.size(0)
out = F.relu(features, inplace=True)
out = F.avg_pool2d(out, kernel_size=7).view(features.size(0), -1)
out = self.classifier(out)
return out
This ensures that the cache space occupied by the input data is released.
I really recommend that you also add such a line to the original model code.

Is this a problem specific to densenet? I wonder if you are not hitting an issue similar to https://github.com/pytorch/pytorch/issues/839, which was solved long ago. What is your pytorch version?
print(torch.__version__)

I was the latest version of pytorch installed in August

@chencodeX I can't reproduce this bug and can't see anything obviously wrong in the forward code you posted (although the forward code above is not the forward code of densenet).
Is this still an issue for you or did you find a leak elsewhere in your training loop?

Yes, I am in the process of training program GPU memory found in increasing until overflow.

@chencodeX is the memory increase due to something else in your training loop. It doesn't seem like it's due to the densenet model as far as I can tell?
Can you pose a minimum working example exhibiting this increase in memory, and perhaps that will help us figure out what the issue is

I am sorry, because the server exception, the previous data and the code has been lost, I tried to reproduce the situation today, but the network is working properly, there is no increase in GPU memory situation.

Ok, closing this for now then. Please open another issue if you can reproduce in the future :)


Add config for skipDeletionAdditionalGuardCheck pattern

Temporary builds delete the tracking records after it is done. This breaks the guard check. This pr add a pattern to skip the checks for temporary builds.

Do you know if that's intent to delete the folo records just after builds for temporary one ? Not sure if we should break the guardCheck here.

Do you know if that's intent to delete the folo records just after builds for temporary one ? Not sure if we should break the guardCheck here.

Let me ask Dustin.


Baking at runtime ignores the 'Include Layers' property.

When i bake a navmesh at runtime it appears to ignore the 'Include Layers' property in the navmeshsurface component. This creates an unusable navmesh as it goes around the agents. When baking the navmesh offline using the 'Bake' button on the surface component it works as expected.
Offline bake:

Runtime bake:

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using UnityEngine.AI;

public class NavigationManager : MonoBehaviour
{
    public NavMeshSurface surface;

    public void Rebake()
    {
        if (surface)
        {
            surface.BuildNavMesh();
        }
    }
}

Any feedback on this?

Any luck here? I'm having the same issue

I could not reproduce the issue with Unity 2019.4.18f1. The NavMes is baked correctly taking into account the layers. I've tested with this code.
using UnityEngine;
using UnityEngine.AI;

public class BuildNavMeshAtRuntime : MonoBehaviour
{
    NavMeshSurface m_Surface;

    void Start()
    {
        m_Surface = GetComponent<NavMeshSurface>();
    }

    void Update()
    {
        if (m_Surface && Time.frameCount % 200 == 0)
        {
            m_Surface.defaultArea = 2 + (m_Surface.defaultArea + 1) % 28;
            m_Surface.BuildNavMesh();
        }
    }
}

https://user-images.githubusercontent.com/9038468/106135697-8f15dd80-6168-11eb-8f78-557268a1d76b.mp4

I could not reproduce the issue with Unity 2019.4.18f1. The NavMes is baked correctly taking into account the layers. I've tested with this code.
using UnityEngine;
using UnityEngine.AI;

public class BuildNavMeshAtRuntime : MonoBehaviour
{
    NavMeshSurface m_Surface;

    void Start()
    {
        m_Surface = GetComponent<NavMeshSurface>();
    }

    void Update()
    {
        if (m_Surface && Time.frameCount % 200 == 0)
        {
            m_Surface.defaultArea = 2 + (m_Surface.defaultArea + 1) % 28;
            m_Surface.BuildNavMesh();
        }
    }
}

https://user-images.githubusercontent.com/9038468/106135697-8f15dd80-6168-11eb-8f78-557268a1d76b.mp4


Download from a directory

In the section "Download from a directory" [1] we found the line:
downloaded_bytes = file_client.read_file()
But file_client is a DataLakeFileClient class, and that class doesn't have read_file() function [2]
[1] https://docs.microsoft.com/en-us/azure/storage/blobs/data-lake-storage-directory-file-acl-python#download-from-a-directory
[2] https://github.com/Azure/azure-sdk-for-python/blob/74a9df155747671b60136544cfa27070a3d1ecd1/sdk/storage/azure-storage-file-datalake/azure/storage/filedatalake/_data_lake_file_client.py

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

ID: 12220c1c-ebce-cf63-0ef7-41c6b6747329
Version Independent ID: 92bd839f-13a7-296d-c11d-f88fabca61e3
Content: Azure Data Lake Storage Gen2 Python SDK for files & ACLs
Content Source: articles/storage/blobs/data-lake-storage-directory-file-acl-python.md
Service: storage
Sub-service: data-lake-storage-gen2
GitHub Login: @normesta
Microsoft Alias: normesta

@GuardatiSimone Thank you for your query , our team will further look into it and get back to you at the earliest.

I agree with GuardatiSimone. One additional step is needed:
download = file_client.download_file()
downloaded_bytes = download .readall()

The PR for the fix has been created and it will be merged soon. Thanks again for pointing this out. I am closing the issue based on this update, please let us know if there is something else that we can help you with.


Tests: check side-effects of pollution protection

There are separate checks against the leading keys in a dotted option name than against the last key. If the checks for __proto__ or constructor are wrong in the checks of the last key then it won't lead to prototype pollution as such, but will lead to less consistent behaviour between --a.constructor.prototype.b and --a.constructor.
The pollution checks on the v0.2.x branch missed the constructor check on the last key in the dotted option name. So again this does not allow prototype pollution, but does mean there is different behaviour between the main branch and the v0.2.x branch.
This PR adds a test which fails due to the missing code, and the failing test will be fixed by #24. (Added separately so can see the test fail. Tests added high in the file to avoid a future merge conflict on mainline to slightly simplify copying up.)
If this seems worthwhile, this could be merged separately or as part of #24?
not ok 103 should be deeply equivalent
  ---
    operator: deepEqual
    expected: |-
      {}
    actual: |-
      { constructor: [ [Function: Object], 'IGNORED' ] }
    at: Test.<anonymous> (/Users/john/Documents/Sandpits/minimist/my-fork/test/proto.js:18:4)

If not, then I think it should be part of the PR that fixes it.

I'll add to other PR.


how to fuzz a shared library, like libstagefright.so, with android-afl

how to fuzz libstagefright.so?
compile libstagefright.so and program with afl-gcc, dlopen it and call some function using its symbol in the program?

if want to fuzz function from lib's, need write some wrapper in C/C++

I got it, compile the lib to be a static lib, and your wrapper static link the lib

@JiounDai can you send me your contact, I have some question for you.
thank you

@JiounDai compile the lib to be a static lib with afl-gcc or default?

@Daemonceltics  yeah, Thank you!

Can we fuzz a close-source Android shared library?


C_CPP: Introduction to C

Description 📜
I want to contribute using videography on the following:

Introduction to C
Programming components, Structure & Keyword
Operators & Expressions
Decision making & Branching Structure
Looping Control structure/statements

Domain of Contribution 📊

[x] C/CPP

/assign

/assign


net core WPF “TouchEnter”  evnet have trublle

Please ask how to solve it.

Duplicate of #97. A quirk of WPF means you can't use actions with TouchEnter, sorry. Use an event in the code behind.


Description of tracing file parameters

Description of issue (what needs changing)
What is the difference between fps and frequency in the mediapipe graph tracing results displayed on the viz.mediapipe.dev?
Clear description
I would like to see formulas and examples of calculating various parameters of the tracing file
Correct links
No response
Parameters defined
No response
Returns defined
No response
Raises listed and defined
No response
Usage example
Can I compare the speed of two different graphs, based on the frequency parameter of the last calculator? Or what is the best way to compare them?
Request visuals, if applicable
No response
Submit a pull request?
No response

Sorry, found documentation


TypeError: fit() got an unexpected keyword argument 'show_accuracy' ---- Graph().fit

I am training the Graph model with using fit with parameter show_accuracy. However, I got the error:
TypeError: fit() got an unexpected keyword argument 'show_accuracy'

When I check the source code on github here, it shows that Graph's fit has show_accuracy:
def fit(self, data, batch_size=128, nb_epoch=100, verbose=1, callbacks=[],
        validation_split=0., validation_data=None, shuffle=True,
        show_accuracy=False,
        class_weight={}, sample_weight={}):

But when I check the release version 0.3.2, it is like:
def fit(self, data, batch_size=128, nb_epoch=100, verbose=1, callbacks=[],
        validation_split=0., validation_data=None, shuffle=True,
        class_weight={}, sample_weight={}):

I am wondering, which one is the newest version?
I used this pip install git+git://github.com/fchollet/keras.git to install Keras. Even this, the Graph's fit  I got has no show_accuracy parameter.

Hi fluency03,
I checked my version (which was downloaded using the command pip install git+git://github.com/fchollet/keras.git) and I see the following code:
def fit(self, data, batch_size=128, nb_epoch=100, verbose=1, callbacks=[],
            validation_split=0., validation_data=None, shuffle=True,
            show_accuracy=False,
            class_weight={}, sample_weight={}):

Did you previously install Keras using the command, sudo pip install keras?
If so, maybe you should try upgrading your installation with the following command:
sudo pip install --upgrade git+git://github.com/fchollet/keras.git

I have done this sudo pip install git+git://github.com/fchollet/keras.git. Is it necessary to add --upgrade?

If you have any previous installation of Keras, then even if you call sudo pip install.... will not make any updates. You have to specify that you want to upgrade your existing installation with the --update flag.
Hope this helps!

@jisungk It's solved. thanks

model.fit(X_train, Y_train,
batch_size=128, nb_epoch=4,
show_accuracy=True, verbose=1,
validation_data=(X_test, Y_test))
get the error
TypeError                                 Traceback (most recent call last)
 in ()
2           batch_size=128, nb_epoch=4,
3           show_accuracy=True, verbose=1,
----> 4           validation_data=(X_test, Y_test))
/usr/local/lib/python3.6/dist-packages/keras/models.py in fit(self, x, y, batch_size, epochs, verbose, callbacks, validation_split, validation_data, shuffle, class_weight, sample_weight, initial_epoch, steps_per_epoch, validation_steps, **kwargs)
983             epochs = kwargs.pop('nb_epoch')
984         if kwargs:
--> 985             raise TypeError('Unrecognized keyword arguments: ' + str(kwargs))
986
987         if not self.built:
TypeError: Unrecognized keyword arguments: {'show_accuracy': True}

what about the error fit_generator() got an unexpected keyword argument 'samples_per_epoch'?My keras version is 2.4.1.


[TASK] Investigate chart-verifier tool benefits

Description
RedHat provides an open source tool called chart-verifier which lets you run some predefined and custom checks against your chart.
As a first step, we can look into the existing checks to see if any of them are useful for us, integrate it into our workflow (you can use the openshift-tools-installer action in CI to install this tool), and then we could think about custom checks.
This is a suggestion, as we don't have yet an ad-hoc place to talk about the charts as a product.

I ran the tool on our latest chart and this is the output that I got:
apiversion: v1
kind: verify-report
metadata:
    tool:
        verifier-version: 1.13.4
        profile:
            VendorType: partner
            version: v1.3
        reportDigest: uint64:13503106373664558913
        chart-uri: /charts/
        digests:
            chart: sha256:7dce5d259b555215d4e12f7e2b8776c37b8bc9dc7f06930a067a48e626b72a17
        lastCertifiedTimestamp: "2024-06-20T22:05:47.756963+00:00"
        testedOpenShiftVersion: N/A
        supportedOpenShiftVersions: N/A
        webCatalogOnly: false
    chart:
        name: camunda-platform
        home: https://docs.camunda.io/docs/self-managed/overview/
        sources:
            - https://github.com/camunda/camunda-platform-helm
        version: 10.1.1
        description: 'Camunda 8 Self-Managed Helm charts. Camunda''s process automation platform allows developers to design, automate and improve processes. '
        keywords:
            - bpm
            - bpmn
            - camunda-platform
            - dmn
            - process-engine
            - workflow
        maintainers:
            - name: Zelldon
              email<EMAIL_ADDRESS>              url: ""
            - name: aabouzaid
              email<EMAIL_ADDRESS>              url: ""
        icon: https://helm.camunda.io/imgs/camunda.svg
        apiversion: v2
        condition: ""
        tags: ""
        appversion: 8.5.x
        deprecated: false
        annotations:
            artifacthub.io/changes: |
                - kind: added
                  description: "add console auth vars"
            artifacthub.io/containsSecurityUpdates: "false"
            artifacthub.io/links: |
                - name: Camunda 8 docs
                  url: https://docs.camunda.io/
                - name: Camunda 8 Self-Managed docs
                  url: https://docs.camunda.io/docs/self-managed/about-self-managed/
                - name: Camunda 8 Self-Managed Helm charts
                  url: https://github.com/camunda/camunda-platform-helm
            camunda.io/helmCLIVersion: 3.15.1
            dummy: dummy
        kubeversion: ""
        dependencies:
            - name: keycloak
              version: 19.4.1
              repository: https://charts.bitnami.com/bitnami
              condition: identity.keycloak.enabled,identityKeycloak.enabled
              tags: []
              enabled: false
              importvalues: []
              alias: identityKeycloak
            - name: postgresql
              version: 12.x.x
              repository: https://charts.bitnami.com/bitnami
              condition: identity.postgresql.enabled,identityPostgresql.enabled
              tags: []
              enabled: false
              importvalues: []
              alias: identityPostgresql
            - name: web-modeler-postgresql
              version: 11.x.x
              repository: file://../web-modeler-postgresql
              condition: postgresql.enabled
              tags: []
              enabled: false
              importvalues: []
              alias: postgresql
            - name: elasticsearch
              version: 20.0.4
              repository: oci://registry-1.docker.io/bitnamicharts
              condition: elasticsearch.enabled
              tags: []
              enabled: false
              importvalues: []
              alias: ""
            - name: common
              version: 2.x.x
              repository: https://charts.bitnami.com/bitnami
              condition: ""
              tags: []
              enabled: false
              importvalues: []
              alias: ""
        type: application
    chart-overrides: ""
results:
    - check: v1.0/helm-lint
      type: Mandatory
      outcome: FAIL
      reason: |
        Helm lint has failed:  [ERROR] templates/: template: camunda-platform/templates/zeebe-gateway/deployment.yaml:23:28: executing "camunda-platform/templates/zeebe-gateway/deployment.yaml" at <include (print $.Template.BasePath "/zeebe-gateway/configmap.yaml") .>: error calling include: template: camunda-platform/templates/zeebe-gateway/configmap.yaml:44:29: executing "camunda-platform/templates/zeebe-gateway/configmap.yaml" at <include "camundaPlatform.authIssuerBackendUrl" .>: error calling include: template: camunda-platform/templates/camunda/_helpers.tpl:195:10: executing "camundaPlatform.authIssuerBackendUrl" at <include "identity.keycloak.url" .>: error calling include: template: camunda-platform/templates/identity/_helpers.tpl:193:9: executing "identity.keycloak.url" at <include "identity.keycloak.host" .>: error calling include: template: camunda-platform/templates/identity/_helpers.tpl:161:12: executing "identity.keycloak.host" at <include "identity.keycloak.hostDefault" .>: error calling include: template: camunda-platform/templates/identity/_helpers.tpl:128:17: executing "identity.keycloak.hostDefault" at <{{template "common.names.fullname" .Subcharts.identityKeycloak}}>: template "common.names.fullname" not defined
        [WARNING] /root/.cache/helm/repository/chart-verifier/_charts_/camunda-platform: chart directory is missing these dependencies: keycloak,postgresql,web-modeler-postgresql,elasticsearch,common
    - check: v1.0/contains-test
      type: Mandatory
      outcome: PASS
      reason: Chart test files exist
    - check: v1.0/has-readme
      type: Mandatory
      outcome: PASS
      reason: Chart has a README
    - check: v1.0/required-annotations-present
      type: Mandatory
      outcome: FAIL
      reason: 'Missing required annotations: [charts.openshift.io/name]'
    - check: v1.0/is-helm-v3
      type: Mandatory
      outcome: PASS
      reason: API version is V2, used in Helm 3
    - check: v1.1/has-kubeversion
      type: Mandatory
      outcome: FAIL
      reason: Kubernetes version is not specified
    - check: v1.0/contains-values-schema
      type: Mandatory
      outcome: FAIL
      reason: Values schema file does not exist
    - check: v1.0/not-contain-csi-objects
      type: Mandatory
      outcome: PASS
      reason: CSI objects do not exist
    - check: v1.1/images-are-certified
      type: Mandatory
      outcome: PASS
      reason: No images to certify
    - check: v1.0/chart-testing
      type: Mandatory
      outcome: FAIL
      reason: 'invalid configuration: no configuration has been provided, try setting KUBERNETES_MASTER environment variable'
    - check: v1.0/not-contains-crds
      type: Mandatory
      outcome: PASS
      reason: Chart does not contain CRDs
    - check: v1.0/contains-values
      type: Mandatory
      outcome: PASS
      reason: Values file exist
    - check: v1.0/signature-is-valid
      type: Mandatory
      outcome: SKIPPED
      reason: 'Chart is not signed : Signature verification not required'
    - check: v1.0/has-notes
      type: Optional
      outcome: PASS
      reason: Chart does contain NOTES.txt

I will investigate if any of the checks are useful. And I will also see if it is possible to run only a defined number of checks.

This is how I ran the tool:
Pull the image:
docker pull --platform linux/amd64 quay.io/redhat-certification/chart-verifier:latest

cd into the latest chart and run:
docker run --platform linux/amd64 -v ./charts quay.io/redhat-certification/chart-verifier:latest verify /charts/


Perfect scrollbar overlapping resize element

I have divs that use the perfect scroll bar. However these divs are also resizeable. The problem I am having is that the scroll bar is overlapping the re size element and when trying to resize I just end up grabbing the scroll bar. Has anyone dealt with this issue already?

As I understand, the divs are resized using a custom method? Then as a naive workaround, you can disable(or destroy) PS when starting to resize, and re-initialise when finishing.
If my understanding was wrong, could you specify the case a bit more, preferably with an example or a jsfiddle?

Closed for inactivity. Please feel free to reopen.


Upgrade Linux Kernel for flatcar-3510 from 5.15.133 to 5.15.134

Subject says it all.

See instead https://github.com/flatcar/scripts/pull/1248