[BUG] Tag generation as separate folders

Mandatory Step

[x] I am using latest production release of Elegant

Recommended Step

[ ] I have attached link to my blog source code repository, so that developers can try to reproduce the issue at the end
[ ] I can't my source code but I have linked my pelican configuration files to help developers reproduce the issue

Description
Describe the bug
Currently, Pelican generates a page for tags/categories which lists all of them. This is ok, but it would be great to also have pointer to 'tags' as in url, for example:

https://next.elegant.oncrashreboot.com/tags/admonitions

vs what we actually have at:

https://next.elegant.oncrashreboot.com/tags#admonitions-ref

Would it be possible to enable 'both' ?
To Reproduce
Generate any website with elegant
Expected behavior
Additional url for just the tags, this will help people coming from other themes to Pelican not to get 'broken urls'

This is a not a bug. It's an intentional feature. Elegant is a zen inspired minimal theme.
Let me know why a separate tag page is better than what we currently have? Any specific feature that we can't add to current design?

Hi Talha , no, it's not better, but if we can generate 'both' 'or' make the 404 page to 'redirect' to the tags#reftag it would be good.
Just happened because my previous theme was using it and got email from 'search console' complaining that url was no longer available

To solve this problem, you need to setup redirects from your example.com/tags/eg to example.com/tags#eg.
How to accomplish this? It depends on your choice of hosting provider.
We use Netlify, which lets us define redirect rules in a text file. Here is our rules
https://github.com/Pelican-Elegant/elegant/blob/master/documentation/content/extra/_redirects
You would need to do something similar with your site.
If your host does not provide this option, then you can use Elegant redirection feature
https://elegant.oncrashreboot.com/article-redirect
but I don't think this a better solution. For your use case, a 301 redirect is a better solution.

Yes, I know, or use `.htaccess.. as with regular website but wanted to see if there's something we could do from Elegant side to have it covered (if we were willing to pursue that goal)

Alternatively, what I did at my website before using pelican, was to use a custom 404 that redirected to 'search', like the 404, to pass the url field except site to the:
https://next.elegant.oncrashreboot.com/search.html?q=admonition
If we convert '/' in the url to 'spaces' and redirect or provide link to user to 'perform search' on website will make it easier

Sorry. I don't think we should add a feature that changes a core feature of Elegant that was added explicitly, when 301 redirection is a much more suitable solution.

If we convert '/' in the url to 'spaces' and redirect or provide link to user to 'perform search' on website will make it easier

Let me know if I understand you correctly, when user gets to a 404, you want to autofill the search field with the url.

Yes, or even 'autosubmit' it and show it with a custom header saying 'the page you were looking for wasn't found, here are some alternates that might be helpful'

Agree. That would be nice to have.
Can you take up this issue?

I can try to!
Sent from mobile
El jue., 1 ago. 2019 9:12, Talha Mansoor<EMAIL_ADDRESS>escribió:

Agree. That would be nice to have.
Can you take up this issue?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Pelican-Elegant/elegant/issues/476?email_source=notifications&email_token=AACMJD77XRSJK2K7I2V2NKTQCKEHLA5CNFSM4IINGSF2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3JR7OY#issuecomment-517152699,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACMJD4ARIRWXYI6XWMBJW3QCKEHLANCNFSM4IINGSFQ
.

I have added this task to roadmap, assigned to you and updated labels.
There is no rush. Take your time to implement. But if you lose interest then please update this issue tracker.

I think we might need to do it with javascript, as the site gets rendered before uploading, hence the 404.html template should contain the code to convert the current location into filling the form default value
are we ok with that approach? other ideas?

the 404.html template should contain the code to convert the current location into filling the form default value

That's the only way to do it.

A JS script that works only inside 404 page
If URL is present (after removing the domain part) then place it inside search box
else do nothing

can you check https://github.com/Pelican-Elegant/elegant/pull/478 ?
Contains a link to an example on the netlify site. If behaviour is ok, I can proceed to documenting

We might also consider to remove some 'other words' like 'tags' etc to make it easier to find results and maybe tuning the text above the search box to 'remind' user to press 'search' (or directly perform it)


fix(ol): fix errors with gcc's compilation of ol namespaces

GCC assumes ol already exists due to including typedefs.js as an externs file. It also never creates the ol.ext namespace, presumably due to how those files are included in the compilation which results in Closure not replacing their
goog.provide statements properly.

I added @suppress {checkTypes} to the file so GCC will ignore the actual OL types referenced in the JSDocs. That removes the need to goog.require anything, which either needed to require every referenced namespace in the file or rely on those to be implicitly required elsewhere.


Undefined Index: introImage

Keep getting this error with Auditlog enabled when uploading an image to a field in Craft.
/craft/plugins/auditlog/services/AuditLogService.php(165)
$diff[$key]['label'] = $before[$key]['label'];

Fixed in 0.7.1


How much memory is required for editing Qwen-14B with ROME or MEMIT?

I checked out the memory requirement in the README but Qwen series models are not mentioned. I experimented with an H800 with 80G, which works fine with Qwen-7B but gets OOM with Qwen-14B (both ROME and MEMIT). Can you provide the memory requirements for Qwen-14B?

When running Rome and Memit locally, they respectively occupy 66GB and 74GB of VRAM. You can try modifying the hyperparameters to reduce VRAM usage.

Hi, do you have any further questions?

I am confused about the function of hyperparameters. Is there any instruction that can help me tune the hyperparameters in order to reduce VRAM usage?


[AppConfig-LiveTest] Refactor to async call to pass live test

Fixed the com.azure.core.exception.HttpResponseException: Status code 401, (empty body) live test error message.

/azp run java - appconfiguration - tests

API change check
API changes are not detected in this pull request.


Create BasicCharacterObject Delegate

Make it so BasicCharacterObject is fully serializable. This includes all BasicCharacterObject fields/stored properties.
Definition of Done

[ ] BasicCharacterObjectSurrogate is fully implemented
[ ] CharacterObject has a test that demonstrates full serialization

Not a surrogate anymore, this is now a dynamic serializer

Using new serialization method


Android Demo doesn't work

This isn't a copy of #68. I've been trying to run both this project's demo and this separate one with no success. Here's the error I'm getting.
Please note that I've console.log'd every single component imported in demo.js, and none of them is undefined.

I'm running Windows 10 on my PC, react-native-cli is 2.0.1 and react-native is the same exact version as the required dependencies in each package.json.

@jkomyno i'm sorry about the errors! let's concentrate on getting https://github.com/FormidableLabs/victory-native-demo to work, as that's the one we've tested the most with Android. It looks like that is what your screenshot is from, correct?
I haven't done is test on Windows + Android; I'm hoping this isn't an issue there.
Could you try using https://github.com/FormidableLabs/victory-native-demo as a base and just render a simple react-native-svg test? Here's one from their readme:
import Svg,{
    Circle,
    Ellipse,
    G,
    LinearGradient,
    RadialGradient,
    Line,
    Path,
    Polygon,
    Polyline,
    Rect,
    Symbol,
    Text,
    Use,
    Defs,
    Stop
} from 'react-native-svg';

class SvgExample extends Component {
    render() {
        return (
            <Svg
                height="100"
                width="100"
            >
                <Circle
                    cx="50"
                    cy="50"
                    r="45"
                    stroke="blue"
                    strokeWidth="2.5"
                    fill="green"
                />
                <Rect
                    x="15"
                    y="15"
                    width="70"
                    height="70"
                    stroke="red"
                    strokeWidth="2"
                    fill="yellow"
                />
            </Svg>
        );
    }
}```

Note that Expo includes their own version of react-native-svg, so you won't need to install anything else.

If this doesn't work, could you verify that a basic https://github.com/react-community/create-react-native-app application works on your system?

My understanding is that VictoryArea is broken, since I can render every other component without issues. If I comment it out, the example works well.

I have no issues with react-native-svg or with the other components imported by victory-native, at all.

Ah! you hit https://github.com/FormidableLabs/victory-native/issues/115 then. That is something I was never able to reproduce locally, but a number of users hit it. they were able to fix it with master, which was released as<EMAIL_ADDRESS>

Good to know, thanks! As of now I have @0.12.0 installed in an app I'm working on at the office, so I will upgrade and see if VictoryArea works as expected!
I'll let you know in some hours, now I have lunch break

sounds good! with these spooky react-native bugs, i always recommend a fresh install of all dependencies (especially if a simple upgrade doesn't work the first time)

It works: I can finally use VictoryArea both in the demo and in my app!
Thanks for the update.

thanks for the update @jkomyno! This has been helpful for me -- I know see the underlying issue was conflicting deps. Will fix the demo repo.

@chrisbolin I have just noted that after upgrading to<EMAIL_ADDRESS>the VictoryPie layout seems broken. Related issue: #125

@jkomyno thanks! i'll make sure the demo depends on a "good" version of victory-native

@jkomyno that was a release timing issue on our part. The VictoryPie layout is fixed in version 0.14.0


Using a fullWidth TextField and InputAdornment messes with the input width

When I use a TextField with an Adornment through the InputProps, if I set the width of the TextField component, either with fullWidth or with style property, the input field remains at the default width, only the root element takes the full space.

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior
The input should receive the properties from the TextField even if I use the InputAdornment or not. Without it, the component renders ok.
Current Behavior

The root element changes its width, the input not.
Steps to Reproduce (for bugs)
Minimum example:
https://codesandbox.io/s/k2wn0166y7
Your Environment

Tech
Version

Material-UI
1.0.0-beta.18

React
16.0.0

browser
Google Chrome 61

etc

+1 for this.
@oliviertassinari would it be possible to support <CircularProgress /> too as adornment on inputs?
Very nice to have on selects which load options asynchronously :)

@damianobarbati Have you tried doing so? How does it render? Do you see override generic enough we can move into the core components?

Yes, with the following (excerpt):
render = () => {
    ....
    const endAdornment = <CircularProgress />;

    const input = (
        <Input
            type={'text'}
            name={name}
            onFocus={onFocus}
            onBlur={onBlur}
            maxLength={maxLength}
            disabled={disabled}
            readOnly={readOnly}
            placeholder={placeholder}
            startAdornment={startAdornment}
            endAdornment={endAdornment}
            fullWidth={true}
            margin={'dense'}
        />
    );

    const selectValue = !multiple ?
        String(value || '') :
        (value || []) ;

    return (
        <FormControl className={cx(className)} fullWidth={true} required={!!required} error={!!errors}>
            {!disableLabel &&
            <InputLabel htmlFor={name}>{displayName || name}</InputLabel>
            }
            <MuiSelect input={input} value={selectValue} multiple={multiple} onChange={onChange} classes={{ icon: cx({ [classes.hideMoreIcon]: computing }) }}>{options}</MuiSelect>
            { !errors && description &&
            <FormHelperText>{description}</FormHelperText>
            }
            { errors &&
            <FormHelperText>{errors}</FormHelperText>
            }
        </FormControl>
    );
}

