Parser doesn't support ListSeparator after typedef

According to the definition at https://thrift.apache.org/docs/idl#typedef, it isn't supposed to but some of the tests have them.  Not sure how to resolve this (by spec or by tests).

It looks like the "Field" definition includes an optional ListSeparator at the end: https://thrift.apache.org/docs/idl#field

The IDL docs declare typedef as:

Typedef ::= 'typedef' DefinitionType Identifier

But the thrift compiler support ListSeparators (even in strict mode) so I think this should be supported.


Dashing start command fails in Tmux session

When I run the dashing start command in a tmux session, it errors out. But the same command after exiting Tmux, works perfectly. This is within a vagrant session running Oracle Linux 6.5. I am also using rvm. From the logs, it seems that is relevant as well.
To reproduce, open terminal, go to a dashing app, run the following -
➜ dashing start
LOGGING_LEVEL = 1
Thin web server (v1.6.4 codename Gob Bluth)
Maximum connections set to 1024
Listening on <IP_ADDRESS>:3030, CTRL+C to stop

➜ tmux new
➜ dashing start
ERRORS start here (about 300 lines, thus not attaching log for now).

you look to be trying to run to instances of dashing on the same port (3030)
try this for the 2nd command.
dashing start -p 3031

My bad, This is after I stopped the first instance and started tmux. I'll update the description


Firefox support

Hi,
I found your add on very useful. Do you have plan to port it to firefox?

Yes, I already added Firefox support for the next version. It is not released yet as I am still working on some new features.

The latest version is compatible with Firefox and published to https://addons.mozilla.org/, waiting for the review now

It is available now at https://addons.mozilla.org/en-US/firefox/addon/load-timer/


Created a client's presence detector json sample, based on WiFi connection.

I've created a simple presence detector sample, based on the idletime property of a client device.

Hi @Supergiovane
Is this still current?
Naturally this isn't a code check, as its just  a flow, but does it still hold up?
if so I'll merge with our test branch

Yes. It’s still current


Resize Processes tab feature

I'd like to request a resizing of the Processes tab. Sometimes it takes much more of my terminal size than I'd like to.
Not proficient in Rust, but if you give me a guideline I could try to do that too.

It's very easy to add. I'll add in in the next version. In fact, I already wanted to include it in the current version, but didn't have time.

Now the process list width can be set in mprocs.yaml:
procs_list_width: 20

Thank you very much!! I'm really happy using mprocs on my daily basis at work and home.


[HOLD] Reapply RoomHeaderAvatars Logic to fix regression correctly this time

CC: @Julesssss
Details
Fixes regression found here , but this time
HOLD on https://github.com/Expensify/App/pull/7852 being on staging.
context is here: https://expensify.slack.com/archives/C03TQ48KC/p1646832274218809
but this was deployed to staging before the original PR was deployed to staging so it broke the ReportDetails page. This will reintroduce the correct changes but requires that first PR to be deployed to staging first. Then this can be cherry picked.
Fixed Issues
related to https://github.com/Expensify/App/issues/8040
Tests
N/A same as Web QA.

[x] Verify that no errors appear in the JS console

PR Review Checklist

Contributor (PR Author) Checklist

[x] I verified the PR has a small number of commits behind main
[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I clearly indicated the environment tests should be run in (Staging vs Production)

[x] I wrote testing steps that cover success & fail scenarios (if applicable)
[ ] I included screenshots or videos for tests on all platforms
[ ] I ran the tests & veryfy they passed on:

[ ] iOS / native
[ ] Android / native
[ ] iOS / Safari
[ ] Android / Chrome
[ ] MacOS / Chrome
[ ] MacOS / Desktop

[x] I verified there are no console errors related to changes in this PR
[x] I followed proper code patterns (see Reviewing the code)

[ ] I added comments when the code was not self explanatory
[ ] I put all copy / text shown in the product in all src/languages/* files (if applicable)
[ ] I followed proper naming convention for platform-specific files (if applicable)
[ ] I followed style guidelines (in Styling.md) for all style edits I made
[ ] I followed the JSDocs style guidelines (in STYLE.md)

[ ] I followed the guidelines as stated in the Review Guidelines
[ ] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[ ] I corroborated the UI performance was not affected (the performance is the same than main branch)
[ ] If I created a new component I verified that a similar component doesn't exist in the codebase

PR Reviewer Checklist

[ ] I verified the PR has a small number of commits behind main
[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the testing environment is mentioned in the test steps

[ ] I verified testing steps cover success & fail scenarios (if applicable)
[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

[ ] iOS / native
[ ] Android / native
[ ] iOS / Safari
[ ] Android / Chrome
[ ] MacOS / Chrome
[ ] MacOS / Desktop

[ ] I verified there are no console errors related to changes in this PR
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] I verified comments were added when the code was not self explanatory
[ ] I verified any copy / text shown in the product was added in all src/languages/* files (if applicable)
[ ] I verified proper naming convention for platform-specific files was followed (if applicable)
[ ] I verified style guidelines were followed
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified other components are not impacted by changes in this PR (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[ ] I verified the UI performance was not affected (the performance is the same than main branch)
[ ] If a new component is created I verified that a similar component doesn't exist in the codebase

QA Steps
This will be done on staging.
Find a defaultRoom, policyRoom, or workspaceChat and verify that when you click on the header and see the report details, it looks like this:

and not this:

[ ] Verify that no errors appear in the JS console

Screenshots
N/A

@TomatoToaster is there a reason this is still on hold?

Nope! We can merge this now, I realized the other one just went to staging last night, but also we could've just merged this without the CP-staging label and it would fix it 😅

🚀 Cherry-picked to staging by @TomatoToaster in version: 1.1.42-4 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
failure ❌

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by @chiragsalian in version: 1.1.42-6 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Fixed problems with image layouts and layout transitions.

It's a replacement PR for #238.

Thx for all the effort you put into this! Much appreciated 👍


Clear web gl

if you try to remove all nodes and links:
grapher.data({nodes: [], links: []}).render();

the page doesn't refresh because we're not clearing the webGL context.
this fix addresses that and also seems to prevent the strange flicker that sometimes happens when grapher is rerendering during zoom, pan, and node selection.
#8

oops. thanks

Looks good! For v1.1,0 we might consider adding a clear function from Grapher to both renderers. But this should be a good fix for 1.0.


Fix Playground & resources namespacing for refactor

Signed-off-by: Andrew Welch<EMAIL_ADDRESS>

Merci!


Maximum call stack size exceeded when trying to convert pixi.js v8.

Hello, I'm trying to convert PIXI.js v8 with the following configuration :
{
	"input": "node_modules/pixi.js/lib/**/*.d.ts",
	"output": "dist",
	"libraryName": "pixi.js"
}

But I'm getting the following error :
file:///C:/Users/MyUser/IdeaProjects/random/karakum-test/node_modules/karakum/build/utils/fileName.js:17
    return commonPrefix(common, ...rest);
           ^