It renders the <CircularProgress /> on the left "fullsize":

Maybe whatever is passed along as adornment should be constrained to respect the Input height "no matter what"?
See https://github.com/callemall/material-ui/pull/8504/files#diff-3d0c58268ffed10a2bd03ed4b6799ebbR53 for an example.

@Jannis same with explicit endAdornment :)
const computingAdornment = (
    <InputAdornment position={'end'}>
        <CircularProgress />
    </InputAdornment>
);

<Input
    ...
    ...
    endAdornment={computedEndAdornment}

Ok, I thought it might be worth checking. :)

If anyone's still struggling with this, this'll fix it for you.
    amountField: {
      boxSizing: 'border-box',
    },

....

<TextField
  fullWidth
  InputProps={{
    className:classes.amountField,
    startAdornment: <InputAdornment position="start">$</InputAdornment>,
  }}
/>

@turbochef You should be able to remove this style in v4. It was fixed in #14988.


Drag and Drop Sorting using Jquery-ui

Hi, I wanted to share an implementation which solves [Feature] Manual sort order #192.
This adds a dependency (jquery-ui) so that photos can be dragged and dropped to rearrange their order. An INT "position" field needs to be added to your lychee_photos table for this to work and update "sortingPhotos" in settings to "ORDER BY position ASC".
On the server side it re-orders the whole album collection on every update to preserve 0-n ordering, so may become slow for extremely large albums.
lychee.zip

@globalchangemedia You were unable to make it into a pull request from your branch ??

@fly-man-  "position" should be added to the lychee_photos database table as an INT. It then functions to define the Photo position within the Album. Does that answer your question?
Apologies that I was unable to make this a formal pull request.

@globalchangemedia As you can see from the commit, position is added to the database as an INT.
But when you open the Options menu and go to Settings, you are presented with the options how to sort an Album and Photo's. To change that to work with positioning, i'd have to add a new option. My question was what to name that option so it will trigger the ORDER BY position ASC
Example:

This PR has been merged in a bulk merge.


message can not send,becaues channel is closed

消费者和服务提供方不在一个机房，进行网络调整，比如机房间网络专线调整，消费端产生message can not send 报错，消费端无法进行自动恢复，每次重启消费端服务才能恢复？

I am afraid this is the case now. Because the Provider address registered on zk is wrong (due to network adjustment).

@thanks0rf
I will close this issue.
Reopen if still have problems with it.


Adjust tunables, fix issues with start/stop timers, what entries removed, etc

Adjust tunables, start/stop of discard timer and when to discard
Adjusted the tunables based on more testing.

Fix issues related to when we start/stop the timers

Fix bug where we were incorrectly discarding inodes from the inode cache.

Moved execution of for() loop under if statement since no need to test for() loop
if if statement did not pass.

I updated the for loop.  I am now rebuilding so that we can start our long running test.


Add support TypeScript

Add support TypyScript
Install TypeScript before using:
$ npm install ts-loader typescript --save-dev

Usage:
// webpack.mix.js
let mix = require('laravel-mix');
mix.ts('src/hello-word.ts', 'dist/');

please add tests

Current, test script not run my enviroment (on master banch without modified)
I will add test after it run.

I think this needs an official documentation on the Laravel Mix Github wiki.


Applying to the Cifar 10

Thank you for your sharing. I think it is really effective API. But I got  a problem
When I apply to the CIFAR 10 images, I got this error.
How I can fix it?
Train on 50000 samples, validate on 10000 samples
Epoch 1/10
49984/50000 [============================>.] - ETA: 0s - loss: -55.6413 - iou_score: 4.4150
InvalidArgumentError                      Traceback (most recent call last)
 in ()
27     batch_size=32,
28     epochs=10,
---> 29     validation_data=(x_val, y_val),
30 )
5 frames
/usr/local/lib/python3.6/dist-packages/tensorflow/python/framework/errors_impl.py in exit(self, type_arg, value_arg, traceback_arg)
526             None, None,
527             compat.as_text(c_api.TF_Message(self.status.status)),
--> 528             c_api.TF_GetCode(self.status.status))
529     # Delete the underlying status object from memory otherwise it stays alive
530     # as there is a reference to status from this from the traceback due to
InvalidArgumentError: Incompatible shapes: [16,1] vs. [16,32,32,1]
[[{{node loss_18/sigmoid_loss/mul_1}}]]
[[{{node loss_18/mul}}]]

You are definitely passing wrong data into your network. Could you share your code or you can manually take look your batches by yourself?

Hi @eric333316152
As far as I know, CIFAR is classification dataset, not segmentation, so you have to use another kind of network architectures.

@Diyago @qubvel I think u guys are right .......  CIFAR 10 data is inappropriate for this problem. I am just a AI beginner. So, could you please let me know where I can get data set for segmentation?

@qubvel @Diyago ah before, can u please see my code?
I used data set in other github, which is here : https://github.com/divamgupta/image-segmentation-keras
And here is my code.
from segmentation_models import Unet
model = Unet()
model = Unet('resnet34', encoder_weights='imagenet')
model = Unet('resnet34', classes=51, activation='softmax')
model = Unet('resnet34', input_shape=(None, None, 3), encoder_weights='imagenet')
-> until here, just import your code
import cv2
import numpy as np
ann_img = np.zeros((30,30,3)).astype('uint8')
ann_img[ 3 , 4 ] = 1 # this would set the label of pixel 3,4 as 1
cv2.imwrite( "ann_1.png" ,ann_img )
-> code for preparing for trained code, But I DONT know how to apply to total dataset
model = Unet('resnet34', input_shape=(416, 608, 3), encoder_weights='imagenet')
from keras.preprocessing.image import ImageDataGenerator, load_img
from keras.utils import to_categorical
from sklearn.model_selection import train_test_split
import matplotlib.pyplot as plt
import random
import matplotlib.pyplot as plt
import random
sample = random.choice(x_train)
image = load_img("/content/gdrive/My Drive/dataset1/dataset1/images_prepped_train/"+sample)
plt.imshow(image)
-> just mounting for google-drive of dataset, and checking image
from segmentation_models import Unet
from segmentation_models.backbones import get_preprocessing
from segmentation_models.losses import bce_jaccard_loss
from segmentation_models.metrics import iou_score
BACKBONE = 'resnet34'
preprocess_input = get_preprocessing(BACKBONE)
preprocess input
x_train = preprocess_input(x_train)
x_test = preprocess_input(x_test)
define model
model = Unet(BACKBONE, encoder_weights='imagenet')
model.compile('Adam', loss=bce_jaccard_loss, metrics=[iou_score])
fit model
if you use data generator use model.fit_generator(...) instead of model.fit(...)
more about fit_generator here: https://keras.io/models/sequential/#fit_generator
model.fit(
x=x_train,
y=y_train,
batch_size=32,
epochs=10,
validation_data=(x_test, y_test),
)
And I got this error : AttributeError: 'str' object has no attribute 'ndim'
At first, at the other github, they said I have to preprocess the data for training using this code:
import cv2
import numpy as np
ann_img = np.zeros((30,30,3)).astype('uint8')
ann_img[ 3 , 4 ] = 1 # this would set the label of pixel 3,4 as 1
cv2.imwrite( "ann_1.png" ,ann_img )   -> But I dont know how to apply to in my total code
Please help me !!!

Take some courses, you have to improve your python skills at least.
Every time you create a new model by defining
model = ...
And all previous models are discarded. You have to take kwargs that suit to your case and define model just once, in README there are three different independent examples (one for each particular case).


Support explicitly using HTTP/1.1 protocol version

Still defaults to HTTP/1.0 and offers only limited support for HTTP/1.1
Closes #5.

LGTM :+1:
@cboden unless you have any comments this can be merged and included in the next tag

LGTM


Make it easy to write literal dictionaries

I'm playing with web things and I would like to have some syntax for writing literal dictionaries.
The new Dolphin's literal dynamic arrays, like {3+4. 'Hello' reverse}, could be an easy way to solve this.
We could write {'Hello'->'World'. 'Dolphin' -> 'Smalltalk'} asDictionary/asHash/asMap. Even the syntax looks very nice. I have to admit that I would have loved to still have the opportunity to use {} characters for hashes only, but on the other hand, Pharo uses the same syntax.
Another option to avoid a message send is to use double curly brackets.
Pharo implements #asDictionary in Collection using #associationsDo: for the magic but in Dolphin that wouldn't be possible because the way Array>>associations are built.
What do you think @blairmcg ?

I like the general idea of having a compact form for expressing dictionaries, particularly as the normal Dictionary construction is clunky, even more so than the Array with:+, which brace arrays address. The compactness of brace arrays is nice, both for writing and reading. The small perf boost is a bonus too.
I really don't like the asDictionary thing though because it only works when the array contains associations. The correct general purpose implementation of asDictionary on a sequenceable collection would be a Dictionary of integer keys and the contents as values, although that wouldn't be terribly useful. The arrangement in Pharo is, to put it kindly, inconsistent. Pharo's Collection>>associationsDo: is based on an assumption that the contents of the collection are associations, which only applies for some instances so this is not a general purpose method that belongs in a core hierarchy; it smells badly to me. There is also a SequenceableCollection>>keysAndValuesDo: which does the right thing and passes the indices as the keys, but which is then inconsistent with respect to the inherited associationsDo:, which is assuming that the elements themselves have keys. If one is actually working with a Dictionary, then there are consistent implementations of associationsDo: and keysAndValuesDo:.
There will need to be a better way which does not make me feel unclean :-)

I agree with you about the Pharo inconsistency. But in any case you should assert all elements are associations.
Time ago I wrote something close to the idea, clean but it requires parentheses, and implement #, in Association and then you can write a dictionary this way:
('class' -> 'button'), ('data-input' -> 'true')
Is it easy to read? Maybe.
Does it resembles a Dictionary? Not really.
The best would be:
{'class': 'button', 'data-input': true}
Other option using the current Array pattern:
{{'class' -> 'button'. 'data-input' -> true}}
Area you comfortable with any of them? :)

I like your "best" option as well from the point of view of it being compact and not confused with regular Smalltalk. I'm wary of adding new Smalltalk syntax to the compiler as it goes against the spirit of the language really. I think it has to have proven general worth to justify the reduction in simplicity that results from any addition of more syntax. That said, I'd support any effort to add a JSON format, including adding a compiler for it that could come into play for particular tagged methods. I'm less keen on an inline escape into it (e.g. using your extra pair of curly braces), but not implacably opposed. By using an alternate and well known format that is deliberately not Smalltalk we get the freedom to use different semantics. In another language we can express constant, or partially constant, structures recognised by its compiler, which would allow one to avoid dynamic execution just because that is what you have to do in order to obey the rules of Smalltalk where everything (with very few historic exceptions) is expected to be a message send, including those #<- operations to make an association out of two literal constants. Doing as much as possible at compile time to create largely constant data structures is important for many uses I think. One good example is as a replacement for STL to replace its use for view resources. STL is an (unreadable) literal format that then requires quite slow interpretation to build things, although it is completely general purpose. The current alternative options of performing a lot of dynamic/imperative construction at runtime can make everything slow in a way that is very difficult to optimize- you don't really want a peanut-buttering effect, i.e. an overhead on everything that adds a performance drag one can't eliminate, because it is a little bit here, a little bit there, adding up to a lot. If the data is constant, it should be expressible as constant literals, or at least much as possible. It is also undesirable to create a load of intermediate objects that are discarded - the most common form of hashed-key lookup collection in Dolphin is actually LookupTable, and this doesn't contain associations at all.
As I said, I'm not keen on the asDictionary conversion because these have the appearance of being very general, but are in fact not (it only works for a collection of a particular type of element). I also have to add another of these methods to the collection that is going to be converted for each type of thing I want to convert into. So I need at least asLookupTable, asIdentityDictionary, as well as asDictionary.  asXXX conversions have these problems in general, and are rather over-used I think, probably because of their convenience, brevity, and the binding priority of unary selectors. Now, those are useful qualities, but not enough to outweigh the unclean feeling they give me. I wouldn't object so much to a conversion selector that didn't try to imply complete generality in the way that asDictionary does.
It strikes me that the object that knows how to consume the array of information is the target collection class, and we should be sending a message to it that asks it to produce an instance from a known input. What you are really saying, expressed in long form is Dictionary fromArrayOfAssociations: {'class' -> 'button', ...}, and one could have a similar instance creation method for LookupTable, or any other collection type one added. This could then be abbreviated using an appropriate binary selector, e.g. <<, which has the advantage of brevity and reduces the need for parenthesis, but loses information and then has some of the less desirable qualities of asXXX. I don't think it is as bad, particularly if the chosen binary selector is only used for this, or only in this context. #<< is currently used for bit shifting, and would make sense as a general stream operation as well (given existing use elsewhere for that). A class object is a factory for its instances, so "streaming" in an array of its constituent elements to create an instance kind of works conceptually.
Having written all that, and thought about it, I'd rather see a compiled JSON solution. In the meantime I'd just write:
Dictionary withAll: {'class' -> 'button'. 'data-input' -> true}

This works already, although has a number of the qualities I don't like:

It builds a collection at runtime that is either entirely static, or has large amounts of static data, and could have been constructed at compile time. ##() will help for the entirely static case.
it fails if the elements of the collection argument are not associations (or at least things that understand key and value) .
it will work for LookupTable, but the associations are discarded creating needless overhead.

One could add the asDictionary method to do Dictionary withAll: self, but I think there is relatively little value.

@blairmcg thanks as usual for your detailed explanation. I agree with you about everything. I'm not too worried about introducing a bit of new syntax to Smalltalk while it doesn't introduces secondary effects. The json syntax seems a bit problematic with the colon and comma characters hanging around.
My use case is not always for static data, sometimes keys and values are built dynamically. And this remembers me that there is another new syntax I like to have in Smalltalk, template strings, {fooBar} reverse. For things like building html, specifying dynamic classes or ids, both json syntax and template strings are really handy. I recently discovered the new method #?? and love it because allows me to write more concise.
As for the alternatives you've mentioned I like #<<. In Ruby it's used to append to an Array. Another option would be to add a Json subclass to LookupTable and using it this way:
Json withAll: {'class' -> 'button'. 'data-input' -> true}
or
Json << {'class' -> 'button'. 'data-input' -> true}.
This has the convenience that we don't have to implement #<< in current dictionary like classes and in addition we could provide a valid json #displayString

@fxgallego, are template strings the same as interpolated strings in C#? It looks like they are. Yes, I've often thought those would be nice to have. Its one of those things pending replacement of the compiler - it could be done in the current compiler, but it is hard to extend. I really must get back to finishing off #119.
I was thinking that for JSON one would write it in the native form, but be able to refer to variables and such from Smalltalk, and that there would be a separate compiler for it that would emit a compiled method that would product the result much more efficiently than trying to interpret it at runtime. I like the idea of separate tagged methods that are built with a different compiler, but that might not be flexible enough. One might want to mix it up in order to be able to put in loops and such to generate collection content. It's difficult to know what would work in the abstract, however. With a compiler framework in Smallltalk it will be much easier to experiment.

Yes, the same as interpolated strings. But the JS syntax is lighter, it uses only backticks and thus more suitable for smalltalk, I think.


Event log incorrectly uses UTC date on frontend

Home Assistant release (hass --version): 0.18.2
Python release (python3 --version): Python 3.5.1
Component/platform: event log/Debian GNU/Linux 7.10 on raspberry pi
Description of problem: Event log shows incorrect day for certain times of the day.
Expected: Event log should show day for the current timezone instead to displays the current UTC day which if you live in a significant offset from UTC can end up being the wrong day.
Repro:

Set hass to Los Angeles timezone.
Also well as the web browser/OS the browser is running in.
Visit page late in the afternoon, notice it will be one day ahead and display nothing (since that day hasn't happened yet on the hass server).

You can see: https://github.com/home-assistant/home-assistant-js/blob/master/src/util/date-to-str.js for the source of the bug. This should not be a UTC date but probably whatever timezone hass is running in (or just the local timezone the browser is in).

Fixed in the frontend. Thanks.


Upgrade to Scrapy 1.1

This stack is available to provide compatibility with old projects created on Scrapy Cloud 1.0, but it will not receive updates for recent Scrapy versions. If you need a stack containing a recent Scrapy version, check this repository please: https://github.com/scrapinghub/scrapinghub-stack-scrapy
Note that there's a branch for each minor Scrapy version (1.2, 1.3, etc.) and there are also branches that provide Python 3 support too.
@vshlapakov I think this issue can be closed.

@jesuslosada Right, you described it well. Thanks, closing.


Regression in 1.0.40 handling of extern C++ return reference types

With 1.0.40, this code:
    unsafe extern "C++" {
        include!("rpmostree-clientlib.h");
        fn client_require_root() -> Result<()>;
        type ClientConnection;
        fn new_client_connection() -> Result<UniquePtr<ClientConnection>>;
        fn get_connection<'a>(self: Pin<&'a mut ClientConnection>) -> Pin<&'a mut GDBusConnection>;
        fn transaction_connect_progress_sync(&self, address: &str) -> Result<()>;
    }

no longer compiles:
error[E0308]: mismatched types
   --> rust/src/lib.rs:26:1
    |
26  | #[cxx::bridge(namespace = "rpmostreecxx")]
    | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ expected struct `Pin`, found `&mut _`
...
381 |         fn get_connection<'a>(self: Pin<&'a mut ClientConnection>) -> Pin<&'a mut GDBusConnection>;
    |                                                                       ---------------------------- expected `Pin<&'a mut cxxrsutil::FFIGDBusConnection>` because of return type
    |
    = note:         expected struct `Pin<&'a mut cxxrsutil::FFIGDBusConnection>`
            found mutable reference `&mut _`
    = note: this error originates in an attribute macro (in Nightly builds, run with -Z macro-backtrace for more info)

error: aborting due to previous error

(Associated C++ code is here: https://github.com/coreos/rpm-ostree/blob/bef1fc792655e6a22cc8676949a6a5792686ee2b/src/app/rpmostree-clientlib.h#L37-L56)
It's not clear from the error message what exactly is wrong. It said it found a &mut _ but the underlined type matches exactly the expected type (GDBusConnection is the same type as cxxrsutil::FFIGDBusConnection).

This was a poor interaction between #408 and #781. Sorry about the breakage! I've released a fix in 1.0.41.

Thanks for the quick fix!


Resolve RUSTSEC-2020-0071 by removing time 0.1 dependency from chrono

Also upgrades two dependencies

Thanks @kpcyrd! Side note, I just realized we don't have dependabot configured here.

Thanks for the fast merge! :)

hi! :D could you release a new version with this change (either 0.3.1 or 0.4.0)?
Thanks!

Hey @kpcyrd, I'll try and have 0.4.0 out soon. I just opened https://github.com/in-toto/in-toto-rs/pull/55. Is it okay to have that merged and dependencies bumped before the release?

Yes sure :)

Hey @kpcyrd: https://github.com/in-toto/in-toto-rs/releases/tag/v0.3.1, https://crates.io/crates/in-toto
v0.3.1 is live!

Thanks, running cargo update and pushing the new Cargo.lock has resolved my dependabot alert! :)


Updated README

Split up code sections
Added sections for recurring & cancelling jobs
Expanded on Listener events
Removed resque installation line

Thanks, @argvk !


API details

I was able to login, get my transactions.  I don't think the accounts list is complete.
You are using "accounts_response = pc.fetch('/newaccount/getAccounts')"
What is the significance of "newaccount"
Do you have information about an "API"
thanks
Rand

Inspect the network requests. Personal Capital doesn't document this so you have to figure it out. Out of scope for this library.


Changeset comment not included

I have created a couple of challenges, but they never include a predefined text as part of the commit, even if I have specified.
This is one of the challenges: https://maproulette.org/admin/project/46513
This is the changeset description: #ColombiaDivisionPolitica #maproulette
What should I do to include a template for the commit message?

Hi @angoca
This sounds more like a MapRoulette front-end issue than an API issue. Could you please resubmit here: https://github.com/osmlab/maproulette3/issues
Thanks!


The React dev tools Tabs Doesn't Show Up unless i checked all reasons in https://github.com/facebook/react/tree/main/packages/react-devtools#the-react-tab-doesnt-show-up #27564

I cant see react tabs in chrome dev panel.
I have checked this problem on https://reactjs.org/ and have got the same result
My OS:
Edition Windows 11 Pro
Version 22H2
Installed on 25.02.2023
OS build 22621.2428
Experience Windows Feature Experience Pack 1000.22674.1000.0
My browser version is Chrome Version is 118.0.5993.89 (Official Build) (64-bit)
My React Developer Tools extension version is 4.28.4 (10/3/2023)

Hey thanks for raising this issue. How ever the right place to raise this issue would be https://github.com/facebook/react/ repo as the devtools code resides there. This repo contains just the documentation site.


Update the release-next documentation

Merge-forward the release-next branch.
In particular, this will make the new cmctl upgrade migrate-api-version instructions available in the release-next docs.

/override dco

/lgtm
/approve


Expand / collapse inconsistency

To expand a cell, a button exists:

To collapse it, there is no button:

You can double click in the area to the left of the code to collapse it, but this is not obvious. The expand button should simply change to a collapse button and remain at the top left corner of the cell.

We don't show the collapse button anymore. When it's collapsed, we would show the run button when available.


Fix deploy target in Makefile

Remove the default generated Dockerfile that is not currently used to build the hive image
Fix the docker-build target in the Makefile to use the Dockerfile we use for CI
Fix the RBAC permissions for the controller (and generated sa role) so it can access jobs

/lgtm


Discrepancy Between Generated Python OpenAPI Client And Argo Server