RangeError: Maximum call stack size exceeded
    at commonPrefix (file:///C:/Users/MyUser/IdeaProjects/random/karakum-test/node_modules/karakum/build/utils/fileName.js:17:12)
    at commonPrefix (file:///C:/Users/MyUser/IdeaProjects/random/karakum-test/node_modules/karakum/build/utils/fileName.js:17:12)
    at commonPrefix (file:///C:/Users/MyUser/IdeaProjects/random/karakum-test/node_modules/karakum/build/utils/fileName.js:17:12)
    at commonPrefix (file:///C:/Users/MyUser/IdeaProjects/random/karakum-test/node_modules/karakum/build/utils/fileName.js:17:12)
    at commonPrefix (file:///C:/Users/MyUser/IdeaProjects/random/karakum-test/node_modules/karakum/build/utils/fileName.js:17:12)
    at commonPrefix (file:///C:/Users/MyUser/IdeaProjects/random/karakum-test/node_modules/karakum/build/utils/fileName.js:17:12)
    at commonPrefix (file:///C:/Users/MyUser/IdeaProjects/random/karakum-test/node_modules/karakum/build/utils/fileName.js:17:12)

Here is my package.json :
{
	"name": "karakum-test",
	"version": "1.0.0",
	"scripts": {
		"start": "karakum --config karakum.config.json"
	},
	"dependencies": {
		"pixi.js": "^8.0.0-beta.6"
	},
	"devDependencies": {
		"karakum": "^1.0.0-alpha.25",
		"typescript": "^5.2.2"
	}
}

Thank you for the report. I will check what is wrong with this commonPrefix helper.

@Ayfri I removed recursion in the commonPrefix helper. It is used only for detection of common file root that can be set up manually using inputRoots config option, so also now I compute common root lazily only if inputRoots is not specified. All changes are published, so you can check them.
But to be honest, I didn't reproduce your problem. After running of Karakum for your setup, I received only two files
/dist/filters.kt
/dist/index.kt

I noticed that index.kt re-exports a lot of stuff, so maybe the number of reexported files causes the issue with recursion?
It would be great if you provide more details about your conversion. Maybe I will even add you configs as an example in Karakum repository.

Sorry, I used v7 of PIXI.js, that is why the number of files was so small. I tested it with  v8 and now I see, the number of files is really big. I still didn't reproduce it on Linux (I don't know why), but I will try to do it on Windows later.

@sgrishchenko Have you had time to test ?

No, I didn't yet, but I already published a fix, that should solve this issue. You can try a new Karakum version.


Got different accuracy between history and evaluate

I fit a model as follow: history = model.fit(x_train, y_train,                     epochs=50,                     verbose=1,                     validation_data=(x_val,y_val))
Got the answer :
Epoch 48/50
49/49 [==============================] - 0s 3ms/step - loss: 0.0228 - acc: 0.9796 - val_loss: 3.3064 - val_acc: 0.6923
Epoch 49/50
49/49 [==============================] - 0s 3ms/step - loss: 0.0186 - acc: 1.0000 - val_loss: 3.3164 - val_acc: 0.6923
Epoch 50/50
49/49 [==============================] - 0s 2ms/step - loss: 0.0150 - acc: 1.0000 - val_loss: 3.3186 - val_acc: 0.6923
While, when I try to evaluate my model in train set with model.evaluate(x_train,y_train)
I got this [4.552013397216797, 0.44897958636283875]
I have no idea how this happen? Thank you.

I have the same issue that is why i post it also here.
First i tested my model after training in a separate session.
I tested if i had read in the data and labels in a wrong order or
if the preprocessing is different, but everything seemed fine.,
Then i thought i had an issue similar to this one #https://github.com/keras-team/keras/issues/4875
But then i implemented testing directly after training and also on the training dataset,
which should give the exact same result as the training accuracy is?!
I compile using the fit_generator:
(i know the model is massively overfitting here, but this should not be an issue since it is just for tests)
Epoch 16/17
72/73 [============================>.] - ETA: 0s - loss: 0.1138 - acc: 0.9648 - weighted_acc: 0.9648Epoch 00016: val_acc did not improve
73/73 [==============================] - 20s 274ms/step - loss: 0.1139 - acc: 0.9646 - weighted_acc: 0.9646 - val_loss: 0.9137 - val_acc: 0.6249 - val_weighted_acc: 0.6249
Epoch 17/17
72/73 [============================>.] - ETA: 0s - loss: 0.1059 - acc: 0.9661 - weighted_acc: 0.9661
73/73 [==============================] - 20s 280ms/step - loss: 0.1053 - acc: 0.9664 - weighted_acc: 0.9664 - val_loss: 0.5450 - val_acc: 0.7273 - val_weighted_acc: 0.7273
879/879 [==============================] - 6s 6ms/step
Test_accuracy: 63.663%
My guess is that the model trains correctly but at the end of training storing/ updating the model or model weights something goes wrong.
Any help appriciated

@z888888861 Do you use Batch Normalization layers? Are you fine-tuning the network (trainable=False for some of the layers)?
If not, there is a very high chance you are overfitting the network.

If not, there is a very high chance you are overfitting the network.
How should overfitting be the problem when he is testing the training data? Maybe I miss something here.

@SpecKROELLchen Did not notice he was testing on training data. Are you using BN layers and finetuning? If yes you might be affected by what is currently discussed here #9965

@datumbox Sorry for the late response, and thanks for your response. I don't use BN layers but fine tuning.
I also checked you link, but did not understand how my problem is related to this one.
Basically the code snipped looks like this:
READ IN AND STUFF...
pmodel = resnet50.ResNet50(include_top=False, input_tensor=custom_input, weights="imagenet",
classes=20)
pmodel = pmodel.output
pmodel = GlobalAveragePooling2D()(pmodel)
pmodel = Dropout(drop_out_rate)(pmodel)
predictions = Dense(num_classes, activation='sigmoid')(pmodel)
pmodel = Model(inputs=custom_input, outputs=predictions)
pmodel.compile(optimizer="Adam", loss=binary_crossentropy, metrics=["accuracy"])
callback = ModelCheckpoint(Folder, monitor = "val_acc", verbose=1, save_best_only=True)
history = pmodel.fit_generator(TRAIN_DATA, validation_data = VAL_DATA, shufle= True,
callbacks=callback)
score = pmodel.evaluate(TRAIN_DATA,verbose=1, batch_size=12)
print(pmodel.metrics_names[1], score[1]*100)
And the test-accuracy ON TRAINING DATA is always something between 50-70%, while
the accuracy is like 95%.
I still did not solve this problem. So any help is appriciated.
@z888888861 since you first posted and did not response later on, did you maybe solve your problem?

@datumbox Ah, thank you. I thought you meant the layers behind the basemodel. Thx, i will keep an eye on that.

Sorry i have to repost here again. I think my problem might still be a little bit different or a mix of the BN problem and another one.
I tested VGG16 and the training AND validation accuracy did almost not change.
It stayed between 68-69%.
Then i tested it with Xception model (should also have BN) but then it massively overfitted
and train_acc = 98%, while val_acc=70%.
And no matter what i tried (increasing drop out, l2 kernel regularizer, reduce learning rate for Adams optimizer, i could not handle the overfitting).
And on ResNet50, i have high train_acc AND val_acc, while in reality or test mode the results are still poor.
I tried to switch every parameter but do not get good results.
Any help is appreciated.

Okay,
i still did not solve my problem and will describe my problem a little bit more in detail.
First step: reduced my problem to a 1 class binary classification as i have done it multiple times.
1.) Read in the data, doing padding when resizing the image
to the desired and needed target_shape.
2.) doing a train-test-split 80-20.
READ IN DATA AND LABELS...
# PREPROCESS the read in data
im_dat_gen = ImageDataGenerator(rescale=1. / 255,
                                rotation_range=40,
                                width_shift_range=0.2,
                                height_shift_range=0.2,
                                shear_range=0.2,
                                zoom_range=0.2,
                                fill_mode='nearest')

train_gen = im_dat_gen.flow(X_train, Y_train, batch_size=12)  # img_data_generator
val_gen = im_dat_gen.flow(X_val, Y_val)

Custom_input = Input(255,255,3)
base_model = vgg19.VGG19(include_top = False, input_tensor = input, weights = "imagenet", classes = num_classes)
x = base_model.output
x = GlobalAveragePooling2D()(x)
x = Dropout(drop_out_rate)(x)
x = Dense(fully_connected_size, activation='relu', kernel_regularizer=regularizers.l2(kernel_regu))(x)  # ,input_dim=7 * 7 * 512
x = Dropout(drop_out_rate)(x)
# 1 CLASS BINARY PROBLEM
predictions = Dense(1, activation='sigmoid')(x)
model = Model(inputs=Custon_input, outputs=predictions)

model.compile(optimizer="adam", loss="binary_crossentropy, metrics=["accuracy"])
# TB CALLBACKS
ckpt_callback = ModelCheckpoint(callbackfolder, monitor='val_acc', verbose=1, save_best_only=True)
tb_callback = TensorBoard("MyModel.h5", write_graph=True,
                              write_images=True)     #
cb_list = [ckpt_callback, tb_callback]
# FIT MODEL
history = model.fit_generator(train_gen,validation_data = val_gen, steps_per_epoch=steps_per_epoch,shuffle = True, epochs=num_epoch,
                                  callbacks=cb_list, validation_steps=validation_steps)
# TEST MODEL on training data (just to see if the weights etc. are saved correctly)
score = model.evaluate(X_train, Y_train, verbose=1, batch_size=batch_size)
print("%s: %.3f%%" % (model.metrics_names[1], score[1] * 100))

Comments:
I tested everything i can imagine, changing activation, not using dropout, not using freeze,
train n layers.
But the problem from my post above this one still exists.
The model does not seem to train at all.
while if i use resnet50 for the exact same code (just doing a different preprocessing),
my model trains, validation is very nice, but test on training data fails totally
(probably due to the BN issue)
I really need help :D.

have the same issue, any updates on this issue?

Same issue here. When training using fit_generator, I get a training and validation accuracy that are both much higher than the ones I get when I evaluate the model manually, on training and testing data.

Same problem here. I'm training with fit_generator, using separate generators for the training and validation sets:
history = classifier.fit_generator(train_generator,
    steps_per_epoch=train_batches,
    validation_data=val_generator,
    validation_steps=val_batches,
    epochs=60, verbose = 0)

The loss and accuracy that are stored in history for the training and validation sets have simply nothing to do with the values I get from e.g., using
scores = classifier.evaluate(X_val, Y_val, verbose=0)
print('validation loss:', scores[0])
print('validation accuracy:', scores[1])

Try to downgrade to keras 2.1.6 and see if you still face such issues. I could solve it by downgrading.

Try to downgrade to keras 2.1.6 and see if you still face such issues. I could solve it by downgrading.

Thanks @adityapatadia! This solved my problem, too.

I suggest you guys can save model/weights and load it and test, that might avoid the problem.

I also got this problem, and when I save model/weights and load it to test, this problem just occurs. Also applying fine-tune with fit_generator; the accuracy of training data using fit_geterator and evaluate are greatly different.

Loading the model from the saved model and evaluating it on the test set gives me 16% while
the reported validation accuracy after the last epoch (during training the model) is 85%.
I'm using fit_generator and real-time data augmentation.
Keras version: 2.2.4, with tensorflow backend: 1.13.1
python: 2.7.16

Try the following might help:

build your network again in your test phrase with the same code for building your network when you were training.
load weights from the trained model/weights.

what is the difference between what you suggested and load_model method?
Do you mean to set_weights for each layer separately?

if there is  your own custom layers/functions in your model.
if you didn't define well or what, you might not load the model correctly so it might come to the terrible test accuracy.
i used the "load_weights" methods and it's fine to avoid those problems.
it is just my suggestion, you can try it tho. Or check if you load your custom layers/functions correctly.

Hello, I think I got similar issue. Actually when I use keras.fit() to train the model and use the training data as validation data, I get different training accuracy and validation accuracy during the training process. Have you found the solution for the different training/test accuracy?

I have found a fix for this issue.
I encountered this issue while using ImageDataGenerators and obviously my model accuracies on both training and validation set were far more lower when using mode.evaluate() as compared to values returned from model history.
So the fix is to set  shuffle=False while creating your validation generator and then your accuracy will match with the validation set.
For training set it may not match as we generally keep shuffle=True for training.
Below is an example of how to create validation DataGenerator for reproducible results (Set shuffle=False) :
validation_datagen=ImageDataGenerator(rescale=1./255)
validation_generator=train_datagen.flow_from_directory(
    validation_directory,
    target_size=target_size,
    batch_size=validation_batch_size,
    class_mode=class_mode,
    shuffle=False
)```

Thank you very much @RaviBansal7717.

@RaviBansal7717 Nothing made sense, world was turning gray, you are a lifesaver ! Why on earth would shuffle=True be the default though.

I still have the problem that the training accuracy in history is 100% but when I use the model. evaluate(Train_image, Train_label),  it gives me 86%, plus I already turned off the shuffle, regularization, dropout, and set batch size equal to the whole dataset number during training. Really have no idea what went wrong?

I have a slightly different issue. Whether my training accuracy is 70, 80, or 98%, I routinely run model.evaluate(X_test, y_test) after training and get an accuracy score around 0.05.  I've seen it happen with and without BN and dropout layers. I'm using Keras 2.4.3, TF 2.5.0 and Python 3.9.5.  Suggestions very much appreciated .


need help with dropdown menu

I m using bootstrap for toogle down menu bar in asp.net c#
kindly check this link http://forums.asp.net/t/2037958.aspx?need+help+with+bootstrap

Sadly the issue tracker is for framework bugs not implementation questions. I'd recommend stack overflow with the twitter-bootstrap-3 tag. Good luck!


Configure on AIX 7.1 failure OpenSSL 3.0.2

AIX build is failing with OpenSSL 3.0.2
./config no-ssl3 --prefix=/usr/local/epic/EpicAugust2022 --libdir=/usr/local/epic/EpicAugust2022/lib --openssldir=/usr/local/epic/EpicAugust2022 -D_FORTIFY_SOURCE=2 -Wl,-R,/usr/local/epic/EpicAugust2022/lib -L/usr/local/epic/EpicAugust2022/lib...Your $OBJECT_MODE was found to be set to 64
Failure!  build file wasn't produced.
Please read INSTALL.md and associated NOTES-* files.  You may also have to
look over your available compiler tool chain or change your configuration.

Can't use string ("1/8") as a HASH ref while "strict refs" in use at /epic/perl_build/875518/source/openssl-3.0.2/util/perl/OpenSSL/config.pm line 924.

This can be fixed with this patch for config.pm:

 cat config.pm.patch
--- config.pm   2022-03-15 09:30:24.000000000 -0500
+++ config.pm   2022-04-14 14:29:02.000000000 -0500
@@ -875,7 +875,7 @@
              } else {
                  $config{disable} = [ 'asm' ];
              }
-             return %config;
+             return \%config;
          }
        ],

please open a PR for this


compare mysql num open connections with max open file limit

This will probably depend on the following upstream commit to get all of /etc/mysql/*:
https://github.com/sosreport/sos/pull/2510
As without that, we can't see the current value of 'max_connections' since the config files weren't collected in most cases. So check for the existence of max_connections:
./percona-xtradb-cluster.conf.d/mysqld.cnf:max_connections = 20000
If we can find a max_connections > 4190 then we should verify that /etc/systemd/system/mysql.service.d/charm-nofile.conf is configured with a value. In theory we could check the actual value but in practice that file was never created until I patched the charm, so if the file exists, it's likely OK. If it doesn't exist, it's not OK if max_connections is > 4190.

Need to also investigate the mysql-8 innodb cluster situation.

While we are at it lets also check for logs like "[Warning] InnoDB: Difficult to find free blocks in the buffer pool (338 search iterations)! 0 failed attempts to flush a page! Consider increasing the buffer pool size." since that implies that the innodb-buffer-pool-size is too small.

A bug related to max_connections with mysql-innodb - https://bugs.launchpad.net/charm-mysql-router/+bug/1929826

The log string check for
"[Warning] InnoDB: Difficult to find free blocks in the buffer pool (338 search iterations)! 0 failed attempts to flush a page! Consider increasing the buffer pool size." since that implies that the innodb-buffer-pool-size is too small.

seems straightforward to implement as a scenario check or even a bug if a known LP issue exists. Something like,
checks:
  input:
    path: 'var/log/mysql/error.log'
  free_blocks_difficult:
    expr: '(.+ \d+) (\d+:\d+:\d+) .+ (\S+) Difficult to find free blocks in the buffer pool\:'
conclusions:
  free_blocks_difficult:
        raises:
      type: SQLError
      message: >-
        XXXX <-- some reasonable message or if there's a LP bug already for this, I could do it as a bug itself instead of a scenario

But for the two properties, /percona-xtradb-cluster.conf.d/mysqld.cnf:max_connections and the existence (or values in it) of the /etc/systemd/system/mysql.service.d/charm-nofile.conf need to be exported and available as properties in the MySQLChecksBase class) which currently looks like
from hotsos.core.host_helpers import (
    APTPackageChecksBase,
    ServiceChecksBase,
)
from hotsos.core import plugintools

SVC_VALID_SUFFIX = r'[0-9a-zA-Z-_]*'
MYSQL_SVC_EXPRS = [r'mysql{}'.format(SVC_VALID_SUFFIX)]
CORE_APT = ['mysql']

class MySQLChecksBase(plugintools.PluginPartBase):

    def __init__(self, *args, **kwargs):
        super().__init__()
        self.apt_info = APTPackageChecksBase(core_pkgs=CORE_APT)
        self.systemd_info = ServiceChecksBase(service_exprs=MYSQL_SVC_EXPRS)

    @property
    def plugin_runnable(self):
        return self.apt_info.core is not None

Or is there another way of doing this without the MySQLChecksBase containing those two exported properties?

Hi @dosaboy ,
thanks for the help with how to approach this one.
So I've done something like this. This is not my final code, just something to check if I am on the right track as far as the config check is concerned. I am sure I can use better logic for the existence of the /etc/systemd/system/mysql.service.d/charm-nofile.conf file itself, here I am checking if it contains any string at all.
I just wanted to show you this rough approach, note this git diff doesn't contain the code for the bug check for ""[Warning] InnoDB: Difficult to find free blocks in the buffer pool ..." because that part is clearer than this other check for this issue..
So this is for the check of whether max_connections is greater than 4190 and if so, yet the charm-nofile.conf file does not exist.
diff --git a/defs/scenarios/mysql/mysql.yaml b/defs/scenarios/mysql/mysql.yaml
index 4e0ea32a..654bdd69 100644
--- a/defs/scenarios/mysql/mysql.yaml
+++ b/defs/scenarios/mysql/mysql.yaml
@@ -2,3 +2,32 @@
 # directory including subdirectories.
 requires:
   property: hotsos.core.plugins.mysql.MySQLChecksBase.plugin_runnable
+
+checks:
+  max_connections_greater_than_4190:
+    requires:
+      config:
+        handler: hotsos.core.plugins.mysql.MySQLConfig
+        assertions:
+          max_connections:
+            ops: [[gt, 4190]]
+  file_not_exists:
+    input:
+      path: /etc/systemd/system/mysql.service.d/charm-nofile.conf
+    requires:
+      expr: '(\S+)'
+
+conclusions:
+  max_connections_problem:
+    decision:
+      and:
+        - max_connections_greater_than_4190
+        - file_not_exists
+    raises:
+      type: MySQLWarning
+      message: >-
+        Max Connections is higher than 4190 but there is no
+        charm-nofile.conf seen.
+
+
+
diff --git a/hotsos/core/plugins/mysql.py b/hotsos/core/plugins/mysql.py
index 5dfd1e4a..8b645c78 100644
--- a/hotsos/core/plugins/mysql.py
+++ b/hotsos/core/plugins/mysql.py
@@ -2,8 +2,11 @@ from hotsos.core.host_helpers import (
     APTPackageChecksBase,
     ServiceChecksBase,
 )
-from hotsos.core import plugintools
-
+from hotsos.core import (
+    host_helpers,
+    utils,
+    plugintools
+)

 SVC_VALID_SUFFIX = r'[0-9a-zA-Z-_]*'
 MYSQL_SVC_EXPRS = [r'mysql{}'.format(SVC_VALID_SUFFIX)]
@@ -20,3 +23,10 @@ class MySQLChecksBase(plugintools.PluginPartBase):
     @property
     def plugin_runnable(self):
         return self.apt_info.core is not None
+
+class MySQLConfig(host_helpers.SectionalConfigBase):
+    def __init__(self, *args, **kwargs):
+        path = os.path.join(HotSOSConfig.DATA_ROOT, 'etc/mysql/mysql.conf.d/mysqld.cnf')
+        super().__init__(*args, path=path, **kwargs)
+
+


USUM UB Beast Ball Auto Mod + Partner Cap Shiny Auto Mod Check + IV Color Checker

Hey so when doing the regular gen 7 Ultra Beasts auto mod makes them caught in Beast Balls. However, this is not applied to Blacepheon and Stakataka.
Also auto mod struggles Shiny Partner cap pika (Which was proven legal.)
Pikachu-Partner (M) @ Pretty Wing
Ability: Static
Level: 21
Shiny: Yes
Hardy Nature

Thunderbolt
Quick Attack
Thunder
Iron Tail

025-07 ★ - Pikachu - 2F6C060D5DA3.zip

Also looking at the new IV doesn't seem to properly work.Everything still looks blue when selecting mons.


Identity is not authenticated and has empty claims when it reaches API

Core Library
MSAL.js v2 (@azure/msal-browser)
Core Library Version
2.14.0
Wrapper Library
MSAL Angular (@azure/msal-angular)
Wrapper Library Version
2.0.0
Description
I've been trying to implement this library with Azure B2C but I've hit a roadblock with authentication. I'm able to login to the app with the front-end code and push the access token to the API but by the time the request gets to the 'AuthorizationHandler' implementation on the API the user identity is completely blank. It has one identity but all the values are null and 'Authenticated' is false. I presume this is the reason that my requests are redirected to '/Account/Login?ReturnUrl='actualUrlNeeded''. Originally I was using the MsalService and our own interceptor but I've since switched to using the MSALInterceptor and I have the same problem, any and all help is greatly appreciated as this one has me stumped.
MSAL Configuration
const isIE = window.navigator.userAgent.indexOf('MSIE ') > -1 || window.navigator.userAgent.indexOf('Trident/') > -1;

export function MSALInstanceFactory(): IPublicClientApplication {
   return new PublicClientApplication({
      // MSAL Configuration
      auth: {
         clientId: <our client id>,
         authority: 'https://<our tenant>.b2clogin.com/<our tenant>.onmicrosoft.com/B2C_1_sign-in',
         knownAuthorities: ['https://<our tenant>.b2clogin.com'],
         redirectUri: 'https://localhost:5001/login',
         postLogoutRedirectUri: 'https://localhost:5001/login'
      },
      cache: {
         cacheLocation: BrowserCacheLocation.LocalStorage,
         storeAuthStateInCookie: isIE, // set to true for IE 11
      },
      system: {
         loggerOptions: {
            loggerCallback: () => {},
            piiLoggingEnabled: false,
         },
      },
   });
}

export function MSALInterceptorConfigFactory(): MsalInterceptorConfiguration {
   return {
      interactionType: InteractionType.Popup, // MSAL Interceptor Configuration
      protectedResourceMap: new Map([
      ['/api/', ['https://<our tenant>.onmicrosoft.com/<our client id>/access_as_user', 'openid', 'profile']],
      ]),
   };
}

@NgModule({
   declarations: [AppComponent, NavMenuComponent, HomeComponent],
   imports: [
...
MsalModule
...
],
   providers: [
...
       { provide: HTTP_INTERCEPTORS, useClass: MsalInterceptor, multi: true },
      {
        provide: MSAL_INSTANCE,
        useFactory: MSALInstanceFactory
      },
      {
        provide: MSAL_GUARD_CONFIG,
        useFactory: MSALGuardConfigFactory
      },
      {
        provide: MSAL_INTERCEPTOR_CONFIG,
        useFactory: MSALInterceptorConfigFactory
      },
      MsalService,
      MsalGuard,
      MsalBroadcastService,
...
  ],
   bootstrap: [AppComponent, MsalRedirectComponent],

Relevant Code Snippets
Api in Startup.cs:

services.AddSingleton<IAuthorizationPolicyProvider, CustomAuthorizationPolicyProvider>();
            services.AddSingleton<IAuthorizationHandler, ValidUserAuthorizationHandler>();
            services
               .AddAuthentication(JwtBearerDefaults.AuthenticationScheme)
               .AddMicrosoftIdentityWebApi(options =>
               {
                  options.TokenValidationParameters.NameClaimType = "name";
               },
               options => {
                  options.Instance = "https://<our tenant>.b2clogin.com";
                  options.ClientId = "<our client id>";
                  options.Domain = "<our tenant>.onmicrosoft.com";
                  options.TenantId = "<our tenant id>";
                  options.SignUpSignInPolicyId = "B2C_1_sign-in";
               });

In CustomAuthorizationPolicyProvider:

public Task<AuthorizationPolicy> GetDefaultPolicyAsync()
      {
         var policy = new AuthorizationPolicyBuilder();
         policy.AddRequirements(new ValidUserAuthorizationRequirement());
         policy.Combine(options.Value.DefaultPolicy);
         return Task.FromResult(policy.Build());
      }

//placeholder
internal class ValidUserAuthorizationRequirement : IAuthorizationRequirement
   {
   }

Identity Provider
Azure B2C Custom Policy
Source
External (Customer)

@robsierra Can you please clarify, is your API receiving the token but with missing claims or is it not receiving a token at all?

@tnorling Yes, thank you for examining the question. I receive the token but all of the information in the identity is null:

@robsierra Since MSAL is successfully acquiring the token and attaching it to the request I'm going to suggest you reach out to Azure B2C Support. MSAL has no control over the contents of the token, that is likely controlled by your tenant settings, policy configuration, app registration and/or specific claims/scopes you need to request. Unfortunately we are not the experts in this area so I can't give you any suggestions, however, support should be better able to help you get this resolved. You can open a ticket with them here

@tnorling Okay, thank you very much I'll do just that.


Render QR code with temporary icon and words

Closes https://github.com/getodk/central/issues/746

The previous approach was to use the QR code library qrcode-generator to generate an <img> tag with the QR code data encoded in a data URL.

This new approach uses a <canvas> to render the QR code but also render a box on top it it and additional text below it, all into the same image, and then generate the data URL of that canvas to put into an <img> tag.

What has been done to verify that this works as intended?
I've tried it and seen it work, but I haven't updated the tests yet.
Why is this the best possible solution? Were any other approaches considered?
It's a little indirect, using a canvas to made the desired image and then instead of showing the canvas directly, putting it into an img tag with a data URL.
But it also means the rest of the code doesn't have to change (the popover really just wants an img) and the QR code is something people can save right from their browser.
I tried using an OffscreenCanvas but I'm less familiar with this and wasn't sure how to get the final result out in the way I wanted it. Using a regular canvas but just not showing it worked easier and was easier for me to debug.
How does this change affect users? Describe intentional changes to behavior and behavior that could have accidentally been affected by code changes. In other words, what are the regression risks?
We talked about how adding something over the QR code might make it not decode, so we settled on a smaller square image with more text below. This QR code does successfully decode so it's probably ok.
Does this change require updates to user documentation? If so, please file an issue here and include the link below.
Before submitting this PR, please make sure you have:

[ ] run npm run test and npm run lint and confirmed all checks still pass OR confirm CircleCI build passes
[ ] verified that any code or assets from external sources are properly credited in comments or that everything is internally sourced

🎉
I think doing an image in the center and some text at the bottom works well and I'm excited that it sets us up to add useful text for regular QR codes too.
Some surface thoughts we can discuss at any time:

We could maybe use the 📝 emoji for symmetry with Collect
I think it would likely be helpful to use the word "temporary". Maybe "Temporary draft"?

Updated look:

(could the icon be centered better? probably... i was having some weird issue where it doesn't always render in the same spot. sometimes it's a little bit higher than that.)

could the icon be centered better? probably... i was having some weird issue where it doesn't always render in the same spot.

That's interesting that it's not always consistent. But I personally think it's OK if it's not perfectly centered vertically.


Make Overlay.panIntoView an API method

Add parameters for custom animation parameters and margin

Add protected wrapper method for previous autoPan functionality

See #10741

It could be nice to have the function take a single options object.  This would have animation and margin properties and would allow more options to be added in the future without an awkwardly long signature.
To do this, you would create a new type as is done with the existing constructor options.

I agree. It would also then make sense to alter the autoPan options to use the new type rather than being two separate parameters to the constructor (and two internal properties).

so autoPanOptions in the constructor options instead of autoPanAnimation and autoPanMargin, and then carry the change through the whole class? autoPanAnimation and autoPanMargin could be left in as deprecated options for backward compatibility to avoid breaking the API.

@tschaub the test failures (rendering test) appear to be unrelated to this change: Could you force the build to run again?

@ahocevar Nice and obvious idea - should have thought of that myself!


zetaclient persistent state management

Each zetaclient has its own persistent state with leveldb; examples include

last scanned block on external chains : https://github.com/zeta-chain/zeta-node/blob/1b5a1feca45be76cd70c23530716f3e2f7c7fad4/zetaclient/evm_client.go#L862
observed external txs and receipts:
https://github.com/zeta-chain/zeta-node/blob/1b5a1feca45be76cd70c23530716f3e2f7c7fad4/zetaclient/evm_client.go#L875
bitcoin clients stores UTXOs and "used" UTXO (signed & broadcasted, but not yet mined) in leveldb as well.

The persistent state management is a bit ad-hoc and while leveldb is simple & efficient, it does not have
proper database features such as filter by multiple keys etc.
Also, there might be more states that need to be persisted, such as the signed outbound transaction.
action items:

[x] explore using sqlite to replace leveldb as zetaclient persistent state
[x] explore using some kind of ORM such as GORM to make persistence in sqlite easier and maybe persist more app state.

Closed by #504


Feature/wait for purchase

Warped up all the waiting for events into a purchase method called wait_for_completion
Added a compatible property is_completed which actually calls is_purchase_valid, to check to see if the smart contracts have finished completing the purchase.

We had something like this many months ago in the original test Ocean where we had a sub process/thread to wait for the blocking event. So yes I'll put it on the issue list.
I'm not sure, on the best method of handing errors. I would think we should move towards using exceptions for all calls.


EC2 Metadata roleName request returned error

I'm getting this error when cloning this app:
EC2 Metadata roleName request returned error
I've made no changes aside from entering the AWS account IDs. I've confirmed that the accounts are configured with AWS CLI credentials.
AWS CLI Version: 2.1.26
AWS CDK Version: 1.89.0
Windows

giving --profile to the cdk cli helped


Lets not deploy PRs to Sonatype

Apply the same changeset that was done for pac4j
[WARNING] The requested profile "js" could not be activated because it does not exist.
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-deploy-plugin:2.7:deploy (default-deploy) on project j2e-pac4j: Failed to deploy artifacts: Could not transfer artifact org.pac4j:j2e-pac4j:jar:1.1.1-20150720.190634-9 from/to sonatype-nexus-snapshots (https://oss.sonatype.org/content/repositories/snapshots/): Failed to transfer file: https://oss.sonatype.org/content/repositories/snapshots/org/pac4j/j2e-pac4j/1.1.1-SNAPSHOT/j2e-pac4j-1.1.1-20150720.190634-9.jar. Return code is: 401, ReasonPhrase: Unauthorized. -> [Help 1]
[ERROR]

PR: https://github.com/pac4j/j2e-pac4j/pull/11


Restructured the content under third-party integrations

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind user-interface

/kind content

/kind translation

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area blog

/area documentation

/area videos

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

Clarified the info about running Falco as a daemonset
added MacOS instructions for minikube
added MicroK8s instructions

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign rajakavitha1 after the PR has been reviewed.
You can assign the PR to them by writing /assign @rajakavitha1 in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

e2d0f91 restructured the thirdparty-integrations
369e59e Merge branch 'master' of https://github.com/falcosecurity/falco-website into rajie-4

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign rajakavitha1 after the PR has been reviewed.
You can assign the PR to them by writing /assign @rajakavitha1 in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

e2d0f91 restructured the thirdparty-integrations
369e59e Merge branch 'master' of https://github.com/falcosecurity/falco-website into rajie-4

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

closing this PR to add the DCO

closing this PR to add the DCO


don't turn off village music if we're at a spawnpoint tagged as being in the village

…lues for area, consume is TH village flag to disable music

testing done:

spawned at a location tagged as being in the village, confirmed that music plays and that the logs reflect that it is tagged
spawned at a location not tagged as being in the village, confirmed that music does not play and that the logs reflect that it is not tagged


#1368 Up-to-date Lookout page content

Hi @marnovo @ricardobaeta
This commit is referred to the issue #1368 on the Backlog with the same name.
It updates the content of the Lookout page

already deployed into staging https://landing-staging.srcd.run/lookout/

Hi @dpordomingo, sorry I missed your last message until now. I was sure I did the sign-off, maybe it is because I used the accept option on the git page. I just followed the steps and I hope everything is fine now.
Thanks.

Thank you @znegrin !

sorry, it seems some commit was not signed off yet

Hi @dpordomingo,
I've been trying to follow the steps and to fix the users and sign-offs but it keeps getting weirder. I really don't understand what is going on. I guess the problem is that weird "zuri.negrin" user that I don't understand where is coming from.
I am trying and following the requieren DCO action but keeps asking for more actions over and over again so I am no longer sure of what is happening here. Probably is a silly thing but I don't understand why is it happening and I've been several hours trying to fix this unsuccessfully. I would appreciate some light and help with this.
Thank you!

Ok, I fixed the problems with DCO reworking the broken commit:
$ git show 55e9bfb55d8e68264dfa7e8024b3625dca6d54a3

commit 55e9bfb55d8e68264dfa7e8024b3625dca6d54a3
Author: zuri.negrin <Cle_12-trI>
Date:   Mon Mar 4 19:07:29 2019 +0100

    Up-to-date Lookout page content

    Signed-off-by: zuri.negrin <Cle_12-trI>

    Signed-off-by: zuri.negrin
    Signed-off-by: Zuri Negrín<EMAIL_ADDRESS>

I'll merge against candidate branch


docs: update README.md

ziped -> zipped

@eltociear Thanks for your PR. For the simple type error modification, I will not consider merge it to the main branch, sorry. You may consider add some new features and Pull Requests again, I will happy to merge it.


Attributes mismatch for Content Launcher Cluster AcceptHeader

Problem

expected behavior: the type of Content Launcher Cluster -> AcceptHeader should be [list-def][string]

https://github.com/CHIP-Specifications/connectedhomeip-spec/blob/49f5a1fb8e73c515233b851500e9c4219eb07c47/src/app_clusters/media/ContentLauncher.adoc#13-attributes

actual behavior: the type is   DataModel::List  《 const chip::ByteSpan》 ;

https://github.com/project-chip/connectedhomeip/blob/master/zzz_generated/app-common/app-common/zap-generated/cluster-objects.h#L26571

Proposed Solution
change to DataModel::List 《 const chip::CharSpan》; ?

Fixed in https://github.com/project-chip/connectedhomeip/pull/12961


Update reference to GraphIQL 0.9.1 to newer version?

In https://github.com/apollographql/graphql-server/blob/master/packages/graphql-server-module-graphiql/src/renderGraphiQL.ts#L32 there is a hard coded reference to 0.9.1. I am having troubles with this version which look the same as https://github.com/graphql/graphiql/issues/356. Is there any reason not to upgrade to a newer version?
(I am trying to get GraphIQL to fetch the schema from a PHP GraphQL server, so I fetch the schema remotely, build it into a local schema object, but then get an error message that looks like 356 above. It mentioned the bug was fixed in 0.9.2.)

Thanks for filing an issue @alankent. #407 should take care of this. I will release a new version shortly.


Changed type check to protect against null values

Changed the type check as the value can be null .GetType will throw a null reference exception.

Thank you for PR!


For adding %datetime% to log entries

Please see the post here: https://hydrogenaud.io/index.php/topic,124427.msg1043429.html#msg1043429
Also please add ability to log by newest entry first.  Thank you!

@marc2k3 again with the snippet to the rescue!


Permissions: Strongly typed / enums

Fixes #913.

I'm not sure if we want to have to maintain a full list of these on our end. Even if we maintain a list it should be inferred from the underlying type so we can get errors if the underlying CDK changes.
Also I sort of wonder if we can just change the API to pass the function that's supposed to be called eg
permissions: [EventBus.grantPutEventsTo]

Let me take a crack and defining an inferred type

Some weird typescript for you
  type GrantFunc = `grant${string}`;
  type GrantExtract<T, K = keyof T> = K extends GrantFunc ? K : never;

  type EventBusGrant = GrantExtract<typeof events.EventBus>;

EventBusGrant evaluates to "grantPutEvents" | "grantAllPutEvents"

You can use the above method for now but hypothetically there is a way for them to pass in any construct + function pair and have the typings + autocomplete work.  Don't have time to get that working at the moment.
Longer term I do wonder if this [construct, string] api is a good one. I feel like we might as well ask them to pass in the function right?

The issue is that not all the functions that look like grantSomething behave the same way. Some Receive more arguments than others.

Ah ok.  You can also use typing to filter out function signatures that match what we support, but further hints at the idea that this maybe is the wrong api if we're not supporting all grant functions

Yeah. It's good enough for most grants which is great since we can always use other permission formats.
But it's true that we can always imagine other ways to handle permissions.
In a trade-off between speed-and-simplicty over quality-and-complexity, you're the one deciding :D

Yeah let's hold on this until I can chat with Frank and Jay on the long term direction we want to go

We are transition this repo to https://github.com/sst/v2 and are closing this issue for now. If it is still relevant please reopen it there - sorry for the trouble


Enable androidMode='spinner' when mode='time'. This closes #119, clos…

…es #193

Coverage remained the same at 98.276% when pulling c3e089e49fb77894206982d4201c0ccfec1e8d4e on dsandmark:master into 58ed2b1fa9c6f372104788e6d2b5fa960be2792f on xgfe:master.

#204


Open code

when will you open the  code ?

Thank you for your attention! All code and weights are now open source!


Support VirtualBox ovfs

Two changes were required here, and each has its own commit:

Update the govmomi schema to support StorageItem, which is the node where virtualbox defines to which controller a disk is attached. (vmware puts that info in an Item node)
Update ovf_utils to use the new node.


Add drawing app script

Drawing App using OpenCV
Aim
Create a basic Python script used for making a drawing app using OpenCV
Detail
On calling the file on the command line a window opens where you can draw anything.

Hello, I am a CSE prefinal year student, I would like to work on this issue.

Assigned to you..Go ahead..

thank you

@yashasvimisra2798 When I can expect the PR?


Garbage Collection Basics

The book should include a chapter (or series of multiple chapters) discussing the basics of garbage collection and how to tackle this using Rust. The end goal should be a set of resources to implement a naïve, non-moving mark-and-sweep garbage collector. While such a GC won't be very useful for most applications it's easy enough to understand/explain that it should serve as a good introduction.

At the Basics level, this series should be limited to single-threaded mutator/gc stop-the-world collection.
Are there any reasons to not cover a basic implementation of copying collection such as Cheney's algorithm at this level? That might make a smoother transition to advanced gc #3.
Possible breakdown:
Mark and sweep collection

a free-list allocator

memory blocks/arenas
object headers
free-list alloc()
free-list free()

marking
sweeping

Copying collection

bump allocation

memory block alignment
from-space and to-space

copying collection

object forwarding
two-finger tracing and copying

@pliniker

Are there any reasons to not cover a basic implementation of copying collection such as Cheney's algorithm at this level?

Since copying and mark-and-sweep are a bit different I think we could do both, then stick with one of the two for the rest of the book.

@pliniker This depends a bit on what you consider to be "registers". For example, in Inko the "registers" are just indexes in a Rust Vec. This is done because the VM has to be able to suspend code at arbitrary points in time and resume code, meaning it can't rely on the Rust stack.
For the sake of simplicity we could do something similar where we use a dedicated data structure for the stack/register, instead of relying on the Rust call stack and registers.

Another option is to slightly adjust Cheney's algorithm and don't immediately trigger GC in an allocation, instead waiting until reaching a safepoint.

@YorickPeterse Having re-read the Immix paper, I think I see better why you favor it :slightly_smiling_face:  and I have to agree.
I'm now thinking that the Basics chapter(s) might best be designed to build toward Immix:  block-size-aligned blocks (#9), basic bump allocation (#4), mark & sweep.
This leaves major issues unaddressed and doesn't result in a practically usable GC yet - line marking, allocating into partially reclaimed blocks, defragmentation, parallelizing etc need to be covered in the Advanced chapter(s).
This feels a lot more targeted and focused to me, now.  Perhaps it's what you were originally suggesting?

@pliniker That sounds like a good idea, even if it's just because it saves us from having to cover two very different algorithms.

Depends on:

#24


Fix data race on search.

The VPTree type was not thread-safe during search.  The tau variable
was shared across all instances.  This patch removes it from the struct
and adds it as a parameter to search() instead.
I've also a test for use with `go test -race -cpu=4``.  Without the fix,
this test reports a data race.

Wow, thank you! I hadn't actually considered using search in parallel. I hope you found the project useful. :)

Yes, I'm using a specialised version of it in https://github.com/dgryski/go-simstore .

Cool. ^_^


Issue

When i try to acces my server using this system, it prints;
[ERROR] AxiosError: Request failed with status code 404
[ERROR] AxiosError: Request failed with status code 404

Hi, can you share more information about your system please.
The first thing that comes to mind would be a variable in the .env that is missing or not in the right format. Check the PTERO_HOST, it shouldn't end with a slash, it's an error in my documentation, i will update it soon.

Thanks. That was the issue.


bump emberjs-build

I restarted the build (it should have been fixed by https://github.com/tildeio/glimmer/pull/202).


Fixing issue #18

This fixes the top border not being displayed correctly on the notification box in the profile.
See #18

Thank you for your first contribution @Wolfuryo! I would be happy to see you more around here.


Compiling problem

I compile the GAPS library, and there are some errors. The output of the compiled information is very long. So, I output it to a log file. Some problems list below.
make[5]: Entering directory /home/hoo104/3dmatch-toolbox/gaps/apps/pfm2pfm' make[5]: *** No rule to make target ../../pkgs/CSparse/*.h', needed by pfm2pfm.o'.  Stop. make[5]: Leaving directory /home/hoo104/3dmatch-toolbox/gaps/apps/pfm2pfm'
make[4]: *** [opt] Error 2
make[4]: Leaving directory /home/hoo104/3dmatch-toolbox/gaps/apps/pfm2pfm' make[3]: *** [target] Error 2 make[3]: Leaving directory /home/hoo104/3dmatch-toolbox/gaps/apps'
make[2]: *** [opt] Error 2
make[2]: Leaving directory /home/hoo104/3dmatch-toolbox/gaps/apps' make[1]: *** [target] Error 2 make[1]: Leaving directory /home/hoo104/3dmatch-toolbox/gaps'
make: *** [opt] Error 2

out.log.zip

Hoototo.  I cannot open a zip file.   So, please send the errors inline
directly in an email.   Thanks.
On 3/24/2017 3:31 AM, Hoototo wrote:

I compile the GAPS library, and there are some errors. The output of
the compiled information is very long. So, I output it to a log file.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tomfunkhouser/gaps/issues/2, or mute the thread
https://github.com/notifications/unsubscribe-auth/AFj-efNBtmgm3eFwQpLDbuv1CbtvVXz0ks5ro3FHgaJpZM4Mn1BG.

I have the compiling problem to.
I search the folder pkgs that can't a file or folder named CSparse.

I met the same compiling problem , have you solved it? @Hoototo @happyfrank

CSparse is only used if RN_USE_CSPARSE is defined.
Can you post the compile error indicating from where CSparse is being referenced?

make[5]: Leaving directory /home/shi/tools/3dmatch-toolbox/gaps/apps/scnview' make[4]: Leaving directory /home/shi/tools/3dmatch-toolbox/gaps/apps/scnview'
cd pfm2pfm; make opt
make[4]: Entering directory /home/shi/tools/3dmatch-toolbox/gaps/apps/pfm2pfm' make ../../bin/x86_64/pfm2pfm "CFLAGS=-DRN_USE_CSPARSE  -Wall -I. -I../../pkgs  -O3 -DNDEBUG" "LDFLAGS= -L../../lib/x86_64 -O " make[5]: Entering directory /home/shi/tools/3dmatch-toolbox/gaps/apps/pfm2pfm'
make[5]: *** No rule to make target ../../pkgs/CSparse/*.h', needed by pfm2pfm.o'.  Stop.
make[5]: Leaving directory /home/shi/tools/3dmatch-toolbox/gaps/apps/pfm2pfm' make[4]: *** [opt] Error 2 make[4]: Leaving directory /home/shi/tools/3dmatch-toolbox/gaps/apps/pfm2pfm'
make[3]: *** [target] Error 2
make[3]: Leaving directory /home/shi/tools/3dmatch-toolbox/gaps/apps' make[2]: *** [opt] Error 2 make[2]: Leaving directory /home/shi/tools/3dmatch-toolbox/gaps/apps'
make[1]: *** [target] Error 2
make[1]: Leaving directory `/home/shi/tools/3dmatch-toolbox/gaps'
make: *** [opt] Error 2

You can look andy's answer
https://github.com/andyzeng/3dmatch-toolbox/issues/2#issuecomment-289919224


Do not import anything, just explicitly extend it

I've increasingly come to think that Julia code is more readable if we avoid doing any broad import calls and only use Base.foo names instead. This makes it easy to identify the source of any function and makes it more obvious when you're doing something that might have negative consequences.

Complete as of 84ebf99cf1427a3554d0fbeaa0ec038e6bf5d565


Bug in NavBar

Description:-
On clicking bookmarks in navbar when on login/signup we get the error 404 page not found.
Screenshot

Please assign this issue to me under gssoc'24


WFLY-19131 Clustering TS: disable color output logging from nodes

Resolves
https://issues.redhat.com/browse/WFLY-19131

Thanks @rhusar


refactor: find-my-way => micro + micro-fork

and nodemon => micro-dev
and support query params: style, label, emoji, list
Regarding https://github.com/amio/badgen-service/issues/10#issuecomment-406813669
Preview
https://badgen-service-jczedbqvoc.now.sh/
https://badgen-service-jczedbqvoc.now.sh/badge/chat/gitter/purple?style=flat&label=asdf

Hm or not. Yea it works anyway i think.

When will go live?

Curious, what badge/query are you going to use?

I already started replacing all the badges to Badgen ;]]
As about the query usage. I don't like npm | v1.2.3 and release | v1.2.3, so i make them npm version | v1.2.3 and github release | v1.2.3 :D Self preference, it's more clear and explicit. (https://github.com/olstenlarck/eslint-config-xaxa)

It's live now :D

Why it's not automattic after the merge? Why is that Now deployment Github App then?

We need a reliable test suite before making deployment automatically.
Now's Github App keep complaining in almost every PR:

Since this Pull Request originated from a forked repository, Now cannot deploy it as there are potential security risks.

So I turn it off, at least for now. 😔


Global engine registry

This branch adds a Sprockets.register_engine method that allows you to globally override built-in engines or register new engines. Engines can still be overridden or registered on a per-environment basis with the existing API.
Intended usage by third-party libraries:
require 'sprockets/engines'
Sprockets.register_engine '.scss', MyScssEngine

@chriseppstein - will this work for you?

I know this is old, but where is the proper place to put Rails.application.assets.register_engine for a custom asset processor?


Bug

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Now when you have a problem Ex{..} it will bring you to a solution that
will have to detail with common comments session and details on the dev
repository
On Thu, 10 Aug 2023, 1:39 pm smy088, @.***> wrote:

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always
frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features
you've considered.
Additional context
Add any other context or screenshots about the feature request here.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/Web-Dev-For-Beginners/issues/1179, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AMICBU5CGYVTIELU4NPJTFDXUTI63ANCNFSM6AAAAAA3LOA7HU
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


dispatchToast crashes when bundled

Hi, thanks for your contributions, we use these bindings extensively! We're having some trouble with Toasts at the moment:
Calls to dispatchToast seems to crash the app when called, only if the app is built and previewed with vite. The error message output by react is a "Minified React error #31":
Objects are not valid as a React child (found: object with keys {timeout, politeness, intent, toastId}). If you meant to render a collection of children, use an array instead.
It seems the generated js does not output a valid Toast ReactElement, but instead attempts to render the DispatchToastOptions as an object.
To reproduce

Write a component using a Toast, like in the provided wiki example
Build the app using fable and vite build
Try out the Toast

This issue is only present when built, running the project normally without the build argument passed to vite allows toasts to work as intended.
Thanks. Let me know if you need any more info or are unable to reproduce.

Hey @otytlandsvik , thanks for the first legit issue! Sorry to hear that-- the Toast component was definitely one of the more complicated to get working (or almost working, I guess). I'll look into it and keep you posted. Feel free to look around the project; maybe you can spot the issue before I can.

@otytlandsvik Although I'm having a hard time reproducing it, I'm assuming it has something to do with tree shaking when Vite is building. I'm thinking that maybe if I use more variables/ functions in order to create the dispatchToast method, then it won't be as affected by tree shaking. Just a theory, though. I'll keep trying to reproduce it. Any thoughts?

I think I managed to solve it!
By the looks of things, the use of your utility JSTuple.from2Args is problematic in this case. The compiled js code looks something like this:
    const dispatchToast = (tupledArg) => {
        const element = tupledArg[0];
        const options = tupledArg[1];
        const options_1 = createObj(options);
        const args = [element, options_1];
        controller.dispatchToast(args[0], args[1]);
    };

...and when vite performs its bundling magic, it turns into:
        (props_2) => {
            let args_1;
            const tupledArg = props_2;
            controller.dispatchToast(
                ((args_1 = [tupledArg[0], createObj(tupledArg[1])]),
                args_1[0],
                args_1[1]),
            );
        },

Instead emitting the arguments directly into the function like this seems to work:
        let dispatchToast (element, options) =
            let options = !!options |> createObj |> unbox<DispatchToastOptions>
            emitJsExpr (element, options) "$0, $1"
            |> controller.dispatchToast

I'll create a PR in a moment.

This is exactly why I made this project public-- so that people smarter than me can solve the issues that I can't quite figure out yet. 😁 Thanks for the repo branch and the PR! I'll push a new version with the change soon.


Fix hierarchy conflict validation excluding non-propagated objects

Resolves issue with hierarchy conflict validation
Excludes objects/resources not being propagated

for example, (behaviour with current logic):
hnc-parent  namespace has kube-root-ca.crt configmap with applied propagate.hnc.x-k8s.io/none=true annotation
when creating a child namespace this will result in the following validation error, even though the parent configmap is not going to be propagated to the child namespace
kubectl apply -f child-ns.yaml
kubectl apply -n hnc-child  -f child-ns-hierarchy.yaml
Error from server (Conflict): error when creating "child-ns-hierarchy.yaml": admission webhook "hierarchyconfigurations.hnc.x-k8s.io" denied the request: Cannot update hierarchy because it would overwrite the following object(s):
  * Namespace "hnc-child": kube-root-ca.crt (/v1, Kind=ConfigMap)

with this PR, the parent annotated configmap resource would be excluded from this check and there is no validation error

Adding info on tested scenarios ->
TESTED:
### with latest hnc version, errors seen for resources with propagate none annotation ->
$ kubectl apply -n hnc-child  -f child-ns-hierarchy.yaml
Error from server (Conflict): error when creating "child-ns-hierarchy.yaml": admission webhook "hierarchyconfigurations.hnc.x-k8s.io" denied the request: Cannot update hierarchy because it would overwrite the following object(s):
  * Namespace "hnc-child": kube-root-ca.crt (/v1, Kind=ConfigMap)
  * Namespace "hnc-child": istio-ca-root-cert (/v1, Kind=ConfigMap)

------
### now after updating hnc image from this PR
### creation works without issue (and no validation errors)
kubectl apply -n hnc-child  -f child-ns-hierarchy.yaml
hierarchyconfiguration.hnc.x-k8s.io/hierarchy created

### after removing the annotation the conflict validation errors return as expected
kubectl -n hnc-parent annotate cm istio-ca-root-cert propagate.hnc.x-k8s.io/none- # remove annotation
kubectl apply -n hnc-child  -f child-ns-hierarchy.yaml
Error from server (Conflict): error when creating "child-ns-hierarchy.yaml": admission webhook "hierarchyconfigurations.hnc.x-k8s.io" denied the request: Cannot update hierarchy because it would overwrite the following object(s):
  * Namespace "hnc-child": istio-ca-root-cert (/v1, Kind=ConfigMap)

/ok-to-test

Can you please also add a short note about testing to the commit message? E.g.:
Tested: got a spurious error on an unpropagated object with this change, which is fixed with this change. Also added a unit test that fails without this change but passes with it.

For that second part, that's always a good sanity check if you didn't already try it - just make sure the test fails if you break the feature.

/lgtm
/approve
/hold
/assign @rjbez17

@rjbez17 lmk if you have any concerns about this
/hold cancel

Sorry I meant to respond earlier. Lgtm as well.


test for same value

Dear chen:
After I train a xgboost model, I test on my test set, but I get same value for the different test record. I do this on sklearn gradient boostring regressor, I get the result. So I want ask you, why I get the same value using xgboost, and how should I tune parameters?
Thank you very much!

Old question, no answer. Closing


kui is not supporting kubectl plugins (other than itself)

using kui shell app i would expect that my usual plugins (most of them installed with krew) are working the same way than in my traditional shell. bit this is not the case :

thanks for the bug report! we will look into this. i agree that this is an important use case.


Fix: fix population of data when data-urlencode flag is used with G flag i…

Description

fix population of request data field when using the --data-urlencode flag along with the -G or --get flag when importing from curl closes #2010

Contribution Checklist:

[x] The pull request only addresses one issue or adds one feature.
[x] The pull request does not introduce any breaking changes
[x] I have added screenshots or gifs to help explain the change if applicable.
[x] I have read the contribution guidelines.

Note: Keeping the PR small and focused helps make it easier to review and merge. If you have multiple changes you want to make, please consider submitting them as separate pull requests.
Publishing to New Package Managers
Please see here for more information.

Thank you @jwetzell !
This will go out in the 1.13.0 release. Please subscribe to Discord announcement channel to stay notified on the releases.


[Pulsar SQL] Add max split entry queue size bytes limitation

Motivation
In Pulsar SQL, there is a configuration pulsar.max-split-entry-queue-size to control the entry queue capacity, but some entries are so big some are small, it's hard to control the entry queue size bytes.
Modifications
Add a new configuration pulsar.max-split-entry-queue-cache-size to control the entry queue size bytes.
Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (no)
The public API: (no)
The schema: (no)
The default values of configurations: (no)
The wire protocol: (no)
The rest endpoints: (no)
The admin cli options: (no)
Anything that affects deployment: (no)

@gaoran10 Could you please check the comments?

/pulsarbot run-failure-checks

@gaoran10 Could you please check sijie's comments?

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks


support namespace

The namespace can isolate storage.

In addition, a dependency dirs is added, because std::env::var("HOME") does not support windows

Maybe it is more appropriate to use different storage for different namespaces?

I changed namespace use different storage.  and supports data migration between different versions
https://github.com/fewensa/microkv/pull/1
Hope to submit pr to you after seeing your comments

Huge apologies for the long wait on merging this, as other work was prioritized!
Never really thought about the need for namespacing/buckets for the purposes this project was introduced for, but seeing the demand in other projects, will definitely merge to fully support.
Little iffy about introducing another dirs dependency, but will introduce for cross-platform compatibility.

In fact, I made a lot of changes after this pr, including the migration after the version upgrade, etc., which became more complicated than before. But I think it can be optimized a bit, and I will submit the pr to you after the modification is completed. If you think it’s okay, I hope it can be merged


Wonky formatting inside TSX brackets with )}

From @prencher on April 21, 2016 22:28

VSCode Version: 1.0
OS Version: Any

When formatting a TSX document,  gets formatted as .
It appears isolated to ), for example  is unaffected, as is every other case I could think of to test.
I would assume this is supposed to not have spaces, or respect the non-empty bracket setting added in 1.0.
Copied from original issue: Microsoft/vscode#5630

Agree. Moving to the TS team which provides the formatter.

Duplicate of #6716


TSX (web-mode) tree-sitter-hl not working

I've been using tree-sitter-hl-mode for rustic-mode and js2-mode and it seems to be working great. When I try to enable it in a .tsx file (typescript-tsx-mode), however, I get an error and the mode fails to start.
font-lock-eval-keywords: Wrong number of arguments: #<subr quote>, 0

Here's the backtrace:
Debugger entered--Lisp error: (wrong-number-of-arguments #<subr quote> 0)
  quote()
  font-lock-eval-keywords(quote)
  tree-sitter-hl--minimize-font-lock-keywords()
  #<subr tree-sitter-hl--setup>()
  apply(#<subr tree-sitter-hl--setup> nil)
  tree-sitter-hl--setup()
  tree-sitter-hl-mode(toggle)
  funcall-interactively(tree-sitter-hl-mode toggle)
  command-execute(tree-sitter-hl-mode record)
  counsel-M-x-action("tree-sitter-hl-mode")
  #f(compiled-function (x) #<bytecode -0x20a815e057386a3>)("tree-sitter-hl-mode")
...

Can you provide more details to reproduce the issue? For example:

Your OS and Emacs build, e.g. the result of (emacs-version).
The installation mechanism, e.g. package.el vs. straight.el.
What is typescript-tsx-mode? Where does it come from? It doesn't seem to be on MELPA.
A minimal init.el with the issue. This can be a good start: https://gist.github.com/ubolonton/a4cfe85782f40e280570b3ef393ded5f
The values of font-lock-keywords and font-lock-defaults before enabling tree-sitter-hl.

Sorry that I'm just now responding to this. It still doesn't seem to work on the latest commit.

Result of M-x emacs-version: GNU Emacs 28.0.50 (build 1, x86_64-pc-linux-gnu, GTK+ Version 3.24.21, cairo version 1.16.0), this is native-comp running on NixOS 20.09 with doom.
Installation: straight.el, which comes with doom
typescript-tsx-mode comes from DOOM, just derived from `web-mode, described like so:

Major mode derived from web-mode by define-derived-mode.

It inherits all of the parent's attributes, but has its own keymap,
abbrev table and syntax table:

  typescript-tsx-mode-map, typescript-tsx-mode-abbrev-table and typescript-tsx-mode-syntax-table

Here's the relevant part of my config:
(use-package tree-sitter
  :hook ((rustic-mode python-mode json-mode js-mode js2-mode typescript-mode go-mode sh-mode) . tree-sitter-mode)
  :config
  (require 'tree-sitter-langs)
  (push '(typescript-tsx-mode . typescript) tree-sitter-major-mode-language-alist)
  (add-hook 'tree-sitter-after-on-hook #'tree-sitter-hl-mode))

Here's font-lock-keywords:
(t
 (web-mode-fontify
  (whitespace-point--flush-used)
  ("\\(	+\\)" 1 whitespace-tab t))
 (web-mode-fontify
  (0 font-lock-keyword-face))
 (whitespace-point--flush-used
  (0 nil))
 ("\\(	+\\)"
  (1 whitespace-tab t)))

Here's font-lock-defaults:
('(web-mode-fontify)
 t)

Sorry that I'm just now responding to this. It still doesn't seem to work on the latest commit.

Result of M-x emacs-version: GNU Emacs 28.0.50 (build 1, x86_64-pc-linux-gnu, GTK+ Version 3.24.21, cairo version 1.16.0), this is native-comp running on NixOS 20.09 with doom.
Installation: straight.el, which comes with doom
typescript-tsx-mode comes from DOOM, just derived from `web-mode, described like so:

Major mode derived from web-mode by define-derived-mode.

It inherits all of the parent's attributes, but has its own keymap,
abbrev table and syntax table:

  typescript-tsx-mode-map, typescript-tsx-mode-abbrev-table and typescript-tsx-mode-syntax-table

Here's the relevant part of my config:
(use-package tree-sitter
  :hook ((rustic-mode python-mode json-mode js-mode js2-mode typescript-mode go-mode sh-mode) . tree-sitter-mode)
  :config
  (require 'tree-sitter-langs)
  (push '(typescript-tsx-mode . typescript) tree-sitter-major-mode-language-alist)
  (add-hook 'tree-sitter-after-on-hook #'tree-sitter-hl-mode))

Here's font-lock-keywords:
(t
 (web-mode-fontify
  (whitespace-point--flush-used)
  ("\\(	+\\)" 1 whitespace-tab t))
 (web-mode-fontify
  (0 font-lock-keyword-face))
 (whitespace-point--flush-used
  (0 nil))
 ("\\(	+\\)"
  (1 whitespace-tab t)))

Here's font-lock-defaults:
('(web-mode-fontify)
 t)

typescript and typescript-tsx have different syntax see tree-sitter-typescript
It's a conflict between  JSX syntax and the foo Typescript casting syntax. In typescript-tsx you cast using foo as number. see here https://www.typescriptlang.org/docs/handbook/jsx.html#the-as-operator
I think the fix is to have separate mode, grammar, and HL queries for typescript-tsx
I also get parsing errors from a normal TSX file that compiles correctly.

Turns out tree-sitter-langs already comes with the TSX grammar, you can set it up with a typescript-tsx mode.
Still missing TSX highlighting queries I think. but this is a pretty good start.
Here's my config that lets me parse TSX
;; Typescript
(setq typescript-indent-level 2)
;; you can also use the DOOM one if you wish
(define-derived-mode typescript-tsx-mode typescript-mode "TSX"
  "Major mode for editing TSX files.

Refer to Typescript documentation for syntactic differences between normal and TSX
variants of Typescript.")
(add-to-list 'auto-mode-alist '("\\.tsx?\\'" . typescript-tsx-mode))

(use-package tree-sitter
  :ensure t)

(use-package tree-sitter-langs
  :ensure t
  :after tree-sitter
  :config
  (tree-sitter-require 'tsx)
  (add-to-list 'tree-sitter-major-mode-language-alist '(typescript-tsx-mode . tsx)))

With this config, if I open a typescript file and check M-x tree-sitter-debug-mode, I get some nice jsx nodes in the parse tree:
program:
  import_statement:
    import_clause:
      namespace_import:
        identifier:
    string:
  import_statement:
    string:
  import_statement:
    import_clause:
      identifier:
    string:
  lexical_declaration:
    variable_declarator:
      identifier:
      arrow_function:
        formal_parameters:
        statement_block:
          return_statement:
            parenthesized_expression:
              jsx_element:
                jsx_opening_element:
                  identifier:
                jsx_text:
                jsx_element:
                  jsx_opening_element:
                    identifier:
                  jsx_text:
                  jsx_closing_element:
                    identifier:
                jsx_text:
                jsx_element:
                  jsx_opening_element:
                    identifier:
                    jsx_attribute:
                      property_identifier:
                      string:
                  jsx_text:
                  jsx_element:
                    jsx_opening_element:
                      identifier:
                      jsx_attribute:
                        property_identifier:
                        string:
                    jsx_text:
                    jsx_closing_element:
                      identifier:
                  jsx_text:
                  jsx_element:
                    jsx_opening_element:
                      identifier:
                      jsx_attribute:
                        property_identifier:
                        string:
                    jsx_text:
                    jsx_closing_element:
                      identifier:
                  jsx_text:
                  jsx_element:
                    jsx_opening_element:
                      identifier:
                      jsx_attribute:
                        property_identifier:
                        string:
                    jsx_text:
                    jsx_closing_element:
                      identifier:
                  jsx_text:
                  jsx_closing_element:
                    identifier:
                jsx_text:
                jsx_closing_element:
                  identifier:
  export_statement:
    identifier:

@srcreigh I wonder if a very small MELPA package that sets up the derived mode as a target for hooks wouldn't be such a bad idea.

https://github.com/tree-sitter/tree-sitter-javascript/blob/master/queries/highlights-jsx.scm
simple jsx highlights.scm, can be used with existed typescript and javascript query.

This is a crude workaround, but if you set
(setq tree-sitter-hl-use-font-lock-keywords nil)

then you'll bypass the logic in tree-sitter-hl--minimize-font-lock-keywords that causes the error. Highlighting works correctly after that.
My value of font-lock-keywords when I get the error is:
 (("\\_<[[:digit:]]+\\(?:\\.[0-9]*\\)?\\_>" quote highlight-numbers-number)
  web-mode-fontify
  ((lambda
     (bound)
     (hl-todo--search nil bound))
   (1
    (hl-todo--get-face)
    t t))
  (whitespace-point--flush-used)
  ("\\(  +\\)" 1 whitespace-tab t))

Unlike what @loafofpiecrust posted, this does include quote as seen in the error, so maybe that's helpful for somebody who understands the tree-sitter-hl--minimize-font-lock-keywords code better than I do.

Thanks @sangaline for the workaround.
I'm exploring a bit and found this bug is mainly due to this construct, right?
;; In web-mode
(setq font-lock-defaults '('(web-mode-fontify) t))
;; In emacs-tree-sitter
(setq keywords-spec (car font-lock-defaults))
(car (cdr keywords-spec)))
;; Which led to an equivalent
(car (car '('(web-mode-fontify)))) ;; => quote

If font-lock-defaults value allow to have quote, I think any mode will be broken on this part. Maybe someone has the idea on what value font-lock-defaults allows?


Heatmap Research

Research javascript heatmap open-source libraries for use in our project

http://www.patrick-wied.at/static/heatmapjs/examples.html
http://www.patrick-wied.at/static/heatmapjs/docs.html#h337-register

Heatmapjs looks cool, only it seems to only support points. Not a show stopper, but you might want to figure out how to handle a polygon.

Ok is then would you like something more like a choropleth map style?

http://wrobstory.github.io/2013/04/python-maps-choropleth.html
Could be an example it needs more detail and using the world.

Last meeting we decided to move forward with polygons on the heatmap and not "blobs". We can implement the polygons but are having trouble finding a data source to provide us with the polygon coordinates of a location.
The ability to outline a polygon of a location exists in Google Maps (try this search for "phoenix, arizona"), but the Google Maps API does not provide a method of accessing a polygon's coordinates.
We are not alone in the quest for polygon coordinates:
Google Maps API V3: How to get region border coordinates (polyline) data?Add “Search Area” outline onto google maps result
Reading the above posts directs you to Twitter's API that gives you the ability of retrieving a "bounding box" of a searched "place". We are investigating if this is a sufficient and accurate source of polygon data.

We can (and should) still approximate an street address as a point, however at some size (maybe 1 square mile?) moving to a polygon would be much more useful.
I would suggest starting with the large regions (countries, states, and cities) first and then work your way down. You could look at the TIGER shapefile data for inside the US and convert it to postgis with the shp2pgsql utility.  There also is thematicmapping.org that can provide world wide countries and areas.
Shape files are a pretty common format for vector data. Fair warning: pay attention to your map projection when using these utilities, chances are you're going to get data in two different projections.
The "bounding box" of a searched "place" could be a good approximate to start with, especially for computing how two regions compare (do they intersect? is one inside the other? etc)

Just another thought; if you knew the relative size of your locations (categorically speaking, so a city vs an street address) you could use your existing points and create a circle with a radius proportional to the size. The scope of this database isn't to accurately define the extent of a region so a circle approximation would be better than nothing.


Addition of armv6-m

I'm trying to model Armv6-m ISA with an intermediate language representation.
I was planning to compare O0 vs O3 Assembly codes easily using godbolt but I realized we can do -march = armv6 but no -march = armv6-m.
I don't know if there is a general need for it but I would be delighted to have that.

Ah, thank you very much it's actually compiler independent


If applied to a selector, what's the behaviour?

.whatever {
  font-rendering: mandatory 60s;
}

.whatever .foo {
  font-family: foo;
}

.whatever .bar {
  font-family: bar;
}

.whatever .blah {
  font-family: sans-serif;
}

Assuming:

foo & bar are defined in @font-face rules
foo takes 5 seconds to load
bar takes 10 seconds to load
(I'm also assuming the font-rendering in .whatever has precedence over rules in @font-face)

When does the text in .foo render? As in, does it also wait for bar as font-rendering is set on a parent that uses both fonts? What about the text in .blah?

The way I just defined it in http://tabatkins.github.io/specs/css-font-rendering/ is that each font face comes with a timer that starts when the UA first attempts to use it.  Whether a given element using the font considers itself blocked, swappable, or stuck depends on its individual 'font-rendering' values, compared against the font's timer.  The 'font-rendering' descriptor in the @font-face itself just provides a default for any element that doesn't manually specify things.
For more complicated things, definitely just use the Font Loading API.


[Question]: I only persisted the docstore for my VectorStoreIndex How can I rebuild the index?

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
I only persisted the Docstore in my storage context.
Now I can't reload my VectorStoreIndex even though I think the Docstore contains all the we need?
I have feeding the reloaded Docstore into a new VectorStoreIndex using the from_documents but it doesn't work

@Data-drone The docstore probably doesn't contain the embeddings, unless you specifically attached embeddings to your nodes before they were inserted.
So, embeddings aside, you can get the nodes back with
from llama_index.core.storage.docstore import SimpleDocumentStore

docstore = SimpleDocumentStore.from_persist_path("./storage/docstore.json")

nodes_dict = docstore.docs
nodes = list(nodes_dict.values())

index = VectorStoreIndex(nodes=nodes, ...)


Disable defender also during build

Motivation
At the moment Windows defender is only disabled during unit tests to speed them up. This PR also disables it during the build.
Changes

Disable Windows defender during build

Test Plan
n/a
TODO

[x] Explain motivation or link existing issue(s)
[x] Test changes and document test plan
[x] Update documentation as needed

DevOps Notes

[x] This PR does not require configuration changes (e.g., environment variables, GitHub secrets, VM resources)
[x] This PR does not affect public APIs
[x] This PR does not rely on a new version of external services (PoET, elasticsearch, etc.)
[x] This PR does not make changes to log messages (which monitoring infrastructure may rely on)

bors merge


Ninja as a dependency in JIT mode

Sorry I'm a little bit confused, can you clarify what need to be specified as requires in _get_requires_for_build function and what need to be set as install_requires? Thanks!

@yzh119 Sorry for insufficient description.
install_requires in setup.py and project.dependencies in pyproject.toml specifies packages required at runtime.
build-system.requires in pyproject.toml specifies packages required to build the package, and
_get_requires_for_build can specifies addtional packages considering options like FLASHINFER_ENABLE_AOT=1.
In JIT mode, we do not build shared objects when building the package, but users need ninja to accelerate building. Thus ninja should be in install_requires/project.dependencies. It is ok that _get_requires_for_build specifies ninja but not necessary.
In AOT mode it is just opposite. ninja is supposed to be present in _get_requires_for_build and absent in install_requires/project.dependencies.

Hi @ur4t It seems that the latest main FLASHINFER_BUILD_VERSION isn't working.


Use Haumea for gathering chart metadata

Haumea converts file trees to attrsets.
❯ git checkout -q master && nix eval --json .#chartsDerivations.x86_64-linux > old
❯ git checkout -q haumea && nix eval --json .#chartsDerivations.x86_64-linux > new
❯ diff old new && echo OK
OK

In Goorzhel:haumea2 I have another proof of concept—lifting the default.nix files—but I didn't want to do too much in one PR.

I do agree this is more readable now 👍


Feature request : Support for blob types

Hello, I have an use case in my app where I need to store a lot of configuration file that are binary but they are all tiny and would like to store them as blob instead of file.
Maybe an option in the file type to store them as blob instead of real file on the file system or on a s3 bucket ?
Thanks for your beautiful work on pocketbase

Sorry, there are no plans for this (at least for v1). For me the cons maintaining it outweigh the limited use cases where having such option will be beneficial.
It will not only complicate the internals, but will cause also unnecessary confusion for the developers which storage to use and when. Additionally, we'll have to implement some special tool to allow migration from blob to file storage because I'm sure someone will end up with a use case where they've started with blob and later decided to increase the file size limit and now want to migrate to local/s3 filesystem.


spin templates list should show source of the template

it might be useful to show the source of the template when we run spin template list.
+-----------------------------------------------------------------+
| Name         Description                                        | Source|
+=================================================================+
| http-c       HTTP request handler using C and the Zig toolchain |<EMAIL_ADDRESS>            |
| http-go      HTTP request handler using (Tiny)Go                | https://github.com/fermyon/spin@main                         |
| http-ts      HTTP request handler using Typescript              |https://github.com/rjindal/spin@some-branch-name    |

This feels like a lot of clutter for the default display - I worry it would overwhelm the content that folks are actually interested in.  So maybe behind a --verbose switch or something.
That said, although this sounds simple it requires a non-trivial change to the innards.  At the moment, templates, once installed, are entirely standalone - they don't remember where they were installed from.  So we would need to change the installer to save that information.  However, that does open up some useful possibilities, such as spin new being able to say "hey that template you're instantiating is from a different version of Spin than what you're using. Do you want to get the matching one?"

At lease partially addressed in #1085.

I'll open a new issue for the chat that wasn't addressed in the PR.


IPFS upload

We want to support files being dragged into the chat window, then we upload to IPFS, and then Receiver shares the link in the chat.
See prior hackathon projects as an example.

Similarly, if a user uploads a .png, .svg, or other image format, after we upload to IPFS we display the image inside the Receiver window as a message.

One example of using it inside chat: https://github.com/De-Chat/DeChat/blob/fec96230a87bc5d5727974c6aa6b5909542650d5/README.md#chat--image--file-sending
They pass along the file type inside the message to help the chatbox render as an image or as a file link.