Summary
I can make a request using the HTTP API that fails when using the Python client generated from the swagger.json.
Running Argo server locally, I navigate to this URL in my browser: http://localhost:2746/api/v1/workflows/argo?fields=items.metadata.name
I have a few example workflows, so the response looks like this:
{
  "items": [
    {
      "metadata": {
        "name": "example-mlcwl"
      }
    },
    {
      "metadata": {
        "name": "example-7slfm"
      }
    },
    {
      "metadata": {
        "name": "example-s86zz"
      }
    }
  ]
}

Generating the Python OpenAPI client:
docker run --rm \
      -v ${PWD}/argo-openapi-client:/local openapitools/openapi-generator-cli generate \
      -i /local/swagger.json \
      -g python \
      -o /local \
      --additional-properties=packageName=argo,projectName=argo

And using it to make the same request:
>>> import argo
>>> configuration = argo.Configuration(host='http://localhost:2746')
>>> with argo.ApiClient(configuration=configuration) as api_client:
...   api_instance = argo.WorkflowServiceApi(api_client)
...

>>> api_instance.list_workflows('argo', fields='items.metadata.name')
Traceback (most recent call last):
<snip>
ValueError: Invalid value for `spec`, must not be `None`

>>> api_instance.list_workflows('argo', fields='items.metadata.name,items.spec.affinity')
Traceback (most recent call last):
<snip>
ValueError: Invalid value for `metadata`, must not be `None`

>>> api_instance.list_workflows('argo', fields='items.metadata.name,items.spec.affinity,metadata')
{'api_version': None,
 'items': [{'api_version': None,
            'kind': None,
            'metadata': {'annotations': None,
...

Looking at the swagger.json, I can see that those fields are required, but I'm confused about how the server fulfills the same request in the browser if the request doesn't meet the requirements of the spec, and whether the generated client should match that behavior?
$ argo version
argo: latest+a00a8f1.dirty
  BuildDate: 2020-10-08T01:44:49Z
  GitCommit: a00a8f141c221f50e397aea8f86a54171441e395
  GitTreeState: dirty
  GitTag: v2.11.3
  GoVersion: go1.15.2
  Compiler: gc
  Platform: darwin/amd64

Message from the maintainers:
Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.

Can you ask in the #argo-sdks channel?

Can you ask in the #argo-sdks channel?

https://argoproj.slack.com/archives/CT5HFHDK8/p1605050218009600

While continuing to look at this, I tried the following:
config = argo.Configuration(host="http://localhost:2746")
config.client_side_validation = False
with argo.ApiClient(configuration=config) as api_client:
    api_instance = argo.WorkflowServiceApi(api_client)
    results = api_instance.list_workflows( "argo", fields="items.metadata.name")

I noticed that this client_side_validation variable exists on the Configuration object and is always set to True. That same variable is referenced in a few places in WorkflowServiceApi like this: self.api_client.client_side_validation. It also has a corresponding idea in the generated model files, e.g. io_argoproj_workflow_v1alpha1_workflow.py:
class IoArgoprojWorkflowV1alpha1Workflow(object):
    def __init__(self, api_version=None, kind=None, metadata=None, spec=None, status=None, local_vars_configuration=None):  # noqa: E501
        """IoArgoprojWorkflowV1alpha1Workflow - a model defined in OpenAPI"""  # noqa: E501
        if local_vars_configuration is None:
            local_vars_configuration = Configuration()
        self.local_vars_configuration = local_vars_configuration

    @spec.setter
    def spec(self, spec):
        """Sets the spec of this IoArgoprojWorkflowV1alpha1Workflow.
        :param spec: The spec of this IoArgoprojWorkflowV1alpha1Workflow.  # noqa: E501
        :type spec: IoArgoprojWorkflowV1alpha1WorkflowSpec
        """
        if self.local_vars_configuration.client_side_validation and spec is None:  # noqa: E501
            raise ValueError("Invalid value for `spec`, must not be `None`")  # noqa: E501

        self._spec = spec

If the client_side_validation setting in the WorkflowServiceApi could propagate to where the models are instantiated after the request returns, then this wouldn't be a problem. As it is, the model instantiation behavior is inconsistent with the Configuration used in the request and raises the ValueError no matter what because local_vars_configuration is never populated.

@kennytrytek-wf try out https://pypi.org/project/argo-workflows ?

OK this is the argo server api correct (not submitting the workflow), this is on my todo list for above project (add an issue there? maybe we can work together on this)

OK this is the argo server api correct

Yes, this is the argo server API as it compares to the generated OpenAPI client. I know the argo-client-python project exists, but I want to use the client as generated from the OpenAPI spec to ensure it is always compatible with the version of Argo server we have deployed.
My question is more fundamental about how the OpenAPI spec and Argo server relate to each other; should the server return responses that can't be defined by the spec (i.e. the example above where the response objects are missing required attributes), and should the OpenAPI client relax its definitions or add support for those queries?

@kennytrytek-wf @yxue-kabam I've come across this exact same issue trying to list cron workflows. I'm using the latest 5.0.0 version of the API (https://pypi.org/project/argo-workflows/)
Is there any work being done to fix this issue? Or at leat the ability to set client_side_validation to False as alluded to above?
Regards

I've added a new issue here: https://github.com/argoproj/argo-workflows/issues/6399

I am facing the same issue. does anyone have a solution?

Can you try the new SDK (note that it's still experimental and not published to PyPI yet): https://github.com/argoproj/argo-workflows/tree/master/sdks/python


update mime package to 2.2.x

Just a little update to use a newer version of the mime package. Thanks!

Coverage remained the same at 94.34% when pulling f0c2a9d05b6010856493e1644fadd97950c16fed on devoidfury:master into dab9c14ad5802d75bf9cb7d39934b2248af0df96 on emotionLoop:master.

Thanks @devoidfury !


Incomplete  comment

It seems we missed something here
https://github.com/docker/notary/blob/master/signer/api/api.go#L79
// CreateKey returns a handler that generates a new
func CreateKey(cryptoServices signer.CryptoServiceIndex) http.Handler {
...
}

Fixed here https://github.com/docker/notary/pull/343


Test highlight.js within Anki and Anki iOS

Using this and exported Marked App html: https://github.com/components/highlightjs

Default hljs.initHighlightingOnLoad(); doesn't work on iOS app. I have no idea why! Luckily, thanks to this thread, the below seems to work :+1:  ... for now at least.
hljs.initHighlighting.called = false;
hljs.initHighlighting();

When opening cards in the Mac app, it takes a while to close down. Hopefully this won't impact performance of the app.

Also refer to this issue

Minified script doesn't work

Correct code snippet below, requires hljs.
var blocks = document.querySelectorAll('pre code:not(.hljs)');
Array.prototype.forEach.call(blocks, hljs.highlightBlock);

Ok so progress so far using unminified highlights shim, with script and calling script on only the front card:
hljs.initHighlightingOnLoad();

Mac app: Works in browse -> cards ... on front and reverse.

Doesn't work on drill mode at all
Doesn't work in browse mode when Anki is relaunched.

iPhone app: Doesn't work

var blocks = document.querySelectorAll('pre code:not(.hljs)');
Array.prototype.forEach.call(blocks, hljs.highlightBlock);

hljs.initHighlighting.called = false;
hljs.initHighlighting();

Mac app: Works in browse -> cards ..., doesn't work on reverse (but works if you place either calling script on the reverse card too)

Doesn't work on drill mode at all
Doesn't work in browse mode when Anki is relaunched.

iPhone app: Both work fine :+1:

Mac app console log, not particularly helpful
Anki[9545]: JS error on line 7567: SyntaxError: Parse error
Anki[9545]: JS error on line 0: ReferenceError: Can't find variable: hljs
Anki[9545]: JS error on line 7567: SyntaxError: Parse error
Anki[9545]: JS error on line 0: ReferenceError: Can't find variable: hljs

Closing this as #21 is infinitely easier to mantain. Too many bugs.


Zero length path eagerload

Fixes #1666
This pull introduces/changes:

Allows zero length in eager load.

Codecov Report

Merging #1667 (efb44b0) into master (84f6193) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #1667   +/-   ##
=======================================
  Coverage   93.73%   93.74%
=======================================
  Files         126      127    +1
  Lines        5684     5708   +24
=======================================
+ Hits         5328     5351   +23
- Misses        356      357    +1

Impacted Files
Coverage Δ

lib/active_graph.rb
98.18% <100.00%> (+0.03%)
:arrow_up:

...tive_graph/node/query/query_proxy_eager_loading.rb
100.00% <100.00%> (ø)

...uery/query_proxy_eager_loading/association_tree.rb
100.00% <100.00%> (ø)

lib/active_graph/string_parsers/relation_parser.rb
100.00% <100.00%> (ø)

lib/active_graph/core/schema.rb
94.44% <0.00%> (-2.78%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 84f6193...efb44b0. Read the comment docs.

Closing in favour of activegraph-edge gem. This will moved to  activegraph-edge gem


Update to Arrow 35.0.0

Which issue does this PR close?

Closes #.
Rationale for this change

Update to Arrow 35.0.0
What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

Regression issue here preventing tests to run
select '2020-01-02 01:01:11.1234567890Z'::timestamp now fails
but select '2020-01-02 01:01:11.123456789Z'::timestamp is ok
@alamb @tustvold

Regression issue here preventing tests to run

Thanks @comphead -- I think that is apache/arrow-rs#3859
There were a few regressions, in arrow 35.0.0 so I think we should probably skip arrow 35.0.0 and wait for arrow 36.0.0: apache/arrow-rs#3889

Do we have a list of regression to be fixed in arrow 36.0.0?

Closing in favor of https://github.com/apache/arrow-datafusion/pull/5685


Feature/merge

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

Project coverage is 74.85%. Comparing base (4c0213e) to head (7510d8e).
Report is 3 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #126   +/-   ##
========================================
  Coverage    74.85%   74.85%
========================================
  Files           10       10
  Lines          835      835
========================================
  Hits           625      625
  Misses         210      210

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


Php-fpm failed to build laradock

Info:

Docker version: 18.03.0-ce-win59 (16762):
Laradock commit: 0c41fceae1b23089f5360dc4f8aeb85719c7bf5a:
System info (Mac, PC, Linux): Windows
System info disto/version: 10

Issue:
When I try to run the command docker-compose up -d nginx mysql, I get this error...
ERROR: Service 'php-fpm' failed to build: The command '/bin/sh -c if [ ${INSTALL_IMAGE_OPTIMIZERS} = true ]; then     apt-get install -y --for
ce-yes jpegoptim optipng pngquant gifsicle ;fi' returned a non-zero code: 100

Relevant Code:
My .env file,
`
Point to the path of your applications code on your host
APP_CODE_PATH_HOST=../public/
`

I have the same problem, but when I use --verbose I see this message in the screen
urllib3.connectionpool._make_request: http://localhost:None "GET /v1.25/images/laradock_php-fpm/json HTTP/1.1" 404 46

I have the same problem, but when I use --verbose I see this message in the screen:
compose.service.build: Building php-fpm
compose.cli.verbose_proxy.proxy_callable: docker build <- (path='/home/ernesto/code/myproject/laradock/php-fpm', tag='laradock_php-fpm', rm=True, forcerm=False, pull=False, nocache=False, dockerfile=None, cache_from=None, labels=None, buildargs={'PHP_VERSION': '7.0', 'INSTALL_XDEBUG': 'false', 'INSTALL_BLACKFIRE': 'false', 'INSTALL_SOAP': 'false', 'INSTALL_IMAP': 'false', 'INSTALL_MONGO': 'false', 'INSTALL_AMQP': 'false', 'INSTALL_MSSQL': 'false', 'INSTALL_ZIP_ARCHIVE': 'true', 'INSTALL_BCMATH': 'true', 'INSTALL_GMP': 'false', 'INSTALL_PHPREDIS': 'true', 'INSTALL_MEMCACHED': 'false', 'INSTALL_OPCACHE': 'true', 'INSTALL_EXIF': 'false', 'INSTALL_AEROSPIKE': 'false', 'INSTALL_MYSQLI': 'true', 'INSTALL_PGSQL': 'false', 'INSTALL_PG_CLIENT': 'false', 'INSTALL_TOKENIZER': 'true', 'INSTALL_INTL': 'true', 'INSTALL_GHOSTSCRIPT': 'false', 'INSTALL_LDAP': 'false', 'INSTALL_SWOOLE': 'false', 'INSTALL_IMAGE_OPTIMIZERS': 'true', 'INSTALL_IMAGEMAGICK': 'true'}, network_mode=None, target=None, shmsize=None, extra_hosts=None, container_limits={'memory': None}, gzip=False, isolation=None, platform=None)
docker.api.build._set_auth_headers: Looking for auth config
docker.api.build._set_auth_headers: No auth config in memory - loading from filesystem
docker.utils.config.find_config_file: Trying paths: ['/home/ernesto/.docker/config.json', '/home/ernesto/.dockercfg']
docker.utils.config.find_config_file: No config file found
docker.api.build._set_auth_headers: No auth config found
urllib3.connectionpool._make_request: http://localhost:None "POST /v1.25/build?t=laradock_php-fpm&q=False&nocache=False&rm=True&forcerm=False&pull=False&buildargs=%7B%22PHP_VERSION%22%3A+%227.0%22%2C+%22INSTALL_XDEBUG%22%3A+%22false%22%2C+%22INSTALL_BLACKFIRE%22%3A+%22false%22%2C+%22INSTALL_SOAP%22%3A+%22false%22%2C+%22INSTALL_IMAP%22%3A+%22false%22%2C+%22INSTALL_MONGO%22%3A+%22false%22%2C+%22INSTALL_AMQP%22%3A+%22false%22%2C+%22INSTALL_MSSQL%22%3A+%22false%22%2C+%22INSTALL_ZIP_ARCHIVE%22%3A+%22true%22%2C+%22INSTALL_BCMATH%22%3A+%22true%22%2C+%22INSTALL_GMP%22%3A+%22false%22%2C+%22INSTALL_PHPREDIS%22%3A+%22true%22%2C+%22INSTALL_MEMCACHED%22%3A+%22false%22%2C+%22INSTALL_OPCACHE%22%3A+%22true%22%2C+%22INSTALL_EXIF%22%3A+%22false%22%2C+%22INSTALL_AEROSPIKE%22%3A+%22false%22%2C+%22INSTALL_MYSQLI%22%3A+%22true%22%2C+%22INSTALL_PGSQL%22%3A+%22false%22%2C+%22INSTALL_PG_CLIENT%22%3A+%22false%22%2C+%22INSTALL_TOKENIZER%22%3A+%22true%22%2C+%22INSTALL_INTL%22%3A+%22true%22%2C+%22INSTALL_GHOSTSCRIPT%22%3A+%22false%22%2C+%22INSTALL_LDAP%22%3A+%22false%22%2C+%22INSTALL_SWOOLE%22%3A+%22false%22%2C+%22INSTALL_IMAGE_OPTIMIZERS%22%3A+%22true%22%2C+%22INSTALL_IMAGEMAGICK%22%3A+%22true%22%7D HTTP/1.1" 200 None
compose.cli.verbose_proxy.proxy_callable: docker build -> <generator object APIClient._stream_helper at 0x7f5d7d912bf8>
Step 1/68 : ARG PHP_VERSION=${PHP_VERSION}
ERROR: compose.cli.main.main: Service 'php-fpm' failed to build: Please provide a source image with `from` prior to commit

I'm using Ubuntu Linux

After more attempts to make it work I noticed this error message:
urllib3.connectionpool._make_request: http://localhost:None "GET /v1.25/images/laradock_php-fpm/json HTTP/1.1" 404 46
So I noticed that 404 response, now I see that error each time that I run the command.

Getting This Also

@David-Melo I give up with this and use cyberduck/php-fpm-laravel I hope this helps


Refactor params parser specs

Use named subjects for methods, avoiding calling for parser in each expectation.

Coverage remained the same at 99.213% when pulling 93ec3705c7f560d9db5416e4d484e5bc811c6cc1 on lidimayra:refactor-params-parser-specs into b9beec2c9f584e91320aeaf95822a1b8d77888c4 on ruby-grape:master.

Needs a rubocop -a ; rubocop --auto-gen-config and a rebase please.

Thanks for pointing this out.
Done! :smiley:

thank @lidimayra


Export LICENSE.txt in //third_party/nanopb

Same as https://github.com/nanopb/nanopb/commit/4710f9cef63e71f39100183f757d78add0ed7855. Needed to simplify building grpc in TensorFlow.
TensorFlow currently uses a different version of nanopb from gRPC just because of that missing commit. We need to include all licenses in our packaged library.

@ctiller mind taking a look?

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

jenkins: test this please

Friendly ping. This is blocking us from upgrading gRPC in TensorFlow. Thanks!

Flakes: #9082 #9727 #9399


Fix $ObjMap and $ObjMapi exactness

Does similar behaviour as $ReadOnly type, tho I'm not sure why flags.exact is different from ExactT.
Also added trust

Or maybe this is intended behaviour?

Hdhfookndujfo

Gebcjsnkxlmxbhdjc


Generating & Rendering bcad files

Hey Lyle,
First off - thanks for publishing your code! Anyways I was wondering how you rendered the bcad files? I've been experimenting with a few models and it'd be nice if I could visualize the generated bikes somehow. Currently I have to manually upload the files to bikecad.ca and wait until the bike loads, which is super slow.
Second question - in the genBCAD function instead of writing a bcad file from scratch you first open a template file and manipulate it, but why do you do it like that? The template file has over 6000 entries and you only edit those who are also in the generated dataframe, meaning there are ca. 5000 entries that are always the same - no? Also when I tried generating a bcad file with your method it wouldn't open on the website, not sure why. I had started writing my own method (before I realized you already implemented it), where I generated a new file from scratch and those files would open on the website, though there were some issues like for example boolean values were represented as 0 and 1. So yea my question is - why do you base your genBCAD method on a template file and why does the output not work (for me)?
Again thanks a lot for publishing your code!
-- Leonard

Hey Lyle,
thanks for the quick reply! I completely understand why you wouldn't want to publish your modded BikeCAD renderer, I guess I'll just have to use the website.
I'll append my notebook where I've been testing out the dataset. At the end you'll see two cells, in the first one I just copied your deOH and genBCAD functions and in the second cell I wrote my own genBCAD function that generates a new bcad file without a template. My problem is that neither cells produce bcad files that I can open using the web editor. I don't get an Error like 'corrupted file' or anything, it just never loads.
If you have some spare time I'd greatly appreciate if you could give me some pointers!
-- Leonard
VAE BIKED Test.zip

Hey Lyle,
Sure I'll attach some example files! Thanks for your help. In the attached archive you'll find 20 files, 10 generated using your genBCAD function (0.bcad to 9.bcad), the other 10 using my version (gen_sample_0.bcad to gen_sample_9.bcad).
-- Leonard
example_BCAD_files.zip

Hey Leonard,
Sorry for the slow response here... I haven't really been able to identify the issue. I can open both yours and my versions in bikecad, but they are kind of messed up... some screenshots:
My version:

Your version:

The files also seem laggy in the software for some reason. I suspect that there are some weird values in the files that are making some unresolvable geometry somewhere but I havent pinpointed what it is. In fact, most of the values that I checked seem quite reasonable. I'm assuming these are VAE-generated samples? Perhaps you can do a couple things to help us debug here. First: can you try taking a couple of the original bikes from the dataset and regenerating bikecad files from them? Then see if they open for you. Second: to help me understand what might be going wrong: Which parameters is your model generating? And which are you leaving at default values?
Also, all of the bikes seem to have this same structure, so it may be that your generative model is just stuck in some region of the design space where the geometry is invalid. It's not uncommon for generative models to collapse in some off-distribution area that may be geometrically invalid, but this would be more typical for a GAN variant, I'd imagine.
Anyways, sorry for a lack of a concrete solution, but hope this is a step in the right direction.
Lyle

Hey Lyle,
So I did some more testing yesterday and though the browser console I found out the reason that nothing happens when I try to open files on website is that eventually it will throw this error:

07:56:33.301 XHRGET
https://www.bikecad.ca/uBCADfiles/1.bcad
[HTTP/1.1 200 OK 0ms]
07:56:33.482 java.lang.IllegalArgumentException: setSelectedIndex: -16 out of bounds cheerpOS.js:1772:11
07:56:33.485 	at javax.swing.JComboBox.setSelectedIndex(Unknown Source) cheerpOS.js:1772:11
07:56:33.487 	at basic.DecalPanel.propRead(Unknown Source) cheerpOS.js:1772:11
07:56:33.490 	at basic.paintSchemes.propRead(Unknown Source) cheerpOS.js:1772:11
07:56:33.493 	at basic.bikeCADPro.ReadIn(Unknown Source) cheerpOS.js:1772:11
07:56:33.496 	at basic.bikeCADPro.PropOpen(Unknown Source) cheerpOS.js:1772:11
07:56:33.500 	at basic.bikeCADPro.PropOpenFirst(Unknown Source) cheerpOS.js:1772:11
07:56:33.505 	at basic.bikeCADPro.init(Unknown Source) cheerpOS.js:1772:11
07:56:33.507 	at sun.applet.AppletPanel.run(Unknown Source) cheerpOS.js:1772:11
07:56:33.509 	at java.lang.Thread.run(Unknown Source) cheerpOS.js:1772:11

Then I tried recreating a file from the dataset, without involving the VAE, just running it through the XML conversion, and that worked. So it seems the VAE was just returning invalid models, and not the genBCAD or to_xml functions producing invalid bcad files. I tried training a few different models and I've gotten some valid files that actually load, but none of the outputs visually resemble bikes, the geometry is still messed up.
Anyways thanks so much for your help & patience!
-- Leonard

Hi Leonard,
Great! Glad we were able to get to the bottom of this. Yeah, in my experience generative models really struggle with this dataset (it's part of a larger trend of generative models which excel in vision/NLP struggling to observe geometric/physical constraints in design problems). This is the subject of much of my thesis work & an upcoming paper looking at training generative models with both valid & invalid datapoints to help it avoid invalid regions of the space. In general, if you are struggling with precision, I'd try training a very small model, which will probably not capture all the variety in the dataset, but should hopefully get the `average' bike right.
Generally, if you constrain the model to only learn a subset of the parameters, it can give it fewer opportunities to "mess up," but then the representative power of the model is diminished. I've tried a few different models on the full ~2400 parameter dataset, including the CTGAN and TVAE from this paper. They were generally able to create valid bikes, but they still looked pretty weird. Eventually I kind of just decided to use various smaller subsets of the parameter space.
Anyways, hope this insight is valuable!
-Lyle

Going to close this issue, but feel free to keep commenting in the thread or get in touch if you want to chat about generative AI.


TextServer seems to merge Latin characters with the default project font

Godot version
4.0.dev.20211117
System information
Windows 10
Issue description
TextServer seems to consider some sequences of Latin characters as one grapheme, at least with the default font. It doesn't happen in the editor, but happens when you run a project with a TextEdit in the scene. Trying to select the text causes this behavior:
https://user-images.githubusercontent.com/11782833/142471901-4534a563-ab70-4536-9239-68cefda09838.mp4
I've run into this issue with fi and ff. It seems that enabling caret_mid_grapheme allows to select individual characters once again (but only with Shift + Arrows, mouse selection still handles it as one unit).
I noticed this behavior while working with TextServer in engine code, and I believe that the problem lies somewhere in shaped_text_get_glyphs and related methods and computations. @Paulb23 noted that this behavior may be desirable for some languages, but IMO it's a problem when this happens with English text and default project font, as that can trip users.
Steps to reproduce

Create a new project with a scene containing a TextEdit or LineEdit.
Either pre-fill or fill it at runtime with ff or fi.
Try selecting individual characters.

Minimal reproduction project
textserver-ff-grapheme.zip

/cc @bruvzg

ff, fl, fi, ffi and many others are standard ligatures, if a font is designed to use them, merging it to the one grapheme is a correct behavior (it's done by HarfBuzz, not TextServer code).
But, seems like I have missed mid-grapheme cursor handling in the mouse selection (shaped_text_hit_test_position).

are standard ligatures, if a font is designed to use them, merging it to the one grapheme is a correct behavior (it's done by HarfBuzz, not TextServer code).

Yes, that may be technically a correct behavior, but this isn't a great user experience. Should we change the default font then to the one that doesn't have ligatures?

Web browsers can handle selecting ffi mid-grapheme with both mouse and keyboard. We should try to do the same.

Should caret_mid_grapheme be on by default then?

Yes, that may be technically a correct behavior, but this isn't a great user experience.

Mouse selection is a bug, I'll fix it. Mid-grapheme caret probably should be enabled by default. So probably no reason to change the font.

Yes, I understand that it's a bug 🙃 Okay, setting it to on by default (and fixing the mouse bug) sounds good then!

Also, it's possible to disable this behavior, we'll need to add OpenType feature overrides to font and set it for built-in font by default. Currently, it can be done for specific control only.

Note: there's another bug in property loading, and it will be applied only after re-entering text.


VerifyClient.VerifyRequestAsync hangs when called in an asp.net 4.7 application

When calling var result =    await client.VerifyClient.VerifyRequestAsync(req, _credentials).ConfigureAwait(false);
The task simply never returns. It does successfully call the verify API   as I do get a text message.
After downloading the vonage-dotnet-sdk to troubleshoot it,  I found numerous awaits in VerifyClient.cs and ApiRequest did not have a .ConfigureAwait(false);  (which can in certain cases cause a task to wait indefinitely)  After fixing all those cases and VerifyRequestAsync, and VerifyCheckAsync both returned as expected.
I didn't test the other API methods, but I suspect many of them also have the same issue.

Yes, indeed - you're absolutely correct.
I investigated that topic earlier today and I learned that it's recommended to use ConfigureAwait when working on a library, given you don't know if the caller is going to need the synchronization context or not. So, it's better to leave that open, knowing there are rare cases where you can have a deadlock... For what I've seen, it's mostly with ASP.NET.
I'll tackle that problem. Hopefully, some automated refactoring can take care of that for me.

@tim-arheit The version 7.2.1 has been released and should solve this problem.
Can you give it a try and let me know if everything's okay on your side?

7.2.1 does fix the issue we were having.
Thanks!

Great to hear! :)


[fix][broker][monitoring] fix message ack rate

Motivation
fix PR.15674, there was a incorrect API usage in it, need to fix.
Modifications
fix issue
Documentation
Check the box below or label this PR directly.
Need to update docs?

[ ] doc-required
(Your PR needs to update docs and you will update later)

[x] doc-not-needed
(Please explain why)

[ ] doc
(Your PR contains doc changes)

[ ] doc-complete
(Docs have been already added)

@codelipenghui  PTAL

@codelipenghui  Fixed batch messages ack rate, PTAL


checkov: 3.2.235 -> 3.2.236

Diff: https://github.com/bridgecrewio/checkov/compare/refs/tags/3.2.235...3.2.236
Changelog: https://github.com/bridgecrewio/checkov/releases/tag/3.2.236
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 337678 run on x86_64-linux 1

2 packages built:

checkov
checkov.dist


Support for listing tags

Added support for getting tags out in an iterable format, just like branches.

@martwana Thanks for the contribution!


bug: Data in previous step lost

expect
When create task in dialog, the whole process is split into several steps. Data in last step should be kept.
got
After click next step button, then click previous button to go back into last step, the data just filled in is lost.

closed by #147


branch-3.0 Pick all BE execution log reduction

pick https://github.com/apache/doris/pull/42383
pick https://github.com/apache/doris/pull/42900

run buildall


typo: Double word "on"

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

biztalk/core/configure-the-time-zone-and-recurrence-scheduling-in-biztalk-server.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


I have question by #750, it's closed,but my problem is still....

I hava installed,browser extension,andI also open use the browser extension in Settings. but it's response network error too.

Already there's lot of duplicate issues, please track the issue on #750.
Please tell steps to reproduce issue on #750.


test: update snapshots

🧰 Changes
Looks like there was some upstream css changes, so updating the snapshots.
Also, a previous PR seems to have broken the uploading of snapshots during the workflow.
And, unfortunately, I'm currently getting different results running the snapshot tests locally vs CI. 😞
🧬 QA & Testing

Broken on production.
Working in this PR app.

This PR was released!
🚀 Changes included in v6.59.1


[BUG] uperf incorrectly reporting RR op/s

Describe the bug
uperf output has both "throughput in Mbps" and "Average ops". We are mistakenly reporting "throughput Mbps" as  "op/s". For example, in my run "267.433574 Mbps" is reported as "267.433574 (OP/s) "
DEBU[2024-06-05 14:15:28] Storing uperf sample throughput: 267.433574 Mbps, P99 Latency 33.969550, Average ops: 32645.700000
DEBU[2024-06-05 14:15:29] Rendering RR Transaction results
+---------------+---------+----------+-------------+--------------+---------+--------------+-----------+----------+---------+---------------------+--------------------------+
|  RESULT TYPE  | DRIVER  | SCENARIO | PARALLELISM | HOST NETWORK | SERVICE | MESSAGE SIZE | SAME NODE | DURATION | SAMPLES |      AVG VALUE      | 95% CONFIDENCE INTERVAL  |
+---------------+---------+----------+-------------+--------------+---------+--------------+-----------+----------+---------+---------------------+--------------------------+
| 📊 Rr Results | netperf | TCP_RR   | 1           | true         | false   | 1024         | false     | 10       | 1       | 25986.000000 (OP/s) | 0.000000-0.000000 (OP/s) |
| 📊 Rr Results | uperf   | TCP_RR   | 1           | true         | false   | 1024         | false     | 10       | 1       | 267.433574 (OP/s)   | 0.000000-0.000000 (OP/s) |

Yikes - are you putting together a patch for this @venkataanil ?


Limited Composite PK support

See:
https://groups.google.com/forum/m/#!topic/activejdbc-group/mSgxD1zwJIs

Please skip the first jdk6 commit. I got strange issues in my eclipse IDE with the default jdk5 setup.

@comtel2000 , I added and cleaned your code: https://github.com/javalite/activejdbc/issues/405.
Also, added docs here: http://javalite.io/surrogate_primary_keys#what-about-composite-pks
it is part of the framework now, thanks for your contribution!

@ipolevoy Did you plan to add relationship support in the near future? At least it would be great to have a working include method, so that entries can be cached.
Is there more work to do as changing the orderBy properties?

@boeserwolf91 , currently no plans to add relationships for composite keys. The ActiveRecord too does not support composite keys out of the box  (available as a separate gem outside Rails project). If we get enough requests for this, we might consider.


Now the liked map email is not sent if the owner like his own map

Closes #4886
// @Kartones please check this PR

Frontend tests were OK :+1: (details)

:+1:


Add a script to delete CF package

Create a script functions_delete_package.py which will delete all sequences and functions from package specified either in CLI or in config file. Delete also this package.

Merged to devel, closing.


Use configFile option, support an additional repo deploy key

Make build-in-devcontainer action use the new configFile option
Support an additional extra-repo-deploy-key in build-in-devcontainer action

Both of these changes are necessary to build all the RAPIDS projects in rapidsai/devcontainers CI jobs.

The devcontainers builds in https://github.com/rapidsai/cuml/pull/5760 passes, so I think this change is safe.

I've opened the following PRs to update the devcontainers and use this action:

https://github.com/rapidsai/cudf/pull/15099
https://github.com/rapidsai/cugraph/pull/4184
https://github.com/rapidsai/cugraph-ops/pull/600
https://github.com/rapidsai/cuml/pull/5778
https://github.com/rapidsai/cumlprims_mg/pull/173
https://github.com/rapidsai/cuspatial/pull/1344
https://github.com/rapidsai/kvikio/pull/344
https://github.com/rapidsai/raft/pull/2192
https://github.com/rapidsai/rmm/pull/1470

@bdice @vyasr All the PRs that depend on this branch have been merged, so I'm going to merge this into branch-24.04.

/merge


Update golang compiler for golang tip job

What type of PR is this?
/kind bug

What this PR does / why we need it:
After https://go.googlesource.com/go/+/82ee946d7acd93cad27f748efb9c3131ae69668e it is required to use at least 1.20 to build golang from source.
Which issue(s) this PR fixes:
https://testgrid.k8s.io/sig-scalability-golang#build-and-push-k8s-at-golang-tip
Special notes for your reviewer:

/assign @tosi3k

/lgtm
/approve


scenechange is a little slow

Hi friend, I want to use Pannellum for a virtual tour,when the scene change,I want it to seamless switch.
can you help me?

Images take time to load, so it's not going to be instant. The best you can do is use multiresolution images, as is done for the example on the pannellum.org homepage.

Year,last day I just try to use multiresolution images,it's faster,but how can I makes it more fast . what if I preload images?

You can try that, but it's unlikely to be faster than multires.

en，It's OK for multires,I found that is this code (var data = renderer.render(hotSpot.clickHandlerArgs.pitchCustom * Math.PI / 180, hotSpot.clickHandlerArgs.yawCustom * Math.PI / 180, (hotSpot.clickHandlerArgs.hfovCustom+20) * Math.PI / 180, {returnImage: true});)make it slow.
And,when I use phone's browser,this code,will not work,how can I get the picture for fade?

What's wrong with the standard sceneFadeDuration option?

yes，I know that set standard sceneFadeDuratio can cancel fade，but what if Iwant to use fade effect .and can I use some  function to get the  fade picture？

Your above code is the way to capture the WebGL canvas contents (although it doesn't always work in some browsers). If it's slow, it's because saving the canvas contents is slow on the browser / device you're using.


✨feat (Dashboards): update dashboard content

@xKeCo could you please check the issues Sonar has reported?


Firefox Sync Server

We should discuss the feasibility of providing a Firefox sync server package option to allow synchronizing bookmarks etc. to ones local server instead of the public ones. Most important is probably the question if we can set it up in a way that ensures security.

I like this idea! It would be a really nice feature.

We should put it behind a docker nginx, see here - related to #50
According to here, the server would be used via http, I guess that it should work equally over https
It is unfortunate that the Arch Wiki does not have a section on running an account server

This repository quite nicely shows how docker-compose can be used to setup an nginx reverse proxy using LetsEncrypt for https. docker-nginx-ssl-proxy and docker-letsencrypt-nginx-proxy-companion are two examples of containers that can be used to work with LetsEncrypt.


[FLINK-35936][cdc-connector][paimon] Fix paimon cdc schema evolution failure when restart job

Thanks @MOBIN-F for the contribution, could you add description for the PR ?


[BUG] api-version for NetworkClient.ConnectionMonitors is invalid

The api-version '2020-04-01' is invalid. The supported versions are '2020-07-01,2020-06-01,2020-05-01,2020-01-01,2019-11-01,2019-10-01,2019-09-01,2019-08-01,2019-07-01,2019-06-01,2019-05-10,2019-05-01,2019-03-01,2018-11-01,2018-09-01,2018-08-01,2018-07-01,2018-06-01,2018-05-01,2018-02-01,2018-01-01,2017-12-01,2017-08-01,2017-06-01,2017-05-10,2017-05-01,2017-03-01,2016-09-01,2016-07-01,2016-06-01,2016-02-01,2015-11-01,2015-01-01,2014-04-01-preview,2014-04-01,2014-01-01,2013-03-01,2014-02-26,2014-04'.

https://github.com/Azure/azure-sdk-for-net/blob/d9e73dac2cd7491938e89ce2f78af523b3865dd8/sdk/network/Azure.ResourceManager.Network/src/Generated/ConnectionMonitorsRestOperations.cs#L61
            var monitor = await network
                .ConnectionMonitors
                .StartCreateOrUpdateAsync("", "", "asdf", new ConnectionMonitor
                {
                    AutoStart = true,
                    Destination = new ConnectionMonitorDestination
                    {
                        Address = "google.nl",
                        Port = 80
                    },
                    Location = "westeurope",
                    MonitoringIntervalInSeconds = 30,
                    Source = new ConnectionMonitorSource("/subscriptions/###/resourceGroups/asdf/providers/Microsoft.Compute/virtualMachines/###"),

                });

            await monitor.WaitForCompletionAsync();

<PackageReference Include="Azure.ResourceManager.Network" Version="1.0.0-preview.2" />

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

@PhoenixHe-msft, you created PR for April release earlier. .
Did they retired the 2020-04-01? It is still clearly in the swagger repo.

I got one step further by using NetworkWatcherRG and NetworkWatcher_westeurope for the resource group and network watcher name respectively. But then I got the error:
Content:
{
  "error": {
    "code": "IncorrectAPIVersionToCreateSingleSourceDestinationConnectionMonitor",
    "message": "Cannot create single-source-destination connection monitor /subscriptions//resourceGroups//providers/Microsoft.Network/networkWatchers//connectionMonitors/ using this api version 2020-04-01. Supported version is 2019-06-01 or lower.",
    "details": []
  }
}


[Feature] Menu Type field

Hello, guys a made small useful field it's called menu_type and it's used in MenuItem you can choose whatever you want for example tag,article,hotel,destination,category etc... whatever you want. Here is the syntax
 $this->crud->addField([
            'name' => 'type',
            'label' => 'Type',
            'choose_label' => 'Choose Option',
            'type' => 'menu_type',
            'options' => [
                'page' => 'Page',
                'tag'  => 'Tag',
                'hotel' => 'Hotel',
                'destination' => 'Destination',
                'category' => 'Category'
            ],

            'types' => [

                'page'   => [
                    'model' => 'App\Models\Page',
                    'name' => 'page_id',
                    'attribute' => 'name'
              ],
                'tag'  => [
                    'model'=> 'App\Models\Tag',
                    'name' => 'tag_id',
                    'attribute' => 'name'
                ],
                'hotel' => [
                    'model' => 'App\Models\Hotel',
                    'name'   => 'hotel_id',
                    'attribute' => 'name'
                ],
                'destination' => [
                    'model' => 'App\Models\Destination',
                    'name' => 'destination_id',
                    'attribute' => 'name',
                ],
                'category' => [
                    'model' => 'App\Models\Category',
                    'name' => 'category_id',
                    'attribute' => 'name'
                ]
            ],
        ]);

Hi @petyots ,
Sounds great! How do you store the information in the DB, to know which "entity" the ID corresponds to (whether it's a Tag, Hotel, Destination, Category, etc)?
Cheers!

Seems similar to what I had suggested just recently? May be mistaken. https://github.com/Laravel-Backpack/MenuCRUD/pull/23

@AbbyJanke - I noticed you closed your PR?
@tabacitu - is this still useful/relevant?
(I'm on a close issues rampage atm).

@petyots , @AbbyJanke - any of you guys want to help the community by writing an article on how to do this? It can be wherever you want (medium, your blog, any other blog) and I'd love to link to it from the backpack blog too. I think this is a great way to keep these snippets close right now, so that we can reuse them in our projects, even if we don't include them in the core project.
Cheers!
PS. @lloy0076 I say let's close it, we need a little more room to breathe in Issues, so let's only keep issues in Github Issues :-) We do need to provide a helpful solution for everything before we close, though. That's the kind thing to do and I hope it will foster the same kind of attitude from the community. We all choose to work with Backpack, and communicate with each other, so I truly believe each message should be kind :-)


I/flutter (30089): IsarError: Operation not supported. You might need to use a different flavor of Isar.

Steps to Reproduce
I haven't yet started a project completely fresh but I ran into this while implementing v4 into my code, I got a few classes working already but now any class i try adding can't be added to isar. A couple of things are to note:

I can still write the older models, even when regenerating their part files.
It does not seem to have anything to do with the fields in the models that are failing
any new model that I try gives errorL  I/flutter (30089): IsarError: Operation not supported. You might need to use a different flavor of Isar.
and any new model does not show an amount when looking at the inspector but instead says loading and 0B

Code sample
I tried adding this model so I could know if the problem was related to the class that was failing
import 'package:isar/isar.dart';

part 'Person.g.dart';

@collection
class Personel {
  @Id()
  String id;
  String? email;

  Personel({
    required this.id,
    this.email = '',
  });
}

this is how i tried writing to the db:
isarManager.isar.write((isar) {
        isar.homeConversationModels
            .put(HomeConversationModel(id: 'test'));
        isar.personels.put(Personel(id: 'hello!', email: 'test@example.com'));
      });

where the first one works and the second one fails.
This is how i initialize isar:
final dir = await getApplicationDocumentsDirectory();
    isar = Isar.open(
      schemas: [
        HomeConversationModelSchema,
        ConversationModelSchema,
        PersonelSchema,
      ],
      directory: dir.path,
    );

meanwhile my more complex classes with lists embedded classes etc. are still working
Details

Platform:  pixel 3 x86_64 Android Emulator on Windows
Flutter version: [3.13.3]
dart version [3.1.1]
Isar version: [e.g. 4.0.0-dev.14]

[x] I searched for similar issues already
[x] I filled the details section with the exact device model and version
[?] I am able to provide a reproducible example

Same here

same here

exact same issue here. Anyone have success?

I had the same problem, but for me only closing the app and reloading it was enough to "fix" it.

For me, this occurred when adding schemas into the Isar.open call which were annotated as @embedded.
When I remove these and only included those annotated as @collections, it worked fine!
Hope that helps for you!

For me, I found that if I sort the entire @collections on the top of schemas array, the problem is solved


(Fix) Decimals validation for reserved tokens

This PR closes #1158 & #1159.
Both were related as decimals value wasn't updated properly across the child components.

Black Duck Security Report
Merging #1161 into integration/#1082-new-design will not change security risk.
Click here to see full report


[firebase-database] Calls to setPersistenceEnabled() must be made before any other usage of FirebaseDatabase instance

Hi,
If you launch your app on debug, work on it, change something and save, the app gets reloaded, and then you get this:

Error: Uncaught (in promise): Error: com.google.firebase.database.DatabaseException: Calls to setPersistenceEnabled() must be made before any other usage of FirebaseDatabase instance.
JS: set persistenceEnabled(file: node_modules@nativescript\firebase-database\index.android.js:621:0)
JS:     at (file: src\app\app.component.ts:36:20)
JS:     at invoke(file: node_modules\zone.js\fesm2015\zone.js:372:0)
JS:     at onInvoke(file: node_modules@angular\core\fesm2015\core.mjs:25548:0)
JS:     at invoke(file: node_modules\zone.js\fesm2015\zone.js:371:0)
JS:     at run(file: node_modules\zone.js\fesm2015\zone.js:134:0)
JS:     at (file: node_modules\zone.js\fesm2015\zone.js:1275:0)
JS:     at invokeTask(file: node_modules\zone.js\fesm2015\zone.js:406:0)
JS:     at onInvokeTask(file: node_modules@angular\core\fesm2015\core.mjs:25535:0)
JS:     at invokeTask(file:///data/data/org.n...

I think Eddy got this once.
app.component:
    firebase().initializeApp().then(
      () => {
        firebase().database().persistenceEnabled = true;
        firebase().crashlytics().setCrashlyticsCollectionEnabled(true);
        console.log("firebase.init done");
      },
      error => {
        console.log(`firebase.init error: ${error}`);
      }
    );

package.json
{
  "description": "NativeScript Application",
  "license": "SEE LICENSE IN <your-license-filename>",
  "repository": "<fill-your-repository-here>",
  "dependencies": {
    "@angular/animations": "~13.2.0",
    "@angular/common": "~13.2.0",
    "@angular/compiler": "~13.2.0",
    "@angular/core": "~13.2.0",
    "@angular/forms": "~13.2.0",
    "@angular/platform-browser": "~13.2.0",
    "@angular/platform-browser-dynamic": "~13.2.0",
    "@angular/router": "~13.2.0",
    "@nativescript-community/ui-pulltorefresh": "^2.5.1",
    "@nativescript/angular": "^13.0.0",
    "@nativescript/background-http": "~5.0.0",
    "@nativescript/camera": "^5.0.12",
    "@nativescript/core": "^8.2.3",
    "@nativescript/datetimepicker": "2.1.9",
    "@nativescript/firebase-core": "^2.0.2",
    "@nativescript/firebase-crashlytics": "^2.0.2",
    "@nativescript/firebase-database": "^2.0.2",
    "@nativescript/firebase-storage": "^2.0.2",
    "@nativescript/geolocation": "~8.0.0",
    "@nativescript/imagepicker": "~1.0.0",
    "@nativescript/mlkit-barcode-scanning": "^1.0.5",
    "@nativescript/mlkit-core": "^1.0.5",
    "@nativescript/mlkit-face-detection": "^1.0.5",
    "@nativescript/theme": "^3.0.1",
    "@ngx-translate/core": "^11.0.1",
    "@ngx-translate/http-loader": "^4.0.0",
    "@nstudio/nativescript-camera-plus": "^4.1.0",
    "@nstudio/nativescript-loading-indicator": "~4.0.0",
    "@nstudio/nativescript-snackbar": "~2.0.0",
    "crypto-browserify": "^3.12.0",
    "latest": "^0.2.0",
    "nativescript-imagecropper": "~4.0.0",
    "nativescript-nfc": "~5.0.0",
    "nativescript-permissions": "^1.3.8",
    "nativescript-sound-kak": "^1.2.0",
    "nativescript-theme-core": "~1.0.4",
    "nativescript-ui-chart": "~9.0.0",
    "nativescript-ui-sidedrawer": "~10.0.0",
    "nativescript-webview-interface": "^1.4.2",
    "rxjs": "^7.4.0",
    "stream-browserify": "^3.0.0",
    "tns-platform-declarations": "^6.5.15",
    "zone.js": "~0.11.4"
  },
  "devDependencies": {
    "@angular-devkit/build-angular": "^13.3.6",
    "@angular-eslint/builder": "13.0.1",
    "@angular-eslint/eslint-plugin": "13.0.1",
    "@angular-eslint/eslint-plugin-template": "13.0.1",
    "@angular-eslint/schematics": "13.0.1",
    "@angular-eslint/template-parser": "13.0.1",
    "@angular/compiler-cli": "~13.2.0",
    "@nativescript/android": "8.2.3",
    "@nativescript/ios": "8.2.3",
    "@nativescript/schematics": "^0.5.2",
    "@nativescript/types": "~8.2.0",
    "@nativescript/webpack": "~5.0.0",
    "@ngtools/webpack": "~13.2.0",
    "@types/jasmine": "^3.5.11",
    "@typescript-eslint/eslint-plugin": "5.3.0",
    "@typescript-eslint/parser": "5.3.0",
    "eslint": "^8.2.0",
    "typescript": "~4.5.5"
  },
  "readme": "NativeScript Application",
  "main": "./src/main.ts",
  "scripts": {
    "lint": "ng lint"
  }
}

Probably because the app.component is call again. It can probably occur also if you put the app on background and you come back. If you can help please.
Thanks for your help

What you can do is add a guard around firebase().initializeApp() in this instance it should only be called once. Also the initialize call should be in your main.ts

@triniwiz by a guard you mean a try and catch ? Because i tried and i still have the same. I also moved it to main.ts and same thing

A property you can set/get to ensure it’s only set once per app launch

You mean something like:
try {
  let initialized = false;
  if(initialized === false) {
    firebase().initializeApp().then(
    () => {
      firebase().database().persistenceEnabled = true;
      firebase().crashlytics().setCrashlyticsCollectionEnabled(true);
      initialized = true;
      console.log("firebase.init done");
    },
    error => {
      console.log(`firebase.init error: ${error}`);
    }
  );
  }
}
catch (err) {
  console.log("err in try and catch " + err);
}

because if yes it doesnt work

I also have the same issue. Any solution?

@triniwiz some news about this ?

Will try fixing soon


column optimization is run once per output? (shouldn't it be only once!?)

From the script here:
import dask
import uproot
from dask.diagnostics import ProgressBar

tree = uproot.open({"https://github.com/CoffeaTeam/coffea/raw/master/tests/samples/nano_dy.root": "Events"})
dtree = uproot.dask(tree)

out = {"a": dtree.event, "b": dtree.Muon_pt, "c": dtree.Jet_pt}

with ProgressBar() as _:
    print(dask.compute(out))

we get the following output:
(coffea-dev) lgray@Lindseys-MBP coffea % python col_opt_repro.py
[########################################] | 100% Completed | 105.21 ms
[########################################] | 100% Completed | 105.33 ms
[########################################] | 100% Completed | 101.16 ms
[########################################] | 100% Completed | 524.83 ms
({'a': <Array [3749778, 3749762, 3749777, ..., 3749882, 3749892] type='40 * uint64'>, 'b': <Array [[], [], [], [], [...], ..., [], [], [], []] type='40 * var * float32'>, 'c': <Array [[80.8, 45.6, 29.6, 17.6, 15.2], ..., [...]] type='40 * var * float32'>},)

Where we can see there are three dry runs over empty data.
If we only compute one of the elements of out we get:
import dask
import uproot
from dask.diagnostics import ProgressBar

tree = uproot.open({"https://github.com/CoffeaTeam/coffea/raw/master/tests/samples/nano_dy.root": "Events"})
dtree = uproot.dask(tree)

out = {"a": dtree.event, "b": dtree.Muon_pt, "c": dtree.Jet_pt}

with ProgressBar() as _:
    print(dask.compute(out["b"]))

(coffea-dev) lgray@Lindseys-MBP coffea % python col_opt_repro.py
[########################################] | 100% Completed | 104.73 ms
[########################################] | 100% Completed | 309.18 ms
(<Array [[], [], [], [], [...], ..., [], [], [], []] type='40 * var * float32'>,)

So it's clear the column optimization is being run once per output.
I think this should only be done once on the entire fused output, no?

Ideally yes, I'll dig into the upstream dask code again to see what it does with a dict vs a tuple (what we solved before!)


Add the Subscription setting

Have the list of settings match what the user is prompted for in the UI

@andresnava,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@ggailey777 - Please review the proposed changes.  Thanks

@andresnava thanks for catching this UI change. #sign-off


Update Manager: Branches: no branch is listed

Enhancement request.
The GUI should list at least one branch.

Done!
Now that "branches" tab will always be disabled until an actual branch is available.


SlowMotion Playback

Describe the bug
Can't play mixamo animation at realtime speed
To Reproduce
Steps to reproduce the behavior:

Open the file using f3d --any mixamo animation
Press spacebar for play
Slow Motion playback issue

Expected behavior
Can't play mixamo animation at realtime speed
System Information:

OS: Mac
GPU and GPU driver: Apple M1 Pro 32 GB Ventura 13.0.1

F3D Information
Paste the content of f3d --version: Latest
Additional context
I can't play any Mixamo animation at realtime speed. All them play slow motion
Here settings which i download for Unity

https://user-images.githubusercontent.com/104074156/201876450-8fbed94e-a36f-4db8-bd8e-332dfd5bcc52.mp4

Paste the content of f3d --version: Latest

Please be more specific. In your case, the FBX animations highly depend on the Assimp version used internally by F3D.
Also, please share the FBX file.

No answer from OP, closing. @alianilir, feel free to reopen.

Reproduced on windows with any animated fbx

@Meakk @alianilir : So I checked different versions and I only see this in our 1.3.1 release using Assimp 5.1.0.
This is fixed in F3D master using Assimp 5.2.4, you can download our nightly to try it:
https://github.com/f3d-app/f3d/releases/tag/nightly


No simple way to use preferred peers only

trinity Version: trinity-v0.1.0-alpha.20-840-g3b52bdee (current master at commit 3b52bdee).
OS: Arch Linux
Python Version (python --version): 3.7.3
Environment (output of pip freeze): in gist

What is wrong?
I'd like to fast-sync from another node also under my control. Naively:
trinity --ropsten --port=$PORT_TRINITY --log-level=debug \
    --disable-discovery --max-peers=1 --preferred-node $ENODE_GETH

This doesn't work:

Entries from previous runs are still in the peer DB, at ~/.local/share/trinity/ropsten/nodedb-eth1.
Bootnodes are still in the viable peer list, via p2p/constants.py.
Inbound connections are still handled.

To work around, one can:

Remove the peer DB.
Comment out the bootnodes.
Close the port via firewall to all IPs but the preferred peer's.

... but this is a lot of busywork.

Another issue is the fact that if no preferred peers are specified, then there's a "default preferred peers list" (in trinity/constants.py).
I'd argue this violates the principle of least surprise to node operators: "no preferred peers" should mean just that: there is no peer preference.
Also, this puts a strain on those "preferred by default" nodes, some of which are also in the bootnodes list. This is not what bootnodes are for!
How can it be fixed
Not sure. Introduce --preferred-only?..
But take also into consideration:

#440 - Decouple "preferred" from "discovery".
#24 - Maintain updated pool of recommended peers.

I think I'd split responsibilities up as follows.

New flag: --peer-backend.

Can be provided multiple times or maybe with a comma separated list of values
--peer-backend discovery-v4: enables the v4 discovery protocol
--peer-backend discovery-v5: enables the v5 discovery protocol
--peer-backend static: enables the static peers API (previously known as preferred peers)

--disable-discovery: becomes an alias for: --peer-backend discovery-v4 (or maybe v5)

This isolates the concept of "how do I find out about peers" and fits in well with this: https://github.com/ethereum/trinity/pull/496
Next, we isolate the idea of "bad peer tracking".  Maybe to start we do something like:

--peer-tracking

disable disables this mechanism, swapping in the NoopPeerInfo backend
memory could use a new in memory only database that only persists for the lifetime of the node.
sqlite3 (default) would use the SQLite3 database of peer history.

Next we isolate the issue of incoming connections.

--allow-inbound-connections or --forbid-inbound-connections as mutually exclusive options which determine whether we open the listening service.

--disable-discovery: becomes an alias for: --peer-backend discovery-v4 (or maybe v5)

I didn't follow this. I think --disable-discovery would be --peer-backend static in this model, no?
I assume that the previously connected peers would only be tried again on restart if discovery-v* is enabled.